Fixing a few bugs in ReadConverter

Adding a more comprehensive test for ReadConverter
Added gat-tools-java as a dependency to access the Read->SAMRecord converter

Coverage increased (+1.3%) to 25.76% when pulling 0a20aa74808741c93cad2fbb14f7b56c63df63dd on lbergelson:lb_ReadConverterFix into d8b04b5d3a12f97bb1cc7bd6a68638c43821b1cf on googlegenomics:master.

Nice!  Good test makes me happy.

@wbrockman You merged, did you want me to add that comment and fix the spacing?

Yes please - sorry, crossing up my different review workflows.
On Tue, Mar 31, 2015 at 10:26 AM, Louis Bergelson<EMAIL_ADDRESS>wrote:

@wbrockman https://github.com/wbrockman You merged, did you want me to
add that comment and fix the spacing?
—
Reply to this email directly or view it on GitHub
https://github.com/googlegenomics/dataflow-java/pull/45#issuecomment-88108597
.

This line is my favorite :)
    10  @RG     ID:0    SM:Hi,Momma!    LB:whatever     PU:me   PL:ILLUMINA


0.1.3 Orchestration using cron

Source  - #27
The Problem - In the absence of an automated task scheduler like cron, essential tasks that need to run periodically, such as data pulls and backups, log file rotations, or system health checks will have to be triggered manually.

#50

@brmhastra Thank you for raising a PR but lets first take some time to explain to the community why CRON is useful feature and after development how it will work

@brmhastra we have to demo how to automate google colab notebook with local data platform


Add new operator controller MachineSet

Which issue this PR addresses:

This was previously in https://github.com/Azure/ARO-RP/pull/1532 but after some issues squashing / rebasing, went ahead and re-opened as a new PR. Most comments there have been addressed.
User Story 8749435 - cluster upgrade is blocked if there is only one worker VM, because the ingress and console operators are Degraded. We need to ensure that there are at least two worker replicas in place to avoid this.
What this PR does / why we need it:
This controller watches MachineSet objects for changes, and tallies worker replicas. If Spec.Replicas inside of an object matching our infraID is reduced to where total replicas is less than three, the operator will revert the change / scale the MachineSet back up.

Test plan for issue:

Added unit testing coverage for the Reconciler.
Added to E2E to confirm that the behavior is as expected.
Tested the controller in a development cluster with a custom MachineSet, to confirm that no scaling action was taken.
Confirmed that the controller only reconciles on changes to MachineSet objects.

Is there any documentation that needs to be updated for this PR?
Probably just README.md, which is part of this PR.

@cadenmarchese there were conflicts, beucase I merged another PR related to the operator. I rebased and fixed conflcits so we can merge while you are on leave. I hope you don't mind.


Need help understanding how the plugin works

Hey, so i want to make a plugin of my own which uses the graph view but i've got no help from the discord aside from someone linking me to this plugin. I would love to understand how this plugin and its renderer works so that i can use this knowledge in my plugin aswell.
Any help is much appreciated. Thank you!

Well, the first thing I'll say is that the graph view was not designed to be modified. The solution I've used is a hacky one. Essentially we find the renderer that Obsidian makes by waiting a little bit after the graph view opens, and then add things to draw on top of the layer they make. Feel free to use any of my source code as long as you keep the same license and give credit here, and let me know if you have any specific questions about the code.


Multiple Docker Build ERROR

Having a few issues with my current build of DefectDojo and am looking to take the master branch to do some testing on the newer version to see if its my build or bugs and having a few issues with the docker build.
Bug description
Seeing a range of ERRORS in the Docker build process.
ERROR: django-auditlog 0.4.5 has requirement python-dateutil==2.6.0, but you'll have python-dateutil 2.8.0 which is incompatible.                                                                                ERROR: django-overextends 0.4.3 has requirement django<2.0,>=1.8, but you'll have django 2.2.4 which is incompatible.qq
ERROR: django-auditlog 0.4.5 has requirement python-dateutil==2.6.0, but you'll have python-dateutil 2.8.0 which is incompatible. ERROR: django-overextends 0.4.3 has requirement django<2.0,>=1.8, but you'll have django 2.2.4 which is incompatible.
A bunch of warnings are also returned but not sure if these are going to cause an issue at this stage.
warning @yarn_components/mocha ><EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade warning @yarn_components/mocha > glob ><EMAIL_ADDRESS>please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js warning @yarn_components/mocha > glob ><EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
Steps to reproduce

git clone  (master)
docker-compose build

Expected Behaviour
Build without errors.
Environment

Docker version 19.03.5-ce on Arch

Console Logs
A subset of console logs in case it might be useful:
Processing /tmp/wheels/pyasn1-0.4.8-py2.py3-none-any.whl                                                                                Processing /tmp/wheels/smmap2-2.0.5-py2.py3-none-any.whl                                                                                Processing /tmp/wheels/zipp-1.1.0-py2.py3-none-any.whl                                                                                Processing /tmp/wheels/Jinja2-2.11.1-py2.py3-none-any.whl                                                                                Processing /tmp/wheels/MarkupSafe-1.1.1-cp35-cp35m-manylinux1_x86_64.whl                                                                                Building wheels for collected packages: django-custom-field, django-tastypie-swagger                                                                                Building wheel for django-custom-field (setup.py): started                                                                                Building wheel for django-custom-field (setup.py): finished with status 'done'                                                                                Created wheel for django-custom-field: filename=django_custom_field-2.9-py3-none-any.whl size=9826 sha256=0097bb6b799b2f6b6835c19dc3bf714f6c3293875da5e12031a493ffadc7fefe                                                             Stored in directory: /tmp/pip-ephem-wheel-cache-c_rqdfwj/wheels/6f/59/9e/9b3c0e3b0c4f98b9a324754773e75c4185bcd9cb2b11cc125d                                                                                Building wheel for django-tastypie-swagger (setup.py): started                                                                                Building wheel for django-tastypie-swagger (setup.py): finished with status 'done'                                                                                Created wheel for django-tastypie-swagger: filename=django_tastypie_swagger-0.1.4-py3-none-any.whl size=170084 sha256=2baeb808e6cc45788adfbf4db61f8cd3fd76ac6802d3a5d8d34e812626bbc606                                                 Stored in directory: /tmp/pip-ephem-wheel-cache-c_rqdfwj/wheels/e7/aa/b1/0c68c8e17e28e977dde43efe040c92038d55f7b7e669b340e1                                                                                Successfully built django-custom-field django-tastypie-swagger                                                                                ERROR: django-auditlog 0.4.5 has requirement python-dateutil==2.6.0, but you'll have python-dateutil 2.8.0 which is incompatible.                                                                                ERROR: django-overextends 0.4.3 has requirement django<2.0,>=1.8, but you'll have django 2.2.4 which is incompatible.                                                                                Installing collected packages: numpy, six, asteval, webencodings, bleach, bleach-whitelist, billiard, vine, pytz, zipp, importlib-metadata, amqp, kombu, celery, coverage, defusedxml, django-celery-results, python-dateutil, sqlpars e, Django, django-jsonfield, django-auditlog, django-custom-field, django-dbbackup, django-environ, django-filter, django-appconf, pilkit, django-imagekit, django-multiselectfield, django-overextends, django-polymorphic, djangores tframework, simplejson, MarkupSafe, jinja2, coreschema, uritemplate, urllib3, certifi, chardet, idna, requests, itypes, coreapi, openapi-codec, django-rest-swagger, django-slack, django-tagging, django-taggit, django-taggit-serial izer, django-tastypie-swagger, python-mimeparse, django-tastypie, django-watson, gunicorn, html2text, humanize, pyjwt, pycparser, cffi, asn1crypto, cryptography, oauthlib, requests-oauthlib, pbr, requests-toolbelt, jira, lxml, Mar kdown, mysqlclient, pandas, pdfkit, Pillow, psycopg2-binary, pygments, python-nmap, redis, sqlalchemy, meld3, supervisor, uWSGI, vobject, whitenoise, titlecase, python3-openid, social-auth-core, social-auth-app-django, ecdsa, futu re, pyasn1, rsa, Python-jose, smmap2, gitdb2, gitpython                                                                                Successfully installed Django-2.2.4 Markdown-3.1.1 MarkupSafe-1.1.1 Pillow-6.0.0 Python-jose-3.0.1 amqp-2.5.2 asn1crypto-1.3.0 asteval-0.9.14 billiard-<IP_ADDRESS> bleach-3.1.0 bleach-whitelist-0.0.10 celery-4.3.0 certifi-2019.11.28 cf fi-1.14.0 chardet-3.0.4 coreapi-2.3.3 coreschema-0.0.4 coverage-4.5.3 cryptography-2.7 defusedxml-0.6.0 django-appconf-1.0.3 django-auditlog-0.4.5 django-celery-results-1.1.1 django-custom
Collecting python3-openid>=3.0.10; python_version >= "3.0" (from social-auth-core>=1.2.0->social-auth-app-django==3.1.0->-r ./requirements.txt (line 52))                                                                              Collecting pyasn1>=0.1.3 (from rsa->Python-jose==3.0.1->-r ./requirements.txt (line 53))                                                                                Collecting smmap2>=2.0.0 (from gitdb2>=2.0.0->gitpython>=2.1.13->-r ./requirements.txt (line 54))                                                                                Collecting zipp>=0.5 (from importlib-metadata>=0.18; python_version < "3.8"->kombu<5.0,>=4.4.0->celery==4.3.0->-r ./requirements.txt (line 5))                                                                                Collecting jinja2 (from coreschema->coreapi>=2.3.0->django-rest-swagger==2.1.2->-r ./requirements.txt (line 18))                                                                                Collecting MarkupSafe>=0.23 (from jinja2->coreschema->coreapi>=2.3.0->django-rest-swagger==2.1.2->-r ./requirements.txt (line 18))                                                                                Building wheels for collected packages: django-custom-field, django-tastypie-swagger                                                                                Building wheel for django-custom-field (setup.py): started                                                                                Building wheel for django-custom-field (setup.py): finished with status 'done'                                                                                Created wheel for django-custom-field: filename=django_custom_field-2.9-cp35-none-any.whl size=9825 sha256=c508aec034b47ba2b7387d05f686452e5eff12385f9074086106da627d41f713   Stored in directory: /tmp/pip-ephem-wheel-cache-dea559s0/wheels/1f/80/54/96282fa201cd4a45b500d621356ee348d43cebf1133dc7b797   Building wheel for django-tastypie-swagger (setup.py): started   Building wheel for django-tastypie-swagger (setup.py): finished with status 'done'   Created wheel for django-tastypie-swagger: filename=django_tastypie_swagger-0.1.4-cp35-none-any.whl size=170085 sha256=bbdcac28f48b0188ebfb42a6250d302db9158b5dca1720196a6502ef448c0789   Stored in directory: /tmp/pip-ephem-wheel-cache-dea559s0/wheels/db/47/ff/9112c9c8a2c16fdb7d836153822ba59216b6cd6dab80bee002 Successfully built django-custom-field django-tastypie-swagger ERROR: django-auditlog 0.4.5 has requirement python-dateutil==2.6.0, but you'll have python-dateutil 2.8.0 which is incompatible. ERROR: django-overextends 0.4.3 has requirement django<2.0,>=1.8, but you'll have django 2.2.4 which is incompatible. Installing collected packages: six, numpy, asteval, webencodings, bleach, bleach-whitelist, zipp, importlib-metadata, vine, amqp, kombu, pytz, billiard, celery, coverage, defusedxml, django-celery-results, python-dateutil, sqlpars e, Django, django-jsonfield, django-auditlog, django-custom-field, django-dbbackup, django-environ, django-filter, django-appconf, pilkit, django-imagekit, django-multiselectfield, django-overextends, django-polymorphic, uritempla te, itypes, MarkupSafe, jinja2, coreschema, certifi, idna, urllib3, chardet, requests, coreapi, openapi-codec, simplejson, djangorestframework, django-rest-swagger, django-slack, django-tagging, django-taggit, django-taggit-serial izer, django-tastypie-swagger, python-mimeparse, django-tastypie, django-watson, gunicorn, html2text, humanize, pbr, asn1crypto, pycparser, cffi, cryptography, pyjwt, oauthlib, requests-toolbelt, requests-oauthlib, jira, lxml, Mar kdown, mysqlclient, pandas, pdfkit, Pillow, psycopg2-binary, pygments, python-nmap, redis, sqlalchemy, meld3, supervisor, uWSGI, vobject, whitenoise, titlecase, python3-openid, social-auth-core, social-auth-app-django, ecdsa, futu re, pyasn1, rsa, Python-jose, smmap2, gitdb2, gitpython Successfully installed Django-2.2.4 Markdown-3.1.1 MarkupSafe-1.1.1 Pillow-6.0.0 Python-jose-3.0.1 amqp-2.5.2 asn1crypto-1.3.0 asteval-0.9.14 billiard-<IP_ADDRESS> bleach-3.1.0 bleach-whitelist-0.0.10 celery-4.3.0 certifi-2019.11.28 cf fi-1.14.0 chardet-3.0.4 coreapi-2.3.3 coreschema-0.0.4 coverage-4.5.3 cryptography-2.7 defusedxml-0.6.0 django-appconf-1.0.3 django-auditlog-0.4.5 django-celery-results-1.1.1 django-custom-field-2.9 django-dbbackup-3.2.0 django-en viron-0.4.5 django-filter-2.0.0 django-imagekit-4.0.2 django-jsonfield-1.4.0 django-multiselectfield-0.1.8 django-overextends-0.4.3 django-polymorphic-2.0.1 django-rest-swagger-2.1.2 django-slack-5.13.0 django-tagging-0.4.6 django -taggit-1.2.0 django-taggit-serializer-0.1.7 django-tastypie-0.14.2 django-tastypie-swagger-0.1.4 django-watson-1.5.2 djangorestframework-3.9.4 ecdsa-0.15 future-0.18.2 gitdb2-2.0.6 gitpython-3.0.7 gunicorn-19.9.0 html2text-2018.1 .9 humanize-0.5.1 idna-2.8 importlib-metadata-1.5.0 itypes-1.1.0 jinja2-2.11.1 jira-2.0.0 kombu-4.6.7 lxml-4.3.3 meld3-2.0.0 mysqlclient-1.4.2 numpy-1.18.1 oauthlib-3.1.0 openapi-codec-1.3.2 pandas-0.25.3 pbr-5.4.4 pdfkit-0.6.1 pi lkit-2.0 psycopg2-binary-2.8.2 pyasn1-0.4.8 pycparser-2.19 pygments-2.4.2 pyjwt-1.7.1 python-dateutil-2.8.0 python-mimeparse-1.6.0 python-nmap-0.6.1 python3-openid-3.1.0 pytz-2019.1 redis-3.2.1 requests-2.22.0 requests-oauthlib-1. 3.0 requests-toolbelt-0.9.1 rsa-4.0 simplejson-3.17.0 six-1.14.0 smmap2-2.0.5 social-auth-app-django-3.1.0 social-auth-core-3.2.0 sqlalchemy-1.3.13 sqlparse-0.3.0 supervisor-4.0.3 titlecase-0.12.0 uWSGI-2.0.18 uritemplate-3.0.1 ur llib3-1.25.3 vine-1.3.0 vobject-<IP_ADDRESS> webencodings-0.5.1 whitenoise-5.0.1 zipp-1.1.0 Removing intermediate container fab4716f29bd

