Fix textencoder uninitialized variable

Final_len was uninitialized if input string was empty. Doesn't cause issues on Linux gcc or regression test coverage, but did cause issues with MSVC.
Fixes #1293.

@fatcerberus Since the original issue is not caught by automatic regression testing (compiler luck I guess), could you check if the fix here works in MSVC and your specific options?

Sure, I'll test it out in a bit.

Yep, that fixed it.  Thanks :)
What I can't figure out is that "invalid args" error.  If anything it should just create a huge unbacked buffer like you say in https://github.com/svaarala/duktape/issues/1293#issuecomment-272705932, but instead it's throwing an error.

Huh, this bug produces really weird symptoms.  On x86 the thrown error is different:
C:\src\spectacles-i>C:\src\minisphere\msw\cell -bd dist
Cell 5.0a1 Sphere packaging compiler (x86)
the JavaScript-powered compiler for Sphere v2
(c) 2015-2017 Fat Cerberus

evaluating Cellscript.js...
building targets...
    transpile script @/lib/map/index.js
        TypeError: buffer required, found [object Uint8Array] (stack index 1)
        @ [#/cell_modules/transpile.js:27]
writing game.json... OK.

Thanks for reporting the bug :) Always difficult to find if automatic regressions don't catch them. One more case where running the test set automatically on Windows too would be nice. Now that's a bit easier tooling-wise so maybe that coverage can be achieved in the coming months.
Just to be sure, was that x86 issue also fixed here?

Cell 5.0a1 Sphere packaging compiler (x86)
the JavaScript-powered compiler for Sphere v2
(c) 2015-2017 Fat Cerberus

evaluating Cellscript.js...
building targets...
    transpile script @/lib/map/index.js
writing game.json... OK.

Indeed it is. 😄

@svaarala I'd like to do a release soon (in the next week or so) with ES2015 support; what's the timeframe on Duktape 2.0.1?

I can do it anytime - I've just been waiting for any small fixes to crop up for a bit. But it's not a very heavy process to release a 2.0.2 if more fixes crop up.

Alright, I'll keep an eye out and let you know when I'm ready to release.  For now I can just build against master in-house.

Sounds good - it takes a few hours to do a release so couple of days notice is more than enough.


Enhancement: Update to league/container:^2.0.2

This PR

[x] requires league/container:^2.0.2
[x] applies fixes to work with it

See https://github.com/thephpleague/container/releases/tag/2.0.2.
:information_desk_person: It's been released just a minute ago!

Yay!

@tomphp
Hang on before you tag, I forgot to update README.md.


How can we help?
None
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: https://filezilla-project.org/download.php?platform=win64
Publisher:filezilla-project.org
Package Name:filezilla
Description: free ftp client
Package Version: 3.66.4
Installer URL:https://filezilla-project.org/download.php?platform=win64

Hello,
This application has been blocked by the vendor so we are unable to add it to Winget.

More information in #78578


LIME-243 - Extend VC Validity to 6 months

Proposed changes
What changed

Expiry time increased from 2 hours to 6 months

Issue tracking

LIME-243

See also: https://github.com/alphagov/di-ipv-cri-fraud-api/pull/91

Closing pull request will be picked up in a separate ticket


[Bug] Github.io site not showing video on Firefox

at https://blazored.github.io/Video/
the resource https://res.cloudinary.com/blazoredgitter/video/upload/v1557015491/samples/elephants.mp4 is mentioned, but I see it saying it doesn't support video format or MIME type (see picture on actual wording at Firefox/Win11)

Cloudinary became problematic, so that video is no longer there. I'll
rebuild the sample site when I get a chance.
If you want to test it, you can just download the repo and run the samples
On Fri, 10 Mar 2023 at 08:56, George Birbilis @.***>
wrote:

at https://blazored.github.io/Video/
the resource
https://res.cloudinary.com/blazoredgitter/video/upload/v1557015491/samples/elephants.mp4
is mentioned, but I see it saying it doesn't support video format or MIME
type (see picture on actual wording at Firefox/Win11)
[image: image]
https://user-images.githubusercontent.com/3461504/224270308-b6f523e7-5022-4337-91a1-a7fdb6937fa2.png
—
Reply to this email directly, view it on GitHub
https://github.com/Blazored/Video/issues/25, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWQFPV35SPSIIPOS6NNF4LW3LUE7ANCNFSM6AAAAAAVWEZ5XM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


How do I just do vcpkg install packages && cmake -DCMAKE_TOOLCHAIN_FILE=%vcpkg%\scripts\buildsystems\vcpkg.cmake"?

Maybe I'm overcomplicating things, but I want to do:
  build_windows:
    runs-on: windows-latest
    steps:
      - uses: actions/checkout@v2
      - name: install dependencies
        run: |
          git clone --depth=1 https://github.com/Microsoft/vcpkg.git
          .\vcpkg\bootstrap-vcpkg.bat
          .\vcpkg\vcpkg install libarchive
      - name: build
        run: |
          md cmake-build-debug
          cd cmake-build-debug
          cmake .. -DCMAKE_TOOLCHAIN_FILE="..\vcpkg\scripts\buildsystems\vcpkg.cmake"
          cmake --build .
        shell: cmd
      - name: test
        run: ctest .
        working-directory: cmake-build-debug

Except I want to cache everything vcpkg related. Happy to pin https://github.com/Microsoft/vcpkg to a specific commit hash.
How do I do this with run-cmake, without introducing vcpkg as a submodule?
(also preferably without a vcpkg.json)
Thanks,
PS: I tried one the examples with submodule ❌'d on https://github.com/marketplace/actions/run-vcpkg#samples… but it still required a submodule

@SamuelMarks if i get this right, this is the question: "how can I implement a pure workflow without using run-vcpkg (with or without run-cmake) and without having vcpkg as a submodule?"
I think the generic answer is to leverage actions/checkout.
A concrete example is in the pure workflow at lukka/CppCMakeVcpkgTemplate: note that this workflow is using vcpkg as a submodule, hence all you have to do in addition to that is to ensure the sources are on disk before they are going to be used (which is already nicely done in the workflow you shown in the question).

@lukka Yeah I'm trying to put it together https://github.com/offscale/libacquire/blob/1cc1da3/.github/workflows/github-actions.yml#L47-L99 (previous commit I tried using checkout)
Happy to use run-vcpkg, but I don't want vcpkg as a submodule. Also would prefer not to have a vcpkg.json.

@SamuelMarks closing stale issues. Feel free to continue discussion if needed.

@lukka My question remains, how do I replace my vcpkg cache hack with your run-cmake without adding a submodule?
(for bonus points: manifest mode caching)

@SamuelMarks you can use run-vcpkg with an explicit value for vcpkgDirectory input that points to the file system location where you want vcpkg to be checked out. If it is not a submodule, it will be automatically checked out in that location.
You also need to specify the Git commit id in vcpkgGitCommitId, which would not be needed if vcpkg was a submodule.
Guess other details are missing that depends on your requirements, but to give an idea:

