Leads for animals

Tempting animals with items is a really shitty experience. Porting leads from 1.6 would help significantly.
Should have some tech gating, potentially rope and glue (credit to Rocky for the idea)

https://github.com/BTW-Community/BTW-source/issues/69

Closing this as the update to 1.6.4 includes leads as a vanilla feature


Add meteor-blaze-components compatibility

If anybody else wants to use this package and their project is already using meteor-blaze-components, this fixes the name collision on BlazeComponent.

Legend, thanks, @duce3!  Merged and published as v1.4.2.  Thanks for your contribution!

Oh, @duce3, it says also this is your first pull request ever :)  Great job! :tada:


add density estimators to tests.

to allow testing different density estimators, e.g., see #819

I think this would slow down tests too much, no?
Alternatively, we could have a separate test for this and mark it @slow?

Hm that's true. But yes, I think having a slow test for it would probably be good.

Codecov Report

Merging #820 (464f8d4) into main (e3a041a) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #820   +/-   ##
=======================================
  Coverage   74.83%   74.83%
=======================================
  Files          80       80
  Lines        6196     6196
=======================================
  Hits         4637     4637
  Misses       1559     1559

Flag
Coverage Δ

unittests
74.83% <ø> (ø)

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


Add paths-filter action

Adds paths-filter to the Utility section.
This action enables conditional execution of workflow steps and jobs, based on the paths that are modified by pull request or in pushed commits.
There is already one action with similar declared purpose: Has Changed Path.
However it just detects changes introduced by latest commit while action added by this PR has much broader feature set.
Therefore I would not consider it being a duplicate.

I use this action & I can say it's pretty useful!


Collapsing header improvements

This is a PR without a backing issue.
This PR changes CollapsingHeader container like this:

CollapsingHeader is now opening or closing only when clicking the arrow icon
CollapsingHeader is now filling all the available horizontal space
Emit clicks on CollapsingHeader's header only when NOT clicking the icon

Visual display:

I think being able to click the entire header should be the default behavior, just as it is the default behavior to be able to click the label of a checkbox or a radiobutton to toggle it. This new behavior is only useful for selectable collapsing headers, so I suggest we restrict it to those.

just a random suggestion: single click could select and double click (or long press/click) could be expand. kinda like reddit comment threads or some tree ui in android apps.

I think being able to click the entire header should be the default behavior, just as it is the default behavior to be able to click the label of a checkbox or a radiobutton to toggle it. This new behavior is only useful for selectable collapsing headers, so I suggest we restrict it to those.

Now that you mention it my target is to be able to create something like this for my project.

Now I'm not quite sure how to make it API wise so we can add custom controls to the header. In my case I would like to add a checkbox in front of the title and a button on the right side of the label as it can be seen in the screenshot above.
I'm thinking of adding something like this, but the drawback is that you need to paint the title yourself:
CollapsingHeader::new("title").show_custom_header(|title, ui| {
    // Show checkbox

    // Show title

    // On the right side of the header, show button for actions popup menu
});

@DrOptix I think the best solution for that screenshot is to refactor CollapsingHeader so one can have a CollapsingSection without a header, controlled by a button. This is sort of how Window implements collapsing.

I found a nicer approach in https://github.com/emilk/egui/pull/1538


deploy-github should take in -c $CIRCLE_SHA1 to tag on a specific commit

can I take this over? I've started working on it


Adding an arrow to scroll to top

I want to add an arrow which on clicking will return to top of the page
@pooranjoyb Please assign this issue to me under GSSOC 2024 and add necessary labels to it.

Yes you can do it! Good Issue :)
Make sure you create a Component for that and add it to all the pages :) @SalehaTabassum
Do raise the PR to dev

Hey, I want to work on this issue. can you please assign it to me if no one is working on it ?

hey @pooranjoyb, i would like to work on this issue. can you please give this issue a level 2 and assign it to me?
i'm familiar with the codebase as i recently raised a PR to the popshop.

Assigned to @somuwashere based on fcfs basis. You have 12hrs to solve this! Go ahead :rocket:


KAFKA-17046: Upgrade netty version to 4.1.111.Final

The diff with netty 4.1.110.Final is:
https://github.com/netty/netty/compare/netty-4.1.110.Final...netty-4.1.111.Final
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

thank you @m1a2st , update LICENSE-binary to the latest version.

LGTM, Gentle ping @chia7712, PTAL

@ulysses-you could you please rebase code to trigger QA again?

thank you @chia7712  for the reminder, rebased.

thank you @m1a2st @chia7712 !


Restore: choose a recent backup > bal 0

All attempts on the test net:
I installed version 2.3.0 on a new device. After that I clicked receive, sent me some DFI and left the wallet again. When I left, I did not create a backup, because backups are only necessary for emergencies, but not for everyday use.
Then I tried to open the app again.

The wallet was opened, but the account balance was 0.
I did this test 3 times. 3 times the same result. The default Wallet.dat does not store private keys.

Then I changed the test. I have saved the backup. After that I could choose between the backup and the default file when restoring. The backup contains the coins, the default wallet does not.
Why does the standard wallet not store private keys?

Hi @RobinTorque , thanks for your feedback, please give us time to investigate this.

/triage accepted

The problem was fixed with version V2.3.2.
@izzycsy The issue can be closed from my point of view.

Thanks @RobinTorque

I'll re-open this for v2.3.2 @RobinTorque. It will not be included on this version. Unfortunately the issue requires a different fix as it causes other major issues.
Currently, what happens is:

By default, the node creates wallet.dat. Initially it's empty.
If you don't have any wallet or walletdir files, the node uses this file as the default wallet.

The issue occurs because we delete the default wallet.dat. We backup them every time we delete it (e.g, wallet-bak-dateofbackup.dat) on the default wallet directory so funds are safe. My initial fix caused some regressions that I had to rollback. I will do a proper fix for this on the next release.


std.regex postprocesses ctRegex every time at runtime

greensunny12 reported this on 2016-08-31T23:32:56Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=16457
CC List

dmitry.olsh (@DmitryOlshansky)
greeenify

Description
Consider the following program, it will crash as it allocates _a lot_ of memory before the garbage can be collected.

void main()
{
    import std.regex;
    enum re = ctRegex!(`((c)(s)?)?ti`);
    import core.memory : GC;
    string text = "Lorem ipsum dolor sit amet, consectetur adipiscing elit";

    foreach (i; 0..500_000_000)
    {
        if (auto m = matchAll(text, re)) {}
        //if (i % 1_000_000)
            //GC.collect();
    }
}

On my machine (16G) it crashes at about 5M iterations.
The GC profile finds two hotspots (here 2M iterations):

<PHONE_NUMBER>	        2000000	uint[] D main std/regex/internal/parser.d:1607
      184000000	        2000000	std.regex.internal.ir.Bytecode[] D main std/array.d:852

(the latter is insertInPlace)

After looking at the code it seems pretty weird, because "postprocess" should be called while constructing the RegEx and once only.

dmitry.olsh (@DmitryOlshansky) commented on 2016-09-03T19:51:57Z
(In reply to greensunny12 from comment #0)
> Consider the following program, it will crash as it allocates _a lot_ of
> memory before the garbage can be collected.
>
> void main()
> {
>     import std.regex;
>     enum re = ctRegex!(`((c)(s)?)?ti`);
>     import core.memory : GC;
>     string text = "Lorem ipsum dolor sit amet, consectetur adipiscing elit";
>
>     foreach (i; 0..500_000_000)
>     {
>         if (auto m = matchAll(text, re)) {}
>         //if (i % 1_000_000)
>             //GC.collect();
>     }
> }
>
> On my machine (16G) it crashes at about 5M iterations.
> The GC profile finds two hotspots (here 2M iterations):
>
><PHONE_NUMBER>	        2000000	uint[] D main
> std/regex/internal/parser.d:1607
>       184000000	        2000000	std.regex.internal.ir.Bytecode[] D main
> std/array.d:852
>
> (the latter is insertInPlace)
>
> After looking at the code it seems pretty weird, because "postprocess"
> should be called while constructing the RegEx and once only.

The problem is enum re = ... line. Enum means ctRegex!`((c)(s)?)?ti` is copy-pasted at the place of usage, basically reconstructing regex on each loop iteration b/c compile-time version can't cache compiled patterns.

Replace enum with static and all should be good.

greeenify commented on 2016-09-03T20:08:39Z
thanks a lot for looking at this so quickly :)

> The problem is enum re = ... line. Enum means ctRegex!`((c)(s)?)?ti` is copy-pasted at the place of usage, basically reconstructing regex on each loop iteration b/c compile-time version can't cache compiled patterns.

Okay that makes partially sense, but to give some background I found this problem because the D changelog generation crashed on my machine (16G) while parsing a 300kB log file. The dummy file is a dustmited & reduced version of the changelog generator:

https://github.com/dlang/tools/blob/master/changed.d#L77

And using `enum ... = ctRegex!...` seems to be a very common pattern:

https://github.com/search?q=enum+ctRegex&type=Code&utf8=%E2%9C%93

> Replace enum with static and all should be good.

Unfortunately I can still observe the same behavior, I also tried to move ctRegex to the matchAll.
Funnily with regex I don't get this allocation / out of memory -problem.

FYI static immutable doens't work:

foo.d(10): Error: template std.regex.matchAll cannot deduce function from argument types !()(string, immutable(StaticRegex!char)), candidates are:
/usr/include/dlang/dmd/std/regex/package.d(860):        std.regex.matchAll(R, RegEx)(R input, RegEx re) if (isSomeString!R && is(RegEx == Regex!(BasicElementOf!R)))
/usr/include/dlang/dmd/std/regex/package.d(868):        std.regex.matchAll(R, String)(R input, String re) if (isSomeString!R && isSomeString!String)
/usr/include/dlang/dmd/std/regex/package.d(875):        std.regex.matchAll(R, RegEx)(R input, RegEx re) if (isSomeString!R && is(RegEx == StaticRegex!(BasicElementOf!R)))

dlang-bugzilla (@CyberShadow) commented on 2017-07-21T09:00:44Z
I think the obvious reasonable improvement here is to improve the documentation, and mention to not use ctRegex with enum. Not sure much can be done in Phobos to work around an underlying language / compiler implementation issue.


How to use the vars start_date and static_incremental_days

Hi,
I have two issues with the  vars of the package

