Add support for reading OIDC clients from json formatted file

Convenient way to pre-populate OIDC clients from readable json formatted file, instead of inserting records in MongoDB or dynamic registration. setitem, delitem and pop are honoured but will not persist.

Have you considered using pickle?
There are other cases where data could be persisted. There could be a global shelve object that pickles data under appropriately configured keys. That would enable SATOSA to be restarted, like no interruption occurred.

pyOP, which is what we are using to implement the OIDC frontend, already supports client databases in json format that follows OIDC client metadata.
We probably need to make this more visible in the documentation

Please enlighten me, for I only scratch our itches and didn't find anything in SATOSA, nor the pyOP lib that met our wish for simple prepopulation of OIDC clients. pyOP/storage.py only contains references to MongoDB?

In SATOSA (and anything that uses pyOP), when you create a new Provider class, you can pass a clients parameter which is a deserialized json string ( created from a json formatted string or file  with json.load() for instance )
The format of that json file is described in the spec

Aha... so you suggest to stay with Mongo or memory but use the provider constructor to bootstrap with a file containing the "hardcoded" clients? How would mongowrapper react to seeing the same client twice after restart?

Mark, I'm sorry but I didn't follow your thought above... We are using json formatted client db files for many projects ( with no MongoDB backend ) just fine

I'll pull the pullrequest and rethink our strategy for populating the OIDC client DB, without the need for restart.


Update DB2DataSourceCreator.java

The validation query is incorrect for DB2.  It works for derby but not DB2.    The query "select 1 from sysibm.sysdummy1" works for both.

I'm hesitant to accept this change without knowing how it might affect other DB2 users. I assume the existing validation query has been working for other DB2 users (including the original contributor of the DB2 support code @mariobriggs), so this may be dependent on the DB2 server configuration. I would like to give others some time to comment on the change.
In the meantime, you can customize the validation query used when the DB2 connection is created as shown here: https://github.com/spring-cloud/spring-cloud-connectors/issues/144#issuecomment-158548898.

Closing this PR due to lack of information on how DB2 server configuration affects the validation query that should be used, and the fact that the validation query can be customized as necessary.


First attempt at fixed scrolling

This is an attempt to fix #29. There's a few issues with using tbody and thead and making it float, but I think this works pretty well. The only issue that I can see is that it doesn't resize the table when resizing the window, and the em breaks for fs-repo-migrations.
@harlantwood thoughts?

Thanks!  Can you push it to IPFS or gh pages so we can test it from different devices?
npm run publish from your branch should push to gh pages, or npm run build  + ipfs add -r public

I can test on Android & osx...

Pushed!

Error?

The page build completed successfully, but returned the following warning:

CNAME already taken: project-repos.ipfs.io. Check out https://help.github.com/articles/my-custom-domain-isn-t-working/#cname-already-taken for more information.

Works for me at: http://RichardLitt.github.io/ci-status
Testing on Android now...

Hm, on my Android device I see rows bleeding through above the headers... Screenshot:

(Chrome on Nexus 6, marshmallow I believe)

I wonder if jquery datatable has any built in (or pluggable) way to do this...

This looks like a good candidate: https://datatables.net/extensions/fixedheader/

Yeah, that bleeding is expected. I don't mind it, it helps show me that that element is floating and that you can go up or down. To fix it, we'd need to just offset the global 15px padding on body, I think.

We could also use jQuery. Want me to try that?

Hm, I see your point about visibility but I think the bleeding looks pretty bad visually.
We are already using jquery datatabble, so I think just a line of code somewhere from the page I linked to might do it.

I created a second attempt in #36.

Fixed in #36


Change in avalanchego binary and image causes docker compose up to fail

$ docker run --rm avaplatform/avalanchego:latest pwd
/avalanchego/build

$ docker run --rm avaplatform/avalanchego:latest ls
avalanchego
plugins

Description of problem
<REPO>/docker-compose/files/start-nodeX.sh runs build/avalanchego in container which does not exist. Latest avalanchego image build has moved PWD from /avalanchego to /avalanchego/build.
Description of problem 2
--db-enabled flag has been deprecated / removed

Hi @otherview is it possible to merge PR #10?


Update to Ruby 2.1.9, fix dependencies, and cleanup

Update to Ruby 2.1.9
Add .ruby-version for version managers
Update dependencies
Satisfy Rubocop
Remove superfluous starter directory containing a copy of the project

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

This project did not work out of the box with the steps given, due to dependency problems. I have updated the README.md to reflect changes.


Add groovy installation profile configuration.

As a developer,
I want a groovy installation profile to be configurable via environment variables
So that I don't have to manually configure the Groovy profile.
Scenario: Test with env variable GROOVY_VERSION='2.4.3'
Expected:
ADOP groovy profile with version 2.4.3 to exist.
Scenario: Test with env variables GROOVY_VERSION='2.4.3,2.4.8'
Expected:
The following profiles to be installed:

ADOP Groovy_2.4.3
ADOP Groovy_2.4.8

The installation is inspired from the maven setup in which the GROOVY_VERSION env variables takes a comma delimited list of Groovy profile versions to install.
Signed-off-by: Northard, Robert A<EMAIL_ADDRESS>

LGTM.


Add Cirrus CI or LUCI tests to web_installers repo

As mentioned in PR: https://github.com/flutter/web_installers/pull/6
web_installers repo currently is not supported by automated tests running on presubmit.
/cc @yjbanov @godofredoc

Superseded by https://github.com/flutter/flutter/issues/98552


newEpoch event

newEpoch event to clean up the blockPriorityQueue and keep a count of orphan blocks in the previous epoch.

Thanks for your contribution @AyushKaul
Sorry for the very late reply / review.
Unfortunately, I don't think we'll need this block queue cleanup as is. @mpetrunic is currently replacing the current block queue (with polling) with a block pool that exposes an async iterator, has a different internal implementation.
I can tell you did a good bit of digging in our codebase to figure out this PR, we'd definitely welcome further contributions :) come join our discord chat https://discord.gg/yjyvFRP

Closing this PR, see above comment :)


Control trafic user

can control the total traffic of the connecting user. ? In my case I do not want to exceed the traffic of my vps

Have you solved your problem?

noup


Update support.md

Grammar correction.

@Grindelek for future PRs, can you please group multiple changes into a single PR. If you make a change on one branch, continue adding to that branch and then PR it.

Of course, I'll try it, but I'm new at GitHub, so could you give some hint about it or link a tutorial?

@Grindelek check out: https://github.com/WGBH/pbucore/wiki/Contributing-to-the-project-through-Github-web-interface

@Grindelek I haven't reviewed all of these yet, but it looks like you were on a different branch when making changes. This PR is also trying to merge those changes into lbryio:feb-update as opposed to master. Going to close - please resubmit after reading through the github docs.


Checks on Latitude-Longitude & Distance-Area & links between tables

