Update docfx.json

:warning: Validation status: warnings

File
Status
Preview URL
Details

server-2013/lync-server-2013-planning-for-security.md
:warning:Warning

Details

server-2013/remove-csnetworkregionlink.md
:warning:Warning

Details

server-2013/lync-server-2013-dns-summary-extensible-messaging-and-presence-protocol-xmpp-federation.md
:warning:Warning

Details

server-2013/remove-csannouncement.md
:warning:Warning

Details

server-2013/migration-from-lync-server-2010-group-chat-or-office-communications-server-2007-r2-group-chat-to-lync-server-2013-persistent-chat-server.md
:warning:Warning

Details

server-2013/lync-server-2013-deployment-process-for-integrating-on-premises-unified-messaging.md
:warning:Warning

Details

server-2013/lync-server-2013-re-activate-server-after-security-configuration-wizard-closes-ports-in-iis.md
:warning:Warning

Details

server-2013/remove-legacy-archiving-and-monitoring-servers.md
:warning:Warning

Details

server-2013/migrating-xmpp-federation_1.md
:warning:Warning

Details

server-2013/lync-server-2013-delete-an-existing-conferencing-policy.md
:warning:Warning

Details

server-2013/lync-server-2013-new-response-group-application-features.md
:warning:Warning

Details

server-2013/remove-front-end-pool-or-standard-edition-server.md
:warning:Warning

Details

server-2013/lync-server-2013-quality-of-experience-qoe.md
:warning:Warning

Details

server-2013/lync-server-2013-viewing-information-about-user-accounts-enabled-for-lync-server.md
:warning:Warning

Details

server-2013/lync-server-2013-dns-summary-dns-and-hlb-load-balanced.md
:warning:Warning

Details

server-2013/remove-csnetworkregion.md
:warning:Warning

Details

server-2013/migrating-xmpp-federation.md
:warning:Warning

Details

server-2013/lync-server-2013-quality-of-experience-qoe-database-schema.md
:warning:Warning

Details

server-2013/lync-server-2013-delete-an-existing-collection-of-sip-trunk-configuration-settings.md
:warning:Warning

Details

server-2013/remove-csxmppallowedpartner.md
:warning:Warning

Details

server-2013/lync-server-2013-viewing-information-about-push-notification-settings.md
:warning:Warning

Details

server-2013/remove-csnetworkintersitepolicy.md
:warning:Warning

Details

server-2013/lync-server-2013-dns-summary-autodiscover.md
:warning:Warning

Details

server-2013/lync-server-2013-new-persistent-chat-server-features.md
:warning:Warning

Details

server-2013/migrating-multiple-sites-and-pools_1.md
:warning:Warning

Details

server-2013/lync-server-2013-qoe-view-details.md
:warning:Warning

Details

server-2013/remove-cswebserviceconfiguration.md
:warning:Warning

Details

server-2013/lync-server-2013-delete-an-existing-collection-of-meeting-configuration-settings.md
:warning:Warning

Details

server-2013/lync-server-2013-viewing-and-working-with-reports-created-by-best-practices-analyzer.md
:warning:Warning

Details

server-2013/remove-csnetworkinterregionroute.md
:warning:Warning

Details

server-2013/lync-server-2013-new-monitoring-features.md
:warning:Warning

Details

server-2013/lync-server-2013-dns-requirements.md
:warning:Warning

Details

server-2013/lync-server-2013-qoe-table-details.md
:warning:Warning

Details

server-2013/migrating-multiple-sites-and-pools.md
:warning:Warning

Details

server-2013/lync-server-2013-delete-an-existing-collection-of-lync-phone-edition-configuration-settings.md
:warning:Warning

Details

server-2013/remove-cswatchernodeconfiguration.md
:warning:Warning

Details

server-2013/lync-server-2013-viewing-and-analyzing-monitoring-server-reports.md
:warning:Warning

Details

server-2013/remove-csnetworkconfiguration.md
:warning:Warning

Details

server-2013/lync-server-2013-new-management-and-administration-features.md
:warning:Warning

Details

server-2013/lync-server-2013-dns-requirements-for-standard-edition-servers.md
:warning:Warning

Details

server-2013/migrating-group-chat-servers.md
:warning:Warning

Details

server-2013/lync-server-2013-delete-an-existing-collection-of-client-version-configuration-settings.md
:warning:Warning

Details

server-2013/lync-server-2013-qoe-reports.md
:warning:Warning

Details

server-2013/remove-csvoicetestconfiguration.md
:warning:Warning

Details

server-2013/remove-csnetworkbandwidthpolicyprofile.md
:warning:Warning

Details

server-2013/lync-server-2013-new-ipv6-features.md
:warning:Warning

Details

server-2013/lync-server-2013-view-user-pin-information.md
:warning:Warning

Details

server-2013/lync-server-2013-dns-requirements-for-simple-urls.md
:warning:Warning

Details

server-2013/migrating-archiving-and-monitoring-servers_1.md
:warning:Warning

Details

server-2013/lync-server-2013-delete-an-existing-collection-of-cdr-configuration-settings.md
:warning:Warning

Details

This comment lists only the first 50 files in the pull request.

server-2013/lync-server-2013-planning-for-security.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csnetworkregionlink.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-dns-summary-extensible-messaging-and-presence-protocol-xmpp-federation.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csannouncement.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migration-from-lync-server-2010-group-chat-or-office-communications-server-2007-r2-group-chat-to-lync-server-2013-persistent-chat-server.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-deployment-process-for-integrating-on-premises-unified-messaging.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-re-activate-server-after-security-configuration-wizard-closes-ports-in-iis.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-legacy-archiving-and-monitoring-servers.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migrating-xmpp-federation_1.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-delete-an-existing-conferencing-policy.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-new-response-group-application-features.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-front-end-pool-or-standard-edition-server.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-quality-of-experience-qoe.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-viewing-information-about-user-accounts-enabled-for-lync-server.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-dns-summary-dns-and-hlb-load-balanced.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csnetworkregion.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migrating-xmpp-federation.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-quality-of-experience-qoe-database-schema.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-delete-an-existing-collection-of-sip-trunk-configuration-settings.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csxmppallowedpartner.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-viewing-information-about-push-notification-settings.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csnetworkintersitepolicy.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-dns-summary-autodiscover.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-new-persistent-chat-server-features.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migrating-multiple-sites-and-pools_1.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-qoe-view-details.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-cswebserviceconfiguration.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-delete-an-existing-collection-of-meeting-configuration-settings.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-viewing-and-working-with-reports-created-by-best-practices-analyzer.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csnetworkinterregionroute.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-new-monitoring-features.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-dns-requirements.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-qoe-table-details.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migrating-multiple-sites-and-pools.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-delete-an-existing-collection-of-lync-phone-edition-configuration-settings.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-cswatchernodeconfiguration.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-viewing-and-analyzing-monitoring-server-reports.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csnetworkconfiguration.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-new-management-and-administration-features.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-dns-requirements-for-standard-edition-servers.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migrating-group-chat-servers.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-delete-an-existing-collection-of-client-version-configuration-settings.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-qoe-reports.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csvoicetestconfiguration.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/remove-csnetworkbandwidthpolicyprofile.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-new-ipv6-features.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-view-user-pin-information.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-dns-requirements-for-simple-urls.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/migrating-archiving-and-monitoring-servers_1.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

server-2013/lync-server-2013-delete-an-existing-collection-of-cdr-configuration-settings.md

[Warning] Author Serdar Soysal cannot be recognized in GitHub.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/OfficeDocs-SkypeforBusiness-Test-pr.ru-ru, working directory: W:\tn2o-s, target branch: master, target branch commit: bad6dfa9825553f19fe864f326e4a45992d72cd1, source branch: 905c2537966ae8d3c662fdd71c1d0f5a/master, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/OfficeDocs-SkypeforBusiness-Test-pr.ru-ru, working directory: W:\31in-s, target branch: master, target branch commit: bad6dfa9825553f19fe864f326e4a45992d72cd1, source branch: 905c2537966ae8d3c662fdd71c1d0f5a/master, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Metadata changes are obsolete and hence closing PR without merging. Please open new PR for other new fixes. Thanks.


Move VRGDA logic into a lib

PR for https://github.com/transmissions11/VRGDAs/issues/12
Followed the design of https://github.com/transmissions11/libcompound - split Linear and Logistic into 2 libs.

wait why close?

sorry my bad for not merging earlier

@transmissions11 was just cleaning up some loose ends, thought maybe this wasn't what you wanted haha Should I reopen?

yes def, sorry i will clean this up asap and get it merged!

yes def, sorry i will clean this up asap and get it merged!

Hi @transmissions11 @0xBeans, I'm working on a VRGDA project that needs this library in order to make our contract upgradeable. To that end, I rebased this branch so it is up-to-date with current master branch, except for the LogisticToLinear implementation since it was only added recently.  I pushed that branch here https://github.com/transmissions11/VRGDAs/compare/master...nvonpentz:VRGDAs:library-rebase?expand=1.
When making the remaining changes to the LogisticToLinear implementation, I noticed the abstractions didn't fit as well, so I created a fresh branch with a different approach. You can see that here https://github.com/transmissions11/VRGDAs/compare/master...nvonpentz:VRGDAs:library?expand=1. The main differences are that it

Creates one library instead of two, with only one getVRGDAPrice function (similar to master), and separate getTargetSaleTimeLinear and getTargetSaleTimeLogistic functions.
Does not remove all the abstract classes. Instead, logic common between the different VRGDA types (i.e. the getVRGDAPrice function) is defined in an abstract VRDGA class that Linear, Logitistic, and LogisticToLinear inherit from. I implemented things such that the interfaces defined by the current abstract contracts on master are preserved.

If it's useful, I'm happy to open a PR for either, but I also want to be respectful of the work @0xBeans has done already. In this case I went ahead and implemented in order to unblock myself. Cheers!


[DOCFIX] en/Running HBase on Alluxio.md