First Isuue - start_date
I'm trying to compile the object dbt_packages/ga4/models/staging/ga4/base/base_ga4__events.sql
When i compile it i got this error

Compilation Error in rpc request (from remote system.sql) Required var 'start_date' not found in config: Vars supplied to request = { "ga4": { "conversion_events": [ "add_payment_info", "add_shipping_info", "add_to_cart", "add_to_wishlist", "begin_checkout", "generate_lead", "purchase", "refund", "remove_from_cart", "select_item", "select_promotion", "view_cart", "view_item", "view_item_list", "view_promotion" ], "dataset": "analytics_297083529", "frequency": "daily", "project": "corneliani-374310", "start_date": "20230101" } }

I resolved the issue with this workaround:
I putted the variable start_date at first level of vars (out from ga4), so the question is:
This var must be at first level or i must keep 2 copy of it?

Second Issue - static_incremental_days
When I try to use this var as explained in this paragraph of the README: Incremental Loading of Event Data (and how to handle late-arriving hits) i got  this Server Error:

'str' object cannot be interpreted as an integer
16:56:56  Unable to do partial parsing because a project config has changed

I tryed to set the value of the parameter both as string
static_incremental_days: "current_date - 7"
and not
static_incremental_days: current_date - 7
And i got the same error showed before in both cases
What is the right way to set this parameter to be sure the last 7 days sharded events_ tables will be reprocessed every run?
Thanks.
Srefano G.

If you override models in your project, it's fairly easy to miss that you need to move the package variable to the project. It looks like you have that sorted, though, so good job there.
static_incremental_days should be an integer, just as the error message is telling you. So you want static_incremental_days: 7.

If you override models in your project, it's fairly easy to miss that you need to move the package variable to the project. It looks like you have that sorted, though, so good job there.
static_incremental_days should be an integer, just as the error message is telling you. So you want static_incremental_days: 7.

Thanks Damon,
now all is clear.
Stefano G.


Creating a html form input with the 'html-output' in php

I have looked and tried to find the answer to my question, read all the issues, read me file and I don't seem to find anywhere that tells me how I can create a variable in php with the data?
$htmltxt = $_REQUEST['pout'];
<input type="hidden" name="pout" value="html-output">
I understand php better than javascript, how do I send html-output to the form or how do I request it from php?
Answers would be helpful and adding this info to the readme file will be immensely useful.
If anyone wants to supply the answers I will even write the how to...
Thanks and Regards
Guy

I had the same problem. I used post, not REQUEST, because I don't know REQUEST. Here is my solution:
Remove this in edit.html
  <script>
      var editor = window.pell.init({
        element: document.getElementById('editor'),
        defaultParagraphSeparator: 'p',
        onChange: function (html) {
          document.getElementById('text-output').innerHTML = html
          document.getElementById('html-output').textContent = html
        }
      })
    </script>

and add this:

  <script>
      var editor = window.pell.init({
        element: document.getElementById('editor'),
        defaultParagraphSeparator: 'p',
        onChange: function (html) {
          document.getElementById('text-output').innerHTML = html
          document.getElementById('html-output').textContent = html
          document.getElementById('html-output2').textContent = html
        }
      })
    </script>
form action="NAMEOFYOURPHPFILE" method="post">
<input name="content" type="hidden" id="html-form">
</form>

NAMEOFYOURPHPFILE is the name of your php file.
(Sorry for bad English)


No need of eval

Codecov Report

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

@@         Coverage Diff          @@
##           master    #2   +/-   ##
====================================
  Coverage      80%   80%
====================================
  Files           3     3
  Lines          30    30
====================================
  Hits           24    24
  Misses          6     6

Impacted Files
Coverage Δ

src/ops.jl
100% <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 b9c45f1...59b9ab4. Read the comment docs.


Some Questions

Hello it's me again and I had a couple of questions:
1- How do I properly backup my wireguard  Users in this script ?
2- How many users can I add in this script ?
Thanks in advance

Simply backup your Wireguard config for Client and Server. (Debian server default: /etc/wireguard/wg0.conf )
It's a /24 Subnet. That means, 253 Clients, 1 Server, 1 Broadcast.


bad rabbit credentials reported as "socket closed"

this is pretty bad ux. can we do better?

Yes, yes we can https://github.com/onefinestay/nameko/pull/240


Add alicloud oss output binding

Closes #970

@yaron2 Why is this merged to RC3 docs? Wasn't this only added in the latest RC4 and geared towards releasing with 1.0 ?


refactor(ndt_scan_matcher, ndt_omp): move ndt_omp into ndt_scan_matcher

Description
Currently, we are using the ndt_omp package as an external library. We would like to release this package through ROS2 bloom, but a package with the same name, ndt_omp, has already been released.
Since ndt_omp is currently under TIER IV organization, adding the prefix “autoware_” to ndt_omp is not ideal.
As ndt_omp is a small library, we are considering merging it into the ndt_scan_matcher package.
Related links

https://github.com/tier4/ndt_omp/pull/68

How was this PR tested?

logging_simulator works the same as before
TIER IV internal deterministic test

Notes for reviewers
None.
Interface changes
ndt_scan_matcher no longer depends on ndt_omp.
Effects on system behavior
None.

@SakodaShintaro The code looks fine but ndt_omp is currently being used in map_assessment_tool, too. Is the tier4/ndt_omp still remain after moving ndt_omp to Autoware?

@anhnv3991
Yes, I will plan to archive tier4/ndt_omp.
https://docs.github.com/en/repositories/archiving-a-github-repository/archiving-repositories

@esteve
I think the current directory structure is what you described, but am I misunderstanding something?
shintarosakoda@dpc2304005:~/autoware$ tree ./src/universe/autoware.universe/localization/autoware_ndt_scan_matcher/include/
./src/universe/autoware.universe/localization/autoware_ndt_scan_matcher/include/
└── autoware
    └── ndt_scan_matcher
        ├── hyper_parameters.hpp
        ├── map_update_module.hpp
        ├── ndt_omp
        │   ├── LICENSE
        │   ├── README.md
        │   ├── estimate_covariance.hpp
        │   ├── multi_voxel_grid_covariance_omp.h
        │   ├── multigrid_ndt_omp.h
        │   └── ndt_struct.hpp
        ├── ndt_scan_matcher_core.hpp
        └── particle.hpp

3 directories, 10 files

What is the correct structure?

@SakodaShintaro ah, sorry, I think I misread the changes. That folder structure looks good to me, thanks.


Bootlint, CSS styles, dashboard, and visual tidyness

Breaks up Sass nesting and updates code for new style. Adds comments, switches uppercase hex values to lower per style, removes values prefixed 0., spacing per style. Adds container fluid and moves row to another DIV for bootlint.
Overrides homepage.html.erb and adds col-sm-5 and col-md-6 for bootlint and better display.
Refactors the heading greeting and removes the hover states from the icons.
Resolves #279 and #418


docs: add additional resources

(Disclosure: I run the respective sites. Thanks for considering!)

@msaaddev, are you still open for suggestions?


[flutter_tool] Crash less when git fails during 'version'

Description
This PR prevents the tool from crashing when the command to determine the latest git commit date fails. Instead, an error is printed, and a dummy date is returned. The dummy date should be innocuous since it is only used to print or to decide whether to print an update message. A preceding error message about the git command will probably prevent confusion about the dummy date.
Related Issues
Seen in crash logging.
Tests
I added the following tests:
New tests in version_test.dart.
Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Fixes #34142.

Also fixes https://github.com/flutter/flutter/issues/43565 I believe


Add troubleshooting information

Update the README file with some instructions concerning troubleshooting like:

Missing API key error on a Dependabot PR
Bad request error on API key rate limit in imgbb

Perhaps also consider a case when a user is out of free disk space limit...


Add YARD gem and generator

I developed a tool to generate RBS from YARD tags and applied it to YARD gem to generate RBS.

Could you commit the generator? Everyone else cannot modify the RBSs except you if the generator is unavailable in this repository.

@pocke
I've committed the generator and regenerated!

I fixed dependency for rack.
And also add to call check-untyped-call.rb on _scripts/test


Simplify BackedEnum::toArray

🎁
https://www.php.net/manual/en/function.array-column.php

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.88%. Comparing base (1d34858) to head (2f77eb6).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #11      +/-   ##
============================================
- Coverage     89.65%   88.88%   -0.77%
  Complexity       22       22
============================================
  Files             2        2
  Lines            58       54       -4
============================================
- Hits             52       48       -4
  Misses            6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Glad to contribute to your projects.
src: https://github.com/archtechx/enums/blob/27d0405b983b14a2ee90621c4034eeb6010c0a0c/src/Options.php#L18


Cannot parse "ExponentDecimalReal" numbers

Using the generated ANTLR4 grammar, I cannot parse exponent decimal real numbers. Parsing a string of 1E2 with the exponentDecimalReal rule returns the following error:
line 1:0 mismatched input '1E2' expecting {'.', DecimalInteger, Digit}

I am not yet very familiar with ANTLR but I think the issue is either the ordering of the rules or the lexer/parser nature of the related rules. This leads us to the shouldBeLexerRule method and the Antlr4Massager classes.
In issue #56 @Mats-SX stated that it's not a goal to provide a good ANTLR parser, so I think a good approach would be to patch the Antlr4 and the Antlr4Massager classes. Do you agree?
Also, @thobe can you please elaborate on how to proceed with the // TODO: melt these snow flakes. task?

Yes, these issues are caused by the lexer/parser division of the parsing paradigm that Antlr is using. Our source grammar doesn't (yet) have a notion of lexing, so the conversion to the Antlr side of thing has so far been quite crude, and with it has come a number of issues. We have cards on our wall that tells us to clean all of this up:

The current ANTLR grammar suffers from having to go to great length to wrestle a lexer out of the source grammar that does not understand that concept. It is quite foreseeable that other consumers of the grammar will hit the same issue. We should restructure the source grammar to be based on a core set of non-overlapping productions that can be translated directly to lexing rules.
A consequence of this is that the code that generates the Antlr4 grammar contains a sizeable piece of custom code to make the lexing work, which is a technical dept.

The problem right now is that we're short on resources. Hopefully we will be able to come back to these issues soon enough.


remove 'React' vs 'react' warning