During my build I see such kind of warnings too, but the build is successful.
ERROR: django-auditlog 0.4.5 has requirement python-dateutil==2.6.0, but you'll have python-dateutil 2.8.0 which is incompatible.
ERROR: django-overextends 0.4.3 has requirement django<2.0,>=1.8, but you'll have django 2.2.4 which is incompatible.

As I can see the first error is from other project: https://github.com/jjkester/django-auditlog/issues/162

There is a good explanation about second error: https://stackoverflow.com/questions/50507194/mezzanine-4-2-3-has-requirement-django1-11-1-8-but-youll-have-django-2-0-2

there are often this kind of errors but they don't seem to cause issue in the application. Ideally though , dependencies version should be finely tuned to avoid those as it could cause issues.


The snapshot

How to specify snapshot when starting with Docker

@satonz i'm not sure, if this is possible. we use initcontainer with import snapshot part. you may try to run alternately docker containers with custom entrypoint key -n and --env SNAPSHOTURL="https://fil-chain-snapshots-fallback.s3.amazonaws.com/mainnet/minimal_finality_stateroots_latest.car" and then run as usual with sync state

Hi, @satonz
--env SNAPSHOTURL="https://fil-chain-snapshots-fallback.s3.amazonaws.com/mainnet/minimal_finality_stateroots_latest.car"
look at README.md . We had update it


Avoid logging warnings when handler does not support metadataproviders

Discovered by https://github.com/vapor/console-kit/pull/177

We were setting the metadata provider "always" so even without implementing the right factory function it would have tried to set a provider during logger creation. Since this happens in code which did not adopt metadata providers, it ends up as logging a warning for creating loggers.
This can be very noisy, e.g. a plain new vapor project would be like this:
[1357/1357] Linking Run
Build complete! (47.19s)
[ WARNING ] Attempted to set metadataProvider on ConsoleLogger that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.
[ NOTICE ] Server starting on http://<IP_ADDRESS>:8080
[ WARNING ] Attempted to set metadataProvider on ConsoleLogger that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.
[ WARNING ] Attempted to set metadataProvider on ConsoleLogger that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.
[ INFO ] GET / [request-id: AAF6BC94-109E-4492-B061-F26B4060DC9F]
[ WARNING ] Attempted to set metadataProvider on ConsoleLogger that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.
[ WARNING ] Attempted to set metadataProvider on ConsoleLogger that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.
[ WARNING ] Attempted to set metadataProvider on ConsoleLogger that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.

The change in https://github.com/apple/swift-log/commit/6f9b9d4cd7cce207b5256adb5ac6fc4c7f6e9040 here makes it such that the only way to make use of metadata providers is to make sure to bootstrap with a factory that accepts one. OR you can always set it manually later on, if you know your log handler will support it.
This avoids the noisy logging.
The warning logging still would happen here if you do yourself
log.metadataProvider = ....

on a log handler that does not support it. I think this is fine (?), but it would happen for every instance of such call...
I could be convinced to remove the warning entirely if we think this is still risking it too much.

