Not sure if we have descriptions for each media item

Description

Relates to https://github.com/WordPress/openverse/issues/142 in the main Openverse repository.
I'm currently unsure if we collect a textual description of each media item in the catalog. This would be a nice thing to expose via the api and display in the frontend.

Just to be clear - are these descriptions that would be made available by a provider's API, or something we would need to generate?

Perfect, thank you for those examples! I think it's probably worth keeping for all media types (I could imagine images, books, 3D models, all having descriptions). I would be concerned about the performance impact of making that field searchable though. Perhaps it's not a huge issue if we're indexing it on ElasticSearch

I would be concerned about the performance impact of making that field searchable though.

I agree! I'm cool with treating it like single-result metadata only, not a searchable field.

I'm not sure either, and would love to learn more, but here are some things I have noticed:

I don't see image.description in the image DDL or audio.description in the audio DDL, at least in the local staging DB
I don't them in the ingestion server either, though these are labelled "mock": image and audio.
the elastic search mapping does seem like it might be expecting a description field.
but the description parsing function seems to be written as if to say "hey, I'll take a description if you've got one!"

Does this mean that whenever the data is ready, we're ready to index it on elastic search to make it searchable, but we would have to either fully refresh the entire catalog database or accept that it would be only available on a limited set of image records at first? Are there any performance concerns on the elastic search side if we go with the former approach? Any bias / relevance concerns on the elastic search side if we go with the incremental approach?

Thanks for doing that dive Rebecca! By the looks of it, the Elasticsearch mapping is looking for that information within the media's meta_data JSON blob, so specifically meta_data["description"]. The implication for me here is that description can be added where reasonable/possible in provider scripts, but we don't have a column for it on any of the media tables. If anything, I think if we start to add descriptions, we might get more relevant results!
The question for me is, do we want to surface it as a nullable column or leave it as an optional meta_data field. I'm inclined to use the latter, since modifying the schema of the catalog can be an ordeal and we already have the machinery set up to parse it from meta_data. @stacimc @obulat do you have any additional thoughts here?

Thanks @AetherUnbound, that makes a ton of sense. I just looked through some of the provider scripts and I see that some do rename similar fields to be description in the metadata. And I now see there is this note in the image media store.

Perfect, thank you for those examples! I think it's probably worth keeping for all media types (I could imagine images, books, 3D models, all having descriptions). I would be concerned about the performance impact of making that field searchable though. Perhaps it's not a huge issue if we're indexing it on ElasticSearch

The description for results is part of the searched fields in the API. We search title, tags, and description: https://github.com/WordPress/openverse-api/blob/main/api/catalog/api/controllers/search_controller.py#L340
This is effective, by the way, in that you will sometimes get results for which the only field that produced a hit was the description. We don't surface the description on the frontend, however. Currently this feature is pretty obscure.


Offline/reward-labelled dataset support?

Just wondering if trl has support for offline learning? Was hoping to use trl since if I'm not mistaken, compatability with peft is being worked on, but I was also hoping to train offline.

Not sure I fully understand the question :) Indeed, PEFT is being worked on in #145. In terms of RL algorithms only PPO is implemented at this point.

In terms of RL algorithms only PPO is implemented at this point.

Thank you! This answers my question, I was wondering it trl had an implementation of ILQL.


Add default location for chrony.conf on Ubuntu

What does this PR do?
This PR adds the Ubuntu default location for chrony.conf to the list of locations returned by getLocalDefinedNTPServers
Motivation
Without this change the agent will fail to find the ntp config when using chrony on Ubuntu

Is there anything else I need to do to get this approved / merged?

Hi @kharry5, Thanks for your contribution!
Looks good to me! We just need to add a release note, and run the rest of the workflows before merging.
For reference, please check the CONTRIBUTING.md document which provides the guidelines for contributing to this repository.
Thanks!

@NouemanKHAL I've added the release notes, but I've added them from my work laptop that didn't have a real email address configured so now bits-bot is complaining about the CLA not being signed by that user.
Is there a way to correct this for this PR or should I close this and create a new one?

@kharry5 looks fine to me, could you update your branch with main? Feel free to rebase or merge the main branch, either works.

@kharry5 It seems that the commit history has become messy, could you rebase your branch with main and (force) push again?
You can refer to the steps documented here.

@NouemanKHAL I've attempted the rebase + force push. If that didn't do the trick then I'll just create a new PR.

/merge


Entry is not public in LinkedHashMap; cannot be accessed from outside package

Hi,
android compilation fails with error
...\platforms\android\app\src\main\java\com\synconset\ImageFetcher.java Error:(293, 58) error: Entry is not public in LinkedHashMap; cannot be accessed from outside package Error:Execution failed for task ':app:compileDebugJavaWithJavac'. Compilation failed; see the compiler error output for details.
Cordova 8
Gradle 3.0
defaultCompileSdkVersion=26
Any suggestions how to fix this problem?
Thank you very much for your help!

Same problem, any suggestions?

I have the same problem, did you find the solution?

+1

Hey guys,
I had the same problem while I was trying to find a solution for an other issue: https://github.com/wymsee/cordova-imagePicker/issues/263
The problem was on a variable mistake in ImageFetcher.java.
https://github.com/mkihellas/cordova-imagePicker/commit/61c18fc63fc9a5e69296715d76e38d043741dcd9#diff-76acd7e874d9233cd649120f24357319
We can use our repositery that has the fix: https://github.com/mkihellas/cordova-imagePicker.
Keep in mind that I also changed some things in plugin.xml in order to make it build successfully with CLI 8.0.0 on PhoneGap.

Thanks @mkihellas will give it a shot and let you know

+1

the plugin  no longer maintained ! I recently made a new plugin: https://github.com/dmcBig/cordova-plugin-mediaPicker

This bug was a showstopper for me. I ended up using to the camera plugin (https://github.com/apache/cordova-plugin-camera) instead. Just pass PictureSourceType: PHOTOLIBRARY, and it will open the photo library instead of the Camera.

In having to maintain an older ionic project, without changing much on my machine and coming back to the ionic app months later, I ended up having to do what @dansterrett suggests and used cordova-plugin-camera and it seems to work well and is probably the most actively maintained.
There were too many broken dependencies between mediaPicker and others to successfully build for me.  I suspect it's where I've updated cordova and ionic over time that the older ionic app and plugins began breaking all of a sudden.

the plugin no longer maintained ! I recently made a new plugin: https://github.com/DmcSDK/cordova-plugin-mediaPicker

Dear Sir,
Can you share with us the proper way to use your plugin (which looks really promising) with an ionic 3 project?
Best Regards.


add warning for sandbox upgradation

coverage: 93.548% (+0.004%) from 93.544%
when pulling 3fe7ae6e2f76438e5f20736a2165640909b419a0 on SushilMallRC:addWarning
into 7818d41e52ef3222d2cceb9c6b62589fe915c775 on ringcentral:master.


Add HttpPoster (#460)

See #460.

Flux example:

default infile = "{"criterias": [], "facettes": []}";
infile
|post-http(url="https://www.zoerr.de/edu-sharing/rest/search/v1/queriesV2/-home-/-default-/ngsearch?maxItems=10&skipCount=0&propertyFilter=-all-",contentType="application/json",accept="application/json")
| print
;

functional review: @TobiasNx

TODO: refactor to make an abstract class to share code with the metafacture-io/src/main/java/org/metafacture/io/HttpOpener.java.

Superseded by #463.


"Don't pass a nullable value type as an in argument."

Don't pass a nullable value type as an in argument. The Nullable type isn't declared as a read-only struct.  That means the compiler must generate defensive copies for any nullable value type argument passed to a method using the in modifier on the parameter declaration.

I think this guidance is a bit too strong.  Roslyn (v3.2 and later) does not make defensive copies when accessing some members of Nullable<T> even when targeting .NET Framework (where none of its members are readonly).

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: ac8804c7-6acd-ab2a-4e14-0b99265969a5
Version Independent ID: 183efcbd-a781-d303-f9cd-1c22da1676de
Content: Write safe and efficient C# code
Content Source: docs/csharp/write-safe-efficient-code.md
Product: dotnet-csharp
Technology: csharp-advanced-concepts
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

That's a good observation @carlreinke
This is an area that will get an update over the next few months. I'll address it then.


added json as implicit variable of type JsonBuilder to groovlets

groovlets have long had an implicit object of type MarkupBuilder called html, which is useful for writing out XML representations of objects. A similar object called json, of type groovy.json.JsonBuilder, was added to ServletBinding so that it is available in groovlets.
Unlike MarkupBuilder, JsonBuilder only serializes its output but does not write it to the output writer. Expected usage, therefore, will be to use the builder and then use "out << json" or "print json" to write the result to the response.
https://jira.codehaus.org/browse/GROOVY-5341

Ken, thanks for this contribution.
I have a couple remarks though:

Don't use tabs, use 4 spaces :-)
Secondly, I applied your pull request, ran the tests, but got caught by the fact that the ServletBinding java class depends on the JsonBuilder groovy class which is compiled after the other one. So with a non-clean build, one had the JsonBuilder class already compiled, so all was fine... but when running a full clean build, the JsonBuilder class isn't compiled and that yields some unknown symbol error. So the advice is to always run a full clean build before submitting :-)

Sorry about the tabs -- I used TextMate for the first time and didn't realize it was using tabs instead of spaces.
Thanks for fixing the loading problem. I ran the individual test only, unfortunately. Thanks for accepting my first contribution to the Groovy language, even if I did wind up breaking the build. :)

Cool! :-D
This might be a case where StreamingJsonBuilder would be a good match, as it would then mimic the behaviour of the existing MarkupBuilder by constructing it with:
StreamingJsonBuilder jsonBuilder = new StreamingJsonBuilder( output.getWriter() );

And you wouldn't need the extra out << json step.
Really handy addition though! :-)

Yes, I'm actually wondering if instead, we should be using the streaming one, which indeed allows us to get rid of the disgracious out << json step. That would be much nicer IMHO.

I've followed your suggestion, Tim :-)


sudo and brainy theme

The check for sudo in brainy theme link
triggers an email with security information, if sudo is set up to send emails on errors.
Also logs are cluttered with a
a password is required ; TTY=pts/1 ; PWD=/home/user/path; COMMAND=/usr/bin/id -u

