Error: Cannot read property 'joinKey' of undefined

I use Ractive with Backbone.js and I recently upgrade Ractive version from 0.7 to 0.9.13
Before the upgrade, everything is fine.
Sometimes when I update the Backbone model, the error Cannot read property 'joinKey' of undefined occurs. This error in function ModelBase__proto__.joinAll
In some cases, I fixed by using .clone() method to the Backbone model. But in other cases, it has no effect.
Do you please give me some suggestions to handle this issue?
Thanks.

I cannot public all code becauce the privacy of project.
The code cause error:
return formItems.findWhere({
    itemNo: sort.get('itemNo'),
});

Then I add clone() like this:
return formItems.findWhere({
    itemNo: sort.get('itemNo'),
}).clone();

Then in some case it works fine. But some other case it doesn't.
Don't understand the root cause of the error.

@xilanhthanki Your above description may or may not help to solve the problem, but I wanted to put a general recommendation, if you don't mind:
In my understanding, active Ractive users work on large projects and I do definitely know that sometimes it's a very time consuming task to create a minimal reproduction code in the Playground. But that's fair, because the investigator (@fskreuz, @evs-chris and the Ractive users) has to have something to work on/mull over. IMHO, a reproduction code is equal to a "pages of description of a problem".
Bonus: If you actually try to make it a habit to create a reproduction code first, you will often find yourself pressing the "cancel button" while you are about to create a new issue. (guess how I might know :) )

Thanks @ceremcem for the recommendation. Highly appreciate it.
I cannot make the reproduction code in the Playground cause the code in the project too complex and the error may come from any corners.
Maybe you can close this issue. Thanks for your support.

Based on the stack trace, I'm not really sure how this is happening. Looking through everywhere that calls joinAll, there should be no case where the model can not be an actual model as far as I can tell. It would be tremendously helpful to be able to step through and see what's going wrong. If you can't reduce it to a smaller case or share code publicly, could you share it privately?
Failing that, can you tell me what formItems and sort are? I have no experience with backbone, but I may be able to stumble through it given some more details.

Unfortunately the project is huge because it have been run few years ago. So I don't know how to share with you. There're some problems about performance so we have to do many things to improve. My work is doing version up from 0.7 to 0.9.13 and fix all issues appears.
Btw the formItems  is a Backbone Collection and the itemNo: sort.get('itemNo') is just a condition to find a model in collection.
I found this in Backbone API: http://backbonejs.org/#Collection-findWhere
The function return a Backbone model (with some changed values) then the error occurs here. After cloning the model, the error disappears. So my guess is the mutation of the Backbone model causes this issue but I'm really not sure.
I know the hypothesis is too little for you to investigate. Thanks you all for your support.

Addition hypothesis. The problem may occur in Backbone.js adaptor for Ractive
See my attached images
The good case: no prefix @

The bad case: has prefix @

All the time we see the prefix @, the error occurs.
Hope this help. Thanks.

Gtocha. This is an incompatibility between the backbone adaptor and ractive >= 0.8. What's happened is that computed contexts are no longer kept at the model level, due to potential overlap wherein one model gets overwritten by another that happens to have the same source path but is a completely different model. What needs to happen to fix this is the adaptor needs to get a context helper rather than a prefixer in order to be able to update relative keypaths. Additionally, setting computed contexts has been removed, also I believe from version 0.8.
As a workaround, you can avoid using a computed context (wherever the getSelectedItem() is being called in the template) along with the backbone adaptor. Depending on what you're doing that may not be feasible. I'm guessing you're using that in an iteration, and if so, you might try jumping to 0.10.0 and the experimental source mapped {{#each}} block. A reference expression (some.path[variable]) can also avoid computed contexts in 0.9, and as of 0.10 you can use an array of keys to get to any model if you know the path. You can safely have a computed member of a reference expression e.g. ~/[getPathTo(item.1)], which combined with 0.10's support for arrays in reference expressions looks like it may also get you where you need to be.

Your errors with 0.10 would likely be resolved by setting resolveInstanceMembers: true if going from 0.7 to 0.9 worked and 0.10 didn't. There are very few breaking changes between 0.9 and 0.10, and that's the most visible one from older versions. Alternatively, prefixing your method calls in event listeners with @. would achieve the same effect in a safer way.
Other than avoiding computed contexts, there's not much I can think of that would work as a workaround in 0.9. I'll look at the backbone adaptor to see if I can come up with a reasonable fix there.

Thanks. Solved the errors by setting resolveInstanceMembers: true
Let me try your workaround.

After investigating, This is what I found
In parent component, I'm writing this
<FormSelectbox options="{{form.items}}" selected="{{getFormItem(this)}}" index="{{index}}">
Then in child component, everytime I use selected property, the error always occurs. Although I just use console.log(selected) only.

So instead of use selected property directly, I pass itemNo to child component, then do findWhere inside child component. The error doesn't appear anymore. This fix also works on 0.9
This may be inconvenience but I think I have to do this in my project.
Thank you all for your support.

Pushing this over to #2512.


bug: admin api  setting upstream config checks , functionclean_handlers  is nil

Issue description
upstream conf eg:
{
    "desc": "AUTO-aimgo-web-v1.21-passport",
    "retries": 1,
    "key":"",
    "hash_on": "vars",
    "enable_websocket": false,
    "timeout": {
        "connect": 0.5,
        "send": 1,
        "read": 2
    },
    "checks": {
        "active": {
            "unhealthy": {
                "http_statuses": [
                    429,
                    404,
                    500,
                    501,
                    502,
                    503,
                    504,
                    505
                ],
                "interval": 1,
                "timeouts": 3,
                "http_failures": 2,
                "tcp_failures": 2
            },
            "type": "http",
            "http_path": "/health",
            "healthy": {
                "successes": 1,
                "interval": 2,
                "http_statuses": [
                    200,
                    302
                ]
            },
            "req_headers": [
                "User-Agent: apisix.health,curl/7.29.0"
            ],
            "timeout": 1,
            "https_verify_certificate": true,
            "concurrency": 10
        },
        "passive": {
            "unhealthy": {
                "http_failures": 3,
                "http_statuses": [
                    500,
                    501,
                    502,
                    503,
                    504,
                    505,
                    506
                ],
                "tcp_failures": 3,
                "timeouts": 7
            },
            "healthy": {
                "http_statuses": [
                    200,
                    201
                ],
                "successes": 3
            },
            "type": "http"
        }
    },
    "nodes": {
        "<IP_ADDRESS>:40940": 1
    },
    "type": "roundrobin"
}

first add config happen error, error info eg:
2020/04/01 17:34:12 [error] 5307#0: *21439 failed to run balancer_by_lua*: /usr/local/apisix/lua/apisix/balancer.lua:106: bad argument #1 to 'insert' (table expected, got nil)
stack traceback:
        [C]: in function 'insert'
        /usr/local/apisix/lua/apisix/balancer.lua:106: in function 'create_obj_fun'
        /usr/local/apisix/lua/apisix/core/lrucache.lua:66: in function 'lrucache_checker'
        /usr/local/apisix/lua/apisix/balancer.lua:127: in function 'fetch_healthchecker'
        /usr/local/apisix/lua/apisix/balancer.lua:248: in function 'pick_server'
        /usr/local/apisix/lua/apisix/balancer.lua:313: in function 'load_balancer'
        /usr/local/apisix/lua/apisix.lua:486: in function 'http_balancer_phase'
        balancer_by_lua:2: in main chunk while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET /v1/passport/hello/222 HTTP/1.1", host: "web.uqudu.com:8888"

Environment

apisix version (cmd: apisix version):  master And 1.0
OS: centos7

Minimal test code / Steps to reproduce the issue

first call admin api write upstream config after request server api error occurred.
restart apisix server,  request server api  get right
guess  should modify code ,not restart also is success! eg:

healthcheck_parent.clean_handlers = {}
upstream.parent.clean_handlers = {}

What's the actual result? (including assertion message & call stack if applicable)
What's the expected result?

I will check it later, thanks for your report.

I want to fix it.


IDE Plugin

Hi
There is a plugin for Coda too.
Cheers
Steve

Do you have any URL?

Hi
The url is https://github.com/bobthecow/Twig-HTML.mode.git
It is more of just a syntax highlighter at present but still helps. Have you used Coda?


MSID: 91429 Version: 2 DOI: 10.1101/2<IP_ADDRESS>5862

MSID: 91429
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5862
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91429
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5862

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/91429v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5862
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/91429/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Validation messages:
error: This journal reference (id c5) has no article-title element.
error: This journal reference (id c6) has no article-title element.
error: This journal reference (id c23) has no article-title element.
error: This journal reference (id c62) has no article-title element.
error: This journal reference (id c94) has no article-title element.
error: This journal reference (id c95) has no article-title element.
error: This journal reference (id c98) has no article-title element.
error: This journal reference (id c118) has no article-title element.
error: This journal reference (id c119) has no article-title element.
error: This journal reference (id c120) has no article-title element.
error: This journal reference (id c121) has no article-title element.
error: This journal reference (id c124) has no article-title element.
I've uploaded the MECA package (with no corrections) to s3://prod-elife-epp-meca/91429-v2-meca.zip, so it can be downloaded/edited.


All resizes seem to have an issue where stuff gets pasted in other notes... sometimes.

This is probably a preventDefault thing.

This was a preventDefault thing.


SFrame.save(...) should auto detect .json file format

If format is None, which is the default value, the format is inferred from the file type. However this only works for .csv files. We make it work for .json files too.

Issue fixed with PR #1038


More dotfiles

Path
Project
Comment

~/.Aliview
https://github.com/AliView/AliView
does not seem to support XDG

~/.dgl
https://github.com/dmlc/dgl
seems to support XDG via environment variables DGLDEFAULTDIR and DGL_DOWNLOAD_DIR

~/.pymol and ~/.pymoltimestamp
https://pymol.org/
does not seem to support XDG

~/.biojs_templates, ~/.jalview_properties, ~/.jalview_develop_properties and ~/.jalview_nonrelease_properties
https://www.jalview.org/
does not seem to support XDG.

~/.groovy, ~/.jswingreader and ~/.biojs_templates
???
I think they are created by jalview

~/.grass8
https://github.com/OSGeo/grass
Path is hardcoded

steam
~/.steampath ~/.steampid ~/.steam and games
https://github.com/OpenMandrivaSoftware/steam_xdg_enforcer


tweak: use standard flit dev dependency extra name

flit install --deps develop uses the extra names test, doc and dev.
This changes the template and the main pyproject to respect that.
https://flit.pypa.io/en/stable/cmdline.html#cmdoption-flit-install-deps

