Calling methods in externs upon wrong object should warn

I was calling a method which was missing from my externs, and was debugging why the compiler wasn't picking this up:
Externs:
var ExternsProblem = {};

/** @constructor */
var SomethingElse = function() {};

SomethingElse.prototype.something = function() {};

Code:
ExternsProblem.something(); // should fail, doesn't

You can see that something is defined elsewhere in the externs, but not here!
Debugger

Ticking ENABLE_ALL_DIAGNOSTIC_GROUPS highlighted the problem:
input0:2: WARNING - could not determine the type of this expression
ExternsProblem.something(); // should fail, doesn't
^^^^^^^^^^^^^^

Debugger
This error is UNKNOWN_EXPR_TYPE which has been disabled for 5 years.

The solution turns out to be to add @const to the externs:
/** @const */
var ExternsProblem = {};
...

Ah. Now I see the warning I expected:
input0:2: WARNING - Property something never defined on ExternsProblem
ExternsProblem.something(); // should fail, doesn't
               ^^^^^^^^^

Debugger

I gather that the compiler doesn't know the type of external global var ExternsProblem so never complains about any property access upon it. This makes sense.
However we need a warning to prevent developers from calling functions that simply don't exist (typo) or aren't documented.
I'm guessing JSC_UNKNOWN_EXPR_TYPE was disabled 5 years ago because it's old. Perhaps there's another check that should be kicking in here and isn't. I'm going to explore forcibly enabling this warning in all my externs, but would appreciate knowing how severe you think this issue is, and whether it affects other cases outside of externs.

I think the real takeaway here is we should warn whenever we see a non-constant namespace declaration in externs. Or just assume they are always constant.

Enable "strictMissingProperties" it should warn.  Please report back if you have already done this or if this doesn't fix the problem.

or more general "strictCheckTypes"

I can't recall exactly but I have played with those strict options before (I think one was suggested as alternative NTI when it was removed) and I think it generated millions of warnings for commonly used patterns like:
let a = {};
a.something = ...
a.somethingElse = ...

This is just from memory, I'll need to revisit this in more detail.

It will certainly generate warning for values typed as "Object" but it understands literals:
https://closure-compiler-debugger.appspot.com/#input0%3D%250Alet%2520a%2520%253D%2520%257B%257D%253B%250Aa.something%2520%253D%25201%253B%250Aa.somethingElse%2520%253D%25202%253B%250A%250Ause(a.something)%253B%250Ause(a.bsProp)%253B%250A%250A%250Alet%2520b%2520%253D%2520%257B%257D%253B%250Ab.bsProp%2520%253D%25202%253B%250A%250A%250A%252F**%2520%2540type%2520%257B!Object%257D%2520*%252F%2520var%2520x%2520%253D%2520fn()%253B%250Ause(x.something)%253B%250Ause(x.bsProp)%253B%250A%26input1%26conformanceConfig%26externs%3Dvar%2520fn%253B%26refasterjs-template%26includeDefaultExterns%3Dtrue%26CHECK_TYPES%3Dtrue%26STRICT_CHECK_TYPES%3Dtrue%26MISSING_PROPERTIES%3Dtrue%26TRANSPILE%3Dtrue%26CLOSURE_PASS%3Dtrue%26PRESERVE_TYPE_ANNOTATIONS%3Dtrue%26PRETTY_PRINT%3Dtrue

Closing in favor of https://github.com/google/closure-compiler/issues/3085
Reopen if you have more specific information.


add hardcoded authorizers for metrics and token reviews

/assign @sttts
proof in https://github.com/openshift/kubernetes/pull/588

/lgtm


Add support for bucket replication

I split this commit to make my changes easier to check. Please have a look at https://github.com/mineiros-io/terraform-aws-s3-bucket/commit/8fe3fe3b91b414548760697543bdd8540fddd834 and let me know if anything need to be done to merge this in master.
Thanks for this awesome module.

Hi @aminehmida,
thanks for your great contribution. I will discuss needed changes to your PR with the team and add comments within the next days.
We do have some internal guidelines to keep base modules within one service and not add other services like roles into them.. but it makes totally sense to create the replication role with the replication configuration tbh.
so far:

I split this commit to make my changes easier to check. Please have a look at 8fe3fe3 and let me know if anything need to be done to merge this in master.

no need to split commits here, as you can disable whitespace changes in the review console of github.. - check it out now ;) could you squash the two commits and force push again? not a hard requirement though ;)
Thanks again :rocket:

Hey, anything blocking this from being merged? We'd love to see these changes in master.

Hey, anything blocking this from being merged? We'd love to see these changes in master.

Hi @Limess ,
Let me discuss this with the team. This PR needs some changes in order to be merged back to master but we might be able to handle this internally this week.

This is very old. I am not working on this anymore. Feel free to use my code and make another PR if you like. Just going to close it for now.


fix: Tags on RMRK NFTs

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

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #3765
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

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

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

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

How does it work? Where can we test this?

How does it work? Where can we test this?

you mint NFT with selected tags and test whether they are included in metadata of the NFT at https://app.gc.subsquid.io/beta/rubick/007/graphql and now, whether they are showing with @Jarsen136's implementation

now something gets saved, but is this the way it should work? 👀

I think it's not.
By the way, I wonder what's name of the key should be shown when user selected some tag 🤔

I guess it's better now with the redesign of the property component.

I have doubts about the correct functioning, as this been tested on mint?

I have doubts about the correct functioning, as this been tested on mint?

Yup, petersopko have tested on rmrk. I have tested on bsx as well. It works well on minting.

pay 50 usd

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


[default][ux] bullets indicating active gallery element

Is it only me or those bullets indicating active gallery element are counterintuitive from UX standpoint?

Steps to reproduce:

Visit URL https://demo.vuestorefront.io/p/24-MB03/crown-summit-backpack-3
Click on the gallery indication bullets back and forth

Expected result:
Dark, filled bullet shoud indicate active element. Bordered bullet should indicate inactive element to simlate native HTML radio buttons group pattern.
Actual result:
It works opposite to expected
Additional question: is there any reason why we are using two separate viual patterns for slides indicators? Just look at homepage main slider bullets indicator:
https://demo.vuestorefront.io/

Good eye for details ;). It was just implemented by other person ( @nuovecode ). @vue-kacper do you want to correct it?

Thanks ;) Ok, since new theme will be available soon I'll apply this minor ux improvement directly to new theme if necessary

Yes, it's fixed now


Create make_tag_push_summary and make_receiver_push_summary tests: render using rmarkdown

Create test using rmd = FALSE argument.
Tag push:
Coverage aim: https://app.codecov.io/gh/mhpob/otndo/blob/main/R%2Fmake_tag_push_summary.R#L138
Current test file: https://github.com/mhpob/otndo/blob/main/tests/testthat/test-make_tag_push_summary.R
Receiver push:
Coverage aim: https://app.codecov.io/gh/mhpob/otndo/blob/main/R%2Fmake_receiver_push_summary.R#L208
Current test file: https://github.com/mhpob/otndo/blob/main/tests/testthat/test-make_receiver_push_summary.R

Closed by 8d07ff989c7a37d2fd6bda826645d4f21a079130


request_successful = false / Unknown error

Hey @drewm, thanks for this script, it's certainly helped. Up until now I've had no problems using the script to display subscriber counts with the old version, compatible with MailChimp's pre-v3 API. However, I cannot get the new version (for MailChimp 3) to work.
After I include MailChimp.php, I'm doing the following:
use \DrewM\MailChimp\MailChimp;
$MailChimp = new MailChimp('xxxxxxxxxxxxxxxxxxxxxxxxxx-us5');
$result = $MailChimp->get('lists');

If I try to echo/print the $result variable, I get nothing. If I var_dump it, I get bool(false). If I var_dump($MailChimp), I get the following:

object(DrewM\MailChimp\MailChimp)#1 (7) { ["api_key":"DrewM\MailChimp\MailChimp":private]=> string(36) "xxxxxxxxxxxxxxxxxxxxxxxxxx-us5" ["api_endpoint":"DrewM\MailChimp\MailChimp":private]=> string(33) "https://us5.api.mailchimp.com/3.0" ["verify_ssl"]=> bool(true) ["request_successful":"DrewM\MailChimp\MailChimp":private]=> bool(false) ["last_error":"DrewM\MailChimp\MailChimp":private]=> string(64) "Unknown error, call getLastResponse() to find out what happened." ["last_response":"DrewM\MailChimp\MailChimp":private]=> array(2) { ["headers"]=> array(20) { ["url"]=> string(40) "https://us5.api.mailchimp.com/3.0/lists?" ["content_type"]=> NULL ["http_code"]=> int(0) ["header_size"]=> int(0) ["request_size"]=> int(0) ["filetime"]=> int(-1) ["ssl_verify_result"]=> int(0) ["redirect_count"]=> int(0) ["total_time"]=> float(0) ["namelookup_time"]=> float(0) ["connect_time"]=> float(0.015) ["pretransfer_time"]=> float(0) ["size_upload"]=> float(0) ["size_download"]=> float(0) ["speed_download"]=> float(0) ["speed_upload"]=> float(0) ["download_content_length"]=> float(-1) ["upload_content_length"]=> float(-1) ["starttransfer_time"]=> float(0) ["redirect_time"]=> float(0) } ["body"]=> bool(false) } ["last_request":"DrewM\MailChimp\MailChimp":private]=> array(5) { ["method"]=> string(3) "get" ["path"]=> string(5) "lists" ["url"]=> string(39) "https://us5.api.mailchimp.com/3.0/lists" ["body"]=> string(0) "" ["timeout"]=> int(10) } }

And of course, note that I've removed my API key in the examples above. The API key is correct in my code, and I even generated a new one just to ensure I had it right.
Ultimately, I had to settle for this solution, which works perfectly. I have no idea what's preventing your script from working in my case. The PHP version I'm testing on is 5.3.1 and I've enabled cURL in php.ini, which I believe is required.
Any thoughts on what's going wrong?

