5.4-visualizing-what-convnets-learn

There is a problem in code number 13 where K.gradients is used it should be modified with tf.GradientTape

I'm struggling with this problem.
Could you please tell me how I can change my code using GradientTape?

I had the same problem. I implemented this answer and it solved the issue.
It suggests adding the following code before calling K.gradients:
import tensorflow as tf
tf.compat.v1.disable_eager_execution()


Add db column in grants for contract template name

just using the grant name isn't good enough any more.  This is a db change but should be safe to do now.

fixed in  #127


[Bug] The Web Interface doesn't respect --log-level

What are you really trying to do?

Describe the bug
GET requests being populated in the stdout when --log-level warn is set
Minimal Reproduction

Run temporal server start-dev --log-level warn
Go to localhost:8233
Open up your terminal and notice a bunch of logs you shouldn't see just from visiting the webui

Environment/Versions

OS and processor: M1 Mac
Temporal Version: v0.8.0
Are you using Docker or Kubernetes or building Temporal from source?
Built from source (i.e. [temporal_cli_0.8.0_darwin_arm64.tar.gz].
(https://github.com/temporalio/cli/releases/download/v0.8.0/temporal_cli_0.8.0_darwin_arm64.tar.gz))

Additional context

Thanks for reporting, we're working to fix it.

This also happens when using --log-format=pretty since the web UI keeps printing in json format:
❯ temporal server start-dev --log-level=warn --log-format=pretty  --ip <IP_ADDRESS> --db-filename /tmp/temporal.db
Starting UI server...

   ____    __
  / __/___/ /  ___
 / _// __/ _ \/ _ \
/___/\__/_//_/\___/ v4.9.0
High performance, minimalist Go web framework
https://echo.labstack.com
____________________________________O/_______
                                    O\
⇨ http server started on [::]:8233
{"time":"2023-04-19T19:03:34.134408-03:00","id":"","remote_ip":"<IP_ADDRESS>","host":"localhost:8233","method":"GET","uri":"/_app/version.json","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/111.0","status":200,"error":"","latency":4144604,"latency_human":"4.144604ms","bytes_in":0,"bytes_out":27}
{"time":"2023-04-19T19:03:34.186391-03:00","id":"","remote_ip":"<IP_ADDRESS>","host":"localhost:8233","method":"GET","uri":"/namespaces/default/workflows?query=&search=basic","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/111.0","status":200,"error":"","latency":28996,"latency_human":"28.996µs","bytes_in":0,"bytes_out":1193}
{"time":"2023-04-19T19:03:34.521378-03:00","id":"","remote_ip":"<IP_ADDRESS>","host":"localhost:8233","method":"GET","uri":"/_app/immutable/entry/start.d4bd6572.js","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/111.0","status":200,"error":"","latency":128214,"latency_human":"128.214µs","bytes_in":0,"bytes_out":23056}
{"time":"2023-04-19T19:03:34.521849-03:00","id":"","remote_ip":"<IP_ADDRESS>","host":"localhost:8233","method":"GET","uri":"/_app/immutable/entry/app.9ded9747.js","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/111.0","status":200,"error":"","latency":176679,"latency_human":"176.679µs","bytes_in":0,"bytes_out":53970}
{"time":"2023-04-19T19:03:34.543261-03:00","id":"","remote_ip":"<IP_ADDRESS>","host":"localhost:8233","method":"GET","uri":"/_app/immutable/chunks/index.a36c9cb0.js","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/111.0","status":200,"error":"","latency":159421,"latency_human":"159.421µs","bytes_in":0,"bytes_out":16093}
...

Testing on the rewrite the web UI still doesn't respect the log level, but that is because we are always hiding the logs for the UI now by https://github.com/temporalio/cli/pull/441 @cretz is this intentional?

It is intentional to hide web output because we didn't build the full logger integration and the web code was writing to stdout/stderr directly in some cases. But if we think this is important, we can probably provide our same slog to the UI logger. I have no strong opinion.

I don't get it... will the web logs respect the "log-level" setting?


SDK-2088: Adding new metrics to the SDK

This PR adds support for Mood, DominantEmotion, and some new expressions and emotions

looks good


allow configuration section in tsconfig for customizing codefixes

There might be a section in tsconfig say "codefixes" which would be a dictionary of arbitrary configuration objects keyed by names of codefixes. Such object containing customization would be passed to the code of a corresponding codefix.

More of an implementation detail for https://github.com/Microsoft/TypeScript/issues/6508. I do not think we want to separate extensions into categories, e.g. "code fixes", "refactorings" and "completions" etc..


Add Ldaprc lexer

Hello,
Here is a new lexer to highlight ldaprc files.
Regards,
Étienne

Thanks for the review !

While the lexer isn't very complicated, I do wonder if you can simplify it. It looks like all the lines have the form <keyword> <whitespace> <value>. Is it possible to use just a few rules like
(r'(\w+)(\s+)(\d+)', bygroups(Keyword, Whitespace, Keyword.Constant),
(r'(\w+)(\s+)(\w+)', bygroups(Keyword, Whitespace, Literal),
...

?

That was my first attempt. I updated to match how DockerLexer is done.
(J'ai été longtemps Charentonnais :wave: )


Remove all PG14 from installers

I know this issue is closed and merged, and I think I'm OK with this happening, but I'm wondering what the reasoning is for removing PG14 installers from the examples. Postgres 14 is still supported, but does Crunchydata and/or PGO use a different support matrix? I'm probably going to be ok with whatever the answer is, but I'm struggling to find it.
References I've looked through:

https://www.postgresql.org/support/versioning/
https://access.crunchydata.com/documentation/postgres-operator/latest/references/components#container-versions


Update intake catalog with Marine conservation/ coral bleaching datasets

Not now, focusing more on adding stability + experimenting with ML models


Implement CONTAINS operator

Like:
    select * from animaltype where races contains (name.toLowerCase().subString(0,1) = 'e')

It is similar to IN operator.

I might dont get the picture here - but looks to mee like the example is trying to assign 'e' to a string
anyway. Might not be important that I dont get the example. The CONTAINS looks useful and is well described here
https://msdn.microsoft.com/en-us/library/ms187787.aspx

This should be a small change to the parser

Bumping issue. I find myself needing this too.


Support IPC endpoint

Some tools (i.e. Mist, Solidity tests, etc.) have switched to use IPC instead of the HTTP-RPC.
Would it be hard adding IPC support? (That would need to replace the http server when the option is turned on. Ideally listening on both an IPC file and HTTP would be nice.)

Quick google makes it look promising: https://www.npmjs.com/package/node-ipc
I have no idea how IPC works, but this would be a good feature.

@tcoulter well the term IPC is overused, but what these clients really mean is a Unix socket or a Windows pipe. node-ipc seems to support that.

Definitely would love to see this happen

Looking forward to IPC support

I had IPC support implemented in https://github.com/trufflesuite/ganache-cli/pull/261, but it was closed after half a year of not merging it :)

@axic just a question on a related problem, but does ganache supports connecting to local node using unix socket?

@axic just a question on a related problem, but does ganache supports connecting to local node using unix socket?


cardano-ledger (as used in cardano-node@master and 1.34.5) with 9.2 compatibility

This PR builds the foundation for a cardano-node@master and<EMAIL_ADDRESS>with 9.2 support. Both reference cardano-ledger@f49879. We therefore want to create a cardano-ledger@f49879 + 9.2 compatibility release, so that cardano-node@master, and<EMAIL_ADDRESS>can be made 9.2 compatible, until ledger@master, which is since #3041 9.2 compatible is integrated into cardano-node@master.
cardano-node@master references f49879a79098d9372d63baa13b94a941a56eda34 but should really reference c7c63dabdb215ebdaed8b63274965966f2bf408f (tag: ledger/1.0.0)
while ouroborus-network@master references 49613f11e034485c8377945c86c0465726c0860c (tag: ledger/1.1.0)
c7c63dabd (tag: node/1.35.3, tag: ledger/1.0.0) Merge pull request #2938 from input-output-hk/jc/fix-alonzo-minfee-on-release-1-0-0
7bec14d52 the alonzo UTxO rule to use alonzo minfee function
ebcf1a893 Merge pull request #2917 from input-output-hk/jc/update-plutus-on-release-1-0-0
e54e85a18 update plutus
f49879a79 Merge pull request #2909 from input-output-hk/jj/nothunks

(node@master -- while referencing f49879a79, should reference c7c63dabd for coherence).
7f7a8ae75 Merge pull request #2960 from input-output-hk/jc/set-utf8-in-nix-shell
49613f11e (tag: ledger/1.1.0) Merge pull request #2981 from input-output-hk/lehins/revert-renaming-of-environment
c98ea168c Update Changelog with new renamings
4c8306206 Rename back the Environment types:
b24fa846e Merge pull request #2979 from input-output-hk/lehins/fix-performance-issues-due-to-inlining
a46d52bc2 Merge pull request #2978 from input-output-hk/lehins/minimize-constraints-to-avoid-ghc-bug
af68c1cd9 Add a whole bunch of `INLINE` and `SPECIALIZE` pragmas
651d325c4 Keep number of constraints to the minimum in order to work around a ghc bug:
2eafa6e55 Merge pull request #2972 from input-output-hk/td/unify-rules-types-across-eras
67b322fc1 Prefix rule-types with era name in babbage
835fe353d Prefix rule-types with era name in alonzo
651af86d6 Prefix rule-types with era name in shelley-ma
cf7d2c77c Prefix rule-types with era name in shelley
b44a3b895 Merge pull request #2967 from input-output-hk/ts-conway-consenus-test
a07bc36cd Added Consenus tests for Conway era.  Also extended Test.Cardano.Ledger.Generic.Proof and related modules to the Conway era. Added Conway unit tests (TwoPhase
Validation BabbageFeatures) Added Conway property tests
67bce2c27 Add two invalid Babbage feature tests: ReferenceInputsNotSupported and ReferenceScriptsNotSupported (#2971)
93eda91f5 Added precision to the Babbage era specification concerning the use of new features with PlutusV1 scripts (#2970)
52125a3fb Merge pull request #2966 from input-output-hk/jj/conway-json
96a5618d0 Ormolize
1a1e7dfcd Implemented FromJSON manually
1d3331748 Added FromJSON instance to ConwayGenesis
d4743d896 Merge pull request #2963 from input-output-hk/jj/conway-genesis
66cb66463 Merge pull request #2959 from input-output-hk/jc/move-alonzo-spec-out-of-draft
5302e70dc Update GenDelegs in the DState
31803c8df Added ConwayGenesis as the translation context
d696f1d47 Merge pull request #2950 from input-output-hk/lehins/fixups-to-recent-restructure
275f90cfc Merge pull request #2962 from input-output-hk/td/remove-redundant-dependency
ec7dd5651 Remove unused data-default-class library from shelley-ma dependencies
01a43bebd Avoid ghc bug that causes an infinite loop
0f5307fee makefile for alonzo spec
7f7a8ae75 Merge pull request #2960 from input-output-hk/jc/set-utf8-in-nix-shell
f8e6f561c set LC_ALL="C.UTF-8" inside nix shell
dd5109edf mark alonzo spec as pubic (not draft)
83c9627bc Merge pull request #2956 from LudvikGalois/ludvikgalois/title-case-follow-the-satoshi
be3a19c29 Merge pull request #2957 from input-output-hk/newhoggy/new-calculatePoolDistr'-function
f436bff8f New calculatePoolDistr' function which is the same as calculatePoolDistr, but additionally takes a filter function to constrain returned results
fdfeca9e2 Fix typo in changelog of delegation design spec
74dc096a1 Change "Follow the Satoshi" to title case
892eed66a Merge pull request #2955 from LudvikGalois/ludvikgalois/unboxed-vector-maps
3bf6a4b7e Merge pull request #2953 from input-output-hk/ts-validate-by-phase-onTopOf-alexisPr
f841c0cd1 Merge pull request #2924 from input-output-hk/ts-testPropertyNamed
8a3f91687 Merge pull request #2762 from SebastienGllmt/patch-5
1f1e0f6a0 Moved Language.hs to core. Added Phase types, and changed the classes EraScript and EraTx methods to use phased types. This means validate methods only work on scripts valid for a given phase, and this is enforced by types.
ad42466fc Replaced testProperty with our own version written in terms of testPropertyNamed
3392ae6ac Make VMap.VU an Unboxed Vector
11e4d4a8a Merge pull request #2942 from input-output-hk/jc/clarify-PVCannotFollowPPUP-comment
49c687faf Add Byron CDDL to README
25c19a08f better comment for the PVCannotFollowPPUP failure
93b5d5421 Merge pull request #2945 from input-output-hk/lehins/new-conway-era
b8a3ea89d Add `FromSharedCBOR` constraint to the `EraTxOut`
fba18d078 Fix the changelog and add an entry for ConwayEra
c3a46e042 Introduce barebones for the new Conway Era
d10846cec Reduce duplication by creating a helper function `initialStateFromGenesis`
b56be4778 Create helper lens functions for defining EraTxOut and EraTxBody instances in future eras
c7c63dabd (tag: node/1.35.3, tag: ledger/1.0.0) Merge pull request #2938 from input-output-hk/jc/fix-alonzo-minfee-on-release-1-0-0

https://github.com/input-output-hk/cardano-ledger/compare/c7c63dabdb215ebdaed8b63274965966f2bf408f...49613f11e034485c8377945c86c0465726c0860c does not look like we could (or should) trivially assume we can work from what's referenced in network. (It also bumps plutus).
we therefore should create a 9.2 compatible release based off of tag 1.0.0 (c7c63dabdb215ebdaed8b63274965966f2bf408f), which is what this PR tries to achieve.

depends on https://github.com/input-output-hk/plutus/pull/4945

@JaredCorduan can we get a proper release/<IP_ADDRESS> branch (that's he version number I assume?) from f49879a79? It's effectively what we use today in cardano-node@master, but CHaP references c7c63dabdb215ebdaed8b63274965966f2bf408f for <IP_ADDRESS>.

Ahh, ok. So here we are now:
c7c63dabd (tag: node/1.35.3, tag: ledger/1.0.0) Merge pull request #2938 from input-output-hk/jc/fix-alonzo-minfee-on-release-1-0-0
7bec14d52 the alonzo UTxO rule to use alonzo minfee function
ebcf1a893 Merge pull request #2917 from input-output-hk/jc/update-plutus-on-release-1-0-0
e54e85a18 update plutus
f49879a79 Merge pull request #2909 from input-output-hk/jj/nothunks

@newhoggy this looks pretty much like https://github.com/input-output-hk/cardano-ledger/compare/release/1.0.x...release/1.0.x-ghc-9.2.4, (#3081), so you might have doe this before already :-) We just need to resurrect the PR!

@newhoggy @angerman - is this draft PR still needed, or can we close it?

This PR appears to be stale, I'm going to close it for now, but please feel free to re-open it if it is still relevant.


Add UCAS downtime banner wherever we link off to the site

Context
UCAS is planning downtime on 20 - 22nd March so we should inform our users that this is happening as there are a number of places we link off to the site.
Changes proposed in this pull request
This PR adds a banner to inform candidates of the downtime which is hidden under a feature flag called banner_for_ucas_downtime wherever we link off to UCAS i.e. wherever we call UCAS.apply_url.
Screenshot

Guidance to review

Copy and pasta okay for this? 🍝 Could be a component but seems overkill.
Didn't add a system spec for this. Should I?

Link to Trello card
https://trello.com/c/vyOK6UCd/1154-update-content-ucas-down-time
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

No need for a system spec but yes do make this a component 👍 will approve after!

Definitely happier with this now! 🙇♀


Automatically make deleted dandisets "private"

"Motivation" user case:

dandi/dandisets#388

I think that upon detection of deletion of a dandiset we should automatically

make github repo private
adjust .gitmodules entry to annotate that it is private as was done in 894ad5f

Unless some critical use-cases (like redistribution of illegal materials etc), I do not think we should remove them entirely for a number of reasons:

AINAL: original terms&conditions allowed open resharing/archival so we should be in clear to retain  a private copy e.g. for the cases where deletion might have been accidental, would later be needed for reproduction, etc
might have had releases and was used in publications (reproduction) so might be worth having exact copy to later discuss with original authors on ability to reshare etc.

yeap, moved to here


[move] Async Move Examples.

This PR adds some examples for asynchronous programming in Move. We use a simple account, where accounts live on different actors. One can deposit, withdraw, and transfer between those accounts. The transfer is the interesting operation because it requires a roundtrip communication between actors: only if deposit on end is successful, should the money be withdrawn on the other.
There are three versions of this example:

With continuations.
With futures.
With plain message passing and an explicit state machine.

Motivation
Async Move
Have you read the Contributing Guidelines on pull requests?
Yes

Another concern that I'm having is about msg deduplication. Do you think the rpc retry should be handled by some external dedup mechanism?

Now contained in #142


Server returning 500 error on sync/watched

When I run the following command I get a 500 server error. Is the server experiencing problems?
GET https://api.trakt.tv/sync/watched/shows
Request Headers
Content-Type: application/json
trakt-api-version: 2
trakt-api-key: xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
Authorization: Bearer xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
User-Agent: PostmanRuntime/7.29.2
Accept: /
Postman-Token: b88240b4-4a6e-40d8-865e-6a0efdc770bb
Host: api.trakt.tv
Accept-Encoding: gzip, deflate, br
Connection: keep-alive
Response Headers
Date: Sat, 21 Oct 2023 02:32:27 GMT
Content-Type: application/json
Transfer-Encoding: chunked
Connection: keep-alive
x-frame-options: SAMEORIGIN
x-xss-protection: 0
x-content-type-options: nosniff
x-download-options: noopen
x-permitted-cross-domain-policies: none
referrer-policy: strict-origin-when-cross-origin
Cache-Control: private, no-store
vary: Accept-Encoding
x-ratelimit: {"name":"AUTHED_API_GET_LIMIT","period":300,"limit":1000,"remaining":990,"until":"2023-10-21T02:35:00Z"}
x-request-id: ef9dcb4c-c703-468c-a263-03f2e853930a
x-runtime: 0.032077
CF-Cache-Status: DYNAMIC
Server: cloudflare
CF-RAY: 819617147ca5174e-SJC
alt-svc: h3=":443"; ma=86400
Response Body

Should be fixed now.


feat(firestore-translate): text multiple fields update

Testing
Single

Multiple

Codecov Report

Merging #421 into next will increase coverage by 2.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             next     #421      +/-   ##
==========================================
+ Coverage   39.77%   41.79%   +2.01%
==========================================
  Files          19       19
  Lines         636      658      +22
  Branches       85       88       +3
==========================================
+ Hits          253      275      +22
  Misses        381      381
  Partials        2        2

Impacted Files
Coverage Δ

firestore-translate-text/functions/src/index.ts
100.00% <100.00%> (ø)

LGTM.

Hi! Is this functinoality available in extension?  I've tried like this but didn't work(this caused field name of "content,title" as the target field to translate. not "content" and "title" ).


Not detecting blog articles

I'm transferring a site from docpad to middleman, currently stuck at getting middleman-blog to work.
My config: https://github.com/Komodo/komodo-website/blob/middleman/config.rb#L4
Note some articles have titles that do not match the filenames, but a lot of them do. I'd imagine it should pick those up as needed.
Did I forget something here? Am I doing something wrong?
Edit: To clarify - this page remains completely empty: https://github.com/Komodo/komodo-website/blob/middleman/source/blog.html.erb
Dumping "page_articles" returns []
And the actual blog articles do not reflect the blog.permalink setting.

Hey @Naatan
Two things you can try:

What do your blog article filenames look like?
When using middleman-blog for the first time, I noticed that the file extension setting needs to be explicit. i.e. .md won't match on files that have e.g. a .md.erb extension.

Have you got YAML front-matter in each of your articles?
Try adding the YAML block to one your article files

Most of them have an extension of .html.md, you can see them here: https://github.com/Komodo/komodo-website/tree/middleman/source/blog
All files have YAML front matter. I can view the articles through middleman but they do not seem to be interpreted by middleman-blog.

Try the following in your config.rb:
blog.default_extension = ".html.md"

No dice

Hmm, can you try putting the articles loop in the main template file rather than a partial?

I have tried that, but it did nothing.
Is there some way I can debug this?

Ah, I see it. Sorry.
Because you set a prefix in config to blog, you don't need to also add it to the sources value. Doing do makes the source path look something like: prefix + source = 'blog/blog/{year}-...
Remove the blog/ prefix on your sources value.

Thanks @tdreyno - that did it! :)


Search also for init*.so files when detecting package during cimport

Fixes #2886

I don't know if there's any value in using importlib.machinery.all_suffixes() (https://docs.python.org/3/library/importlib.html#importlib.machinery.all_suffixes) at runtime as the definite list of acceptable suffixes.
To be clear, I haven't given this a lot of thought, so there might be obvious issues with it. But I believe it's what Python does

I don't know if there's any value in using importlib.machinery.all_suffixes() (https://docs.python.org/3/library/importlib.html#importlib.machinery.all_suffixes) at runtime as the definite list of acceptable suffixes.

I have checked that and it generates following suffixes:
>>> importlib.machinery.all_suffixes()
['.py', '.pyc', '.cpython-39-darwin.so', '.abi3.so', '.so']
>>> importlib.machinery.EXTENSION_SUFFIXES
['.cpython-39-darwin.so', '.abi3.so', '.so']

We can use it and I like it because it can help us avoid using regexes etc. but I am not sure about one point. Does it make sense to to think about "cross compiling" - Cython to detect .so extensions on windows and vice versa and also "cross compiling" between python versions - to detect python 3.8 .so files in Cython running on python 3.9? In my opinion no but I want to double check that. If the answer is no we don't need I think we can use it.

Not complete sure about "cross-compiling" I think. More realistically this would be when there's an __init__.cpython-38-something.so and you're using Python 3.9 or similar.
My feeling if we should probably match as widely as possible. So what you have now is likely better. But I'm not certain.

My feeling if we should probably match as widely as possible. So what you have now is likely better. But I'm not certain.

As mentioned those functions are available only since python 3.1 so cannot be used until Cython 3.1 in any case...

The PR seems require more complex changes. Hence removing the milestone and marking as draft. Let's see when I will have time for deeper investigation.


[BUG] build-local.sh is broken on branches release/2.8 and release/2.9

Running a command like:
TARGET_REPO=$REPO/rancher:$TAG ./dev-scripts/build-local.sh
no longer works because the docker buildx build command in that script
is still building in the ./package directory but as of commit b4a6197c098ff in
release/v2.9 and commit 93f4cc8dd2e30c in release/v2.8, the Dockerfile is now trying
to run go mod download ; go build ... in the package directory, which
doesn't contain go files.

I created a local partial fix for this sometime this week, the changes can be seen here: https://github.com/rancher/rancher/compare/release/v2.9...mallardduck:dev-local-build
I think next week I will submit a formal PR to improve ease of use for the local dev build process.


Input tensor on GPU in C++ API

I am trying to feed a Tensor (using the C++ API) that has memory allocated on the GPU (using GPUBFCAllocator) into a network.
Now, the placeholder in the network is on the GPU (I checked this in Tensorboard), and the memory allocated for the input tensor is on the GPU, but whenever I run the network, nvprof --print-gpu-trace shows me [CUDA memcpy HtoD] and [CUDA memcpy DtoH], before the computations (i.e. convolutions etc.) start.
This suggests to me that the input tensor is being copied to CPU memory, and then back to GPU memory.
While debugging this, I found multiple hints in the source that seem to suggest the CPU is always used as device to feed tensors from.
See e.g.:
https://github.com/tensorflow/tensorflow/blob/eb56a8af24695bf8258addf28b0c53fbabff72e6/tensorflow/core/common_runtime/direct_session.cc#L264
https://github.com/tensorflow/tensorflow/blob/429905c6283d3182a816487807e97e592849ce19/tensorflow/core/common_runtime/graph_runner.cc#L109

Is this analysis correct?
How can one feed in a tensor that has memory allocated on GPU memory, without copying back and forth to CPU memory? If this is currently not possible, then I think this would be a good feature to add.
Especially when one wants to combine Tensorflow input/output with other algorithms (not in TF), one might want to keep data on the GPU, to avoid host to device and device to host transfers.

Thanks in advance.

Your analysis is correct.  GPU tensors are always fed by establishing a CPU tensor and copying its value to the GPU.  In the other direction, when reading the output of a graph computation, if the value you want to read is produced on the GPU, it must be copied back to the CPU before producing I/O e.g. writing to a file or window.
The basic reason is that the GPU is simply an auxillary processor with very limited capability to engage in  I/O operations.   It is not feasible to e.g. read from local disk directly into GPU memory without first staging through CPU memory.  In principle this should not pose a performance problem, so long as you're able to to buffer input into CPU RAM ahead of it's being needed on the GPU.
If you want to combine a TF program with a non-TF program that both run on GPU, without copying data back and forth to CPU RAM, I think you will need to arrange for input/output values to be in Vars that are local to the GPU, and somehow make them accessible to the non-TF program.  Alternatively, maybe you can package your non-TF program as a single TF Op.

qq

Thanks @poxvoculi for your quick reply and detailed explanation.
I understand that reading from/to disk etc. into GPU memory without passing through CPU memory is not feasible. However, that is not what I am trying to do. I am trying to combine a TF progam with a non-TF program that both run on GPU, as you mention.
Your last suggestion (using Vars) seems feasible, but it also seems like a bit of a workaround. Especially in high-performance environments, I can imagine that I would not be the only one using this functionality.
The fact that data is copied to CPU memory and then back to GPU memory when running a graph with an input tensor on GPU memory, suggests to me that TF is detecting that this tensor is on GPU memory.
So wouldn't it be easier to directly work from an input tensor if it is on GPU memory, and transfer it CPU -> GPU if it is on CPU memory?
Similarly there could be an option I think to specify whether output tensors should be copied back to CPU memory or not.
This would give the flexibility needed I think.
Thanks and looking forward to hear your thoughts.

OK, if your real question is "How do I get a GPU-using TensorFlow program to cooperate effectively with a non-TF GPU program?" that's a whole collection of difficult issues.
The short, easy answer is to pipeline them, with the inputs/output staged through CPU RAM, as it sounds like you're doing.  Anything beyond that gets into tricky stuff that I probably shouldn't even suggest.
Just for illustrative purposes,  here are some thoughts about some of the problems.
Suppose you're willing to invoke the programs as separate processes, and just want to avoid the CPU RAM staging of values passed between them.  TensorFlow allocates all GPU RAM by default, because that allows us to do memory management more efficiently than by using cudaMalloc.  Via an option one can request that TF only allocate part of the total.  TF can only apply operations to Tensor typed memory regions that it has allocated.  It should be possible to declare a Var to be resident to a GPU, which provides a long-term storage with a static location, but there's no way to specify an address at which that Var should allocate.  Once the Var has been established, a TF graph program can read and store its value.  With considerably hackery on your part, it might be possible to modify the TF runtime to capture the GPU RAM address of an allocated Var in which you're interested, and make it available to another process.  While the TF program is still live, holding the allocated memory, but idle, it might be possible to start another program which uses the GPU and can read/write a location it hasn't allocated itself via pointer.  (GPUs are not (yet) virtualizable, so I think a second process will see the same memory contents and address space as the first.)  So this might point to a crude way of alternating the actions of two separate GPU-using programs on some shared memory, so long as only one of them is a TF program.
Alternatively, you might want to call a non-TF program in the middle of a TF graph, e.g. by wrapping it in a new Op.  This could work, if you're willing and able to rewrite that non-TF program to obey the TF GPU runtime assumptions about use of stream executor contexts, memory allocators, etc. which are pretty non-standard in the CUDA world.  Simply trying to link and call a pre-existing GPU utility is unlikely to work.
Hope this clarifies things.

Thank you again for your quick reply.
That is exactly the question. My problem does not relate to calling a non-TF program in the middle of a TF graph.
Let's say program A runs on the GPU (non-TF) and I want to feed its output (which is in GPU memory) into a TF network (program B, which uses the TF C++ API).
I understand that the easiest way is to do: copy output of A back to CPU memory -> run program B. TF will then internally copy the data to GPU memory and start running the graph.
Now, I'm looking for a way to bypass the process of staging it through CPU RAM, because from a performance point of view this seems to be unnecessary. It would be more efficient if we could do a memory copy from GPU memory to GPU memory, i.e. copying the output of program A to the allocated memory for the input tensor of program B, both living on GPU memory.
As you mention there are some problems there.
I understand that TensorFlow cannot operate on memory that it didn't allocate, but I am using a TF allocator:
tensorflow::GPUBFCAllocator* allocator = new tensorflow::GPUBFCAllocator(0, sizeof(float) * height * width * 3);
tensorflow::Tensor input_tensor = tensorflow::Tensor(allocator, tensorflow::DataType::DT_FLOAT, tensorflow::TensorShape( { 1, height, width, 3 }));
<copy output data from program A into the GPU memory allocated by input_tensor using a GPU->GPU copy>

So TF allocated a Tensor on GPU memory, holding our input data (i.e. the output of program A), and I do not see why TF will now still insist on staging this through CPU RAM. All of this we can do at runtime.
I think it would be a good feature addition to skip staging through CPU RAM if the input tensor has its memory allocated on the GPU using a TF GPU allocator.
Similarly, there could be a flag to keep output tensors on GPU memory and not stage them back to CPU RAM. It would then be the users responsibility of course to deal with this, but it will allow users to not waste any time copying back and forth to CPU RAM in performance-critical environments.
The solution using Vars could work I think, but I think it is a bit of a workaround and making this a feature could be useful to more users who are trying to optimize setups where TF is getting input/output from other programs operating with GPU memory.
What are your thoughts on this?

The session->Run() function computes the the subgraph closure induced by the specified outputs, in other words, it computes forward from consts, Vars or fed Nodes only along paths that lead to the requested outputs.   Given this situation, if you already have figured out how to copy the values computed by A into a TF allocated tensor, why not copy them into a GPU resident Var, instead of into some other kind of tensor, and skip the explicit feed argument to Run()?  If you're working in C++, you can use DMAHelper to get the address of the backing buffer for the memcpy.
I agree it could be handy to have Session::Run() recognize that a feed tensor may already be GPU resident.  I guess that whoever wrote that code assumed it would never happen, so it could take a while to discover everything that needs to be fixed.

Yes, using Vars would indeed be a solution for now, but I still think (as you mention) that support for using feed tensors for this would be welcome.
Do you think this could be flagged as 'enhancement'?
Thank you for the pointer to DMAHelper.

larsmennen did solved this issue? i have the same problem

I would also be interested in knowing how to solve this issue. From my understanding, tensorflow::ops::Variable does not expose the tensor so we can memcpy into it. Is there any other way to get the cuda pointer from the Variable?

Same problem here, I'm working with video in GPU memory and for me copying data to CPU and back to GPU again is a problem, I'm working in realtime and copies kill performance. Is there any update on this issue?

Update: We do have an experimental means of feeding/fetching the GPU memory of Tensors, added in commit a1d6179adb1ca6208281ed955860c319525edf75
This is subject to change, but something y'all may want to try.
See the unittest https://github.com/tensorflow/tensorflow/blob/f3ed7f7e836da4f0ca1cb04cadce938744932b72/tensorflow/core/common_runtime/direct_session_test.cc#L1897
for an example.
This is all in the runtime implementation, there are no current plans to surface this into higher level APIs (or Python) at this time.

+1 on this fueature, especially for python.
Let me give a definitive use case, for same data different parameter / architecture jobs, we have the same data-set being used to train and evaluate over and over again. If we have 100 different combinations to try, this means we will copy from disk -> cpu mem -> gpu mem 100 times. ALL nvidia documentation suggests limiting data transfer between device and host as the primary optimisation.
Currently TF is a blocker for us using in certain production use cases because of this.

+1 too, need to feed from GPU memory

Tried @asimshankar 's suggested experimental code, it works, thank you! Any time frame for when this will become an API not likely to change at any moment?


Review models for our project

[ ] Read into the literature to find models suitable for our project

[ ] Document why these models are suitable

[ ] Choose one of the models implemented in MMAction2 that we can use

[ ] (Optional) Draft up da paragraph for this model selection for the final report.

Thanks for the heads up, I will start digging into this tomorrow as I am currently busy typing up a draft for my thesis proposal.
I'll have to find out more, but at first glance, the lack of variety and the age of the models seems a bit worrying(?) – I'd be curious to find out how they perform on various datasets.

I've been reviewing models today. Something that I'm wondering about is whether temporal action localization is an appropriate choice for human fall detection specifically.
Looking at the models available in MMAction2, they seem to address the problem of temporal-action proposal generation (TAPG). The popular benchmarks for that problem are THUMOS-14 and ActivityNet-1.3. These datasets include large sets of videos with highly varying scenes, scenarios, and actions.
My impression is that TAPG is a hard problem to solve, and the accuracies for state of the art models do not seem to be very high, especially compared to what human fall detection (HFD) algorithms seem to be able to produce.
I’d be curious to see how TAPG performs on human fall datasets specifically, but I wonder if we’re running the risk of taking an overly generalized solution (TAPG) and applying it to something very specific (HFD), losing a lot of accuracy in the process only to get the temporal features.

For the models available in MMAction2, the TCANet paper did a comparison with the other two models and their model outperformed BSN and BMN, so that would be the model to go for if we're opting for TAPG and the MMAction2 framework.


Manual hacks to get no_inline amber script passing

Requires https://swiftshader-review.googlesource.com/c/SwiftShader/+/42368

Thanks I am taking a look now

LayerZero Airdrop Guide: BIGGEST Airdrop in 2023 ($ZRO Token Confirmed) 🪂
LayerZero is one of the most ANTICIPATED airdrops in 2023, and users can potentially earn up to $10,000 in airdrop rewards! This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Claim Now on Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Share on Social Media:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Reddit

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.


Error handling serviceAccountId attribute when assigning stellio-creator role in Keycloak

I am following the Stellio Context Broker tutorial to integrate with Keycloak.
I was able to create a client in Keycloak successfully, but when I tried to assign the stellio-creator role to the service account of the client, I encountered an issue.
Stellio logs indicate that the serviceAccountId attribute is not recognized.
Here is the Stellio's log.
stellio-search-service        |  2024-08-25 23:28:55,741 [ntainer#0-0-C-1] DEBUG c.e.stellio.search.listener.IAMListener  - processIam - Received IAM event: {"tenantName":"urn:ngsi-ld:tenant:default","operationType":"ENTITY_CREATE","entityId":"urn:ngsi-ld:Client:8c78c950-98d9-431f-9600-02804221901b","entityTypes":["Client"],"operationPayload":"{\"id\":\"urn:ngsi-ld:Client:8c78c950-98d9-431f-9600-02804221901b\",\"type\":\"Client\",\"clientId\":{\"type\":\"Property\",\"value\":\"test-client\"}}","contexts":["https://easy-global-market.github.io/ngsild-api-data-models/authorization/jsonld-contexts/authorization.jsonld","https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context-v1.7.jsonld"]}
stellio-search-service        |  2024-08-25 23:28:55,832 [atcher-worker-2] DEBUG c.e.stellio.search.listener.IAMListener  - invoke - Successfully handled event ENTITY_CREATE for resource urn:ngsi-ld:Client:8c78c950-98d9-431f-9600-02804221901b
stellio-search-service        |  2024-08-25 23:29:04,200 [ntainer#0-0-C-1] DEBUG c.e.stellio.search.listener.IAMListener  - processIam - Received IAM event: {"tenantName":"urn:ngsi-ld:tenant:default","operationType":"ATTRIBUTE_APPEND","entityId":"urn:ngsi-ld:Client:8c78c950-98d9-431f-9600-02804221901b","entityTypes":["Client"],"attributeName":"roles","operationPayload":"{\"type\":\"Property\",\"value\":[\"stellio-creator\"]}","updatedEntity":"","contexts":["https://easy-global-market.github.io/ngsild-api-data-models/authorization/jsonld-contexts/authorization.jsonld","https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context-v1.7.jsonld"]}
stellio-search-service        |  2024-08-25 23:29:04,202 [ntainer#0-0-C-1] DEBUG c.e.stellio.search.listener.IAMListener  - processIam - Received IAM event: {"tenantName":"urn:ngsi-ld:tenant:default","operationType":"ATTRIBUTE_APPEND","entityId":"urn:ngsi-ld:Client:8c78c950-98d9-431f-9600-02804221901b","entityTypes":["Client"],"attributeName":"serviceAccountId","operationPayload":"{\"type\":\"Property\",\"value\":\"urn:ngsi-ld:User:6c25546d-4917-4cd2-88ee-71ba2f1a474d\"}","updatedEntity":"","contexts":["https://easy-global-market.github.io/ngsild-api-data-models/authorization/jsonld-contexts/authorization.jsonld","https://uri.etsi.org/ngsi-ld/v1/ngsi-ld-core-context-v1.7.jsonld"]}
stellio-search-service        |  2024-08-25 23:29:04,204 [atcher-worker-1] ERROR c.e.stellio.search.listener.IAMListener  - invoke - Error while handling event ATTRIBUTE_APPEND for resource urn:ngsi-ld:Client:8c78c950-98d9-431f-9600-02804221901b: BadRequestDataException(message=Received unknown attribute name: serviceAccountId)

Could you please confirm if the authentication method described in the tutorial is out-of-date?
If so, could you provide guidance on the updated authentication setup?

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

I think the above comments are spam.

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Indeed, we did a few changes with respect to Keycloak integration very recently that may have broken the tutorial, I gonna check it.
PS: are you using the last release of Stellio?

Thinking about it, it may be because the tutorial is still referencing the previous Keycloak version (23.0.6) and not the current one (25.0.2)

Indeed, we did a few changes with respect to Keycloak integration very recently that may have broken the tutorial, I gonna check it.
PS: are you using the last release of Stellio?

Yes, I'm using the latest version of Stellio.

Thinking about it, it may be because the tutorial is still referencing the previous Keycloak version (23.0.6) and not the current one (25.0.2)

Well, I have noticed that the Keycloak version in the tutorial is old and there is a newer version in the docker hub.
I also tried the newer version but encountered the following error, so I have quit using the latest version of Keycloak.
keycloak  | 04:58:03,783 FATAL [org.jboss.as.server] (main) WFLYSRV0239: Aborting with exit code 1

If you already have the documentation to make it compatible with the latest Keycloak, I would like to know.

Thinking about it, it may be because the tutorial is still referencing the previous Keycloak version (23.0.6) and not the current one (25.0.2)

Well, I have noticed that the Keycloak version in the tutorial is old and there is a newer version in the docker hub. I also tried the newer version but encountered the following error, so I have quit using the latest version of Keycloak.
keycloak  | 04:58:03,783 FATAL [org.jboss.as.server] (main) WFLYSRV0239: Aborting with exit code 1

If you already have the documentation to make it compatible with the latest Keycloak, I would like to know.

I'll check this tomorrow morning.

I fixed a few things in the documentation and validated it locally, it should be OK now.

@bobeal
Thank you so much for your help!
The problem has been resolved.
To remove an old database and work it properly, I had to do docker compose down -v for Keycloak.
Now both Stellio and Keycloak are working with their latest versions.

Good to hear that. Closing this issue then.


[ISSUE-837]Add NOTICE

Fixes ISSUE #837
Modifications

Add NOTICE file
Add registry plugin into distribution

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@ruanwenjun It would be better to split into several small prs.

@ruanwenjun It would be better to split into several small prs.

Done

I didn't change eventmesh-protocol-plugin/eventmesh-protocol-http/out/production/resources/META-INF/eventmesh/org.apache.eventmesh.protocol.api.ProtocolAdaptor this is a BUG?


allow preflight OPTIONS request

Would be good to handle preflight request, currently it's hard to query rpc node with browser app, browser tries to do OPTIONS request before POST (it asks server, which request methods are allowed, allowed origins and etc.), but getting
{"id":null,"jsonrpc":"2.0","error":{"code":-32602,"message":"Invalid Params","data":"invalid method 'OPTIONS', please retry with 'POST'"}}
It's this part of code responsible for that in pkg/services/rpcsrv/server.go, which retrict any methods that not POST

Actually what can be done is - return successful response with allow methods (only POST in this case), like described in mdn docs click
Doing that will let browser to successfully continue POST request :thinking:

Well, that means an extension of EnableCORSWorkaround kludge we have. It's really a misuse of CORS and proper setups should have some reverse proxy that handles it, but looks like proper setups are not that popular.


Include idea plugin into release build pipeline

Description:
At the moment for each release, we manually build it.

We are not releasing idea plugin anymore, hence i am closing this issue


Link from migrations module to embed_migrations!

The migration docs should mention embed_migrations!.

@Eijebong I'll take this if that's cool

Sure ^^


pkg/metrics: include revision and arch info in cilium_version

This will be helpful if users are running different revisions and/or on different platforms, e.g. how many linux/amd64 as well as linux/arm64 cilium nodes they have in the cluster(s).
Signed-off-by: ArthurChiao<EMAIL_ADDRESS>

@aanm Is this one ok to backport?

@tommyp1ckles since this one is changing the metrics labels I believe it might be more suitable to backport it to 1.13 only.


[Coverity CID :197612]Integer handling issues in /lib/os/printk.c

Static code scan issues seen in File: /lib/os/printk.c
Category: Integer handling issues
Function: z_vprintk
Component: Other
CID: 197612
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

*** CID 197612:  Integer handling issues  (CONSTANT_EXPRESSION_RESULT)
/lib/os/printk.c: 197 in z_vprintk()
191     u32_t u;
192
193     if (long_ctr == 0) {
194     u = va_arg(ap, unsigned int);
195     } else if (long_ctr == 1) {
196     long lu = va_arg(ap, unsigned long);

CID 197612:  Integer handling issues  (CONSTANT_EXPRESSION_RESULT)
"lu ><PHONE_NUMBER>" is always false regardless of the values of its operands. This occurs as the logical operand of "if".

197     if (lu > INT32_MAX) {
198     print_err(out, ctx);
199     break;
200     }
201     u = (u32_t)lu;
202     } else {

This is a duplicate of 13843, Coverity triggered due to:
commit 4aa4883 (upstream/topic-sensors)
Author: Patrik Flykt<EMAIL_ADDRESS>Date: Thu Mar 14 19:42:51 2019 -0600
subsystems: Rename reserved function names


Update Exchange-Teams-interact.md

At the end the autodiscover v2 is based on rest api calls
https://docs.microsoft.com/en-us/graph/hybrid-rest-support
for discovering the mailbox even in outlook the request will be sent via autodiscover v2 and after the oauth token will be requested.
we need to have an unified and consistent way on the article that states that we support exchange 2016 CU 3+

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dstrome PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Add API for serials

connect to https://github.com/bm-sms/daimon-news-misc/issues/102

Can you fix styling issue?

I will disable these checks.

Can you rebase?

Rebased

Almost works 👍
However some extra code and data in JSON API response.
Can you check these?

I found following patch about SiteSerializer#serials_count
commit 97967a6e8b3bfa7b19f192b665d4f3d90950b437
Author: Kenji Okimoto<EMAIL_ADDRESS>Date:   Thu Jun 9 18:07:26 2016 +0900

    Add serials_count to current_site attribute

    If we include serials in current_sites_controller.rb,
    GET `/api/localhost/sites/current` executes a lot of queries to fetch
    posts related to all serials.
    We need presence or absence of serials.

diff --git a/app/serializers/site_serializer.rb b/app/serializers/site_serializer.rb
index a9048a1..a1ddcd0 100644
--- a/app/serializers/site_serializer.rb
+++ b/app/serializers/site_serializer.rb
@@ -17,10 +17,15 @@ class SiteSerializer < ActiveModel::Serializer
     :mobile_favicon_image_url,
     :head_tag,
     :home_url,
-    :ad_client
+    :ad_client,
+    :serials_count
   )

   has_many :categories
   has_many :serials
   has_many :links
+
+  def serials_count
+    object.serials.count
+  end
 end

review -> in progress

This PR has some TODOs:

[ ] Drop unnecessary relationships from JSON response
[ ] Rebase with master
[ ] Fix filtering

This PR has some TODOs:

All is done!


🎉 release: v9.1.1

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


Qemu improvements

Docs for b5e6ef5e45981f1472061d06fe1591939b6dd745 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/1244/docs/artifacts/html/index.html

LGTM


Plug Codecademy on Get Involved

Something that might be interesting is to try to give people a place to go to gain the skills they would need to get involved. For example in the web section we talk about the things we use to build our website. Maybe we should also link to Codecademy and let people know that there is an awesome and free way for them to gain these skills if they don't already have them.

perhaps have several resources? There are quite a few, and not all resources work for all people. Freecodecamp, The Odin Project and code.org all spring immediately to mind.

We should probably collect a comprehensive list, rank by the amount of applicable lessons and price, then shortlist from there.

Codecademy
Code School
Teaching Tree
Linux Academy
Free Code Camp
The Odin Project
Code.org

Codecademy is like 90% free. The offer extra paid projects, but you can learn 100% of the lessons without paying.
I think I would disqualify freecodecamp.com just on the way their website looks. I'm not sure I trust someone to teach web skills when their website looks like it does.
The Odin project seems like a text book and structured like a college course list, which I don't think is as good as something that's interactive and teaches the skills directly instead of hiding what is actually in the lesson behind "Web 101".
I can't even find proper web development courses on code.org

I think that a link to something like MDN would be useful:
https://developer.mozilla.org/en-US/Learn
In addition to providing a handy reference for tags and stuff, their intro to web dev was short and sweet. I think they start with installing a text editor and end with publishing a web site via github pages. I dont think it would be necessary to link stuff like the odin project and free code camp because both programs are set up as a free "bootcamp" for people who want a web dev job.

I have done courses on freecodecamp I would say their courses are not that bad for beginners however their website might look

I think this is a great idea! I'd be happy to help out with this :)

@danrabbit I'm trying FreeCodeCamp right now and yes it looks ugly, but the courses are free, the philosophy is great and the method is awesome ! So to disqualify it just because it's UI is not great.


Shem's Suggestions

Implemented some of the suggestions from @andela-sogumbe

Coverage increased (+1.0%) to 100.0% when pulling c8d8ebec5758ac5f712e5b961302163347ee2931 on develop into 26a1ec7d2c979b096aeaae31ab50c9824cbc1ebc on master.


Logging file for hls_lfcd_lds_driver

When bringing up the turtlebot3 waffle-pi, all processes launch except for the hls_lfcd_lds_driver. It crashes with the following message:
[ERROR] [1530514807.727153169]: An exception was thrown: open: No such file or directory [turtlebot3_lds-2] process has died [pid 1985, exit code 255, cmd /home/pi/catkin_ws/devel/lib/hls_lfcd_lds_driver/hlds_laser_publisher __name:=turtlebot3_lds __log:=/home/pi/.ros/log/1a14ac62-7dc4-11e8-95b1-10c37b9ff016/turtlebot3_lds-2.log]. log file: /home/pi/.ros/log/1a14ac62-7dc4-11e8-95b1-10c37b9ff016/turtlebot3_lds-2*.log
The file or directory which does not exist is not printed in any of the logs so I do not know which non-existent file is causing the issue.
Every other process seems to launch fine with no errors.

I have run your above mentioned commands and received the following output:
pi@raspberrypi:~/catkin_ws/src $ rm -rf hls_lfcd_lds_driver/
pi@raspberrypi:~/catkin_ws/src $ cd ~/catkin_ws/src
pi@raspberrypi:~/catkin_ws/src $ git clone https://github.com/ROBOTIS-GIT/hls_lfcd_lds_driver.git
Cloning into 'hls_lfcd_lds_driver'...
remote: Counting objects: 243, done.
remote: Compressing objects: 100% (13/13), done.
remote: Total 243 (delta 2), reused 4 (delta 1), pack-reused 229
Receiving objects: 100% (243/243), 61.77 KiB | 0 bytes/s, done.
Resolving deltas: 100% (106/106), done.
pi@raspberrypi:~/catkin_ws/src $ cd ~/catkin_ws && catkin_make -j1
Base path: /home/pi/catkin_ws
Source space: /home/pi/catkin_ws/src
Build space: /home/pi/catkin_ws/build
Devel space: /home/pi/catkin_ws/devel
Install space: /home/pi/catkin_ws/install
####
#### Running command: "make cmake_check_build_system" in "/home/pi/catkin_ws/build"
####
-- Using CATKIN_DEVEL_PREFIX: /home/pi/catkin_ws/devel
-- Using CMAKE_PREFIX_PATH: /home/pi/catkin_ws/devel;/opt/ros/kinetic
-- This workspace overlays: /home/pi/catkin_ws/devel;/opt/ros/kinetic
-- Using PYTHON_EXECUTABLE: /usr/bin/python
-- Using Debian Python package layout
-- Using empy: /usr/bin/empy
-- Using CATKIN_ENABLE_TESTING: ON
-- Call enable_testing()
-- Using CATKIN_TEST_RESULTS_DIR: /home/pi/catkin_ws/build/test_results
-- Found gtest sources under '/usr/src/gtest': gtests will be built
-- Using Python nosetests: /usr/bin/nosetests-2.7
-- catkin 0.7.8
-- BUILD_SHARED_LIBS is on
-- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-- ~~  traversing 10 packages in topological order:
-- ~~  - turtlebot3 (metapackage)
-- ~~  - turtlebot3_msgs
-- ~~  - turtlebot3_navigation
-- ~~  - hls_lfcd_lds_driver
-- ~~  - interactive_markers
-- ~~  - turtlebot3_bringup
-- ~~  - turtlebot3_example
-- ~~  - turtlebot3_slam
-- ~~  - turtlebot3_teleop
-- ~~  - turtlebot3_description
-- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-- +++ processing catkin metapackage: 'turtlebot3'
-- ==> add_subdirectory(turtlebot3/turtlebot3)
-- +++ processing catkin package: 'turtlebot3_msgs'
-- ==> add_subdirectory(turtlebot3_msgs)
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
-- turtlebot3_msgs: 3 messages, 0 services
-- +++ processing catkin package: 'turtlebot3_navigation'
-- ==> add_subdirectory(turtlebot3/turtlebot3_navigation)
-- +++ processing catkin package: 'hls_lfcd_lds_driver'
-- ==> add_subdirectory(hls_lfcd_lds_driver)
-- Boost version: 1.62.0
-- +++ processing catkin package: 'interactive_markers'
-- ==> add_subdirectory(interactive_markers)
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
-- +++ processing catkin package: 'turtlebot3_bringup'
-- ==> add_subdirectory(turtlebot3/turtlebot3_bringup)
-- +++ processing catkin package: 'turtlebot3_example'
-- ==> add_subdirectory(turtlebot3/turtlebot3_example)
-- Using these message generators: gencpp;geneus;genlisp;gennodejs;genpy
-- Generating .msg files for action turtlebot3_example/Turtlebot3 /home/pi/catkin_ws/src/turtlebot3/turtlebot3_example/action/Turtlebot3.action
-- turtlebot3_example: 7 messages, 0 services
-- +++ processing catkin package: 'turtlebot3_slam'
-- ==> add_subdirectory(turtlebot3/turtlebot3_slam)
-- +++ processing catkin package: 'turtlebot3_teleop'
-- ==> add_subdirectory(turtlebot3/turtlebot3_teleop)
-- +++ processing catkin package: 'turtlebot3_description'
-- ==> add_subdirectory(turtlebot3/turtlebot3_description)
-- Configuring done
-- Generating done
-- Build files have been written to: /home/pi/catkin_ws/build
####
#### Running command: "make -j1" in "/home/pi/catkin_ws/build"
####
[  0%] Built target _turtlebot3_msgs_generate_messages_check_deps_VersionInfo
[  0%] Built target _turtlebot3_msgs_generate_messages_check_deps_Sound
[  0%] Built target _turtlebot3_msgs_generate_messages_check_deps_SensorState
[  0%] Built target std_msgs_generate_messages_py
[  7%] Built target turtlebot3_msgs_generate_messages_py
[  7%] Built target std_msgs_generate_messages_cpp
[ 11%] Built target turtlebot3_msgs_generate_messages_cpp
[ 11%] Built target std_msgs_generate_messages_eus
[ 17%] Built target turtlebot3_msgs_generate_messages_eus
[ 17%] Built target std_msgs_generate_messages_lisp
[ 21%] Built target turtlebot3_msgs_generate_messages_lisp
[ 21%] Built target std_msgs_generate_messages_nodejs
[ 25%] Built target turtlebot3_msgs_generate_messages_nodejs
[ 25%] Built target turtlebot3_msgs_generate_messages
[ 25%] Built target geometry_msgs_generate_messages_nodejs
[ 25%] Built target rosgraph_msgs_generate_messages_lisp
[ 25%] Built target geometry_msgs_generate_messages_py
[ 25%] Built target rosgraph_msgs_generate_messages_cpp
[ 25%] Built target rosgraph_msgs_generate_messages_py
[ 25%] Built target roscpp_generate_messages_py
[ 25%] Built target rosgraph_msgs_generate_messages_nodejs
[ 25%] Built target sensor_msgs_generate_messages_nodejs
[ 25%] Built target roscpp_generate_messages_eus
[ 25%] Built target geometry_msgs_generate_messages_lisp
[ 25%] Built target rosgraph_msgs_generate_messages_eus
[ 25%] Built target roscpp_generate_messages_lisp
[ 25%] Built target roscpp_generate_messages_cpp
[ 25%] Built target geometry_msgs_generate_messages_cpp
[ 25%] Built target roscpp_generate_messages_nodejs
[ 25%] Built target sensor_msgs_generate_messages_cpp
[ 25%] Built target sensor_msgs_generate_messages_py
[ 25%] Built target sensor_msgs_generate_messages_eus
[ 25%] Built target geometry_msgs_generate_messages_eus
[ 25%] Built target sensor_msgs_generate_messages_lisp
Scanning dependencies of target hlds_laser_segment_publisher
[ 27%] Building CXX object hls_lfcd_lds_driver/CMakeFiles/hlds_laser_segment_publisher.dir/src/hlds_laser_segment_publisher.cpp.o
[ 28%] Linking CXX executable /home/pi/catkin_ws/devel/lib/hls_lfcd_lds_driver/hlds_laser_segment_publisher
[ 28%] Built target hlds_laser_segment_publisher
Scanning dependencies of target hlds_laser_publisher
[ 30%] Building CXX object hls_lfcd_lds_driver/CMakeFiles/hlds_laser_publisher.dir/src/hlds_laser_publisher.cpp.o
[ 31%] Linking CXX executable /home/pi/catkin_ws/devel/lib/hls_lfcd_lds_driver/hlds_laser_publisher
[ 31%] Built target hlds_laser_publisher
[ 41%] Built target interactive_markers
[ 41%] Built target diagnostic_msgs_generate_messages_lisp
[ 41%] Built target diagnostic_msgs_generate_messages_nodejs
[ 41%] Built target diagnostic_msgs_generate_messages_cpp
[ 41%] Built target diagnostic_msgs_generate_messages_py
[ 41%] Built target diagnostic_msgs_generate_messages_eus
[ 44%] Built target turtlebot3_diagnostics
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3Feedback
[ 44%] Built target actionlib_msgs_generate_messages_nodejs
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3ActionResult
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3Action
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3Goal
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3Result
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3ActionFeedback
[ 44%] Built target _turtlebot3_example_generate_messages_check_deps_Turtlebot3ActionGoal
[ 54%] Built target turtlebot3_example_generate_messages_nodejs
[ 54%] Built target actionlib_msgs_generate_messages_py
[ 65%] Built target turtlebot3_example_generate_messages_py
[ 65%] Built target actionlib_msgs_generate_messages_cpp
[ 75%] Built target turtlebot3_example_generate_messages_cpp
[ 75%] Built target actionlib_msgs_generate_messages_lisp
[ 85%] Built target turtlebot3_example_generate_messages_lisp
[ 85%] Built target actionlib_msgs_generate_messages_eus
[ 97%] Built target turtlebot3_example_generate_messages_eus
[ 97%] Built target turtlebot3_example_generate_messages
[100%] Built target flat_world_imu_node
[100%] Built target turtlebot3_description_xacro_generated_to_devel_space_

The below is my subsequent launch output with the error:
pi@raspberrypi:~/catkin_ws $ roslaunch turtlebot3_bringup turtlebot3_robot.launch
... logging to /home/pi/.ros/log/7d3db83a-7dc7-11e8-95b1-10c37b9ff016/roslaunch-raspberrypi-2194.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

started roslaunch server http://<IP_ADDRESS>:45601/

SUMMARY
========

PARAMETERS
 * /rosdistro: kinetic
 * /rosversion: 1.12.12
 * /turtlebot3_core/baud: 115200
 * /turtlebot3_core/port: /dev/ttyACM0
 * /turtlebot3_lds/frame_id: base_scan
 * /turtlebot3_lds/port: /dev/ttyUSB0

NODES
  /
    turtlebot3_core (rosserial_python/serial_node.py)
    turtlebot3_diagnostics (turtlebot3_bringup/turtlebot3_diagnostics)
    turtlebot3_lds (hls_lfcd_lds_driver/hlds_laser_publisher)

ROS_MASTER_URI=http://<IP_ADDRESS>:11311

process[turtlebot3_core-1]: started with pid [2204]
process[turtlebot3_lds-2]: started with pid [2205]
process[turtlebot3_diagnostics-3]: started with pid [2206]
[ERROR] [1530516973.110996474]: An exception was thrown: open: No such file or directory
[turtlebot3_lds-2] process has died [pid 2205, exit code 255, cmd /home/pi/catkin_ws/devel/lib/hls_lfcd_lds_driver/hlds_laser_publisher __name:=turtlebot3_lds __log:=/home/pi/.ros/log/7d3db83a-7dc7-11e8-95b1-10c37b9ff016/turtlebot3_lds-2.log].
log file: /home/pi/.ros/log/7d3db83a-7dc7-11e8-95b1-10c37b9ff016/turtlebot3_lds-2*.log
[INFO] [1530516974.186115]: ROS Serial Python Node
[INFO] [1530516974.220300]: Connecting to /dev/ttyACM0 at 115200 baud
[INFO] [1530516976.437317]: Note: publish buffer size is 1024 bytes
[INFO] [1530516976.439693]: Setup publisher on sensor_state [turtlebot3_msgs/SensorState]
[INFO] [1530516976.538217]: Setup publisher on version_info [turtlebot3_msgs/VersionInfo]
[INFO] [1530516976.554305]: Setup publisher on imu [sensor_msgs/Imu]
[INFO] [1530516976.570550]: Setup publisher on cmd_vel_rc100 [geometry_msgs/Twist]
[INFO] [1530516976.674707]: Setup publisher on odom [nav_msgs/Odometry]
[INFO] [1530516976.690645]: Setup publisher on joint_states [sensor_msgs/JointState]
[INFO] [1530516976.706390]: Setup publisher on battery_state [sensor_msgs/BatteryState]
[INFO] [1530516976.810571]: Setup publisher on magnetic_field [sensor_msgs/MagneticField]
[INFO] [1530516979.994298]: Setup publisher on /tf [tf/tfMessage]
[INFO] [1530516980.010809]: Note: subscribe buffer size is 1024 bytes
[INFO] [1530516980.012520]: Setup subscriber on cmd_vel [geometry_msgs/Twist]
[INFO] [1530516980.028822]: Setup subscriber on sound [turtlebot3_msgs/Sound]
[INFO] [1530516980.047168]: Setup subscriber on motor_power [std_msgs/Bool]
[INFO] [1530516980.067067]: Setup subscriber on reset [std_msgs/Empty]
[INFO] [1530516980.075170]: --------------------------
[INFO] [1530516980.078131]: Connected to OpenCR board!
[INFO] [1530516980.081164]: This core(v1.2.0) is compatible with TB3 Waffle or Waffle Pi
[INFO] [1530516980.084242]: --------------------------
[INFO] [1530516980.087465]: Start Calibration of Gyro
[INFO] [1530516981.987609]: Calibration End

Hi, again @erikgroving,
I'm not sure, but it seems to be a permission issue.
Have you run the 'create_udev_rules' script on the following link?
http://emanual.robotis.com/docs/en/platform/turtlebot3/raspberry_pi_3_setup/#4-usb-settings
$ rosrun turtlebot3_bringup create_udev_rules

I have just run the script, but am still getting the same error message.
up the turtlebot3 usb connection.

Reload rules```

Any ideas?

I'm sorry for the inconvenience. Um ... Let's check some things.

Connect Lidar to your Remote PC, not Raspberry Pi, and run it on your PC as described below. If the problem is the same on the Remote PC, it may be a Lidar problem. If it works without problems, there will be a problem with your Raspberry Pi configuration.
http://emanual.robotis.com/docs/en/platform/turtlebot3/appendix_lds_01/#user-guide-for-ros

Assuming a problem with SBC settings, delete all packages and build files that are currently installed and reinstall them.
http://emanual.robotis.com/docs/en/platform/turtlebot3/raspberry_pi_3_setup/#install-linux-ubuntu-mate

$ cd ~/catkin_ws/
$ rm -rf src/ build/ devel/
$ mkdir src
$ cd ~/catkin_ws/src
$ git clone https://github.com/ROBOTIS-GIT/hls_lfcd_lds_driver.git
$ git clone https://github.com/ROBOTIS-GIT/turtlebot3_msgs.git
$ git clone https://github.com/ROBOTIS-GIT/turtlebot3.git
$ cd ~/catkin_ws/src/turtlebot3
$ sudo rm -r turtlebot3_description/ turtlebot3_teleop/ turtlebot3_navigation/ turtlebot3_slam/ turtlebot3_example/
$ source /opt/ros/kinetic/setup.bash
$ cd ~/catkin_ws && catkin_make -j1

Well, the lidar had become unplugged from the small board so no wonder the corresponding process was always crashing! Thank you so much for your help and sorry about the silly mistake.


Removing autocompletion for regular commands

I'm trying to figure out how to disable all the regular LaTeX symbol autocompletions:

I'd basically like to only display autocompletion data related to things like labels, bibliographic citations, etc. rather than all the commands from packages!
I tried to read the cmp-vimtex documentation but could not figure out what the trigger would be.  Currently my sources are:
        sources = cmp.config.sources {
          { name = 'ultisnips', priority = 1000 },
          -- { name = 'omni', priority = 1000 },
          { name = 'vimtex', priority = 750 },
          -- { name = 'nvim_lsp', priority = 600 },
          -- { name = 'buffer', priority = 500 },
          -- { name = 'otter', priority = 500 },
          -- { name = 'path', priority = 250 },
        },

so I have only enabled ultisnips and vimtex sources. The long list of suggestions goes away if I comment out the vimtex sources line.

Sure, the Latex package-related commands are provided by Vimtex. Currently, I'm really short of free time, so my advice is to check Vimtex's documentation to see if there's a way to configure the omnifunc to fit your needs.


Move to Gorilla/mux for server routing

Current Behavior

We are using HTTProuter for routing & serving our UI pages.
Desired Behavior

Use Gorilla/mux for server routing, remove complex network logic from handlers and use mux UI handlers to server SSR next UI bundle.

Resources

Read this. article for a brief Comparision between HTTProuter & gorilla/mux.

Hey!!
Can I work in this
@kushthedude

Sure go ahead!
On Thu, 20 Aug, 2020, 00:24 Dev Kalra<EMAIL_ADDRESS>wrote:

Hey!!
Can I work on this
@kushthedude https://github.com/kushthedude
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/layer5io/meshery/issues/1410#issuecomment-676600646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLVMSWJSKGBJ5FFO3CDSBQNVDANCNFSM4QA7YO5A
.


Duplicate contact information are added to same campus partner while registering it(part-1).

Duplicate contact information are added to same campus partner while registering it.

Dev env, logged in as campus partner and admin user.
Master, logged in as campus partner and admin user.
Duplicatecontactinformaationadded.docx

Completed: added unique_together for class Contacts in home\models.py
In progress: checking how to use contactdataarray[] to check for duplicates on form.


Editor window - font-size vmin - it's a terrible experience...

I see some CSS classes related to ACE editor...
On my machine font-size is bound to height of the window, I cannot simply ctrl + to see more code on the screen, it's a terrible experience, limiting to only 36 lines of code...

Thanks @stefek99
@yann300 I prepared a PR for this https://github.com/ethereum/browser-solidity/pull/577

WAT
Did you manage to respond to my issue and provide PR in 20 minutes?

Lol, @stefek99 that was a coincidence :) But yes, I am also proud of that :)

should be better now, please reopen if not


Collocations and Twitter feeds problem

I've been trying to use collocations with Twitter feeds, and have run into a bit of an issue. When I download someone's Tweets with the Twitter API, and run nltk.collcations(), instead of common phrases and words, I get mostly Twitter handles. "@whosits @soandso; @bob @sam", etc. People often phrase Tweets in that way-"@Bob @Blah How did you like the movie?" and so collocations seems to be returning Twitter handles instead of common phrases and combos of words, like I expected. Should there be a built in method to strip Twitter handles from a Twitter feed, so you don't have to regex them out of a text editor? On the other hand, finding the most commonly mentioned Twitter handles in a feed is something very NLTK worthy, but the current collocations module seems not well suited for Twitter.

@ewan-klein do you have any thoughts on this?

@Amanda-Clark There is some ongoing work on the twitter branch of NLTK to provide better treatment of tweets. If you change to that branch, you'll find a tweet-aware tokeniser https://github.com/nltk/nltk/blob/twitter/nltk/tokenize/casual.py. In principle, we could define a method normalize() that strips out Twitter handles and URLs from a tweet text. However, looking at the demo at the end of https://github.com/nltk/nltk/blob/develop/nltk/collocations.py, it seems that the obvious thing to try, at least in the first instance, would be to apply a word_filter which returns True for any string that starts with @.

@ewan-klein Is there someone working on doing that? Sorry I'm not used to how (open source) projects are done in GitHub, or in general.

Regarding tokenization for Twitter: This would be nice to have in NLTK. In case you haven't seen it, Twokenizer does a lot of processing for emoticons, etc. (But unlike PTB-style tokenizers it does not attempt to separate clitics.)

@Amanda-Clark Can you clarify what you mean by 'doing that'?

@ewan-klein Sorry I was referring to what you said here: "However, looking at the demo at the end of https://github.com/nltk/nltk/blob/develop/nltk/collocations.py, it seems that the obvious thing to try, at least in the first instance, would be to apply a word_filter which returns True for any string that starts with @." Is there someone working on adding that to NLTK?


TypeScript, readonly arrays combined with where and in

Problem
I'm working a lot with readonly properties on a lot of my types, for immutability, because I think that makes the code better. I've just started messing around with the in filter, and it doesn't seem to work well with readonly arrays. For example:
        const emailAddresses: readonly string[] =<EMAIL_ADDRESS><EMAIL_ADDRESS>        await client.user.findMany({
            where: {
                emailAddress: {
                    in: emailAddresses, // Type 'readonly string[]' is not assignable to type 'Enumerable<string> | undefined'. The type 'readonly string[]' is 'readonly' and cannot be assigned to the mutable type 'string[]'. ts(2322)
                },
            },
        });

Suggested solution
I don't see any downsides of allowing readonly arrays with the in filter, and I can't see any technical constraints against it either (but I'm no TypeScript guru).
Alternatives
None.
Additional context
None.

I've submitted a PR for this :tada: #18572


vim_performance_daily extract time_profile options

High level: Use scopes/relations and get away from find
vim_performance_daily extract time_profile options
Introduce a scope so it can be used as a traditional AR scope / model
/cc @matthewd rails5 love (and performance too)

@matthewd thanks so much - TimeProfiles are very scope friendly

/cc @gtanzillo changed profile_for_each_region to return 1 time profile per region. (per the comment added)

:+1: LGTM


3DC multi branches cleanup

In 3+DC, the mutable state may have multi-branches info. When a workflow is cleanup, all branches need to be deleted. Currently, the logic is to remove all non current branches (not max version), this is due to archival only archive current branch. In the future, when archival supports archive multiple branches. The cleanup can delete all branches.

Fixes #1757


improve efficiency of binary morphology functions by accepting a tuple for footprint

This MR backports an efficiency improvement for binary morphology functions that were made in cuCIM's vendored copy of cupyx.scipy.ndimage code. See additional background in https://github.com/cupy/cupy/pull/8126#issuecomment-2081554814
The CUDA kernels themselves do not have to change to support this as we already previously merged support for not providing a weights array. The changes here just allow the higher level binary morphology functions to make use of that existing optimization.
For NumPy, allocating a small (e.g. 3x3) binary array of ones takes on the order of 1 microsecond, so this would be a micro-optimization if proposed for SciPy itself. For CuPy, though, the array creation time is > 10 microseconds and also involves synchronization.  In addition to the synchronization during array creation, the change here will also avoid a second synchronization that occurs during an internal check of whether all elements of the footprint array are true.

/test mini

Let me see if I can reproduce and resolve the failures locally (I think I only ran the full set of morphology tests locally previously, but failures here are mostly in min/max/rank filters)

/test mini


Handle obsolete app engine standard runtimes

Umbrella issue for dealing with obsolete/deprecated runtimes.

Code inspections + quickfix on old runtime
Project generation of old runtime removal (already complete)
(possible) warning on deployment of application with obsolete runtime
Eventually remove supporting logic of obsolete runtime (e.g. whitelisting)

oops wrong repo!


[Feature] Enhance Accessibility: Add ARIA Attributes to Local Content

Problem
The local content within the code lack ARIA attributes to improve accessibility. While global content such as the footer and header have some ARIA attributes implemented, local content sections are missing them, which can result in inconsistent accessibility support across the page.
Description
To enhance accessibility for users with assistive technologies, ARIA attributes should be consistently applied to local content sections such as form elements, navigation within sections, and interactive components (buttons, links, etc.). This would help screen readers and other assistive technologies navigate and understand the content better, improving the user experience.
Adding ARIA attributes to local content aligns with accessibility best practices and ensures a better user experience for those using assistive technologies. This also brings consistency between the accessibility of global content (header, footer) and local content.
Alternatives

Continue without ARIA attributes for local content, which may lead to reduced accessibility.
Implement ARIA attributes only for new content, leaving existing sections as they are, but this could cause inconsistency across the site.
Conduct a full accessibility audit to identify missing ARIA attributes and apply them site-wide, including global and local content.

Additional context

[link to resources and related issues]

Implementation

[ ] I would be interested in implementing this feature.

Hey, @TimidRobot I can work on this in a couple of days. Can you assign me? Since this still has a priority low I am guessing it is fine to delay it by 3-4 days


Upgrade Node version to LTS on webui folder

What does this PR do?
Bump node version to the latest LTS version based on the docker-hub tag
Motivation
The v12 release of Node entered a maintenance status. As a good practice, the base image should be moved to the next active LTS version
https://nodejs.org/en/about/releases/

Hello @leonardxfce,
Thanks for raising this issue. According to the link you referred to, Production systems should be using either active or maintenance LTS releases (of which v12 is currently in maintenance).
Do you think that it would be more appropriate to jump from v12 to v16 in the new year (Jan/Feb 2022) to prevent having to leapfrog smaller versions?
Also, we try to avoid rotating tagged images, like :latest, as we have no control over when they get changed. Do you think that it would be better to have something like this instead?
FROM node:14.16
# Current Active LTS release according to (https://nodejs.org/en/about/releases/)

Sound good to me. I can do the change to a more specific image. Give me some mins.

Pushed. Let me know if looks good for you. If not we can delay this kind of PR to next LTS release

Hello @leonardxfce,
Is it possible that you may have reverted a commit when you rebased? It appears that your change did not get pushed.
Thanks!


Work on real release system for ÖbEngine

Right now there is no real versions for ÖbEngine, we're stuck on 0.9.9 for a long time.
The goal is to implement real semantic versioning.
Build jobs from AppVeyor / Travis / Azure should only release on GitHub Releases when a new tag is created.

Done in master branch


Allow feeding of hyperparameters through argparse

As of now, users do not have the freedom to pass hyperparameters during the run.

still doing some pytorch revision, ill take a look as soon as im done :)

Sure. No hurry!

Quick question. I've noticed that you're example notebook goes through a molecular optimization, but does not return an output fingerprint example for inspection. Would we call get_fingerprint at the very end? Might be good to add in the starting molecule and ending molecule in the notebook, would convince folks that this adaptation is good.
(Apologies for the delay btw, I'm working on the RL nanodegree - I want to be 100% familiar with pytorch in RL before I get cracking on this.)

That's a good idea! Maybe we could give the user the freedom to decide the interval after which the molecule is displayed (as an argparse parameter)? We could then modify the example notebook to use this interval parameter. Let me know if you have other ideas on the way this functionality could be added.
Currently, all hyperparameter values are fixed in the hyp.py file and the optimization proceeds for 5000 episodes by default. Our first priority should be to allow users some flexibility on these decisions (through argparse).
Since you're working on RL, you might also be interested in issues #4 and #7 since they require some RL background to implement.

Yeah, I agree with the argparse comment.  It's important to allow for user customization.
Did you base your work on the original Google TF repo, or from scratch. In any case, it would be great if you  you briefly illustrate how much of the google repository has been covered, and what remains to be done (TODO).
I don't recall of the back of my head, but I think the Google repo was substantially longer than this project.
I love your Github Profile man, but can you illustrate your background a little bit?
If you don't mind, I'll get back to you in the weekend and crack out argparse.
Should we setup some sort of chatroom?
Regards

From: Akshay Subramanian<EMAIL_ADDRESS>Sent: 20 April 2020 09:41
To: aksub99/MolDQN-pytorch<EMAIL_ADDRESS>Cc: Xu, Yijie<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [aksub99/MolDQN-pytorch] Allow feeding of hyperparameters through argparse (#5)
That's a good idea! Maybe we could give the user the freedom to decide the interval after which the molecule is displayed (as an argparse parameter)? We could then modify the example notebook to use this interval parameter. Let me know if you have other ideas on the way this functionality could be added.
Currently, all hyperparameter values are fixed in the hyp.py file and the optimization proceeds for 5000 episodes by default. Our first priority should be to allow users some flexibility on these decisions (through argparse).
Since you're working on RL, you might also be interested in issues #4https://github.com/aksub99/MolDQN-pytorch/issues/4 and #7https://github.com/aksub99/MolDQN-pytorch/issues/7 since they require some RL background to implement.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/aksub99/MolDQN-pytorch/issues/5#issuecomment-616343515, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFT5MHYL7LE5XVK65DFWMQLRNPVCTANCNFSM4MA2WT4Q.

Yeah, I've based this on the Google repository. I've started out with only the single property optimization functionality (QED only as of now). The original implementation supports single property optimization for logP as well and also implements multi-objective optimization (#7) .
If we look at the actual architecture, the original Google repo also implements bootstrap heads (#4).
You're right, it would be a good idea to mention these as a TODO in the README. I'll get that done asap!
No issues dude, take your time!
We can connect on gitter if that's ok?

I specify the molecule to optimize in the start molecule section of hyperparameter file. Where can I obtain the SMILES of the molecule at the end of each optimization step and the final molecule. We have developed some software tools for automated drug discovery ( https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>1768v1 ). We plan to use this in one of our works. If interested, I welcome you to collaborate
start_molecule = "CCN1c2ccccc2Cc3c(O)ncnc13"
epsilon_start = 1.0
epsilon_end = 0.01
epsilon_decay = 2000
optimizer = "Adam"
polyak = 0.995
atom_types = ["C", "O", "N"]
max_steps_per_episode = 40
allow_removal = True
allow_no_modification = True
allow_bonds_between_rings = False
allowed_ring_sizes = [3, 4, 5, 6]
replay_buffer_size = 1000000
learning_rate = 1e-4
gamma = 0.95
fingerprint_radius = 3
fingerprint_length = 2048
discount_factor = 0.9

I specify the molecule to optimize in the start molecule section of hyperparameter file. Where can I obtain the SMILES of the molecule at the end of each optimization step and the final molecule. We have developed some software tools for automated drug discovery ( https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>1768v1 ). We plan to use this in one of our works. If interested, I welcome you to collaborate
start_molecule = "CCN1c2ccccc2Cc3c(O)ncnc13"
epsilon_start = 1.0
epsilon_end = 0.01
epsilon_decay = 2000
optimizer = "Adam"
polyak = 0.995
atom_types = ["C", "O", "N"]
max_steps_per_episode = 40
allow_removal = True
allow_no_modification = True
allow_bonds_between_rings = False
allowed_ring_sizes = [3, 4, 5, 6]
replay_buffer_size = 1000000
learning_rate = 1e-4
gamma = 0.95
fingerprint_radius = 3
fingerprint_length = 2048
discount_factor = 0.9

Hi @bengeof ! Thanks for your question. Your work on automated drug discovery looks very interesting!
The current version doesn't automatically display the molecule after each optimization step. But this can be easily configured by utilizing the visualize_state method defined in environment.py.
https://github.com/aksub99/MolDQN-pytorch/blob/d4422981c4e3aa25f468335535f67bfa707eb2ae/environment.py#L492
Also, to get a general sense of how to use this code, I would like to point you to examples/MolDQN-pytorch.ipynb which contains an example. Do let me know if you have any other queries or if you face any issues while working with this code.

Hi @bengeof ! Thanks for your question. Your work on automated drug discovery looks very interesting!
The current version doesn't automatically display the molecule after each optimization step. But this can be easily configured by utilizing the visualize_state method defined in environment.py.
https://github.com/aksub99/MolDQN-pytorch/blob/d4422981c4e3aa25f468335535f67bfa707eb2ae/environment.py#L492
Also, to get a general sense of how to use this code, I would like to point you to examples/MolDQN-pytorch.ipynb which contains an example. Do let me know if you have any other queries or if you face any issues while working with this code.

Hello,
I have some small molecules which interact with a rather high binding affinity toward my protein. I was wondering if I could use MolDQN to optimize these compounds and dock them against the protein target and see if the binding affinity has increased.
I can not figure out how I can feed the "smi" of these compounds to the main.py  and get the outputs.
could you please provide an example on how I can achieve this?
Thanks in advance.
Best,
Amir


Hyperlink is wrong when last character is period (.) to finish a sentence..

Describe the bug
Try:  https://catfact.ninja/fact.
It will parse it as https://catfact.ninja/fact.
Instead of https://catfact.ninja/fact with a period in the end.
Steps to reproduce
See above.
Screenshots or additional context
No response

I'm playing with link parsing in each memo for threading / preview. Got a better regex for plain link. But it looks like there will be an update on markdown overall. Will hold off any fix on this

Personally suggest adding a space after the link instead of .

Personally suggest adding a space after the link instead of .

I don't think it's a bug, https://catfact.ninja/fact. is a valid url.


Add warning about exporting top directory level

Found this out while trying to set up the icecube export: all.export /icecube/PUBLIC didn't work so I had to do all.export /icecube.

  Although I don't doubt this is the case, is this expected behavior?

Doesn’t look correct.  I think more debugging is needed.

@matyasselmeci did you ever figure out what the actual problem was?

It's on my todo list. Please don't close this; we should merge it and then revert it when the bug is fixed.


Suppressed Exceptions?

When I run the example script, the code runs fine, however any exceptions within the event handlers are suppressed for some reason.
This throws an exception as expected (and halts execution):
include __DIR__.'/vendor/autoload.php';
use Discord\Discord;
$discord = new Discord(['token' => '*TOKEN*']);

throw(new \Exception('test exception'));

$discord->on('ready', function ($discord) {
    echo "Bot is ready!", PHP_EOL;

    // Listen for messages.
    $discord->on('message', function ($message, $discord) {
        echo "{$message->author->username}: {$message->content}",PHP_EOL;
    });
});

$discord->run();

However this does not:
include __DIR__.'/vendor/autoload.php';
use Discord\Discord;
$discord = new Discord(['token' => '*TOKEN*']);

$discord->on('ready', function ($discord) {
    throw(new \Exception('test exception'));

    echo "Bot is ready!", PHP_EOL;

    // Listen for messages.
    $discord->on('message', function ($message, $discord) {
        echo "{$message->author->username}: {$message->content}",PHP_EOL;
    });
});

$discord->run();

The exception is completely ignored, as are any exceptions on code that run within the $discord event handler.  This makes it difficult to debug.
There is a previous issue that mentions this, but it was closed (without explanation): #260

Unfortunately due to the fact that the ready callback is called within a react PHP timer, the exception is supressed as any exceptions thrown in a timer will not be caught. I'm going to implement a try/catch statement around the ready statement to emit and log any exception.
If you want to handle the exception yourself, the best way would be to use your own try/catch or listen for the exception event on the Discord class.

@davidcole1340 Is this something reactphp does?
I found an old issue where they discussed it removing that suppression a few years ago, so I'm curious if this is a feature or a regression or something?

Yeah, comes from the react timers: "The timer callback function MUST NOT throw an Exception."
I'm not sure on the reasoning for this or if it can be changed but could be something to look into.

Thanks for the quick reply, I see the issue now.
If you are able to work something out I'd love to see an example!
Thanks!

Looking at the issue from react, seems like it was fixed only for the ExtEventLoop. I might have a play around and see what happens with other loops.

Could you see what loop you are using?
$loop = \React\EventLoop\Factory::create();
echo get_class($loop).PHP_EOL;
die;

React\EventLoop\StreamSelectLoop

Seems like it is an issue with react/promise, as when a promise is rejected and not handled it does not print the exception. I've tested on one of the pull requests related to the issue and it seems to work as expected, however it seems that the fixex will only be coming with<EMAIL_ADDRESS>


[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
No response
Other Software
No response
Repro Steps
➜ wsl.exe
There was an operation attempted on a degraded object.
Error code: Wsl/Service/CreateInstance/CreateVm/ConfigureNetworking/0x803b0017

after every few days, it runs into this, and the only way to fix it is to restart Windows, which is really tiresome at this point.
Having to restart Windows because WSL refuses to boot is unacceptable. It will be working 10 minutes ago but will refuse to open the next time you need it.
related issues seem to be: #10150
Expected Behavior
WSl should run.
Actual Behavior

Diagnostic Logs
WslLogs-2023-09-17_11-49-12.zip

@keith-horton,
What kinds of logs do we need if we/you/I want to dig into:
hresult: 	0x803B0017
message: 	Entity B95D0C5E-57D4-412B-B571-18A81A16E005 is in invalid State
module: 	hostnetsvc.dll

The trace shows the network HNS created is in a bad state.
Running this should at least unblock you:
hnsdiag delete networks B95D0C5E-57D4-412B-B571-18A81A16E005
Network : B95D0C5E-57D4-412B-B571-18A81A16E005
Name             : WSL
Type             : ICS
Subnet Address   : <IP_ADDRESS>/20
Gateway          : <IP_ADDRESS>
If that works, I can look to handle this error in WSL (HCN_E_DEGRADED_OPERATION)

@keith-horton I've tried it on 3 occacions now, all it seems to do is get stuck.

Sorry about this. Are you saying that running the above hnsdiag delete networks command would not unblock HNS - WSL would still fail to start with that error?

Yes to that.
Also, the command just stays stuck with no response. Let it run overnight then just had to kill it.
and then trying to open WSL in the morning again shows:

Can you look in your wslconfig file, or in the registry? It looks like this is trying to create a NAT but specifying a specific IP address for the container: <IP_ADDRESS>.
Possibly look under HKLM\SOFTWARE\Microsoft\Windows\CurrentVersion\lxss --> if there are registry values for "NatNetwork" and/or "NatGatewayIpAddress".
If so, can you remove those and retry?

The trace shows the network HNS created is in a bad state.
Running this should at least unblock you:
hnsdiag delete networks B95D0C5E-57D4-412B-B571-18A81A16E005
Network : B95D0C5E-57D4-412B-B571-18A81A16E005 Name : WSL Type : ICS Subnet Address : <IP_ADDRESS>/20 Gateway : <IP_ADDRESS>
If that works, I can look to handle this error in WSL (HCN_E_DEGRADED_OPERATION)

@keith-horton -- FWIW, I was encountering a similar issue out of the blue today, and this fixed it for me without a reboot. Other things I tried first (which didn't fix it) were:

disabling/re-enabling the WSL vEthernet network interface
stopping/restarting ICS
stopping/restarting HNS

Background: I returned after a few days away, and found all my WSL2 terminals closed. I was unable to launch WSL2 (although WSL1 worked OK). No Windows updates or reboots had taken place while I was gone, although it looks like WSL itself may have been updated from 2.0.9 to 2.1.4 last night.
From the event log:
Windows Installer installed the product. Product Name: Windows Subsystem for Linux. Product Version: <IP_ADDRESS>. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0.

My error trying to launch WSL2:
Failed to configure network (networkingMode Nat). To disable networking, set `wsl2.networkingMode=None` in C:\Users\myname\.wslconfig
Error code: Wsl/Service/CreateInstance/CreateVm/ConfigureNetworking/HNS/0x803b0017

My environment:
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.60
MSRDC version: 1.2.5105
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19045.4046

Can you look in your wslconfig file, or in the registry? It looks like this is trying to create a NAT but specifying a specific IP address for the container: <IP_ADDRESS>.
Possibly look under HKLM\SOFTWARE\Microsoft\Windows\CurrentVersion\lxss --> if there are registry values for "NatNetwork" and/or "NatGatewayIpAddress".
If so, can you remove those and retry?

I've tried deleting those records.
I still end up with:

Apologise for the late reply, I haven't been using WSL as I was tired of running into this issue every few days.

@utkarshsethi , were you able to find the HNS Network, and delete it?
e.g. 'list networks' to find the GUID for the "WSL" network. then specify that GUID to delete it.
Can you confirm if after deleting the HNS Network, if WSL can start?
If not, can you please take a WSL Networking trace?
https://github.com/microsoft/WSL/blob/master/diagnostics/collect-networking-logs.ps1
C:\Windows\System32>hnsdiag list networks
Network : C08CB7B8-9B3C-408E-8E30-5E16A3AEB444
Name             : Default Switch
Type             : ICS
Subnet Address   : <IP_ADDRESS>/20
Gateway          : <IP_ADDRESS>
Network : B95D0C5E-57D4-412B-B571-18A81A16E005
Name             : WSL
Type             : ICS
Subnet Address   : <IP_ADDRESS>/20
Gateway          : <IP_ADDRESS>
C:\Windows\System32>hnsdiag delete networks B95D0C5E-57D4-412B-B571-18A81A16E005
C:\Windows\System32>hnsdiag list networks
Network : C08CB7B8-9B3C-408E-8E30-5E16A3AEB444
Name             : Default Switch
Type             : ICS
Subnet Address   : <IP_ADDRESS>/20
Gateway          : <IP_ADDRESS>
C:\Windows\System32>

Did not work the first time, but did eventually work when I retried after restarting my VPN post deleting the network.
I, unfortunately, forgot to start logging while I was facing the issue.

This is still an issue in the exact same manner. hnsdiag to delete the WSL network just hangs the terminal, and does not cause a fix either. deleting reg entries doesn't produce any effect either. I get the exact same error at completely random times.
The only way it seems to work again is to do multiple restarts and hope it works now.


More tests, comments

Changes Unknown when pulling aa8b72605056fb44e3794aa8f30257eba418630d on RyanCopley:master into ** on BlockSync:master**.


Restore window title signal functionality

Description
I'm... pretty sure this has been wrong since this entire time? Window title changes never worked (in either yabai 6 or 7), but they do work now with this change. Apologies if I'm missing something obvious 😅
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Enable the process widget, without the two following options and check the registered yabai signals

❯ yabai -m signal --list | grep title | grep -v '""'
	"label":"Refresh simple-bar when current window title changes",
	"event":"window_title_changed",

This signal does not get registered at all without this PR, because the logic is flipped. (The above shell command returns nothing)
However, with any of the process widget options enabled, the signal will not get registered (make sure yabai is restarted, to avoid stray signals). Without this PR, the signal will get registered but will not be of any use.
Test Configuration:

OS version: Sonoma 14.3, and 14.4
Yabai version: v6.0.1, and v7.0.2
Übersicht version: 1.6 (82)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] My changes generate no new warnings

Oh, I must have broken it sole time ago without noticing! Thanks for the catch!
I'll merge it soon :)

I had indeed made the opposite of what needed to be done, thanks for your PR!