Training script

@alecgunny @eric-moreno
Right now, the training script takes in 'pre-loaded' RandomWaveformDatasets for training and validation.
Where do we want to this 'pre-loading' (e.g. loading in hdf5 files, setting glitch fraction, batch_size, etc).
I see that DeepClean has a training project that seems to handle this. Is this something we want to adopt, and if so, should it be included in this pull reqeust?

Waiting on implementation of architecture library to fully test wrapper functionality.

This all looks really good. My only comment would be around the point you bring up about the dict data arguments to the train function. I go back and forth on this a lot too, but my general gut is that dicts that are supposed to have specific fields are just prone to errors and typos and are generally nonintuitive (to someone reading the code trying to understand it, it's not immediately clear what fields this dict is supposed to have and what it's not, even though it's obviously in the function doc). Some alternatives that jump off the top of my head:

Make a dataclasses.dataclass class called DataStruct or something that has hanford_background, livingston_background, signal, and glitch attributes and expose these as as the train_data, valid_data arguments. It's functionally the same as a dict, but at least the structure is made explicit somewhere and can be linted, autodoc'd, autocompleted, etc.
At that point, since we're grouping these files together anyway, just construct the RandomWaveformDataset objects in the train project and accept any iterable of (X, y) pairs as the arguments for train_data and valid_data in the train function. The nice thing about this is that most of the dataset args don't get used elsewhere in training: sample_rate, min_snr, max_snr, batch_size, batches_per_epoch. This also seems fair since our dataloader is one way, but perhaps not the only way, one might be interested in sampling data for BBHNet to train on. This is as opposed to the DeepClean case, where (and maybe this is just lack of imagination) the sampling process follows pretty straightforwardly from the problem statement.
Break the train arguments out explicitly: hanford_background, livingston_background, signal, glitches, and then have a single validation argument valid_data that's either the current dictionary or a 4-length tuple that follows the order in which the train arguments are listed and defaults to None (for no validation). This at least makes the arguments explicit once, and then having a line like

if valid_data is not None:
    val_hanford_background, val_livingston_background, val_signal, val_glitches = valid_data
    ... # do dataloader stuff here

would help make it clear in one place what the structure is. This is also consistent with how Keras' model.fit method accepts validation data (which is why I opt for it in DeepClean where there's only two arguments).
I think my intuition says option 2 makes the most sense, but not leaning super hard against any of them (including the current one). Thoughts?

This all looks really good. My only comment would be around the point you bring up about the dict data arguments to the train function. I go back and forth on this a lot too, but my general gut is that dicts that are supposed to have specific fields are just prone to errors and typos and are generally nonintuitive (to someone reading the code trying to understand it, it's not immediately clear what fields this dict is supposed to have and what it's not, even though it's obviously in the function doc). Some alternatives that jump off the top of my head:

Make a dataclasses.dataclass class called DataStruct or something that has hanford_background, livingston_background, signal, and glitch attributes and expose these as as the train_data, valid_data arguments. It's functionally the same as a dict, but at least the structure is made explicit somewhere and can be linted, autodoc'd, autocompleted, etc.
At that point, since we're grouping these files together anyway, just construct the RandomWaveformDataset objects in the train project and accept any iterable of (X, y) pairs as the arguments for train_data and valid_data in the train function. The nice thing about this is that most of the dataset args don't get used elsewhere in training: sample_rate, min_snr, max_snr, batch_size, batches_per_epoch. This also seems fair since our dataloader is one way, but perhaps not the only way, one might be interested in sampling data for BBHNet to train on. This is as opposed to the DeepClean case, where (and maybe this is just lack of imagination) the sampling process follows pretty straightforwardly from the problem statement.
Break the train arguments out explicitly: hanford_background, livingston_background, signal, glitches, and then have a single validation argument valid_data that's either the current dictionary or a 4-length tuple that follows the order in which the train arguments are listed and defaults to None (for no validation). This at least makes the arguments explicit once, and then having a line like

if valid_data is not None:
    val_hanford_background, val_livingston_background, val_signal, val_glitches = valid_data
    ... # do dataloader stuff here

would help make it clear in one place what the structure is. This is also consistent with how Keras' model.fit method accepts validation data (which is why I opt for it in DeepClean where there's only two arguments).
I think my intuition says option 2 makes the most sense, but not leaning super hard against any of them (including the current one). Thoughts?

@alecgunny I think option 1 makes the most sense to me, but I think that may be because I'm a bit confused about option 2. What exactly are the (X, y) pairs you are talking about ? I would expect if we're creating the RandomWaveformDatasets in the train project, that we would explicitly have the train function accept RandomWaveformDataset objects as well. Is your idea that in the future, its possible we may use the train function w/o RandomWaveformDataset objects?
I see your point about how the train function is agnostic to the parameters sample_rate, min_snr, max_snr, batch_size, batches_per_epoch.


MailCatcher not working with SwagNewsletter Plugin

PHP Version
8.1
Shopware Version
5.7
Actual behaviour
MailCatcher is not working with SwagNewsletter Plugin.
Fatal error: Uncaught TypeError: SwagNewsletter\Subscriber\MailTransport::__construct(): Argument #3 ($mailTransport) must be of type Shopware\Components\DependencyInjection\Bridge\MailTransport, FroshMailCatcher\Components\MailFactory given, called in /var/www/html/public/var/cache/dev_fef1e05141adf0a78c8d792b9d142056a0da8beb/proxies/ShopwareDev7700a37c85af5ddc03f07b752e21bff6d4fb8e8fDebugProjectContainer.php on line 18747 and defined in /var/www/html/public/custom/project/SwagNewsletter/Subscriber/MailTransport.php:72 Stack trace: #0 /var/www/html/public/var/cache/dev_fef1e05141adf0a78c8d792b9d142056a0da8beb/proxies/ShopwareDev7700a37c85af5ddc03f07b752e21bff6d4fb8e8fDebugProjectContainer.php(18747): SwagNewsletter\Subscriber\MailTransport->__construct() #1 /var/www/html/public/vendor/symfony/dependency-injection/Container.php(257): ShopwareDev7700a37c85af5ddc03f07b752e21bff6d4fb8e8fDebugProjectContainer->getSwagNewsletter_Subscriber_MailTransportService() #2 /var/www/html/public/vendor/symfony/dependency-injection/Container.php(231): Symfony\Component\DependencyInjection\Container->make() #3 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Components/DependencyInjection/Container.php(212): Symfony\Component\DependencyInjection\Container->get() #4 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Components/DependencyInjection/Container.php(136): Shopware\Components\DependencyInjection\Container->doLoad() #5 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Components/ContainerAwareEventManager.php(209): Shopware\Components\DependencyInjection\Container->get() #6 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Components/ContainerAwareEventManager.php(143): Shopware\Components\ContainerAwareEventManager->lazyLoad() #7 /var/www/html/public/custom/plugins/FroshProfiler/Components/Event/EventManager.php(294): Shopware\Components\ContainerAwareEventManager->getListeners() #8 /var/www/html/public/custom/plugins/FroshProfiler/Components/Event/EventManager.php(84): FroshProfiler\Components\Event\EventManager->parentNotify() #9 /var/www/html/public/vendor/shopware/shopware/engine/Library/Enlight/Components/Mail.php(392): FroshProfiler\Components\Event\EventManager->notify() #10 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Controllers/Frontend/Account.php(557): Enlight_Components_Mail->send() #11 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Controllers/Frontend/Account.php(468): Shopware_Controllers_Frontend_Account->sendResetPasswordConfirmationMail() #12 /var/www/html/public/vendor/shopware/shopware/engine/Library/Enlight/Controller/Action.php(186): Shopware_Controllers_Frontend_Account->passwordAction() #13 /var/www/html/public/vendor/shopware/shopware/engine/Library/Enlight/Controller/Dispatcher/Default.php(467): Enlight_Controller_Action->dispatch() #14 /var/www/html/public/vendor/shopware/shopware/engine/Library/Enlight/Controller/Front.php(225): Enlight_Controller_Dispatcher_Default->dispatch() #15 /var/www/html/public/vendor/shopware/shopware/engine/Shopware/Kernel.php(197): Enlight_Controller_Front->dispatch() #16 /var/www/html/public/shopware.php(17): Shopware\Kernel->handle() #17 {main} thrown in /var/www/html/public/custom/project/SwagNewsletter/Subscriber/MailTransport.php on line 72
Expected behaviour
No Error is shown
Steps to Reproduce?
Install SwagNewsletter Plugin and try to send password recovery Mail

Hey,
I have right now no time to maintain Shopware 5 plugins. Feel free to make a PR if you have already the fix :)


Create GitHub action workflow govulncheck.yml

Add https://github.com/Templum/govulncheck-action
For more info, see : https://go.dev/blog/vuln

Codecov Report
Patch and project coverage have no change.

Comparison is base (5796067) 0.00% compared to head (cfd7025) 0.00%.

: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     #66   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           1       1
  Lines         639     639
======================================
  Misses        639     639

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.

Example alert generated in security --> code scanning : https://github.com/platform9/luigi/security/code-scanning/3


git keyword not working in command pallet when SCM Preview is enabled

Testing #20999

type "git " in the command pallet

Expected: git related commands show up
Actual: quick open for files is shown

This behaviour is now deprecated.


Added managementapp-list. Solves #2919

Added managementapp-list. Solves #2919
Possibility of merge issues as it touches same files as #2917. Happy to solve once that one is in.

Awesome! We'll review it shortly

Hey @appieschot, are you still working on this?

Yes, now that we have the managementapp add in I will rebase and restructure this command based on the findings / feedback of that command. Give me a few days 🦾

@waldekmastykarz sorry for the multiple commits but looks pretty complete to me now ;-).

Multiple commits are not an issue. As long as we can rebase them without merge conflicts, we can handle it 💪

Merged manually. Thank you! 👏


RibbonPanel new property

I would find it useful to be able to hide the text of a RibbonPanel, to have a new property like .HideText or .ShowText, reducing the height of the panel. Or even that the text-area disappears when text is set to empty.
Thanks.

Hi, westconsulting-ita, which text exactly do you mean? The Panel's Caption, like ribbonPanel1, ribbonPanel2, etc. by default?
The text that is drawn bold on the bottom-most part of each panel?