some bugs in the en/Running HBase on Alluxio.md

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11475/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11501/
Test PASSed.


⚠ gorzdrav.org has degraded performance

In 57db00a, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 5646 ms

Resolved: gorzdrav.org performance has improved in 69cf4f8.


single track view is not working

The user @Nautic reported following error:
Each data donor has a list of their uploaded files ("Tracks" menu).
If he clicks on an ID there, this track is displayed on the map.
In my case, only a pink area with a coordinate system appears.
Maybe something is wrong with the sea mark layer?
Or with the water depth layer?

sample url: https://depth.openseamap.org/singletrack.html?track_id=127587&zoom=12&lat=52.37155&lon=12.9525185
Analysis shows that call of mapserv.fcgi fail.
https://depth.openseamap.org/cgi-bin/mapserv.fcgi?LAYERS=trackpoints_single_track_10&NUMZOOMLEVELS=22&PROJECTION=EPSG%3A900913&TYPE=png&TRANSPARENT=TRUE&TRACK_ID=127587&SERVICE=WMS&VERSION=1.1.1&REQUEST=GetMap&STYLES=&FORMAT=image%2Fpng&SRS=EPSG%3A900913&BBOX=1408887.3051562,6859727.0816042,1448023.0636328,6898862.8400808&WIDTH=1024&HEIGHT=1024
proposal: replace the call of mapserver with a call of geoserver

The sample URL shows the map and track points now. Is that what is expected, i.e. is the bug fixed?

#240 fixed the pink area -- the correct server is now called up. It does appear that track filtering is not yet correct though. According to @martinover , https://depth.openseamap.org/singletrack.html?CQL_FILTER=TRACK_ID=127587&zoom=12&lat=52.37155&lon=12.9525185 should show a single filtered track.

The sample URL shows the map and track points now. Is that what is expected, i.e. is the bug fixed?

@wschildbach: I checked the feature and it seems to be fixed. Thanks for support.


Protect instance data on model change

This is being scrapped as in some scenarios you want to overwrite the instance data with model data.


Measure gas

Fixes #201.

This a good start. How do the gas costs display? What does it look like?

During testing, it logs to console something like this:
...
      ✓ pays each recipient their corresponding amount (427ms)
Setup costs: 164716 wei.
BulkPayOut costs: 202482 wei.
Complete costs: 29608 wei.
...

Would it be reasonable to set some max cumulative gas as a part of testing?

You mean the tests to fail if max total gas reached? That would mean to define a max total gas for approx. 30 diferent tests.

You mean the tests to fail if max total gas reached? That would mean to define a max total gas for approx. 30 diferent tests.

Or maybe just one or two of the tests that we wanna assure. Just something that's less than manual

I don't even know what standard to use when setting those limits... Do you have some limits in mind?

I don't even know what standard to use when setting those limits... Do you have some limits in mind?

Basic sanity checks. Pick a couple of things that would be bad to say increase by 5x.

Sanity checks, checked.


how to parser a file with macro？

imput file like this：
module A ( ifdef DEBUG
input debug_tag,
endif input xxx, ); endmodule
but verible-verilog-syntax cant parse debug_tag, unless add
define DEBUG.
how to add "define DEBUG" in cmd line?

I think my issue like this #1898

This is really too bad. Verilator was doing exactly what I needed, but the preprocessor is sort of half-way implemented. Example:
ifdef TEST1
foo test1
  (.clk (port.clk));
endif

ifndef TEST2
foo test2
  (.clk (port.clk));
endif

But unfortunately, since there is no way to control in the "verible-verilog-syntax" command what preprocessor defines are present, "test2" will be present in the tree and "test1" will not. I was prepared to use the "rawtokens" output to essentially do my own preprocessing. But I can't because not enough information is put into the tree.
Any suggestions?


Added TimeTask method override for Task

The current extension method only accepts Task, this adds an override that also accepts a Task.

Hi @Vake93 ,
Thank you.
I'm off this week, I'll take a look at this around 23rd-24th of August.


Move bounding box out of the node data

DO NOT MERGE

Codecov Report

Merging #498 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #498      +/-   ##
=========================================
+ Coverage    96.2%   96.2%   +<.01%
=========================================
  Files         128     128
  Lines        9239    9241       +2
=========================================
+ Hits         8888    8890       +2
  Misses        351     351

Impacted Files
Coverage Δ

...h/src/details/DTK_DetailsTreeConstruction_decl.hpp
100% <ø> (ø)
:arrow_up:

packages/Search/src/details/DTK_DetailsNode.hpp
100% <ø> (ø)
:arrow_up:

packages/Search/src/DTK_LinearBVH_decl.hpp
100% <100%> (ø)
:arrow_up:

...ch/src/details/DTK_DetailsTreeConstruction_def.hpp
100% <100%> (ø)
:arrow_up:

...es/Search/src/details/DTK_DetailsTreeTraversal.hpp
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

In principle we could template TreeConstruction on the tree type.

I guess. However tree construction is too tightly coupled to a specific tree.

In principle we could template TreeConstruction on the tree type.

I guess. However tree construction is too tightly coupled to a specific tree.

The intent is not to use with another type, it is to be able to deduce the device type and the bounding volume type.

@aprokop I addressed your comments.  Should we close that PR and put it in the backlog?
For reference, these changes slightly improve performance on the GPU (< 2-3% for BVH::query()) but degrade it on the CPU (few percents)

Should we close that PR and put it in the backlog?

