pip install not working

Looks like pkgconfig isn't in the requirements on PyPI.

thanks for telling. It's in requirements.txt already, you think it's about the install_requires line in the setup.py?

It may have been a matter of the version on PyPI being out of date. In the end I needed to install most of the requirements separately anyway, and it took some messing around with paths to get things to build (probably because of conda). Here is what ended up working (in a conda environment that had numpy already):
conda install pkgconfig
conda install cmake
conda install eigen3
conda install -c menpo eigen # required this channel to get the correct version
pip install pybind11
git clone https://github.com/iosonofabio/lshknn.git
cd lshknn
export CPLUS_INCLUDE_PATH=/Users/james.webber/anaconda3/envs/maca-dash/include/eigen3
python setup.py install

It looks like it's just that one must install the requirements on manually, rather than building them into setup.py which is what I was expecting. For instance in .travis_before_install.sh you set up all of the dependencies before installing this repo.
A potential enhancement would be write a proper build recipe for conda, so that all of these requirements could be installed in one line.

Sure, Iàm not very familiar with conda but feel free to to make a PR with the recipe continuously deploying to anaconda cloud or, if you prefer to maintain recipes by hand, add one to anaconda cloud.


[Minor] Convertion tool

Adding beginning support for converting XML to JSON manifests. Currently, it is a draft PR because the tool used is not 100% ready.

Do these changes impact command syntax of any of the packages? (e.g., add/remove command, add/remove a command parameter, or update required parameters)
If Yes, briefly describe what is impacted.
Yes, adding one more option output to the convert command.

Do these changes impact documentation? (e.g., a tutorial on https://docs.microsoft.com/en-us/office/dev/add-ins/overview/office-add-ins)
Not yet, as the documentation for these commands are yet to be written.

If you answered yes to any of these please do the following:
> Include 'Rick-Kirkham' in the review
> Make sure the README file is correct
Validation/testing performed:
Manual runs of the convert tool and npm run test while the tool is in the root directory.

But there is no reference in the PR to a package to create a link for.  That's my question . . . where is "./converter/lib/main.js" coming from?  It's not in the PR itself and the PR doesn't contain and reference to a package that suggest the manifest converter we are trying to integrate.  So if I were to clone and run this then I don't think it would work because I'm missing something not included here.

You both @akrantz and @millerds are correct, this PR itself does not work right now. I am using some local code on my machine to be able to use the converter calls, that's why I opened as a draft PR, sorry for not being more clear that it is not working right now.
To have the final PR version, I am just waiting to receive the package version of the local files I'm using. As suggested, I would change the CLI call to use npx package_name.

You both @akrantz and @millerds are correct, this PR itself does not work right now. I am using some local code on my machine to be able to use the converter calls, that's why I opened as a draft PR, sorry for not being more clear that it is not working right now.
To have the final PR version, I am just waiting to receive the package version of the local files I'm using. As suggested, I would change the CLI call to use npx package_name.

I'm not sure what suggestion relates to "npx package_name" . . . I was suggesting we don't want a cli command for converting a manifest and that is should just be the project convert.

You both @akrantz and @millerds are correct, this PR itself does not work right now. I am using some local code on my machine to be able to use the converter calls, that's why I opened as a draft PR, sorry for not being more clear that it is not working right now.
To have the final PR version, I am just waiting to receive the package version of the local files I'm using. As suggested, I would change the CLI call to use npx package_name.

I'm not sure what suggestion relates to "npx package_name" . . . I was suggesting we don't want a cli command for converting a manifest and that is should just be the project convert.

Agree, ideally we have some APIs to call instead of using the CLI. What I meant is that there is not NPM package for us to import yet. When we have it, we can work on making this PR work.

@millerds This is a non-erroring version. I think we can move with the PR even if still have some bugs so it becomes easier for people to test.

Note that we can't check this in until after private preview.

@millerds just made the output in the same folder as the input manifest.
This PR is now functional and working. Can we merge it? I think we should merge it and hold on to publishing.
Then we will see how to publish/release to the people that need it.

I checked and we are still using versions <1.0.0 which basically means pre-release so I guess we should be OK checking this in and even publishing it as long as we support it like pre-release versions.


Re-enable &  fix unit tests in test_schemas.py for schemas_e6d6074

Resulting from schema update in PR #353

Fixing this issue is uncovering inconsistencies in the Avro schemas themselves since 0.5.1. I'll shortly create a PR in ga4gh/schemas to address all the ones I've found.

Fixed in #515


JAMES-3683 Implement PerSenderRateLimit mailet

I think I might find a bug here. I think we should not increase the counter when it exceeds one of the rate limits. Please have a look at the test below:

I think this is more part of the underlying RateLimiter component and we might be limited on this by the RateLimitJ choice.
Could you be having a look, @quantranhong1999 ?

I think this is more part of the underlying RateLimiter component and we might be limited on this by the RateLimitJ choice.

I did read some implementation code of RateLimitJ. I think it works fine for an individual rate limit. e.g when a user requests a number of requests that exceed the limit, RateLimitJ does not increase the counter.
I think the issue here is that we combine many rate limits. E.g rate limits A and B: when user requests exceed A but do not exceed B => A counter is not changed but B increases.
If we have an API for checking whether the rate limit will be exceeded without actually increasing it yet, we can check all rate limits together. But I do not see RateLimitJ exposing that API...  Any idea? Should we just skip this case?

I think the issue here is that we combine many rate limits. E.g rate limits A and B: when user requests exceed A but do not exceed B => A counter is not changed but B increases.

This mailet relies on individual, distinct rate limiters: one per limit. This wouldn't make sense.

Should we just skip this case?

I need to have a deeper look too.

IMO this does not impact much the overall rate limit use cases. This only happens at the edge use case when the user exceeds a rate limit. If we choose a suitable group of rate limits then it is likely that the user will exceed the other rate limit soon => that user should wait for resources regeneration.

:+1:

I think I might find a bug here. I think we should not increase the counter when it exceeds one of the rate limits. Please have a look at the test below:

Ah I got it: it is an atomicity problem!
If one of the limits is reached, we will increment the ones that were checked before - and passed.
Ok I understand, and this would actually be a really hard problem to tackle. I would also agree ignoring this,


Support insertion in StreamingHttpResponse

Since Django 1.5 StreamingHttpResponse is introduced, this response class does not have content attribute, django-debug-toolbar middleware should be aware of that.
Now I get this error:
Traceback (most recent call last):
  File "/usr/lib/python2.7/wsgiref/handlers.py", line 85, in run
    self.result = application(self.environ, self.start_response)
  File "/home/sirex/.buildout/eggs/Django-1.5-py2.7.egg/django/contrib/staticfiles/handlers.py", line 72, in __call__
    return self.application(environ, start_response)
  File "/home/sirex/.buildout/eggs/Django-1.5-py2.7.egg/django/core/handlers/wsgi.py", line 255, in __call__
    response = self.get_response(request)
  File "/home/sirex/.buildout/eggs/Django-1.5-py2.7.egg/django/core/handlers/base.py", line 187, in get_response
    response = middleware_method(request, response)
  File "/home/sirex/.buildout/eggs/django_debug_toolbar-0.9.4-py2.7.egg/debug_toolbar/middleware.py", line 131, in process_response
    smart_unicode(response.content),
  File "/home/sirex/.buildout/eggs/Django-1.5-py2.7.egg/django/http/response.py", line 341, in content
    "Use `streaming_content` instead." % self.__class__.__name__)
AttributeError: This StreamingHttpResponse instance has no `content` attribute. Use `streaming_content` instead.

I use Django 1.5 and django-debug-toolbar 0.9.4.

I'm still getting this on Django 1.7.7 and DjDT 1.3.0:
"This FileResponse instance has no content attribute. Use streaming_content instead."
Traceback:
File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py" in get_response
204.                 response = middleware_method(request, response)
File "/usr/local/lib/python2.7/dist-packages/debug_toolbar/middleware.py" in process_response
94.             new_response = panel.process_response(request, response)
File "/usr/local/lib/python2.7/dist-packages/w3c_validator/panels.py" in process_response
38.         self.validator.validate_source(response.content)
File "/usr/local/lib/python2.7/dist-packages/django/http/response.py" in content
377.             "Use `streaming_content` instead." % self.__class__.__name__)

Exception Type: AttributeError at /assets/css/skin.css
Exception Value: This FileResponse instance has no `content` attribute.
Use `streaming_content` instead.

Is there a config change I need to make, or a workaround?
JJW


Hopefully fixing #7639.

We'll try getting all the dependencies before cloning in order to fix #7639 once and for all.

Need to install interop's dependencies as well...
go get google.golang.org/grpc/interop

Interop green again. I am merging this since otherwise pull requests will remain broken.


Add query param serializer to work with openapi handling of query parameters

Currently, OpenApi spec supports the following serialization for query parameters. Our cloud app uses the 2nd line in the table in the above link. This means that parameters that take in string arrays should look like this: foo=bar,baz,bat. However, the react-query generator uses axios and, by default, produces url strings like this instead: foo[]=bar&foo[]=baz&foo[]=bat. This change explicitly converts the axios default to use the correct serialization that OpenApi supports.
PR checklist

[x] Read the contribution guidelines.
[x] Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
[x] Run the following to build the project and update samples:./mvnw clean package
./bin/generate-samples.sh
./bin/utils/export_docs_generators.sh

Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/java*.
For Windows users, please run the script in Git BASH.
[x] File the PR against the correct branch: master, 5.1.x, 6.0.x
[x] If your PR is targeting a particular programming language, @mention the technical committee members, so they are more likely to review the pull request.