Major breaking change for our actions if we proceed to merge this. Pinging @jorgepiloto. I agree with the change though. It's just that we should inform everybody so that they can update their current pyproject.toml files.


Fixed #25697 -- Added warning when passing non existing template_name

Passing a non existing template_name to default error views should
raise a TemplateDoesNotExist error. Emitting a
PendingDeprecationWarning for the deprecation phase.

@charettes updated as per your comments

@timgraham modified as agreed

Some minor edits: http://dpaste.com/1CVC18F
The commit message also needs to be updated as it still talks about a deprecation.
Could you rework the tests to call the views directly in the test method instead of requiring URL patterns? I think this will be simpler.
e.g. (completely untested):
rf = RequestFactory()
request = rf.get('/')
page_not_found(request, Http404(), template_name='...')

Thanks!

@timgraham updated as per your comments. The above snippet looks nice and working. Tests updated (as a different task, probably updating test_default_content_type_is_text_html along the same lines can be useful. I can provide the fix in a different PR

@timgraham everything is green again

merged in 21bf685f5e660a187fcac31e84d5cd89758aeb74, thanks!

@timgraham thanks for your gide


Issue 65 - refactoring proposal

Refactoring of 2 methods.

Coverage increased (+0.3%) to 95.188% when pulling d9ef5630b0090ba6e9a859cd011839bee51b58df on chutch-patch-1 into f66355f636c362ef60e0aca301ac3014c394cbf7 on issue-65.


Handling invalid times in Nimrod radar data

✨ Feature Request
Deal with invalid seconds in nimrod data triggering an  error in iris.fileformats.nimrod_load_rules.time
Motivation
I am processing Nimrod data in the BADC archive and data for 2004-2006 has invalid seconds with a value of -32768.
cftime.datetime triggers an error when given a time of -32768 seconds. I tried to use the callback feature, but that is called after the times are set in the cube and so cannot fix the time before cftime.datetime is called (and fails).
Additional context
My solution was to remove the vt_second from the call to datetime as resolution of the radar data is 5 minutes, so seconds are not that important. I then “monkey” patched iris.fileformats.nimrod_load_rules.time. This is not an elegant solution…
Three potential solutions:

Modify iris.fileformats.nimrod_load_rules so that the callback function is called before times are set.
Deal with invalid times when creating the field (before converting it to a cube)
Ignore seconds when creating the cftime.datetime

This is the relevant bit of code in time after my modification:
valid_date = cftime.datetime(
field.vt_year,
field.vt_month,
field.vt_day,
field.vt_hour,
field.vt_minute,
#field.vt_second, # seconds occasionally invalid but not needed for nimrod...
)

Thanks @SimonTett, we agree that this isn't particularly elegant. However it may be the correct thing to do in this case:

We're dealing with a file that is acknowledged to contain invalid information.
We have fewer users using Nimrod formats than other formats, and we have to make tough prioritisation calls 😥

We'll leave this open as a marker, so anyone is welcome to propose a fix if they have time. We'd be happy to review a pull request if you put one up 🙂 Your point 2 is the only one we think we'd consider acceptable.


Current version (4 days ago) crashing after choosing folder

Going to try older ones now, is it just my pc?
Never mind, they all crash, cant work out what dll its not finding either

I have tried to download visual studio and cloned the repo but the build fails too, am i cursed?

Incase you didn't find a fix, I don't know which one fixed it but I installed c++ Red 2010/2012/2013/2015-2022 I think it was 2010.


Remove pyproject.toml, add ruff.toml (dev branch)

Description
Since we don't currently use pyproject.toml for its intended purpose of describing the entire DefectDojo application and its dependencies, but only to configure the ruff linter, this PR switches to using the ruff.toml file to configure ruff. This will prevent confusion with tools that might try and fail to parse our current pyproject.toml file.
This one is the same as #9929 but for the dev branch.
Test results
This appears to work as expected when running ruff 0.3.7 locally - no linting issues reported.
[sc-5411]

Had some wicked conflicts from the release and had to pull this PR into it #9936


Update Dashboard UI

What does this PR do?
   The Login page is now responsive.
   The main Dashboard layout is responsive now and the Env variables are categorized in different components.
   Tooltip added to the "Invite members" button on users page to inform why it is disabled.

Which issue(s) does this PR fix?
     [Issue](https://github.com/authorizerdev/authorizer/issues/157)

If this PR affects any API reference documentation, please share the updated endpoint references

This is replication of #161 and not needed any more


[#108858698] Enable new Whitehall attachment processing

We've tested this in integration and we believe it works as we expect.
This might be easier to review by adding ?w=1 to the URL to remove whitespace changes.

LGTM, very exciting :fireworks:

Pushed a commit for the CPU check.

https://govuk.zendesk.com/agent/tickets/1229635 may have a bearing on this. So hold off for a while.

We've had a look at that and we think it's unrelated.

:+1: Nice work! :eggplant:


Add clear label for other than production environments

This issue is imported from pivotal
E.g. TEST, TEST2, ACCEPTANCE, etc.


Bucket logging configuration using library

I was wondering whether it is possible to set the bucket logging in the target bucket with the access logs are then stored in the bucket log as it is explained here https://cloud.google.com/storage/docs/access-logs but this is only possible using gsutil. There is the option for XML API here https://cloud.google.com/storage/docs/xml-api/put-bucket-logging but not sure if there is one in this google-cloud-java storage library. If it is possible how do I do it?

I don't believe this feature is available here yet. @hzyi-google can you confirm? If it is unavailable, let's turn it into feature request.

I don't think there is either.

I think it will be good as well if you could check the status of bucket logging and remove bucket logging if needed as doing this from console is quite hectic


Create README.md for valaxy-addon-waline

#120

Emm, which probably does need a README, but just creating a README may not be worth it as a pr.
Do you want to complete its README?

No, I don't want to, because I'm too lazy. :)


parse css numbers using the invariant culture

If current culture does not have '.' as decimal separator HtmlToWmlCssApplier.cs will fail to parse css numbers such as 33.3%. Fixed by always parsing with the Invariant Culture.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Removes sfdx from the cli:version:inspect command

What does this PR do?
Removes sfdx from the cli:version:inspect command
Other cleanup to remove complexity
Example of this skipping non-existing tarballs:

Testing:

pull branch
yarn build
Run the same command we use during promotions here

bin/run.js cli:versions:inspect -c stable-rc -l archive

You can run it with npm also if you want

bin/run.js cli:versions:inspect -c stable-rc -l archive -l npm

NOTE: I did not look at any of the "dependency" listing code. That was out of scope for this.
What issues does this PR fix or reference?
@W-16893140@

QA
🟢 bin/run.js cli:versions:inspect -c stable-rc -l archive
🟢 bin/run.js cli:versions:inspect -c stable-rc -l archive -l npm
Separate but do we need a WI for this?
(node:99362) NOTE: The AWS SDK for JavaScript (v2) will enter maintenance mode
on September 8, 2024 and reach end-of-support on September 8, 2025.

Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check blog post at https://a.co/cUPnyil
(Use `node --trace-warnings ...` to show where the warning was created)


[fix][sys] doesn't work release plan on sys/notice

Coverage decreased (-16.7%) to 66.424% when pulling e9aa91fc4c0ff92a082d812877bd96051e8353be on sunny4381:fixes/sys-notice into 731f9998d9b3b1bf88a0230882086337b25f1a3c on shirasagi:master.


As a graduate, I want to be able to showcase my skill tags, personal statement, resume, LinkedIn profile fetching from Github so that my work are clearly displayed to potential employers.

The platform must display gratudate's skill tags;The platform must display gratudate's statement;The platform must display gratudate's resume;The platform must display gratudate's LinkedIn profile,

After task 24, this user story will be completed.


[DevTestLabs] Adding command for claiming vm from lab

Part of feature #3317

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

(env) vishrut@mac azure-cli (lab-vm-claim) $ az lab vm claim -h

Command
    az lab vm claim: Take ownership of an existing virtual machine in Azure DevTest Lab.

Arguments
    --lab-name          [Required]: Name of the Lab.
    --name -n           [Required]: Name of the virtual machine.
    --resource-group -g [Required]: Name of lab's resource group.

Global Arguments
    --debug                       : Increase logging verbosity to show all debug logs.
    --help -h                     : Show this help message and exit.
    --output -o                   : Output format.  Allowed values: json, jsonc, table, tsv.
                                    Default: json.
    --query                       : JMESPath query string. See http://jmespath.org/ for more
                                    information and examples.
    --verbose                     : Increase logging verbosity. Use --debug for full debug logs.

Formatter changes example:
(env) vishrut@mac azure-cli (lab-vm-claim) $ az lab vm list --lab-name cliautomationlab -g lab-template --claimable -o table
Name           Location    OsType
-------------  ----------  --------
centosclaim01  westus      Linux

Codecov Report

Merging #3318 into master will increase coverage by 0.02%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master    #3318      +/-   ##
==========================================
+ Coverage   70.32%   70.34%   +0.02%
==========================================
  Files         391      391
  Lines       25110    25121      +11
  Branches     3815     3816       +1
==========================================
+ Hits        17658    17672      +14
+ Misses       6336     6327       -9
- Partials     1116     1122       +6

Impacted Files
Coverage Δ

...re-cli-lab/azure/cli/command_modules/lab/params.py
98.48% <100%> (+0.04%)
:arrow_up:

...zure-cli-lab/azure/cli/command_modules/lab/help.py
100% <100%> (ø)
:arrow_up:

...-cli-lab/azure/cli/command_modules/lab/commands.py
85.43% <71.42%> (-1.3%)
:arrow_down:

...rc/azure-cli-testsdk/azure/cli/testsdk/__init__.py
100% <0%> (ø)
:arrow_up:

.../azure-cli-testsdk/azure/cli/testsdk/decorators.py
100% <0%> (ø)
:arrow_up:

...li-lab/azure/cli/command_modules/lab/validators.py
30.26% <0%> (+1.18%)
:arrow_up:

...re-cli-lab/azure/cli/command_modules/lab/custom.py
45.71% <0%> (+2.85%)
:arrow_up:

Continue to review full report at Codecov.

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

Updated help with --ids
(env) vishrut@mac azure-cli (lab-vm-claim) $ az lab vm claim --help

Command
    az lab vm claim: Take ownership of an existing virtual machine in Azure DevTest Lab.

Arguments

Resource Id Arguments
    --ids              : One or more resource IDs (space delimited). If provided, no other arguments
                         should be specified.
    --lab-name         : Name of the Lab.
    --name -n          : Name of the virtual machine.
    --resource-group -g: Name of lab's resource group.

Global Arguments
    --debug            : Increase logging verbosity to show all debug logs.
    --help -h          : Show this help message and exit.
    --output -o        : Output format.  Allowed values: json, jsonc, table, tsv.  Default: json.
    --query            : JMESPath query string. See http://jmespath.org/ for more information and
                         examples.
    --verbose          : Increase logging verbosity. Use --debug for full debug logs.

Thanks for the review!


Hangs with no feedback when TestClass is not initialised

The GUI is showing no feedback and is blank if a test in the test list is not initialised.
An error should be raised and be viewable from both the gui and cli.

Linked with #16


Register v0.1.3

@JuliaRegistrator register
Release notes:

refactoring
cleanups
correctness fixes

Registration pull request created: JuliaRegistries/General/99232
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" 5e5a2c72ea3ed14926506caae7079c02d38c5d5e
git push origin v0.1.3


Add backreference to S3A to prevent premature finalize

Problem
We converted our YARN clusters to use S3 for logs. In general, this worked, but when our users had large enough logs, two negative behaviors developed:

Some specific node manager logs wouldn't load, erroring out with no logs available for that container.
When we tried to use the yarn logs command, it would similarly fail.

At first, we assumed there was something wrong with the logs in S3, but in all cases, they were there, so we were able to directly access and work around this, but that wasn't a long term solution. Logs in the job history server indicated that the S3AFileSystem object was closed during active operation. Incorrect ordering of these operations was suspected, but when diagnosed what was triggering the close, it was the finalize() method of the S3A object.
This was triggered from the LogReader class, which constructs a S3A object in its constructor, but does not retain a reference outside it, keeping instead a stream object that uses a produced S3AFileSystem object. This turns usage of that LogReader object into a race against the garbage collector as it decides to finalize the S3A object, which in turn closes the S3AFileSystem object. This doesn't happen for LogWriter because it retains the FileContext containing the S3A object as a reference for the lifecycle of that object.
Solution
Similar to https://github.com/apache/hadoop/pull/5780, I'm going to try add a back reference to address the issue of the S3A object being garbage collected while the LogReader is still processing TFiles. I considered changing the logic of the LogReader to hold a reference to this object, but that's fragile when the real problem is the assumption that the S3A object won't get reaped while created S3AFileSystem objects spawned from it are still in use.
The only downside here is that if the S3AFileSystem objects are not closed properly, the finalize() will never happen for S3A to trigger cleanup. However, that's a diagnosable logic failure whereas the bug being fixed is unpredictable behavior and failure.
Testing
We deployed an updated version of this to our JHS and RM to test if both the log browsing and yarn logs CLI worked successfully with sufficiently large logs. It did. We haven't done long term memory impact assessment yet.

LGTM


Fix flaky TestContainerLifecycleEvents e2e test

What does this PR do?
In the TestContainerLifecycleEvents e2e test, consider the oldest nginx pod.
Motivation
Despite the previous fix #25710, the TestContainerLifecycleEvents e2e test has failed once.
Additional Notes
In order to be sure that a pod lifecyle events is emitted, we need to delete a pod which has already been seen by the datadog agent.
That’s why #25710 added a filter to consider only running pods.
In the failure case mentioned above, the
kubernetes_state.pod.ready{kube_cluster_name:ci-552666450-4670-kind-cluster, kube_namespace:workload-nginx, pod_name:nginx-55847996d-2jt7j}

proves that the pod was running:

It was caught by the kubernetes_state_core check running in the cluster checks runner.
But the
container.memory.usage{kube_cluster_name:ci-552666450-4670-kind-cluster, kube_namespace:workload-nginx, pod_name:nginx-55847996d-2jt7j}

shows that the pod didn’t run for long enough to be seen by the node agent.

So, in this case, the nginx pod was up and running for very few time and it was deleted before the kubelet poller of the node agent could see it.
We deleted a pod that wasn’t ever seen by the node agent. No pod event lifecycle was emitted.
This issue should be fixed by systematically choosing the oldest pod.
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
TestContainerLifecycleEvents e2e test shouldn’t fail anymore.

/merge


HDS support for gRPC Health Checks

Title: HDS support for gRPC Health Checks
Description:
I'm trying to use HDS to configure a gRPC health check, but its failling and I see this error in the logs:
...  cluster must support HTTP/2 for gRPC healthchecking

Digging into the code I see that the health checker code (source/common/upstream/health_checker_impl.cc) has a validation check to ensure that gRPC health checks can only be configured for clusters that have the HTTP2 feature present.
It seems that clusters generated via HDS do not ever set the HTTP2 feature and so it it doesn't seem possible to configure a gRPC health check with HDS.
What I propose is that support for HttpProtocolOptions are added to the ClusterHealthCheck message proto so that Clusters built with HDS can have the HTTP2 feature set and thus gRPC health checks may be configured via HDS.

I'm happy to submit a PR if the suggestion is reasonable :)

Can you explain the problem in a bit more detail and provide the config that is problematic?
I think the tests in https://github.com/envoyproxy/envoy/blob/main/test/integration/hds_integration_test.cc are covering the right cases, no?

Correct me if I'm mistaken but I don't see an any tests in https://github.com/envoyproxy/envoy/blob/main/test/integration/hds_integration_test.cc  that use a gRPC health check specifier, only different forms of HTTP and TCP.
To restate the problem: I want to use the health discovery service to configure a gRPC health check https://github.com/envoyproxy/envoy/blob/21fba6209fdeee43c86a917b6a26823fd718d941/api/envoy/config/core/v3/health_check.proto#L205 but the config fails due to the check at https://github.com/envoyproxy/envoy/blob/main/source/common/upstream/health_checker_impl.cc#L70 which requires the upstream cluster to be explicitly configured with the HTTP/2 feature.
As far as I can tell the health discovery service does not offer a means for setting the HTTP/2 feature on an upstream cluster and so I'm at an impasse.
If that doesn't clear it up I can provide a config.

Here is sample config:
{
  "cluster_health_checks": [
    {
      "cluster_name": "my-cluster",
      "health_checks": [
        {
          "timeout": {
            "seconds": 1
          },
          "interval": {
            "seconds": 1
          },
          "unhealthy_threshold": {
            "value": 1
          },
          "healthy_threshold": {
            "value": 1
          },
          "HealthChecker": {
            "GrpcHealthCheck": {}
          },
          "no_traffic_interval": {
            "seconds": 1
          }
        }
      ],
      "locality_endpoints": [
        {
          "endpoints": [
            {
              "address": {
                "Address": {
                  "SocketAddress": {
                    "address": "<IP_ADDRESS>",
                    "PortSpecifier": {
                      "PortValue": 3000
                    }
                  }
                }
              }
            }
          ]
        }
      ]
    }
  ],
  "interval": {
    "seconds": 2
  }
}

no stalebot


Support for product flavors

Currently, we run tests against using the debug variant. This doesn't work correctly when the build has flavors.

@erd Debug is a BuildType. Do you mean ProductFlavor or BuildType? Currently I utilize product flavors in my apps. For instance, I usually run gradlew testFreeDebug to run Robolectric tests against the ProductFlavor "Free" and the BuildType "Debug".

@jaredsburrows The description means that the current 1.0.0-SNAPSHOT version of the gradle plugin assumes no product flavors, as it tries to figure out the AndroidManifest, resources, and assets paths via the "debug" variant name. "Variant" is a matrix combination of BuildType [+ ProductFlavors].  When there are no product flavors, the variant is simply "debug".  When there are, it is a combination of the build type and product flavor;  e.g., "debugFree" and "debugPaid".  That's what he means by "the debug variant" -- it was not a typo.  When you have product flavors, the current snapshot does not work, because the variant is not simply "debug" anymore.
See the discussion from #129: https://github.com/robolectric/robolectric-gradle-plugin/pull/129/files#diff-82b959b86a626f6171146a18ca9a224cR1

Alright, thanks for specifying 1.0.0-SNAPSHOT .

You like to use robolectric with flavors? if you like try a solution without this plugin https://github.com/nenick/AndroidStudioAndRobolectric/tree/flavors


WIP Feature - Implement BoundedTaskQueue.java & PriorityTaskQueue.java #98

Please refer to #98 for details about this feature branch.
closes #98

Closing and will re-open with a PR merging into staged-for-release.


ko-trans: add includes/index.md

ko-trans: add includes/index.md
I don't think 'true' should be changed.

/assign @claudiajkang

/assign @ClaudiaJKang

@xichengliudui Thank you! :+1:
/lgtm
/approve


Add example using own data sets

@ojhall94 has a nice example of analysing 16 Cyg A where he uses his own data that he plus into lightcurve and then run PBjam star.  We should add this example to the docs somewhere.
It's probably more about interacting with lightkurve than anything else but probably an issue users will care about.
We should add a few more orders to the example and we should fix the problem #204 to make it look nicer.

Is there a link to this notebook? It sounds like that's what happens when you give pbjam.session a numpy array or file name instead of a lightkurve object.
This is of course not possible if we want the tutorial to show the pbjam.star case, but for pbjam.session it's trivial.

@ojhall94 - do you have the 16 Cyg notebook?  We'll make it pretty if you can shoot it across.


Open folder with docker-compose hangs after creating container

VSCode Version: 1.35.0
Local OS Version: Windows 10 64 bit, using docker toolbox version 18.03.0-ce
Name of Dev Container Definition with Issue: docker-compose template

Steps to Reproduce:

Clone this repository
Navigate to container-templates/docker-compose folder and run code .
When the window opens, click Reopen in Container
docker-compose build runs and hangs after container starting

Hangs with this:

Things I have tried:

Setting up workspaceRoot to an empty string in devcontainer.json
Re-creating the local docker-machine
Following this issue

@talohana I can confirm the definition itself works with Docker Desktop 2.0.3 (engine 18.09.2) on Win 10 64-bit and macOS Mojave.  Likely you are running into a problem with Docker Toolbox itself.  Moving to vscode-remote-release since this is not related to the definition itself.
Unfortunately, Docker Toolbox is not something we are supporting as it is a legacy solution that Docker is phasing out.  You could try https://code.visualstudio.com/docs/remote/containers-advanced#_developing-inside-a-container-on-a-remote-docker-host to see if this can work. I don't know why it would hang, but it's possible the problem is trying to volume mount the local filesystem. Docker toolbox support is with https://github.com/microsoft/vscode-remote-release/issues/95
Can you retry with Docker Desktop?


Remove file://

To implement the benchmarks, we need to remove the file:// scheme (which also isn't allowed by the htsget spec). See here. Last time I just changed it for http://localhost, but this time I was thinking maybe we should take a more permanent route? If I understand correctly, LocalStorage shouldn't be constrained to localhost.

Indeed, localstorage refers to an arbitrary amount of bytes on a volume, but the host of those volume(s) can be any host other than localhost too (connected to a network even). So yes, generalising it would be the best course of action.

Okay! Sadly, I don't think I can commit the necessary time for this. Maybe Marko or you can take a look.

No worries @CastilloDel, thanks for pointing it out, we need to tackle this for sure if we want htsget-rs to be fully spec-compliant, which we do ;)


Fix bug overwriting exe manifest on reproduce

Fixes #265

Hello @aidanheerdegen! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file payu/models/model.py:

Line 284:78: W291 trailing whitespace

Coverage increased (+0.02%) to 42.024% when pulling e5ac960c6a8501b883623df03f6643058be76830 on issue-265 into 70a3ce923bc154d1f158a1e5f6722e8865e900c5 on master.


support sort

if it's passible to support sort?

The map is implemented as a hashmap, and as such is unordered. If you want the keys to be sorted for a specific operation you would have to iterate them and add them to an array, then sort the array and work from that.


📚[DOC]: Information on Local Data for inference ensembles

How would you describe the priority of this documentation request
Medium
Is this for new documentation, or an update to existing docs?
New
Describe the incorrect/future/missing documentation
The documentation lacks information for setting up the initial conditions. For GFS, IFS and CDS, it's rather clear. However, I'm having a lot of trouble understanding the hrmip and hdf5 files. I'm not sure how to use them, but I want to use my local data for inference_ensembles. Any help would be much appreciated!

The hdf5 and hrmip files are somewhat specific to our internal datasets.
If you have your own local data format, I would recommend writing your own subclass of
API: https://github.com/NVIDIA/earth2mip/blob/86b11fe4ba2f19641802112e8b0ba6b962123130/earth2mip/initial_conditions/base.py#L26.
You can then use earth2mip as a library with your own datasource.
Does that make sense?

Ohh I see, thank you!


Easy to use Inference code

This pull requests contains easy to use inference code. The shell script provided by authors for inferecing doesn't save video files. Also for generating videos on custom images one has to rewrite the .yaml file.
The script can be run by using
python script_name.py path_to_reference_image.jpg path_to_motion_sequence.mp4 --seed 123 --steps 50 --guidance_scale 5.0

#71


Write documentation

This seems really interesting! Are there plans for more (or any) documentation on the VM?

Yes, the language is in pretty dire need of documentation but I've been putting it off since I'm the only user I know of ^^
Ideally I'd document the basic features of the language and provide some explanations for more unique features like the environments. The VM part would be pretty simple to document as well so I don't see that being an issue.


(Docs) What is a base_unit?

Metric has a method to_base_units, but no info on what is a base_unit. What is it? How would a Metric use these functions?

looks like there is more info in the src that is not being displayed by rust doc. Perhaps it is \\ instead of using \\\?

Conversation happening in irc https://botbot.me/mozilla/xi/2017-12-09/?msg=94473883


[New Video Request]: How much JavaScript is needed to learn React.js?

👋 Hey There,
You can request a video in the areas of,

Web Development(JavaScript, React, CSS, HTML, many more)
Jamstack
Content Creation/Blogging
Career

Please provide the answer to the questions below to get started:

Suggest a Topic Name
How much JavaScript is needed to learn React.js?

What do you want to Learn(Please provide some pointers)?
The JavaScript concepts to learn before learning React.js. Sometimes people think one particular feature is provided by React, but instead it’s just modern JavaScript syntax.

How soon do you need this video tutorial?(Give minimum a couple of weeks)
At your convenience.

Do you want to see your name, social handle as a requester in the Video Description? (If yes, please mention the full name, social handle, etc.)
Vikas Ganiga | twitter.com/vikasganiga05 | github.com/vikasganiga05

Note: Please be polite while raising the request. The creator has all the rights to accept/reject a request. There is no strong commitment made by the creator to publish a requested video within any given time frame. But, will try as much as possible.

@vikasganiga05
Thanks a lot for opening the issue. I have captured it in this video: https://www.youtube.com/watch?v=ODKIxaSMgpU
The above video also links several other videos on JS required to get started with React.
Also, I have captured it in greater detail here: https://www.freecodecamp.org/news/react-fundamentals-for-beginners/
I hope it helps.

I am closing this issue based on the above comment.

@vikasganiga05
Thanks a lot for opening the issue. I have captured it in this video: https://www.youtube.com/watch?v=ODKIxaSMgpU
The above video also links several other videos on JS required to get started with React.
Also, I have captured it in greater detail here: https://www.freecodecamp.org/news/react-fundamentals-for-beginners/
I hope it helps.

Oh! sorry, I have not made research. I will not make this mistake in the future. Thanks! :pray:


buliding problem

Hello, i'm a newer to this. When i try to build the operon on Linux and Windows, they came to the same problem:
"
CMake Error at CMakeLists.txt:78 (find_package):
By not providing "FindEigen3.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Eigen3", but
CMake did not find one.
Could not find a package configuration file provided by "Eigen3" with any
of the following names:
Eigen3Config.cmake
eigen3-config.cmake

Add the installation prefix of "Eigen3" to CMAKE_PREFIX_PATH or set
"Eigen3_DIR" to a directory containing one of the above files.  If "Eigen3"
provides a separate development package or SDK, be sure it has been
installed.
"

Hello,
We are using manifests to manage dependencies, which is a relatively new feature of vcpkg. First of all please make sure you are using a recent CMake (>3.20) and vcpkg. This feature should be already enabled by default but if not, please see e.g. https://stackoverflow.com/questions/66358575/c-vscode-cmake-vcpkg-manifests-are-not-enabled on how to enable it.

Thank you very much, i recheck the version: the Cmake is 3.23.1, and the vcpkg is 2022-03-30. And all the dependencies were also rechecked:

Then another problem occured when i execute the cmake oredr:
-- Running vcpkg install
Error: while loading aria-csv:
The port directory (E:\program files\vcpkg\ports\aria-csv) does not exist
Error: failed to load port from E:\program files\vcpkg\ports\aria-csv
note: updating vcpkg by rerunning bootstrap-vcpkg may resolve this failure.
-- Running vcpkg install - failed
CMake Error at E:/program files/vcpkg/scripts/buildsystems/vcpkg.cmake:850 (message):
vcpkg install failed.  See logs for more information: E:\program
files\vcpkg\operon\build\vcpkg-manifest-install.log
Call Stack (most recent call first):
D:/program files/CMake/share/cmake-3.23/Modules/CMakeDetermineSystem.cmake:124 (include)
CMakeLists.txt:8 (project)
I rerun the bootstrap-vcpkg.bat as requested but it still didn't work. Could you point out my mistake, thank you very much!

Hi,
It looks like you need to specify the ports directory, e.g. from the operon folder:
cmake -S . -B build -G "Visual Studio 16 2019" -A x64 -DCMAKE_TOOLCHAIN_FILE=..\vcpkg\scripts\buildsystems\vcpkg.cmake -DVCPKG_OVERLAY_PORTS=.\ports

You can also see the output of our windows CI pipeline (Configure CMake step): https://github.com/heal-research/operon/runs/6285374400?check_suite_focus=true


No image in the "phone" screen upon initial load.

Challenge create-visual-balance-using-the-textalign-property has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
There is no visual in the right-hand, "phone" view upon initial load.  Upon completion of the challenge, the h2, p, and links become visible.

Google

Google was founded by Larry Page and Sergey Brin while they were Ph.D. students at Stanford University.

    Larry Page
    Sergey Brin

    ```

While they are different challenges, I'm closing this is favor of your other issue thread on the same topic.


um2nc - fix for UNKNOWN VARIABLE

In the conversion from UM PP file to netcdf, most fields get a sensible long_name and units etc. Variables (fld_s08i851 and following) that are associated with CABLE/CASA-CNP (mostly carbon, nitrogen and phosphorus pools) have long_name = "UNKNOWN VARIABLE". Is there somewhere in the code that this mapping is defined? Some CABLE variables e.g. fld_s03i801 are provided with correct long_names. I am happy to help with providing appropriate long names for the unknown fields.
Rachel

ACCESS-Archiver is not standalone and I believe it depends on the umfile_utils "gadi system module" that has a stashvar.py python module containing the metadata mapping information. umfile_utils module as far as I know does not have a git repository so trying to get the gadi system module updated has been quite slow and difficult (I'm not capable of this and I don't know who is).
I inserted the required information into the stashvar.py module below (renamed to .txt). We just need someone to update and upload it to gadi.  The units are sometimes unconventional so they will have to be double checked.

Thanks, Tammas, for progressing this.
I think atm_stashvar[3851] is actually a labile carbon pool. I think the previous year surface fraction is atm_stashvar[3835].
Is there a character limit on the name of the variable? I think some would benefit from being expanded. Perhaps something we can work through together.
I agree some of the units look odd. I wouldn't expect the wood flux to be per day so we should check.

Thanks! I've just now corrected the code block above.

WOOD FLUX CARBON (CASA-CNP)

is the "(CASA-CNP)" important?
not clear to or from... may CARBON FLUX TO WOOD PRODUCTS

THIN RATIO FOR FOREST

The input ratio for thinning is 0 = remove all carbon and 1 = remove none. This is really counter intuitive. The output data for this variable might be 1 = all removed and 0 = none. I'll know for sure once I understand the run I've done with thinning enabled.

Checking units of NITROGEN DEPOSITION.
In the UM ACCESS-ESM1.5 version there is the following in atmosphere/boundary_layer/casa_um_inout.F90:
    !call um2cable_lp(nidep,nidep(1:10),annNdep ,soil%isoilm, skip)
    call um2cable_lp(nifix,nifix(1:10),annNfix ,soil%isoilm, skip)
    call um2cable_lp(pwea ,pwea(1:10) ,annPwea ,soil%isoilm, skip)
    call um2cable_lp(pdust,pdust(1:10),annPdust,soil%isoilm, skip)

    !casaflux%Nmindep = annNdep/365.0      ! gN/m2/day
    casaflux%Nminfix = annNfix/365.0      ! gN/m2/day
    casaflux%Pdep    = annPdust/365.0     ! gP/m2/day
    casaflux%Pwea    = annPwea/365.0      ! gP/m2/day

I assume this packs variables from the UM and converts the units as needed.
Nmindep is commented out so I assume that this means that the nitrogen deposition in the UM is already in gN/m2/day before it's passed to CABLE. Also, nitrogen fixating is in gN/m2/year in the UM before it's passed to CABLE and is therefore converted to /day here.
So Nitrogen deposition should be gN/m2/day and fixing should be gN/m2/year?

And I think the wood flux variables should be per year.

Not sure why the 3917-3920 variables are in kg... I couldn't find anything in the code to suggest they are. So I think they are g/m2/year.

The CMOR master_map.csv has this info for nitrogen deposition. There is no other unit conversion besides the tile average, which suggests that it's kg m-2 yr-1

The published data on fs38 are in different units again (kg/m2/s) so there must be unit conversion info missing from the master map. Multiplying the cmorized file on fs38 by 100*60*60*24 (->g/m2/day) looks like it gives the right order of magnitude of values. So I think g/m2/day is correct.

I think all of the information in the stashvar codeblock above is correct now. Someone needs to merge this into the umfile_utils module.

I've created a pull request for a workaround #3.

To add to Tammas' list above, I believe
atm_stashvar[3883] = ["SOIL ORDER", "", "", "", ""]
atm_stashvar[3887] = ["PHOSPHORUS FROM DUST", "", "g P m-2 yr-1", "", ""]
atm_stashvar[3888] = ["PHOSPHORUS FROM WEATHERING", "", "g P m-2 yr-1", "", ""]

Thanks Rachel. I have added these above and to #3

Now being addressed by a new version of the netCDF converter, https://github.com/ACCESS-NRI/um2nc-standalone/issues/24


Geometry as aguments?

Is it possible to pass the desired geometry as arguments rather than hardcoding them in the settings.ini?  I love the idea of being able to scale a specific application window using this but I'd like to just have a configuration for different scaling factors and be able to give the geometry at runtime.  I'd try to tackle this myself, but my python skills are pretty much non-existent...

Thanks for the feedback!

I'd try to tackle this myself, but my python skills are pretty much non-existent...

A bad patch is better than no patch. It gets things going! I may clean up code after pulling.


V2 new app architecture

eliminate the use of exec()
new README and CONTRIBUTING guides

Hi Ann, since you called it V2, can we now create new Dash instances in examples? like app=ProxyDash() 🙂

HI @IcToxi  I just stared working on this so I haven't tried it with with different app instances yet.  Please feel free to let me know how it goes if you get to that before I do.
The main reason for the changes in this revision was to remove the exec() to display the interactive apps which was causing an issue with auto generated component ids.
Some unrelated comments:

In this PR, I plan on adding a way for users to display the app or the code in a full screen mode as you suggested .
I made the default for your cool bad-apple app to be "app first" followed by the code -- which makes the app display  full screen. However, when I run it, I need to reduce my browser screen size to see the app run correctly.   Do you know what css is required to make the app scale so it's displayed in it's container without requiring scroll bars?

Hi @AnnMarieW,
In fact, we don't have the necessary to think about the independent environment now. Have you considered using Blueprint? Actually the new Dash instance just redefines the routing method, and the theme is just a matter of static resource configuration, and the low-level is still the same Flask server. So I believe that Blueprint can achieve this isolation effect. Or use DispatcherMiddleware.  I don't know much about combining such methods with Dash, so I'm still exploring. If you know more, please try it in this version.
I don't think it's a good idea to use the bad-apple app as your first experiment. 😁 Because its scaling will be hindered by the font settings. And I think it would be a bit too much work if you were to simply use CSS for this feature. It's better to use a component like some kind of Overlay, I'm going to find a suitable component, and will let you know when there is good news.

Hi Ann, I think react-grid-layout might be a good solution if we have a callback bound to the button to change the width of the element. And I was trying to wrap it, but I didn't successfully render the children.


Cannot install tagin.min.js from CDN

Hello, and first off thank you for the excellent little library here!
I've got a purely static site with no build, and am using Tagin straight from the CDN.  It appears that I am able to use tagin.js just fine, but when I instead switch to the minified artifact, Tagin is no longer defined:
<!doctype html>

    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1">

    <link rel="stylesheet"<EMAIL_ADDRESS>integrity="sha256-YvdLHPgkqJ8DVUxjjnGVlMMJtNimJ6dYkowFFvp4kKs=" crossorigin="anonymous">
    <link rel="stylesheet"<EMAIL_ADDRESS>
    <title>Hello, world!</title>

Hello, world!

    <form>
      <input type="text" name="tags" class="form-control tagin" value="red,green,blue">
    </form>

I'm not sure if this is just my lack of JS know-how showing or an issue w/ the minified JS, but I'd love any help figuring out how to make use of the minified library!

Hello,
thanks for using tagin, to fix the issue please update to version 2.0.1


BarcodeScannerFragment Skipped 63 Frames! when swiping between 3 fragments

Great work for porting this to use fragments, thanks!
I have a 3 tab viewpager. The barcode is fragment item number 2.
I start my application and the default fragment item 0 is shown. When I swipe to fragment item 1 the barcode scanner stored as item number 2 starts up, maybe the viewpager is caching it I'm not sure. This causes a bit of processing to occur as the camera initializes in the background and the DecodeThread starts up. Now this is all happening as fragment item 1 is showing not the barcode scanner which is fragment item 2.
Due to this there is a moment of about 1 second or ~60 frames frames skipped while switching between fragment item 0 and 1.
Here is an extract of my logcat :
04-23 23:13:59.076: I/CameraConfiguration(21035): Settable value: off
04-23 23:13:59.076: I/CameraConfiguration(21035): Supported values: [infinity, auto, macro, continuous-video, continuous-picture]
04-23 23:13:59.076: I/CameraConfiguration(21035): Settable value: auto
04-23 23:13:59.076: I/DecodeThread(21035): Hints: {POSSIBLE_FORMATS=[CODABAR, CODE_39, CODE_93, CODE_128, DATA_MATRIX, EAN_8, EAN_13, ITF, QR_CODE, RSS_14, RSS_EXPANDED, UPC_A, UPC_E], NEED_RESULT_POINT_CALLBACK=com.welcu.android.zxingfragmentlib.ViewfinderResultPointCallback@430cc730}
04-23 23:13:59.136: I/PlatformSupportManager(21035): Using implementation class com.welcu.android.zxingfragmentlib.common.executor.HoneycombAsyncTaskExecInterface of interface com.welcu.android.zxingfragmentlib.common.executor.AsyncTaskExecInterface for SDK 11
04-23 23:13:59.136: I/AutoFocusManager(21035): Current focus mode 'auto'; use auto focus? true
04-23 23:14:00.006: E/MediaPlayer(21035): Should have subtitle controller already set
04-23 23:14:00.006: I/Choreographer(21035): Skipped 63 frames!  The application may be doing too much work on its main thread.
04-23 23:14:00.536: W/MediaPlayer-JNI(21035): MediaPlayer finalized without being released
Do you know of a way to keep the barcode scanner fragment from continually stopping and starting when swiping though fragments please?
Many Thanks,
Steve

@SuperEvenSteven Try to put  into your activity layout, then fragment should works fine.


Translation with --sample leads to gluon error

I am running into the following error, only if I use --sample instead of beam search:
[2020-08-25:17:03:08:ERROR:root:exception_hook] Uncaught exception
Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/translate.py", line 269, in <module>
    main()
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/translate.py", line 46, in main
    run_translate(args)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/translate.py", line 155, in run_translate
    input_is_json=args.json_input)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/translate.py", line 237, in read_and_translate
    chunk_time = translate(output_handler, chunk, translator)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/translate.py", line 260, in translate
    trans_outputs = translator.translate(trans_inputs)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/inference.py", line 836, in translate
    batch_translations = self._translate_nd(*self._get_inference_input(translator_inputs))
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/inference.py", line 1015, in _translate_nd
    max_output_lengths))
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/mxnet/gluon/block.py", line 693, in __call__
    out = self.forward(*args)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/beam_search.py", line 666, in forward
    sample_best_hyp_indices)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/mxnet/gluon/block.py", line 693, in __call__
    out = self.forward(*args)
  File "/net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/mxnet/gluon/block.py", line 1146, in forward
    'Find all contexts = {}'.format(ctx_set))
ValueError: Find multiple contexts in the input, After hybridized, the HybridBlock only supports one input context. You can print the ele.context in the input arguments to inspect their contexts. Find all contexts = {gpu(0), cpu(0)}

I am calling Sockeye with a script such as
OMP_NUM_THREADS=1 python -m sockeye.translate \
                -i $data_sub/$corpus.pieces.src \
                -o $samples_sub_sub/$corpus.pieces.$seed.trg \
                -m $model_path \
                --sample \
                --seed $seed \
                --length-penalty-alpha 1.0 \
                --device-ids 0 \
                --batch-size 64 \
                --disable-device-locking

Sockeye and Mxnet versions:
[2020-08-25:17:03:03:INFO:sockeye.utils:log_sockeye_version] Sockeye version 2.1.17, commit 92a020a25cbe75935c700ce2f29b286b31a87189, path /net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/sockeye/__init__.py
[2020-08-25:17:03:03:INFO:sockeye.utils:log_mxnet_version] MXNet version 1.6.0, path /net/cephfs/scratch/mathmu/map-volatility/venvs/sockeye3/lib/python3.5/site-packages/mxnet/__init__.py

Details that may be relevant:

Running on a Tesla V100
OS: Ubuntu 16.04.6 LTS
the MXnet version in the CUDA 10.2 requirements file (https://github.com/awslabs/sockeye/blob/master/requirements/requirements.gpu-cu102.txt) is no longer available on Pypi. I had to install mxnet-cu102mkl==1.6.0.post0.

Is anyone familiar with this error / is this a bug in Sockeye or Mxnet/Gluon?

Hi @bricksdont!
This sounds like some tensors taking part in the hybridized computation during sampling live on another device than the GPU you are using.
My bet is that this is the problem: https://github.com/awslabs/sockeye/blob/master/sockeye/beam_search.py#L548
This array should also live on self.context. Do you want to create a PR?

Thanks Felix!
Will give this a try and make a PR if it solves the problem.

Yes, that's all it took to fix the problem. Thanks for the quick help.


Sets owner ref's on secrets

This deletes secrets when the corresponding account is deleted.

/approve


#5014 Fix MaxChainedCallsOnSameLine false positives

I've now run on the detekt codebase (with a max of 7 since there are some violations longer than 5) to verify that there aren't any more obvious false positives.
There are still interesting cases (example) with fully-qualified class names which still violate this rule; thoughts on what to do there? I haven't looked into it yet, but I'm not certain whether it will be possible to differentiate those from normal qualified expressions.

I think that we will need to use type solving.

There are still interesting cases (example) with fully-qualified class names which still violate this rule; thoughts on what to do there? I haven't looked into it yet, but I'm not certain whether it will be possible to differentiate those from normal qualified expressions.

Not sure if this example qualifies for MaxChainedCallsOnSameLine violation, but it is something that needs cleaning up anyway.

I think that we will need to use type solving.

How would that rule benefit from type solving? Could you please highlight an example?

I think that we will need to use type solving.

How would that rule benefit from type solving? Could you please highlight an example?

This was in response to this example of a fully-qualified class name which this rule currently flags. Probably this rule should ignore such cases (although this one can just be imported), but I agree that type resolution is probably necessary to determine whether a chained calls are from a fully qualified class name. I'll open a separate issue to track/discuss that.


Install web targets directly from VS

Don't know how I totally missed this🤦♂

Why closed and not merged?

@christianacca The WebBuildTools feature add a lot more than just the Web Targets so we're exploring other to get the web targets without bringing in all that excess cruft.

Thanks @shirhatti for the info


Waypoint: Local Scope and Functions (checker issue)

Challenge Waypoint: Local Scope and Functions has an issue.
User Agent is: Mozilla/5.0 (X11; CrOS x86_64 5841.83.0) AppleWebKit/537.36 (KHTML like Gecko) Chrome/36.0.1985.138 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function myFunction() {
  var myVar = "loo";
  console.log(myVar);
}
myFunction();

// run and check the console
// myVar is not defined outside of myFunction

// now remove the console.log line to pass the test

checker responds:
Remove the second console log

That's what's causing the issue, when the line is removed checker returns success:
// now remove the console.log line to pass the test

So it may be optimal to do it like this:
function myFunction() {
  var myVar = "loo";
  console.log(myVar);
}
myFunction();

// run and check the console
// myVar is not defined outside of myFunction
console.log(myVar); // now remove this whole line to pass the test

By the way, commenting second console log line produces checker error also, but this is probably known, because of instructions to remove the line to pass the test.

@rfprod:
This is a duplicate of #5559 .
Please read the following before opening a new issue :-)

Help I've Found a Bug
Search for a similar issue
Have a good day !

@ltegman: This issue can be closed

@rfprod:
This is a duplicate of #5559 .
Please read the following before opening a new issue :-)

Help I've Found a Bug
Search for a similar issue
Have a good day !

@ltegman: This issue can be closed

https://github.com/FreeCodeCamp/FreeCodeCamp/blob/staging/seed/challenges/01-front-end-development-certification/basic-javascript.json#L1902
Here is a proper regex that resolves this issue:
function myFunction[\s\S]+myFunction\(\s*\d+\s*,\s*\d+\s*\)
I'm having a bit of a hangover, so i'm browsing on my laptop. Because of this, I'm not creating a PR. If someone else wants try, keep in mind that you still need to do the escaping. ;-)

https://github.com/FreeCodeCamp/FreeCodeCamp/blob/staging/seed/challenges/01-front-end-development-certification/basic-javascript.json#L1902
Here is a proper regex that resolves this issue:
function myFunction[\s\S]+myFunction\(\s*\d+\s*,\s*\d+\s*\)
I'm having a bit of a hangover, so i'm browsing on my laptop. Because of this, I'm not creating a PR. If someone else wants try, keep in mind that you still need to do the escaping. ;-)

Closing as a duplicate. Thanks and happy coding!

Closing as a duplicate. Thanks and happy coding!

I've been reading all the fixes for this. None of them are working for me! No matter what I can't pass this one!!


[Payment card / Subscription] Implement “Add payment card” screen (UI)

Drafting cc @fabioh8010

Hey! I'm Nicolay from Callstack - will work on this flow - please assign it for me. Thanks!

Assigning @rushatgabhane as C+ since he reviewed the PR

            1. Go to settings/subscription/add-payment-card
            2. Verify that user can see add debit card
            3. Click Add payment card
            4. Verify that user will see errors with required fields
            5. Fill all required fields
            6. Verify that no errors are shown

Payment summary:

Contributor+: $250 @rushatgabhane (please request payment in NewDot)

Closing :)

$250 approved for @rushatgabhane


Pathological Behavior without Merkel Trees?

So, I have a React app where everything is an Immutable object, rooted in the same place. I am thus using I.is() to know whether a tag needs re-rendering. However, it appears that you may end up with SUM(1 ...N) comparisons, where N = level of change to the tree. So for example, if you had something like:
A -> B -> C -> D -> E
   \ -> F -> G

A is the root of the tree, it splits off into two branches. Let's assume I change E to H. As such, that entire spine must become new as such:
A' -> B' -> C' -> D' -> H
   \ -> F -> G

So, if I had my react app:
<root A = { A }>

root's render:
   <B B = { B } />
   <F F = { F } />

etc.

In the componentShouldUpdate, it is doing a I.is() for each of the attributes. Initially, I.is(A, A'), This requires traversing every node, since you won't get a difference until E != H. The left node is easy, I.is(F, F) returns and no render is needed. HOWEVER, I.is(B, B') must retraverse nearly the entire spine (4 comparisons: B == B', C == C', D == D', E !== H). This causes a re-render, and thus now next tag I.is(C, C'), which is 3 comparions. And so forth. You end up with SUM(Level(change)) comparisons.
If we implemented a lazy Merkel hash on each node, the first comparison would of course requires 5 comparisons (all the way down until it found the difference), but then the next time through merkel !== merkel and we'd be done.

I would strongly recommend against using value based equality checking (Immutable.is(a, b)) within shouldComponentUpdate and instead use reference identity based equality checking (a === b).
That's because you're doing a expensive computation (equality check) to determine if you can skip doing a expensive computation (re-render). If the equality check is slow itself, then you haven't really gained anything. However reference identity is extremely cheap and is perfectly suited for use in shouldComponentUpdate when using with React.
The issue with computing Merkel-style hashes per node when doing equality checking is that you may convert a one-pass equality algorithm into a two-pass equality algorithm, first computing and comparing hashes, and then if equal comparing data structures as well to avoid collisions. An important difference from a Merkel-tree is that Immutable data structures are designed to live in memory and operate efficiently for general purpose use, so they do not use cryptographically secure hashes.


dataset cache, repeat problem

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

I apologize, but I am having a hard time understanding what the problem is, where the problem is, and what version it affects. Please resubmit and pay attention to the issue template (https://github.com/tensorflow/tensorflow/issues/new). Please provide all the information it asks. Thank you.
(It seems that the issue just contains the template, no details were provided. I'm guessing that is a mistake, so feel free to re-open with all the details or file a new issue with them)

Hi Asimshankar;
tks for follow up. I guess I figured out the issue basically i tried to start the data with np.nan for the dataset. with cache and repeat, for some reason, the value of np.nan is changed by the program sometimes and my code can not detect the np.nan which i used it to mark the new batch as there is no such api with the repeat dataset indicates the end of one cycle.
i used additional data with the number to detect and it works now.
the issue is the repeat cache dataset changes the value of np.nan sometimes.
Jian


Please include a copy of the MIT license

Hi there. From a distribution packaging point of view it would be awesome if you could include a copy of the MIT license with your software, usually in a file called LICENSE.
Thanks!

Fixed via 8477f59.


US Version

Can I run the current version [Stratux Europe Edition 1.6r1-eu028-us]
In the US?

Sure, this image is particularly created for US use.


Provide flutter/flutter LUCI builder with fuchsia_ctl version

When I run main.star with lucicfg 1.14, this fails.  Am I doing something wrong?

Hm.  I suspect you might have manually updated the .cfg file?


docs: Add cmake extraction example

#4173 for the mkdir problem
#4175 for hiding cmake problems
Updated the comments about the env. vars
I thought about adding the merge step, but since the other examples also don't merge, I left it out.


findbugs:gui doesn't load plugins #37

GUI gets configured plugins

does not run all the integrations tests.  I will have to look into it further to get exact details.

Tried running the tests with mvn -Prun-its clean install, they fail the same way with and without my changes:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-invoker-plugin:2.0.0:run (integration-test) on project findbugs-maven-plugin: 37 builds failed. See console output above for details. -> [Help 1]
But maybe they just fail on Windows?
Or with maven-invoker-plugin 2.0.0? I updated the version because of https://issues.apache.org/jira/browse/MNG-5792
with_changes.txt
without_changes.txt

Just noticed this issue as well. Is this ready to merge?

I'll check the itests on Linux when I have the time, in a week probably.

Looked at this again. Figured out, that I needed to download the findbugs sources manually for ITests, because googlecode is unavailable. Downloaded the 1.3.9 sources. Run the tests on master (without my changes).
master.txt
[INFO] Build Summary:
[INFO]   Passed: 18, Failed: 27, Errors: 0, Skipped: 0

Most failures are off by one errors when comparing findbugs.xml and findbugsXml.xml files, e.g.:
[INFO] Building: check\pom.xml
[INFO] ..FAILED (14.3 s)
[INFO]   The post-build script did not succeed. Expression: (xdocErrors == findbugsXmlErrors). Values: xdocErrors = 33, findbugsXmlErrors = 34

Checked the files at target/it/check/target, and the difference is that findbugsXml.xml has this additional bug instance:
type='HE_SIGNATURE_DECLARES_HASHING_OF_UNHASHABLE_CLASS'
...
<LongMessage>UseOfNonHashableClassInHashDataStructure doesn't define a hashCode() method but it is used in a hashed context in UseOfNonHashableClassInHashDataStructure$UMap</LongMessage>

@gleclaire, is there some secret to running the tests? :) Or is there an unrelated problem?

I'm still willing to do the work, if something needs to be done. I just wish @gleclaire would explain, what's wrong or how to run tests or something...

It doesn't work for reports on multi-module projects

@gleclaire, as in during the site lifecycle? I'll check it out.

@VsevolodGolovanov All integration tests run now.  Can you check out the issue with GUI again to see if this issue still exists?

Verified that the issue is still present in the current 3.0.6-SNAPSHOT.
Successfully ran itests without my changes, and got 7 failures with my changes - now I can see the problems and fix them.

@VsevolodGolovanov Is this something you might be able to address this week?  I'm pretty sure same issue was raised against spotbugs-maven-plugin.  I'm releasing that but if this could be fixed soon, I can incorporate that fix.  We are pending 3.1.0 release there so I'll hold on a while in case this can be fixed.

Same issue reported here potentially [here](https://github.com/spotbugs/spotbugs-maven-plugin/issues/10}.  Would be great to get this fixed on this PR and moving this week 👍

Rebased on the current master and fixed the issue with ResourceHelper missing the resourceManager. All itests pass.

Thanks for the update


AssertionError on "Compute per-frame scales" step (frame_names is None)

The Problem
While doing a customized run with python main.py --video_file {...} --path {...} --batch_size 2 --make_video, I'm getting the following error in terminal:
************************************
****  Compute per-frame scales  ****
************************************
Traceback (most recent call last):
  File "main.py", line 13, in <module>
    dp.process(params)
  File "/home/aeverless/Desktop/content/consistent_depth/process.py", line 117, in process
    return self.pipeline(params)
  File "/home/aeverless/Desktop/content/consistent_depth/process.py", line 62, in pipeline
    valid_frames = calibrate_scale(self.video, self.out_dir, frame_range, params)
  File "/home/aeverless/Desktop/content/consistent_depth/scale_calibration.py", line 242, in calibrate_scale
    os.path.dirname(scaled_depth_fmt), ".raw"
  File "/home/aeverless/Desktop/content/consistent_depth/scale_calibration.py", line 142, in check_frames
    assert frame_names is not None
AssertionError

Setup
I'm running Pop!_OS 20.10 with the following setup:

CPU: i5-8400
GPU: GTX 1660 Super
RAM: 16GB DDR4
CUDA: 11.2.0
Graphic Driver: Nvidia 460.32.03

Context
I have followed the steps in Readme and Google Colab, successfully reproducing the result (although with lowered batch size) as shown in the demo — thus I think that the issue lies in Colmap or the way that I have it installed, which I installed by running install_colmap_ubuntu.sh.
Installing it was painful, however, when I tried to reinstall it by a different method — my system seemed to hang forever on the make -j step (the same was for building ceres-solver) and I had to cold reboot (REISUB) my system. At that point I had to scrap this method, and I stuck with the script provided with this repository.
I have also ran the setup/install shells in flownet2, and even recloned it from the flownet2-pytorch repo afterwards, trying to fix this problem I'm having.
Right now I am at the point that Colmap is installed and it seems to work fine (at least it does respond to the -h and gui opens the GUI), but frame_names still seems to be None. I have reviewed some of the code to try and figure out what the problem was in, and I figured out that it checks the frame_names from the given frame_range (which is unspecified in my case) to not be None, and based on the result of this check it then creates several folders in the hierarchical directory and proceeds the work. My current result folder looks like this:
colmap_dense  color_down  color_down_png  color_flow  color_full  depth_mc  frames.txt  R_hierarchical2_mc
R_hierarchical2_mc contains B0.1_R1.0_PL1-0_LR0.0004_BS2_Oadam folder, inside which is an empty checkpoints directory.

The Problem
While doing a customized run with python main.py --video_file {...} --path {...} --batch_size 2 --make_video, I'm getting the following error in terminal:
************************************
****  Compute per-frame scales  ****
************************************
Traceback (most recent call last):
  File "main.py", line 13, in <module>
    dp.process(params)
  File "/home/aeverless/Desktop/content/consistent_depth/process.py", line 117, in process
    return self.pipeline(params)
  File "/home/aeverless/Desktop/content/consistent_depth/process.py", line 62, in pipeline
    valid_frames = calibrate_scale(self.video, self.out_dir, frame_range, params)
  File "/home/aeverless/Desktop/content/consistent_depth/scale_calibration.py", line 242, in calibrate_scale
    os.path.dirname(scaled_depth_fmt), ".raw"
  File "/home/aeverless/Desktop/content/consistent_depth/scale_calibration.py", line 142, in check_frames
    assert frame_names is not None
AssertionError

Setup
I'm running Pop!_OS 20.10 with the following setup:

CPU: i5-8400
GPU: GTX 1660 Super
RAM: 16GB DDR4
CUDA: 11.2.0
Graphic Driver: Nvidia 460.32.03

Context
I have followed the steps in Readme and Google Colab, successfully reproducing the result (although with lowered batch size) as shown in the demo — thus I think that the issue lies in Colmap or the way that I have it installed, which I installed by running install_colmap_ubuntu.sh.
Installing it was painful, however, when I tried to reinstall it by a different method — my system seemed to hang forever on the make -j step (the same was for building ceres-solver) and I had to cold reboot (REISUB) my system. At that point I had to scrap this method, and I stuck with the script provided with this repository.
I have also ran the setup/install shells in flownet2, and even recloned it from the flownet2-pytorch repo afterwards, trying to fix this problem I'm having.
Right now I am at the point that Colmap is installed and it seems to work fine (at least it does respond to the -h and gui opens the GUI), but frame_names still seems to be None. I have reviewed some of the code to try and figure out what the problem was in, and I figured out that it checks the frame_names from the given frame_range (which is unspecified in my case) to not be None, and based on the result of this check it then creates several folders in the hierarchical directory and proceeds the work. My current result folder looks like this:
colmap_dense color_down color_down_png color_flow color_full depth_mc frames.txt R_hierarchical2_mc
R_hierarchical2_mc contains B0.1_R1.0_PL1-0_LR0.0004_BS2_Oadam folder, inside which is an empty checkpoints directory.

I'm having this same issue @aeverless, did you find a solution?

@mattbev no I didn't. I still believe that the issues lies in Colmap (because the demo was successfully done) and I'll try something else but as for now I'm hoping that the devs will answer. If I do lay my hands on the solution I'll of course post it here

@mattbev what are you running though? It might be apt to specify your setup, since it is conceivable that we just happen to be unlucky with our hardware or software choice

I left an issue here that describes my specific issue. Thanks for the update!

@aeverless I found a workaround to this issue... for me I believe the error is a result of a colmap gpu threading issue. So my workaround is explicitly disallowing gpu threading.
if you edit the dense() function in tools/colmap_processor.py and insert:
 cmd.extend(['--PatchMatchStereo.gpu_index', '0'])

before the final run(cmd), it resolves my issue. this addition adds the argument to colmap's patch_match_stereo operation
Hope this helps!

@aeverless I found a workaround to this issue... for me I believe the error is a result of a colmap gpu threading issue. So my workaround is explicitly disallowing gpu threading.
if you edit the dense() function in tools/colmap_processor.py and insert:
 cmd.extend(['--PatchMatchStereo.gpu_index', '0'])

before the final run(cmd), it resolves my issue. this addition adds the argument to colmap's patch_match_stereo operation
Hope this helps!
Edit: however, I now get a seg fault during the later Fine-Tuning step but I assume this is unrelated.

The later error in the Fine-Tuning step is an issue with the Mannequin Challenge (mc) model. If I specify --model_type "monodepth2" I don't encounter this issue

@aeverless I found a workaround to this issue... for me I believe the error is a result of a colmap gpu threading issue. So my workaround is explicitly disallowing gpu threading.
if you edit the dense() function in tools/colmap_processor.py and insert:
 cmd.extend(['--PatchMatchStereo.gpu_index', '0'])

before the final run(cmd), it resolves my issue. this addition adds the argument to colmap's patch_match_stereo operation
Hope this helps!
Edit: however, I now get a seg fault during the later Fine-Tuning step but I assume this is unrelated.

The later error in the Fine-Tuning step is an issue with the Mannequin Challenge (mc) model. If I specify --model_type "monodepth2" I don't encounter this issue

I'm glad that you've found the solution for your problem, but after trying it out I found that it doesn't work for me — the error persists as specified in the issue


Quickstart

Report
Our current "Get Started" pages are multiple pages, going through a full project. They are useful for someone who wants to set aside time to learn dvc, but maybe not so much for people who simply want to start using a particular feature of dvc as quickly as possible in their own code. There is some discussion about this in https://github.com/iterative/dvc.org/pull/4883#issuecomment-1753663079 and it has been discussed before, but maybe we need more of a "quickstart" guide that gets people up to speed as quickly and directly as possible.

Hey @dberenbaum !
Thanks for opening the discussion about this
I indeed that the docs are a great learning tool but it somehow slows things down when users might want to get things working fast.
Found that somehow docs from other experimentation platforms are more user-oriented and speed things up WRT timings, naming, and content.
Drafted to the discussion axis that I´m seeing here, feel free to reach out with whatever you might need on my side!
𝙸𝚖𝚙𝚘𝚛𝚝𝚊𝚗𝚝 𝙽𝙾𝚃𝙴:  𝚃𝚑𝚒𝚜 𝚒𝚗𝚌𝚕𝚞𝚍𝚎𝚜 𝚜𝚘𝚖𝚎 𝚗𝚘𝚝𝚎𝚜 𝚊𝚗𝚍 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚊𝚝𝚒𝚘𝚗𝚜 𝚏𝚘𝚛 𝚀𝚞𝚒𝚌𝚔𝚂𝚝𝚊𝚛𝚝 𝚠𝚛𝚝 𝙱. 𝚃𝚑𝚎 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚊𝚝𝚒𝚘𝚗𝚜 𝚊𝚛𝚎 𝚖𝚘𝚛𝚎 𝚖𝚎𝚗𝚝𝚊𝚕 𝚖𝚘𝚍𝚎𝚕𝚜 𝚝𝚑𝚊𝚝 𝙸 𝚜𝚑𝚘𝚞𝚕𝚍 𝚏𝚘𝚕𝚕𝚘𝚠, 𝚋𝚞𝚝 𝚌𝚘𝚖𝚎 𝚠𝚒𝚝𝚑𝚘𝚞𝚝 𝚊𝚗𝚢 𝚏𝚘𝚛𝚌𝚒𝚗𝚐 𝚒𝚗𝚝𝚎𝚗𝚝𝚒𝚘𝚗 𝚏𝚘𝚛 𝚝𝚑𝚎 𝚖𝚊𝚒𝚗𝚝𝚊𝚒𝚗𝚎𝚛 𝚝𝚘 𝚒𝚖𝚙𝚕𝚎𝚖𝚎𝚗𝚝 𝚝𝚑𝚎𝚖.
Docs pages
From the DOCs analysis page, I´m finding that B presents a cleaner view and 2 prioritized calls to action for a time speedup : a search bar and a Quickstart button.   As per DVC , can see the search bar up , the edit on github , the get started button ... many calls to action with the same size (getting started is somehow prioritized by color and position) without specifically knowing which is the faster track.
𝙲𝙾𝙽𝙲𝙻𝚄𝚂𝙸𝙾𝙽: 𝚝𝚑𝚎 𝚌𝚘𝚗𝚝𝚛𝚒𝚋𝚞𝚝𝚘𝚛 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚜 𝚝𝚘 𝚝𝚑𝚎 𝚖𝚊𝚒𝚗𝚝𝚊𝚒𝚗𝚎𝚛 𝚝𝚘 𝚙𝚛𝚒𝚘𝚛𝚒𝚝𝚒𝚣𝚎 𝚌𝚊𝚕𝚕 𝚝𝚘 𝚊𝚌𝚝𝚒𝚘𝚗𝚜 𝚆𝚁𝚃 𝚊 𝚏𝚊𝚜𝚝-𝚝𝚛𝚊𝚌𝚔 𝚜𝚌𝚎𝚗𝚊𝚛𝚒𝚘
QuickStart in B
If I go to Quickstart in B:

I can select CLI or Notebook. In DVC only see CLI in Getting Started. There is this logical/rational belief that might make sense that you are not prioritizing Quickstart with notebooks and DVCLive. DVCLive main page is somehow better on this and I understand it is a quickstart for experiment tracking because I know the tool - not because it is prioritized at the beginning - , and have some common structure with respect to B [1] [2]
The structure and guidance from B in the start states to solve 3 main points and goes to Ws Common questions, Including a What´s next ?  DVC literally recommends you to follow your own adventure somehow, which clearly invites you to an open-ended scenario.

𝙲𝙾𝙽𝙲𝙻𝚄𝚂𝙸𝙾𝙽: 𝚝𝚑𝚎 𝚌𝚘𝚗𝚝𝚛𝚒𝚋𝚞𝚝𝚘𝚛 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚜 𝚝𝚘 𝚝𝚑𝚎 𝚖𝚊𝚒𝚗𝚝𝚊𝚒𝚗𝚎𝚛 𝚝𝚘 𝚊𝚍𝚍 𝚊 𝚀𝚞𝚒𝚌𝚔𝚜𝚝𝚊𝚛𝚝 𝚘𝚛 𝚌𝚑𝚊𝚗𝚐𝚎 𝚝𝚑𝚎 𝚞𝚗𝚍𝚎𝚛𝚕𝚢𝚒𝚗𝚐 𝚍𝚢𝚗𝚊𝚖𝚒𝚌𝚜 𝚘𝚏 𝚝𝚑𝚎 𝙶𝚎𝚝 𝚂𝚝𝚊𝚛𝚝𝚎𝚍 𝚜𝚎𝚌𝚝𝚒𝚘𝚗. 𝙸𝚏 𝚊 𝚀𝚞𝚒𝚌𝚔𝚜𝚝𝚊𝚛𝚝 𝚜𝚎𝚌𝚝𝚒𝚘𝚗 𝚒𝚜 𝚊𝚍𝚍𝚎𝚍, 𝚝𝚑𝚎 𝚌𝚘𝚗𝚝𝚛𝚒𝚋𝚞𝚝𝚘𝚛 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚜 𝚝𝚘 𝚊𝚟𝚘𝚒𝚍 𝚘𝚙𝚎𝚗-𝚎𝚗𝚍𝚎𝚍 𝚜𝚌𝚎𝚗𝚊𝚛𝚒𝚘𝚜.
"Quickstart" with DVC
Non existing page is named as such. Glimpsing that DVCLive is playing an important role here, but I only discover it when I go to the Studio page, to experiments and model registry. Also known that there is a fast track for using the extension, but not presented from the start.
𝙲𝙾𝙽𝙲𝙻𝚄𝚂𝙸𝙾𝙽: 𝚝𝚑𝚎 𝚌𝚘𝚗𝚝𝚛𝚒𝚋𝚞𝚝𝚘𝚛 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚜 𝚝𝚘 𝚝𝚑𝚎 𝚖𝚊𝚒𝚗𝚝𝚊𝚒𝚗𝚎𝚛 𝚝𝚘 𝚝𝚑𝚒𝚗𝚔 𝚊𝚋𝚘𝚞𝚝 𝚝𝚑𝚎 𝚛𝚘𝚕𝚎 𝙳𝚅𝙲𝙻𝚒𝚟𝚎 (or even DVC API ?) 𝚝𝚊𝚔𝚎𝚜 𝚒𝚗 𝚊 𝚏𝚊𝚜𝚝-𝚝𝚛𝚊𝚌𝚔 𝚜𝚌𝚎𝚗𝚊𝚛𝚒𝚘, 𝚊𝚗𝚍 𝚙𝚛𝚒𝚘𝚛𝚒𝚝𝚒𝚣𝚎 𝚍𝚘𝚌𝚜 𝚊𝚌𝚌𝚘𝚛𝚍𝚒𝚗𝚐𝚕𝚢.
Prioritizing Integrations
Integrations is a section with its own importance in docs in B , thing that has proven important when including a tool in a fast-track scenario. In DVC, I have to go to DVCLive > ML Frameworks
𝙲𝙾𝙽𝙲𝙻𝚄𝚂𝙸𝙾𝙽: 𝚝𝚑𝚎 𝚌𝚘𝚗𝚝𝚛𝚒𝚋𝚞𝚝𝚘𝚛 𝚛𝚎𝚌𝚘𝚖𝚖𝚎𝚗𝚍𝚜 𝚝𝚘 𝚝𝚑𝚎 𝚖𝚊𝚒𝚗𝚝𝚊𝚒𝚗𝚎𝚛 𝚝𝚘 𝚛𝚎𝚌𝚘𝚐𝚗𝚒𝚣𝚎 𝚒𝚗𝚝𝚎𝚐𝚛𝚊𝚝𝚒𝚘𝚗𝚜 𝚊𝚜 𝚙𝚊𝚛𝚝 𝚘𝚏 𝚝𝚑𝚎 𝚏𝚊𝚜𝚝-𝚝𝚛𝚊𝚌𝚔 𝚜𝚌𝚎𝚗𝚊𝚛𝚒𝚘, 𝚊𝚗𝚍 𝚙𝚛𝚒𝚘𝚛𝚒𝚝𝚒𝚣𝚎 𝚊𝚌𝚌𝚎𝚜𝚜 𝚊𝚌𝚌𝚘𝚛𝚍𝚒𝚗𝚐𝚕𝚢.

Current user journey for Model Registry
The goal of the following two comments is to draft two diagrams comparing the accesibility (number of clicks) that a user must fo to find the useful code snippet for their project within the context of Model Registry.
Initial page : docs
Final page : docs page including code snippet
B analysis
Results : 1 or two clicks to the code
Diagram that shows the story for a user accessing model registry in B.
The user can access from QuickStart and from Search bar , in 1 or 2 clicks they are in Model registry able to access the code snippet. Beyond the code, the user can find links to getting Started video and End-to-End guide. The Model Registry page seems to be the place to make a decision , including code and other links
flowchart TD

    SE[Search Bar] -->|Model Registry| B(Model Registry)

    DG[Developer Guide]
    A[QuickStart] -->|Quick Start Docs| B(Model Registry)
    API[API Reference]
    TUT[Tutorials]

    B --> C{Let me think}
    C -->|How it works| D[CODE SNIPPET]
    C -->|Get Started| E[Video Link Quickstart]
    C -->|Walkthrough| F[End-to-end Guide]

DVC Analysis
Results : 4 or 5 clicks to the code , knowing the path, and TBH , I really don´t know if I´m in the right place
The user must search in the bar, then go to the Model Registry Info, then go to Studio Model Registry, then to Add models to registry, then to Live.log_artifact. Please let me know if there is a quicker path for these things.  I have add a "let me think" in DVC in each moment that I have to stop to go where I was supposed to, I felt I had some sort of paralysis-for-analysis user experience.
flowchart TD

    SE[Search Bar Lateral] -->|Model Registry| B(Model Registry)

    UG[User Guide]
    GS[Get Started]
    UC[Use Cases] -->|Model Registry| B(Model Registry Intro)
    CR[Command References]
    I[Info]

   B --> C{Let me think}
   C --> SM(Studio Model registry)
   C --> D(On top of Git)
   C --> E(Versions)
   C --> SM(Studio Model registry)

   SM --> SM2{Let me think}

   SM2 --> F(Add models to registry)
   SM2 --> H(Register New Versions)
   SM2 --> K(Assign stages to models)
   SM2 --> L(...)

   F --> SM3{Let me think}

   SM3 --> Dvc(dvc.yaml)
   SM3 --> Dvc2(dvclive)
   SM3 --> Dvc3(Live.log_artifact CODE SNIPPET)

Hey. Hope this can visualize how many steps the user might have to take into account to reach the desired destination.
I learned some things about diagraming thanks to this discussion, so thanks for the opportunity to contribute!


Inference error

$ python -m lrm.inferrer --model_name openlrm-base-obj-1.0 --source_image ./assets/sample_input/owl.png --export_video
======== Loaded model from checkpoint ========
Traceback (most recent call last):
File "/usr/local/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/local/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/mnt/bn/dika/mlx/workspace/project/OpenLRM/lrm/inferrer.py", line 265, in
inferrer.infer(
File "/mnt/bn/dika/mlx/workspace/project/OpenLRM/lrm/inferrer.py", line 210, in infer
results = self.infer_single(
File "/mnt/bn/dika/mlx/workspace/project/OpenLRM/lrm/inferrer.py", line 155, in infer_single
planes = self.model.forward_planes(image, source_camera)
File "/mnt/bn/dika/mlx/workspace/project/OpenLRM/lrm/models/generator.py", line 91, in forward_planes
planes = self.transformer(image_feats, camera_embeddings)
File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/mnt/bn/dika/mlx/workspace/project/OpenLRM/lrm/models/transformer.py", line 126, in forward
x = layer(x, image_feats, camera_embeddings)
File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/mnt/bn/dika/mlx/workspace/project/OpenLRM/lrm/models/transformer.py", line 79, in forward
x = x + self.self_attn(before_sa, before_sa, before_sa)[0]
File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/usr/local/lib/python3.10/site-packages/torch/nn/modules/activation.py", line 1160, in forward
return torch._native_multi_head_attention(
TypeError: _native_multi_head_attention(): argument 'qkv_bias' (position 7) must be Tensor, not NoneType

Meat with the same error.. any update or clues?

update torch>=2.1.0

solved


Skip test if PHP integer is too small (i.e. 32-bit PHP environment)

Fixes #147.

Coverage remained the same at 94.279% when pulling c6e9258a1fe2dd137c9b305ed9b09e4cfff572da on RadiusWebTools:skip-broken-tests-on-x86-php into df4610157c05e42569955ad4da91bd6b61de53c2 on mikey179:master.

Coverage remained the same at 94.279% when pulling c6e9258a1fe2dd137c9b305ed9b09e4cfff572da on RadiusWebTools:skip-broken-tests-on-x86-php into df4610157c05e42569955ad4da91bd6b61de53c2 on mikey179:master.

Haven't run a 32bit version of PHP for a long time. :) Thanks for catching this!