Add video to homepage

The video still needs to be inserted under the text on the homepage when it is done. #56

Use this code to insert:

Note to self: once promo video is on youtube, link to the youtube page (file is too big to embed).
For Instructions video: ?


The background color of stack view is brown in 1.1.2 version.

Hi, @KiranJasvanee
I found  the background color of the stack view is brown in 1.1.2 version, but its color is cleared in the master.

Screenshots

I've solved it. The background color can be modified using runtime.
guard let ivarName = ("stackView" as NSString).utf8String else {
    return
}
guard let ivar = class_getInstanceVariable(OnlyPictures.self, ivarName) else {
    return
}
if let stackView = object_getIvar(onlyPictures, ivar) as? UIStackView {
    stackView.backgroundColor = .clear
}


Doc: Improve configuration structure for backends

Docs for GH-1784

Docs reviewed and merged to develop. Closing this ticket now.

Re-opening so that updates can be made to https://github.com/SAP/cloud-commerce-spartacus-storefront/wiki/Configuring-the-Base-URL

No further updates required. Re-closing 😄

Final review of https://github.com/SAP/cloud-commerce-spartacus-storefront/wiki/Configuring-the-Base-URL now complete.
Closing ticket and moving to "Done".


Issue #29 - Add tox support

Add a tox support. Add testenvs for running our test suite against
python 3.9, building our docs, and executing our "pre-commit linting suite".
Add a "dev" testenv for creating a development environment via tox if
users are so inclined.
Rework the way we're handling checks in pre-commit.
We call our pre-commit "linters" from tox so we need to:

Move as many checks as possible over to pull specific revisions of the
repositories. The caching in pre-commit is much nicer and saves us the
time of tox building a new environment, etc. Note, these specific revs
should be updated regularly via pre-commit autoupdate.

Add filtering to keep some of checks from running across the
repository.

Re-baseline our repository with a run of pre-commit run --all-files. This is the same thing tox is going to use to call our
steps.

Resolve #29

Note, the README is getting a bit all over the place with all the changes to tooling and the like. I'll see what I can do to clean that up in a follow on issue.

@MJJoyce @nutjob4life per the updates, do we think it makes sense to move some of the library recommendations, best practices, etc. to a wiki / Github page at https://nasa-pds.github.io/ and just link to it? not sure what the best practice is for that either :-)

@jordanpadams, I'd say if we're expecting people to use this template repo as the go-to for anything Python it's probably best to keep it there. Moving it to a wiki also runs the risk of stuff getting out of sync with what's in here, etc.


[WIP] Implement SSA for the Numba IR

So far, I've implemented the dominance frontier using the algorithm described in the paper "A Simple, Fast Dominance Algorithm" - K. D. Cooper, T. J. Harvey, and K. Kennedy.
I still need to write some tests but I would appreciate some feedback.
issue #3904

Fixed flake8 errors and added code for testing the immediate dominator algorithm.

I added some tests basing on real python bytecode at https://github.com/sklam/numba/tree/pr/4149.  The algorithm seems to be working as expected.

@guilhermeleobas, I think this PR is feature complete for IDOM and DOMFRONT.  Do you have any addition to these algorithms?  I think other SSA related features should go in a separate PR.

No, that’s all for dominance frontier and immediate dominators.

Ok, I'll remove the WIP tags on it.

Should I wait for this PR to be merged or can I already create another PR for the rest of the SSA?

You can create another PR.  If there's any change on this PR, you can always merge it to the new PR.

@stuartarchibald, yes, we'll need the license and references to source SHA's from NetworkX.  @guilhermeleobas, we can merge the PR once you added those.

Done! Let me know if you guys need any other modification.
Regards

with reference added, this looks ready to merge.


removeAll/retainAll return Boolean; transfer less strict

A couple of changes I hadn't pushed yet when #232 was merged.
removeAll and retainAll should return Boolean like the stlib functions do.
And I thought transfer could be less strict about the target array type.

Thanks, these will be available in the next release.


feat(starr-anime): Add AsukaRaws to Anime Raws

Pull Request
Purpose

Asuka Raws is a raws-only anime group
Closes https://github.com/TRaSH-Guides/Guides/issues/2066
Approach

Added to the sonarr and radarr anime raws CF
Open Questions and Pre-Merge TODOs

Requirements

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

ordering in the Sonarr CF is wrong, other than that lgtm

fixed


Enhance "Go to Definition" for Improved Navigation in Split Editors

Description:
I often work with Visual Studio Code on an ultrawide monitor, utilizing the split editor feature to enhance my project visualization. While using the "Go to Definition" feature, I've encountered a scenario where the behavior could be improved for users with split editors.
Current Behavior:
When clicking "Go to Definition," the file opens in a new tab, even if it's already open in another editor.
Suggested Enhancement:
It would be beneficial if Visual Studio Code intelligently detected whether the file containing the definition is already open in another editor. Instead of opening a new tab, the "Go to Definition" feature could prioritize navigating to the definition in the another existing editor where the file is already open.
Additional Scenarios:
For users who split large files into two editors to simultaneously view different sections, "Go to Definition" should prefer navigating to the definition in the other editor rather than the same editor.
Benefits:

Improved navigation efficiency for users working with split editors.
Minimized clutter of open tabs, providing a cleaner workspace.
Enhanced usability for users who frequently split their workspace.

I believe this enhancement would significantly improve the user experience for those working with split editors on larger screens. Thank you for considering this feature request.

You can use the workbench.editor.revealIfOpen setting for this.

Thank you.


free(): invalid pointer error on invalid query (Nim bindings)

I was trying out the Nim bindings and tested the query SELECT * FROM nimtest where nimtest was the name of the database. This didn't work, but instead of raising an error it SIGABRT on free(): invalid pointer. This is the backtrace from GDB:
Program received signal SIGABRT, Aborted.
0x00007ffff7cae355 in raise () from /usr/lib/libc.so.6
(gdb) bt
#0  0x00007ffff7cae355 in raise () from /usr/lib/libc.so.6
#1  0x00007ffff7c97853 in abort () from /usr/lib/libc.so.6
#2  0x00007ffff7cf1878 in __libc_message () from /usr/lib/libc.so.6
#3  0x00007ffff7cf8d3a in malloc_printerr () from /usr/lib/libc.so.6
#4  0x00007ffff7cf9ef4 in _int_free () from /usr/lib/libc.so.6
#5  0x00007ffff78aa19e in std::default_delete<fleece::impl::Dict::key>::operator() (this=0x7fffffffcde8, __ptr=0x7fffffffd039) at /usr/include/c++/10.1.0/bits/unique_ptr.h:78
#6  std::default_delete<fleece::impl::Dict::key>::operator() (__ptr=0x7fffffffd039, this=0x7fffffffcde8) at /usr/include/c++/10.1.0/bits/unique_ptr.h:78
#7  std::unique_ptr<fleece::impl::Dict::key, std::default_delete<fleece::impl::Dict::key> >::~unique_ptr (this=0x7fffffffcde8, __in_chrg=<optimized out>) at /usr/include/c++/10.1.0/bits/unique_ptr.h:360
#8  fleece::impl::Path::Element::~Element (this=0x7fffffffcdd8, __in_chrg=<optimized out>) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Core/Path.hh:98
#9  fleece::smallVector<fleece::impl::Path::Element, 4ul>::erase (last=0x7fffffffcdf8, first=0x7fffffffcdd8, this=0x7fffffffcdd0) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Support/SmallVector.hh:112
#10 fleece::impl::Path::drop (this=this@entry=0x7fffffffcdd0, startAt=startAt@entry=1) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Core/Path.cc:61
#11 0x00007ffff79dbddc in litecore::QueryParser::writePropertyGetter (this=0x7fffffffd5c0, fn=..., property=..., param=0x0) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:1298
#12 0x00007ffff79dc73f in litecore::QueryParser::propertyOp (this=0x7fffffffd5c0, op=..., operands=...) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:963
#13 0x00007ffff79da164 in litecore::QueryParser::handleOperation (this=this@entry=0x7fffffffd5c0, op=<optimized out>, op@entry=0x7ffff7bb7080 <litecore::QueryParser::kOperationList>, actualOperator=..., operands=...) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:634
#14 0x00007ffff79da312 in litecore::QueryParser::parseOpNode (this=0x7fffffffd5c0, node=<optimized out>) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:619
#15 0x00007ffff79de987 in litecore::QueryParser::resultOp (this=0x7fffffffd5c0, op=..., operands=...) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:746
#16 0x00007ffff79da164 in litecore::QueryParser::handleOperation (this=this@entry=0x7fffffffd5c0, op=<optimized out>, op@entry=0x7ffff7bb6fc0 <litecore::QueryParser::kResultListOperation>, actualOperator=..., operands=...) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:634
#17 0x00007ffff79dac43 in litecore::QueryParser::writeSelectListClause (this=this@entry=0x7fffffffd5c0, operands=operands@entry=0x5555555e328e, key=..., sql=0x7fffffffd1e0 "\214\062^UUU", sql@entry=0x7ffff7b0b55e "", aggregatesOK=aggregatesOK@entry=true) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:335
#18 0x00007ffff79e091f in litecore::QueryParser::writeSelect (this=0x7fffffffd5c0, where=0x0, operands=0x5555555e328e) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:259
#19 0x00007ffff79e1658 in litecore::QueryParser::parse (this=0x7fffffffd5c0, expression=0x5555555e328e) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/QueryParser.cc:194
#20 0x00007ffff79e172d in litecore::QueryParser::parseJSON (this=0x7fffffffd5c0, expressionJSON=...) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Core/Doc.hh:106
#21 0x00007ffff7a09285 in litecore::SQLiteQuery::SQLiteQuery (this=0x5555556527f0, keyStore=..., queryStr=..., language=<optimized out>) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/API/fleece/slice.hh:458
#22 0x00007ffff7a0762e in litecore::SQLiteKeyStore::compileQuery (this=0x5555555cc390, selectorExpression=..., language=litecore::QueryLanguage::kN1QL) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Query/SQLiteQuery.cc:515
#23 0x00007ffff78ce5fa in c4Query::c4Query (queryExpression=..., language=1, db=<optimized out>, this=0x5555555e51f0) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/API/fleece/slice.hh:273
#24 operator() (__closure=<optimized out>) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/C/c4Query.cc:53
#25 fleece::function_ref<c4Query*()>::callback_fn<c4query_new2(C4Database*, C4QueryLanguage, C4Slice, int*, C4Error*)::<lambda()> >(intptr_t) (callable=140737488345616) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Support/function_ref.hh:59
#26 0x00007ffff78cfdd9 in fleece::function_ref<c4Query* ()>::operator()() const (this=<synthetic pointer>) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Support/function_ref.hh:71
#27 c4Internal::tryCatch<c4Query*>(C4Error*, fleece::function_ref<c4Query* ()>) (outError=0x7fffffffdaf4, fn=...) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Support/c4ExceptionUtils.hh:60
#28 0x00007ffff78cf404 in c4query_new2 (database=<optimized out>, language=language@entry=1, expression=..., outErrorPos=<optimized out>, outErrorPos@entry=0x7fffffffdaf0, outError=outError@entry=0x7fffffffdaf4) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/Fleece/Support/function_ref.hh:65
#29 0x00007ffff7887ef8 in CBLQuery::createC4Query(CBLDatabase const*, unsigned int, fleece::slice, int*, C4Error*)::{lambda(c4Database*)#1}::operator()(c4Database*) const (c4db=<optimized out>, __closure=<synthetic pointer>) at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/vendor/fleece/API/fleece/slice.hh:274
#30 litecore::access_lock<c4Database*, std::recursive_mutex>::use<c4Query*, CBLQuery::createC4Query(CBLDatabase const*, unsigned int, fleece::slice, int*, C4Error*)::{lambda(c4Database*)#1}>(CBLQuery::createC4Query(CBLDatabase const*, unsigned int, fleece::slice, int*, C4Error*)::{lambda(c4Database*)#1}) const (callback=..., this=0x5555555e4ae0)
    at /home/peter/Projects/couchbase-lite-C/vendor/couchbase-lite-core/LiteCore/Support/access_lock.hh:60