Potentially improve Axios handling by allowing user to pass in request-specific options instead of a customAxiosInstance.


Query finalized header once if there are multiple clients for same network

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (9aaebbc) 18.02% compared to head (59fbf2a) 17.98%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #713      +/-   ##
==========================================
- Coverage   18.02%   17.98%   -0.05%
==========================================
  Files          25       25
  Lines        3461     3470       +9
==========================================
  Hits          624      624
- Misses       2772     2781       +9
  Partials       65       65

Files
Coverage Δ

blockchain/finalized_headers.go
0.00% <0.00%> (ø)

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


Add option to get help about script usage.

This is the start of a work on adding help to the scripts to fast display their usage to users.
@mamueluth can you please check this? What do you think? (Please also answer to the TODO comments in the code.)
This is an interesting source about exiting early: https://echorand.me/posts/bash-functions-exit/

Maybe we could beside the print_and_exit function add functions like:
Warning
Info
....
Which print the given message in a defined way (e.g color coded). This way we would have a consistent coloring for given messages and could change adjust formatting of messages as needed.

And yes, probably to add additionally a function process_inputs for this general things.


e2e test\workflow\bceidbusiness\industry_user_admin

Description:
Tests of \bceidbusiness\industry_user_admin workflow
WIP

[ ] assert "My Operator\create new Operator"
[ ] ...

Closing this - we don't have an industry_user_admin role anymore


Getting Started: capitalized "Editor" looks weird

Testing #111554

VS Code version: Code - Insiders 1.52.0-insider (92192baf760315ac47fb2caaf3ce6d0778962352, 2020-12-03T05:43:59.635Z)
OS version: Windows_NT x64 10.0.19042

/assign @JacksonKearl

Yeah I agree, updated. Thanks!


Updates to phish cmdlets

Docs Build status updates of commit 208c402:
:clock10: Full build: the average full build time is 14 min(s) 18 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 208c402:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Disable-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Enable-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/New-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/New-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Remove-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Remove-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Set-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Set-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit 3a8018b:
:clock10: Full build: the average full build time is 14 min(s) 20 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 3a8018b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Disable-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Enable-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/New-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/New-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Remove-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Remove-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Set-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Set-AntiPhishRule.md
:white_check_mark:Succeeded
View (exchange-ps)

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


WT^ size limit

Currently all pending wt(s) are added to the WT^ when one is created. Instead, the sidechain should limit the transaction size to something reasonable considering the state of the parent chain and noting that the transaction size will slightly increase when it is unblinded.
See:
https://github.com/drivechain-project/bitcoin/blob/sidechainBMM/src/miner.cpp#L683-L749


Speed up pytest execution wall time by ~37%.

Before: 644 passed in 19.89s.
After: 644 passed in 12.47s.
Running with python -m pytest --durations=5 showed this test case was a huge outlier, and timing its execution showed a huge disparity between the time TechnicalDrawing thought it took to run and the time its caller saw et up. Dropping the Builder context makes these times lineup.
Tested the individual test case speed-up with hyperfine -- 'python -m pytest tests/test_drafting.py::TestTechnicalDrawing::test_basic_drawing' which reports before & after of:
Benchmark 1: python -m pytest tests/test_drafting.py::TestTechnicalDrawing::test_basic_drawing
  Time (mean ± σ):      9.488 s ±  0.534 s    [User: 9.525 s, System: 1.478 s]
  Range (min … max):    9.117 s … 10.929 s    10 runs

Benchmark 1: python -m pytest tests/test_drafting.py::TestTechnicalDrawing::test_basic_drawing
  Time (mean ± σ):      1.951 s ±  0.040 s    [User: 2.292 s, System: 1.111 s]
  Range (min … max):    1.871 s …  2.020 s    10 runs