For context, the origin of the warning is this:
extension LogHandler {
    /// Default implementation for `metadataProvider` which defaults to `nil`.
    /// This default exists in order to facilitate source-compatible introduction of the `metadataProvider` protocol requirement.
    public var metadataProvider: Logger.MetadataProvider? {
        get {
            nil
        }
        set {
            self.log(level: .warning, message: "Attempted to set metadataProvider on \(Self.self) that did not implement support for them. Please contact the log handler maintainer to implement metadata provider support.", metadata: nil, source: "Logging", file: #file, function: #function, line: #line)
        }
    }
}

so if someone tried to set a provider on a logger whose handler does not support it, the intent was to give a warning.

cc @tim0x

another option is to try and emit once n first occurrence. this will have done performance cost but maybe static atomic check is not too bad. better maybe we only do it #if DEBUG

Only doing it in debug is definitely a good idea as well! Perhaps they and the atomic approach combined would be the best 🤔

Sounds like a plan, done: https://github.com/apple/swift-log/pull/252/commits/3efaf3773776995bdc911893138745bec3aeaeb4


download.jitsi.org down

This Issue tracker is only for reporting bugs and tracking code related issues.
Before posting, please make sure you check community.jitsi.org to see if the same or similar bugs have already been discussed. General questions, installation help, and feature requests can also be posted to community.jitsi.org.
Description

Jitsi Download Page in down since yesterday
Is it down right now
Current behavior

Download page and repos can't be reached
Expected Behavior

Download page and repos reachable as usual
Steps to reproduce

Go to https://download.jitsi.org/jitsi/

This is known and the certificate had expired. We are working on it. And will let you know once it is fixed.

All issues had been fixed and download.jitsi.org is back.

Seems to be down again.


Bind for <IP_ADDRESS>:50070: unexpected error Permission denied

I catch the error while following setup guide for Data Hub in datahub-docker-v2-setup.md
#docker run -ti --publish <IP_ADDRESS>:50070:50070 --name hdfs --hostname hdfs --net dev-net datahub run-hdfs C:\Program Files\Docker\Docker\Resources\bin\docker.exe: Error response from daemon: driver failed programming external connectivity on endpoint hdfs (7bf60e8a820af7157f693a0da72cd0b6cadb534d0ebb2c3f704e8ef21668a851): Error starting userland proxy: Bind for <IP_ADDRESS>:50070: unexpected error Permission denied.
It is because of port 50070 is reserved by windows.
#netsh interface ipv4 show excludedportrange protocol=tcp

Протокол tcp Диапазоны исключения портов

Начальный порт    Конечный порт
----------    --------
     49708       49807
     49808       49907
     49908       50007
     50008       50107
     50108       50207
     50208       50307
     50308       50407
     50542       50641

You could use this command like work around.
docker run -ti --publish <IP_ADDRESS>:9870:50070 --name hdfs --hostname hdfs --net dev-net datahub run-hdfs
But you should use <IP_ADDRESS>:9870  instead of <IP_ADDRESS>:50070 to access Apache Hadoop User Interface
Windows 10 Pro
Version: 1809
Build: 17763.195

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.

So After a fun windows update, I'm having this problem @.@  Docker commands worked to host my containers before update, now permission denied uhhgg.


DuckDNS provider

Hello, feel free to post here your work on the DuckDNS integration in libdns
We can then create a repository and give you access to it 👍

I wrote an initial implementation here: https://github.com/francislavoie/duckdns, I haven't tested it yet. WIP.
FYI @mholt if you wanted to take a quick peek.

Alright, it's ready to go! Works for me:
$ xcaddy build --with github.com/francislavoie/caddy-duckdns

Caddyfile:
example.duckdns.org {
        tls {
                dns duckdns {env.DUCKDNS_API_TOKEN}
        }

        root * /usr/share/caddy
        file_server
}

Awesome!! Thanks Francis! This will help a lot of people.
I went ahead and made the repos for your duckdns code. (Make sure to update any import paths that use the francislavoie account to use the new libdns/caddy-dns orgs.)
I believe that closes this up.

Alright it's pushed
xcaddy build --with github.com/caddy-dns/duckdns


Deleting caracters files and mods folders, don't delete them in-game...

Describe your problem here.
(Unnecessary warning that this text was made with google translator)
I don't know if this fits more in "Bug" or "Help Wanted", but I'll post on this in "Help Wanted" because I think it makes more sense.
I wanted to delete a character files because when I tried to use the character in a song or in the character editor, "null object refence" appeared, so I deleted the .png .xml and .json files, but when I compiled the game, it was still in the character editor, (and obviously, it was still giving the "null object refence" error and closing my game). It also happened to me before with a mod folder.
I deleted it, but the mod was still in the game. But at least I was able to disable the mod using the "-debug" thing. But I still wanted to know how to permanently remove files and folders from the game, and only reappear in the game when the files are actually in the "example-mods" folder!
Here are some screenshots for you to understand the problem better.

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Not on the main part of the engine, but I changed some files in the Sky Reman. mod folder before deleting it, (2 .json files, 2 .ogg and 1 png). Nothing too important I think.

Delete them from the export folder

you have to delete them in the export folder
but this is annoying tbh


Fix Offering API, provide ID of template object

@scytacki, it fixes the problem you've noticed on learn.staging.
It was caused by Offering API and a mismatch between ExternalActivity ID and Activity ID (690 vs 898):

I've fixed API to always return template model ID as that's what is expected everywhere else.

But the same class could have external activities that don't have templates, external activities whose template is a sequence, and external activities whose template is an activity. So it seems likely there will be id conflicts, if I understand this right.

I've looked at it more and I now see my comment above isn't really relevant. I'm looking at it more to get a better grasp of this.  It does seem like your fix will work.

If the offering runnable is an ExternalActivity that has no template, is it necessary to include this activities section?  If that isn't needed then the id here would aways be a Activity id instead of sometimes a ExternalActivity id.

I see it is used here: https://github.com/concord-consortium/portal-pages/blob/d4c0fe0b7500566e7d29d9a32b3e22d01c7b0884/src/library/components/common/offering-progress/index.js#L59-L63 which looks like it would be used even in the case of an external activity that has no template. It looks like that code also assumes that all activities have reports.  But in the case of an external activity without a template there is no report at the activity or student level. There is "binary" progress for these external activities without templates, but that doesn't seem to be working on the recent activity page on staging.

I'm going to go ahead and merge this in so staging is less broken. But we still need to address this issue of ExternalActivities without templates. And ideally I'd like to improve the JSON structure so it is clear the data returned in this case does not correspond to a Activity in the Portal.

Here is a PT bug for this: https://www.pivotaltracker.com/story/show/157700766


Two launch configurations with the same preLaunchTask trigger "already running" warning

VSCode Version: 1.38.1
OS Version: OS X 10.14.6

Steps to Reproduce:
With the following .vscode/launch.json:
{
  "version": "0.2.0",
  "compounds": [
    {
      "name": "all",
      "configurations": ["web-server", "web-ui"]
    }
  ],
  "configurations": [
    {
      "name": "web-server",
      "type": "node",
      "request": "attach",
      "port": 20006,
      "restart": true,
      "preLaunchTask": "npm: dev-web"
    },
    {
      "name": "web-ui",
      "type": "chrome",
      "request": "launch",
      "url": "http://localhost:20005/",
      "timeout": 20000,
      "webRoot": "${workspaceFolder}/web",
      "preLaunchTask": "npm: dev-web"
    }
  ]
}

Launching the "all" configuration works properly, but presents the following:

@isidorn a task is considered active after we fire the task active event. It looks like the launch configurations are not waiting for this and are just firing off the launch configurations in parallel. I'm not sure there's a good solution from the tasks side.

Yes, debug is just launching debug configurations in parallel which just run preLaunchTasks in parallel.
It is very rare that both configuratinos depend on the same task.
The only way to work around this would be if the taskService exposes a flag to suppress this warning and debug would pass true to this flag.
However I think this is pretty minor and I think we can ignore for now
@alexr00 leaving up to you to decide what to do here


plans: Precisely determine min- and max-widths.

This takes some of the eyeballing, guesswork, and media-queries out of the sizing of the plans panes, and ensures that systems set to always display scrollbars only take them when they are required.
Screenshots and screen captures:
Responsiveness:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @karlstolley!


Fix a bug that an e-mail completion message of a password reset link is displayed duplicate

Coverage remained the same at 84.144% when pulling 42ff6e57939f4e7fce41da2cf3b650e2cb4aa9df on fix-bug-that-email-completion-message-of-password-reset-link-is-displayed-duplicate into 5e0d750e44fd3a95b7606fb56a5356a836fd362c on master.


[NO_ISSUE] Add script to mirror images to local registry

Add a simple script to pull the images PSO requires and push them into a local registry.
Update README to reference this script.
This will need to be updated as part of each PSO release cycle with new image tags

Great idea!


Can't see other GitHub orgs in amplify

Describe the bug
Logged in to github but I cant see other repos that not in the default org.
To Reproduce
Steps to reproduce the behavior:

Go to connect Apps.
Click on GitHub
Scroll down to all possible repos
Cant find my repos

Expected behavior
Able to discover the repos in github and connect my app
Additional context
Add any other context about the problem here.
Sample code
Include additional sample code or a sample repository to help us reproduce the issue. (Be sure to remove any sensitive data)

@yahavb were you able to figure this out?

@swaminator I am having this same issue but seemingly due to the fact that the repository is part of a GitHub team and not a repo that I own or is shared with me or by another individual. Any guidance on that situation?

I'm in the same situation. I committed code to an organization's repo 2 hours ago and it's not in the list.

Any news on this issue ? Having the same problem, recently moved a repository to one of my organizations, and cannot use it with Amplify.

@edonadei My issue was this here: https://github.com/aws-amplify/amplify-console/issues/270#issuecomment-558411896. Once I authorized the team it showed up.

I'm having the same problem.
I have a total of 16 personal repositories, and if I do the same with Netlify, it lists 16 candidates, but Amplify only shows 8.
Netlify

Amplify

Also, this is not an issue with #270.
What I am concerned about is that I belong to an organization that has a large number of repositories. So we have more than 120 candidate repositories to connect to Amplify, but Amplify only shows up to 120.
Maybe the limit of this list is 120?


Frontend build error while executing npm ci due to aws-amplify/cli dependency

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
dfsgdd5kz1ebu
Region
us-east-1
Amplify Console feature
Build settings
Describe the bug
Suddenly, our frontend build for our React app connected to our GitHub branch is failing with the following log result. It is a create-react-app:
# Starting phase: preBuild
2021-05-05T21:28:46.658Z [INFO]: # Executing command: npm ci
2021-05-05T21:28:47.014Z [WARNING]: npm
2021-05-05T21:28:47.015Z [WARNING]: WARN
2021-05-05T21:28:47.015Z [WARNING]: prepare removing existing node_modules/ before installation
2021-05-05T21:29:04.653Z [WARNING]: npm
2021-05-05T21:29:04.654Z [WARNING]: ERR! Error while executing:
                                    npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>                                    npm ERR!
                                    npm ERR! Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
                                    npm ERR! Permission denied (publickey).
                                    npm ERR! fatal: Could not read from remote repository.
                                    npm ERR!
                                    npm ERR! Please make sure you have the correct access rights
                                    npm ERR! and the repository exists.
                                    npm ERR!
                                    npm ERR! exited with error code: 128

This has never occurred before running
npm install -g @aws-amplify/cli

for updates.
Our package-lock.json is
{
  "name": "re-plus-app",
  "version": "0.1.0",
  "private": true,
  "dependencies": {
    "@aws-amplify/cli": "^4.50.0",
    "@aws-amplify/ui-react": "^0.2.34",
    "@emotion/react": "^11.1.5",
    "@emotion/styled": "^11.3.0",
    "@material-ui/core": "^4.11.3",
    "@material-ui/icons": "^4.11.2",
    "@stripe/react-stripe-js": "^1.4.0",
    "@stripe/stripe-js": "^1.13.2",
    "@testing-library/jest-dom": "^5.11.9",
    "@testing-library/react": "^11.2.3",
    "@testing-library/user-event": "^12.6.2",
    "aws-amplify": "^3.3.20",
    "bootstrap": "^4.6.0",
    "final-form": "^4.20.1",
    "final-form-material-ui": "^0.3.0",
    "formik": "^2.2.6",
    "moment": "^2.29.1",
    "node-sass": "^4.14.1",
    "prop-types": "^15.7.2",
    "react": "^17.0.1",
    "react-avatar": "^3.9.7",
    "react-bootstrap": "^1.4.3",
    "react-calendar": "^3.3.1",
    "react-circular-input": "^0.2.3",
    "react-circular-progressbar": "^2.0.4",
    "react-datepicker": "^3.7.0",
    "react-dom": "^17.0.1",
    "react-final-form": "^6.5.2",
    "react-icons": "^4.2.0",
    "react-number-format": "^4.5.3",
    "react-router": "^5.2.0",
    "react-router-bootstrap": "^0.25.0",
    "react-router-dom": "^5.2.0",
    "react-scripts": "4.0.1",
    "recharts": "^2.0.9",
    "stripe": "^8.140.0",
    "typeface-roboto": "^1.1.13",
    "web-vitals": "^0.2.4",
    "yup": "^0.32.9"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test",
    "eject": "react-scripts eject"
  },
  "eslintConfig": {
    "extends": [
      "react-app",
      "react-app/jest"
    ]
  },
  "browserslist": {
    "production": [
      ">0.2%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  }
}

We have been scouring AWS resources and cannot find a solution to this issue. Any advice would be greatly appreciated.
Expected behavior
We expect the react-app to build without errors as it does locally.
Reproduction steps
An update on the branch connected to Amplify console would initiate a build. Cloning and backend build steps complete successfully, however frontend build fails right away after running prebuild command: npm ci
Build Settings
version: 1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - npm ci
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Additional information
No response

This issue occurred because winstonjs/winston-transport was attempting to be installed via ssh as specified in the package-lock.json. Amplify however, does not have ssh credentials to access public repositories this way. To solve this error I added winstonjs/winston-transport to package.json with:
npm i winston-transport

And then had to manually update the version to 4.4.0 in the package-lock.json as shown below:
    "winston-transport": {
      "version": "4.4.0",
      "from"<EMAIL_ADDRESS>      "requires": {
        "logform": "^2.2.0",
        "readable-stream": "^2.3.7",
        "triple-beam": "^1.2.0"
      },

The frontend of my react app now builds successfully.


Spirv-Val Does Not Validate Against Empty Structs

Upstream issue https://github.com/shader-slang/slang/issues/5351. You can see the problematic spirv includes a %5 = OpTypeStruct which is an OpTypeStruct with no arguments. This causes nvidia's drivers to segfault and causes mesa to outright the reject the pipeline.
As far as I can tell this is indeed invalid, so it would be nice to get validation against this, as it went unnoticed.

Vulkan allows empty structs (see Offset and Stride Assignment).

A structure has a base alignment equal to the largest base alignment of any of its members. An empty structure has a base alignment equal to the size of the smallest scalar type permitted by the capabilities declared in the SPIR-V module. (e.g., for a 1 byte aligned empty struct in the StorageBuffer storage class, StorageBuffer8BitAccess or UniformAndStorageBuffer8BitAccess must be declared in the SPIR-V module.)

So I need more context about how this is failing in drivers to determine if it is truly a validation issue.

For reference this was resolved in the internal Vulkan issue 2174.

Interesting! So this is the minimized spirv involved. It passes spirv-val:
; SPIR-V
; Version: 1.4
; Generator: Khronos SPIR-V Tools Assembler; 0
; Bound: 30
; Schema: 0
               OpCapability VariablePointers
               OpCapability PhysicalStorageBufferAddresses
               OpCapability Int64
               OpCapability Int8
               OpCapability Shader
               OpExtension "SPV_KHR_variable_pointers"
               OpExtension "SPV_KHR_physical_storage_buffer"
               OpMemoryModel PhysicalStorageBuffer64 GLSL450
               OpEntryPoint Vertex %vertMain "main" %light_buffer %gl_Position
               OpSource Slang 1
               OpName %cbuffer__t "cbuffer__t"
               OpName %light_buffer "light_buffer"
               OpName %vertMain "vertMain"
               OpName %LightBuffer_natural "LightBuffer_natural"
               OpDecorate %_ptr_PhysicalStorageBuffer_LightBuffer_natural ArrayStride 0
               OpDecorate %cbuffer__t Block
               OpMemberDecorate %cbuffer__t 0 Offset 0
               OpDecorate %_ptr_PhysicalStorageBuffer_uchar ArrayStride 1
               OpDecorate %gl_Position BuiltIn Position
       %void = OpTypeVoid
          %9 = OpTypeFunction %void
               OpTypeForwardPointer %_ptr_PhysicalStorageBuffer_LightBuffer_natural PhysicalStorageBuffer
 %cbuffer__t = OpTypeStruct %_ptr_PhysicalStorageBuffer_LightBuffer_natural
%_ptr_PushConstant_cbuffer__t = OpTypePointer PushConstant %cbuffer__t
        %int = OpTypeInt 32 1
      %int_0 = OpConstant %int 0
%_ptr_PushConstant_6 = OpTypePointer PushConstant %_ptr_PhysicalStorageBuffer_LightBuffer_natural
      %int_1 = OpConstant %int 1
      %ulong = OpTypeInt 64 0
      %uchar = OpTypeInt 8 0
%_ptr_PhysicalStorageBuffer_uchar = OpTypePointer PhysicalStorageBuffer %uchar
      %float = OpTypeFloat 32
    %v4float = OpTypeVector %float 4
%_ptr_Output_v4float = OpTypePointer Output %v4float
%LightBuffer_natural = OpTypeStruct
%_ptr_PhysicalStorageBuffer_LightBuffer_natural = OpTypePointer PhysicalStorageBuffer %LightBuffer_natural
%light_buffer = OpVariable %_ptr_PushConstant_cbuffer__t PushConstant
%gl_Position = OpVariable %_ptr_Output_v4float Output
   %vertMain = OpFunction %void None %9
         %20 = OpLabel
         %21 = OpAccessChain %_ptr_PushConstant_6 %light_buffer %int_0
         %22 = OpLoad %_ptr_PhysicalStorageBuffer_LightBuffer_natural %21
         %23 = OpPtrAccessChain %_ptr_PhysicalStorageBuffer_LightBuffer_natural %22 %int_1
         %24 = OpBitcast %ulong %23
         %25 = OpBitcast %_ptr_PhysicalStorageBuffer_uchar %24
         %26 = OpPtrAccessChain %_ptr_PhysicalStorageBuffer_uchar %25 %int_0
         %27 = OpLoad %uchar %26 Aligned 1
         %28 = OpConvertUToF %float %27
         %29 = OpCompositeConstruct %v4float %28 %28 %28 %28
               OpStore %gl_Position %29
               OpReturn
               OpFunctionEnd

which came from the following slang. They removed any unbounded arrays from the underlying spirv struct, which let the LightBuffer_natural struct have zero members. They then bitcast the LightBuffer* to a uint8_t* to read from the unbounded array.
Something about this spirv caused nvidia to segfault in the driver and mesa to reject the pipeline, returning ERROR_UNKNOWN. I don't know if this is strictly the non-membered struct, or some other use of it, but slang ended up not emitting the struct if it has zero args, and that fixed the problem for me.
struct LightBuffer {
    uint8_t lights[];
}

[vk::push_constant]
LightBuffer* light_buffer;

[shader("vertex")]
float4 vertMain() : SV_Position {
    return float4(light_buffer.lights[0]);
}

What looks off to me is:
%cbuffer__t = OpTypeStruct %_ptr_PhysicalStorageBuffer_LightBuffer_natural
%_ptr_PushConstant_cbuffer__t = OpTypePointer PushConstant %cbuffer__t
%light_buffer = OpVariable %_ptr_PushConstant_cbuffer__t PushConstant

SPIR-V requires that %light_buffer be decorated with one of AliasedPointer or RestrictPointer, but the validator is only considering arrays and not structs to traverse. If you add one of those decorations with the empty struct does it still fail in those drivers?

I added
               OpDecorate %light_buffer AliasedPointer

But nvidia still segfaults and mesa returns ERROR_UNKNOWN.


Update website name

👷 Deploy Preview for polite-dolphin-8b4710 processing.

Name
Link

🔨 Latest commit
5e69fe29159481531dda8c2af1134642a96d4a95

🔍 Latest deploy log
https://app.netlify.com/sites/polite-dolphin-8b4710/deploys/635b3e17bc6ccd0009cb905e


[Errno 2] No such file or directory: 'data/unc/corpus.pth'

Hi, I try to run this code, but get the following error. where should I download this missed 'data/unc/corpus.pth' file ?
Traceback (most recent call last):
File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/media/wangxiao/b8efbc67-7ea5-476d-9631-70da75f84e2d/reference_code/DMS/dmn_pytorch/train.py", line 177, in
max_query_len=args.time)
File "/media/wangxiao/b8efbc67-7ea5-476d-9631-70da75f84e2d/reference_code/DMS/dmn_pytorch/referit_loader.py", line 89, in init
self.corpus = torch.load(corpus_path)
File "/usr/local/lib/python3.6/site-packages/torch/serialization.py", line 356, in load
f = open(f, 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'data/unc/corpus.pth'

@andfoy Thanks for your kind explanation. It worked, this is a cool work.

Hello,sir. I got the same problem and cannot solve it.
Traceback (most recent call last):
File "d:\baihuple_File\BigFour\graduate_design\RIS_Paper\Paper1-toR\SHNET-main\main.py", line 368, in
main(args)
File "d:\baihuple_File\BigFour\graduate_design\RIS_Paper\Paper1-toR\SHNET-main\main.py", line 245, in main
train_dataset = ReferDataset(
^^^^^^^^^^^^^
File "d:\baihuple_File\BigFour\graduate_design\RIS_Paper\Paper1-toR\SHNET-main\dataloader\referit_loader.py", line 136, in init
self.corpus = torch.load(corpus_path)
^^^^^^^^^^^^^^^^^^^^^^^
File "D:\anaconda3\Lib\site-packages\torch\serialization.py", line 987, in load
with _open_file_like(f, 'rb') as opened_file:
^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\anaconda3\Lib\site-packages\torch\serialization.py", line 436, in _open_file_like
return _open_file(name_or_buffer, mode)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\anaconda3\Lib\site-packages\torch\serialization.py", line 417, in init
super().init(open(name, mode))
^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'D:\baihuple_File\BigFour\graduate_design\RIS_Paper\Paper1-toR\refer-master\data\data\unc\corpus.pth'
Hope to get your help and I'll be very grafful.


⚠ Noco has degraded performance

In f62305b, Noco (https://nocodb.sps-platforma.ru) experienced degraded performance:

HTTP code: 502
Response time: 3758 ms

Resolved: Noco performance has improved in d9f5073 after 8 minutes.


updated validations.py script

Fixed the behavior of validations.py script.
Closes # 1
Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

googlebot I signed it!

@googlebot I signed it!

@googlebot I fixed it.


Filter empty prompt in random bench serving

Motivation
Fix issue:
 RuntimeWarning: divide by zero encountered in scalar floor_divide
  ratio = (input_lens[i] + prompt_len - 1) // prompt_len

Reproduce:
python3 -m sglang.bench_serving --backend sglang --dataset-path ShareGPT_V3_unfiltered_cleaned_split.json --dataset-name random --random-input 128 --random-output 64 --num-prompts 3200 --request-rate 32 --random-range-ratio 1.0

By the way, if an empty prompt request is sent while other requests are in decoding, the server will fail for both normal and overlap cases:
[2024-11-12 11:34:43 TP0] Traceback (most recent call last):
  File "/workdir/repos/sglang/python/sglang/srt/managers/scheduler.py", line 1210, in run_scheduler_process
    scheduler.event_loop_overlap()
  File "/workdir/tools/miniconda/miniconda3/envs/sgl-vl/lib/python3.9/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/workdir/repos/sglang/python/sglang/srt/managers/scheduler.py", line 368, in event_loop_overlap
    batch = self.get_next_batch_to_run()
  File "/workdir/repos/sglang/python/sglang/srt/managers/scheduler.py", line 615, in get_next_batch_to_run
    self.running_batch.merge_batch(self.last_batch)
  File "/workdir/repos/sglang/python/sglang/srt/managers/schedule_batch.py", line 959, in merge_batch
    raise e
  File "/workdir/repos/sglang/python/sglang/srt/managers/schedule_batch.py", line 956, in merge_batch
    self.output_ids = torch.concat([self.output_ids, other.output_ids])
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument tensors in method wrapper_CUDA_cat)

cc: @merrymercy


nagios: 4.0.8 -> 4.2.3 (for CVE)

Motivation for this change
Nagios update contains many security related fixes (see https://www.nagios.org/projects/nagios-core/history/4x/) . 5b6d52b should probably be backported to stable.
Nagios version 4.2.0 fixes:

CVE-2008-4796
CVE-2013-4214

Nagios version 4.2.2 fixes:

CVE-2016-9565

Nagios version 4.2.3 fixes:

CVE-2016-8641

Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

I'd merge and cherry-pick as soon as I heard back on the unzip.

Ok, LGTM, thank you. merging and backporting.

Backported at 7fc197f and a9523ed. Thank you!


Amplify push failure - CloudFormation stack rollback failed

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.20.2
Amplify CLI Version
12.10.3
What operating system are you using?
macOS
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Made one change to an AppSync resolve after seeing this post (https://github.com/aws-amplify/amplify-category-api/issues/2157) which seemed related to the issue I was seeing. I only made this change after the stack was in a bad state (not able to roll back).
Describe the bug
We recently upgraded from Amplify Transformer V1 to V2. A bug was detected a few days after this upgrade, and to verify the V1 -> V2 Transformer change was the cause, I synced back to the commit before the upgrade, did an amplify push, and confirmed things worked as expected.
After doing some digging I discovered the issue was down to a change in behavior from Amplify Transformer V1 to V2 where the owner field is not automatically populated when making GraphQL requests. I found this post https://github.com/aws-amplify/amplify-category-api/issues/37, which suggested to update amplify/cli.json to include "populateOwnerFieldForStaticGroupAuth": true. I made this change, along with a few other updates to amplify.json (see cli.diff.txt, just remove the .txt extension to preview), synced back to our change after the upgrade to Transformer V2, and did an amplify push.
After running this push, the CloudFormation stack for the application failed. It failed in the api<app>client stack. From the top level, its status is reported as UPDATE_COMPLETE, but when you click the Physical ID link, it shows as UPDATE_ROLLBACK_FAILED. The reason is The following resource(s) failed to update: [ConnectionStack].. Looking at the ConnectionStack, I can see it is also in UPDATE_ROLLBACK_FAILED. The following reason is three resolvers failed to deploy (these are tables in our GraphQL schema):
The following resource(s) failed to update: [GuestUserRegistrationssourceFileResolver, AppUserlicenseResolver, ViewSessionsourceFileResolver].

If I attempt to continue the rollback, I go from one of the nested stacks to the root stack, but as it is not reported as being in UPDATE_ROLLBACK_FAILED, I can't rollback at all. If I try and run amplify push again I see:
Rolled back (2 of 1)
🛑 ["Index: 0 State: {\"deploy\":\"deploying\"} Message: Stack:arn:aws:cloudformation:eu-west-2:<account>:stack/amplify-<app>client-<environment>-115352-api<app>client-.../... is in UPDATE_ROLLBACK_FAILED state and can not be updated."]

At this stage I am not sure how to recover the stack. Is there something I can do to fix the resolvers outside Amplify? Any guidance/support/advice would be hugely appreciated.
Expected behavior
The CloudFormation stack can be rolled back and amplify push works as expected.
Reproduction steps
Not sure exactly if this will work, but this is roughly what I did:

Upgrade project from Amplify Transformer V1 to V2 (without "populateownerfieldforstaticgroupauth": true set - see attachment for state of amplify/cli.json)
amplify push
Sync back in Git history to before the upgrade (Transformer V1)
amplify push (restore earlier state)
Sync back to latest Git commit (Transformer V2)
Update amplify/cli.json with changes shown in diff
amplify push
See push fail, CloudFormation enters apparent unrecoverable state.

Project Identifier
Attempting to run amplify diagnose --send-report shows:
⠏ Sending zip
DiagnoseReportUploadError
✖ Sending zip

Log output
See earlier description
Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

I haven't had a huge amount of help with my other issues on the API repo I'm afraid @ykethan, if you have any ideas I'd love to know!
I have opened an AWS support case for this particular issue too, but it seems like something with Amplify caused it to fail and get into a bad state, hopefully support can help me recover it.

Hey @pr0g, I apologize for the inconvenience you've experienced. Have you been contacted by a member of our support team regarding this issue?

Hi @AnilMaktala, thanks for getting back to me. That's okay, I'm speaking to someone from support who's contacted the CloudFormation team to help restore it. One of the nested stacks is reporting to its parent that it's in UPDATE_COMPLETE, but internally it's in UPDATE_ROLLBACK_FAILED, so the rollback can't be continued from the root stack. Apparently that's a symptom of Drift, but I don't know how that could have happened as I was using the Amplify CLI to perform all operations. I'll report back with an update hopefully when it's sorted. Thanks!

From the description, it is most likely that the failure of the deployment is caused by this

I made this change, along with a few other updates to amplify.json

As you mention about the error comes from ConnectionStack, it should be related to the feature flag regarding the connection changes rather than the auth resolver ones. I examine the diff file and find that there are other flags added apart from populateOwnerFieldForStaticGroupAuth: true.
As a workaround for the update rollback failure, I notice there are already steps mentioned by another customer (see https://github.com/aws-amplify/amplify-category-api/issues/2157#issuecomment-2017234685) about adding the dummy resolvers for those with errors (in your case the ones in connection stacks), which should be helpful for you to resolve the rollback issue.
Once you rollback successfully, I suggest only keeping the populateOwnerFieldForStaticGroupAuth: true but remove the other changes in the diff, which should prevent unintended changes/failures to the resolvers/resources.

Hi @AaronZyLee,
Thanks for your reply. Yes in hindsight I should have been a good scientist and only changed one thing at a time (lesson learned again 🙈). The reason I updated these flags is I'd been meaning to do it after @ykethan suggested I do it in this post. I realize I probably should have done this after though (less haste more speed).
I did see the post you mentioned, but unfortunately I don't think it will work for me because the root stack doesn't think it's in an UPDATE_ROLLBACK_FAILED state, only the nested stacks do, so I'm basically stuck without intervention from AWS support (I will follow-up with them again tomorrow to see where things have go to).
Might it be possible to delete the ConnectionStack and have it get redeployed? I've shied away from doing that because I didn't want to make things worse, but maaaybe that might work?
Thanks for the feedback and it's good to know for future, but ideally now I just need a way of recovering things and getting back to a good state.

Hi @AnilMaktala,
Thanks for following-up, I was able to talk to AWS support and was able to get my CloudFormation stack back to UPDATE_ROLLBACK_COMPLETE, unfortunately when I try and do an Amplify push things are still failing. I've been talking with the AWS Amplfy support team and have managed to narrow things down a bit.
I'm going to try and sync back to earlier in our Git history when this problem occurred and do an amplify push --force to see if I can get the deployment to succeed (this is after deleting the deployment.json file in S3). I think that now that the environment is so out of sync with what's in Git, trying to do an amplify push now is causing problems (the failure happens when trying to update a model/table that's been removed).
I'm going to try and get to this later this week and will leave an update if that works.
Thanks!


Add RunAndReport helper

Add helper method that represents one of the most common usage situations in applications: a job that should be quiet in normal operation, and also give useful output logs along with the error on the occasion that it does go wrong (without writing that boilerplate yourself every time).

I have previously thought that gosh could be more helpful in scenarios where exit codes mean nothing - such as git, docker, the majority of all binaries ever produced by humans - and really, the exec was a failure IFF the combined output contained string X.
Am I correct in thinking the buffer is dropped on the floor if the exit code was kosher? Maybe this should be modified to return the Proc and the buffer.

You're free to assemble that yourself and it should be minimal effort: as before, if you need the buffer, allocate the buffer -- in any of the dozens of combinations gosh supports.
I'd like to avoid the set of functions for invoking a Command becoming fractal.  I'm already dubious about this expansion -- I'm only ready to concede there might be enough cause to put it on the same shelf as Command.Output() after having written this exact behavior more than a dozen times in the last year, and would want a similar burden of proof before any making any other special elevations.
And, though this is unrelated, if you have scenarios where the exit codes mean nothing... file a bug upstream with whoever's the goofball!  zero/non-zero is supposed to be pretty unequivocal.  If an application screws that up, it's well outside of gosh's job description to address that.

You're free to assemble that yourself

I think you misunderstood what I was asking, but that works :D
I'll add a commit that adds , buf to the return statement, that's actually all I was asking (other than modifying the corresponding fn sig of course). There's just no justifiable reason to drop irreplaceable data on the floor if the caller can _ it anyway.
I'll hold off on fixing all of known computer science (git, docker, the majority of all binaries ever produced by humans) until I have literal fleets of nerds to do it with, sadly...

I tell a lie; I decided to match the .String() behaviour you used in this PR. You're right, I think this will be very helpful for Gosh usage but I don't think more functions will be needed except as other obvious boilerplate reductions.

If you want the combined stdout and stderr streams at all times, perhaps you could also use the existing Command.CombinedOutput() method, which does exactly that.  To get it for all exit codes (i.e. never panicking), meld in the AnyExit option.

In that light, this PR seems a little silly. It feels like making excuses for panicing by default - I'd be tempted to either drop the exit code whitelist array and just push that to userspace, or revert this change (plus mine) and use AnyExit + CombinedOutput in the scenario where RunAndReport behaviour is desired.
I have strong feelings that it ought to be one or the other; golang developers who use a try/catch workflow of any kind would appear to be approximately two, and I assert that nobody will be happy with needing an anon closure + deferred recover call to get a string.

Well, I revise that a little bit, because I the first option would renig on the ability to run a vast set of commands without gardening each one. So instead limit my above comment to the second option.

This does not use AnyExit.  It is not supposed to accept any exit status.
The caller may set any selection of "okay" exit codes before calling this, consistent with all other methods that launch a process.
This method is explicitly about collecting output for attachment to errors in the case of "errors" and is in no way anything to do with changing the notion of what "errors" is defined as.
I'd entertain PRs (separately from this entire discussion) for methods that translate the current exit code reactions to error returns instead of panics, but it would have to be carefully designed.  The panic choice is explicit: gosh in its most normal form (read roughly as: when using the tersest invocations) is supposed to behave as much as possible like a shell with set -e mode enabled, which means immediately halting on errors.
I have already suggested precisely a mechanism getting the combined output unconditionally and without a panic, should that be what you require.  Please try it.

The issue isn't about it being possible but instead about redundancy caused by control flow choices. Your comment that it can be interpreted specifically as an error supplement does seem reasonable, in that light I'm more comfortable leaving in this change.
In the future it would be great to have a shot at these (I take it gosh stuff will PR now) before getting merged, I think it would have been a shorter discussion.


Training reference set is inflexible, making label inclusion unpredictable

As seen in train_utils.py's train() function (link):

The labels (which were the train_labels of the TrainClassifierMsg) are split up into a train set and a reference set. The reference set is determined by taking every 10th image of labels, up to 5000 annotations (assuming the number of samples per image is constant throughout labels). The 5000 cap is to ensure the reference set fits in memory. The reason the reference set is made to be separate from the train set is that it's used to calibrate the classifier (according to the code; I can't provide a deeper explanation myself).

The classifier's set of recognized classes (i.e. its labelset) ends up being the set of classes present in both the train set and reference set.

As a result, from the point of view of the program calling pyspacer, it's difficult to control what the classifier's recognized labelset ends up being.
Also, if the rarity of a class is on the order of 1 in 5000 points, there's a good chance that class ends up being excluded from the classifier's recognized labelset. Further, if the labelset size is on the order of 5000 labels, most of the labels have a good chance of being excluded. That's a significant obstacle to any vision of, say, a generalized CoralNet-wide classifier (CoralNet has 7000+ labels).
Possible solution avenues; more than one might be desired:

Make the number 5000 customizable, so that a system with higher-memory instances than CoralNet can raise to potentially a much higher number.

Rework training so that the reference set doesn't have to fit in memory and thus doesn't have to be size-capped at all. Not sure how doable this is. If there are any caveats to this training implementation compared to the old one, then perhaps the new implementation would become a new Trainer class (alongside the existing MiniBatchTrainer).

Allow TrainClassifierMsg to specify train labels, val labels, and ref labels - not just train and val. This means the reference set isn't left to chance. To reduce entry-level complexity, maybe this granularity can be optional: have the option to specify train & val & ref, and also the option to just specify 'labels' and let pyspacer figure out how to split it up into train/val/ref. I think this solution should be implemented, because CoralNet still has trouble handling the corner case where the classifier's recognized labelset is size 0, and being able to reliably detect that situation before calling pyspacer (by defining the ref set on CoralNet's side) would help a lot.

 pyspacer training: Make reference set larger & specifiable to ensure rare labels are recognized

Interesting, and good explanation. My take is that solution #3 is preferable; we should be able to call pyspacer with a reliable set of all possible labels, whether they are in the val labels or training labels or neither.

Sounds good - yeah, based on CoralNet's needs as well, I should be implementing solution 3 in the short term.

Done in PR #71.
In terms of the 3 solution avenues in the OP, 1 and 3 were done. 2 did not seem possible, at least with scikit-learn.


Is Radio FX shared with other users even if they don't have CrossTalk installed?

Title.
Is it shared? I've just installed Crosstalk, I can hear my friends on TS talking walkietalkie-ish, but they can't hear me the same way if they don't have Crosstalk installed. Is it intended? Or how do I solve this?
In "Radio FX" my checkmark is on Home and Other.
Halp.

That's by design.
With such effects the developer has to make a choice to avoid double applicance of the effect.
There're a couple of more plus points for doing this on receive than send, so that's how it's been done.

Ah, okay, I thought I was missing something. Thanks for clarification!

No problemo :)


Refs #25653 -- Clarify help text for --selenium option

Ticket #25653 changed the behaviour of runtests.py --selenium to only run the Selenium tests instead of running them additionally but the help message for --selenium still says "Run the Selenium tests as well".

merged in 87994b40b3874e45666edfceb313e19611cd460e, thanks.


gk_cli crashes in python3 when creating entities

? Do you need any additional custom tokens for your entity?  No
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/bin/gk_cli", line 11, in <module>
    sys.exit(main())
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/__main__.py", line 114, in main
    return create_new_project('custom/')
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/__main__.py", line 139, in create_new_project
    create_new_entity(file_location + 'entities/', entity)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/entity.py", line 22, in create_new_entity
    _create_new_entity_file(entity_name_choice, custom_tokens, file_location)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/entity.py", line 27, in _create_new_entity_file
    file_content = _get_file_content(entity_name, custom_tokens)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/entity.py", line 38, in _get_file_content
    entity_definition_content = _get_entity_definition_content(entity_name, custom_tokens)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/entity.py", line 97, in _get_entity_definition_content
    'extraTokens': ''' + _format_custom_tokens(custom_tokens) + ','
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/entity.py", line 110, in _format_custom_tokens
    return str(tuple([remove_quotation_marks(token['label']) for token in custom_tokens])).replace("'", '')
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/entity.py", line 110, in <listcomp>
    return str(tuple([remove_quotation_marks(token['label']) for token in custom_tokens])).replace("'", '')
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/gk_cli/cli_utils.py", line 71, in remove_quotation_marks
    return string.replace('"', '').strip("\'")
TypeError: a bytes-like object is required, not 'str'

This should be fixed with https://github.com/finos-voice/greenkey-discovery-sdk/commit/bb27efc991b62fdf51d5c405760f88eb8a083c55


refac: Use internal actor id.

Codecov Report

Merging #178 (da4a2a0) into main (ee38af7) will increase coverage by 10.88%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main     #178       +/-   ##
===========================================
+ Coverage   55.54%   66.43%   +10.88%
===========================================
  Files          55      111       +56
  Lines         785     2121     +1336
  Branches       85      217      +132
===========================================
+ Hits          436     1409      +973
- Misses        334      650      +316
- Partials       15       62       +47

Flag
Coverage Δ

appcommon
55.65% <100.00%> (+0.11%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../WebApp/Authentication/AuthenticationExtensions.cs
83.92% <ø> (ø)

...ost.WebApi04/Security/ExampleDomainUserProvider.cs
100.00% <100.00%> (ø)

...rce/App/source/WebApp/Middleware/UserMiddleware.cs
90.90% <100.00%> (+0.58%)
:arrow_up:

...pp.TestCommon/Database/SqlServerDatabaseManager.cs
93.84% <0.00%> (ø)

...toFixture/Extensions/SetParameterCreateProvider.cs
78.57% <0.00%> (ø)

...Configuration/B2C/B2CAuthorizationConfiguration.cs
89.09% <0.00%> (ø)

...App.TestCommon/Configuration/B2C/B2CAppSettings.cs
100.00% <0.00%> (ø)

...tCommon/FunctionAppHost/FunctionAppHostSettings.cs
0.00% <0.00%> (ø)

...erviceBus/ResourceProvider/TopicResourceBuilder.cs
98.03% <0.00%> (ø)

...unctionAppHost/FunctionAppHostManagerExtensions.cs
0.00% <0.00%> (ø)

... and 49 more

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


fix profiling in IE

Current implementation causes an error in IE11 because console.profile there isn't a "normal" function with Function prototype.

Hey! Thanks for the PR. Can you clarify what you mean? What is the exact error produced?

It's easy to see from the following examples from the IE's developer console:
console.log

function __BROWSERTOOLS_CONSOLE_SAFEFUNC() { ...

console.log.__proto__

function() { [native code] }

console.profile

[object] { }

console.profile.__proto__

undefined

console.profile.bind(console)

Object doesn't support property or method 'bind'

console.profile.toString()
"
function profile() {
    [native code]
}
"

The console.profile function is really present in IE, but it seems that it is a special deep engine-implemented function without JS's function prototype. So there isn't .bind property and its calling in the initialization stage causes an error.

BTW, it seems that you already fixed that issue by 74615e2.

Oh thanks! now I understand, it doesn't have .bind. Should be fixed by 0.18.0 as you mentioned 😄 Thanks for letting me know!


Automated cherry pick of #1761: 添加lb eip同步

Cherry pick of #1761 on release/2.11.0.
#1761: 添加lb eip同步

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11.0@2ed681a). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.11.0   #1794   +/-   ##
================================================
  Coverage                  ?   7.61%
================================================
  Files                     ?     412
  Lines                     ?   63393
  Branches                  ?       0
================================================
  Hits                      ?    4829
  Misses                    ?   58153
  Partials                  ?     411

Flag
Coverage Δ

#aFlag
7.61% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/loadbalancers.go
1.62% <0%> (ø)

pkg/compute/models/cloudsync.go
0% <0%> (ø)

pkg/compute/models/purge.go
0% <0%> (ø)

pkg/compute/models/elasticips.go
1.38% <0%> (ø)

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 2ed681a...3c20e3d. Read the comment docs.

/approve
/lgtm


Documentation update

I use a Surface Pro 4 with Application Guard and it works fine.  Your documentation says a hardware requirement is a 64bit processor with 4 Cores... Only the Surface Book 2 (with 8th gen proc) has 4 cores.  You should update your document to indicated to remove the 4 cores requirement.

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

ID: 5f43df7b-6ead-de5a-2e41-4d49978ab485
Version Independent ID: 5497d300-cb21-55f3-7e80-98bc3592635c
Content: System requirements for Windows Defender Application Guard (Windows 10)
Content Source: windows/security/threat-protection/windows-defender-application-guard/reqs-wd-app-guard.md
Product: w10
GitHub Login: @Justinha
Microsoft Alias: justinha

@tdogdfw Thanks for raising this.
@ronsong Thanks in advance for any help you can provide.

When you are working with Intel MOBILE processors (Core-I, Pentium, Celeron and Atom) the CPU only supports 2 cores.  The new 8th generation MOBILE Core-I processors have 4 cores.  Every mobile Surface device (outside of the new Surface Book 2) is based on dual core processors since they are all 7th generation or older.  The Book2 uses an 8th generation processor which actually does have 4 cores.  This is true with all laptop/tablet devices across all OEMs which are using any of the MOBILE Intel processors prior to 8th generation Core I.  Perhaps you were querying “Logical Processors” and not cores?  Logical processors(4) = cores(2) * threads(2).
As proof, I’ve screen captured task manager from a new Core I7 Surface Pro 5 that shows only 2 cores.  All of my Surface Pro 5’s, 4’s and 3’s have been able to run hypervisor and support Application Guard, despite the comment on your documentation that says 4 cores are required.
<EMAIL_ADDRESS>I’ve also copied a screen shot of task manager from a Surface Studio, which uses a 6th generation DESKTOP Core-I processor which shows 4 cores.
<EMAIL_ADDRESS>Hope this helps.
<EMAIL_ADDRESS>Terry Gore
Surface Solution Architect
Cell:<PHONE_NUMBER> |SfB:<PHONE_NUMBER>
From: Rona Song<EMAIL_ADDRESS>Sent: Thursday, August 16, 2018 1:25 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Terry Gore<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Documentation update (#1480)
Thanks for connecting! @tdogdfwhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftdogdfw&data=02|01|Terry.Gore%40microsoft.com|0dd470dfb18845dcb24a08d603a596ba|72f988bf86f141af91ab2d7cd011db47|1|0|636700407070596418&sdata=BUkJ2xCfw0vKv18ajVs9s8xgKQEZXcnDxI5oRhdblpg%3D&reserved=0 , which SP4 do you use? When we say 4 cores, we look for 4 logical cpus and in our codepath, there's an API we use to check for number of cores and it must return with great than 4 to work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fissues%2F1480%23issuecomment-413640347&data=02|01|Terry.Gore%40microsoft.com|0dd470dfb18845dcb24a08d603a596ba|72f988bf86f141af91ab2d7cd011db47|1|0|636700407070596418&sdata=kRAFUmWrQvucE3c6NlLqFUSjqOdM3RsNy30fGR95%2Frc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAodlHWdw4QGadhp_6sZZ8hQTgD9TXPgLks5uRbkAgaJpZM4V_JpD&data=02|01|Terry.Gore%40microsoft.com|0dd470dfb18845dcb24a08d603a596ba|72f988bf86f141af91ab2d7cd011db47|1|0|636700407070596418&sdata=q%2BvaAb1E6wVwZJCcwAAp3K4OZYLA6ERcYQQgjELtjsU%3D&reserved=0.

Hi, the function we call is getactiveprocessorcount - which as you mentioned is logical processors, which we've derived from logical cores. We can check on what would be the best way to address this documentation, but how do you recommend we communicate this?

For future readers of this issue ticket:
Closed by update in commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/e007979d57003033e13ce6808fd2cf3bd549e7b7


DSERV-50-2bit

Set default window to 50 right now. And the return reference sequence look like this:

Previously, for search and summary endpoint, the default assembly is GRCh37 and all the invalid assembly will be changed to GRCh37. This is misleading to user. Will not set default for assembly and validate the input assembly for query.


Moving icon toggle to the v2 component model.

Very similar to checkbox and friends.
Once again, minimal changes to CSS/DOM, as that's better handled in separate PRs if need be.
@surma and/or @Garbee: PTAL!

LGTM :)

LGTM2

Thanks, guys! 👍

LGTM as well. One issue but it is trivial and we will eventually cover it anyways once the new simple-mode thing is worked on. (Existence of mdl-js-icon-toggle in the registered CSS Classes property.)

Yeah, I think we should do a pass and remove all the js classes at some point, or just wait for the simple mode PR and handle it all there. Thanks, @Garbee!


Publish begin/end request and unhandled errors to telemetry source

Publish these:

Request start/end, response start/end (or the closest we can get to these)
Unhandled errors that get bubbled up to the servers

Example in MVC: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/Actions/MvcRouteHandler.cs#L73-L78

Related: https://github.com/aspnet/Diagnostics/issues/180


Feature Request:  > SWS: Toggle Ruler Red When Track(s) Armed For Recording

Thanks to SWS, there is the Toggle Ruler Red Whilst Recording action - very useful!
Would it be possible to request a similar action for when track(s) are currently armed but not yet recording?
It would help warn and prevent against those cases where one mistakenly records on the wrong tracks, and has to undo, that are out of view from TCP. Especially if mixer is hidden and the tcp/project view is zoomed into just one or two tracks.
Thanks!
ps. I originally posted in the forum, here: https://forum.cockos.com/showthread.php?t=286024

@DomMcsweeney
Will a script do? The script will have to constantly run in the background to monitor the state.

Please try this one

Color ruler if tracks are record armed.lua

The thing is that in the original script line 7 (indicated in the error message) is occupied by the script header which is commented out and normally cannot throw any error. Needless to say that on my end this problem doesn't occur.
It's possible that the script was downloaded incorrectly, like an HTML page for example. Please make sure that you use the download button on the script page. If you do and the script still produces the error,  i guess i will need to see your version of the file, the actual code.

Great!

No prob, glad it's useful.

Yes, in this respect in particular the SWS extension is shining. Look for Startup actions in the SWS menu.

Actually no. REAPER has a kind of under-documented feature of external file which is picked up by the program on startup without  having to be imported into the Action list. It must be named __startup.lua (or __startup.eel but now it's less relevant) and paced in the root of the \Scripts folder.
So to run another script or action on startup using this file it must contain the following code:
-- between the quotes insert the command ID of a script/native action/custom action/SWS cycle action
local COMMAND_ID = ""
reaper.Main_OnCommand(reaper.NamedCommandLookup(COMMAND_ID), 0)


fix deprecated ServiceLocatorAwareInterface warning

Fixed the deprecated ServiceLocatorAwareInterface warning by having the PluginManager factories inject the ServiceLocator themselves.
resolves #2
I found it hard to come up with a test for the issue. The deprecation warning is/was thrown when ZF2's ServiceManager applies the ServiceLocatorAwareInitializer to the PluginManagers created by MonoLog's *PluginManagerFactorys, and the initializer finds out that a ServiceLocator was not yet set.

Coverage increased (+0.5%) to 90.179% when pulling e488d2c74ca78f9d248ac1a2df431fbfbbe3c5bc on bvarent:fix/deprecated-servicelocatoraware into d37bcb25e63dfab27a3b51832f6262109b5f7f33 on neeckeloo:master.

Coverage increased (+0.5%) to 90.179% when pulling 5eeeb445e2793bc28b635bf9675c2a6b3b4eea6d on bvarent:fix/deprecated-servicelocatoraware into d37bcb25e63dfab27a3b51832f6262109b5f7f33 on neeckeloo:master.

Coverage increased (+0.5%) to 90.179% when pulling dfa57c05e29a2c2fdfba61b60a9e3ef70c3f9b2e on bvarent:fix/deprecated-servicelocatoraware into d37bcb25e63dfab27a3b51832f6262109b5f7f33 on neeckeloo:master.

Coverage decreased (-2.2%) to 86.232% when pulling 0bdeef15fb743e934ad258a2a2c10a8e68903bac on bvarent:fix/deprecated-servicelocatoraware into 237c9ac4eed24c47c5fc55d990c81ef8e4a3919a on neeckeloo:master.


no Magento Connect in System ( or anywhere ) Ver 2.1.6

I'm using Magento ver. 2.1.6  and in the GUIDE it says

Once you have the key, log into you Magento Store's Admin Panel and navigate to System > Magento Connect > Magento Connect Manager.

However there is no Magento Connect in System ( or anywhere )
Am I missing something?

The plugin is not yet available in Magento Connect. I've updated the README file to reflect this.


not able to overwrite redux-form destroyOnUnmount default

Hi! when setting destroyOnUnmount false (default is true in redux-form) it does not get updated. Seems redux-auto-form is not taking in account this prop.
Thanks.

@suchostone, this is because Redux-Autoform doesn't expect this prop. We should accept all props that Redux-Form accepts and just repass them. My fault. Sorry.
I'll take this problem this weekend.
Meanwhile, if you want to contribute and file a PR, all you have to do is to add new props to the AutoForm component and repass them to AutoFormInternal. I believe every property you pass to AutoFormInternal will automatically be received by Redux-Form.
The AutoForm component is quite straightforward: https://github.com/redux-autoform/redux-autoform/blob/master/src/AutoForm.js
Again.. If you can wait, I'll take a look at it this weekend. About the file upload component you asked. I just decided to prioritize the documentation flaws we got today, and right after I'll take that too.

@andrerpena I will take it now!

Thanks @jony. If you could see what other props are missing and add those as well that would be amazing. Thanks

I have added a class that performs a save of props as a workaround to extract the properties from render and pass them to the reduxForm decorator function. Also, I have added the prop reduxFormProps which expects an object containing redux form props.
now you should be able to do the following:
render() {
     let { autoFormProps } = this.props;
     let reduxFormProps = {
          destroyOnUnmount: true
     };

     return <Autoform reduxFormProps={reduxFormProps} {...autoFormProps}/>
}

@JonatanSalas , thanks for your quick response and your effort. I made my considerations on your commit here.

So @JonatanSalas , you removed the PropStore, which is the right thing to do, but still we didn't solve the subject of this issue. I'll address it today

@andrerpena have you looked at it?

As you can see here, I added all the props from Redux-Form that I thought was safe, including the requested destroyOnUnmount.
However, because the bootstrap demo has been removed from RAF, I wasn't really able to test it, that's why I published it as beta.
In order to test it, install 0.9.0-beta.
@JonatanSalas , we have to figure out how to test it, maybe Enzyme is the way to go, but I'm not really sure how convenient that is. Something to think about.

I'll leave this issue open until I finally have time to test it.

Should be fixed with 8ba48838cb5a18a89605d5b4ae3429d3bb76c770

it works! thanks!


What is ODATA?

There are so many things I don't know about MS S/W.  Potentially new terms should be explained.  I have no idea what ODATA is.  It should be defined.
It shouldn't be hard to add this link to the doc:  https://docs.microsoft.com/en-us/odata/
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3c6b51d7-e615-8421-1789-4e74ff35b361
Version Independent ID: d1e8f7de-9c60-cf00-71bf-59c70935be0e
Content: Connect with data by using OData queries - Azure DevOps
Content Source: docs/report/powerbi/odataquery-connect.md
Product: devops
Technology: devops-analytics
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@gitclem - Thank you for the feedback. I'll make sure to add the link to this and other articles.

@gitclem - Thank you for the feedback. I'll make sure to add the link to this and other articles.


dapr upgrade failed to upgrade

Expected Behavior
Expect ./dapr upgrade -k --runtime-version=1.0.0-rc.3 to upgrade a cluster from rc.2 to rc.3
Actual Behavior
mark@Azure:~$ ./dapr upgrade -k --runtime-version=1.0.0-rc.3
ℹ Dapr control plane version 1.0.0-rc.2 detected in namespace dapr-system
ℹ Starting upgrade...
❌ Failed to upgrade Dapr: Warning: resource customresourcedefinitions/configurations.dapr.io is missing the kubectl.kubernetes.io/last-applied-configuration annotation which is required by kubectl apply. kubectl apply should only be used on resources created declaratively by either kubectl create --save-config or kubectl apply. The missing annotation will be patched automatically.
The CustomResourceDefinition "configurations.dapr.io" is invalid: spec.preserveUnknownFields: Invalid value: true: must be false in order to use defaults in the schema

Steps to Reproduce the Problem

Start with AKS cluster rc.2
Download dapr cli from releases (don't use script)
Run ./dapr upgrade -k --runtime-version=1.0.0-rc.3

Release Note
RELEASE NOTE: FIX Bug in upgrade command that fails to upgrade cluster

Can't reproduce this.
Can you?

Working for me, closing issue
I started with an AKS cluster running 1.18.14.
Opened an Azure Cloud shell with a manually downloaded dapr 1.0-rc.4 cli:
aaron@Azure:~$ ./dapr --version
CLI version: 1.0.0-rc.4
Runtime version: n/a

aaron@Azure:~$ ./dapr init -k --runtime-version 1.0.0-rc.2
⌛  Making the jump to hyperspace...
ℹ  Note: To install Dapr using Helm, see here: https://docs.dapr.io/getting-started/install-dapr-kubernetes/#install-with-helm-advanced

✅  Deploying the Dapr control plane to your cluster...
✅  Success! Dapr has been installed to namespace dapr-system. To verify, run `dapr status -k' in your terminal. To get started, go here: https://aka.ms/dapr-getting-started

aaron@Azure:~$ ./dapr status -k
  NAME                   NAMESPACE    HEALTHY  STATUS   REPLICAS  VERSION     AGE  CREATED
  dapr-placement-server  dapr-system  True     Running  1         1.0.0-rc.3  7m   2021-02-04 21:45.40
  dapr-sidecar-injector  dapr-system  True     Running  1         1.0.0-rc.3  7m   2021-02-04 21:45.29
  dapr-operator          dapr-system  True     Running  1         1.0.0-rc.3  7m   2021-02-04 21:45.29
  dapr-sentry            dapr-system  True     Running  1         1.0.0-rc.3  7m   2021-02-04 21:45.29
  dapr-dashboard         dapr-system  True     Running  1         0.6.0       7m   2021-02-04 21:45.29

aaron@Azure:~$ ./dapr upgrade -k --runtime-version=1.0.0-rc.3
ℹ  Dapr control plane version 1.0.0-rc.2 detected in namespace dapr-system
ℹ  Starting upgrade...
✅  Dapr control plane successfully upgraded to version 1.0.0-rc.3. Make sure your deployments are restarted to pick up the latest sidecar version.


godef: no declaration found for fmt.Println

On my machine, linux mint 64 bit Emacs-24.5, currently " M-x godef " can't find declarations in golang standard packages in $GOROOT, while it works for third party packages inside $GOPATH directory.  Should I set any variables inside Emacs or any possible mistakes I have made?

What's the output of

M-: (getenv "GOROOT")
`M-: (go-root-and-paths)
go env in a shell.

(getenv "GOROOT" ) : /usr/local/go
(go-root-and-paths) : ("/usr/local/go" "/home/huangjian/gopath")
go env in a shell :
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/huangjian/gopath"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

Just to make sure: The fmt package was being imported?
Run the following in a shell to check if godef is working outside of emacs:
echo 'package foo; import "fmt"; func fn() { fmt.Println() }' | godef -i -o 45

Then insert that same code in an Emacs buffer and run godef-describe with point on the Println identifier.

huangjian@HJ-LinuxMint ~ $ echo 'package foo; import "fmt"; func fn() {fmt.Println() }' | godef -i -o 45
parseLocalPackage error: no more package files found
godef: no declaration found for fmt.Println
it seems that godef doesn't work outside of Emacs, but it does work for third party packages, confusing.

Is this a bug of godef? I don't think it is a wrong of go-mode.

If godef doesn't work outside of go-mode, then it's either an issue with your environment or a bug in godef. One last question before I redirect you to godef's issue tracker
What version of Go are you using? (go version) – I expect that godef needs at least version 1.4.

go version go1.6 linux/amd64
I installed go built from source and place it at /usr/local/go.

Then I'm afraid I can't help you. You will want to ask the godef developer for help. Closing this issue. Good luck!

I had the same issue, and I resolved it by reinstalling godef package.
cd $GOPATH/src/github.com/rogpeppe/godef && go install


Huge Memory & CPU Usage

Hello,
I compared three versions of a test application. I saved a lot (10^7) objects of a type with three properties of type double.
The first version was coded with saving to a file with json serialization (Newtonsoft Json). The second was with litedb. The third with simply saving to a txt-File (File.WriteAllLines).
Code LiteDB:
// Open database (or create if not exits)
using(var db = new LiteDatabase(@".\LiteDB.db"))
{
    // Get customer collection
    var col = db.GetCollection<Node>("nodes");
    col.Insert(nodes);
}

The results are:

Version
Memory Usage
Disk Space Usage
Time Needed [ms]

Newtonsoft Json
~500MB
673MB
68,630ms

LiteDB
~6 675MB
3 476MB
497,464ms

Text File
~500MB
950MB
38,913ms

Is there a faster way of saving bigger datas?

Hi @KeKl, thanks for your comment.
Performance and memory use drops in LiteDB when you are working with huge mass of data. This reason is because LiteDB implement a full transaction and journal backup save. Transactions are memory only (thats why use too many memory) and journal write in disk twice.
To works fast with huge data in LiteDB, try to use:
using(var db = new<EMAIL_ADDRESS>journal=false"))
{
    var col = db.GetCollection<Node>("nodes");
    col.InsertBulk(nodes);
}

nodes var must returns and IEnumerable<Node> and will be better if uses yield return

This code must run fast, but never as Newtonsoft or Text File. LiteDB implements indexes (at least _id index), data in pages, and has a much more complex data structure.
I have some plans to remove some features (like transactions and journals) in prior be simple, less memory usage and faster.

Hi @mbdavid,
thank you for your answer.
I tested also your implementation. It´s faster.
Thank you!
Regards,
Kevin


[assimp] Fix cmake config name and add cmake version

Fixes https://github.com/microsoft/vcpkg/issues/14256 and https://github.com/microsoft/vcpkg/issues/14255

Fix assimp cmake config name from AssimpConfig.cmake to assimpConfig.cmake
Export assimpConfigVersion.cmake file.

Currently the usage is:
The package assimp:x64-windows provides CMake targets:

    find_package(assimp CONFIG REQUIRED)
    target_link_libraries(main PRIVATE assimp::assimp)

assimp doesn't contains any features.

Thanks phoebe :)

Thanks a lot @PhoebeHui !


An error occurred attempting to install Democracy3LanguageSelector

https://i.gyazo.com/e7b645b8f73cc6bb5663baebfb9615a2.png
text of install.log:

The following properties have been set:
Property: [AdminUser] = true {boolean}
Property: [InstallMode] = HomeSite {string}
Property: [NTProductType] = 1 {int}
Property: [ProcessorArchitecture] = AMD64 {string}
Property: [VersionNT] = 6.3.0 {version}
Running checks for package 'Microsoft .NET Framework 4.5 (x86 and x64)', phase BuildList
Reading value 'Version' of registry key 'HKLM\Software\Microsoft\NET Framework Setup\NDP\v4\Full'
Read string value '4.6.00081'
Setting value '4.6.00081 {string}' for property 'DotNet45Full_TargetVersion'
The following properties have been set for package 'Microsoft .NET Framework 4.5 (x86 and x64)':
Property: [DotNet45Full_TargetVersion] = 4.6.00081 {string}
Running checks for command 'DotNetFX45\dotNetFx45_Full_x86_x64.exe'
Result of running operator 'ValueEqualTo' on property 'InstallMode' and value 'HomeSite': true
Result of checks for command 'DotNetFX45\dotNetFx45_Full_x86_x64.exe' is 'Bypass'
Running checks for command 'DotNetFX45\dotNetFx45_Full_setup.exe'
Result of running operator 'ValueNotEqualTo' on property 'InstallMode' and value 'HomeSite': false
Result of running operator 'VersionGreaterThanOrEqualTo' on property 'DotNet45Full_TargetVersion' and value '4.5.50709': true
Result of checks for command 'DotNetFX45\dotNetFx45_Full_setup.exe' is 'Bypass' 'Microsoft .NET Framework 4.5 (x86 and x64)' RunCheck result: No Install Needed Launching Application.
URLDownloadToCacheFile failed with HRESULT '-2146697210'
Error: An error occurred trying to download 'https://dl.dropboxusercontent.com/u/260255445/Democracy 3/ClickOnce/Democracy3LanguageSelector.application'.

So what's the problem and how to fix it?

I've got the same problem. I think we need a new link to download.

Yes, it was hosted on dropbox and dropbox closed public folder. I have to move it elsewhere but I didn't have time.

We'll wait. Can you write in this thread when it'll works?

Yes, sorry I don't have so much time now but soon I will try.

anyone still tracks this problem?

@dragouf I think it's not a terrible problem. If you need to change the files to a new public address I think I can help you. Then you just need to update the download address in your application.

the same problem

Yes, sorry I don't have so much time now but soon I will try.

In the end have you done something to solve the problem? Dragouf, please, you said it long ago and since then, it is impossible to play Democracy 3 in a language other than English.  Thanks for the attention.

Really, I do not know if it's helpful. But, it would be enough if you simply upload the already translated files to any server to be able can, although it's, to apply the translation manually.

Before I die, I'll tell my son: "If this is updated, tell me at my grave!"


Add smoke tests to 'Release Version' workflow [HZ-2748]

Adds module to build and run a simple 'smoke test' during the release
deployment pipline. Runs separate steps for OS and EE jars.
Requires a new repository secret (HZ_ENTERPRISE_LICENSE) to run the HZ
EE service container.

BTW When you have EE and OS hazelcast members as services, don't they try to connect one to the other?

BTW When you have EE and OS hazelcast members as services, don't they try to connect one to the other?

@kwart I don't have the logs from the runs on my fork, but I think they do since they have a shared network. I tested in my local docker, and indeed they attempt and fail to join via multicast:
2023-08-02 11:37:04,939 [ WARN] [hz.dazzling_gagarin.MulticastThread] [c.h.i.c.i.MulticastService]: [<IP_ADDRESS>]:5701 [smoke-test] [5.3.1] Received a JoinRequest with an incompatible binary-format. An old version of Hazelcast may be using the same multicast discovery port. Are you running multiple Hazelcast clusters on this host? (This message will be suppressed for 60 seconds).

Even though it didn't cause any issue, I will go ahead and add some config to disable multicast and add cluster names, just to be safe. Good catch!
          HZ_CLUSTERNAME: smoke-test
          HZ_NETWORK_JOIN_MULTICAST_ENABLED: false
          HZ_NETWORK_JOIN_TCPIP_ENABLED: true


Add Aqua and ExplicitImports testing

I added some automated code quality testing.

CI fails because ExplicitImports.jl needs at least Julia v1.7, but we run CI with Julia v1.6. How do you think, we should handle this, @ranocha, @sloede?

CI fails because ExplicitImports.jl needs at least Julia v1.7, but we run CI with Julia v1.6. How do you think, we should handle this, @ranocha, @sloede?

Since we run CI on the latest release and on v1.6, we could also add something like
if VERSION >= v"1.7"
    @test isnothing(check_no_implicit_imports(P4est))
    @test isnothing(check_no_stale_explicit_imports(P4est; ignore = (:PointerWrapper,)))
end

The problem with your last suggestion is that we would still have ExplicitImports.jl in the Project.toml, leading to the same problem. I have just ignored ExplicitImports.jl in my packages supporting Julia v1.6.

Ah yes, you're right. I just removed the ExplicitImports.jl regression test then.


Catch Authorization error and log meaningful message

When reconnecting or calling the guest wifi toggle method an AuthorizationError can rise.
fritzconnection.fritzconnection.AuthorizationError.
Related #4

Done in https://github.com/mammuth/ha-fritzbox-tools/commit/1e5d2744d9325226fd6f8ecc08d3d52993561f71


UnknownHostException mentions hostname with search domain added

Netty version: 4.1.4.Final
Context:
While testing the dns resolution of vert.x I found the cosmetic issue that the exception for UnknownHost mentions the hostname with the last attempted search domain appended, which is kind of confusing. I would prefer to see the original hostname supplied to the method in the exception. The resolution function is based on the netty resolver, the exception text is created in DnsNameResolverContext.
I tried to reproduce the issue with a unit test in netty, which I didn't get quite right, however it is shown when doing the following:

add an unknown hostname to the list of hostnames in DnsNameResolverTest, e.g. "unknown.hostname"
remove the line .nameServerAddresses(DnsServerAddresses.singleton(dnsServer.localAddress())) in newResolver() to enable dns resolution with real dns
add at least one search domain (I used internal.example.com)
run the unit test testResolveA()
if should fail with the error

java.net.UnknownHostException: failed to resolve 'unknown.hostname.internal.example.com'
$ java -version
java version "1.8.0_71"
Java(TM) SE Runtime Environment (build 1.8.0_71-b15)
Java HotSpot(TM) Client VM (build 25.71-b15, mixed mode)
Operating system: Windows 10 Home 64 bit
Microsoft Windows [Version 10.0.10586]
(edit: I  choose a rather bad example as search domain before, I have changed the example to example.com now)

To show the issue inside vert.x the following unit test should show the error message when running it inside the current master of eclipse/vert.x
https://gist.github.com/alexlehm/d4d4ffdf95c97c78dff7ba91de1b0d09
Starting test: UnknownHostExceptionTest#testResolveExceptionHostname
java.net.UnknownHostException: failed to resolve 'sdfsdfsdfsdsdffd.internal.example.com'. Exceeded max queries per resolve 3

@alexlehm makes sense... interested in proving a PR ?

https://github.com/netty/netty/pull/5608