#31 CBLQuery::createC4Query (outError=0x7fffffffdaf4, outErrPos=0x7fffffffdaf0, queryString=..., language=1, db=0x5555555e4ad0) at /home/peter/Projects/couchbase-lite-C/src/CBLQuery_Internal.hh:59
#32 CBLQuery::CBLQuery (outError=0x7fffffffdaf4, outErrPos=0x7fffffffdaf0, queryString=..., language=1, db=0x5555555e4ad0, this=0x5555556501d0) at /home/peter/Projects/couchbase-lite-C/src/CBLQuery_Internal.hh:36
#33 CBLQuery_New_s (db=0x5555555e4ad0, language=1, queryString=..., outErrorPos=0x7fffffffdaf0, outError=0x7fffffffdaf4) at /home/peter/Projects/couchbase-lite-C/src/CBLQuery.cc:57
#34 0x0000555555571a4b in newQuery__OhCPcHLdYW3ucemgk5MuLg (db=0x7ffff7bde050, str=0x555555575ec0 <TM__zZS2NtWhkO9cJEWatgDgDNQ_13>, language=<optimized out>) at /home/peter/Projects/nim/nim/lib/system/sysstr.nim:96
#35 0x0000555555573070 in NimMainModule () at /home/peter/Projects/couchbase-lite-C/bindings/nim/src/example.nim:29
#36 0x0000555555573d9f in NimMainInner () at /home/peter/Projects/nim/nim/lib/system.nim:2149
#37 0x0000555555572ba6 in NimMain () at /home/peter/Projects/nim/nim/lib/system.nim:2157
#38 0x0000555555572bdd in main (argc=<optimized out>, args=<optimized out>, env=<optimized out>) at /home/peter/Projects/nim/nim/lib/system.nim:2164

This is a bug in Couchbase Lite Core (LiteCore) ... I added that query string to one of the LiteCore query tests and it hits an array-bounds assertion failure in stack frame 9. Looks like a bug in QueryParser.
FYI, you don't need the FROM clause in a query. The query scope is implicit since there are no tables.

Filed in our Jira as https://issues.couchbase.com/browse/CBL-1191

It looks like this fix has been merged, so closing this.


Using symbol_table in a single-threaded context

Is there a way to not pay the cost of locking/sharding when I'm guaranteed to be using in a single-threaded context only? If not, would it reasonable to expose a const-param mechanism to the SymbolTable structure that disables the locking and sharding?

I was operating on the assumption that locking is very low cost if not contended, probably a single atomic operation.

I was noticing substantial overhead compared to the string_interner library when I reimplemented GlobalSymbol using it without any of the sharding/locking. I also assume that locking might disable certain code-motion/inlining optimizations that can otherwise be performed
Anyways, okay with closing if symbol_table is not really meant to support the single-threaded use case but might be worth micro-benchmarking a bit if single-thread support matters too

I'm not super concerned with maximum single-threaded performance at the moment, so I'll close this for now.


Code for LTARI

Hi, it's a good work for 3D reconstruction. I noticed the usage of LTARI in your manuscript, but I fail to find it in your tutorial page or the source code. I wonder if you can provide a description of the usage so that I can evaluate the performance. Thanks.

Hi,
Thank you for your interest in paste2! Here is the implementation of LTARI I used in the paper:
import sklearn

def compute_alignment_ari(sliceA, sliceB, pi):
    mapped_clusters = []
    for j in range(pi.shape[1]):
        mapping = pi[:, j]
        if np.sum(mapping) > 0:
            i = np.argmax(mapping)
            mapped_clusters.append(sliceA.obs['annotation'][i])
        else:
            mapped_clusters.append("NULL")

    assert len(sliceB.obs['annotation']) == len(mapped_clusters)
    true_clusters_mapped_region = []
    mapped_clusters_mapped_region = []
    for i in range(len(sliceB.obs['annotation'])):
        if mapped_clusters[i] != "NULL":
            true_clusters_mapped_region.append(sliceB.obs['annotation'][i])
            mapped_clusters_mapped_region.append(mapped_clusters[i])

    ari = sklearn.metrics.adjusted_rand_score(true_clusters_mapped_region, mapped_clusters_mapped_region)
    return ari

sliceA, sliceB are AnnData objects and pi is the n*m alignment matrix. This code assumes the .obs['annotation'] field stores the cell type annotation and should be changed to the name you use to store the annotation in your dataset.
Let me know if there is any issue with it! Thank you.


fixing check for isProcessingFirstFile which was case sensitive.

also added a bunch of logging.

Maybe remove most of the SelfLog calls, I think they are reserved for errors

a consumer of the lib is only going to enable SelfLog output if there's a problem - in which case they'll be kind of useful won't they?


Redesign the transaction priority selection

Redesign the transaction priority selection
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=5590-6434&t=mSdoAB1vB1aqs8pp-4

Verified this issue on keeper app v 1.2.20(419)


embeddings models keep_alive

I use embeddings models a lot and every time it loads the model do the vectoring and then unload it immediately. when I try to keep alive by using this command
$ curl http://localhost:11434/api/generate -d '{"model": "mxbai-embed-large:latest", "keep_alive": -1}'
it tells me that this model isn't a generative model and refuse to keep alive. please have support for it to decrease the latency as it copies the 600 megabytes every time and then delete it which adds a couple of seconds to an operation that should take only a second.

You need to load an embedding model via the embedding API endpoint:
$ curl http://localhost:11434/api/embed -d '{"model": "mxbai-embed-large:latest", "keep_alive": -1}'
$ ollama ps
NAME                            ID              SIZE    PROCESSOR       UNTIL
mxbai-embed-large:latest        468836162de7    1.2 GB  100% GPU        Forever

thank you. I tried and it works.


Render Markdown (type: markdown)

Would be nice if #123 could produce [#123](https://github.com/owner/repo/issues/123).

Yeah, I guess we could add markdown as a value in the type option. However, we need to note that the attributes option will be a noop then.


Allow column specification for MySQL index

Now it is allowed to create index(:table, ["token(16)"]) like for
postgres adapter.
Additionally fixed two test cases where column specification should not be
quoted.
btw, MySQL does not have functional indexes

I'm not sure why tests are failing for ELIXIR_ERL_OPTIONS=""

Hi @immon! I think we should continue escaping columns names. What if we support fragments, like we do in the :default option, to provide functional indexes? So it would look like this: create index(:table, [fragment("token(16)")]). The patch would be very similar, except it would match on {:fragment, _} key in the adapters and return the second argument.

I don't feel strong enough to write something more complicated. I just copy&pasted the code we have in postgres adapter (and fixed failing tests). Sorry.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Flags provided but not defined: docker-compose

There are other duplicate issues but no one worked for me.
After spending my  6 to 8 hours without eating or drinking anything, I finally found there is a bug in echoip, whenever I run sudo docker-compose up:

My compose file:

I have tried every possible combination:
"-C 0 -p -r -H X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb"
"-C 0 -p -r -H=X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb"
If I try:
"-H=X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb -p -r"
It considers all other flags and theier values as a value of -H:

If I try:
"-a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb -H=X-Forwarded-For -p -r"

If I try:
"-a=/data/GeoLite2-ASN.mmdb -c=/data/GeoLite2-City.mmdb -f=/data/GeoLite2-Country.mmdb -H=X-Forwarded-For -p -r"
It considers all other flags and their values as value of -a

did you download GeoIP Files?

did you download GeoIP Files?

did you download GeoIP Files?

Even if I try this:
"-p -r -H=X-Forwarded-For"

dont know if it related, why do you use the full path in the volume?
try
    volumes:
      - ./geoipupdate:/data:ro

dont know if it related, why do you use the full path in the volume? try
    volumes:
      - ./geoipupdate:/data:ro

in my last reply bro you can see volumes or geolite2 is irrelevant. It is pure issue of flags.

I have no idea where the problem is. In my setup it works fine.
-C 0 -p -r -H X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb -t /html

I have no idea where the problem is. In my setup it works fine.
-C 0 -p -r -H X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb -t /html

Strange, I just copied your docker-compose in a new a.yaml file and changed volume mapping to mine one, and now everything is working fine.
sudo docker-compose -f  a.yaml up. Thanks for your quick response and help. But I will pposr both files here may be some UTF text issue.