Can I request the redux-oidc/dist/redux-oidc.js file to use
define(["react"]
instead of
define(["React"]
This is to remove the warnings like:
WARNING in ./~/react/lib/getNodeForCharacterOffset.js
There is another module with an equal name when case is ignored.

Sure thing. Will fix it asap.

It should be fixed in the latest version (1.1.14). Please let me know if this fixed your issue.


Feature/add test coverage

Explain how this PR changes scenario
Improving test coverage and updating ruff ignores
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] I have read the CONTRIBUTING document.
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

custom_components/scenario/cover.py
95.31% <100.00%> (+95.31%)
:arrow_up:

... and 3 files with indirect coverage changes


Longer blog name breaks mobile in post list

Settings longer xx.title in languages.toml causes incorrect page width on /posts/. It seems that the breadcrumb is not properly limited according to the window size (especially visible on mobile - swipe left):
https://blog.solidsoft.pl/posts/
The problem doesn't occur on the single post view, where the breadcrumb behaves correctly:
https://blog.solidsoft.pl/2020/03/02/enable-java-14-preview-features-in-gradle-maven-and-idea/
Btw, it just a version, WITHOUT changes from from #221.

incorrect page width

What do you mean the incorrect page width?
I checked the blog but the width seems to look normal

Yes, the whitespace, but not only in the screenshot. In the Firefox mobile I can just swipe right to see it. And without that not the whole text is visible.

With Firefox desktop (with the narrow window) I cannot swipe, but the problem with text out-of-the-screen it present.

The same breadcrumb (which is missing in your screenshots for some reason), in the single post view behaves correctly:

Yes, when I made display: none, it works as expected.
I'll update soon

Can you check again?
I just truncated the long site name to 150px

Not really :-/
On desktop:

Maybe I will just shorten the name of my blog ;-)

I just come across that it would be good if we made it scrollable

Maybe I will just shorten the name of my blog ;-)

Yes, I think it is the best way for now.
I tried some things but the CSS not working uniformly over cross-browser.
I'm closing this issue as I won't fix it.


Does it really needs webpack?

Webpack seems like an overkill
Why do we need to have a minified version?
Isn't it enough to use babel?

Can Babel bundle?

why do we need to bundle?
babel will produce a es5 compatible files

Perhaps we don't. I have not been paying much attention to this repo, honestly. I don't know what we're using Webpack for. If it's overkill, let's kill it.

If it's overkill, let's kill it.

Alright, let's do it ;)

I have not been paying much attention to this repo

Fair enough, I guess you are not using it that much (Have you red: Open Source Stamina ?)
Would you be ready to give some of us more rights? (I'm mainly thinking about @tjconcept, @brianmhunt and myself)
You gave us the 'Collaborator' role but with the condition not to use our right, which is kind of useless :D

The condition was that the PR process would be used. Feel free to merge each other's PRs. =)

Cool awesome ;)
Would you be ok using https://github.com/semantic-release/semantic-release so you don't have to care about releases or is that a security issue?
And could you also enable greenkeeper?
 Please re-open with new PRs if you're still interested.


[e2e]: delegator-lowperms push main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6465895577
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Tue Oct 10 07:14:21 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6492421422
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Thu Oct 12 07:14:10 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6505139822
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Fri Oct 13 07:13:28 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6516241393
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Sat Oct 14 07:12:44 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6522629644
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Sun Oct 15 07:12:36 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6530199050
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Mon Oct 16 07:14:43 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6543728397
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Tue Oct 17 07:18:18 UTC 2023
Tests are passing now. Closing this issue.


add valgrind and mpi_proc options to run_tests.py and travis tests

cc @dgergel for code review

@jhamman I've finished reviewing. Although we've added mpi and valgrind functionality, I wonder if we might want to do two VIC runs in run_system to specifically make sure that running VIC on one processor versus 4 yields the same results?

Although we've added mpi and valgrind functionality, I wonder if we might want to do two VIC runs in run_system to specifically make sure that running VIC on one processor versus 4 yields the same results?

Yes, we should do that but not in this PR. Can you open an issue for adding that test?


AnalyzeTemporaryDtors should be removed from .clang-tidy

Since it's no longer supported in clang-tidy trunk.

Thanks for letting us know!

Thanks for the quick fix!


Debugger can randomly cause crashes

No picture since this issue occurs randomly. Please photograph the error as soon as its spotted
From what I understand, a desync can occur between the Game and the Editor in terms of FEAGI Debug communication, which spams the error logs it crashes the whole editor soon after.
This only effects local testing, FEAGI Debugging is disabled on all exports
It can be avoided by disabling debugging when not needed.

if I were to take a guess, some array of bytes is less than 4 bytes long, which causes the error to trigger


whisper-info: add --json output option

fixes #159

Coverage remained the same at 88.577% when pulling 959a5a80152ad8b29a5f1facc0139a46743dc767 on piotr1212:info_json into 0d5a0deac07e4bea822e1350c939f89acb370033 on graphite-project:master.

Coverage remained the same at 88.577% when pulling 959a5a80152ad8b29a5f1facc0139a46743dc767 on piotr1212:info_json into 0d5a0deac07e4bea822e1350c939f89acb370033 on graphite-project:master.

Hi @piotr1212 ,
Could you please rebase?

Coverage remained the same at 88.3% when pulling f9ff46eab3f9d9f835701f49e3904a5a1230c8e6 on piotr1212:info_json into eafbc0367bb702cb9d1860529248d87b852f31b2 on graphite-project:master.


C#/Mono fixes and enhancements

Fixes #21139
Fixes #13355
Fixes #21667

Throws hundreds of errors when I try to compile master with Mono enabled now...

Ahhh! I can't believe I forgot to mention this... This PR introduces an intentional build regression. If the file mono_glue.gen.cpp was generated before the changes in b1356a3590ad1a4ed7bc4f85e1ffe7e9776855c3, you will have to delete it manually. The reason is that now mono_glue.gen.cpp is compiled even when passing mono_glue=no. The newly generated file is surrounded with #ifdef MONO_GLUE_ENABLED so it's not a problem, but old files don't have that.

Deleting that file worked. This may confuse many people though...

Hi,
Even after deleting everything "generated", I still have 2 errors :

...\csharp_script.cpp(1466) : error C4716: 'CSharpInstance::mono_object_disposed' : doit retourner une valeur
...\csharp_script.cpp(1489) : error C4716: 'CSharpInstance::mono_object_disposed_baseref' : doit retourner une valeur

Compiling one Win10 x64 with VS2015
Those two functions are now bool but without any return, is it intentional ?

No, those methods shouldn't return anything. I changed it and forgot to revert the changes.

Fixed the return type build regression: 5e57beebb1a43c65d949fdfe5081f9c27f181c80


ForceGC throws errors: "The GUID passed was not recognized as valid by a WMI data provider. (Exception from HRESULT: 0x80071068)"

I was trying to use PerfView to force GC on a process as suggested by http://stackoverflow.com/a/26033289/1837339, however, it seems not working and I got an error like this:
Loading the ETWClrProfiler.
Turning on debug privilege.
Highest Runtime in process is version v4.0.30319
  0.0s: Trying to attach a profiler.
  0.0s: Done Attaching ETLClrProfiler ret = 0
Attached ETWClrProfiler.
  0.1s: Enabling JScript Heap Provider
  0.1s: Enabling EtwClrProfiler
  0.1s: Enabling CLR GC events
  0.1s: Requesting a JScript GC
  0.1s: Requesting a DotNet GC
  0.1s: Requesting .NET Native GC
HeapDump Error: System.Runtime.InteropServices.COMException (0x80071068): The GUID passed was not recognized as valid by a WMI data provider. (Exception from HRESULT: 0x80071068)
   at System.Runtime.InteropServices.Marshal.ThrowExceptionForHRInternal(Int32 errorCode, IntPtr errorInfo)
   at Microsoft.Diagnostics.Tracing.Session.TraceEventSession.CaptureState(Guid providerGuid, UInt64 matchAnyKeywords, Int32 filterType, Object data)
   at GCHeapDumper.TriggerAllGCs(TraceEventSession session, Stopwatch sw, Int32 processID)
   at GCHeapDumper.ForceGC(Int32 processID)
   at Program.MainWorker(String[] args)

Any idea what could be causing the issue? Thanks.

However, I found that the issue doesn't appear when I build the HeapDump.exe locally with latest code from master branch. It seems only happen on the version 1.9.0 I downloaded from https://www.microsoft.com/en-au/download/details.aspx?id=28567

I have the same problem with PerfView downloaded from the link pasted above.

Getting the latest compiled version from this site worked for me.

I am closing this as it does not seem to reproducible from the current source
Note that you can get built binaries of PerfView from https://github.com/Microsoft/perfview/releases
We will update the download center, but probably not for a few months yet.


Broken/Invalid url

When Clicking on the Add Source list link for community source codes. The link appears to
Be a broken or an invalid url. Tried first on safari, then on multiple different browsers and the same problem has occurred.

Nevermind it was on my side of things


Add builder interface to the API

A builder interface minimizes the memory footprint and allows other interesting optimizations in the future. It should become the recommended interface for ingesting ballots.

Closed with #35


crates.io page has broken logo

Describe the bug
https://crates.io/crates/datafusion/43.0.0 apache datafusion logo is missing

To Reproduce
not sure
Expected behavior
The logo should be like https://datafusion.apache.org/

Additional context
No response

https://github.com/apache/datafusion/raw/HEAD/datafusion/core/docs/source/_static/images/2x_bgwhite_original.png was found in crate.io.
https://github.com/apache/datafusion/raw/HEAD/docs/source/_static/images/2x_bgwhite_original.png is the correct one.

The logo is missing because readme = "../../README.md" in datafusion/core/Cargo.toml.
If image ./static/image.png was specified, in this case crates.io resolve path to datafusion/core/static/image.png, but static/image.png is what we would like is static/image.png
Possible solution:

use absolute path
maintain a separate README.md for crate.io(create datafusion/core/README.md and readme = "./README.md")

I prefer first one, and I could open a PR if you agree.
Also, I thinks it's an good first issue, so I can also leave it to other if missing logo is not an serious problem

I prefer first one, and I could open a PR if you agree.

I agree. Thank you 🐰

Also, I thinks it's an good first issue, so I can also leave it to other if missing logo is not an serious problem

Good call -- I added the tag. I defer to you if you want to fix it.