Same here

Same issue here.

For me setting the :
public $verify_ssl = false;
in MailChimp.php worked I have no SSL sertificate set on my local machine.

Unknown error, call getLastResponse() to find out what happened.

Did you call getLastResponse() to find out what happened, @impressivewebs?

@drewm I thought I tried it, but I don't really understand what I'm supposed to read when I use that. I didn't get anything out of that to help me debug it, if that's what you're asking. If you want, I can try again and see what it says. But IIRC, it didn't help me.

@drewm Ok, if I do:
var_dump($MailChimp->getLastResponse());

I get:
array(2) { ["headers"]=> array(20) { ["url"]=> string(40) "https://us5.api.mailchimp.com/3.0/lists?" ["content_type"]=> NULL ["http_code"]=> int(0) ["header_size"]=> int(0) ["request_size"]=> int(0) ["filetime"]=> int(-1) ["ssl_verify_result"]=> int(0) ["redirect_count"]=> int(0) ["total_time"]=> float(0) ["namelookup_time"]=> float(0) ["connect_time"]=> float(0.032) ["pretransfer_time"]=> float(0) ["size_upload"]=> float(0) ["size_download"]=> float(0) ["speed_download"]=> float(0) ["speed_upload"]=> float(0) ["download_content_length"]=> float(-1) ["upload_content_length"]=> float(-1) ["starttransfer_time"]=> float(0) ["redirect_time"]=> float(0) } ["body"]=> bool(false) }

I have no idea what I'm looking for in there.

That looks like an SSL problem with your server:
["ssl_verify_result"]=> int(0)

Oh... well, the server I'm testing on is my local machine, so that would explain that. Another user in this thread mentioned setting public $verify_ssl = false;, so I guess that's what's needed? Forgot to try that. Is it unsafe to use the script in a non-SSL environment or something?

Yeah, I just tried it and it seems to work fine now. I guess that's solved, but I suppose it would be good if the docs explained the problem, or else provided instructions to set that to false if necessary.

They do: https://github.com/drewm/mailchimp-api#troubleshooting

@drewm Ah ok... Sorry, my bad. I guess I hadn't read the docs since the update. :( Thanks.

I set the verify_ssl to false and it works as normal (on localhost)
$MailChimp->verify_ssl = false;

Why the error related to ssl is not returned by getLastError? It looks like another issue.
The error should be clearly returned as well as other errors. Taking a look at getLastResponse is more technical and in some cases doesn't bring the clarity unfortunately in order to understand the cause as for example the cause with SSL.

https://github.com/bryan3561/mailchimp-api
this is correct
or changue for this code:
https://github.com/drewm/mailchimp-api/commit/d2691e57fb7b45ba475a09254e058d851f322c30


Added delegate for every cell.

Added a delegate to enable Custom Delegation in every cell.

There's already something called a 'parentController' that does mostly what you want here.  You just need to cast it into whatever class you need.


drivers: led: Add system call handler support

Add system call handler support to LED subsystem. No buffers are
involved in any of the API's and hence the syscall support is
straightforward.
This PR has been validated on 96Boards Neonkey with CONFIG_USERSPACE
enabled.
Signed-off-by: Manivannan Sadhasivam<EMAIL_ADDRESS>

Codecov Report

Merging #7339 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7339      +/-   ##
==========================================
+ Coverage   55.09%   55.09%   +<.01%
==========================================
  Files         474      474
  Lines       51786    51786
  Branches     9937     9937
==========================================
+ Hits        28530    28531       +1
+ Misses      19304    19303       -1
  Partials     3952     3952

Impacted Files
Coverage Δ

lib/posix/pthread.c
69.84% <0%> (+0.5%)
:arrow_up:

Continue to review full report at Codecov.

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


Adjusting internal spring maths

A couple simple internal changes that shouldn't affect end users of Fusion, except that springs may be more precise, especially in extreme circumstances.
We could make some modifications to overdamped spring calculations inside springCoefficients, which may improve numerical precision (thanks to AxisAngles for suggesting this on stream a couple of months ago: https://youtu.be/SWNRgQEyFGI?t=5117):

zDivide can be rearranged from 1/(z1 - z2) to -1/(2 * zRoot * speed)

This should be more numerically precise

z2 can be rearranged from (zRoot - damping) * speed to -speed / (zRoot + damping)

This can be made consistent with z1, which can be slightly rearranged to -speed * (zRoot + damping)

If we multiply every usage of z1 and z2 by speed, we can remove the speed from z1 and z2's definitions, making them -1 * (zRoot + damping) and -1 / (zRoot + damping) respectively.
We can then copy timeStepSpeed up into this scope and simplify both occurences of timeStep * speed.
We can also then factor out the speed for posPosCoef to simplify.
If we remove speed from zDivide's definition,  we can divide all coefficients by speed

In the video, trey arrives at velPosCoef = speed * (-z1Exp + z2Exp) * zDivide, though I got velPosCoef = speed*speed*speed * (-z1Exp + z2Exp) * zDivide - I should follow up about this discrepancy and triple check

Trey "makes no guarantee of correctness", though I've checked over everything myself and it seems to be fine minus the one discrepancy I noted. They also noted at the time that the only change we should use is to the definitions of z1 and z2, so perhaps most of these changes should be discarded?
Furthermore, we could change the way we move springs over time from an incremental system to one based on initial conditions (thanks to Quenty for suggesting this):

springCoefficients allows us to accurately calculate the final position and velocity of a spring from an initial position and velocity.  It does this by returning coefficients we can apply to the initial postion and velocity.
springCoefficients's return value is solely dependent on damping ratio, speed, and time elapsed since the initial position and velocity was measured.
Currently, we use the last frame's position/velocity as the initial conditions, and the time elapsed is the time since last frame and now. The position and velocity returned become the spring's new position and velocity.
The advantage of this approach is that the time elapsed will be identical for all calls to springCoefficients, meaning that - if two springs share a damping ratio and speed, they can reuse the same springCoefficients return values, vastly cutting down on computations needed when multiple identically-configured springs are animating at once.
However, the disadvantage is that small floating point inaccuracies may be introduced by the repeated resetting of the initial position/velocity - this is typically not a problem except for very high frequency springs
We could eliminate these inaccuracies by instead only setting the initial position/velocity when the spring's goal changes (and adjusting our calculation of elapsed time accordingly)
However, this has the downside of no longer being able to reuse the same coefficients across springs
It's an open question whether we should implement this slightly more precise spring at a possible performance/caching expense

I'm perfectly happy to make the first set of changes (by AxisAngles) but we'll need to test out Quenty's changes to make sure that we don't lose too much performance by preventing reuse of spring coefficients.

I ran some testing to see if there's a significant impact on calculating coefficients per-spring versus reusing coefficients for entire batches of springs.
Here's the update time graphed with 1000 underdamped (speed 2.345, damping 0.272) springs (orange is per-spring, blue is batched):

Here's the update time graphed with 1000 critically damped (speed 1, damping 1) springs (orange is per-spring, blue is batched):

I think it's pretty clear to see that batching does not have a significant performance benefit. Yes, I double checked the results. Due to this I don't see any reason not to improve the numerical precision of springs at the expense of batching updates.


feature/eng-3827-replace-all-nextroute-engine-panics-with-useful-errors

Description
This PR adds an input validation for lat/lon and a couple of other fields.
Changes

input.go

Zero values should be allowed in all marked cases

I added links to where we are throwing an error when allowing the 0 values. so I think this is correct here.

We should not compound incorrect behaviour, I would fix the incorrect errors

I am not sure that this is a good approach. I forsee that this will create work for us because customers accidentally will pass "0" for max stop or max distance e.g. and complain that those vehicles are not assigned anything.
While the other way around, if we reject "0" for those fields they will find a way to get what they actually want without our help.
I'd like to get some broader feedback on my thought here: @sebastian-quintero @merschformann @dirkschumacher @muellerd (added Daniel here as well because he does the most customer work right now)

We should not compound incorrect behaviour, I would fix the incorrect errors

I am not sure that this is a good approach. I forsee that this will create work for us because customers accidentally will pass "0" for max stop or max distance e.g. and complain that those vehicles are not assigned anything. While the other way around, if we reject "0" for those fields they will find a way to get what they actually want without our help. I'd like to get some broader feedback on my thought here: @sebastian-quintero @merschformann @dirkschumacher @muellerd (added Daniel here as well because he does the most customer work right now)

I see the point for both really:

You are strict with what you allow and the user has to make sure that the input is always correct. Passing in a vehicle with max distance set to zero will give an error because that doesn't make sense. The user corrects the inputs.
You are flexible and you let the engine figure out what to do. A vehicle with max distance set to 0 cannot be assigned anything. The engine does not use that vehicle and simply assigns stops to other vehicles. This does not give an error because it is always assuming that the input is correct, this is, there is intention behind setting max distance to 0.

I think the more flexible we are, the better. If for performance reasons we would like to be stringent and not allow "incorrect" settings, then let's go with option 1 and validate the semantics of the input thoroughly. However, production systems often benefit from going with 2 because in case some service fails and sends 0s on some settings, you guarantee some service. That begs the question though: what is better? No solution or a bad solution?
There is no definitive answer of course so my actual preference is: let's make it an option. You know? Something like -flexibility {off, on} or whatever. When on, there are no zeroes allowed. When off, we make sure that even nextroute can tolerate a zero being passed as a limit.
Default behavior? 🤷🏻♂  up for debate.

For the float64 values you would not allow 0.0 but would 0.001, totally arbitrary. My view remains that an error ony if it is incorrect input not when we believe the user did not want to do this.


Remove all IE related CSS and update browserslist

Drop support for IE and communicate this in the README.md → Browser Support section.

Addressed in the "Version 3" work (see the v3 branch).


refactor(glue-alpha): Refactored Glue L2 alpha construct RFC 0497

Issue # (if applicable)
N/A
Reason for this change
Updating README with RFC-aligned documentation
Description of changes

Description of how you validated changes

Checklist

[ ] My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Merging to bring up to date with aws-cdk core PR

Submitted new PR: https://github.com/aws/aws-cdk/pull/32521


fix: Module instance delete navigation

Description
This PR addresses the below issues,

In App editor, deleting a package is not redirecting to add screen
In package Editor, deleting a js/query should fall back to the main
Icon not displaying for tabs of package instance

PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/30894
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

New Features

Added IDE type detection for enhanced redirection logic in code editing workflows.
Introduced customizable module icons in the Explorer view for better visual differentiation.

Enhancements

Improved IDE tab management to support JavaScript and Query modules, ensuring a smoother development experience.

/ok-to-test<EMAIL_ADDRESS>


pycsw does not split comma-separated typeNames (CSW GetRecords requests)

Description
pycsw does not split the different (comma-separated) typeNames, like mentionned in the specification OGC 07-006r1 (page 219)
ex: csw:Record,gmd:MD_Metadata
OK when requesting each typeName individually.
For information, GeoNetwork is compliant to this requirement.
Environment

operating system: ?
Python version: ?
pycsw version: 1.10.2
source/distribution
[ ] git clone
[ ] DebianGIS/UbuntuGIS
[ ] PyPI
[ ] zip/tar.gz
[ ] other (please specify):
web server
[ ] Apache/mod_wsgi
[ ] CGI
[ ] other (please specify):

Steps to Reproduce
Check CSW GetRecord (Post) on following servers:
https://data.noaa.gov/csw
Fail
http://demo.pycsw.org/services/csw
Fail
http://www.fao.org/geonetwork/srv/eng/csw
OK
<?xml version="1.0" encoding="UTF-8" standalone="no"?> <csw:GetRecords xmlns:gmd="http://www.isotc211.org/2005/gmd" xmlns:csw="http://www.opengis.net/cat/csw/2.0.2" xmlns:ogc="http://www.opengis.net/ogc" service="CSW" version="2.0.2" resultType="results" startPosition="1" maxRecords="1" outputFormat="application/xml" outputSchema="http://www.opengis.net/cat/csw/2.0.2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.opengis.net/cat/csw/2.0.2 http://schemas.opengis.net/csw/2.0.2/CSW-discovery.xsd">   <csw:Query typeNames="csw:Record,gmd:MD_Metadata">     <csw:ElementSetName>full</csw:ElementSetName>     <csw:Constraint version="1.1.0">       <ogc:Filter>         <ogc:PropertyIsEqualTo>           <ogc:PropertyName>csw:AnyText</ogc:PropertyName>           <ogc:Literal>WAVEWATCH</ogc:Literal>         </ogc:PropertyIsEqualTo>       </ogc:Filter>     </csw:Constraint>   </csw:Query> </csw:GetRecords>
Additional Information

Thanks for the report.  OGC 07-006r1 Table 65 says typeNames should be comma separated.  However in the rest of the document typeNames is expressed as space separated.  As well, if you try to validate the request XML in this ticket it fails validation if comma-separated, but passes if space separated.
Given pycsw is validating against the official schemas, it sounds like there is an inconsistency between the spec and the schema which should be clarified with OGC.


"our goals" links to a 404 (/about/) from homepage

If you click "our goals" at http://opensourcedesign.net you get a 404 at http://opensourcedesign.net/about/

Funny, cause "goals" exists: http://opensourcedesign.net/goals/

Yah, oversight on my part (forgot it links there). Now fixed.


Correction for raw affiliation Ingénieur d’études, Institut Agro – Montpellier Supagro, UMR CEE‐M (Center for environmental economics – Montpellier) France

Correction needed for raw affiliation Ingénieur d’études, Institut Agro – Montpellier Supagro, UMR CEE‐M (Center for environmental economics – Montpellier) France
raw_affiliation_name: Ingénieur d’études, Institut Agro – Montpellier Supagro, UMR CEE‐M (Center for environmental economics – Montpellier) France
new_rors: 03rnk6m14;00rwjvc96
previous_rors: 03rnk6m14
works_examples: W3122104333
contact: e92d5462b483d864a3e5d3c18835f585:17d4466e047fe14eb86807aafe5ba320c5aa @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Server profile - logical disk and sasLogicalJBODUri

Scenario/Intent
create logical disk in server profile
Environment Details
Module Version: current master - v5.1.1
**Ansible Version:**2.64
OneView Appliance Version: 4.10.03-364293
OneView Client API Version: 600
Python Version: 2.710
python-hpOneView SDK Version: v4.7
Platform: Ubuntu 18
Steps to Reproduce
using the example sysnergy_environment_setup, in the localstorage --> sasLogicalJBOD, the attribute sasLogicalJBODUri is not supported
Expected Result
Actual Result
Error: unrecognized JSON field
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: HPOneViewException: (u'Unrecognized JSON field.', {u'errorSource': None, u'recommendedActions': [u'Check the valid fields for this JSON object and remove or correct the unrecognized field.'], u'data': {}, u'errorCode': u'UNRECOGNIZED_JSON_FIELD', u'details': u'The field named sasLogicalJBODUri near line 1, column 1,607 is not a valid field in this JSON object.', u'message': u'Unrecognized JSON field.', u'nestedErrors': []})

Going forward, all the enhancement requests will be addressed in the NEW HPE OneView Ansible Collection. Please consider migration to the Ansible Collection.
Migration support material can be found in the Migration Support folder.
We have added this resource in our ansible collections backlog.
https://github.com/HewlettPackard/oneview-ansible-collection


Update to Go 1.17

What this PR does:
Updates to Go 1.17 and bumps dependencies of controller-runtime as well as Kubernetes.
Which issue(s) this PR fixes:
Fixes #292
Fixes #171
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

I updated the ticket, those dep updates are dependencies of updating the other dependencies ;)