if vcpkg is going to be launched by CMake (e.g. by run-cmake or by plain invocation to cmake -DVCPKGTOOLCHAIN, everything should be working fine, but you need to double check the right locations are automatically cached.
if vcpkg is run by run-vcpkg itself (i.e. runVcpkgInstall input is true), then something else (not sure what) must be done that needs to be figure out.

Glad to help further if you have a workflow to reason on.

Closing this issue as stale since long time, let me know if you need any support @SamuelMarks, thanks!


Add AWS Auth

Waiting for #366

Hey @Rugvip
Are you referring to AWS Cognito, the service they provide?
Or to be able to authenticate with your AWS account? AFAIK they do not support OAuth2

AWS Cognito seems to be what Amazon recommends for API access as well: https://docs.aws.amazon.com/sdk-for-javascript/v2/developer-guide/loading-browser-credentials-cognito.html. Another alternative would be to support Login With Amazon, which looks like a flavor of OAuth, https://developer.amazon.com/docs/login-with-amazon/web-docs.html, https://docs.aws.amazon.com/sdk-for-javascript/v2/developer-guide/loading-browser-credentials-federated-id.html.
Not too familiar with AWS API access and what works best and is most accessible, so looking for feedback on that.

@jaime-talkdesk are you working on this? We are really interested in getting AWS auth support in Backstage and would greatly appreciate any help.

I will tackle it after Auth0 PR but for now is free for grab. If no one takes in the meantime, I will tackle it

@Rugvip i have seen other Opensource to implement https://github.com/dexidp/dex to handle different providers. What is the use case for this? we are using AWS

@carlsendk Afaik dex is for the use-case of being an OIDC provider. We actually have ended up implementing something similar in the auth-backend as a PoC, where we issue out own Backstage Identity Tokens. It's a slightly different use-case though and I'm not sure it's a good idea to handle bridging of other services using that. We did look at dex early on but ended up not considering when we also decided to write the backend in TypeScript rather than Go.
What we're looking for here is a more straight-forward OAuth Client Application implementation where the user of Backstage is a Resource Owner and wants to delegate access of AWS resources to a Backstage plugin. On top of that we probably end up wanting to be able to use AWS as a sign-in provider, but as mentioned above we have a separate solution in place for that for now.

Cognito have some limitations last time we looked at it on how much you get in the header information. So we had to take another route. But we do have some people who have spent a lot time on this. So if you need more information ask next time your are chatting with the team.
we will at some point have something in backstage that do something in AWS, but first we need to get our okta flow working with backstage.

@carlsendk We're working on an AWS Lambda plugin and obviously need to authenticate with AWS for that. The best solution we could come up with so far is described in https://github.com/spotify/backstage/issues/1988#issuecomment-675559584 If you get a chance, would you mind taking a look so we know we're on the best path?

Just to line up, @dtuite I think Roadie is working at least somewhat towards this with https://github.com/RoadieHQ/backstage-plugin-aws-auth ?

Just to line up, @dtuite I think Roadie is working at least somewhat towards this with https://github.com/RoadieHQ/backstage-plugin-aws-auth ?

@adamdmharvey
We created a solution to access AWS services without cognito, just by the regular IAM users for our aws-lambda plugin. To prevent leaking the access token in the frontend source code and the need for the user to authorize theyself each time they want to acces AWS, we created a separate backend plugin (aws-auth) that stores the access key of the IAM user and frontend app (any future plugin that would have to authorize in the AWS services) can fetch it from there on demand.

@adamdmharvey
We created a solution to access AWS services without cognito, just by the regular IAM users for our aws-lambda plugin. To prevent leaking the access token in the frontend source code and the need for the user to authorize theyself each time they want to acces AWS, we created a separate backend plugin (aws-auth) that stores the access key of the IAM user and frontend app (any future plugin that would have to authorize in the AWS services) can fetch it from there on demand.

@Rugvip #366 is closed; is there something else this is waiting on?

@joebowbeer thanks, will close this as it's not specific enough. We should open tickets for individual AWS auth technologies if they are missing.


How to prohibit maximize window by double clicking toolbar or area within "DragToMoveArea"?

version :  window_manager: ^0.2.2 on win10
I want to prevent user to resize app, my main code like this :
void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  if (Platform.isWindows) {
    await WindowManager.instance.ensureInitialized();
    windowManager.waitUntilReadyToShow().then((_) async {
      await windowManager.setTitleBarStyle(
        TitleBarStyle.hidden,
        windowButtonVisibility: false,
      );
      await windowManager.setSize(const Size(755, 545));
      await windowManager.setMinimumSize(const Size(755, 545));
      await windowManager.setPreventClose(true);
      await windowManager.setSkipTaskbar(false);
      await windowManager.setResizable(false);
      await windowManager.setMinimizable(true);
      await windowManager.center();
      await windowManager.focus();
      await windowManager.show();
    });
  }
...

And my home page is  wrapped within "DragToMoveArea". But the app can still be maximized by double clicking any area of the page.
By the way, as there is setMinimizable, why there is no setMaximizable?

  return GestureDetector(
      behavior: HitTestBehavior.translucent,
      onPanStart: (details) {
        windowManager.startDragging();
      },
      onDoubleTap: () async {
        // bool isMaximized = await windowManager.isMaximized();
        // if (!isMaximized) {
        //   windowManager.maximize();
        // } else {
        //  windowManager.unmaximize();
        // }
      },
      child: child,
    );

But when dragging, the mouse icon would be duplicated.

I don't understand what you mean, do you have a screenshot?

It is strange that this phenomena disappeared today.


json createDataDictionary overwrite

I'm opening two logfiles (_event and _stim) consecuitively. When I only open the logfile (_event) with extraColumn, createDataDictionary nicely creates a json file with all the extra columns.
Then when I open the following, _stim, that json file for the logfile / _event got back to its default settings without the extraColumns.
Any idea why is it happening?
    % create  logfile with extra columns to save - BIDS
    logFile.extraColumns = cfg.extraColumns;
    [logFile]  = saveEventsFile('open', cfg, logFile); %dummy initialise

    % set the real length of columns
    logFile(1).extraColumns.LHL24.length = 12;
    logFile(1).extraColumns.PE4.length = 12;

    % actual inititalization
    logFile = saveEventsFile('open', cfg, logFile);

    % create response file - used for counting button press
    responseFile.extraColumns = cfg.responseExtraColumns;
    responseFile  = saveEventsFile('open_stim', cfg, responseFile);

Will look into this: will most likely write a test for that.
Trying to figure out on which branh / version you are working: what is the latest commit of CPP_BIDS when you do  git log ?

I can see the issue is for both _event and _stim initialisation, it creates the same file name json. If it's not required to create json dictionary for _stim, it would not over write the file.
function logFile = initializeFile(cfg, logFile)

    logFile = initializeExtraColumns(logFile);

    createDataDictionary(cfg, logFile);

    % Initialize txt logfiles and empty fields for the standard BIDS
    %  event file
    logFile(1).fileID = fopen( ...
        fullfile( ...
        cfg.dir.outputSubject, ...
        cfg.fileName.modality, ...
        logFile.filename), ...
        'w');

    % print the basic BIDS columns
    fprintf(logFile(1).fileID, '%s\t%s\t%s', 'onset', 'duration', 'trial_type');
    fprintf(1, '%s\t%s\t%s', 'onset', 'duration', 'trial_type');

    printHeaderExtraColumns(logFile);

    % next line so we start printing at the right place
    fprintf(logFile(1).fileID, '\n');
    fprintf(1, '\n');

end

Here is my latest CPP_BIDS git log:

(base) mac-114-168:CPP_BIDS battal$ git log
commit 17ab12d096b9611a0dd62c6136b7d90285ee7ab7 (HEAD, CerenB/master, master, ceren_mod)
Merge: bc0aaa6 962c947
Author: Ceren<EMAIL_ADDRESS>Date:   Mon Aug 31 12:34:40 2020 +0200

    Merge tag 'v1.0.0'

commit 962c947fe38094da9561eeba5daa44993505f2c0 (tag: v1.0.0)
Merge: f6e3b4c febf3e6
Author: Remi Gau<EMAIL_ADDRESS>Date:   Sat Aug 22 17:20:21 2020 +0200

    Merge pull request #85 from cpp-lln-lab/remi-update_randomize

    fix typos in cfg files for mh_linter

Actually createDataDictionary adapts the name of the json to the name of the tsv you give it. It removes the .tsv extension and puts the .json instead.
And because you open the 2 files with 'open' or 'open_stim' they will have different .tsv filename, so they will have distinct json names.
    logFile = saveEventsFile('open', cfg, logFile);
    responseFile  = saveEventsFile('open_stim', cfg, responseFile);

I think I am able to reproduce the error.

Actually createDataDictionary adapts the name of the json to the name of the tsv you give it. It removes the .tsv extension and puts the .json instead.
And because you open the 2 files with 'open' or 'open_stim' they will have different .tsv filename, so they will have distinct json names.
    logFile = saveEventsFile('open', cfg, logFile);
    responseFile  = saveEventsFile('open_stim', cfg, responseFile);

OK and my message describes the theory not what it does in practice: "and that's why, kids, you should write tests for your code to make sure it does what you think it does!"

I think the issue is with open_stim, it goes to createDataDictionary and re-writes the json file name with the _event file name. See from createDataDictionary.m
    fileName = strrep(cfg.fileName.events, '.tsv', '.json');

yup I am fixing createDataDictionary to take the filename of the logfile instead.
also added tests to make sure it actually creates the right json.


Revert "name is not a thing"

:v:
/cc @zendesk/vegemite
Description
This reverts commit c51079ff7cf84fa0b04ab9b0a113308079d39702.
Make name  a thing again!
https://github.com/zendesk/zendesk_apps_tools/pull/137#issuecomment-260105744
References

JIRA:

Risks

won't work this way, need to use package.original_manifest I think

I see there is a manifest class now. Which is def 👍
I added name to the class: https://github.com/zendesk/zendesk_apps_support/pull/141

ok, in that case this pr is blocked behind bumping and releasing zas


fix(setup-window-mock): reset in after hook

I have some tests which indirectly depend on the default state of the global window mock (all imported via ember-window-mock) and they started failing after I mutated that state in an unrelated test.
While I technically could and probably should (?) call setupWindowMock in the failing tests as well, I think it would be easiest and safest, if every test module would clean up after itself and reset the global window mock back to its default state.

@simonihmig Unless you know of a reasonably performant way to assert that the window mock is pristine, this PR is ready to be merged. ☺

Released as 0.5.2!


Add a fuelup upgrade command

This command will run fuelup self update and fuelup update. The idea is to reduce the number of commands users need to run.

added in #625


Application shortcut removed each time installing Atom on Windows

Description
It's just a small issue but it seems, as if each time in Update Atom, any existing shortcuts to Atom get removed or are no longer valid.
Steps to Reproduce

Install Atom
Create a shortcut based on the newly created Atom icon
Update Atom the the Shortcut is gone

Versions
Windows 7 x64
All the way up to Atom 1.6.2

I've noticed this as well. It isn't a huge problem, but more of an annoyance that I have to remove and replace the shortcut whenever Atom updates.

This should be fixed in Atom 1.7.0.  Sorry for the inconvenience.


Janky config and tool balance

Config jank

Changing mitts maximum durability makes no difference. Vanilla iron tongs still have a config entry and there is no ingame config. Config is properly reading changes to manually added cold items. Config is also poorly formatted

Balance

Mitt durability is MUCH too strong for how simple they are to make 12000 the same as colored steel. Wrought iron has less durability than wooden tongs 3000 for iron and 4000 for wooden.

Resolved in #2 (good job me!)


Added additional admin functionality to sortedm2m to preserve author order

This PR addresses issue #578
Before the author order of a given publication would be saved into the database in a random order. This would not be a problem in a case where the order didn't matter; however, in author order matters within the academic world, calling for a way to preserve the initial order that the authors were added in.
Using the through parameter of many to many fields, an intermediary model was added to save the relationship of the Publication model with the Person model. With the addition of the intermediary model, we can now manage the order that the authors (person models) were added to the publications via foreign keys and an integer field. Django will now save the order as a model.

This is great, thanks @johnsonkuang. After this PR is merged (which will have to wait until @mechanicjay is back from vacation because this updates the database schema), we will also have to go back through all publications currently in the database and properly fix their author order.

Forgot to upload images of what the new UI would look like.
Here is an image of the new admin interface:

as you can see, you can specify the order of the authors via an order integer field
One down side, however, that I think results from the way we're loading the js is that the order is set once you added the publication, changing the order after saving will not affect the way that it displays on the website

Hmm. A few things.
First, why can’t we just use the multi-select widget before and track the author order based on the order in the widget.
Second, I don’t understand the downside you just stated.
Sent from my iPhone

On Aug 28, 2018, at 11:26 AM, johnsonkuang<EMAIL_ADDRESS>wrote:
Forgot to upload images of what the new UI would look like.
Here is an image of the new admin interface:
as you can see, you can specify the order of the authors via an order integer field
One down side, however, that I think results from the way we're loading the js is that the order is set once you added the publication, changing the order after saving will not affect the way that it displays on the website
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@jonfroehlich

The multi-select widget is something that can only be used with an unmodified Many-to-many field. By specifying extra information such as order, you run into this error
<class 'website.admin.PublicationAdmin'>: (admin.E013) The value of 'fieldsets[0][1]["fields"]' cannot include the ManyToManyField 'authors', because that field manually specifies a relationship model.

Once you've saved a publication you can't change the order around because it won't do it

I think #2 is a deal breaker for this approach tbh. People will inevitably
make mistakes and need to change order.
On Tue, Aug 28, 2018 at 1:59 PM johnsonkuang<EMAIL_ADDRESS>wrote:

@jonfroehlich https://github.com/jonfroehlich

The multi-select widget is something that can only be used with an
unmodified Many-to-many field. By specifying extra information such as
order, you run into this error
<class 'website.admin.PublicationAdmin'>: (admin.E013) The value of
'fieldsets[0][1]["fields"]' cannot include the ManyToManyField 'authors',
because that field manually specifies a relationship model.
2.
Once you've saved a publication you can't change the order around
because it won't do it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jonfroehlich/makeabilitylabwebsite/pull/585#issuecomment-416738549,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABi-9TEUTH6V62VYNEx3lm5E7ZGH7OBnks5uVa8-gaJpZM4WOhy6
.

--
Jon Froehlich
Associate Professor
Paul G. Allen School of Computer Science & Engineering
University of Washington
http://www.cs.umd.edu/~jonf/
http://makeabilitylab.io
@jonfroehlich https://twitter.com/jonfroehlich - Twitter

☹
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Jon Froehlich<EMAIL_ADDRESS>Sent: Tuesday, August 28, 2018 2:30:16 PM
To: jonfroehlich/makeabilitylabwebsite
Cc: johnsonkuang; Mention
Subject: Re: [jonfroehlich/makeabilitylabwebsite] Add model to save the author order (#585)
I think #2 is a deal breaker for this approach tbh. People will inevitably
make mistakes and need to change order.
On Tue, Aug 28, 2018 at 1:59 PM johnsonkuang<EMAIL_ADDRESS>wrote:

@jonfroehlich https://github.com/jonfroehlich

The multi-select widget is something that can only be used with an
unmodified Many-to-many field. By specifying extra information such as
order, you run into this error
<class 'website.admin.PublicationAdmin'>: (admin.E013) The value of
'fieldsets[0][1]["fields"]' cannot include the ManyToManyField 'authors',
because that field manually specifies a relationship model.
2.
Once you've saved a publication you can't change the order around
because it won't do it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jonfroehlich/makeabilitylabwebsite/pull/585#issuecomment-416738549,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABi-9TEUTH6V62VYNEx3lm5E7ZGH7OBnks5uVa8-gaJpZM4WOhy6
.

--
Jon Froehlich
Associate Professor
Paul G. Allen School of Computer Science & Engineering
University of Washington
http://www.cs.umd.edu/~jonf/
http://makeabilitylab.io
@jonfroehlich https://twitter.com/jonfroehlich - Twitter
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/jonfroehlich/makeabilitylabwebsite/pull/585#issuecomment-416747378, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfdHHEpTpXPQnyYcsmwC95_SdamZumSeks5uVbZogaJpZM4WOhy6.

Using the sorted m2m field,
Below is a sequence  of screenshots that show how the admin changes correspond to the order. I'm using the custom widget developed to support the Sorted M2M

As you can see, you can change the order via the two up and down buttons to the right of the second box and the order will save.

Yay, this is so great. Thank you @johnsonkuang!
I think we need to add some functionality to talks... right?

space taken out of models.py


Fix #11

Hi Andrie,
Here is the correction for the issue #11 :)
Bye,
Margot

@andrie This fix is pretty straightforward, and should be useful :)


BLUETOOTH_CONNECT permission missing in permissions.js

See https://developer.android.com/reference/android/Manifest.permission#BLUETOOTH_CONNECT

It's now REQUIRED for Android 12 API 31

I guess it has been solved by https://github.com/NeoLSN/cordova-plugin-android-permissions/pull/110


⚠ Secret Website 3 has degraded performance

In 47b7f10, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3450 ms

Resolved: Secret Website 3 performance has improved in 7dd39a3.


Migrate reviews when migrating repository from github

This PR will migrate review and review comments when migrating repository from github.
TODO:

[x] Need tests.
[x] Replace OriginalAuthorID
[x] Don't think migrator is the author if it's a migrated review or comment.
[ ] Don't think reviewers as the same person on reviewers summary.
[ ] Create action comment for reviews
[ ] ~Reactions, will be done after another PR.~

Codecov Report

Merging #9463 into master will decrease coverage by 0.1%.
The diff coverage is 1.9%.

@@            Coverage Diff            @@
##           master   #9463      +/-   ##
=========================================
- Coverage   41.51%   41.4%   -0.11%
=========================================
  Files         570     571       +1
  Lines       74644   74853     +209
=========================================
+ Hits        30989   30994       +5
- Misses      39815   40015     +200
- Partials     3840    3844       +4

Impacted Files
Coverage Δ

models/migrations/migrations.go
1.3% <ø> (ø)
:arrow_up:

modules/migrations/base/downloader.go
26.66% <ø> (ø)
:arrow_up:

modules/migrations/git.go
53.33% <0%> (-3.81%)
:arrow_down:

modules/migrations/gitea.go
7.77% <0%> (-1.9%)
:arrow_down:

models/review.go
41.63% <0%> (-4.53%)
:arrow_down:

models/migrations/v115.go
0% <0%> (ø)

modules/migrations/github.go
67.42% <0%> (-9.89%)
:arrow_down:

modules/migrations/migrate.go
23.33% <19.04%> (-2.46%)
:arrow_down:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

services/pull/temp_repo.go
31.62% <0%> (-2.57%)
:arrow_down:

... and 13 more

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 1df701f...525ed13. Read the comment docs.

migration has conflicts

It's ready for review.

@6543 All done.

current state if I test i get this error:
Migrate repository from https://github.com/6543/remaster failed: pull request does not exist [id: 128, issue_id: 0, head_repo_id: 0, base_repo_id: 0, head_branch: , base_branch: ]

@lunny found the error source: https://github.com/go-gitea/gitea/pull/9463/files#diff-2557bcc0dfec751b3834722f0d5477f0R694 return the error!

@6543 Thanks! I have fixed that.

It's ready for review.

Maybe remove the wip label? 😁

It's ready for review.

Maybe remove the wip label? 😁

Currently github stored position related the patch head but gitea stored line of treepath. I will resolve that.

Fixes #9443

pleace resolve confilct

@6543 @guillep2k It's ready to review again.

depends on #9688

can you resolve migration confilct


FIX & ENH: linear_regression_raw solver sklearn style

The solver parameter in mne.stats.linear_regression_raw can now be set with a function. However, the y is transpose as compared to the expected equation w = X^T Y
Here is  the copy from the default solver used in linear_regression_raw:
linalg.solve(a, X.T * y.T, sym_pos=True, overwrite_a=True, overwrite_b=True).T

For example if one use a sklearn's ridge, we would have to transpose y before passing the solver:
from sklearn.linear_model.ridge import ridge_regression
solver = lambda X, Y: ridge_regression(X, Y.T, alpha=1.)
event_id = dict((c, ii) for ii, c in enumerate(categories))
evokeds = linear_regression_raw(raw, events, event_id, solver=solver)

I suggest a couple of things:

transpose Y back to its original shape
detect whether the solver has a 'fit' attribute, in which case it'd do solver.fit(X, Y).coef_
add a dictionary to call the commonly used regressions (e.g. solver='ridge')

It seems everyone who uses this uses sklearn's Ridge anyways, so it would be good to be more sklearn compliant and have a dict path to Ridge here.
I will do this.

Can you also defaults  event_id to dict((k, v) for ii, v in enumerate(np.unique(events[:, 2]))) in that case ?

Hm. Seems fine to me I guess.
@Eric89GXL would we need a deprecation cycle here? Currently, the solver is private, and the default use case does not change, but user-defined solvers will need an additional .T.
(I assume the number of affected users is super low though, and I probably know 90%+ of them personally ...)

I'd also like to make event_id optional, I barely even use it anymore.

would we need a deprecation cycle here?

If it's a bugfix then no. If it's just for API reasons then yes.

linalg.solve(a, X.T * y.T, sym_pos=True, overwrite_a=True, overwrite_b=True).T

Just saw this. Keep in mind that ndarray.T is a view, and thus allowing overwriting the contents could mean that they get destroyed, so make sure that users know that the X or Y they pass could be modified inplace.

...although I guess you could check sizes and tell by the dimensionality, and warn about the new style? The only time you'd be in trouble is when Y is square, which might be rare enough to live with

The only time you'd be in trouble is when Y is square, which might be rare enough to live with

Not only rare, but also basically pointless - that'd mean more channels than time points, in which case you really shouldn't be doing rERP :)