I dug a little into why it was so much slower under the builder and the answer seems to be ~13k calls to deepcopy, but I haven't dug deeper yet to figure out if there's a way to "transfer" to the builder rather than making copies.
(full disclosure: the next-slowest test case is test_pack.py that I added very recently; I'll be looking into speeding that up next :) )

This is something we may want to open an issue for, I've noticed certain operations take an order of magnitude longer when inside a Builder context.

This is something we may want to open an issue for, I've noticed certain operations take an order of magnitude longer when inside a Builder context.

Filed https://github.com/gumyr/build123d/issues/384 which also has a pointer to a self-contained repro test case, in case that's of interest.

(full disclosure: the next-slowest test case is test_pack.py that I added very recently; I'll be looking into speeding that up next :) )

This now #386.


Dataflow mapper will propagates the error and reason back to the caller.

Coverage increased (+0.02%) to 87.243% when pulling e05c60cd5bd072323d398748a09f5a2067d86b5e on KRuelY:mapperPropagatesError into b06c423a09d7ae653c107c4080dbcd557570ab27 on cloudfoundry-incubator:master.


Add upgrade validation test

Description

To test upgrade in real scenario there should be user defined objects dependent on babelfish objects.
This commit adds upgrade validation test to identify dependencies on babelfish objects using pg_catalog.pg_depend. It runs as a part of the version upgrade test framework in the set up base version composite action.
The dependencies being tracked are :
User defined to babelfish objects

Views to view
Views, computed columns to functions
Views, computed columns to operators
Views, tables, functions, procedures, type to type
Views, table to collation
The dependencies on cast are not tracked by pg_depend.
Also, procedures and functions are stored as string, so dependencies within body is not tracked by postgres.

The check has expected dependency file based on the base version that contains the objects for which tests are not present currently. It then compares the expected and output file to validate upgrade tests present.
If the test fails, it gives the expected dependency file and a csv file containing dependency count to analyse the coverage of dependencies on the babelfish objects in the upgrade logs of the respective upgrade.

Signed-off-by: Dhruvi Sheth<EMAIL_ADDRESS>Check List

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

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

One of the upgrade test is failing. Please resolve that.

Looks good to me!


tf.Print converts Variable to mutable Tensor

The PR is proposed to resolve #14788.
tf.Print converts Variable to mutable Tensor, instead of constant.
v = tf.Variable([99])
# <tf.Variable 'Variable:0' shape=(1,) dtype=int32_ref>

p = tf.Print(v, [v])
# Tensor("PrintRef:0", shape=(1,), dtype=int32_ref)

How to test

[x] add test case
[ ] pass all tests

Can one of the admins verify this patch?

Handing this off to @alextp as variables czar. I think this approach won't play at all well with ResourceVariable, so it might need some more work....

Can't you achieve the same result by doing the following in the definition of Print:
 if v.dtype._is_ref:
  with ops.control_dependencies([Print(constant_op.constant(1), data)]):
    return gen_array_ops._ref_identity(v._ref)

?

Though I do discourage you from writing such code. Why do you want this? Forwarding mutable references to variables is weird and not quite future-proof with resource variables.

Thanks for feedback, @mrry , @alextp .
Yes, I also think it weird to convert variable to mutable tensor. @alextp 's solution is better and straight.


KAFKA-5362: Follow up to Streams EOS system test

improve tests to get rid of calls to sleep in Python
fixed some flaky test conditions
improve debugging

Call for review @enothereska @dguy @bbejeck @guozhangwang

We got from 2 thread to one and increase number of Streams instances from 2 to 3 to compensate. This is required to make rebalances predicable to allow replacing sleeps with wait-conditions.
Also triggered 25 rounds of system tests: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/987/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6137/
Test FAILed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6122/
Test FAILed (JDK 8 and Scala 2.12).

This should got to 0.11.0, too.
Test failures are unrelated. Seems to be a more general Jenkins issues with those test -- I saw multiple reports that this SSL tests failed (guess, re-testing does not help atm). But his PR only does system test stuff anyway.

Seems like there was a system test failure in the above link?

Some of the changes are exactly what I had suspected during my on-call.
One general question - do we want to consider checking for the actual messages consumed (assuming sent with an incrementing number in the value) as well?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6296/
Test FAILed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6280/
Test FAILed (JDK 8 and Scala 2.12).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6304/
Test FAILed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6288/
Test FAILed (JDK 8 and Scala 2.12).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6292/
Test FAILed (JDK 8 and Scala 2.12).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6308/
Test FAILed (JDK 7 and Scala 2.11).

retest this please

@dguy No need to trigger retests -- I am playing with this branch to figure out why system tests are still unstable...

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6311/
Test FAILed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6295/
Test FAILed (JDK 8 and Scala 2.12).

Rebased and extended closing timeout. Triggered system test: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/1090/

Retriggered system test: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/1092/

I've  just triggered another build as the last one didn't work for some aws issues: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/

1093 still fails.

There was a 5 minute gap in the logs for one Streams instance where nothing happened... Might have been environmental.
Retriggered: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/1094/

I had a quick look and it seemed like there may have been some connectivity issues with the brokers and between the brokers

Retest this please

The last run 1094 got more than 100 successful runs -- we never got this many before. I guess we should merge this now. The latest changes increasing the timeouts and adding some flush statements for stdout seems to do the trick.

retest this please

Merged to trunk and 1.0.


Why is Prefer=return-no-content?

I am very new to OData.  I am using the v3 nuget package and am trying to read all entries like this:
		try {
			ODataClientSettings odataSettings = new ODataClientSettings("https://test.fielddirect.com/DataServices/OData/") {
				PayloadFormat = ODataPayloadFormat.Atom
			};
			odataSettings.BeforeRequest += delegate (HttpRequestMessage message) {
				message.Headers.Authorization = new AuthenticationHeaderValue("Basic", App.CurrentUser.Token.ToString());
				message.Headers.Remove("Prefer");
				Console.WriteLine(message);
			};
			odataSettings.AfterResponse += delegate (HttpResponseMessage message) {
			};
			ODataClient client = new ODataClient(odataSettings);
			IEnumerable<sumDailyProduction> results = await client.For<sumDailyProduction>().Top(50).FindEntriesAsync();
			foreach (sumDailyProduction prod in results) {
			}
		} catch (Exception e) {
		}

I have a few issues.  First, my OData service uses tokens for basic authentication.  I could not find a way to submit a token using the ICredentials interface so I had to do a BeforeRequest.  Is that the right way?
Second, more important, is when I am trying to call FindEntriesAsync, simple odata client is putting a Prefer=return-no-content header so I get 0 results back.  I had to remove the Prefer header manually.  Am I doing something wrong?

ODataClientSettings has Credentials  property. Can you use that?
I confirmed that in some scenarious "Prefer" is incorrectly set. I will make a fix, thanks for reporting it.

I can't use the credentials for 2 reasons:

searching the source code, I can only see one place where the credentials are actually used and I think it was in something related to web API. if debug my app that provides credentials, it NEVER uses them and never hits the breakpoint in the code that actually uses them.
my Odata service uses an external service to perform login which gives me a token (Guid). I then must add that token to the Odata headers with the Authorization:Basic  format. since ICredentials is based on username/password, I can't use it. I need something more general like an IAuthorization interface where I can specify whatever I want to be added to the headers as Authorization:XXXXXXXXX

I see. I need to figure out how this can be solved.

But actually I don't see any better way than assigning AuthenticationHeaderValue in BeforeRequest. Anything else will be just syntactic sugar that will do the same. So yes you are doing it right.
And of course I will fix the other issue ("Prefer").

Fixed in 4.20.


WAMPv2 basic profile

It looks like this supports WAMPv1, which has been deprecated.
The spec for the new version can be found here. The WAMPv2 basic profile provides the same feature set as WAMPv1 plus a little. Just supporting this would probably be enough to make the library usable again.

Any update on this?

Latest commit was more than a year ago so I tried to make this compatible with wamp.2.json protocol. You can check out my fork and suggest improvements.
Disclaimer: I didn't even know that there was a language called dart until today. So my code might be messy.


react-instantsearch triggers unnecessary requests when everything is unmounted

Hello everybody,
When all react-instantsearch components/widgets are unmounted (for instance, in case of route change), many unnecessary search requests are triggered by react-instantsearch. The below fiddle triggers an unmounting of the whole tree below the InstantSearch component after 1s.
Steps:

Open https://jsfiddle.net/uza8mda0/1/
Open network tab, filter for XHR requests
Click play button, wait for >= 1s
Observe that 4 requests have been triggered in total, instead of one.

Bug: What is the current behavior? Unnecessary requests are triggered when everything is unmounted, even with cache enabled on the algolia client
Bug: What is the expected behavior? Unmounting everything doesn't trigger any request.
What project are you opening an issue for?

react-instantsearch

What is the version you are using? Always use the latest one before opening a bug issue. 3.2.1

Not sure if this can be of any use, but in the ongoing vue.js implementation we offer the ability to "freeze" the sync with Algolia so that multiple SearchParameters can be mutated without triggering a search query for each.
Also as long as the helper doesn't trigger a change event, we don't trigger new search queries.
i'd gladly discuss this specific point if needed.


Help with issue "#30269"

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Hi, I have very very little experience with coding (infact I just made a github account 10 min ago), I have had plenty of fun with ytdl, but I'm having an issue that I know you guys have solved, but I can't decipher how you solved it. It took me like 2 days to install ytdl, so I'm worried I'll break it, when all I want is to download is a silly video of that guy saying "wake up Joe Biden", but it's a "YouTube Short" so I can't.
So what I'm asking is, can someone please explain how you made it so you could download youtube shorts like you did in #30269?
Thank you so much. Sorry to bother.

The easiest way is to just switch to https://github.com/yt-dlp/yt-dlp. If you want you can read the differences in default behavior, but that probably won't be necessary for you, apart from moving your config file if you created one
Actually, just edit the url from https://youtube.com/shorts/XXXXXXX to https://youtube.com/watch?v=XXXXXXX

Or even easier, just remove https://youtube.com/shorts/ leaving only youtube-dl XXXXXXXXX

Oh yeah, I forgot about that :rofl:


Fixed spelling of "separator" in "lineSeperator"

This was bothering me, so I went ahead an fixed it. Glad to know this library was written by a human.
I understand the concern about introducing breaking changes (even though it's still pre-1.0) just for a spelling error, but I thought I might as well just leave this here in case you had any interest in doing so.
Anyways, thanks for the great library!

Hi, thanks for pointing this out! I had no idea I made a typo there.
I see you committed a bunch of other files as well in this pull request. I can't accept the pull request because of that. The pull-request should only include the relevant change to the /src folder.
I'm going to go ahead and fix the typo myself. I don't think this breaking change will have a major impact for many users, and I will therefore just make a mention for it in the changelog.
Thanks & cheers, Hein


NotFoundError: NewRandomAccessFile failed to Create/Open: images/labelmap.pbtxt : The system cannot find the file specified; No such file or directory

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): Install-TensorFlow-Python 3.7 file from github repository given by the tutorial below.
TensorFlow version: 2.3
Python version: 3.7
Installed using virtualenv? pip? conda?: pip
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 11.4
GPU model and memory:

Describe the problem
OK, so I followed this tutorial to the T: https://www.youtube.com/watch?v=a1br6gW-8Ss&t=157s. And then I get this random error. Everything was fine, I used labelImg, I generated the tfrecords, I downloaded all the software, I organized the files and extracted all the files. I'm using an up to date version of tensorflow. And the images/labelmap.pbtxt exists in the images folder (maniacal laughter). Why can't it find this file? I added a path, that still doesn't work.
Provide the exact sequence of commands / steps that you executed before running into the problem
python model_main_tf2.py --pipeline_config_path=training/ssd_efficientdet_d0_512x512_coco17_tpu-8.config --model_dir=training --alsologtostderr.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@TriangularFarOutPoint
please share the error in text so it will benefit any other user with similar error. [Also please upgrade your tf version to tf 2.4.1/2.5]
You may refer to these issues and let us know:link,link1


Bring role from fedcloud-catchall-ops repo

Summary

Initial commit, bringing content

Related issue :

I think it would be great to check how this follows/could follow https://github.com/EGI-Foundation/ansible-style-guide

i

I think it would be great to check how this follows/could follow https://github.com/EGI-Foundation/ansible-style-guide

it does miss testing and publishing to ansible-galaxy. Otherwise I think it complains with it

I think it would be great to check how this follows/could follow https://github.com/EGI-Foundation/ansible-style-guide

it does miss testing and publishing to ansible-galaxy. Otherwise I think it complains with it

So I was quite a lot thinking about the molecule-based testing: https://docs.egi.eu/ansible-style-guide/test

used admin powers to merge this one. It now also includes testing with molecule


Unknown CommandMessage

when play use vlc, got below warning message , and cannot play the video with vlc( always black screen)
Unknown CommandMessage 'getStreamLength', use default parser
Unknown message CommandMessage(8,0,0,37,20,0,37)
Unknown NetStreamCommand 'play', use default parser
Unknown message NetStreamCommand(8,0,0,35,20,1,35)
Unknown CommandMessage '3', use default parser
Unknown message UserControlMessage(2,1,1,10,4,0,10)

Currently, pyrtmp does not support the playback feature (because I didn't use it)
I don't have any plan to implement this feature in the near future.


[UWP] Fix build break problems for some VS users

Fixes #2922
Still not entirely sure why this is only broken for some folks. Possibly because we shouldn't have been using the UnionMetadata in the first place, but this seems to work properly.
Verified by building/running the visualizer
Microsoft Reviewers: Open in CodeFlow

If you could update the description to explain the change a bit more, that would be great.

If you could update the description to explain the change a bit more, that would be great.

done :)


[Feature][Manager] Add Cluster Management for MQ/DataProxy/Sort Cluster

Description
Add Cluster Management for MQ/DataProxy/Sort Cluster, and we could get the status of all modules quickly.
Use case
No response
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

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

Supported in #3945 and #3967.


fix(vehicle_cmd_gate): fix the problems while transition from/to stopped state

Description

This PR solves 2 different issue:

If the user sends stop command to autoware while driving, autoware does not do steering control while stopping: https://github.com/autowarefoundation/autoware.universe/pull/3653
Closes issue: https://github.com/autowarefoundation/autoware.universe/issues/4915

Related links

Tests performed
Tested on Psim, working as we want.

Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Hi @TakaHoribe, about the problem mentioned this issue, I moved the steering converge checking from controllers to vehicle_cmd_gate to solve this issue. If this method is okay, please let me know, I will create another PR to remove steering converge checking from controllers.

Hi @TakaHoribe @tkimura4, sorry to disturb you, but just I want to kindly ping you.

@kosuke55
Could you review this PR?

Hi @TakaHoribe, about the problem mentioned this https://github.com/autowarefoundation/autoware.universe/issues/4915, I moved the steering converge checking from controllers to vehicle_cmd_gate to solve this issue. If this method is okay, please let me know, I will create another PR to remove steering converge checking from controllers.

What do you think about this opinion?

@brkay54 Thank you for your PR.
It is reasonable not to use the stop steering command when the vehicle is trying to stop. However, I can't see the purpose of the move of the steering convergence check. What is the reason why you think the check should be in the gate?

@TakaHoribe, thank you for the reply.

However, I can't see the purpose of the move of the steering convergence check. What is the reason why you think the check should be in the gate?
The main cause is https://github.com/autowarefoundation/autoware.universe/issues/4915
Checking steering convergence in the controller creates unnecessary complexity. For this case:

pid_controller does not allow the vehicle to move because the steering is not converged.
vehicle_cmd_gate does not allow to control steering because the vehicle stopped.
These two nodes wait for themselves forever.

Because vehicle_cmd_gate checks conditions for start and stopping, I think it makes sense to move steering checking into vehicle_cmd_gate.

@brkay54 Thank you, I understood. It sounds reasonable. Would you reproduce the non-start issue in psim? Or does it happen only in the real vehicle testing?

Yes, I can reproduce in psim.
https://github.com/autowarefoundation/autoware.universe/assets/45468306/d457b2d3-c3cc-40bc-9f47-4781abdca430
Sorry for the screen capture, there is a problem on my computer when recording screen.

Vehicle can start at the beginning in auto mode.
I set steering about 20 deg with manual controller.
After I set 20 deg, when I clicked auto, vehicle can not move because of steering converging issue.
I set steering about 0 def with manual controller.
Vehicle can move in auto mode.

@TakaHoribe -san, I rebased on the current master and solved the conflicts. And tested again, worked as we wanted.
https://github.com/autowarefoundation/autoware.universe/assets/45468306/19a2484b-9342-430d-a1e1-9957f49bdf7e
Merge order:
1 - https://github.com/autowarefoundation/autoware.universe/pull/5183 and https://github.com/autowarefoundation/autoware_launch/pull/607
2 - https://github.com/autowarefoundation/autoware.universe/pull/5189 and https://github.com/autowarefoundation/autoware_launch/pull/616
3 - https://github.com/autowarefoundation/autoware.universe/pull/5203
Thank you for your effort, if you have any concerns please let me know.

@TakaHoribe -san, sorry to bother you, it is a friendly ping.

@brkay54 @TakaHoribe @rej55 I checked the code, there are 3 relevant modules, which are trajectory_follower, vehicle_cmd_gate, default_ad_api;

If the required steering angle is too large, the steering angle cannot be converged, the the target velocity of longitudinal velocity should be set as 0;
When the target velocity is 0, the vehicle_cmd_gate will output a state of is_start_requested=false;
If is_start_requested==false, the default_ad_api will change the state to PAUSED and keep calling the service /control/vehicle_cmd_gate/set_pause to set PAUSED state of vehicle_cmd_gate;
When the state of vehicle_cmd_gate is PAUSED, the vehicle_cmd_gate will keep sending stop control cmd which the steering angle is the current_steering_angle of the vehicle.

So the vehicle enters a dead loop and keeps PAUSED state.
I suggest create paused_control_cmd for PAUSED state in vehicle_cmd_gate that the steering_cmd is the command of trajectory_follower. When the steering angle is converged, the longitudinal controller can output normal control command.

Firstly, we need to avoid sending createStopControlCmd(), while the vehicle is stopping (not stopped yet in disengage state). Because it causes sending current_steering while the vehicle is decelerating. You can see the problem this issue.
Thank you for the suggestion @ahuazuipiaoliang! It makes sense to me, however, it will cause an undesired behavior. If we change both DISENGAGE and PAUSED states to send control_cmd's steering command, the steering will be controllable all the time. So, the vehicle won't be fully stopped, always it listens to the control_cmd's steering command. cc @rej55 @TakaHoribe

@brkay54 cc @TakaHoribe
Sorry for late reply, I agree for the suggestion from @ahuazuipiaoliang.
IMO, the trajectory_follower should send the control_cmd not to move the steering.
(@TakaHoribe said the same thing when I talked with him)

@rej55 Thanks, Fujiyama. Actually, I have modified the code in my way and tested on real vehicle for several days. I have not found any issue yet.

@rej55 Thanks for the reply, it makes sense to me. I am updating the PR.

@rej55 @ahuazuipiaoliang Sorry to disturb you guys, thank you for your replies!
To be able to not listen to the trajectory_follower's output when the vehicle fully stopped, I found another way. I changed the structure like this:
  // Check engage
  if (!is_engaged_) {
    filtered_commands.control.longitudinal = createLongitudinalStopControlCmd();
  }

  // Check pause. Place this check after all other checks as it needs the final output.
  adapi_pause_->update(filtered_commands.control);
  if (adapi_pause_->is_paused()) {
    if (is_engaged_) {
      filtered_commands.control.longitudinal = createLongitudinalStopControlCmd();
    } else {
      filtered_commands.control = createStopControlCmd();
    }
  }

In this way, the vehicle will be able to control steering if the vehicle is in PAUSED state. Also, it won't be able to control the steering when it is fully stopped and disengaged. I made some tests to check:
https://github.com/autowarefoundation/autoware.universe/assets/45468306/c0e0fb57-1a79-46a7-bdaa-7925b5ff2046
In the video above, I tested whether the vehicle can control the steering when we set a large steering value by using the manual controller or not. When I changed state to AUTO, the vehicle did not move until the steering converged as you can see.
In the second part, I set a large steering value again and I changed its state to STOP, as you can see, the vehicle did not change its steering. (It changed a little bit because the manual controller sets ENGAGE state to true while running and the steering was changing in the transition between ENGAGE to DISENGAGE)
Also, I checked the behavior of the vehicle while stopping. (It can control the steering or not while moving and disengaged) You can see the result in the video below:
https://github.com/autowarefoundation/autoware.universe/assets/45468306/0beac618-ceca-4ac2-8c2a-edec41320e86
As you can see, It still can control steering after I click the STOP button.
Also, I tried this structure too:
  // Check engage
  if (!is_engaged_) {
    filtered_commands.control.longitudinal = createLongitudinalStopControlCmd();
  }

  // Check pause. Place this check after all other checks as it needs the final output.
  adapi_pause_->update(filtered_commands.control);
  if (adapi_pause_->is_paused()) {
    filtered_commands.control.longitudinal = createLongitudinalStopControlCmd();
  }

It also solves the problems above, but, when the vehicle fully stopped, trajectory_follower can control the steering as you can see below:
https://github.com/autowarefoundation/autoware.universe/assets/45468306/c22c959c-fb50-4e8c-bc31-5894d6a5c1f6
Therefore I used the first structure. Please share your comments with me! Thank you for your efforts.

@tkimura4 @TakaHoribe We need code-owner approve. Could you check the PR?


Inspect query to adjust timerange

When doing a query like
select
transform.moving_average(foo, 5m)
from -1d to now

when the edge of availability of full resolution data is at -1d, since you'd fetch data beyond that point with the moving average, the query would suddenly error.
Now, by inspecting the query (using annotations on the registered functions) it's possible to automatically adjust the query's resolution, increased to (say) 5m, so that no error occurs.
It's intelligent enough to recognize that
select
transform.moving_average(foo, 5m) | transform.moving_average(12m)
from -1d to now

extends by a total of 17m, and that
select
transform.moving_average(foo, 5m) | transform.timeshift(-7m)
from -1d to now

doesn't extend it at all.
@drcapulet

Won't work on forecast.anomaly_* functions with extra time.


DO NOT MERGE: osd: activate - add --cluster support

As per https://github.com/ceph/ceph/pull/13566 we now need to expose the
--cluster argument when activating a device.
DO NOT MERGE: we need to wait for https://github.com/ceph/ceph/pull/13566 and its backport.
So CI failures are expected.
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

Closing this because of https://github.com/ceph/ceph/pull/13573


[Pipelines] add nif-version parameter

We need the nif-version parameter in pipelines as well. It determines the nif version that is submitted to e-Internationalization in the beginning and in the end of the pipeline. The nif-version parameter of individual pipeline requests is not influenced by the nif-version parameter of the pipeline. This will be a parameter similar to visibility or persist which gets his own field in the database. Currently it can values 2.0 and 2.1. This requires changes in

e-Internationalization itself
endpoint POST /pipelining/chain
endpoint POST /pipelining/templates
endpoint PUT /pipelining/templates/{id}
maybe GET /pipelineing/templates/{id}
respective API documentation

moved do #113


Fix session creation for auto-login when resetting password

Fixes so that a session cookie is created when web.changePassword.autoLogin is enabled and you're resetting the password for an account.
How to verify

Create a new application using this gist.
Start the application.
Navigate to the forgot password page (http://localhost:3000/forgot) and reset the password for one of your accounts.
Click the reset password link in the email you received and change the password for your account.
You should be redirected to / and logged in.

Fixes #164

Coverage decreased (-0.5%) to 70.974% when pulling 684a71ca38e57c5955129f19a64eef10599aac9b on fix-password-reset-autologin into f1abe55a25210598e30dde093ae8cdcca57371c6 on master.

I've verified that this PR fixes the reported problem, however I'm also noticing that the pre/post handlers are not being run for this case.  Can we refactor the code to handle those cases as well?  I know it may touch many files, but it seems like we can consolidate a lot of this.

Coverage decreased (-0.4%) to 71.15% when pulling 860a79daa02c63ee0812119f871e7e9b30b17175 on fix-password-reset-autologin into f1abe55a25210598e30dde093ae8cdcca57371c6 on master.

Coverage decreased (-0.4%) to 71.15% when pulling 860a79daa02c63ee0812119f871e7e9b30b17175 on fix-password-reset-autologin into f1abe55a25210598e30dde093ae8cdcca57371c6 on master.

@robertjd The latest changes are now pushed.

Coverage decreased (-0.4%) to 71.15% when pulling 7248ca7f22c2d4a82c374e540da254fcf20ee5c3 on fix-password-reset-autologin into f1abe55a25210598e30dde093ae8cdcca57371c6 on master.

Thanks @typerandom !  This looks great and I've verified that it solves the original issue.
I do have one issue with a test change, please see my inline comments.  In sum: we need to make sure that we have a test case that asserts the developer's ability to end the response from inside their handler, preventing the rest of the next() handlers from running.  I'd like to address this test issue before we merge.

@robertjd I saw your comment and I'll fix the code/test before we continue.

Coverage decreased (-0.9%) to 70.661% when pulling f7e07089422c4e6d0782d37c7f44bfa5c5190323 on fix-password-reset-autologin into f1abe55a25210598e30dde093ae8cdcca57371c6 on master.

@robertjd This is fixed now.


What are the CF-functionalities we need?

@rsignell-usgs mentioned in #10

The CF model itself actual consists of functionality that can be separated: unit conversion, vertical coordinate calculation, standard_name manipulation, handling of different common data model featureTypes (Grid, Point, TimeSeries, TimeSeriesProfile, Profile, Trajectory, TrajectoryProfile). Grid handles only data which is colocated with coordinate values.

I would like to start a discussion on how are we on each of these topics.  Maybe we should open separate issue for each.  My take is:

unit conversion: cf_units is out there and works.  It has all the UDUNITS idiosyncrasies but it is the standard adopt in CF and many other conventions.
vertical coordinate calculation: we have this functionality in iris and parts of it scattered among several ROMS modules out there.  I hope that odvc can tie all this together.
standard_name manipulation: what do we need here beyond compliance-check?  Iris ships the standard_name xml and uses it to create the cube.  I am unaware of any other package that uses it.
handling of different common data model: that is interesting.  If I extract a time-series from a model should that become a TimeSeries featureType automatically?  What would be the functionalities expected here?
grid: this goes beyond CF and the discussion has already started involving pyugrid and pysgrid.

unit conversion: cf_units- done
vertical coordinate calculation: odvc - will be done
standard_name manipulation: I don't think we need this... I don't want to be checking CF compliance... we can use compliance-checker for that: https://github.com/ioos/compliance-checker/.
handling of different common data model: - I think we will need an implementation of the CF DSGs in python, and it should be implemented in the same way pyugrid and pysgrid will be so the common API is consistent. I would love it if extracting a timeseries from a model produced a TimeSeries object I could then export to NetCDF (or just analyze locally in memory).

I guess a new issue for handling of different common data model is in order then.

in addition to the vertical coordinate caluclations we need to be able to support mapping to and from depth and sigmas which unfortunately requires a bathymetry database. In some cases some model outputs include the bathymetry used, but not most.

@lukecampbell, if a model is using coordinates with sigma, it is supposed to provide the depth, or else it's not CF-compliant.   I think we've agreed the API should allow the user to fix non-CF datasets by having a callback or similar functionality.   So you could define a function that CF-izes your dataset (or variable) by providing the bathymetry if it was not provided.    And if you wanted to estimate the model depths by reading from a bathymetry database you could do so.

Correct me if I'm mistaken here but currently unstructured grids are inherently non-CF compliant. At least 1.6. I'm pretty sure FVCOM uses sigma, do you know if it includes depth in the model run outputs?

Unstructured grids are inherently non-CF compliant.

Not at all!  One convention does convention does not exclude the other.  In fact they are additive.  The dataset below is Conventions: CF-1.0 and ugrid 0.9.  (Even though the metadata does not say anything about the latter.)

FVCOM uses sigma, do you know if it includes depth in the model run outputs?

Look for the variable h:
http://www.smast.umassd.edu:8080/thredds/dodsC/FVCOM/NECOFS/Forecasts/NECOFS_GOM3_FORECAST.nc.html

That's good to know. We had a similar project to this here and I remember we had some troubles dealing with models that used sigma because we couldn't get the actual depth. This makes me wonder what model we were using that didn't include depth.

Yoda would say:
   Valid your mode is not.
   Vertical coordinates without depth you cannot compute.
   Use the metadata Luke!

Oh, I know. My biggest issue is the general lack of metadata.

"""
       Valid your mode is not.
       Vertical coordinates without depth you cannot compute.
       Use the metadata Luke!
""".replace("Use the metadata Luke!", "Scream at the data provider Luke!")

That's at least half of my current job is working with providers to improve their metadata. We always have the same arguments though.  -- Provider.  "Because I legitimately have no idea what's needed here." -- Me.

I'm intrigued that @kwilcox (or anyone else) didn't mention Paegan here, or in #10 or #12. Admittedly, Paegan seems to be dormant (last activity was 1.5 years ago), but it did bill itself as "The Python CDM for Met/Ocean data", and it's still a/the backbone behind pyoos. At least for TimeSeries, I imagine there's a lot to be had or extracted-from there.
Nifty discussions you've got going here and in #10! Coincidentally I learned today about Unidata's "pyCDM" project (only as a brief reference in a recent Unidata meeting report, but couldn't find any actual repo or some such tangible, public evidence for it.

pyCDM sounds interesting.  I found this and the last bullet point mention that some code exist.  Hopefully that will be on GiHub at some point.

I'm betting @dopplershift or @lesserwhirls would be willing to fill us in on pyCDM thoughts and plans...

Well, not much to fill in. We'd like to get a lot of the useful functionality of the CDM into Python. Unfortunately, that's a big task with not a lot of resources for us to put into it. Us Python folks here have kept focused on much lower-hanging fruit.
Now if we have a critical mass of the community interested here, we'd of course be very interested in helping out....


change task list look and fix some glitchy behavior with screen resizing

This also resolves #86

LGTM! Adding timestamp for the task is really a good idea and replacing onclick with addEventListener in the script file is also great! But I have a question: Can you explain why we need to have all these e.stopPropagation();? I am a little bit confused about the use of these lines of code.

😄  - e.stopPropagation() basically stops the click event from bubbling up the parent divs. So when I click for example the Help Out button, it will trigger the helpOut() funtion but it won't trigger its parent on click function showTaskInfo() (it's considered a parent click event because the help-button is inside the task div that holds the showTaskInfo() click event).


From property in the  HomeController.inbox gives NULL error

The expected property 'From' could not be found while processing an entry. Check for null before accessing this property.
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.InvalidOperationException: The expected property 'From' could not be found while processing an entry. Check for null before accessing this property.
Source Error:
Line 149:            try
Line 150:            {
Line 151:                var mailResults = await client.Me.Messages
Line 152:                                    .OrderByDescending(m => m.ReceivedDateTime)
Line 153:                                    .Take(50)
Source File: C:\Users\Owner\Documents\Visual Studio 2015\dotnet-tutorial-master\dotnet-tutorial\Controllers\HomeController.cs    Line: 151
I've tried to test for null but can't get around this --- also, even if I change it to .Take(10) I don't get the "From" fields.
Vr, Tim:

This is fixed - apparently, I was working with two different "tutorials"; after copying these files and the other comment about NULL in the "from" it fixed the problem.
here is the code from the other issue in this project.
var lstMessages = await client.Me.Messages
.OrderByDescending(m => m.ReceivedDateTime)
.Take(10).ExecuteAsync();
        var mailResults = lstMessages.CurrentPage.Select(m => new Models.DisplayMessage(m.Subject, m.ReceivedDateTime, m.From));

        return View(mailResults);


chore: add bookmarklet generator source

In the future, the bookmarklet generator page should be updated from here vs in helix-pages, so changes that affect both the sidekick and this page can be released together.

:tada: This PR is included in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


inf-3481: Move AWS role to actions secret

Moving this role arn to a repo secret.

@MSAdministrator Requesting your review on this because you approved the last change to this code by our team.
Security would like this role to be moved into a repo secret. When looking at this I also noticed this workflow has failed the last few times it's run and wanted to make you aware of that.

@larntz Hey Luke, I cannot seem to access the URLs (within the README) from S3 anymore from my browser. Did you or anyone else change this behavior? Currently pyattck is broken because of this.

I did not make any changes to this bucket. I'll see what I can find out.

@larntz Thanks so much! Based on my limited knowledge (and based on when issues were filed) it may have happen around April 3rd or somewhere around there.

Thanks for the additional info.


Game won't start

Hello, I tried using the dll and it has even generated a successful log, but the game won't start (Windows 10). This game's community has been trying to fix the Windows 10 incompatibility for years, could you help us please?
It's Bomberman Online World, you can find it here http://fileforum.betanews.com/detail/Bomber-Online-World/1209068165/1
Thanks in advance!

This game is protected by some kind of anti-cheat system that prevents using a local copy of ddraw.dll (and probably other system files). Without a local copy of ddraw.dll, the game produces an error message that says "Can't set display mode". However, as soon as you try using a local copy - even if it's the original, unmodified system ddraw.dll file - the game will not even get this far and just silently exits.
I experimented briefly with the builtin Windows compatibility shims, but it appears the game is "protected" against those too...
Unfortunately, this means DDrawCompat will be useless here. Circumventing the anti-cheat protection of games is out of scope for this project, so I'll have to close this issue without a satisfactory resolution.

For what it's worth, I was able to hex-edit BomberMan.exe to use a 32 bit color mode instead of 16 bit and it's enough to get the game started, but the display will still be uselessly corrupted. This is kind of expected, because the game is still writing 16 bit color data into the 32 bit color buffer, resulting in the horizontal resolution "halving" and color corruption effects.
If you want to experiment with this yourself, these are the bytes I hex-edited in BomberMan.exe:
46 04 6A 00 6A 10 8B 08  52 8B 56 19 52 50 FF 51
Replace the value I highlighted (10) with 20.
In the current file version (<IP_ADDRESS>) this byte is located at offset 0xB355 (decimal 45909).
Do this only when the launcher is already open, because as soon as the launcher is started it "updates" BomberMan.exe and overwrites your changes.


fs/vfs/fd_open.c:  fs_fdopen() must not set errno

Summary
Functions within the OS must never set the errno value.  fs_fdopen() was setting the errno value.  Now, after some parameter changes, it reports errors via a negated errno integer return value as do most all other internal OS functions.
Impact
The per-thread errno will not be set by fs_fdopen.  This should correct the error noted in the dev list:  nx_start() logic calls group_setup_idlestreams() which, in turn, call fs_fdopen().  If fs_fdopen() fails it would set the errno value (first bug) and, if the the idle thread stack parameters are not initialized, it could also cause a NULL pointer derefence crash.
Testing
sim:nsh

Why this patch label with backport/9.0? Since the new set_errno doesn't(shouldn't too) cherry-pick to release/9.0, fd_open also don't need backport to 9.0 too.

Why this patch label with backport/9.0? Since the new set_errno doesn't(shouldn't too) cherry-pick to release/9.0, fd_open also don't need backport to 9.0 too.

I thought that the errno changes were in 9.0.  It is a bug in 9.0 too, but not critical enough to backport.


new(scap): Support no_events mode

What type of PR is this?

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

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

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

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf
/area libscap-engine-gvisor
/area libscap-engine-kmod
/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
This PR introduces a no_events mode for libscap (controlled by a new scap_open_args flag). In this mode, no events are generated from the engine. Opening an inspector not to get any events from it sounds pointless but (due to the current scap architecture) we need an engine to access e.g. /proc scan data (which may have different implementations, see e.g. gVisor).
The engine must also be aware that it's being opened in no_events mode, since otherwise it can do things that would break the original (main) inspector used to process events (again, e.g. gVisor opens a listening socket which would redirect all new connections to the new, presumably temporary, inspector).
So the end result is that we can use scap to run a /proc scan (or equivalent) without interfering with the main event loop.
Special notes for your reviewer:
In order to avoid adding a new parameter to all sinsp::open_* methods, I created a (C++) wrapper over scap_open_args with one (hopefully) final API change. Nothing changes if you use the default ppm_sc_set but otherwise it's the caller's responsibility to create a sinsp_driver_params struct (which is equivalent to scap_open_args).
I really wanted to use something more modern than a raw pointer but every approach I tried around references had worse (IMHO) downsides.
Does this PR introduce a user-facing change?:

cleanup(sinsp)!: all `sinsp::open_*` methods take an optional `sinsp_driver_params*` pointer which contains the ppm_sc_set that was previously explicitly passed (to some methods; now all open methods respect ppm_sc_set even if not all engines do)

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

Needs approval from an approver in each of these files:

OWNERS [gnosek]

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

@gnosek can we close this now?

Yeah, it's no longer needed. Thanks!


QAction::eventFilter: Ambiguous shortcut overload: Ctrl+W

From Cybjit on 2014-05-24T12:37:14Z
Spyder Version:  2.3.0rc (3680: r96ff231ec293 )
Python Version:  3.4.0
Qt Version    :  4.8.6, PyQt4 (API v2) 4.10.4 on Linux
IPython >=0.13  :  None (NOK)
pyflakes >=0.6.0:  0.7.3 (OK)
pep8 >=0.6      :  1.4.6 (OK)
pygments >=1.6  :  1.6 (OK)
sphinx >=0.6.6  :  1.2.2 (OK)
psutil >=0.3    :  1.2.1 (OK)
rope >=0.9.2    :  0.9.4-1 (OK)
matplotlib >=1.0:  None (NOK)
pylint >=0.25   :  None (NOK)
What steps will reproduce the problem?

Open a tab in Spyder
Press ctrl+w
'QAction::eventFilter: Ambiguous shortcut overload: Ctrl+W' is printed in terminal

What is the expected output? What do you see instead?
Closed tab. Nothing happens.
Please provide any additional information below
. 'Close file' is defined in two places. If I remove either, it works. https://bitbucket.org/spyder-ide/spyderlib/src/ee70372671f2a3d53a7be658bc99df03192b89d6/spyderlib/plugins/editor.py?at=v2.3.0rc#cl-624 https://bitbucket.org/spyder-ide/spyderlib/src/ee70372671f2a3d53a7be658bc99df03192b89d6/spyderlib/widgets/editor.py?at=v2.3.0rc#cl-685
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1828

From ccordoba12 on 2014-06-11T18:40:02Z
Status: Accepted
Labels: Cat-Editor MS-v2.3

From ccordoba12 on 2014-06-19T14:24:37Z
This issue was closed by revision 38d8adebeaeb .
Status: Fixed

From<EMAIL_ADDRESS>on 2014-07-01T21:56:07Z
Looks like wrong commit linked. I'd reopen this issue to find a better solution to have both case for Close File/Close Tab action.


Tab key indents selected block

Implement multi-line selection indention via tab key.

Tab - Indent block (Inserts spaces to align with absolute multiples of 2)

@PropGit do you have a list of items you want tackled as part of this? or bugs you have found with indent?

@phated Yes.
BUG Selecting a block of text and then pressing Tab replaces all of selection with "tabbed" whitespace.
Example
Selected text:

Tabbed result:

Expected result:


Test cleanup - should not include functional changes

What type of PR is this?
cleanup
Which issue does this PR fix:
What does this PR do / Why do we need it:
Standardizes on use of t.Run for parameterized tests, standardizes imports, standardizes naming of mocked resources. No functional changes. Small update to developer cheat sheet. Also removed lingering references to "mesh".
Testing done on this change:
?   	github.com/aws/aws-application-networking-k8s/pkg/apis/applicationnetworking/v1alpha1	[no test files]
ok  	github.com/aws/aws-application-networking-k8s/pkg/aws	0.441s	coverage: 24.2% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/aws/services	0.584s	coverage: 9.7% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/config	0.597s	coverage: 32.9% of statements
?   	github.com/aws/aws-application-networking-k8s/pkg/k8s	[no test files]
?   	github.com/aws/aws-application-networking-k8s/pkg/model/lattice	[no test files]
ok  	github.com/aws/aws-application-networking-k8s/pkg/deploy	0.854s	coverage: 19.4% of statements
?   	github.com/aws/aws-application-networking-k8s/pkg/runtime	[no test files]
?   	github.com/aws/aws-application-networking-k8s/pkg/utils	[no test files]
?   	github.com/aws/aws-application-networking-k8s/pkg/utils/gwlog	[no test files]
?   	github.com/aws/aws-application-networking-k8s/pkg/utils/retry	[no test files]
?   	github.com/aws/aws-application-networking-k8s/pkg/utils/ttime	[no test files]
ok  	github.com/aws/aws-application-networking-k8s/pkg/deploy/externaldns	1.314s	coverage: 75.6% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/deploy/lattice	1.153s	coverage: 84.7% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/gateway	1.615s	coverage: 77.7% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/latticestore	2.018s	coverage: 64.1% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/model/core	1.914s	coverage: 61.7% of statements
ok  	github.com/aws/aws-application-networking-k8s/pkg/model/core/graph	2.100s	coverage: 17.2% of statements
ok  	github.com/aws/aws-application-networking-k8s/controllers	0.638s	coverage: 0.0% of statements
ok  	github.com/aws/aws-application-networking-k8s/controllers/eventhandlers	0.963s	coverage: 53.5% of statements

Automation added to e2e:
n/a
Will this PR introduce any new dependencies?:
no
Will this break upgrades or downgrades. Has updating a running cluster been tested?:
n/a
Does this PR introduce any user-facing change?:
no
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.77%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/deploy/lattice/service_network_manager.go
2
3
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3825

Relevant Lines:
8357

💛  - Coveralls


Influence of Artificial Intelligence In Smart and  Intelligent Manufacturing

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of article
Influence of Artificial Intelligence in Smart and Intelligent Manufacturing
Introduction paragraph (2-3 paragraphs):
Artificial Intelligence is growing rapidly. Artificial intelligence refers to any device that takes actions or rather perceives its environment that aims at maximizing chances of success towards some goal. This can include and is not limited to the growth of technologies, such as rule-based systems and traditional logic, that enable computers to solve issues in ways that resemble thinking.
Smart manufacturing is collaborative manufacturing systems that respond to meet changing demands in smart factories and customer needs. Increasing automation, digital transformation have led to the full transformation of industries, manufacturing businesses, companies, and processes.
This article will discuss in detail the impact of Artificial Intelligence on smart manufacturing and intelligent manufacturing as well as its applications
Key takeaways:

Definition of Artificial Intelligence
Definition of smart manufacturing
Benefits of Intelligent manufacturing
Applications of Artificial Intelligence in intelligent  manufacturing

References:
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

@valerio140
👋  Sounds like a very interesting and useful topic - but it seems a bit broad(and almost out of our areas of focus).
We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
Is there anyway to fold this topic into a more "practitioner-focused" sense.
Perhaps focusing more on the applications being used for such a topic. How does tie back into the larger developer community.
A couple notes to think about 🚀
Thank you

@valerio140
Just pinging you on this topic form - as were making some room in the topic queue.
No rush if you're still working on this one - if you need more time we can always re-open at a later date.


phone number shown in rtl when choosing rtl languages

hi.
I noticed in the demo app that when choosing an rtl language the number format also gets turned to rtl. this makes displaying the number looks erroneous. here is an example:

the desired behavious is that the number layout remains ltr as this is the natural way of displaying the numbers. or so do I assume.

just ignore the mentioning of this issue on the pr. it is a mistake.


Launching via URI fails on "cold start"

With the app closed (not just in background, e.g. tap open apps button and swipe it away) and starting a LN payment through clicking a payment URL (e.g. by clicking the QR code here https://jochen-hoenicke.de/lightning/ ) and entering the unlock code quickly (or not really slow at least) Eclair fails with "None of your Lightning channels has enough balance to send this payment". Once the app is launched clicking the QR code works as expected and brings up the pay window.

The control checking the channels' availability fails because the channels have not established a connection with their peers yet, and as such there are no channels ready to pay. It is technically correct but obviously poor UX.
We can either:

have a softer control in the case of a cold start and take the chance to send the payment even if it could fail.
wait for the channels to go to NORMAL. I think this option would be worse UX-wise and could bring bugs.

Thanks

Maybe the payment window or the payment button could wait for the channels and display a message "waiting for channel connection" or something as long as things are not ready.

The pay button will now be disabled when the channels are offline which should solve most of the cold start failures. Thanks for the feedback.


feat(BottomSheet): (Android only) allows to interact with the screen behind the sheet

When you have a sheet open in the middle of the screen you cannot interact with the screen behind it, this PR adds that you can interact.
For now it only supports Android, I want to make it work for iOS too
old behavior:
https://user-images.githubusercontent.com/15719383/184112618-12c07bbb-1b10-418d-809c-f19a5f81a4bd.mp4
new behavior:
https://user-images.githubusercontent.com/15719383/184112497-45eca734-d8c3-4fa8-849f-a3bd73fce1cc.mp4

@vallemar very nice. Not sure you can do that on iOS though :s Do you want me to merge this now or do you want to add iOS if possible first?

@farfromrefug I wouldn't know what to tell you.  It is what you say, I have been trying to make it work on iOS but I have not succeeded.  Do you think it would be good to merge it and if later it is achieved by us or any other, will you add it?  I think I would merge it because the operation is quite useful

@vallemar agree, merged! Will make a release soon
Thanks for your contribution
PS: i might look at the iOS side one day when i have a project that needs it. The hard thing right now with iOS is that the native lib is not maintained anymore. I might fork and release my own at one point

@farfromrefug Thank you!  I've been seeing that on iOS you can also override the event of the view and pass it to the top view, tonight I'll try this, hopefully we could ignore the google plugin and do it via the generated views!  Then I will leave a summary of what I have achieved to have it somewhere!

I've tried using userInteractionEnabled= false but I haven't been able to get the event to pass to the post view. I have also tried hitTest but I have not been able to make it fire

@vallemar must be because it uses a presented view controller.

@vallemar https://stackoverflow.com/questions/34660095/pass-touches-through-a-uiviewcontroller#53922625

@vallemar Just built with your PR. What is touch_outside view? Also are you using vscode? There was typings error about coordinator not being an android.view.View. Could you ensure you see those PR? Also can you format code with eslint before commiting?

Hi @farfromrefug,touch_outside I imagine it is the part of the view where the sheet is not, I saw something about this in some pages. I'm using webstorm, ok I'll check with eslint and try again what you say about android.view.View, this worked without any problem in all my tests...

@farfromrefug touch_outside I didn't see it in NSCore, I saw it in android native side, I think it's the view above this view: https://github.com/nativescript-community/ui-material-components/blob/master/src/bottomsheet/bottomsheet.android.ts#L155

@vallemar ok thanks will look in material android code for it. thanks

Copying this info written by @ammarahm-ed in Discord, so it doesn't get lost. Might help with that functionality on iOS:

On iOS that will require using a regular UIView/Controller since something like a modal.doesn't allow passing touches through it.
In my RN sheet, i had to use simple View for it to work property. Inside a modal/presenter it will not work so the same might be true for bottom-sheet too.

@MrSnoozles  I have tried this in ios in all possible ways but without success, have you been able to do it?


diff_drive_controller

I decided to write code for my rover in ros 2 and apparently a diff drive controller hadn't been implemented yet so I decided to make one. This one is very basic and only does open loop control, but wouldn't be very hard to extend. I decided this was a good enough pull request to have some base code for a diff drive controller.
I'll be improving this controller as I get some odometry feedback and need to update odometry in my robot.
I wrote a test as well, but the test is very basic and mainly tests the core functionally and not much logic behind the twist messsages being sent.
I'm currently using ros dashing for my coding, so I couldn't use the code in the master branch.

Hope you don't mind if we don't get to merge this. @piraka9011 has his port almost ready but we would be glad if you could give us a review on that upcoming PR!

Yeah that's not a problem! If someone is already further ahead, then that's better. Could you link me to his branch? I'd like to use his diff drive controller if it's better. I'd actually prefer not to implement this, but was planning because I had to.

I think this is fine actually. I don't mind working on top of this/doing some refactoring to improve the code + tests.

Ok @piraka9011 please give this a review and let me know when it is ready for a merge.
Let's keep it snappy, I'd like to merge it asap and then you should move this branch over to master where we have all CI set up. It should involve more refactoring required by all the linters but it hsould be fine. Also a new package will be required. I'm happy to give you a priority review on that one

I'm currently building ros2 foxy. Pretty new to ros 2, so I didn't even know how to build to the latest until just now. I'm currently in the process of porting it to be compatible with the latest build according to this documentation: https://index.ros.org/doc/ros2/Installation/Foxy/Linux-Development-Setup/#linux-latest
I'm hoping master will build off that.

Just merged master into my diff drive branch and put the diff_drive_controller into its own package.
Need to do the following:

Apply @piraka9011 's change requests
Add a test to load the diff drive controller

Build failure looks like a downstream dependency issue. Taking a look...

Hey guys, just an update. I'd say the diff drive controller is 80% complete (building with foxy now):

Most of the original logic is implemented from the original diff drive controller
This needs some rigorous testing for bugs
Needs more unit and integration testing (odometry and speed limiter are not tested as of now)

It looks like the build is failing due to dependency on boost accumulators. It looks like ros2 is moving away from boost if possible? I can easily implement a rolling mean class, or find one.
Spend the last week or so implementing a sensor driver for an IMU I just got because a driver wasn't available outside of arduino usage, so didn't make any progress on this until recently.
I'll be testing this on my rover and seeing how it goes. I'll be adding in the ros2 navigation stack since I have my imu working. I just recently got a realsense t265 and d435 and the ros2 nodes kept crashing, so I might be focusing my contributions on that after I integrate navigation stack and finish up with this.

Let's not reimplement boost if possible. A lot of people (especially in corporate environments) weren't happy that ROS1 was exporting boost everywhere, ROS2 cut back on this. The only thing that changed is that now we have to properly include boost, it won't fall on our laps automagically.

Let's not reimplement boost if possible. A lot of people (especially in corporate environments) weren't happy that ROS1 was exporting boost everywhere, ROS2 cut back on this. The only thing that changed is that now we have to properly include boost, it won't fall on our laps automagically.

That's weird because it automatically linked for me on my machine. Not sure why. Although I am using many other packages rather than only building the ros2_controls repo

Thanks for the work on this, we'll now work on bringing this to master.


strange problem in parsing

I got two fresh problems :) this is the rss  I work with http://rssdeneme.appspot.com/rss/ACA7DD72D944  which contains this article http://rssdeneme.appspot.com/eksi/F4EDFE6191DE4C5
the problem is kindleear ignores some texts eg. http://i.imgur.com/EPmz8qM.jpg  there was a text below "2)". and the second problem is if there is a link inside this article, it goes to the next line before and after  http://i.imgur.com/Tz7J6Ax.jpg .is there anyway to solve this? or any work around

A module of KindleEar named 'Readability' did something wrong.
my suggestion is put your rss into a py file and set variable
fulltext_by_readability = False


Filter out more filename extensions for blackbox source headers

Since it's basically always bad for header files to be listed in the .f files, this makes filtering them out a bit more general.

Is there ever a reason to addResource("something.h")? Isn't that the only way they would end up in the .f?

I believe this was an issue that @abejgonzalez was currently fixing with grep. Maybe there's a better solution by just avoiding the situation as you say. Abe?

Yes. In Rocket Chip they add a .h for remote bitbang (https://github.com/chipsalliance/rocket-chip/blob/2eafc09ca30f932253c8d5828a0f34597fae0b92/src/main/scala/devices/debug/Periphery.scala#L197). I think the main reason they add this is to move the header to the build directory. However, this also adds it to the .f file which could cause issues.

I guess I should rephrase my question- is there a reason someone might want the header file in the .f? Is .f explicitly meant to be consumed by a tool like VCS, or is it an unregulated "list of files" ? It seems reasonable to me that firrtl would emit a list of all resources and expect you to process them into the files you actually want, but I could see the other way as well.

My assumption is that it is an unregulated list of files that need to be filtered/changed to be put in the simulation/CAD environment. However, I think one of the original authors of this pass should chime in.

Currently, .vh files are already filtered out under the assumption that anything that is a 'header' will be double-included if listed in the filelist. The idea here is that this assumption holds true for other kinds of headers (.h, .svh), so it just makes it a bit more consistent.


Person cannot log in

Bug description
Person cannot log in: https://telefoonboek.ugent.be/nl/people/002004407693
Information
Error received:

Person can be found in phone book: https://telefoonboek.ugent.be/nl/people/002004407693
Person is set to active in Biblio front-office: https://biblio.ugent.be/person/184A5CA0-8E27-11E9-8035-D3015707D3EF?format=json
"View as" works, and can be added as an author

@mietcls I cannot find this error message anywhere in code of biblio-backoffice. Are you sure that this user ends in biblio-backoffice, and not somewhere else (keycloak, or even the ugent login page)?

Confirmed, person can log in.


erlang: 20.0 -> 20.1

Add building of erlangR20 with odbc and javac support.
Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thank you for updating the pull request!
@LnL7 has deleted such attributes from all packages in #26797, and we won't reintroduce them without a reason. If you want to install one, you can use either beam.interpreters.erlangR20_odbc or (erlangR20.override { odbcSupport = true; }).
In order to merge this, there has to be one commit with the right message. You can edit the commits in the push request with git rebase -i master and then git push -f.

Ahh ok, I think there are a few documentation updates that I could update for this: nixpkgs/doc/languages-frameworks/beam.xml - https://nixos.org/nixpkgs/manual/#how-to-install-beam-packages
nix-env -f "<nixpkgs>" -qaP -A beamPackages doesn't work for me this works though: nix-env -f "<nixpkgs>" -qaP -A beam.interpreters
Should I try to update these too?

If you mean that the documentation does not mention that erlang is available with ODBC or without X and how to install them, then please update it. Otherwise it seems up to date. (It has been updated in #26668.)
beam.interpreters and beam.packages.erlang (aka beamPackages) are two different sets of packages. It seems that nix-env -f "<nixpkgs>" -qaP -A beamPackages must have been failing with error: hex-packages.nix:1550:9 called without required argument ‘redix_0_3_6’ since #16122. This is quite unfortunate. @gleber, could you help fix this?

beamPackages unfortunately rotted over time, since hex2nix mechanism is so hard to operate currently.
Plan to fix this is in #11748 and this issue has been reported in #26279. The only thing I can do quickly is drop most of beamPackages, since AFAIK they are not currently used by anyone.

Thank you, I'm deleting transitively unbuildable packages in #31383.

Sounds good!
On 8 Nov 2017 08:29, "Orivej Desh"<EMAIL_ADDRESS>wrote:

Thank you, I'm deleting transitively unbuildable packages in #31383
https://github.com/NixOS/nixpkgs/pull/31383.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/31380#issuecomment-342733505, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACBoeErja3oIko0WqG1liXLy0WW74zEks5s0VhRgaJpZM4QVy4Y
.


Documentation for "Async function call" misformatted

See: https://www.jspsych.org/plugins/jspsych-call-function/

This is fixed and will be live after the next docs update.


Implement replace() method to move out of struct/enum

Replacing a pinned value is safe as long as the destructors of types which do not implement Unpin are called.
Pin::set allows replacing the whole value, but it does not allow recovering the un-pinned fields, so the idea is to provide a method to achieve this.
The API would look something like:
#[pin_project]
enum Example {
    A(#[pin] PinnedValue),
    B(UnpinnedValue),
    C,
}

...

#[project]
match self.replace_and_project(Example::C) {
    Example::A() => { /* no data */ },
    Example::B(value) => { /* oh look a value! */ },
    Example::C => {},
}

I was working on something very similar to this, but never quite finished. I can finish it up and open a PR.

It seems that this can be replaced by wrapping the unpinned field with an Option.
#[pin_project]
enum Example {
    A(#[pin] PinnedValue),
    B(Option<UnpinnedValue>),
    C,
}

#[project]
match self.as_mut().project() {
    Example::A(_) => { /* ... */ },
    Example::B(value @ Some(_)) => {
        let value = value.take().unwrap();
        // ...
    },
    Example::B(None) | Example::C => {},
}
self.set(Example::C); // When using this way, whether you should replace self depends on the situation.

It seems reasonable to add this, given that it increases the size if use Option and that we cannot easily change the public type signature.

Assigning to @Aaron1011 based on the previous comment (feel free to unassign).

IIUC, macro needs to generate an additional struct/enum to support this.
@Aaron1011: Do you think it preferable to add a feature like #124 to implement this? (I feel the existing project-attribute-based approach becomes bothered as more types are generated.)

In order to be panic-safe, it will need to generate code something like:
struct DropInPlaceHelper<T>(*mut T);

impl<T> DropInPlaceHelper<T> {
    fn drop(&mut self) {
        ptr::drop_in_place(self.0);
    }
}

struct OverwriteHelper<T>(*mut T, MaybeUninit<T>);

impl<T> OverwriteHelper<T> {
    fn drop(&mut self) {
        ptr::write(self.0, self.1.assume_init());
    }
}

fn partial_replace(ptr: *mut Foo, new_value: Foo) -> Y {
    let _helper0 = OverwriteHelper(ptr, MaybeUninit::new(new_value));
    let _helper1 = DropInPlaceHelper(&mut (*ptr).pinned_field1);
    let _helper2 = DropInPlaceHelper(&mut (*ptr).pinned_field2);
    let _helper3 = DropInPlaceHelper(&mut (*ptr).pinned_field3);
    ptr::read(&mut (*ptr).unpinned_field)
}

(Maybe we need to wait for https://github.com/rust-lang/unsafe-code-guidelines/issues/232's discussion?)


searching an element

program releated to searching an element

Hey Pal,
Thankyou for contributing , but code already exist.
Happy Coding!


docs: update aligned layer documentation

[x] The FAQ was added to the documentation.
[x] The list of learning resources was updated.

Ready. It has been modified, @JuArce


feat(renovate): setup for deps management

added .idea to gitignore, renovate setup
closes #2
@denis-markushin take a look, please

@l3r8yJ conflicts are found. Fix it, pls.

@denis-markushin resolved


Logstash jdbc input Rufus::Scheduler::CronJob error

Problem description
Hello everyone, I have an issue when launch logstash input jdbc schedule on docker. In fact, the configuration (include schedule) works totally fine when i installed it in local (by using tar package). But when i use docker elk, i got the error rufus-scheduler intercepted an error 2032 IOError (log bellow).
I think the problem is from schedule because when i remove it, there aren't no error.
My logstash config file like following:
Extra information
Error log:
logstash_1       |   2166   job:
logstash_1       |   2166     Rufus::Scheduler::CronJob "* * * * * *" {}
logstash_1       |   2166   error:
logstash_1       |   2166     2166
logstash_1       |   2166     IOError
logstash_1       |   2166     Read-only file system
logstash_1       |   2166       org/jruby/RubyIO.java:1234:in `sysopen'
logstash_1       |   2166       org/jruby/RubyIO.java:3795:in `write'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.16/lib/logstash/plugin_mixins/jdbc/value_tracking.rb:122:in `write'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.16/lib/logstash/plugin_mixins/jdbc/value_tracking.rb:46:in `write'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.16/lib/logstash/inputs/jdbc.rb:318:in `execute_query'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/logstash-input-jdbc-4.3.16/lib/logstash/inputs/jdbc.rb:276:in `block in run'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:234:in `do_call'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:258:in `do_trigger'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:300:in `block in start_work_thread'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:299:in `block in start_work_thread'
logstash_1       |   2166       org/jruby/RubyKernel.java:1425:in `loop'
logstash_1       |   2166       /usr/share/logstash/vendor/bundle/jruby/2.5.0/gems/rufus-scheduler-3.0.9/lib/rufus/scheduler/jobs.rb:289:in `block in start_work_thread'
logstash_1       |   2166   tz:
logstash_1       |   2166     ENV['TZ']:
logstash_1       |   2166     Time.now: 2019-12-23 21:10:39 UTC
logstash_1       |   2166   scheduler:
logstash_1       |   2166     object_id: 2016
logstash_1       |   2166     opts:
logstash_1       |   2166       {:max_work_threads=>1}
logstash_1       |   2166       frequency: 0.3
logstash_1       |   2166       scheduler_lock: #<Rufus::Scheduler::NullLock:0x1367c038>
logstash_1       |   2166       trigger_lock: #<Rufus::Scheduler::NullLock:0x2e8a3683>
logstash_1       |   2166     uptime: 88.81754199999999 (1m28s818)
logstash_1       |   2166     down?: false
logstash_1       |   2166     threads: 2
logstash_1       |   2166       thread: #<Thread:0x6abbb5ad>
logstash_1       |   2166       thread_key: rufus_scheduler_2016
logstash_1       |   2166       work_threads: 1
logstash_1       |   2166         active: 1
logstash_1       |   2166         vacant: 0
logstash_1       |   2166         max_work_threads: 1
logstash_1       |   2166       mutexes: {}
logstash_1       |   2166     jobs: 1
logstash_1       |   2166       at_jobs: 0
logstash_1       |   2166       in_jobs: 0
logstash_1       |   2166       every_jobs: 0
logstash_1       |   2166       interval_jobs: 0
logstash_1       |   2166       cron_jobs: 1
logstash_1       |   2166     running_jobs: 1
logstash_1       |   2166     work_queue: 0
logstash_1       | } 2166 .

Stack configuration

Docker setup
docker --version
Docker version 19.03.5, build 633a0ea838

docker-compose version
docker-compose version 1.23.2, build unknown

Docker logs
My logstash config file
input {
    jdbc {
        jdbc_connection_string => "jdbc:postgresql://${POSTGRES_HOST}:5432/jmdb"
        jdbc_driver_library => "/home/tnvan/postgresql-42.2.8.jar"
        jdbc_driver_class => "org.postgresql.Driver"

        jdbc_user => "${JDBC_USER}"
        jdbc_password => "${JDBC_PASSWORD}"

        jdbc_paging_enabled => true

        use_column_value => true
        tracking_column_type => "numeric"
        tracking_column => "decision_id"
        last_run_metadata_path => "/usr/share/logstash/config/sql_last_value.yml"

        statement => "select * from decision d"

        type => "decision"
        schedule => "* * * * *"
    }
}

"Read-only file system" suggests that the plugin is trying to write in a location where it's not allowed to, most likely in a volume mounted from the host as "read-only". Please check your Compose file for such volume.

@NhatThien is that issue resolved?

That issue is resolved by deleting read_only in logstash volumn in docker-compose.yml file. Thank you a lot.