Being able to send multiple pictures of a product in Offline mode / Initial product upload

Currently, the offline mode in the app doesn't let the user send multiple pictures of a product.
To achieve parity with the current app, it should let the user upload as many images as he wishes.
Bonus points if he can define which one it is :-)

Does it mean that the user adds images to an existing product? Or does it mean that he adds more images in the offline edit activity?

Both use cases make sense I guess.

44b5f2f160657cebedd9f471fa8df16812b021d2


setup.py: Discover tests

This PR makes it possible for the tests
to be discovered correctly by adding
a function get_test_suite in setup.py.
Closes https://github.com/vmware/tern/issues/406
Signed-off-by: PrajwalM2212<EMAIL_ADDRESS>

Earlier there was an error in loading test suite

Now the tests are discovered ad run correctly

This PR looks good to me. Will merge after a review from @nishakm.

@PrajwalM2212 Thanks for this PR! This will be helpful to run all the unit tests at once. When I run python setup.py test this change works for me to run the test suite.
However, I was looking through setuptools documentation and saw that test is deprecated in version 41.5.0 in favor of tox and the test command will be removed in future versions of the package. Therefore, I wonder if this change should be implemented with tox instead of setuptools test option to save ourselves trouble in the future? @PrajwalM2212 @nishakm thoughts?

I agree with @rnjudge that it would be worth it to move to tox as that seems to be the testing standard for Python projects. Also, this would be a nice GSoC project to work on, unless @PrajwalM2212 wants to take it on now :)

@PrajwalM2212 is working on a PR to use tox. Closing this PR.


Make clock_gettime syscall directly to skirt ABI issues

We (likely) want to hand code the syscall in asm for x86, x64... but that can be v2.
@jtattermusch - heads up

msan is pretty upset.

Seems OK now

Yeah okay :-P

Can we merge this soon?

LGTM & merging.


Zero downtime failover clarification

Replacing text from 3 different sections to align with one another.

For future maintenance, might be easier to update the partial and then reference the partial in the other places.
{{<render file="_dns-zero-downtime-failover.md" productFolder="Fundamentals">}}


feat(remix-dev): add suppport for .mjs and .cjs configs

Allows remix.config.js to be written as ESM or CJS following package.json type field conventions for resolution.
I.e:
If package.json "type": "module"

.js loaded as ESM
.mjs loaded as ESM
.cjs loaded as CJS

If package.json is not "type": "module"

.js loaded as CJS
.mjs loaded as ESM
.cjs loaded as CJS

Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:
Existing functionality covered by existing tests.

Something weird going on with the tests?

Looks like we maybe should include a test or two for this and fix the tests that are broken in this PR

For anybody wondering, I was on remix 1.6.0 and came here after upgrading to 1.7.4 because I was getting this error message:
Error [ERR_REQUIRE_ESM]: require() of ES Module … not supported.
index.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in /Users/jimnielsen/Sites/jim-nielsen.com/readlists/remix/package.json to treat all .js files as CommonJS (using .mjs for all ES modules instead).

I had type: "module" in my parent package.json but my remix.config.js file was a common JS file (it had module.exports = {}). Everything was working in my 1.6.0 setup (which was based off the template generator for a remix project) but it stopped working somewhere jumping to 1.7.4
I had two choices, both of which worked:

Rename remix.config.js to remix.config.cjs
Change my config from module.exports = {…} to export default {…}

Personally, I went with no. 2 and my project started to work again.


[ENH] Keeping a history of every operation applied to a DataFrame

A whacky idea dreamed up during the sprint.
This would essentially record for you your data processing pipeline (on the DataFrame bit at least). Metadata could go in a .pj accessor in the dataframe / series.
Fun challenges include:

Detecting use of all Pandas functions, not just PyJanitor ones
Minimizing modification of Pandas objects as much as possible
Avoiding brittleness to pandas code updates
Handling multi-dataframe operations without losing computation metadata

Thoughts on how this could be possible would be nice. Dream big.
@szuckerman @ericmjl @HectorM14

This is not exactly the same thing, but since you’re already thinking about a scope outside of just pyjanitor added methods, it would be really cool to see something like tidylog for Pandas.