I know that I can disable the check, but is there any other way to do it, without triggering sudo error checks?
Ty

There have been some previous discussions on this topic in #744, #792 and #902.
The gist is that I have not found a way to reliably have the sudo check functionality without actually using sudo. At the moment, disabling it looks like the only option I'm aware of. If you have another way of checking for these permissions without triggering an alert, I'm all ears...

Maybe this would be an option? https://stackoverflow.com/a/18216122/1228454
The EUID environment variable could potentially be used for this check instead of using sudo - someone would have to validate this.

Ok thanks, I will look if I can come up with something....

Just replacing
[ $(sudo -n id -u 2>&1 | grep 0) ]

with
[ $(id -u) = 0 ]

Ref is working for me

Just replacing
[ $(sudo -n id -u 2>&1 | grep 0) ]

with
[ $(id -u) = 0 ]

Ref, is working for me

Hi All,
The purpose of the check is not to confirm "Am I logged in as root", which is what [ $(id -u) = 0 ] will tell you.
The purpose of the check is to confirm "Do I have an active sudo session, whereby if I entered a sudo command it would not ask me for my password", which is what the original code is trying to test.
Wondering myself why checking for an active sudo session is so difficult, I did some googling, and the general consensus appears to be:
On a system that considers failed sudo attempts important enough to send emails, checking for active sessions is frowned upon, as it is a form of information leakage, letting a bad actor know that they could safely execute sudo commands.
Which I must say I totally agree with.  If the emails are important, then the check goes against your security concerns.   If the check (and prompt display) are important, then the email goes against your convenience concerns.
Hope that helps

Completely agree, thanks, @davidpfarrell for the details. At the moment, it looks like the currently used code does what it can do.
If you're not comfortable with the security escalation (emails etc.), please disable the sudo check as outlined in the documentation.

This issue is very stale, so I close it. Thanks for contributing and posting!


Soviet Pop

By the time the Soviet Union disolves , the pop of Belarussian, Estonian, Georgian, Lithuanian and some other nations cores is majorily Russian.

I will increase the likelihood of internal migration of minorities. This will make a more rapid population dilution in a nation, making the dominant pop (Russians) become more common everywhere.


Take password protection out of labs

refs #5073
This takes password protection out of labs and moves it to Settings\General.
per @JohnONolan's request I added a random words generator library that generates random passwords when privacy is first enabled.

I'm not sure it's worth it to use an external dependency and attach ~20KB worth of random words to the global namespace.
In my opinion, I'd rather pick 50 or 100 words and put the code in core/client/app/utils/.

fixed

:+1: for the 6 + 3 conbo as discussed in slack :) This seems to be working really nicely.


AAE-786: Jackson Databind polymorphic typing vulnerability (CVE-2019-16335)

AAE-786: Update spring boot to fix Jackson Databind polymorphic typing vulnerability (CVE-2019-16335)

Ref https://github.com/Activiti/Activiti/issues/2991


Update ECRRepository property

Issue #, if available:
Description of changes:
Received an status reason in Cloudformation creation for AWS::ECR::Repository saying template validation failed stating the sub-property of scanOnPush was non-permitted and deemed as an extraneous key.
The ImageScanningConfiguration property for the AWS::ECR::Repository resource appears to be ScanOnPush and was a typo.
Reference: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ecr-repository-imagescanningconfiguration.html
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the update! (and sorry for the delay getting to this - vacation...)


After restarting macOS, additionalDisks of the VM failed to mount (could not attach disk "test", in use by instance "default" )

Description
Steps to Reproduce

limactl disk create test --size 10GB
limactl start --name default --set='.additionalDisks[0] = "test"' template://ubuntu-lts
Restart the macOS operating system.
limactl start default

~ % limactl start default
INFO[0000] Using the existing instance "default"
INFO[0000] Attempting to download the nerdctl archive    arch=aarch64 digest="sha256:589dabd962d936b29fd377dcddbb49c07d1c4c27dd4b402bc4b6b20287fe9c37" location="https://github.com/containerd/nerdctl/releases/download/v1.4.0/nerdctl-full-1.4.0-linux-arm64.tar.gz"
INFO[0000] Using cache "/Users/jesse/Library/Caches/lima/download/by-url-sha256/8704652b907393aea54c8d98b48efa0fe9bef4d120fbbf8799bb5264631da634/data"
ERRO[0002] [hostagent] could not attach disk "test", in use by instance "default"
INFO[0002] [hostagent] Starting QEMU (hint: to watch the boot progress, see "/Users/jesse/.lima/default/serial.log")
FATA[0002] exiting, status={Running:false Degraded:false Exiting:true Errors:[] SSHLocalPort:0} (hint: see "/Users/jesse/.lima/default/ha.stderr.log")

Environment Information

Operating System: macOS 13.4.1
Lima Version: 0.16.0

limactl disk unlock DISKNAME may work

@AkihiroSuda Thanks, it works, but this issue is weird, the instance can't mount an additional disk that is occupied by itself, maybe we should keep this issue open until it is resolved.


How to use iframe in Framework7?

Hi, i want to ask about how to use  tag in Framework7. I have  code like
<iframe src="https://www.facebook.com/" style="width:100%; height:100%;">
But, the external page is not loaded, and i already add this line to the config.xml
<allow-navigation href="*" />
<allow-intent href="*" />
<access origin="*" />

also,
<meta http-equiv="Content-Security-Policy" content="default-src 'self' data: gap: https://ssl.gstatic.com 'unsafe-eval'; style-src 'self' 'unsafe-inline'; media-src *; img-src 'self' data: content:;">
in the index.html file. But the page is still not loaded.
Thanks before.

Facebook blocks being loaded in an iframe with X-Frame-Options, tl;dr it can't be done and this isn't F7 related


cd command does not work.

Executing this command
orphic locate into test directory
Returns command
cd test
and when I execute this through orphic it won't take me to the directory.

Commands are executed in a new process separate from the process of your shell environment, so cd will not take effect in your shell unfortunately. There are no workarounds to this (to the best of my knowledge).

Well, that was my guess as well. Thank you for the answer anyway 👍


Maybe need to restructure Neo4j address  argument for data  I/O scripts

Neo4j (default) uses two  TCP ports: 7474 and 7687  (for BOLT:).
KGX currently (as of March  8, 2019) accepts a single "--address"  parameter, which the  Makefile  set to boit://localhost:7687
This setting  breaks the  Neo4j tests.
Resetting the parameter to http://localhost:7474 helped run the download tests, but the upload test failed when kgx/neo_transformer.py looked for a bolt: session.
It is suggested that we keep  the TCP  ports out of the  "--address"  setting and simply use the Neo4j defaults internally, with perhaps  some override mechanism added to the system.
Then, we should  fix the tests.

#111 and #116 covers this case.
Makefile is edited so that configuration for the hostname is a separate parameter.
Supporting HTTP only as per #111.

@RichardBruskiewich Which tests did you have in mind?


[Feat] NBEATSx dropout

Added dropout to NBEATSx.
Since individual models benefit from dropout regularization, this is the most common use case.

Doesn't NBEATSx also rely on ensembling? I thought dropout lowered the performance of ensemble


HDFS cluster info is missing on PAI Dashbaord

PAI installed a HDFS cluster, but cluster info is not on the PAI Dashboard.
It can be access on the port 50070 with the same ip as PAI.

Maybe we don't want to bind on HDFS.
At least we need to provide some way to cooperate with storage system.

By design.


Sourcedata structure constrained or not?

The chapter about sourcedata vs rawdata is a bit confusing from my point of view: on the one hand the sourcedata folder MUST be BIDS compliant, but ALTERNATIVELY it may not, see:
https://bids-specification.readthedocs.io/en/stable/02-common-principles.html#source-vs-raw-vs-derived-data
It reads like "if the creatue is a dog it must bark but if it is an animal it may do anything" :-)

It must be BIDS-ish if the folder is located in the BIDS dataset, but if it's not then you can organize it however you want.
sourcedata in BIDS dataset:
my-dataset/  # BIDS, aka raw data
    sourcedata/  # must be BIDS-ish
        sub-X/
    sub-X/
    [etc]

sourcedata not in BIDS dataset:
my-dataset/
    rawdata/  # BIDS, aka raw data
        sub-X/
        [etc]
    sourcedata/  # no need for BIDS-ish structure

Thanks,  @tsalo, that makes it much clearer.


enable dragging

@majodu I'd say this is unlikely to get merged given inclusion of so many orthogonal formatting changes (whitespace, brackets, quotes).
You're also missing  d3-drag:d3from the rollup command.
@bumbeishvili are you interested in a minimal version of this PR that just adds drag-and-drop?

@beekhof  I have an idea how to implement drag drop and would not merge other PRs with similar functionality . I have implemented it already in other private project and it certainly was not as simple as I would have imagined and I faced many issues.
That said, if you want to give it a go, feel free and I can link that PR to relevant issues to that people can have a temporary solution

It took a bit of debugging, but PR #307 seemed to work for me

Had a quick look. Not sure how it will behave in the new v3 version, since root.descendants() behaves differently and it only returns visible nodes (some nodes are hidden because of paging )

@beekhof How can we use that functionality of drag and drop for PR? Is there any Stackblitz or codesandbox for that?
@bumbeishvili


Revert SingleThreadHandler::WriteDebugFile to patch version to suppress prompting for lk files

Now SingleThreadHandler (used in fcall_reopt_size_battery) does not prompt for lk debug script generation.
New class SingleThreadHandlerWithDebugOutput prompts for debug output and can be used if desired.
.Functionality not works as previous release.


clustering surreal DB with NATS KV

I was playing around with surreal clustering.
They support a few different way and tikv is the easiest for me. See : https://docs.surrealdb.com/docs/installation/running/tikv
I got it working easily...

# install tikv
curl -sSf https://tiup-mirrors.pingcap.com/install.sh | sh

# start it up ( in own terminal )
tiup playground --tag surrealdb --mode tikv-slim --pd 1 --kv 1

# start up surreal DB ( in own terminal ) using tikv://<IP_ADDRESS>:2379 protocol
surealdb start --auth --allow-all --user $(SURREAL_VAR_USERNAME) --pass $(SURREAL_VAR_USERPASS) --bind $(SURREAL_VAR_BIND) --log $(SURREAL_VAR_LOGLEVEL) tikv://<IP_ADDRESS>:2379