take


Dialog title specs

The specs for the Dialog's title text are non-existent. I'm not sure if they just want you to use normal typography headings or what.
For 1.x I made up just some very readable size good for short titles. If developers want something else, the standard typography heading classes are available.
We should ask for clear guidance on this from the MD team for future updates though.

Only information I could really find that may be of some use is toward the bottom of the dialog components page in the spec.
https://www.google.com/design/spec/components/dialogs.html#dialogs-specs
It says that for a full screen dialog that the title font size is 20sp. The font sizes appear to be comparable with just a quick glance, so to me it looks like a plain  tag would be best suitable for now.
I was playing around with the dialog element earlier today and I agree the current state of the heading is a little obnoxious but it works.


Allow using DSPy with Python 3.12

Closes #572.
Hello! We use DSPy with Python 3.12 in production and would be happy to contribute this support back to the project if desired.
In terms of the changes required to get this working, there were only two:

Bump the Python version in pyproject.toml.
Upgrade the fastembed package to at least version 0.2.0.

I see that this has already been looked at in  #582 but I'm not sure what the status of that PR is at this point.

Would highly appreciate enabling python3.12 🙏

Hi @krishnan-chandra , thanks for the PR!
I think this can be merged but tagging @hmoazam here since this impacts dependencies and an existing PR #919 related to ensuring proper pypi/dependency releases. Let us know if this is fine to merge or should wait for your PR @hmoazam !

Hi @krishnan-chandra , thanks for the PR!
I think this can be merged but tagging @hmoazam here since this impacts dependencies and an existing PR #919 related to ensuring proper pypi/dependency releases. Let us know if this is fine to merge or should wait for your PR @hmoazam !

This looks fine to me, I should have my PR in tomorrow and can merge in these changes so not a problem.

Thanks everyone!


[FEA] Remove URLs from pydantic error messages

Fixes https://github.com/NVIDIA/spark-rapids-tools/issues/541
Exceptions thrown by the CLI argument validator may show some pydantic specific information that is likely to confuse the user, specifically the pydantic URL in the following case.
2023-09-01 14:21:50,410 ERROR pyrapids.argparser: Validation err: 1 validation error for QualifyUserArgModel
  Value error, Invalid arguments: Cannot run tool cmd. Cannot define eventlogs from input (platform, cluster, and eventlogs) [type=value_error, input_value=ArgsKwargs((), {'eventlog...commendation': 'MATCH'}), input_type=ArgsKwargs]
    For further information visit https://errors.pydantic.dev/2.1/v/value_error

We replaced use sites of IllegalArgumentError with PydanticCustomError for more clear exception error messages.
Example of new error message:
2023-09-13 14:21:20,439 ERROR spark_rapids_tools.argparser: Validation err: 1 validation error for QualifyUserArgModel
  Cannot run tool cmd. Cannot define eventlogs from input (platform, cluster, and eventlogs)
  Error: [type=invalid_argument, input_value=ArgsKwargs((), {'eventlog...commendation': 'MATCH'}), input_type=ArgsKwargs]

Follow up work:
We need to add unit tests to evaluate change in this PR. Tracked here: https://github.com/NVIDIA/spark-rapids-tools/issues/562

Can you please add sample of the new error message in the PR description or as a PR comment? this way the reviewer can see how the changes affect the users.

I included a sample new error message in description of this PR.

These changes can be evaluated using unit-tests which is a good thing to have. By capturing the output of the STDOUT in the unit tests and assert that it follows a certain pattern. This should be done in the UTs defined user_tools/tests/pyrapids_unit/test_tool_argprocessor.py

We will update unit tests in a separate PR, the issue is track here: https://github.com/NVIDIA/spark-rapids-tools/issues/562

Thanks @cindyyuanjiang. I think the arguments input_value and input_type seem unclear to me. Could you explain what values are they showing?
Error: [type=invalid_argument, input_value=ArgsKwargs((), {'eventlog...commendation': 'MATCH'}), input_type=ArgsKwargs]

Thanks @cindyyuanjiang. The arguments input_value and input_type seem unclear to me. Could you explain what values are they showing?
Error: [type=invalid_argument, input_value=ArgsKwargs((), {'eventlog...commendation': 'MATCH'}), input_type=ArgsKwargs]

Thanks @parthosa ! These key words come from Pydantic error message formatting. I don't have a way to change that. AFAIK, input_value and input_type refers to the value and type of the input, but since we are validating a command line argument here, it is not very clear. A  common error may look like Input should be greater than 42 [type=greater_than, input_value=21, input_type=int].


support for python 3.5, 3.6

add support for python 3.5 and 3.6 in PyPI

Is this same limitation? (or) some new issue?
Kindly let me know if there workaround.
(pyats) nkaliape@narayan-vm:~/pyats$ python3 -V
Python 3.5.2
(pyats) nkaliape@narayan-vm:~/pyats$ pip install pyats
Collecting pyats
Could not find a version that satisfies the requirement pyats (from versions: )
No matching distribution found for pyats
(pyats) nkaliape@narayan-vm:~/pyats$ cat /etc/issue
Ubuntu 16.04.3 LTS \n \l

yep, same problem. Please use python 3.4
on ubuntu it goes:
apt-get install python3.4 python3.4-pip python3.4-venv
then:
python3.4 -m venv <folder>' to create a virtual environment.

support for python 35/36 is added in v4.1.+


Establish component communications

Use context for inter-component communication.
Getting the required results.
App.js will be the parent component and rest will be its child

Communication successful


Create a Twitter account

Opening this for discussion. Having a social media presence will help us communicate with users and the community at large. It could retweet blog posts about Shipwright, or relevant to building container images on cluster.
A few Shipwright contributors should have access to the login credentials.
Both @shipwrightio and @shipwrightbuild are currently available (@shipwright isn't)
cc @qu1queee @sbose78 @adambkaplan thoughts?

+1
We could post about the following:

features
releases
enhancement proposals
Retweeting things happening in this ecosystem
Suez memes
Demos!
Blog mentions

On Wed, Mar 31, 2021, 21:09 Jason Hall @.***> wrote:

Opening this for discussion. Having a social media presence will help us
communicate with users and the community at large. It could retweet blog
posts about Shipwright, or relevant to building container images on cluster.
A few Shipwright contributors should have access to the login credentials.
Both @shipwrightio https://twitter.com/shipwrightio and @shipwrightbuild
https://twitter.com/shipwrightbuild are currently available @.***
https://twitter.com/shipwright isn't)
cc @qu1queee https://github.com/qu1queee @sbose78
https://github.com/sbose78 @adambkaplan https://github.com/adambkaplan
thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shipwright-io/build/issues/707, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEFEAAOPE4H3RXW547ANCDTGPBUFANCNFSM42F52ZQA
.

shp it
https://twitter.com/shipwrightio/status/1379267460243816455?s=20


[Improvement][dolpscheduler-ui] some datasourceType which backend can't support ,but show in frontend

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is this only for the datax task? @fuchanghai


Consistent use of the plural form of repository

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
A lot! Check this search: https://docs.github.com/en?query=repository's
What part(s) of the article would you like to see updated?
Filenames (it's just strange) as well as the texts itself
Additional information
Hi folks, I was going to edit this page  to fix the name, since the plural of repository should be repositories in my opinion, and not repository's, but then I noticed more pages who suffer from this inconsistent use of the word. About a 10 pages have it in their content already, not sure about how many file names this is happening.
Therefor I wanted to reach out: what has the preference and can I start proposing PR's that link to this issue for this?
I can image it is just updating the file name and add the original file path to the redirect from path. But is there anything else that needs an update then?

@rajbos Probably the use of "repository's" is correct in this page https://docs.github.com/en/repositories/releasing-projects-on-github/viewing-your-repositorys-releases-and-tags. However, the intro is only stating users to view releases and tags of a single repository.

Indeed, there will be exceptions 😅.
Question is more about the process to get this fixed 😀.

👋 Hey @rajbos, thanks for opening an issue! I can definitely see how this could grab someone's attention. However, the references in the search titles are not plural but show possession of a single repository. So the files are correct with the singular form repository with an 's for possession.
I hope that clears up things for the titles; I'm going to go ahead and close this issue 💛
Thank you for your passion for improving the docs! ⚡


Add VerifyPeer and AllowWrongHost options to validate upstream certificates

This is my attempt to implement issue #80. I've been using this in my SSLproxy project for about 6 months now. I think VerifyPeer works well, but I am suspicious if AllowWrongHost option does what you actually want.
Also note that since we have conf file support now, I haven't created command line options for these.

A few points to discuss before merging:

I am not sure what the intention behind AllowWrongHost is. Allow wrong hostname where? Is this intended to weaken VerifyPeer or is it intended to prevent SSLsplit from adding the SNI hostname to the forged server certificates if it does not match already? Code looks more like the latter, but documentation and inclusion in the same pull request as VerifyPeer seems to suggest the former.
Assuming AllowWrongHost is intended to weaken VerifyPeer, then I do not understand the point of having AllowWrongHost. What is the use case for using both VerifyPeer and AllowWrongHost at the same time?
On the question of not exposing VerifyPeer to the command line directly I've opened #200 to compensate for that.

Yes, AllowWrongHost does the latter. That's why I was asking your opinion. Do you think it helps in anyway? I can remove it altogether, if it is useless. But if it does help, I can make the documentation clear (how?).
The defaults are VerifyPeer=no and AllowWrongHost=yes. This is the weakest case and also the behavior of the current master and develop branches, without those options of course. But with VerifyPeer=yes, I think we get validation (the issue about AllowWrongHost in your first point remaining).

I did not intent to have any relation between VerifyPeer and AllowWrongHost.

To further explain my intention, AllowWrongHostname (AddSNIToCertificate) was for increasing security. With AllowWrongHostname=no, the SSLproxy/SSLsplit passes the wrong.host (hence the name) test at https://badssl.com, otherwise it fails (VerifyPeer cannot help with this test). That is why I thought this option was useful.
I will modify the name and documentation as you recommend. But I want to ask again, if this was not what you had in mind in the issue #80, I can remove the AllowWrongHostname (AddSNIToCertificate) option from my pr. What do you think?

I can see the value of having the option now :) I think it would be good to also document that the option is needed to pass such browser tests which test whether the browser refuses to connect to servers with a hostname mismatch in their server certificate's CN/SAN.

I have updated option name and documentation accordingly.

On the other tests I don't see any sensible way to pass those, basically it would mean to write code to replicate the exact structure of the server certificate chain and all crypto parameters (like DH group size). Doable, but not a two-line change. I'd suggest to open an issue on that.


Clean-up: stale encumbrances

Source
stale encumbrances
Summary
stale encumbrances, specifically those expired over a year ago AND has no records attached, need to be removed to streamline processes
User Interface Changes
Post a note on /Encumbrances.cfm
Encumbrances which meet these conditions will be deleted once a year: a) expiration that are over 1 year old; b)  no attached records
Data Structure Changes
none