@jcmkk3 tidylog is a very good idea! A hacky first-pass would be have every pyjanitor function print out what exactly it’s doing. This, of course, ignores all pandas functions. The more sophisticated version would use the logging facilities in Python. pyjanitor functions would need to be changed to explicitly report what exactly it’s doing in plain English; with an additional standard kwarg, we could allow a user to provide a reason for doing something, passed in as a string to the function call. (It may look clunky, though.)
Anyways, just some ideas to riff off @jcmkk3’s idea.

Apparently you can put a trace function in python that will log every function that's called. I saw this answer on stackoverflow which gave some decent code to show which functions are being called.
However, one of the main problems is how many intermediate functions are getting called. I modified the code from the stackoverflow answer to write all the results to a list.
import sys
import os

log = []

def tracefunc(frame, event, arg, indent=[0]):
      if event == "call":
          log.append(frame.f_code.co_name)
      return tracefunc

# sys.settrace(tracefunc)
sys.setprofile(tracefunc)

def main():
	print(os.getcwd())

main()
log

If I run the above, the output is:
['<module>', '<module>', '<module>', 'main', '<module>']

Now if I just merely import the pandas module:
import pandas as pd
len(log)
>>> 141765

And if I make a DataFrame:
dat = pd.DataFrame({'name': ['first1', 'first2'], 'last': ['last1', 'last2']})
len(log)
>>> 142220

Just creating a DataFrame was 455 function calls.
And just calling a DataFrame adds nearly 10k function calls.
dat
len(log)
>>> 153782

However, if you log[-10:], the output is:
['justify', 'justify', '<listcomp>', '_join_unicode', '_join_unicode', '_join_unicode', '_join_unicode', 'should_show_dimensions', '<module>', '<module>']

I'm thinking that if in the tracefunc function there was a condition to check for the set of all possible DataFrame and PyJanitor methods it would cut the list down to only what we want. Also, one thing to think about is how the DataFrame methods within PyJanitor functions will be logged, since they're getting called as well.

Interesting idea!
Would the goal be to first log operations for a dataframe, and be able reload those operations, or generate a .py with the same workflow? Or as @ericmjl says, a report in plain English. And should we be able to account for the possibility of multiple dataframes being processed, instead of recording all pyjanitor and pandas functions?
An alternative to some type of logging could be a decorator that gets applied to all pandas/pyjanitor methods. I haven't tried it out, but this Stack Overflow post (Defining Python decorators for a complete module) has some example code:
import othermodule
import types
for k,v in vars(othermodule).items():
    if isinstance(v, types.FunctionType):
        vars(othermodule)[k] = my_decorator(v)

That code (and more) could be put into a method like janitor.record(), invoked at the top of a script or notebook.
It may be helpful to see some examples of pandas/pyjanitor operations, and the resulting records, regardless of how we get there.

@hectormz I would vote for @ericmjl's suggestion about having the report in English. Reloading the operations would be nice, but users already have the code in a notebook or .py file; I'm not sure as to how helpful a log in that case would be.
I would vote for the following: Have a log like a web-style log. It's basically a DataFrame with columns like:
datetime, function_call, meaning, current_shape_of_df, time_to_execute
and maybe also columns for system settings (operating system, python version, etc).
What makes this difficult, though, is that to make it useful a config file would probably be necessary.
Some initial questions that come to mind that could be solved by a config would be stuff like:

Where is this being output? a DataFrame? a file'? a sqlite` db?
How many records to store before removing values? Maybe removing after a certain amount of time? A new python session?
How verbose should the logging be? Only PyJanitor? Also Pandas? Also base Python functions?

This all sounds like interesting stuff that users would appreciate, but I'm wondering if it's too big a task for this package. Maybe make this its own package that works with PyJanitor?

Now I understand, got it. Beyond the implementation of the history logging, one of the big questions to me will be where to store the history (and for how long), if a config file is not set.
In my view, the big picture items are:

Actual implementation of pandas/pyjanitor logging
Creating dictionary of methods and plain English equivalents (manually or automatically)

Maybe creating a small tool to check codebases for methods without translations as the pyjanitor/pandas methods change

Formatting and saving output/managing file size

It seems like a majority of effort is packed into 1., but does that cover the basic considerations?
tidylog looks very nice @jcmkk3 !

Friends, I think @eyaltrabelsi, a contributor to pyjanitor, has done some wonderful work with pandas-log related to this issue! The programming  model is very good: context manager so that we can selectively do logging at the appropriate places.

I am going to close this issue for now, but please feel free to re-open later!


New version: Google.AndroidGPUInspector version 3.3.1

Pull request has been created with komac v2.4.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @TejasMate,
This manifest adds Apps and Features entries that aren't present in previous PR versions. These entries should be added to the previous versions, or removed from this version.
(Automated response - build 887.)


Fix Flags trait

In bitflags 2.3, the BitFlags trait was rejigged so its associated type moved to a supertrait. That broke the specific way we were using it in this library. Since the trait is deprecated anyways this just moves it forwards to Flags, which is semver-compatible.

As reported over in https://github.com/bitflags/bitflags/issues/359


Tweaks for monthly mean calculation.

Modifications to the monthly mean calculation to speed up the processing on sun.
Main changes:

Use glob instead of rglob: Recursive glob can take a long time in a directory containing many .zarr files because each zarr file is itself a directory.
Sparse accumulation: Reindexing files to the all time steps of the day is slow and requires a lot of memory. Replaced it wich accumulation that uses only the corresponding time steps.
Data types: Use single precision for accumulation, which I think should be sufficient. However, probably better to make this a flag to allow checking its impact.
Add option to parallelize processing of multiple months.

The changes took the processing time for a single month down from about 20 to 10 minutes on sun. Moreover, the reduced memory requirements allow running about 8 processes in parallel.

I compared the means calculated using single and double precision (see plot below) and it does not make a significant difference. So it seems safe to run the calculations using single precision.

What does the plot show? In any case, I also think single precision should be OK, but recently (working with another problem) using single and double precision when computing means from a large array did make a difference. So if we can allow it, I'd compute the means with double precision.


fire_torpedo_all_succes test failure

The test for fire_torpedo_all_succes ran into problems, expected output was true, but found false.

fixed, added function


Llm graph transformer library

@rakshita-arora can we fix this with the separate folder so we can review and merge it in?

@rakshita-arora can we fix this with the separate folder so we can review and merge it in?

Sure @jexp , I am working on it

Got Feedback from Tomaz on the LLM_graph_transformer library.  Will incorporate those changes and raise another Pull request


infinite weighted quantile loss

Description
When the true-target values are zeros, our weighted quantile loss become inf.
This is especially a problem for HPO jobs, where we rely on the mean_wQuantileLoss as our objective metric to access the models performance.
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.1]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.2]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.3]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.4]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.5]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.6]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.7]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.8]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, wQuantileLoss[0.9]): inf
[2020-02-08 12:04:19 +0100] [59599] [INFO] #test_score (local, mean_wQuantileLoss): inf

To Reproduce
The test time-series needs to contain zeros in the prediction-range. The following example triggers the problem until a prediction_length of 4.
{"start": "2020", "target": [2, 1, 0, 0, 0, 0]}

Here is the code triggering it:
https://github.com/awslabs/gluon-ts/blob/90587baff19a764aed02ffab752f70cf927f50e5/src/gluonts/evaluation/_base.py#L321-L323

Looking at tensorflow, they seem to assume a minimum value by which they divide:
diff = math_ops.abs(
      (y_true - y_pred) / K.clip(math_ops.abs(y_true), K.epsilon(), None))

According to this, epsilon is 1e-07.

This only happens when all true values are 0 (or really close to it). I would propose to just return the unscaled value in that case.

I think “inf” is actually the correct and more useful output. Would it maybe be an option to check beforehand whether the test set is only zeros and use a different metric in that case?

Another option would be to put the logic you propose in your script and apply it before logging metrics.

No fix.
We also emit the unscaled version, which can be used for these cases.


How to animate RecyclerView items in Android

Proposed title of article
How to animate RecyclerView items in Android
Introduction paragraph (2-3 paragraphs):
It is a good practice in Android to show data to the user in a presentable way rather than displaying everything at the same time on the User Interface. This article will help you to understand how you can present your list of data in an animated manner that will improve user experience of your application.
Animations are one of the ways through which you can visualize updates in the list in case data changes from the data source. In this article we will learn 3 types of animations namely:

left to right animation
right to left animation
Scaled animation

Key takeaways:
By the end of this tutorial, the reader will be able to:

Understand what is animation.
Create  XML animation files.
Integrate animations in RecyclerView.

References:
N/A

Thank you for the topic suggestion @paulnjoroge789016, however it seems that this topic has already been sufficiently covered across some previously published articles - https://www.section.io/engineering-education/search/?q=recyclerview. Please select a different topic area to explore. Thank you.

Hi @ninjaginja, This is a completely different and an unique area that has not been tackled in the previously published articles. I have gone through all articles related to RecyclerView and there are no conflicts. Thanks.


this is a very clever idea !

@aktxyz Thanks, glad you like it.
Anything in particular you would use it for? Still very much looking for feedback on how to apply this concept.

I have not tried it out yet.  It does look like it would be perfect for the various source generators I am using ... but these are working fine now so don't want to spend the time redoing them.  Next time.

Yeah, no worries about trying it now. At the moment, it crashes too much to be worth having anyone test the demo.


Missing zoneinfo.zip  when including a timezone parameter into a job config

I've set up dkron via docker-compose and applied the official image
But when I tried the example from the getting start page, the error occurred.
first job config
$ curl localhost:8080/v1/jobs -XPOST -d '{
  "name": "job1",
  "schedule": "@every 10s",
  "timezone": "Europe/Berlin",
  "owner": "Platform Team",
  "owner_email"<EMAIL_ADDRESS>  "disabled": false,
  "tags": {
    "dkron_server": "true:1"
  },
  "concurrency": "allow",
  "executor": "shell",
  "executor_config": {
    "command": "date"
  }
}'
$

