v0.4r2 missing a GB?

First, thanks for this fantastic project. Now have wx in the Glasair.
I just downloaded v0.4r2. I notice its .img is 1GB smaller than previous releases, even tho the zip file is same as others. (I dnld'd & decompressed it twice.) It boots quickly, but it never makes the wireless WiFi blink. No stratux SSID either. Is it just me or is there something missing from the image?

There's something wrong with it, there was a problem with the upload it appears. Github afaik only has the web interface which takes some time to upload with. I'm re-uploading the zip now, looks like 10-15 mins more.

Thx. Whew - glad to know it wasn't me. Surprising that 7zip unzipped it as an image. (Serves me right for not checking hash.)

Alright, try again.

Yep - boots and connects. I can't get any towers from the couch, so I can't report whether it stays connected now, but I did have the uptime disparity in flight a few days ago.
I probably posted in the wrong forum for this kind of report. (Should have been in reddit instead.) My first time on github. I didn't mean to make an Issue (# 75) about it.
Thanks again for bringing wx to our cockpit for less than the cost to fill the tanks.

Correction: Issue #77.


error

Dear @mathii,
please see below:
python3.12 eigenstrat2vcf.py -r humandna
Traceback (most recent call last):
File "/Volumes/WD_BLACK/AADR/1/eigenstrat2vcf.py", line 7, in
import argparse, gdc, pyEigenstrat
ModuleNotFoundError: No module named 'pyEigenstrat'

Yes you need the pyEigesnstrat package here: https://github.com/mathii/pyEigenstrat
But both that package and this one need python2 (2.7+ probably) not python3.

you are trying to install some other random package named gdc. These scripts do not need to be installed - just run them directly

as long as the script gdc.py is in the same directory as the script you are trying to run, it should work.

as I told you already, you need to install it.
try

git clone git@github.com:mathii/pyEigenstrat.git
cd pyEigenstrat
pip install .

or something like that. If you continue to have problems you should ask for help with someone who is more familiar with python.

delete the file eigenstrat2vcf.py from the directory you are running in.  If you continue to have problems you should ask for help with someone who is more familiar with python.


Adding features to the OpenTitanUart driver

I fleshed out the OpenTitan UART driver a little while working on https://github.com/lowRISC/sonata-software/pull/26. I don't think this is quite ready to be merged yet; opening this purely for visibility.

There's a bunch of stuff in here that's just improving the existing code, rather than adding new things.  It would be great to have those merged soon (I don't anticipate problems with the other bits either).

There's a bunch of stuff in here that's just improving the existing code, rather than adding new things. It would be great to have those merged soon (I don't anticipate problems with the other bits either).

:+1: Rebased onto https://github.com/microsoft/cheriot-rtos/pull/278#event-13694639924 and marked as ready for review.

We probably need to document somewhere that this changes the behaviour of stderr, for anyone who is using stdio.h.

We probably need to document somewhere that this changes the behaviour of stderr, for anyone who is using stdio.h.

Where would be a good place to document this change?

Good question.  A blog post maybe, and possibly in the Sonata bit of the Getting Started doc?


An easy way to use hatching in polygons

First I want to thank you for the library. Simply incredible.
I have a question: it is necessary to hatch polygons according to a given pattern (example)
Is there any easy way to use ol-ext with your library?
Or is this functionality already available (but didn't find it)?
Thank you

Did it, but I really hope there is a better way to do this.

No, I don't see how you could reduce the boilerplate code in this case? You have to have a separate component type that wraps this feature. I have tried to extract all the common logic in the base class, but you still have to enumerate the properties.
Otherwise, there is no support for any ol-ext features - maybe having an additional package should be the way to go for those.


Async-listener breaks method dispatch for promise subclasses

Test case:
require('async-listener');
class P extends Promise {
  then() {
    console.log('In Subclass');
    return super.then.apply(this, arguments);
  }
}

P.resolve(5).then(v => {
  console.log(v);
});

Expected output (and observed when require('async-listener'); is removed):
> In Subclass
> 5

Observed output:
> 5

This issue breaks the got module (by breaking p-cancelable) and can be reproduced with node version 6.x and 8.x.

I believe this snippet illustrates the underlying problem:
function WrappedPromise(executor) {
  var p = new Promise(executor);
  p.__proto__ = WrappedPromise.prototype;
  return p;
}

WrappedPromise.__proto__ = Promise;
WrappedPromise.prototype.__proto__ = Promise.prototype;

WrappedPromise.prototype.then = function then(onFulfilled, onRejected) {
  console.log('WrappedPromiseThen');
  return Promise.prototype.then.call(this, onFulfilled, onRejected);
};

class P extends WrappedPromise {
  then(onF, onR) {
    console.log('PThen');
    return Promise.prototype.then.call(this, onF, onR);
  }
}

(new P(res => { res(42); })).then(v => { console.log(v); });

Async-listener replaces the global promise with its own function-based class. Other modules extend the global Promise using ES6 class syntax. However, extending the function-based wrapper class does not result in the correct dispatch behavior. I think the best approach may be to rewrite the wrappedPromise as an ES6 class. Does anyone have thoughts on that change or any other possible approaches?

@watson @Qard @othiym23 Any thoughts on this?

@matthewloring Thanks for the ping 😃 I've commented on the PR

Can we re-opening after revert of the original fix due to https://github.com/othiym23/async-listener/issues/121. I don't have permission.

@watson opened a new issue #123

I could also just reopen this if you guys think that is better?

I think the new issue is good


Research alternative authentication mechanisms for BOINC projects

 Issue by grctest
Tuesday Apr 11, 2017 at 21:05 GMT
Originally opened as https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/105

The current state of password reuse within the BOINC environment due to account managers poses a massive security risk, if a BOINC user has a single project's credentials phished then they can log into their Boincstats account and all associated BOINC projects (due to boincstats using the same username/email/password combination).
If there was a more secure auth mechanism found, integration into existing projects may be difficult & compatibility with existing BOINC account managers will break.

 Comment by LokiToki
Wednesday Apr 12, 2017 at 06:27 GMT

It's in the responsibility of the account manager like BAM. When i add a new account the default PW is used. Why account manager don't generate a secure PW and store them like a PW manager? The biggest problem i see is that the user needs a PW for every project he or she wants to log in besides the account manager. Maybe the account manager could support a export function of the users PW database in a KeePass encrypted database file. So it would be an ease to receive the pws and import them to a local KeePass instance.

 Comment by grctest
Wednesday Apr 12, 2017 at 23:25 GMT

Why account manager don't generate a secure PW and store them like a PW manager?
It's just the way things have been implemented, it's certainly the case for Boincstats however it's not confirmed that gridrepublic has the same security practices - that's certainly worth further investigation.

What would be great would be a stand-alone account manager, like a local Boincstats BAM! on your computer, so you'd be in charge of all your own keys.
Being able to export your passwords from boincstats would enable an attacker to gain access to your BOINC projects this way, no? But you have a good point that if they wanted to log into their BOINC project account directly this would be difficult without a SSO implemented.

 Comment by LokiToki
Thursday Apr 13, 2017 at 07:00 GMT

What would be great would be a stand-alone account manager, like a local Boincstats BAM! on your computer, so you'd be in charge of all your own keys.

It would be another "app" the user have to install. But what if we think a bit further. Imagine a Secure Boinc Manager which encapsulate the native BOINC Manager in a secure environment that uses the Project Manager interface from the Boinc Manager - like a virtual machine. It's open source. So instead of one App u install another app including Password Manager and multi machine remote management like boinc task. Than the dev has nearly unlimited possibilities.
But: this is not the topic of this "issue"

 Comment by tomasbrod
Saturday Apr 15, 2017 at 21:16 GMT

#78 SRP authentication mechanism keeps your password secure even if you reuse it.

OAuth2 was proposed by several of the BOINC 2017 workshop attendees.


deps: use go1.17 and remove GO111MODULE

deps: update to use go1.17
remove GO111MODULE setting as go1.16 defaults to go mod build

signed the CLA.

signed the CLA.

can we merge this @genevieveluyt ?

@chenrui333 there are a couple other places that still reference 1.16 if you wouldn't mind updating those as well

[ ] https://github.com/Shopify/kubeaudit/blob/main/goreleaser.Dockerfile#L1
[ ] https://github.com/Shopify/kubeaudit/blob/main/Dockerfile#L1
[ ] https://github.com/Shopify/kubeaudit/blob/main/Makefile#L13
[ ] https://github.com/Shopify/kubeaudit/blob/main/README.md?plain=1#L50


bug(cdk-virtual-scroll-viewport): scrollWindow attribute not working in angular ssr (universal)

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
When using angular SSR the scrollWindow attribute is not working. I am getting the error:-
document is not defined
    at _CdkVirtualScrollableWindow (...\test_cdk_scroll_ssr\node_modules\@angular\cdk\fesm2022\scrolling.mjs:1659:26)
    at NodeInjectorFactory.CdkVirtualScrollableWindow_Factory (...\test_cdk_scroll_ssr\node_modules\@angular\cdk\fesm2022\scrolling.mjs:1667:14)
    at getNodeInjectable (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5984:44)
    at searchTokensOnInjector (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5911:16)
    at lookupTokenUsingNodeInjector (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5860:34)
    at getOrCreateInjectable (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5772:23)
    at ɵɵdirectiveInject (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:11050:19)
    at ɵɵinject (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:1106:42)
    at NodeInjectorFactory.factory (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:3325:29)
    at getNodeInjectable (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5984:44

Reproduction
StackBlitz link: SSR is not supported in stackblitz
Steps to reproduce:

create Angular SSR  app
install angular material
add cdk-virtual-scroll with scrollWindow attribute as shown in the official example: https://material.angular.io/cdk/scrolling/overview#cdk-virtual-scroll-window-scrolling

My Code
app.component.ts:-
import { ChangeDetectionStrategy, Component } from '@angular/core';
import {ScrollingModule} from '@angular/cdk/scrolling';

@Component({
  selector: 'app-root',
  styleUrl: './app.component.scss',
  templateUrl: './app.component.html',
  changeDetection: ChangeDetectionStrategy.OnPush,
  standalone: true,
  imports: [ScrollingModule]
})
export class AppComponent {
  items = Array.from({length: 100000}).map((_, i) => `Item #${i}`);
}

app.component.html

Content before

<cdk-virtual-scroll-viewport scrollWindow itemSize="50">

{{item}}

</cdk-virtual-scroll-viewport>

Content after

app.component.scss:-
.example-item {
    height: 50px;
  }

  .example-header,
  .example-footer {
    height: 100px;
    background: lightgray;
  }

Expected Behavior
It should work and use the window scroller just like it does in non ssr angular apps.
Actual Behavior
On adding the scrollWindow attribute giving the following error:-
document is not defined
    at _CdkVirtualScrollableWindow (...\test_cdk_scroll_ssr\node_modules\@angular\cdk\fesm2022\scrolling.mjs:1659:26)
    at NodeInjectorFactory.CdkVirtualScrollableWindow_Factory (...\test_cdk_scroll_ssr\node_modules\@angular\cdk\fesm2022\scrolling.mjs:1667:14)
    at getNodeInjectable (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5984:44)
    at searchTokensOnInjector (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5911:16)
    at lookupTokenUsingNodeInjector (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5860:34)
    at getOrCreateInjectable (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5772:23)
    at ɵɵdirectiveInject (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:11050:19)
    at ɵɵinject (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:1106:42)
    at NodeInjectorFactory.factory (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:3325:29)
    at getNodeInjectable (...\test_cdk_scroll_ssr\node_modules\@angular\core\fesm2022\core.mjs:5984:44

Maybe because window is not always available in ssr, or something like that but in the documentation, it is not mentioned that this will not work in Angular SSR.
Environment

Angular: 17.3.8
CDK/Material: ^17.3.10
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

I think it is expected behavior as it uses window object, which is only available when the app is running on browser. We can solve it by using ngIf with viewport and isPlatformBrowser to make sure the viewport only renders in browser.
this answer in stackoverflow helped me:-
https://stackoverflow.com/a/78527170/19566600


Build gen_snapshot with correct target_os on Windows

In Windows builds, we override target_os to win. For android-arm and android-arm64 builds of gen_snapshot we should be building with target_os="android". Previously this resulted in #17387.
We should eliminate this override and fix up the rest of the build to correctly support building gen_snapshot (and ideally the engine) on Windows; this would also make it easier for Windows developers to contribute to the engine.

Is this still the case? It looks like the linked code is no longer that way, and AFAICT we're building gen_snapshot on Windows now.

Closing this as it seems done, feel free to reopen if this isn't the case.


Get streak/point values from db

Use ActivityController to get streak and points values.
Use points() and  streak() methods.
And replace hard coded strings in profile sheet with it.
Make sure that points value change, on adding or deleting the activity

sir, please assign me this issue

sir, please assign me this issue


Custom TypeSerializer not used for serialization

I registered a custom type serializer to a gson config loader
so now deserializing is working correct but serializing is not even called when saving.
TypeSerializerCollection serializerCollection = TypeSerializers.newCollection();
serializerCollection.registerType(TypeToken.of(Color.class), new ColorSerializer());

this.configLoader = GsonConfigurationLoader.builder().setFile(file).setDefaultOptions(ConfigurationOptions.defaults().setSerializers(serializerCollection)).build();

this.configLoader.save(this.config);

Unfortunately this issue is quite old and doesn't have enough information to be able to reproduce the original issue. I'll be closing this issue for now but if you can provide a standalone, fully working example (or even better, a unit test) that reproduces this issue on the latest master I'll be happy to look into the issue again.


[Improvment] [Style] API Test spotless check failed.

Search before asking

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

What happened
found

What you expected to happen
CI pass
How to reproduce
N/A
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

This is not a bug. It is expected behavior. BTW, if those so-called 'blocked' PRs not rebased on latest dev branch, it still get blocked.

This is not a bug. It is expected behavior. BTW, if those so-called 'blocked' PRs not rebased on latest dev branch, it still get blocked.

get it. thx


Tax - Split scan - Selected rate from confirmation page does not show up in split detail view

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

Version Number: 1.4.76-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/40240
Action Performed:
Precondition:

Workspace is tax-enabled

Go to staging.new.expensify.com
Go to workspace chat
Go to + > Split expense > Scan
Upload a receipt
On confirmation page, click Tax rate
Select any rate
Click Split expense
Click on the split preview

Expected Result:
The selected tax rate will show up in Tax rate row in split details view
Actual Result:
The selected tax rate does not show up in Tax rate row in split details view. The row is empty
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/b133585f-2336-49b7-9782-db0ed700653e
View all open jobs on GitHub

@kadiealexander  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

the revert https://github.com/Expensify/App/pull/42670 has been deployed, please retest and confirm this is no longer reproducible.

Issue is not fixed.
https://github.com/Expensify/App/assets/43996225/d39549af-782f-4f01-9309-1c226d56d8f2

Proposal
Please re-state the problem that we are trying to solve in this issue.
The selected tax rate does not show up in Tax rate row in split details view. The row is empty
What is the root cause of that problem?
We have two bug here

When we start split bill here, we don't accept taxCode and taxAmount param in this function to create optimistic transaction and pass it to param of StartSplitBill API

https://github.com/Expensify/App/blob/2514f2988eb26858d30fa501c0040b1f8fc2e239/src/pages/iou/request/step/IOURequestStepConfirmation.tsx#L342

When we edit tax rate and tax amount, we're calling UpdateMoneyRequestTaxRate API because we don't have the case for edit split bill here

https://github.com/Expensify/App/blob/2514f2988eb26858d30fa501c0040b1f8fc2e239/src/pages/iou/request/step/IOURequestStepTaxRatePage.tsx#L76
What changes do you think we should make in order to solve the problem?

Accept taxCode, taxAmount param in startSplitBill function and then use this to create optimistic transaction here.

Pas taxCode as transactionTaxCode and taxAmount as transactionTaxAmount into startSplitBill function here

https://github.com/Expensify/App/blob/2514f2988eb26858d30fa501c0040b1f8fc2e239/src/pages/iou/request/step/IOURequestStepConfirmation.tsx#L342

Pass taxCode and taxAmount as param of StartSplitBill API here. We also need BE change to return taxCode and taxAmount of the transaction when we call StartSplitBill API

https://github.com/Expensify/App/blob/2514f2988eb26858d30fa501c0040b1f8fc2e239/src/libs/actions/IOU.ts#L4540

When we edit split bill tax rate, we should call setDraftSplitTransaction as the revert PR here https://github.com/Expensify/App/pull/42670/files#diff-2d4b9c38f6fb668f527da301a60d3c9bb05c865f33aa475d29148bbda45e3cc0

https://github.com/Expensify/App/blob/2514f2988eb26858d30fa501c0040b1f8fc2e239/src/pages/iou/request/step/IOURequestStepTaxRatePage.tsx#L76
What alternative solutions did you explore? (Optional)
NA

This should be hold https://github.com/Expensify/App/issues/39690
cc @MonilBhavsar

We can close this. This is new feature and not a bug


[Scheduled Maintenance] Domain transfer

Context:
hypeparty.cr4.live changes to hypeparty.net

Domain transfer done.
hypeparty.cr4.live is still available as a redirect to hypeparty.net including uri params.
Overlays should still work with old domain.


Add detailed helptext for the stop command

Add an entry into default-config.json under helpTopics. The key should probably be 'stop', and the description should describe how one can use the command and how it will behave from a user perspective.

Better instructions, since @ErikAlmaguer is extremely new to the project and to team development.
This task is part of the detailed-helptext project, which is currently occupying dev-master due to engineering constraints which pushed this out on the roadmap to overlap with v5 integration - so there will be some merging required.
Further references to code and line numbers will be done off the currently non-working dev-master, at commit b881be1f6635cb015fa8e18a67c1eb2a059d0f8e.
The first step will of course be to make a branch off dev-master, named something like detailed-helptext-stop (whatever you want really, it's just useful to have names that mean something related to the purpose of the branch). You'll work on this branch, and eventually submit a Pull Request (PR) to merge that branch into dev-master to render your changes an "official" part of CadenceBot mainline (eventually, once dev-master gets updated to match current master, the CadenceBot-dev instance you've seen will be moved over to run dev-master in accordance with my official software development process as documented in the README.
Then, you can use several resources to try and gain an understanding of the stop command (it's very simple, so I don't anticipate any trouble). Lines 2 through 40 of default-config.json include some configuration of the core commands, among which is stop. Based on that structure you could probably find where its behavior is defined in bot.js, but to short-circuit that for you the line which recognizes whether a message is triggering the command is 684. This task is partially intended as a "warm-up", so part of the objective here is for you to familiarize yourself with the project and how things work behind the curtain.
If you have any problems with that, feel free to reach out for help and I'll give you a hand, but I doubt you'll need it.
Once you feel you properly understand the behavior of the command, you can see between configuration on lines 53 and 94 of default-config.json of existing detailed helptext, and if you'd like (though it's a bit less direct) the implementation of the command that converts that configuration data into a message to be sent to the user in bot.js between lines 736 and 865 how detailed helptext is written. The deliverable for this issue is another block in default-config.json, like the ones for help or nowplaying, for stop.
Unfortunately, due to the aforementioned v5 situation, we don't really have a great way to run and validate this - Normally, I'd ask that you at least test your new stuff and that it doesn't break anything else before putting our a pull request, but circumstances given, just do your best and when you think it's done, put out a pull request and I'll give you feedback including any results of an improvised test I'll put together on my own.
Short version:

Make a new branch based on dev-master for your work.
Familiarize yourself with the stop command, by interacting with the bot on Discord and reading the relevant sections of default-config.json and bot.js
See how to write detailed helptexts primarily based on the relevant section of default-config.json and if necessary in bot.js
Write your own detailed helptext section for the stop command, and commit it onto your branch.
Open a PR for that branch here on GitHub.
We'll work together to get that PR to a state ready to merge, then get it into dev-master.

If you have any questions, or if you have any trouble with any of this, please feel free to reach out to me or to anyone else on the team.
And, of course, welcome aboard!


Multi-factor Authentication

Has anyone plugged in some sort of Multi-Factor capability in Identity Server. there all types of scenarios. But I was thinking about something like GMails style, where they text you a code that you must enter before you are authenticated.

Where is it now (2015)?

This was answered in 2013 that it will be implemented in the next version. Two years passed since..

It's supported in IdSvr3.


SMOTEN input_scitype and doc string

julia> input_scitype(Imbalance.MLJ.SMOTEN)
Tuple{Union{Table{<:AbstractVector{<:Finite}}, AbstractMatrix{Finite}}, AbstractVector}

The AbstractMatrix{Finite} should be Abstract{<:Finite}.
And the doc-string is contrary to this:
 Transform Inputs
  ≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡

    •  X: A matrix or table of floats where each row is an observation from the dataset

    •  y: An abstract vector of labels (e.g., strings) that correspond to the
       observations in X

I understand why <:Finite (was a little sleepy last time I touched this, sorry) but why Abstract instead of AbstractMatrix? As for the docstring, yes, it should have agreed with the spec in Imbalance.jl docs, let me fix this.

my bad. i meant AbstractMatrix{<:Finite}} 😳


Adds UUID support to NDT7

Makes ndt7 save its data in a well-formed directory structure.
Also adds UUIDs to the filename and to the metadata.

This change is

Please take another look.
Also, avoiding the phenomenon of "actually, I no longer understand why this integration test used to pass" is a major motivation behind the development of unit tests to cover "already tested" code.  Integration tests can frequently pass "by accident" when multiple bugs happen to work together just right.

Travis also doesn't support socket cookies (a kernel feature from 2017). So creating files full of data is largely a non-starter because they can't be named because a UUID can't be generated.

Pull Request Test Coverage Report for Build 371

20 of 39 (51.28%)  changed or added relevant lines in 5 files are covered.
138 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-15.05%) to 57.548%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fdcache/fdcache.go
8
10
80.0%

ndt7/server/results/metadata.go
0
4
0.0%

ndt7/server/results/file.go
6
19
31.58%

Files with Coverage Reduction
New Missed Lines
%

tcpinfox/tcpinfox.go
3
0.0%

ndt7/server/results/metadata.go
6
0.0%

fdcache/fdcache.go
8
78.33%

bbr/bbr.go
11
0.0%

tcpinfox/tcpinfox_linux.go
17
0.0%

bbr/bbr_linux.go
19
0.0%

ndt7/server/results/file.go
29
10.77%

ndt7/server/download/download.go
45
41.78%

Totals

Change from base Build 353:
-15.05%

Covered Lines:
568

Relevant Lines:
987

💛  - Coveralls

Updated comments in response to your comments. :)

@pboothe :shipit:


update mantine dependency, rework settings UI

Settings will look like this:

Tested, looks good.
If you can, you could try to fix e2e tests as we have them for settings page. If not, I'll do this later.
@derneuere @DigitalWarrior

I had something similar in mind. But with tabs or some other form of grouping. I dislike the fact that there's no space at the bottom. Transcode switch seems abandoned. Maybe adding some space at the bottom would help?

Agree. I also toyed with the idea of tabs, but seems like we'de be hiding the options again. Just not on the selects.
Good idea with the spacing on the bottom. I'll tweak it and push the changes.

That looks nice. I'll update e2e tests after the merge. Or, if you feel brave, you could give it a go. :)

Thanks @sickelap for the help with the tests :)


Fix shared_ptr assignment operator

The copy assignment operator doesn't compile because _p = p
should be _p = p._p. It's also not safe for self-assignment,
because it resets the pointer before copying it.
This increments the ref count first, so that reset()
won't delete the object if this is the only owner.

Feel free to close this if you really don't want people using the project :slightly_smiling_face:

Until people really do stop using the project, ignoring bugs is just plain mean.


Simplify e2e tests

Summary by CodeRabbit

New Features

Enhanced end-to-end testing capabilities with new command-line options for more granular control over testing configurations.
Improved handling of user accounts and deployment configurations for Ethereum-based applications.

Bug Fixes

Streamlined user account management to reduce potential errors related to address mismatches in transactions.

Refactor

Restructured private key management for clarity and security.
Updated configuration management to enhance modularity and reduce dependencies.

Tests

Adjusted user account handling in tests for improved reliability and correctness.

There are probably even more ways we can simplify the e2e tests, but this was the low-hanging fruit.


hls in the browser is buffering too much

Which version are you using?
v0.16.0
Which operating system are you using?

OS

[x] Linux
[ ] Windows
[ ] macOS

Architecture

[x] amd64
[ ] arm64
[ ] arm7
[ ] arm6

Describe the issue
I am running the simple pipeline from the gstreamer to the rtsp server and then read the hls stream in the browser. Problem is, that the results is buffering to much and it is not much usable.
Where could I have the problem?
Description
Describe how to replicate the issue

start the server with the default yml settings
publish with

gst-launch-1.0 uridecodebin uri=file://any_h264_video.mp4 ! nvvideoconvert ! nvv4l2h264enc ! rtspclientsink location=rtsp://localhost:8554/stream

read with browser at the URL localhost:8888/stream

it works ok on jetson xavier, closing


disqusShortname parameter ignored in config.toml

Hi!
I can't get disqusShortname parameter working in config.toml.
I tried to put it with the main parameters and under [params], but it is ignored.
This is the page source
(function() {
    if (window.location.hostname == "localhost")
        return;

    var dsq = document.createElement('script'); dsq.type = 'text/javascript'; dsq.async = true;
    var disqus_shortname = '';
    dsq.src = '//' + disqus_shortname + '.disqus.com/embed.js';
    (document.getElementsByTagName('head')[0] || document.getElementsByTagName('body')[0]).appendChild(dsq);
})();

However, if I put the shortname in the template directly, everything works.
Hugo version
$ hugo version
Hugo Static Site Generator v0.15-DEV BuildDate: 2015-05-30T17:12:00+02:00

obtained with go get.

@brunetto Custom parameters like your disqusShortname will not work as "main parameter". Storing them under Params would be the right way.
Furthermore you need to access your custom variable with var disqus_shortname = '{{ .Site.Params.disqusShortname }}';. The curly braces will allow you to insert custom logic or like in your case parameters into your template.
I did it that way in one of my own themes (which is a work in progress) but you could take a look at the source code

Is this still an issue?


[Hub Generated] Publish private branch 'Vishakha-Git-resources-Microsoft.Authorization-2023-03-01-preview'

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I have reviewed the general guidance on the spec PR review process: https://aka.ms/specprreview.
[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data-plane related specifications.
[x] I commit to follow the Breaking Change Policy.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can request review from an ARM API Review board.

ARM API changes review

If you want for the ARM team to review this PR, you must add the ARMReview label.
The automation may automatically add the ARMReview label, if appropriate.
If this happens, proceed according to guidance given in GitHub comments also added by the automation.

Breaking change review
If you have any breaking changes as defined in the Breaking Change Policy,
follow the process outlined in the High-level Breaking Change Process doc.
Getting help

For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For additional help, see https://aka.ms/azsdk/support/spectools.

The PR is created based on the updates in the private branch. The updates in the PR has already been reviewed and approved with this PR Azure/azure-rest-api-specs-pr/12088

@rkmanda @bdefoy Can we make "The new version has a different type 'object' than the previous one" which gets raised when someone adds the missing 'type:object' that other linter rules complain about, NOT a linter warning? It confuses everyone greatly, and I don't understand the benefit? [ARMShouldGuidelineViolation]

💬 1026 - TypeChanged
The new version has a different type 'object' than the previous one ''.New: Microsoft.Authorization/preview/2023-03-01-preview/privateLinks.json#L79:13Old: Microsoft.Authorization/stable/2020-05-01/privateLinks.json#L79:13

💬 [1026 - TypeChanged](https://github.com/Azure/openapi-diff/tree/master/docs/rules/1026.md)	The new version has a different type 'object' than the previous one ''.
New: [Microsoft.Authorization/preview/2023-03-01-preview/privateLinks.json#L79:13](https://github.com/Azure/azure-rest-api-specs/blob/93a6d461001dbf4801718cc175e2f6b2513f0924/specification/resources/resource-manager/Microsoft.Authorization/preview/2023-03-01-preview/privateLinks.json#L79:13)
Old: [Microsoft.Authorization/stable/2020-05-01/privateLinks.json#L79:13](https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Authorization/stable/2020-05-01/privateLinks.json#L79:13)

Please check and fix the Avocado error.

Please check and fix the Avocado error.

Can not fix avacado error

❌ MULTIPLE_DEFAULT_TAGS
The readme file has more than one default tag.readme: specification/resources/resource-manager/readme.md

❌ [MULTIPLE_DEFAULT_TAGS](https://github.com/Azure/avocado/blob/master/README.md#MULTIPLE_DEFAULT_TAGS)	The readme file has more than one default tag.
readme: [specification/resources/resource-manager/readme.md](https://github.com/Azure/azure-rest-api-specs/blob/1f498289072cb1a616996452fd8ee5c6603bdc35/specification/resources/resource-manager/readme.md) Since the resource-manager spec is shared by multiple services and hence the error of multiple tags. Can not delete any existing multi tags.

/azp run


Rank and Getshout doesn't work while following https://github.com/sentanos/roblox-js/wiki

Do you use the cookie for authentication, like mentioned in this repo's README?

Working on a fix for both very soon. Thank you for reporting this.

getShout.js and onShout.js have been fixed. Rank's are next.

Thanks. Have an eta for it? Also, can the number be a string? Like: rank(groupid,“3”)?

It could technically be a string but that implies that's the player's name.
It's an integer for a reason because it's a user's ID.
On Mon, Feb 11, 2019 at 01:27 sasial-roblox<EMAIL_ADDRESS>wrote:

Thanks. Have an eta for it? Also, can the number be a string? Like:
rank(groupid,“3”)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/suufi/noblox.js/issues/38#issuecomment-462229195, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMKxJAcqhUJW1du1XcLR1wxvRrqrG3jaks5vMQ07gaJpZM4aI9KQ
.

Sorry, I meant can the rank number be a string? The player would be an interger. It would be like:
rank(groupid,playerid.“2”)

@LapinoLapidus Yes.

Seem to be working now,
@ipadpuppydogdude1
The fix is to add this line of code:
    var Username = args[1]
    var strRank = args[2]
    var Rank = Number(strRank)

Then, use Rank in the command


Add sub section list to list pages

Hello, I'd like to organize my blog something like this

travel / china / xxx
travel / france / xxx
projects / java / xxx
projects / scala / xxx

So I would like to have multiple level of sections, where each index.md can display its subsections.
Is it possible to add this feature? Thanks

Hi @peter-jerry-ye Hugo respects your folder section out of the box, so I don't believe you need the theme to do anything special for you here. You might get more help in the Hugo forums on non-theme questions. I'll close, but feel free to open if you have more information.

Hi @budparr ,
I do understand that Hugo handles well the folder section. What I don't understand is how we can get access to subsections from its parent. I mean, I could put a menu on the whole, but what if I want to present the subsections on the page of parent section? For example, in the section 'Travel' (which is not home page), I'd like to present basic information for each country I've visited, and then each country is a section that contains many posts.
I've looked around and it seems that no theme has ever done this, so I'm not sure if it was I that had a bad design or misunderstood something. Could you help me? I can modify the list.html to achieve this easily, but I just wonder why no one has ever done this before.
Thanks.

@regisphilibert Might have ideas, but otherwise the forum is where you're likely to find a variety of opinions.


Behavior difference of lime test html5 between Windows and Linux

The behavior of lime test html5 differs between Windows and Linux. On Windows, the web browser automatically opens when the command is run. On Linux nothing happens but http://<IP_ADDRESS>:3000/ can still be opened.
Tested under Windows 10 and Ubuntu Mate 18.04 with this project and Firefox as the default web browser.
#955 is probably related.

Can you try xdg-open path/to/index.html?
This should be set in your distribution to use the default browser.

I tried it again and found that it launches just fine if I use lime test html5. What doesn't work is sudo lime test html5 which I previously used because it is required if I choose Linux as the build target.

The application that opens HTML files is Visual Studio Code for my user account and the root account.

Closing because it seems you figured this out.


B2C User flow API Connector: Unable to set extension attributes in token even after returning them in response

Hi,
Per the link in the document here it is possible to return extension attributes to be part of claims. But to my surprise, I don't see them in the token at all.
I return the key-value pair with key that matches exactly the one existing for other users.
For all federated users for the very first time, they will not have values in any custom attributes, we have a scenario to set them in API Connector to generate a token by B2C.
My sample response would look like below:
where {{extension_app_id}} is the extensions application id
{
    "version": "1.0.0",
    "action": "Continue",
    "userMessage": "Success",
    "given_name": "Navi", // To see at least this value is overwritten in the claim
    "givenName": "Navi", // To see at least this value is overwritten in the claim
    "extension_{{extension_app_id}}_scopes": "1 2 3",
    "extension_{{extension_app_id}}_userid": "fc9f678d-f369-464d-a099-5c3432f2e484"
}

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

ID: 8e32e9a4-3b2f-dafb-605e-d87350a7d58d
Version Independent ID: 872b2c4c-7762-4ef8-29c5-29011da165a2
Content: Add API connectors to user flows (preview)
Content Source: articles/active-directory-b2c/add-api-connector.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@nkumars Thank you for reaching out.
We are actively investigating and will get back to you soon.

@nkumars If a custom attribute doesn't have a value, it will not be added to the token as a claim. Try adding a default value to these claims before passing it to the API Connector.

@amanmcse, it seems the documentation is incomplete. I had achieved returning custom claims and this is possible only in the first trigger After signing in with an identity provider and not in Before creating the user. I tried all possible cases and found only this works and not any other combinations.

@nkumars If a custom attribute doesn't have a value, it will not be added to the token as a claim. Try adding a default value to these claims before passing it to the API Connector.

Is there a good way to add a default value without the user seeing an entry box?

@samiozturk, it just works when we return claim in After signing in with an identity provider and if the same is returned in  Before creating the user, doesn't return those claims in token
Ofcourse, I can return default claim and I also know if the claim has no value doesn't return in token.


fixes stack too deep error when compiling with the optimizer off

This PR fixes stack too deep error when compiling with the optimizer off. This is important to be able to run forge coverage and generate test coverage reports.

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

Project coverage is 61.89%. Comparing base (54e90a2) to head (c38cc21).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #94   +/-   ##
=======================================
  Coverage   61.89%   61.89%
=======================================
  Files          27       27
  Lines        4091     4091
=======================================
  Hits         2532     2532
  Misses       1427     1427
  Partials      132      132

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Update composer.json

Update the mono log used to take advantage of several new versions. Basic test locally and all works fine. Only a couple of breaking BC changes in the change logs but nothing I can see would apply except maybe 1.8.0.
https://github.com/Seldaek/monolog/blob/master/CHANGELOG.mdown

The composer.lock file should also be updated in this PR.

@elazar
So it should.
Also found that I need to update https://github.com/phergie/phergie-irc-client-react first - otherwise we get stuck - I will need to check other plugins as well!

@elazar
I am going to close this pull request, because I don't think I can do it.
Need to first patch the irc-client-react and then this one. I can't submit a full patch here until that later one is completed but then people wont be able to install since the two packages will have different versions until you accept a patch for this one which is complete.
If you could do it would be great, for now in my project I am doing this in the main composer.json file which satisfies both laravel and phergie.
"monolog/monolog": "1.12.0 as 1.6.0"


GRPC over dev tunnels

Is it possible that this error is connected to dev tunnels ? Is grpc supported over tunnels ?
Error: 12 UNIMPLEMENTED: Received HTTP status code 404
    at callErrorFromStatus (/usr/src/app/node_modules/@grpc/grpc-js/build/src/call.js:31:19)

This is from grpc package where the error is thrown
/**
 * Construct a ServiceError from a StatusObject. This function exists primarily
 * as an attempt to make the error stack trace clearly communicate that the
 * error is not necessarily a problem in gRPC itself.
 * @param status
 */
function callErrorFromStatus(status, callerStack) {
    const message = `${status.code} ${constants_1.Status[status.code]}: ${status.details}`;
    const error = new Error(message);
    const stack = `${error.stack}\nfor call at\n${callerStack}`;
    return Object.assign(new Error(message), status, { stack });
}

It might be duplicate of #263 or at least connected

Closing this in favor of https://github.com/microsoft/dev-tunnels/issues/263 as they appear to be the same underlying issue.


Broken CSS selectors

Hi, This script appears to no longer work. I was able to fix the login selectors by using these ids
self.webdriver.find_element_by_id("username").send_keys(self.email)
self.webdriver.find_element_by_id("password").send_keys(self.password)
self.webdriver.find_element_by_css_selector(".login__form_action_container button").submit()

but I am stuck on this one since I'm not sure which DOM element this selector corresponds to.
Message: no such element: Unable to locate element: {"method":"class name","selector":"pv-top-card-section__name"}

Saving the user_id also appears broken but I just commented out the part where it searches for the voyager body response.

I'm getting this error and I'm still not sure how to fix it. Traceback is saying there's something wrong in linkedin.py line 97

Hi @lifeHacker42 the CSS selectors had changed again - do a git pull and it should work

Getting a similar error:
Message: no such element: Unable to locate element: {"method":"css selector","selector":"[id="username"]"}
Do you think you can help with it?


Deploy without github

Hi! Is it possible to use nuts without github (use private git or mercurial repo on the same server where nuts was deployed)?

@d9k nuts has backends, there's only one implemented by default which is the github backend.
Basically backends provide a means to get all the current versions, optionally a changelog and read build assets per version.
So in theory you can implement whatever logic you want through a driver. So yes, that would allow you to have a local (to nuts) git repo that holds all your build assets and has tags for each version.
It's possible, but it's not what we recommend/support, GitHub integration is a breeze since it works as expected out of the box and should work for pretty much any standard GitHub project.

@d9k I had a similar use case and initial modified nuts but soon realized that it would be better to develop the app from scratch. Perhaps it'd be of use to you: https://github.com/ArekSredzki/electron-release-server

@ArekSredzki, interesting, thanks!)


compile_time attribute in chef_gem resource breaks on chef < 12.1

the changes in 2.6.1 from [https://github.com/opscode-cookbooks/aws/commit/2fc04a4f5e45494da6e27a5246789d8b17797cf2] Intended to suppress the warnings from the newer compile_time resource on chef_gem break the cookbook if you are not using chef 12.1 (which isn't out yet anyway).
This fixes https://github.com/opscode-cookbooks/aws/pull/106
pinging @someara who has made a few of these changes recently ;)

Sorry about that. Fixed in the latest patch release

NP, thanks for the quick fix. :thumbsup:

@someara it's not fixed for us as the resource does not see it.
Also related to https://github.com/opscode-cookbooks/xml/issues/22


[C2M] Crash when calling a deference to a function double pointer

The following test case makes C2M crash:
#include <assert.h>
static int f() { return 1; }
typedef int (*F)();
static F pf = &f;
static F* ppf = &pf;
int main(int argc, char** argv) {
  assert((*ppf)() == 1);
}

Output:
c2m test2.c -eg
test2.c:5:18: warning -- incompatible types in assignment to a pointer
Segmentation fault (core dumped)

By the way, all the recent issues I've created I have found on occasion while attempting to make the language I am working on to pass all tests with C2M. So I am not trying to break C2M, I'm just trying to make it work with my C code generator.

The following was committed for the issue:
https://github.com/vnmakarov/mir/commit/016dc2546d4b61bf2a1189d2891d7e6c275f3663
https://github.com/vnmakarov/mir/commit/90a6c70b22207ad870fc3dce95bd662bb3e60313

The fix works for me, thanks!


Create CD pipeline

This is essentially a sub-task of a larger "Create CI/CD Pipeline" user story.
The focus of this user story is to create the portion of the CI/CD pipeline that executes after CI is complete. The following are requirements of the pipeline when a PR is submitted:

[ ] PowerShell module artifacts are published to the Microsoft PowerShell Gallery.

Does this need to be gated, or can it be automated after successful CI?


Add support for query parameter serialization styles

While implementing filtering API recently and I came across a problem with parsing list of comma-separated ids from query string:
Example:
GET /api/items?id=1,3,5

I know this type of parsing can be easily done by declaring id parameter as str and parsing it inside the function but then we're missing awesome OpenAPI doc features.
Please find the PR https://github.com/tiangolo/fastapi/pull/2078 with proposed solution.
OpenAPI Docs: https://swagger.io/docs/specification/serialization/
Comments regarding code changes:

all changes should be backward compatible
there's no support for style=deepObject as it would require more changes (because query params cannot be non-scalar values at the moment)
in case multiple values are present (e.g. ?id=1&id=2,3) and explode=False, only last value is parsed (in this case 2,3); it's consistent with the currently implemented behaviour in case of declaring parameter as non-sequence (e.g. str) and receiving more than one value in query string

Any movement on this?

@codyflynnNC OpenAPI specification allows this as Jakub mentioned, and he already opened a PR for this, so for now we are just waiting for @tiangolo's feedback.

This would be a great addition. There is obviously no singular way to do things, especially when you want to support filtering like GET /api/items?id=gt:1,lt:3. But it would be nice to have a way to do this.

This would be a great addition. There is obviously no singular way to do things, especially when you want to support filtering like GET /api/items?id=gt:1,lt:3. But it would be nice to have a way to do this.

It seems that the reticence on allowing for encoding styles for deepObject might be that there is no standard way of encoding it?
How about allowing for pluggable parameter encoding styles, so we can implement whatever is needed. As it stands, those of us trying to implement endpoints for populating tables that support filtering find it impossible to create a nice documented API that allows for lt, gt and so on comparison operators.


Show structure of deleted content

Right now if for example I delete a bold element I see in the PDF something like this:
       <b>text</b>

with a strike out.
Ideally the PDF output would show the structure and styling for the deleted content.
This was also reported here:
https://www.oxygenxml.com/forum/viewtopic.php?f=6&t=13282&p=41718#p41718

The problem is that if we take the deleted content from the Oxygen processing instruction, parse it and create a node set from it, the nodeset will not contain @class attribute values and we will not be able to apply templates over it.


Onboarding security analytics dashboards plugin. (#380)

Forwardport of 165f8559cca46f141fe9460463b6793ca4b14ad5 from https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/380

Even though the 3.0 plugin version isn't available yet. It's making branching strategy difficult that it's not work was only done on 2.5

I think I should lint checker fail because I need to forwardport another commit that handles that file and the conflicts might be plenty.

Closing as duplicate.


Unable to populate with initial values

I have a requirement that the react-multi-email field show some existing email addresses, but I am unable to populate the field with initial values.  My container loads the component and shows the field without any javascript errors, and the react-multi-email field shows, but it is empty.  Below is the code for the component.  I am setting the "emails" attribute to an array with a single value of<EMAIL_ADDRESS>import PropTypes from "prop-types";
import React from "react";
import ReactModal from "react-modal";
import { ReactMultiEmail } from 'react-multi-email';

const TestMultiEmailModal = ({ showModal }) => {
    return (
        <ReactModal contentLabel="Submit Internally" isOpen={showModal} className="react-modal" overlayClassName="react-modal-overlay">
            <ReactMultiEmail
                style={{}}
<EMAIL_ADDRESS>                onChange={_emails => {
                    this.setState({ emails: _emails });
                }}
                getLabel={(
                    email,
                    index,
                    removeEmail,
                ) => {
                    return (
                        <Label key={index}>
                            {email}
                            <Icon name="delete" onClick={() => removeEmail(index)} />
                        </Label>
                    );
                }}
            />
        </ReactModal>
    );
};

TestMultiEmailModal.propTypes = {
    showModal: PropTypes.bool
};

export default TestMultiEmailModal;

The reason why it does not work is that the value of ReactMultiEmail.props.email is not managed by state.
It is recommended to manage the value of emails as state as the following code.
Have a happy coding.
import PropTypes from "prop-types";
import React from "react";
import ReactModal from "react-modal";
import { ReactMultiEmail } from 'react-multi-email';

class TestMultiEmailModal extends React.Component {

  state = {
    emails<EMAIL_ADDRESS>  }

  render(){
    const {showModal} = this.props;
    const {emails} = this.state;

    return (
      <ReactModal contentLabel="Submit Internally" isOpen={showModal} className="react-modal" overlayClassName="react-modal-overlay">
          <ReactMultiEmail
              style={{}}
              emails={emails}
              onChange={_emails => {
                  this.setState({ emails: _emails });
              }}
              getLabel={(
                  email,
                  index,
                  removeEmail,
              ) => {
                  return (
                      <Label key={index}>
                          {email}
                          <Icon name="delete" onClick={() => removeEmail(index)} />
                      </Label>
                  );
              }}
          />
      </ReactModal>
    );
  }

}

TestMultiEmailModal.propTypes = {
    showModal: PropTypes.bool
};

export default TestMultiEmailModal;


Update README.md 📝

Fixed few grammatical errors in the README.md file

anybody here??

Hi @tusharxoxoxo, thanks for those corrections!
We'll need you to sign the DCO before we can merge your commits though. When you click on the Details link of the DCO check, it'll show you what you need to do.
Also, some of the parts you changed in README.md are output of the CLI. Would you mind applying your changes to those texts as well?  See here for example: https://github.com/CycloneDX/cyclonedx-gomod/blob/2fe0a1da390fbc17df326f35139a5a4d9e1ffe65/internal/cli/cli.go#L35-L63

is it correct ??


Cursor for disabled elements

It would be cool if disabled elements (checkboxes, selects, radios and buttons) had the cursor set to not-allowed to make it clear that the element is disabled.
Otherwise great work!

I think we will leave this up to developers if they want to add it. We don't feel it is necessary.


[WIP] PIP 35: Improve topic lookup for topics that have high number of partitions

[WIP]  PIP 35: Improve topic lookup for topics that have high number of partitions
Pending UT and change in Proxy classes.
Add batch lookup ability to LookupService.
Accept list of topic names.
Whenever possible, return partial succeed result instead of failing whole batch lookup result.
For all topics pass in in request, first decided which NDBundle they belong to, then do lookup on each bundle.
Response will have map of broker address to list of topics that broker serve.
Failed lookup will be in separate list.
(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

This is WIP, still have some changes but most of the flow is added.
Want to get some initial comments before adding UT and making change in Proxy classes.

@rdhabalia @merlimat since you are familiar with the lookup logic, can you please take a look at this proposal, to make sure @MarvinCai is in a right redirection to continue?

@MarvinCai Thanks for the work. overall looks good. looking forward to see the final pr.

Closed as stale and conflict.


Add DeepL as translation service

Hi 👋
Nice package!
I figured it would be nice to also include DeepL as a translation service. This will give an opportunity to the privacy focused people an alternative to OpenAI and Google Translate.
I’m marking the PR as a draft since I want to test the integration in a real Laravel app before I mark the PR as done.

This is great! Let me know how the testing goes, and i will merge it.

@Bottelet I did a test locally on a personal project, works like a charm with DeepL 👌

This is great I appreciate the contribution! :raised_hands:


Is this invalid css?

Following is the CSS and while saving, why its giving me error:

And even with the SASS files too.
As soon as I hit Cmd+S, texmate tries to load the executable and after few secs, displays this error though its a valid SASS.

How can I fix this?
Or How can I tell VOS to not to fire up the checking command for certain files like SASS, CSS, etc ?

The css error is that you're validating css3 against the 2.1 spec. You can change what you validate against. See the last item in the configuration settings under "VOS_CSS_PROFILE"
As far as the Sass validation error, it's an issue with RVM that I haven't had a chance to address yet.

Thanks, setting VOS_CSS_PROFILE = css3 did the CSS trick.
But still struggling with the sass files.
I went through the README and with some new tweaks made in the SHELL vars, now its fine.
Thanks again.


Add commitizen

Add commitizen to the repo to make it easier for people to write their commit messages in the correct format.

[x] Add commitizen.
[x] Update the README to add the commitizen badge.
[x] Update the contribution guide to explain that people can use commitizen.
[x] Relax the commitlint rule that forces an issue to be referenced - I mainly added it so I wouldn't forget to do it, but that should be less of an issue once commitizen is added. This allows us to do certain tasks like refactoring without requiring a specific issue.

I've added commitizen, and tried to configure it to match what we need for Thrift.Net.


Typo under Wi-Fi antennas section

On the MT3620 development board user guide page - last sentence of 2nd paragraph under "Wi-Fi antennas" section

The images on the second row show the re-oriented capacitor positions in green.

There ain't no green in the 2nd image. :)

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

ID: 877d4170-58b8-f3a6-49f0-45b7f514b7e9
Version Independent ID: 18c7d3f7-864d-da98-447c-5aac535c8a99
Content: MT3620 development board user guide - Azure Sphere
Content Source: AzureSphere/articles/hardware/mt3620-user-guide.md
Product: azure-sphere
GitHub Login: @esalc
Microsoft Alias: azurespheredocs

@jlk Thank you for your feedback! We will review and update as appropriate.

Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thank you for your feedback! We've fixed this in the docs that will be published later this week.


The version lookup code for executors was broken. This commit fixes it.

I wasn't able to fully appreciate the changes, but don't see anything obviously wrong at least

I wasn't either, until I ran into the previous issue with having the same path twice in sys.path and the code was trying to compare the "new" version with the "existing" version.
You can test this by disabling the duplicate sys.path entry removal in init.py, then adding the src dir twice to PYTHONPATH and finally attempting to get an instance of the local executor.


tap streams dont work when a dimension is a datetime

%%opts Curve [tools=['tap','hover']]
a=hv.Curve((list(pd.date_range('2017-08-01', '2017-08-31')), range(31)))
b=hv.Curve((range(60,91), range(31)))
tap_a=streams.SingleTap(source=a)
tap_b=streams.SingleTap(source=b)
a
b

tapping on plots a,b ; tap_a always returns None
where as tap_b returns a value of x and y.
not only hv.Curve, it is also evident when using hv.Points.
if it is a limitation atleast should be warned?

Sounds like a limitation that can and should be addressed...

I believe this is now addressed by the PR above which has now been merged.


feat: add OTEL instrumentation

trying it again, see if anything has improved since the last time I tried

@karrui - what shall we do with this going forward? will we be using OTEL integration with Datadog, both directly, and via Vercel?

@karrui - what shall we do with this going forward? will we be using OTEL integration with Datadog, both directly, and via Vercel?

OTEL didn't seem to work with the default datadog integration (via Vercel)

Killing this branch, integration still doesn't support OTEL. @timotheeg was going to ping DD rep to see if they are going to  improve the integration.


help icon failure

It doesn't seem like the help system currently functions.

I didn't manage to recreate the error, everything works fine for me in both dev and production modes.
Could you please send me the full address of the index file that the app is trying to access?

This issue was occurring when the web server did not have permission to serve the markdown file.


[Blades in the Dark] Minor tweak to blades.css to prevent invisible text during dark mode

Changes / Comments
Added a different value for the --dark-secondarytext in the label styling; the background of some labels on this sheet is the same color as the initial value of --dark-secondarytext so it's making the text invisible. It'll be a much more significant undertaking to modify the sheet to fully take advantage of dark mode, but this should at least allow things to function as they should without full toggle support.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Add "Copy" and "Share" Buttons for Generated Answer

Feature Request: Add "Copy" and "Share" Buttons
Summary
Enhance user experience by adding:

A "Copy" button next to the generated answer to allow users to easily copy the content with one click.
A "Share button" to let users share the generated answer across different platforms, such as social media, email, or messaging apps.

Proposed Functionality

Copy Button: Clicking this button should copy the generated answer to the clipboard.
Share Button: Clicking this button should open a dialog or sharing options that allow users to share the answer on selected platforms.

Additional Context
This functionality aims to improve usability and increase the accessibility of Helpmate AI's answers by simplifying how users interact with the generated content.

hello @codeaashu I would like to work on this feature. Please assign me the task.


Consolidate CODEOWNERS

This just removed @tdstein from the CODEOWNERS.
This fixes the issue where changes in /internal were only auto-requesting reviews from Taylor.

🫡


Licensing issue

Hi there.
I used this datepicker in an application, and after an audit, the auditors informed me that there might be a possible infringement on the LGPL type license on the library found here: https://gist.github.com/xaprb/8492729
Apparently (according to the auditors anyway) you are not allowed to embed the content from date-functions.js into your own library (according to the LGPL license).
I am not 100% sure if I agree with all this, but maybe it would be worthwhile checking into this.
Thank you for your time and effort!
EDIT: The exact words from the auditors:
"The main issue with the LGPL is that the object code form of the licensed component can only be linked to, not embedded inside other code files. In the case of jquery.datetimepicker.js's use of the LGPL licensed date-functions.js, it contravenes the license condition because the date-functions.js code is embedded within the jquery.datetimepicker.js code."

At the moment, your work use same license, it's not a problem.


Installing modpack from FTB fails on KeyError: 'url'

Trying to install https://feed-the-beast.com/modpacks/96-ftb-plexiglass-mountain
(base) zajo@zor-awakened ~ % picomc mod pack ftb install 96
/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/requests/__init__.py:89: RequestsDependencyWarning: urllib3 (1.26.12) or chardet (3.0.4) doesn't match a supported version!
  warnings.warn("urllib3 ({}) or chardet ({}) doesn't match a supported "
INFO Installing FTB Plexiglass Mountain 1.2.2 as FTB_Plexiglass_Mountain-1.2.2
INFO Fetching Forge metadata
INFO Resolving version
INFO Installing Forge 1.18.2-40.1.84 as 1.18.2-forge-40.1.84
WARNING As the Forge project is kept alive mostly thanks to ads on their downloads
WARNING site, please consider supporting them at https://www.patreon.com/LexManos/
WARNING or by visiting their website and looking at some ads.
INFO Downloading installer
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:01<00:00,  1.78s/it]
INFO Installing with PicoForgeWrapper
INFO Done installing Forge
WARNING Skipping unsupported target java
Traceback (most recent call last):
  File "/opt/homebrew/Caskroom/miniconda/base/bin/picomc", line 8, in <module>
    sys.exit(main())
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/picomc/__init__.py", line 18, in main
    picomc_cli()
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  [Previous line repeated 1 more time]
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/decorators.py", line 84, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/picomc/cli/utils.py", line 16, in wrapper
    fn(x, *a, **kwa)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/decorators.py", line 84, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/picomc/mod/ftb.py", line 155, in install_cli
    install(pack_id, version, launcher, im, name, use_beta=beta)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/picomc/mod/ftb.py", line 122, in install
    dq.add(f["url"], filepath, f["size"])
KeyError: 'url'

MacOS, M1
click                   8.1.3
picomc                  0.4.5

I suppose the urllib warning at the top isn't critical but also a bit annoying as it's present with every picomc command.
Let me know what else I can provide :) !

The urllib warning is probably due to miniconda shipping weird versions of packages. I'd recommend using a vanilla python distribution (brew install python3), potentially even with something like pipx.
I'll have a look at the API issue soon

So basically FTB stopped providing download URLs for some files and instead provides curseforge project and file IDs. The curseforge API has been replaced in the meantime as well, so that needs to be fixed first. At least it seems to have documentation this time. https://docs.curseforge.com/#getting-started


CommonsChunkPlugin manifest load error

this is my webpack config file
entry: {
    'main': [
      'webpack-hot-middleware/client?path=http://' + host + ':' + port + '/__webpack_hmr',
      './src/client.js'
    ],
    vendor: [
      'react',
      ...
    ]
  },
....
//about plugin:
new webpack.optimize.CommonsChunkPlugin({
  names: ['vendor', 'manifest']
}),
...
webpackIsomorphicToolsPlugin.development()

then i can get right file :
manifest-455455b9862b20aae523.js
main-455455b9862b20aae523.js
vendor-455455b9862b20aae523.js
but when i change some code, hash will change in webpack-assets.json file always.
and browser output some erro like this:
Uncaught TypeError: Cannot read property 'call' of undefined
  at __webpack_require__ (manifest-455455b….js:586)

I am not sure if there some problem with tools.

but when i change some code, hash will change in webpack-assets.json file always.

That's because you're using
output:
  filename: [name]-[hash].js
  chunkFilename: [name]-[hash].js

Every time any file is updated (js) the [hash] changes and therefore all filenames change

As for the browser error, there's not enough info on the error you provided.


Remove default terminal item from SUI

Summary of the Pull Request
Looks like we forgot to comment out the Default terminal setting from the UI.
Commented it out here.

References
#9463

PR Checklist

[ ] Closes #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

You gotta run runxamlformat now to format XAML files 😉

SOMEONE promised me that x:Load="False" was good enough here. :|

Wait it totally should have been. Why isn't it?


favour https over ssh for submodules

This breaks use of cf-redis-release for non-Pivots everywhere, which is more of a concern now the repo is OSS, rather than being private and proprietary as was the case when my dear friends Tiago and Will opted for SSH.

Additionally, you can use following Concourse task to prevent this from happening
https://github.com/cloudfoundry/infrastructure-ci/tree/master/scripts/common/check-git-submodules

+1 for this, encountering same issue when trying to create cf-redis-release in ci

Closing as this was done in https://github.com/pivotal-cf/cf-redis-smoke-tests/commit/69fc83bd25a24b60e0273b739aa2e6d07d246f00
Sorry it took so long.


Write an underscore '_' to a file. Excel, requests file recovery

A string containing '' (underscore) is written as 'x005f' (in 'sharedStrings.xml '). After that, when you open the file, Excel will ask you to restore file. And after the recovery in the file 'sharedStrings.xml ' the value changes to ''. I tried to write the special characters '\b\v\f\u000e\u 001f\u007f', and they are also converted to a format similar to 'x005f', but Excel works fine with them.

After changing the file and starting the project, Excel still asks to restore the file.
Microsoft Office Professional 2021
Microsoft ® Excel ® LTSC MSO (16.0.14332.20438)
I tried running the same thing in Excel 2019 and got the same results.

Yes that's exactly what happened

It works.
Also to check if there is a problem in the version of Excel I used, I downloaded the application 'Microsoft Excel: Spreadsheets' on android. When I opened the file, I received the same warning about recovery, so maybe this will help your tests.


Make --dump and --show work with aliases

Given the justfile:
alias b = build

build:
  echo 'building!'

The following should work:
$ just --show b

I've been cating my justfiles to see aliases when I forget them. I think it would be ideal if --show worked like this..
Given:
alias b = build
build:
    echo 'Building!'

$just --show build
build:
    echo 'Building!'
---------------
Aliases for build: b

$just --show b
b is an alias for build
build:
    echo 'Building!'

We need a better separator than a bunch of hyphens but this is close to what an average user would want to see when they use --show. What do you think?
If someone else isn't working on this, I can give it a shot next week/weekend.

It's nice when the output of a command is also valid input. In the case of --show it would allow moving recipes to a new justfile. For example:
$ just --show build >> new-project/Justfile

So I think for --show it would be good to use the syntax of the original file:
$ just --show b
alias b := build
build:
    echo 'Building!'

It seems natural to include the alias b := build line if a user does --show b, since they specifically asked about it, and displaying the build recipe on its own wouldn't make much sense.
I'm not entirely sure if --show build should also show aliases:
$ just --show build
alias b := build
build:
    echo 'Building!'

Or not:
$ just --show build
build:
    echo 'Building!'

What do y'all think? I'm don't feel super strongly about it, so I'm easily swayed.

I think displaying the alias is appropriate. If --show displays details about a given recipe then it should also show any meta-details such as aliases as well, in my mind.  😁


Question regarding PYSEC-2024-71 fix

The vulnerability marked as PYSEC-2024-71 seems to be solved: https://github.com/advisories/GHSA-hxwh-jpp2-84pm . However https://osv.dev/vulnerability/PYSEC-2024-71 marked v5.0 as vulnerable as well. They said they are downstream from you guys in this comment: https://github.com/google/osv.dev/issues/2568 .
Question: what is your upstream for this vuln (not asking about NIST NVD but beyond that)? What is the source of truth to submit the update that this has been fixed? Is it the original report bug report: https://huntr.com/bounties/a42935fc-6f57-4818-bca4-3d528235df4d ?
P.S. This affects environments tracking "time to resolution" via pip-audit for regulatory purposes, so I'm willing to submit a PR either here or whichever upstream needs to be informed.

@MindaugasBernatavicius Thanks for the question, we pull from the NVD dataset but don't automatically update if more information is added to those records, perhaps the initial record didn't have the fixed version and that's why our record is "out of date". If you could submit a pull request fixing the issue I can review it, ping me on the PR.
We use some OSV tooling to sift through NVD for Python projects, maybe that tooling has improved to be able to automatically update records? I haven't been following OSV tooling very closely lately.
If that answers your question I can close this issue.

Sure, this can be closed. If the PR is incorrect we might want to discuss it there.


For some reason .env is not created automatically

For some reason .env is not created automatically. I don't know yet how to change the x and y settings

you should be developer possible to change it, need knowledge


Add "x-ms-is-hns-enabled" to ServiceGetAccountInfoResponse

Is your feature request related to a problem? Please describe.
The "x-ms-is-hns-enabled" is not exposed via ServiceGetAccountInfoResponse.
Describe the solution you'd like
Expose the response value there.
Describe alternatives you've considered
N/A
Additional context
Currently the only way to access it is to call response._response.headers.get("x-ms-is-hns-enabled")

@ramya-rao-a Swagger fixed https://github.com/Azure/azure-rest-api-specs/pull/11871

Added isHierarchicalNamespaceEnabled to the response of BlobServiceClient.getAccountInfo().
This is now availabe in<EMAIL_ADDRESS>and the it will be GA next month.


Move rbac rules for moac to a separate yaml file

Depending on exact timing we could see that certain operations in moac
start to fail when doing:
kubectl delete -f moac-deployment.yaml
That is because rbac rules for moac have already been destroyed but moac
still has work to do in the short time window before k8s destroys it too.
Now it is possible to enforce correct order when deleting moac by running:
kubectl delete -f moac-rbac.yaml
kubectl delete -f moac-deployment.yaml

you are right, it should be :-D I will update the message before the merge. thanks

I have updated the commit message


jsconfig.json gives broken intellisense for js files with module.exports

VSCode Version: 1.38.0
TS Version: 3.6.2
OS Version: Ubuntu 18.04 LTS bionic beaver
Tested with no extensions
Can not really find a solution to this problem after a lot of searching

I have a personal js library that I use both in browser-side and node-side for a lot of my personal projects . In fact the library itself has its own project . The js library looks like this :
const library = {};

library.foo = function() {/*browser code*/}

if (typeof module === 'object') {
 library.bar = function() {/*node code*/}
 module.exports = library;
}

For a browser project (library is used via script tag) If I try to get intellisense via jscofig.json for such a file I get broken intellisense (by typing f or b I get suggestions for foo or bar respectively , but there is no library suggestion when I type l , and even if I type library I get no suggestion about its properties ) .
For a node project everything works fine regarding intellisense using require .
The issue here is with module.exports . If I change it to var substitute = module; substitute.exports = library; then I get intellisense for browser project via jsconfig.json but broken intellisense (as described before) for node via require .
How can I get intellisense to work for both require and jsconfig.json ?
Before you attempt to check intellisense you should know the following  :

I open index.js in a project that does not have any other js file and no jsconfig.json file . Then I view in the same VScode session some other , completely unrelated to my project , js files . For whatever file I view I get intellisense (of the globally scoped variables that have been defined via let , var or const) when I am editing index.js . If the unrelated js file happens to have module.exports then the intellisense that I get in index.js for that file is broken like it has been described before  .
I stop getting intellisense for files that I view when I create a jsconfig.json file file in the folder of index.js . But still there is the same broken behavior with intellisense for files that are included via jsconfig.json and have module.exports .

Please restart vscode to test if intellisense really works .

There currently isn't a way to write a JS file in such a way that it's treated as a UMD module (i.e. available in both browser and module contexts). In TS declaration file this would be export as namespace library but there's not yet an equivalent for JS


fix dtlauncher

fixed issue and added touch screen events

Hi - it looks like you also changed helloworld - but I think you have renamed the app file for some reason and then left the old app.js?

Hi - it looks like you also changed helloworld - but I think you have renamed the app file for some reason and then left the old app.js?

Yes i forgot to remove the app.js
I  prefer to use a final name than a generic app.js name, at least when there is common app file.

I prefer to use a final name than a generic app.js name, at least when there is common app file.

I'm not a big fan of that, it's much easier to look in a directory and see icon.js/app.js/etc and know what's there, and it's also what we do in the example app and the tutorials.
It's not a problem with this one, but please don't start renaming all the files in existing apps.


snow bros 2 is out.

arcade game.
https://github.com/psomashekar/pram0d-pocket-dist-public

The repository doesn't use the release feature of GitHub, which both this application and the consumers of its API use. Unfortunately that means we don't have a good way to add this without creating a hack specifically for this repository. This isn't something we're looking to do any time soon, but pull requests are welcome.

Alternatively, pram0d said that someone could fork the repository and add releases themselves, but that would require monitoring any updates he makes and updating the release zips manually. If someone takes on that work, then we could get it added.

I added Snow Bros. 2 to the website, and we're working on getting this added to the API in #26. Going to close this issue and track work in the other one.


fix(storage): renames vars and fields for clarity

Good job 👍
Please also fix the test's

LGTM, thanks!


Update LAB_AK_06_Lab1_Ex2_Connect_Windows.md

from CraigBeereMCT issue named Module 6 - Lab 1 - Exercise 2: change device login to user login #122 I ran into the same issue this past week.
I have created a PR to update the new User authentication method.
Related Issue Module 6 - Lab 1 - Exercise 2: change device login to user login #122
Link related Github Issue 🢂 Fixes # . (Include issue number after #)
Checklist
Mark completed with "x" between brackets, "[x]", or checking the box once the PR is created:

[x ] Has related GitHub Issue 💥 Create Issue 📝
[x ] Tested it
[x] Read the PR collaboration guide 👓 Collaboration Guide 📝

Changes proposed in this pull request:

I saw issue #122 and will correct.


Supplying CloneTileData/CloneBatchData as a raw array

TODOs:

[ ] Find out why clone instancing isn't working in Perf game
[ ] Confetti is slower, I'm thinking it's the flatMap. Refactor code to optimally use raw data. (Or is this just the wrong use case? Test on roguelikes instead?)

I've slept on it, and I may park this.
I like the syntax change that happened here, and that might be worth keeping (?), but I think the core idea may be deeply flawed. The issue is that to gain all this efficiency you'd need to basically bind your model structure to the mutable structure this thing requires including all the rendering specific values. If you're going to do that, there's probably much simpler ways to gain efficiency.

Parking this PR.


Update VVCEasy.bat

check and get the UAC for administrator privilege

@neoOpus, thank you for making pull request.
Is there any that you could do without creating new window cmd in batchfile?

Martin Eesmaa

I found this project (https://github.com/gerardog/gsudo) and it gives the elevated privilege to the same terminal window/tab... so I think you can add it to your project if you think it fits with your requirements...
let me know if you want some help regarding this or some other stuff

@neoOpus, okay, I guess I have no choise of creating new Windows Terminal window, it will be stuck in old cmd, but I can accept your pull request. I hope the UAC Script will help to users... :)

@neoOpus, okay, I guess I have no choise of creating new Windows Terminal window, it will be stuck in old cmd, but I can accept your pull request. I hope the UAC Script will help to users... :)

I send you a second reply to your question, I guess you didn't read it, it is about gsudo which can make what you wanted but it is part of another project that maybe you can include in your distribution...


Set up the kanban board

https://github.com/matee-safi/spacex-project/projects/1
We are a group of two

Hi @matee-safi @astroboyReloaded ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights
Kanban template :heavy_check_mark:
Tasks assigned :heavy_check_mark:
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Do we need to add support for Firestore labels?

Present on WriteRequest and ListenRequest:
message WriteRequest {
  string database = 1;
  ...
  map<string, string> labels = 5;
}

/cc @schmidt-sebastian @samtstern

Neither WriteRequest nor ListenRequest should at this point be used in the Python SDK.

@schmidt-sebastian That is true, they are not. Closing out for now.


Can't use this folder to protect your privacy choose another folder ....!!!

I am unable to access .statuses folder directly. When I click on the "grant button" it does not directly open the ,statuses folder and open the root directory and the "use this folder" button is unclickable. User have to open manually this folder to get access.
here is the directory which I want to access:
saf = Saf( '/storage/emulated/0/Android/media/com.whatsapp/WhatsApp/Media/.Statuses');

how to initialize same problem im facing how to solve ?

Android less than 11 don't need permission to access. Above Android 11, access can be given the same way.


JAEGER_SERVICE_NAME from jaeger-operator not respected

The jaeger-operator for Kubernetes modifies app/service containers to have a JAEGER_SERVICE_NAME environment variable. However, java-spring-jaeger wants e.g. OPENTRACING_JAEGER_SERVICE_NAME.
With many microservices, mapping these env vars using beans is a little cumbersome. I'm not sure which project should make it possible to configure the env var name. Do you think it would make sense to respect JAEGER_SERVICE_NAME as well as OPENTRACING_JAEGER_SERVICE_NAME?

We could certainly do it easily here.
@pavolloffay would something like this make sense?

Here's the place where it's hardcoded in the operator: https://github.com/jaegertracing/jaeger-operator/blob/9cdd514655654a92de5a70b58fda2797d6a6a32b/pkg/inject/sidecar.go#L30

Yes it would be good if the env vars from the operator (JAEGER_SERVICE_NAME and JAEGER_PROPAGATION) were respected.

@camelpunch would you like to provide a PR? If that's not possible, I will get to it some time soon

Yeah I'll take a look.

Awesome, thanks!

Closed by #57


double use of Productidentifier

Sorry,  the Productidentifier was used double by me

Somehow Callicolor has not been added to the device catalog and ist not found n the device dashboard.