Add autodelete criteria to top of UI


Use old array syntax for payment data array acess

Hi Hugh,
Thanks for this fix so it works on older PHP versions as well.
Regards,
Rik
Adyen


ESS AC POWERSETPOINT can't be set

This field is used to engage battery load from the grid. The integration shows changes in the value correctly, but it's value can't be set from within the home assistant.

Addition:
The values are not displayed in the entities when changed from outside, they are just logged correctly.

Have you tried deleting once and adding the victron connection again? Then it worked for me.

Hello @AminShAT and thank you for opening this issue,
Could you perhaps provide me with a screenshot of your hass-victrons integration configuration?
This would help me in narrowing down the cause of your issue since the values provided in the integrations configuration are used to calculate safe min/max values for entities.

Du you mean this one?

Exactly.
In your configuration you specify an AC per phase current limit of 0.
Since this is the base for calculating the powersetpoint range (namely grid V * grid A) this results in the behaviour you are experiencing.
By setting the current limit to an value in AMPS you will see the slider entity become useable for setting the powersetpoint.

I understand, but this is inconsistent with the behavior in Victron itself and from outside via Nodered, where it is possible. Not sure, if it would be best practice, to have 2 places to maintain system characteristics. Aren't those values available within Victron?

Unfortunatly the possibility of discovering settings in the GX device is limited via the modbus server.
Therefore, a register scan for all valid unit id's was implemented and amps/voltages are explicitly asked for.
Since, the integration cannot retrieve the values from the GX device.

Alright. So set it to 65A. Will try next time. I used actually to force load my batteries.

Great to hear.
I'm closing the issue as resolved.


Inconsistency between docker environment and dev-env

It was discovered that the docker environment was inadvertently based on debian:jessie rather than ubuntu trusty.  This should be based on ubuntu to match the vagrant environment, ideally driven by the same configuration

@jeffgarratt @srderson @muralisrini @jyellick

Gah, I started with opening an issue and then seem to have become confused and wrote it like a PR...

@ghaskins closing as I believe you have resolved this issue with the openblockchain/baseimage image.


Deprecated ISO 639 codes

Some of these ISO 639 codes have been deprecated and split into more specific codes. The new codes should be added, keeping the deprecated codes as was done for 'ARK '.

bhk → fbl, lbl, rbl, ubl
btb → beb, bum, bxp, eto, ewo, mct
duj → dwu, dwy
flm → cfm, rnl
dap → njz, tgj
xst → stv, wle

One of btb’s replacements, fan, is already associated with a language system tag: 'FAN0'.

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

ID: 662c6f54-f90c-52ed-dca2-0f0a0a49fee5
Version Independent ID: 6aa33924-d45e-60c3-a9fd-e317ff662993
Content: Language system tags - Typography
Content Source: typographydocs/opentype/spec/languagetags.md
Product: windows
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Maintaining at this level of detail isn't a high priority, but thanks for providing the details. Adding in the next version.


[Mega Issue] Agent Quality: Editing

What problem or use case are you trying to solve?
OpenDevin hits a bunch of issues when trying to perform edits:

[ ] https://github.com/OpenDevin/OpenDevin/issues/3214
[ ] Repeatedly failed to edit: https://opendevin.slack.com/archives/C06R25BT5B2/p1722547184048369

Describe the UX of the solution you'd like
Integrate / Build a light-weight evaluation for editing:

[x] https://github.com/OpenDevin/OpenDevin/issues/3183
[x] https://github.com/OpenDevin/OpenDevin/pull/2685#discussion_r1659651873

(Longer-term) Collect data and train a specialized model for editing

[ ] https://github.com/OpenDevin/OpenDevin/issues/1854

Do you have thoughts on the technical implementation?
Describe alternatives you've considered
Additional context

Here is an example of a prompt that causes lots of editing issues: https://www.all-hands.dev/share-opendevin?share_id=4e4841bd6240dc2ee334742dee59f5104b5c982ff9309aaeea5ce977968bfcf3

Related issue: https://github.com/OpenDevin/OpenDevin/issues/3412

Related issue: #3452

Editing can be implemented as Terminal or GUI agent operations. For example:

Instruct the Terminal agent to use an editor like nano or vim
Instruct the GUI agent to use an IDE like vscode or Jetbrains IDEA

Interesting @James4Ever0 ! Have you compared the performance of something like this compared to other methods like the ones implemented in OpenHands or SWE-Agent?

Interesting @James4Ever0 ! Have you compared the accuracy of something like this compared to other methods like the ones implemented in OpenHands or SWE-Agent?

Benchmarking has not yet been done since this is just a library to interact with terminal and the actual terminal agent implementation is the most important factor in performance.
This method is promising since it allows direct interaction with almost all terminal programs, including those rendering complex graphics (like gameplay and QR codes) and layouts.

We've made a lot of good progress on this generally. @xingyaoww or @enyst do we feel like this is resolved?

Let's close it for now since editing has got a lot better since new sonnet


Flatten flags for all compile approaches

There's a helper that flattens the options for Gulp, but nowhere else: aka '--foo_bar' => 'fooBar'.

Done


HomeBrew support

We need a brew tap that allows one-line installation

I can add it to a homebrew tap if you provide a installable dmg on this github repo page


Update url from http to https

I updated url of the following domain from http to https.

www.fluentd.org
docs.fluentd.org
kafka.apache.org
www.splunk.com
cncf.io
fluentd.org
github.com
groups.google.com
slack.fluentd.org
docs.ruby-lang.org
test-unit.github.io

Thanks!


Snap.svg on Node.js

Hi!
I'm asking because I found no documentation about this topic. Is it possible to use Snap.svg with Node.js? Did someone succeeded making it work server side?
Thanks in advance for your answers

On its own, not without modification.
Node.js' require system is modeled after the CommonJS spec and there is issue #447 for it. Support would be awesome.
If you happen to be using webpack, or don't mind utilizing it, it has an imports-loader module that can help shim in CommonJS compatibility for the time being. The approach is detailed in #341.

I recently posted the answer in SO about running Snap.svg in Node.js with jsdom.
In short:
const jsdom = require('jsdom');
const xmlserializer = require('xmlserializer');

jsdom.env('', ['node_modules/snapsvg/dist/snap.svg.js'], (error, window) => {
    if (error) throw error;

    const paper = window.Snap(100, 100);

    const rect = paper.rect(20, 20, 60, 60);
    rect.attr({fill: 'red'});

    const svg = xmlserializer.serializeToString(paper.node);
    window.close();

    console.log(svg);
});