Error from dkron logs
dkron_1   | time="2019-02-22T04:05:08Z" level=info msg="Error #01: open /usr/local/go/lib/time/zoneinfo.zip: no such file or directory" node=4164f29ffdb9

I removed timezone config and retried and it worked
$ curl localhost:8080/v1/jobs -XPOST -d '{
  "name": "job1",
  "schedule": "@every 10s",
  "owner": "Platform Team",
  "owner_email"<EMAIL_ADDRESS>  "disabled": false,
  "tags": {
    "dkron_server": "true:1"
  },
  "concurrency": "allow",
  "executor": "shell",
  "executor_config": {
    "command": "date"
  }
}'
{"name":"job1","timezone":"","schedule":"@every 10s","owner":"Platform Team","owner_email":"platform@example.com","success_count":0,"error_count":0,"last_success":"0001-01-01T00:00:00Z","last_error":"0001-01-01T00:00:00Z","disabled":false,"tags":{"dkron_server":"true:1"},"retries":0,"dependent_jobs":null,"parent_job":"","processors":null,"concurrency":"allow","executor":"shell","executor_config":{"command":"date"},"status":""}
$

Could you include zoneinfo into this official image??
Ref:docker-conpose.xml
---
version: '2'
services:
  consul:
    image: consul
    ports:
      - "8500:8500"
      - "8300"
    hostname: node1
    command: agent -server -bootstrap -client=<IP_ADDRESS>
  dkron:
    image: dkron/dkron
    depends_on:
      - consul
    ports:
      - "8080:8080"
      - "8946:8946"
    volumes:
     - ./:/gopath/src/github.com/victorcoder/dkron
    environment:
      - GODEBUG=netdns=go
    command: agent --server --backend=consul --backend-machine=consul:8500 --join=dkron:8946 --log-level=debug

Thanks @bomu-bomu good catch, probably a bug in the build environment.


feat: same filter is now retrieved from the url

close #810

bad idea, an object can't have the same key twice for obvious reasons


EWC-4-KS-ParentComp-Draw | Added Draw Component

In this PR Draw component is working fine, issue related to the HTML inside container pertains leading to an unwanted design in the UI.

Not the right PR. Hence Closing. The PR for the Draw component is: https://github.com/sencha/ext-components/pull/67


Consider downloading config from other's repo to make it more shareable?

Thank you for the extension, it's really useful. But as you say, the snippet is opinionated. If the variable URL can be configured in VS Code configurations, users can then update from any favorite repo. I think it will be more shareable. 😀

PR welcome

So we may need to move the content to a real json file, and after that we could link with a configurable URL to any gist :eyes:
That could really be useful, could it?
We could even create some presets :thinking:

Actually, the ideal method in my mind is we move the generation logic directly to the extension, and provide a more human-friendly API to configure those entries so ppl could customize the generation as they want.
That could use quite some effort, considering the current solution works well, and VS Code might improve the config interface in the future, I personally do not have enough motivation to push it right now. So PR welcome again (or take this idea and create a new project).


Task exception was never retrieved Error

Please tick all applicable boxes.

[x ] I am using Python 3.5.3 or higher (run python --version on the command line)
[x ] I have followed the official guides to install the bot for my system
[x ] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[ ] The latest master version (release-260419)

[ ] The latest review version

[x] It's running release 260819-5-gc87dc05

What type of issue are you creating?

[ x] Bug
[ ] Feature request
[ ] Question

Description of issue
Bot launches and successfully joins the voice channel but doesn't respond when asked to play a song.
Gives the following error in the console:
Task exception was never retrieved
future: <Task finished coro=<VoiceClient._create_socket() done, defined at C:\Users\Admin_Anthoros\AppData\Roaming\Python\Python36\site-packages\discord\voice_client.py:172> exception=gaierror(11001, 'getaddrinfo failed')>
Traceback (most recent call last):
File "C:\Users\Admin_Anthoros\AppData\Roaming\Python\Python36\site-packages\discord\voice_client.py", line 191, in _create_socket
self.endpoint_ip = socket.gethostbyname(self.endpoint)
socket.gaierror: [Errno 11001] getaddrinfo failed
Before this error I followed another forum when having and issue in which my bot wouldn't launch at all. This forum solution asked to change 'requirements.txt' with this:
pynacl==1.4.0
discord.py[voice]==1.3.4
pip
youtube_dl
colorlog
The bot is now able to launch but is unresponsive to music commands and only listens to extraneous commands such as, 'clean' 'clear' 'shutdown' etc.
Steps to reproduce

run 'Run.bat'

Log file
musicbot.log
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.

Try this:
py -3.7 -m pip install websockets

It says:
Installed Pythons found by C:\WINDOWS\py.exe Launcher for Windows *
Requested Python version (3.7) not installed, use -0 for available pythons
Python 3.7 not found!
-3.6-64

my fault, I wrote it wrong
Try:
py -3.6 -m pip install websockets

No worries. Tried it and it says
Requirement already satisfied: websockets in c:\users\admin_anthoros\appdata\roa    ming\python\python36\site-packages (6.0)

Hi, my bot did always fully work. But I'm getting a TimeoutError too today. It's a bit different to yours but probably something changed/broke in the discord handshake.

I do remember that discord was in the process of changing a lot for bots and such. Might have to just wait this out but I'm willing to try solutions.

In requirements.txt change discord.py version to discord.py[voice]==1.4.1
Update dependencies after. That fixed it for me :)

Worked like a charm. Thank you so much!


Guide on how to use it In Windows please?

I would like tro try it but I don't have "make" in windows, can I try it anyway?
Thanks

Hey there. If you just want to try the app, you can go here: https://alexcrist.github.io/autotone/


test: add new test to check for stable version support

Description
Issue : https://github.ibm.com/GoldenEye/issues/issues/6660

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

Will merge https://github.com/terraform-ibm-modules/terraform-ibm-icd-mysql/pull/25 first and then rebase this PR and merge

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Enabling Dependency Bot notifications

The 'Dependency Bot' section doesn't provide enough details on what needs to be done to enable the notifications, and whether the notifications can be managed by users through their preferences.
Please add that detail.
For the record, I have not received any notifications to date for the scenarios handled by the Dependency Bot - hence I'm not sure its working properly/enabled.

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

ID: a1714e91-58b6-9911-0da6-7555d09eccb3
Version Independent ID: 2a317fb7-67cb-a355-8fe2-8cd0e147acdc
Content: Plan and track dependencies across teams and organizations using the Dependency Tracker extension - Azure DevOps
Content Source: docs/boards/extensions/dependency-tracker.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

I don't think the dependency bot is installed with this extension, or at least I've never seen it work? Not sure if this should be struck from the documentation or if anything needs to be added from dev team to make this work.

@sonal-b, @soccerjoshj07 - Thank you for your feedback. I'm commenting out the information on the Dependency Bot for now. This extension isn't owned by a product team, so any issues should be forwarded to the extension website.  I will revisit this if/when updates are made to the extension.


Add changelog for 2.0.0-beta1

I can push the tags and promote the release on sonatype once this gets merged in.

Turns out I don't have permissions to push to Sonatype! I was at step 4 on https://github.com/square/tape/blob/master/RELEASING.md, so to release this someone with the right permissions would have to run this to pick up from where I left off:

Merge the PR to the master branch.
git pull from the master branch.
3 ./gradlew clean uploadArchives.
Visit Sonatype Nexus and promote the artifact.
git tag -a X.Y.X -m "Version 2.0.0-beta1"
Update the gradle.properties to the next SNAPSHOT version.
git commit -am "Prepare next development version."
git push && git push --tags

I'll do it tomorrow


😬 Actually send the request body

Fixes #113
(whoops)

Tested with GET and POST

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: efa9bb00-cb49-11e9-81b8-6fab94e294e1


Extend StreamExt with state type parameter

@sfackler what do you think of something like this? I wanted this so I can use futures::stream::iter() but for a StateStream; this seemed like a neater solution than copying iter().

Sounds reasonable to me!

Good call, amended to include that change. (I seem to have trouble keeping my rustfmt install running, so it might not be formatted according to your standards -- sorry.)

Thanks!


Rejigger the config

Split into own file for organisational purposes, and adds options depending on the system so that we don't have any "only works on windows" options.

Funny, I had the platform specific stuff implemented and not pushed yet ;) So, yeah, LGTM!
Btw, what does the notification option do?

If you're waiting for a long running task (evaluation, booting the client etc.) and switch to another window, that option just puts up a notification so that you can quickly switch back when it's done. It's quite nice but doesn't appear to work on windows for some reason.

That explains why I never noticed any difference between having it on and off... I'll try to remember looking into it sometime soonish.

API reference. Notifications don't show up for me at all, and it sounds like they aren't easy to support on Windows versions below 10, so maybe we can switch to using the tray API.


Convert DropdownListWidget, MultiSelectDropdownListWidget to ES6 classes