it's just for API reasons then yes.

It's just for API reasons. When I wrote the function, I wasn't thinking about sklearn.
But I guess if we transpose y based on its dimensionality, it'd be sufficient to just warn for 2 releases or so.

+1 for dim check + deprecation

Q: maybe we should error when the array has the wrong shape. Rationale: factorizing/inverting/... a matrix with the wrong dimensions can lead to massive arrays that overload RAM and might freeze your Python session/computer. Thoughts? @Eric89GXL @kingjr

I think it's better to be stringent / make sure the user passes the right thing, so +1 for error

Close for #3857


Basic wide string support

This also ensures proper logging (including on exception) and adds some more standard functions from glibc and msvcrt.

Nice! Looking great on first glance. I'll try to do some testing tonight/tomorrow.

On my machine with a linux-armv7 binary, I get:
Traceback (most recent call last):
  File "...plugins/binaryninja-printf/__init__.py", line 553, in run
    func_type, func_decl_name = bv.parse_type_string(func_decl)
  File "...python/binaryninja/binaryview.py", line 6687, in parse_type_string
    raise SyntaxError(error_str)
SyntaxError: error: string.hpp:1:66 unknown type name 'rsize_t'

I guess the plugin should either add a rsize_t typedef if it isn't already present, or fall back to regular size_t?

I think it would be reasonable to just use size_t. AFAIK rsize_t is equivalent to that on all platforms that have those functions anyway; it only exists because the C standard says it should exist. Alternatively, could ask the BN devs if they want to support rsize_t natively in their type parser like they do for size_t, but ehh.
I think it would be good to avoid adding types to the user's BNDB if at all possible.


Java updates

removed redundant generic type specifications
favored try-with-resources
applied streams where that seemed appropriate and the other languages used that style

Thank you! I'm thinking about how to manage language versions. I want to keep Java 7 examples. As soon as I decide this I'll merge this PR.

@evmorov I had same thought, some languages, for example JavaScript drastically differs between versions.
I think, language mapping based on file extension should be change on path.
This:
CODE_EXTENSION = {
    'coffee' => 'coffeescript',
    'rb' => 'ruby',
    'java' => 'java',
    'py' => 'python',
    'php' => 'php'
}.freeze

can be changed to this:
CODE_EXTENSION = {
    'coffee' => 'coffeescript',
    'ruby' => 'ruby',
    'java/7' => 'java',
    'python/3' => 'python',
    'php' => 'php'
}.freeze