This can be solved with this loader config (seen in #341):
{
    test: require.resolve('snapsvg'),
    loader: 'imports-loader?this=>window,fix=>module.exports=0'
 }

@roadev @r4j4h  I'm beginner of Snap. could you please help me? I couldn't figure out how can I manage webpack dev mode and prod mode with same Snap code. I'm using angular2 with typescript.
I put the webpack loaders in webpack.common.js after reading #341
[{
    test: require.resolve('snapsvg'),
    loader: 'imports-loader?this=>window,fix=>module.exports=0'
 },]

Is this loader able to put snap javascript files into dist folder's one of js file?
I'm confusing because I put <script src="../public/scripts/snapsvg/dist/snap.svg.js"></script> in index.html and I can test Snap in develop mode, but when I try production mode, index.html file in dist folder try to find snap.svg.js from the location but it can't find.
(x GET http://localhost:3000/public/scripts/snapsvg/dist/snap.svg.js )
If I put snap.svg.js file manually in dist folder and change the script file's location to root like <script src="snap.svg.js">, it works. but in this case, dev mode doesn't work unless I put the same file into code's root folder.
Frankly speaking, I can't understand why <script src = ...> is required in index.html even though I declared Snap in source code as Snap = require( "imports-loader?this=>window,fix=>module.exports=0!snapsvg/dist/snap.svg.js" );

Did anyone manage to have it run well with node.js?
Any best practices (or alternatives for *.svg processing in node.js)?

@roadev I arrive here without knowing what "loader" is. Can you give a more fleshed out example of your solution based on #341 that is specific to node?

@sparr See #483. It’s for working with webpack, which is a preferred way of bundling & managing JS for front-end code (it also allows you to use NPM for easy version management).

@Sammons that’s awesome. Could you submit a PR that adds that to the README? That seems useful for others.

but jsdom does not have implemented getBBox https://github.com/jsdom/jsdom/issues/918

jsdom is on version 16 now.
anyone have an updated template to get this running with latest version of JSDOM?


Show column stats

Jira: AI-38, AI-57
Students are going to need to be able to "inspect" their data to check for errors in preparation for training.
For categorical columns, we now show the number of unique values in the column and what those values are. For continuous columns, we now validate that the column only contains numerical values and show the max and min values in the column.

This will help students identify when they have more categories than anticipated, for example:

If they've mis-identified the datatype stored in a column or if this there is a string lurking in a column that should only contain numbers, for example:

These functions look like good candidates for unit testing too.

definitely - I opened a task for this.


But why?

Hello,
Could you please clarify why one would like to use Polly instead of handling exception in a mainstream fashion?
I'm asking this because it seems to me that while using Polly will reduce boilerplate, it will be more obscure to those who read the code.
Thanks!

@dkrikun I wondered the same thing when I first came across Polly. So, I'll leave you with a use case where I implemented Polly in a way that would've cost me many more key strokes:
I developed a Xamarin-based mobile app used by field engineers to take pictures after work is done on a job site, and to fill out a quality control check sheet. The app is built to run offline or within occasionally connected scenarios. When they are ready to submit their data to the intake service, they might not have the best of connections.
Polly provides an interesting retry policy when an exception is encountered. Now, consider this: you've got 30 reasonably large photos to upload to the cloud. You may not have the most reliable data connection, but it's mostly there. You need to upload these photos while you can. So, the way I handled it is to set an exponential retry policy that waits for longer durations each time an exception is encountered before trying again. You can clearly see the exception handling here, and also easily understand the retry logic.
Here's a code sample from my project. If I were to write this from scratch, it would take a lot more time to do, so why reinvent the wheel?
                    // Retry a specified number of times, using a function to
					// calculate the duration to wait between retries based on
					// the current retry attempt (allows for exponential backoff),
					// calling an action on each retry with the current exception,
					// duration and context provided to Execute()
					// In this case will wait for
					//  1 ^ 2 = 2 seconds then
					//  2 ^ 2 = 4 seconds then
					//  3 ^ 2 = 8 seconds then
					//  4 ^ 2 = 16 seconds then
					//  5 ^ 2 = 32 seconds
					await Policy
						.Handle<Exception> ()
						.WaitAndRetryAsync (5, retryAttempt =>
							TimeSpan.FromSeconds (Math.Pow (2, retryAttempt)),
							(exception, timeSpan, context) => {
								// Send exception logging to Xamarin.Insights:
								MessagingCenter.Send<MobileServiceClient, ExceptionContainerException> (QCApp.Client,
									Common.Constants.MessageCenterMessages.MessagingCenterError, new ExceptionContainerException { Title = "Error uploading image",
										Exception = exception
									});
								Insights.Report (exception, new Dictionary<string,string> {
									{ "Filename", "SectionViewModel.cs" },
									{ "Where", "UploadImage" },
									{ "Issue", "Error uploading image" }
								});
							}
						)
						.ExecuteAsync (async () =>
							// JDH - 04/12/2015 - Moved away from using the Microsoft.WindowsAzure.Storage SDK in
							// favor of direct REST calls due to the SDK's slowness. REST calls appear to run about
							// 15x faster for uploads!
							BlobTransfer.UploadFileAsync (picture, picture.Filename)
						);

Pretty clear and concise, isn't it?
BTW, implementing this exponential retry cut down failed upload attempts to almost zero.
Hopefully this helped illustrate a good use case for this excellent library.


Refactor rate limiters

To prepare for the introduction of a Redis-based rate limiting (required for Puma), the current implementation has been refactored.

ExpiringRequestCounter
-- Renamed from RequestCounter.
-- Single increment method (combines old get + increment methods).
-- Returns expires_in seconds instead of valid_until timestamp.

ConcurrentRequestCounter
-- Renamed from ServiceBrokerRequestCounter.
-- Not a singleton anymore.
-- Renamed methods from try_acquire?/release to try_increment?/decrement.

UserResetInterval returns next expires_in seconds instead of reset_interval timestamp.

Retry-After header returns "delay-seconds" instead of epoch seconds (fixes #2544).

Tests cleanup.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

The Mozilla docs state that both HTTP Dates and seconds are valid for the Retry-After header, so I'm curious about the impetus for that change.  Are we sending a Date, but still not the right format?
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Retry-After
Any concern that some clients have code expecting Retry-After to be a date and this will be a behavior change?


[Bug]: Wrong certificate uploaded to tenant during "Generate certificates" command

Description
Summary
It seems that for Generate certificates command or Initialize workspace command (which includes certificates generation as a step, so same logic, essentially) the wrong certificate is uploaded to the tenant.
When you then build an extension and sign it with that certificate, the following error is returned by the tenant:
Dynatrace Log output:
2024-07-01T14:38:21.293Z [INFO][dynatrace-extensions.buildExtension] Command called.
2024-07-01T14:38:21.293Z [INFO][utils.conditionCheckers.checkWorkspaceOpen] Is a workspace open? true
2024-07-01T14:38:21.306Z [INFO][utils.conditionCheckers.isExtensionsWorkspace] Is this an extensions workspace? true
2024-07-01T14:38:21.323Z [INFO][utils.conditionCheckers.checkCertificateExists] Does DEV certificates exist? true
2024-07-01T14:38:21.331Z [INFO][utils.conditionCheckers.checkNoProblemsInManifest] Are there problems in the manifest? true
2024-07-01T14:38:21.335Z [INFO][commandPalette.buildExtension.buildExtension] Executing Build Extension command
2024-07-01T14:38:21.822Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T14:38:21.872Z [INFO][commandPalette.buildExtension.assembleStandard] Built the inner archive: /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/extension.zip
2024-07-01T14:38:21.882Z [INFO][utils.cryptography.sign] Signing extension with credentials from /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/developer.pem
2024-07-01T14:38:22.004Z [INFO][commandPalette.buildExtension.assembleStandard] Wrote the signature file: /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/extension.zip.sig
2024-07-01T14:38:22.016Z [INFO][commandPalette.buildExtension.assembleStandard] Wrote initial outer zip at: /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/custom_my.awesome.test-0.0.1.zip
2024-07-01T14:38:22.547Z [INFO][dynatrace-extensions.buildExtension] Completed normally
2024-07-01T14:38:22.557Z [ERROR][] Extension validation failed.

Dynatrace output:
{
  "code": 400,
  "message": "Extension signature is not valid"
}

Steps to reproduce
Initial configuration

Clean or existing installation of VS Code add-on (both tested)
Set it up to be connected to the dev tenant
Create a new folder
Open this empty folder with VS Code
Invoke Initialize workspace or click the button with the same text in "Dynatrace extensions" sidebar
Go through the steps, choose to Generate new ones when selecting which certificates to use
Choose "Yes" to set these certificates as default for all workspaces
Choose "Yes" to upload the CA certificate to Dynatrace
You receive an error: Cannot initialize workspace without certificates.

Here, I'm overwriting the existing certificate but I've verified that the behavior is the same when the certificate doesn't exist.
2024-07-01T19:32:10.495Z [INFO][dynatrace-extensions.initWorkspace] Command called.
2024-07-01T19:32:10.496Z [INFO][utils.conditionCheckers.checkWorkspaceOpen] Is a workspace open? true
2024-07-01T19:32:10.510Z [INFO][utils.conditionCheckers.checkEnvironmentConnected] Is a tenant connected? true
2024-07-01T19:32:10.522Z [INFO][commandPalette.initWorkspace.initWorkspace] Executing Initialize Workspace command
2024-07-01T19:32:10.534Z [INFO][commandPalette.initWorkspace.initWorkspace] Using cached schema version 1.291.0
2024-07-01T19:32:10.886Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:32:10.911Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:32:32.711Z [INFO][dynatrace-extensions.generateCertificates] Command called.
2024-07-01T19:32:32.726Z [INFO][utils.conditionCheckers.checkWorkspaceOpen] Is a workspace open? true
2024-07-01T19:32:32.734Z [INFO][utils.conditionCheckers.checkOverwriteCertificates] Can continue and overwrite certificates? true
2024-07-01T19:32:32.743Z [INFO][commandPalette.generateCertificates.generateCerts] Executing Generate Certificates command
2024-07-01T19:32:33.254Z [INFO][commandPalette.generateCertificates.generateCerts] Generating CA certificate
2024-07-01T19:32:33.348Z [INFO][commandPalette.generateCertificates.generateCerts] CA Cert created successfully
2024-07-01T19:32:33.360Z [INFO][commandPalette.generateCertificates.generateCerts] Generating RSA key pair for Developer certificate
2024-07-01T19:32:33.488Z [INFO][commandPalette.generateCertificates.generateCerts] Generating the Developer certificate
2024-07-01T19:32:33.576Z [INFO][commandPalette.generateCertificates.generateCerts] DEV Cert created successfully
2024-07-01T19:32:33.590Z [INFO][commandPalette.generateCertificates.generateCerts] Writing certificates to disk
2024-07-01T19:32:33.603Z [INFO][commandPalette.generateCertificates.generateCerts] Wrote all certificates at location /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates
2024-07-01T19:32:33.617Z [INFO][commandPalette.generateCertificates.generateCerts] Updating vscode settings
2024-07-01T19:33:21.526Z [INFO][dynatrace-extensions.distributeCertificate] Command called.
2024-07-01T19:33:21.537Z [INFO][utils.conditionCheckers.checkWorkspaceOpen] Is a workspace open? true
2024-07-01T19:33:21.546Z [INFO][utils.conditionCheckers.checkEnvironmentConnected] Is a tenant connected? true
2024-07-01T19:33:21.557Z [INFO][utils.conditionCheckers.checkCertificateExists] Does CA certificates exist? true
2024-07-01T19:33:21.566Z [INFO][commandPalette.distributeCertificate] Executing Distribute Certificate command
2024-07-01T19:33:21.949Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:33:21.960Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:33:30.356Z [INFO][utils.conditionCheckers.checkActiveGateInstalled] Is ActiveGate installed locally? false
2024-07-01T19:33:30.357Z [INFO][utils.conditionCheckers.checkOneAgentInstalled] Is OneAgent installed locally? > false
2024-07-01T19:33:30.369Z [INFO][dynatrace-extensions.distributeCertificate] Completed normally
2024-07-01T19:33:30.376Z [INFO][dynatrace-extensions.generateCertificates] Completed normally
2024-07-01T19:33:30.383Z [ERROR][commandPalette.initWorkspace.initWorkspace] Workspace initialization aborted due to earlier failure.
2024-07-01T19:33:30.400Z [INFO][dynatrace-extensions.initWorkspace] Completed normally
2024-07-01T19:34:03.064Z [ERROR][commandPalette.initWorkspace.initWorkspace] Cannot initialize workspace without certificates.
2024-07-01T19:34:04.357Z [INFO][commandPalette.distributeCertificate] Certificate successfully updated in the Credential Vault.

Initialize workspace the second time
However, this error doesn't mean something is wrong. The certificates were successfully generated and the ca.pem file is uploaded to the tenant.

Click Initialize workspace again
Choose Use existing certificates this time, since we've just generated them
Choose Extension 2.0

2024-07-01T19:41:24.680Z [INFO][dynatrace-extensions.initWorkspace] Command called.
2024-07-01T19:41:24.688Z [INFO][utils.conditionCheckers.checkWorkspaceOpen] Is a workspace open? true
2024-07-01T19:41:24.697Z [INFO][utils.conditionCheckers.checkEnvironmentConnected] Is a tenant connected? true
2024-07-01T19:41:24.708Z [INFO][commandPalette.initWorkspace.initWorkspace] Executing Initialize Workspace command
2024-07-01T19:41:24.728Z [INFO][commandPalette.initWorkspace.initWorkspace] Using cached schema version 1.291.0
2024-07-01T19:41:25.130Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:41:25.149Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:41:27.862Z [INFO][utils.conditionCheckers.checkSettings] Settings developerCertkeyLocation exist? true
2024-07-01T19:41:46.416Z [INFO][dynatrace-extensions.initWorkspace] Completed normally

Result: successful initialization of the extension
Build the extension

Call Dynatrace extensions: Build command
The build process goes through up to the "Validation" phase

{
  "code": 400,
  "message": "Extension signature is not valid"
}

2024-07-01T19:44:16.218Z [INFO][dynatrace-extensions.buildExtension] Command called.
2024-07-01T19:44:16.218Z [INFO][utils.conditionCheckers.checkWorkspaceOpen] Is a workspace open? true
2024-07-01T19:44:16.225Z [INFO][utils.conditionCheckers.isExtensionsWorkspace] Is this an extensions workspace? true
2024-07-01T19:44:16.230Z [INFO][utils.conditionCheckers.checkCertificateExists] Does DEV certificates exist? true
2024-07-01T19:44:16.236Z [INFO][utils.conditionCheckers.checkNoProblemsInManifest] Are there problems in the manifest? true
2024-07-01T19:44:16.243Z [INFO][commandPalette.buildExtension.buildExtension] Executing Build Extension command
2024-07-01T19:44:16.639Z [INFO][utils.conditionCheckers.checkUrlReachable] Is URL https://xxx.com/api/v1/time reachable? true
2024-07-01T19:44:16.702Z [INFO][commandPalette.buildExtension.assembleStandard] Built the inner archive: /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/extension.zip
2024-07-01T19:44:16.704Z [INFO][utils.cryptography.sign] Signing extension with credentials from /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/developer.pem
2024-07-01T19:44:16.825Z [INFO][commandPalette.buildExtension.assembleStandard] Wrote the signature file: /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/extension.zip.sig
2024-07-01T19:44:16.831Z [INFO][commandPalette.buildExtension.assembleStandard] Wrote initial outer zip at: /Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/custom_my.awesome.extension-0.0.1.zip
2024-07-01T19:44:17.263Z [INFO][dynatrace-extensions.buildExtension] Completed normally
2024-07-01T19:44:17.271Z [ERROR][] Extension validation failed.

The built and signed extension produced by the VS Code add-on can still be found in the workspace storage:
$ ll "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/"
total 32
drwxr-xr-x  6 vagiz.duseev  staff   192B Jul  1 21:44 .
drwxr-xr-x  6 vagiz.duseev  staff   192B Jul  1 22:25 ..
-rw-r--r--@ 1 vagiz.duseev  staff   6.0K Jul  1 22:30 .DS_Store
drwxr-xr-x  9 vagiz.duseev  staff   288B Jul  1 21:32 certificates
-rw-rw-rw-  1 vagiz.duseev  staff   226B Jul  1 21:44 extension.zip
-rw-r--r--  1 vagiz.duseev  staff   2.8K Jul  1 21:44 extension.zip.sig

Let's manually put extension.zip and extension.zip.sig into a zip archive
$ zip bundle.zip extension.zip extension.zip.sig
  adding: extension.zip (stored 0%)
  adding: extension.zip.sig (deflated 25%)

Uploading it to the tenant gives the same error: Extension signature is not valid.
Let's check the certificates
First, triple checked that the certificate paths that end up in settings.json are the same as in the actual folder.
{
    "dynatraceExtensions.developerCertkeyLocation": "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/developer.pem",
    "dynatraceExtensions.rootOrCaCertificateLocation": "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/ca.pem",
}

Folder contents
$ ls -la 8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates
total 64
drwxr-xr-x  9 vagiz.duseev  staff   288B Jul  1 21:32 .
drwxr-xr-x  6 vagiz.duseev  staff   192B Jul  1 21:44 ..
-rw-r--r--  1 vagiz.duseev  staff   3.2K Jul  1 21:32 ca.key
-rw-r--r--  1 vagiz.duseev  staff   1.8K Jul  1 21:32 ca.pem
-rw-r--r--  1 vagiz.duseev  staff   814B Jul  1 21:32 ca.pub.key
-rw-r--r--  1 vagiz.duseev  staff   3.2K Jul  1 21:32 dev.key
-rw-r--r--  1 vagiz.duseev  staff   1.9K Jul  1 21:32 dev.pem
-rw-r--r--  1 vagiz.duseev  staff   814B Jul  1 21:32 dev.pub.key
-rw-r--r--  1 vagiz.duseev  staff   5.1K Jul  1 21:32 developer.pem

Try to build and sign the extension using dt-cli
Let's try to reproduce the process with dt-cli using the same generated certificates.
# Assemble the archive
$ dt ext assemble --src extension/ --output dist/extension.zip
Building dist/extension.zip from extension
Adding file: extension/extension.yaml as extension.yaml

# dt-cli refuses to sign an extension with `developer.pem` file from above because the file has a read permission for groups+all. Similarly to SSH keys, these certificates, ideally, would only be readable by the owner (`0400` permissions).
# Let's set correct permission for developer.pem file.
$ chmod 0400 "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/developer.pem"

# Sign the bundle
$ dt ext sign --src dist/extension.zip --output dist/bundle.zip --key "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/developer.pem"

Let's now upload that signed bundle to the tenant.

If we do that manually, the tenant will report an error: Extension signature is not valid.
Same happens if we validate the signed bundle through dt-cli below (the output is very ugly and scary and should be refactored to avoid stack trace printing).

$ dt ext validate dist/bundle.zip --tenant-url https://xxx.com/ --api-token dt0c01.xxx
Extension validation failed!
Traceback (most recent call last):
  File "/Users/xxx/Projects/dt-cli/dtcli/server_api.py", line 32, in validate
    response.raise_for_status()
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/requests/models.py", line 1021, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://xxx.com/api/v2/extensions?validateOnly=true

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/dt", line 6, in <module>
    sys.exit(main())
  File "/Users/xxx/Projects/dt-cli/dtcli/__main__.py", line 5, in main
    dtcli.scripts.dt.main()
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/xxx/Projects/dt-cli/.venv/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/Users/xxx/Projects/dt-cli/dtcli/scripts/dt.py", line 159, in wrapper
    return func(*args, **kwargs)
  File "/Users/xxx/Projects/dt-cli/dtcli/scripts/dt.py", line 747, in validate
    server_api.validate(extension_zip, kwargs["tenant_url"], kwargs["api_token"])
  File "/Users/xxx/Projects/dt-cli/dtcli/server_api.py", line 36, in validate
    raise dtcliutils.ExtensionValidationError(response.text)
dtcli.utils.ExtensionValidationError: {"error":{"code":400,"message":"Extension signature is not valid"}}

Verify the signature using OpenSSL
Verification of the signature generated by VS Code add-on with the certificates generated by VS Code add-on above:
openssl cms -verify -CAfile "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/ca.pem" -in "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/extension.zip.sig" -binary -content "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/extension.zip" -inform PEM -out /dev/null
CMS Verification failure
000C3D0302000000:error:17000064:CMS routines:cms_signerinfo_verify_cert:certificate verify error:crypto/cms/cms_smime.c:289:Verify error: certificate is not yet valid

Same procedure but for the bundle signed with dt-cli using the same certificates (generated by VS Code add-on):
$ openssl cms -verify -CAfile "/Users/xxx/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/ca.pem" -in dist/bundle/extension.zip.sig -binary -content dist/bun
dle/extension.zip -inform PEM -out /dev/null
CMS Verification failure
000C3D0302000000:error:17000064:CMS routines:cms_signerinfo_verify_cert:certificate verify error:crypto/cms/cms_smime.c:289:Verify error: certificate is not yet valid

The validity dates of the generated certificates are not exactly same, although I'm not sure how that affects the process. The output below is generated with `openssl x509 -noout -text -in "<path-to-cert.pem>"
ca.pem:
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            4b:12:1c:16:69:f3:ab:04:87:ab:b5:be:08:ab:89:d0:f1:35:72:07
        Signature Algorithm: sha256WithRSAEncryption
        Issuer: CN=Extension Developer Root
        Validity
            Not Before: Jun 30 19:32:33 2024 GMT
            Not After : Jul  1 19:32:33 2027 GMT
        Subject: CN=Extension Developer Root

dev.pem:
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            72:43:b7:da:a6:e9:32:74:ca:76:c8:25:14:96:e5:d1:b9:a5:96:44
        Signature Algorithm: sha256WithRSAEncryption
        Issuer: CN=Extension Developer Root
        Validity
            Not Before: Jul 29 19:32:33 2024 GMT
            Not After : Jul  1 19:32:33 2030 GMT
        Subject: CN=Extension Developer Dev

developer.pem:
openssl x509 -noout -text -in "/Users/vagiz.duseev/Library/Application Support/Code/User/workspaceStorage/8a857babd1976b2dedff29e86f7ac03e/DynatracePlatformExtensions.dynatrace-extensions/certificates/developer.pem"
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            72:43:b7:da:a6:e9:32:74:ca:76:c8:25:14:96:e5:d1:b9:a5:96:44
        Signature Algorithm: sha256WithRSAEncryption
        Issuer: CN=Extension Developer Root
        Validity
            Not Before: Jul 29 19:32:33 2024 GMT
            Not After : Jul  1 19:32:33 2030 GMT
        Subject: CN=Extension Developer Dev

Question: why do we need an intermediate dev.pem?
How to fix
Manually overwrite the deployed CA certificate with the dev.pem instead.
Support information

[ ] Screenshots
[ ] Extension manifest file
[ ] YAML snippets
[X] Support archive (see here)

I can't reproduce this. Let's discuss offline.


Sound files can not be removed from a Pad

Hey friend. I'm loving Controlcast at the moment. I just have one issue. Every time I make a Pad an audio sample. If I then try and change the audio file to something else (a different audio file) it'll just play the first music file again. Seems like clearing the pad does not clear the audio file path. Even once I try and put a new one in. Any help would be appreciated!

Clear the audio, add a new file and save. Then restart CC. Should be correct after that.


Fix Redshift  tests

Describe the bug
In CI there are flaky tests, we need to fix -
https://github.com/astronomer/astro-sdk/actions/runs/3197623724/jobs/5221086373
Version

Astro: [e.g. 1.1.1]

To Reproduce
Steps to reproduce the behavior:
Run CI multiple times you'll see CI jobs failing.
Expected behavior
An unrelated CI job should not fail.

recently also failed with same error https://github.com/astronomer/astro-sdk/actions/runs/3523069879/jobs/5906787666

@pankajastro it would be good to add the findings you were mentioning in this thread as well


Prevent HomeKit enabled Meross devices from being bridged

Please explain your feature request in a one or two sentences.

I have a mix of Meross devices such that some came with HomeKit support and some don't. When using the plugin via Meross cloud, all devices are bridged by Homebridge, even those already with HomeKit support, causing duplicates to show up.

Is your feature request related to a problem? Please describe.

Yes. I was hoping that I would not see duplicate devices.

Any particular Meross devices that this relates to?
The Meross 3 way switches have a version that works with HomeKit, specifically MSS550X. A list in configs of serials to not bridge would be appreciated, or an automated form of detecting whether or not a device supports HomeKit already would work too.

Anything else?

Thanks for the hard work!

Hi @Conanap can you get me a list all the homekit native devices you, specifically with their model number and the hardware version as shown in the homebridge ui? (in the my devices tab of plugin settings).
Thanks!

For sure! I only have 1 item that I use for now.
Model Number: mss550x
hardware version: 4.0.0
Let me know if you need other information, thanks!


Donate menu item goes to /#donate hashlink

Hash link on homepage still not working - donate menu link should go here.


[wip] Use Json type to represent naked dictionaries

Tracking issue
https://github.com/flyteorg/flyte/issues/5318
Why are the changes needed?
As described in https://github.com/flyteorg/flyte/issues/5318, we can use a json string as the transport for a variety of types, including naked dictionaries.

What changes were proposed in this pull request?
Leverage the new Json type defined in https://github.com/flyteorg/flyte/pull/5337 to replace the usage of protobuf Struct as the transport for naked dictionaries with a json-encoded string.
Keep in mind that this is WIP. TODO:

[ ]  define how custom encoders are passed in, if necessary,
[ ] make this backwards compatible with the old Struct
[ ] fix+add tests

How was this patch tested?

Setup process
Screenshots
Check all the applicable boxes

[ ] I updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] All commits are signed-off.

Related PRs

Docs link

msgpack-binary idl subsumes this.


Svelte 5 support

Is this compatible with Svelte 5?

The latest version is now compatible.

Thank you!


DNS error

Hi,
So our DNS/DC is on Azure, and i'm trying to join an Ubuntu machine that's on AWS to the domain.
How can I troubleshoot the below error?  Is there a log file somewhere?
Error: DNS_ERROR_BAD_PACKET [code 0x0000251e]
A bad packet was received from a DNS server. Potentially the requested address does not exist.

Can the Ubuntu machine ping the AWS domain?

Hi @rbest-bt
not 100% what you're asking.
Domain Controller = Windows on Azure
Client (that wants to join) = Ubuntu 16.04 on AWS
The ubuntu machine is on the internet and DNS is working, can ping domains fine.

We do not have an azure setup at this time. It's in the backlog to test against.

Some things that could pose an issue. Are the systems on the same network? Is there a NAT router in the way? Can you resolve the actual IP of the domain or an external IP?

Hi @rbest-bt
I resolved this issue by adding "dns-nameserver" as the domain controller in /etc/network/interfaces before attempting "domainjoin-cli".
Not sure what the network consists of in Azure, but the 2 servers were in the same subnet - i.e. Domain Controller = <IP_ADDRESS>, Server to join = <IP_ADDRESS> (Example addresses).

Thanks for posting your solution.


build figures from source, use non-proprietary source format

The src/figs/*.pdf files were created from src/graffles/*.graffle but the build system in build/Makefile does not rebuild the PDF figures when the OmniGraffle files change.
I would recommend to:

remove the prebuilt PDF figures from the git repository
switch from the proprietary OmniGraffle file format as source for your figures to an open format like SVG that can be converted to PDF at build time.

It looks like OmniGraffle can export to SVG, which can then be edited in vector graphics editors like Inkscape and then Inkscape can convert to PDF for import into the TeX document:
https://en.wikipedia.org/wiki/OmniGraffle
https://tex.stackexchange.com/questions/2099/how-to-include-svg-diagrams-in-latex

We included the PDFs and don't automatically regenerate them because we don't expect people to have OmniGraffle installed.
There's loss of metadata when exporting from .graffle to .svg, so we'd still need two sets of files committed.

By sticking to OmniGraffle, you also prevent people not using it from improving/adjusting the diagrams or adding new ones.
I'm not very familiar with TeX diagrams, but is there any other non-proprietary source format that would preserve the mentioned metadata so that you could use it instead of OmniGraffle?
I'd like to make RISC-V a first-class architecture in Debian, including having the architecture specifications available to read offline. Unfortunately due to the use of the proprietary OmniGraffle format instead of a more open format, this means the riscv-isa-manual package would have to be put into Debian contrib instead of Debian main.

Apparently @kasanovic has authored the only three graffles, so he should make the call.
Are the PDF figures themselves a problem for debian main, or is that a matter of your preference?

I'm not sure if it is written down anywhere but the Debian ftp-master's policy is that everything in main must be buildable from the source format using only other packages from main. The package build process is allowed to skip building them (if they take days to build for example) but they must be buildable.

I guess the question is, is PDF an acceptable source format for images?  I have to assume that some binary formats are OK, e.g. bitmaps, at least for legacy reasons.

The source is the form that you as the upstream developers prefer for creating/modifying the figures. In this case it is clearly the OmniGraffle format that is your currently preferred format and the PDF form is clearly not.

Yeah, that makes sense.


Devise doesn't use scoped errors for scoped views

Environment

Ruby 2.6.2p47
Rails <IP_ADDRESS>
Devise 4.6.1

Current behavior
When I generate views for a user resource, they have the errors rendered as
render "devise/shared/error_messages", resource: resource

in place of the
render "users/shared/error_messages", resource: resource

as is the case with shared links.
Expected behavior
Views should use the scoped view to render errors, changing all
render "devise/shared/error_messages", resource: resource

to
render "users/shared/error_messages", resource: resource

for a user resource.

I've pinpointed the problem, and fixed it, can someone tell me how to test the changes?

I've submitted a PR a month ago. Can someone please review it so it can be merged?

@shobhitic Sorry about that, I was not able to review it yet but I'll try to do it soon.

@tegon can you please go through the review if you have time now? There's about 10 lines of changes!

Can someone please review the attached PR?

@shobhitic Sorry about that, I'm solving some other things I have pending here and haven't had the time to review this yet. But I'll do it as soon as I can.


Correction for raw affiliation Laboratory of Glasses and Ceramics, Institute of Chemical Science, University of Rennes 1, UMR CNRS 6226, Rennes, France

Correction needed for raw affiliation Laboratory of Glasses and Ceramics, Institute of Chemical Science, University of Rennes 1, UMR CNRS 6226, Rennes, France
raw_affiliation_name: Laboratory of Glasses and Ceramics, Institute of Chemical Science, University of Rennes 1, UMR CNRS 6226, Rennes, France
new_rors: 015m7wh34;02feahw73;00adwkx90
previous_rors: 015m7wh34;02feahw73
works_examples: W4376472742;W4386476466
contact: c7cc5f7fdc09011867a4867e7483f826:5505a2c4e4b84a0213c26de342912f3c @ univ-rennes.fr

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


Create and implemente an URI-Scheme for inexor

Create an URI Scheme for inexor, so if you click a link in your browser (for example inexor://example.com:30000) inexor opens and connects to example.com, port 30000.
Further explanation: https://msdn.microsoft.com/en-us/library/aa767914(v=vs.85).aspx

Linux: https://askubuntu.com/questions/514125/url-protocol-handlers-in-basic-ubuntu-desktop

First I thought this us useless but I think it could be pretty useful indeed!

Red Eclipse reference: https://github.com/red-eclipse/base/pull/633

This'll be really usefull in terms that flex will be in charge of actually providing informations about servers .. no more sneaky command line stuff .. you'll receive a "join tree" object, which tells your client where to go, and how the server's set-up, yiha!

I've found this gist which describes how we can implement URL schemes in flex using Node JS.
On linux, we could just use the xdg-open command from command line (or .desktop files) to start a server (assuming Inexor Flex is running):

xdg-open inexor://server/start/31415

The Steam browser protocol could be an inspiration what would be possible (Inexor Flex could be the open source competitor of the steam client).

@aschaeffer Didn't you merge something like this already in master?

Yes it's in the master branch of flex.


How to show incoming call notification in android application ?

I'm trying to build one to one video calling Mobile App using react-native-agora.
I'm able to join to call one to one perfectly.
Now i'm trying to show the incoming call notification on android app.
Any idea to show incoming call notification on android app.
My dependencies
"react": "16.8.3",
"react-native": "0.59.3",
"react-native-agora": "2.4.0-alpha-1",

@nitheeshskiploop same problem here . have you find its workaround

It seems you need integrate some notification component, you can try to use firebase or aws cloud component

@varunsdcet i have used firebase notification component.

I will close this issue because it is not active. if there are still problems, you can reopen this issue.


exclude deleted elemets from sorting

eg. for this course: https://en.serlo.org/43357/welcome
there are several deleted course pages on the sorting page: https://en.serlo.org/entity/link/order/43238/link
imo they don't have to be there

General information:

You get to the specified page by using the settings icon in any course and going to Whole Course > Order children where you can change the order of the children (i.e. course pages in this case)

To approach this:

The template for the sorting page is defined in https://github.com/serlo-org/athene2/blob/master/src/module/Ui/templates/entity/link/order.twig (i.e. in src/module/Ui/templates and then mostly following the stuff in the url ;))
Currently, we are iterating over all children, https://github.com/serlo-org/athene2/blob/master/src/module/Ui/templates/entity/link/order.twig#L20
We can either filter the children in the templates directly, or (imho better) in its controller: https://github.com/serlo-org/athene2/blob/master/src/module/Entity/src/Entity/Controller/LinkController.php#L83.

Closed by #752