I did some basic testing of DropdownListWidget instances in organization settings. We currently have no MultiSelectDropdownListWidget instances to test outside of the Node tests.
git show -w (--ignore-all-space) recommended for reviewing.
Cc @aryanshridhar (#18954)

I think merging this may effectively entail giving up on integrating https://github.com/zulip/zulip/pull/19591. It's possible that's already going to be pretty hard.

Rebasing #19591 onto current main was tedious but straightforward:
https://github.com/zulip/zulip/compare/main...andersk:rebased-19591
and from there, rebasing onto this was easy:
https://github.com/zulip/zulip/compare/main...andersk:rebased-19591-on-21935
(I haven’t attempted to verify anything about these branches beyond linters and Node tests.)

OK great -- did some additional testing myself and merged. It'd be great if you can push that update to the #19591 branch.

(I thought this was messier since I hadn't pulled it down to do the git show -w yet)


Add support for type parameters to anonymous class expressions

This is done by unifying the parsing logic for class expressions and
class declarations, since anonymous class declarations already suppoted
type arguments.
Fixes https://github.com/facebook/flow/issues/3567

Hi, this is my first PR :slightly_smiling_face:
I was hit by this bug while working on a personal project and I tought that it was a great opportunity to learn something new.
I have some problems running the tests locally: if I run make && make js and then make test in the src/parser folder, for some reason it doesn't consider my changes to the code. I double-checked by deleting whole sections of the parser files both nothing changed :confused:.
I tested my changes using echo "(class <T> {});" | bin/flow ast | prettier, so I'm not sure yet that this PR didn't break anything. I'll wait for the CI result.

/сс @gabelevi or @mroch

Oh, this is cool @dsainati1


Stable Diffusion 3

Hi, are there any plans for adapting the code for Stable Diffusion 3?

Thanks for opening this issue!
I'm going to keep this open as a feature request and update the thread if I get around to it; it's been on my todo list for some time but I haven't had a chance. If other folks are also interested in this feature please like / comment on this issue so I can gauge interest.


feat: update api

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add test file for GitHub module

Note

Test all the methods of the github module and add a test file for the same.
Add test file in directory src/test.
Check the codechef_test.py file as a reference to get an idea about structuring the code.
Use unittest only.

I would love to work on the issue

Go ahead @SWAYAMKHANDURI
Note

Create a separate file for this test, named github_test.py file.
Make different classes for different classes existing for github module.

All the best 👨💻

alright thanks :relaxed:

im new to testing
so i would love to see ur work @SWAYAMKHANDURI
waiting for ur PR👍


package updates

Codecov Report

Merging #129 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #129   +/-   ##
=======================================
  Coverage   67.97%   67.97%
=======================================
  Files          15       15
  Lines         818      818
=======================================
  Hits          556      556
  Misses        262      262

Impacted Files
Coverage Δ

bowtie/control.py
62.26% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Got an error while trying to build a docker image (bin/build: not found)

➜  metabase_duckdb_driver git:(main) ./build_image.sh
Cloning into 'docker/metabase/source'...
remote: Enumerating objects: 353471, done.
remote: Counting objects: 100% (2540/2540), done.
remote: Compressing objects: 100% (1348/1348), done.
remote: Total 353471 (delta 1364), reused 1997 (delta 963), pack-reused 350931
Receiving objects: 100% (353471/353471), 321.25 MiB | 2.57 MiB/s, done.
Resolving deltas: 100% (231675/231675), done.
Sending build context to Docker daemon  464.1MB
Step 1/12 : FROM metabase/ci:java-11-clj-<IP_ADDRESS>0.04-2022-build as builder
java-11-clj-<IP_ADDRESS>0.04-2022-build: Pulling from metabase/ci
e0b25ef51634: Pull complete
d1bd2bc15eb1: Pull complete
77d0de1fd7e0: Pull complete
b638505435dc: Pull complete
1d8dbe37838f: Pull complete
Digest: sha256:8881f81f34088fc2b1c0f1ce1b0356af600086c6494ef8c883ba7a6c099a158f
Status: Downloaded newer image for metabase/ci:java-11-clj-<IP_ADDRESS>0.04-2022-build
 ---> 918a5d466f5c
Step 2/12 : ARG MB_EDITION=oss
 ---> Running in 828160f4606a
Removing intermediate container 828160f4606a
 ---> 851d9f1a0819
Step 3/12 : WORKDIR /home/circleci
 ---> Running in 48204a92d230
Removing intermediate container 48204a92d230
 ---> 1861b928c007
Step 4/12 : COPY --chown=circleci . .
 ---> 074f69699758
Step 5/12 : RUN INTERACTIVE=false CI=true MB_EDITION=$MB_EDITION bin/build
 ---> Running in ee62ff90b6af
/bin/sh: 1: bin/build: not found
The command '/bin/sh -c INTERACTIVE=false CI=true MB_EDITION=$MB_EDITION bin/build' returned a non-zero code: 127
Sending build context to Docker daemon  464.1MB
Step 1/3 : FROM ubuntu_metabase
pull access denied for ubuntu_metabase, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Here is the error:
/bin/sh: 1: bin/build: not found
The command '/bin/sh -c INTERACTIVE=false CI=true MB_EDITION=$MB_EDITION bin/build' returned a non-zero code: 127

Hi @AlexR2D2!
Thank you for the update, but it doesn't work for me.
I got another error:
➜  metabase_duckdb_driver git:(main) ./build_docker_image.sh
Cloning into 'docker/metabase/source/v1.44.6'...
remote: Enumerating objects: 7631, done.
remote: Counting objects: 100% (7631/7631), done.
remote: Compressing objects: 100% (6646/6646), done.
remote: Total 7631 (delta 465), reused 3784 (delta 320), pack-reused 0
Receiving objects: 100% (7631/7631), 53.05 MiB | 1.27 MiB/s, done.
Resolving deltas: 100% (465/465), done.
Note: switching to '5c6ae211191d06e92bf1435afc9bad5473d7840a'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /var/lib/snapd/void/Dockerfile: no such file or directory
unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /var/lib/snapd/void/docker: no such file or directory

Some strange error. Maybe this can helps

This error https://github.com/AlexR2D2/metabase_duckdb_driver/issues/5#issuecomment-1325193856 appeared because I cloned the repository to the /tmp folder
But in my home directory the script started and now it in progress...

Got an error
      Metabase v0.44.7-SNAPSHOT (5c6ae21 ?)

      Copyright © 2022 Metabase, Inc.

      Metabase Enterprise Edition extensions are PRESENT.

      Usage of Metabase Enterprise Edition features are subject to the Metabase Commercial License. See https://www.metabase.com/license/commercial/ for details.
          Finished compilation in 32.8 seconds.
        Copy resources
          Copy src
          Copy shared/src
          Copy resources
        Create uberjar
      Building uber jar: /home/circleci/target/uberjar/metabase.jar
          Created uberjar in 17.5 seconds.
        Update META-INF/MANIFEST.MF
        Built target/uberjar/metabase.jar in 54.3 seconds.
    Uberjar built successfully.
  All build steps finished.
Removing intermediate container a549b5e683f9
 ---> 37b1382a623c
Step 6/12 : FROM --platform=linux/amd64 eclipse-temurin:11-jre as runner
 ---> 896000291b27
Step 7/12 : ENV FC_LANG en-US LC_CTYPE en_US.UTF-8
 ---> Using cache
 ---> 7d4df1e93157
Step 8/12 : RUN apt-get update &&     apt-get install -y bash fonts-dejavu curl ca-certificates-java &&     apt-get upgrade -y &&     rm -rf /var/cache/apt-get/* &&     mkdir -p /app/certs &&     curl https://s3.amazonaws.com/rds-downloads/rds-combined-ca-bundle.pem -o /app/certs/rds-combined-ca-bundle.pem  &&     /opt/java/openjdk/bin/keytool -noprompt -import -trustcacerts -alias aws-rds -file /app/certs/rds-combined-ca-bundle.pem -keystore /etc/ssl/certs/java/cacerts -keypass changeit -storepass changeit &&     curl https://cacerts.digicert.com/DigiCertGlobalRootG2.crt.pem -o /app/certs/DigiCertGlobalRootG2.crt.pem  &&     /opt/java/openjdk/bin/keytool -noprompt -import -trustcacerts -alias azure-cert -file /app/certs/DigiCertGlobalRootG2.crt.pem -keystore /etc/ssl/certs/java/cacerts -keypass changeit -storepass changeit &&     mkdir -p /plugins && chmod a+rwx /plugins
 ---> Using cache
 ---> 27d78738719b
Step 9/12 : COPY --from=builder /home/circleci/target/uberjar/metabase.jar /app/
 ---> 03376aa6b35c
Step 10/12 : COPY bin/docker/run_metabase.sh /app/
 ---> e31e665e4c4b
Step 11/12 : EXPOSE 3000
 ---> Running in 0c5d3752f438
Removing intermediate container 0c5d3752f438
 ---> e70c9bc55b74
Step 12/12 : ENTRYPOINT ["/app/run_metabase.sh"]
 ---> Running in 2e7fe790162b
Removing intermediate container 2e7fe790162b
 ---> a6e1a318366b
Successfully built a6e1a318366b
Successfully tagged ubuntu_metabase:latest
Sending build context to Docker daemon  167.8MB
Step 1/3 : FROM ubuntu_metabase
 ---> a6e1a318366b
Step 2/3 : COPY target/duckdb.metabase-driver.jar /plugins/
COPY failed: file not found in build context or excluded by .dockerignore: stat target/duckdb.metabase-driver.jar: file does not exist

@AlexR2D2 I keep getting an error when trying to build the docker image using the provided script (build_docker_image.sh).
The command '/bin/sh -c INTERACTIVE=false CI=true MB_EDITION=$MB_EDITION bin/build' returned a non-zero code: 255
unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /Users/andreasgustafsson/repos/metabase_duckdb_driver/docker/metabase/source/v1.44.6/docker: no such file or directory

Any ideas what might be wrong?

I have the same issue as @pyoner when running build_docker_image.sh
Step 2/3 : COPY target/duckdb.metabase-driver.jar /plugins/ COPY failed: file not found in build context or excluded by .dockerignore: stat target/duckdb.metabase-driver.jar: file does not exist

Hi, @pyoner  @zufanka, could you update the repository and try again? I fixed the script a bit :)
apps $ git clone git@github.com:AlexR2D2/metabase_duckdb_driver.git
apps $ cd metabase_duckdb_driver
apps/metabase_duckdb_driver $ ./build_docker_image.sh

... wait for the image to build

@AlexR2D2 I keep getting an error when trying to build the docker image using the provided script (build_docker_image.sh).
The command '/bin/sh -c INTERACTIVE=false CI=true MB_EDITION=$MB_EDITION bin/build' returned a non-zero code: 255
unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /Users/andreasgustafsson/repos/metabase_duckdb_driver/docker/metabase/source/v1.44.6/docker: no such file or directory

Any ideas what might be wrong?

it seems the Metabase the repository has not been cloned.
Could you try to clone Metabase source of v1.44.6 version manually into metabase_duckdb_driver /docker/metabase/source/v1.44.6 folder and try to run script again?


Standarize on emacs 29 with tree-sitter

Also removes no longer needed packages and configurations.

Given that I've mentioned this a few places and asked for refinements or feedback and nothing has come up and that I've been using this happily for more than a week, I think I'll go ahead and merge this.


Make the logo bigger

I always wanted to say that. Logo on maps is too small IMO.

Here a test between 2 sizes (Top: current size , Bottom: new size)

cc/ @saleiva

Cooleo.

Assigning to @alonsogarciapablo, he might need the asset

New SVG logo coming in https://github.com/CartoDB/cartodb/pull/8716. But already updated in CartoDB.js, just in case.


Checkbox InputType not working correctly in generic

Bug Report
System Information

Ubuntu 20.04.3
AMP <IP_ADDRESS>
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do?

Add checkboxes to a generic template via config.json, like this:
    {
        "DisplayName": "Enable Voice Chat",
        "Category": "Server Settings",
        "Description": "If enabled, players will be able to use in-game voice chat",
        "Keywords": "voice,chat",
        "FieldName": "DisableVoN",
        "InputType": "checkbox",
        "IsFlagArgument": false,
        "ParamFieldName": "disableVoN",
        "IncludeInCommandLine": false,
        "DefaultValue": "0",
        "EnumValues": {
            “0”: "False",
            “1”: "True"
        }
    },

And then have the value of 0 or 1 written to a file via metaconfig.json, according to whether the checkbox is off (False) or on (True).

What are you expecting to happen?

That when the checkbox is toggled the value of 0 or 1 is written to the file as applicable.

What is actually happening? ('Nothing' is not an acceptable answer!)

That instead of 0 or 1, the text False or True is written instead.
Reproduction

Create a generic instance template with a checkbox InputType in config.json, with the relevant variable being written to a file via metaconfig.json (or even just added to startup parameters via the main kvp).

It works fine for me, can you share your relevant metaconfig too?

The relevant line is: "disableVoN": "DisableVoN",.
Can you share yours so I can compare?

Nvm, found your repo. I can’t see any relevant difference.

actually, you are right, it's printing out True/False for me too, even if i change the right hand side to be 0, 1

Yes, I have tried multiple ways.

Fixed in latest build.


Runtime Cache Integration

Pull Request Description

related #681
Changelog:

add: CachePreferenceAnalysis that sets weight 1 to the right-hand-side expressions
update: Enable caching in the IdExecutionInstrument
update: Extend InvalidateCache command. All cache invalidation is performed with InvalidateCache#run methods before the program execution

Important Notes

To complete the integration task, the Changeset algorithm should be updated to support batch edits.
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the Scala and Java style guides.
[x] All code has been tested where possible.

Awesome stuff in general. Only problem is, I don't think we can just cache the enterables like this – it's unbound memory again.

Though we definitely must cache the currently-entered enterable so there's some thinking to do :)


[SPARK-47933][PYTHON][TESTS][FOLLOW-UP] Enable doctest pyspark.sql.connect.column

What changes were proposed in this pull request?
Enable doctest pyspark.sql.connect.column
Why are the changes needed?
test coverage
Does this PR introduce any user-facing change?
no, test only
How was this patch tested?
manually check:
I manually broke some doctests in Column, then found pyspark.sql.connect.column didn't fail:
(spark_dev_312) ➜  spark git:(master) ✗ python/run-tests -k --python-executables python3 --testnames 'pyspark.sql.classic.column'
Running PySpark tests. Output is in /Users/ruifeng.zheng/Dev/spark/python/unit-tests.log
Will test against the following Python executables: ['python3']
Will test the following Python tests: ['pyspark.sql.classic.column']
python3 python_implementation is CPython
python3 version is: Python 3.12.2
Starting test(python3): pyspark.sql.classic.column (temp output: /Users/ruifeng.zheng/Dev/spark/python/target/4bdd14b8-92ba-43ba-a7fb-655e6769aeb9/python3__pyspark.sql.classic.column__i2_c1zct.log)
WARNING: Using incubator modules: jdk.incubator.vector
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
**********************************************************************
File "/Users/ruifeng.zheng/Dev/spark/python/pyspark/sql/column.py", line 385, in pyspark.sql.column.Column.contains
Failed example:
    df.filter(df.name.contains('o')).collect()
Differences (ndiff with -expected +actual):
    - [Row(age=5, name='Bobx')]
    ?                      -
    + [Row(age=5, name='Bob')]
**********************************************************************
   1 of   2 in pyspark.sql.column.Column.contains
***Test Failed*** 1 failures.

Had test failures in pyspark.sql.classic.column with python3; see logs.
(spark_dev_312) ➜  spark git:(master) ✗ python/run-tests -k --python-executables python3 --testnames 'pyspark.sql.connect.column'
Running PySpark tests. Output is in /Users/ruifeng.zheng/Dev/spark/python/unit-tests.log
Will test against the following Python executables: ['python3']
Will test the following Python tests: ['pyspark.sql.connect.column']
python3 python_implementation is CPython
python3 version is: Python 3.12.2
Starting test(python3): pyspark.sql.connect.column (temp output: /Users/ruifeng.zheng/Dev/spark/python/target/2acaff3c-ef1d-41eb-b63e-509f3e0192c0/python3__pyspark.sql.connect.column__66td62h9.log)
Finished test(python3): pyspark.sql.connect.column (3s)
Tests passed in 3 seconds

after this PR, it fails as expected:
(spark_dev_312) ➜  spark git:(master) ✗ python/run-tests -k --python-executables python3 --testnames 'pyspark.sql.connect.column'
Running PySpark tests. Output is in /Users/ruifeng.zheng/Dev/spark/python/unit-tests.log
Will test against the following Python executables: ['python3']
Will test the following Python tests: ['pyspark.sql.connect.column']
python3 python_implementation is CPython
python3 version is: Python 3.12.2
Starting test(python3): pyspark.sql.connect.column (temp output: /Users/ruifeng.zheng/Dev/spark/python/target/390ff7ae-7683-425c-b0d2-ee336e1ad452/python3__pyspark.sql.connect.column__f69b3smc.log)
WARNING: Using incubator modules: jdk.incubator.vector
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
org.apache.spark.SparkSQLException: [INVALID_CURSOR.DISCONNECTED] The cursor is invalid. The cursor has been disconnected by the server. SQLSTATE: HY109
	at org.apache.spark.sql.connect.execution.ExecuteGrpcResponseSender.execute(ExecuteGrpcResponseSender.scala:281)
	at org.apache.spark.sql.connect.execution.ExecuteGrpcResponseSender$$anon$1.run(ExecuteGrpcResponseSender.scala:101)
**********************************************************************
File "/Users/ruifeng.zheng/Dev/spark/python/pyspark/sql/column.py", line 385, in pyspark.sql.column.Column.contains
Failed example:
    df.filter(df.name.contains('o')).collect()
Expected:
    [Row(age=5, name='Bobx')]
Got:
    [Row(age=5, name='Bob')]
**********************************************************************
   1 of   2 in pyspark.sql.column.Column.contains
***Test Failed*** 1 failures.

Had test failures in pyspark.sql.connect.column with python3; see logs.

Was this patch authored or co-authored using generative AI tooling?
No

merged to master


Fix DebugSyncStream

This PR addresses the following issue

Marking as blocked pending #525.

Addressed this differently in the following PR


Revert "json support"

Reverts Landoop/stream-reactor#467

This change is

@stheppi can this be merged?


maven plugin goal gver:tag -and-set-version fails when no tag applies

The goal should just skip the tag step instead.

The goal actually works as intended, but the none  type probably should be renamed to fail


There is an extra outer border on a protected pdf in chat reported by @huzaifa-99

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

Action Performed:

Send a protected pdf in any chat (a protected pdf sample is attached: Pass Ccccccc11)
Notice it has an extra outer border in the chat

Expected Result:
The outer border should not exist (or at-least be rounded the same as the inner border)
Actual Result:
There is an extra outer border with a different border radius (Visible after zoom in the page)
Workaround:
unknown
Platform:

Where is this issue occurring?

Web

Version Number: 1.2.26-0
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/201203546-3e235951-a71a-45bb-bff1-79d62bc5fa8e.mp4

sample-protected.pdf
Expensify/Expensify Issue URL:
Issue reported by: @huzaifa-99
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1668111547203699
View all open jobs on GitHub

Dupe https://github.com/Expensify/App/issues/11520

Dupe #11520


Blog post invalid link to demo website

The blog post (https://explosion.ai/blog/introduction-to-holmes) links to https://demo.holmes.prod.demos.explosion.services/ at the bottom of the post. That doesn't work; in Github link of https://holmes-demo.explosion.services/ is what should be used instead.

Many thanks for letting us know!


pass variable in $og->addImage( )

Hi, this package is vary helpful to me.
How to pass variable in $og->addImage( );
Eg. :
$image='https://site.com';
$og->addImage($image);  // Not working
but
->setTitle($site->title);  // is working

Hi!
After OgPackage initialization you should register it.
$og = new Butschster\Head\Packages\Entities\OpenGraphPackage('some_name');

$og->setType('website')
   ->setSiteName('My awesome site')
   ->setTitle('Post title');

// You can render itself

$og->toHtml();
// <meta name="og:type" content="website">
// <meta name="og:site_name" content="My awesome site">
// <meta name="og:title" content="Post title">

// Or just register this package in Meta class and it will be rendered automatically
Meta::registerPackage($og);

Thanks for reply.
I want to pass a variable in addImage( );
When I pass variable in set attribute , It works
$og->setType($site->type)     // working
->setSiteName($site->name)   // working
->setTitle($site->title);   // working
But when I pass variable in add attribute , It doesn't work
Eg.:
$og->addImage($site->image)   // not working

Thanks for reply.
I want to pass a variable in addImage( );
When I pass variable in set attribute , It works
$og->setType($site->type) // working
->setSiteName($site->name) // working
->setTitle($site->title); // working
But when I pass variable in add attribute , It doesn't work
Eg.:
$og->addImage($site->image) // not working

Please show me full code, it will be easier to understand your problem.
Try to render $og object locally.
$og->setType($site->type) // working
->setSiteName($site->name) // working
->setTitle($site->title)
->addImage($site->image) // not working

dd($og->toHtml());

This package has unit tests coverage and you can see examples right in this tests https://github.com/butschster/LaravelMetaTags/blob/master/tests/Packages/Entities/OpenGraphPackageTest.php#L50


Add context cancellation to backup routines

This should get the ball rolling on issue #110. Please review 😄
I did notice that the project doesn't have an overarching test strategy/implementation in place, hence the reason for the lack of test(s).
I wanted to get your opinion on how to add (or start) adding some test suites onto the project @nilslice. I was thinking of maybe using something like https://github.com/spf13/afero for mocking the uploads/ directory. WDYT?

@eticzon - thank you! this looks great. I will give it a spin and get back to you with any changes.
Re: testing.. yes, we're desperately short on tests. I knew I would be working on the internal API and the interfaces a bit, and thus didn't want to slow down dev to keep tests up to date, but it's gotten a little out of hand ;)
Frankly, testing is easily the domain I have the least amount of experience in and would be hugely grateful to anyone who could step up to establish some starting point to get things going in the right direction for Ponzu's test strategy. I need to refactor quite a bit of the HTTP handler code to be testable, and I'm sure there is a lot elsewhere to fix up. The APIs are mostly ready to be frozen for an upcoming v1.0, so now is definitely a good time to start writing tests.
I looked at afero and it looks useful -- not only for the uploads but the CLI too. Commands new, add, build, and run in the CLI make use of certain directories on disk that could also benefit from testing with that package. If you'd like to use it, please go ahead! I am currently putting all** dependencies into vendor, flattening and checking them in (but removing their .git directory first to not submodule).
** except Bleve search, since it is a very large codebase.

@nilslice cool! Thanks! Yup just @mention me if you need some changes done.
Re: tests, glad that you liked the afero idea.  If it's OK with you, I'll write up a summary on a issue with a rough plan on getting started with testing. It'll take a while, so by that time the handler code will be in an a more testable state like you said.

Re: tests, glad that you liked the afero idea. If it's OK with you, I'll write up a summary on a issue with a rough plan on getting started with testing.

That would be great, thank you :+1:

It'll take a while and by that time the handler code will be in an a more testable state like you said.

Perfect. Thanks again for the help.

👍  Glad I was able to help.


WebContainer gets corrupted on second install.

Describe the bug

Navigate: https://stackblitz.com/edit/qwik-starter-tlhhfa?file=src%2Froutes%2Findex.tsx
ctrl-C the server
run npm install && npm run dev Does not work..

Link to the blitz that caused the error
https://stackblitz.com/edit/qwik-starter-tlhhfa
Steps to reproduce

Navigate: https://stackblitz.com/edit/qwik-starter-tlhhfa?file=src%2Froutes%2Findex.tsx
ctrl-C the server
run npm install && npm run dev Does not work..

Expected behavior
it should restart the application
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Hi! :wave: Thanks for reporting this issue!
This appears to be a bug in webcontainer, we are tracking it internally and will update this issue when we have fixed it.
In the meantime, you can workaround this by manually adding @emnapi/runtime to your package.json

Hey @mhevery ! We have fixed the underlying issue with npm so this should no longer happen.


Fixing Host ID initialization (ensuring it is set when FastLogger is created)

Resolves #1393
This is a very tactical fix to address a production impacting issue (described in #1393). I'm working on a larger fix as this still doesn't address some configuration related issues, but wanted to make sure this was out.

Yes! @davidebbo suggested the same thing. I'll have an E2E test that invokes the logging APIs to retrieve invocation logs and make sure they're properly saved.


linuxHeaders: 6.5 -> 6.6

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Successfully built on x86_64-linux reverse dependencies: apparmor-bin-utils apparmor-parser apparmor-utils audacity audit autofs5 ceph ceph-dev dbus-broker duperemove envoy gfxtablet glibcInfo glibcLocales glibcLocalesUtf8 glibc_memusage input-utils jujuutils klibc libapparmor libceph librseq linuxHeaders linuxptp multipath-tools musl mythtv nbd oprofile sc-controller steamcontroller stratovirt systemd systemdLibs systemdMinimal tenacity trafficserver uclibc uclibc-ng udev.


GEOMESA-1021 Convert command

Added convert command and documentation

also looks like another rebase may be needed

Question: for bin file export what params do i need? The error should list which ones are required.

That's not clear to me. Looking at the code, only the dateAttribute is required but it still has a default value. It appears the other params are optional I haven't verified this.

@aheyne needs rebase

Merged master

@aheyne is this good for merging? I'll give it one last look over if so

@jahhulbert-ccri said he was going to try it out to make sure it's working and unless @jnh5y has an idea for handling the data changes (eg lat/lon -> point), requiring new converters, then yes, I think it's probably good for now.

i don't think it compiles can you take a look

Looks like OptionalCqlFilterParam got left out, probably during a merge. It compiles for me now.

@aheyne i tried to run this command:
$ bin/geomesa convert -C example-csv -s example-csv examples/ingest/csv/example.csv

and got this output:
2016-11-25 18:01:47,850 ERROR [org.locationtech.geomesa.accumulo.tools.AccumuloRunner$] org.locationtech.geomesa.tools.ConvertParameters$ConvertParameters cannot be cast to org.locationtech.geomesa.tools.export.ExportParams
java.lang.ClassCastException: org.locationtech.geomesa.tools.ConvertParameters$ConvertParameters cannot be cast to org.locationtech.geomesa.tools.export.ExportParams
  at org.locationtech.geomesa.tools.ConvertCommand.execute(ConvertCommand.scala:53)
  at org.locationtech.geomesa.tools.Runner$class.main(Runner.scala:24)
  at org.locationtech.geomesa.accumulo.tools.AccumuloRunner$.main(AccumuloRunner.scala:29)
  at org.locationtech.geomesa.accumulo.tools.AccumuloRunner.main(AccumuloRunner.scala)

@jahhulbert-ccri I fixed that issue you were having, I was casting params to the wrong subtype.
I've also put the what I have so far on the unit test if you'll take a look. It is functional atm and the test passes fine but it's currently only testing CSV, TSV and JSON. All it needs for the other types is an example input file and converter/sft. I'm still working on making these but it's somewhat tedious so if you have some already done examples we could drop in there that'd be awesome.

Merged as commit 04c8a378fcb1a28961e42ec33f863883463880e4


Merge latest master to trim_whitespace branch

Not sure what the point of this PR is, but #287 has been updated and further tweaked. All further changes should be made there.


Gateway API SupportedFeatures field

Problem Statement
GatewayClass's SupportedFeatures field graduation is one of the targets for Gateway API 1.2 (https://github.com/kubernetes-sigs/gateway-api/issues/3164). The main point of such a feature is adding a new SupportedFeatures field in the GatewayClass that is populated by the implementation. Such a field has two main purposes:

allow the conformance suite to read the GatewayClass and run the needed tests, without the need from the implementation maintainer to manually specify which features are supported
give users feedback on the features currently implemented by the implementation.

Additional Information
https://github.com/kubernetes-sigs/gateway-api/pull/3200 introduced a breaking change in the feature, so we need to wait for the next release (1.2) to be able to properly implement it.
Acceptance Criteria

[ ] The SupportedFeatures field is supported by the operator

This is blocked by the Gateway API 1.2 release


Possible to build on ESP32?

If so, how ?

I don't think it's possible to do so yet, xtensa isn't a supported platform by Zig/LLVM yet. I think you have to reimplement the parsing and rendering for Xtensa platform

@X-Ryl669 htcw_gfx 2.0 is being released shortly and will render TVG and SVG files. It runs on the ESP32. It's not ready yet, but it's being worked on right now.
homepage
1.0 is all that's available right now though, but I figured I should let you know. It will run on the ESP32.

Zig now has a C backend, so you should be able to compile the Sdk to C in theory. I'll need to take some days off to give TinyVG the last polishing it's missing, like updating the Sdk, fixing some more pertinent bugs and rework the specification into a better readable format.
If you're willing, you can try playing around with Zig and the C backend and see if you can make it work


PCA and variant frequency analsis for NFE samples

This script performs a PCA on NFE samples only from the combined HGDP/1kG + TOB-WGS dataset. It also filters for variants which are in the gnomad loadings, but not in the TOB-WGS data.

🎊


[WIP] Improve md2 js

Aiming to fix template string parsing in examples and make the code testable.
@joelmukuthu

Superseded by #28


"Drajna Nouă" is listed in the drop-down menu

Description:
"Drajna Nouă" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Drajna Nouă" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Drajna Nouă, CĂLĂRAȘI (Dragalina)".


Move atu disable to silicondevice destructor

When I initially implemented ATU in https://github.com/tenstorrent/tt-umd/pull/13, I had the wrong impression that only a single TTDevice is allowed to be open. TTDevice are being opened multiple times when ran from Pybuda, furthermore tt_SiliconDevice can be opened multiple times.
I've now moved at which exact point is ATU disabled, to better reflect when it was even set up.
This was tested with benchmark models on Pybuda.
This is a general fix for umd though, not Buda specific.
Buda still builds on main, hence the PR on main branch.

I know this PR was merged but do you think you can also push it to metal-main?

I know this PR was merged but do you think you can also push it to metal-main?

I can cherry pick the commit on that branch.
But isn't the idea to merge the metal-main into main anyway?

I know this PR was merged but do you think you can also push it to metal-main?

I can cherry pick the commit on that branch. But isn't the idea to merge the metal-main into main anyway?

yeah I believe so, Im just not sure when this would happen

Maybe it is a good moment to make a specific BBE branch, and we can review metal-main branch if that is going to speed up the process. So we can make changes on BBE branch if those changes are Buda specific. If not, we will merge it to main in a standard way