[ ] Values outside the specified range should give errors in case of Latitude and Longitude
[x] There should be a cross check for Distance & Area (at least one should hold a positive value)
[x] Turning Distance into float(6) should coincide with a specified range Lower:0.000100000000 Upper: 9999.000000000000, as in Area #102
[ ] There should be a check of the links between the tables (are all CampaignID's in Samples present in Campaigns, etcetera...) (see issue 117 and file screening 128)

@Osanna123 can you implement these checks? Would be good that they are implemented before testing the full upload.

Should also be tested with a file that purposefully contains errors (cf. #117).

Anna is working on this, in progress

Implemented; tested but @Osanna123 to test "Values outside the specified range should give errors in case of Latitude and Longitude"

done


Fleet: resources age & last modified

There's a request to provide a Resource Age along with a Last Modified Timestamp
Internal reference: SURE-4651

@gaktive Where?

I think we need the backend to add this - otherwise we'll have to fetch all of the resources which is expensive.

This is on the resources tab that is used in a few places.

@nwmac is there an existing backend ticket to work on this? or shall we transfer this over there?

@gaktive Created https://github.com/rancher/rancher/issues/36173 for the backend work for this

"Last modified" is not a concept k8s keeps track of for resources in general; they can't provide something that doesn't exist.
"Age" is (metadata.created), though it's just regular mutable user data so isn't necessarily when it was actually created.

The backend ticket was missing an area3 label, so I didn't see this until now. I highly doubt it will get into 2.6.4.

Related backend moved to 2.6.8, though @Jono-SUSE-Rancher & I are uncertain about that milestone yet, so moving to 2.6.x.

@gaktive @aalves08 Should this issue be closed as stale?

@catherineluse the related backend is still open but without milestone. I'll check with Team 3 what their plans are.

Depends on backend changes - https://github.com/rancher/fleet/issues/1184

Depends on backend changes - rancher/fleet#1184

Backend is done, pending QA approval

As I understand the state of things, this will require a Fleet version bump on rancher/rancher. https://github.com/rancher/rancher/issues/36173#issuecomment-1629247008 has the appropriate details.


Arch Linux box hangs at reboot

I was trying to build an arch linux x86-64 box on Virtualbox 4.3.2, using Packer and veewee-to-packer, when the guest os simply hung. The provisioning script running/waiting when arch linux hung was reboot.sh.
On further investigation, I found that this issue could be caused by the latest updates to systemd. But if there's any way to not run provisioning scripts that would cause it to go to that state, I'll be thrilled.

closing as there was no feedback in three months, please let me know if you can provide more details and I will reopen


Security Management UIs

Implementation of the following JIRAs:
https://issues.jboss.org/browse/DASHBUILDE-100
https://issues.jboss.org/browse/DASHBUILDE-99
The ability to configure custom security policies will make possible to entirely manage the dashbuilder security configuration from the UI. Once the above features are available it will be possible to:

Create new groups of users
Create new users
Add users to existing groups
Set access permissions to groups over existing dashboards
Set the home perspective/page the user will be redirected after login

NOTE: This PR depends on https://github.com/uberfire/uberfire/pull/321

@psiroky the build is also failing here... ¿?

@dgutierr seems like the same/similar failure as for UberFire?

+1 after uf pr's merge


⚠ Slack has degraded performance

In 1946afb, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 383 ms

Resolved: Slack performance has improved in b6debe5 after 4 hours, 21 minutes.


One set of image & json will be missing

There is always one set of image and json missing when we have multiple layer configuration and shuffleLayerConfiguration.
For example , with the setting below :
const layerConfigurations = [
  {
    growEditionSizeTo: 2,
    layersOrder: [ ...]
   },
  {
    growEditionSizeTo: 5,
    layersOrder: [ ...]
   }
];

const shuffleLayerConfigurations = true;
The 2.png & 2.json will always be missing in the build.

I have made a quick fix on the issue above , in the /src/main.js
The quick fix is to change the line in while loop to this :
editionCount <= layerConfigurations[layerConfigIndex].growEditionSizeTo + 1
const startCreating = async (storedDNA) => {
  if (storedDNA) {
    console.log(`using stored dna of ${storedDNA.size}`);
    dnaList = storedDNA;
  }
...
    while (
      editionCount <= layerConfigurations[layerConfigIndex].growEditionSizeTo + 1
    ) {
      let newDna = createDna(layers);
   ...
};

No longer an issue


Update 'Picking the Right Solution' with information from VMware/Pivo…

…tal offerings
Added VMware Cloud PKS and Cloud Foundry Container Runtime to places within the document that are correctly represented for Cloud and on-primises offerings and tooling. I also properly alphabetized the list.
Signed-off-by: Kendrick Coleman<EMAIL_ADDRESS>

/assign @zacharysara


Forge test: call traces not decoded after upgrading to 0.2.0

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (613da19 2022-03-28T15:06:43.699738100+00:00)
What command(s) is the bug in?
forge test -vvv
Operating System
Windows
Describe the bug
Provided two contracts:

regTest.sol

pragma solidity ^0.8.10;

contract RegressionOriginal {

    address public toDecodeAddr;

    function decodeThiss(uint256 amountDecode, address toDecode) public returns (address[] memory, uint256, uint256[] memory) {
        address[] memory addArr = new address[](2);
        addArr[0] = msg.sender;
        addArr[1] = toDecode;

        toDecodeAddr = toDecode;

        uint256[] memory uintArr = new uint256[](2);
        uintArr[0] = gasleft();
        uintArr[1] = amountDecode;

        return (addArr, gasleft(), uintArr);
    }
}

and

regTest2.sol

pragma solidity ^0.8.10;

contract NotRegressionTheOriginal {

    address public toDecodeAddr;

    function decodeThis(uint256 amountDecode, address toDecode) public returns (address[] memory, uint256, uint256[] memory) {
        address[] memory addArr = new address[](2);
        addArr[0] = msg.sender;
        addArr[1] = toDecode;

        toDecodeAddr = toDecode;

        uint256[] memory uintArr = new uint256[](2);
        uintArr[0] = gasleft();
        uintArr[1] = gasleft();

        return (addArr, gasleft(), uintArr);
    }
}

combined with a test contract,

regTest.t.sol

pragma solidity >0.8.10;

import "../../lib/ds-test/src/test.sol";

import "../regTest.sol";
import "../regTest2.sol";

contract RegressionTest is DSTest {

  function testDecodes(uint256 fuzzUint, address fuzzAddr) public {
    RegressionOriginal contractOrig = new RegressionOriginal();

    contractOrig.decodeThiss(fuzzUint, fuzzAddr);
    fail();
  }

  function testContractsAreNotEqual() public {
    assertTrue(keccak256(type(NotRegressionTheOriginal).creationCode) != keccak256(type(RegressionOriginal).creationCode));
  }

}

Installing forge/cast from source or through foundryup, using the latest v0.2.0 shows issues with decoding the contractOrig.decodeThiss call. In my case it shows two unexpected behaviors:

The function call is not decoded, along with the input being not decoded.
The wrong contract is shown in <contractName>::<function>.

regTest.t.sol tests for both contracts not being the same (although highly similar, note the additional s in decodeThiss and the uintArr[1] difference).

running forge clean does not solve this issue.
More peculiar, when you:

delete the 'regTest2.sol' file and
remove the references in the regTest.t.sol to that file

regTest.t.sol

pragma solidity >0.8.10;

import "../../lib/ds-test/src/test.sol";

import "../regTest.sol";
//import "../regTest2.sol";

contract RegressionTest is DSTest {

  function testDecodes(uint256 fuzzUint, address fuzzAddr) public {
    RegressionOriginal contractOrig = new RegressionOriginal();

    contractOrig.decodeThiss(fuzzUint, fuzzAddr);
    fail();
  }

  // function testContractsAreNotEqual() public {
  //   assertTrue(keccak256(type(NotRegressionTheOriginal).creationCode) != keccak256(type(RegressionOriginal).creationCode));
  // }

}

the decoding of the function & function input works. The correct contractname is also referenced in the call trace.

After adding back the original regTest2.sol file (and running forge clean as desired), the unwanted behavior returns. This is even without having the contract in regTest2.sol referenced in the test file:

Upon downgrading to v0.1.0 using cargo install --git https://github.com/gakonst/foundry --rev 22323e21c3e00d9eccc5914589c23f2a0115413b --bins --locked , the unexpected behavior does not show. I.e., all function calls are decoded correctly and the contract names are referred to as expected.
As I assume it might be something cache related, I hope to be able to reproduce it on another machine "from scratch", but haven't had the chance to yet.

This is due to how we identify contracts - they are identified using fuzzy matching of the bytecode, and the bytecode of both of these contracts are very similar

Understood. This was the smallest repro example I could create.
I notice on our main project that almost all of the calls are not decoded (and the contract names aren't shown either) on 0.2.0. Just a solmate's ERC20.transferFrom and a sporadic ERC20.mint seems to come through.

The project does work with factories & proxies, perhaps a similar rootcause?

Should work with proxies - are the bytecodes of all your contracts very similar? I think the threshold is 10%

Should work with proxies - are the bytecodes of all your contracts very similar? I think the threshold is 10%

We have a series of contracts that do inherit the same abstract contract, but only about 5 of them inherit that same contract (from around ~30 which don't get decoded). The other contracts are very different, asides from an occasional shared openzeppelin's Ownable for example.
In our tests, we do create many instances of the same contracts. For example, a series of ERC20's that are based on the same contract, a series of ERC721's, series of ERC1155's, series oracles, ...

Also, do traces for your main project decode properly with 0.1.0?

Yes! All traces for all calls, including contract names, function names and input decodes work perfectly on 0.1.0.

@jparklev if you have any more minimal example it'd help! thanks!

I went to produce a minimal example and was having some trouble, so I went back to my original, cleaned the cache, and now it works as expected. Not sure how I got into the state before, maybe a vestige of a previous version in my cache? At any rate, will come back here and try to track it down if I run into it again.
(what I was seeing was that the trace wasn't decoding solmate ERC20 token function sigs or events)

faced the same issue when I had artifacts/cache from forge 0.1.0. once i cleared the cache, everything showed perfectly

is this still an issue @ArcadiaFinance @jparklev? Or has it been fixed?

I assume this is no longer an issue, feel free to reopen

Hi @brockelmore, @gakonst, Thomas from Arcadia here.
Seems that it is no longer an issue, thanks for the efforts looking into it!


Removes REQUIRE file and updates Project.toml

Removes upper bound on Images package

I think you could just delete Manifest.toml; it'll be generated if not existed.

@JuliaRegistrator register()

Pull request comments will not trigger Registrator as it is disabled. Please trying using a commit or issue comment.


[TheYeteeBridge] Fix HTML parsing

TheYetee is now on Shopify, HTML changed so the parsing must change too.

Fix #1987

gj @Chouchen !


[Improvement] Typewriter Effect in the Main heading

We can add a typewriter effect in the main heading to give it a dynamic effect and improves overall visual appearance of the website
Please assign me this issue under Gssoc'23 @thestarsahil

@Anshika-Jain02
Thank you for your interest in working on this issue. I appreciate your suggestion to add a typewriter effect to the main heading, which will enhance the visual appearance and create a dynamic effect on the website.
Having an open discussion will help ensure that we collectively agree on the best approach and implementation for this feature. It will also provide an opportunity for collaboration and gathering valuable feedback from the community.
Please initiate the discussion and provide the necessary details about your proposed typewriter effect. I'm looking forward to seeing the conversation unfold and collaborating with you on this exciting enhancement.

Thank you sir for considering my addition to your project. As you must have seen that when we open the main page of GitHub preceding the login page a typewriter effect comes. I want to add a similar typewriter effect in the heading "Still Confused with College Choice" using the javascript setTimeout feature and then it creates an appearance as someone is typing the text in real time.

@thestarsahil can i start working on this?

@thestarsahil can i start working on this?

Yes i already assigned you


[OAP-Cache-OAP] fix plasma metrics

What changes were proposed in this pull request?
Fix plasma metrics, cache size is 0 on web UI.
How was this patch tested?
Verified via spark web UI.

LGTM for a check workaround. Please file a ticket for future fix as well.


Add Sakupen Circles X Ganstas Paradise by pengwino

dont do this one cause it dont sync but the other one i made did sync

reject


DRY-up jsonapi.go

The file jsonapi.go has a lot of repeated code. For example, the line

w.Header().Add("Content-Type", "application/json")

appears 33 times in just that file. jsonapi.go needs DRYying up.

This is something I want to refactoring using a better router (ideally gocraft/web) when theres time but if it's an issue now feel free to fix it.


Update check-device-connectivity.yaml

Hello,
Proposal to tackle https://github.com/camaraproject/DeviceStatus/issues/17
I tried to align in parallel with the work in progress for Device Location. I'm looking for your comment.
Thanks

Close this one and I will open a new one following discussion on issue #17


[Bug]: Asset Combiner prevents usage of CSS var()

What happened?
Version is 3.1.1
Anywhere in a theme's scss, put in a block like
$sass_var: 100px;
padding-top: var($sass_var);

And then compile to CSS with the method of your choice. You should see that the compiled css (in case of orange theme at assets/css/app.css) the $sass_var has been compiled into
padding-top: var(100px);

Now, make sure that when 'enableAssetCombiner' => FALSE in config/system.php, your site will correctly load app.css and you'll see the padding-top: var(100px); line in your css.
Turn on asset combiner so that 'enableAssetCombiner' => TRUE and load the same page, which will now point to a minified version of your css that is specifically missing padding-top: var(100px);
What did you expect to happen?
var() is valid CSS as per https://developer.mozilla.org/en-US/docs/Web/CSS/var() and I think should be allowed to exist here?
Version
Other (please detail)
What browser are you seeing the problem on?
Firefox
Relevant log output
No response

I'm unable to reproduce the error, works fine for me on the frontend and admin. Could be caching issue?

Oh, sorry for the delay in reviewing. :)

I'm pretty certain it's not a browser-side caching issue but it may be a Laravel caching issue that I'm not aware of.
I've made a fresh installation to test it at https://www.tasty-testing.com/ to make sure it's not a side effect of my other plugins - you can currently see that at the bottom of https://www.tasty-testing.com/_assets/1452465ec47c99502dae464829a8b2a5-1637093243.css, there is .bug_testing{padding-bottom:100px} whereas I've added
.bug_testing{
$sass_var: 100px;
padding-top: var($sass_var);
padding-bottom: 100px;
}

to the bottom of themes/tastyigniter-orange/assets/src/scss/app.scss (and then hit save on the template in the admin ui in order to compile sass). The fact that padding-bottom: 100px; shows up tells me that it's compiling and that it's getting through the cache.
I'm sorry to be pointing you at such an esoteric and probably-inconsequential bug! I'm happy to try and help but I frankly don't have any idea how the asset combiner works, and it's a bit above my level of understanding of Laravel.

There's no need to apologise. The asset combiner was adapted from Symfony's Assetic Manager and it seems the issue is coming from minifying the CSS, disabling minify outputs the CSS as expected. We were using an outdated CSSMin package, I've replaced it with one adapted from octobercms.
https://github.com/tastyigniter/flame/commit/7d8f5b9a4ee82f417c01f093effe1a4887b9f46c
https://github.com/tastyigniter/TastyIgniter/commit/1db10760e626072ecab88c027f247a3bd9a4c7b3


Plat DA15000: Update the output binary name in the "imgprep" script.

Codecov Report

Merging #1661 into master will decrease coverage by 1.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1661      +/-   ##
==========================================
- Coverage   68.02%   66.54%   -1.48%
==========================================
  Files         169      169
  Lines       20226    21134     +908
  Branches     2506     2594      +88
==========================================
+ Hits        13758    14064     +306
- Misses       5605     6162     +557
- Partials      863      908      +45

Impacted Files
Coverage Δ

src/core/net/ip6.cpp
54.71% <0%> (-8.97%)
:arrow_down:

src/core/api/thread_api.cpp
32.27% <0%> (-4.39%)
:arrow_down:

src/core/mac/mac_frame.hpp
68% <0%> (-2%)
:arrow_down:

src/core/thread/mle_router.cpp
77.46% <0%> (-1.61%)
:arrow_down:

src/core/net/udp6.cpp
82.85% <0%> (-0.96%)
:arrow_down:

src/core/net/ip6_mpl.cpp
92.3% <0%> (-0.77%)
:arrow_down:

src/core/coap/coap_client.cpp
52.38% <0%> (-0.6%)
:arrow_down:

src/core/thread/mle.cpp
81.56% <0%> (-0.36%)
:arrow_down:

src/core/thread/network_data_leader_ftd.cpp
68.54% <0%> (-0.24%)
:arrow_down:

src/core/thread/mesh_forwarder.cpp
78.47% <0%> (+0.48%)
:arrow_up:

... and 1 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 7c7519f...60a19d8. Read the comment docs.


add subscription groupings, lower default time, fix name

PR Checklist
Made some fixes to HDInsight At-scale workbook.

Grouped all overview tabs by subscription
Changes the name to "HDInsight Clusters" from "HDInsight At-Scale Insights"
Reduced default time value from 30 to 15 mins

Can you post some screen shots so I can preview what your changes look like? thanks!


The method iter_lines of CurlHttpEventStream does not guarantee the security of re-entry

iter_lines () does not guarantee the security of re-entry. Multiple calls to this method will result in partial loss of data.

You mean it's not reentrant? I don't see how that's a problem, it never calls that method from a different context (thread).


Add Reactify transform for Browserify

This fixes an unexpected token error when requiring react-tabs in browserify.

+1

@mzabriskie any idea when this will be released?
thanks!


Correct the objectHash when the value is null

the jsondiffpatch executes objectHash and may pass in null as the obj argument value
This PR takes null into account and returns the objects hash-index if that were the case.

[x] commit messages are commitizen-friendly
[x] code is unit tested
[x] code is integration tested
[ ] public code is documented
[ ] code is reviewed by at least one person
[x] code satisfies linter rules

cc @ct-dali @PhilippSpo

Coverage decreased (-1.8%) to 93.399% when pulling cee991defce7e154294639e42e9c428beec0e4b7 on adnasa:correct-objectHash into bfa1a2191847c70363b8a7ee225363fe4dfc5438 on sphereio:rewrite-2.0.

This is not acceptable without a test case that reproduces the problem.

Coverage decreased (-1.8%) to 93.399% when pulling 71b628191a580e723c5cc2f0ba297b624de01107 on adnasa:correct-objectHash into bfa1a2191847c70363b8a7ee225363fe4dfc5438 on sphereio:rewrite-2.0.

Coverage decreased (-1.5%) to 93.68% when pulling fe4f28e12d5fea56d6712d2f6488d8f0f0bceddd on adnasa:correct-objectHash into bfa1a2191847c70363b8a7ee225363fe4dfc5438 on sphereio:rewrite-2.0.

Coverage decreased (-1.5%) to 93.68% when pulling 71c5ca0fe97399e71fa76645997e95bec22a5d1c on adnasa:correct-objectHash into bfa1a2191847c70363b8a7ee225363fe4dfc5438 on sphereio:rewrite-2.0.

I don't understand why the coverage went down...

@adnasa @PhilippSpo what's the status here?

Coverage of untouched files went down

Don't worry about the coverage

The only thing we could add is an integration test then.

I'll try to find the time today to figure why test coverage went down.
Other than that, I think we are good :)
cc @PhilippSpo

I'd like to have this a go
cc @PhilippSpo @emmenko
I think one of the main reasons that coverage went down is because more code is added and in the case of http and task-queue utils, some utility were not covered.
Trying to cover them would be out of scope of this PR

🚀

🍹
cc @PhilippSpo @emmenko


[feature/new-editor]: implemented CodeEditTextView

Implementation of CodeEditTextView with use of tree-sitter for syntax highlighting. See the documentation for more information.

This will be merged to a feature branch first until more languages are supported. Implementation is subject to change.

Currently supported Languages

Go
GoMod
HTML
JSON
Ruby
Swift
YAML

Note that running in Debug configuration will take a huge performance hit on syntax highlighting initialization. Change to Release when not debugging.
Performance hits only occur when first opening a file of a supported language. Every following file will not initialize the highlighting queries again and thus will happen immediately.

@lukepistrol github.com/mattmassicotte/tree-sitter-go branch "feature/swift" https://github.com/CodeEditApp/CodeEdit/blob/d87fc333a8d1558379c86f1be5fb57dd0eaeaf27/Package.resolved#L107
has been removed made this commit could not be rebuilt
maybe mattmassicotte will do the same to other mattmassicotte/tree-sitter-* in the near future

@ryh Please update your packages. CodeEditTextView already uses the official tree-sitter source for the Go language.


Fitted Trend Line and Residuals

Hello,
I see you have provided examples on predicting 12 intervals in the future. But I seek past values. Can you please give more examples on how to generate fitted line and residuals on a time series using arima and varima?
Thank you!

Hi,
The code does not currently have a simple interface for producing in-sample predictions for the levels, using VARIMA models (it would require some small changes in the forecast function). However, it is quite easy to get the in-sample forecast / residuals of the underlying VARMA models. Would that help?

Thanks for your reply!
I am not an expert with time series decomposition. Can you give a brief example on how to extract the trend (forecast) and residuals for in-sample (past data) using VARIMA?

Hi,
Trend-cycle decompositions based on VARIMA models are not implemented yet. I would suggest you to take a look at the multivariate Beveridge and Nelson decomposition.
Alternatively, you could use any state-space model described in Harvey (1990). They are actually much easier to implement with the current status of the package. I am happy to give you more details on that if you can let me know which model would work best. An example is already reported in the Estimation of state-space models section in the readme.
--
Bibliography
Harvey, Andrew C. Forecasting, structural time series models and the Kalman filter. Cambridge university press, 1990.

@fipelle oh I was commenting at the same moment you were replying :)
I will have a look at it, and get back to you.

Happy to help! I am currently updating the library offline, I will start again adding features relatively soon. The Beveridge and Nelson decomposition in on my list, but there is quite some stuff to add before that I am afraid.
PS: As mentioned, you can get the in-sample prediction and residuals for a VARMA (not a VARIMA). If that is what you actually want to do I can work out an example.

I am moving it to a private todo list and closing the issue. I should get back to it soon. Please take a look at MessyTimeSeriesOptim.jl in the meantime, as I may implement it there.


Add TypeVar to enable better subclassing

Can you give an example how it will be used?

Can you give an example how it will be used?

As context, I was originally working on https://github.com/home-assistant/core/pull/75477 when I discovered this. Home Assistant itself subclasses BluetoothServiceInfo (although it isn't an issue in this particular instance since from_advertisement is overwritten).
Example
Say we create a new subclass with additional attributes and / or methods without overwriting from_advertisement.
@dataclass
class SubClass(BluetoothServiceInfo):
    some_attr: int = 10

    def some_method(self, var: int) -> None: ...

Calling from_advertisement on SubClass will return an instance of BluetoothServiceInfo and not actually SubClass. Thus accessing some_attr or some_method will cause type errors.
info = SubClass.from_advertisement(...)
reveal_type(info)  # BluetoothServiceInfo
info.some_method(2)  # type error

The solution for it is to use a TypeVar bound to the base class as done here. Once mypy supports PEP 673, typing.Self can be used instead.
--
It's usually not necessary to update the typing if everything still works. There are a few similar cases in Home Assistant, for example. However, since this is an external library any update will take longer than changing just HA alone which is why I openned the PR.

Would like to get Martin a chance to comment before merging anything

I'm assuming this came about when you saw the with Bleak version.

Yeah, although not (yet) necessary. Good practice, especially in a library, never the less.

Would like to get Martin a chance to comment before merging anything

https://github.com/home-assistant/core/pull/75477 is independent from this one. So there isn't any rush.
@bdraco Would you mind approving the workflow run? I would like to see if the commit message is fine now 😃

I think the commit message should use a lowercase letter after the colon.

I think the commit message should use a lowercase letter after the colon.

Thanks! Let's see if it works now.

Forgot about it earlier, an example where using TypeVars does make a difference: https://github.com/home-assistant/core/pull/75472


Use statefulset for prometheus alertmanager

Signed-off-by: Olivier Vernin<EMAIL_ADDRESS>Recent prometheus update was blocked because a volume was already attached the exist container, switch alertmanager to a statefulset better handle such usecases

I am merging this right now as I already manually did the change to unblock infra.ci


Token-based authentication to Azure SQL Server

I have need to authenticate to Azure SQL Server using token based authentication. This is possible in other languages like Python because the ODBC packages like pyodbc support pre-connection attributes (https://github.com/mkleehammer/pyodbc/issues/228). However, none of the R ODBC packages: RODBC (https://www.rdocumentation.org/packages/RODBC), ODBC (https://www.rdocumentation.org/packages/odbc), or r-dbi (https://www.rdocumentation.org/packages/DBI), support pre-connection attributes.
I can't use ActiveDirectoryPassword authentication because this doesn't work from Mac/Linux when the user is a guest of the tenant.
It'd be great if MS could provides means of token-based authentication to Azure SQL Server.

I think this belongs more in the odbc repo. Maybe @jimhester could comment.

Open feature request here: https://github.com/r-dbi/odbc/issues/299

@jeffon12 I'm looking in using this in my workflow. How is your Work around holding up?
If I'm allready using reticulate in my workflow, how hard is it to implement this solution?

Dependency management turned out to be too difficult with reticulate, so we ended up wrapping the R code with Python which performs the query, writes the results to a file, and then calls R.


Fix bug: Get wrong log for retried task & frontend crash

When calling getTaskAttempts API. The task attempt list returned and sorted by DESC order of attemptId.
Previous code treat the list as ASC order and return wrong log content for retried task.
Fix this issue, as well as frontend bug.
Fix https://github.com/microsoft/pai/issues/5125

Coverage decreased (-0.02%) to 33.977% when pulling aefa9cf0340bd52c1caa758244b909ce83a48f48 on binyli/log-bug into 0be199f3adc0e098e54d1b20e7f033356638c1cc on master.


Update XPS_GLYPH_MAPPING link.

The first mention of XPS_GLYPH_MAPPING was linked to the documentation for XPS_GLYPH_INDEX. Link to the documentation for XPS_GLYPH_MAPPING instead.

@bungeman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Dry run on non-main branches

Executes a dry run on branches other than main. All steps are run, but the final push doesn't transfer any data.
closes #31

What about repositories such as https://github.com/jhnc-oss/poky holding a plethora of non-main branches that still get updated a lot? Imvho, these would be very soon outdated with this change?

But instead of relaying on feature branches to refresh the destination repositories (with potential to break it), we could increase the syncs from once per day to something more frequent.

But instead of relaying on feature branches to refresh the destination repositories (with potential to break it), we could increase the syncs from once per day to something more frequent.

I refer to release branches such as https://github.com/jhnc-oss/poky/tree/dunfell

Ah, just to avoid some confusion here: The dry run relates to branches of this repository only, mirrored repos still update all of their branches.

Ah, just to avoid some confusion here: The dry run relates to branches of this repository only, mirrored repos still update all of their branches.

k. go ahead

Still, it might be a good idea to increase the frequency. PR on the way ...


Date picker i18n

Fixes #12185. This version pull in all localizations provided by package:intl. I'm working on a separate PR that reduces package:intl to only the list of locales supported by Flutter.

Can we fix https://github.com/flutter/flutter/issues/12050 first? Or at least next? I would rather we didn't keep adding dependencies on intl.

This had a bigger-than-usual negative impact on the aot_snapshot_size benchmarks.

This had a bigger-than-usual negative impact on the aot_snapshot_size benchmarks.

This is partly because we added localized dates, and partly because we added localized dates for all locales in the world. https://github.com/flutter/flutter/pull/12466 partially addresses the issue.
Here are the numbers for building complex_layout:
aot_snapshot_size_isolate:
Before this PR: 1832972 bytes
After this PR: 1941640 bytes
After #12466: 1867322 bytes
aot_snapshot_size_rodata:
Before this PR: 1758568 bytes
After this PR: 1892776 bytes
After #12466: 1794616


Fix error, notes, and warning based on BiocCheck()

fine i guess

Is this ready?

Is this ready?
@erawijantari - can you resolve the conflicts, and pull this through so that we can close the PR?

@erawijantari

@erawijantari - can you resolve the conflicts, and pull this through so that we can close the PR?

Sorry, seems like I accidentally closed the PR when I synced the forked directory and changed the master branch to devel as was suggested.
But I had checked all change and did not see any conflict.

Seems good, thanks!


Verify PX4IO debugging

Check if the command px4io debug <debuglevel> still  works.

Seems to work fine.


Is there any way to check if DataClass has foreign key relations?

I need that because in order to sync some DataClass objects from database with remote server, its dependencies must be synced first. Example: In order to sync Car DataClass which has foreign key relation with User DataClass (owner foreign key), I have to sync User object first because maybe User it does not exist on remote server yet and sending Car object to remote server without existance of User object would throw non-existent foreign key object exception.
I would need to check if DataClass has foreign key relations and to which DataClass it is related to. Is that possible?

This information isn't really exposed at the moment, and I don't think drift should be exporting that information. It would increase the API surface a lot (if we expose column constraints as structured classes), increases code size and isn't relevant for most users.
You can, at runtime, iterate over all columns from a given table and check if they have a foreign key constraint to order your synchronization logic. But I think it will be a lot easier if you just implement that manually. If you know car has a foreign key relation to user (you should because you're writing the tables :D), then you can also respect this ordering when it comes to synchronization.


Move date pickers to @mui/x-date-pickers

Fixes imports because date pickers would no more be available in @mua/lab
@Hyperkid123
@rvsia

@victoria-dos
Thank you for this PR! 🏆
We can remove these components from the editor component file - consumers should import them by themselves when using the editor and the MUI mapper.
After that, we have to add them to these files:
https://github.com/data-driven-forms/editor/blob/31407bcca010c24df0ee3c55fdf8b2c0b0f4b4da/packages/editor-pro/public/index.tsx#L78
https://github.com/data-driven-forms/editor/blob/31407bcca010c24df0ee3c55fdf8b2c0b0f4b4da/packages/editor-pro/cypress/component/ProEditor.cy.tsx#L33 (to all render functions - we should implement one wrapper component)
and then also update this file after the new release: https://github.com/data-driven-forms/react-forms/blob/master/packages/react-renderer-demo/src/pages/editor/live-editor.md
Can you please also add fix(pro): ... to trigger a new release?

:tada: This PR is included in version @data-driven-forms/editor-pro-v0.0.13 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix AutoNet regression

This was not caused by the websocketpp update; rather, it was caused when CoreContext made a change to the way context state is constructed when AutoFired was deprecated.  Fixing this just involves moving the enumeration responsibility to AutoNet, where it probably belonged all along.
Fixes #922

Thanks for jumping on this one


[Feature] Dropdown

Dropdown

[x] Dropdown example
[x] Dropdown divider
[x] Dropdown header
[x] Multi-level dropdown
[x] Dropdown with checkbox
[x] Background hover
[x] Helper text
[x] Dropdown with radio
[x] Background hover
[x] Helper text
[x] Dropdown with toggle switch
[x] Dropdown with scrolling
[x] Dropdown with search
[x] Menu icon
[x] Notification bell
[x] User avatar
[x] Avatar with name
[x] Dropdown navbar
[x] Sizes
[x] Placement
[x] Double placement

Thanks to @jjagielka .


7.5.0 login freeze and CPU go to 100%, but 6.9.0 works fine

Priority
(Medium) I'm annoyed but I'll live
Description
hi team, when I run m365 login --authType password --userName<EMAIL_ADDRESS>--password xxxxxx --debug
then it freeze,

but then I switch to m365 6.9.0, everything works.
Steps to reproduce

install m365 7.5.0
run m365 login --authType password -u xxxx -p xxxx --debug

Expected results
it should login success
Actual results
it freeze
Diagnostics
No response
CLI for Microsoft 365 version
3.7.0
nodejs version
18.19.0
Operating system (environment)
Linux
Shell
bash
cli doctor
can't provider doctor, because it need login first

Additional Info
No response

Thanks for reaching out @linchen2chris and sorry to hear CLI is not working as expected. Could you please try to run v7.0 and see if you still see the error? It would help us a lot to debug the issue, if we could find the first CLI version that shows this behavior. We appreciate your help

Just tried 7.5 on Ubuntu 20.04.

I can't reproduce the freezing.
The cli doctor command throws the same error on my end

I'm curious to hear your results when installing 7.0 or 7.4 @linchen2chris

The cli doctor command throws the same error on my end

cli doctor is not an anonymous command which is why you need to sign in first. That's by design.

I'm curious to hear your results when installing 7.0 or 7.4 @linchen2chris

hi @martinlingstuyl  7.0.0 works

but 7.4.0 freeze, same with 7.5.0

@martinlingstuyl I also tried 7.3.0 and 7.2.0:
7.2.0 works

but 7.3.0 freeze, maybe 7.3.0 is the first version breaks. hope that helps

So what kind of Linux environment are you running @linchen2chris ?

So what kind of Linux environment are you running @linchen2chris ?

centos in docker

Would you mind sharing your dockerfile. So we can try to reproduce it?

FROM centos
RUN cd /etc/yum.repos.d/
RUN sed -i 's/mirrorlist/#mirrorlist/g' /etc/yum.repos.d/CentOS-*
RUN sed -i 's|#baseurl=http://mirror.centos.org|baseurl=http://vault.centos.org|g' /etc/yum.repos.d/CentOS-*
RUN npm install -g<EMAIL_ADDRESS>

No unfortunately not, @waldekmastykarz...

...but is 7.3 the same version where we updated Ora? We may just have found an additional side effect of the spinner.

@Adam-it...

...but is 7.3 the same version where we updated Ora? We may just have found an additional side effect of the spinner.

Yes it is.
In 7.2 it was 5.4.1 and in 7.3 we updated it to higher version which introduce some problems.

@linchen2chris, we'll be removing a faulty package from the CLI in the next beta. We're expecting it may be the reason for your issues. We'll get you up to speed as soon as we've done that so you can try out the new beta. Sorry for the long hold up.

Any update on this @linchen2chris?

@linchen2chris, we'll be removing a faulty package from the CLI in the next beta. We're expecting it may be the reason for your issues. We'll get you up to speed as soon as we've done that so you can try out the new beta. Sorry for the long hold up.

this should be part of the v8 release which we target this month

@linchen2chris could you comment on this?

As we've probably fixed this, and @linchen2chris is not responding. Let's close this issue for now.


autocomplete attributes are off

https://v.usetapes.com/qkQisJcd2z
should be

with the number selected so its easy to change. Later on we can make it so one more tab adds another attribute

Closing, based on https://github.com/atom/autocomplete-plus/issues/661


[Feature] Release X units proportionally

Feature Description

Release a subset of a bond's collateral, with the remaining amount staying in the bond.
This is a partial release, difference with a partial slashing being the collateral staying in the bond rather than being moved to the Treasury.
Additional Information

WARNING Implementing this introduces a scalability risk.
To release proportions means introducing a mapping for allowed withdrawals (mapping(address=>uint256)), with the first proportional release having to populate (and pay for) the storage allocation of allowed withdrawals.
A linear growth cost i.e.
  cost = tx_base + number_of_guarantors * uint256_storage_cost

The tx_base being the base transaction cost and any non-repeating computation.
At some number of guarantors the cost would exceed the gas limit for a block. That's likely to be a very large number ...but it does exist, with this feature adding the scalability limitation.

proportional release could be a bit tricky as its not quite the mechanics of Uni v2 LP tokens (as release is only when you redeem the receipt LP token), or Sushi rewarder Onsen mechanics which is per block rewards assigned to your key. Maybe to utilize those battletested mechanics we do it like this:
Redemption of receipt token will take proportional share of what ever is in the pool (post slashing and rewards, similar to Uni LP mechanics)
Proportional release will be a special "slashing" of the pool, where assets go to a "claim bucket", snapshotted against eligible bonders during the time. similar to Onsen/sushi rewarder contracts.
To avoid too many micro deposits we probably need another field which is "minimum bond" parameter.

Question: is the following logic expected and correct?
In a simple scenario where this is only ONE user purchasing the bonds with BIT:

User sends 100 BIT, the contract takes it and issues him 100 BOND (the ERC20 token of this contract);
Users sends 100, 100, 100, 100 BIT in a row, now the collateral is 500 BIT, user get 500 BOND;
Now we slash 50 BIT;
We partial release 75 BIT back to the pool;
We slash 50 BIT again;
We release 75 BIT again;
Now we open the redemption;

Using the 500 BOND the user still holds, he can redeem 500 - (50+75+50+75) = 250 BIT
The user can withdraw 75 + 75 = 150 BIT due to the "partial release" back to the pool.
Is the above computation what we want?
If yes the a simple solution seems to be:

Create a total_deposit variable to track the total_deposit
Use a map to track each user's bond payment;
Keep a running _collateral that will be reduced each time a "slash" AND "partial release" happens;
Keep a running _partial_release_total that increment with each partial release;
Create a withdraw function for withdrawing partial release, withdrawal_amount = user_total_deposit * partial_release_total / total_deposit

The above solution depends on allowing withdrawal after allowing redemption, so all the slashes and partial releases are already done by the admin, which seems logical to me, so user can withdraw the partial release at one shot.
Please share more insights so we can implement it.

As Bond debt tokens are tradable, that also needs supporting & testing.

There are two main scenario that spring to mind:
Product creation/delivery; a boolean achievement, as you either have the fully featured product delivered to match the performance criterion, or you do not (As the saying goes, the first 80% of the project takes 80% of the time …the next 20% takes another 80% of the time :grinning:).
Achievement metric: a tangible measurable target is given and only partially achieved, here the expectation is the target would be fully realised given more time e.g. sign up ten users in two months, but only 7 are signed up by the evaluation date.
Really it comes down to the idea and chosen application of Performance Bonds, specifically on criterion evaluation.
If we have performance criterion evaluated on a specific date, they will either be fully met, partially met or not at all satisfied, which if enforced as is, equates to fFull redemption, partially slashed, or fully slashed.
The case for partial release could be point two, when the collateral is considerable, there could be milestone releases ….but that could also be implemented by using multiple bonds, a separate bond. for each milestone :thinking_face:

I feel that multiple successive bonds would be a better use of the platform than one large bond with long term performance criteria on a single evaluation date
For that reason I lean towards not having partial release. We should encourage any complex projects to begin with a roadmap of staggered bonds. From this we can see if new scenarios appear through use of the platform and we use that to evaluate adding a partial release feature

Yeh probably staggered bonds (and funding requests) would be the approach

Above comments are from thread discussing implementing a partial release using staggered bonds, instead of a single with proportional release.
Achieves the same outcome, but without bloating the gas cost of all other use cases.


[ctr] add --null-io and --detach

This adds the ability to start and detach from a task with ctr.  It also adds an option to redirect all task io to /dev/null.

Codecov Report

Merging #1654 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1654   +/-   ##
=======================================
  Coverage   48.87%   48.87%
=======================================
  Files          27       27
  Lines        4080     4080
=======================================
  Hits         1994     1994
  Misses       1667     1667
  Partials      419      419

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 ce57d6d...929ea33. Read the comment docs.

@dmcgowan rebased and updated


reformer on HPU

Eager Mode
    * 1 x RuntimeError: mat1 and mat2 must have the same dtype, but got Half and Float
        | Traceback (most recent call last):
        |   File "/homes/delaunap/hpu/results/venv/torch/bin/voir", line 8, in <module>
        |     sys.exit(main())
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/cli.py", line 128, in main
        |     ov(sys.argv[1:] if argv is None else argv)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/phase.py", line 331, in __call__
        |     self._run(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/overseer.py", line 242, in _run
        |     set_value(func())
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/scriptutils.py", line 37, in <lambda>
        |     return lambda: exec(mainsection, glb, glb)
        |   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 208, in <module>
        |     main()
        |   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 204, in main
        |     runner.train()
        |   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 120, in train
        |     loss = self.step(data)
        |   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 92, in step
        |     self.amp_scaler.scale(loss).backward()
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/_tensor.py", line 535, in backward
        |     torch.autograd.backward(
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/autograd/__init__.py", line 267, in backward
        |     _engine_run_backward(
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/autograd/graph.py", line 744, in _engine_run_backward
        |     return Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/autograd/function.py", line 301, in apply
        |     return user_fn(self, *args)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/transformers/models/reformer/modeling_reformer.py", line 1671, in backward
        |     output = layer.backward_pass(
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/transformers/models/reformer/modeling_reformer.py", line 1534, in backward_pass
        |     res_hidden_states = self.feed_forward(next_attn_output)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1535, in _wrapped_call_impl
        |     return self._call_impl(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1544, in _call_impl
        |     return forward_call(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/transformers/models/reformer/modeling_reformer.py", line 1396, in forward
        |     return apply_chunking_to_forward(
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/transformers/pytorch_utils.py", line 239, in apply_chunking_to_forward
        |     return forward_fn(*input_tensors)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/transformers/models/reformer/modeling_reformer.py", line 1405, in forward_chunk
        |     hidden_states = self.dense(hidden_states)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1535, in _wrapped_call_impl
        |     return self._call_impl(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1544, in _call_impl
        |     return forward_call(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/transformers/models/reformer/modeling_reformer.py", line 1366, in forward
        |     hidden_states = self.dense(hidden_states)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1535, in _wrapped_call_impl
        |     return self._call_impl(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1544, in _call_impl
        |     return forward_call(*args, **kwargs)
        |   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/torch/nn/modules/linear.py", line 116, in forward
        |     return F.linear(input, self.weight, self.bias)
        | RuntimeError: mat1 and mat2 must have the same dtype, but got Half and Float

Lazy Mode
 * no training rate retrieved
  * Error codes = 1, 1
  * 1 exceptions found
	* 1 x RuntimeError: [Rank:0] FATAL ERROR :: MODULE:PT_DEVMEM Allocation failed for size::268435456 (256)MB
    	| Traceback (most recent call last):
    	|   File "/homes/delaunap/hpu/results/venv/torch/bin/voir", line 8, in <module>
    	| 	sys.exit(main())
    	|   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/cli.py", line 128, in main
    	| 	ov(sys.argv[1:] if argv is None else argv)
    	|   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/phase.py", line 331, in __call__
    	| 	self._run(*args, **kwargs)
    	|   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/overseer.py", line 242, in _run
    	| 	set_value(func())
    	|   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/voir/scriptutils.py", line 37, in <lambda>
    	| 	return lambda: exec(mainsection, glb, glb)
    	|   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 208, in <module>
    	| 	main()
    	|   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 204, in main
    	| 	runner.train()
    	|   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 120, in train
    	| 	loss = self.step(data)
    	|   File "/homes/delaunap/milabench/benchmarks/huggingface/bench/__main__.py", line 96, in step
    	| 	accelerator.mark_step()
    	|   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/habana_frameworks/torch/utils/internal.py", line 27, in wrapper
    	| 	func(*args, **kwargs)
    	|   File "/homes/delaunap/hpu/results/venv/torch/lib/python3.10/site-packages/habana_frameworks/torch/core/step_closure.py", line 66, in mark_step
    	| 	htcore._mark_step(device_str, sync)
    	| RuntimeError: [Rank:0] FATAL ERROR :: MODULE:PT_DEVMEM Allocation failed for size::268435456 (256)MB


Made login page responsive for all devices

Related Issue
Fixes #380
Description
[Please include a brief description of the changes or features added]
Made few changes in login.css, login.js files and fixed issues and made login page responsive.
Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
https://github.com/SUGAM-ARORA/UniCollab/assets/134281589/4b297924-6440-40c3-9387-fb75be074966
https://github.com/SUGAM-ARORA/UniCollab/assets/134281589/e787d221-ea69-4ec4-b01a-9dc61a0d4613
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

@SUGAM-ARORA I had updated my branch. Please review my PR.


test: Don't delete and redeploy Cilium at end of test context

test-me-please

retest-net-next

Coverage increased (+0.001%) to 37.011% when pulling 9933a522c71fdef555b8c45ae5214bb81ec8010e on pr/tgraf/ci-dont-uninstall-cilium-at-end-of-test into 859e1789fd1b5ee7ba2c613f360a86d40e0b7603 on master.


Cli output log file volum 6g

I run a dependency check cli 9.8.0,  with an nvd api, and add the parameter --log=log.txt, then depedency check try to download the nvd, when it shows 50%, I found my log file has already 6g byte,  it seems huge and it's normal ?

Likely it was due to massive retries of the download as IIRC NVD had issues in January 2024


MIC-400 rename affected components, decision actions and review actions enumerations

Codecov Report
Attention: Patch coverage is 95.34884% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.29%. Comparing base (b25dc32) to head (8221642).

Files
Patch %
Lines

...lengesupportinterventionplanapi/entity/Referral.kt
50.00%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #107      +/-   ##
============================================
- Coverage     87.34%   87.29%   -0.06%
  Complexity      582      582
============================================
  Files           117      117
  Lines          1762     1755       -7
  Branches        100      100
============================================
- Hits           1539     1532       -7
  Misses          178      178
  Partials         45       45

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


Use IndexMap::swap_remove instead of remove

The remove function is deprecated. Swap_remove should be used instead.
Cargo-make currently fails to compile without this change.

As a side-note, I think cargo-make should commit its Cargo.lock in the repo. The lack of a Cargo.lock means that cloning the repo and building might lead to compilation errors due to newer versions of the dependencies breaking the build (as is the case with index-map here).

Thanks a lot!
as for the lock, keep in mind that it fails due to harsh linting rules i enabled.
those rules only apply when you indeed clone and compile but not when you install it.
so users are not impacted, only cargo-make developers (like you).
i actually want it to break to make sure i'm using the latest stuff.
in this specific case, index map did a backward comparability break without increasing the versions correctly but it happens :)

Codecov Report

Merging #292 into 0.22.2 will decrease coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           0.22.2     #292      +/-   ##
==========================================
- Coverage   93.42%   93.32%   -0.11%
==========================================
  Files          86       86
  Lines       14227    14259      +32
==========================================
+ Hits        13292    13307      +15
- Misses        935      952      +17

Impacted Files
Coverage Δ

src/lib/descriptor.rs
94.01% <100%> (-0.67%)
:arrow_down:

src/lib/installer/rustup_component_installer.rs
73.23% <0%> (-6.77%)
:arrow_down:

src/lib/legacy.rs
75% <0%> (-4.6%)
:arrow_down:

src/lib/cache.rs
68.83% <0%> (-3.78%)
:arrow_down:

src/lib/installer/crate_version_check.rs
82.35% <0%> (-2.5%)
:arrow_down:

src/lib/config.rs
78.78% <0%> (-2.47%)
:arrow_down:

src/lib/io.rs
78.26% <0%> (-1.74%)
:arrow_down:

src/lib/environment/gitinfo.rs
94.73% <0%> (-1.7%)
:arrow_down:

src/lib/functions/mod.rs
93.15% <0%> (-1.3%)
:arrow_down:

src/lib/condition.rs
97.77% <0%> (-1.1%)
:arrow_down:

... and 10 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 8bf2c46...8f08954. Read the comment docs.


can't receive the notification when the app is killed

[REQUIRED] Please fill in the following fields:

Unity editor version: 2018.4.13
Firebase Unity SDK version: 6.15.2
Source you installed the SDK: FirebaseMessaging (.unitypackage or Unity Package Manager)
Problematic Firebase Component: FirebaseMessaging (Auth, Database, etc.)
Other Firebase Components in use: no (Auth, Database, etc.)
Additional SDKs you are using: Facebook, AdMob (Facebook, AdMob, etc.)
Platform you are using the Unity editor on: Mac (Mac, Windows, or Linux)
Platform you are targeting: Android (iOS, Android, and/or desktop)
Scripting Runtime: Mono (Mono, and/or IL2CPP)

[REQUIRED] Please describe the issue here:
I use link send message like this : https://console.firebase.google.com/project/xxxx-xxxx/notification/compose
When the app in the foreground and background, my android  device can be receive messages and notifications.
But, when the app is killed(just do Closed game (swipe away)), I'm waiting with my device.  waiting, waiting, waiting...
.After a long time, my device is not very happy and can't display any notifications.
What can I do here, and then my device can be informed？

@Leo-lwj
Sorry for the late reply.
I actually tried both Firebase 6.15.2 and the latest 6.16.0 with FCM quickstart and I actually can still receive push notification after the app is in foreground, background or after killed, if I subscribed to the topic.
Could you try and see if you can reproduce the issue with quickstart?  Also, could you give us an example about how you compose the notification, ex what you filled in each field?
Shawn

Have had the same issue. In my case it happened when I launch the application via Android Studio run button. What I had to do was to run the freshly builded app on the device as usual. Then Kill the app, open again by tapping on the app icon on the device, then kill or close the app again and then voila. Now I can receive push notifications even when the app is closed.

@matmlyreal
Did you try to build Android apk from Unity directly?
Also, could you detail about your process to create Android project and some more information such as Unity version, Firebase SDK version, like those from the template.
It would be great if you can provide a minimum viable produce, like modified quickstart.
Shawn


User Authentication

Issue Report
Title
User Authentication
Description
After registration , tough user putting valid credentials ,it shows that credentials are incorrect.
Steps to Reproduce the Issue
To assist us in understanding and resolving the issue, please outline the steps you took that led to the problem:

Register user and store info in DB
While sign in match input credentials with DB
Action 3: Describe any additional actions that may be relevant.

Feel free to be as detailed as necessary so we can replicate the issue accurately.
Expected Behavior
After sign up store details in DB . While sign in check is user exists and let him log in if exists
Actual Behavior
Tough putting correct credential it says invalid credentials
Screenshots (if applicable)

Steps to Reproduce:

Navigate to the product search bar on the homepage.
Type "Wireless Mouse" into the search field.
Press the Enter key.

Expected Behavior: The system should display a list of products matching "Wireless Mouse."

Actual Behavior: The message "No results found" appears, regardless of valid input.

Example 2
Title: Login Issue with Valid Credentials
Description: Users are experiencing issues logging in, despite entering the correct username and password.

Steps to Reproduce:

Access the login page.
Input a valid username and password.
Click the "Login" button.

Expected Behavior: The user should successfully log in and be redirected to their dashboard.

Actual Behavior: An error message states, "Invalid credentials," preventing access.

Thank you for taking the time to report this issue! We appreciate your help in improving our project.

Assign me this issue with tags I will start working on it


ci: 🐝 update dprint config and move to GitHub action from pre-commit hook

Description
Update dprint conifg, removing the dprint pre-commit hook, which is no longer supported, and replacing it with a GitHub action.  Also update dpring plugins.
Checklist:

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

Current dependencies on/for this PR:

main

PR #73   👈

This comment was auto-generated by Graphite.


string is vulnerable (<=99.999.99999). Patched versions: <0.0.0

Prerequisites

[x] Plugin is in the latest version
[x] Issue was not reported yet

Description
In the package.json from the swagger codegen tool for a nodejs-server i get a Error with the newest version of node security (0.4.0):
string is vulnerable (<=99.999.99999). Patched versions: <0.0.0

I get the error only for the "swagger-tools: "0.10.3" entry, here the full package.json:
{
  "name": "api-storage",
  "version": "0.1.0",
  "description": "Store all your api in one place and let other services consume them",
  "main": "index.js",
  "keywords": [
    "swagger"
  ],
  "license": "Unlicense",
  "private": true,
  "dependencies": {
    "js-yaml": "^3.3.0",
    "swagger-tools": "0.10.3"
  }
}

Versions
Plugin:
0.4.0

IDE:
IntelliJ IDEA 2017.3.4 (Ultimate Edition)
Build #IU-173.4548.28, built on January 29, 2018
JRE: 1.8.0_152-release-1024-b11 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Mac OS X 10.13.3

So it looks like this package has not been fixed yet and it's vulnerable.
In the next releases I'll implement some mechanism that will allow to accept the risk and turn off this information - as suggested in #3

I don't think so, except you use another source than nodesecurity.io. There is no open advisory for "swagger-tools" right now: https://nodesecurity.io/advisories?search=swagger-tools

Yeah, but swagger-tools has a dependency to string package.
https://nodesecurity.io/advisories?search=String

I read "string", i thought "string" as in text and didn't think twice about that there could be a node package called "string".
Thank you for your time, issue is done 👍


Config Client Retry Not Working

Hi,
Description
I have been trying to dockerize my Spring boot Application that contains a Config Server and a Config Client.  I've been following the steps mentioned in the Document  but the Retry behavior is not working as expected.
Property Configuration
spring:
  cloud:
     config:
       fail-fast: true
       retry:
         max-attempts: 10000
         max-interval: 1000

Details
This is the content that is present in both bootstrap.yml and application.yml under the config client service that uses the config-server running on port 8888.
I have a set of Bean that is creating a TransportClient bean for Elasticsearch that causes the error `Caused by: java.lang.IllegalArgumentException: Could not resolve placeholder 'elasticsearch.host' in value "${elasticsearch.host}"``
What I noticed is, that the spring.cloud.config.retry.* properties have no effect and I don't see any retry attempts being done from the Config Client.  I also tried the spring.config.retry.* properties with same results.
Gradle Configuration
compile("org.springframework.boot:spring-boot-starter-web:1.5.7.RELEASE")
compile("org.springframework.cloud:spring-cloud-starter-config:1.3.3.RELEASE")
// Retry Methods
compile("org.springframework.retry:spring-retry:1.2.1.RELEASE")
compile("org.springframework.boot:spring-boot-starter-aop:1.3.3.RELEASE")

Can you please help me understand what I am missing in this case and why the retry is not working ?

First thing I noticed is that you are not using the BOM.
dependencyManagement {
  imports {
    mavenBom 'org.springframework.cloud:spring-cloud-dependencies:Finchley.M2'
  }
}

dependencies {
    compile 'org.springframework.cloud:spring-cloud-starter-config'
}

It is unclear to me what the elastic search property has to do with the retry not working.  Could you please provide the complete logs?

@ryanjbaxter
Elasticsearch property has nothing to do with the Retry not working. The error I am observing with respect to elasticsearch is because of Retry not working.
The Scenario is as follows.
I have a set of Properties in application.yml and application-local.yml in the git repo that is configured under Config Server.
I have a client Service that relies on the Config Server to create @Bean for Elasticsearch client under the @EnableAutoconfiguration class. For some reason, i've been noticing failure on the availability of the Config Server endpoints (Root cause still unknown).
Due to this unavailability of Config server, when I am trying to deploy the Client application, it fails to initialize the @Bean and fails.
The Retry method I added based on the document is not working.
Will give a try to the mavenBom and see if that fixes my problem.

@ryanjbaxter
I tried including the mavenBom details in my build.gradle file. But the issue still exists. Haven't been able to put some time and run the Spring Boot app in debug mode to figure out exactly what's going wrong while Autowiring and Bean Processing. Any suggestion is appreciated.

I am experiencing this problem with my spring-boot app.
Spring boot 1.5.7
Spring cloud config client 1.3.3
Spring core 4.3.11
Intentionally shut down a localhost spring config server such that I get a Connection refused https://localhost:8888/a/b/c
Bootstrap.yml
Tried with and without failFast true/false, and this setting takes effect.
Clearly only 1 attempt is made, with and without other settings like maxAttempts
spring:
  cloud:
    config:
      retry:
        initial-interval: 1000
        maxInterval: 3000
        maxAttempts: 30
        multiplier: 1.1
      uri: https://localhost:8888

Could you provide a sample app?

might a little bit late. but also this might help someone looking around like me.
you need to get the following understandings first

failFast the SpringBoot application, so that the default config won't work in any case
initial-interval & maxAttempts ... etc won't work unless you fix some dependencies in your maven/gradle file
in my case, gradle alongside with yml  ... the following were a must:

compile('org.springframework.cloud:spring-cloud-starter-config')
compile("org.springframework.boot:spring-boot-starter-actuator")
compile("org.springframework.retry:spring-retry")
compile("org.springframework.boot:spring-boot-starter-aop")

so eventually you will have a bootstrap.yml file that looks like this
server:
port: 9000
spring:
application:
name: command-api
cloud:
config:
uri: http://config:8888
failFast: true
retry:
initialInterval: 2000
maxAttempts: 30
I hope this help out somebody

Thanks, was just about having all that dependencies.


sample data for local development testing

Hi, this is a great project. Can you provide some sample data for local development testing? I want to test it out. thank you very much!

Hi @htthYjh ,
The repository initially consisted of just the pre-training architecture but I am actively updating it on a daily basis. The full repository when completed will allow for complete, scalable, and distributed pre-training.
I am working on reproducing the OPT and GODEL pre-training corpora. I will be uploading both of them to Huggingface datasets. Currently, there is a Huggingface streaming data loader implemented which allows you to use the Pile dataset by EleutherAI for pre-training. I will be updating the repository to include a local environment data loader to go along with the streaming one.
Best,
Enrico

Thank you so much, great work, looking forward to your progress.

Hi @htthYjh ,
I rebuilt the data loader to work locally: https://github.com/conceptofmind/LaMDA-pytorch/blob/main/lamda_pytorch/build_dataloader.py
A few things you are going to have to take into consideration if you are going to use the provided Pile dataset:

The Pile dataset is over 1TB of data. You will likely need a storage device with up to 2TB of space for everything including the tokenizer and saved models.
If you want to use a different dataset, I provided a configuration file with different fields that can be adjusted. You can find a bunch of different text generation datasets on the Huggingface Datasets hub. I will still be uploading the GODEL conversational dataset as well.

The configuration for the data loader looks like this:
"""
Configuration for data loader.
"""

use_huggingface: bool = field(
    default = True,
    metadata = {'help': 'Whether to use huggingface datasets'}
)

train_dataset_name: Optional[str] = field(
    default="the_pile",
    metadata={"help": "Path to Hugging Face training dataset."}
)

eval_dataset_name: Optional[str] = field(
    default="the_pile",
    metadata={"help": "Path to Hugging Face validation dataset."}
)

choose_train_split: Optional[str] = field(
    default="train",
    metadata={"help": "Choose Hugging Face training dataset split."}
)

choose_eval_split: Optional[str] = field(
    default="train",
    metadata={"help": "Choose Hugging Face validation dataset split."}
)

remove_train_columns: ClassVar[list[str]] = field(
    default = ['meta'],
    metadata={"help": "Train dataset columns to remove."}
)

remove_eval_columns: ClassVar[list[str]] = field(
    default = ['meta'],
    metadata={"help": "Validation dataset columns to remove."}
)

seed: Optional[int] = field(
    default=42,
    metadata={"help": "Random seed used for reproducibility."}
)

tokenizer_name: Optional[str] = field(
    default="gpt2",
    metadata={"help": "Tokenizer name."}
)

tokenizer_seq_length: Optional[int] = field(
    default=512,
    metadata={"help": "Sequence lengths used for tokenizing examples."}
)

select_input_string: Optional[str] = field(
    default="text",
    metadata={"help": "Select the key to used as the input string column."}
)

batch_size: Optional[int] = field(
    default=16,
    metadata={"help": "Batch size for training and validation."}
)

save_to_path: Optional[str] = field(
    default="''",
    metadata={"help": "Save the dataset to local disk."}
)

Let me know if this solves your issue.
Best,
Enrico

Ok,let me check,thank you very much


mac_addr should have its own type, instead of String

Hi,
Currently, NetworkInterface's mac_addr member is a Option<String>.  It would be nice to have a simple MacAddr type, like:
pub struct MacAddr([u8; 6]);

with a From<[u8; 6]> impl and an octets method that returns [u8; 6].
Since mac_addr currently uses a String, it's unclear if it's uppercase, lowercase, in xx:xx:xx:xx:xx:xx format, or in xx-xx-xx-xx-xx-xx format.  This will make comparisons easier.  I can attempt a pull request if you agree with the idea.  Thanks

Hi @mllken! Thanks for opening this issue!
Yeah I agree, I think we  could have a dedicated type which also allow us to work with MAC addresses with ease.
Would you like to work on this? As a first iteration I think we could:

Introduce a type MacAddr just as the one you shared in your comment: MacAddr([u8; 6]).
Replace instances of Option<String> with Option<MacAddr>
Make sure we can serialize it into its string representation

I think now its a good time to introduce such a change due to the improvement suggested on #37, a single major release could introduce both changes!

Looking into this.  It seems winapi defines PhysicalAddress as [u8; 8] (8, instead of 6).  The current linux code in your crate can also handle mac addresses of any size.  The proposed type might be too restrictive to support all your currently supported platforms.

Looking into this.  It seems winapi defines PhysicalAddress as [u8; 8] (8, instead of 6).  The current linux code in your crate can also handle mac addresses of any size.  The proposed type might be too restrictive to support all your currently supported platforms.

Yeah theres two main schemes for Mac Addresses, one being the EUI-48 and the second is the EUI-64 scheme.
Theres more details in this RFC document: https://www.rfc-editor.org/rfc/rfc7042#page-6
I think we could have an approach to support both scenarios using an approach similar to std's for IpAddr and have an enum like:
enum MacAddress {
    Eui48(Eui48),
    Eui64(Eui64),
}

I think we could construct at the moment of parsing the Mac Address bytes in question.
WDYT?


[flutter_tools]customize systemTempDirectory

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
The 'systemTempDirectory' can be customized through environment variables
There are too many directories in the '% tmp%/flutter_tools. xxxx' directory, and dozens of GB have not been automatically cleared.
It's too easy to fill up my Windows system disk.
flutter_tools can customize a temporary directory for easy cleaning.
My envionment win10 + Flutter 3.13.3 + IntelliJ IDEA 2023.2
Proposal
FLUTTER_SYSTEM_TEMP_DIRECTORY=e:\dev\flutter_tmp

@RelicOfTesla
Can you take a look at this issue and check if it helps to answer your case or not ?

Not help..IntelliJ IDEA 2023.2 plugin - Flutter 75.1.4,  Debug,Stop,Debug,Stop,Debug,Stop..........Debug,Stop.. Now, You can see many fluters_tools.xxxx  directory appears and has not been deleted(directory count>10), and executing "flutter doctor - v" will not automatically clear it.

Manually execute the cmd.exe command "flutter run -d chrome" and after opening it, press "ctrl+c" or close cmd.exe window to still not delete the directory.

The flutter command has a thin .bat wrapper on Windows to make it a runnable command; Ctrl-C terminates the Flutter tool's Dart script, as it is supposed to. Then, because it's how the console on Windows works, you get a prompt asking if you want to terminate the enclosing .bat script.

Now, You can see many fluters_tools.xxxx directory appears and has not been deleted

Can you provide screenshot of these directories generated ?

F:\tmp>flutter create test1
Flutter assets will be downloaded from https://storage.flutter-io.cn. Make sure you trust this source!
Creating project test1...
Resolving dependencies in test1... (25.9s)
Got dependencies in test1.
Wrote 129 files.

All done!
You can find general documentation for Flutter at: https://docs.flutter.dev/
Detailed API documentation is available at: https://api.flutter.dev/
If you prefer video documentation, consider: https://www.youtube.com/c/flutterdev

In order to run your application, type:

  $ cd test1
  $ flutter run

Your application code is in test1\lib\main.dart.

F:\tmp>cd test1

F:\tmp\test1>flutter run -d chrome
Flutter assets will be downloaded from https://storage.flutter-io.cn. Make sure you trust this source!
Launching lib\main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             18.2s
This app is linked to the debug service: ws://<IP_ADDRESS>:56416/qM0nngy8xgE=/ws
Debug service listening on ws://<IP_ADDRESS>:56416/qM0nngy8xgE=/ws

  To hot restart changes while running, press "r" or "R".
For a more detailed help message, press "h". To quit, press "q".

A Dart VM Service on Chrome is available at: http://<IP_ADDRESS>:56416/qM0nngy8xgE=
The Flutter DevTools debugger and profiler on Chrome is available at:
http://<IP_ADDRESS>:9103?uri=http://<IP_ADDRESS>:56416/qM0nngy8xgE=
终止批处理操作吗(Y/N)? y

F:\tmp\test1>

translated:
终止批处理操作吗(Y/N)? y      =>    Terminate batch job (Y/N)?

Thanks for the update. Keeping this issue open for team's attention / tracking.

I agree that it is a problem that these temp directories grow over time. How would customizing the path we use help, though?

I'm actually going to close this in favor of re-opening https://github.com/flutter/flutter/issues/84094. Thanks for the research @RelicOfTesla


Add documentation for building/packaging with native modules

Add documentation to the readme for how to build with native packages.

@aj-ptw do you have any info on how to build with native packages?
I'm struggling a little at the moment and its hard to find any solid info.

I think this should be closed because of the #440


[Bug]: Turn on strict mode

What did you expect to happen?
I expect this to be on
What happened?
It is not on
Version
Beta 0.1.0
What browsers are you seeing the problem on?
No response
Relevant log output
No response
Code of Conduct

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

Resolved in #328


Add "Modding APIs" section

The wiki needs a definitive source of user-created APIs for common usecases, as discussed in #28
This PR adds a modding API section with:

A section for non-code APIs so non-programmers can have a section to check for simple swaps (like audio replacement, custom suits, etc)
A section for code APIs so modders can have an easy overview

I think going forward, at least for now, the sections should be:

Initial setup (Max's guide on how to get everything initialized)
Modding APIs (for user-created APIs, such as customsounds)
Advanced Modding (for specific topics like Networking and creating Configs. the section may need a rename)

Here's an example of what I think the Modding APIs section could look like once we start getting some guides (like #28)

would love any comments on this @MaxWasUnavailable @Xilophor since I'm not 100% sure the best way to go about this
I want to keep the playing field as fair as possible, but I do think we should incentivize modders to write guides on their APIs - I've done this in the PR by adding a gold star to any on-wiki modding APIs

there might also be some APIs missing right now, feel free to suggest/add some. I just did a quick general overview of APIs I've seen mentioned a lot

I feel like advanced topics should come before APIs. Since we then have a fairly clear logical ordering going from non-API to API stuff, rather than no/yes/no.

I was thinging about this yesterday. With all of this being added, the main non-developer stuff is being "squished" in the sidebar; should we look into separating it into a Beginner's Guide and a Developer's Guide - may be appropriate for another PR/issue

I was thinging about this yesterday. With all of this being added, the main non-developer stuff is being "squished" in the sidebar; should we look into separating it into a Beginner's Guide and a Developer's Guide - may be appropriate for another PR/issue

I think we can have two separate sidebars like (bsmg.wiki)[https://github.com/bsmg/wiki] does, I can look into it in a later PR

I was thinging about this yesterday. With all of this being added, the main non-developer stuff is being "squished" in the sidebar; should we look into separating it into a Beginner's Guide and a Developer's Guide - may be appropriate for another PR/issue

I think we can also have two separate sidebars like bsmg.wiki does, I can look into it in a later PR

This would be good imo. Separate mod creation and mod usage.

I'm also honestly not 100% sure what the line between an API and advanced modding topic is, that might have to be figured out as we go
Like, the page on custom networking uses UnityNetcodeWeaver, which I would technically class as an API, but the concept is so fundamental to modding

We can work this out as we go. We may want to eventually boil it down to the fundamentals, and link to APIs that make it easier.

This would be good imo. Separate mod creation and mod usage.

Yeah, I think the sidebar is probably fine right this moment, but once #44 #38 and #28 are merged it's going to start getting overwhelming if we keep adding more pages
for now I've made an issue - #47


build: restructure release output

Work in Progress PR (for review)

Based on top of #3608

Note: AOT Travis check will likely fail at the moment. Not updated yet.
cc. @jelbourn

Closing as we are merging this together with @jelbourn's changes: #3751


mpu9250 in px4fmu-v2

Do we support any px4fmu-v2 derivatives that have an mpu9250 and can't be "upgraded" to px4fmu-v3 (bootloader update)?
Unless there's a good example I propose we disable the mpu9250 in px4fmu-v2 to save about 11 kB of flash and find a way to painlessly get any affected users through the bootloader update (QGC prompt to set SYS_BL_UPDATE).
https://github.com/PX4/Firmware/pull/10355#issuecomment-422597424
https://github.com/PX4/Firmware/pull/10519

Do we support any px4fmu-v2 derivatives that have an mpu9250 and can't be "upgraded" to px4fmu-v3 (bootloader update)?

I don't know of any boards.
Now that we have the BL update we can also start thinking about removing the VTOL modules for example.

Do we support any px4fmu-v2 derivatives that have an mpu9250 and can't be "upgraded" to px4fmu- > (bootloader update)?

RE: bootloader update:
The bootloader for fmuv3 and fmuv2 are now EXACTLY the same. @LorenzMeier created the V3 bootloader build to avoid the "Where the V3 bootloader question" Originally it did not do the silicon check, but it does now.

We have such a board! awav/mro X2.1 arhitechture is v2+ mpu9250. But I think there could be some optimization. Can we put the v3 bootloader to our board?
We still have this pending https://github.com/PX4/Firmware/pull/8138, with regard to the boot scripts.

@tubeme you can update the BL, instructions are here: https://docs.px4.io/en/advanced_config/bootloader_update.html, and then check if you can flash the v3 target.

Thanks @bkueng I have a problem following this manual. My board is AUAV X2.1 and in the master firmware there is no SYS_BL_UPDATE. Is there a BIN version somewhere so I can flash with JTAG? The chip is  STM32F427 Cortex M4 with FPU
168 MHz/256 KB RAM/2 MB Flash so for sure it has 2MB flash.

Ah I see. The AUAV X2.1 has a different build target and thus is not affected by this.

Yes and ur chip is rev.3 so the new bootloader should work.

AUAV X2.1 has its own bootloader. https://github.com/PX4/Bootloader - do not use the V2 or V3 bootloader on the AUAV X2.1. The AUAV X2.1 is not compatible with FMUv2 it has hardware differences that will cause the board not to function (inputs on V2 are used as outputs on the X2.1) So you must uses the correct bootloader so it loads the correct FW.
See https://docs.px4.io/en/flight_controller/mro_x2.1.html and follow the instructions there.


verifyClient function or alternative

Hello, thanks for your library
I followed an earlier version which had the option to check the client ... if the verification failed, called the abortConnection(socket, code, name) function
const http = require('http')
const WebSocketServer = require('uws').Server
const PORT = proccess.env.PORT || 3000

const server = http.createServer((req, res) => {
    // do stuff...
})

const WS = new WebSocketServer({
    server: server,
    verifyClient (info, cb) {
        // for example ever fails
        cb(false, 401, 'Unauthorized')
    }
})

WS.on('connection', function(ws) {
    // never show
    console.log(ws)

    ws.on('message', data => {})
    ws.on('error', CloseEventCode => {})
})

server.listen(PORT)

The connection was aborted before arriving at the event onconnection and in front-end show this in console:

My question: is it possible to reproduce the same behavior in the current library?

No, but you can close the websocket in the open handler if you don't find the http request suitable.


#3445 Add codestyle to git

Description (required)
Fixes #3445 Add codestyle to git
What changes did you make and why?
import google java code style, use predefined styles for kotlin/xml

Codecov Report

Merging #3486 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #3486   +/-   ##
========================================
  Coverage      7.15%   7.15%
  Complexity      298     298
========================================
  Files           266     266
  Lines         11935   11935
  Branches        974     974
========================================
  Hits            854     854
  Misses        11011   11011
  Partials         70      70

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 61a3f45...fedce32. Read the comment docs.


Remove black markerline from trackerdash

Turns this

into this

Closing in favor of #472.
@mgrauer will delete the branch once his external use case is properly using #472's resulting semantic release version.

I'm deleting this branch now, we have pointed to v0.9.1 in our downstream project.

Awesome, thanks @mgrauer!


Data viewer should clearly indicate when data is filtered

Originally from https://github.com/microsoft/vscode-jupyter/issues/1286#issuecomment-765306346:

One thing on filtering pandas dataframes:
I can click on Filter Rows button and enter a condition in any of the white boxes that appear below the column names. The dataframe gets filtered in real time. But when I click again on teh Filter Rows button, the dataframe stays filtered. I'd expect that once the filters are masked, they are not applied. One could get confused unless he shows the filters.
My suggestion is that the filters gets applied only when the conditions are visible or, alternatively, add some indication that the dataframe is filtered even when the conditions are hidden.

We added a clear filter button in https://github.com/microsoft/vscode-jupyter/pull/4805. Not sure if that addresses the ask here.

With the "clear filters" button and having to explicitly pick "hide filters" this feels resolved to me unless customers complain.