Hi, I mean the text that is on the bottom part of each panel, where there is also the ButtonMore and can be set by the Text property (RibbonPanel1.Text = …).
This because I would also like to use the ribbon as a simple Toolstrip.
Thanks.
Silvano Rinaudo
WEST CONSULTING
Via Annunziata 22
12020 VOTTIGNASCO (CN) - ITALY
Telefono 0171 1740013
Skype: silvano.rinaudo
https://westconsulting.it https://westconsulting.it
http://www.wess.it http://www.wess.it
Questo messaggio di posta elettronica e gli eventuali documenti ad esso allegati sono rivolti unicamente ai destinatari e possono contenere informazioni riservate. La riproduzione e l'uso di questo messaggio, in mancanza di autorizzazione del mittente, sono vietati. Se avete ricevuto questo messaggio per errore, vogliate cortesemente darcene comunicazione telefonica.
This e-mail and any attachment hereto is intended only for the person or entity to which is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. Copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this e-mail by mistake, please notify us immediately by telephone.
Da: tajbender<EMAIL_ADDRESS>Inviato: venerdì 20 settembre 2019 13:59
A: RibbonWinForms/RibbonWinForms<EMAIL_ADDRESS>Cc: Silvano Rinaudo<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Oggetto: Re: [RibbonWinForms/RibbonWinForms] RibbonPanel new property (#48)
Hi, westconsulting-ita, which text exactly do you mean? The Panel's Caption, like ribbonPanel1, ribbonPanel2, etc. by default?
The text that is drawn bold on the bottom-most part of each panel?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/RibbonWinForms/RibbonWinForms/issues/48?email_source=notifications&email_token=ANHYUVDPR2S2UWSD42NCT3LQKS3JBA5CNFSM4IYLO2FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7GO2NQ#issuecomment-533523766 , or mute the thread https://github.com/notifications/unsubscribe-auth/ANHYUVDAF7T5KN3ETHCKWNDQKS3JBANCNFSM4IYLO2FA .

@adriancs2, @harborsiem: What are your thoughts about this request for a new feature?
I think it's worth a look, do you agree?

Hi @tajbender, I think this is ok to be implemented

In the COM based Microsoft Ribbon Api you can define the RibbonPanel without a text. But the height of the Ribbon is not reduced. What is about the ButtonMore Property, that is in the same text line ?
By the way: In Microsoft Ribbon Api the RibbonPanel is called Group.
RibbonWinForms should work similar to the Microsoft Ribbon.


Make AnalysisMapper public

AnalysisMapper looks like the right way to avoid needing to be aware of zinc analysis content, but currently it is an sbt.internal.inc API.

The whole analysis mappers API is going to change drastically. It's highly tied to the serialization mechanism. The mechanism that I envision is more high-level operating only on the API classes and mapping from one class to the same class.
However, I must add that I'm working on making the content of analysis machine-independent. So this won't be necessary in theory. I'm just adding these hooks in case users need to perform more changes before serializing/deserializing.

Yea, these seem challenging to use correctly... not clear yet which of the methods need overrides in order to catch all relevant cases. Our implementation currently looks like https://github.com/pantsbuild/pants/compare/master...twitter:stuhood/zinc-upgrade-2017-06#diff-297b1b6a94d202511b784fd43284b6ebR166 ... but I'm not able to (easily) test portability until after X17 is available.

@stuhood Here is an example how this is done inside hoarder (code is ported to work with sbt 0.13.x but hopefully you will be able to get the main idea).
If you like I can try to explain what and how need to be mapped.

@romanowski : Thanks. Looks similar.
https://github.com/sbt/zinc/pull/351 is relevant. I'm hoping that @jvican agrees that a getMachineIndependentMapper implementation that supports multiple bases to relativize to is worth including... it would be really great if both hoarder and pants could use the default implementation.

I'm happy to include such implementation. I'm just not sure how the interface should look like. If we want to make it generic, we'll need some assumptions that downstream users can agree on.
@stuhood Can you try to implement a rough prototype of ReadMapper and WriteMapper so that I get an idea of what you mean and we extrapolate a generic implementation from there?

@stuhood I had a shot at implementing generic read and write mappers. This is the best I came up with. The mappers take RootPaths that define sourcesRootPath, libariesRootPath (for jars) and productsRootPath (for class files).
Can you let me know what you think about it?
/cc @romanowski who may use this as well in Hoarder.

@jvican : That looks like it would work for our usecase: pants would use the same path for the librariesRootPath and productsRootPath because all 3rdparty jars are symlinked under a particular directory. It would be good to look concretely at what those values would be in an sbt build. Also, if it's possible to be more precise about the relationship between "library" and "external" and "binary" deps in analysis, that would be great (ex: mapBinaryFile relativizes to librariesRootPath, which is confusing).
The last bit that we have manipulated in the past has been the JDK's classpath entries: @romanowski used the bootstrap classpath of the current JVM to do that in the link he posted above, and pants has always rebased JAVA_HOME explicitly.

Yeah @stuhood, unfortunately JDK classpath entries must be special cased, unless the build tool sym links them from every project or use one of the tricks you mention.
I agree that we should be clearer on the differences between "library" and "external" and "binary" deps in analysis.
This has been fixed in #351, so it can be closed.


ART: Claude Monet, The Series Paintings

Art entry, a link that sends you to Claude Monet's paintings.

Apologies, I didn't read the directions carefully. But I hope this is better, thank you!


Support for Abstracted Objects

I realise that "It currently doesn't know how to translate CloudForamtion resource into the abstracted objects used in cfndsl, for example EC2_Instance."
However this would be a very useful feature to add to the roadmap.
I have used cfn2dsl to 'reverse engineer' a manually created stack. I then manually converted the cfn2dsl output to use the abstracted object notation and this took quite some time.

if we start supporting abstracted objects, that will fall into a version dependency trap, as the generic type can give you a safe way to translate back and forward the templates as you needed.
I am happy to raise this as a feature request, any thoughts?
thx

I understand that you use cfn2dsl as a say of keeping json and cfndsl representations of a stack in sync with each other.
My use case for cfn2dsl is just for a one way conversion of json to cfndsl.  In this case the dependency trap would not be problem.

Doesn't it actually do this now?? I'm pretty sure it does.


Remove IgnorePlugin compat-layer and fix various TODOs

Remove Compilation#performance
Remove ContextDependency#prepend
Remove IgnorePlugin compatibility layer
Update EnvironmentPlugin TODO comment

What kind of change does this PR introduce?
refactoring
Did you add tests for your changes?
no
Does this PR introduce a breaking change?
yes
What needs to be documented once your changes are merged?
IgnorePlugin documentation is outdated.
The plugin no longer accepts 2 arguments but an object:
new webpack.IgnorePlugin({
  requestRegExp: /^\.\/locale$/,
  contextRegExp: /moment$/
})

Thanks


Invalid Use of new keyword

I do the same that you do in tutorial video but I got this error message
Invalid Use of new keyword
Here is my code
Set MyRequest = CreateObject("WinHttp.WinHttpRequest.5.1")
MyRequest.Open "GET", "https://jira.atlassian.com/rest/api/2/issue/JRA-9.json"
MyRequest.send
' MsgBox MyRequest.ResponseText
Dim Json As Object
Set Json = JsonConverter.ParseJson(MyRequest.ResponseText)
MsgBox Json("key")
Also I running Microsoft Script Running.

Most likely issue is that you forgot to use the set statement with the new operator.
Set obj = new Collection
Hope that helps, Andrew.

From: Hali Iliya<EMAIL_ADDRESS>Sent: Monday, December 30, 2019 10:02:13 AM
To: VBA-tools/VBA-JSON<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [VBA-tools/VBA-JSON] Invalid Use of new keyword (#157)
I do the same that you do in tutorial video but I got this error message
Invalid Use of new keyword
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/VBA-tools/VBA-JSON/issues/157?email_source=notifications&email_token=AIKTRA3SQMMKIIUNFSUSZWLQ3IEPLA5CNFSM4KBNK2CKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IDLMLDA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIKTRA3PHOKOAJAX5A64AKLQ3IEPLANCNFSM4KBNK2CA.

From: Hali Iliya<EMAIL_ADDRESS>Sent: Monday, December 30, 2019 10:02:13 AM
To: VBA-tools/VBA-JSON<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [VBA-tools/VBA-JSON] Invalid Use of new keyword (#157)
I do the same that you do in tutorial video but I got this error message
Invalid Use of new keyword
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/VBA-tools/VBA-JSON/issues/157?email_source=notifications&email_token=AIKTRA3SQMMKIIUNFSUSZWLQ3IEPLA5CNFSM4KBNK2CKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IDLMLDA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIKTRA3PHOKOAJAX5A64AKLQ3IEPLANCNFSM4KBNK2CA.

I am getting this same error, but it is not in my code that the error is pointing to. It is pointing to Private Function json_ParseObject, the 3rd line of code in that function calls "Set json_ParseObject = New Dictionary". This is where the error shows.

Difficult to say what might be wrong without some surrounding code around the issue. Post a reduced code sample that demonstrates what you are experiencing.

In the module that has json_ParseObject, do you have a reference for Microsoft Scripting Runtime?  The fix looks like you do not as it is changing from early binding (using new) to late binding (using CreateObject).

Or do you have multiple Dictionary classes?  If using "new Scripting.Dictionary" works instead of just "new Dictionary", then this is likely the reason.

Here's the example that does not work
Sub Parser()
    Dim para As Paragraph
    Dim xmlhttp As New MSXML2.XMLHTTP60
    Dim url As String
    url = "https://jsonplaceholder.typicode.com/todos/1"

    xmlhttp.Open "GET", url, False
    xmlhttp.Send

    Dim Json As Object
    Set Json = JsonConverter.ParseJson(xmlhttp.responseText)
    MsgBox Json("title")
End Sub

Here's how to fix it
'Private Function json_ParseObject(json_String As String, ByRef json_Index As Long) As Dictionary
Private Function json_ParseObject(json_String As String, ByRef json_Index As Long) As Object
    Dim json_Key As String
    Dim json_NextChar As String

    'Set json_ParseObject = New Dictionary
    Set json_ParseObject = CreateObject("Scripting.Dictionary")

thanks


[$500] Delayed text message sent to phone even though the frequency is set to immediately

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

Version Number: 1.4.20-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1703994911075889
Action Performed:

Send a message to phone account in chat

Expected Result:
SMS is received with the message
Actual Result:
SMS is only deliver 10min after the message is sent
Workaround:
unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011050c562712f8468
Upwork Job ID:<PHONE_NUMBER>621502464
Last Price Increase: 2024-01-03

potentially related GH: https://github.com/Expensify/App/issues/33621

Still waiting for proposals

@laurenreidexpensify Can you make this internal as message is made from BE only?

Waiting for an internal engineer to pick up

I just tested this and I got the SMS (it was slow, but it got there). Does anyone have reproduction steps? Alternatively a requestID from a comment that was sent but not received by SMS:

Okay I got around to testing this properly now and similar to Ioni, it did sent eventually (like maybe a 10min lag), will write up proper repro steps now

Maybe. I'll check logs and see if I can find out what happen with David's message.

I found the entry in Twilio, Message SID SMf51ac329d7022be54317b0dad3b64764 for reference
Unfortunately, the logs of the messaging flow are already gone. They're only available for 7 days.
Our logs even indicate it was queued up, sent, and delivered within 1 second. I suspect any delays here are from carriers outside of our control and Twilio's.

I can take another look if someone has an example of an issue with one but I'll need the phone number the message should have went to sent to me and the time.

Logs for @laurenreidexpensify example that she provided

So as far as I can see, the messages were sent quickly. I don't know how to troubleshoot end user delays, and their support article is broken https://www.twilio.com/docs/messaging/guides/debugging-common-issues#my-messages-are-delayed
I created a support ticket for more info https://help.twilio.com/tickets/15269844

Last update from Twilio.

Hi Jpersaud,,

Thanks for reaching out to Twilio Support, and thank you for providing those examples.
I have reviewed them and will need more feedback from our carrier partner, so I have escalated to them for further investigation.

I will keep you updated here.

Making a weekly while we wait to hear back from Twilio

No update from Twilio yet. Bumped the support ticket for an update.

I am almost sure this is basically due to https://expensify.slack.com/archives/C03SDMF9YJ2/p1705935685709549
Let's close for now


Update jsDoc documentation

Update jsDoc documentation.

activity
nft
ainft721Object
ainftToken
credit
discord
eth
event

Fix get function in ainft721Object.

Please review the code you've worked on with a comment.

LGTM
Thanks!

Thank you for addressing all questions and suggestions! 😁

Thanks for the review!


[WIP] Upgrade to Django 1.10

This PR does not include FastCGI upgrade.

Automated message from Dropbox CLA bot
@umairwaheed, it looks like you've already signed the Dropbox CLA. Thanks!

@umairwaheed  a head's up in case this is helpful when upgrading, I think there will be some fiddling with django-bitfield, which we use...the project is not currently maintained and not supported by 1.10 last I checked https://github.com/disqus/django-bitfield/issues/67

PR for Django BitField. https://github.com/disqus/django-bitfield/pull/73

This is getting to be a pretty big branch; can you start building a PR with the parts that also work on django 1.8 so we can start merging those?

This is getting to be a pretty big branch; can you start building a PR with the parts that also work on django 1.8 so we can start merging those?

Yeah, sure.

PR containing commits that are ready to be merged https://github.com/zulip/zulip/pull/2215.

@timabbott, one of the side effects of upgrading to Django 1.10 is that whenever the password of the user is changed the password hash changes causing the user to log out. Currently, our UI doesn't handle it well and the user is allowed to navigate away before he is redirected to the login page. This is also the reason why casper tests related to settings page fail.

I think we probably want to keep the behavior that it logs out other clients, but it shouldn't log out the current client that's actually changing the password.  Maybe that means we should just log you back in again (using the new_password you just set) in the same API call that changed the password?

Maybe that means we should just log you back in again (using the new_password you just set) in the same API call that changed the password?

This looks good.

@timabbott, this logging out issue also seems to be due to caching.

Logging out issue due to caching fixed in https://github.com/zulip/zulip/pull/1945/commits/64b7d23fc96b70027957f356072ad54f09e427f8

The front end tests pass now but backend tests fail. They are are failing due to some i18n issue. Will check tomorrow.

@timabbott done with this.

This commit doesn't work with Django 1.8 https://github.com/zulip/zulip/pull/1945/commits/7d2365cc1fde545a8904b699eec61f4632b4eaa6

@umairwaheed it looks like django-bitfield merged 1.10 support into master recently; should we switch to using that?

(Also can you rebase this?)

We should also update PROVISION_VERSION to 3.0 in version.py since this is a big, non-backwards-compatible change to dependencies.

Once that is done I can try deploying this to chat.zulip.org.

@timabbott done

Already merged.


python 2/3 compatibility

It seems possible to get the codebase to a point where it is compatible with both python 2 and 3 and I am willing to tackle it.
This guide https://docs.python.org/3/howto/pyporting.html has a pretty clear path to getting a compatible codebase.
I propose using this issue to track the work, but to open up individual issues for each bit of work in the process.
Steps:

Futurize stage 1 – https://github.com/BD2KGenomics/toil/issues/1784
Futurize stage 2  – https://github.com/BD2KGenomics/toil/issues/1785
deal with semantic changes (text vs bytes, division, etc) http://python-future.org/what_else.html#what-else
Pylint – (pylint --py3k)
work through dependencies – https://pypi.python.org/pypi/caniusepython3
continuous integration with python 2 and 3

Before I dig into this too much, can I get a thumbs-up from maintainers that this sort of stuff would be accepted?

If you want to dish out some tasks I can try to help.  I've been wanting to see this support for a while

Thanks @jorvis. I'm still working on fleshing out some of the strategy here. If something comes up I'll let you know.

Thanks for this @ryanking
I would also note that in addition to what is listed above there is the potential effect of unordered dictionaries in python3 <3.6
This is usually not a real problem but can make testing a pain, especially if it is non determinism flowing up from dependencies.  For this reason I suggest starting with 3.6 or even making it a requirement

I am one of those Python 2.7 luddites, but you'd get my strong +1

thanks @genomematt, added a note about unordered dictionaries to the todo list

Is there a ticket/pull request for the third action point in this ticket (deal with semantic changes)?

@jsmedmar, not unless you count on this issue here. I believe @ejacox was planning on dealing with this next week.

wondering if there is any update on this?

This postmortem might prove useful for migrating the code: https://medium.com/@boxed/moving-a-large-and-old-codebase-to-python3-33a5a13f8c99

I got inspired to work on this a little (very limited time).
I have a fork and if any one wants to help, that would be fantastic.
Mark
Pau Ruiz Safont<EMAIL_ADDRESS>writes:

This postmortem might prove useful for migrating the code: https://medium.com/
@boxed/moving-a-large-and-old-codebase-to-python3-33a5a13f8c99
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.*

related work is #1936;  bd2k-python-lib is hard to port to Python 3 and many parts are not needed

Hey @ryanking @diekhans, just wondering what the status of this card is, because the cards #1936 and #1937 are both resolved. Just wondering how far through you guys got, and whether the changes to complete the port are trivial or there's some head-ache?

We now more-or-less-officially support Python 3. We have our CI tests running and passing under Python 3; any bugs people report when running under Python 3 are going to be seen as as important or more important than Python 2 bugs.
I won't warrant that it's been a bug-free port, but it is "done".


KOGITO-1868 - Add Zulip notification to daily builds

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

This will create a new topic each time ?

topic per subject as is now:
Kogito daily snapshot deploy #${BRANCH_NAME}: ${currentBuild.currentResult}

@radtriste
Reduced to:

Kogito daily snapshot deploy #${BRANCH_NAME}
Kogito daily Quarkus master


Feature: spotify scan improvements

Breaking changes

fixed typo in env var

RECOMEND_GEN_SCHED -> RECOMMEND_GEN_SCHED

Features added

now all playlists, including private, are scanned
artist top playlists are not scanned/imported by default. now controlled by ARTIST_PLAYLIST_ENABLED env var

I think this is more in-line with the average user's expectation for a no-config set up. Most users will have a ton of artists in their library when they just want to set this up to sync the playlists they made themselves

Bugs fixed

fix exception thrown when manual rescan is called before any jobs are started
prevent recommendation playlist scans/imports, since that API endpoint is now depcrecated, also see https://github.com/spotipy-dev/spotipy/issues/1172

Hello and thank you for your PR, i will review this ASAP and merge if I find no issues.


App with high CPU usage/mem/hangs possibly after wake

Bug Description, Steps to Reproduce, Crash Logs, Screenshots, etc.
Hi there, first of all thank you for MenuBarExtraAccess!
Problem: my macOS app starts showing high CPU/mem usage possibly after computer is awaken from sleep. Profiling indicates some good amount of time spent on NSStatusItem.
I think it might be the way I'm using MenuBarExtraAccess. I'm reproducing it consistently putting the computer to sleep and then clicking on the app in the menu bar. Edit: now I'm able to consistently trigger it by simply clicking on the app in the menu bar, don't even need to put it to sleep.
Would appreciate any help or pointers you may have.  Code and profiling screenshots can be found below.

Environment
Environment: macOS Sonoma 14.2.1, MBP M1 2020, one external monitor
System settings: menu bar auto-hide enabled
Code
This is sample code that very closely resembles how my app uses MenuBarExtraAccess and MacControlCenterUI:
import SwiftUI
import MacControlCenterUI
import SettingsAccess

@main
struct MyAppApp: App {
    @NSApplicationDelegateAdaptor(AppDelegate.self) var appDelegate: AppDelegate
    @State var isMenuPresented: Bool = false

    var body: some Scene {
        Settings {
            SettingsView().environmentObject(appState)
        }

        MenuBarExtra() {
            MenuBarView(isMenuPresented: $isMenuPresented)
                .openSettingsAccess()
                .environmentObject(appState)
        } label: {
            Label(Constants.appName, image: "MenuBarIcon")
        }
        .menuBarExtraStyle(.window)
        .menuBarExtraAccess(isPresented: $isMenuPresented)
    }
}

import SwiftUI
import MacControlCenterUI
import Combine
import OSLog

struct MenuBarView: View {
    @Environment(\.openSettings) private var openSettings
    @Binding var isMenuPresented: Bool

    @State private var counter: Int = 0
    @State private var isCounterEnabled: Bool = false
    @State private var timer: AnyCancellable?

    func update() {
        // Update counter every second only when the UI is being presented
        counter += 1
    }

    func startTimer() {
        timer = Timer.publish(every: 1, on: .main, in: .common)
            .autoconnect()
            .sink { _ in update() }
    }

    func stopTimer() {
        timer?.cancel()
        timer = nil
    }

    var body: some View {
        MacControlCenterMenu(isPresented: $isMenuPresented) {
             VStack {
                 Toggle("Enable", isOn: $isCounterEnabled)
                 Text("\(counter)")
             }
        }
        .onChange(of: isMenuPresented) { _ in
            if isMenuPresented {
                if isCounterEnabled: {
                    startTimer()
                }
            } else {
                stopTimer()
            }
        }
    }
}

Screenshots
52.28 s   36.8%	1.00 ms	 	                     -[NSStatusItem _updateReplicantsUnlessMenuIsTracking:]

MenuBarExtras.setupEventsMonitor

statusItem.for(:)

Thanks for the detailed reporting. I may not have time to delve into this right away but there may be some ways to mitigate what's happening.
There are less than ideal inefficiencies necessary (mirror introspection and dynamic lookups) to even get MenuBarExtraAccess to provide its essential functionality in the first place, because of how handcuffed Apple makes their API. So it may be possible to improve but not clear if it can be totally resolved.

Hey there, thanks for the quick reply. I tracked down the issue in my app and it wasn't related to the timer nor MenuBarExtraAccess.
It was caused by a using public computed property from another object. It perpetually triggered new renders of the view. Please don't ask me why, I was probably tired 😂
class AppState: ObservableObject {
    @AppStorage("something") var something = 0

    var somethingDynamic: Int {
       something + 1
    }
}

struct SomeView: View {
    @EnvironmentObject private var appState: AppState

    var body: some View {
        Text("\(appState.somethingDynamic)")
    }
}

Hey no sweat, glad you found it.


[fastlane] Version bump

Added device_grid
Removed * from fastlane docs for required parameters
Shell escape git commit / tag messages

👍


[TDX-1628] Dev Portal: Archive a Service Version

Summary

Add Archive a Service version to ServiceHub documentation

Reason
New feature enables archiving a Service version.
Testing
https://deploy-preview-3367--kongdocs.netlify.app/konnect/servicehub/manage-services/#archive-a-service-version

LGTM! Not sure when we want this to be included in the docs, since this feature is behind a feature flag that we have disabled for now. Perhaps this should be on hold until we start enabling the flag?

@Mierenga I think the way it works is tech writer can merge but waits to "publish" to docs until the feature is live in production.

LGTM! Not sure when we want this to be included in the docs, since this feature is behind a feature flag that we have disabled for now. Perhaps this should be on hold until we start enabling the flag?

I think the way it works is tech writer can merge but waits to "publish" to docs until the feature is live in production.

That's right. @falondarville - this feels like a good case for starting to use labels properly again. Can set it to do not merge and maybe we need a label for unreleased or something similar?

Correct @lena-larionova. I should have communicated that in the initial PR. This is on hold til the feature flag is turned on. I wanted to ensure the PR was prepped and ready to go for when ENG was ready. Thanks for your diligence always 💯

This was reverted in https://github.com/Kong/docs.konghq.com/pull/3469 because we haven't yet turned on the feature flag in Konnect. It will be turned on sometime after upgrading the Konnect control planes to v2.7.


dynamixel-workbench: 2.2.0-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository dynamixel-workbench to 2.2.0-0:

upstream repository: https://github.com/ROBOTIS-GIT/dynamixel-workbench.git
release repository: https://github.com/ROBOTIS-GIT-release/dynamixel-workbench-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.6.1
previous version for package: 2.0.0-0

dynamixel_workbench
* Added XW540
* Bugs fixed, typos revised
* Contributors: Ryan Shim, Yutaka Kondo, T-Kitajima, Nico Zevallos, Madhur Deep Jain, Will Son, YongHo Na

dynamixel_workbench_controllers
* Added XW540
* Bugs fixed, typos revised
* Contributors: Ryan Shim, Yutaka Kondo, T-Kitajima, Nico Zevallos, Madhur Deep Jain, Will Son, YongHo Na

dynamixel_workbench_operators
* Added XW540
* Bugs, typos fixed
* Contributors: Ryan Shim, Yutaka Kondo, T-Kitajima, Nico Zevallos, Madhur Deep Jain, Will Son, YongHo Na

dynamixel_workbench_toolbox
* Added XW540
* Bugs fixed, typos revised
* Contributors: Ryan Shim, Yutaka Kondo, T-Kitajima, Nico Zevallos, Madhur Deep Jain, Will Son, YongHo Na

Holding for sync.


bn128 elliptic curve builtins in k

For the EVM semantics of Byzantium.

I'm ok to merge this. I haven't carefully read the actual implementation of the fields and curves, but it should be ok to add it for now as a builtin, since we're not going to symbolically reason about the crypto algorithms in the near future.

@bmmoore Are you reviewing (or will you review) this? Otherwise, I'd like to merge this.


added numpy tutorial

It's my tutorial for Day 1. Should I simply merge it? I am not used to have push access. ;-)

Yes, go ahead!
Quickly looked at it:

for the setup section, this is for downloading data files? It seems it is very few data (in KBs), so I would just add it to the /data fir that is already in the Day_1 dir. That saves the participants a step to further download these

@jorisvandenbossche Yes, it's already in there. But you need to assume that people cloned the repo.

Ah, didn't see that.
Yes, we are going to give the instructions to download the full repo (either by cloning or by downloading it as a zip file) at the beginning of the day. So everybody should have the data then.

I updated the instruction to tell the student to download the whole repo, but if you explain it anyways, I will skip this part.

Yep, it will be in the opening slides (and will put that in a mail as well). I started on those: https://paris-saclay-cds.github.io/python-workshop/#1 (but have to expand it)

Something else: what I personally find an important aspect to be familiar with is boolean indexing using a mask. I will certainly see that in the pandas one, but it might be nice to already see that in the numpy part (if you think there is room and would fit somewhere of course)

I thought about adding boolean indexing, but indeed there is little room. If you think it will be useful for your pandas tutorial I can include it, though.

See ya tomorrow!!!


How do I increase the size of the spinner ?

I tried to increase the width and height but that's not help me to increase the spinner.

@rajasimon Try this PR https://github.com/tobiasahlin/SpinKit/pull/113

@rajasimon I added scss mixins with PR #121, so you can customize the size to anything you want

Thanks


Update mvn output in pom file.

GROUPINGS-973

main:
[echo] Now that you have a clean project,
[echo] here are some common goals for this project:
[echo]
[echo] To run the unit tests with your Grouper server:
[echo] $ mvn clean test
[echo] To run the javascript tests with jasmine:
[echo] $ mvn clean jasmine:test
[echo] To create a jacoco test report:
[echo] $ mvn clean test jacoco:report
[echo]
[echo] To run UH Groupings locally:
[echo] $ mvn clean spring-boot:run
[echo]
[echo] To access a list of possible commands:
[echo] https://www.hawaii.edu/bwiki/display/SITARd/Unit+Testing#UnitTesting-RunningYourTest

Looks good.  Good reminder though that we need to ensure that the code formatting is standardized across all of our IDE's.  I wonder if we should create a few test files that each of us can use to ensure that our IDEs are standardized such that the IDE should not feel compelled to reformat the test files.

Looks good. Good reminder though that we need to ensure that the code formatting is standardized across all of our IDE's. I wonder if we should create a few test files that each of us can use to ensure that our IDEs are standardized such that the IDE should not feel compelled to reformat the test files.

I feel like I need to make a ticket for someone to fix the problem we have with formatting the pom file... I'll move this convo to slack...


Upgrade the golang version

Hi @myungjin,
Do we need to update the golang version to 1.22+ in Prerequisites, as required in go.mod?
Using go1.18 will cause errors when compiling the Flame control plane components.
Thanks,
Shixiong

Yes..we recently upgraded some of the packages and they needed golang 1.22. would that be an issue?

No issue reported when I used golang 1.22 for the development. But someone may follow prerequisites.md and install golang 1.18 in their runtime environment.
I will update prerequisites.md in the next PR.


Support manually provided benchmark results

Some benchmarks might require priviledged permission and cannot be run in GitHub Actions VMs. In those situations, accepting manually provided benchmark results (run elsewhere) can be useful.

Isn't this already supported? https://github.com/benchmark-action/github-action-benchmark#:~:text=Additionally%2C even though there is no explicit example for them%2C you can use customBiggerIsBetter and customSmallerIsBetter to use this action and create your own graphs from your own benchmark data. The name in these tools define which direction "is better" for your benchmarks.


Ensure when you click the candidate name a page opens with the record

Is your feature request related to a problem? Please describe.
As an admin I want to see the candidate record and application history when I click the candidate name.
Describe the solution you'd like
Candidate names are hyperlinked and open the record corresponding to that candidate when clicked.
Additional context
Click the page
This opens

Speak to Enor about the prototype for more information on Design.

Applications

[ ] Draft
[ ] Applied
[ ] Withdrawn

Stages

[ ] Review
[ ] Shortlisted
[ ] Selected
[ ] Recommended
[ ] Handover


Update sudoers configuration

Apple has recompiled sudo with the tty_tickets flag enabled in Sierra.

I'm not seeing tty_tickets in my sudoers file. I'm on build 16A323 (10.12).

I do not believe it is in the sudoers file, sudo has it baked in. You can test it by opening two Terminal windows/tabs and executing a sudo command in each one. Both should prompt you for a password.
Apple probably publishes the source code for 10.12’s Darwin soon, then we will know for sure.


Clarifying question about pcluster update and post_install

Hi,
Just want to clarify behavior of pcluster update with a post_install. If the path to the post-install script in S3 stays the same but the contents get changed, will later nodes launched in the cluster pull from that S3 location?

Hi @calebwin,
this has been answered in https://github.com/aws/aws-parallelcluster/issues/2558#issuecomment-803722953.
The post installation script will be download and executed in the bootstrap phase of new launched nodes, regardless if the location or the content stay the same.
Behind the scenes, the fetch_and_run script is called during the execution of the user data

Thank you :+1:


Padding color when using FillMode.PRESERVE_ASPECT_FIT

Hi. Thank you so much for the library.
When i use FillMode.PRESERVE_ASPECT_FIT to "force" the library to compose a square video with .size(outputHeight, outputHeight), the resulting video will have paddings (on the top&bottom / left&right) just like what i expected. But, the padding was black. How can i change the color of those paddings?

Yes. Please set GlFilter.class which set the color by setClearColor method.
https://github.com/MasayukiSuda/Mp4Composer-android/blob/master/mp4compose/src/main/java/com/daasuu/mp4compose/filter/GlFilter.java#L170-L175

@MasayukiSuda Thanks a lot. It works like a charm.


Fix test Renegotiation_AddTrack_Multiple

Fixes #1176. I was unable to reproduce this locally, but this must have happened because onTrackChan was being written to before onTrackCount variable was updated. I changed the order and removed the unnecessary mutex.

Thanks @at-wat!

@jeremija This LGTM!


Bintray is being sunsetted - migrate to another bottle host?

Provide a detailed description of the proposed feature
:wave: Not sure if y'all are already aware, but it seems that JFrog has put out a deprecation notice for Bintray: https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/
As of May 1, Bintray will be fully decommissioned.
Does Homebrew have a plan for migrating bottles away from Bintray before this happens?

Never mind, just saw the Discussions thread over at https://github.com/Homebrew/discussions/discussions/691


feat: Add /assetTypes/{assetType}/assetViews/{assetViewId}/datasets/{datasetId}/manifestJson endpoint

fixes fds-900

No problem! I'll change them both here.

No problem! I'll change them both here.

Cool, thank you! I'm not sure what the requirements are for these endpoints, but you might not necessarily need all the processing that happens in the function.

@GiaJordan Could you re-review?


Validate texture paths in custom models

When users select a custom item model in the Editor, the Editor will parse it, find its texture paths, and ask the user to provide the corresponding textures. Finally, it alters the model such that it looks for the textures in the right location.
Unfortunately, when these texture paths contain forbidden characters like : or uppercase characters, these will be copied to the final model, and eventually be rejected by the minecraft client. The Editor should do some renaming to prevent such issues.

This will be fixed in Custom Items 11.5


refactoring

note https://github.com/abap2UI5/abap2UI5/pull/943/checks?check_run_id=21914369412, its likely that this PR contains a breaking change in the samples repo

@larshp thanks for the hint, now also the cross check is green (maybe i will also add this check to the branch protection rule)


UNDERTOW-634 Remove unused advertise configuration MCMPConfig.Adverti…

…seBuilder#setAdvertiseAddress
https://issues.jboss.org/browse/UNDERTOW-634

Is this no longer needed with NIO?

I think https://github.com/undertow-io/undertow/commit/6297ea5b8666742da1529a07d07c8f4a3d6073ca is the correct solution


[ansible/artifactory] installation fails due to undefined variable

Is this a request for help?:
No
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

Which installer:
Ansible 7.18.6
Which product and version (eg: ansible & collection version - 1.1.2):
Artifactory, 7.18.6
Which operating system and version(eg: ubuntu & version - 20.4):
Ubuntu 20.04
What happened:
TASK [jfrog.platform.artifactory : Create required directories] ******************************************************************************************************************************************
ok: [artifactory.example.com] => (item=/data/jfrog-filestore)
ok: [artifactory.example.com] => (item=/opt/jfrog/artifactory/var/data)
ok: [artifactory.example.com] => (item=/opt/jfrog/artifactory/var/etc)
ok: [artifactory.example.com] => (item=/opt/jfrog/artifactory/var/etc/security/)
ok: [artifactory.example.com] => (item=/opt/jfrog/artifactory/var/etc/artifactory/info/)

TASK [jfrog.platform.artifactory : Configure systemyaml] *************************************************************************************************************************************************
changed: [artifactory.example.com]

TASK [jfrog.platform.artifactory : Configure master key] *************************************************************************************************************************************************
ok: [artifactory.example.com]

TASK [jfrog.platform.artifactory : Configure join key] ***************************************************************************************************************************************************
changed: [artifactory.example.com]

TASK [jfrog.platform.artifactory : Configure installer info] *********************************************************************************************************************************************
fatal: [artifactory.example.com]: FAILED! => changed=false
  msg: 'AnsibleUndefinedVariable: ''platform_collection_version'' is undefined'

What you expected to happen:
Successful installation.
How to reproduce it (as minimally and precisely as possible):
- name: Install and configure Artifactory
  hosts: artifactory_test
  become: true
  tasks:
    - include_role:
        name: jfrog.platform.artifactory

Anything else we need to know:
Workaround is relatively simple - to define this value in inventory.

Duplicate of https://github.com/jfrog/JFrog-Cloud-Installers/issues/119

@weakcamel  can you try running the whole collection instead of a single artifactory role  , since platform_collection_version  value is already defined from groupvars/all/package_version.yaml

@chukka Could you explain how to "run a collection"?
My understanding is that a collection is a set of roles you can mix and match, it's not a runnable on its own:

https://github.com/jfrog/JFrog-Cloud-Installers/blob/master/Ansible/examples/playbook-rt.yml uses a role
The collection structure explained here https://docs.ansible.com/ansible/latest/dev_guide/developing_collections_structure.html doesn't mention embedding group variables inside a collection - is there another document which does?

@weakcamel  I generally test collection's specific role using below commands . Hope it helps

cd /tmp
vi hosts.ini #create inventory
ansible-galaxy collection install jfrog.platform -p .
ansible-playbook -vv ansible_collections/jfrog/platform/artifactory.yml -i hosts.ini

@chukka That's not how you use collections. Collections are modules meant to be installed into an existing code structure which is typically according to a variant of standard directory which defines:

user roles
playbooks
inventory, including group and host variables for all the managed hosts, of any type (Artifactory being just 1 of many)
requirements to install external roles
external roles/collections which by default get installed to /etc/ansible or ~/.ansible

Your setup is good for testing (group_vars/all/package_version.yaml gets included in the inventory of your test project)  but not feasible for using
means making a collection the Ansible "root" directory - copying over whole inventory, user roles, playbooks into the source code of collection.  Doing it this way reverses the Ansible directory structure upside down.

@chukka well, this more or less this setup. Except we're not using the built-in playbook - why would we? our environment is different, we need a different one to set up the host from scratch (OS configuration)

I would second @weakcamel, a collection is a collection of stand-alone roles, any variables needed by any individual role should be present in either defaults or vars (preferably defaults to allow it being overridden by the playbook) so that one might incorporate the role in ones own playbook.

Let me take a second look at it and come back next week

any variables needed by any individual role should be present in either defaults or vars

+1

(preferably defaults to allow it being overridden by the playbook) for that role, so that one might incorporate the role in ones own playbook.

In this specific situation:
https://github.com/jfrog/JFrog-Cloud-Installers/blob/master/Ansible/ansible_collections/jfrog/platform/group_vars/all/package_version.yml

products' versions - yes, those need to be overridable (and I believe they're already redefined in the respective roles)
platform_collection_version is pretty much a constant, so it's private - could potentially be in  in vars of each role.

Ideal IMO would be to be able to pull this value from collection galaxy metadata but I don't see a reasonable way to do that (unless - perhaps - with a very simple Python plugin which would provide that).

@weakcamel  Makes sense , looking at Ansible 2.x preference of variables , yes, we can override them. Do you have a better idea not to duplicate these versions in group_vars/all/package_version.yml and product default roles variables  ?

@chukka I'm afraid not. Product default versions IMO belong to the product roles, for the platform_collection_version I don't see a feasible way to not duplicate it.
Perhaps this could be automatically updated as part of the collection's release procedure?


Expose existing functionality via UaSettings() for RFC8599 usage

Allow Contact params and extra headers to be set for SIP REGISTER. Current use case for RFC8599 Push Notification support to pass Firebase Cloud Messaging Tokens and APNs Tokens in SIP registrations. Refs #89

Closing this. Not needed. Done in another PR.


Procfile: change flags used

I changed the flags so that they would run for me. Was getting
"flag provided but not defined:" errors on nearly everything
for 0.4.0, 0.4.6, and 2.0.0
footer

I do not think it is a problem.
try to run ./build
then goreman start

@maxgarvey The Procfile is designed for v2.0, and it should work perfectly. From 0.4 to 2.0, some flags are deprecated and not used any more.


Could not create pin 'G0' for module 'top_module' with ID 1:

I am trying to load the ISCAS 89 Benchmarks.  I am receiving the error:
[/home/user/Desktop/HAL/hal/src/netlist/module.cpp:816] could not create pin 'G0' for module 'top_module' with ID 1: net 'G0' with ID 5 is neither an input nor an output
The s27.v benchmark is shown below:
module s27(VDD,CK,G0,G1,G17,G2,G3);
input VDD,CK,G0,G1,G2,G3;
output G17;

  wire G5,G10,G6,G11,G7,G13,G14,G8,G15,G12,G16,G9;

  FD1 DFF_0(CK,G5,G10);
  FD1 DFF_1(CK,G6,G11);
  FD1 DFF_2(CK,G7,G13);
  IV  NOT_0(G14,G0);
  IV  NOT_1(G17,G11);
  AN2 AND2_0(G8,G14,G6);
  OR2 OR2_0(G15,G12,G8);
  OR2 OR2_1(G16,G3,G8);
  ND2 NAND2_0(G9,G16,G15);
  NR2 NOR2_0(G10,G14,G11);
  NR2 NOR2_1(G11,G5,G9);
  NR2 NOR2_2(G12,G1,G7);
  NR2 NOR2_3(G13,G2,G12);

endmodule

I was wondering what the fix for this would be ?
Thanks

@thud06 : Although you did not mention which gate library you are using I am guessing it was lsi_10k since I can reproduce your error message when I try to parse the netlist above with that one.
The point is that the pin order in your netlist does not match the pin order given in the gate library. The solution would be to adjust the pin order in either of the two.
Adjusting the netlist would look like this:
`module s27(VDD,CK,G0,G1,G17,G2,G3);
input VDD,CK,G0,G1,G2,G3;
output G17;
wire G5,G10,G6,G11,G7,G13,G14,G8,G15,G12,G16,G9;
FD1 DFF_0(G5,CK,G10);
FD1 DFF_1(G6,CK,G11);
FD1 DFF_2(G7,CK,G13);
IV  NOT_0(G0,G14);
IV  NOT_1(G11,G17);
AN2 AND2_0(G14,G6,G8);
OR2 OR2_0(G12,G8,G15);
OR2 OR2_1(G3,G8,G16);
ND2 NAND2_0(G16,G15,G9);
NR2 NOR2_0(G14,G11,G10);
NR2 NOR2_1(G5,G9,G11);
NR2 NOR2_2(G1,G7,G12);
NR2 NOR2_3(G2,G12,G13);
endmodule
`
Please verify that you can parse and display the modified netlist using the lsi_10k gate library.


Error:  Load key "id_rsa": invalid format

Following this tutorial worked well until the last step of connecting to the VM.  Final steps with the outputs shown:
PS C:\Users\shawn\OneDrive\documents\Projects\linux vm qs> terraform output -raw tls_private_key > id_rsa
PS C:\Users\shawn\OneDrive\documents\Projects\linux vm qs> terraform output public_ip_address
"<IP_ADDRESS>"
PS C:\Users\shawn\OneDrive\documents\Projects\linux vm qs> ssh -i id_rsa<EMAIL_ADDRESS>The authenticity of host '<IP_ADDRESS> (<IP_ADDRESS>)' can't be established.
ECDSA key fingerprint is SHA256:fAmKfkfJNaeAKkWFqhbMk2w8k0NQ5eIt4nuui18nFms.
Are you sure you want to continue connecting (yes/no/[fingerprint])? yes
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Load key "id_rsa": invalid format
<EMAIL_ADDRESS>Permission denied (publickey).
PS C:\Users\shawn\OneDrive\documents\Projects\linux vm qs>

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

ID: dc3a8a34-f0f0-7aba-b45b-2c59d722b7de
Version Independent ID: 667d19a1-ff34-3979-105c-c0a918d29707
Content: Quickstart - Configure a Linux virtual machine in Azure using Terraform
Content Source: articles/terraform/create-linux-virtual-machine-with-infrastructure.md
Service: terraform
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@FeedTheHungry Thanks for the feedback. I'm testing this now.

@FeedTheHungry I've updated the article and it will be live tomorrow.


adjust dependencies for forward compatibility

When installing this component in React 18, using TypeScript, warnings appear
in your editor since this package depends on @types/react directly. This is
only necessary for the build environment, and shouldn't be included in a
package's dependencies as it can confuse TypeScript at the application level.
This adjustment to the dependencies still lints and builds just fine, but the
package itself will no longer depend on anything at runtime, allowing usage of
the component in any future version of React.

yuh

yuh

Hey! Thanks for the contribution, apologies for not seeing this earlier.


How to add postcss plugins

I ended up adding them to the postcss array in vue-loader.config.js because I couldn't get it to work.
Is there a better way or should we add a hook to the postcss array?

Can you post your postcss config?
I may add build.postcss option in nuxt.config.js

Just added postcss-cssnext to vue-loader.config.js inside my nuxt package for giggles. That made the build complain about double autoprefixer though (think it's included in cssnext).
Think build.postcss in nuxt.config.js would be sweet 👍
'use strict'

module.exports = function () {
  let config = {
    postcss: [
      require('postcss-cssnext')(),
      require('autoprefixer')({
        browsers: ['last 3 versions']
      })
    ],
…

This option has been added in v0.6.7
Thank you @PatrikLythell for using Nuxt.js 😄

Just for the record, since I spent few hours looking for that, here is the nuxt.config.js that worked for me:
module.exports = {
  build: {
    postcss: [
      require('postcss-nested')(),
      require('postcss-responsive-type')(),
      require('postcss-hexrgba')(),
    ]
  }
}

That's cool, but seems that any scss file included by CSS array in the nuxt.config.js files is not prefixed.
css: [
      { src: '~assets/scss/app.scss', lang: 'scss' },
],

My workaround was import the scss file in the layout vue file, like so

Any other better approach?


Jinja2 version

Is it possible to bump or loosen up the jinja2 version to support 3.x.x from the current jinja2==2.11.3? I'm having version conflicts with some packages I'm using like prefect. Thank you.

What versions are working better in your case? Did you try changing the jinja version in the requirements.txt file to see if it works with current templates? If so loosening it up shouldn't be much of a problem, if not we can think of another solution


username validation

Ensuring username is unique for users. This can be done with using email as a username since the same username cannot be used multiple times to create an account.

Links to code changes can be found in branch 4-username-validation


erorr

Hello engineer, this error is encountered during execution

fixed set AUTO_CLAIM_SQUAD_BONUS to False to fix it

fixed set AUTO_CLAIM_SQUAD_BONUS to False to fix it

It's great


Use relative public path

It uses relative path for public URL.
Closes #572

This line is also using process.env.BASE_URL instead of the relative path:
https://github.com/nest-desktop/nest-desktop/blob/dev/src/router/index.ts#L8

This line is also using process.env.BASE_URL instead of the relative path: https://github.com/nest-desktop/nest-desktop/blob/dev/src/router/index.ts#L8

@jhgoebbert We guess that this should not be required. We will test it without changing that line, but if it is required, we will come back to this later. :+1:


events do not agree on the timeline

in the day view, change day-view-start = "vm.dayViewStart"   to day-view-start = "'08:00'" to begin at 08:00 AM, but not change this coiciden events in the time line.

Can you share a plunker which reproduces your problem. Thanks.

Closing to keep issues clean, if this is still an issue let me know.


Chapter 4 Example 8: Changed 'loc' to 'pos'

Updated the Particle class in both the examples.

Changed loc to pos in flight404/flight404_particles_1_simple.

thank you!


Question: Have you been using passport with koa?

I am trying to implement passport in koa for my react app. Have any of you used the koa-passport package and what are our experiences? I am really struggling to make it work and would love to see how people implemented it, especially with react.

Respectfully, this isn't necessarily the right forum for inquiries like these. I don't think anybody minds questions like this but for your own sake, Stack Overflow or IRC could assist you much faster.
koajs/koa#issues is generally intended for discussion surrounding the library, not application specific problems.

I understand and agree.

@fl0w Have the KoaJS team thought of opening a gitter.im or a slack community for this kind of issues?

note that I am only contributing where I can, not "part of the team"
@maticrivo there's been some discussion previously and if I remember correctly a preference for slack was voiced. Nothing specific to putting that into effect though as far as I know.
Personally I wouldn't mind an official channel to direct questions that fall out of scope for an github issue.

@tj @jonathanong @dead-horse what do you guys say?

https://gitter.im/koajs/koa?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge
@maticrivo we do have a gitter for discussion.


Update pyup config; add a docstring

Coverage remained the same at 90.22% when pulling 9941f652dffa3385221289572388a4f7a33b752c on cranti:nits into 064e51ce88a08500822864a3cbbaf6c9c2aea207 on kensho-technologies:dev.

Thank you!


Screen ruler may trigger the game mode of Nvidia's graphics card and cause the screen to turn white

Microsoft PowerToys version
0.75.1
Installation method
GitHub
Running as admin
No
Area(s) with issue?
Screen ruler
Steps to reproduce
see downward.
it  trigger the game mode of Nvidia's graphics card and cause the screen to turn white.
✔ Expected Behavior

❌ Actual Behavior

Other Software
No response

Actually, the same bug also happens in Shortcut Guide's gui, and my GPU is 3060 laptop with the latest Nivida driver.

This is frequently caused by the app Mica For Everyone. Please see #28217 for more details on how to resolve this incompatibility. Does this help? /needinfo


Misunderstanding with Eloquent's create method - it should be static only.

I just spent an hour to figure out why $myModel->create() doesn't work (no data is being stored in database table). The create is a static method. I suggest that in Eloquent it is not allowed to call create method dynamically, so there is no way to misunderstand the functionality of that method. Or fix the problem by making it possible to use that method dynamically (alias it to the save method).
Or maybe it should be written in docs with big red text, so next time you could just say RTFM... :smile:

Create works just fine even if you call it like you did. But you need to provide an array of data, otherwise nothing will happen.

@acasar Yes, I understand that. But there is no point on doing that ( (new Car($dataThatWillBeLostAnyway))->create($request->all()) vs Car::create($request->all()) ). And in my case I want to add data that is not set as "fillable". :wink:

@taai There is a difference between passing attribute-value pairs to create and passing it to model instantiation.
What you are looking for is (new Car($dataThatWillBeLostAnyway))->save(). I think this is sufficiently covered in the documentation.

To create a new record in the database, simply create a new model instance, set attributes on the model, then call the save method

and

You may also use the create method to save a new model in a single line. The inserted model instance will be returned to you from the method.


More gamut fixes/adjustments

By default use no threshold and force 0 threshold in all fit methods.
Also, disable using other spaces to determine if in_gamut. I'm not
convinced it is needed if we are checking both anyways. And my
assumption that fitting must be done in the same space as the check may
be wrong as well.
We've already fixed the issues of conversions and gamut checks by using
a precise gamut check, so really the only issue is whether spaces like
HSL should be fit in sRGB or in there own space. I'm thinking it may be
the latter.

@gir-bot lgtm


Fixed features search request

Issue 1276
Changed casing on features attributes to snake casing. Also includes an added function in test_client.py to test the new changes.

+1 added a missing test and fixes the sequence annotions client.

+1

please bring up to date with master


Delete manifestChecker / arcs_manifest_parse_test

It was only used in a couple of places, and isn't actually very helpful. This script would check that a .arcs file parsed correctly, and that the resources it mentioned (e.g. other .arcs files, .js/.json/.wasm files) actually exist.
However, most of those things we aren't using right now, and we have other BUILD rules that check a manifest file parses correctly as a side-effect (e.g. arcs_manifest_proto). Additionally, manifestChecker was too strict about certain things (e.g. it checks that a particle spec has either an implementation file or is marked external) which we don't want to enforce right now.
So, I'm deleting it. It can be resurrected in the future if we decide that it was actually helpful.
(Review only the second commit)

CC: @alxrsngrtn


Google Colab or Hugging Face

could please provide me a Google Colab or Hugging Face sample of your project to load a new image(or a set of new images) and get output such as your demo on https://milan-demo.csail.mit.edu/. For example, I want to be able to input images and a query and it produces an output of the corresponding image highlighting the regions of most activated neurons and their predicted description.
I would appreciate it if you could provide such code.

have you achieved your goal? I also want to try this project code as you said.


[Bug]: kid section and accessories not visible under products after switching to any page except home

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
when i look from the home page i see there are 4 product types but after switching to any of the tab through navbar , it shows only two products (missing kid section and accessories)
(@harikasayani please assign me this issue, i'll fix it as quick as possible)
Add Screenshots / Video Clip
here is how it looks at home page:

and this is how at other pages:

What browsers are you seeing the problem on?
Chrome, Microsoft Edge
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

@anshxika Proceed!

@hars-21 thankyou and the PR is done from my side for this bug :)
(added the sections where-ever they were missing)


bragged

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.1
logical conflict every: 100
sleep for: 300s
close stale after: 4 hours
[pullrequest]
requests per hour: 30
deps=[b]

/trunk merge


[Installation]: offline chat gpt

Your current environment
The output of `python collect_env.py`

How you are installing vllm
pip install -vvv vllm

Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Please elaborate on your issue.


Classes not detected by Tailwind 3 JIT?

Hey there, not sure if I'm missing something here.
I've added a number of classes to my config for links. None of them are recognized if the class doesn't already exist in the compiled CSS from Tailwind's JIT. If i manually add a div, say, with the desired classes, Tailwind detects and adds to CSS, which then makes the Bard content display properly.

Thanks for reaching out @dambridge
If I do understand the problem correctly, that problem can't be solved by classify.
Could you give me a short feedback, if one of those two things resolves your issuse?

Adding your classes to the safelist: https://tailwindcss.com/docs/content-configuration#safelisting-classes
Adding the classify config file to the Tailwind content: https://tailwindcss.com/docs/content-configuration#configuring-source-paths

Yeah, was looking more at this too. I'll try these and let you know how it goes!

OK, so #1, safe listing classes, works. #2 does not.

Thanks for the feedback!
I never tried option 2 personally but thought that it might work...

Quick follow up. Looks like safe listing classes isn't doing it either. Only solution I can find is to put the classes somewhere in one of the scanned files that is hidden so the JIT can pick it up.

The syntax for the safelist did change from version 2 to 3, did you adjust the settings?
Either way: I will merge the PR of John the next couple of days. He did a great job and does include a very nice solution for this issue, which should make this really easy and a no-brainer. Let's wait a few days and then check if everything is working as expected.

I did some local tests and can confirm that both solutions do work:

Adding your classes to the safelist: https://tailwindcss.com/docs/content-configuration#safelisting-classes
Adding the classify config file to the Tailwind content: https://tailwindcss.com/docs/content-configuration#configuring-source-paths

I liked your way @JohnathonKoster a lot, but after some testing, the second option does work great and does the same as the export command, but needs zero configuration.
Out of maintainability reasons, I did remove the export command and hope that's ok: https://github.com/visuellverstehen/statamic-classify/commit/5d35f738788283817f8ae7a60ba22a42a8f317d2
I added documentation, to show how to add the config file to your TailwindCSS config file:
https://github.com/visuellverstehen/statamic-classify#working-with-css-frameworks-like-tailwind-css
Hopefully this helps to make things much clearer @dambridge 😊
Please reach out if not.


add generic Jenkins files for linux LH and win32

Issue: #504
Signed-off-by: lanxia<EMAIL_ADDRESS>

Reviewer: @ShelleyLambert


Problem to runt the role ossec_server

hello, first at all thanks for the this cookbook look amazing,
Im trying to runt the rolse ossec_server but i have this error
Cookbook Trace:
---------------
/var/chef/cache/cookbooks/ossec/recipes/server.rb:28:in `from_file'

Im using the same example than you have in your Readme
name "ossec_server"
description "OSSEC Server"
run_list("recipe[ossec::server]")
override_attributes(
  "ossec" => {
    "conf" => {
      "server" => {
        "global" => {
          "email_to" =><EMAIL_ADDRESS>          "smtp_server" => "smtp.yourdomain.com"
        }
      }
    }
  }
)

really im not sure what's im doing wrong, any help?
thanks.

Please don't contact me privately like you did, that isn't very appropriate in free software projects and this isn't even my cookbook. I'm not overly familiar with the server recipe. You'll need to provide a much fuller trace than that for us to see what's really wrong.

Sorry about that, i saw a nice pull request from you in a fork repo, anyway you are right,
thanks for your help.

Given that you haven't set node['ossec']['server_env'] or changed node['ossec']['server_role'], I would guess that node['fqdn'] is blank, which can happen if you haven't configured /etc/hosts properly. The exact way to do this varies between distros but check that the <IP_ADDRESS> line doesn't start have localhost as the first entry.

Thanks for your reply
my /etc/hosts is easy:
<IP_ADDRESS>   ossec_manager localhost localhost.localdomain localhost4 localhost4.localdomain4
::1            ossec_manager localhost localhost.localdomain localhost6 localhost6.localdomain6

so you mean in the server role look like that:
name "ossec_server"
description "OSSEC Server"
run_list("recipe[ossec::server]")
override_attributes(
  "ossec" => {
  "server_env":  "server"
    "conf" => {
      "server" => {
        "global" => {
          "email_to" =><EMAIL_ADDRESS>          "smtp_server" => "smtp.yourdomain.com"
        }
      }
    }
  }
)

[root@ossec_manager user]# hostname
ossec_manager
[root@ossec_manager user]# hostname -f
ossec_manager

It's not that then. You shouldn't need to set server_env. I'm afraid I don't know what the problem is as I prefer to use my authd stuff from #48 instead of this recipe.

i will try that, usually we work with agent_auth, this #48 was the reason to contact with you, thanks!

hi again @chewi sorry for ask again, after use your #48 as you say in your README if i set the agent_server_ip attribute manually the recipe don't try to use agent_auth
this is my role
{
  "name": "ossec_client",
  "description": "OSSEC Client Agents",
  "json_class": "Chef::Role",
  "default_attributes": {

  },
  "override_attributes": {
    "ossec": {
      "agent_server_ip": "<IP_ADDRESS>",
      "conf": {
        "agent": {
          "syscheck": {
            "frequency": 321
          }
        }
      }
    }
  },
  "chef_type": "role",
  "run_list": [
    "recipe[ossec::client]"
  ],
  "env_run_lists": {

  }
}

sorry for bother you.

I don't understand how you thought that.

Set the agent_server_ip attribute manually and this recipe will attempt to register with the given server running ossec-authd.

Don't use the ossec::client recipe here, use ossec::agent_auth instead. I know this is confusing and that's why I've asked for the SSH method to be dropped or at least for the recipes to be renamed.

Oh understood, now works nice in agent and manager.
on the other hand, the certs for authd are not created by the recipe, right? so they need to be created or moved to the ossec folder

Correct, I find certificate creation within cookbooks to be a bit impractical. If you want this then I suggest creating a wrapper cookbook. Glad you got it working.

ok perfect thanks again! and sorry for the first contact!

sorry @chewi one last question, how do you manage the local_rules.xml??

I haven't got very far into actually configuring OSSEC yet. I suppose the Gyoku stuff could have gone further and rendered rule files too, probably through an LWRP. On the other hand, don't the rule files tend to be quite static? Even the custom ones?

I'm jumping in a bit late here, but I'd like to make most of this cookbook a LWRP.  We've learned a lot in the last few years and attribute driven cookbooks still have a lot of issues in the wrapper cookbook model.  Pure library cookbooks are much more powerful for complex software packages like OSSEC

yes and no, the local_rules many times are useful to rewrite other rules, also you might need to create a new rules, etc.
on the other hand, do you know RESTful API? you can do many things in ossec like add new agents without ossec-auth or agent-auth.
If i can help with ossec, happy to help :)

@chewi sorry one last question, when i try to use a different repo than atomic or my own rpms for the server in the common recipe i have the next error
manager.yokatan.com     # Declared in /var/chef/cache/cookbooks/ossec/recipes/common.rb:46:in `from_file'
manager.yokatan.com
manager.yokatan.com     file("/var/ossec/etc/ossecs.conf") do
manager.yokatan.com       action [:create]
manager.yokatan.com       retries 0
manager.yokatan.com       retry_delay 2
manager.yokatan.com       default_guard_interpreter :default
manager.yokatan.com       declared_type :file
manager.yokatan.com       cookbook_name "ossec"
manager.yokatan.com       recipe_name "common"
manager.yokatan.com       owner "root"
manager.yokatan.com       group "ossec"
manager.yokatan.com       mode 288
manager.yokatan.com       manage_symlink_source true
manager.yokatan.com       content #<Chef::DelayedEvaluator:0x00000003e59248@/var/chef/cache/cookbooks/ossec/recipes/common.rb:53>

manager.yokatan.com       path "/var/ossec/etc/ossecs.conf"
manager.yokatan.com     end
manager.yokatan.com
manager.yokatan.com
manager.yokatan.com Running handlers:
manager.yokatan.com [2016-04-01T13:25:32-05:00] ERROR: Running exception handlers
manager.yokatan.com Running handlers complete
manager.yokatan.com [2016-04-01T13:25:32-05:00] ERROR: Exception handlers complete
manager.yokatan.com Chef Client failed. 1 resources updated in 50 seconds
manager.yokatan.com [2016-04-01T13:25:32-05:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
manager.yokatan.com [2016-04-01T13:25:32-05:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
manager.yokatan.com [2016-04-01T13:25:32-05:00] ERROR: file[/var/ossec/etc/ossecs.conf] (ossec::common line 46) had an error: NoMethodError: undefined method `to_hash' for nil:NilClass
manager.yokatan.com [2016-04-01T13:25:32-05:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

The folder /var/ossec/etc is the same, same permissions than atomic rpms, all the same but still having this error.
i dont understand from where came the line manager.yokatan.com       atomic_update truebecause also i delete all reference to install through repository, i install the rpm manually and later try to control with chef.
any idea?
thanks again

My guess is that your RPM does not include /etc/ossec-init.conf with a TYPE= line. That would cause node['ossec']['install_type'] to be nil and line 56 to fail. You need this line set to either server or agent, depending on the RPM. Another way might be to set node['ossec']['install_type'] as an override attribute but I haven't tried that.
We use our own RPMs too, hosted on packagecloud, but these are based on the Atomic ones. The Atomic repository will always be added by the repository receipe and we do not try to prevent this. Instead, we add our own repository beforehand with the packagecloud cookbook and set its priority to 1 so it is chosen over Atomic. I forget exactly how you do that and I don't have the code to hand.

yes thanks, i had two errors,
1.- my rpms don't left ossec-init.conf in /etc/, left in other folder
2.- to me server is manager in this ossec-init.com.
I just change the common and the attributes/default.rb to manager.
My specs are based in wazuh of course :) thanks again for your help


Rank fail to meet the condition.

Describe the bug
After installing Answer 1.0.2, I created a user. This user should ask the first question. But saving the question is not possible, because this - meaningless - error message appears:
Rank fail to meet the condition.
To Reproduce
Steps to reproduce the behavior:

Installation Answer 1.0.2
registration user
manual release of the user in the admin dashboard
ask new question
title, text filled in
tags created
send question
error message appears

Expected behavior
Question should be saved.
Platform (please complete the following information):

Device: Desktop
OS: Windows 11
Browser Firefox
Version 108.0.2

Ok. Thanks.
Yes, I can confirm that if the user is activated via activation link, then asking questions also works perfectly.


DEPRECATION: ember-cli-babel 5.x has been deprecated.

Getting the warning:
DEPRECATION: ember-cli-babel 5.x has been deprecated. Please upgrade to at least ember-cli-babel 6.6. Version 5.2.8 located: bai-ui -> ember-pikaday -> ember-cli-babel
"ember-pikaday": "^2.2.4"
"ember-cli": "^3.1.4"

Duplicate of #175

Closing because it's a duplicate. I will publish a fix for this to NPM soon!


Istio proxy-status correction for 1.0.3

This is the initial PR for https://github.com/istio/istio/issues/12295.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ibm-jason
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ibm-jason: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

This PR was not created properly. Closing.

@ibm-jason: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ibm-jason: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ibm-jason: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-integ-k8s-tests

prow/istio-presubmit.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ibm-jason: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-integ-k8s-tests

prow/istio-presubmit.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-presubmit

prow/istio-integ-local-tests.sh
d43304f093f49f6af3dfe298077b73cab6a5f70e
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Track title is not being changed when switching to prev track

Here's how it works now:

This issue has been fixed in 9498e020c5c94309cc3ee7385cea70274a8ab580 along with the rework of the metadata display.


Study: build TLS without Cipher

After #5207, when USE_PSA_CRYPTO is enabled, the TLS library should no longer have any direct dependency on MBEDTLS_CIPHER_C, as all crypto operations are now done via PSA. So, ideally the dependency from TLS on MBEDTLS_CIPHER_C in check_config.h should be replaced with a dependency on MBEDTLS_USE_PSA_CRYPTO || MBEDTLS_CIPHER_C and we would be able to build TLS without Cipher.
This task is to study what needs to be done in order to achieve that result.
Expected result: a series of estimated tasks.

Right after #5434, I tried building with the default config plus MBEDTLS_USE_PSA_CRYPTO minus MBEDTLS_CIPHER_C (minus NIST_KW_C which depends on it), with the modified dependency in check_config.h. This shows two kind of issues:

I seems we never tried to build without CIPHER_C, as a number of modules have undeclared (meaning: not checked in check_config.h) dependencies on it: ccm.c, cmac.c, gcm.c, pkcs12.c, pkcs5.c, psa_crypto_cipher.c, psa_crypto_mac.c - these should be documented.
So far, the software implementation of ciphers (and some macs, I assume CMAC) in PSA still depends on cipher.c, and that dependency should be remove before PSA can be used as a replacement for cipher.c.

the software implementation of ciphers (and some macs, I assume CMAC) in PSA still depends on cipher.c

PSA calls mbedtls_cipher_xxx functions for the following purposes:

In psa_aead_setup, just mbedtls_cipher_info_from_psa to decode the cipher ID for AEAD algorithms based on block ciphers, to pass it to mbedtls_gcm_setkey or mbedtls_ccm_setkey.
Indirectly via the CCM and GCM modules, which implement those AEAD algorithms on top of ECB done via the cipher API.
For all cipher operations.
For CMAC, both directly since the API relies on mbedtls_cipher_cmac_xxx to perform CMAC opertaions, and indirectly because the CMAC module works  on top of ECB done via the cipher API.

Building PSA without cipher.c requires not only removing the indirections in the PSA code, but also rewriting the lower-level modules not to depend on cipher for ECB.

Yes, building TLS without cipher.c (without drivers) would require much more work than I realised before. The direct dependencies were the easy part, the indirect dependencies are quite deep.

Actually, I've been thinking: I think we already have tests (all the test_psa_crypto_config_accel_xxx components in all.sh) for building a drivers library libtestdriver1.a based on our implementation of some primitive X, then building the library (and programs and tests) in a configuration without X by linking to that. We could probably do something similar here - build every primitive that depends on CIPHER_C in libtestdriver1.a and then build the rest of the library including TLS without CIPHER_C, linking to the testdriver library to get an implementation of them via the PSA API.
I'm not sure how hard that extension would be, as I'm not very familiar with how the libtestdriver1.a mechanism works. @ronald-cron-arm (since according to git blame you did a lot in this area) do you have thoughts to share?

build every primitive that depends on CIPHER_C in libtestdriver1.a and then build the rest of the library including TLS without CIPHER_C, linking to the testdriver library to get an implementation of them via the PSA API

That does indeed sound like an evolution of component_test_psa_crypto_config_accel_cipher: all cipher mechanisms are provided through psa_xxx, and not through mbedtls_xxx. The existing test component keeps MBEDTLS_CIPHER_C because it's also used for AEAD and because of its existing uses in TLS. So I think you want to make a test component where all ciphers (authenticated or not, and also CMAC) are disabled in mbedtls_xxx, and there are PSA drivers for those mechanisms.

Actually there's another class of issues with getting a usable build of TLS without all ciphers as PSA drivers and no CIPHER_C: currently the dependencies are expressed as #if defined(MBEDTLS_GCM_C) so if we did a build with AES-GCM provided by a driver (with our implementation compiled out), currently TLS would think that no AES-GCM ciphersuites are available.
This can be fixed by changing dependencies from MBEDTLS_xxx to PSA_WANT_xxx in all the right places (perhaps with an extra abstraction layer to account for whether USE_PSA_CRYPTO is defined or not). So, definitely not a blocker in the long term, but clearly an extra chunk of work to be planned (and perhaps have design discussions about).

Note: as of today's development, libmbedtls no longer has any runtime dependency on any mbedtls_cipher symbol when MBEDTLS_USE_PSA_CRYPTO is enabled:
scripts/config.py full
docs/architecture/psa-migration/syms.sh full
egrep 'mbedtls_cipher' full-tls-external

has an empty output.

Tasks have been created.


Refactor [DEV-9285] top suffix behavior and implementation adjustment

DEV-9285
Two things happening here:
1. Refactor top-suffix-only logic so it overflows if a space is present in the suffix.
So:
"5lbs": pushes to the left
"5lbs per meter":  overflows
"5 units": overflows
2. Refactor to use direct manipulation via props as to not trigger a re-render
Testing Steps
In the editor:

Create a Bar or Line chart
Open the "Left Value Axis" panel
Select the checkbox labeled "Only Show Prefix/Suffix"
Add a Suffix without a space (i.e. something like 'lbs')
Observe that the suffix pushes the tick labels to the left, to the left
Add a suffix with a space (i.e. something like " units" or "lbs of something")
Observe that the suffix now overflows to the right instead

Self Review

I have added testing steps for reviewers
I have commented my code, particularly in hard-to-understand areas
My changes generate no new warnings
New and existing unit tests are passing

Screenshots (if applicable)

🫷Hold of on reviewing until i delete this comment - Jay noticed a multiplying space issue


Prettier: To get Format on Save to work, set Editor: Default Formatter setting

During this session 03-js-tools/B-prettier.md, someone asked, "If it's working when we run the script, but not working when we save the file, what might be a troubleshooting step?"
This was my issue as well. I searched and found this Stack Overflow post: https://stackoverflow.com/questions/52586965/why-does-prettier-not-format-code-in-vs-code
We also must set the Editor: Default Formatter setting to use Prettier, as in the following image:

Then formatting on save works properly. I probably have way too many extensions installed in my VS Code!

yeah also worked for me when I set formatting to Prettier

Thanks, everyone! I updated the annotation to make it more clear in the course.


DL-209: Update build script to build distributedlog website under bookkeeper.apache.org/distributedlog

Problem:
Currently bookkeeper.apache.org/distributedlog is using content at the root of branch asf-site.
distributedlog.incubator.apache.org is using content at content dir of branch asf-site.
The website build script should support both.
Solution:

add _config-apache.yml for building website at baseurl /distributedlog
update website/build.sh to build apache to build different configurations
add a publish-website.sh to be able to publish website by CI

Also:

add _config-staging.yml for building website at staging
add a staging-website.sh to publish staging site.

@leighst @fcuny can anyone help review this one as well?

rebased the pull request to latest master


HTTP highlighting broken on github.com

It appears that HTTP highlighting is broken on github.com. Rather then creating an issue github support sent me here to report the bug.
For example:
Request
GET /a/b/c HTTP/1.1
Host: cloud.com
Accept: application/json

Response
HTTP/1.1 200 OK
Content-Type: application/json

{
  "a": "A",
  "b": {
    "c": "C"
  }
}

More examples of the issue can be found here: https://github.com/OADA/oada-docs/blob/master/rest-specs/REST-API-Examples.md. That page, and the others in that repo, used to be colorful.
languages.yml (https://github.com/github/linguist/blob/master/lib/linguist/languages.yml#L1194) seem to indicate HTTP highlighting is still supported.

@abalmos It looks good in Lightshow so I opened a pull request (#1952) to use this grammar.

This will be addressed in https://github.com/github/linguist/pull/2011 which should go live very soon.


How to disable all animation ?

How can we disable all animations open/close from your lib?

It would require some experimentation but the answer should be no different than the answer to: "How do you disable animations for UITableView insertion / deletion of the cells?"
I would try subclassing, then overriding insertRowsAtIndexPaths / deleteRowsAtIndexPaths and passing None for animations. If that doesn't work I'd try wrapping those calls with techniques to disable / remove animations (there are lots of answers on StackOverflow).


Support Vector Machine - Documentation

I would like to write the whole analysis of this algorithm and documentation.

Task : Create detailed documentation of the Support Vector Machine ML algorithm by updating readme.md
Level-1 : 10 points

I would like to create detailed documentation of SVM. Please assign me this task. I am a gssoc participant.

Task : Create detailed documentation of the SVM ML algorithm by updating readme.md
Level-1 : 10 points

Sure @SNIGDHA2654 ,You have been assigned the task to create detailed documentation of SVM.
Try to submit a PR in 2 days.

It's already been more than 2 days @ashwani-rathee please if you could assign this task to me. I am also a GSSoC'21 participant.

@surbhi2408 It's clear from my side, we are now waiting for admin's decision on merging

Merged


Minor: Invoking gpg2 with --no-use-agent outputs a warning

On Debian Stretch RC2 with
$ gpg --version
gpg (GnuPG) 2.1.18
...

invoking aptly publish repo ...
leads to
gpg: WARNING: "--no-use-agent" is an obsolete option - it has no effect

I've got very promising result so far with Go internal openpgp package, looking towards that as replacement for invoking gpg command line utility.
I don't feel supporting wild mix of all the gpg versions makes any sense.