"Considerations for optimizing client-side data access" contains poor guidance

This section should not be recommending developers attempt to implement HTTP caching as part of their client applications.  Clients should enable existing caching solutions like WinInet caching or other alternatives.  Implementing correct HTTP caching is a non-trivial task.

I am not sure that this section actually recommends to developers that they should implement client-side caching. This material is more about implementing servers that support client-side caching (whether these requests come directly from a client or an intermediate server). This is an important consideration.

Re-reading this I see what you mean.  There are a number of uses of "should" that I read as suggesting to the reader that they would be the ones implementing this behaviour.  However, I suspect the word was intended to infer that the behaviour might happen.
e.g. "If the data was changed by another user changed (status code 412) or not found (status code 404) then the cached object should be discarded"
Interestingly, the HTTP caching spec has nothing to say about how invalidation occurs with non 2XX response codes, so the actual behaviour is completely implementation dependent. https://tools.ietf.org/html/rfc7234#section-4.4


While trying to preview the generated Travel v4 odata service an error appears on the browser

After generating the RAP artifacts in the eclipse ABAP cloud project and publishing ODATA V4/V2 services when you click the preview for Travel v4 odata service, the Fiori app preview is supposed to load but actually, the following error appears on the browser:
Application could not be started due to technical issues.Prop. 'DISTANCE' has unit amount conver. and EDM type 'Int32' without decimals
More information and possible ideas on how to fix in the following thread:
https://answers.sap.com/questions/13750416/sap-abap-cloud-trial.html

This will be fixed by fixing the root cause in the /DMO/ travel sample objects. Here the data element for the distance field has to be changed from INT32 to DECIMALS(12,2).


Chrome shows violates Content Security Policy directive

waypoint-say-hello-to-html-elements:1 Refused to load the script 'data:application/javascript;base64,KGZ1bmN0aW9uKCkgewogICAgLy8gaHR0cHM6Ly9k…ogICAgZ2EucmVtb3ZlID0gbm9vcGZuOwogICAgd2luZG93W2dhTmFtZV0gPSBnYTsKfSkoKTs=' because it violates the following Content Security Policy directive: "script-src 'unsafe-eval' 'unsafe-inline' .google-analytics.com .gstatic.com https://*.cloudflare.com .cloudflare.com https://.gitter.im https://*.cdnjs.com .cdnjs.com https://.jsdelivr.com .jsdelivr.com .twimg.com https://*.twimg.com 'self'".

Reference : https://developer.chrome.com/extensions/contentSecurityPolicy

@vicky002  thanks for posting this issue!
Seems you have a similar issue as in https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5999 so let's keep the discussion there, thanks.

Please review the Guidelines for Contributing, thank you!.

@vicky002 Where are you located?

Mumbai.


Update guarded-fabric-manage-hgs.md

Fix copy/paste error

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

LGTM


Add dict.clear and bytes.clear

Codecov Report

Merging #494 into master will decrease coverage by 0.38%.
The diff coverage is 63.15%.

@@            Coverage Diff             @@
##           master     #494      +/-   ##
==========================================
- Coverage   57.38%   56.99%   -0.39%
==========================================
  Files          67       67
  Lines       12300    12376      +76
  Branches     2906     2935      +29
==========================================
- Hits         7058     7054       -4
- Misses       3457     3534      +77
- Partials     1785     1788       +3

Impacted Files
Coverage Δ

vm/src/obj/objbytearray.rs
46.7% <61.53%> (-19.29%)
:arrow_down:

vm/src/obj/objdict.rs
57.26% <66.66%> (-6.68%)
:arrow_down:

Continue to review full report at Codecov.

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


As a convenience, send google the 'hd' parameter representing which domain to restrict sign-in to

Only set when requiredDomain is configured.
NB: As per official documentation from Google, you still need to check that the returned user
is indeed belonging to the requiredDomain which is code already present.

This should address #37


how can i mask a array

Environment info
Operating System: centos
Compiler:
Package used (Python/R/Scala/Julia):
MXNet version: 0.9.3
the newest version gives:
element_mask is removed. Please use src*mask.reshape((mask.size, 1, 1, ..., 1)) directly as binary ops now support broadcasting.
how can i realize the function?
for example, I have a matrix A, shape(32,256) and a mask maxtrix, shape(32,)
how can i do the broadcast_mul function?
I try:
A = mx.sym.broadcast_mul(A, mx.sym.Reshape(data=mask, shape=(1, 256)))
but error

why remove element_mask but not giving a new example !!!

src*mask.reshape((mask.size, 1, 1, ..., 1))
This saids you should reshape((32,1))

thanks, i got it.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Integration Test - Release - 2018/07/11 17:28:02

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegartionTestFiles/ChangeSentToAuthor.md
:white_check_mark:Succeeded

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] v0.6 snapshot

Notes for Reviewers
This PR takes a snapshot in time as a copy of the documentation site at the end of the v0.6.x release cycle.
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit d7b2052d21d51ef3fb0684175d5d3d59d65f84d1 at: https://653b27672a07ac7806509ef5--meshery-docs-preview.netlify.app

⚠ The snapshot selector (e.g v0.6) is not remember across page-to-page navigation.

🚀 Preview for commit 327a215adf45288d5cd53ddf8e453b48c42316b3 at: https://653b2df9783fbf7e06571df1--meshery-docs-preview.netlify.app

🚀 Preview for commit cfcbe9e7a66e376bfc3526e4fe6f0f19331f3f39 at: https://653bff238e2f056a7d4e9277--meshery-docs-preview.netlify.app


Different name for multiple composed mutations?

I want each of my composed mutations to have a different name.
mutationState({name: "mutation"})(compose(
  graphql(...),
  graphql(...),
  graphql(...),
)(MyComponent));

Maybe the "name" option could be a function that resolves to a string?
Or, better yet, use the name configured in each graphql(..., {name: 'myMutation2'}).

@thealjey This HOC currently only supports a single mutation object.
You can try this:
mutationState({ name: 'mutation1' })(
  mutationState({ name: 'mutation2' })(
    mutationState({ name: 'mutation3' })(
      compose(
        graphql(...),
        graphql(...),
        graphql(...),
      )(MyComponent),
    ),
  ),
)


Add support for Hugo's image bundling

Currently Academic has good support for associating featured images with content in the frontmatter. However, images have to located in the static/img folder.
But Hugo 0.32 gives us Page Bundling! And I already use this approach to structure my content.
Is the theme going to be enhanced and brought in line with the Page Bundling changes? That is, can featured images be co-located in the content folder?
In the meantime, is there a efficient 'workaround'  to achieve this without resorting to permalinks or changing all the templates?

A use-case would be to obtain a DOI for each blog post.
You can get a DOI for a GitHub repository through zenodo.org:
https://guides.github.com/activities/citable-code/
It would greatly simplify the process if each post could be included as a git submodule with all content self-contained.

What to do if I have a BaseURL like this:
https://users.itk.ppke.hu/~botcs/about/
Sadly all the static/img stuff is linked to only https://users.itk.ppke.hu/ instead of https://users.itk.ppke.hu/~botcs/about/ which is really annoying - and I have no clue where to change and what

@gcushen
I have implemented page bundling for my site. It does not change anything visually, but really helps with content organization.
Changes are the following:

Content of leaf pages moved from page.md to  page/index.md
In index.md instead of
image_preview = "img.png"
I use:

[[resources]]
  name = "image_preview"
  src = "images/img.png"

img.png goes to content/page/images/ instead of static/
in related layout files instead of:

          {{ with $project.Params.image_preview }}

          {{ end }}

I use

             {{ with $project.Resources.GetMatch "image_preview" }}
                {{ $img_res := .Resize "250x q90" }}

             {{ end }}

Benefits:

All images belong to their parent project, no more preview_for_project_1.png in static
Automatic resizing if needed for a particular layout/widget
Easier extendability, we can have different types previews, for example for  #688

Drawbacks:

My code breaks compatibility. I can fix it, but it will be an if-else mess.

Are you interested in a PR?

Yes, please submit a PR. Thanks!

Okay, somewhere next week.
On Fri, Sep 14, 2018, 3:29 PM gcushen<EMAIL_ADDRESS>wrote:

Yes, please submit a PR. Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gcushen/hugo-academic/issues/440#issuecomment-421461283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUUjj51qSH485Ze2pITdjak2dRfAgcOks5ubAO0gaJpZM4RWoN3
.

See #708


How to import flow types from react?

I can't for the life of me figure out how to import a type from React -- @types/react/index.d.ts
I think the namespace is what's throwing me off, and I can't find documentation anywhere on how to import with namespaces.
I'm looking for something along the lines of:
import type { ErrorInfo } from 'react';

Where ErrorInfo is located in
export = React;
export as namespace React;
declare namespace React {
  interface ErrorInfo {
    componentStack: string;
  }
}

How do I use this ErrorInfo definition in my file?

I think you're in the wrong repo. That's a typescript definition! 😮

Is that not where Flow pulls its types? I've been doing import type { TypeName } from 'package' with success for all things outside of React. I guess the TypeScript definitions have all been the same as the Flow definitions for every other use case.
How do I get the Flow type in this case?

React types come from the core flow libs. I think you need to wait for a Flow update.

Definitions of react are kept here
https://github.com/facebook/flow/blob/master/lib/react.js
This probably can be closed.
/cc @vkurchatkin


Logging for refresh

The first commit adds a debug log for refresh() occurring, which is a very good idea.
The second should never be merged and is mostly for debugging a case where I think refresh is firing.