But the thought occurred to me how it might be possible to cluster Surreal DB using NATS KV.
I suspect that the SurrealDB team would need to support it though, but maybe not since Surreal DB supports Web sockets and so does NATS.
https://github.com/surrealdb/surrealdb/tree/main/lib/src/api/opt/endpoint has all the clustering endpoints. It looks pretty easy to add NATS using the nats rust library.

Yes, to allow SurrealDB to store data in NATS KV it's need to write implementation for SurrealDB. So it's better to open the issue in SurrealDB repository or even just write PR

Yes, to allow SurrealDB to store data in NATS KV it's need to write implementation for SurrealDB. So it's better to open the issue in SurrealDB repository or even just write PR

I will raise it there


{CI} Update pipeline to use Python 3.12

Use 3.12 in CI as https://github.com/Azure/azure-cli/pull/29887 is merged

CI


Zero downtime migrations?

Is it possible to adjust the mapping of the fields in the index with this package? In an older package we used scout-elasticsearch-driver#zero-downtime-migration which created a new index with the new mapping, imported the data and changed the used alias if everything went fine.
Thanks for your great work!
Robert

Hey @robigoal, I've decided not to include such feature out of the box, because it doesn't fit every project. Instead, I'll create a wiki page and describe how to create a zero-downtime migration. I'll work on it in the near future.

Hi @robigoal, as promised here this the guide. I hope it will be helpful for you 🙂


url linkage from error message page

there is an extra legion.stanford.edu/ in the url's pointing from the error message pages to the glossary of terms

also check for a missing .md suffix on one of the files in either glossary or design patterns

@aheirich are these still issues?  If so, are they things we can clean up in the near future?  Thanks.

I think this problem has been solved, but the online documentation still reflects the old behavior.   We just need to run doc/makeAllTheThings.bash and update the online repo.

I just ran the update and it's still broken. Assuming you have Jekyll and Doxygen installed, you can repro locally with:
git clone https://github.com/StanfordLegion/legion.stanford.edu.git jekyll
cd jekyll
make serve