And files can be organized in to directories, for example:
`-- code/
    |-- coffee/
    |   `-- array_add.py
    |-- ruby/
    |   `-- array_add.py
    |-- java/
    |   `--7/
    |      `-- ArrayAdd.py
    |-- php/
    |   `-- ArrayAdd.py
    `-- python/
        `-- 3/
            `-- array_add.py


Add Laminas module installer

Is this really required? It seems to me like Laminas should be more modern than this, and if it's needed Id' like to hear it from someone on the Laminas team. In the meantime closing this as it is very old.


Problems Starting the Linux start script

I got the Qt error, that no Qt plugins could be initialzed.
I had to install libxcb-xinerama0
sudo apt install libxcb-xinerama0
But I suggest using the web interface
python src/app.py -w

@the-jan-peter Which operating system?
Try a fresh install also follow the instructions in readme https://github.com/rupeshs/fastsdcpu?tab=readme-ov-file#fastsd-cpu-on-linux


Further Embed Customisation

At the moment, our embed code is designed solely for use in the DC server. This works fine but it would be brilliant to make our bot publicly accessible.
This would probably involve a refactor of our current embed routines to ensure embeds update globally in a safe and efficient manner while storing their identifiers/guild identifiers in an SQL database and accounting for various edge cases such as a channel/embed being deleted, while also creating commands to allow external server administrators to create embeds in a simple manner.
This would also involve a rework of the bot's permissions and since we're using interactions to a great extent, we should be able to strip down the required perms significantly. The ultimate goal is to not require the reading of messages whatsoever, so an implementation should not have to search for messages in a channel and should rather immediately fetch and store the message id from the message object created.
I'm not quite sure where to start on this so please do give me your thoughts and ideas, and we'll see how to go about it.

@mollybeam We're going to add your last issue posted to this one.  Just makes sense we're already gonna be doing a huge overhaul.
https://github.com/Digital-Controllers/discord-bot/issues/20#issue-1716675706

This has been implemented successfully. Keeping issue open as we will be working with this code heavily in the coming weeks.

Issue closed due to new PR #37


Add Use Case Example: NFT Marketplace

Feedback
#15 - More examples on specific cases (NFT marketplace, liquidity pool contract, etc)

Soon to open source HederaMarketplace.com

https://github.com/hashgraph/hedera-nft-minter

https://github.com/hashgraph/hedera-nft-utilities#rarity-score-calculation


Test documentation examples as part of E2E tests

Current E2E test suite test the controller and agent by using the debug-echo discovery handler, as such the other discovery handlers don't get tested as part of E2E tests.
It would be a really nice addition to add tests for all the examples we have in the documentation, this would ensure they are really functional and is in my opinion a good starting point to do E2E tests on all discovery handlers as those examples are set to be easily reproduced.

+1

Hi @LavenderQAQ,
the idea is to modify run-end-to-end.py to be able to test the different use cases highlighted in the demos section of the doc:

https://docs.akri.sh/demos/usb-camera-demo
https://docs.akri.sh/demos/opc-thermometer-demo
There is currently no onvif demo as we currently don't have a way to emulate an onvif camera.

The overall idea is to ensure these documented demos are regularly tested.

@diconico07 Ok, thank you for your detailed explanation. I want to finish this issue 😻.

@LavenderQAQ help on this would be greatly appreciated! @diconico07 is spot on. For onvif, @cliffordt3 did some work several months back to containerize the onvif demo. Some of that work is captured in this PR https://github.com/kate-goldenring/onvif-camera-mocking/pull/4.
Doing udev, opc ua, and onvif all at once may be a lot, so it may be best to tackle one, PR it in and then go for the next.

@kate-goldenring Sorry, I don't have time to follow up this issue at present because of some other projects😵💫. Very sorry for that.

@LavenderQAQ no worries. Feel free to ping if time arises in the future.


Only return active records from v1 Get Teacher endpoint

Context
https://trello.com/c/t4OHWRvz/731-stop-returning-inactive-data-on-the-v1-get-teachers-endpoint
Changes proposed in this pull request
Add a state == 'Active' filter to all the joined records in the V1 Get Teacher endpoint.


Update meetup functions to use field_event_api_key for url generation

Simplified get_meetup_events() to create urls from the field_events_api_key within the organization API.
Then updated format_meetup_events() to compare values from event['group']['urlname'] rather than event['group']['id'].

This worked fine with HackGreenville and OpenWorks' dashboard so merging it in.
Nice work.


⚠ Bofrost.de has degraded performance

In 7d6c23e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2498 ms

Resolved: Bofrost.de performance has improved in 620f989.


visual location encoding in UNITER

Hi,
I notice that you simply project the object location here https://github.com/e-bug/volta/blob/main/volta/embeddings.py#L495 and that you set the object location dimension to 5 there https://github.com/e-bug/volta/blob/main/config/ctrl_uniter_base.json#L16
How exactly do you represent the location of the object? Chen et al. say they use a 7 dimensional vector: [x_1, y_1, x_2, y_2, w, h, w ∗ h] (normalized top/left/bottom/right coordinates, width, height, and area.) They hard-code it: https://github.com/ChenRocks/UNITER/blob/master/model/model.py#L254
Bests,
Paul

Very clear, thanks!


Add build instruction for new versions

This PR extend the list of available versions to be up to date with Julia development. The follwing versions are included

v1.0.5
v1.2.0
v1.2.0-rc2
v1.2.0-rc3
v1.3.0
v1.3.0-alpha
v1.3.0-rc1
v1.3.0-rc2
v1.3.0-rc3
v1.3.0-rc4
v1.3.0-rc5
v1.3.1
v1.4.0
v1.4.0-rc1
v1.4.0-rc2
v1.4.1
v1.4.2
v1.5.0
v1.5.0-beta1
v1.5.0-rc1
v1.5.0-rc2

LGTM 🚀


Route installation test fix

Hi,
I have created this pull request to merger changes to configure network instance before configuring bgp in the test "route_installation_test.go" under bgp/policybase.
Path of the script: /root/GitMount/featureprofiles/feature/bgp/policybase/ate_tests/route_installation_test/route_installation_test.go
Ref issue: https://github.com/openconfig/featureprofiles/issues/191
Thanks,
Prabha

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
596

Relevant Lines:
963

💛  - Coveralls

Hi Likai,
We need to configure network instance type, router id for network instance.
I have updated code for network instance type and router id (loopback interface configuration and getting primary ip on loopback interface) .
Please could you review and approve it.
Thanks,
Prabha

Which test plan is this for? RT-1.2 is already done and published here: https://github.com/openconfig/featureprofiles/tree/main/feature/bgp/policybase/ate_tests/route_installation_test
Also, could you please revert the binding and testbed changes?

Hi Likai,
Sure, I will update and commit changes.
Thanks,
Prabha

Hi,
I am opening new pull request after fixing with above comments.
Thank you for the suggestions.
Thanks,
Prabha


Configurable probes

Django imposes auth for accessing the endpoint controller with DJANGO_ALLOWED_HOSTS env var..
This PR allows configuring k8s probes or setting the host to the k8s svc by default so that Django doesn't reject the probes with 401

Thanx for approval @lvpeschke! If PR is OK, could you merge it or add someone with write access as reviewer??


fix: add missing eos_token for Qwen2.5 Coder

Description
Added the missing eos_token <|endoftext|> for Qwen2.5 Coder. This oversight was brought to our attention through an issue comment and confirmed via the Hugging Face tokenizer_config.json file.
(https://huggingface.co/Qwen/Qwen2.5-Coder-7B/blob/main/tokenizer_config.json)
This fix addresses the problem with string/comment creation attempts and should resolve the unexpected behavior in text generation.
closes #2330
Checklist

[x] The base branch of this PR is dev, rather than main
[ ] The relevant docs, if any, have been updated or created

Screenshots
[ No visual changes. ]
Testing

Load the Qwen2.5-Coder-7b model.
Attempt to create strings or comments.
Verify that the <|endoftext|> token is properly recognized as an end-of-sequence marker.

Will create PR for branch cleanup after updating dev


Make om bosh-diff return with exit code 2

Return 2 if --check is specified and there are differences instead
of 1 for all other errors.
Co-Authored-By: Amber Salome<EMAIL_ADDRESS>

Solved by #512


slot attributes are deprecated | Vue 2.6+

Describe the bug
The slot attribute has been depreciated as shown at https://github.com/vuejs/rfcs/blob/master/active-rfcs/0001-new-slot-syntax.md and https://github.com/vuejs/rfcs/blob/master/active-rfcs/0002-slot-syntax-shorthand.md.  When provided by this component, the code will fail to compile.
Steps to reproduce
Use the below example as shown at https://github.com/antoine92190/vue-advanced-chat using Vue.js 2.6 or greater.
`div v-for="message in messages" :slot="'message-avatar_' + message._id">
New Avatar
`
Expected behavior
Avatar shown next to message
Device (please complete the following information)

OS: Windows 10
Browser: Chrome  Version 111.0.5563.147 (Official Build) (64-bit)
Package version: 2.0.6

Additional context
This issue is related to https://github.com/antoine92190/vue-advanced-chat/issues/412 that was closed, however, the proposed solution did not fix the issue.

I don't have the issue on my side. Can your reproduce on this minimal example? https://github.com/antoine92190/vue-advanced-chat-sandbox

Actually I'm not using vue slots, but native web slots: https://developer.mozilla.org/en-US/docs/Web/Web_Components/Using_templates_and_slots
So I guess it's an misinterpretation of the eslint-plugin-vue.
What error are you getting when running npm install on the sandbox?

Closing this as it seems to be an issue with eslint-plugin-vue. Please reopen otherwise


Improve dynamic logo compatibility

This fixes the dynamic logo in WebKit-based browsers. The old inline styling is still in place as a fallback.
Before/after
Before

After:

Thanks :)


Web receiver: content & research V3

Docs

Starting questions etc

Content being moved to static site by EOS.


Possibility to add imports "at the bottom of the imports"

Related to #82
When new imports are added at the top of the file, a new import should be at the bottom of the imports.
import ...
import ...
// << -- new import location

/* code */

Must-have feature while using reflect-metadata.
Also it should divide external modules and internal app parts imports by empty line into two section and insert in correct section.
https://angular.io/styleguide#!#03-06

I'd also like to see any automagically added imports added to the bottom of the list, but it should definitely be something configurable.

Will be resolved (I hope at least) with #102


Check Log For Information error in Kodi 18.9

I'm getting the "Check the log for more information" error whenever I try to play any movie on Einthusan app in Kodi 18.9. It was working fine and I started getting the error recently. I'm not able to see the log file.

Kindly answer the below details:

Kodi version:
Device Type and Version:
Einthusan Addon Version:
Regular User or Premium User:

Install Log Viewer for Kodi and share the log file or photo or recent logs:
Settings -> Add-ons -> Install from repository -> Program Add-ons -> Log Viewer for Kodi -> Install


added simon game

Title and Issue number

Title: Simon game
Issue No. : #172
Code Stack : html, css, javascript.
Close #172

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md
[X] I have updated the Index.html file for my contribution

Changes done , please review, Thankyou:)


Working! Can it work with Infinite Zoom?

Im not sure this is an issue but is there a way to make it work with the Infinite Zoom extension as well? Infinite Zoom is fun and I would like to add it to the public side of my server.
PS. DiffusionDefender has been awesome so far and after tuning the .txt file I was able to get some pretty consistantly good looking Donkeys ;)
Thank you for all you do,
ProfitC

Did you intend to close and complete this issue?


Are there plans to open source the code?

Dear author, thank you very much for your work. I want to learn your processing methods and apply them to my own processing scheme. Do you have plans to open source your code? Thank you very much and look forward to your reply.

Thank you for your message. We have recently updated our programs on GitHub.  About the open-sourcing, this is a topic that requires further discussion within our team. We appreciate your interest and support. :)

Okay, thank you for your reply!


sign up page is too constricted and css is not good.

Describe the bug
the space is too less in between .
Expected behavior
more spacing
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

#113 #123
Please Check out this

It's been fixed i guess.


Update PUBLIC_URL

Replace:
pub static PUBLIC_URL: &str = "/_api/public/";

with
pub static PUBLIC_URL: &str = "/_api/public";

so we can write nicer urls like format!("{PUBLIC_URL}/images/smile.png") (notice slash before images).

Implementation notes:

~Wait until the PR https://github.com/MoonZoon/MoonZoon/pull/58 is merged.~
Update also related docs and examples that use PUBLIC_URL.
https://github.com/MoonZoon/MoonZoon/blob/7741ea4788760e85e87b8b0042d1e72844caafb1/crates/zoon/src/lib.rs#L128

Perhaps we can leave PUBLIC_URL as is and implement 2 new macros:
public_url_const!("images/smile.png");
public_url_const!("{IMAGES}/smile.png");
public_url!("images/{name}/png");


Az/fix analytics permissions

Changed permission checking for the analytics view: used OR instead of AND for the group whitelist
Motivation and context

Specifying group whitelist doesn't work as expected: default Django permission check requires that user has all permissions from list, but it's not appropriate for the group whitelist.
How has this been tested?
Manually
Checklist

[ ] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Pull Request Test Coverage Report for Build 5353

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 65.657%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cvat/apps/log_viewer/views.py
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

cvat/apps/log_viewer/views.py
1
0%

Totals

Change from base Build 5343:
-0.01%

Covered Lines:
10836

Relevant Lines:
16101

💛  - Coveralls


Add config file support

Hi Kien,
I'd like to add support for a .json formatted config file to your SSG. Will be accessible at runtime with the --config or -c argument .

Issue solve with PR #12


Use Chainer v2

Changes Unknown when pulling a8556134cc664c23adba359464fded1fb2a4172e on toslunar:chainerv2 into ** on pfnet:master**.

Changes Unknown when pulling f7e6a99edcfa11cbf2efad848ebc8eca9ce6b5f0 on toslunar:chainerv2 into ** on pfnet:master**.

Changes Unknown when pulling 04ede2d4346773bd45f06e562c16485b0ef1c49f on toslunar:chainerv2 into ** on pfnet:master**.

Great work! Have you checked tests (including gpu and slow tests) in your environment?

Some slow tests fail because of BLAS errors that seem to be chainer/chainer#2744.
I'm doing gpu tests.

LGTM


KeyError with similar_objects() function

Hi Alex,
I'm working on a project using Wagtail-CMS which incorporates your tagging system. For what I'm working on at the moment the similar_objects() function you created is exactly what I need, however I think there might be an issue with the function (unless I'm making some silly mistake in my own code).
I've written up a detailed post on SO which you can check out here.
Many thanks

Stuck with the same problem. Eventually I decided to use raw sql to perform my query. I wrote this way (It is just workaround, not solution to original problem.):
ArticlePage.objects.raw('select a.page_ptr_id, p.title, count(at.tag_id) as tag_count from article_articlepage a join wagtailcore_page p on a.page_ptr_id = p.id join article_articletag at on at.content_object_id = a.page_ptr_id join taggit_tag t on t.id = at.tag_id where tag_id in (1, 2, 3) group by (page_ptr_id, p.id) order by tag_count desc');

I put this into gist with better explanation, so feel free to copy and modify it if you needed: https://gist.github.com/prokaktus/bd805e3f9a3eabb64be22523ae5fdf66

I'm having the same issue and seems like this pr #424 is kind of stuck for the last few years.

This seems to be a duplicate of #80

Thanks.


[Suggestion] Replace Barrel Jack with USB-C

I always try to build everything in my toolbox around the USB-C power supplies so I can have one power supply to rule them all  haha :)
If 100W is enough power for the board, I think that is a cool upgrade :)

Certainly!
I had initially designed around a barrel jack for simplicity to begin with, but be sure there will be a USB C version in the not too distant future.
I plan to work on one myself and I have heard mention of some others forking off the repo and doing the same. Long live open-source!

Awesome! In the mean time, I ordered everything to build 3 of then rigth now, I'll rebuild one when USB-C comes in haha.
I would suggest the STUSB4500 chip which would be perfect for that, but unfortunately in these times they are unavailable...
Anyway in my opinion, add USB-C and a replaceable heating part, and you have an amazing product :)

FYI:
I am starting right now to build a USB C Version. But it might take some time to get it finished, since it is my first real project.
It is not as easy as I initially thought, since USB C PD is designed for 3A, but according to specification it should also handle 5A.
USB C doesn't have a 12V Level, only 15V, so we get a higher current. I will have to do some testing...
I think, I might use the 15 or 20V line and a step-down circuit.
PS: Maybe we could do a discord channel or something, where we can communicate with each other?

I'm not sure USB is a good option
Most notebooks these days run off 20v - banggood has adaptors for all of
them to a barrel plug - would be handy to be able to use the notebook PSU
On Thu, Mar 3, 2022 at 10:09 PM Markus von Staden @.***>
wrote:

FYI:
I am starting right now to build a USB C Version. But it might take some
time to get it finished, since it is my first real project.
It is not as easy as I initially thought, since USB C PD is designed for
3A, but according to specification it should also handle 5A.
USB C doesn't have a 12V Level, only 15V, so we get a higher current. I
will have to do some testing...
I think, I might use the 15 or 20V line and a step-down circuit.
PS: Maybe we could do a discord channel or something, where we can
communicate with each other?
—
Reply to this email directly, view it on GitHub
https://github.com/AfterEarthLTD/Solder-Reflow-Plate/issues/10#issuecomment-1057934632,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIHLCL2U6UIZUSSKCEJ7BCLU6CMV5ANCNFSM5PSABCDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Awesome @MarkusvonStaden !
I would also make the electronics and heating plate separate as well, while being at it haha!
Tell me if I can help you.
Regarding 12V, yes not all usb-c pd supplies have that option, don't think it's actually part of the standard. However that shouldn't be a problem in my opinion, we could probably make it work regardless of the power input (in a limited range, like 12-20v) by measuring the actual Vin and adapting the pwm settings around the input voltage.
Totally in for the discord channel!
@martinlanza In my opinion, USB C is way more common nowadays than barrel connectors.. However I think we could use both options and combine them on the pcb so anyone can choose the preferred one

Hi, I started to fork this project in order to make USB-C board with STUSB4500 and an other mcu (STM32G030F). The Schematic is already done. We can work together if you want.
Sincerely

Hi, I started to fork this project in order to make USB-C board with STUSB4500 and an other mcu (STM32G030F). The Schematic is already done. We can work together if you want. Sincerely

Awesome! :)
The STUSB4500 is definitely a top notch option, but pretty hard to get these days unfortunately! Why did you change the MCU?
If we can get the board with both USB-C and Barrel Jack, external heater board, and maybe profile automatic heatup, I think it's awesome

Hi, I started to fork this project in order to make USB-C board with STUSB4500 and an other mcu (STM32G030F). The Schematic is already done. We can work together if you want. Sincerely

Awesome! :) The STUSB4500 is definitely a top notch option, but pretty hard to get these days unfortunately! Why did you change the MCU?
If we can get the board with both USB-C and Barrel Jack, external heater board, and maybe profile automatic heatup, I think it's awesome

I prefer to change the mcu to have availability in the near future. This component is smaller, more powerful and cheap.
Keep the Barrel Jack will not be a problem.
For USB PD IP2721 can also be a solution.

I see :)
There have been multiple discussions around the MCU change, mostly because of the Atmega328p availability, I think it would be great if everybody settled on one choice so we can all go in the same directions hahaha
Yes, IP2721 does work too :)
I'd rather use brand name parts though, had too much issues with performance and availability in the past with obscure components haha

My choice of MCU will not be objective I am a  STM32 fan  ;)

Haha don't worry we're in the same boat here haha!

I plan to finish it today, so I can order the first PCBs to test.

Wow guys!
This sounds great. I love the idea of making it USB-C powered.
You'll certainly need a well specced USB PD capable of 60W or more at 20V for the same size board. Also, consider increasing the capacitance to even out the current draw when the MOSFET is being pulsed.
Look forward to seeing what you come up with.
Also, I've created a Discord for SR Plates. https://discord.gg/T9dxeRmY Please feel free to join send invites, create a channel and manage your project through that if you like.

First comment on the project. Just wanted to add the new Discord link that never expires here:
https://discord.gg/YzhG6FcCRA

Done, well spotted, thanks

Hi,
Added my implementation of IP2721_MAX12. I tried to use back similar components to reduce BOM list. Any comments?


Rename pick_ups.html.erb to schedule.html.erb

Summary
This application used to only support picking up distributions, but https://github.com/rubyforgood/diaper/pull/1818 added the ability to specify delivery vs pick-up. After that change, this filename doesn't make as much sense anymore, so we should rename it to schedule.html.erb since this view shows the distribution schedule as a calendar.
See https://github.com/rubyforgood/diaper/pull/1818#discussion_r487657016 for PR comment that led to creating this issue.
Things to Consider
There is a breadcrumb in the application in the top left that currently says "Pick-Ups". I'm not sure if this is generated from this filename or not, but that text should match the rename as well.

Criteria for Completion

[ ] This erb file is renamed to something more schedule.html.erb or something equally generic since we now support 2 types of distribution.
[ ] The sidebar link works after the rename
[ ] The breadcrumb text and link works after the rename

I can work on this issue

Hi @rebecarancan , it's all yours!


Use JSON instead of TOML as config format

As discussed in this thread, we'll standardize the JSON format of the config file to something like:
{
    "Deps": [
        {
            "ImportPath": "rsc.io/arm/armasm",
            "Rev": "616aea947362"
        },
        {
            "ImportPath": "rsc.io/x86/x86asm",
            "Rev": "af2970a7819d"
        }
    ]
}

Have you also considered lowercasing the "Nut.toml" filename? I'm not sure if it matters, but having all lower-case aligns with Go's file naming conventions and should probably be documented as such too.

I'd also argue that TOML is a much nicer format for hand-editing over JSON. If there's any expectation that users (humans) touch this file in any way, I think TOML makes more sense.

I think one of the reasons for choosing JSON is that it doesn't require any external dependencies like TOML does. So unless a TOML parser makes its way into the standard library I don't think it'll happen.
On the other hand, I guess nut could support both and have TOML support as a differentiator against other dependency managers. And supply a "compile-to-standard-config-format" command if you'd for some reason would like to move to another dependency manager.
Just suggestions and speculation though.

@elithrar:

I'd also argue that TOML is a much nicer format for hand-editing over JSON. If there's any expectation that users (humans) touch this file in any way, I think TOML makes more sense.

I personally like TOML over JSON as well. But as you read from the discussions linked in issue, we're trying to standardize the config file format across the Go community, e.g., Godep will use the same format. The suggestion is to use something that the Go standard library can parse.  It seems no conclusion has been made yet. Feel free to comment on the discussion thread to show your concerns. I voted TOML BTW :smile:.

Have you also considered lowercasing the "Nut.toml" filename?

This will be standardized as part of the discussion too.

@renstrom:

I guess nut could support both and have TOML support as a differentiator against other dependency managers. And supply a "compile-to-standard-config-format" command if you'd for some reason would like to move to another dependency manager.

I thought about this too. But I don't have enough facts to prove it to be a good idea. If we were to go with this approach, I'm thinking keeping Nut.toml the same, but running nut install will generate the standardized config file in ./internal/TBDCONFIG.CFG. It's like Ruby Bundler generating a Gemfile.lock from Gemfile. Nut.toml is less strict about defining the versions of the dependencies, e.g., leaving it blank to always get the latest dependencies as it's today.


Add "File: delete" to Command Palette like Sublime Text

Feature Request: Add a command to Command Palette like Sublime Text "File: Delete".
Sublime Text:

VS Code 1.25.1:

See: https://marketplace.visualstudio.com/items?itemName=sleistner.vscode-fileutils


Evaluation Code

Dear Authors,
Thank you for your outstanding contribution!
I was wondering if it would be possible for you to share the evaluation script that employs COLMAP, including the required parameters. Additionally, could you provide details on how you calculated the Rotational Error and Translation Error as presented in your paper? This information would greatly aid in ensuring reproducibility and would be invaluable for applying these metrics in a different yet related study.
Thanks!

I have the same question here. In particular, since the translation from COLMAP can have arbitrary scale, directly comparing it to the ground truth using L2 norm without scale normalization (eq.5 in the paper) would usually result in large error?
Having more clarity here or even the script of evaluation would be great.
@skrya: are you able to reproduce the quantitative measure in the paper?

Same question here.


Update to support route registrar move to backend endpoints only being TLS

In an upcoming version of route_registrar in the routing release, the runtime WG is advising that only TLS enabled backend endpoints will be supported.  cloud.gov needs to make sure all deployments and releases using route_registrar to route through Cloud Foundry are using TLS for the back end endpoint that is being registered.
Notes

From end web services needs to be using TLS
The CA used for these certs needs to be trusted by the gorouter - use MasterBosh for CA

Acceptance Criteria

[ ] Determine if front end web services supports TLS - upgrade if needed
[ ] Create new credhub based certificates to use via ops files
[ ] Update documentation reflecting the change
[ ] Update deployment repos and pipelines as needed

The only registered route for this BOSH deployment already uses a TLS backend
https://github.com/cloud-gov/deploy-logs-opensearch/blob/414318fb334712905bdcc794fe9dd2f3328cffa1/opsfiles/enable-proxy-auth.yml#L107
So this issue can be closed


Scalar threshold percentile

Changed BasicThreshold plugin and unit test setup functions to output scalar threshold and percentile coordinates, rather than single-valued dimensions.
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

This sounds like a good change in theory, but does making these things non-dimensional just cause problems for downstream systems, particularly when we may increase the number of thresholds or percentiles for variables causing it to jump to a dimension? There is something nice about sending percentiles as [50.0] and allowing the number and cube dimension to grow.

@benfitzpatrick Certainly for spot data, we already represent a single percentile as a scalar coordinate and that's what is disseminated.  I will have to check the "percentile" CLI / unit tests but I believe that does too.  The code and synthetic data setup functions were not behaving consistently in this.

We're intending to expose spot data as a list of percentiles at the beta, but as I was saying to @bayliffe, I don't think that intention is widely known


Workday to Microsoft Azure AD User provisioning UPN attribute

Based on the user's first name and last name, i need to generate a value for the UPN attribute and check for its uniqueness in the target AD directory before assigning the value to the UPN attribute.
Tried below expression and in the workday to azure ad user provisioning its failing for the second user
SelectUniqueValue(ToLower(Join("@", NormalizeDiacritics(StripSpaces(Join("", Mid([FirstName], 1, 1), [LastName]))), "domain.com"), ), ToLower(Join("@", NormalizeDiacritics(StripSpaces(Join("", Mid([FirstName], 1, 2), [LastName]))), "domain.com"),), ToLower(Join("@", NormalizeDiacritics(StripSpaces(Join("", Mid([FirstName], 1, 3), [LastName]))), "domain.com"),))
For example:
First Name : Sam
Last Name : Dood
UPN to create<EMAIL_ADDRESS>its working for the first user
Second user details
First Name : Sun
Last Name : Dood
expression should trigger
UPN to create<EMAIL_ADDRESS>however for the second user its not creating in AD

@nick834 Thanks for reaching out to us, but at this time it does not look like you are referring to a document but to a specific scenario. If you are referring to an Azure documentation and if you think it needs an update (w.r.t documentation or product), please let us know that Azure documentation link / URL here so that we can investigate, work and provide you an update here. Else, we request you to please post your question on MSDN and other forums, we have our engineers and others who are monitoring them and will be happy to help.

@nick834 Closing this issue for now. If there are any questions or feedback regarding Azure docs then please refer to an Azure document link / URL and let us know so that we will gladly continue the discussion.


AutoDAN with Llama-2

Hello there,
Thank you for your work! I have tested out your code with GA on my own, there seems to have some problems that I could not get any good prompts that could lead to jailbreaking the Llama-2 model, I am just wondering if you could help me here! Thank you!

Hello guys,
Apologies for my delay! I was occupied with work for ICML before. We've observed that the recent fschat update has caused an issue, leading to incorrect ASR results in the Llama-2 model. We've now aligned the code properly. Please don't hesitate to reach out if you have any questions.
Thanks for your interest in our work!


more descriptive issues

I'd really like to contribute since I actually started working on a Prometheus client library some time ago but never got to finish it because I don't have enough time to do this right now. I found this project but the issues here have little to no descriptions or what actually is needed.

Hi @Rukenshia
Thanks for your interest and welcome to contribute! I sorry that we don't have any descriptions or what is needed right now. We will improve these later.
Now, I think the most important issues are #5 , #34 and #7, feel free to have a try :-)


unknown reconstruction result using BFM model, need help

At first, I get bfm2009_model from 01_MorphableModel.mat.
Then I use texture_coordinates from stn/BFM_UV.mat (size of [2 53490], range [0 1])and texture_triangle_indices from" bfm2009_model['tl'] - 1".
However, the obj result of fitting-model-simple (https://github.com/patrikhuber/eos) is unexpected as following:

Can someone tell me why, i am confused deeply.


Add stargazers and fork count

Purpose
Include the stargazers count and forks count for each repo in the repository list. Closes #7
Approach
Add additional parameter to the graphql query
Open Questions and Pre-Merge TODOs

[ ] Change the tests

👍 Makes sense. In addition to a test, the readme and https://github.com/sindresorhus/gh-latest-repos/blob/master/example-response.json needs to be updated too.

Some tests were failed.

Since we modify the response from github on our fetchRepos() function, we will need 'revert' the result from nock to use the default-shaped response. This will make us have 2 jsons, one for our fixture, one for the github's fixture.

Looks good. Thanks :)


Fixed scratch size calculation for conv for HiFi targets for scenarios when input, filter and output heights are 1.

Fixed scratch size calculation for conv for HiFi targets for scenarios when input, filter and output heights are 1.

replaced with PR#3004


Unable to link plugin after npm install

Summary
After using npm to install a plugin. running sfdx plugins link returns an error.
Note that in order for npm install... commands to succeed we need to make use of the NODE_EXTRA_CA_CERTS environment variable. Specifically, the variable specifies the path to a file that represents the root certificate of our organization.
However, when any sfdx command is run the following warning is shown:
Warning: Ignoring extra certs from `.\Downloads\MyRoot.crt`, load failed: error:02001003:system library:fopen:No such process

Have attached the output of sfdx doctor --command "plugins link". Have also attached yarn-error.log file from the plugin folder which I believe is a result of the failed link operation. In particular, the following appears in the yarn-error.log file:
  Error: write EPROTO B8610000:error:0A000152:SSL routines:final_renegotiate:unsafe legacy renegotiation disabled:c:\ws\deps\openssl\openssl\ssl\statem\extensions.c:922:

My theory is that the link operation needs to access a secure URL for some reason (not sure why, as wouldn't the install have taken care of that?) but because sfdx is ignoring NODE_EXTRA_CA_CERTS this is failing.
I do not believe the issue is specific to the signups plugin.
Steps To Reproduce

Set NODE_EXTRA_CA_CERTS
Run npm install @salesforce/plugin-signups --global
Switch to plugin folder
Run sfdx plugins link

Expected result
Plugin is listed when run sfdx version --verbose
Actual result
@salesforce/cli: linking plugin @salesforce/plugin-signups... !
    Error<EMAIL_ADDRESS>--non-interactive
<EMAIL_ADDRESS>    --preferred-cache-folder=C:\Users\xxxxxxx\AppData\Local\sf\yarn --check-files --silent exited with code 1

System Information
 CLI Version:
        @salesforce/cli/2.21.8

 Architecture:
        win32-x64

 Node Version:
        node-v18.18.0

 Plugin Version:
        @oclif/plugin-autocomplete 3.0.2 (core)
        @oclif/plugin-commands 3.0.7 (core)
        @oclif/plugin-help 6.0.7 (core)
        @oclif/plugin-not-found 3.0.4 (core)
        @oclif/plugin-plugins 4.1.9 (core)
        @oclif/plugin-search 1.0.8 (core)
        @oclif/plugin-update 4.1.4 (core)
        @oclif/plugin-version 2.0.8 (core)
        @oclif/plugin-warn-if-update-available 3.0.5 (core)
        @oclif/plugin-which 3.0.11 (core)
        @salesforce/cli 2.21.8 (core)
        apex 3.0.8 (core)
        auth 3.0.10 (core)
        community 3.0.7 (link) C:\Users\xxxxxxx\AppData\Roaming\npm\node_modules\@salesforce\plugin-community
        data 3.0.5 (core)
        deploy-retrieve 2.2.3 (core)
        info 3.0.7 (core)
        limits 3.0.5 (core)
        marketplace 1.0.10 (core)
        org 3.1.1 (core)
        packaging 2.0.1 (link) C:\Users\xxxxxxx\AppData\Roaming\npm\node_modules\@salesforce\plugin-packaging
        schema 3.0.8 (core)
        settings 2.0.9 (core)
        sobject 1.0.5 (core)
        source 3.0.3 (core)
        telemetry 3.1.4 (core)
        templates 56.0.4 (core)
        trust 3.1.2 (core)
        user 3.1.0 (core)

 OS and Version:
        Windows_NT 10.0.19045

 Shell:
        cmd.exe

 Root Path:
        C:\Users\xxxxxxx\AppData\Roaming\npm\node_modules\@salesforce\cli

Additional information
plugins-link.zip

A quick update...
I realized the path I was using to set NODE_EXTRA_CA_CERTS was relative and therefore when I switched to the plugin folder it would no longer be correct. Changing to an absolute path eliminated the Ignoring extra certs warning, but the end result is still the same (i.e. link operation fails with the same error and same error shown in yarn-error.log file).

Managed to solve this on my own (after much searching and trial-and-error)...
Firstly, if you run openssl s_client -connect registry.yarnpkg.com:443 -tls1_2 you'll observe that the yarn package registry does NOT support secure renegotiation. With newer versions of Node.js (>= 18?) secure renegotiation is now the default.
So the solution is to (re-)enable unsafe legacy renegotiation, as follows:

Create a text/config file with the following content:

nodejs_conf = openssl_init

[openssl_init]
ssl_conf = ssl_sect

[ssl_sect]
system_default = system_default_sect

[system_default_sect]
Options = UnsafeLegacyRenegotiation

Set the environment variable OPENSSL_CONF to the fully-qualified path of the file created in the previous step.

Link operation should now succeed.

As a postscript, I ran into basically the same issue on our CI/CD server. However, the above solution was not an option as access to https://registry.yarnpkg.com would require a firewall change. Requesting access to a vendor domain (salesforce.com) is one thing, but opening access to a seemingly unrelated domain is another matter (which I did not pursue).
To install the plugin, I had copied the folder contents from my work machine (where I did follow the above) and attempted to link the plugin, with the same (and not very informative) ...exited with code 1 error. As it turns out, deleting the yarn.lock file from the plugin folder allowed the link operation to complete successfully. Probably this is not a good idea for an original install, but as I have stated, a proper install had previously been performed on another computer that did have access to the yarn registry.
It strikes me that this additional verification imposed by the presence of the yarn.lock file is a relatively new development. And I'm not sure why it is necessary when you need to first run npm install... which presumably retrieves all relevant content?
In any case, some may find this additional update useful/helpful.


Pin dependencies

This PR contains the following updates:

Package
Type
Update
Change

actions/checkout
action
pinDigest
-> b4ffde6

actions/upload-artifact
action
pinDigest
-> 5d5d22a

oxsecurity/megalinter
action
pinDigest
-> a7a0163

peter-evans/create-pull-request
action
pinDigest
-> 70a41ab

stefanzweifel/git-auto-commit-action
action
pinDigest
-> 3ea6ae1

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ COPYPASTE
jscpd
yes

no
1.08s

✅ CSS
stylelint
1
0
0
1.41s

✅ JAVASCRIPT
standard
1
0
0
1.53s

✅ JSON
eslint-plugin-jsonc
3
0
0
3.56s

✅ JSON
npm-package-json-lint
yes

no
0.44s

✅ JSON
prettier
3
0
0
0.3s

✅ JSON
v8r
3

0
9.64s

✅ MARKDOWN
markdownlint
8
0
0
0.63s

✅ MARKDOWN
markdown-link-check
8

0
2.03s

✅ MARKDOWN
markdown-table-formatter
8
0
0
0.33s

✅ REPOSITORY
gitleaks
yes

no
0.21s

✅ REPOSITORY
git_diff
yes

no
0.01s

✅ REPOSITORY
grype
yes

no
10.15s

✅ REPOSITORY
secretlint
yes

no
1.33s

✅ REPOSITORY
trivy
yes

no
6.65s

✅ REPOSITORY
trivy-sbom
yes

no
1.27s

✅ YAML
prettier
2
0
0
0.54s

✅ YAML
v8r
2

0
4.79s

See detailed report in MegaLinter reports
MegaLinter is graciously provided by


Add support for optional styling of statuses

Added support for custom alert styles, e.g.
{
    "key": "color-lime",
    "title": "Lime test",
    "subTitle": "This alert is lime. Yumm.",
    "state": "info",
    "style": {
    	"background": "lime"
    }
}

Heey @syphernl, thank you for your PR!
As I already mentioned face to face, I primarily think that all statuses should adhere to the given states. Especially since there are several themes that should be supported now. Adding custom elements to those statues could potentially mess up the looks, and especially since it's possible to have a different theme per client.
It also raises the question, do you want to have custom status styling in the back-end domain, whilst that doesn't feel like a fit.
What would be a better idea imo is to introduce another state. Not sure what the state should be, but the main goal would be to have states that can apply to all possible status sources. I want to prevent adding "module specific" elements, as I much rather have a streamlined overview.
I'm not sure what to do with this PR yet, as I am in conflict with the recently introduced themes. On the other hand it couldn't hurt to allow user to push extra styling if they really want it. My biggest worry is that custom styles end up in adapters, that's something I really want to prevent.
@syphernl what do you think about that?

Closing for now, feel free to re-open or elaborate the feature.


git-clone cluster tasks throwing permission denied public key when using ssh basic-auth

Expected Behavior
git-clone and git-clonev0-14-3 cluster tasks to clone the private gitlab repository into the output workspace.
Actual Behavior
git-clone and git-clonev0-14-3 cluster tasks are throwing Permission denied error (public key) error with Gitlab while using ssh basic-auth.
{"level":"error","ts":1600075644.453436,"caller":"git/git.go:41","msg":"Error running git [fetch --recurse-submodules=yes --depth=1 origin --update-head-ok --force master]: exit status 128\nWarning: Permanently added the ECDSA host key for IP address '<IP_ADDRESS> to the list of known hosts.\r\n\n    ***********************************************************************\n    * Access to this computer system is limited to authorised users only. *\n    * Unauthorised users may be subject to prosecution under the Crimes*\n\n\nLoad key \"/tekton/home/.ssh/id_gitlab-creds\": invalid<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic,keyboard-interactive).\r\nfatal: Could not read from remote repository.\n\nPlease make sure you have the correct access rights\nand the repository exists.\n","stacktrace":"github.com/tektoncd/pipeline/pkg/git.run\n\t/opt/app-root/src/go/src/github.com/tektoncd/pipeline/pkg/git/git.go:41\ngithub.com/tektoncd/pipeline/pkg/git.Fetch\n\t/opt/app-root/src/go/src/github.com/tektoncd/pipeline/pkg/git/git.go:119\nmain.main\n\t/opt/app-root/src/go/src/github.com/tektoncd/pipeline/cmd/git-init/main.go:52\nruntime.main\n\t/usr/lib/golang/src/runtime/proc.go:203"}
{"level":"fatal","ts":1600075644.4535692,"caller":"git-init/main.go:53","msg":"Error fetching git repository: failed to fetch [master]: exit status 128","stacktrace":"main.main\n\t/opt/app-root/src/go/src/github.com/tektoncd/pipeline/cmd/git-init/main.go:53\nruntime.main\n\t/usr/lib/golang/src/runtime/proc.go:203"}

Steps to Reproduce the Problem

Do as seen in this example here - https://github.com/tektoncd/pipeline/blob/master/examples/v1beta1/taskruns/git-ssh-creds-without-known_hosts.yaml

Additional Info

Kubernetes version:
Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.0+724e12f93f", GitCommit:"d5465d715197862b15c8cf05e864d7c3cfea6917", GitTreeState:"clean", BuildDate:"2019-10-10T22:03:41Z", GoVersion:"go1.12.8", Compiler:"gc", Platform:"linux/amd64"} Server Version: version.Info{Major:"1", Minor:"18+", GitVersion:"v1.18.3+08c38ef", GitCommit:"08c38ef", GitTreeState:"clean", BuildDate:"2020-07-30T18:43:47Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"linux/amd64"}

Tekton Pipeline version:

 Client version: 0.12.0
 Pipeline version: v0.14.3
 Triggers version: v0.6.1

Openshift pipelines operator version 1.1.1

Secret
kind: Secret
apiVersion: v1
metadata:
  name: gitlab-creds
  namespace: my-namespace
  selfLink: /api/v1/namespaces/my-namespace/secrets/gitlab-creds

  annotations:
    tekton.dev/git-0: git.company.com
  managedFields:
    - manager: Mozilla
      operation: Update
      apiVersion: v1
      time: '2020-09-14T09:26:36Z'
      fieldsType: FieldsV1
      fieldsV1:
        'f:data':
          .: {}
          'f:ssh-privatekey': {}
        'f:metadata':
          'f:annotations':
            .: {}
            'f:tekton.dev/git-0': {}
        'f:type': {}
data:
  ssh-privatekey: >-
    LS0V....
type: kubernetes.io/ssh-auth

Pipeline
apiVersion: tekton.dev/v1beta1
kind: Pipeline
metadata:
  managedFields:
    - apiVersion: tekton.dev/v1beta1
  name: restart-vm
  namespace: my-namespace
spec:
  params:
    - default: ''
      description: url of the git repo for the code of deployment
      name: git-url
      type: string
    - default: ''
      description: revision to be used from repo of the code for deployment
      name: git-revision
      type: string
  tasks:
    - name: git-clone-v0-14-3
      params:
        - name: url
          value: $(params.git-url)
        - name: revision
          value: $(params.git-revision)
        - name: submodules
          value: 'true'
        - name: depth
          value: '1'
        - name: sslVerify
          value: 'true'
        - name: deleteExisting
          value: 'true'
      taskRef:
        kind: ClusterTask
        name: git-clone-v0-14-3
      workspaces:
        - name: output
          workspace: my-workspace
  workspaces:
    - name: my-workspace

PipelineRun
  - apiVersion: tekton.dev/v1beta1
    kind: PipelineRun
    metadata:
      name: restart-vm-$(uid)
    spec:
      params:
      - name: git-url
        value: git@git.company.com:my-group/my-project.git
      - name: git-revision
        value: master
      pipelineRef:
        name: restart-vm
      serviceAccountName: my-bot
      workspaces:
      - name: my-workspace
        persistentvolumeclaim:
          claimName: my-pvc

My service account my-bot is also linked with the secret gitlab-creds.
I tested the ssh-keys doing a git clone git@git.company.com:my-group/my-project.git manually in the cli and it works perfectly fine. So, I'm guessing this is a bug?

This error message gives me pause: Load key \"/tekton/home/.ssh/id_gitlab-creds\": invalid format
Is it possible that this is happening because you're using data instead of stringData in the Secret?

@sbwsg When I put in the ssh private key in a Source Secret in openshift, it automatically encodes it with base64 and creates this Secret YAML for me. Moreover, stringData is plain text, since its a private key, isn't using data more secure?

Initially I think we just need to figure out if this is the core of the problem or not.  Unfortunately I'm not familiar with openshift so can't provide much help there.  But my hunch is that the encoded data field is possibly the reason that the invalid format error is being given.
All Tekton does is expose your Secrets to the container.  It doesn't do any kind of encoding/decoding of the SSH key.  So an error about the formatting of the credential implies, to me, something wrong in the encoding before it gets to Tekton.

@bharathvenkatrajan just wanted to check in here: did you ever manage to resolve this problem?  Were you able to figure out if there were indeed encoding issues or another root cause?

@sbwsg apologies for the late reply. I tried to debug more but was still getting the same error. So i switched to http auth.

/remove-lifecycle stale
/lifecycle frozen
Related https://github.com/tektoncd/pipeline/issues/3712

/remove-lifecycle stale
/lifecycle frozen
Related https://github.com/tektoncd/pipeline/issues/3712

I received the error message Load key \"/tekton/home/.ssh/id_gitlab-creds\": invalid format when a newline was missing at the end of -----END OPENSSH PRIVATE KEY-----

@muxmuse You are a lifesaver.. took me 3 hours and your post pointed my to the solution. Thanks for sharing!

/cc @sbwsg

It appears that this issue is related to formatting errors in the private key.  I'm going to close this for the time being since we don't have really good steps to reproduce nor evidence that it's a Tekton-specific problem. If someone stumbles across this in future and is able to pinpoint a specific problem in Pipelines then feel free to re-open!

@chmouel I would request to re-open this issue as we are facing this on pipelines version 0.40.2.
Few pointers that might help in understanding and resolving the issue:

We have tried to use the OPENSSH key outside the context of tekton and it works with or without the newline character at the end.
These keys are generated without a trailing EOL character, and we have to explicitly add a \n for this to work in a tekton task.
The above statements are tested in git-clone task from Tekton Hub and a custom own task, both show the same results.

This is the guide followed to mount a kubernetes.io/ssh-auth secret to the service account used in tekton pipeline.


Add numbers to boxes

A human-readable 6 digit ID. Do we need to generate this in a non-colliding way, or can accept the odd collision?

I'm not gonna manage this task today @bfirsh There is too much happening around me. I cannot concentrate.


Mismatched version in bower.json in 3.2.0 tag

Hi @daneden
In 3.2.0 tag version there is still 3.1.1 version annotation. It causes an error during bower components installation. I saw the problem is fixed, but on a master branch only. Could you bump version to 3.2.1, please? And remember to change version to 3.2.1 in bower.json again.
Thanks

This should be resolved by 813a666.

@daneden Thanks!


Fix go sec

Motivation
Fixes go sec.
Solution
Checks error when defering file closing.
Open questions

Pull Request Test Coverage Report for Build 1364

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

Totals

Change from base Build 1335:
0.0%

Covered Lines:
382

Relevant Lines:
506

💛  - Coveralls


RTC read/update is unreliable

On my MACCHIATObin (single-shot) reading and/or updating the RTC is unreliable. Sometimes the date is completely off. Changing the bus parameters to match what
mainline Linux does makes things work for me.

Fix merged:
https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commit/f1d266b838420e248281afbbb9c5abc6d640f63b
Thanks!


How to use multiple where clause using String variable

ReActiveAndroid Version:
implementation 'com.reactiveandroid:reactiveandroid:1.4.3'
Bug or Feature Request:
public static int getactualarea(String seasonid,String crop,String section) {
return (int) Select.from(PlatationMasterDb.class).where("section_id LIKE ? AND season_id LIKE ?  AND season_id LIKE ?", section, seasonid,crop).sum("manual_area");
}
Description:
this is my code

Please resolve my issue its emergency.

i solved my issue , can u close the tickets


Can't upgrade to New Relic Android Agent Gradle Plugin 7.2.0

Description
Hi, at first we encountered the same error as described in #158 , except for us it's happening for org.bouncycastle.
I tried to solve the NoClassDefFound error by upgrading to New Relic Android Agent Gradle Plugin 7.2.0, but I'm not able to build the application.
The newrelicTransformClassesForDebug task outputs
> Task :app:newrelicTransformClassesForDebug
[ClassTransform] Excluding signed or incompatible artifact[/Users/xxx/.gradle/caches/transforms-3/e73fc33f99e5a13ebe0f03ce40346279/transformed/jetified-bcpkix-jdk18on-1.76.jar]from instrumentation
[ClassTransform] Instrumentation is disabled for [org/bouncycastle/cert/AttributeCertificateHolder.class] with exception: SHA-256 digest error for org/bouncycastle/cert/AttributeCertificateHolder.class
...

but org.bouncycastle transformed classes are empty

which causes dexBuilder tasks to crash
> Task :app:dexBuilderDebug
AGPBI: {"kind":"error","text":"com.android.tools.r8.internal.vc: Invalid empty classfile","sources":[{"file":"/Users/.../Hyperspace/.../app/build/intermediates/classes/debug/ALL/newrelicTransformClassesForDebug/classes.jar"}],"tool":"D8"}

I tried to fix the empty bouncycastle classes by using the new excludePackageInstrumentation configuration, but without any success. I was able to verify, that 2024-01-08T18:52:50.743+0100 [DEBUG] [newrelic] [ClassTransform] Excluding package [org/bouncycastle/jcajce/provider/asymmetric/x509/] from instrumentation exclude is called.
newrelic {
   excludePackageInstrumentation('org.bouncycastle.*')
}

Could you please help me to solve this issue?

@t-havlicek-sap  Thank you for bringing this to us. BouncyCastle (and signed JARs in general) have been problematic for instrumentation, which is why we've excluded those classes for now. But the expected behavior is an unmodified output class, and it appear you have nothing generated. We'll get tis under the scope as soon as we can prioritize it.
Another reason we've been unable to nail it down is we cannot reproduce it in house. So if you have a simple reproduction we could use that would go along way.
If you are a New Relic customer, I recommend going through our support team, who will request some basic issue artifacts and can get a ticket started for you.

Hi @cthomas-newrelic , thank you for the quick response. Today, after an exhausting debugging I found that the issue happens when we apply both Firebase Performance Monitoring Gradle Plugin and New Relic Android Agent Gradle Plugin.
I'll link this ticket to our customer support ticket.

@t-havlicek-sap FYI, NR agent has conflicts with Firebase Performance SDK, see more details here.

I am closing this issue as resolved.
It would be nice to mention at least in Android agent compatibility and requirements part of your documentation, that New Relic Android Agent Gradle plugin is not compatible with Firebase Performance Monitoring Gradle Plugin. You could generate a warning like you're generating for Android Gradle Plugin version compatibility.
From the user experience, it would be nice if the plugin would fail the newrelicTransformClasses task in case it transforms any empty classes with user actionable feedback.


[CMake/Macros] Set correct RPATH

Previously it expected the host (i.e. the compiler) already loaded in the process. But since that's not guaranteed, let's set the correct RPATH.
Also, stop importing string-processing and backtracing.

@swift-ci Please test


[0.22] undo the hack, part of #510

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>Proposed Changes

correct the release branch apply mechs.

Release Note

Docs

/hold
let's see how this goes on this branch

/unhold


Need change the sequence of rename features

The rename-rules is happened before prepend-rp-prefix and rename-mapping so the 'Serialized Name' in the document might not right.
For example:
Serialized Name: NetworkRuleSetIpRule

But here the Ip is renamed to IP, so if we use NetworkRuleSetIpRule as the key to do prepend-rp-prefix and rename-mapping then nothing happen

PR:
https://github.com/Azure/autorest.csharp/pull/3677
https://github.com/Azure/azure-sdk-for-net/pull/38379


LIVERY SUBMISSION：CHINA EASTERN A350-900(1st A350 delivered form China livery）

1st is 4096,2nd is 2048

@kolos26 i did not see it in game?

This one will be solved tomorrow, others should work now
bili-開飛機のzm @.***> ezt írta (időpont: 2024. jan. 16., K
1:48):

@kolos26 https://github.com/kolos26 i did not see it in game?
—
Reply to this email directly, view it on GitHub
https://github.com/kolos26/GEOFS-LiverySelector/issues/440#issuecomment-1892917694,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASSSWFWCBMQTRH3RFUSHZITYOXE6VAVCNFSM6AAAAABBYAGBWCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJSHEYTONRZGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Create role speakup if not exists during flywayMigrate build task

When you don't have created user role speakup in DB speakup then you can't run application by typing gradlew bootRun. This error is being printed into terminal:
Migration V1__speakup.sql failed
SQL State  : 42704
Error Code : 0
Message    : ERROR: role "speakup" does not exist
Location   : src/main/resources/db/migration/postgres/V1__speakup.sql (/home/telenv01/IdeaProjects/speakup/src/main/resources/db/migration/postgres/V1__speakup.sql)
Line       : 64
Statement  : ALTER TABLE users OWNER TO speakup
ERROR: role "speakup" does not exist

Add below SQL into migrating script.
CREATE ROLE speakup LOGIN
VALID UNTIL 'infinity';


Fix Ecto.Schema's many_to_many documentation

Small things

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:

@josevalim hey, should these docs updates be merged to v2 branch? Hexdocs currently still has wrong information, even though v2.0.6 was pushed recently. If you want, I can maybe do a PR with all the doc fixes that have been commited to master but not to v2.


Add madvise support

I added a branch that implements this but I'm not sure about the API yet. If anyone is checking this out, I'd really appreciate some feedback.

Madvise can actually be called anyway be the programmer after invoking you library. So not sure if it is a priority, is it?
Anyway, I think either a template tag or a parameter passed to the map function is fine.

@amallia Thank you for chiming in!
Yes, that is true for POSIX, but as far as I know, Windows has unfortunately no such equivalent--you have to provide the file cache hint when opening a file handle, i.e. when calling CreateFileA.
But since then I've realized that this would break the symmetry of the API, in that one wouldn't be able to specify the cache hint on Windows when opening a mapping with an existing file handle.
So you're probably right in that if a user wanted this feature, it's trivial to just call: madvise(mmap.data() - mmap.offset(),  mmap.mapped_length(), MADV_RANDOM), and on Windows they'd need to open the file handle themselves with the desired flags. Not the best, considering my goal was abstracting away the need to differentiate between operating systems, but it will have to do.


Dump ITAB_DUPLICATE_KEY in method CALCULATE_STATUS (ZCL_ABAPGIT_FILE_STATUS)

While trying to update the code for the course Writing Testable Code for ABAP I get a short dump
   SORT lt_items. " Default key - type, name, pkg
   DELETE ADJACENT DUPLICATES FROM lt_items.
   lt_items_idx = lt_items. " Self protection + UNIQUE records assertion  <<<<<<<<< DUMP here

I think it might be due to this entry:

my current workaround
    DELETE ADJACENT DUPLICATES FROM lt_items COMPARING obj_type obj_name. " devclass

hmm, I imported the new zip without problems
guess your main package is "$WTC_EXERCISES"? which does not leave room for "REF_REBATE_ENGINE"
From my system, using "$WTC1" as main package,

Hi Lars,
your guess is correct.

Change to be implemented in abapGit:
Error message if the package name gets too long

This problem (short dump) can be caused by various means. For example if I try to download a large set of objects using ZABAPGIT and whilst that is in progress open another session and then try to open ZABAPGIT again then both sessions dump and thereafter you cannot call transaction ZABAPGIT without getting a short dump.
I know how to fix this and will tyr to create a pull request.

Today I run also into this issue when cloning the ABAP SDK for Azure repository into a fresh system. For the moment I've fixed it by using the fix by @hardyp.

@gregorwolf are you running the latest abapGit version? What is the package name you are cloning into?

Just updated abapGit from GitHub. I've used $ZADF as the package to clone into.

Okay,
Original issue: error message to be implemented if the package name gets too long
New issue found by Gregor, when installing the Azure SDK into $ZADF abapGit dumps. As $ZADF is part of the folder structure, https://github.com/Microsoft/ABAP-SDK-for-Azure/tree/master/src/zadf (FULL folder logic is used) some unexpected stuff happens. We need to implement logic which does not allow the root package to be named identical to one of the subpackages in the repo.


view messages sent by operators

(The operators are sending messages and it isn't possible to view them using the POST website.) The link on the website doesn't go anywhere

Put in the wrong repo