Enabling extensions gives me an error.

[Gedmo\Exception\UnexpectedValueException]
Class Application\Test\Entity is not a valid entity or mapped super class.
I'm using Doctrine 2.1.2 with the 2.1.x tag. Enabling extensions gives me that error anytime I try to manipulate an entity. With the extensions disabled everything works properly.
#0 /modules/SpiffyDoctrineExtensions/library/DoctrineExtensions/lib/Gedmo/Mapping/ExtensionMetadataFactory.php(98): Gedmo\Mapping\Driver\Chain->readExtendedMetadata(Object(Doctrine\ORM\Mapping\ClassMetadata), Array)
#1 /modules/SpiffyDoctrineExtensions/library/DoctrineExtensions/lib/Gedmo/Mapping/MappedEventSubscriber.php(174): Gedmo\Mapping\ExtensionMetadataFactory->getExtensionMetadata(Object(Doctrine\ORM\Mapping\ClassMetadata))
#2 /modules/SpiffyDoctrineExtensions/library/DoctrineExtensions/lib/Gedmo/Sluggable/SluggableListener.php(103): Gedmo\Mapping\MappedEventSubscriber->loadMetadataForObjectClass(Object(Doctrine\ORM\EntityManager), Object(Doctrine\ORM\Mapping\ClassMetadata))
#3 /modules/SpiffyDoctrine/library/doctrine-orm/lib/vendor/doctrine-common/lib/Doctrine/Common/EventManager.php(64): Gedmo\Sluggable\SluggableListener->loadClassMetadata(Object(Doctrine\ORM\Event\LoadClassMetadataEventArgs))
#4 /modules/SpiffyDoctrine/library/doctrine-orm/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php(313): Doctrine\Common\EventManager->dispatchEvent('loadClassMetada...', Object(Doctrine\ORM\Event\LoadClassMetadataEventArgs))
#5 /modules/SpiffyDoctrine/library/doctrine-orm/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php(170): Doctrine\ORM\Mapping\ClassMetadataFactory->loadMetadata('Application\Tes...')
#6 /modules/SpiffyDoctrine/library/doctrine-orm/lib/Doctrine/ORM/EntityManager.php(257): Doctrine\ORM\Mapping\ClassMetadataFactory->getMetadataFor('Application\Tes...')
#7 /modules/SpiffyDoctrine/library/doctrine-orm/lib/Doctrine/ORM/UnitOfWork.php(1255): Doctrine\ORM\EntityManager->getClassMetadata('Application\Tes...')
#8 /modules/SpiffyDoctrine/library/doctrine-orm/lib/Doctrine/ORM/UnitOfWork.php(1234): Doctrine\ORM\UnitOfWork->doPersist(Object(Application\Test\Entity), Array)
#9 /modules/SpiffyDoctrine/library/doctrine-orm/lib/Doctrine/ORM/EntityManager.php(464): Doctrine\ORM\UnitOfWork->persist(Object(Application\Test\Entity))
#10 /modules/Application/src/Application/Controller/IndexController.php(14): Doctrine\ORM\EntityManager->persist(Object(Application\Test\Entity))
#11 /library/ZendFramework/library/Zend/Mvc/Controller/ActionController.php(109): Application\Controller\IndexController->indexAction()
#12 [internal function]: Zend\Mvc\Controller\ActionController->execute(Object(Zend\Mvc\MvcEvent))
#13 /library/ZendFramework/library/Zend/EventManager/EventManager.php(418): call_user_func(Array, Object(Zend\Mvc\MvcEvent))
#14 /library/ZendFramework/library/Zend/EventManager/EventManager.php(204): Zend\EventManager\EventManager->triggerListeners('dispatch', Object(Zend\Mvc\MvcEvent), Object(Closure))
#15 /library/ZendFramework/library/Zend/Mvc/Controller/ActionController.php(77): Zend\EventManager\EventManager->trigger('dispatch', Object(Zend\Mvc\MvcEvent), Object(Closure))
#16 /library/ZendFramework/library/Zend/Mvc/Application.php(329): Zend\Mvc\Controller\ActionController->dispatch(Object(Zend\Http\Request), Object(Zend\Http\Response))
#17 [internal function]: Zend\Mvc\Application->dispatch(Object(Zend\Mvc\MvcEvent))
#18 /library/ZendFramework/library/Zend/EventManager/EventManager.php(418): call_user_func(Array, Object(Zend\Mvc\MvcEvent))
#19 /library/ZendFramework/library/Zend/EventManager/EventManager.php(204): Zend\EventManager\EventManager->triggerListeners('dispatch', Object(Zend\Mvc\MvcEvent), Object(Closure))
#20 /library/ZendFramework/library/Zend/Mvc/Application.php(223): Zend\EventManager\EventManager->trigger('dispatch', Object(Zend\Mvc\MvcEvent), Object(Closure))
#21 /public/index.php(32): Zend\Mvc\Application->run()
#22 {main}

what mapping driver do you use? extensions support Annotation, Xml, Yml. custom drivers fallback to Annotation.
The issue is that you do not have metadata driver for that namespace and I guess you use some kind of personal metadata generator. At least add some default driver for the namespace of your entity classes

Ah, I'm using a custom driver chain which is breaking it. I'm not sure what you mean by adding a namespace. Is there a doc link anywhere I can browse?

@l3pp4rd, where is DoctrineExtensions using the mapping driver?
We had to provide a custom driver to allow chaining different metadata without separating them by namespace...

extensions uses mapping driver according to the ORM metadata driver used. It cannot hook into mapping file reading of orm. so it rereads the file. If it cannot find none of the drivers then it fails to read it. I'm open to all sudgestions to avoid this issue. But so far extensions need to determine somehow, which driver to use for reading it, where it is located and to which namespace it belongs.

Interesting... Could you point me to where this happens (class(es)/method(s))? It's a major issue that makes it difficult to plug DoctrineExtensions into the SpiffyDoctrine module, and I'd like to help out if there's a clean way of fixing it :)

determine driver driver chain I agree it can and should be improved. But the only way I see is specifying metadata drivers in entity. Which would be very hard in case of xml metadata, it needs specific file location

@l3pp4rd, ok, I'll try to work on it between this and next week (can't do before, sorry). As soon as I'm able to get some decent idea about how to clean up this behavior I'll write it down here :)

I guess the best would be for doctrine to give the data it loaded with a driver in loadClassMetadata event. This way extensions would not need any additional drivers, and rereading the mapping

any progress? in general I can remove triggering this exception, it would mean it that extensions will not read such entities

@l3pp4rd, sorry, did not look at it yet. The problem is not the exception itself, but the fact that DoctrineExtensions steps in with it's own MetadataDriver, replacing the old one (from what I got till here). From an ideal point of view, this should be handled through www.doctrine-project.org/docs/orm/2.0/en/reference/events.html#load-classmetadata-event .
I still didn't look at it, so what I'm saying may be an impossible utopy...


Replace erusev/parsedown with league/commonmark

restrict the set of raw HTML tags (no <script>, <iframe> et. cetera)
render external links with rel=noreferrer noopener and target=_blank

Fixes #115.

Thanks! :)

Thanks! :)


Gravity issues

When you take a spacesuit armor item off gravity modifiers seem to vanish

Most likely due to conflict between what is making "gravity" and 3d_armor...  3D Armor changes gravity.  Need to investigate this more..

Fixed this-- it was issue with 3d_armor monoids ...


Handling default values

Hey Alan,
I noticed that default values don't seem to be handled at the moment (unless I'm missing something).
I generated code based on:
{
    "type": "record",
    "name": "example",
    "subject": "example",
    "fields": [
        {
            "name": "name",
            "type": "string"
        },
        {
            "name": "hobby",
            "type": "string",
            "default": "golf"
        }
    ]
}

Then ran:
func main() {
	el := example.Example{
		Name: "Or",
		// Hobby: "PRs",
	}

	var buf bytes.Buffer
	if err := el.Serialize(&buf); err != nil {
		log.Fatal(err)
	}

	eln, err := example.DeserializeExample(&buf)
	if err != nil {
		log.Fatal(err)
	}

	fmt.Printf("%+v\n", eln)
}

which output:
&{Name:Or Hobby:}

If I uncomment the line above, I get a value as expected.

Sorry about the slow reply. Default values don't work right now, I should have time in the next few days to add them.

No worries. Thanks a lot for you work Alan!

@rikonor Default values should work in master now - every Record struct has a constructor method that will set the values from the schema file. Let me know how it works for you!

That's fantastic. Thank you very much @alanctgardner


Incorrect wrapped token icon being displayed

On the main page, there is no icon for the unwrapped token amount, and the unwrapped icon is shown for the wrapped token

Ah this component doesn't display the token icon, just the symbol. And as far as i can tell, the wrapped token symbol was set as 🔆

Ouch, that was me indeed. I choose that as the symbol for the Garden 😅


[10.x] Add soft enum cast

Laravel Version
10+
PHP Version
8.1+
Database Driver & Version

Description
https://github.com/laravel/framework/blob/10.x/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php#L1182
replace to tryFrom return null if case exists
Steps To Reproduce
Create error cast value enum

I get 500 error
ValueError

"company_driver" is not a valid backing value for enum App\Enums\Company\Job\DriverTypeEnum

 protected $casts = [
        'driver_type'             => DriverTypeEnum::class,
 ];

I expect to get null


[rust] implement list_distros match_condition method in rust

[rust] implement list_distros match_condition method in rust

Thank you! :1st_place_medal:


Shibboleth DS page JS error

Remnant of an old call to an ES5 JS file is causing and error.
Log out of the tool and then click on 'Your Institution' button on the Sign in form to replicate the error:
 Showing /apps/dmp/apps/roadmap/releases/20171010182816/app/views/orgs/shibboleth_ds.html.erb where line #1 raised:

undefined method `javascript' for #<#<Class:0x00562fa1141348>:0x00562f9edd2240>

Log indicates that its using an HTTP GET instead of POST now.

related to #685

working as expected. closing.


Shortcut method for mutuallyExclusiveTrueProps?

Currently, if you want to use mutuallyExclusiveTrueProps, you need to do something like this:
const fooProp = mutuallyExclusiveTrueProps('foo', 'bar', 'baz');

Foo.propTypes = {
  foo: fooProp,
  bar: barProp,
  baz: bazProp,
};

I wonder if it would be useful to be able to do something more like:
Foo.propTypes = {
  ...mutuallyExclusiveTrueProps('foo', 'bar', 'baz'),
};

And, if you wanted to also have this generate the defaultProps for you:
Foo.propTypes = {
  ...mutuallyExclusiveTrueProps('foo', 'bar', 'baz').propTypes,
};
Foo.defaultProps = {
  ...mutuallyExclusiveTrueProps('foo', 'bar', 'baz').defaultProps,
};

or, obviously:
const fooProps = mutuallyExclusiveTrueProps('foo', 'bar', 'baz');
Foo.propTypes = {
  ...fooProps.propTypes,
};
Foo.defaultProps = {
  ...fooProps.defaultProps,
};

The main drawback to this that I think of right now is that it might not work so well with some of the React lint rules.
I'm not sure what this method should be named, or perhaps if we add propTypes and defaultProps, we could just attach those to the return value of the current method as a convenience.

Here's another possible option you can use currently:
const propNames = [ 'foo', 'bar', 'baz' ]
Component.propTypes = propNames.reduce((acc, prop) => (
  { ...acc, [prop]: mutuallyExclusiveProps(validator, ...propNames) }
), { /* additional props */ })

Also
const propNames = [ 'foo', 'bar', 'baz' ]
Component.propTypes = {
  ...Object.fromEntries(propNames.map(prop => [
    prop,
    mutuallyExclusiveProps(validator, ...propNames),
  ])),
  // additional propTypes
};


Add One platform to the list

Hello,
I would like to suggest adding Cybertalents to the list of platforms.
a well known website hosting jeopardy style challenges for many categories "Forensics/RE/Crypto/Web,..." also providing a monthly regional CTF events would be useful for practicing

Hi, would you like to open up a PR?

yes


[Feature Request] integration with telegram instead of emails.

I've always wanted this to work without the needs to use emails but using chat systems.

By using the API from https://github.com/42wim/matterbridge you could integrate to most chat systems. matterbridge is a single binary written in go. You need a configuration file to talk to a chat.
Broadcasting to all configured chat systems is just a simple http POST request:
https://github.com/42wim/matterbridge/wiki/Api#send-message-post-apimessage

This should be taken care of by either webhooks (might be merged soon) or by an external program such isso-telegram-notifier as mentioned in https://github.com/posativ/isso/pull/724#issuecomment-939299481


fix nxm links

A lot of tests are failing, so for now I just disabled it.

Heh, the rebase conflicts did not went as planned, well that happens when you force push all the time, and not just at the end ;)

Thank you! I've included your fix (I think; I've excluded the find . -name xunit.runner.json -delete bit for now since I'm not sure it's necessary) in my PR, with you as co-author.

Thank you! I've included your fix (I think) in my PR, with you as co-author.
Do we need the find . -name xunit.runner.json -delete bit? It seems to break a bunch of tests.

No, thats is only to make it pass when you disable tests


Linux arm64 support?

Just wondering how trivial it would be to get this working on a Raspberry Pi?

Hard to tell, our use case scenario as of now is x64. Is it most recent RPi 4 which is ARM64? I would say most important is to build underlying webrtc library for ARM64. Spitfire on its top is something that looks doable.


[helm/redis-cache] service not routed to master

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one):

Version of Helm and Kubernetes:
latest
Which chart:
redis-cache
What happened:
in my app i use the dns endpoint, if the elected master goes down, the connection is routed to another instance that is slave
What you expected to happen:
the service should route to the new elected master
How to reproduce it (as minimally and precisely as possible):
deploy a 3 replicas, add some data, and kill the master
Anything else we need to know:

cc: @dhilipkumars


create chart for uptimerobot prometheus-exporter

See #204 for some context.

See also:

https://github.com/ukwa/uptimerobot-prometheus-exporter
https://github.com/masaruhoshi/uptimerobot-prometheus-exporter
https://github.com/lekpamartin/uptimerobot_exporter
https://github.com/rmrf-run/prometheus-uptimerobot


FluxCD Updates

This cumulative PR includes the following changes:

[X] https://github.com/aenix-io/cozystack/pull/166
[X] https://github.com/aenix-io/cozystack/pull/167
[x] https://github.com/aenix-io/cozystack/pull/172
[x] https://github.com/aenix-io/cozystack/pull/171
[X] https://github.com/aenix-io/cozystack/pull/178

When it will be ready, we will merge all in one time

@kingdonb I rebased PR to include https://github.com/aenix-io/cozystack/pull/178

Tested with e2e.sh works well.


Hoping icon stacking can be use with this plugin

We can use stack like this with html css

Hope can use the same technique with this plugin.

I'm not familiar with HTML/CSS, does that code snippet just super-impose one icon on top of another? If that's the case, you could use a FrameLayout to achieve a similar effect.


Localization for placeholder

How should i localize the placeholder to multiple language if the placeholder field isn't from LocalizedStringKey class ?

Have you tried using something like this:
FloatingLabelTextField($email,
placeholder: NSLocalizedString("label_password", comment: ""),
editingChanged: { (isChanged) in

                        }) {

                        }


Add DEA explorer guide #59

I have copied the DE Africa Explorer documentation and updated it for DEA Docs.
I'm not really able to test the formatting until it renders on the site, but I copied it from another page, so I am hopeful.

This looks so good, yay! Something we've needed for a long time 🙂


Change the default tab in #settings/your-bots to "Active bots" instead of "Add a new bot". Redirect to "Active bots" tab on creating a new bot.

This is a follow-up task for #5096. As creating a new bot in "Add a new bot" tab, shows an empty form. This  might be mistaken by users to think that the new bot didn't get created.
For clarity of the situation and confirmation for the user that the bot got created, we intend to switch tab to "Active bots"

@zulipbot claim


Check conditions for Kings of Summer, Kings of Winter and Rains of Castamere agendas

Also, check restrictions in PHP when saving deck.
To make it easy, I added the Functional PHP library, to use it when checking restrictions.

This library is awesome!

Ok, I'm trusting you blindly. I'm putting it online without testing. YOLO

lstrojny/functional-php 1.2.4 requires php >=5.6.0

argh

What version of PHP used the server?
I've looking in the composer repository, and v1.0 of functional-php only require PHP >= 5.3,,, but I don't sure the only 3 functions I use (map, select and reduce) will work well. I will try with that version.

It seemd that work well with v1.0 of functional-php. What do you prefer? Upgrade php version if its possible, or downgrade functional-php?

The server uses php-5.5.9.
We can use functional-php 1.0, or we can remove it entirely. It's nice, but it's nothing that can't be done easily with array_map, array_reduce, array_filter, etc.
Sorry, but I'm not going to update to php-5.6, because it is not a package officially supported by Ubuntu 14.04 (and probably never will).

So, with the help of 20e34b4fabca353c0eabaaa51347b87045fde1cb, the fix is online.


fix: empty meta.image on carousel and remove pinata head request on gallery item

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[x] Refactoring

Context

[ ] Closes #<issue_number>
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Just confused how is this related to #4073?

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 26.64 USD/KSM ~ 1.877 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0xa83466456a1ea41a8a5e1c27f28fd1b2818865347143a2e08b01fdc54409f2f8
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Larger Vim plugin help file cannot be highlighted

Describe the bug
I'm maintainer of vim-lsp. The documentation is not highlighted on GitHub although its mode line description at bottom of the file is correct.
To reproduce this, open https://github.com/prabirshrestha/vim-lsp/blob/master/doc/vim-lsp.txt in your browser and check if highlight is applied.
Expected behaviour
Highlight is applied like this
Related discussion
Today I investigated why the document is not highlighted on GitHub.
The regex for heuristic matched correctly to modeline at the bottom of the document file.
However, the document file is larger than 51200 bytes. And the heuristic detection has 51200 bytes limit on finding the regex pattern. I understood that moving the modeline to top of the document file will make the heuristic detection work, but it's not possible because the format of Vim help file does not allow it. The first line is treated as summary of the document by Vim so modeline cannot be put at the line. On the other hand, Vim's mode line must be put on the first line or the last line of the file (please see :help modeline for more details). After all, modeline must be put at the last line in Vim help file. It means that Vim help files larger than 51200 bytes cannot be highlighted currently.
Additional notes
I think this issue is important for Vim plugin users because larger document is harder to be read and searched. Highlight would help to do them, but currently highlight is not applied due to this issue for larger documents.

Vim's mode line must be put on the first line or the last line of the file

I don't think this is true.
The :set line can be put at, for example, line 3 to be recognized as a modeline unless the value of the modelines option (default: 5) is less than 3 (please see :help 'modelines').

Yes, that's correct. But it's up to user's configuration. For example, when setting the value to 1, what I said is true. Since Vim help file cannot assume some user setting is set to specific value, I can't say putting the modeline at line 3 is always ok.
And also, putting modeline at line 3~5 is not good in terms of Vim's help file format. Usually table of contents is put at top of the file. Putting mode line just before the table looks weird.

Worked it out. Your modeline isn't matching on the modeline strategy because you're specifying the file type/language as help but Linguist doesn't have a language or alias called "Help".
Changing your modeline to "vimhelp" however does work because it now matches the language alias:
https://github.com/github/linguist/blob/32ec19c013a7f81ffaeead25e6e8f9668c7ed574/lib/linguist/languages.yml#L6099-L6104
... as can be seen in my fork at https://github.com/lildude/vim-lsp/blob/master/doc/vim-lsp.txt (I'll delete this fork at some point).
When you move the modeline up into the range of the heuristic limits, the heuristic matches because it explicitly looks for "help" and matches that to "Vim Help File":
https://github.com/github/linguist/blob/32ec19c013a7f81ffaeead25e6e8f9668c7ed574/lib/linguist/heuristics.yml#L573-L574
Scroll all the way to the right and you'll see (?:filetype|ft|syntax)[ \t]*=help(?=[ \t]|:|$)
The final solution here would probably be to add help as an alias to the "Vim Help File" language entry.

The final solution here would probably be to add help as an alias to the "Vim Help File" language entry.

This was a deliberate omission on my part. Given the specificity of Vim's help format, I was a little reluctant to add an alias as generic as help out of concerns about misclassification. In hindsight, I may have been overly cautious.
Also, I completely forgot about the Vim Help heuristic when preparing #5271. 🥵 That'll need to be D.R.Y-ed up.


 This is, of course, a policy decision on behalf of the host but without the information available hosts (such as Open Source Collective) are unable to enforce a policy without the use of a tool like Metabase.

Prototype of an account hover card that can show contextual information about accounts including "hosted since":

@gustavlrsn Looks like this was resolved in https://github.com/opencollective/opencollective-frontend/pull/9628?


Test action update

Test action update.

Codecov Report
Patch and project coverage have no change.

Comparison is base (66640f0) 96.42% compared to head (00da2a7) 96.42%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage   96.42%   96.42%
=======================================
  Files           1        1
  Lines          28       28
=======================================
  Hits           27       27
  Misses          1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Views' doc missing when on same route and using same class

Hi,
It seems to be the case that when I have multiple views on the same route that are both from the same class(typical for CRUD endpoints), the docs for those views don't generate at all.
Example:
config.add_route('my-route', '/my/path/')
config.add_view(
    Foo,
    attr='read',
    route_name='my-route',
    request_method='GET',
)
config.add_view(
    Foo,
    attr='update',
    route_name='my-route',
    request_method='PUT',
)

Seems to not generate any docs whatsoever for those two views when running make html. Also, this may be a separate issue, but if you take the above code and give the views separate routes so that they do appear, both will use the Foo's class docstring instead of the read and update method's doc strings which I found unintuitive.

@TheDan64 Sorry, I didn't notice this issue. I'll fix it tomorrow for you!

@sontek Thanks, looking forward to it!

@sontek Any update on this?

@TheDan64 Sorry, I forgot about this.  Here is the PR to fix it:
https://github.com/SurveyMonkey/pyramid_autodoc/pull/9
any chance you can test it?

Sure! I will take a look, thanks!

@sontek When I build and pip install from source, I get this message:
Processing /home/dkolsoi/repos/pyramid_autodoc/dist/pyramid_autodoc-1.0.1.dev1.tar.gz
    Complete output from command python setup.py egg_info:

    Installed /tmp/pip-pDbYJH-build/.eggs/setuptools_git-1.1-py2.7.egg
    ERROR:root:Error parsing
    Traceback (most recent call last):
      File "/home/dkolsoi/venv/local/lib/python2.7/site-packages/pbr/core.py", line 111, in pbr
        attrs = util.cfg_to_args(path, dist.script_args)
      File "/home/dkolsoi/venv/local/lib/python2.7/site-packages/pbr/util.py", line 264, in cfg_to_args
        wrap_commands(kwargs)
      File "/home/dkolsoi/venv/local/lib/python2.7/site-packages/pbr/util.py", line 566, in wrap_commands
        cmdclass = ep.resolve()
      File "/home/dkolsoi/venv/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2361, in resolve
        module = __import__(self.module_name, fromlist=['__name__'], level=0)
    ImportError: No module named upload
    error in setup command: Error parsing /tmp/pip-pDbYJH-build/setup.cfg: ImportError: No module named upload

I can't find a module name upload, so I assume it's something else?

@sontek Any update on this?

Took a look at this again and we were using the version 0.0.7 back then. 1.0.0 doesn't seem to have those issues anymore. Thanks.


fix #136 incorrect label for date input in cards example

fix #136 incorrect label for date input in cards example

Thanks!


Update SDK to be compliant with spec v0.5.0

Update the SDK to be compliant with the v0.5.0 spec.
What's Changed
⚠ BREAKING CHANGES

https://github.com/open-feature/spec/pull/142 Error codes must be a constrained set of values (e.g. enum).

Additional changes

https://github.com/open-feature/spec/pull/140 Clarify that reason must be a string in the flag resolution details and evaluation details.
https://github.com/open-feature/spec/pull/142 Add optional error message to flag resolution details.

General

[ ] Add/update a spec version badge on the readme.

Full Changelog: https://github.com/open-feature/spec/compare/v0.4.0...v0.5.0

The biggest change I see here is the inclusion of the Error enums

Yeah, it looks like reason is already a string in the PHP implementation. That was also a breaking change in other SDKs which is why we clarified the requirement in the spec.

ErrorCode is implemented as an "enum" (supporting PHP 7.4 and 8.0 which do not have native Enums)

As far as I can tell this is complete :+1:


Fix missing object.coffee outputs.

I have been working on a "optional" loaded javascript file that will import a pdf into a new pdf for output.   Everything works other than pdfkit currently doesn't support outputting a "hex" value.   i.e < 0A >, the Object.convert wants to either treat it as a String key i.e. /0A   or as a Object << 0A >>

I have finished a module that can be integrated with pdfkit that will add a new function to the pdfkit prototype that will allow us to import multiple PDF into the current pdf stream.    The code is now working on my box; but the only piece that is missing so that people don't have to pull "my" copy of pdfkit is the ability for me to output a value the way I need to to the stream.    (In the only case I haven't been able to work around in the pdfkit)    So I would be appreciate if we could get this merged in and deployed as soon as you have some time.

I should also say that as I was testing I realized just now that "true", "false", "null" and "" all also have to be passed through the system as a isRaw as they are string values but not a "/key" and not a "(string)" but actually the raw text.   So I actually have a couple things now using that minor change to your objects.convert function.

Boolean values, null, etc. are already handled by the else case here which converts them to strings correctly. I'm curious what other "hex" values you need to import? Do you have an example?

I'll double check that true/false/null gets passed through, I left them as strings as they came in as string characters; but switching them to a real value is easy.    However the "" (Which can be considered as blank or null depending on the key) and hex values (on PDF 1.7 Spec page 56) are still handled with the isRaw.
As for the usage; in the 100 random pdf's that I'm playing with I've seen hex values primarily in the ColorSpace key, but I've also seen it anywhere  MultiByte strings are supposed to be.  And their are a couple keys in the spec that are required to be Hex (For example page 185&186, SubType)

I've confirmed with some minor alterations that null/true/false will work fine, now I just need Hex and blank.   I should probably state; when I import the pdf; I read the entire structure and all keys, strings, dictionary's, etc;   then I spit out the all the data into pdfkit structures.
I don't modify any data other than all the references are rewritten (using your pdfreference) to be updated to stay in sync in the new document.      So if I get passed in a Hex value; it gets spit back out as a hex value.   The idea is to do as few changes to the stream as possible, this allows most the features of the existing pdf to continue to work even without me directly supporting them.

I'm already not really a fan of the isString hack here, and I'd rather not add another one. For hex strings, we could use Buffers (e.g. convert them to "<#{buf.toString('hex')}>"). Empty strings can be either () or <> (empty PDFObject.s i.e. with isString, or empty Buffer). Does that work for you?

I'm not a big fan of the isString / isRaw hacks either, & the only reason I went with isRaw (rather than .isHex) is it allowed me to standardize how to do different outputs depending on the object passed in; a  "" gets output as "", not as ("") or /""   and hex gets output as <hex>.    So creating new "objects" to output anything was simple.   However, updating the code to support the Buffer object is also a good solution.
Since I really need this support; I am willing to do this which ever way you want:

Leave it as is, two hacks (isString/isRaw).
Update the code and refactor the isString; and change it to isRaw (so you only have one hack in the system still).
Eliminate the isString in the system and my new isRaw, and replace it with code that supports doing Buffer.isBuffer & Buffer.toString() -- I assume that Buffer is emulated property on the browser as I believe you are using it elsewhere in your code.

Let's go with the buffer option to create hex strings. Browserify automatically handles buffer support in the browser.
As for isString, I think a better implementation would be to have a wrapper class, e.g. PDFString for that so we can do if object instanceof PDFString, but that doesn't necessarily need to be done in this PR. I went with name objects (e.g. /name) for strings by default since they are more commonly used, so there has to be a wrapper for strings somehow to distinguish them.
Where are you seeing empty strings without parentheses or angle brackets around them? I don't see anything in the spec for that... they say empty strings should be enclosed with parens.

I've updated the code to eliminate the .isRaw/.isString hacks; so they all now use Buffer objects.     I've tested my code against it and everything appears to be working properly.
As for the Empty string/value; I think I saw it somewhere in the spec that it could be considered null (empty) in some cases; but the main thing is that I've ran into several pdf's that the code was:
"x 0 obj  endobj" and then either a dictionary key reference or a reference in a array to that obj; so I am making the "assumption" that if it is in the wild; it needs to be kept as is.    I don't want to arbitrarily delete a "empty" obj, when it has refs'.   So I need to be able to create "x 0 obj  endobj" which means passing a "" as the value to be stored.

Any news on if this will be accepted (again I'm willing to do whatever changes are needed); or do I need to point my repository to my own copy of PDFKit so that I can have a automatic deploy of both my pdfimport and pdfkit.

Sorry, there were merge conflicts here. I spent some time reworking this in 22a9bfdfab43792e32bb604aa72bf0183a2b1b38. I realized that JavaScript already has two string representations: literal and String object wrapper, and we can take advantage of this. Now, literals are converted to PDF names as before, and String objects are converted to PDF strings. Buffers are converted to hex strings.
What this means for you is that you'll need to parse the hex strings you get in as buffers, taking off the angle brackets: e.g. new Buffer(string.slice(1, -1), 'hex'). You'll also need to wrap strings as String objects, removing the parens: new String(string.slice(1, -1)).
I found the bit about null objects in the spec. From my reading, I think it means references that don't point to a valid object become null:
An indirect reference to an undefined object is not an error; it is simply treated as a reference to the null object. For example, if a file contains the indirect reference

    17 0 R

but does not contain the corresponding definition

    17 0 obj
      ...
    endobj

then the indirect reference is considered to refer to the null object.

I haven't seen anything about the contents of the reference being empty. Have you tried just dropping them? Or how about putting null as the contents instead?

Only problem is now with your '<' + object.toString('hex') + '>' is that you are forcing buffer to be a hex.   I would much much rather have this be just a straight:    "object.toString();" so that I can control what gets output.
The reasons why is:

I already read it in as a a raw string (that is how it is in the document); it is already in hex form; I don't want to convert it out of hex to normal characters so it can be re-converted back hex.  So very wasteful of cpu and memory -- I would much rather just do new Buffer('<'.+ data + '>');   So what is read in is written back out as is; no conversions.
By leaving it object.toString() I can also support other things like the blank issue "new Buffer('')" so that I get a  "x 0 obj  endobj" out.  I have no idea what happens if I drop the ref, most pdf readers are pretty forgiving; but occasionally if you put the data in wrong they crash (took me a while to figure out what was causing my pdf reader to crash; and it ended up actually being a bad ref).    What I do know is that that format is in a couple pdf's -- so I assume (maybe a bad assumption) that it is valid even though I didn't see it specifically documented in the pdf spec.

So any change I can get you to change it back to what my pull was of just the
else if Buffer.isBuffer(object)
object.toString()

The reason I did this was that making hex strings out of binary buffers could be useful for other purposes in the future, other than just what you're doing here. You already have to make a new Buffer either way, and the memory allocation is the expensive part, not the string conversion, which takes place both ways as well. Your way you have to make a buffer from the ascii string, and this way you have to make one from a hex string - probably no difference in terms of performance, and it will use less memory (2 characters in the buffer for every hex byte, vs 1 byte).
The spec seems clear enough to me:

The definition of an indirect object in a PDF file consists of its object number and generation number, followed by the value of the object itself bracketed between the keywords obj and endobj.

Containing nothing is invalid, pdf readers just happen to accept it. Can you try setting the value to null instead?

I switched it to null, and replaced my code to use your new String and Buffer methods and I am good to go.   Thanks for getting these changes in!   Any idea when npm will have it?

Sorry about the wait. Just released v0.7.1. Was trying to get together a bigger release, but it's taking longer than I hoped.


ignore demo-build branch in GitHub Actions

Codecov Report
Patch coverage has no change and project coverage change: -3.92 :warning:

Comparison is base (46abce6) 75.52% compared to head (bdb5cb3) 71.60%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   75.52%   71.60%   -3.92%
==========================================
  Files          14       14
  Lines         817      817
==========================================
- Hits          617      585      -32
- Misses        116      148      +32
  Partials       84       84

Flag
Coverage Δ

cosmosdb_sql
?

dynamodb
62.17% <ø> (ø)

mongo_replicaset
59.24% <ø> (ø)

mongo_standalone
59.24% <ø> (ø)

mysql
?

pgsql
61.56% <ø> (ø)

sqlite
61.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Catch IOError in Admin API

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Add function to enable power event handler

I suspect this was overlooked as CanHandlePowerEvent was never being set to true anywhere in the codebase so the service event handler was never being setup under the covers.
Should resolve #384 and #170
If I'm mistaken and this was intentional please let me know.

This does work I should mention - power events are now fired properly, not sure if anyone is around to merge it but would be great to have this upstream, have to use our private nuget server right now.
Let me know if you need me to make any changes to make it acceptable for merging!

I think we should ping @phatboyg :)
This change is trivial but necessary.

Yeah, pretty trivial.


Cut a couple of bytes

Why bother with “tan”?

wow! I figured there was some smarter way to use a single color in a gradient but couldn't figure it out, thanks


"./manage.py migrate" no longer prompts for a Django superuser account

Most likely something on my end.  But following the rtd install from scratch per https://docs.readthedocs.org/en/latest/install.html, "./manage.py migrate" no longer prompts for a superuser password.

I'm experiencing the same, so it may not be on your end.

I created a superuser manually using "./manage.py createsuperuser", which seems to do the trick.

Thanks for the report! The docs will be fixed with #1566
The ./manage.py createsuperuser is the recommend way to create a new superuser.


feat(plugins/rate-limiting): Add ability to rate limit by subdomain

Summary
Kong provides a basic rate limiting plugin. This plugin however does not offer the ability to rate limit by subdomain. Something which is used by a lot of organizations to achieve multi tenancy.
This PR adds support to the rate limiting plugin to rate limit by subdomain.
Checklist

[x] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[x] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - https://github.com/Kong/docs.konghq.com/pull/4962

Full changelog

[Implement ...]

Issue reference

Fix #[issue number]

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you, theundeadmonk, I think that we should add test cases for this feature.

@chronolaw Do you mind taking a look at this again?

@theundeadmonk Good idea! I was thinking of another alternative solution. Have you ever tried the limit_by header feature? which is basically setting the config.limit_by to header and writing a custom plugin to extract the subdomain and add it as a request header. I think this is a more general approach to implementing a dynamic limit_by.
And of cause, adding an extra built-in limit_by is something convenient for users. (Let's seek some opinions from other members to evaluate this PR)
(By the way, can you prepare a corresponding docs PR on https://github.com/Kong/docs.konghq.com, please? https://github.com/Kong/docs.konghq.com/pull/4626 is an example in case you need)

@vm-001 Added a PR to the docs repo if you still feel this PR might be useful
https://github.com/Kong/docs.konghq.com/pull/4962

@hbagdi What do you think?

Deferring the decision to @teb510

Closing this due to lack of activity. Please re-open if needed.


New trial template design for Katib

Fixes https://github.com/kubeflow/katib/issues/1341, https://github.com/kubeflow/katib/issues/1280.
Blocked by: https://github.com/kubeflow/website/pull/2312.
I have added doc about new Katib Trial Template.
Please take a look.
/assign @gaocegege @johnugeorge @8bitmp3 @RFMVasconcelos

@andreyvelich Hi, thanks for the PR! Can you do us a favor and run the Markdown text in all files through Google Docs with grammar and spelling checking? There may be around 10 things (like missing indefinite articles ("the") or misspelt words ("below")) that you can edit easily in the Trial Template Overview file alone, I think. This will definitely help speed things up! Thank you! 👍


Change the asynchronous implementation

Right now the platform uses a loop to monitor changes on activities and notifications, on Django 1.9 you have django channels, created to implement in a better way asynchronous communications.

I can work upon this @sebastian-code . Can I send a PR for it?

I am also thinking of django-channels. I thought of making a signals and listener for changes in the database when data is modified into it and trigger a specific user accordingly.

Ok, are you planning to use Redis or another messaging queue?

Redis or celery? I am having confusion? Can you suggest one

I apologize for creating it, the two of them cover different purposes, for django-channels would be Redis, but at the end, that doesn't matter, because it should be defined inside the .env file, and the code implementation is agnostic of the communication layer, so your approach is just perfect.

Solved. Closing


Fix variables input not being used

We are setting config.variableStrings using the value of the variables input.
However, since the Action uses synthetics.executeTests instead of synthetics.execute to run the tests, config.variableStrings is never parsed into config.global.variables (cf. this part in datadog-ci, which is only used when calling synthetics.execute).
This is why the variables set in the workflows are not actually used.
This PR fixes this behavior, by parsing the variables directly in the Action, using the synthetics.utils.parseVariablesFromCli function.

There is probably the same issue in the Azure DevOps task


Recursion issues while parsing complex avro schema nested records

Affected module

Ingestion Framework

Describe the bug
While parsing AVRO schemas from confluent schema registry following error appears when the schema is more complex:
[2023-09-27, 17:20:23 UTC] {avro_parser.py:261} DEBUG - Traceback (most recent call last):
  File "/opt/venv/lib/python3.8/site-packages/metadata/parsers/avro_parser.py", line 259, in get_avro_fields
    field_models.append(parse_single_field(field, cls=cls))
  File "/opt/venv/lib/python3.8/site-packages/metadata/parsers/avro_parser.py", line 210, in parse_single_field
    obj = cls(
  File "pydantic/main.py", line 339, in pydantic.main.BaseModel.__init__
  File "pydantic/main.py", line 1076, in pydantic.main.validate_model
  File "pydantic/fields.py", line 884, in pydantic.fields.ModelField.validate
  File "pydantic/fields.py", line 1101, in pydantic.fields.ModelField._validate_singleton
  File "pydantic/fields.py", line 1157, in pydantic.fields.ModelField._apply_validators
  File "pydantic/class_validators.py", line 337, in pydantic.class_validators._generic_validator_basic.lambda13
  File "pydantic/main.py", line 711, in pydantic.main.BaseModel.validate
  File "pydantic/main.py", line 339, in pydantic.main.BaseModel.__init__
  File "pydantic/main.py", line 1076, in pydantic.main.validate_model
  File "pydantic/fields.py", line 884, in pydantic.fields.ModelField.validate
  File "pydantic/fields.py", line 1101, in pydantic.fields.ModelField._validate_singleton
  File "pydantic/fields.py", line 1157, in pydantic.fields.ModelField._apply_validators
  File "pydantic/class_validators.py", line 337, in pydantic.class_validators._generic_validator_basic.lambda13
RecursionError: maximum recursion depth exceeded while calling a Python object
[2023-09-27, 17:20:23 UTC] {avro_parser.py:262} WARNING - Unable to parse the avro schema into models: maximum recursion depth exceeded while calling a Python object
[2023-09-27, 17:20:24 UTC] {local_task_job_runner.py:225} INFO - Task exited with return code Negsignal.SIGABRT

This cannot be worked around by try except blog as we get SIGABRT (because of the recursion issue) which is final and fatal.
To Reproduce
Expected behavior
Schema is properly parsed and ingested to be later visualized in UI.
Version:

OS: [e.g. iOS]
Python version: 3.8
OpenMetadata version: 1.1.5
OpenMetadata Ingestion package version: 1.1.5

Additional context
The error did not appear in 1.1.0 where parse_single_field was used to parse 'record' entries instead of parse_record_field. Recursion issues are somewhat handled in pydantic 2+ but I am not sure if:

pydantic upgrade to 2+ would be easy (there are breaking changes 1 vs 2)
https://github.com/pydantic/pydantic-core/pull/134/files this fix actually catches recursion issues instead of properly handling complex nested types

cc @OnkarVO7 @ulixius9


Feature : Remove Show Sidebar on No Distraction Mode

Removes "Show Sidebar" and "Hide Sidebar" from the View Menu when in No Distraction mode to enhance user experience (Issue #12129)

Instead of explicitly removing the menu items, they most likely should just enabled/disabled with commandManager#setEnabled
However, I am not entirely sure if #12129 is an entirely valid bug in the first place: opening the sidebar for a while to find a while seems somewhat valid use case, but it does bring some weird inter-op with the No-Distractions concept in general so I would be okay with disabling them too.
@abose opinions?

@bhldev  Thank you for contributing to Brackets. You will need to Accept the Brackets CLA before we can merge this pull request.

Allowing the user to open the sidebar was conscious decision we made as @petetnt said. This is to help with users to see the project contents even in no distraction mode for some time if he choose. Closing this PR as that was a design choice.
@bhldev If you are interested in contributing to features in brackets, please take a look at our trello board at https://trello.com/b/LCDud1Nd/brackets to see features requested by the community.


self - serve flow grapql error

two users reported this morning that they can't create a collective:
One is un GBP and the other one in AUD

It seems like it's not related to currency, another collective reported it, in USD

It looks like the API wasn't deployed with the new graphql method.
I've just redeployed it and it should work properly now.
Can you ask them to try again?
It works on staging:
btw, something tells me I'm gonna love this collective :-)

closing since issue is solved.


Better error output for complexFilter?

Maybe this is a limitation of ffmpeg but for some reason it doesn't give much information about why / where it failed in my code.  Maybe child process can't function this way or something... but figured I'd post this any way...
Thanks for all you do for the community!
events.js:85
      throw er; // Unhandled 'error' event
            ^
Error: ffmpeg exited with code 1: Error configuring filters.

    at ChildProcess.<anonymous> (/Users/Joshua/Projects/archiving-composer/node_modules/fluent-ffmpeg/lib/processor.js:167:22)
    at ChildProcess.emit (events.js:110:17)
    at Process.ChildProcess._handle.onexit (child_process.js:1067:12)

Try to add an 'error' handler and look at the arguments
ffmpeg(...)
  .on('error', function(err, stdout, stderr) {...})

You'll find more information there. And if you don't, post the result here (especially stdout/stderr)

Yep, thank you this works perfect.


Cannot read property 'width' of null when moving away from screen

I have an Ionic app that uses the latest versions of angular 4 and ionic 3.
I'm using PIXI (4.5.2) in one of the screens in it - the game screen.
The player frequently moves in and out of that screen.
Before starting the game I'm:
-Preparing resources
-Moving to game screen
-Adding the canvas (app.view) to DOM
Before moving away from the game screen I'm:
-Destroying resources and resetting the PIXI.loader
-Removing the canvas (app.view) from DOM
Most of the time everything works fine, but rarely when I'm moving away from the game screen I'm getting the following error:

Runtime Error
Cannot read property 'width' of null
Stack
TypeError: Cannot read property 'width' of null at Image.e.calculateVertices (http://localhost:8100/assets/imageSwapper/pixi.min.js:14:13368) at Image.e._renderWebGL (http://localhost:8100/assets/imageSwapper/pixi.min.js:14:14230) at Image.e.renderWebGL (http://localhost:8100/assets/imageSwapper/pixi.min.js:10:21822) at Swapper.e.renderWebGL (http://localhost:8100/assets/imageSwapper/pixi.min.js:10:21898) at e.renderWebGL (http://localhost:8100/assets/imageSwapper/pixi.min.js:10:21898) at e.render (http://localhost:8100/assets/imageSwapper/pixi.min.js:13:8590) at t.render (http://localhost:8100/assets/imageSwapper/pixi.min.js:10:10909) at t.emit (http://localhost:8100/assets/imageSwapper/pixi.min.js:16:7855) at t.update (http://localhost:8100/assets/imageSwapper/pixi.min.js:16:6834) at _tick (http://localhost:8100/assets/imageSwapper/pixi.min.js:16:5017) at t.invokeTask (http://localhost:8100/build/polyfills.js:3:12256) at Object.onInvokeTask (http://localhost:8100/build/main.js:4539:37) at t.invokeTask (http://localhost:8100/build/polyfills.js:3:12177) at n.runTask (http://localhost:8100/build/polyfills.js:3:7153) at invoke (http://localhost:8100/build/polyfills.js:3:13213)

I don't know yet why this error only rarely occurs and why. What do you think?

Would it be possible to get the unminified stackstrace? Also, could you give a code snippet of how you're destroying the app?

For some reason I'm not able to reproduce this issue anymore, if and when this happens again I'll send here the stack trace of the unminified version.

Sounds good. I'm going to close this issue, if it happens again and there's more info, please reopen.

I finally have an unminified stacktrace:

core.es5.js:1084 ERROR TypeError: Cannot read property 'width' of null
    at Image.calculateVertices (Sprite.js:211)
    at Image._renderWebGL (Sprite.js:295)
    at Image.renderWebGL (Container.js:409)
    at Swapper.renderWebGL (Container.js:414)
    at Container.renderWebGL (Container.js:414)
    at WebGLRenderer.render (WebGLRenderer.js:318)
    at Application.render (Application.js:128)
    at TickerListener.emit (TickerListener.js:78)
    at Ticker.update (Ticker.js:394)
    at Ticker._tick (Ticker.js:113)
    at t.invokeTask (polyfills.js:3)
    at Object.onInvokeTask (core.es5.js:4116)
    at t.invokeTask (polyfills.js:3)
    at n.runTask (polyfills.js:3)
    at invoke (polyfills.js:3)

Guessing it's this line: https://github.com/pixijs/pixi.js/blob/dev/src/core/sprites/Sprite.js#L215
If you are able to reproduce this again, could you log out texture and orig?

console.log(texture, orig); - orig is null

Is it possible to make a jsfiddle which reproduces this error?

It's a bit difficult and cumbersome because we're talking about an ionic app that wraps this.
Do you think it's possible to solve it without a reproduction? Or otherwise by remote control?

Is it possible you are destroying the Texture and then rendering? Or destroying resources but not removing children from your app? Are you using app.destroy?

Yup.
-Destroying resources and resetting the PIXI.loader
-Removing the canvas (app.view) from DOM
Apparently PIXI kept rendering after canvas was removed from DOM?
Now whenever I'm destroying textures I'm removing the sprites that are using them. I think it's solved. Thank you :)

Sounds good 👍


autoType is not support

com.alibaba.fastjson.JSONException: autoType is not support. org.springframework.security.core.authority.SimpleGrantedAuthority

111


EurekaInstanceRegisteredEvent not dispatching as intended

I went through the source code and figured out that eureka dispatches the following 3 events from springs ApplicationContext

EurekaInstanceRenewedEvent
EurekaInstanceRegisteredEvent
EurekaInstanceCancelledEvent

Of these only the EurekaInstanceRenewedEvent works and the listener is triggered when the server receives a heartbeat. Other events are not dispatched when a new instance is registered or canceled from eureka.
I think InstanceRegistry is overriding the wrong register method.
The ApplicationResource controller calls the register method in the  InstanceRegistry when a new instance is registered. But the call goes to the super class of InstanceRegistry, bypassing the event dispatch. Not sure if this is intended and this is how it is supposed to work. If it is then what is the use of the event and how can it be used?

You are right. org.springframework.cloud.netflix.eureka.server.InstanceRegistry class extends PeerAwareInstanceRegistryImpl which has register(final InstanceInfo info, final boolean isReplication) which is not overridden in InstanceRegistry (so it does not have any event producing). This method is being called addInstance method from com.netflix.eureka.resources.ApplicationResource class. I think this method should be overriden.

I almost have a working solution. I will make a pull request soon.

good


Add option to paste animation as duplicate

Might resolve #24950

(this image is outdated, see below for new one)
@avencherus
@golddotasksquestions
EDIT:
Added another commit that alternatively makes Paste duplicate by default and adds "Paste As Reference" option for current behavior.

Great, I'll check it out, and very likely start using it immediately.  It certainly should resolve the issue, as no option even exists.  Much appreciation for this.
Though I don't know if the UI presentation side of things were ever settled.
I will just add that there are non-programmers that end up working with the AnimationPlayer, and in my experience when I have tried to explain the whole Animations are a Resource business, eyes get glassy.  Later they're just paranoid about using the tool, and duplicating and double checking everything.
I don't animate a lot, so almost always when I return to it, I make this mistake myself of copy / paste, start editing... uh oh.
I think the major problem first is that the word Copy is already implying a Duplicate.
It's my feeling that for a tool like this, copying and pasting resource references is a secondary operation.  Duplication seems to be the most intended, as copy and pasting in this workflow context is more like templating.  Reaching for something similar and reworking it.
IE - Take enemy A's floating animation into enemy B, and tweak it a little.
So to add to any following discussions (if any).
I would say it probably might even be best to start with the implications of the word Copy.
Copy
Copy (As Resource)
Paste
Leaving that option there for those who know what is what regarding Resource behavior.
As others have mentioned as well, there is still some ambigiouty about in the AnimationPlayer about when things are being done to a shared resource or a unique one.
The tools seems to be trying to hide the back end implementation (Animations as Resources), for user friendliness, but then has these behaviors that are invisible and baffling to non-programmers.

@avencherus Why should there be two Copy actions?
One Copy is fine, with Paste and Paste as Reference instead.
Implementation-wise it would copy a reference first, and then if Paste is selected it would perform a deep duplicate after navigation to the reference pointer, or if Paste as Reference is used it will put the reference as-is.

@asheraryam Great points.
The initial thinking was to express the intention of the action with the first choice.  If having referential copying at all.  It is currently a bit of a dangerous feature, since there are no accompanying indicators or persisting expectations of which is which.  One might have 1 shared Animation among a list of 20 Animations, and return to editing them a week later.
I stopped short of Paste / Paste as Resource in my post, but reflecting on it, I think from a user perspective (implementation aside) it should be better.  One is probably traveling to another location in the editor to do this, and it may not be entirely clear what is on the clipboard if these two options exist.  (The Paste option would have to contextually change.)  The Paste is the ending action and there it seems most clarifying.
With maintaining both, Paste / Paste as Reference sounds better to me at least.
I was thinking about that naming too, Paste as Resource Reference seems like the most clear, but wow that poor box is already being stretched out quite awkwardly since everything else is very terse.   X)

Added another commit that does this in alternative way (see OP).

Added another commit that alternatively makes Paste duplicate by default and adds "Paste As Reference" option for current behavior.

Thank you!

@KoBeWi Is this still desired? If so, it needs to be rebased on the latest master branch.

Rebased refactored and squashed.

Thanks!


BIGTOP-2476 Add Zookeeper Charm

Based on the vanilla apache-zookeeper charm, but utlizes Bigtop to
reduce complexity in the charm itself.
This covers all the changes that we discussed, including requiring a manual rolling restart after adding or removing zk peers (per http://www.benhallbenhall.com/2011/07/rolling-restart-in-apache-zookeeper-to-dynamically-add-servers-to-the-ensemble/)

LGTM, verified deployment and smoke tests on azure and aws.


MaaS: Updated process.yml

This patch updates process.yml to work well for Xenial and Trusty
and makes use of the existing psutil and lxc python modules.
Connects rcbops/u-suk-dev#1233

Also seeing some oddness when running the plugin via the venv:
root@rpc-openstack:/usr/lib/rackspace-monitoring-agent/plugins# ./run_plugin_in_venv.sh /usr/lib/rackspace-monitoring-agent/plugins/process_check.py blah blah2 blah3
status okay
metric blah_process_status uint32 1
metric blah2_process_status uint32 1
metric blah3_process_status uint32 1
root@rpc-openstack:/usr/lib/rackspace-monitoring-agent/plugins# /usr/lib/rackspace-monitoring-agent/plugins/process_check.py blah blah2 blah3
status okay
metric blah_process_status uint32 0
metric blah2_process_status uint32 0
metric blah3_process_status uint32 0

I'm seeing the venv oddness as well. Taking a look.

So the parent venv wrapper script was appearing in the results. I added a check for that.
root@rpc-openstack:/usr/lib/rackspace-monitoring-agent/plugins# ./run_plugin_in_venv.sh /usr/lib/rackspace-monitoring-agent/plugins/process_check.py blah blah2 blah3
status okay
metric blah_process_status uint32 0
metric blah2_process_status uint32 0
metric blah3_process_status uint32 0
root@rpc-openstack:/usr/lib/rackspace-monitoring-agent/plugins# ./run_plugin_in_venv.sh /usr/lib/rackspace-monitoring-agent/plugins/process_check.py swift-object-server
status okay
metric swift-object-server_process_status uint32 1


fix: General fixes + feat: Fastify + MercuriusDriver

Do you mean to merge into main branch?
Since classic chain still has oracle, market, treasure module.


Setup express app

Integrate Express
Description
Since this is a MEAN application we need to set up node JS and express JS for the back end
Definition of Done
The Express app should be up and serving in its own port. It should be able to accept any incoming request from the angular app

Setting up the project with the express app in typescript.
Reference doc: https://typeofnan.dev/your-first-node-express-app-with-typescript/

Basic endpoint to just give response

And server is running in local

Linter and formatter configs are setup: https://github.com/suvadeepghoshal/travel-vault/commit/bd18e02b0330860a1572e13d2e0b12081939a35f

Angular is successfully able to communicate with the express app


Why no mentioning of Azure Data Explorer?

Why isn't Azure Data Explorer mentioned as a viable alternative for storing time-series data? https://azure.microsoft.com/en-us/services/data-explorer/

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

ID: 87bde15d-d343-b84d-b77f-55b8e3f12638
Version Independent ID: 9d35a3eb-707b-47f6-68bc-3f22fee4c78c
Content: Time series data
Content Source: docs/data-guide/scenarios/time-series.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @ZoinerTejada
Microsoft Alias: pnp

Hi @olavt Thanks for your feedback. We will review and update as appropriate.

AB#160222 - Thanks for reporting - this issue is under review

Added in #2059


SlackWebhookHandler: use footer for username & footer_icon for userIcon

related to https://github.com/Seldaek/monolog/issues/1247 & https://github.com/Seldaek/monolog/issues/1332
Since userIcon & username cannot be overridden, the text & smaller icon in footer can be used instead.

There are quite a few skipped tests. They all seem to be related to external database/cache thus do not concern SlackWebhookHandler.
PHPUnit 8.5.21 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.2.24-0ubuntu<IP_ADDRESS> with Xdebug 2.6.0
Configuration: /var/www/html/monolog/phpunit.xml.dist

.............................................................   61 / 1114 (  5%)
...................................................SS........  122 / 1114 ( 10%)
...........S................SS......S........................  183 / 1114 ( 16%)
............................................S................  244 / 1114 ( 21%)
.............................................................  305 / 1114 ( 27%)
.............................................................  366 / 1114 ( 32%)
.............................................................  427 / 1114 ( 38%)
.............................................................  488 / 1114 ( 43%)
...................SS........................................  549 / 1114 ( 49%)
.............................................................  610 / 1114 ( 54%)
.............................................................  671 / 1114 ( 60%)
.............................................................  732 / 1114 ( 65%)
.............................................................  793 / 1114 ( 71%)
.............................................................  854 / 1114 ( 76%)
.............................................................  915 / 1114 ( 82%)
.............................................................  976 / 1114 ( 87%)
......S...................................................... 1037 / 1114 ( 93%)
............................................................. 1098 / 1114 ( 98%)
................                                              1114 / 1114 (100%)

Time: 3.07 seconds, Memory: 339.20 MB

There were 10 skipped tests:

1) Monolog\Handler\AmqpHandlerTest::testHandleAmqpExt
amqp-php not installed

/var/www/html/monolog/tests/Monolog/Handler/AmqpHandlerTest.php:26

2) Monolog\Handler\AmqpHandlerTest::testHandlePhpAmqpLib
php-amqplib not installed

/var/www/html/monolog/tests/Monolog/Handler/AmqpHandlerTest.php:82

3) Monolog\Handler\CouchDBHandlerTest::testHandle
Could not connect to couchdb server on http://localhost:5984

/var/www/html/monolog/tests/Monolog/Handler/CouchDBHandlerTest.php:28

4) Monolog\Handler\ElasticaHandlerTest::testHandleIntegration
Cannot connect to Elastic Search server on localhost

/var/www/html/monolog/tests/Monolog/Handler/ElasticaHandlerTest.php:192

5) Monolog\Handler\ElasticaHandlerTest::testHandleIntegrationNewESVersion
Cannot connect to Elastic Search server on localhost

/var/www/html/monolog/tests/Monolog/Handler/ElasticaHandlerTest.php:246

6) Monolog\Handler\ElasticsearchHandlerTest::testHandleIntegration
Cannot connect to Elasticsearch server on localhost

/var/www/html/monolog/tests/Monolog/Handler/ElasticsearchHandlerTest.php:210

7) Monolog\Handler\MongoDBHandlerTest::testHandleWithDriverManager
Could not connect to MongoDB server on mongodb://localhost:27017

/var/www/html/monolog/tests/Monolog/Handler/MongoDBHandlerTest.php:73

8) Monolog\Handler\ZendMonitorHandlerTest::testWrite
ZendServer is not installed

/var/www/html/monolog/tests/Monolog/Handler/ZendMonitorHandlerTest.php:24

9) Monolog\Handler\ZendMonitorHandlerTest::testGetDefaultFormatterReturnsNormalizerFormatter
ZendServer is not installed

/var/www/html/monolog/tests/Monolog/Handler/ZendMonitorHandlerTest.php:24

10) Monolog\Processor\MercurialProcessorTest::testProcessor
hg is missing

/var/www/html/monolog/tests/Monolog/Processor/MercurialProcessorTest.php:29

OK, but incomplete, skipped, or risky tests!
Tests: 1114, Assertions: 1896, Skipped: 10.

Thanks! I will go through this and the rest of the pull requests tomorrow. Today I have a crazy day.
Thanks for your effort

sorry for the delay.
Question ... I cannot test this honestly ...but how does it look when you do have something like:
'footer' => null,
'footer_icon' => null,

in the attachments of a record, but they are NULL ... what will happen? I am worried that maybe Slack will complain, or the outlook will not be good. Can you tell me? In case of errors, should these fields be added in demand?
Someone else has an opinion on this?
OR
can you help me setting up my machine in order to test this ? What do I need?

sorry for the delay.
Question ... I cannot test this honestly ...but how does it look when you do have something like:
'footer' => null,
'footer_icon' => null,

in the attachments of a record, but they are NULL ... what will happen?

The content in the green box disappears. Only time is present.

Thanks for Your effort. I believe @Seldaek organización another mánager will join during this Week maybe to take Care about the pull requests in the queue.
Have a great day

Thanks


Elastsearch gather-log updates

Description
Some improvements to the Elasticsearch gather-log files I noticed while looking at updating the reporter.

Use curl -sS so we can get any errors that might happen when talking to ES
Filenames should use _ instead - to match what gets created by chef-automate gather-logs
Added an additional output for _cluster/state which we use to detect if the cluster is read-only due to the disk filling up.

Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

Tested on my server install. This works as expected and gets us data that matches our pattern and automate's gather logs.


Add ice flag dreamsicle plot code

This code creates this PNG within an R Markdown targets pipeline:

Polished viz in InkScape:

Super cool 🤩 . Tiny typo in case someone else doesn't catch it: "it does not represent ~of~ when and where streams freeze"

Fixed!


other grammar/spelling, and small clarification

Peter, thanks for the fixes but XML isn't what PR should be opened to. The draft.md file is where text being edited, XML is generated by tooling and is never edited directly, since every automated build will simply overwrite changes in the XML.

Thank you! Fixed by #49


.pull-right and .pull-left on [class*="span"] does'nt work anymore

I was using .pull-right and pull-left on.span* div to reorder columns visually, but a change (float:left on [class*="span"]) is blocking the .pull-right/left to work...
Thanks...

Those were never intended to be used on the grid columns, and there is already #1215 open which asks for support on this. Look for that in 2.x.


RestEase.SourceGenerator not using *.g.cs in generated source files.

Description
A clear and concise description of what the bug is. Use screenshots as necessary.
I am utilizing the source generator to generate my API clients.  This unfortunately causes
dotnet test --configuration Release --verbosity normal --collect:"XPlat Code Coverage"
To skip generating coverage for my library containing the interface with errors like:
[coverlet]Unable to instrument module: , pdb without local source files, RestEase_*.cs

Looking at coverlet they ignore all source generated files following Microsoft's recommendation, for example: RestEase_*.g.cs.
I know this is not necessarily a bug in this project, but was wondering if you wouldn't mind adding the g.cs?  I think it is being done here.   If it is, I could open a PR to add it if you would like?
To Reproduce
Can use samples from your main ReadMe to generate.  Then run the:
dotnet test --configuration Release --verbosity normal --collect:"XPlat Code Coverage"
Version Info

RestEase version: 1.5.5
Target framework version: net6.0

Additional Info
Add any other context about the problem here.

Sure, I don't see why not. I'm slightly surprised that the source generator machinary doesn't do this by default, but other MS SGs seem to have run into the same problem and are manually appending .g as a workaround.

Awesome @canton7, much appreciated on the quick turn around on that.  Tried it out today and works beautifully.
And I agree, seems weird the SG engine doesn't automatically handle that for ya.


Document use of local plugins

Description
Documentation on plugin configuration insufficient for me to configure Gridsome to use a local plugin which is a helpful thing to know when developing a plugin.
Initially I was trying to use a path relative to gridsome.config.js but was getting a Error: Cannot find module.
use: './plugins/my-amazing-plugin'
After some digging through loadConfig.js it seems a tilde is needed to resolve to the project root.
use: '~/plugins/my-amazing-plugin',

There was some documentation for it, but I can't find it now either... It seems like the "How to use plugins" page is redirected to /plugins instead.
Also, Gridsome should support ./ paths too. Not sure why only ~/ was implemented.

Also, Gridsome should support ./ paths too.

It does when using require like this: use: require('./src/plugins/gridsome-plugin-pagination'),


Incorporating more values

Hi,
obviously your project was tailormade for you needs (running) which is perfectly fine. I want to add some additional values from my activities, especially "rear_gear_change" (cycling). Problem is, that this is not found in "records" but in "events". Events have their own timestamp, so I would have to somehow add these timestamps to the timestamps of "records" and display "rear_gear_change" then in the map. To be honest I would need some push in the right direction from you .... If needed I can provide the relevant part of my fit file. Thanks a lot!

Hi

If needed I can provide the relevant part of my fit file

Yes, please send the *.fit file. I am travelling till the Russian New Year (Jan 14) being now in Russia, but since I have an access to the gmale and Git, I can think how to fix it.
Happy New Year!

I invested some time today to understand the functionality, but I did not find the exact point where to insert the new values. I see that I have to create a new array in "chartdata" (function getDataFromFitForCanvasJS). From what I see in the code the array comes from the options in ylist. But I did not clearly see where the list is filled from. This is probably due to the fact that I already modified the code a little bit to fit my needs (I have 2 arrays in chartdata: heart rate and speed).
This is how my fit file looks like after parsing. Shortened and redacted it due to privacy reasons, but it should be enough to see how the values look like:
{
  "activity": {
    "sessions": [
      {
        "laps": [
          {
            "records": [
              {
                "timestamp": "2023-12-29T10:36:05.000Z",
                "elapsed_time": 0,
                "timer_time": 0,
                "position_lat": ...,
                "position_long": ...,
                "distance": 0.00027,
                "altitude": 0.04660000000000002,
                "speed": 0,
                "heart_rate": 159,
                "temperature": -257.15
              },
              ...
            ],
            "lengths": []
          }
        ]
      }
    ],
    "events": [
      {
        "timestamp": "2023-12-29T10:36:05.000Z",
        "data": 0,
        "event": "timer",
        "event_type": "start",
        "event_group": 0
      },
      {
        "timestamp": "2023-12-29T10:36:05.000Z",
        "data": 671159047,
        "event": "rear_gear_change",
        "event_type": "marker"
      },
      ...
    ],
    "hrv": [],
    "device_infos": [
    ],
    "developer_data_ids": [],
    "field_descriptions": [],
    "sports": [
      { "name": "GRAVEL", "sport": "cycling", "sub_sport": "gravel_cycling" }
    ]
  }
}

The vaule most interesting is in "events" and has the event type "rear_gear_change". I have to check in the official Garmin SDK what kind of data this is because of course this should be an integer between 1 and 12.
So it would be very nice if you could give me a hint how to add these values as a new array in chartdata. I want to upgrade my bike also with a powermeter, so it would be useful for me to understand how the chartdata object is created in order to be able to add power data on my own without having to bother you again.

but I did not find the exact point where to insert the new values

The metrics are collected automatically by scanning the fit file, function prepareFitData, line 645

Probably,   you can add into the function prepareFitData a cycle like
for (let k in data.events) { ...}
to fill the needed metric from events section

If your metric is the averaged one between events, then the x-axis must be laps.
Select in x-axis droplisr lap-number, and then check metrics in the y-axis droplet.

This does not work. If I change to lap_number, I have the options "event" and "event_type" in the dropdown. Both options show no data in the plot. I do not know what you mean by "metric is the averaged one between events". Obviously the "rear_gear_change" is an event that occurs when I change gears on the bike. So the timestamp does not match with the timestamps in records (only by a mere coincedence). So in the example from my post above, I did change gears here:
{
        "timestamp": "2023-12-29T10:36:05.000Z",
        "data": 671159047,
        "event": "rear_gear_change",
        "event_type": "marker"
},

data is uint8z according to the FitSDK. Will try to add a for loop, only the x field worries me. I have to check how you calculate this out of distance.

what you mean by "metric is the averaged one between events"

It was my mistake. The section events is not checked in  prepareFitData.

Having a hard time to figure out how it could work ...
Added the following right after the loop for data.laps (around line 683):
for (let k in data.events) {
  if (data.events[k].event == 'rear_gear_change') {
    let event = data.events[k];
    D = new Date(event.timestamp);
    data.records[k].timestamp = D.getHours() + D.getMinutes() / 60 - timeoffset;
    data.records[k].gear = event.data;
  }
}

Now I have gear data in data.records. Problem now is that this data is not correctly shown in the chart. In my already modified view I only show speed and heart_rate on the y-axis (now gear as well) and distance on the x-axis. While speed and heart_rate appear in the chart at the right place related to distance, gear values are shown more or less one after the other right from the beginning until the amount of gear changes is reached (~50). So gear values are not properly displayed in relation to distance. So data.records[k].timestamp is wrong which is quite obvious as the counters (k) for records and events do not match . How and where do I have to make modifications to get the correct x-value (meaning pushing the gear change events to the correct place in the records array)? Second question: is there already a helper function somewhere in the project for converting uint8z values to correct values?

Explore my function getDataFromFitForCanvasJS, line 216., it is called from line 396.
You see there a check for x-axis, e.g. line 218. Introduce a flag variable for the gear, e.g.
if (fieldx === "timestamp_gear"){ here is the code to put data for the chart, put here gear's timestamps, as in line 221}
The second question

data is uint8z according to the FitSDK. Will try to add a for loop, only the x field worries me. I have to check how you calculate this out of distance

Do you mean out of the track (if not give me more explanations what do you want)? You have in the *.fit file
{
"timestamp": "2023-12-29T10:36:05.000Z",
"data": 0,
"event": "timer",
"event_type": "start",
"event_group": 0
},
the event_type": "start", and the corresponding timestamp. Check other timestamps with the "start" timestamp, to recognize whether the gear change before or after "start" event.

More confused than before. What is the difference between the code I entered around line 683 and the code you proposed in 218? I tried some variations, but no luck.
Just use one of your own .fit files. I had a look to the one in this repo (2021-01-26T17_13_43+00_00_6171809523.fit). There are 8 elements in the events array. My array looks like this:

So if you use for example the "undefined" events in your array as an equivalent to my "rear_gear_change" event, you can try to incorporate those values.  This would be really of great help as I am stuck somehow at the moment.


Upgrade local backbone version

Readiness checklist

[ ] I added/updated tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

as a preparation for bumping the backbone in #175


MIgrations fail on install

Install shopper in a clean Laravel 10 project

composer require shopper/framework --with-dependencies
php artisan shopper:install

The issue:

A workaround is described here: #224


feat(notification)!: remove iconDescription props for InlineNotification and ToastNotification

Closes #672
Closes #1563
The InlineNotification and ToastNotification components get revamped label treatment for their icon, to support a11y and i18n.

BREAKING: remove iconDescription prop
Feat: add statusIconDescription prop
Feat: add closeButtonDescription prop

I'm also piloting some handy documentation links: "detailed usage" and "see also".

Released in v0.71.0


Rewrite construct_df and add tests

Closes #16, #14, #9

Function construct_df is rewritten to work with the most recent pandas version.
Add infrastructure to run tests in docker containers
Add github actions to run tests using different Python versions (only done if github actions is enabled for this repository, free for public repositories)

See e.g. https://github.com/gedeck/doepy/actions/runs/3165946032 how github actions work

Hello! What's the status on this? :)

I don't know. I use the package in an upcoming book (https://gedeck.github.io/mistat-code-solutions/IndustrialStatistics/) and was hoping that it would get fixed soon. Unfortunately, this hasn't happened yet, so I incorporated the code into the package that accompanies the book (https://github.com/gedeck/mistat, https://pypi.org/project/mistat/; import using from mistat.design import doe).
I'm not happy with this solution but it was the best I could do for now.

Thanks for your swift response, @gedeck!!


Broken PyPI metadata and related problems

Look at https://pypi.org/project/ansible-review/#description, it shows:
Project description
The author of this package has not provided a project description

This probably means that you use outdated setuptools when building the dist.
There's also a few other problems I see:

setup.cfg refers to README.md, but RST is default for warehouse. To make it understand makrdown you need to provide extra setting with mime type specified correctly (and use up-to-date setuptools)
PyPI page would also benefit from project_urls metadata option
It's better readable when you keep metadata in declarative config. Here's example shim for making old setuptools understanding how to read that: https://github.com/cherrypy/cheroot/blob/master/setup.py
You publish only sdist, while per current recommendations you should also publish a universal wheel as well. This would also save end-users from having to have modern setuptools in place.
Version is hardcoded, alternatively it could be stored in a Git tag. There's a plugin called setuptools-scm, which helps to handle this smoothly.
Manual building/publishing is error-prone (because of human factor). I suggest changing this towards using CI/CD (the project is about automation, but doesn't have automation in place? hmm..)

So if you're open to such changes let me know as I've gotten a very good experience with all I mentioned above and I can send a PR or/and consult you on those if you have any questions/conserns.

Hm.. it's curious that some old versions (https://pypi.org/project/ansible-review/0.9.0/#description) have wheels and description published (misrendered markdown though)

We (Ansible) can fix this in ansible-lint then raise a PR to update it here in ansible-review as well

Yep, working on submitting a similar issue to ansible-lint now.


Metronome 0.3.6 bump with docker params

High-level description

Adds Docker Param support to Metronome for Jobs.

Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-3688 Release Metronome 0.3.6  on DCOS 1.9.

Related tickets (optional)
Other tickets related to this change:

DCOS_OSS-2564 Allow Docker Params for Job Runs

DCOS_OSS-2519 Metronome allows creating jobs with no command and no image.

Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): change log
[x] Test Results: https://jenkins.mesosphere.com/service/jenkins/view/Marathon/job/marathon-team-releases/job/metronome-release/51/
[ ] Code Coverage (if available): [link to code coverage report]

This repo has @mesosphere-mergebot integration. You can interact with the following commands.
@mesosphere-mergebot bump-ee
@mesosphere-mergebot merge-it
@mesosphere-mergebot override-status
@mesosphere-mergebot label [Ship It |Holding |Work In Progress |Ready For Review |Request For Comment]

PR creators can apply one of  [Ready For Review |Work In Progress]. Owners can apply any label.

@mesosphere-mergebot bump-ee

This release allows docker params to be set for jobs... and is tested working via API/CLI.
There is a need for a UI update in order to have UI allow for these params (even in JSON edit mode).  It also needs to be fixed for "editing" a job submitted for API.
It isn't clear if that UI update will be added to this PR or be on another PR.

@mesosphere-mergebot label review for review

:wave: This PR has been inactive for 3 days. Please review the status checks to see what needs to be done to move it forward or change the label to Work in Progress. Thank you!

:wave: This PR has been inactive for 3 days. Please review the status checks to see what needs to be done to move it forward or change the label to Work in Progress. Thank you!

@mesosphere-mergebot merge-it


LSC Bulb e14

Hi,
i bought 2 lsc bulb e14 last month and 3 plugs. I can Flash tasmota.
But yestarday, i buy 2 other bulbs and it's impossible. I have an error "devide did not appear with the intermediate firmware"
We have error with ssl in smarthack-psk.log
thanks you for your help

Please follow the instructions and post your logs, without doing this I cannot help

Hi
Sorry, it's not a good Idea to publish issue between 2 professionnal meetings. 😉
smarthack-wifi.log
smarthack-web.log
smarthack-udp.log
smarthack-psk.log
smarthack-mqtt.log
Thanks you

Hi cognacais, did you buy the LSC E14 color ones? They seem to be Realtek not ESP. kueblec can you confirm?

Just found out that it should work:
https://templates.blakadder.com/lsc_smart_connect_C45.html
Anyway I get the same message as cognacais: "Device did not appear with the intermediate firmware"

Hi kbfifi
Yes, it's this model. And i flashed 2 bulbs last month and integrated to home assistant.

Meanwhile I can confirm it's an esp by a simple network scan on the vtrust-flash network. Probably there's a firmware update that needs to have a new solution. @kueblc I read something about a product key?

To be complete my files as well:
smarthack-mqtt.log
smarthack-psk.log
smarthack-udp.log
smarthack-web.log
smarthack-wifi.log

Just found out that it should work:
https://templates.blakadder.com/lsc_smart_connect_C45.html
Anyway I get the same message as cognacais: "Device did not appear with the intermediate firmware"

This doesn't mean it should work, as manufacturers have been using the same model numbers and FCC ID even if they change the chipset. It is quite possible that a device on the template repository has been switched to a Realtek or another chip.

Probably there's a firmware update that needs to have a new solution. @kueblc I read something about a product key?

You're probably talking about #483, where the PSK format changed, which is not the same as a product key. The product key simply identifies the product, it is not a key in the cryptographic sense. All devices of the same make and model share the same product key.
For both of you @cognacais and @kbfifi, it looks like the process stopped at the token registration stage, which likely indicates the plaintext response is being rejected. This is probably a new patch by Tuya, so there's unfortunately not much we can do at this time. Hopefully we can develop a workaround.

Thanks for your comments. Is there something I can do to speed this up?

Thanks you @kueblc for your job. There is any Hope ?
I have the same problem with this model https://templates.blakadder.com/calex-429004.html
I don't have log here. I Can post logs in 2 weeks (vacation 😉)
Same that @kbfifi if i Can help.

Hey Guys,
may I can give some additional information. I bought some LSC RGBW E14 bulbs and successfully flashed them with tasmota some weeks ago. Then the bulbs were sold out for weeks in kinda all markets on the Niederrhein. Now I bought 2 additional bulbs and flashing is not possible anymore.
Maybe someone should remove them from compatible devices list?
I don't believe they changed chip to Realtek since macvendors.com says it's still Espressif Inc.

Unless they are not or  no Ionger flashable by the native app, this problem is solvable I think. @kueblc Let me know if I should make tcpdump or so

Update: Now I also bought the LSC E14 smart filament LED. I successfully flashed Tasmota with tuya-convert. However Tasmota does not connect to my network

Hey guys,
i have an information. The model 899807 (lsc e14) doesn't change but version has changed (V2 to V2.3)
https://templates.blakadder.com/lsc_smart_connect_C45.html
I found lsc e14 V1 do you know if it's possible to flash tasmota?
Thanks you

I can confirm that the LSC C45 400lm RGBW Bulb v2.3 (look underneath the box) did not want to convert. My logs also show similar messages as to what @cognacais already posted.
I was able to convert the v2.0 version just fine.

Hi guys
Just for information bulbs v1 are flashable.
Have a good day

Dear Team and all,
i have the same issue with a newly bought LSC Smart Light Strip (Action).
I can see in the logs that my mobile and a second device (most likely the strip, mac 68:57:2d:a2...)
connect to the AP of my Raspi, but then the script stucks in repeating
"Resending SmartConfig Packets
..................
SmartConfig complete.
Resending SmartConfig Packets
.................."
and finally:
"Device did not appear with the intermediate firmware"
I tried several times in slow blinking  and fast blinking mode (3 times powercycle)
In fast blinking mode the blinking stops while the script runs, which indicates that
something happens. The device was never connected to the app or another network before.
Is it confirmed that the devices are delivered with a new stock firmware that
prevents tuya-convert to work?
Thanks and best regards!

This bug report is not about the light strip but about the LSC e14 color bulb.
Have a look here:
https://github.com/ct-Open-Source/tuya-convert/issues?q=is%3Aissue+lsc+strip+is%3Aclosed+

There is also a new PIR sensor what indicates exactly the same problem like above. It´s a new 360degrees sensor. It´s technical name is WiFi-702 PIR. Also sold by Action with the LSC brand. See http://go.quwl.net/h-pd-1221.html for a chinese product page.

Any updates on this problem? I also bought soms v2.3 bulbs and I hope there is a solution for it already...


feat: labels will auto fit wrapper width

Before

After

Pull Request Test Coverage Report for Build 97

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.158%

Totals

Change from base Build 96:
0.0%

Covered Lines:
29

Relevant Lines:
62

💛  - Coveralls


cmdline: Hitting backspace with empty command closes the popup

Describe the bug
When you hit backspace and there is no text entered, the cmdline_pop closes. This is surprising when you are spamming backspace to clear out the cmdline when making a spelling mistake. I would expect that enter and esc are the only keys that would exit the popup.
Which version of Neovim are you using?
NVIM v0.8.0-dev-1093-g982fef601
To Reproduce
Steps to reproduce the behavior:

type :
hit Backspace

Expected Behavior
Popup remains open
Screenshots

That's standard Neovim behavior. Try the same without Noice. Cmdline closes when you hit backspace

Wow, you are right 😅. Sorry about that. I am not sure why felt like I had not experienced that before.


Figure out how to add ApolloBridge for migrated SSR App

react-effect-apollo is necessary when using the Query component from react-apollo or @shopify/react-graphql.
I imagine this will be a pattern we want to support for current app that uses Query component but want to upgrade to SSR.
<ApolloBridge> should be wrap around the app in both decorate function and streaming respond
I think we have 3 choices

expect the user to wrap it themselve in <App/>
offer it as part of @shopify/react-graphql-universal-provider either by default or as an option.
offer it as part of @shopify/react-server either by default or as an option.

4th option is having the webpack plugin include it.


GearVR / Google Cardboard Support

Is there any plan to add GearVR or Google Cardboard support?

Not yet, but pull-requests and discussions welcome ;)
I would prefer a solution that supports both the 'Mobile VR' SDKs as well as WebVR, other then that I haven't spent much thought yet how VR support would best be integrated into Oryol. A few question would be (apologies if those don't make sense, I haven't actually looked at the VR SDKs yet):

What would be the required minimal changes to the Gfx module?
What about the position tracking? Better have a new 'input device' in the Input module, or have a separate 'VR' module?
What other features besides rendering and position tracking need to be exposed, and where should they live?


Fix error in Conviva type definitions when Yospace is not installed

Previously, TypeScript users would get the following error when they've installed the Conviva connector without the (optional) Yospace connector dependency:
node_modules/@theoplayer/conviva-connector-web/dist/types/integration/ConvivaConnector.d.ts:3:39 - error TS2307: Cannot find module '@theoplayer/yospace-connector-web' or its corresponding type declarations.

3 import type { YospaceConnector } from '@theoplayer/yospace-connector-web';
                                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This PR works around this issue by adding a @ts-ignore comment above that import. If the dependency isn't installed, the imported type resolves to any, which is good enough.
Note that we need to use a JSDoc-style comment /** @ts-ignore */ in order for TypeScript to preserve it during emit. A bit hacky, but it works. 😛

@tvanlaerhoven reported that there was still a runtime dependency on the Yospace connector, which Webpack didn't like.
This should also be fixed now.