Yes. Create an issue (if we don't have it) to keep track of the performance PRs.


Can you add options to artisan commands?

So i user :Laravel artisan command and i want to do for example make:test can i pass -p option  for pest test somehow? I see in code that it should ask for Options in some cases, it does not for make:test?

I see that there is a <C-y> mapping that makes the options available, did not notice the argument difference. Maybe makes sense to add this to documentation, but my bad.


Anyone interested in making a PyTorch version of this on Python 3?

Between research and another tutorial I'm working on (NLP), I have little time left. If someone would like to build a PyTorch version of this, we can add it to this repo.
Would be very helpful. If you're interested, mail me at<EMAIL_ADDRESS>Best,
Spandan

Thanks to @AnshulBasia  for completing this! New PyTorch version is now available! Navigate to the folder docs/ for that!


🐛 Bug:  Failed to run tests with a large tests count

Bug Report Checklist

[X] I have read and agree to Mocha's Code of Conduct and Contributing Guidelines
[X] I have searched for related issues and issues with the faq label, but none matched my issue.
[X] I have 'smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, my usage of Mocha, or Mocha itself.
[ ] I want to provide a PR to resolve this

Expected
I am trying to run tests in the amount of 74,800. but I get an error
#
# Fatal error in , line 0
# Fatal JavaScript invalid size error 174895934 (see crbug.com/1201626)
#
#
#
#FailureMessage Object: 0x7ffdea17c440
 1: 0xd17f41  [/home/atutik/.nvm/versions/node/v20.10.0/bin/node]
 2: 0x20fe211 V8_Fatal(char const*, ...) [/home/atutik/.nvm/versions/node/v20.10.0/bin/node]
 3: 0x1090db8  [/home/atutik/.nvm/versions/node/v20.10.0/bin/node]
 4: 0x127023f  [/home/atutik/.nvm/versions/node/v20.10.0/bin/node]
 5: 0x12703da  [/home/atutik/.nvm/versions/node/v20.10.0/bin/node]
 6: 0x14e9876 v8::internal::Runtime_GrowArrayElements(int, unsigned long*, v8::internal::Isolate*) [/home/atutik/.nvm/versions/node/v20.10.0/bin/node]
 7: 0x790753ed9ef6

If run 73k tests, then everything works
Minimal, Reproducible Example
you can reproduce the problem like this
git clone git@github.com:sanchezzzhak/node-device-detector.git
npm i
node tests/tools/update-fixtures.js
npx mocha tests/*.spec.js

Versions
mocha 10.3.0 or 9.1.3
node 20.10.0
Additional Info
No response

I found the reason by downgrading the node js version, which showed that there is an infinite loop for test functions.
There is no problem.


Metadata injection breaks SvelteKit error pages

Environment
"@sentry/vite-plugin": "^2.22.2",
"@sveltejs/kit": "^2.5.17",
"@sentry/sveltekit": "^8.26.0",

Steps to Reproduce
Added the following to our vite config:
sentryVitePlugin({
  applicationKey: 'key',
}),

Expected Result
Opening a page that would cause an +error.svelte page to be rendered works without problems in preview mode (vite preview).
Actual Result
The page doesn't render and the console shows the following error:
ReferenceError: can't access lexical declaration 'me' before initialization

This is part of the generated and minified code:
// [...]
e._sentryModuleMetadata[(new me).stack] = Object.assign({
}, e._sentryModuleMetadata[(new me).stack], {
  '_sentryBundlerPluginAppKey:key': !0
});
// [...]
class me extends j {
  constructor(e) {
    super (),
    N(this, e, ue, de, r, {
    })
  }
}
export {
  me as component
};

So at first glance it appears that Sentry is using the wrong class to get the stack trace from. Looking at the unminified module code we can see why:
// [...]
  _sentryModuleMetadataGlobal._sentryModuleMetadata[new Error().stack] = Object.assign(
    {},
    _sentryModuleMetadataGlobal._sentryModuleMetadata[new Error().stack],
    { "_sentryBundlerPluginAppKey:key": true }
  );
// [...]
class Error extends SvelteComponent {
  constructor(options) {
    super();
    init(this, options, instance, create_fragment, safe_not_equal, {});
  }
}
export {
  Error as component
};

SvelteKit generates components named Error for +error.svelte pages. Sentry is injected after SvelteKit's code generation and expects Error to refer to the built-in value.
Interestingly it works when referencing Error directly in the component somehow because that forces SvelteKit to use a different name for the component:
// [...]
function instance($$self, $$props, $$invalidate) {
  let $page;
  component_subscribe($$self, page, ($$value) => $$invalidate(0, $page = $$value));
  console.log(">>>>>", Error);
  return [$page];
}
class Error_1 extends SvelteComponent {
  constructor(options) {
    super();
    init(this, options, instance, create_fragment, safe_not_equal, {});
  }
}
export {
  Error_1 as component
};

I filed this as a bug here because even though I think SvelteKit should not shadow built-in classes, Sentry should be aware of the conventions that SvelteKit uses or at the very least account for the fact that Error might be overwritten.

On another note: Is there a reason you're using the vite plugin directly instead of sentrySvelteKit()?

It should work but you'll likely need to pass the options via unstable_sentryVitePluginOptions to the plugin because we didn't add this feature yet to the API exposed from @sentry/sveltekit. Probably a todo for the future.
I opened #594 to fix the bug. Expect it to be released soon. Note, if you switch to @sentry/sveltekit you'll either need to manually update to the latest @sentry/vite-plugin version or wait until we bump it from within the SDK. Could take a couple of days.


Library problem of abs calculation

I get the following error when I compile my Arduino project. I included both the Arduino and the TensorFlow library:
Arduino\libraries\Arduino_TensorFlowLite\src/tensorflow/lite/kernels/internal/quantization_util.h: In function 'tflite::QuantizationParams tflite::ChooseQuantizationParams(double, double, bool)':
AppData\Local\Arduino15\packages\arduino\hardware\mbed\3.3.0\cores\arduino/Arduino.h:63:16: error: expected unqualified-id before '(' token
 #define abs(x) ((x)>0?(x):-(x))
                ^
Arduino\libraries\Arduino_TensorFlowLite\src/tensorflow/lite/kernels/internal/quantization_util.h:67:12: note: in expansion of macro 'abs'
       std::abs(qmin_double) + std::abs(rmin / scale);

It looks like there is a compilation error and a problem with the definition of the "abs" function in the "Arduino.h" file, which is conflicting with the "abs" function used in the "quantization_util.h" file.
Did someone also experience such error?
Can anyone help me with that error?
Thanks

@konegen
Your feedback regarding the TensorFlow Lite Micro (TFLM) codebase is much appreciated.
Please make sure you have the latest update to the tflite-micro-arduino-examples repository.  This issue was resolved through PR #170 approximately two weeks ago.  All example applications in the repository compile correctly.
If this issue has occurred while creating your own application, then you will continue to encounter the compilation error. Mixing TFLM header files together in the same application with the Arduino.h header file, will require the following code:
#include <Arduino.h>
#undef abs

Alternatively you may separate out all TFLM specific code in your application to a new file with the .cpp suffix.  This new file should not include any Arduino header files.
Please feel free to reopen this issue should you wish to supply any additional information.


readonly change

readonly change for entry.html

Hello @gmichel! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file Project/pipeline_models.py:

Line 64:14: E222 multiple spaces after operator
Line 67:1: E265 block comment should start with '# '

In the file Project/pipeline_views.py:

Line 338:80: E501 line too long (86 > 79 characters)
Line 485:80: E501 line too long (82 > 79 characters)
Line 505:80: E501 line too long (85 > 79 characters)

In the file Project/settings.py:

Line 153:1: E265 block comment should start with '# '

In the file Project/wsgi.py:

Line 12:1: E265 block comment should start with '# '
Line 13:21: E225 missing whitespace around operator
Line 18:1: E265 block comment should start with '# '
Line 19:1: E265 block comment should start with '# '
Line 20:80: E501 line too long (85 > 79 characters)
Line 21:1: E265 block comment should start with '# '

Closed in favor of #38 which has just the entry.html change


PR: Error when setting a path with a single quote

Fxies #6451.

@csabella, please add a test for this.

@ccordoba12   I wasn't sure how to add a test.  I put some code on the issue showing that it was working with a manual test.  Is there an existing test that I can look at for creating a new directory or for working with these ipython command calls?

@ccordoba12   Thanks for the suggestion!  In that test file, test_get_env called get_env which had the same type of goal as get_cwd and set_cwd.  Very helpful.
I'm getting PEP8 warnings, but I copied that code from the other tests.  Is the longer line allowed here?

@csabella, this looks pretty good. I just left a couple of comments in your tests to improve them a bit.
And please don't worry about pep8 issues in test files. Unfortunately, we can't make pep8speaks to ignore them.

It looks like windows isn't applying the os.chdir before the assert.  :-(   Trying to figure it out.

I've added a qtbot.wait().  However, all the existing tests near this one just skip windows (@pytest.mark.skipif(os.name == 'nt' or PYQT4, reason="It doesn't work on Windows and segfaults in PyQt4")).  I can enter the commands run by those other tests manually into the IPython console on a windows machine (such as "import os; os.environ['FOO'] = 'bar'"), but yet the automated testing is skipped.  I believe the chdir in this new test is running into the same issues as those other tests.

@csabella, I don't want to put an additional burden on you, so if you want to skip this test on Windows, please do it.

However, all the existing tests near this one just skip windows

Well, I just removed a lot of those skipif's because I noticed the corresponding tests were hanging on Windows due to us opening a QDialog and trying to close it with a QTimer. I removed that and used a lower level API instead.

@ccordoba12   It's not a burden.  I just don't know what to change to get it to work.  My dev box is Linux and it works there.  I can run the Spyder installation on Windows, but I can't download the git repo and test it from there.  So, I'm kind of in the dark trying to fix this in the tests.  :-(

@csabella, no worries then. If you tested your PR on a Windows box and it works, then please skip the test. You can come back later to remove the skip if you find how to do it (I've done that a lot of times :-).

@ccordoba12   No, I haven't tested it on Windows.  I can't install the git repo on the Windows box that I can access, so I can't test my changes there.

@dalthviz, please test @csabella work on Windows and let us know if it works for you or not.

The patch is working, but the test_get_cwd is failing locally on my setup (Windows 7 64 bits). I'll take a look a it now in case I can find out why.

The test are failing because of the \ in the path when running shell.execute("import os; os.chdir('''{}''')".format(tempdir))
Running this instead with a raw string works:
shell.execute("import os; os.chdir(r'''{}''')".format(tempdir))

Thank you, @jnsebgosselin!   I really appreciate you looking into this.  Without being able to test on Windows, I would have never figured that out.

I'm glad I could help! Is it me or the tests are failing for a reason that is unrelated to this PR? The test test_get_help is failing everywhere.

Cheryl, the failures are unrelated to your tests. I'm fixing them in #6626.

@ccordoba12   Thank you for letting me know.  I was just going to start looking into them.

@csabella, please rebase to get my fix. Then this should be ready.

Thanks again @ccordoba12  and @jnsebgosselin  for helping me get the test to pass.  :-)


Add chennel data to order webhook

I want to merge this change because crucial data was missing
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3198 Add channels data to order webhook


Flutter Commands failing w/Flutter 3.0.2

We have a pipeline that has been running fine for a long time.  Today they broke, with the following error:

Starting: FlutterCommand
==============================================================================
Task         : Flutter Command Task
Description  : Launch a custom Flutter command with custom arguments.
Version      : 0.3.0
Author       : Hey24sheep
Help         : Don't forget to insert a 'Flutter Install' task before. [More Information](https://github.com/hey24sheep/azure-flutter-tasks)
==============================================================================
/Users/runner/hostedtoolcache/Flutter/3.0.2-stable/macos/flutter/bin/flutter pub run intl_utils:generate
/Users/runner/hostedtoolcache/Flutter/3.0.2-stable/macos/flutter/bin/internal/shared.sh: line 225: /Users/runner/hostedtoolcache/Flutter/3.0.2-stable/macos/flutter/bin/cache/dart-sdk/bin/dart: Bad CPU type in executable
##[error]Error: The process '/Users/runner/hostedtoolcache/Flutter/3.0.2-stable/macos/flutter/bin/flutter' failed with exit code 1
Finishing: FlutterCommand

Looks like it is related to Flutter 3.0.2.  Manually setting the Flutter version back to 3.0.1 fixes the issue.

I ran into the same thing just a couple of hours ago.  A rollback to 3.0.1 seems to address the issue for me as well.

Looking at the log it is because of "Bad CPU Type", which is an Apple / Flutter issue.
Here is the solution to this. Also, check these links Reddit1 and Reddit2.

There are now 2 latest stable versions. That's why we should be able to choose one. Could you add "architecture" input parameter to FlutterInstall task?.
https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_arm64_3.0.2-stable.zip
https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_3.0.2-stable.zip
- task: FlutterInstall@0
  inputs:
    mode: 'auto'
    channel: 'stable'
    version: 'latest'
    architecture: 'arm64'

{
  "base_url": "https://storage.googleapis.com/flutter_infra_release/releases",
  "current_release": {
    "beta": "bcea432bce54a83306b3c00a7ad0ed98f777348d",
    "dev": "13a2fb10b838971ce211230f8ffdd094c14af02c",
    "stable": "cd41fdd495f6944ecd3506c21e94c6567b073278"
  },
  "releases": [
    {
      "hash": "cd41fdd495f6944ecd3506c21e94c6567b073278",
      "channel": "stable",
      "version": "3.0.2",
      "dart_sdk_version": "2.17.3",
      "dart_sdk_arch": "arm64",
      "release_date": "2022-06-10T17:11:47.366348Z",
      "archive": "stable/macos/flutter_macos_arm64_3.0.2-stable.zip",
      "sha256": "734874caac4928d86a7bba8aa843e5c0577ccb1051df551db41dd1e4d64b6d9e"
    },
    {
      "hash": "cd41fdd495f6944ecd3506c21e94c6567b073278",
      "channel": "stable",
      "version": "3.0.2",
      "dart_sdk_version": "2.17.3",
      "dart_sdk_arch": "x64",
      "release_date": "2022-06-10T17:02:07.215846Z",
      "archive": "stable/macos/flutter_macos_3.0.2-stable.zip",
      "sha256": "7fc88b2944495c20e013c83c4e0bfcd40d258d9fb52ddbde5c0771e207c9ff29"
    },
    {
      "hash": "bcea432bce54a83306b3c00a7ad0ed98f777348d",
      "channel": "beta",
      "version": "3.1.0",
      "dart_sdk_version": "2.18.0 (build 2.18.0-44.1.beta)",
      "dart_sdk_arch": "x64",
      "release_date": "2022-05-26T17:58:53.645129Z",
      "archive": "beta/macos/flutter_macos_3.1.0-beta.zip",
      "sha256": "8feffb0ef36df30c30aab3ff876578dd2762213e90deee5f3b5847353133fe21"
    },
    {
      "hash": "bcea432bce54a83306b3c00a7ad0ed98f777348d",
      "channel": "beta",
      "version": "3.1.0",
      "dart_sdk_version": "2.18.0 (build 2.18.0-44.1.beta)",
      "dart_sdk_arch": "arm64",
      "release_date": "2022-05-26T17:56:13.705273Z",
      "archive": "beta/macos/flutter_macos_arm64_3.1.0-beta.zip",
      "sha256": "4f4d368dfbc7c6697f2840c434163a9b35dcf5dbe399671513d7ae0e24a0f351"
    },
    {
      "hash": "fb57da5f945d02ef4f98dfd9409a72b7cce74268",
      "channel": "stable",
      "version": "3.0.1",
      "dart_sdk_version": "2.17.1",
      "dart_sdk_arch": "x64",
      "release_date": "2022-05-20T03:38:50.201052Z",
      "archive": "stable/macos/flutter_macos_3.0.1-stable.zip",
      "sha256": "20d96acdc49f877e533697300228b58108b18a970e29184d8477c01889218ce9"
    },
   ...
}

@hey24sheep this is the error I get from devops when I try to install rosetta:
"Installing Rosetta 2 on this system is not supported."
Any thoughts about how to (or if you can) add this step to a pipeline?

@ctnataman could you add that to a new issue. It's a feature request different to the current issue.

@bobwiller Are you running the command on the latest MacOs VM Agent?
I can't seem to find anything else related to this particular error log. Could you run your Flutter command with verbose and paste the full log here. If verbose log doesn't give much more info, you could also open this same issue on Flutter's Github, maybe someone from Flutter teams knows about the solution.

Yes, vmImage: macOS-latest
I guess maybe I am confused - installing rosetta isn't a Flutter command.  In my pipeline I added this after the FlutterInstall task:

task: CmdLine@2
inputs:
script: 'softwareupdate --install-rosetta'

Here is the entire output:
Starting: CmdLine
Task         : Command line
Description  : Run a command line script using Bash on Linux and macOS and cmd.exe on Windows
Version      : 2.201.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/command-line
Generating script.
Script contents:
softwareupdate --install-rosetta
========================== Starting Command Output ===========================
/bin/bash --noprofile --norc /Users/runner/work/_temp/28d2bbb4-b0f3-4a1d-9029-e3e56ce0cf9a.sh
Installing Rosetta 2 on this system is not supported.
##[error]Bash exited with code '1'.
Finishing: CmdLine
If you want me to try something different let me know.

@bobwiller I don't think this will work. Problem is from Microsoft's or Flutter's side.
First, I would recommend you to create this issue on Flutter's Github. This will be more helpful. Someone could point you in the right direction.
Second, check this link. You can currently use a self hosted agent to bypass the builds so this way you won't be stuck.

@hey24sheep I think the problem is with the Install task downloading arm64 binaries on x86-64 agents.
We can modify it to check the platform before downloading flutter.
You can exec uname -m to check this. It returns arm64 on M1 macs and x86_64 on intels (according to this SO post). Then you will know which one to download from @ctnataman json example above.

I have updated the extension, it should be available within 15 minutes or so. Let me know if it is working as intended by closing this issue.

@bobwiller I have updated the extension twice today, it should be available within 15 minutes or so. Let me know if it is working as intended by closing this issue. Else, this issue will be closed within few weeks as obsolete.

I still got the "Bad CPU type in executable" error on MS-hosted MacOS agents and Flutter version 3.0.2.
Flutter Install Task version is 0.3.2 and it downloaded flutter_macos_arm64_3.0.2-stable.zip.

@kostadin-damyanov-prime It auto detects the arch type and then downloads the sdk. If you try to give it a customArchType it can download that. It should work without any issues as it has been tested live and otherwise.
You can check if you are on the latest extension or not. Also could you post verbose logs here of the same along with your config.

@hey24sheep I ran the pipeline with system diagnostics enabled and this is the output:
##[debug]Finding version 'custom' from 'https://storage.googleapis.com/flutter_infra_release/releases/releases_macos.json'
##[debug]customArch=undefined
##[debug]Using Arch type 'x64'
##[debug]customVersion=3.0.2
##[debug]Found version hash 'cd41fdd495f6944ecd3506c21e94c6567b073278'
##[debug]Trying to get (Flutter,3.0.2-stable, macos) tool from local cache
##[debug]isExplicit: 3.0.2-stable
##[debug]explicit? true
##[debug]Agent.Version=2.204.0
##[debug]Agent.ToolsDirectory=/Users/runner/hostedtoolcache
##[debug]checking cache: /Users/runner/hostedtoolcache/Flutter/3.0.2-stable/macos
##[debug]not found
##[debug]Starting download archive from 'https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_arm64_3.0.2-stable.zip'
##[debug]
##[debug]Agent.Version=2.204.0
##[debug]Agent.TempDirectory=/Users/runner/work/_temp
##[debug]testing directory '/Users/runner/work/_temp'
Downloading: https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_arm64_3.0.2-stable.zip
##[debug]destination /Users/runner/work/_temp/c5c91659-76bb-4d4c-9033-0883716ba772
##[debug]downloading
##[debug]creating stream
##[debug]download complete
##[debug]Succeeded to download '/Users/runner/work/_temp/c5c91659-76bb-4d4c-9033-0883716ba772' archive from 'https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_arm64_3.0.2-stable.zip'
##[debug]Extracting 'https://storage.googleapis.com/flutter_infra_release/releases/stable/macos/flutter_macos_arm64_3.0.2-stable.zip' archive
Extracting archive

It resolves the arch to x64 and somehow still downloads arm64. How this could happen?

@kostadin-damyanov-prime it is indeed weird. I ran some tests and I cannot reproduce the shown result :/
Here are my tests, with x64

with arm64

Could you try giving it a customArch value as 'x64' and check if it builds successfully.

I am not using the customArch parameter. We are using MS-hosted agents and I do not know if all of them are x64.
However, I think I found what the issue is. It happens when I provide a specific version but not a customArch.
// if user selected custom
if (version.toLowerCase() !== 'latest') {
	// fetch custom version
	version = task.getInput('customVersion', false);

	if (customArch && customArch.trim() !== '') {
		current = json.releases.find((item) => item.channel == channel && uniformizeVersion(item.version) == uniformizeVersion(version) && item.dart_sdk_arch == customArch);
	} else {
		current = json.releases.find((item) => item.channel == channel && uniformizeVersion(item.version) == uniformizeVersion(version));
	}
}

I think in the else block we should add && item.dart_sdk_arch == arch so that it will use the detected arch value.

@kostadin-damyanov-prime I see, I wanted to keep all builds intact.
So think of this, there are 3 kind of releases now,
1 - old releases which are missing this new arch type key in json
2 - new releases which has this key in json
3 - which has 2 different values for the same key (only on macos (bane of my existence :D)
So, I assumed that people will bypass this error automatically by providing it a customArch if they read logs. But, you are not wrong here as well. It is hard to debug and know what is wrong.
I have just pushed a new fix which will auto check for all these three scenarios I have mentioed above. It should be available in 15 minutes or so. I have tested locally it works as intented but please check on your agents too and let me know.

The latest update fixed the issue for me, thanks @hey24sheep!

I am closing this issue as considered fix by multiple latest updates. If anyone faces any issues, feel free to open up a new one.


Add token support to TransferTransaction

Description:
This PR adds support to transfer tokens with TransferTransaction, as well as needed accompanying classes.
Related issue(s):
Fixes #199
Notes for reviewer:
Functionality from the Java SDK will be used to create a token and associate the token with certain accounts to test this (since these capabilities are required and haven't been developed in the C++ SDK yet).
Checklist

[X] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

This has been tested by creating and associating tokens and accounts using the Java SDK, then taking those account and token IDs and inputting them into the TransferTokensExample to test functionality.


Fix for points straddling the international date line

This fixes the distance calculation for such points by reducing the
longitude difference to the interval [-180, 180].  It also uses the
library functions sinh and hypot where appropriate.

sinh will also be more accurate for small args (not that this will matter here).  If you do revert the hypot change, I recommend you retain it as a comment (noting that it is slower).

I think I have the longitude fix applied everywhere it's needed.  I'm not familiar with this testing framework, so I recommend that someone else add more tests.
Currently the longitude values can be anything -- and that's OK since the angle reduction is done on the fly.  On the other hand, latitude values outside the range [-90,90] can be returned (e.g., by offset); arguably these should be clipped to the allowed range.

@cffk btw, is there a simple alternative to std::remainder that we could use in the reference repo (in JS)? We can use its definition (a - Math.round(a / b) * b) but I wonder if there was a simpler formula for the purpose.

I recommend (a - Math.round(a / b) * b) .  (Alternatively use a % b and then check whether the result isn't in [-b/2, b/2].)  Perhaps it's worth defining a longitude_difference function to localize the actual implementation?

Commentary:
An alternative fix to the longitude wrapping issue would be to enforce
the cylindrical topology in the point class (e.g., normalizing the
longitude to [-180, 180] in the constructor).  Then long_diff could be
defined as
double d = a - b;
return d < -180 ? d + 360 : (d > 180 ? d - 180: d);

It's not clear whether this is an improvement and once you go down this
path there's the temptation to enforce latitude in [-90, 90] and to
over-think the whole problem.

This is ready to merge, I believe.  Probably more could be done on the tests, but these are going to be revamped as part of #13.  By the way, what's the provenance of the datasets used for testing?  How were the "correct" results obtained?


nuke: improving readability

Brief description
Code was too complex to read
Testing notes:

configure project_settings/nuke/publish/ExtractReviewDataMov/outputs to have multiple presets with use of filters
open your testing nuke script
publish render write node

I agree with @iLLiCiTiT this comment is likely the best format. Which makes it something like this:
# please consider example as pseudocode - untested
...

                # process only allowed families
                if f_families:
                    if set(families).intersection(f_families):
                        continue

                # process only allowed task types
                if f_task_types:
                    if not task_type in f_task_types:
                        continue

                # process only subsets matching patterns
                if f_subsets:
                    if any(re.search(pattern, subset)
                           for pattern in f_subsets)):
                       continue

This also means the lines below it where you check all can be removed:
                # we need all filters to be positive for this
                # preset to be activated
                test_all = all([
                    test_families,
                    test_task_types,
                    test_subsets
                ])

                # if it is not positive then skip this preset
                if not test_all:
                    continue

I agree with @iLLiCiTiT this comment is likely the best format. Which makes it something like this:
# please consider example as pseudocode - untested
...

                # process only allowed families
                if f_families:
                    if set(families).intersection(f_families):
                        continue

                # process only allowed task types
                if f_task_types:
                    if task_type in f_task_types:
                        continue

                # process only subsets matching patterns
                if f_subsets:
                    if any(re.search(pattern, subset)
                           for pattern in f_subsets)):
                       continue

I believe above logic says that if it matches any of the filters then they are continued and excluded which might be the opposite of what you'd need? :D
This also means the lines below it where you check all can be removed:
                # we need all filters to be positive for this
                # preset to be activated
                test_all = all([
                    test_families,
                    test_task_types,
                    test_subsets
                ])

                # if it is not positive then skip this preset
                if not test_all:
                    continue

Brilliant! Thank you @BigRoy . Code looks and reads much better this way ;)


Workaround SR-9930 to prevent CI failures in test_passthrough_environment

This test fails frequently in CI and has been preventing us from being
able to release packages. Disable the part affected by SR-9930 for now.
rdar://48045782

@swift-ci please smoke test

@swift-ci please test and merge

@swift-ci please test and merge

Previous failure was unrelated; our old friend missing submodule 'SwiftOverlayShims'

Merged to un-block CI


D5 Class Diagram Documentation Added

Simple design-level UML class diagram of the project with addition of not-yet-implemented user role and comment features.

looks good, lets review all of them together and make sure everything is consistent


the R package 'littler' inherited from rocker/hadleyverse repo had a bug that won't run installGithub.r correctly

the bug is discused and fixed here:
https://github.com/eddelbuettel/littler/pull/49/commits/4a5a4014714d8727b2699db96f6e42ebeb34bd3f
the thing is rocker/hadlyeverse is phasing out and mostly unmantained (latest build 9 months ago), also  it got built against an old R, namely R 3.4.1 if I recall correctly.
so even though littler bug is now corrected at CRAN a year ago, rocker/hadleyverse ignores that.
I rewrote your Dockerfile to base it on rocker/tidyverse (the one recommended by rocker to substitute hadleyverse) to fix that
plus added installation of  openjdk-8 (to automate java installation) and other required packages such as lzma and bz2 and did a javareconf at step 1.
I tested Achilles, SqlRender and DatabaseConnector (the ones depending on JAVA) from the Rstudio that offers the OHDSI/broadsea docker-compose.yml and they all work against my PostgreSQL CDM backend
lastly I commented these lines
## COPY Rserv.conf /etc/Rserv.conf
## COPY startRserve.R /usr/local/bin/startRserve.R

they complain since it does'nt find them in order to copy them, I assume it is expecting them on the host machine, I haven't tested if Rserve works as expected.
hope it helps others, thanks.

I've created the below pull request, so I'm closing this issue.
#3


Subscript Shorthand

Shorthand function enabled
Allowed for Sub-Families to contain 1 element
In-line lookups
Subscripted Lookups

mistake


bug: target-pinecone cannot add - meltano add loader target-pinecone failed

Meltano Version3.3.2
Python Version
3.10.6
To Reproduce
Steps to reproduce the behavior:
meltano add loader target-pinecone

Bug scope
CLI (options, error messages, logging, etc.)
Operating System

Appe M1 Pro: macOS Sonoma v 14.2.1

Screenshots

@DaniPoot it looks like the issue you're running into is Loader 'target-pinecone already exists in your Meltano project. You'll need to either remove it or use the inherit-from feature to create another version of it.

@pnadolny13 I've removed the loader, tried to install again and I get another error.

@DaniPoot
This is https://github.com/MeltanoLabs/target-pinecone/issues/16, which I logged a while ago but never got to submit a PR since I had no way to test the changes, but it seems fine in CI.
I've now submitted https://github.com/MeltanoLabs/target-pinecone/pull/30.
Can you update your meltano.yml to point to the branch?
pip_url: git+https://github.com/MeltanoLabs/target-pinecone.git@refs/pull/30/head

And try installing again?


linux: run make install in parallel

Motivation for this change
This improves build times significantly, from 20min to 9min on the aarch64 community box.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Should this actually go to staging? For ordinary kernel bumps I'm normally fine with it, but this is every kernel package + every dependency...

I don't think so, since it's only the kernels and modules that will be rebuilt.

This seems weird! Is Linux building things in the install phase? I always assumed that make install would just do install / cp so that cores wouldn't make a difference, and the bottleneck would be IO.
If this is a common experience, perhaps we should set this everywhere for make install when enableParallelBuilding is set (or add an enableParallelInstalling option).

This seems weird! Is Linux building things in the install phase? I
always assumed that make install would just do install / cp so
that cores wouldn't make a difference, and the bottleneck would be IO.
If this is a common experience, perhaps we should set this everywhere
for make install when enableParallelBuilding is set (or add an
enableParallelInstalling option).

There's process overhead for every file being installed, and I wouldn't
be surprised if that's the bottleneck when installing non-parallel onto
fast storage.

This seems weird! Is Linux building things in the install phase? I
always assumed that make install would just do install / cp so
that cores wouldn't make a difference, and the bottleneck would be IO.

It does build things in the install phase as well, I just noticed.

Yes, it builds at least dtbs, and also does some compression e.g. of kernel modules. Running 64 xz's in parallel is going to be faster than running them in series :D


[AVRO-2943] Add new GenericData String/Utf8 ARRAY comparison test

https://issues.apache.org/jira/browse/AVRO-2943

@dkulp could you take a look and merge if there are no objections?


Feat:Add hover cards for "about us" page

Describe the feature
Add hover cards to improve the UI
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@AkhilaSunesh Do you have any design or reference about how you want to implement about us page?

https://github.com/Satyam1923/Spring-Music-Player/assets/140897461/ac57dc2d-94f5-49e2-a7d0-ec4f74006c65
There will be a smooth transition


[SPARK-42974][CORE][3.4]  Restore Utils.createTempDir to use the ShutdownHookManager and clean up JavaUtils.createTempDir method.

What changes were proposed in this pull request?
The main change of this pr as follows:

Make Utils.createTempDir and JavaUtils.createTempDir back to two independent implementations to restore Utils.createTempDir to use the spark.util.ShutdownHookManager mechanism.
Use Utils.createTempDir or JavaUtils.createDirectory instead for testing where JavaUtils.createTempDir is used.
Clean up JavaUtils.createTempDir method

Why are the changes needed?
Restore Utils.createTempDir to use the spark.util.ShutdownHookManager mechanism and clean up JavaUtils.createTempDir method.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GitHub Actions

cc @srowen @sadikovi backport to 3.4

Merged to branch-3.4.

thanks @dongjoon-hyun @sadikovi


Remove this function

https://github.com/Susannova/Discord_Bot/blob/4c5dd64e05caeee7cfccf847fab642eb6694d708/core/bot_utility.py#L84
We now use always the id. Then you can just use bot.get_channel(channel_id) to get the channel

file does not exists anymore so i assume its removed


modified doc string of memmodel/linear_model.py

Changed docstrings of memmodel/linear_model.py to include code examples

@mail4umar as all these objects are classes, do you have any best practices to display them nicely?

@abhsharma2 you can update the changes and then run the checks.


Can't list casks

Please always follow these steps:

[X] Confirmed this is a problem with Homebrew/brew and not one or more formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new
[X] Ran brew update and retried your prior step?
[X] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[X] Ran brew config and brew doctor and included their output with your issue?

I believe this to be a brew issue rather than cask, but I can't brew cask list.  I'm only seeing this on one of my computers and not another.
Here's the full transcript including the requested steps:
$ brew doctor
Your system is ready to brew.
$ brew config
HOMEBREW_VERSION: 1.1.12-38-g30582a5fb
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 30582a5fbd35843e183f1591cc8413309fa5e705
Last commit: 3 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 35eb0880b0b500384d3b9bfee82fbaaa0ba25651
Core tap last commit: 2 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/Homebrew
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: octa-core 64-bit sandybridge
Homebrew Ruby: 2.0.0-p648
Clang: 8.1 build 802
Git: 2.12.2 => /usr/local/bin/git
Perl: /usr/bin/perl
Python: /usr/local/bin/python => /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python
Ruby: /usr/bin/ruby => /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
Java: 1.8.0_121
macOS: 10.12.4-x86_64
Xcode: 8.3.1
CLT: N/A
X11: 2.7.11 => /opt/X11
$ brew doctor
Your system is ready to brew.
$ brew cask list
Error: Calling "cask :v1 => 'token'" is deprecated!
Use "cask 'token'" instead.
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:9:in `load'

Follow the instructions here:
  https://github.com/caskroom/homebrew-cask#reporting-bugs
/usr/local/Homebrew/Library/Homebrew/utils.rb:96:in `odeprecated'
/usr/local/Homebrew/Library/Homebrew/compat/hbc/cask_loader.rb:6:in `cask'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:9:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:9:in `instance_eval'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:9:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:41:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:128:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/scopes.rb:40:in `block in installed'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/scopes.rb:30:in `map'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/scopes.rb:30:in `installed'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/list.rb:63:in `list_installed'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/list.rb:14:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:115:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:158:in `process'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:91:in `<main>'

What is the output of brew cask doctor?

Do you have HOMEBREW_DEVELOPER set? Otherwise this should only be a warning:
Error: Calling "cask :v1 => 'token'" is deprecated!

Yes, HOMEBREW_DEVELOPER is set as I like to build directly from source.  Output from brew cask doctor also doesn't show any issues:
$ brew cask doctor
==> Homebrew-Cask Version
Homebrew-Cask 1.1.12-38-g30582a5fb
caskroom/homebrew-cask (git revision f3b38; last commit 2017-04-14)
==> Homebrew-Cask Install Location
<NONE>
==> Homebrew-Cask Staging Location
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads
~/Library/Caches/Homebrew/Cask
==> Homebrew-Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask (3633 casks)
/usr/local/Homebrew/Library/Taps/blogabe/homebrew-xtg (0 casks)
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core (0 casks)
==> Contents of $LOAD_PATH
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin16
/Library/Ruby/Site/2.0.0/universal-darwin16
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin16
==> Environment Variables
LANG="en_US.UTF-8"
PATH="~/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/Homebrew/Library/Homebrew/shims/scm"
SHELL="/bin/bash"

Thank you.

HOMEBREW_DEVELOPER is set as I like to build directly from source.

I don't think HOMEBREW_DEVELOPER is necessary to build from source. Either pass --build-from-source or set HOMEBREW_BUILD_FROM_SOURCE.

This error is happening likely because you installed some casks when cask :v1 => 'token' was valid, and have never reinstalled or --force installed or uninstalled them, and they’re being read from their .metadata directories.
To fix this and keep HOMEBREW_DEVELOPER on, you need to first turn HOMEBREW_DEVELOPER off then update/reinstall/force install all the casks (well, technically just the old ones) with something like
for cask in $(brew cask list); do
  brew cask reinstall "${cask}"
done

But some of them might need some further manual fixing. Then you can reactivate HOMEBREW_DEVELOPER. But as @reitermarkus mentioned, that should not be necessary for your case. In your position, I’d simply turn it off and turn on the option to build from source.

Even with setting HOMEBREW_DEVELOPER, and for that matter HOMEBREW_BUILD_FROM_SOURCE, to '0', I still cannot brew cask list.
Same error message as above.

BTW: I went through and uninstalled/reinstalled all Casks upon the big change however long ago from installing casks to /opt to homebrew's location.  I believe that was after v1, but may be wrong.

Even with setting HOMEBREW_DEVELOPER, and for that matter HOMEBREW_BUILD_FROM_SOURCE, to '0'

They need to be completely unset; i.e., unset HOMEBREW_DEVELOPER.

That did it.  Thank you.
Is there an easy way to find out with cask(s) I need to reinstall rather than going through and doing them all?

Is there an easy way to find out with cask(s) I need to reinstall rather than going through and doing them all?

If brew cask list is not showing any Calling "cask :v1 => 'token'" is deprecated! warnings anymore, you should be good.


[1.19.2] [Forge] GeckoLib 4 has failed to load correctly

Log:
crash-2023-02-11_21.34.45-fml.txt
Error shown in Minecraft Launcher:
java.lang.NoClassDefFoundError: net/minecraftforge/event/CreativeModeTabEvent$Register

Nevermind, I overlooked the mod version


init: Remove unnecessary call to package-initialize

According to Emacs 26.x+, this is not necessary.  It certainly seems strange, but making this change does not affect the results of startup at all for me, besides shaving just a little bit of time off.
Since we require every ELPA package before we configure it, this shouldn't have any danger of affecting reachability, unless that assumption is violated anywhere.
Resolves #123.

Subsumed by #145. Closing.


Make new Parser and Serializer instances

Fixes #15 Make new Parser and Serializer instances per call to parse,
parseFragment, and serialize

LGTM.


Invitation of Winter of Blockchain'24

Hello,
I am Durgesh Kumar Prajapati, Program Manager of WoB'24 (Winter of Blockchain) I have seen you are really doing appriciable work on Web 3.0 that why i am inviting you in our program. If you are interested you can apply as:
Mentor: https://forms.gle/LdLwHYNE1CzC4ZwM9
Project Admin: https://forms.gle/yQzJnddb8jPD66Ue8
If you are interested in contributing, please complete this form by 10 July. We will review your application and provide further details upon acceptance.
Thank you for considering this opportunity to contribute to the Winter of Blockchain program. We are excited about the potential impact of our collaborative efforts and look forward to your participation.
Warm regards,
Durgesh Kumar Prajapati
Program Manager, WOB'24

@mannasugo

hey what is this about

The best hackers I have seen are  d a r k w e b p r o g r a m m e r @ g m a i l . c o m   , they got me $15,000 via hack


Error: Token renewal operation failed due to timeout

Seems related to #4214 and #1420 - however navigating to the URL mentioned only returns a page saying "Sorry, something went wrong. An unexpected error has occurred." - there is no error ID but status code is "302 Found".
We originally deployed in early November with multiple deployments since then and no issues, it has only started appearing on our production environment yesterday after our last deployment (changes were minor CSS changes).
Tried with signing out, clearing browser cache, incognito - no luck. However, after using Chrome Devtools and navigating to Application tab > Storage > Session storage and clearing the values under xx.sharepoint.com and xx-isolated-domain.sharepoint.com, refreshing the page shows that the error disappears, and does not come back. This issue does not appear for everyone, and did appear once in our dev tenant (current issue is in our production tenant), but once it resolved itself it hasn't appeared since.
There is an error message prior to the token renewal error "Refused to display "" in a frame because it set 'X-Frame-Options' to 'sameorigin'. This message doesn't show up if there is no token renewal error.
Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
Graph call to complete properly to give details of signed-in user's directReports. React-based webpart (v1.9.1, isDomainIsolated
is TRUE), bundled, packaged, deployed. API permissions granted in the SPO Admin Center. App added to a site. Client-side webpart added to a modern page. Call is observed with Chrome Dev tools.
Observed Behavior
Call fails with the error observed in the Chrome Dev tools.
Steps to Reproduce
Simple Graph call as below
const client: MSGraphClient = await context.msGraphClientFactory.getClient();
const res = await client
    .api("me/directReports")
    .version("v1.0")
    .select("id,displayName,onPremisesSamAccountName,jobTitle")
    .get()
    .catch(err => {
       if (err) {
         console.error(err);
           return;
       }
    });

Appears to be a dupe to an existing open discussion #4892 ... per our normal process, we try to limit dupe issues so closing.


Support const values

This is important for discriminator values in oneOfs. Coming to OpenAPIKit in the next release.

Actually, this is usable in both 3.0 and 3.1. Even JSON Schema explicitly says that const is equivalent to enum with a single value: https://datatracker.ietf.org/doc/html/draft-bhutton-json-schema-validation-00#section-6.1.3
So in 3.0, you do enum: [myValue], in 3.1 you do const: myValue.

Note for when we're implementing this: in code, we should generate the hardcoded value, so the compiler knows it at compile time already. But we also need to generate a custom decoder and validate that the value matches the const, as otherwise it shouldn't validate according to JSON Schema.

Even JSON Schema explicitly says that const is equivalent to enum with a single value ... So in 3.0, you do enum: [myValue], in 3.1 you do const: myValue.

Note for when we're implementing this: in code, we should generate the hardcoded value, so the compiler knows it at compile time already...

Just to clarify, this doesn't mean we should start emitting a hardcoded value when the OpenAPI document contains an enum with one value, right? For that case, I'd still expect us to emit an enum with one value.

I think so, yeah. We already support enums today, and if only one value is documented, the practical use case matches const. But if the const feature is used, the generated code should skip generating an enum altogether, and emit specific validation code into the decoding initializer.

I started investigating this, and this seems to be more work than it's currently worth.

A more common way of spelling a constant value is a 1-element enum, which already works today.
We could generate a const value as as 1-element enum, but that seems like the ease of implementation driving the API. There should be a first class type for this.
We might need to generate the whole thing, which would be a generated struct that wraps a string/int/another constant value and in decoding, would verify the value on the wire matched the expect one, and throw otherwise, and in encoding, it'd always serialize the constant value.
I scanned a couple of thousand OpenAPI docs in the wild and found a single occurrence, making this much less used than may other features still in our TODO list.

For these reasons, moving this to post-1.0 towards the end of the list, as it doesn't seem high priority or urgent.


Support Full transaction processing for payment and stateful tx

Added support for closeOut and clearState application calls
Added tests
Added support for closeRemainderTo, test
Updated crowdfunding test
Added support for Minimum balance in account, -ve amount checks. Updated impacted tests.
Update /docs

LGTM, added few comments

LGTM, added few comments


Direct video / download links

Occasionally I want to download a video from Nebula to watch it offline later. My current process is using VideoDownloadHelper to find a direct link to the video stream that can be then saved using VLC.
However, especially the first part of this process is cumbersome. It would be nice if extension would provide a direct link to the video stream that can be saved with VLC or similar program.
I would be up for creating a PR for this, if accepted.

VDH is literally a downloader, why go via VLC?
Did you look into getting this link? Because it's not easy. Without decoding the HLS separately, I don't think there's a good way, since the Nebula player uses HLS. But if you have a solution, please do a PR.

VDH is literally a downloader, why go via VLC?

I'm only using the extension. Kinda put off by installing extra software on my PC.

Did you look into getting this link?

Huh, this seemed much easier in the Zype days, but now it seems way more complicated. Will take a look at it at some point.

I'm only using the extension. Kinda put off by installing extra software on my PC.

The extension can usually handle downloads, though I haven't tried it in this instance. But isn't VLC extra software as well?

Will take a look at it at some point.

Good luck, I look forward to it :)

I've looked into it for a while and gave up. I could get the playlist links, but video and audio were in the separate files, so this method would be a lot of hassle.
Instead, I've discovered that https://github.com/yt-dlp/yt-dlp seems to support Nebula, so I will likely use this.

yt-dlp is a nice project :)
Could you please share your methods of finding the playlists? Maybe this could help with #39, since I'm a bit stuck with that feature as well


acq_time for sessions.tsv ?

It seems that acq_time and pathology are now mentioned as columns for sessions TSV files.
I am not sure why this happened, but I am not sure it makes sense to have acquisition time in sessions.tsv AND scans.tsv.
And if we do have an acq_time then the current definition is taken from the scan level and read bizzarrely for sessions.
Am I missing something here?
For pathology, was that introduced through the sample entity PR?
@tsalo is it possible we did not cross check things properly when we started using the schema for this?
See the latest version:
https://bids-specification.readthedocs.io/en/latest/03-modality-agnostic-files.html#sessions-file
See the stable version
https://bids-specification.readthedocs.io/en/stable/06-longitudinal-and-multi-site-studies.html#sessions-file

I believe I added acq_time based on the example TSV (see below for screenshot from stable version):

I didn't realize the definition read weirdly (I probably didn't really look at it in context), so we should probably adjust that. I assume we want to allow/support acq_time for sessions files since it's been in the example snippet for quite some time.
I added pathology in the TSV schema PR as well because the definition added in #812 says "the pathology MAY instead be specified in Sessions files in case it changes over time."

I didn't realize the definition read weirdly (I probably didn't really look at it in context), so we should probably adjust that. I assume we want to allow/support acq_time for sessions files since it's been in the example snippet for quite some time.

Well it is just a bit incompatible with the fact that there will be one row per session.
Current version for sessions.tsv:
Acquisition time refers to when the first data point in each run was acquired. Furthermore, if this header is provided, the acquisition times of all files that belong to a recording MUST be identical.

Maybe something like:
Acquisition time refers to when the first data point of the first run was acquired.

I added pathology in the TSV schema PR as well because the definition added in #812 says "the pathology MAY instead be specified in Sessions files in case it changes over time."

ah OK I forgot about this one.

Damn ... I think both of you are right (Taylor for including it; Remi for pointing out it seems weird).
I agree with your direction above @Remi-Gau to rephrase what acq_time reads like ... that is: make it more general, and less specific to scans.tsv, while hopefully not losing any detail.
Furthermore, I think the acq_time description contains too much information that is better specified in the spec text: The anonymization stuff. The description already says:

Datetime should be expressed as described in Units.

And in "Units", there is a good documentation of how anonymization can be done ... so maybe we could just delete that "duplicate" part from the acq_time description?


Change default gateway for shareable links?

Is your feature request related to a problem? Please describe.
Currently, the shareable links default to using ipfs.io to resolve the content. It would be nice if we were able to set which gateway we'd like to use (whether another public gateway or our own).
Describe the solution you'd like
Ideally, there's a box somewhere I can enter a gateway URL to be used whenever I copy from a shareable link.

This is a good idea.

MVP:  define default gateway URL in the config file (src/common/store.js)  and use value from the  config when copying link
Better: allow user to set value in config via UI (prompt for gateway URL + validate it is correct by loading test CID and comparing payload)

I'm happy to review a PR if someone wants to take a stab at this.

This is a good idea.

MVP:  define default gateway URL in the config file (src/common/store.js)  and use value from the  config when copying link
Better: allow user to set value in config via UI (prompt for gateway URL + validate it is correct by loading test CID and comparing payload)

I'm happy to review a PR if someone wants to take a stab at this.

@amackenzie In the meantime, you can specify public gateway (for your own fallback, and for shareable links) in IPFS Companion's prefs.

@amackenzie In the meantime, you can specify public gateway (for your own fallback, and for shareable links) in IPFS Companion's prefs.

(moving to ipfs-webui repo, since this work would happen upstream)

(moving to ipfs-webui repo, since this work would happen upstream)


Add Stable API Version for point-in-time restore.

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

Hi @amisi01, please fix the failing CI. Thanks!

Hi @amisi01, please fix the failing CI. Thanks!

Hi @YalinLi0312 , I have fixed the CI. Could you please take a look. Thanks

Merged swagger PR link - https://github.com/Azure/azure-rest-api-specs/pull/14627

@amisi01 Please also update the versions in AssemblyInfo.cs.


Need help to setup tinyfecvpn+udp2raw on koolshare

I able to install koolshare lede x64 on pc and I installed udp2raw and tinyfecvpn I have question can I run tinyfecvpn and udp2raw server and client at same device (koolshare lede x64) and iinstalled koolsoft game accelerator which is tinyfec and udp2raw and tinymapper

 but I get there may be a problem with your connection I put server ip which my public ip I checked tun device of tinyfecvpn there is no rx packets I can't figure it out plz anyone guide me in details to run udp2raw and tinyfecvpn on koolshare to accelerate ps4 connection thanks in advances

do i need vps or i can use my public ip ?? (what does udp2raw require to run)

To run udp2raw and tinyfecvpn, you must have something like a VPS. You will need to run udp2raw server and tinyfecvpn server on your VPS.
The udp2raw and tinyfecvon on your koolshare router are only the client parts, you use your clients to connect to the servers that's the first step to make it work.
The koolshare plugin of udp2raw and tinyfecVPN is a wrapper developed by koolshare,  the plugin is not developed by me and I have little knowledge about it.  I can't give you a detailed guide on how to use it.

is there another way to run it using only 1 pc (ubuntu 20.04)  maybe proxy or anything ???

is there another way to run it using only 1 pc (ubuntu 20.04) maybe proxy or anything ???

Just in case anyone ends up here:
tinyfecVPN is a VPN solution that mandatory requires a server-side implementation. As it's nature allows only up to one client per server instance, You would have to be the one hosting it's server appliance at a VPS, or some middle point. Always considering how it will impact your latency performance (For ex: If you intend gaming).
udp2raw it's a really clever way to avoid bad UDP traffic practices done by ISPs (Could be done intentionally or just bad technical implementations), Firewall UDP filtering (Which also could be configuration), and many other other reasons that may, for what the author asked initially, affect the gaming performance (Although it could affect many other usages).
The Koolshare Game Accelerator appliance is a wrap up of all those solutions, over a well known router custom firmware which is LEDE/OpenWRT. Still will require of a server side solution, which in this case could be another Koolshare Game Accelerator router/VPS on some remote location.


Fix broken tests

Which issue(s) this PR fixes
Fixes #333
Agreements

[x] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004.
[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

@sergkudinov Do you prefer to merge this one "now" and do a second pr later to fix the remaining tests or do you want to wait for the #337 to be fixed?

I checked this changes on #344 PR's branch, it works. Just waiting #334 merged to master...

#344 just got merged, you can go ahead.

@PaulFarault
it works on my computer
43 of 43 passed

just rebased on latest master


SOL-121556: Golang: Implement code changes to support the endpoint provision feature

Changes:

SOL-121556: Golang: Implement code changes to support the endpoint provision feature
feat[SOL-121556]: fixed failing tests
SOL-121557: Golang: Implement the endpoint deprovision  feature
chore: removed some unnecessary comments from some tests
feat[SOL-121560]: wrote integration tests to provide 100% coverage for the new code changes

Looks like there are statichceck and got vet failures witht he source please check the git actions.

Looks like there are statichceck and got vet failures witht he source please check the git actions.

Yeah, I have fixed it. Thanks

Few things to note integration tests now take 8 more seconds compared to dev.
596.401 seconds current branch.
588.128 seconds last dev branch.
And Coverage went from 89.5% down to 89.1%.
I believe this branch it not up to date with the base branch SOL-106478.
Please synchronize the branch.
Other then that the quick look I took is in line with @sdewilde comments. I'll take a closer look once the branch is up to date.

@cjwmorgan-sol, these are noted. I think the idea is for this branch to be merged into the feature base branch (SOL-106478) and not the other way around. We can merge this into the base branch and continue our review there before we merge that new PR into dev.


getAnimeRelations and other missing endpoints

Hello,
I see 100% coverage, but I think is not clear, or it is missing AnimeRelations, and some others...
thx for the work already done 👍

Hello. The coverage represents the tests percentage of the current endpoints, sorry for the confusion.
I will look into implementing the missing endpoints.

Hello @HugoLeBoennec sorry for the long wait. I just updated the Anime client with a few more endpoint, including getAnimeRelations.  Should be available in version 1.2.0

Hi @HugoLeBoennec sorry for the lateness. This is available in version 2.1.0 :)


Add support for Smart Alarm

Would you please add this device, it is working on Tuya app, but on plugin showing as not supported, and on HB-UI accessories (i can not see it in the list)
Device info
1- from json:
{
"active_time":<PHONE_NUMBER>,
"biz_type": 0,
"category": "sgbj",
"create_time":<PHONE_NUMBER>,
"icon": "smart/icon/bay1582725542891RIK5/00740aed253fa09a4283b8fad6ae3b51.png",
"id": "bf8e818ec57400c71eehjl",
"ip": "",
"lat": "",
"local_key": "",
"lon": "**********",
"model": "SH.63.220",
"name": "Smart Alarm",
"online": true,
"owner_id": "71465053",
"product_id": "zynxlsbb1owzhz4n",
"product_name": "Smart Alarm",
"status": [
{
"code": "alarm_switch",
"value": false
},
{
"code": "alarm_time",
"value": 10
},
{
"code": "alarm_volume",
"value": "middle"
}
],
"sub": false,
"time_zone": "+01:00",
"uid": "***********",
"update_time":<PHONE_NUMBER>,
"uuid": "815a42525b099074",
"schema": [
{
"code": "alarm_switch",
"mode": "rw",
"type": "Boolean",
"property": {}
},
{
"code": "alarm_time",
"mode": "rw",
"type": "Integer",
"property": {
"unit": "",
"min": 1,
"max": 30,
"scale": 0,
"step": 1
}
},
{
"code": "alarm_volume",
"mode": "rw",
"type": "Enum",
"property": {
"range": [
"low",
"middle",
"high",
"mute"
]
}
}
]
2- from mqtt logs:
[2-1-2023 22:58:43] [@0x5e/homebridge-tuya-platform] [TuyaOpenMQ] onMessage:
topic = cloud/token/in/fce9cfa9425b6b495a84ab1d0ceff5a6
protocol = 4
message = {
"dataId": "09c488a4-2481-4cf0-9632-d99b1c3372c6",
"devId": "bf8e818ec57400c71eehjl",
"productKey": "zynxlsbb1owzhz4n",
"status": [
{
"1": "mute"
},
{
"13": false,
"code": "alarm_switch",
"t": "1672696722",
"value": false
}
]
}
t =<PHONE_NUMBER>

There's no suitable homekit sensor for sgbj. Maybe Leak Sensor is the last choice. Which one do you prefer?
https://developers.homebridge.io/#/characteristic/

You asked which on I prefer? I see only one option! if no other choice then let have it. Thx in advance

https://developer.tuya.com/cn/docs/iot/s?id=K9gf48qox8rvt
seems your device is missing alarm_state. can you test each dp code on tuya web device debugging tool, I need to know which one means alarm event.

I tried to do this, but firstly I am not so expert to carry out this, and secondly I can't see the alarm_state in the persist file, even the alarm event!
Here is Copy&paste from
Configure Control Instruction Mode
LSC SMART Connect Sirene
Devices:Product Type:Siren
Select Control Instruction Mode
This configuration will affect the control instruction mode of the specified product under this project. Before you change the mode, refer to the following table of instructions and check the instructions applicable to this product in the two modes respectively.

Table of Instructions

DP Instruction
Standard Instruction

alarm_state
alarm_state

alarm_volume
alarm_volume

alarm_time
alarm_time

alarm_switch
alarm_switch

Code
Type
Values

alarm_state
Enum
{   "range": [     "light",     "voice",     "light_voice",     "mute"   ] }

https://developers.homebridge.io/#/characteristic/

Is it possible to use something related to security?
https://developers.homebridge.io/#/service/SecuritySystem


How to properly remove selected collection

Try to figure out delay removing collection from view selected parameters list (seems like right now you have to search that collection granule to get it to actually clear)

It was checking to make sure the event.data ended in search


Document probe and scoring methods

Description
Resolves #214.
The methods of the two main classes of the project were poorly documented which created misunderstanding and questions.
Here I'm proposing some documentation for those methods so anyone trying to contribute a new Probe or Scoring implementation can better understand what is expected of the methods.
There is no tests because it's just javadoc modifications and additions.

Submitter checklist

[x] If the issue exists, it is well described and linked in the description
[x] The description of this pull request is detailed and explain why this pull request is needed
[x] The changeset is on a specific branch

feature/ for new feature, or improvements
fix/ for bug fixes
docs/ for any documentation changes

[x] If required, the documentation has been updated
[x] There is automated tests to cover the code change / addition

If there is no test, include an explanation why in the description

[x] Run mvn verify locally and all tests are passing successfully

It is OK to create a pull request which has failing tests if it is created as a draft, is to fix a bug and the first commit is the test to prove the existence of the bug.

[x] There is no new warnings (checkstyle nor spotbugs) on the code

Thank you @alecharp for this PR. Looks good to go.


[Docs] Update developer guide

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (41e85b9) compared to base (1cc16e1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #20   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

User stories are okay so far for v1.2. LTGM! 👍


update index.rst

Hi @vladimirhasko can you review the commit and merge it, if everything is fine. Or who is responsible for that?

Hi @vladimirhasko can you review the commit and merge it, if everything is fine. Or who is responsible for that?

hi Martin,
as initially this document was defined to be API document and not UMN document please move the content to api-ref/source
otherwise jobs won't render the document properly.

Hi @vladimirhasko,
we updated it. Does it work this way?

Hi @vladimirhasko,
we updated it. Does it work this way?

Yes and No.
Finally it tries to build the doc but fails on syntax RST errors.
Please have a look on these errors and fix syntax issues:
https://swift.eco.tsi-dev.otc-service.com/v1/AUTH_73733c21728443609ee0b47f64b5ce0b/zuul_logs/47d/5/5ba27938fd211d3bf48cb5e79b8869bb27cff860/check/otc-tox-docs/47dd3d6/tox/docs/3-commands[2].log

@martinh-dev ,
if you want to use code-block: json you need to use strict json syntax without "..." strings otherwise it will break.
or rather use code-block: console if you don't want to follow strict json syntax

Hi @vladimirhasko,
the check was performed succesfull now. Can we move on?

In the preview I can just see the empty price calcualtor page, but we want to show the API documentation. I can't se it in the preview. Is that possible?

recheck


Text editing cleanup/testability

Move response parsing to protocol
Extract non-sublime-tied logic from commands
Remove debugging status/logging when applying doc edits.
Simplify sorting of document edits (assuming stable sorting)
To Dos:

[ ] write unit tests for sorted editing.

Coverage increased (+0.4%) to 31.891% when pulling b884d8d207f69b5944a740920c2c03ceb44deb67 on refactor-edit into bc46965456d5bde26accc4e7ff8287c8231c406e on master.


errors in paths containing mixed absolute and relative commands

if path contains only relative commands, like d="M 1031,175 v 150 h 8 v 16 h 92 v -16 h 6 v -150 z", SvgPathParser works ok.
if after relative commands there are absolute commands like d="M 1031,175 v 150 h 8 v 16 h 92 v -16 h 6 V 175 H 1031 z", SvgPathParser generates wrong  Path. It looks like parser ignores absolute commands.

I will look it up.

Hello, have you did any chance to look it up?

I am currently looking it up

@eae1212 Found the issue.
What I did
_currentPoint.translate(x,y)

What I should have
_currentPoint = _currentPoint.translate(x,y)

You can get version 1.1.0 from pub.dev.

Thanks a lot! It works great!


Validate empty content of a spreeadsheet with only columns.

how to validate if the spreadsheet have only columns without content, in my situation when a code this: Excel::load($fileImport, function($reader) {
$Array = $reader->toArray();
)};
i get this error "Row 2 is out of range (2 - 1)"
I can do the validation without a package, but i just wanna knows if exist a clean way to do this.

Why version of the package are you on? This exception should have been fixed a while back.

@patrickbrouwers I was using old version i'm sorry, my mistake. thank you.


install.sh: Sync Debian keys and repo source file location with docs

Describe your changes
See https://github.com/netbirdio/public-keys/issues/1#issuecomment-1736322449
Issue ticket number and link
Checklist

[x] Is it a bug fix
[ ] Is a typo/documentation fix
[ ] Is a feature enhancement
[ ] It is a refactor
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @shvchk for your contribution.


Added connection timeout (optional)

Added the ability to set connect/read timeout for requests to prevent long (some tim every long) requests...

😒

😒

😒


add new section

Codecov Report

Merging #35 into master will decrease coverage by 10.58%.
The diff coverage is 1.56%.

@@             Coverage Diff             @@
##           master      #35       +/-   ##
===========================================
- Coverage   34.24%   23.65%   -10.59%
===========================================
  Files          36       35        -1
  Lines         403      613      +210
===========================================
+ Hits          138      145        +7
- Misses        265      468      +203

Impacted Files
Coverage Δ

src/ApiBundle/Decoder/TextDecoder.php
100% <ø> (ø)
:arrow_up:

src/ApiBundle/Decoder/MarkdownDecoder.php
100% <ø> (ø)
:arrow_up:

src/ApiBundle/Decoder/JsonDecoder.php
100% <ø> (ø)
:arrow_up:

src/ApiBundle/Transformer/YamlTransformer.php
0% <ø> (ø)
:arrow_up:

src/ApiBundle/Transformer/JsonTransformer.php
0% <ø> (ø)
:arrow_up:

src/ApiBundle/Twig/Filters/RawCodeFilter.php
100% <ø> (ø)
:arrow_up:

...rc/ApiBundle/Transformer/SerializedTransformer.php
0% <ø> (ø)
:arrow_up:

src/ApiBundle/Decoder/SerializedDecoder.php
100% <ø> (ø)
:arrow_up:

src/ApiBundle/Transformer/DownloadTransformer.php
0% <ø> (ø)
:arrow_up:

src/ApiBundle/Transformer/XmlTransformer.php
0% <0%> (ø)
:arrow_up:

... and 34 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 31d059e...d4984bc. Read the comment docs.


Daphne does not work on roms2/ 2nd SD card on RG351V

When attempting to get Daphne to work, I eventually got it working by putting the files in the roms directory (as opposed to roms2).  I have specified that ArkOS uses the SD card, but this emulator seems to ignore that.

Have you updated the ArkOS install since you installed the image?

Running OaD 1.6 (I updated today).  Thanks

They'll be an update that will correct the script that caused that.  When the update is available and you've completed it, just switch to main, then switch back to sd2 and it should correct the issue for you.

Thanks... You aren't the only firmware that takes a bit of Daphne-manipulation.  The "other 2 firmwares" were configurable from their EmulationStation es_systems scripts.  I could not find an equivalent for ArkOS, hence a ticket.  Cheers.

New update for ArkOS OaD Test Version:
-Added UAE4arm_libretro.so for retroarch32 for Amiga and Amiga32
-Added potator core for Watara Supervision
-Added section for MD-MSU  using the genesis_plus_gx retroarch core
-Updated /usr/bin/emulationstation.sh and switch to sd2 and switch to main scripts to be more stable.
-Fixed Daphne not loading from SD2
-Updated emulationstation to support scraping for Watara Supervision
Just update from the Options section