(Before actually merging, I'll remove commit 2)

first PR LGTM

Dropped the last commit, good to merge now

@baude @jwhonce @vrothberg @TomSweeneyRedHat PTAL

/lgtm


Fix memcpy of garlicat to actually be garlicat

Unsure if this code ever properly worked since it's copying the wrong magic bytes.
Description
The garlicat magic bytes have been incorrect since original import.
Related Issue
No related issue, however if this isn't considered important it might be worth just removing the excess code.
Motivation and Context
This code cannot properly work as designed with the current code.
How Has This Been Tested?
Just code analysis.
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

interesting. peer finding works over tor, but this is definitely a correct pr.

nice catch Tim


Subset Sum (dp)(#114), Binary Search on Rotated Arrray (#112) using C++

Please let me know if any changes are required.

I have changed the folder structures .


sort endpoints list by id

In the endpoint list view, sort them by id

will be implemented at version 3.0


Hi Team,
I would like to http://filmey.tv/ into youtubedl
I have latest version installed on my PC
youtube-dl==2016.12.20
got below error:
DownloadError: ERROR: Unsupported URL: http://filmey.tv/مشاهدة-فيلم-Rogue-One-A-Star-Wars-Story-2016-مترجم-أون-لاين-artid44987.html
Thank you in advance
AGR
Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.12.22. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2016.12.22

Before submitting an issue make sure you have:

[ ] At least skimmed through README and most notably FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.12.22
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: http://filmey.tv/مشاهدة-فيلم-Rogue-One-A-Star-Wars-Story-2016-مترجم-أون-لاين-artid44987.html

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

From what I can see most of the content in the site are copyrighted, youtube-dl does not support sites dedicated to copyright infringement.


Weaviate Document Store

Summary and motivation
We want to support Weaviate as a Document Store in Haystack 2.x, much like we did for Haystack 1.x
Checklist
If the request is accepted, ensure the following checklist is complete before closing this issue.

[x] The code was merged in the main branch
[x] Docs are published at https://docs.haystack.deepset.ai/
[x] There is a Github workflow running the tests for the integration nightly and at every PR
[x] A label named like integration:<your integration name> has been added to this repo
[x] The labeler.yml file has been updated
[x] The package has been released on PyPI
[x] An integration tile has been added to https://github.com/deepset-ai/haystack-integrations
[x] The integration has been listed in the Inventory section of this repo README
[x] There is an example available to demonstrate the feature
[x] The feature was announced through social media

Integration has been announced on socials, everything is done.


Update ElasticSearch when a task is CANCELED

When a workflow is TERMINATED, all tasks are added to ElasticSearch in their current state, even if they are in SCHEDULED state because they have not yet been polled. After that, SCHEDULED tasks have their status changed to CANCELED and the database is updated. But ElasticSearch is not updated (indexDAO.indexTask() is never called), so the database is out of sync with ElasticSearch. This change will index the task after the database is updated so that both are in sync.

@rickfish Thank you very much for the contribution.
The task document is indexed in Elasticsearch only upon the completion of the workflow - https://github.com/Netflix/conductor/blob/master/core/src/main/java/com/netflix/conductor/core/orchestration/ExecutionDAOFacade.java#L218. So the task document would be created with CANCELED status in ES once the workflow reaches a terminal state.

When a workflow is TERMINATED, all tasks are added to ElasticSearch in their current state, even if they are in SCHEDULED state because they have not yet been polled. After that, SCHEDULED tasks have their status changed to CANCELED and the database is updated.

This seems odd, when a workflow is TERMINATED, all tasks within the workflow should also be in a terminal state. All the SCHEDULED state tasks should be marked as CANCELED at this stage.

This seems odd, when a workflow is TERMINATED, all tasks within the workflow should also be in a terminal state. All the SCHEDULED state tasks should be marked as CANCELED at this stage.

They are CANCELED but only in the database.
The problem lies in WorkflowExecutor.terminateWorkflow (https://github.com/Netflix/conductor/blob/bd32fe164020781468b70f44e7dc76dc24714d49/core/src/main/java/com/netflix/conductor/core/execution/WorkflowExecutor.java#L717).
The use case I used to reproduce this is starting a workflow that has a fork that has on both sides a SIMPLE task where each is SCHEDULED (so not yet polled). Then terminate the workflow using DELETE /workflow/. The terminateWorkflow() method referenced above is then called. You can see that when executionDaoFacade.updateWorkflow() is called in terminateWorkflow() all the SCHEDULED tasks (which are not usually indexed into ES) are indexed, When that call returns, the SCHEDULED tasks are CANCELED. If you follow the link above, you will see that the task status is set to CANCELED and updateTask() is called but there is no call to indexDAO.indexTask(). I'm sure that this is a strange use case that isn't usually executed. One of our users is causing this use case several times a day; that is how I noticed it.

Thanks for the context. This is a valid bug and is indeed an edge-case. Looking at the linked code, I feel that it would be easier to move executionDAOFacade.updateWorkflow(workflow);  after line 721. This would prevent having to add a dependency to IndexDAO in the WorkflowExecutor. Would need to ensure that the workflow object is updated with the updated tasks (new status).
Also, we are looking into indexing task documents during updateTask to enable task level searching for in progress workflows. Once this is implemented, it would solve for this issue as well. Thoughts?

To me it looks like either of your solutions is better than doing a one-off index of the task when needed as I did but I admit that I don't know the reasoning behind not putting the indexDAO.indexTask() call in updateTask() in the first place. Can you explain? I'm sure there is a reason and I don't want to start changing code that is integral to the functioning of Conductor unless I understand all the implications.
The problem with not calling indexDAO.indexTask() during the updateTask() method from my perspective is the /tasks/search endpoint. If you use that endpoint and your freeText is, for example, status:CANCELED then you get inaccurate results because that endpoint does not find the CANCELED tasks as ElasticSearch is used to do the initial query. I think the same issue surfaces for other statuses but I haven't yet looked into it further.

I think the reason that updateWorkflow() is called before the updating of the tasks (during terminateWorkflow()) is because we don't want to have a task update exception thwart the updating of the workflow. Please correct me if you disagree. I therefore like your second solution of adding a call to indexDAO.indexTask() in the updateTask() method. That will create a ton more volume for ElasticSearch but I would rather have both ElasticSearch and the database in sync with more volume than worry about volume.
The question is: are there situations where adding something to ElasticSearch (where it currently is not added) would cause a problem for another use case? I don't know the answer to that...

Are there situations where adding something to ElasticSearch (where it currently is not added) would cause a problem for another use case?

Your observation is correct that this will create a large volume of requests to ES and in case of synchronous indexing, this can introduce a latency cost. However, I agree that it is better to have both Elasticsearch and primary datastore in sync in such situations.

I therefore like your second solution of adding a call to indexDAO.indexTask() in the updateTask() method.

This change can be added such that it is performed only in cases where async indexing is not being used. Otherwise with the current in-memory queue async indexing model, there is a high possibility of indexing requests being dropped.

This change can be added such that it is performed only in cases where async indexing is not being used. Otherwise with the current in-memory queue async indexing model, there is a high possibility of indexing requests being dropped.

So, if I add this code to ExecutionDAOFacade.updateTask(Task task):
if(!config.enableAsyncIndexing()) {      indexDAO.indexTask(task); }
then I would need to bracket every existing call to indexDAO.indexTask() with this:
if(config.enableAsyncIndexing()) {      indexDAO.indexTask(task); }
Is that acceptable to you? It could get messy I am thinking.

I am going to make the changes to index all task updates if async indexing is not enabled but I will put it in another pull request as I created a different branch (index-all-task-updates). Sorry if this is a problem. The new pull request is #1526. I will close this one.

@rickfish - Is this issue fixed? I don't see relevant code changes in the new pull request is #1526.

@techyragu sorry, the pull request was #1546, not 1526.


fix: Hindi text

🔎 Overview
Fixed text information for Hindi, thanks!

Thanks for the fix, will do a release shortly.


How to get IDF coefficients after using method turicreate.text_analytics.tf_idf() ?

Once, in graphlab, I did such thing like that:
sf = graphlab.SFrame({'text': ['democratic governments control law in response to popular act']})
sf['word_count'] = graphlab.text_analytics.count_words(sf['text'])

encoder = graphlab.feature_engineering.TFIDF(features=['word_count'], output_column_prefix='tf_idf')
encoder.fit(wiki)
sf = encoder.transform(sf)
sf

but in turicreate , there are no feature_engineering module.
So, after extracting tf_idf of a SFrame, how to get the tf_idf of a new article?(IDF coefficients are needed).
thanks in advance
Any info is appreciated.

We could make the IDF coefficients as an optional return for the function.


first draft of spouts 2.0 changes

I'll firm up the issues in here right after the holidays.  The details I'm aware of are in the To Do section.

Closing this - All of Jk's suggestions have been added to PR #5563

Closing this - All of Jk's suggestions have been added to PR #5563


Proposed addition of timestamps to PSA

Instead of the following proposed conversion extern method:
    void timestamp_to_nanoseconds(in timestamp_t t,
                                  out bit<64> seconds,
                                  out bit<30> nanoseconds);

it is likely easier to implement this one instead:
    void timestamp_to_nanoseconds(in timestamp_t t,
                                  out bit<80> nanoseconds);

where the output value 'nanoseconds' is the total number of nanoseconds elapsed since Jan 1, 1970.  Any implementation of the first method above would probably require first going through the calculation of required for the second method, then separating the total nanoseconds into whole seconds and nanoseconds in the range [0, 10^9-1], which requires additional arithmetic.

@jafingerhut ,
I would suggest replacing "Jan 1, 1970" with "the time when the timestamp was equal to 0". This will be mush more reasonable to implement (although still quite difficult).
In reality, though, I suspect that aside from the application layer protocols that really need time-of-day based values (but usually are not the ones handled by P4 programs) the scaling will be done at the control plane level. For example, to calculate the timeout, a difference timestamp2-timestamp1 will be compared with the number calculated as timeout_in_ns * NUMBER_OF_TICKS_IN_NS

@vgurevich Suggestion noted.  Doing only part of a conversion to some common external reference, but not all of it, doesn't seem like very much of a benefit for external data analysis software to me, but I'm open to suggestions.  All of the conversions are described as optional in the draft version of the section.  Implementing an absolute reference time like Jan 1, 1970 would require driver and/or control plane involvement to update conversion parameters 2 or 3 times per timestamp wrap time.  The multiplication or division would be the most difficult part, I would think, and once you have done that addition by a control-plane provided constant is much lower cost than that.

I agree with @vgurevich about having a non-unix epoch start time. For this to be really useful one would have to do fairly onerous time synchronization.
In the case of in-band telemetry, the usefulness of nanoseconds vs seconds and nanoseconds depends on the protocol, for example IOAM uses 32-bit seconds and 32-bit nanoseconds
https://tools.ietf.org/html/draft-brockners-inband-oam-data-07
How does one decide which is better? I would say seconds and nanoseconds is a more comfortable format.
I disagree with:
"Any implementation of the first method above would probably require first going through the calculation of required for the second method"
It is entirely possible that hardware implement a 32-bit nanosecond and 32-bit second counter. The divide to go from nanoseconds to seconds is very more expensive than the multiply to go to all nanoseconds.

@DGeorgeNetro Do you know any existing examples of hardware providing a separate nanosecond and second counter?  I'm just curious to broaden my horizons here.  It is trivial for an ASIC or FPGA to keep a count of clock cycles, but those will only provide a nanosecond counter if they happen to run at 1.000 GHz clock rate.

I have removed what appears to be the most controversial part of these changes for now: the conversion functions between timestamps and other time units.
If someone has ideas for a more generally agreed-upon way to do this, they can open a separate issue or pull request for it.

@jafingerhut Netronome's hardware use seconds and nanoseconds with the idea of supporting IEEE1588v2/PTP which uses a seconds/nanoseconds format. I couldn't say exactly what other hardware out there supports it.
You are quite right that this implies supporting a 1 GHz internal clock, FPGAs can achieve this using their configurable PLLs. An ASIC would have have to have the feature burned in.
I would suggest a generic timestamp_t time with target-defined width (as is already in place) and both conversion functions, timestamp_to_nanoseconds() and timestamp_to_timespec(). (where timespec is a type with seconds and nanoseconds, like Linux timespec).
Having both means the most logical for the protocol can be used and an unnecessary conversion from what the target provides, to an intermediate format and back wouldn't happen.
I would also leave the details of conversion out of the spec as this is target specific. A target supporting timestamps would have to make sure the above works and should not require the user to do anything.

@DGeorgeNetro As mentioned above, if you want this to be part of the PSA spec, I would recommend opening a new issue or pull request with the suggested changes.  It doesn't require a 1 GHz internal clock if one is willing to dedicate gates for a large enough integer multiply or divide by the appropriate constant, or some other mechanisms.  I suspect one issue might be that without some notes on implementation details of how to do it written down somewhere (whether in the PSA, or somewhere else), everyone will have their own favorite implementation in mind that works easily on the implementation they think about most often, and disregard difficulties it might cause on others.


[WIP]: drivetrain benchmark

Does the filtered oscillator belong here? I would use a new PR.

Let's outsource the model from this branch and the associated Jupyter notebooks to the /Drivetrain/Drivetrain.ipynb to the ReachabilityBenchmarks repo.

Can we delete this branch?


Any chances of releasing an Angular version of the SDK?

Angular is one of the most popular front end frameworks nowadays. I would love to be able to use Parse in my Angular web app and not have to use third-party wrappers (or in my case, have to write your own)

@gfpacheco: Theoretically couldn't you use the Parse REST API with the Angular JS web application?

With this open sourcing of the SDK, we've designed it to be the core of any application, letting developers wrap it with bindings for the framework of their choice. The hope is that between official Parse bindings (such as Parse+React, launched last April) and community-driven bindings, we'll be able to support a variety of popular frameworks, scalably. Frameworks develop fast, and we don't want to tie releases of the SDK to those. It's easier to maintain framework bindings separately.
We'll happily expose any internal functionality that makes these bindings easier, but supporting bindings for every popular framework is not something we can do on our own. That's where the community comes in, and that's why open source is so great. If members of the Parse community want to drive the development of top-notch Angular bindings, we'll support them in every way we can.

I've been happily using https://github.com/brandid/parse-angular-patch . It was broken by the 1.6.0 update, but if you are willing to stay at 1.5.0 you can use that.

Does anyone know how to require('parse') in angular code?

@5amfung if you are loading the parse.js file to your html Parse will be a globally available var.

@gfpacheco Thanks for the reply.  Not sure if you're familiar with ionic 2, but my tech stack is ionic 2, angular 2, and webpack.  With the default webpack setup, any npm installed library that is written for angular is automatically picked up and included in the final bundle.  I'm just not sure how to get Parse sdk included.  It might be something very simple to do but unfortunately I'm a newbie in javascript.

@5amfung If that's the case I suggest you to ask on StackOverflow and show off your webpack config and relevant code

With the raise of ionic framework with is bundled with Angular. And the notice that angular patch repository was discontinued.
https://github.com/brandid/parse-angular-patch
A oficial support for angular would be very helpful for all comunity.


Use of uninitialized value $name in hash element at .../Spreadsheet/ParseODS/Workbook.pm line 88

I see the above warning when parsing an ods file that has been downloaded from Google Sheets.
It would seem that there is no active sheet in the settings, and that the get_active_sheet method does not check that the active_sheet_name method returns a valid value.
The file is otherwise imported correctly.
I would upload an example file, but unfortunately the GitHub issue tracker does not support it.  I can share a link to that sheet if that is useful.

If you can provide a link to such a file, that would be great! Ideally, it is a file that I can download and put into the test suite, so please tell me if the file is OK to share.

Here's a link:  https://docs.google.com/spreadsheets/d/1vDAbSUeZO1mN7ryXtPfxAKCuvjs5M4v0tifkqJRXaAA/edit?usp=sharing
It's part of the Biodiverse test suite so is already open source.  The version linked to currently lives on a branch, but should be merged onto master shortly.

Thank you for reporting this issue and showing me a file to reproduce it. This should be addressed by f971828e3b4951e9bd081325ae475ccfa0752abf and on its way onto CPAN as 0.27 .


Few movie cards redirecting to trailer of another movie fixed

Issue Number
fixes #324
ISSUE #324
Describe the changes you've made
Few movie cards redirecting to trailer of another movie fixed
Describe if there is any unusual behavior of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have added my name in the contributors list at the end of README.md file.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.
[x] Open Source Program names LGMSOC

@kshitija-2001 check TV page.
https://qazimaazarshad.github.io/Movie-Streaming-Website/tv.html


Docker image can't sync to test network

When using the Docker image:
docker run --rm -ti --net=host sovrinfoundation/sovrin

root@node1:/# sovrin
Loading module /usr/local/lib/python3.5/dist-packages/config/config-crypto-example1.py
Module loaded.

Sovrin-CLI (c) 2016 Evernym, Inc.
Node registry loaded.
    BYU: <IP_ADDRESS>:9744
    BIG: <IP_ADDRESS>:9727
    FCU: <IP_ADDRESS>:9733
    EvernymV2: <IP_ADDRESS>:9723
    EvernymV1: <IP_ADDRESS>:9721
    RespectNetwork: <IP_ADDRESS>:9729
    WSECU: <IP_ADDRESS>:9725
Type 'help' for more information.
sovrin> connect test
New keyring Default created
Active keyring set to "Default"
Client sovrinAzta3P initialized with the following node registry:
    EvernymV1C      listens at <IP_ADDRESS>    on port 9722
    EvernymV2C      listens at <IP_ADDRESS>   on port 9724
    WSECUC          listens at <IP_ADDRESS>  on port 9726
    BIGC            listens at <IP_ADDRESS> on port 9728
    RespectNetworkC listens at <IP_ADDRESS>   on port 9730
    FCUC            listens at <IP_ADDRESS> on port 9734
    BYUC            listens at <IP_ADDRESS>   on port 9745
Active client set to sovrinAzta3P
sovrinAzta3P listening for other nodes at <IP_ADDRESS>:8307
sovrinAzta3P looking for FCUC at <IP_ADDRESS>:9734
sovrinAzta3P looking for BIGC at <IP_ADDRESS>:9728
sovrinAzta3P looking for RespectNetworkC at <IP_ADDRESS>:9730
sovrinAzta3P looking for EvernymV2C at <IP_ADDRESS>:9724
sovrinAzta3P looking for WSECUC at <IP_ADDRESS>:9726
sovrinAzta3P looking for EvernymV1C at <IP_ADDRESS>:9722
sovrinAzta3P looking for BYUC at <IP_ADDRESS>:9745
Connecting to test...
sovrinAzta3P now connected to RespectNetworkC
sovrinAzta3P now connected to EvernymV1C
sovrinAzta3P now connected to EvernymV2C
sovrinAzta3P now connected to BYUC
Connected to test.
sovrinAzta3P now connected to BIGC
sovrinAzta3P could not verify catchup reply CATCHUP_REP(ledgerType=0, txns={'8': {'reqId':<PHONE_NUMBER>805694, 'txnId': 'bfa9e020f5963c12132024
d5a2067c2bd6f53b5244cf247e9e50872c9249704b', 'data': {'node_port': 9725, 'client_ip': '<IP_ADDRESS>', 'client_port': 9726, 'node_ip': '40.12
1.149.41', 'alias': 'WSECU'}, 'dest': 'AZsp46qSPGjHuNYajiMUb5XcMWCRDuk2xdEv7P5MepSN', 'identifier': 'CECeGXDi6EHuhpwz19uyjjEnsRGNXodFYqCRgdLm
LRkt', 'type': 'CHANGE_HA', 'txnTime':<PHONE_NUMBER>369.697}}, consProof=['XmKV6S2v0aU+ShLDNYNTRqept9Wv/n6sPxUGoc0mSIA=']) since Bad Merkle proo
f: second root hash does not match. Expected hash: b'5b5c7b0062201f34f98a7505340db13494e5a425bdac28ea0041c5b8e973df71' , computed hash: b'044
4f93850f58a4a62f25a18f3ade3f4c6ca8c69a2a52a1aac305df3d86e889b'
sovrinAzta3P looking for FCUC at <IP_ADDRESS>:9734
sovrinAzta3P looking for WSECUC at <IP_ADDRESS>:9726
sovrinAzta3P looking for FCUC at <IP_ADDRESS>:9734
(...)

The "Getting Started" instructions fail because sync never occurs.

Hello  @andrevtg. Thank you for your participation
I just wanted to say that we are working on this issue.
I've reproduced it and had a chat with @lovesh
We will write updates here.

@andrevtg
UPD: Glad to inform, that cluster is up.
It was updated to sovrin 0.1.139 and plenum 0.1.158, so update your sovrin setup as well.
Don't worry about BIGC node. It's down currently and doesn't affect on consensus.
You can run Faber test from "Sovrin Getting Started" to check it.

Seems like there are other issues with the Docker Image - cannot connect to test.
Constantly looking for endpoints, but not connecting to test....
ALICE@test> connect test
Already connected to test
Usage:
connect <test|live>
ALICE@test> accept invitation from faber
Expanding faber to "Faber College"
Invitation not yet verified.
Link not yet synchronized.
Invitation acceptance aborted.
Cannot sync because not connected. Please connect first.
Usage:
connect <test|live>

@phillbo I was able to replicate your issue today. I don't believe it's a problem with the docker image, per se; it seems to be caused by one of the validator nodes in the test network misbehaving. We have identified part of the problem today (one of the workflows that dup'ed the problem started working, but another is still blocked). We will report back on further progress within 24 hours.

Great stuff! Looking forward to really getting to task with your platform. Thanks!

picking this up again - some success... another fail...
looks like connecting to test now works...
ALICE@test> status
Connected to test Sovrin network
however, next step - fails
ALICE@test> accept invitation from faber
Expanding faber to "Faber College"
Invitation not yet verified.
Attempting to sync...
Synchronizing...
Link Faber College synced
Pinging target endpoint: ('<IP_ADDRESS>', 5555)
[Not Yet Implemented]

@phillbo The "Not Yet Implemented" message tells me that the code that ran was old. We took that message out a while ago. I'm scratching my head, wondering what went wrong.
I wonder if you could update your code (pip install sovrin -U) and see if the outcome changes.

Updated - results below...
I am running the latest docker on OSX, no firewalls...
ALICE@test> accept invitation from faber
Expanding faber to "Faber College"
Invitation not yet verified.
Attempting to sync...
Synchronizing...
Link Faber College synced
Pinging target endpoint: ('<IP_ADDRESS>', 5555)
Ping sent.
No response received within specified time (8000 mills). Retry the command and see if that works.
ALICE@test>

Please try running the container with "--net=host".
Em sex, 6 de jan de 2017 às 14:36, phillbo<EMAIL_ADDRESS>escreveu:

Updated - results below...
I am running the latest docker on OSX, no firewalls...
ALICE@test> accept invitation from faber
Expanding faber to "Faber College"
Invitation not yet verified.
Attempting to sync...
Synchronizing...
Link Faber College synced
Pinging target endpoint: ('<IP_ADDRESS>', 5555)
Ping sent.
No response received within specified time (8000 mills). Retry the command
and see if that works.
ALICE@test>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sovrin-foundation/sovrin/issues/26#issuecomment-270943813,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ3LO0aG5mgqfF8TBNVciTd_k_iZsF7xks5rPm13gaJpZM4K6fr3
.

I looked at some notes on Kinematic and Docker for OS X, looks like there are some minor issues with proxy…
to remove those, I’ll set up a Ubuntu server this weekend and start form scratch!
I’ll let you know when I get success… thanks!

On Jan 6, 2017, at 8:38 AM, André Fernandes<EMAIL_ADDRESS>wrote:
Please try running the container with "--net=host".
Em sex, 6 de jan de 2017 às 14:36, phillbo<EMAIL_ADDRESS>escreveu:

Updated - results below...
I am running the latest docker on OSX, no firewalls...
ALICE@test> accept invitation from faber
Expanding faber to "Faber College"
Invitation not yet verified.
Attempting to sync...
Synchronizing...
Link Faber College synced
Pinging target endpoint: ('<IP_ADDRESS>', 5555)
Ping sent.
No response received within specified time (8000 mills). Retry the command
and see if that works.
ALICE@test>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sovrin-foundation/sovrin/issues/26#issuecomment-270943813,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ3LO0aG5mgqfF8TBNVciTd_k_iZsF7xks5rPm13gaJpZM4K6fr3
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sovrin-foundation/sovrin/issues/26#issuecomment-270944239, or mute the thread https://github.com/notifications/unsubscribe-auth/ACSRUTo1_vR3HWcu0-ZYIA963QBM3YYIks5rPm3ogaJpZM4K6fr3.


Add step scan support into the prepare for the standard detector

This is a follow on from #116 . The prepare method deals with the fly scanning behaviour of the detector, but not the step scanning. This is not major issue as the prepare is not currently widely used i step scanning, but it could be. As prepare is meant to be able to deal with both a reasonable behaviour for prepare in the standard detector would be to have prepare set trigger values for both options.  It then will go on to perform its other tasks for fly scans. The step scan trigger method can check that the cached trigger info from prepare is for a software trigger (so not a flyscan) if it is to use it. If there is no trigger info then prepare has not been called and we should use the existing defaults ( as they are present now), if it does exist but its fly scanning trigger info, fail with an error.

One thought: if #129 is true, then wouldn't prepare become required to be run after stage and before trigger even in a step scan, because otherwise the writer will never be opened. In that case, should trigger just fail outright if self._trigger_info is None (indicating prepare was not run)?

I think that depends on whether we want to support doing step scans of Preparable detectors using existing plans. If yes, then we need to lazily call prepare() if not already done in trigger(). In not, then we can do as you say and just fail if it hasn't been run

@danielballan @tacaswell thoughts on whether we want people to be able to upgrade to ophyd-async without changing their plans at all?

I think we need a lazy prepare in trigger()

Close in favour of #314


unknown Ancestry bugs

[x] https://www.ancestry.com/family-tree/person/tree/75549/person/110053435746/facts
[x] https://www.ancestry.com/family-tree/person/tree/23667211/person/250076768422/facts
[x] https://www.ancestry.com/family-tree/person/tree/44659506/person/400073848997/facts

The bug was due to marriage events in the timeline that weren't attached to a spouse and thus didn't have a spouse name.


Reduce band-wind in streaming

This issue comes from reddit
It seems like the streaming data can be reduced on the sender side to reduce the band-wind.

Regarding "YUYV format, which contains 2 pixels per 4-byte block", until I can see, the YUYV format seems to hold the same data as RGB format for each 4-byte. So we can not reduce the band wind there. But we can do it in two other ways:

Reducing the size of the frame sent. Currently, the frame is really high for a terminal application: 640 * 480. By default could be less or even we can ask the receiver the expected size (which can take more effort to do, but more accuracy).
Since the data will be rendered as xterm color. the RGB can be converted to xterm (1-byte) on the sender side, reducing the band wind to a third part.


Added the function to reverse the migration for this package

Title and the code are self-explanatory, nothing that complex overall.
All it does is using already existing function (logic) down() from Laravel Migration class
Giving you the ability to revert the changes done to database (removing tables that were added) using php artisan rollback
NOTE: Tested on fresh Laravel v9.48.0 (PHP v8.2.0) project and works without the issues.
P.S: Thanks for this awesome package you made, I'll try to contribute to it whenever I can 👍

@Goatform Thank you so much.


Can't compile the ND4J project using Maven

When I run (mvn clean install -DskipTests -Dmaven.javadoc.skip=true) in my nd4j directory after cloning the repository, I get a build failure that tells me that the files: (/home/hadoop/nd4j_7.20.2016/nd4j/nd4j-backends/nd4j-backend-impls/nd4j-native/pwd/blas/NativeBlas.h) and (/home/hadoop/nd4j_7.20.2016/nd4j/nd4j-backends/nd4j-backend-impls/nd4j-native/pwd/blasbuild/cpu/blas) are missing.
I can't find the pwd directory inside nd4j-native.

That's because instead of "pwd", there should be valid path set to LIBND4J_HOME environment variable

P.s. here's instructions to build libnd4j on windows: https://github.com/deeplearning4j/libnd4j/blob/master/windows.md
and on linux: https://github.com/deeplearning4j/libnd4j/blob/master/README.md


Use hash splitting for deterministic patient split assignments upon dataset updates.

See here for example:
https://github.com/som-shahlab/femr/blob/main/src/femr/splits.py#L58

See #132 for an example of something like hash splitting for ordering shards in this codebase. It is not about patient splits, so it is not a direct analog, but it is related..


Mueller Licht bulbs not showing the color red

The migration from hue to deconz was very successfull, thank you for the good work!
All bulbs that are connected to Phoscon/Deconz are working (Lidl, Ikea, Müller, etc.)
However only Mueller bulbs (all different models) are not able to show the color red. When I set red in HomeKit they display purple.
Those lamps also have build in effect, that the hue plugin exposed.
All other lamps from IKEA, Lidl etc. behave as they should.

Sounds like these lights aren't yet exposed properly through a DDF in deCONZ.  You should see warning messages in the Homebridge log, that Homebridge deCONZ is using the default colour gamut.
I only have one Mueller Light (a GU10 spot), but I think most of them will behave the same.  You might want to try and add the modelid of your bulbs to that DDF, https://github.com/dresden-elektronik/deconz-rest-plugin/blob/master/devices/mli/zbt-extendedcolor.json.

I migrated today from your HUE plugin to DECONZ with the help of the app "Controller für HomeKit" (Backup/Restore functionality, 7-days trial). The Müller bulbs were all functioning correctly with HUE, so I think they are exposed properly. I do not know, how to add modelids to the DDF. I have to research.

Yes, by design.  Homebridge Hue whitelists the Mueller lights, where Homebridge deCONZ relies on deCONZ exposing the light capabilities.

I think in this case, I am rolling back to the old plugin and deactivate homebridge-deconz. I‘d have to replace all of my working Mueller bulbs and I don‘t want to. Thank you

You don’t have to replace anything.  You just need deCONZ to expose the Mueller lights through a DDF.

Hence I did not know all the data for the bulbs for a proper DDF, I went to the nearest IKEA yesterday and bought 8 new trådfri bulbs. Too much of a hassle for me. This issue can be closed. There are too few people with Mueller tint bulbs that want all colors. Thank you for your help and answers. Best regards from Hamburg


Refactor server to its own package in utils/server #261

Resolves #261

Seems like you've pulled this request from matter. Pull v0.9.0 into this and merge it into v0.9.0 itself

@Inconnu08 ?

@YourTechBud ahh damn. I'll get it done today 👍


Modify bulk load runner to gracefully handle failures

When pushing a cluster hard during a long running benchmark, it is not uncommon to occasionally get errors for either full bulk requests (429 responses due to overload) or individual events. At the moment it seems like the bulk request runner treats all of these as fatal errors and terminates, which is not always desirable. It would be great it the bulk runner could be configured to report various types of errors and just continue processing instead.
Maybe this should extend also to other types of runners?

Maybe this should extend also to other types of runners?

Yes, indeed. I will take care of adapting all existing runners in core so that they don't abort the benchmark anymore.

The bulk runner has been modified so it does not actively throw an exception when bulk errors occur (but instead adds them as meta-info). Also, we will just store any failures from Elasticsearch in a meta-data field instead of aborting the benchmark.


Fix variant name duplication on rewrite failure

Closes https://github.com/nrc/rustfmt/issues/375.

Thanks!


Setup Path Getter function

Closes #11
Don't know if this is the exact implementation we wanted, feel free to comment any changes we need.

I think it might be better to have all the needed directories be defined as variables inside a constants.py file. That way, each file that needs a file from a certain dir would import constants and can do `DIR_NAME / "file.ext".

I think it might be better to have all the needed directories be defined as variables inside a constants.py file. That way, each file that needs a file from a certain dir would import constants and can do `DIR_NAME / "file.ext".

Are you meaning something like this?
from constants import assets

path = assets['level_1']
``'

Are you meaning something like this?

from constants import assets

path = assets['level_1']

More like this
from constants import LEVELS_DIR, ASSETS_DIR

block = ASSETS_DIR / "block.png"
map = LEVELS_DIR / "level_1.tmx"

Are you meaning something like this?
from constants import assets

path = assets['level_1']

More like this
from constants import LEVELS_DIR, ASSETS_DIR

block = ASSETS_DIR / "block.png"
map = LEVELS_DIR / "level_1.tmx"

👍🏼, should we do that in this PR?

Probably not, since the name suggests it's a function. I'd say you can make a new PR for issue #13 that has the paths. Also moving the asset files could also be a different PR.

Superseded by #24


Issue 435: Filter gene-related disorder submission from curation and evidence generation

Closes #435

I'm running a test spreadsheet creation to check the output, but meanwhile the code can be reviewed.

Spreadsheet from the test is here. The gene-related disorder terms are certainly not gone (though there are about 2000 fewer terms total compared to July), but their frequency in ClinVar is much reduced so they are deprioritised. I spot-checked a couple terms to make sure they were indeed present in other submissions besides the one filtered out.


Break our airgap tests. Do not merge.

This commit should fail our airgap tests, and make this PR impossible
to merge.
[#183164582]

As expected, this PR failed our tests here.
Our airgap tests seem to work, so I'm closing this now.


[ NKU-UAVTeam/rotors-modified ]  do not exist, or the library is private

Hi @charlie-dog
Thanks for the open source!
But I got some problems configuring the environment.
git clone https://github.com/NKU-UAVTeam/rotors-modified.git
remote: Repository not found.
fatal: Authentication failed for 'https://github.com/NKU-UAVTeam/rotors-modified.git/'

Maybe [ NKU-UAVTeam/rotors-modified ] do not exist, or the library is private.

Hi @TomwKang
The simulation platform and the source code will be opened around September 20th.

Hi @TomwKang The simulation platform and the source code will be opened around September 20th.

Got it !


Update finagle-core, finagle-mysql, ... to 20.8.0

Updates

com.twitter:finagle-core
com.twitter:finagle-mysql
com.twitter:scrooge-serializer
com.twitter:util-core

from 20.7.0 to 20.8.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.twitter" } ]

labels: library-update, semver-minor

Codecov Report

Merging #423 into develop will decrease coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #423      +/-   ##
===========================================
- Coverage    77.23%   77.13%   -0.10%
===========================================
  Files           46       46
  Lines         1041     1041
  Branches        29       29
===========================================
- Hits           804      803       -1
- Misses         237      238       +1

Impacted Files
Coverage Δ

.../main/scala/com/twitter/bijection/Bufferable.scala
90.80% <0.00%> (-1.15%)
:arrow_down:

Continue to review full report at Codecov.

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


ehl_crb: samples/subsys/shell/shell_module does not work

samples/subsys/shell/shell_module on EHL does not work at all.
HW dbg shows it went to the fatal exception handling path.
I disabled all the thread related options in the prj.conf:
CONFIG_PRINTK=y
CONFIG_SHELL=y
CONFIG_LOG=y
CONFIG_INIT_STACKS=y
#CONFIG_THREAD_STACK_INFO=y
CONFIG_KERNEL_SHELL=y
#CONFIG_THREAD_MONITOR=y
CONFIG_BOOT_BANNER=n
#CONFIG_THREAD_NAME=y
CONFIG_DEVICE_SHELL=y
CONFIG_POSIX_CLOCK=y
CONFIG_DATE_SHELL=y
#CONFIG_THREAD_RUNTIME_STATS=y

And it worked again. So one of this option is broken on EHL.

@tbursztyka I also saw this and worked around it by enabling polling-based input. It seems like a regression since I'm pretty sure this worked when we introduced ehl_crb to the tree. @jenmwms do you know anything about this? How recently have you tested this?

I'm pretty sure it work when we introduced it to the tree, too. I've not personally tested it recently. I'll see what we can find as far as bisecting this.

Just tried this back at fe516b9 - it works (I can use commands like 'help' in the shell module sample).

Ok, bisect and I found it breaks at 7009012 which added CONFIG_THREAD_RUNTIME_STATS=y to this samples' prj.conf. I'll need to look into configurations depending on this setting to see what may have changed. We can also check the diffs of the .conf before and after.

@jenmwms thanks for bisecting this! I can also confirm that disabling CONFIG_THREAD_RUNTIME_STATS makes things work again. @nashif any idea why that option is not working on EHL?

@nashif this sample works OK on Up2, just tested and can use the sample.

I found that the runtime statistics are gotten using the default kernel timer when using CONFIG_THREAD_RUNTIME_STATS. But
some architectures, SoCs or boards have timers with higher resolution available via timing functions. Using these timers can be enabled with this instead CONFIG_THREAD_RUNTIME_STATS_USE_TIMING_FUNCTIONS. I thought that might be the case for EHL, but had no impact when I changed the config.

@nashif Do you have any ideas why the option CONFIG_THREAD_RUNTIME_STATS breaks this shell_module sample for EHL?

@jenmwms thanks for bisecting this! I can also confirm that disabling CONFIG_THREAD_RUNTIME_STATS makes things work again. @nashif any idea why that option is not working on EHL?

can you try this:
https://docs.zephyrproject.org/latest/guides/flash_debug/thread-analyzer.html
west build -b ehl_crb samples/synchronization/ -- -DCONFIG_THREAD_ANALYZER=y -DCONFIG_THREAD_ANALYZER_USE_PRINTK=y -DCONFIG_THREAD_ANALYZER_AUTO=y -DCONFIG_THREAD_ANALYZER_AUTO_INTERVAL=5
and see if that works?

Thanks @nashif ! That seems to be working, here's the output:
Booting Zephyr OS build zephyr-v2.5.0-3100-g37de0c49878c  ***
thread_a: Hello World from cpu 0 on ehl_crb!
Thread analyze:
 thread_b            : STACK: unused 832 usage 192 / 1024 (18 %); CPU: 0 %
 thread_a            : STACK: unused 392 usage 632 / 1024 (61 %); CPU: 0 %
 thread_analyzer     : STACK: unused 0 usage 512 / 512 (100 %); CPU: 0 %
 idle 01             : STACK: unused 4088 usage 8 / 4096 (0 %); CPU: 0 %
 idle 00             : STACK: unused 4088 usage 8 / 4096 (0 %); CPU: 0 %
Thread analyze:
 thread_b            : STACK: unused 832 usage 192 / 1024 (18 %); CPU: 0 %
 thread_a            : STACK: unused 392 usage 632 / 1024 (61 %); CPU: 0 %
 thread_analyzer     : STACK: unused 0 usage 512 / 512 (100 %); CPU: 0 %
 idle 01             : STACK: unused 4056 usage 40 / 4096 (0 %); CPU: 98 %
 idle 00             : STACK: unused 4088 usage 8 / 4096 (0 %); CPU: 0 %
Thread analyze:
 thread_b            : STACK: unused 832 usage 192 / 1024 (18 %); CPU: 0 %
 thread_a            : STACK: unused 392 usage 632 / 1024 (61 %); CPU: 0 %
 thread_analyzer     : STACK: unused 0 usage 512 / 512 (100 %); CPU: 0 %
 idle 01             : STACK: unused 4056 usage 40 / 4096 (0 %); CPU: 99 %
 idle 00             : STACK: unused 4088 usage 8 / 4096 (0 %); CPU: 0 %

do we have more details about where it actually fails, i.e. debug the code enabled by  CONFIG_THREAD_RUNTIME_STATS in subsys/shell/modules/kernel_service.c and see what goes wrong.

thread_analyzer     : STACK: unused 0 usage 512 / 512 (100 %); CPU: 0 %

Seems like that thread had a stack overflow?

Noting https://github.com/zephyrproject-rtos/zephyr/pull/35059 helps fix the output from thread analyzer (above comments). Great!
But, no change for this reported failure in samples/.../shell_module with that patch. Same behavior as reported - no output, but removing this lets it work again:
- CONFIG_THREAD_RUNTIME_STATS=y

I tried increasing all of the stack sizes and that did not seem to help (used grep in .config to identify all *_STACK_SIZE).

Able to get printk's throughout arch/x86/core/prep_c.c using CONFIG_X86_VERY_EARLY_CONSOLE.

@andyross Thanks for the suggestion to try removing this piece from locore.S. The sample boots with this removed (and still using the existing THREAD_RUNTIME_STATS=y).
git diff arch/x86/core/intel64/locore.S
diff --git a/arch/x86/core/intel64/locore.S b/arch/x86/core/intel64/locore.S
index 5a56205b59..c7215c452f 100644
--- a/arch/x86/core/intel64/locore.S
+++ b/arch/x86/core/intel64/locore.S
@@ -338,9 +338,7 @@ __resume:
         */
        call z_x86_swap_update_page_tables
 #endif
-#ifdef CONFIG_INSTRUMENT_THREAD_SWITCHING
-       call z_thread_mark_switched_in
-#endif

So now to understand why this unblocked us to boot.  Looks like there is a bug here, tho.

So far nothing jumps out as I look in kernel/thread.c related to z_thread_mark_switched_in() or out(). But I do wonder if this is a different bug? I can revisit on Monday:
diff --git a/kernel/thread.c b/kernel/thread.c
index b69664f7fe..cfefffd5a1 100644
--- a/kernel/thread.c
+++ b/kernel/thread.c
@@ -1050,7 +1050,7 @@ void z_thread_mark_switched_out(void)
        diff = timing_cycles_get(&thread->rt_stats.last_switched_in, &now);
 #else
        now = k_cycle_get_32();
-       diff = (uint64_t)now - thread->rt_stats.last_switched_in;
+       diff = (uint64_t)(now - thread->rt_stats.last_switched_in);
        thread->rt_stats.last_switched_in = 0;
 #endif /* CONFIG_THREAD_RUNTIME_STATS_USE_TIMING_FUNCTIONS */

Does it still fail if you stub out 100% of the function?  That is, replace it with void z_thread_mark_switched_in(void) {}?  It's not impossible this is a stack setup problem that is blowing up due to the simple function call and not anything to do with the content of the function.
If that works, try commenting out individual lines of that function one by one (looking at it, it's an #ifdef hell, so many may not actually be compiled, but whatever).   Eventually you should get to one line that causes the failure.  Then we dig down farther.  (e.g. if it's the SYS_PORT_TRACING_FUNC macro, we know to look in the tracing layer, if it's k_cycle_get_32() then maybe the timer driver isn't initialized yet, etc...)

So far nothing jumps out as I look in kernel/thread.c related to z_thread_mark_switched_in() or out(). But I do wonder if this is a different bug? I can revisit on Monday:
diff --git a/kernel/thread.c b/kernel/thread.c
index b69664f7fe..cfefffd5a1 100644
--- a/kernel/thread.c
+++ b/kernel/thread.c
@@ -1050,7 +1050,7 @@ void z_thread_mark_switched_out(void)
        diff = timing_cycles_get(&thread->rt_stats.last_switched_in, &now);
 #else
        now = k_cycle_get_32();
-       diff = (uint64_t)now - thread->rt_stats.last_switched_in;
+       diff = (uint64_t)(now - thread->rt_stats.last_switched_in);
        thread->rt_stats.last_switched_in = 0;
 #endif /* CONFIG_THREAD_RUNTIME_STATS_USE_TIMING_FUNCTIONS */

See https://github.com/zephyrproject-rtos/zephyr/pull/36244.  Note the patch does not resolve this issue in shell module sample.

Does it still fail if you stub out 100% of the function? That is, replace it with void z_thread_mark_switched_in(void) {}? It's not impossible this is a stack setup problem that is blowing up due to the simple function call and not anything to do with the content of the function.

Nice! It boots up and enters sample with the suggestion above. Now diving deeper into that function to find the line and/or case that breaks this. Lots of #ifdefs as mentioned, but making progress.

k_cycle_get_32() then maybe the timer driver isn't initialized yet, etc...)
After investigating, indeed it seems k_cycle_get_32() is causing the issue (failed functional call). So next we look at timer driver, or where/when the timer driver is initialized to maybe see if we can move this thread tracing to after the timer is set up.
https://github.com/zephyrproject-rtos/zephyr/blob/46029914a7ac092f95e586759b8b6cc379af430c/kernel/thread.c#L1018

Ooh! I think I found it. Its related to timing from what I can tell. Easy fix if I'm correct - We need to use CONFIG_TIMING_FUNCTIONS_NEED_AT_BOOT, so that we can initialize the timer before we call switch_to_main_thread

I was thinking of adding this to the sample prj.conf, but this was only observed on ehl_crb. Is there a better place to specify this config, or no harm to add this for other platforms?


Add addition info to the term "intern"

As intern in english includes also "Werkstudent" it might not clear to german audience. For that reason it would be good to include additional info.
I'm not sure which way would be the best. With a mouse over?
Must be specified by @berlintam @sareg0


RTEMS 5.3 support

Implementation of MPCP and DPCP on RTEMS 5.3, along with the formal verification using Frama-C.


Retry transaction submission in integration tests

Currently, the integration tests might fail if they are executed on two machines roughly at the same time (eg when you execute them on your local machine and in some CI job). The reason is that the hard-coded testing account is used in the integration tests to submit transactions to the Stellar network, which might throw a tx_bad_seq error if two submissions happen simultaneously on two jobs. To mitigate this timing issue, we can just make the integration tests retry the submission in case they fail.

Hey team! Please add your planning poker estimate with Zenhub @b-yap @ebma @TorstenStueber


Header sometimes dosen't appear

The header
Cities.io
 sometimes disappears from view when playing. This happens to me whenever I hit the ready button. The header will be used to show the players stock of resources so it needs to be visible.

Im going to be busy for the next few days, could you take care of this @Dubstepper ?

I'll take care of it. :)

Looks good!

Yay :D

http://imgur.com/a/kRh0G Some header issues I've noticed...It should be simple to fix if you could fix your other issues (lol) and only occurs when using the arrow keys for movement (up/down specifically)


Add sample code for encryption

this might not be relevant ticket for couchbase-lite-java-core. should be in github-couchbase-mobile-portanl?


node-directory-size-metrics eats up burst balance on aws

We ran the all manifests setup on our kubernetes cluster, where we also have a jenkins instance.
Once jenkins runs a build, the node-directory-size-metrics pods starts reading from the persistent volume we have for the home directory.
This in turn causes a lot of read operations consuming all our burst credits. Effectively stalling our disk operations on our jenkins pod roughly 6 hours later.
removing node-directory-size-metrics from the deployment effectively stopped the excessive read operations. To compare running a build on our machine is 16k read operations every 5 minutes.
With node-directory-size-metrics we're seeing over 80k read operations every 5 minutes when the jenkinsprocess is idle.

I'm experiencing the same, here's what cat /proc/1/io gives me
/ # cat /proc/1/io
rchar: 701229807
wchar: 55178558
syscr: 15223413
syscw: 659897
read_bytes:<PHONE_NUMBER>04
write_bytes: 4096
cancelled_write_bytes: 0

We are not actively maintaining the manifest yaml anymore, but we are glad to receive contributions. We will encourage you to run Prometheus through the Prometheus Operator. This implementation uses node exporter to get ndoe metrics and you can create alerts or dashboards based on that.


nixos/gitea: fix pre start script

Motivation for this change
The gitea path is hardcoded in hooks directory in files of paths like:
repositories/<user>/<repo>.git/hooks/update.d/gitea

The locales can be updated on a new gitea release and therefore should be updated in the stateDir.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

I had one problem, the cp -r created a directory at the wrong path :
conf/locale/locale instead of conf/locale
Also the mkdir seems redundant as cp can create it

I'm not sure if it's relevant to this PR, but I'm running gitea on nixos 18.03 and currently, it's failing to start with this error message:
panic: fail to set message file(uk-UA): open conf/locale/locale_uk-UA.ini: no such file or directory

@uvNikita yes, this PR fix exactly this. For a quick fix you can run the command manually.

This feature fixes exactly this error.Am 23.08.2018 10:00 schrieb Nikita Uvarov notifications@github.com:I'm not sure if it's relevant to this PR, but I'm running gitea on nixos 18.03 and currently, it's failing to start with this error message:
panic: fail to set message file(uk-UA): open conf/locale/locale_uk-UA.ini: no such file or directory
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

Great! Thanks for the explanation, I'll try to run the command manually for now.

Merge conflict because a similar fix was merged in #45608. I'm afraid this PR (which was first) was overlooked. Please close it if the issue is fixed now.
Thank you for contributing!

Ah okay, I have updated the PR accordingly.

Thank you.


Weird cfd log metrics fixes

That fixes the logs.
@bonomat If you prefer to restrict the rejected and failed CFD by ID let me know. We could also clean with up in the database, but I am not sure it's worth it. I think the condition I added is good enough.

bors r+


Issue with webcomponents.js

I'm having frequent problems when using Google Polymer with non-Chrome browsers (Firefox/IE). Using Polymer on its own (such as viewing Polymer Element demo pages) does not seem to cause any issues. However, issues arise when using webcomponents.js is included on an existing web-application with many included GUI and utility packages (jQueryUI,
The issue seems to be related to webcomponents.js specifically. If no HTML imports are used and only webcomponents.js is imported, it throws the same error in both Firefox and IE:
webcomponents.js:2114 - SyntaxError: An invalid or illegal string was specified
webcomponents.js:113 - Error: Assertion failed

The specific line of webcomponents.js simply reads:
list = originalDocumentQuerySelectorAll.call(target, selector);

- which seems to have no immediately obvious syntax error.
Due to this error, other GUI elements on the page now exhibit erroneous behavior. There is no other useful console output provided when this happens, and no exceptions thrown by other JS packages. None of this behavior is shown when using Chrome.

webcomponents.js is broken in Safari as well: a simple demo of paper drop down menu doesn't render. Error doesn't say much
[Error] TypeError: Attempting to configurable attribute of unconfigurable property.
	defineProperty (webcomponents.min.js, line 11)
	j (webcomponents.min.js, line 11)
	H (webcomponents.min.js, line 11)
	(anonymous function) (webcomponents.min.js, line 12)
	global code (webcomponents.min.js, line 12)

@hcho112 You might want to check out this SO answer:
http://stackoverflow.com/questions/24942308/platform-js-polymer-jquery-2-x-x-and-bootstrap-error
(And be sure you're including webcomponentsjs before other scripts that touch DOM.)
@jannakha What you're seeing sounds like a separate issue. If you're seeing this consistently, can you open a new bug on webcomponents repo:

URL where this occurs, or a minimal reproducible test case.
Browser & OS version

For what it's worth, I'm not seeing this error on:
https://www.polymer-project.org/components/paper-dropdown-menu/demo.html
Tested on Safari 7.1.2 Mac and Safari iOS (8.1.1), both render correctly.

@arthurevans polymer-project.org uses @version 0.5.1-1 version of webcomponents, so it works in Safari, would be great if polymer-project.org could upgrade to 0.5.3.
I'll submit a bug to webcomponentsjs team.
Cheers.

having the same error (webcomponents.js:112) on IE (10+) /firefox
I'm developing an app in Meteor, using the meteor-polymer package (just a wrapper package to include webcomponents.js 0.5.4 and polymer.js)

Worth a shot: Try upgrading to webcomponents.js 0.5.5 to see if it fixes the issue. Just ran into this issue on another project and the upgrade fixed it. YMMV

Closing stale issues. Please reopen if you have a reproducible test on the latest polyfill.


chore: Add duckdb_engine/sqlalchemy version info to DuckDB user_agent

DuckDB supports custom_user_agent property since DuckDB v0.9.2.
This PR adds version information for both duckdb_engine and sqlalchemy, since they are independent from each other.

Thanks!


How to get Lat and Lng from District names stored in database table

I have a master table called Districts which stores district Id and District Name. I dont have Lat and Lng for displaying the markers on the map. So how should I get the lat and lng from district names to use them as lat and lng in leaflet while creating a map. I will be using leaflet js. I am using Yii2 framework

Hi, great to hear that you find Leaflet useful!
However, this issue tracker is used for reporting bugs and discussing new features. For questions on using Leaflet, please use gis.stackexchange.com or stackoverflow.
If you are really sure that this is a bug in leaflet, or a feature request, please change the wording to make it look like a bug report.


first call to deepstream throws error in update

hello - newbie here who cannot tell if i have discovered a DS issue, or i am doing something really stupid, or perhaps both.    this is a NEW installation running on an amazon-ec2 instance which has nothing except for "yum deepstream" and two scripts (both below)
thank you all very much.
stack trace (does NOT include my script name, which is very surprising):
===========================
/home/ec2-user/node_modules/deepstream.io-client-js/src/record/record.js:542
               if( currentValue !== this._oldPathValues[ path ] ) {
                                                       ^

TypeError: Cannot read property 'testValueTwo' of null
   at Record._completeChange (/home/ec2-user/node_modules/deepstream.io-client-js/src/record/record.js:542:43)
   at Record._applyUpdate (/home/ec2-user/node_modules/deepstream.io-client-js/src/record/record.js:427:7)
   at Record._$onMessage (/home/ec2-user/node_modules/deepstream.io-client-js/src/record/record.js:302:8)
   at RecordHandler._$handle (/home/ec2-user/node_modules/deepstream.io-client-js/src/record/record-handler.js:228:25)
   at Client._$onMessage (/home/ec2-user/node_modules/deepstream.io-client-js/src/client.js:128:42)
   at Connection._onMessage (/home/ec2-user/node_modules/deepstream.io-client-js/src/message/connection.js:323:17)
   at emitOne (events.js:96:13)
   at emit (events.js:188:7)
   at TcpConnection._onData (/home/ec2-user/node_modules/deepstream.io-client-js/src/tcp/tcp-connection.js:180:7)
   at emitOne (events.js:96:13)

// server

var deepstream = require('deepstream.io-client-js');

ds = deepstream( '<IP_ADDRESS>:9021' ).login();
ds.record.listen('test', function (recordName, isSubscribed)    {

       record = ds.record.getRecord(recordName)        ;

       record.subscribe ('testValueOne', function(value)       {
               record.set ('testValueTwo' , 'SERVER: initial testValueTwo setting.' );
       });

       record.subscribe ('testValueTwo', function (value)  {
               console.log('in whenReady area');
       });
});

// client

var deepstream = require('deepstream.io-client-js');
ds = deepstream( '<IP_ADDRESS>:9021' ).login();

//record = ds.record.getRecord('test' + ds.getUid()     )       ;
record = ds.record.getRecord('test' + (new Date).getTime()      )       ;
//record = ds.record.getRecord('test'   )       ;

record.set('testValueOne', 'value set here');
record.subscribe ('testValueTwo', function (value)  {
       testValueTwo = record.get('testValueTwo'                );
});

This is super wierd
	if( this._oldPathValues === null ) {
		return;
	}

	var path, currentValue;

	for( path in this._oldPathValues ) {
		currentValue = this._getPath( path ).getValue();

		if( currentValue !== this._oldPathValues[ path ] ) {
			this._eventEmitter.emit( path, currentValue );
		}
	}

	this._oldPathValues = null;

Your stack trace is saying oldPathValues is null, but their is a check earlier to ensure it isn't, and the key you got is from within the map itself..
I'll try to reproduce it, but this is just WTF mind blown

Got it ( theoratically ), this is a recursion issue. Basically:
1-> server update calls subscribe
2 -> notifies 1
2-> locally call set
3-> notifies 2
4 -> finishes notifes 2
5 -> continues notifies 1
6 -> BOOM
The boom happens because the recursive notification uses the same record singelton object which is nulled before it calls 3
We'll look into this. For now though the best approach MIGHT be to move the subscription order around. Let me know!
ds.record.listen('test', function (recordName, isSubscribed)    {

       record = ds.record.getRecord(recordName)        ;

       record.subscribe ('testValueTwo', function (value)  {
               console.log('in whenReady area');
       });

       record.subscribe ('testValueOne', function(value)       {
               record.set ('testValueTwo' , 'SERVER: initial testValueTwo setting.' );
       });
});

This is super wierd
but that is what all the women said of me back in high school.

For now though the best approach MIGHT be to move the subscription order around.
your suggestion works!

We'll look into this.

keep me notified please.  and of course if you see a better way of doing what i am trying to accomplish, please let me know.

What your doing is legit, just not sure what the end goal is tbh.
Given this isn't a blocker well plan in soon

here is what i am trying to do:

client creates record which will contain username/password value
server checks username/password and subscribes to clients record if login OK
server adds a data item to the record
server subscribes to future changes to the data item
server saves future changes to the data item made by the client.

my logic could probably use some improvement !   any suggestions are most welcome.

First of all let's use not the word server, to avoid confusion with the deepstream server.
It looks like you want to do some communication between two (deepstream) clients. We can call them publisher and subscriber or just client1 and client2, because a client can do both at the same time.
I'm not sure what you want to do, you tried to explain it in deepstream terminology, maybe it's better to explain it in a general way?
You need to be aware if you really need a record, because records are for state. The first step (a record with username/password) looks more like an event.
I created an example with two clients, they do something different, but maybe it helps you to get what you need. I also add some logic to allow starting the clients in an arbitrary order and by an arbitrary delay (client1 can be started 2 seconds before client2 and vice versa).
client1.js
'use strict'

const deepstream = require('deepstream.io-client-js')
const client = deepstream('localhost:6021').login({}, function(success, data) {
	if (success) {
		const recordName = 'foo/bar'
		const record = client.record.getRecord(recordName)

		const id = setInterval(() => {
			if (record.get('data') == null) {
				client.event.emit('custom-login', {
					username: 'user',
					password: 'pass',
					recordName: recordName
				})
			} else {
				clearInterval(id)
				// start to trigger changes which are subscribed by the other client
				let counter = 0
				setInterval(() => {
					record.set('info', counter++)
				}, 1000)
			}
		}, 1000)
	} else {
		console.error(arguments)
	}
})

client.on('error', (data) => {
	console.error(data)
})

client2.js
'use strict'

const deepstream = require('deepstream.io-client-js')
const client = deepstream('localhost:6021').login({}, function(success, data) {
	if (success) {
		client.event.subscribe('custom-login', eventData => {
			console.log('resceived event')
			if (eventData.username === 'user' && eventData.password === 'pass') {
				console.log('custom-login successfull')
				const record = client.record.getRecord(eventData.recordName)
				record.subscribe(recordData => {
					console.log(`record by ${eventData.username} has changed to`, JSON.stringify(recordData))
				})
				record.whenReady(() => {
					record.set('data', {meta: 'connected'})
				})
			}
		})
	} else {
		console.error(arguments)
	}
})

client.on('error', (data) => {
	console.error(data)
})

it never ceases to amaze me what i can learn from looking at other peoples stuff.  thank you Tim.


Use server time for cargo today

cargo today currently uses local time instead of server time for determining the date.

nice catch, thank you!


Adds new HardwareIO for nugus2

Adjusted for metal legs
Exactly the same as n3
Adjusts shoulder pitch so horn zero (Zombie) position is straight up (Same as n1 and n3)

If it's the same as n3, then we should probably just make this the default and only have a specific one for n1

Gonna have to double check what i've done here. Looks like i have the same IO for n1 and n2, with the minor differences in hip and knee values.

If it's the same as n3, then we should probably just make this the default and only have a specific one for n1

Gonna have to double check what i've done here. Looks like i have the same IO for n1 and n2, with the minor differences in hip and knee values.

I checked them side by side and they are slightly different


perf(module:mention): do not trigger change detections if there are no event listeners

PR Checklist

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
[x] Refactoring (no functional changes, no api changes)

Does this PR introduce a breaking change?
[ ] Yes
[x] No

Since this PR is based angular v12, so I prefer to rebase it with master first :) If you don't mind, I will perform this action.


refactor: ♻ rewrite sheet component

👀 PR includes

🎨 Enhance

[ ] Code style optimization
[x] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

📝 Description
重写组件层代码

新增 <BaseSheet/> 组件 和 useSpreadSheet hook

useSpreadSheet():

useEvent 负责对 core 层的事件监听, 映射成回调: S2Event.DATA_CELL_CLICK => onDataCellClick
useLoading 表格加载前 转圈, 加载完成 结束转圈
usePagination 分页状态 state 和 options 同步
useResize options 的 width/height 和 窗口 resize 改变后 自动更新表格

<BaseSheet/>: 三种表格形态的通用结构

增加测试

🖼 Screenshot

🔍 Self Check before Merge

[x] Add or update relevant Docs.
[x] Add or update relevant Demos.
[x] Add or update relevant TypeScript definitions.

TODO: 组件功能全量回归

:tada: This PR is included in version @antv/s2-react-v1.4.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.6.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.6.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.7.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.7.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.8.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.8.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


added division and average functions

A description would have been appreciated, there are no changes. Push your code.


Upgrade GraphQL Java baseline to 20.0

See https://github.com/graphql-java/graphql-java/releases/tag/v20.0

We might raise the baseline again to GraphQL Java 21.0 if it's released during our milestone phase, but we'll stick to 20.0 for now.


Passing data from one middleware to another

It would be better to pass custom data from one middleware to another. I know it can be done by using global variables or altering to, from variables but it would be nice if there was an optional forth argument that could be used for custom data in the middleware?

Without this feature I don't actually understand how to create async methods await-ing each other

@Nathaniel-Rink async behavior is achieved via the next callback parameter. The guard chain will not proceed until the current guard calls it even if it is called in an async manner. Here's the README.md example which uses setTimeout to demonstrate async behavior in guards:
import VueRouter from 'vue-router';
import multiguard from 'vue-router-multiguard';

const guard1 = function(to, from, next) {
    setTimeout(function() {
        console.log('guard1 called');
        next();
    }, 1000);
}

const guard2 = function(to, from, next) {
    setTimeout(function() {
        console.log('guard2 called');
        next();
    }, 1000);
}

const router = new VueRouter({
    routes: [
        {
            name: 'home',
            path: '/',
            component: {},
            beforeEnter: multiguard([guard1, guard2]),
        }
    ]
});

You're right. Just tested it. The code is deceptively elegant in its simplicity. Thanks for the response. Didn't know you were still actively maintaining this!
I'll post your answer to my own question on stack overflow if you don't mind


Removal of old repos and deprecation of legacy API's

Removal of old repos and deprecation of legacy API's

/lgtm


combine=false in twig ignored

I am using assetic and twig, but not the symfony2 framework.
I am using the following markup in twig:
{% stylesheets 'style/print.css' 'style/ie.css' combine=false output='css/test.css' %}
  <link href="{{ asset_url }}" type="text/css" rel="stylesheet" />
{% endstylesheets %}

This correctly generates:
<link rel="stylesheet" type="text/css" href="css/test_print_1.css">
<link rel="stylesheet" type="text/css" href="css/test_ie_2.css">

This is the code I am using to generate my assets from the twig templates:
<?php
$loader = new \Twig_Loader_Filesystem(array(SOURCE)); //Source is just a constant to the directory that holds the templates
$twig = new \Twig_Environment($loader);

$factory = new AssetFactory(SOURCE);
$factory->setDebug(true);
$am = new LazyAssetManager($factory);

$twig->addExtension(new AsseticExtension($factory));

//enable loading assets from twig templates
$am->setLoader('twig', new TwigFormulaLoader($twig));

$fm = new FilterManager();
$factory->setFilterManager($fm);
$fm->set('cssmin', new CssMinFilter());

$templates = array('abc.twig', etc); //just an array of my templates

foreach ($templates as $template) {
   $resource = new TwigResource($loader, $template);
   $am->addResource($resource, 'twig');
}

$writer = new AssetWriter(DESTINATION); //Destination is just the root of the directory assets are to be written into
$writer->writeManagerAssets($am);

The problem is that the asset writer ignores the combine=false declaration. When I run my script to generate assets, I get one combined file called test.css that contains ie.css and print.css.

combine can be used to foce combining the assets. But currently, it cannot be used to force keeping them separated. When the value is false, the debug mode is used instead to determine it (meaning it is always combined when you are not in debug mode)
However, I'm wondering if the AssetWriter does its job properly. @kriswallsmith is it normal to have less stuff in it than what is done in AsseticBundle here ?

Hm, Assetic should always respect combine — it is more specific than debug.

@stof The command in the bundle looks flawed. The combine value should be available in the formula there.

So

HTTP2.0 any1 ? can someone please implement this?  It's 2016 and one big file is not really a "go-to" option for everyone


[FEATURE] Make RandomList serializable

Update RandomList so that it can appear on the inspector.

It's not viable to work on this task until Unity 2020.1 is released, due to serialized generic classes not being supported yet.

Finished in 65e8685a22614f3e00d79f34d36289b46949dfea


Retain Cycle cause by strong reference to delegate

Hey Matt, maybe I'm wrong but I think you should not have a strong reference to your delegate with the risk of causing a retain cycle.
https://github.com/mattconnolly/ZipArchive/blob/master/ZipArchive.h#L89

I think you’re right. I haven’t looked at this project for a while now. Feel free to send a pull request.
-Matt

On 17 Dec 2015, at 7:26 PM, Raphael Oliveira<EMAIL_ADDRESS>wrote:
Hey Matt, maybe I'm wrong but I think you should not have a strong reference to your delegate with the risk of causing a retain cycle
https://githubcom/mattconnolly/ZipArchive/blob/master/ZipArchiveh#L89
—
Reply to this email directly or view it on GitHub https://github.com/mattconnolly/ZipArchive/issues/74.

I'm getting following error (https://github.com/mattconnolly/ZipArchive/issues/66) when installing ZipArchive manually. Can you suggest me how to integrate it properly? I couldn't find any steps to do so. @mattconnolly @raphaeloliveira


yacs error occurred when run command

Describe the bug
when run command:  python federatedscope/main.py --cfg federatedscope/gfl/baseline/isolated_gin_minibatch_on_cikmcup.yaml --client_cfg federatedscope/gfl/baseline/isolated_gin_minibatch_on_cikmcup_per_client.yaml
yacs error:  AssertionError: a (cur type <class 'NoneType'>) must be an instance of <class 'yacs.config.CfgNode'>
Desktop (please complete the following information):

OS: [MACOS]
Version [13.0]

Additional context
2022-08-13 13:25:21,112 (fed_runner:302)INFO: Client 13 has been set up ...
Traceback (most recent call last):
File "/Users/panmin/Downloads/FederatedScope/federatedscope/main.py", line 46, in
runner = FedRunner(data=data,
File "/Users/panmin/opt/anaconda3/envs/fs/lib/python3.9/site-packages/federatedscope-0.1.9-py3.9.egg/federatedscope/core/fed_runner.py", line 58, in init
self._setup_for_standalone()
File "/Users/panmin/opt/anaconda3/envs/fs/lib/python3.9/site-packages/federatedscope-0.1.9-py3.9.egg/federatedscope/core/fed_runner.py", line 115, in _setup_for_standalone
self.client[client_id] = self._setup_client(
File "/Users/panmin/opt/anaconda3/envs/fs/lib/python3.9/site-packages/federatedscope-0.1.9-py3.9.egg/federatedscope/core/fed_runner.py", line 278, in _setup_client
client_specific_config.merge_from_other_cfg(
File "/Users/panmin/opt/anaconda3/envs/fs/lib/python3.9/site-packages/federatedscope-0.1.9-py3.9.egg/federatedscope/core/configs/config.py", line 153, in merge_from_other_cfg
merge_dict_a_into_b(cfg_other, self, self, [])
File "/Users/panmin/opt/anaconda3/envs/fs/lib/python3.9/site-packages/federatedscope-0.1.9-py3.9.egg/federatedscope/core/configs/config.py", line 70, in merge_dict_a_into_b
_assert_with_logging(
File "/Users/panmin/opt/anaconda3/envs/fs/lib/python3.9/site-packages/yacs-0.1.8-py3.9.egg/yacs/config.py", line 545, in _assert_with_logging
assert cond, msg
AssertionError: a (cur type <class 'NoneType'>) must be an instance of <class 'yacs.config.CfgNode'>

May I ask that which branch you run the command? master or cikm22competition?

Cikm22competition will meet this error.

@xieyxclack Cikm22competition will meet this error.

@xieyxclack Cikm22competition will meet this error.

With the experience of @data-alexpan , you might need to install the environment  according to link

thanks buddy. It's worked.


Chapter: GitHub concepts

Purpose/implementation Section
What changes are being implemented in this Pull Request?
Adding a chapter that hopefully gets people hitting the ground running with GitHub.
What was your approach?
The pattern of this chapter is to introduce a concept and then immediately take that step. And the goal of the entire chapter is to open a pull request.
This will be a chapter I'm planning on creating a video to guide folks through these steps.

This'll do for now.


[Hardware][Intel CPU] Fix build failure due to not cpu-only torch installed

The CPU targets of vLLM fail to build with the following error.
Re-run cmake no build system arguments
-- The CXX compiler identification is GNU 12.3.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Build type: Release
-- Target device: cpu
-- Found Python: /usr/bin/python3 (found version "3.10.12") found components: Interpreter Development.Module Development.SABIModule
-- Found python matching: /usr/bin/python3.
CUDA_TOOLKIT_ROOT_DIR not found or specified
-- Could NOT find CUDA (missing: CUDA_TOOLKIT_ROOT_DIR CUDA_NVCC_EXECUTABLE CUDA_INCLUDE_DIRS CUDA_CUDART_LIBRARY)
CMake Warning at /usr/local/lib/python3.10/dist-packages/torch/share/cmake/Caffe2/public/cuda.cmake:31 (message):
  Caffe2: CUDA cannot be found.  Depending on whether you are building Caffe2
  or a Caffe2 dependent library, the next warning / error will give you more
  info.
Call Stack (most recent call first):
  /usr/local/lib/python3.10/dist-packages/torch/share/cmake/Caffe2/Caffe2Config.cmake:86 (include)
  /usr/local/lib/python3.10/dist-packages/torch/share/cmake/Torch/TorchConfig.cmake:68 (find_package)
  CMakeLists.txt:67 (find_package)

CMake Error at /usr/local/lib/python3.10/dist-packages/torch/share/cmake/Caffe2/Caffe2Config.cmake:90 (message):
  Your installed Caffe2 version uses CUDA but I cannot find the CUDA
  libraries.  Please set the proper CUDA prefixes and / or install CUDA.
Call Stack (most recent call first):
  /usr/local/lib/python3.10/dist-packages/torch/share/cmake/Torch/TorchConfig.cmake:68 (find_package)
  CMakeLists.txt:67 (find_package)

The reason is that not cpu-only torch is installed.
torch                             2.4.0
torchvision                       0.19.0+cpu

I guess we should install torch from https://download.pytorch.org/whl/cpu as before.
After this patch, cpu-only torch can be installed and the build passed.
torch                             2.4.0+cpu
torchvision                       0.19.0+cpu

Please review it.
Thanks.

Hi @DamonFool , sorry for the late. I think you can change the title as update torch 2.4.0 for CPU backend and mark the PR as ready to trigger full CI run. Thanks!

/ready

Hi @DamonFool , sorry for the late. I think you can change the title as update torch 2.4.0 for CPU backend and mark the PR as ready to trigger full CI run. Thanks!

Done.
Thanks @bigPYJ1151 .

Hi @bigPYJ1151 , the failing tests are all running on GPUs which should not be affected by this change.

Hi @bigPYJ1151 , I would suggest fixing this bug as soon as possible since this is a build failure.
Please let me know if you're fine with this change.
Thanks.

Hi @mgoin , would you please help to merge this PR as it passed the CPU tests? Thanks!

Thanks @simon-mo for your help.


Update Pages “terms-and-conditions”

Automatically generated by Netlify CMS

👷 Deploy Preview for peaceful-meerkat-940447 processing.

Name
Link

🔨 Latest commit
57c63d611462985d96212afafe306f3a9a67ea3f

🔍 Latest deploy log
https://app.netlify.com/sites/peaceful-meerkat-940447/deploys/64066debad63120007438764


Anerdw: Blacklist (?-i:^Juzzlicka Arrbrunk$)

Anerdw requests the blacklist of the username (?-i:^Juzzlicka Arrbrunk$). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:^Juzzlicka Arrbrunk$) has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Approved by Fastnlight in Charcoal HQ


update the version of underscore dependence!

I found this fork https://github.com/seald/nedb
It is being maintained.


How to cover thumb image   video player's  screen ?

@ismailtosun I use videoView.getCoverView().setImageBitmap(bitmap) depends if the image will be coming from a network call you can use Glide, and adjust the view output if fitCenter or centerCrop or etc..., according to your needs.


add support for podman/containerd

Please add command line argument to extend your tool for other container runtimes.
It is probably easy to add Podman support. It has compatible command line for cp/exec commands.
Containerd doesn't have cp command, but it looks for k8s you do same as described here https://github.com/containerd/containerd/issues/2044.

@k2s podman support done.
I took a look the the ctr cli of the containerd, looks like it's created for debug purpose as stated in the ctr's usage output. So I'm going to table the ctr part for now.


roster file updated

Added bio details in the roster file

Thanks. I'm going to close this PR. You can move onto the next assignment.


Added latest answers to knowledge base

Signed-off-by: Paul O'M<EMAIL_ADDRESS>

Useful new info


Set permissions

As a User with management permission, I want a UI, which enables me to set all permission types for each group.
It's enough to select one permission-type for full specification:
The management permission includes the lending permission.
The lending permission includes the view permission.
Resources

Completes
Attention! There are already existing (UI) functionalities to set permissions. Reuse it, if possible!

#65
#76
#85

Acceptance Criteria

[ ] All groups can be selected.
[ ] A permission gets deleted, if a user clicks on the cross.
[ ] A new permission gets added if I click on “Add new Permission”.
[ ] A group can be set/changed with a click on the left field of a permission.
[ ] A permission-type of a group can be set/changed with a click on the right field of a permission.

Corresponding PR: https://github.com/hpi-swt2/bookkeeper-portal-red/pull/223

The internationalization on the permission level has to be added. @bweengener it sounded as if you had an idea how to do this in the last meeting

For the migration script, we can use the following snippet, when the group name is known. Is everyone the correct title?
INSERT INTO permissions(item_id, group_id, permission_type, created_at, updated_at)
SELECT id, (
    SELECT groups.id
    FROM groups
    WHERE groups.name = 'everyone'
    LIMIT 1),
    0,
    datetime(),
    datetime()
FROM items;

We can use execute within a migration class in order to execute this SQL Insert.
cc @Yannis-Hofmann , @matveysiva


Crashes when enabling recipe book

basically the title,
1.21.4 fabric (0.16.9),
fabric api 0.112.2+1.21.4,
utility belt 2.6.0+1.21.4,
accessories-fabric-1.2.5-beta+1.21.4,
jamlib-fabric-1.2.2+1.21.4

An update was pushed on accessories end to fix the crash, check for the latest version being 1.2.6


Element class named constructor should return Element instance

Line https://github.com/celyes/ray/blob/946a91b4057723632d6e813771bc1ec7368b9143/src/Element.php#L27
I would not make an instance of class A inside class B where A is Ray and B is Element.

This is not the best solution to work with nested arrays indeed. Can you suggest me a better way to achieve this functionality without making sort of a circular returns? Q