Not Working:
services:
  echoip:
    image: mpolden/echoip
    volumes:
      - "/home/afzl/@Apps/geoip/geoipupdate:/data"
    command:
      - "-C 0 -p -r -H X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb"
    ports:
      - "2087:8080"```

Working Fine:
```version: '3.7'

services:
  echoip:
    image: mpolden/echoip
    command: "-C 0 -p -r -H X-Forwarded-For -a /data/GeoLite2-ASN.mmdb -c /data/GeoLite2-City.mmdb -f /data/GeoLite2-Country.mmdb"
    ports:
      - 2087:8080
    volumes:
      - /home/afzl/@Apps/geoip/geoipupdate:/data```


Add dropdown by state to filter flags

Description

In the home page we currently have a first row to add new flags and a second row to search for flags. We have modified the second row in order to show a dropdown list in which we can see the values [All the flags, Enabled flags, Disabled flags] being the value All the flags the one selected by default.
As the current API supports the boolean parameter enabled to filter flags by enabled and disabled, we only needed to change the GUI to add the filter.
Motivation and Context

The number of flags is growing and as this is a service shared by all of our customers, it is expected it will continue growing in the future. To do it easier to work with Flagr we want to provide a way of filtering flags by status (enabled / disabled).
How Has This Been Tested?

I added four flags: Flag Y1, Flag Y2, Flag Z1 and Flag Z2.
I enabled flags Flag Y1 and Flag Z1.
I selected Enabled flags and only Flag Y1 and Flag Z1 were shown (as expected).
I selected Disabled flags and only Flag Y2 and Flag Z2 were shown (as expected).
I selected All the flags and the four flags were shown (as expected).
I filtered by the term "Z" and only Flag Z1 and Flag Z2 were shown (as expected).
I selected Enabled flags and only Flag Z1 was shown (as expected).
I selected Disabled flags and only Flag Z2 was shown (as expected).
I removed the term "Z"only Flag Z1 and Flag Z2 were shown (as expected).

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks for the PR. What do you think if we just add a filter for the "Enabled" column in the flags table?
E.g.
https://element.eleme.cn/#/en-US/component/table#filter
For example from element's documentation:

I think we may just need to add the filter and something like "All", "Enabled", and "Disabled" for the filters.
      :filters=...
      :filter-method=...
      filter-placement=...

Thanks for the PR. What do you think if we just add a filter for the "Enabled" column in the flags table?

For sure! I'll take a look and I'll be back with the suggested changes.

I have done the commit 44b6b7e to filter flags by state, using a filter for the "Enabled" column in the flags table as suggested above.

Codecov Report

Merging #433 (44b6b7e) into master (cff1f32) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #433   +/-   ##
=======================================
  Coverage   81.75%   81.75%
=======================================
  Files          27       27
  Lines        1611     1611
=======================================
  Hits         1317     1317
  Misses        214      214
  Partials       80       80

Continue to review full report at Codecov.

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

lgtm


ARROW-9648: Added compression level parameter to LZ4_FRAME compression codec.

This will only work for the Arrow IPC format, not for Parquet, because Parquet uses the raw LZ4 format instead of the framed format.
Still to figure out:

[x]  Do we need to add tests to test if setting the compression level works as intended? (Or is this already covered by tests for the other formats that support levels)

[ ] Figure out the raw format for Parquet files
The raw format does not support the compression level, but supports an acceleration value. This value works in opposite direction; increasing it speeds up the compression in expense of the ratio. The value can be set from 1 to 65537, so that's also completely different from the compression level in the framing format (1 to 12).

@github-actions crossbow submit -g cpp

Hmm, instead of requiring a source build of LZ4 on older distros, what do you think about doing a compile-time switch? LZ4 exposes LZ4_VERSION_MAJOR, LZ4_VERSION_MINOR and LZ4_VERSION_RELEASE C macro definitions.

@pitrou without building from source it would become difficult to reliably find the max compression level value; the API call for it was introduced in 1.8.0 and before that the name of the LZ4HC_CLEVEL_MAX macro was changed without changing the version number.

@joosthooz looks like you've still got a lint failure and there's one more constant to sometimes hardcode for older liblz4 https://github.com/apache/arrow/runs/4436211338?check_suite_focus=true#step:8:1341

@github-actions autotune everything

@github-actions crossbow submit -g cpp

@github-actions crossbow submit -g cpp

@github-actions autotune everything


doc: Add the sandbox apis of kata-api-design.md

The doc package of kata design, just list part apis of the the sandbox
management apis. This commit add the other sandbox apis to the api
tables, the file kata-api-design.md
Fixes #700
Signed-off-by: Ychau Wang<EMAIL_ADDRESS>

mv this PR from 2.0-dev branch to the kata master branch


Find a suitable display

The calculator needs a display. It should be big enough, cheap and easy to use

[ ] do task one
[ ] do task two


[UPDATE CORE GUIDE] Minor updates in BTC RPC Explorer guide

What
Adapt the guide to different cases of uses and general improvements
Why
This can lead to significant improvements to the Raspibolt nodes.
How

Minor fix in FW configuration
Added a separate optional section in the configuration chapter
Added "Demo mode" configuration
Added Hidden Service comment to standardize with the rest of the guide

Scope

[X] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

Test & maintenance
I tested all and is working fine for me

@twofaktor I could't figure out how the DEMO mode worked. The links you gave me mention a list of RPC command but I don't see any mention of the DEMO option related to them.. so I prefer to not comment and let someone else review this part of your PR.
However the change to add the comment line in the torrc line is needed.. would you be interested to open a new PR for that? or reopen that PR to focus on that change only for now?


Refactor Ignore Logic and Ignore Images for Bare URLs

@platers , how does the refactor for the ignore logic look? Do we want to keep the intermediary methods like ignoreTables and ignoreCodeBlocksYAMLTagsAndLinks instead of ignoreTypes, but use ingoreTypes there instead?
Fixes #276
Fixes #275
Refactored logic for ignoring different types of elements. It should allow for extensibility down the road as needed. Also fixed an issue where images were being caught in no bare URLs.
Changes Made:

Removed ignore methods in utils.ts in favor of ignore-types.ts functions
There is now an object that holds the ignore types that can be used that have their actions and placeholders defined which allows for easier maintenance of all the different types to ignore
Standardizes how ignoring types works and allows for the ability to specify as many as you would like at once
Updated method referenced as a utility method name since the other one was removed
Added the ability to ignore images for no bare URLs and added associated UT

LGTM, no need for the old methods anymore


node_modules.bin\electron-rebuild ends without output

node_modules.bin\electron-rebuild exits fast and without output. I previously installed fibers.
.bin\electron-rebuild

c:\git\iron-node>node -v
v0.12.6

Using

Microsoft Windows 7 Home Premium
6.1.7601 Service Pack 1 Build 7601
X86-based PC

"electron-prebuilt": "^0.30.0",

I also have this problem.

Not a bug, this is how UNIX works :)  The exit code should be non-zero on failure (and a message will be printed)

I browse the source code, I find the following code:
export async function shouldRebuildNativeModules(pathToElectronExecutable, explicitNodeVersion=null) {
  // Try to load our canary module - if it fails, we know that it's built
  // against a different node module than ours, so we're good
  //
  // NB: Apparently on OS X, this not only fails to be required, it segfaults
  // the process, because lol.
  try {
    let args = ['-e', 'require("nslog")']
    await spawn({cmd: process.execPath, args});

    require('nslog');
  } catch (e) {
    return false;
  }

When judge if the module should to be rebuild, the function will require the nslog module first. But my
repo don't dependent this package, so spawn will fail and shouldRebuildNativeModules will return false, then nothing happens.

nslog is a module that electron-compile has as a dependency so that's fine, though you definitely found a bug there, it should be return true. In the meantime, you can work around this by passing -f to always build modules

:+1:   -f works as expected.

I do not think this is a problem of your module, but maybe you have an idea?
I installed fibers post compiled with node_modules\.bin\electron-rebuild -f. My electron app requests a version win32-ia32-v8-4.3/fibers.node but there is not output for this version? Do you know what I am doing wrong ?

The install output is not piped into the cmd. But if you encounter a error when rebuild, the error information will be showed.
So if you don't have output, it's OK.

@liuxiong332  so fibers does not provide a matching version for electron?

I do not think this is a problem of your module, but maybe you have an idea?

This is a common bug with libraries that use node-pre-gyp, they don't correctly fall back to building the module, so rebuild fails - if they just used node-gyp and stopped trying to be clever it would Just Work :-/

Do you know how I can force to build a specific version instead the whole bunch? Maybe I can get more details then.


Remaining Bugs/Issues for the V2 Development API

I had wrote this as a comment but for better visibility I'm going to create an issue for it so it doesn't get lost in the other issue.
Issues

#75 - I moved this to the discussions since I feel like it makes more sense there
#92 - I think the idea here was to have a AQI value (probably using the US EPA scale) in the API
#114
#32
#119
#11

Nice to have Issues but not critical for V2

#154 - I added this one because you said it would be quick to add. Can always remove it from the list if needed
#152 - Not part of the project but would be a nice addition to V2

Tasks

[ ] negative Longitudes displaying incorrectly (-75 shows as 285)
[ ] visibility is not capped
[x] hourly and daily precipitation data points are rounded to two decimal places instead of four
[ ] minutely block seems to show array positioning
[x] Precipitation Probability can go over 1
[x] UV index can display -0
[x] precipIntensityand precipAccumulation can go below zero. (Should check the new ones added as well)
[x] nearestStormDistance and/or nearestStormBearing can display -0
[x] It's possible to not have a precipitation icon/summary when precipIntensity is over 0.25mm/h. Might be related to the precipProbability change
[x] The threshold to show the precipitation icon is inconsistent between currently and hourly/daily blocks. Currently shows it no matter the intensity whereas hourly/daily it needs to be above 0.25mm/h. Should be consistent across all blocks
[ ] Daily precipitationAccumulation in metric units is wrong unless only snow is forecast.

An example of the precipitation intensity bug I added to the list of tasks:
  "currently": {
    "time":<PHONE_NUMBER>,
    "summary": "Cloudy",
    "icon": "cloudy",
    "nearestStormDistance": 35.47,
    "nearestStormBearing": 63,
    "precipIntensity": 0.5602,
    "precipProbability": 0.16,
    "precipIntensityError": 0.14,
    "precipType": "snow",
    "temperature": -0.15,
    "apparentTemperature": -4.02,
    "dewPoint": -0.71,
    "humidity": 0.79,
    "pressure": 989.92,
    "windSpeed": 12.88,
    "windGust": 26.87,
    "windBearing": 184,
    "cloudCover": 0.92,
    "uvIndex": 0,
    "visibility": 6.51,
    "ozone": 413.48
  },

Examples of negative precipAccumulation and precipInsensity:

Hourly
      {
        "time":<PHONE_NUMBER>,
        "icon": "wind",
        "summary": "Windy",
        "precipIntensity": -0.21,
        "precipProbability": 0.06,
        "precipIntensityError": 0.04,
        "precipAccumulation": -0.2,
        "precipType": "snow",
      },

Daily
      {
        "time":<PHONE_NUMBER>,
        "icon": "partly-cloudy-day",
        "summary": "Partly Cloudy",
        "sunriseTime":<PHONE_NUMBER>,
        "sunsetTime":<PHONE_NUMBER>,
        "moonPhase": 0.42,
        "precipIntensity": -0.04,
        "precipIntensityMax": 0.25,
        "precipIntensityMaxTime":<PHONE_NUMBER>,
        "precipProbability": 0.18,
        "precipAccumulation": -0.8999999999999999,
        "precipType": "snow",
      },

On Wednesday the API is predicting rain and snow and using the V2 flag it shows this for the liquid/snow accumulation:
"liquidAccumulation": 1.8,
"snowAccumulation": 11.4,
"iceAccumulation": 0

I would expect the precipAccumulation to be 11.58 but instead it shows "precipAccumulation": 2.94,. It seems like it's converting the snowAccumulation to cm instead of the liquidAccumulation like I would expect. It's only converting the snowAccumulation when there is mixed precipitation otherwise it just shows the value of snowAccumulation. Rain is also not being converted into cm as it is also just using the value of liquidAccumulation.

This issue is meant to be locked so it doesn't get lost in the other issue. Any issues should be reported in #155

Sorry, didn't mean to re-open this!

This is separate from the original list I created. I figured making it it's own issue would help so it doesn't get lost in the other bugs issue since it'll be pretty active with other people reporting bugs. I re-opened but I can close it if you'd like.


i18n(ko) typescript.mdx link heading fix

Description (required)
Corrects a link pointing to an English heading instead of a translated one.
If I'm right, all the link checks should pass.
Related issues & labels (optional)

I wasn't aware of the existence of this PR, so I added the same actions as this PR to #5110 .
Could this cause any problems?

@jsparkdev It shouldn't be a problem! Checks should still pass over there, too. If not, I'll tag you in there to fix! 🙌 Thanks for being so responsive!

I'm so glad! Thank you :+1:


Help handling Push received when app is killed

First of all, great work on the Chat SDK. I am very impressed!
Push notifications work as expected if the app is in the foreground or background. When the app is killed, I notice that the DefaultBroadcastReceiver will send an Intent for ChatSDK.ui().getLoginActivity().
I would like to replace that LoginActivity with my own. What would you suggest as the best way to do this? Manifest Merge? Overriding getLoginActivity()? Some config? I could also implement my own BroadcastReceiver.
Thanks for your attention!

Hi @chriswoolfe the easiest way to replace the login activity would be to follow this guide. You can override the interface adapter and return a custom class by overriding this method:
    @Override
    public Class getProfileActivity() {
        return ProfileActivity.class;
    }

When app is swiped out or killed then come back again didn't receive any push notification.


SW-5795 Unique cohort and participant names in tests

Cohort and participant IDs and names are required to be globally unique, which
causes problems if two tests run concurrently and both try to insert using the
existing "start with a suffix of 1" naming strategy.
For IDs, switch to dynamically-allocated ones and remove the ability to insert
hardwired IDs.
For names, use random UUID suffixes instead of simple numbers; these should be
sufficiently unique such that the chances of a collision in a single test run
are effectively zero.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2267  👈
#2266
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sgrimm and the rest of your teammates on  Graphite

Merge activity

Aug 12, 4:51 PM EDT: @sgrimm started a stack merge that includes this pull request via Graphite.


Disabled adding jenkins user to audio group armv7l

Disabled adding jenkins user to audio group for armv7l

type-o


Adding a u1 tag along with the current u8, u16, u32, u64 and field. Booleans are widely used, and without u1 memory cells the compiler will need to codegen bit masking on top of u8s to handle casts to u1 and some u1 arithmetic operations. We could also swap u16 with u1 since booleans will probably see much more use than u16s.

Done


⚠ 한경주요지수-코스피200 has degraded performance

In 97fcf20, 한경주요지수-코스피200 (https://markets.hankyung.com/indices/kospi200) experienced degraded performance:

HTTP code: 200
Response time: 6725 ms

Resolved: 업종등락 performance has improved in 6e1449f.


How to create a volume without permision like root

Im working with docker to create two containers: django & postgres.
What is my issue?
I want to create a volume in my project to has a database persistent and commit to my repository to download in another machine and has the same database. So, when I create a volume in my docker-compose.yml inside my project, after maked docker-compose up the folder created with volume of postgresql in my project, this have a permision like root, so I cant commit the folder in my repository...
This is my docker-compose file:

version: '3'
services:

    db:
      #build: ./postgresqlDocker/
      image: postgres # Utiliza una imagen la cual en caso de no tener, la descarga
      environment:
        POSTGRES_USER: postgres #CREA UN USUARIO Y UNA BASE DE DATOS CON PRIVILEGIOS
        POSTGRES_PASSWORD: password123 # LE ASIGNA UNA PASSWORD AL USUARIO Y SU BASE DE DATOS
        POSTGRES_DB: test_db
      ports:
        - "5432:5432" # (HOST:CONTAINER)
      volumes:
        - ./data/postgres:/var/lib/postgresql/data/

    web:
      build: .
      command: python manage.py runserver <IP_ADDRESS>:8000
      volumes:
        - .:/code
        # (HOST:CONTAINER)
      ports:
        - "8000:8000"
        # (HOST:CONTAINER)
      depends_on:
        - db

This is the definition of database connection in my settings.py:

DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.postgresql_psycopg2',
        'NAME': 'postgres',
        'USER': 'postgres',
        'PASSWORD': 'password123',
        'HOST': 'db',
        'PORT': 5432,
    }
}

And when I want to access to postgres folder:
cd data/postgres/ bash: cd: data/postgres/: Permission denied
I hope can you help me!
Thanks everyone!

This is an issue related to the uid that postgres is initializing with (probably uid:999 for docker).  The directory will be owned and populated by that user which is not the user you're logged in as.  The easiest way around this is to make sure the uids line up as discussed here: https://github.com/docker-library/docs/tree/master/postgres#arbitrary---user-notes.
So, you can approach this two ways.  If you are planning to checkout a pre-existing database, you just need to make sure that the --user uid:gid you specify has permission to the directory on the host.  Ideally, you want that to be your user, but at least a user you can su to, otherwise you'll have the same problems accessing the directory on the host.
If you're going to checkout later with no pre-existing database, you need the db initialization (initdb) to run inside the container.  However, if you specify a --user uid:gid so the container runs as a user that has access on the host, that uid:gid still needs to exist in /etc/passwd inside the container, or initdb will fail.  The two ways to handle that are described in the URL above, though in the second case, you'd still want to replace 1000:1000 with your own uid:gid.
Hope that helps!

@qedrakmar I really appreciate your help! I will try to fix the issue, because Im a novice in docker ... But thanks a lot! I will post again if I can't fix the issue

@qedrakmar Where I must to specificate the uid:gid ? From my docker-compose file or creating a docker volume?

The UID:GID would be specified in the compose file, as mentioned here.  Note, this docs example show just a user with no GID, but both formats are valid.

Closing, since this appears to be resolved.  In the future, these sorts of questions/requests would be more appropriately posted to the Docker Community Forums, the Docker Community Slack, or Stack Overflow.


File Content as Input Value

Scenario:
I need to extract the contents of the src_addr.txt file and use it as an input value to my Firewall Module.
Is there a proper way to do this?
My Current Firewall Module Usage:
src_addr.txt Content
[
    "<IP_ADDRESS>"
]

Terragrunt Inputs
nat_config = {
    source_addresses = file("./assets/src_addr.txt")
}

Expected output:
Build should push through because the content of the txt file is a set of string
Actual Output:
Error: Inappropriate value for attribute "source_addresses": set of string required.

You need to use json or yaml with yamldecode/jsondecode. In fact, the contents of the file is valid json already so wrapping the file call with jsondecode should do the trick.

You need to use json or yaml with yamldecode/jsondecode. In fact, the contents of the file is valid json already so wrapping the file call with jsondecode should do the trick.

Thank you for this. I'll try it as soon as I can.

This seems an easy problem to solve, thank you for helping. Does Terragrunt inherit all Terraform Functions? Do we have a documentation stating these capabilities?
Also, This worked. :)
source_addresses = jsondecode(file("./assets/test.json"))

Does Terragrunt inherit all Terraform Functions? Do we have a documentation stating these capabilities?

Yes see https://terragrunt.gruntwork.io/docs/reference/built-in-functions/#terraform-built-in-functions

Does Terragrunt inherit all Terraform Functions? Do we have a documentation stating these capabilities?

Yes see https://terragrunt.gruntwork.io/docs/reference/built-in-functions/#terraform-built-in-functions

Yes I've been reading this. Is it safe to assume that all terraform built-in functions are compatible with Terragrunt?

Yes you can assume all the functions are compatible. If you find any that is not (which can happen if you are using functions released in newer terraform versions), file an issue and we can merge in the newer function relatively quickly.


Incorrectly importing unused exports

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Generated main.js contains code for SomeClass and OtherClass
generated main.js
(window["webpackJsonp"] = window["webpackJsonp"] || []).push([["main"],{

/***/ "./src/OtherClass.ts":
/*!***************************!*\
  !*** ./src/OtherClass.ts ***!
  \***************************/
/*! no static exports found */
/***/ (function(module, exports, __webpack_require__) {

"use strict";

Object.defineProperty(exports, "__esModule", { value: true });
class OtherClass {
    constructor() {
        console.log('OtherClass');
    }
}
exports.OtherClass = OtherClass;

/***/ }),

/***/ "./src/SomeClass.ts":
/*!**************************!*\
  !*** ./src/SomeClass.ts ***!
  \**************************/
/*! no static exports found */
/***/ (function(module, exports, __webpack_require__) {

"use strict";

Object.defineProperty(exports, "__esModule", { value: true });
const OtherClass_1 = __webpack_require__(/*! ./OtherClass */ "./src/OtherClass.ts");
var MyEnum;
(function (MyEnum) {
    MyEnum["on"] = "on";
    MyEnum["off"] = "off";
})(MyEnum = exports.MyEnum || (exports.MyEnum = {}));
class SomeClass {
    constructor() {
        const o = new OtherClass_1.OtherClass();
        console.log('SomeClass', o);
    }
}
exports.SomeClass = SomeClass;

/***/ }),

/***/ "./src/entry.ts":
/*!**********************!*\
  !*** ./src/entry.ts ***!
  \**********************/
/*! no static exports found */
/***/ (function(module, exports, __webpack_require__) {

"use strict";

Object.defineProperty(exports, "__esModule", { value: true });
const SomeClass_1 = __webpack_require__(/*! ./SomeClass */ "./src/SomeClass.ts");
console.log(SomeClass_1.MyEnum);

/***/ })

},[["./src/entry.ts","webpack-runtime"]]]);

If the current behavior is a bug, please provide the steps to reproduce.
Basic setup
npm i webpack webpack-cli ts-loader typescript
files
webpack.config.js
const path = require('path');

module.exports = {
    entry: {
        main: './src/entry.ts'
    },
    mode: 'development',
    devtool: 'none',
    resolve: {
        extensions: ['.ts', '.tsx', '.js'],
    },
    optimization: {
        runtimeChunk: {
            name: 'webpack-runtime'
        }
    },
    module: {
        rules: [
            {
                test: /\.tsx?$/,
                loader: 'ts-loader'
            }
        ]
    },
    output: {
        path: path.resolve(__dirname, 'dist'),
        filename: '[name].js',
    }
};

tsconfig.json
{
    "compilerOptions": {
        "module": "commonjs",
        "moduleResolution": "node",
        "pretty": true,
        "lib": ["dom", "es6"],
        "target": "es6"
    },
    "exclude": [
        "node_modules"
    ],
    "include": [
        "src/**/*.ts"
    ]
}

src/entry.ts
import {MyEnum} from './SomeClass'
console.log(MyEnum);

src/SomeClass.ts
import {OtherClass} from './OtherClass';

export enum MyEnum {
    on = 'on',
    off = 'off'
}

export class SomeClass {
    constructor() {
        const o = new OtherClass();
        console.log('SomeClass', o);
    }
}

src/OtherClass.ts
export class OtherClass {
    constructor() {
        console.log('OtherClass');
    }
}

Run
npx webpack
What is the expected behavior?
Generated main.js should not have any references to SomeClass, OtherClass as it only imported the enum.
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
node v9.2.0
package.json
 "ts-loader": "^4.0.1",
  "typescript": "^2.7.2",
  "webpack": "^4.1.1",
  "webpack-cli": "^2.0.12"

Additional Notes
It is worth noting the following setup does work correctly as SomeClass is never accessed it is only used for type definitions.
Change entry.ts to the following
import {SomeClass} from './SomeClass'

const a:SomeClass = null;
console.log(a);

Results in
generated main.js
(window["webpackJsonp"] = window["webpackJsonp"] || []).push([["main"],{

/***/ "./src/entry.ts":
/*!**********************!*\
  !*** ./src/entry.ts ***!
  \**********************/
/*! no static exports found */
/***/ (function(module, exports, __webpack_require__) {

"use strict";

Object.defineProperty(exports, "__esModule", { value: true });
const a = null;
console.log(a);

/***/ })

},[["./src/entry.ts","webpack-runtime"]]]);

Workarounds
Use const enums as they are removed upon compilation
import {OtherClass} from './OtherClass';

export const enum MyEnum {
    on = 'on',
    off = 'off'
}

export class SomeClass {
    constructor() {
        const o = new OtherClass();
        console.log('SomeClass', o);
    }
}

or
Move interface and enum declarations into another file and import them into both entry.ts and SomeClass.ts

Your tsconfig.json is incorrectly set up. It's compiling the module to commonjs, instead of leaving es2015 or esnext modules.

I wonder if a tree-shaking equivalent of the optimizationBailout stats feature might make it easier for users to realise why tree shaking wasn't working in cases like this - and reduce the number of issues filed?

I'm closing this issue. Thanks @Kovensky for the diagnostic!

Hey @Kovensky and @ooflorent, thanks for the response but this does not seem to be working for me. Please let me know if I am still doing something foolish though, I should have thought to compile to modules and not commonjs, that was dumb...
If I change my tsconfig.json to the following
{
    "compilerOptions": {
        "module": "es2015",
        "moduleResolution": "node",
        "pretty": true,
        "lib": ["dom", "es6"],
        "target": "es6"
    },
    "exclude": [
        "node_modules"
    ],
    "include": [
        "src/**/*.ts"
    ]
}

when running npx typescript I am getting the following for entry.js
import { MyEnum } from './SomeClass';
console.log(MyEnum);

which looks correct, but when running webpack the generated main.js is as follows
(window["webpackJsonp"] = window["webpackJsonp"] || []).push([["main"],{

/***/ "./src/OtherClass.ts":
/*!***************************!*\
  !*** ./src/OtherClass.ts ***!
  \***************************/
/*! exports provided: OtherClass */
/***/ (function(module, __webpack_exports__, __webpack_require__) {

"use strict";
__webpack_require__.r(__webpack_exports__);
/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "OtherClass", function() { return OtherClass; });
class OtherClass {
    constructor() {
        console.log('OtherClass');
    }
}

/***/ }),

/***/ "./src/SomeClass.ts":
/*!**************************!*\
  !*** ./src/SomeClass.ts ***!
  \**************************/
/*! exports provided: MyEnum, SomeClass */
/***/ (function(module, __webpack_exports__, __webpack_require__) {

"use strict";
__webpack_require__.r(__webpack_exports__);
/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "MyEnum", function() { return MyEnum; });
/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "SomeClass", function() { return SomeClass; });
/* harmony import */ var _OtherClass__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ./OtherClass */ "./src/OtherClass.ts");

var MyEnum;
(function (MyEnum) {
    MyEnum["on"] = "on";
    MyEnum["off"] = "off";
})(MyEnum || (MyEnum = {}));
class SomeClass {
    constructor() {
        const o = new _OtherClass__WEBPACK_IMPORTED_MODULE_0__["OtherClass"]();
        console.log('SomeClass', o);
    }
}

/***/ }),

/***/ "./src/entry.ts":
/*!**********************!*\
  !*** ./src/entry.ts ***!
  \**********************/
/*! no exports provided */
/***/ (function(module, __webpack_exports__, __webpack_require__) {

"use strict";
__webpack_require__.r(__webpack_exports__);
/* harmony import */ var _SomeClass__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ./SomeClass */ "./src/SomeClass.ts");

console.log(_SomeClass__WEBPACK_IMPORTED_MODULE_0__["MyEnum"]);

/***/ })

},[["./src/entry.ts","webpack-runtime"]]]);

So the workaround of changing entry.ts to
import {MyEnum} from './SomeClass'
console.log(MyEnum.on);

and exporting the enum as a const like this in SomeClass.ts
import {OtherClass} from './OtherClass';

export const enum MyEnum {
    on = 'on',
    off = 'off'
}

export class SomeClass {
    constructor() {
        const o = new OtherClass();
        console.log('SomeClass', o);
    }
}

results in a proper generated main.js
(window["webpackJsonp"] = window["webpackJsonp"] || []).push([["main"],{

/***/ "./src/entry.ts":
/*!**********************!*\
  !*** ./src/entry.ts ***!
  \**********************/
/*! no static exports found */
/***/ (function(module, exports) {

console.log("on" /* on */);

/***/ })

},[["./src/entry.ts","webpack-runtime"]]]);

This is because a const enum is not a value; it only exists as a type, and thus gets replaced by TypeScript because you're not using --isolatedModules.
The strange part is that the second argument to webpack_require.d is not a single character string; this means the module namespaces are escaping. Are you passing the module objects themselves to another function?

I understand const enum only exists as a module, when I use const everything is ok. The issue is when I do not use const I am getting SomeClass and OtherClass in my generated entry.js even though I never use them.
SomeClass.ts
import {OtherClass} from './OtherClass';

export enum MyEnum {
    on = 'on',
    off = 'off'
}

export class SomeClass {
    constructor() {
        const o = new OtherClass();
        console.log('SomeClass', o);
    }
}

entry.ts
import { MyEnum } from './SomeClass';
console.log(MyEnum);

Results in SomeClass and OtherClass ending up in the generated js file.
As far as passing the module to another function, all the code for the example is included. This code is not in another project and is recreatable by using just the code above.
Honestly the const enum workaround is fine for my project I am just concerned there is a deeper issue that will bring in unneeded code for other projects.

This is a known limitation of webpack. See #6264.
The tree shaking is not a good as we would like to.

Tree Shaking is only enabled in production mode.

@ooflorent thanks! easy enough to work around for now.


InvalidOperationException: Configuration value 'Verbose' for category 'Default' is not supported in AuthenticationMiddleware ctr

this is for rc2 version of code.
If there is this line in Startup.Configure
loggerFactory.AddConsole(Configuration.GetSection("Logging")); // It's exist there by default btw
than Implementation of AuthenticationMiddleware throws InvalidOperationException when call base class constructor. "Configuration value 'Verbose' for category 'Default' is not supported."
Stacktrace:
at Microsoft.Extensions.Logging.Console.ConfigurationConsoleLoggerSettings.TryGetSwitch(String name, LogLevel& level)
   at Microsoft.Extensions.Logging.Console.ConsoleLoggerProvider.GetFilter(String name, IConsoleLoggerSettings settings)
   at Microsoft.Extensions.Logging.Console.ConsoleLoggerProvider.CreateLoggerImplementation(String name)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Microsoft.Extensions.Logging.Logger..ctor(LoggerFactory loggerFactory, String name)
   at Microsoft.Extensions.Logging.LoggerFactory.CreateLogger(String categoryName)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1..ctor(RequestDelegate next, IOptions`1 options, ILoggerFactory loggerFactory, UrlEncoder encoder)
   at Test.AuthMiddleware..ctor(RequestDelegate next, IOptions`1 options, ILoggerFactory loggerFactory, UrlEncoder encoder)

@ivankarpey what does the Logging section in your configuration look like?

"Logging": {
    "IncludeScopes": false,
    "LogLevel": {
      "Default": "Verbose",
      "System": "Information",
      "Microsoft": "Information"
    }
  }

Well Verbose is RC1, and removed in RC2

@304NotModified is right.  Either use Debug or Trace. Please file another one if that doesn't solve the issue!

This seems like a user-unfriendly behaviour since other Microsoft logging frameworks have always supported "Verbose".
I'd rather see it translate "Verbose" as an alias to "Debug" for me, than terminate my startup if someone tried to use values the obtained after googling and hitting this page:
https://msdn.microsoft.com/en-us/library/microsoft.windowsazure.diagnostics.loglevel.aspx

@wpostma We have looked into several logging frameworks and user feedback through RC1 to decide on the log levels.
Not sure how "translating" would work at all?  In any case, since we RTM'd, we won't be changing the public API surface.


Automate backup create is failing

Describe the bug
Automate backup create is failing with error:
BackupError: An issue occurred while creating or restoring a backup: Streaming backup events failed: Unable to handle backup event: Backup failed. Check the Chef Automate logs for more information.
when data in automate is more. Data here refers to Nodes inside Chef Infra Server or Compliance reports.
It works fine with less or no data.
To Reproduce
Steps to reproduce the behavior:

Spin up an EC2 instance with automate and load some data of nodes or compliance around 10k nodes will be sufficient.
Try to take multiple backups of automate (it fails)

Expected behavior

Backup create should be successful.

Screenshots

Versions (please complete the following information):

Automate Build Number<PHONE_NUMBER>1356

Additional context
Issue in Zendesk https://getchef.zendesk.com/agent/tickets/29555

Possible fix(es) for ZD29555, ZD29807 and ZD29861 in https://github.com/chef/automate/pull/6733

Fixed in https://github.com/chef/automate/pull/6733


Addressing issue #120

I polished the hamburger menu. Redid the logic; positioned the <BurgerModal> into the <BurgerMenu>;  made some styling changes.  <BurgerModal> doesn't scroll with the rest of the page.

Thanks for the PR @asharonbaltazar
This is good work <3

Thank you for sending in this PR and making the project better 🥳
Let this go in towards HacktoberFest 🥇


Split events

Nowadays in the PermissionsDiff we calculate in Classroom we have many issues:

If someone register a teacher as a student in a course, this has no effect in classroom.
The event action is not idempotent.

We need to split the StudentChanged event into more events:

StudentCreated
StudentUpdated
StudentRegistered

The described semantics are tricky too, since it could lead to race conditions among multiple servers and process the update event before the create event. Won't fix.


add new term MONDO:0100478 brain malformations with or without urinar…

…y tract defects
close #5361
@sabrinatoro
The term requester noted that this term is related to 1p32-p31 deletion syndrome in that NFIA is a gene within this region and individuals with variants in NFIA recapitulate many of the features of the microdeletion syndrome. We are unsure of the proper way to attribute parent and children terms in this situation. We are looking to the MonDo team for advice.
I don't think 1p32-p31 deletion syndrome should be a child of this term (and it is not inferred to be). Do you agree? Thanks for reviewing. :)

If anything, "1p32-p31 deletion syndrome" would be a parent term, and not a child term of 'brain malformations with or without urinary tract defects'. However, in this case, I think the best way is to add the comment, as you did.
IF necessary, in the future, we could have 2 new terms: '~ due to variation in the gene', and '~ due to 1p32-p31 deletion'.

perfect, thanks @sabrinatoro!


Hide name (or state) in picture-entity

I think it would be useful to be able to hide the name or state in picture-entity and center the one that is showing.
Especially for example showing the presence state, there are no reason to show who it is if the picture already showing the person. I made a custom card for this but I think this is something that would be useful for more people as part of the "product".

If we allow name and state to be individually toggled, we should get rid of the show_info config option.
https://home-assistant-lovelace-gallery.netlify.com/#demo-hui-picture-entity-card


Input onFocus() and onBlur() never called on IOS

              <Input
                label="Additional information"
                placeholder="Don't forget to bring..."
                multiline
                onFocus={this.scrollToFocusedInput}
                value={this.state.info}
                onChangeText={this.onInfoChange}
              />

scrollToFocusedInput never gets called on IOS but works as expected on Android

Hi @veedeo 👋 Sorry for the late reply. Was it resolved?

Yep was my fault
On Mon, Oct 28, 2019 at 9:38 AM Artur Yorsh<EMAIL_ADDRESS>wrote:

Hi @veedeo https://github.com/veedeo 👋 Sorry for the late reply. Was
it resolved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/akveo/react-native-ui-kitten/issues/697?email_source=notifications&email_token=AAID5NVR7VUOV7MHYPKCRXLQQ2JFZA5CNFSM4JFNLE5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECL6TNA#issuecomment-546826676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAID5NUEQZ6ENN5DS42S2XDQQ2JFZANCNFSM4JFNLE5A
.


Startup preferences infrastructure

JforD has a lot of toggle settings and startup preferences.  I don't find a compelling need for any of them currently... there are a lot more important things.
However, we probably eventually will want some way to set preferences on startup.  This may be handled differently in a Pharo dev environment than it would be on a windows-based application; it would be nice to not have to use a file, although a file may make it easier to move from image to image. It would be good to have a design for this in place.
I suggest autocommitEnabled as a preference that would be useful and easy (you'll need something for testing with).

I've ported the beginnings of the JfD preferences on startup system to JfP. Several things work like opening browsers, etc. https://github.com/GemTalk/JadeiteForPharo/commit/a51f50b7e0e0f180cd9760f68e7bbb1023ed5d80

@dalehenrich reports that the autocommit preference isn't working yet

[ ] Also, make sure Jadeite.prefs writes to the pharo directory on the client. Not sure why it's trying to go to the server.

Preferences file was using superclass workspace file menu options it shouldn't. Ctrl-S saves locally and a local save option was added to the menu. https://github.com/GemTalk/JadeiteForPharo/commit/0ab209ee9cba6e847ddaa328dcd48b3694ca078a
Autocommit button display updates properly in console now. Preference was working but not displaying correctly.

I did some experimenting and was able to integrate a JfP setting into the Pharo settings browser.
Pharo settings are, by default, universal to all images so (in theory) you'd just need to hit Store Settings  in the Pharo browser to save the settings for all your images.
Not sure which settings are universally helpful, but autocommit is one. Need to explore further.

Added open browser on class after connect setting.


Move files around and add features to roslibrust

Since you were asking about which crates were necessary earlier, I took a stab at reducing things down and then reorganizing the roslibrust crate around features.
This MR is probably going to be a mess to review, but I promise I didn't change any code, just moved it around. Probably easier to pull and view in editor.
Highlights:

Added the following features to roslibrust: client, utils, codegen, all, default
Put all of the rosbridge communication stuff inside a module, but re-exported so it'd be the same.
Removed need for roslibrust_codegen and roslibrust_rospack, unsure how to mark them as deprecated on crates.io
Moved message files out of roslibrust since we don't need them inside the crate dir structure anymore.
Deleted the old code in utils.rs since it was hanging around from the previous codegen method.

Closes #31

I went ahead and merged figuring this was moving in the right direction, and it would be easiest to address stuff by building on top of. Thanks for the clean-up. I'm not sure I can fully remove the dead crates, but I can at least yank them.


New Feature [Artists] Get artist's related artists

https://developer.spotify.com/documentation/web-api/reference/get-an-artists-related-artists

deprecated endpoint :(


Anyway to stop full decryption and decompiling?

ie you can simply
#!/usr/bin/python3
from pyce._crypto import decryptf

with open('output.pyc', 'wb') as output:
    output.write(decryptf('./__init__.pyce', '3e8a6f52eb5b41e0883c594dc026c70aa04258790ca71259d9d1c1bf47cfc0aa'))

Then use  uncompyle6 output.pyc to get full source..
if you have to use 3rd party to encrypt the source, what is the point in this module?

@VeNoMouS could you describe your attack model in more detail?  Would the attacker have full root access to a system when you execute?  Would they be able to do code injection?  Would they start with only access to your .pyce files?
Typically, what you're talking about is restricted by key distribution techniques which is not within the scope of this module.
This module handles the low-level encrypt/decrypt operations, but where the keys come from is up to you or your organization.  For example, you could run a key server on the Internet and only provide them to licensed users.
Obviously, your code has to decrypt somewhere before hitting the CPU.  Attackers could get the code at any point during that window of execution.
For example, a cloud provider could read the RAM of your VM and take the decrypted code from pages in your memory.
What matters here is:

Your attack model
Your key distribution technique

To recap this, if you had the pyce and you had the key, you can simply decode the whole thing without execution... with what i outlined in the original post.
The whole "key distribution technique" argument is head in a paper bag response, your simply pushing the issue/problem of this flaw back onto the user.
I originally looked at this project to distribute private code to users without them reverse engineering my work...
But once ANYONE has a key you can simply reverse engineer all the way back to source,
pyarmor is a better approach in the fact decryption is closed source and only distributed in binary form
While yes, you can place traces around the stack and still reverse engineer but that will take far more time / effort than the ~3 lines of code I provided to convert back to source that it takes with this project.
I'm sorry but I fail to see how this project is designed for distributed code encryption, when it can be reversed to full source so easily.


Fix/lum 12420 select search

General summary

Screenshots

Check list

[x] (if has style) Add need: review-integration label
[x] (if has textual documentation) Add need: review-doc label
[x] (if has code) Add need: review-frontend label
[ ] (if has react) Check through the react dev check list
[ ] (if fix or feature) Add need: test label

Check out the contribution guide for general guidelines for submissions to the design system.

Test OK


Git based workflow for .atd files

The git workflow should probably be first class (files workflow being a secondary citizen). I was thinking about something like:
stork gen 1:34fdsd3 2:fasfsdf 3:abc123 4 -o ./prefix/to/file

So the syntax is <version_number>:<commit_hash>
Where the last one is for the current file.

yeah looks good, I'm pretty sure we will run into some issues when rebasing stuff in PRs but other than that I don't foresee any problems.

I think it's not actually an issue.  You only care about the contents. And the previous ones should be on master already.

oh yeah that's right!

You can use git show <commit>:<atdfilepath>

addressed by #22


Add ap-northeast-3 region

Please add "Osaka, Japan" region labeled by the ID ap-northeast-3

well, it appears to be existent in master codebase


Which temperature to read

@BoostCookie  please elaborate

Let the user specify in the config file which temperature should be used. For example some cards have temp1_input, temp2_input, and temp3_input with temp1_label=edge, temp2_label=junction, temp3_label=mem. The user should be able to specify which of those should be used.
Optionally if people (and @Eraden) think it makes sense: Let the user specify multiple temp inputs and the temperature being used for the fan curve is then the max value of all specified temperatures.

This sounds good for me. Want to do it? :)

Can we please first talk about whether or not the config file should support different fan curves for different devices? As I've elaborated here, right now the program only really works if there is exactly one amd graphics card.

We can make it very easy for ourselves and throw away the whole cards = ["card0"] argument and just always use the first hwmon we find with name=amdgpu. Users with multiple GPUs are rare and the name of the program suggests that it is only meant for amd cards.
We can put a lot of effort into making this a general fan control program that supports any hwmon device with a pwm_input and allow multiple fancurves.

My idea how we should approach problem:
Step 1.

Remove cards entirely. It serve no real purpose right now since we can't ensure which card will be loaded first by OS for normal user. I shown fix in issue #11 but I can't force this on user.

Config is working for every single AMD card the same way.

Add temp_input as i8 so user can choose which one should be observed.

Step 2.

Add amdfan-config binary for simpler way of configuration so new user can quickly set everything up if he has more than 1 card. Requirements:

read all existing card files (more below) and print them, ask if edit one of them or create new
read and print amd card device value
take N points of fan temp -> speed %
allow to go back d and finish w if there is 100% speed
if there is no 100 inform it's required and suggest 80 => 100 when w was chosen

Rely on device/device sysfs file as a way of recognizing card. If there is 2 or more identical cards then config most likely apply for all of them, if not card is recognized by it number.

Split config:

/etc/amdfand/fan.toml - only log level for now
/etc/amdfand/cards/cardX.toml - temperature matrix, device (either 0xNUM or 'any'), temp_input (temp reading file number)

This will be considered as version 1.1.0

I agree with step 1.
Maybe instead of having temp_input as i8 we should have input_temp_file as String, but it probably doesn't matter.
Regarding step 2: I don't think a separate binary is necessary. Let's just add something like amdfand setup that helps you setup the config.
And I don't think we should ever support multiple AMD cards since the OS just scrambles them on every boot and we can't rely that next time the card with number 0 will still be the same one.

And I don't think we should ever support multiple AMD cards since the OS just scrambles them on every boot and we can't rely that next time the card with number 0 will still be the same one.

This is why I want use device/device

This is why I want use device/device

What do you mean? We don't need to use anything. Just look through /sys/class/drm/card[1-∞] and take the first one with name=amdgpu. You only had a problem because we hardcoded the card number in the config and on the next boot your nvidia card had that number. If we just always take the first amd card it works for everybody who has exactly one amd card.

If user has only one card then we don't need to check anything.
If user has 2 AMD cards then i can check which card is it by checking value in those files:
/sys/class/drm/card0/device/hwmon/hwmon1/device/device
/sys/class/drm/card1/device/hwmon/hwmon1/device/device
And if user don't want to use service for one of them or want other matrix for those then we can recognize it with this value.
Then etc will looks like this:
/etc/amdfand/fan.toml
/etc/amdfand/cards/card0.toml
/etc/amdfand/cards/card1.toml

Which card is which can be checked with following command:
glxinfo -B | grep Device

For example in my setup for 0x731f it's Device: AMD Radeon RX 5700 XT (NAVI10, DRM 3.40.0, 5.10.55-1-lts, LLVM 12.0.1) (0x731f)

This would make it all much more complicated just to support the tiny percentage of people with multiple AMD cards. Also naming the configs card0.toml and card1.toml isn't a good idea because the numbering changes from boot to boot. And are the 4 digit hex codes in /sys/class/drm/card0/device/device actually unique? Maybe they are identical if a user has two identical cards.

I asked people on reddit and asked GamingOnLinux if they have and could share statistics about multiple AMD GPU share

To prevent creating cliffhanger I'll write it here.
I'm strongly against dropping multi card support.
Now, sorry for huge delay. I was thinking a lot about this and struggle internally to keep it or not.
I launched poll on Reddit and asked GamingOnLinux and Steam for Adobe information about used hardware.
Reddit as far as revealed about 9% it's not reliable due to small amount of users participating.
GamingOnLinux does not gather such informations.
Steam refused to share such information
I was considering asking Canonical but I need to become partner and I don't know how long it will take and if effort is worth it

I'm strongly against dropping multi card support.

Then we need to find somebody who does have two identical cards so that they can tell us if /sys/class/drm/card0/device/device is identical for both cards or not. My guess is that it is.
If that is the case I suggest we use the pci path instead.


OFFBOARD using LPE

Hi,
I have a question regarding the OFFBOARD mode.
I want to use the OFFBOARD mode and send waypoints to my drone. I'm flying indoor using only optical flow and the Local Position Estimator.
To stream the MAVLink messages I use an iPad connected over BLE to the drone.
I used the C-UART-Interface (https://github.com/mavlink/c_uart_interface_example) as template and tried to implement the same thing on my iPad.
So basically I’m streaming a constant X,Y,Z SET_POSITION_TARGET_LOCAL_NED message. However, when I switch into OFFBOARD mode via RC (or via iPad), the drone starts accelerating really fast.
I tested it without props and while holding the drone with my hand:
http://logs.uaventure.com/view/SPXqCQfahAWj2nc9mEP5uN
I also did another test, where I switched into position hold before switching into OFFBOARD. The result seems to be the same:
http://logs.uaventure.com/view/E79cnzahir3pGqWFsZPeFJ
The plot for Local Position X,Y and Z clearly shows that the setpoint is still changing. The plot for the actuators shows how the drone is accelerating uncontrollably.
I expected that after sending a constant position target all the setpoints should show the constant position and the drone should hold its position there.
My goal is to use the iPad to let the drone hold its position at different waypoints (for example flying a square figure).
Can you give me some more information about the OFFBOARD mode? Especially how the LPE position is used.

@MegaAdragon Looking into this, maybe related to #4352

Hi @MegaAdragon how did you implement Pixhawk with the LPE?

@vinhk Set SYS_MC_EST_GROUP = 1 and reboot.

I flew an auto mission with LPE recently so I believe this has been resolved @MegaAdragon? If there are problems with your offboard config it most likely isn't LPE related.

James if you have time. Do you know how I can find out if my px4flow is
working with pixhawk? Is there a way to see data being sent to pixhawk from
optical flow?
On Saturday, May 14, 2016, James Goppert<EMAIL_ADDRESS>wrote:

I flew an auto mission with LPE recently so I believe this has been
resolved @MegaAdragon https://github.com/MegaAdragon? If there are
problems with your offboard config it most likely isn't LPE related.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219258651

@vinhk  Yes, just plug in usb and start qgroundcontrol. Under the setup page you should see a px4flow display. Use this to focus the camera using text at the distance you intend to flight at.

I see. But without outputs from pixhawk, I'm not sure if it is working my
my pixhawk. I have successfully been able to view the display in
qgroundcontrol. Based on my knowledge of making px4flow work with pixhawk
is upload firmware and just connect serial wires from px4flow to pixhawk.
On Sunday, May 15, 2016, James Goppert<EMAIL_ADDRESS>wrote:

@vinhk https://github.com/vinhk Yes, just plug in usb and start
qgroundcontrol. Under the setup page you should see a px4flow display. Use
this to focus the camera using text at the distance you intend to flight at.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219315301

With px4flow connected to pixhawk, QGC should
show MAVLINK_MSG_ID_OPTICAL_FLOW_RAD messages in the mavlink inspector
On Sun, May 15, 2016 at 4:42 PM vinhk<EMAIL_ADDRESS>wrote:

I see. But without outputs from pixhawk, I'm not sure if it is working my
my pixhawk. I have successfully been able to view the display in
qgroundcontrol. Based on my knowledge of making px4flow work with pixhawk
is upload firmware and just connect serial wires from px4flow to pixhawk.
On Sunday, May 15, 2016, James Goppert<EMAIL_ADDRESS>wrote:

@vinhk https://github.com/vinhk Yes, just plug in usb and start
qgroundcontrol. Under the setup page you should see a px4flow display.
Use
this to focus the camera using text at the distance you intend to flight
at.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219315301

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219315607

Ok thank you. I'll look into it.
On Sunday, May 15, 2016, Mark Whitehorn<EMAIL_ADDRESS>wrote:

With px4flow connected to pixhawk, QGC should
show MAVLINK_MSG_ID_OPTICAL_FLOW_RAD messages in the mavlink inspector
On Sun, May 15, 2016 at 4:42 PM vinhk<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

I see. But without outputs from pixhawk, I'm not sure if it is working my
my pixhawk. I have successfully been able to view the display in
qgroundcontrol. Based on my knowledge of making px4flow work with pixhawk
is upload firmware and just connect serial wires from px4flow to pixhawk.
On Sunday, May 15, 2016, James Goppert<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

@vinhk https://github.com/vinhk Yes, just plug in usb and start
qgroundcontrol. Under the setup page you should see a px4flow display.
Use
this to focus the camera using text at the distance you intend to
flight
at.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219315301

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219315607

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/issues/4329#issuecomment-219316225

You need to interface it via I2C, NOT serial.


Refactor aurora scheduler to use SchedulerUtils to construct arguments

Refactor aurora scheduler code to be more consistent with other schedulers.
Topology arguments are hardcoded in aurora config file currently instead of using the shared functions in SchedulerUtils. It is hard to maintain and it is causing compatibility issue (since config file is in client and the setup code is in core).
fyi @jerrypeng

Christmas cleaning!


First portion of issue #122

Provide aker_version string, i.e. git commit hash string.

Codecov Report

Merging #124 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   74.76%   74.81%   +0.05%
==========================================
  Files          10       11       +1
  Lines         939      941       +2
  Branches      158      158
==========================================
+ Hits          702      704       +2
  Misses         88       88
  Partials      149      149

Impacted Files
Coverage Δ

service/version/upstream/aker_version.c
100% <100%> (ø)

Continue to review full report at Codecov.

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


Gateway network interruption

Installing the alerting backend with HA, doesn't set the replicas to 3 initially (not HA)
Uninstalling the alerting backend and reinstalling it as standalone, installs as HA (replicas) = 3

The order of operations should be:

install API -> get_config API -> edit config replicas to desired number (1 for standalone, 3 for HA) -> set config API with new config

Jumped the gun a little bit, a big with updating the gateway CRD for alerting caused the network connection to be interrupted between the UI and gateway.


Reminder - #5840 - Page 501 "Available - Purchase Lines" - OnAfterOpenPage

Regarding #5840 from January 2020, there is nothing to see in Insider Build 17.0.14751.0-W1.
Status since 24. Mar is "ships-in-future-update".
Will you still integrate it?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


CHEBI:138664 review status

Hello,
Could you please review the status of CHEBI:138664? We need it to validate RHEA:55265 (PMID:28248318).
Many thanks!
Nevila Hyka-Nouspikel

Hi Nevila,
Now done - sorry for taking so long.
Best regards,
Gareth

Thank you very much Gareth.
Have a great weekend,
Nevila
On 2017-10-27 13:20, G-Owen wrote:

Hi Nevila,
Now done - sorry for taking so long.
Best
regards,https://webmail.isb-sib.ch/rcmail/?_task=mail&_action=compose&_id=206003466959f35def1e8f7&_extwin=1#
Gareth
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or mute the
thread [2].

Links:
[1]
https://github.com/ebi-chebi/ChEBI/issues/3381#issuecomment-339944832
[2]
https://github.com/notifications/unsubscribe-auth/ARGKucFgMK5qtbSjpUsAHg-ShyDC4DZKks5swbxfgaJpZM4P27Eb

--
Nevila Nouspikel
SIB | Swiss Institute of Bioinformatics
CMU - 1, rue Michel Servet - 1211 Geneva 4
t: +41 22 379 51 25 - f: +41 22 379 58 58
<EMAIL_ADDRESS>- http://www.sib.swiss


Manuscript: Future Directions

Description:
Outline planned future developments for SysNDD, such as automated literature screening using PubTator API, integration of variant data, and incorporation of case-specific phenotype data.
Tasks:

[ ] Describe planned features and improvements.
[ ] Discuss potential collaborations and funding opportunities.
[ ] Highlight the expected impact of these future developments.

Duplicate, see #75


Add new rule to add single line after function decls

Motivation
To aid in visual processing and add clarity between function declaration and body, a newline is sometimes useful separating the two.
To enforce that, a new rule BlankLineAfterFunctionDeclarations was created, allowing two configurable exceptions:

single line functions, as these expressions are already compact and quite readable, so leaving them as is doesn't hinder visual processing.

super calls (when on first line), as these can be seen as being very "coupled" to the function declaration itself so it might make sense to keep this coupling without visual separation.

Some helpers were created to help adjust indentation while formatting:

Trivia.withSpacesOnly() to return trivia with whitespace only,

Indent.asTrivia to return a Trivia representation of an indent.

Discussion
This is the first of some rules/configurations I intend to add to swift-format, with the goal of eventually using it to apply our own style guide automatically, while also enriching current swift-format's capabilities.
However, I understand that some care must be taken before adding new rules, so that only rules that make sense for the broad community are added. From my perspective, rules should be generic while also being configurable to minimize total number of rules, avoiding having rules that are too specific.
That being said, not sure what's the "process" or "requirements" for adding new rules are (if any), and if my approach was the best one, as I am completely new to the codebase. I basically took some inspiration from BlankLineBetweenMembers and went from there, so please be gentle 😇.
Target branch and tooling notes
I developed this rule under swift-5.1-branch, as I was unable to get swift run generate-pipeline working with the development snapshot. Even on Xcode 11  with the swift-5.1-branch branch, I had to resort to swift package generate-xcodeproj as I couldn't run the tests in the "native" Xcode SPM integration due to some linker errors. Everything should work on master though.
🚧 WIP - missing work 🚧
In the meantime I realized my rule isn't yet applying to all function-like declarations, like init's, deinit's and subscript's. Will do once my approach is validated to avoid throwing away work.

I'll start by saying that this rule is one that is a bit, well, opinionated 😄

I agree 😄. But in a way isn't code formatting in general mostly opinionated? 😂

At the same time, the set of core rules that we provide should be generally useful to a fairly large audience. If we allow too many rules or knobs for configuration, the code base will quickly grow to be difficult to maintain/test reasonably.

Completely agree, and I know this is not an easy balance to achieve.

Looking at this rule and the specificity of its behavior (e.g., treating super differently), I'm not sure this is general enough to take on as a core rule. (And in complete fairness, I also want to look at our own existing rule set and prune some that may not have proved as useful as originally intended, or at least make them optional.)

Yeah, we might be pushing it with the super exception. I added it because it's a part of our style guide, and also as an exercise (in fact implementing this entire new rule was a great exercise to get to know the codebase and formatting mechanics). However, I think we might be able to drop this super exception if the remaining behavior is considered "valid" for the bigger audience.

That parenthetical statement leads to another issue, which is that right now, we don't have a way to make the rewriter rules disabled by default;
(...)
Having a rule like this be enabled by default would be a breaking change, since it would force that behavior on all existing users unless they knew to update their configuration after this went in.

Can't we in a way ensure we don't add any side effects by ensuring the default rule value doesn't modify existing behavior? At least that was my approach/interpretation.

Making some rules opt-in would be straightforward, but I think I'd like to strategize a bit more about other interesting ways that third-parties could hook into the formatter without forking it or adding rules to the core set. For example, maybe we could provide a way that users could build their own dylibs that exported custom rules, and then have swift-format detect and load those into the pipeline.

I do agree that some sort of dynamic plugin/rule system would be ideal so that any 3rd party could develop their own custom rules freely and use them on their own formatting pipeline.
In the meantime (and pardon me if this is a stupid idea), would it be easily feasible to use something along the lines of the XCTestManifests.swift for Linux? In the sense that we could have some configuration file that pointed to some custom rules .swift files, that were then built into the swift-format binary. I don't see that big of a overhead in building the binary every time we want to include a new custom rule, as it's not something you do very often, and you can store/distribute the binary easily. Just a thought, anyway.
Given we don't have anything like that for now, do you suggest 3rd parties keep their custom rules in their own forks?

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


nosec_alert CI action raises false positive

nosec_alert CI action raises false positive because it checks the entire diff for nosec without rule:
      - name: Check for '#nosec' without a specific rule
        run: |
          DIFF=$(git diff ${{ github.event.pull_request.base.sha }})
          echo "$DIFF" | grep -P '#nosec(?!(\sG\d{3}))(?![^\s\t])([\s\t]*|$)' && echo "nosec without specified rule found!" && exit 1 || exit 0

This means that removed lines in the diff are taken into account.
If we remove a nosec without rule it then raises a false positive
Example in consolidate build PR: https://github.com/zeta-chain/node-private/actions/runs/7005440405/job/19055177393?pr=22
    nosec_files: contrib/localnet/orchestrator/smoketest/main.go, contrib/localnet/orchestrator/smoketest/test_accounting.go, contrib/localnet/orchestrator/smoketest/test_bitcoin.go, contrib/localnet/orchestrator/smoketest/test_deposit_eth.go, contrib/localnet/orchestrator/smoketest/test_stress.go, contrib/localnet/orchestrator/smoketest/test_update_bytecode.go, contrib/localnet/orchestrator/smoketest/test_zeta_in_and_out.go, contrib/rpctest/main.go, rpc/namespaces/ethereum/debug/api.go, rpc/namespaces/ethereum/debug/trace.go, rpc/namespaces/ethereum/debug/utils.go, rpc/websockets.go, server/start.go, testutil/network/util.go, x/observer/module_simulation.go, zetaclient/bitcoin_client.go, zetaclient/inbound_tracker.go, zetaclient/config/config_chain.go
    nosec_detected: 1
-	// #nosec
-	defer resp.Body.Close() //#nosec
-	defer resp.Body.Close() //#nosec
-	//defer resp.Body.Close() //#nosec
-		/* #nosec */
-	/* #nosec */
-	/* #nosec */
-	/* #nosec */
-		/* #nosec */
-	err = os.WriteFile(file, contents, 0644) // #nosec
-/* #nosec */
-	// #nosec
nosec without specified rule found!

We should only check added lines (starting with a +)
Setting nosec_alert as not required in CI for now
cc @zeta-chain/devops

Not been encountered since then


fix: do not spread key on render props

Should fix #288

Try with passing the key as a prop directly to the render prop component instead of spreading it.

@kris-ellery Is there any update on this?

Is this fix still working? @m1nsuplee Does your review count as an approval?

Is this fix still working? @m1nsuplee Does your review count as an approval?

fix is still working

fix is still working

Does that mean any of you has the power to approve and/or merge this PR? If not, should the appropriate person be notified?

fix is still working

Does that mean any of you has the power to approve and/or merge this PR? If not, should the appropriate person be notified?

yes they should be notified


feat: drag to resize in the UI

Removed show/hide toggle from popup settings
Removed width slider from popup settings
Added handlebar to side panel to allow for resizing
double click on handlebar when open to hide it
single click on handlebar when closed to open it
keep dragging past the min-width to hide the side panel

https://github.com/marwanhawari/LeetRooms/assets/59078997/9f4015e9-d10d-4c14-9928-637c10bbe854

closes #2


Any plans to move from Spring Boot 2.4.6 -- > 2.5.1?

Any plans to move from Spring Boot 2.4.6 -- > 2.5.1?

or 2.5.2 which was released about 12 hours ago....

I see it is 2.5.4 in embedded-spring-boot-keycloak-server 5.0.0.  Thank you!