To run the update you have to go into legion/doc and execute ./makeAllTheThings.bash
Then copy the publish/* results over into the legion.stanford.eduhttp://legion.stanford.edu repo
I’m trying to do that now
On Feb 5, 2020, at 2:35 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
I just ran the update and it's still broken. Assuming you have Jekyll and Doxygen installed, you can repro locally with:
git clone https://github.com/StanfordLegion/legion.stanford.edu.git jekyll
cd jekyll
make serve
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXWMO6MC37U7CV36HK3RBM5LJA5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5HVWI#issuecomment-582646489, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXQXFKQH5XRMY5AVVJTRBM5LJANCNFSM4D7ULBPA.

We run that line as part of the standard build: https://github.com/StanfordLegion/legion.stanford.edu/blob/master/Makefile#L56

Then I’m perplexed.  I just ran makeAllTheThings locally and it did not reproduce the error.
On Feb 5, 2020, at 2:39 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
We run that line as part of the standard build: https://github.com/StanfordLegion/legion.stanford.edu/blob/master/Makefile#L56
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXTVJ2JBYZUCUEG7E5TRBM5ZNA5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5H67I#issuecomment-582647677, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXTUH5PBCZLWPBWUL6TRBM5ZNANCNFSM4D7ULBPA.

I just did “make serve” and it did not regenerate the messages etc.  Maybe that’s the problem.
On Feb 5, 2020, at 2:41 PM, Heirich, Alan <aheirich<EMAIL_ADDRESS>wrote:
Then I’m perplexed.  I just ran makeAllTheThings locally and it did not reproduce the error.
On Feb 5, 2020, at 2:39 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
We run that line as part of the standard build: https://github.com/StanfordLegion/legion.stanford.edu/blob/master/Makefile#L56
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXTVJ2JBYZUCUEG7E5TRBM5ZNA5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5H67I#issuecomment-582647677, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXTUH5PBCZLWPBWUL6TRBM5ZNANCNFSM4D7ULBPA.

Ok, I think the only problem is that it was not doing “make messages” when it builds.  I just did that and verified the output is ok.
On Feb 5, 2020, at 2:46 PM, Heirich, Alan <aheirich<EMAIL_ADDRESS>wrote:
I just did “make serve” and it did not regenerate the messages etc.  Maybe that’s the problem.
On Feb 5, 2020, at 2:41 PM, Heirich, Alan <aheirich<EMAIL_ADDRESS>wrote:
Then I’m perplexed.  I just ran makeAllTheThings locally and it did not reproduce the error.
On Feb 5, 2020, at 2:39 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
We run that line as part of the standard build: https://github.com/StanfordLegion/legion.stanford.edu/blob/master/Makefile#L56
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXTVJ2JBYZUCUEG7E5TRBM5ZNA5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5H67I#issuecomment-582647677, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXTUH5PBCZLWPBWUL6TRBM5ZNANCNFSM4D7ULBPA.

That seems to be a flaw specific to make serve. I pushed a change so it should update those now.

So I've noticed a local build does actually work. So there's something more going on here.

Well this is interesting:

legion.cc line 1214

Notice one slash after https:. Maybe that cause different behavior in the browser when it's not local?
In collate_messages.py, I see the lines:
if args.prefix != None:
    args.prefix = args.prefix.replace("//", "/")

Is this the culprit?

Ok, seems to be fixed by removing those lines.

Ok, that line was there specifically to fix a problem where we were getting multiple //’s in the interior part of a URL.  This caused a problem of some kind that I don’t recall.  So I think the original need probably still exists, and we should fix this specifically for the case of // appearing after https:
On Feb 5, 2020, at 3:05 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
Ok, seems to be fixed by removing those lines.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXRKCK5PB6OSVFBVIT3RBNA4BA5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5J6PI#issuecomment-582655805, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXXRTU22SJTXGDFNR43RBNA4BANCNFSM4D7ULBPA.

Ok, but this is running specifically on args.prefix which comes into this script from the --prefix flag. Maybe this used to be local directory or something? It now seems to be hard-coded to Github:
	--prefix="https://github.com/StanfordLegion/legion/blob/${BLOB}/runtime" \

(inside makeAllTheThings.bash)

It may be that this used to get parameters from a script that was inconsistent about whether a path ends in / or not.
I agree with you that if we run it only this way then the fix is bad.
On Feb 5, 2020, at 3:11 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
Ok, but this is running specifically on args.prefix which comes into this script from the --prefix flag. Maybe this used to be local directory or something? It now seems to be hard-coded to Github:
    --prefix="https://github.com/StanfordLegion/legion/blob/${BLOB}/runtime" \

(inside makeAllTheThings.bash)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXUXCX6BPPNVRIPBLATRBNBSPA5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5KLSI#issuecomment-582657481, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXXBP44IXILN42DJ5DLRBNBSPANCNFSM4D7ULBPA.

I've temporarily deployed this fix to https://legion.stanford.edu, can you check to see if anything looks wrong?

Error 399 is right now
On Feb 5, 2020, at 3:33 PM, Elliott Slaughter <notifications<EMAIL_ADDRESS>wrote:
I've temporarily deployed this fix to https://legion.stanford.eduhttps://legion.stanford.edu/, can you check to see if anything looks wrong?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/StanfordLegion/legion/issues/320?email_source=notifications&email_token=AAORXXV3B4CPQQGFNTPVCV3RBNEC3A5CNFSM4D7ULBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5L5RQ#issuecomment-582663878, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORXXWLI4INHAVCZCGZ5DDRBNEC3ANCNFSM4D7ULBPA.

I don't want to close this issue until we have a concrete plan to update all the error messages here and keep them consistent with the runtime in a sane way.

@lightsighter What do you mean by this? The script is fully automatic and it runs every time we push the website. The underlying issue was actually one in the internal script to generate the files, and it has been fixed.
As far as I know the pages themselves are generated from parsing the source code, so there isn't really anything hard-coded into the website.
If you mean updating the error messages inside the C++ source code, I think that should be a separate issue.


关键词登陆不上钱包

Did you write down mnemonics correctly. Check the valid English phrase words here: https://github.com/bitcoin/bips/blob/master/bip-0039/english.txt


ui style fixes

removed the unnecessary horizontal scroll
button and footer style fix

This causes the following cropping in medium screensizes

The scaling of svgs is tricky when you want to apply style to svg path.
May be we can do your fix only in combination with a mediaquery matching medium screen size and do a scale down for the container like we do for mobile

following the approach used for mobile. updated the PR


Errors after setting up locally

I was seeing weird behavior, but managed to add a device. When trying to repeat the steps to document the issue, here is how it goes.
It seems that it starts correctly...
$ npm run start:local

><EMAIL_ADDRESS>start:local /home/user/smart-home-nodejs
> node dist/index.js isLocal

Smart home server listening at :::3000

COPY & PASTE NGROK URL BELOW
https://70432596.ngrok.io

=====
Visit the Actions on Google console at http://console.actions.google.com
Replace the webhook URL in the Actions section with:
    https://70432596.ngrok.io/smarthome

In the console, set the Authorization URL to:
    https://70432596.ngrok.io/fakeauth

Then set the Token URL to:
    https://70432596.ngrok.io/faketoken

Finally press the 'TEST DRAFT' button

Then I change the URL in index.html and start the front-end, point my browser to it, and this appears on the log:
{ userId: '1234', deviceId: '1rg', localDeviceId: null }
{ userId: '1234', deviceId: '1rg', localDeviceId: null }
{
  "error": {
    "code": 429,
    "message": "Resource has been exhausted (e.g. check quota).",
    "status": "RESOURCE_EXHAUSTED"
  }
}

POST /smarthome/update 400 7344.313 ms - 162
{
  "error": {
    "code": 404,
    "message": "Requested entity was not found.",
    "status": "NOT_FOUND"
  }
}

Have you linked your account to the Google Assistant?

I didn't even get to that point. I got stuck on steps 1-2 here:
Setup sample service
[...]

Open the web portal URL.
Configure the virtual devices shown as you please. Click the cloud icon shown above at least one of them to enable it for cloud control.

I'm trying to configure the virtual devices, but as soon as I open the web portal URL, that appears on the log.
I didn't even get to "Start Testing" or "Set up Account linking".

You will see a 404 error before you do account linking, as it will try to call HomeGraph APIs, which will fail.
The error is benign.

Same problem for me.
At last i found my test app on Google Home app!
Thank you so much for solution!!!!

Do you still have issue deploying the sample?

Do you still have issue deploying the sample?


imageNet.classify() encountered an error classifying the image

Hi @dusty-nv ,
I followed  Re-training on the Cat/Dog Dataset  and have no problems. In terminal, I run command line
imagenet --model=MODEL_PATH/resnet18.onnx --input_blob=input_0 --output_blob=output_0 --labels=LABEL_PATH/labels.txt IMG_PATH/dog/04.jpg dog.jpg
and it works normally.
According to my-recognition.py and Coding Your Own Image Recognition Program (Python), I write python code like below:
onnx = 'MODEL_PATH/resnet18.onnx'
labels = 'Label_PATH/labels.txt'
argv = ['--model={}'.format(onnx), '--labels={}'.format(labels), '--input-blob=input_0', '--output_blob=output_0']
model_cls = jetson.inference.imageNet('resnet-18', argv)

color_img = 'IMG_PATH/dog/04.jpg'
cuda_img = jetson.utils.loadImage(color_img)

class_id, confidence = model_cls.Classify(cuda_img)

The [TRT] output in terminal indicates it loads model and label successfully, but finally it throws out error:
Traceback (most recent call last):
  File "my_code.py", line 101, in <module>
    class_id, confidence = model_cls.Classify(cuda_img)
Exception: jetson.inference -- imageNet.Classify() encountered an error classifying the image

Please, what may be the problem and how to handle it?
Or, what's the suggested way to load a imageNet with named network type and model path?

Hi @surefyyq, try it like this:
model_cls = jetson.inference.imageNet(argv=['--model=MODEL_PATH/resnet18.onnx', '--labels=Label_PATH/labels.txt', '--input-blob=input_0', '--output-blob=output_0'])

Thanks for your reply @dusty-nv ! But it still ouputs error like below:
[image] loaded 'IMG_PATH/dog/04.jpg'  (500x375, 3 channels)
cuda_img:  (375, 500, 3) <class 'jetson.utils.cudaImage'> <cudaImage object>
   -- ptr:      0x101070000
   -- size:     562500
   -- width:    500
   -- height:   375
   -- channels: 3
   -- format:   rgb8
   -- mapped:   true
   -- freeOnDelete: true

classification argv:  ['--model=MODEL_PATH/resnet18.onnx', '--labels=LABEL_PATH/labels.txt', '--input-blob=input_0', '--output_blob=output_0'] <class 'list'>
jetson.inference -- imageNet loading network using argv command line params

imageNet -- loading classification network model from:
         -- prototxt     (null)
         -- model        MODEL_PATH/resnet18.onnx
         -- class_labels LABEL_PATH/labels.txt
         -- input_blob   'input_0'
         -- output_blob  'output_0'
         -- batch_size   1

[TRT]    TensorRT version 7.1.3
[TRT]    loading NVIDIA plugins...
[TRT]    Registered plugin creator - ::GridAnchor_TRT version 1
[TRT]    Registered plugin creator - ::NMS_TRT version 1
[TRT]    Registered plugin creator - ::Reorg_TRT version 1
[TRT]    Registered plugin creator - ::Region_TRT version 1
[TRT]    Registered plugin creator - ::Clip_TRT version 1
[TRT]    Registered plugin creator - ::LReLU_TRT version 1
[TRT]    Registered plugin creator - ::PriorBox_TRT version 1
[TRT]    Registered plugin creator - ::Normalize_TRT version 1
[TRT]    Registered plugin creator - ::RPROI_TRT version 1
[TRT]    Registered plugin creator - ::BatchedNMS_TRT version 1
[TRT]    Could not register plugin creator -  ::FlattenConcat_TRT version 1
[TRT]    Registered plugin creator - ::CropAndResize version 1
[TRT]    Registered plugin creator - ::DetectionLayer_TRT version 1
[TRT]    Registered plugin creator - ::Proposal version 1
[TRT]    Registered plugin creator - ::ProposalLayer_TRT version 1
[TRT]    Registered plugin creator - ::PyramidROIAlign_TRT version 1
[TRT]    Registered plugin creator - ::ResizeNearest_TRT version 1
[TRT]    Registered plugin creator - ::Split version 1
[TRT]    Registered plugin creator - ::SpecialSlice_TRT version 1
[TRT]    Registered plugin creator - ::InstanceNormalization_TRT version 1
[TRT]    detected model format - ONNX  (extension '.onnx')
[TRT]    desired precision specified for GPU: FASTEST
[TRT]    requested fasted precision for device GPU without providing valid calibrator, disabling INT8
[TRT]    native precisions detected for GPU:  FP32, FP16
[TRT]    selecting fastest native precision for GPU:  FP16
[TRT]    attempting to open engine cache file /home/nvidia/jetson-inference/python/training/classification/models/cat_dog/resnet18.onnx.1.1.7103.GPU.FP16.engine
[TRT]    loading network plan from engine cache... /home/nvidia/jetson-inference/python/training/classification/models/cat_dog/resnet18.onnx.1.1.7103.GPU.FP16.engine
[TRT]    device GPU, loaded /home/nvidia/jetson-inference/python/training/classification/models/cat_dog/resnet18.onnx
[TRT]    Deserialize required 3191513 microseconds.
[TRT]
[TRT]    CUDA engine context initialized on device GPU:
[TRT]       -- layers       29
[TRT]       -- maxBatchSize 1
[TRT]       -- workspace    0
[TRT]       -- deviceMemory 29827072
[TRT]       -- bindings     2
[TRT]       binding 0
                -- index   0
                -- name    'input_0'
                -- type    FP32
                -- in/out  INPUT
                -- # dims  4
                -- dim #0  1 (SPATIAL)
                -- dim #1  3 (SPATIAL)
                -- dim #2  224 (SPATIAL)
                -- dim #3  224 (SPATIAL)
[TRT]       binding 1
                -- index   1
                -- name    'output_0'
                -- type    FP32
                -- in/out  OUTPUT
                -- # dims  2
                -- dim #0  1 (SPATIAL)
                -- dim #1  2 (SPATIAL)
[TRT]
[TRT]    binding to input 0 input_0  binding index:  0
[TRT]    binding to input 0 input_0  dims (b=1 c=3 h=224 w=224) size=602112
[TRT]    binding to output 0 output_0  binding index:  1
[TRT]    binding to output 0 output_0  dims (b=1 c=2 h=1 w=1) size=8
[TRT]
[TRT]    device GPU, MODEL_PATH/resnet18.onnx initialized.
[TRT]    imageNet -- loaded 2 class info entries
[TRT]    imageNet -- MODEL_PATH/resnet18.onnx initialized.
model_cls:  <jetson.inference.imageNet object at 0x7f867548d0>

Traceback (most recent call last):
  File "my_code.py", line 98, in <module>
    class_id, confidence = model_cls.Classify(cuda_img)
Exception: jetson.inference -- imageNet.Classify() encountered an error classifying the image

Hmm I am not sure - since imagenet.py is very similar, can you make a similar modification to it or copy it's source for your needs?

Many appreciation for your quick answer @dusty-nv !
I don't make any modification , just copy imagenet.py source code to my custom file folder and rename it as my_imagenet.py.
At this custom file path, when I run command line in terminal:
imagenet --model=MODEL_PATH/resnet18.onnx --input_blob=input_0 --output_blob=output_0 --labels=LABEL_PATH/labels.txt IMG_PATH/dog/04.jpg cls_test.jpg
It ends without error and the cls_test.jpg saved successfully.
But when I run this in terminal:
python my_imagenet.py --model=MODEL_PATH/resnet18.onnx --input_blob=input_0 --output_blob=output_0 --labels=LABEL_PATH/labels.txt IMG_PATH/dog/04.jpg cls_test.jpg
It still goes error:
Traceback (most recent call last):
  File "my_imagnet.py", line 47, in <module>
    class_id, confidence = model_cls.Classify(img)
Exception: jetson.inference -- imageNet.Classify() encountered an error classifying the image

I'm sooooo confused. How could this happen?

Do you get the same problem running imagenet.py without any modification?

Yes...
Just for sure, the  imagenet.py is at jetson-inference/python/examples/imagenet.py, right?
And I run:
python imagenet.py --model=MODEL_PATH/resnet18.onnx --input_blob=input_0 --output_blob=output_0 --labels=LABEL_PATH/labels.txt IMG_PATH/dog/04.jpg cls_test.jpg
Still the same problem.
Should I try its default setting which uses googlenet?

Yes, next please try imagenet.py with the default googlenet instead of your custom model - do you get the same error?

From: sure @.>
Sent: Wednesday, July 14, 2021 11:26:43 PM
To: dusty-nv/jetson-inference @.>
Cc: Dustin Franklin @.>; Mention @.>
Subject: Re: [dusty-nv/jetson-inference] imageNet.classify() encountered an error classifying the image (#1148)
Yes...
Just for sure, the imagenet.py is at jetson-inference/python/examples/imagenet.py, right?
And I run:
python imagenet.py --model=MODEL_PATH/resnet18.onnx --input_blob=input_0 --output_blob=output_0 --labels=LABEL_PATH/labels.txt IMG_PATH/dog/04.jpg cls_test.jpg
Still the same problem.
Should I try its default setting which uses googlenet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-inference/issues/1148#issuecomment-880363171, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVEGK4OGHSZH3S5AP6OWLLTXZIPHANCNFSM5AKT2RNA.

Hi @dusty-nv , sorry for the late.
I try the imagenet.py with the default googlenet and it works fine, the camera onboard is called and [TRT] reports Timing.
Is that means something is wrong with my custom model? But I trained it following  the Re-training on the Cat/Dog Dataset and works as excepted when used by command line.
I will train the custom model again and try it after, wish me luck.

@dusty-nv good news! It works all right after I retrain the model...
Still don't know why, but thanks very much for your time and attention!


Banned.video Causing 403 Forbidden on HEAD Request

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.09.20
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

(base) X:\World Events\2020 Civil War>youtube-dl --verbose https://banned.video/watch?id=5f6e6aecd1e7550a1c00b9ed
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://banned.video/watch?id=5f6e6aecd1e7550a1c00b9ed']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2020.09.20
[debug] Python version 3.6.8 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: ffmpeg git-2020-07-13-7772666, ffprobe git-2020-07-13-7772666, phantomjs 2.1.1
[debug] Proxy map: {}
[generic] watch?id=5f6e6aecd1e7550a1c00b9ed: Requesting header
WARNING: Could not send HEAD request to https://banned.video/watch?id=5f6e6aecd1e7550a1c00b9ed: HTTP Error 403: Forbidden
[generic] watch?id=5f6e6aecd1e7550a1c00b9ed: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by <HTTPError 403: 'Forbidden'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\extractor\common.py", line 632, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "c:\programdata\anaconda3\lib\site-packages\youtube_dl\YoutubeDL.py", line 2238, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "c:\programdata\anaconda3\lib\urllib\request.py", line 532, in open
    response = meth(req, response)
  File "c:\programdata\anaconda3\lib\urllib\request.py", line 642, in http_response
    'http', request, response, code, msg, hdrs)
  File "c:\programdata\anaconda3\lib\urllib\request.py", line 570, in error
    return self._call_chain(*args)
  File "c:\programdata\anaconda3\lib\urllib\request.py", line 504, in _call_chain
    result = func(*args)
  File "c:\programdata\anaconda3\lib\urllib\request.py", line 650, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)```

## Description

<!--
Provide an explanation of your issue in an arbitrary form. Provide any additional information, suggested solution and as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

I seem to recall Banned.video working flawlessly before, but maybe a change on their service side is now blocking HEAD requests for some reason due to increased IT security attacks being received given 2020.

Their website DOES have a download button, so perhaps this isn't strictly urgent, but in an automated download pipeline scenario, programatically ingesting a URL and outputting an MP4 is ideal.

@eeik I can reproduce. It works if I use http instead of https though.

Is this the command you're thinking? If so, it's not working for me. Is there an alt parameter I should be using?
(base) X:\World Events\2020 Civil War>youtube-dl -f '[protocol=http] --verbose https://banned.video/watch?id=5f6e6aecd1e7550a1c00b9ed

No, I mean changing https://banned.video/watch?id=5f6e6aecd1e7550a1c00b9ed to http://banned.video/watch?id=5f6e6aecd1e7550a1c00b9ed.

🤦♂ ... skipped right over the easy solution! Thanks @Beorn-hayek 👍

Hmm, it now seems neither http nor https work properly. Perhaps they've made a server side change that didn't initially propagate on both protocols?

See https://github.com/ytdl-org/youtube-dl/issues/26805


Backticks vs double quotation marks in SQL files

In some files, backticks are used for table and column names (e.g., Action.sql) while double quotation marks are used in other (e.g., Actor.sql).
Can we standardize them to double quotation marks?

It's gotta be double quotation marks. The backticks come from LucidCharts.


[19.09] dovecot: <IP_ADDRESS> -> 2.3.8

Motivation for this change
backport of https://github.com/NixOS/nixpkgs/pull/70862
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @fpletz @peti

@GrahamcOfBorg test dovecot


error: fastBPE/fastBPE.cpp: no such file or directory

Hi,
Not sure the reason I'm having problems installing fastBPE following your exact steps with google compute engine.
I already tried it on paperspace notebook (v100) and fastBPE installed fine as far as I recall (however the interactive console didn't load, unsure why).
Anyway, I will google about "error: fastBPE/fastBPE.cpp: no such file or directory"

Sorry, it was some python 2 vs python 3. pip vs pip3 install issue.
cd fastBPE
sudo python3 setup.py install
I'll work it out.

Change the line in the .sh
from: pip install Cython
to: sudo python -m pip install Cython

Actually even after that, I still got the same error that I was already experiencing through my paperspace setup.

https://github.com/tensorflow/addons
https://github.com/tensorflow/io (for I/O related ops)
If you depend on functionality not listed there, please file an issue.

WARNING:tensorflow:From generation.py:132: The name tf.train.AdagradOptimizer is deprecated. Please use tf.compat.v1.train.AdagradOptimizer instead.
Model: "model"

Layer (type)                    Output Shape         Param #     Connected to
input_1 (InputLayer)            [(None, 256)]        0

tied_embedding_softmax (TiedEmb multiple             315810054   input_1[0][0]
encoder[0][0]

encoder (Encoder)               (None, 256, 1280)<PHONE_NUMBER>  tied_embedding_softmax[0][0]
Total params: 1,637,964,550
Trainable params: 1,637,964,550
Non-trainable params: 0

None
WARNING:tensorflow:You are creating an Estimator from a Keras model manually subclassed from Model, that was already called on some inputs (and thus already had weights). We are currently unable to preserve the model's state (its weights) as part of the estimator in this case. Be warned that the estimator has been created using a freshly initialized version of your model.
Note that this doesn't affect the state of the model instance you passed as keras_model argument.
WARNING:tensorflow:Estimator's model_fn (<function model_fn at 0x7fc65a8fe938>) includes params argument, but params are not passed to Estimator.
WARNING:tensorflow:From generation.py:156: The name tf.placeholder is deprecated. Please use tf.compat.v1.placeholder instead.
Traceback (most recent call last):
File "generation.py", line 158, in
predict_fn = tf.contrib.predictor.from_estimator(estimator_model, serving_input_fn)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/contrib/predictor/predictor_factories.py", line 105, in from_estimator
config=config)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/contrib/predictor/core_estimator_predictor.py", line 72, in init
serving_input_receiver, estimator, output_key)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/contrib/predictor/core_estimator_predictor.py", line 37, in _get_signature_def
estimator.config)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 250, in public_model_fn
return self._call_model_fn(features, labels, mode, config)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1149, in _call_model_fn
model_fn_results = self._model_fn(features=features, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/keras.py", line 288, in model_fn
optimizer_config=optimizer_config)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/keras.py", line 227, in _clone_and_build_model
optimizer_config=optimizer_config)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/keras/models.py", line 621, in clone_and_build_model
clone = clone_model(model, input_tensors=input_tensors)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/keras/models.py", line 409, in clone_model
model, input_tensors=input_tensors, layer_fn=clone_function)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/keras/models.py", line 210, in _clone_functional_model
tensor_map)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/keras/models.py", line 97, in _make_new_nodes
new_layer = layer_fn(layer)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/keras/models.py", line 57, in _clone_layer
return layer.class.from_config(layer.get_config())
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/keras/engine/base_layer.py", line 580, in get_config
raise NotImplementedError('Layers with arguments in __init__ must '
NotImplementedError: Layers with arguments in __init__ must override get_config.

Change the line in the .sh from: pip install Cython to: sudo python -m pip install Cython

this works after I installed Cython


API Exposure Requirements - Docker Socket Proxy

Question:
If I wanted to place Portainer's access to the docker socket behind Tecnativa/docker-socket-proxy, how would I go about doing this? What API access does Portainer require?
I have already secured access to the socket for Traefik (v1.7) and Netdata. Here is the relevant compose, including my Portainer, pre-modification for proxy.
Original
  socket-proxy:
    image: tecnativa/docker-socket-proxy
    container_name: socket-proxy
    restart: always
    networks:
      - socket-proxy
    ports:
      - 2375:2375
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock:ro
    environment:
      - CONTAINERS=1

  traefik:
    image: traefik:1.7.21
    container_name: traefik
    restart: always
    networks:
      - web
      - socket-proxy
    depends_on:
      - socket-proxy
    ports:
      - "80:80"
      - "443:443"
    expose:
      - "8080"
    volumes:
      - ${DOCKERDIR}/traefik/traefik.toml:/traefik.toml
      - ${DOCKERDIR}/traefik/acme.json:/acme.json
    environment:
      - CF_API_EMAIL
      - CF_API_KEY
    labels:
      - "traefik.docker.network=web"
      - "traefik.enable=true"
      - "traefik.frontend.rule=Host:traefik.${DOMAINNAME}"
      - "traefik.backend=traefik"
      - "traefik.port=8080"
      - "traefik.frontend.auth.basic.users=${HTTP_USERNAME}:${HTTP_PASSWORD}"

  portainer:
    image: portainer/portainer
    container_name: portainer
    restart: unless-stopped
    command: -H unix:///var/run/docker.sock
    networks:
      - web
    depends_on:
      - traefik
    expose:
      - "9000"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ${DOCKERDIR}/portainer:/data
    environment:
      - TZ
    labels:
      - "traefik.docker.network=web"
      - "traefik.enable=true"
      - "traefik.frontend.rule=Host:port.${DOMAINNAME}"
      - "traefik.backend=portainer"
      - "traefik.port=9000"

Here is my modified block for Portainer, which currently does not allow me to connect to the socket. I was unsure what the 'command' line did above.
Modified
  portainer:
    image: portainer/portainer
    container_name: portainer
    restart: unless-stopped
    #command: -H unix:///var/run/docker.sock
    networks:
      - web
      - socket-proxy
    depends_on:
      - traefik
    expose:
      - "9000"
    volumes:
      #- /var/run/docker.sock:/var/run/docker.sock
      - ${DOCKERDIR}/portainer:/data
    environment:
      - TZ
      - DOCKER_HOST=socket-proxy:2375
    labels:
      - "traefik.docker.network=web"
      - "traefik.enable=true"
      - "traefik.frontend.rule=Host:port.${DOMAINNAME}"
      - "traefik.backend=portainer"
      - "traefik.port=9000"

By default, the socket image grants access to ping, events, and version. All other access, such as "Containers" which I have enabled for Netdata and Traefik, require manual enablement. I'm not sure how to first point Portainer at the tcp proxy properly, and then allow all access it needs. Thanks!

AUTH
SECRETS
POST
BUILD
COMMIT
CONFIGS
CONTAINERS
DISTRIBUTION
EXEC
IMAGES
INFO
NETWORKS
NODES
PLUGINS
SERVICES
SESSION
SWARM
SYSTEM
TASKS
VOLUMES

Portainer is meant to manage all the features available inside a Docker environment. If you are planning to make use of all the features available in Portainer, I would allow everything.
Feel free to comment if something's missing.

Thanks @deviantony. I am likely not utilizing Portainer to anywhere near its full capabilities - typically I just view logs, check on existing containers/images/volumes, and sometimes clean up unused volumes and containers.
I've exposed the following and, while I cannot remove volumes right now, it gives me a mainly read only view of everything I personally care about without exposing POST, EXEC, or other capabilities to an internet facing web app. In the unlikely event of some kind of breach of my container security, it's less risk for me.
  portainer-proxy:
    image: tecnativa/docker-socket-proxy
    container_name: portainer-proxy
    restart: always
    networks:
      - socket-proxy
    ports:
      - 2375:2375
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock:ro
    environment:
      - CONTAINERS=1
      - INFO=1
      - IMAGES=1
      - VOLUMES=1
      - NETWORKS=1

  portainer:
    image: portainer/portainer
    container_name: portainer
    restart: unless-stopped
    networks:
      - web
      - socket-proxy
    depends_on:
      - traefik
      - portainer-proxy
    expose:
      - "9000"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ${DOCKERDIR}/portainer:/data
    environment:
      - TZ
      - DOCKER_HOST=socket-proxy:2375
    labels:
      - "traefik.docker.network=web"
      - "traefik.enable=true"
      - "traefik.frontend.rule=Host:port.${DOMAINNAME}"
      - "traefik.backend=portainer"
      - "traefik.port=9000"

I'd like to solve the volume removal issue, but I'll continue to exec into portals via CLI.
Thank you!

Hi @undaunt , I am looking to achieve the same thing. But I don't understand how your last config works without DOCKER_HOST=socket-proxy:2375. How do you tell the Portainer container to use tecnativa/docker-socket-proxy without it ? I tried with it and it wasn't working, Portainer couldn't connect to the docker socket.

Hey @LeVraiRoiDHyrule
I've learned a few things since I posted that config, but what I added was the command flag to not have to specify where Portainer connects to. Since Portainer and the proxy are both on the same network (the socket proxy network) they can communicate that way, and Command specifies the primary host connection being the socket proxy.
  portainer:
    image: portainer/portainer-ee:latest
    container_name: portainer
    profiles: ["all","port","morty"]
    networks:
      - $PROXY_NETWORK
      - $SOCKET_NETWORK
    security_opt:
      - no-new-privileges:true
    command: -H tcp://socket-proxy:$SOCKET_PORT
    volumes:
      - $APPDATA_MORTY/portainer/config:/data
    environment:
      - TZ


Allow coverlet integration with other MSBuild test strategies

This change exposes the InstrumentModules and GenerateCoverageResult targets which execute unconditionally, and can then be tied into build steps using conditional logic. The previous logic for supporting the VSTest target is retained through conditional targets.

@sharwell where do you need it?

@MarcoRossignoli I had to duplicate most of the targets file in dotnet/roslyn-analyzers#3028. With this change, I'll be able to just duplicate the InstrumentModulesNoBuild and GenerateCoverageResultAfterTest targets.

Understood...I don't have idea of release at the moment we're discussing off line...if you want to use this you can dogfood it tomorrow https://github.com/tonerdo/coverlet/blob/master/Documentation/ConsumeNightlyBuild.md

Here the unreleased feat https://github.com/tonerdo/coverlet/blob/master/Documentation/Changelog.md

@MarcoRossignoli Thanks. I'll test out the nightly but there's no rush on the release itself.


acpid: 2.0.29 -> 2.0.30

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/acpid/versions.
meta.description for acpid is: '"A daemon for delivering ACPI events to userspace programs"'.

Checks done (click to expand)

built on NixOS
/nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/bin/acpi_listen passed the binary check.
/nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/bin/acpid passed the binary check.
Warning: no invocation of /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/bin/kacpimon had a zero exit code or showed the expected version
2 of 3 passed binary check by having a zero exit code.
2 of 3 passed binary check by having the new version present in output.
found 2.0.30 with grep in /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30
directory tree listing: https://gist.github.com/9a6064536f0d2a1dc0f6bebe16b24a2a
du listing: https://gist.github.com/d0165f4766121872a87cb61128c50a58

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 7225c2acd063709c85a9a5e9924a5ddea6cc4b65 && nix-shell --pure -I nixpkgs=. -p acpid

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: acpid
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: acpid
Partial log (click to expand)
shrinking /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/bin/acpi_listen
shrinking /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/sbin/acpid
shrinking /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/sbin/kacpimon
gzipping man pages under /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/share/man/
strip is /nix/store/90vmpr41dzsx350k5argycaf693hnl1l-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/bin  /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/sbin
patching script interpreter paths in /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30
checking for references to /build in /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30...
moving /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/sbin/* to /nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30/bin
/nix/store/jjl60m8jhn884bxg60nxpzihry13sq63-acpid-2.0.30

Success on aarch64-linux (full log)
Attempted: acpid
Partial log (click to expand)
shrinking /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/bin/acpi_listen
shrinking /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/sbin/acpid
shrinking /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/sbin/kacpimon
gzipping man pages under /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/share/man/
strip is /nix/store/7iyn7gn33i7xxjgmwf25k20246y6nd9d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/bin  /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/sbin
patching script interpreter paths in /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30
checking for references to /build in /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30...
moving /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/sbin/* to /nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30/bin
/nix/store/wsspggaakysr4r9nfwjylsqizf3j7gxd-acpid-2.0.30


Client Allocation Request for: Equinix

Client Allocation Questions
Core Information

Name: Equinix
Website / Social Media: https://new.bizhost.co.kr/
Region: Asia excl. Greater China
DataCap Requested: 10TiB
Addresses to be Notarized: f3uhhh4i56zmqb66klgsltwsxfp2nyfhzdsj5m346tzqhp72aapebj6nwsvkfu7ocvohwtmpgzjoh6ijtuaiuq
Notary Requested: IreneYoung

@1equinix Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Hi @1equinix  - Thanks for your application. Please answer the following questions to let us know about you, your company, and further information about the DataCap allocation.

Basic Info

What's your name and your position in your company?

Please introduce yourself and your team briefly. How many team members do you have in your R&D or core team? Could you provide several Github accounts or email addresses of them?

For your organization, Please describe its business, its history since inception, registration country, scale, employee counts, operation regions, named executive officers, relevant projects, industry reputation, etc.

Please show us some of your web pages, Github, social media accounts such as Twitter, Facebook, Naver blog, WeChat official account, etc. Please include as many relevant details to provide contextual information (with external links) to show your Web3 reputation or external reputation.

Please provide your company's business registration license. If you have any presentation documents about your business, please provide them to us. And also could send me via email for privacy concerns. My contact<EMAIL_ADDRESS>

How long have the team core members been engaged in development?

What types of data do you plan to store with the DataCap and how many do you need? Could you please send us some of your sample data?

Regarding the data you intend to store, What’s the original source of those data, and do you have any proof of the ownership or authorization?

Where is your data stored now? Whether the data is stored on a server or a cloud service provider?

Track Record

Has your organization obtained DataCap before? if so, please answer the following sub-questions.
1.1 Please share all the links and recordings of your applications before.
1.2 How about your DataCap usage? Is there any remaining? Is there any information on the deals you've done and the miners you've allocated your DataCap to?
1.3 What was your allocation strategy plan then and how did you execute it?

Are there any disputes open against you from your previous DataCap allocations? If any, please describe any of your honors or disputes in the field of Filecoin.

Allocation Plan

Please describe your plan to use the DataCap as detailed as possible, including use case, amount, timespan, geography, storage providers, partners, etc.

How many regions do you plan to store the data in?

How many storage providers you intend to make dealing with?

Do you plan to record your allocation decisions and execution details in public? And how to realize it?

Do you agree to use the DataCap to only store data that abide by local regulations and is in compliance with the recipient miner’s terms of service?

We may further ask some questions and then give an accurate score to the application. Thank you for your cooperation!


Bug: App auth path matching not work for Enterprise

          This change breaks github enterprise, because urls begin with /api/v3, this regex no longer matches for the /app endpoint

Originally posted by @brainiac in https://github.com/yanyongyu/githubkit/issues/42#issuecomment-1760678551

octokit matches the path by replacing the base url first:
https://github.com/octokit/auth-app.js/blob/c0068e06081a5d930799285a7c79c9c948b676b6/src/hook.ts#L43


Feature Request: add upload remaining time indicator

Hello!
This is the same issue I posted on competitions v1.5, but I see it's the same for the new server at codalab.lisn.upsaclay.fr
It would be helpful to add a progress bar or spinning wheel for bundle uploads, so the user/organizer knows things are progressing.
Using the public instance, uploading both competition bundles and submission bundles take longer than expected, and it's not obvious when it's a connection issue.

The is a progress bar in Codabench (codabench.org).
codalab.lisn.fr is running CodaLab Competitions.


In mppt.get_solardisplay_info solar CSV filename and path is hardcoded

At the very least this the path should be an argument to this function and the specific path given by the code that uses the function.

Essentially the same issue exists in mppt.log_solar_info in the same module.  Probably makes sense to fix both as part of the same issue.  Seems that the dencam main program itself should get this file path from the config file.

To note: part of why this hardcoded path is non-ideal is that assumes that the code is being run from the pi user.  This is very often true (and is likely true on all the field devices in Svalbard) but is not a good assumption and notably on the test unit I am using that is not how things are set up.

#73 is a related issue


[Change Request] page 183 "Reverse Transaction Entries" - procedure Post

Hello, can you please change the procedure post as a global procedure ?
    procedure Post(PrintRegister: Boolean)
    var
        ReversalPost: Codeunit "Reversal-Post";
    begin
        ReversalPost.SetPrint(PrintRegister);
        ReversalPost.Run(Rec);
        CurrPage.Update(false);
        CurrPage.Close();
    end;

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. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 11082.


Drag'n'Drop views in InspectorView not working

Describe the bug
I can't drag'n'drop the views in the inspector in order to change their ordering.
To Reproduce
Steps to reproduce the behavior:

Add multiple views to the inspector
Try changing the order with drag'n'drop

Expected behavior
I would expect to be able to change the order of the views
Desktop (please complete the following information):

OS: macos
Browser: chrome
Browser Version: 114.0.5735.90 (Official Build) (arm64)
Spotlight Version 1.0.0

Additional context
react-beautiful-dnd prints the following debug error message:
Unable to find draggable with id: 0e6631f7-1128-4558-9966-8e9175969c9

when i inspect the Elements, I can find the draggable with the searched id:

  <button class="Button__StyledHTMLButton-sc-1xm6gdl-0 jqfzFA">
    <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke="currentColor" class="X__StyledSvg-sc-1dqpkgl-0 PyruS">
      <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M6 18L18 6M6 6l12 12">
      </path>
    </svg>
  </button>

        less_categories

Drag and drop is not supported at the moment and the feature was removed for now.


Add  methods for children testing in mock_data_snapshot.dart

Codecov Report

Merging #14 (bab8353) into main (172d87a) will decrease coverage by 15.82%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main      #14       +/-   ##
===========================================
- Coverage   96.87%   81.04%   -15.83%
===========================================
  Files           5        5
  Lines         128      153       +25
===========================================
  Hits          124      124
- Misses          4       29       +25

Impacted Files
Coverage Δ

lib/src/mock_data_snapshot.dart
26.47% <0.00%> (-73.53%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hey @bonnybun, thanks a lot for the PR 👍.
Can you please write unit tests for the new methods?
If you want I will write them my self later and push my commit to this PR before merging it.

Hi @sitatec,
yes I will write unit tests today.


Rebase update-datastore branch onto master

Rebasing to keep the update-datastore branch up to date and to make future PRs more concise.

Looks like I need to add people to this PR to make googlebot happy. cc/ @aozarov @mziccard

Closing this because I think rebasing now could make merging this branch into the master more painful in the future.


Add option for disabling transactions

Every once in awhile I go into https://github.com/jeanfortheweb/typeorm-fixture-builder/blob/master/src/install.ts#L17 and turn off transactions.
It helps when I don't want to wipe the database in between runs. Usually its because I am developing and messed up some of the data.
I can PR it if you are interested. Not sure if your continuing to dev on this.

Sorry for not seeing your issue dustin!
Sure, handling of transactions could be a thing. Guess they should still run by default? Or the opposite by using --transactional?
Could be confusing for existing users who expect transactions to run

Hey dustin, still interested on figuring out a good update for this? Can you show an example on how you solved it? I would like to make it meaningful for anyone.


add link to the README from the docs

The lack of detailed explanations on the starting page in the docs was noted at https://users.rust-lang.org/t/lets-talk-about-ecosystem-documentation/2791/25

There were the following issues with your Pull Request

Commit: 310db21d6460975f415c1334401048cd3d4bf044
Your commit message body contains a line that is longer than 72 characters
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/serde-rs/serde

This message was auto-generated by https://gitcop.com

Thanks!


[HOLD for payment 2023-07-05] [$1000] Pressing and holding back key in a phone number field resets country selection

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on “Profile” avatar
Click on “Workspace” link
Click on “Issue cards” link
Click on “Connect manually” link
Fill out all forms and submit the forms leading to “Personal Information” form
Enter personal information and click on “save & continue” button
Click on “Passport” button
Click on “Get secure link” button
Click on “Get link via SMS” link
Select a country using dropdown menu
Enter number
Press and hold back key

Expected Result:
Country selection should not reset
Actual Result:
Country selection resets to the default value
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.91-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/228334502-790fae13-ab86-4b9e-8385-d4562e314787.mp4
https://user-images.githubusercontent.com/43996225/228334524-8fa1746f-8b69-4ad6-a586-d47deceb4348.mp4
Expensify/Expensify Issue URL:
Issue reported by: @Natnael-Guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1680004375167929
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016fa3b4a81adeeec1
Upwork Job ID:<PHONE_NUMBER>223389184
Last Price Increase: 2023-04-19

Triggered auto assignment to @michaelhaxhiu (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@michaelhaxhiu Eep! 4 days overdue now. Issues have feelings too...

@michaelhaxhiu Still overdue 6 days?! Let's take care of this!

Job added to Upwork: https://www.upwork.com/jobs/~016fa3b4a81adeeec1

Current assignee @michaelhaxhiu is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane (External)

Triggered auto assignment to @Gonals (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Proposal
Please re-state the problem that we are trying to solve in this issue.
While trying to add a bank account using the manual option, the user gets to a point where they have to enter their phone number in order to obtain a secure link. After selecting the country and entering a phone number, if the holder holds the Backspace key to delete the entry, the keyboard input eventually causes the option in the select box to reset.
What is the root cause of that problem?
This is occurring with dynamically injected code from the onfido-sdk-ui dependency.
What changes do you think we should make in order to solve the problem?
It may be possible to create a dynamic JavaScript event to intercept the keypress event for the Backspace button and cancel the default event behaviour for the select HTML element with the class name PhoneInputCountrySelect. The dynamic code is accessible in the DOM, as it's not an placed within an iframe. However, if onfido changes the dynamically generated code in the future, this will cause a regression.
What alternative solutions did you explore?
Possibly filing an issue with the onfido-sdk-ui team to fix the problem.

reviewing

@michaelhaxhiu I think this issue is all yours :) https://github.com/Expensify/App/issues/16640#issuecomment-1497856194
Thanks for raising the issue on onfido's repo 🙌  @daraksha-dk

@michaelhaxhiu, @Gonals, @rushatgabhane Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@michaelhaxhiu @Gonals @rushatgabhane this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

Sorry, just catching up.
Sweet I can try reaching out to the same CSM, let me get that going and report back when its done.

I noticed that SIbtian reached out to Onfido for another job 2 days ago. I think we may want to touch base with the CSM and discuss the future / long term agenda. I'm going to hold this for a little, so that we can speak with the contact at Onfido. I bet we can get on a call this week or early next (based on what Madhi from Onfido emailed me in the past!)

@MariaHCD, tagging you in on this one as I am going on parental leave! Thanks! 😁

Not overdue, we're working on a way to escalate these issues to Onfido.

@MariaHCD @michaelhaxhiu @rushatgabhane this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

Ok, I had the call with Onfido and need to prepare a formal SO post (so this is easily referenced). But the gist of the game plan - the best bet for Onfido bugs is to (1) make the GH in the upstream repo and (2) reach out to support, linking to the GH you made in step 1.

I emailed Onfido support, received automated response w/ subject line:

Onfido Support- Case (00150935) for Michael H has been created

@MariaHCD, @michaelhaxhiu, @rushatgabhane Eep! 4 days overdue now. Issues have feelings too...

@rushatgabhane usually I'd say No for weekly, but based on the message I just got from Onfido I agree:
Let's agree to be careful with not letting this one go. If we don't see any movement in the next ~2 weeks, I'd like to see about potentially prioritizing this in another way.

Greetings for the day!
Our team has looked into this and confirmed that it is a legitimate issue, however, with low priority.
Our team will be picking this up in the next few weeks and we will keep you posted as further progress.
In the meantime, if there is anything else we can help you with, please do let us know. Happy to help!

Note: I may choose to switch this back to Daily in 1 week.

I just gave a friendly bump to the GH in the upstream Onfido repository. Not much else we can do right now. Let's give it another week. At that point I will consider offering some compensation to get it prioritized.

No progress. I am thinking it could be wise to offer a bounty for prioritization. I'm proposing that via email:

I don't think there have been any updates from Onfido on this issue: https://github.com/onfido/onfido-sdk-ui/issues/2155

No updates from Onfido since their email a week ago ^

Still no update :'(

Hello Michael,
Greetings for the day!
This is to inform you that the reported issue (https://github.com/onfido/onfido-sdk-ui/issues/2155) will be fixed in this week's release.
I will drop a note here once the fix is released.
Kind regards
Gurpreet Singh

progress!

Looks like the fix was released! https://github.com/onfido/onfido-sdk-ui/issues/2155#issuecomment-1598922765

@daraksha-dk are you available to create a PR to update the version of the Onfido UI SDK in our App repo to 12.4.0? If so, I'll assign you to this issue!

Let's get this paid :)

Invited all 3 to job on upwork. Please comment here when you accept.
$1500 = Rushat
$1500 = Daraksha
$250 = Natnael

Accepted the offer. Thanks @michaelhaxhiu

WTF is wrong with upwork 🖕 The job link is expired as of today 😒

They should rename to BusyWork instead of UpWork. 😠
New job link is here - invited you again because upwork wants us to repeat this step a thousand times.
https://www.upwork.com/jobs/~01d6ed8d5f098694bb

I have accepted the offer. Thanks @michaelhaxhiu

all paid


Date picker in admin UI shows wrong dates/days

I had to triple check the date/day/year for this, but today -May 10- gets shown as a Saturday, even though it should be a Friday:

PS. Thank you for this amazing piece of software, it has been an absolute joy to work with.

It is already fixed in master as part of https://github.com/pocketbase/pocketbase/issues/4865.
I'll push a new minor v0.22.12 release sometime later this weekend.


[ansible] Update flanneld config file

On RHEL, the flannel-0.5.5-1 is using environment vairables
ExecStart=/usr/bin/flanneld -etcd-endpoints=${FLANNEL_ETCD_ENDPOINTS} -etcd-prefix=${FLANNEL_ETCD_PREFIX}

On Fedora 24, flannel-0.5.5-6.fc24.x86_64 is still using
ExecStart=/usr/bin/flanneld -etcd-endpoints=${FLANNEL_ETCD} -etcd-prefix=${FLANNEL_ETCD_KEY}

Should update contrib/ansible/roles/flannel/templates/flanneld.j2 accordingly, no rush.

flanneld.service as well.  Plus, add if fedora > VERSION based on when the updated flannel gets into stable repository.


tweak for zef compat

Build.pm no longer needs to be based on Panda::Builder. with latest panda.
This tweak allows zef to install it without having Panda installed.  Note:
%zef build .
will build locally.

@dwarring thanks 😄


Add Missing Fields to RawEEGLAB and EpochsEEGLAB Classes

Describe the new feature or enhancement
The current implementation of the RawEEGLAB and EpochsEEGLAB classes in the MNE library does not load all available fields from EEGLAB .set files. Important fields such as 'icaact', 'icawinv', and several others are missing. This feature request aims to extend the functionality of these classes to include all possible fields present in EEGLAB .set files.
The following fields should be added to both RawEEGLAB and EpochsEEGLAB classes:

icaact
icawinv
icasphere
icaweights
icachansind
chanlocs
urchanlocs
chaninfo
ref
event
urevent
eventdescription
epoch
epochdescription
reject
stats
specdata
specicaact
splinefile
icasplinefile
dipfit
history
saved
etc
run
roi
datfile

Describe your proposed implementation
Using Composition and Create a DataContainerEEGLAB Class
I think Composition is a good solution. This class will be responsible for storing and managing all the fields from the EEGLAB .set file. The class will be initialized with a Bunch object containing EEG data and will extract and store all possible fields. Then RawEEGLAB and EpochsEEGLAB classes will have a data_container field.
Describe possible alternatives
Multiple Inheritance
I think it is better not to use multiple inheritance because it may cause problems such as having the same name of the methods, etc.
Additional context
No response

The ICA fields are read by this function in an ICA object: https://mne.tools/dev/generated/mne.preprocessing.read_ica_eeglab.html
Or is something else missing?

closing as not planned; see discussion in https://github.com/mne-tools/mne-python/pull/12773#issuecomment-2275950319


Publish on docker hub

This is working on my repo and I am publishing the docker images to https://hub.docker.com/repository/docker/pablocastellano/bee-dashboard.
It requires configuring the following secrets

DOCKERHUB_USERNAME
DOCKERHUB_TOKEN

Fixes https://github.com/ethersphere/bee-dashboard/issues/617


Get => io.sockets.connected[id_socket]

Hello, an idea how to have the properties of a socket by id ?
Like  "io.sockets.connected[id_socket]" without redis !
Objectif:
io.in(room_id).clients((err, clients) => {
     clients.forEach(socket => {
          // Get socket with her properties
    }
});

Thank for your time 👍

You can use the customHook/customRequest methods:
// on every node
io.of('/').adapter.customHook = (socketId, cb) => {
  const socket = io.sockets.connected[socketId];
  cb(socket ? socket.handshake : null);
}

// then
const socketId = 'abcd';
io.of('/').adapter.customRequest(socketId, function(err, replies){
  console.log(replies); // an array [null, <handshake>, null, ...] with one element per node
});

Given the number of questions around this, I think we should maybe update the clients() or create another method. For you use-case, which properties do you need? handshake?

@darrachequesne  The custom hook is kind of confusing, I have tried to use it but I'm having a hard time with fully understanding how it works or should work and the docs seems very light on it with few example.
For our purpose when a user connects we add into the 'socket' a nickname, so when connecting we add socket.nickname='nickname' so its always tied to the socket, however because that is not shared across each instance with redis we cant use it and have been trying to hack through it with custom redis lists etc.
It would be amazing if there was a way to expose those with just the 'socket' so there was minimal re-coding. Maybe storing stringable objects from the socket in redis so it can be accessed across the nodes?
I had posted a question about this a few days back because, as noted we are trying to keep a list of users online, in rooms etc so we store in a list things like {socket:socket,nickname:nickname} and push and pop that list, however with all the different events (connect, disconnect, reconnect) its proving not to be as reliable as we would like as things get missed at times and things just sit in the redis collection.

Yes, exactly, the problem with storing data in Redis is the cleanup part.
You can remove values on disconnect event, but what if a server goes down? So now you also need to set a TTL, which is longer than the session duration, but not too long. Or you can frequently check and expand the expiry.
Basically, the customHook/customRequest method is a way to transmit data between servers. Calling customRequest sends a request over the Redis channel, and all the nodes respond with customHook. In that case, nothing is stored in Redis, it's only pubsub chat.
We could add a way to transfer the socket properties from one server to another, something like:
const socket = await io.socket(id).fetch();
const nickname = socket.nickname;

Would an API like that fit your use case? Do you have any suggestion?
Note: in that case, we could also have:
await io.socket(id).join("room1");
await io.socket(id).leave("room2");
await io.socket(id).disconnect();

@darrachequesne,
I think that would be a nice option to have as it seems to be a fairly common use-case.  In doing so would this actually be replicated the sockets on each node or would it be more of a placement holder of sorts?
If its the later do you think it would cause performance issues?
To you previous points about connects / disconnects.. yes this is a pain in the butt! We currently switch everything over to use hset / get and store just the socket and nickname in custom redis sets like 'onlineUsers-[uniq-name]' and add / remove users based on this however to your point, if the server gets disconnected or some other action happens when the server does not send the disconnect we are left with orphen sockets in the redis collection.
I'm thinking of adding a function that basically does a cleanup say every 60 seconds maybe, something where we grab the total all sockets connected, take that list and then loop over all sockets in each room and try and delete them. Only problem with this is it could be expensive.
This obviously is not a new issue, however your suggestion above if I'm understanding it correctly would mean we would not longer need to build our our redist sets to keep track of things outside socket.io-redis correct? Seems like that would be less error prone too.

@darrachequesne @webdpro
I am curious about this api approach.
Wouldn't it work the same as those customHooks?
We have it setup like that to get socket properties from other servers.
io.of('/').adapter.customRequest({
            option: 'get_user_info',
            socketId: socketIdid
        }, (err, results) => {
            if (err) {
            } else {
                // iterate results to find one that returned data
            }
        }
);

io.of('/').adapter.customHook = (data, cb) => {
    if (data.option === 'get_user_info') {
        let info = io.sockets.adapter.nsp.connected[data.socketId] ? io.sockets.adapter.nsp.connected[id].userInfo : null; // attached on socket.connect as socket.userInfo = 'xyz';
        cb(info);
    }
}

@darrachequesne @webdpro
I am curious about this api approach.
Wouldn't it work the same as those customHooks?
We have it setup like that to get socket properties from other servers.
io.of('/').adapter.customRequest({
            option: 'get_user_info',
            socketId: socketIdid
        }, (err, results) => {
            if (err) {
            } else {
                // iterate results to find one that returned data
            }
        }
);

io.of('/').adapter.customHook = (data, cb) => {
    if (data.option === 'get_user_info') {
        let info = io.sockets.adapter.nsp.connected[data.socketId] ? io.sockets.adapter.nsp.connected[id].userInfo : null; // attached on socket.connect as socket.userInfo = 'xyz';
        cb(info);
    }
}

@fliespl yes, it would indeed work the same as the customHook API.
The thing is: we have had reports of users that had issues with understanding how the customRequest/customHook API worked, hence the suggested change. Do you have any suggestion about this?

@fliespl yes, it would indeed work the same as the customHook API.
The thing is: we have had reports of users that had issues with understanding how the customRequest/customHook API worked, hence the suggested change. Do you have any suggestion about this?

@darrachequesne I guess providing some real-life examples should suffice - it's the piece that's missing in my opinion and is causing issues for people.
To implement by my first customHook / customRequest logic, I had to spend 0.5-1h to understand how it works by doing research in main code. I wasn't able to implement it by only based on documentation.

@darrachequesne I guess providing some real-life examples should suffice - it's the piece that's missing in my opinion and is causing issues for people.
To implement by my first customHook / customRequest logic, I had to spend 0.5-1h to understand how it works by doing research in main code. I wasn't able to implement it by only based on documentation.

For future readers: please check https://socket.io/docs/v4/server-instance/#Utility-methods


<EMAIL_ADDRESS>Can't shim 'xmcl.exe': File doesn't exist

The file name is X Minecraft Launcher.exe now
Reference: https://github.com/The-Simples/scoop-minecraft/blob/main/bucket/xmcl.json

Fixed, run scoop update first and then reinstall it again

Fixed in https://github.com/anderlli0053/DEV-tools/commit/924f19327f9fef74304205f53b8f759360a988d9


How do you generate user table?

Hi ossinsight maintainers,
This is excellent work! I am actually doing the similar inside my company for OSPO. I'd like to discuss the way you generate user table and it's pretty helpful to aggregate company, geo information.
Did you make Github requests to receive user detail information? If so, do you update user table frequently. That might be a tradeoff between accuracy and rate limit stuff.

The current data is based on data from the last dump of ghetorrent, you can download it on this link.
I will update the latest users data in the next two weeks. (see: https://github.com/pingcap/ossinsight/issues/245), maybe GitHub GraphQL API is a great solution to keep the public users data up-to-date.

Got it. I think no matter v3 API or V4 (GraphQL) to use, it's will come into rate limit issue easily. we probably needs to use different token or just do it partition by partition. Once you update it, let me know the progress. Thanks!


libtool dropped dependency on texinfo

cmake and doxygen are also build requirements of many other packages.
The latest versions should only pull in xapian-core
Re-ordered so that the first few can be added to the module

Thank you!


Cant log in with Github

Expected Behavior
I should be able to log in.
Actual Behavior
Trying to log in redirects me to Github, where I authorize the application. The next page (still on Github) says it's about to redirect me, but it keeps loading. If I click on the manual redirect link, I am redirected to Adoptoposs, but it says "Authentication failed.", and I'm still not logged in.
Steps to Reproduce the Problem
Described above.
Specifications

Version: N/A
Platform: Firefox (latest) and Brave (latest)
Subsystem: N/A

Let me know if there's anything else I can do to help debug this.

Thanks, @nukeop, for reporting. I suppose you were trying to login from desktop browsers?
I tried in Firefox v100.0 on Desktop and Mobile (Android) in normal and private mode and it worked fine for me :thinking:
Did you already try to revoke the access for the Adoptoposs application from the "Authorized Oauth Apps" in you GitHub settings (https://github.com/settings/applications --> right tab) and then tried to log in again? I was thinking, maybe some token auth scopes were not granted before ( - whereas we have the minimum required read-only auth scopes, and they did not change for a long time).
Please let me know if it's still an issues, then I'll dig deeper.

Hm, that didn't fix it - and I've tried another computer, 2 other browsers (Vivaldi, Chromium), and a different Github account. Is there any other way to log in?

:( nevertheless, thanks for trying. Unfortunately there is not other way to log in right now.
But the good news is: I was able to reproduce the issue and am going to work on a fix tomorrow.

The fix went faster than expected - it's already deployed :rocket:
The culprit was a blank (real) name from the GitHub account side, that needed to be handled during login. I had a name set in my GitHub account, that's why it worked for me. :upside_down_face:
Could you please try to log in again and see if it works for you now?

Thanks! I confirm it's working.