MRD Data opened in Python appears corrupted

Hi all!
I've just taken KomaMRI.jl for a quick test-drive and I noticed that I can't open the resulting ismrmrd files with the ismrmrd-python package. Reason seems to be a second nested level of the <UserParameters> element (see below). I also spotted a potential problem with the gpu user param value (annotated in the snippet below).
I used v0.7.6 with julia 1.9.1
<?xml version="1.0" encoding="utf-8"?>
<ismrmrdHeader xsi:schemaLocation="http://www.ismrm.org/ISMRMRD ismrmrd.xsd" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns="http://www.ismrm.org/ISMRMRD" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
 ...
<userParameters>
    <userParameters> <!-- this one here! -->
      <userParameterLong>
        <name>Nblocks</name>
        <value>2195</value>
      </userParameterLong>
      <userParameterLong>
        <name>gpu</name>
        <value>false</value> <!-- this may be another issue as value 'false' is not a valid xs:long value -->
      </userParameterLong>
      <userParameterLong>
        <name>gpu_device</name>
        <value>0</value>
      </userParameterLong>
      <userParameterString>
        <name>precision</name>
        <value>f32</value>
      </userParameterString>
      <userParameterString>
        <name>return_type</name>
        <value>raw</value>
      </userParameterString>
      <userParameterDouble>
        <name>Δt_rf</name>
        <value>5.0e-5</value>
      </userParameterDouble>
      <userParameterLong>
        <name>Nthreads</name>
        <value>1</value>
      </userParameterLong>
      <userParameterString>
        <name>sim_method</name>
        <value>Bloch()</value>
      </userParameterString>
      <userParameterDouble>
        <name>sim_time_sec</name>
        <value>15.131958583</value>
      </userParameterDouble>
      <userParameterDouble>
        <name>Δt</name>
        <value>0.001</value>
      </userParameterDouble>
    </userParameters> <!-- this one here! -->
  </userParameters>
</ismrmrdHeader>

EDIT 1-Dec-2023 (@cncastillo):
Hi I think this issue can be separated in two:

[ ] #236
[ ] Incorrect byte size of AcquisitionHeader in exported MRD

Hi @gabuzi, thanks for noticing this. I was aware that there could be a problem with the .mrd and Python's MRD reader. As when I tried it it also did not work. This helps us fix it.
If there is problem with the userParameters XML header having not-numeric stuff we can easily change it.
Could you send us the code to open an .mrd in Python? I assume you opened the UI, pressed simulate, and tried opening the resulting .mrd located in the temp folder. Just to try to reproduce this.

Hi can any of you attach a valid MRD to compare?

Oh, this is really bad. I have not seen https://ismrmrd.readthedocs.io/en/latest/mrd_raw_data.html#acquisitionheader and assumed that the offsets correspond to the in-memory layout of the C struct (which in Julia is the immutable header type). And when I worked on this, I actually have seen the padded data in some public MRM files.
So am I reading https://ismrmrd.readthedocs.io/en/latest/mrd_raw_data.html#acquisitionheader  correctly, that the layout is always tight, i.e. that do padding is involved? In that case myfieldoffset seems to be the correct implementation and we then would not need to hardcode the actual table.
One issue though is that I don't understand how we can present the "unpadded" struct in Julia. Does somebody know how this is handled in the C bindings? The C struct will also have such padding.

Looking at https://github.com/ismrmrd/ismrmrd/blob/master/libsrc/serialization.cpp#L11 it seems that the C struct is also serialized directly. I don't get that. Does it mean that the C struct is not padded?
To solve this issue it would be great to ping somebody, how has more experience with the C/C++ implementation and can clarify these questions.

C structs are padded by default unless you tell the compiler not to do so. But have found the line where this is actually done:
https://github.com/ismrmrd/ismrmrd/blob/d364e03d3faa3ca516da7807713b5acc72218a37/include/ismrmrd/ismrmrd.h#L82C1-L82C13
So with that all is clear.
I already did a little bit of research how we can solve this. Unfortunately we can't disable structural padding. But I think we will simply convert the struct to an UInt8 Array and that should do the trick.

And for reference: In our unit test we use the ISMRMRD data:
https://sourceforge.net/projects/ismrmrd/files/data/
And that seems to be padded. Would be great if someone could have a look and inspect that data. I can't load it with the python ismrmrd library but don't know where the issue is.
So what I would definitely also need to fix the issue is correct data. Ideally again, some cartesian and spiral data. What is also important is that the data is small, so that we can use it in the unit tests.

Hi @tknopp, Thanks for looking into this.
I have tested the fix with cartesian and spiral data generated with Koma, and I think it does in fact solve the problem, as I do not see the data being corrupted in Python anymore. I will do further testing, but I believe I will close this issue soon :).

Great, thanks so much for reporting. I did, by the way, already do the releases (including the move of the Limit... types) so you can already rely on that in Koma.


Documentation

continuation: https://github.com/Dasharo/open-source-firmware-validation/pull/352/files


Confusing enum reverse string lookup behaviour in strict mode

Summary
Code below.
Typescript in --strict mode will reject enum reverse lookups using arbitrary strings with this very confusing error message:

Element implicitly has an 'any' type because index expression is not of type 'number'.

A workaround is to cast the string to a subset of all possible enum keys but that's impractical with large enums (or is there an easy way to say "any key of enum X" in TS?).
 I have to use the string representation in the dom, so the state of the filter has to be the string representation. When I check the item's enum value I already know it can only be one of those enum members.
This report is three-fold:

Is this behaviour intended? There are cases where it can definitely catch a reverse lookup that can fail.
If it is intended, then what is the correct way to declare that the lookup will always be valid? let typeFilter: "FOO" | "BAR" = "FOO" is impractical and increases risk of errors.
Regardless of the above, the error message should be fixed. The element highlighted (typeFilter) does not in fact have an "any" type, and Type[typeFilter] will not be any either (in fact, Type[typeFilter] as Type does not fix it). An error such as "Reverse lookup on enum Type may fail because typeFilter is of type 'string'" is much clearer already.

Details
TypeScript Version:  3.2.2
Search Terms: enum reverse string lookup
Code
// test.ts
// Compile with `tsc --strict test.ts`
export enum Type {
	FOO = 0,
	BAR = 1,
}

const Test = () => {
	// Change the following line to `let typeFilter: "FOO" | "BAR" = "FOO";` and it will work
	let typeFilter = "FOO";

	if (Type[typeFilter] != Type.FOO) {
		return;
	}
};

export default Test;

Actual behavior:
test.ts:9:11 - error TS7015: Element implicitly has an 'any' type because index expression is not of type 'number'.

9  if (Type[typeFilter] != Type.FOO) {
            ~~~~~~~~~~

Found 1 error.

Playground Link: Playground
Related Issues: Maybe #27297

Use const instead of let to get a string literal inference.

@jack-williams Yes that's intended for the purpose of reproducing the issue.  The problem isn't that the inferred type is string (that's correct), it's that the behaviour when doing an arbitrary string reverse lookup is very confusing.

My apologies; misread the intent of the query. Yes I agree the error message could probably do with more elaboration. The problem is that indexing into the enum using a number returns a string, and therefore this code is being hit:
else if (getIndexTypeOfType(objectType, IndexKind.Number)) {
    error(accessExpression.argumentExpression, Diagnostics.Element_implicitly_has_an_any_type_because_index_expression_is_not_of_type_number);
}

If think might need the additional check that the indexing value is a number:
 else if (isTypeAssignableToKind(indexType, TypeFlags.NumberLike) && getIndexTypeOfType(objectType, IndexKind.Number)) {
    error(accessExpression.argumentExpression, Diagnostics.Element_implicitly_has_an_any_type_because_index_expression_is_not_of_type_number);
}

With that, you then drop into this error:
error(accessExpression, Diagnostics.Element_implicitly_has_an_any_type_because_type_0_has_no_index_signature, typeToString(objectType));

which gives the message:
 Element implicitly has an 'any' type because type 'typeof Type' has no index signature..
I think at this point there probably wants to be some elaboration based on whether the index type is a string, and the target is a non-const enum.

let typeFilter: keyof typeof Type = "FOO";
I think

The error message is correct, but in the case of enums, could stand to be improved (since the index signature is implicit). Something like
Element implicitly has an 'any' type because index expression is not an enum key or `number`.


BlazorWebAssemblyStandaloneWithIdentity uses cookie auth but no antiforgerytoken

Description
The BlazorWebAssemblyStandaloneWithIdentity sample uses a cookie to store the authentication token but I believe that this opens the site up to csrf attacks.
This is usually mitigated by using antiforgerytokens but I've found it difficult to apply this to the sample as the api and client run as different sites (although I may have just dome something wrong).
Is is right that antiforgery tokens should be used here or is there a reason it is not required?
If it is required will it be added to this sample or can someone point me in the right direction to get this working?
Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/webassembly/standalone-with-identity?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/security/webassembly/standalone-with-identity.md
Document ID
c4e6ec41-7bea-e600-6473-c5c870aab082
Article author
@guardrex

Thanks for opening the issue this way. We use the metadata to track work on these.
It should be automatic and transparent. If you look at a request in browser tools, you should see the antiforgery cookie named something like .AspNetCore.Antiforgery.XXXXXXX on each request.
I agree that it would be nice if the article addressed this subject to head off concerns about it.
@JeremyLikness ... Plz confirm for us that devs don't need to sweat XSRF with basic use of this solution, and are there any gotchas 😈 that are worth calling out?

I have run the Backend and BlazorWasmAuth projects, registered and logged in without the Antiforgery cookie being created.
There is no cookie in the dev tools called .AspNetCore.Antiforgery.XXXXXXX. This is, in fact, what I have been trying to implement.
Am I missing something?

Let me see. I have that test app here ... I'll run it and see what happens. I thought I recalled seeing it, but I might be mistaken. Stand-by ..........................

Here is mine. I still see no antiforgery token.
Feel like I'm missing something obvious

Let's clarify something ...
Are you running at secure endpoints (https)? How are you running these apps ... command-line dotnet run?

Ok ... I just wanted to make sure. You probably know this already, but dotnet run uses the first launch profile, which happens to be insecure HTTP. We've had some issues crop up where the dev just needed to rotate the https launch profile to the top of the list in launchSettings.json to get a sample running securely ... thus make security features work properly.
Stand-by for a bit. I'm going to email Jeremy, too, in case he doesn't see the ping here. He has the security chops 👨🎓 to explain further any considerations for antiforgery with this particular sample app and probably why I'm seeing an antiforgery cookie but you aren't.

Hi,
I will need to make one tweak to secure the logout endpoint:
app.MapPost("/logout", async (
    SignInManager<MyUser> signInManager,
    [FromBody]object empty) =>
{
    if (empty is not null)
    {
        await signInManager.SignOutAsync();
        return Results.Ok();
    }
    return Results.NotFound();
}).RequireAuthorization();

This prevents the endpoint from accepting any forms posts. I spoke with our security team, and they indicated JSON-only endpoints that don't accept form auth or form posts don't need XSRF protections (which is why the MapIdentityEndpoints don't use anti-XSRF). The only way the /logout endpoint above works is if you post a JSON payload of {} which becomes an empty object with no properties but is not null. AFAIK this can't be done from a form.
In ASP.NET Hosted, the anti-XSRF token was rendered in the same page as the client and could be securely grabbed to send back. In standalone Blazor, the client is independent. There is no secure way to send the anti-XSRF token to the client (imagine an endpoint that exposes the token ... but then how do you protect that endpoint?).
There are additional mitigations like SameSite cookies and custom headers and I'll see if those make sense to include in the sample (we definitely should mention them in the documentation).

Thanks for that understanding, @JeremyLikness.
Let's work it from @nathan-parkinson's issue here. When you're ready, paste content into an issue comment, and I'll place it into the article, edit, and ping you on the PR for review+updates.
WRT to the logout POST endpoint, do you want me to make that update? I can knock that out right now.

Go for it!

Ok, that bit is done. I'll add text to the article covering it when I work this issue.

Thank you both for your help with this.
I had wondered how an antiforgery token could work across sites.
It not being required for json only endpoints was the important piece of information I was missing.
Thanks again.

Leaving a note here in case anyone stumbles on this. The code above ☝ throws the logout into a ☠ death loop because an error is triggered and the MapPost is executed over and over. I'll get updated code into the sample app as soon as it becomes available. Until then, I reverted the code back to its original form with a code comment to see the sample repo's issue for more information.
https://github.com/dotnet/blazor-samples/issues/132

The missing piece is posting the empty content from the client. I'll work on that part (with SPA you just fetch with {} but I'm not sure if for Blazor the solution is to POST with new object() as the payload or I'll need to use JavaScript interop. I'll keep you posted.

NOTE TO SELF ... I should be able to close this out, as we merged the update to the Blazor sample app.


CI: set minimal permissions for GitHub Workflows

Hi!
I'm here to suggest the definition of minimal permissions on your workflows, as it would harden your security agains supply-chain attacks.
The idea is to update your workflows to set top-level read-only permissions that would be inherited to all jobs that don't declare their permissions; and for the jobs that require any write permissions, they'd be given job-level. Defining minimal permissions would enhance security against erroneous or malicious actions from external jobs you call from your workflow. It's specially important for the case they get compromised, for example.
Setting minimum permissions for workflows is recommended by GitHub itself and also by other security tools, such as Scorecards and StepSecurity.
I'd be happy to raise a PR with the changes if you agree.
Context
I'm Diogo and I work on Google's Open Source Security Team(GOSST) in cooperation with the Open Source Security Foundation (OpenSSF). My core job is to suggest and implement security changes on widely used open source projects 😊

Hey! This issue has been idle for quite some time. Do you plan on considering these changes? Otherwise I will wait up to 2 more months and then close the issue.
As the changes would be really simple, I've taken the liberty of already creating a PR that should make it easier to preview them.
Thanks!


Usage of .map gets ignored

Thanks for the issue report! The usage here is incorrect; there's an extra set of parentheses in that code, and it's not returning the JSX. It should be this:
return (
	<>
		1
		{new Array(5).fill(0).map((_, row) => <>2</>)}
		3
	</>
)

oh, didn't see that, thanks


intercept-build missing from basekit container

The  intercept-build script is missing from the basekit container.
https://software.intel.com/content/www/us/en/develop/documentation/get-started-with-intel-dpcpp-compatibility-tool/top.html#top_MIGRATION_MAKE_CMAKE

For the Intel® DPC++ Compatibility Tool, please use the Intel® oneAPI Base Toolkit installer found at https://software.intel.com/oneapi.


tab completion of params/vars wipes out remainder of string

Start with the following expression:
        "[resourceId('Microsoft.Network/virtualNetworks', parameters('subnet2Name'))]",

Put the cursor inside the first paren for the resourceId function, I want to add the resourceGroupName parameter for the function.  So type in parameters('  (without the splats of course)
After typing the first single quote, press CTRL+SPACE and select a param to tab complete.  The namespace will be erased.
It looks like we have a few chars we look to terminate the autocomplete, but I think if we simply don't erase anything that might be fine.  Just complete the param name, closing single quote and closing paren.

@bmoore-msft I cannot reproduce this in recent builds. Can you re-confirm that you are still seeing this as an issue?

yes, still repros in the build I have v0.8.4

Not positive this is unexpected behavior...

@neilpeterson @bmoore-msft
Technically this is correct behavior.  Consider this:
"[parameters('<CURSOR>parameter1')]"

If you type CTRL+SPACE and replace with 'subnet2Name', the extension marks the entire current string literal for replacement ('parameter1').
Above, after typing parameters(', you get this (because vscode adds the closing ' - same behavior in typescript):
    "[resourceId(parameters('<CURSOR>''Microsoft.Network/virtualNetworks', parameters('subnet2Name'))]",

Same as above, the span to replace is the entire current string literal, which is '''Microsoft.Network/virtualNetworks' (since two single quotes together are an escape for a single quote).  Thus you get the behavior above.
Need to get a better sense of when users feel the behavior is wrong.  If we need to, we would special-case this scenario's replacement span, or maybe handle the parameters(' replacement differently?

Ignore the above, we shouldn't be replacing the entire string literal.  I could swear vscode used to do it that way...??

Still repros.


x/build/cmd/gopherbot: gopherbot is in sleep-error loop

Gopherbot isn't happy right now. From its logs:
2019/02/12 01:01:02 got corpus update after 274.731276ms
freeze
	https://golang.org/issue/23772  x/build/cmd/gerritbot: <fill this in>
2019/02/12 01:01:02 freeze old issues: PUT https://api.github.com/repos/golang/go/issues/23772/lock: 404 Not Found []
2019/02/12 01:01:02 PUT https://api.github.com/repos/golang/go/issues/23772/lock: 404 Not Found []
2019/02/12 01:01:02 gopherbot ran in 110.450416ms
2019/02/12 01:01:02 sleeping 30s after previous error.
2019/02/12 01:01:32 Updating data from log *maintner.netMutSource ...
2019/02/12 01:01:36 Downloading 649 bytes of https://maintner.golang.org/logs/45 ...
2019/02/12 01:01:36 wrote /cache/golang-maintner/0045.growing.mutlog
2019/02/12 01:01:36 gerrit go.googlesource.com/gofrontend: Ref {CLNumber:161963 Version:0} => 3a9fd412dc7603e55cff110c741a02565ae83980
2019/02/12 01:01:36 Reloaded data from log *maintner.netMutSource.
2019/02/12 01:01:36 got corpus update after 3.996453811s
freeze
	https://golang.org/issue/23772  x/build/cmd/gerritbot: <fill this in>
2019/02/12 01:01:36 freeze old issues: PUT https://api.github.com/repos/golang/go/issues/23772/lock: 404 Not Found []
2019/02/12 01:01:36 PUT https://api.github.com/repos/golang/go/issues/23772/lock: 404 Not Found []
2019/02/12 01:01:36 gopherbot ran in 98.493511ms
2019/02/12 01:01:36 sleeping 30s after previous error.
2019/02/12 01:02:06 Updating data from log *maintner.netMutSource ...
...

It's constantly looping with "sleeping 30s after previous error" and not making any progress.
@bradfitz spotted this in https://github.com/golang/go/issues/30181#issuecomment-462560911.

Issue #23772 was an empty/spam issue that was closed a year ago. It's showing up as 404 now, which gopherbot can't handle.
There are two things we need to fix here:

This specific problem in the "freeze old issues" task: it should skip issues that are 404 and move on to the rest (not treat it as a fatal error).

See if the high-level behavior can be improved such that if one task starts to fail, perhaps it can still perform the other tasks rather than short-circuiting and not attempting other tasks. Maybe it's not possible to make such an improvement in the general case, but it's worth considering.

Since this isn't resolved yet, I have to manually say that I've sent a fix for the "soon" part of this issue as CL 161906.

I've deployed the fix in the aforementioned CL, which is why gopherbot woke up. The CL still needs to be reviewed before it can be merged.

The current fix adds to the plate of "actions gopherbot keeps taking without any effect" (issue #28320):
freeze
	https://golang.org/issue/23772  x/build/cmd/gerritbot: <fill this in>

To fix that, we need to update maintner to detect issues that have gone missing, and set their NotExist field to true. CL 161521 may help with that. /cc @jmdobry


Add Documentation About Required Injection Configuration

Need to configure IConfigurationRoot and IHttpContextAccessor, not sure if there is a better way than just telling people to configure these...

Done in PR #11


[Bug] CreatedBy Salesforce function is not dealt with properly

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
I have a formula on opportunity table  like :
IF(
AND(
    CreatedBy.Id <>  OwnerId,
IF(
    NOT(CONTAINS(Owner.UserRole.Name,"some_string")),
    ),
IF(
    NOT(CONTAINS(Owner.UserRole.Name,"some_other_string")),
    )
    ),True,False)

(It is more complicated in reality but anyway this is the problematic part)
So we should have 3 joins :

1 for the CreatedBy.id  which should be : left join user on user.id = opportunity.created_by_id
2 to catch user_role :

 left join user on user.id = opportunity.owner_id
 left join user_role on user.user_role_id = user_role.id

The first one is not rendered by the tool and CreatedBy.Id  is translated to user__alias.id  so opportunity.created_by_id because of the join, which is giving 0 lines in return because of the CreatedBy.Id <>  OwnerId  line.
It seems that the tool is not really understanding the CreatedBy.Idpart or I am missing something.
Relevant error log or model output
Content of fivetran_formula :
table : opportunity
SF formula:

IF(
AND(
    CreatedBy.Id <>  OwnerId,
IF(
    NOT(CONTAINS(Owner.UserRole.Name,"some-string")),
    ),
IF(
    NOT(CONTAINS(Owner.UserRole.Name,"some-string2")),
    )
    ),True,False)

view_sql:
select
main__table.id,
(
CASE
  user__alias.id <> main__table.owner_id
  AND
  CASE
    WHEN NOT REGEXP_INSTR( user_role__alias.name, 'some-string' ) > 0
    ELSE  NOT REGEXP_INSTR( main__table.closer_role_c, 'some-string' ) > 0
    END
  AND
  CASE
    WHEN  NOT REGEXP_INSTR( user_role__alias.name, 'some-string2' ) > 0
    ELSE  NOT REGEXP_INSTR( main__table.closer_role_c, 'some-string2' ) > 0
    END
WHEN true THEN True ELSE False END )
as value_we_want
from opportunity as main__table
left join user as user__alias on main__table.owner_id = user__alias.id
left join user_role as user_role__alias on user__alias.user_role_id = user_role__alias.id

Expected behavior
we should have :
view_sql:
select
main__table.id,
(
CASE
  user__alias_NEW.id <> main__table.owner_id
  AND
  CASE
    WHEN NOT REGEXP_INSTR( user_role__alias.name, 'some-string' ) > 0
    ELSE  NOT REGEXP_INSTR( main__table.closer_role_c, 'some-string' ) > 0
    END
  AND
  CASE
    WHEN  NOT REGEXP_INSTR( user_role__alias.name, 'some-string2' ) > 0
    ELSE  NOT REGEXP_INSTR( main__table.closer_role_c, 'some-string2' ) > 0
    END
WHEN true THEN True ELSE False END )
as value_we_want
from opportunity as main__table
**left join user as user__alias_NEW on main__table.created_by_id = user__alias_NEW.id**
left join user as user__alias on main__table.owner_id = user__alias.id
left join user_role as user_role__alias on user__alias.user_role_id = user_role__alias.id

dbt Project configurations
--
Package versions
0.6.4
What database are you using dbt with?
snowflake
dbt Version
--
Additional Context
No response
Are you willing to open a PR to help address this issue?

[ ] Yes.
[X] Yes, but I will need assistance and will schedule time during our office hours for guidance
[X] No.

Hi @moreaupascal56 thanks so much for opening this issue!
Unfortunately, this is one that will need to be addressed within the connector itself (which my team does not have much insight into). I would recommend opening a Support Ticket so one of our engineers may look at this deeper and be able to address the bug within the connector.

Hey 👋  Ok yes I understand, so everything that is already in the Formula table is done by the connector that's logical.
Thanks for your answer and have a great day 🌞


Map the vim bindings in more modes

Use (nore)map to map the functions in visual, select and operator-pending mode instead of only in normal mode.
Use <C-U> (:h c_CTRL-U) to delete the automatically inserted range when pressing : in visual mode, as the functions do not support a range.
Update all references in the README and help page.

Thanks for the update, @druckdev!


feat: cut response times which have  first end timestamps

Description
ignore data flow until first data of output column arrived in Response Time.

Related links
Jira: https://tier4.atlassian.net/browse/RT2-1430

Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR has been properly tested.
[ ] The PR has been reviewed.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] (Optional) The PR has been properly tested with CARET_report verification.
[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@emori-ctc
Please review.


Book Live

Source name
Book Live
Source link
https://booklive.jp/
Source language
Japanese
Other details
No response
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

SpeedBinb.


Added more packages to OSGI's exports

This is necessary for structured arguments or creating custom field types.
Just a simple change to OSGi Manifest file

Thanks!


support GeometryBasics 0.2 and avoid doc warning

Seems hasproperty is 1.2+

Thanks!

Thank you :)


Add stargazers-you-know from friendly-github

If you think it's intrusive, just make it disabled by default.
https://github.com/Hermanya/friendly-github

See https://github.com/sindresorhus/refined-github/pull/1393#issuecomment-403977822

Duplicate of https://github.com/sindresorhus/refined-github/pull/1392

@fregante Have can I use friendly-github without conflict with refined-github?


Basic volumetrics replacement for particles

What is the context of the feature/improvement?
Most of the games I have had issues with have been caused by particles for volumetric effects so replacing the 2D analogue would sidestep all the issues with the particles
Please describe the feature/improvement?
The two main groups of particle effects I have in mind is smokes and flames (things like clouds etc would be smoke here). Preferably you would be able to select a group of textures you want to use as the parent for a new volumetrics replacement and based on the particles movement a volumetrics replacement will be put on the scene instead.
Preferably the original texture will be used to generate the root size of the emitter so smoke stacks are fires are at least mostly the correct size by default.
What would be your solution? (optional)
This is definitely oversimplified but

Select a group of textures that the new volumetric will replace.
Record the movement of the texture group that the animation of the volumetrics will follow. Eg what direction is the smokestack blowing towards and where is the emitter located
Use the spawn size of the texture to calculate the emitter size and roughly the correct (Eg) perling noise contrast and animation speed to get a similar look to the original effect.
Allow some user tweaks and correct any issues that the automated process might have left like clipping, density, reflective Vs scattering material and animation speed or animation path.

Version
0.5.1

I feel like I should say that I have zero expectations for this being implemented anytime soon since I will assume from my limited experience that it's a pretty big task to implement even with the existing open source solutions to build opon

REMIX-3116 for tracking


⚠ ClamAV has degraded performance

In be5a382, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 233 ms

Resolved: ClamAV performance has improved in ec4e81d.


Chunked Transition Group filtering

Is there any good way to transition a chunked list
im using animate css but it seems it doesn't work flawlessly
im also using mode="out-in" in transition group is this supported?
the list will be chunked by 3 and so on and add row class per chunk and 3 columns

but the transitioning between components doesnt work properly like this

This is the snippet of my code in my main layout
    <transition-group  style="margin-bottom: 1rem;" v-for="data3 in dataList"
                      class="row" mode="in-out"
                      enter-active-class="animated bounceInRight"
                      leave-active-class="animated bounceOutRight">

      <div class="col-xs-12 col-md-4" v-for="data in data3" :key="data['.key']">
        <md-card md-with-hover>
          <slot :data="data"></slot>
        </md-card>
      </div>

    </transition-group>

using this computed propert for filtering and search
      dataList() {
        let items = this.list;
        if(items && this.search && this.searchKey) {
          let regExp = new RegExp(`${this.search}`,'i');
          items = _.filter(items, item => {
             return regExp.test(item[this.searchKey])
          });
        }
        return _.chunk(items, 3);
      },

Hello @DevEclipse
Thank your for your interest in this project.
However, your issue is a usage/support question, and the issue tracker is reserved exclusively for bug reports and feature requests (as outlined in our Contributing Guide).
We encourage you to ask it on the forum  , Stack Overflow or on gitter and are happy to help you out there.


1.14.7 dev ltc scrypt

Scrypt

submit the review with the changes and the approval

So, this will be reviewed and merged if approved during the 1.14.7 release process. Repeatedly saying "submit the review and approve it" isn't going to hasten that or change procedure.


How to read SocketAddress from ByteBuffer?

I have a ChannelInboundHandler, and want to read ip address from the ByteBuffer, can someone give me some code conduct?

Can the IP address be IPv4 or IPv6, or must it be one or the other? What format is it in: packed binary, dotted-decimal, colon-separated IPv6?

@Lukasa
+------+----------+----------+
| ATYP | BND.ADDR | BND.PORT |
+------+----------+----------+
|  1   | Variable |    2     |
+------+----------+----------+
ATYP
1: IPV4, follow 4 bytes
4: IPV6, follow 16 bytes

Ok, those are packed representations. I am assuming network byte order here.
What data type would you like to end up having? SocketAddress?

Yes, I want SocketAddress type

I haven't tested it, but this code should work:
import NIO

enum PackedSocketAddressError: Error {
    case invalidTypeByte
}

extension ByteBuffer {
    mutating func readPackedRepresentation() throws -> SocketAddress? {
        guard let type = self.readInteger(as: UInt8.self) else {
            return nil
        }

        switch type {
        case 1:
            guard let packed = self.readInteger(as: UInt32.self),
                  let port = self.readInteger(as: UInt16.self) else {
                return nil
            }
            var addr = sockaddr_in()
            addr.sin_addr = in_addr(s_addr: packed)
            addr.sin_family = sa_family_t(AF_INET)
            addr.sin_port = port.littleEndian
            return SocketAddress(addr, host: "")
        case 4:
            guard let packed = self.readSlice(length: 16)?.readableBytesView,
                  let port = self.readInteger(as: UInt16.self) else {
                return nil
            }
            var addr = sockaddr_in6()
            addr.sin6_family = sa_family_t(AF_INET6)
            addr.sin6_port = port.littleEndian
            withUnsafeMutableBytes(of: &addr.sin6_addr) { $0.copyBytes(from: packed) }
            return SocketAddress(addr, host: "")
        default:
            throw PackedSocketAddressError.invalidTypeByte
        }
    }
}


chore: update comments

Types

[x] 🧹 Chores

Background or solution
close: #808
Changelog

Codecov Report

Merging #841 (f690d88) into main (83b7588) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head f690d88 differs from pull request most recent head 6512a56. Consider uploading reports for the commit 6512a56 to get more accurate results

@@           Coverage Diff           @@
##             main     #841   +/-   ##
=======================================
  Coverage   58.59%   58.60%
=======================================
  Files        1213     1213
  Lines       75118    75118
  Branches    15633    15633
=======================================
+ Hits        44016    44020    +4
+ Misses      28342    28339    -3
+ Partials     2760     2759    -1

Impacted Files
Coverage Δ

packages/extension/src/hosted/ext.process-base.ts
0.00% <ø> (ø)

...sion/src/hosted/api/vscode/debug/ext.host.debug.ts
53.92% <100.00%> (ø)

...pi/vscode/debug/extension-debug-adapter-starter.ts
45.23% <100.00%> (ø)

...ages/file-service/src/node/file-service-watcher.ts
68.88% <0.00%> (-0.56%)
:arrow_down:

...erences/src/browser/preference-settings.service.ts
74.83% <0.00%> (+0.64%)
:arrow_up:

...kages/theme/src/browser/workbench.theme.service.ts
72.05% <0.00%> (+0.88%)
:arrow_up:

packages/monaco/src/browser/schema-registry.ts
71.11% <0.00%> (+2.22%)
: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 83b7588...6512a56. Read the comment docs.


Enable GraalVM metadata repository

This commit enables use of the GraalVM metadata repository by default. It changes how the files are copied when working on Docker images, because previously, we had a single directory of resources (the generated one), but now it is possible to have multiple directories. For example, if netty is present on classpath, we will need to copy the configuration files for netty.
In practice, when we prepare the docker build context, we need to scan those directories and copy them. It is however possible that several directories have the same name, because the directory names are derived from GAV coordinates, so the same version can appear multiple times.
In order to:

avoid overwriting configuration files for several modules which have the same version
make things easier to debug when reading the docker files

we add inference to figure out if a configuration directory comes from the metadata repository, in which case we wouldn't use only the directory name, but the 3 last directories which correspond to the GAV coordinates. For example, instead of a directory named 1.0.0, we would have a directory named org.company:artifact:1.0.0.

does this work for users behind a VPN now?

does this work for users behind a VPN now?

By default the plugin uses an artifact published alongside the plugin which is published on Maven Central and using regular dependency resolution, so it should work behind a proxy or VPN. It may not be the latest release of metadata available, though, in which case a manual URL needs to be configured.


Spotlight occlusion

This is semi-immersive experiences such as the skybox demo. users can still see there body and part of their environment while being immersed and another environment.

Moved both to their core/user/ as there might be more features we'll wanna add later.


allow tsetse to build on TS2.6

Diagnostics in TS 2.6 are a ReadonlyArray.
Related to angular/devkit#300
/cc @alexeagle @filipesilva

Can one of the admins verify this patch?


amendment to hip19 for light hotspots

First real amendment! @cokes518 @abhay do you think this necessitates having a whole separate file like this, or is it sufficient to just merge the change into the original? Could include a changelog inside original HIP or similar

I'd be fine with it being merged.

Closing in favor of #132 (right?)


Build and release if rdfa changes

The package should get a new patch release everytime the rdfa changes. This could be done by hand but I think that it would be better to have a cronjob/CI doing the job.
It could be discussed that it should be a minor release instead of a patch one. Otherwise it could be an idea to use composer hooks to generate the types during local package installation and on composer update.

https://github.com/github/hub#git-pull-request

Otherwise it could be an idea to use composer hooks to generate the types during local package installation and on composer update.

This sounds a bit scary to me because if the rdfa changes and removes something, your app could suddenly have type errors and pinning a specific version wouldn't solve the issue.
Doing this with a cron sounds like a good idea though 👍

If you need any help on this feel free to ping me :)

@atymic I've added a bash file #81 feel free to improve it!

@atymic I've pushed a fix in #86 feel free to check it.


:((((

im bummed out i just found out bout this im a huge C# fan i would of LOVED to learn boo but whats the point if its not being maintained.. its dead :((((

look at the PR's  , boo does not seem dead


WIP revisit valuation semantics

This PR introduces a revisited fundamental asset pricing formula.
The existing one did the following:
map a claim to an expression representing a stochastic process
make a model assumption, using geometric brownian motion as an elementary process
serialise the resulting formula to text
The revisited FAPF does only the first step: map a claim to a stochastic process representing its value at time t. The process primitives that are used are the ones outlined in the PJ paper: exch, disc, observable (in the future also snell and absorb.
Moreover, we now handle the propagation of acquisition time, such that
scale nodes that are evaluated before today are replaced with their numerical value
scale nodes that are unknown are mapped to the corresponding stoch. process
There are still a couple of things to iron out, specifically
implementation of snell and absorb primitive processes to handle anytime and until nodes (I think we can come to a better expression than what there is in the paper)
simplification functions
I would like to hear your feedback on this and also discuss whether we need support for printing out the formula in e.g. MathML like it's currently done for the existing fapf.
The tests can give you an idea of the output which is currently generated.

A nice POC would be to implement a bond pricer in e.g. Typescript that
maps the claim to a stochastic process (using CC)
applies model assumptions (e.g. deterministic interest rates)
evaluates the resulting expression to get the price of the bond

Good job!

The process primitives that are used are the ones outlined in the PJ paper: exch, disc, observable (in the future also snell and absorb.

The reason why we went for mathematical expressions over the PJ definitions is that there is less ambiguity; it doesn't require familiarity with the paper or what those exch, disc etc. mean.


mount-file.bash: be quieter when debugging is off

Tested quickly on NixOS only with this config.
Seems to work fine I think; for some reason the service output only sometimes got in my deploy log.

Thanks!


DMI Decode widget - parser failure

$ configctl dmidecode system:
Manufacturer = PC Engines
Product Name = APU
Version = 1.0
Serial Number = 1040830
Family = None Provided

Error (the second is only a follow up of the first one):
Warning: syntax error, unexpected TC_CONSTANT in Unknown on line 5
 in /usr/local/www/widgets/widgets/dmidecode.widget.php on line 31

Warning: Invalid argument supplied for foreach() in /usr/local/www/widgets/widgets/dmidecode.widget.php on line 38

Hardware: APU1d4

next week

https://github.com/opnsense/plugins/pull/1227

thanks, I've merged it.


Extension is not working anymore

Access to XMLHttpRequest at 'https://auctionoton-edge-api.vercel.app/api/item?id=42702&server_name=arugal&faction=alliance&amount=1' from origin 'https://www.wowhead.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

GET https://auctionoton-edge-api.vercel.app/api/item?id=42702&server_name=arugal&faction=alliance&amount=1 net::ERR_FAILED 404

@tstirrat Hi! That's strange. I can not reproduce the error and the requests are working for me. Are you still experiencing this issue?

No, it's pretty persistent. It's been broken for weeks:

I've tried disabling ad blockers etc.
I'll try in incognito mode perhaps.

I just tried it in incognito, to rule out any other extentions interfering. Still broken with the 404

Very strange. I'll see what I can do!

For some reason the extension on chrome extension store reverted to some older version of the code. No idea what happened there. I was probably running a developer version on my end, which is the reason I did not have encounter this issue until I reinstalled the extension from the chrome store today and encountered your same error.
I've pushed an update to Chrome's extension dashboard. This could take some time.

Thanks, it's working again! I really appreciate this extension. It saves me so much extra work!


Please update to Catch2 (2.13.5+)

The catch1 library/test system is unmaintained and not compatible with glibc v2.34+ anymore.
SIGSTKSZ is not a constant anymore as of glibc 2.34+ – Catch2 has been updated to handle such situations (https://github.com/catchorg/Catch2/issues/2178), while Catch1 remains unmaintained, which makes the tests of clasp fail to build on newer systems (e.g. Ubuntu 22.04).
Please update to a newer version of catch2 or – even better – make use of the system provided catch version, e.g. via their CMake integration, as discussed in https://github.com/potassco/libpotassco/issues/12
In file included from /<<PKGBUILDDIR>>/libpotassco/tests/main.cpp:19:
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp:6415:41: error: array bound is not an integer constant before ‘]’ token
 6415 |         static char altStackMem[SIGSTKSZ];
      |                                         ^
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp: In constructor ‘Catch::FatalConditionHandler::FatalConditionHandler()’:
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp:6434:30: error: ‘altStackMem’ was not declared in this scope
 6434 |             sigStack.ss_sp = altStackMem;
      |                              ^~~~~~~~~~~
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp: At global scope:
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp:6466:53: error: array bound is not an integer constant before ‘]’ token
 6466 |     char FatalConditionHandler::altStackMem[SIGSTKSZ] = {};
      |                                                     ^
make[4]: *** [libpotassco/tests/CMakeFiles/test_potassco_opts.dir/build.make:121: libpotassco/tests/CMakeFiles/test_potassco_opts.dir/main.cpp.o] Error 1
make[4]: Leaving directory '/<<PKGBUILDDIR>>/build.dir/hardening_mt'
make[3]: *** [CMakeFiles/Makefile2:260: libpotassco/tests/CMakeFiles/test_potassco_opts.dir/all] Error 2
make[3]: *** Waiting for unfinished jobs....
[ 31%] Building CXX object libpotassco/tests/CMakeFiles/test_potassco_lp.dir/main.cpp.o
cd /<<PKGBUILDDIR>>/build.dir/hardening_mt/libpotassco/tests && /usr/bin/g++-10  -I/<<PKGBUILDDIR>>/libpotassco/tests -I/<<PKGBUILDDIR>>/libpotassco -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. -flto=auto -ffat-lto-objects -fstack-protector-strong -Wformat -Werror=format-security -O3 -Wall -Wdate-time -D_FORTIFY_SOURCE=2 -DNDEBUG -Wdate-time -D_FORTIFY_SOURCE=2 -DNDEBUG -std=c++11 -MD -MT libpotassco/tests/CMakeFiles/test_potassco_lp.dir/main.cpp.o -MF CMakeFiles/test_potassco_lp.dir/main.cpp.o.d -o CMakeFiles/test_potassco_lp.dir/main.cpp.o -c /<<PKGBUILDDIR>>/libpotassco/tests/main.cpp
In file included from /<<PKGBUILDDIR>>/libpotassco/tests/main.cpp:19:
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp:6415:41: error: array bound is not an integer constant before ‘]’ token
 6415 |         static char altStackMem[SIGSTKSZ];
      |                                         ^
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp: In constructor ‘Catch::FatalConditionHandler::FatalConditionHandler()’:
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp:6434:30: error: ‘altStackMem’ was not declared in this scope
 6434 |             sigStack.ss_sp = altStackMem;
      |                              ^~~~~~~~~~~
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp: At global scope:
/<<PKGBUILDDIR>>/libpotassco/tests/catch.hpp:6466:53: error: array bound is not an integer constant before ‘]’ token
 6466 |     char FatalConditionHandler::altStackMem[SIGSTKSZ] = {};
      |                                                     ^

@slyon Thanks for the heads up.


subprocess.CalledProcessError: Command '[ ]' returned non-zero exit status 1

I am having an issue installing the pipeline more specifically conda. I have attached a screenshot of my terminal.  I am new to programming/bioinformatics so I am not well versed in troubleshooting. Please let me know if you have seen this error before. I really appreciate the help!

Hi @KTison !
It looks like the older system python3.6 interferes with bcbio python3.7.
Could you please try exporting PATH in the following way?
https://bcbio-nextgen.readthedocs.io/en/latest/contents/intro.html#install-bcbio-python-package-and-tools
Also this instruction might be useful:
https://bcbio-nextgen.readthedocs.io/en/latest/contents/installation.html#importerrors
SN

Hello @naumenko-sa!
Exporting the path as you advised definitely helped but now I have approached another issue. I am receiving a LinkError where the post-link script failed for package bioconda::deepvariant-1.4.0-py36hf3e76ba_0 (first screenshot). While handling this error there was another exception where a process error occurred (second screenshot). Any suggestions for these issues?

Hi @KTison !
Have you attached the screenshots? I don't see them.
Sergey


Ability to set default history time to live value in the modeler via flag in flags.json

Problem you would like to solve
Desktop modeler recent version has default History clean up "Time to live" configuration set to 180 days, we had default "historyTimeToLive" set to 30 days in our engine configuration(bpm-platform.xml) but currently due the default value in the recent version of the modeler the engine configuration will be overwritten.
Can we enhance the modeler to add flag to indicate the default historyttl value like we do for disable-zebee, c7-engine-version etc

Proposed solution
Implement flag to set default-history-ttl setting
Alternatives considered
Ask every modeler to change the value to 30 but we have thousands of processes across our organization
Additional context
Flag to be named as default-history-ttl

Sounds reasonable to have a flag to configure this. Do you want to contribute this?

Yes I can contribute, might need some help with direction. thank you.

Superceeded by https://github.com/camunda/camunda-modeler/issues/4062.


Hardcode name-based ban for twitter.com/h0nde

Looks good.

await does not belong here, and hard-coding is bad.
Also this does not belong in MinetestBot. https://github.com/MinetestBots/dungeonmaster has an implementation for this now.


Remove unnecessary files, change ga to modify score.py for lab

Change log

remove template folder, update download link in ga
suggestion: using sed replace score for lab

hello @csinrn
I saw what score_lab.py do, and that's a great update in ga~
But I slightly replace the thought with sed.
So that it's no need to create score_lab.py but modify the score in score.py directly for lab.
You can checkout Files changed button to see what lines I modified.
And if you think it's ok, then u can press the Merge pull request XD
Thanks!

hello @WesleyCh3n
I've viewed all the commits. It's a great adjust, and it do make the whole project more concise and improve the overall readability.
It will definitely get 30 out of 30 in the Design/Readability assessment. 😆
Thanks a lot!


version mismatch between packages

Qowaiv is updated to version 3.2.5 but the other packages are still referring to version 3.2.1

Done. Sorry for the inconvenience caused.


Integarte Idsvr4 with web portal by oidc via Https

Oidc works fine on my local machine with Http that be hosted in Kestrel. But run into a lot of issues when switch to Https.
When I deploy the Idsvr4 and web portal on same sever and use Https, the web portal need to redirect to Idsvr to do sign-in, I get the error is "unauthorized_client".
Error logs:
Invalid redirect_uri: http://clientportal/signin-oidc
{
"ClientId": "grapeseed.dashboard",
"ClientName": "GrapeSeed Dashboard",
"RedirectUri": "http://clientportal/signin-oidc",
"AllowedRedirectUris": [
"https://clientportal/signin-oidc"
],
"SubjectId": "anonymous",
"RequestedScopes": "",
"Raw": {
"client_id": "dashboard",
"redirect_uri": "http://clientportal/signin-oidc"
"response_type": "code id_token",
"scope": "openid profile basicinfo",
"response_mode": "form_post",
"nonce": "636203912324322100.ZmQ5MTk5MTAtZGY3Ni00MjFkLWIyOGUtODhkOGVkY
"state": "CfDJ8G9vLBoK3ENEk2u8hnV9DK0OcF8dAi3RKfbg2NZosKw6Gab2oPcR_WKx
}
}
I wondering why could the "https" be converted to "http" eventually.
There are less info about how to integrate IdentityServer4 with Client web application via Https.
Can you provide one example for it.

If the client is at an https address then you should specify https redirects in your IS4 client setup and in your oidc settings.  "redirect_uri": "https://clientportal/signin-oidc"

Thanks, I've set https for all places, in this case, it will be converted to http implicitly.

all set on this issue?

I've followed this way to rewrite the host header on Idsvr side and portal side:
https://github.com/IdentityServer/IdentityServer4/issues/213
app.Use(async (context, next) =>
{
context.Request.Scheme = "https";
HostString httpsHostString = new HostString("xxx");
context.Request.Host = httpsHostString;
await next.Invoke();
});
Current issue has been solved. But when the authorize finished, the page will keep loading and cannot redirect to the "http://clientportal/signin-oidc". Eventually it will show the "504 Gateway Time-out" page.

Maybe it's .net core's bug, can you give me some info to help, thanks.

Open a new issue for new topics, please.


fix: hub spoke is always proposed directly finalize it

Description

Hub spoke is always proposed directly finalize it
Type of change

[ ] Docs change / dependency upgrade
[ ] Configuration / tooling changes
[ ] Refactoring
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires changes in customer code

High-level change(s) description - from the user's perspective

Related Issue(s)
Fixes
Related pull request(s)

Resolved
                                root                                | domain  |  status   |      to_timestamp
--------------------------------------------------------------------+---------+-----------+------------------------
 0x9838c44f57a6d2c86077ebd762171c0e56237a3d18842d301c7c59cb47a3f3a8 | 6648936 | Finalized | 2023-12-22 01:53:59+00
 0x74072f048da0acb0cd828e5ddb4ee8e49e96c65a234fba43e37fa335576f76c0 | 6648936 | Finalized | 2023-12-22 00:53:59+00
 0x8a977df4c1873101969e296602869883b14fc67810a33a1bd9ac7c2c08470104 | 6648936 | Finalized | 2023-12-21 22:24:59+00


Crashed iOS18 - Iphone16 and Ipad Air13

Crashed in new iOS18, methods connect and disconnect.
Error: pusher_channels_flutter/SwiftPusherChannelsFlutterPlugin.swift:174: Fatal error: Unexpectedly found nil while implicitly unwrapping an Optional value

@mostafanaguib2016
fix in
https://github.com/NaN-GeeK/flutter_pusher/commit/bf627181bb0a7716106e206290abf0823ecdd05d


Better pagination UX

Some commands (ie: tag ls and repo ls) use paginated hub APIs. Right now it prints only the first page, and rely on a --all flag to let user print all the pages.
We should at least print a message to let the user know there's more items to show.
Another way could be something more interactive a-la-git, piping each page to LESS.

I wanted to create an issue but I'll add on to this one instead as it's semi related. When using the SDK (just importing to the github project in go), it seems that there is no way to either change the current page, or even limit the amount of items per page. This forces me to put pressure on the API because I am pulling data that I do not need. Instead of pulling the latest 5 tags, I am pulling all the latest tags. If there is a way around it, that's great however it should be documented as I don't want to be hitting rate limits.


How to put custom dates on my calendar. I can't figure it out!

IMPORTANT - BEFORE CREATING YOUR ISSUE PLEASE READ THE FOLLOWING:

PLEASE STATE THE VERSION NUMBER you are using.
Are you using the latest version of JTAppleCalendar? Latest version is currently 7.1.5
PROVIDE ENOUGH INPORMATION SO I CAN RECREATE THE PROBLEM.

Have you looked at any of the videos?

yes. all
On Thu, Mar 1, 2018 at 7:53 PM, JTAppleCalendar<EMAIL_ADDRESS>wrote:

Have you looked at any of the videos?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/patchthecode/JTAppleCalendar/issues/730#issuecomment-369812587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfZW0PULTfUS6gA9pLzHe5tiz_1AKxMQks5taMJEgaJpZM4SYsxG
.

So which part are you stuck on?
got a sample app on github i can take a look at?

closing issue until user responds.


Fix windows line ending issue with Docker

Git on windows automatically converts line endings to CRLF on checkout,
which makes these patch files not readable inside Docker. This should
(hopefully) exclude the patch files from this and fix the problem on
Windows though since I don't have a Windows machine I haven't actually
tested it.

cc @bcai2 let me know if this fixes the problem for you.

cc @bcai2 let me know if this fixes the problem for you.

Windows issues when cloning with CLRF (at least those of which I've found so far):

Attempting to install patches (the RUN yarn install in the Dockerfile) breaks when building. One fix was to convert the patch files' line endings and replace the install line with the following, applying patches individually:
RUN patch -p1 -i patches/patch-package+6.2.2.patch
RUN patch -p1 -i patches/react-table+7.6.2.patch

Building the web applet sometimes fails with exit code 1. Googling suggests that this is another line ending issue, but I can't replicate the error (or find the page I was looking at...).

All of these seem to be circumvented by preventing automatic line conversion, which can be done on Windows by setting core.autoclrf to input (clones as LF, converts CRLF to LF upon committing) or false (no modifications when cloning or committing, which). This can be done with git config --global core.autocrlf true.
Alternatively, adding a few lines to a .gitattributes file should also do the trick for any future Windows users (which just prevents line conversion while cloning).

Windows issues when cloning with CLRF (at least those of which I've found so far):

Attempting to install patches (the RUN yarn install in the Dockerfile) breaks when building. One fix was to convert the patch files' line endings and replace the install line with the following, applying patches individually:
RUN patch -p1 -i patches/patch-package+6.2.2.patch
RUN patch -p1 -i patches/react-table+7.6.2.patch

Building the web applet sometimes fails with exit code 1. Googling suggests that this is another line ending issue, but I can't replicate the error (or find the page I was looking at...).

All of these seem to be circumvented by preventing automatic line conversion, which can be done on Windows by setting core.autoclrf to input (clones as LF, converts CRLF to LF upon committing) or false (no modifications when cloning or committing, which). This can be done with git config --global core.autocrlf true.
Alternatively, adding a few lines to a .gitattributes file should also do the trick for any future Windows users (which just prevents line conversion while cloning).

Yeah this PR adds .gitattributes and should (I hope) fix the patch file issues. I'm wondering what other files need to be added to it.

Yeah this PR adds .gitattributes and should (I hope) fix the patch file issues. I'm wondering what other files need to be added to it.

I'm not sure anything outside of that needs to be added. If any issues come up for me in the future, they're most likely not Windows-exclusive. I'll update if I do find anything else, though.

I'm not sure anything outside of that needs to be added. If any issues come up for me in the future, they're most likely not Windows-exclusive. I'll update if I do find anything else, though.


Use #pragma once

I think, Include guard
#ifdef MY_HEADER_H
#define MY_HEADER_H
#endif // MY_HEADER_H
is more portable and stable then #pragma once, and #pragma once is not in standard.

I agree with gilmanov-ildar that include guard is better solution


"In the browser" installation shows wrong code

Hey,
When you click on In the browser on the installation page, you can see a code snippet under point 3 (usage).
The js file loaded in this snippet (<script<EMAIL_ADDRESS>isn't working in Safari 10.0.2.
If I visit babel/babel-standalone directly, there is another js file included in the usage example. This (<script src="https://cdnjs.cloudflare.com/ajax/libs/babel-standalone/6.18.1/babel.min.js"></script>) one works.
I assume, the script tag in the browser usage.md file should be the same as from babel/babel-standalone
I hope this makes sense, otherwise I'm happy to explain better 😁

Is the second link working on Safari? Both links are correct.

I can't replicate it anymore 🤔
Same code as before and just exchanged the script tag, but works now with both scripts.

Ok cool :)
I close this issue.


Allow outer gaps to be set individually

Please allow at least the vertical and horizontal outer gaps to have their own variables, when not set they simply default to the gaps_out value.
This will be very helpful to bind horizontal gap toggles for those who have wide monitors. Things like browsing/reading in a single 2560px wide window gets uncomfortable real quick.

The monitor setup worked out of the box so I skipped its docs, my bad. That should do the trick, thanks!

still not a solution since no_gaps_when_only will not work

@MightyPlaza indeed, but I don't mind changing it manually for now. Later I will try to write a script to achieve a similar behavior and perhaps share it here if successful.


3d terrain support

Is 3d terrain supported by gl-native?
 I ask because 3D terrain seems to come to js https://github.com/maplibre/maplibre-gl-js/pull/165

@AAverin
They'd be entirely separate and separate implementations. There's almost no shared code (native is C++ with some native libraries and GL JS is JS). I'm not aware of anyone working on 3D terrain for native.

This would be really great!

I will close this for now. Feel free to continue discussing Terrain3D in #252


TypeError: function missing required argument 'initial_buffer_values' (pos 2)

Hi,
I am trying to downgrade iPhone 6s from 15.7.9 to 14.2 using downr1n.
However I get this error every single time I try it.t

I have no clue of what to do since, no one had this problem when I checked issues section of this repo.
Thanks

in an attempt to simplify for anyone having the issue of SystemError: initial_buffer_values macro must be defined for '#' formats, please try the following 3 commands:
sudo pip uninstall lzss
sudo git clone https://github.com/m1stadev/pylzss && cd pylzss
sudo python3 setup.py install
and then re-run dualra1n,

Hi,
I have tried the above commands but no success. Shows the same error.

initial_buffer_values
is it showing th same error initial_buffer_values ?

initial_buffer_values
is it showing th same error initial_buffer_values ?

Yes

sudo pip uninstall lzss sudo git clone https://github.com/m1stadev/pylzss && cd pylzss sudo python3 setup.py install
and then re-run dualra1n,

sudo pzthon3 -m pip uninstall lzss
sudo git clone https://github.com/m1stadev/pylzss && cd pylzss
sudo python3 setup.py install
and then re-run dualra1n, this is a new commands please try it if you want to fix it

Tried the above, but still doesnt work. Shows the same error.

Take a look.

Take a look.

only execute sudo python3 -m pip uninstall lzss, and then execute downr1n again

if it doesn't work check your distro

Changed distro and after running all required commands it works. But got a new error. Please look into this:

try without sudo

Tried it still got the same error. Any other fix.

Tried it still got the same error. Any other fix.

if you have much storage, you could use dualra1n

No, don't have much. That is the reason Im using this. Is there a fix or something? Or do I give up?

No, don't have much. That is the reason Im using this. Is there a fix or something? Or do I give up?

if you are on iOS 15 below please use dualra1n with --downgrade argument instead of --dualboot argument

Im on 15.7.9

https://github.com/dualra1n/dualra1n.git then you can use it. put the your .ipsw file into ipsw/ dir, then execute ./dualboot.sh --downgrade 14.3 (or the vers to downgrade).

fixed, execute sudo python3.9 -m pip uninstall pylzss lzss, and then downr1n command again


How to use swiper in touch (mobile) when I want to "Pinch"(two fingers to zoom) and "move"(one finger to move around)

How to use swiper in touch (mobile) when I want to "Pinch"(two fingers to zoom in) and "move"(one finger to move around) with picture
I found the swiper api will change the pages ,but not the move action ,when I already zoom in the pictrure and I want to see different place of the picture.
The zoom in is support by web broswer and it can be disable by
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no"  />
but I don't have any option to alter this function(zoom in)

I use a baidu touch javascript library to solve this problem.
There are their website http://touch.code.baidu.com .

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Images for future emojis?

I have created a couple emojis that are not in any specification yet:
Panther: (derived from leopard)

Table: (derived from chair)

What's your opinion on expanding the emojis to basically include a picture for every common noun?
If the purpose of twemoji is to be used in all sorts of apps and games, this might be more useful than just limiting it to emojis in some arbitrary specification. For example, I am using twemoji as a way to turn text into illustrations so I need more than the current set.
We could place them in a custom-emoji folder.
Thoughts?

Added in the link : [https://github.com/HaNerW10/Custom-Emojis-Twemoji-TTF/releases/tag/<IP_ADDRESS>](added ttf cett)

This has been mentioned multiple times in issues as well as in the README: if you want emoji that aren't in the Unicode spec, you need to propose it to the Emoji Subcommittee, not here. Luckily submissions are open now for 17.0 so it's a good time for proposals over there!
We will never add non-standard emoji here. It just won't happen. We're a library that implements the standard only.


pkg/query: Add more detail to data retrieval tracing

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Add matchers, min and max time as span tags for data retrieval of the querier.
Verification
n/a as instrumentation
@bwplotka @squat @metalmatze @GiedriusS @povilasv

This is just adding more context to the spans to follow, correct?

correct


[VS Code] The user code is invalid or expired. Try signing in again.

Error:
The user code is invalid or expired. Try signing in again.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.788

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.28.0

platformName
Linux

platformVersion
4.15.0-36-generic

@gryffondor : Are you able to sign in again?

bonjour,
Yes I managed to sign in this morning
Le jeu. 11 oct. 2018 à 19:52, Nandini<EMAIL_ADDRESS>a écrit :

@gryffondor https://github.com/gryffondor : Are you able to sign in
again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/1114#issuecomment-429055050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ag2ogqn3XcgdUoV0ZMJRDEGOSyagTjqeks5uj4VggaJpZM4XXnYq
.

We're tracking this class of issues with #323. I'm going to close this bug as a dupe. If you hit the issue again, can you please run the command "Live Share: Export Logs" and drag-and-drop the zip here or send it to<EMAIL_ADDRESS>


Why are WS/SEE not being closed after turbo-stream-source is gone?

I am using Hotwired/Turbo in a go application.
On one specific page, I want to update several partials from the server asynchronously as they arrive to the server through external apis (openAI in this case).
I used this on my views:
<turbo-frame id="articles_list">
...
<turbo-stream-source src="/repurpose/{{ .ID }}/"></turbo-stream>
...
</turbo-frame>

It normally works as expected, and I can update the page nicely.
But the moment that I navigate to another page (so the turbo-stream-source gets unloaded), the ws/see connection is ignored but not closed.
https://github.com/hotwired/turbo/blob/0826b8152c0e97f19d459c1a1c364fa89cc62829/src/elements/stream_source_element.ts#L14-L16
This creates unlimited connections until I reach the browsers max, and can't navigate anymore.
I fixed it locally by modifying the source and manually closing the connection with this.streamSource.close();:
class StreamSourceElement extends HTMLElement {
...
    disconnectedCallback() {
        if (this.streamSource) {
            disconnectStreamSource(this.streamSource);
            this.streamSource.close();
        }
    }
...
}

The change seems obvious for my use case, but makes me wonder if I am using hotwire turbo as design.
Are only fixed constant connections like /cable in rails the only recommended use case?

Thank you for opening this issue, and for sharing your solution.
I've opened https://github.com/hotwired/turbo/pull/970 to resolve it.

is this issue causing response on network tab which keep receives {"type":"ping","message":Id}, even after we move to another page which doesn't have turbo-stream-source tag in it? if yes, it's still happening right now cuz i just upgrade mine to v8 @seanpdoyle @guillermo
rails 7.1.3
turbo 8.0.0
turbo-rails 2.0.0


Added ability to set device name for tplink component in configuration.yaml

Description:
added ability to use name in configuration file as an optional attribute to switch and lamp.
entity_id is based upon device name, so this should make the tplink component more backward compatible with the old 'platform' method of setting up tplink in configuration.yaml.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
have yet to get to docs
Example entry for configuration.yaml (if applicable):
tplink:
  discovery: false
  switch:
    - host: <IP_ADDRESS>
      name: Porch Switch
    - host: <IP_ADDRESS>
  light:
    - host: <IP_ADDRESS>
      name: Basement Lamp 1

Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass (yes, lots of errors are tossed that I know nothing about and are not related to the component i changed. so "yes?"
[X] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @marc-gist,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

seems as if a minor pylint syntax failed on the tests 😦 ... I have corrected in my code,  what do I do now? Do i push my changes? let the reviewer first comment/fix?
Thanks.

As changes are pushed to your branch, the PR will be updated and Travis will re-run. Feel free to update so tests pass.

We should not merge this. Name customizations are done via the entity registry in the UI. This is available when the device has a unique ID.

We should not merge this. Name customization are done via the entity registry in the UI. This is available when the device has a unique ID.

Why is it not ok to have multiple ways to do a thing? and I did this to allow for automatic entity_id setting, which is done via the Name.  Removing this "feature", as 0.89 did was a breaking change, which i get happens and is needed at times, but in this case, for when people are setting up a large number of devices, it is much simpler, cleaner, easier, etc, to do in the config.  I can easily convert my v88 configuration file to meet v89 and not have to change my automations, or manually go through each and update the via the UI.
@balloob do you own a number of tp-link devices?

For example; here is a current 0.88 config entry
- platform: tplink
  host: <IP_ADDRESS>
  name: "Stereo Booth Speaker"
  scan_interval: 60

i just remove the platform and scan_interval lines and with my CR everything comes back up in 89 with correct entity_id's

Agreed, this should definitely be added back in.  For people like myself and many others with lots of TP-Link devices already working and defined, this flat out breaks our current setups.  There is no easy way to upgrade without breaking things and then working to fix automations, etc.  Especially when adding it causes no ill effects.

This was discussed extensively on Discord, but it will not be added back. TP Link devices have unique IDs and so the way to do this is to edit the entity registry. Configuration values will be ignored after the first boot. It will not be added back.


make install dont link to libcpuinfo.so

the libnnpack is build with cmake -G "Unix Makefiles" -D BUILD_SHARED_LIBS=ON -DNNPACK_LIBRARY_TYPE=shared  -DCMAKE_POSITION_INDEPENDENT_CODE=ON ..
after make the libnnpack.solinked to dependent lib correctly.
but result after make install  show the following
  install git:(master) ✗ ldd lib/libnnpack.so
  linux-vdso.so.1 (0x00007fff515e2000)
  libcpuinfo.so => not found
  libpthreadpool.so => not found
  libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f83221e1000)
  libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f8321fef000)
  /lib64/ld-linux-x86-64.so.2 (0x00007f8322243000)

fix with https://unix.stackexchange.com/questions/279397/ldd-does-not-find-path-how-to-add


SSL Connect Errors

We just upgraded to chef 12 and using chefdk 0.6.2-1. When issuing any knife ec2  commands I get the following error.
ERROR: Excon::Errors::SocketError: SSL_connect returned=1 errno=0 state=SSLv2/v3 read server hello A: unknown protocol (OpenSSL::SSL::SSLError).

@dannyybarra is this issue resolved?  Can you try using latest Chefdkv0.19.6 ?

Generally with errors like this we need to see the output of knife server create ARGUMENTS -V -V to be able to debug the issue.

@dannyybarra, I hope above comments help you resolve issue. Please feel free to re-open if you still get errors. Thanks.


[Error]: Update to 1.19.2 please? And what is fabric-tag-extention?

Minecraft version(s)
Other (specify below)
Minecraft version details
1.19.2
Mod loader
Fabric
Mod loader version

Fabric API version

In what environment did the problem occur?
Other (specify in "What went wrong?" section)
What went wrong? (Crash logs don't go here)

Full list of installed mods

Did the problem cause the game to crash?
Yes. (paste crash report below)
Full crash report contents (if you experienced a crash)

I can't found it anywhere to download.

Resolved as of Towelette's 5.0.0 release.


BUG: read_parquet with filters doesn't handle indices correctly

Modin version checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest released version of Modin.

[X] I have confirmed this bug exists on the main branch of Modin. (In order to do this you can follow this guide.)

Reproducible Example
import pandas
import pyarrow.dataset as ds

my_data = pandas.DataFrame({'a': range(10), 'b': range(10, 20)})

import pyarrow as pa

ds.write_dataset(pa.Table.from_pandas(my_data), './test_parquet_multiple_files/', format='parquet', max_rows_per_group=2, max_rows_per_file=3)

import modin.pandas as mpd

df = mpd.read_parquet(
    './test_parquet_multiple_files/',
    filters=[('a', '>=', 1)],
)
print(df)

Issue Description
This is a bug in my own contribution in #6442.
The current implementation assumes that if the dataframe has a RangeIndex, each partition will have a RangeIndex. However, this is not the case if a single partition contains chunks from multiple files -- the index will restart, and therefore become a regular integer index.
Expected Behavior
>>> pandas.read_parquet('./test_parquet_multiple_files/', filters=[('a', '>=', 1)])
   a   b
0  1  11
1  2  12
2  3  13
3  4  14
4  5  15
5  6  16
6  7  17
7  8  18
8  9  19

A normal RangeIndex is created.
Error Logs

Traceback (most recent call last):
  File "~/src/modin/test_parquet.py", line 20, in <module>
    df = mpd.read_parquet(
  File "~/src/modin/modin/utils.py", line 475, in wrapped
    return func(*params.args, **params.kwargs)
  File "~/src/modin/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "~/src/modin/modin/pandas/io.py", line 297, in read_parquet
    query_compiler=FactoryDispatcher.read_parquet(
  File "~/src/modin/modin/core/execution/dispatching/factories/dispatcher.py", line 195, in read_parquet
    return cls.get_factory()._read_parquet(**kwargs)
  File "~/src/modin/modin/core/execution/dispatching/factories/factories.py", line 213, in _read_parquet
    return cls.io_cls.read_parquet(**kwargs)
  File "~/src/modin/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "~/src/modin/modin/core/io/file_dispatcher.py", line 159, in read
    query_compiler = cls._read(*args, **kwargs)
  File "~/src/modin/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "~/src/modin/modin/core/io/column_stores/parquet_dispatcher.py", line 725, in _read
    return cls.build_query_compiler(
  File "~/src/modin/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "~/src/modin/modin/core/io/column_stores/parquet_dispatcher.py", line 601, in build_query_compiler
    index, sync_index = cls.build_index(
  File "~/src/modin/modin/logging/logger_decorator.py", line 128, in run_and_log
    return obj(*args, **kwargs)
  File "~/src/modin/modin/core/io/column_stores/parquet_dispatcher.py", line 563, in build_index
    start = index_objs[0].start
AttributeError: 'Index' object has no attribute 'start'. Did you mean: 'sort'?

Installed Versions

INSTALLED VERSIONS
commit              : ba1cccd19da778f0c3a7d6a885685da16a072870
python              : 3.10.12.final.0
python-bits         : 64
OS                  : Linux
OS-release          : 5.4.0-135-generic
Version             : #152-Ubuntu SMP Wed Nov 23 20:19:22 UTC 2022
machine             : x86_64
processor           : x86_64
byteorder           : little
LC_ALL              : None
LANG                : en_US.UTF-8
LOCALE              : en_US.UTF-8
pandas              : 2.1.0
numpy               : 1.25.1
pytz                : 2023.3
dateutil            : 2.8.2
setuptools          : 68.0.0
pip                 : 23.2.1
Cython              : None
pytest              : 7.4.0
hypothesis          : None
sphinx              : 7.1.0
blosc               : None
feather             : 0.4.1
xlsxwriter          : None
lxml.etree          : 4.9.3
html5lib            : None
pymysql             : None
psycopg2            : 2.9.6
jinja2              : 3.1.2
IPython             : 8.14.0
pandas_datareader   : None
bs4                 : 4.12.2
bottleneck          : None
dataframe-api-compat: None
fastparquet         : 2022.12.0
fsspec              : 2023.6.0
gcsfs               : None
matplotlib          : 3.7.2
numba               : None
numexpr             : 2.8.4
odfpy               : None
openpyxl            : 3.1.2
pandas_gbq          : 0.15.0
pyarrow             : 12.0.1
pyreadstat          : None
pyxlsb              : None
s3fs                : 2023.6.0
scipy               : 1.11.1
sqlalchemy          : 1.4.45
tables              : 3.8.0
tabulate            : None
xarray              : None
xlrd                : 2.0.1
zstandard           : None
tzdata              : 2023.3
qtpy                : 2.3.1
pyqt5               : None

By the way, with regard to the expected behavior, from a bit of investigation I have determined that the two parquet engines, pyarrow and fastparquet, behave a little differently with respect to the step attribute of the RangeIndex.
import pandas
import pyarrow.dataset as ds

my_data = pandas.DataFrame({'a': range(10), 'b': range(10, 20)})
# Non-default step
my_data.index = my_data.index * 2

import pyarrow as pa
ds.write_dataset(pa.Table.from_pandas(my_data), './test_parquet_multiple_files_step/', format='parquet', max_rows_per_group=2, max_rows_per_file=3)

print('With pyarrow:')
print(pandas.read_parquet('./test_parquet_multiple_files_step/', filters=[('a', '>=', 4)]))

print('With fastparquet:')
print(pandas.read_parquet('./test_parquet_multiple_files_step/', filters=[('a', '>=', 4)], engine="fastparquet"))

outputs:
With pyarrow:
   a   b
0  4  14
1  5  15
2  6  16
3  7  17
4  8  18
5  9  19
With fastparquet:
    a   b
0   3  13
2   4  14
4   5  15
6   6  16
8   7  17
10  8  18
12  9  19

That is, they both start the RangeIndex at 0 regardless of filters, but fastparquet preserves the step, while any filters in pyarrow reset the step to 1. Is this a quirk that Modin would want to replicate in its read_parquet method?
Without filters, of course, they both preserve step. Also note that the different numbers of rows in the outputs is a known difference: fastparquet does not implement filtering within a row group.

@zmbc thanks for reporting! Would you be able to work on a fix for that, too?

Is this a quirk that Modin would want to replicate in its read_parquet method?

Modin strives to be 100%-compatible with pandas, so we should try to replicate whatever quirks pandas has, unless it's too burdensome...

@vnlitvinov Yes, I will work on a fix. It turns out that fastparquet and pyarrow have one additional quirk with regard to indices, but it shouldn't be too much of a burden to get the exact same behavior.


IndexError: list index out of range

Tested with Python Versions:
python2.7
python3.5
python3.6
Latest commit:
python ./acme_tiny.py --account-key /var/lib/letsencrypt/<redacted>.com.account.key --csr /var/lib/letsencrypt/<redacted>.com.domain.csr --acme-dir /var/www/redacted.com/acme-challenge/ > signed.crt                                        Parsing account key...
Parsing CSR...
Found domains: *.<redacted>.com, <redacted>.com
Getting directory...
Directory found!
Registering account...
Already registered!
Creating new order...
Order created!
Verifying <redacted>.com...
Traceback (most recent call last):
  File "./acme_tiny.py", line 198, in <module>
    main(sys.argv[1:])
  File "./acme_tiny.py", line 194, in main
    signed_crt = get_crt(args.account_key, args.csr, args.acme_dir, log=LOGGER, CA=args.ca, disable_check=args.disable_check, directory_url=args.directory_url, contact=args.contact)
  File "./acme_tiny.py", line 131, in get_crt
    challenge = [c for c in authorization['challenges'] if c['type'] == "http-01"][0]
IndexError: list index out of range

Wildcard certificates are not supported by acme-tiny. Please use another ACME client for wildcard certs (you will also need the DNS-01 challenge for that, which is the main reason acme-tiny doesn't support wildcard certs).

Maybe this is a issue can avoid, it should support wildcard certs I think.

acme-tiny does not support the DNS-01 challenge, and so does not support wildcard certs (since DNS-01 is required for wildcard certs by Let's Encrypt).

Good new: I found a repo which support DNS-01 challenge, https://github.com/Neilpang/acme.sh , it is written by 'bash shell'.

Like @felixfontein said, wildcard certs aren't supported by acme-tiny because this script can't modify DNS entries.

Might it be worth giving a better error message in this case? I had to come here and search the issues before I realized what I was doing wrong…


Show medications with prescribed (preferred) name

[ ] Import prescribed name with medication data (currently only generic name)
[ ] Show medication instead of generic name as primary name
[ ] Generic name will be part of synonyms
[ ] If preferred name in synonyms, hide from list of synonyms
[ ] Should be adaptable by user (nice to have)

Will not implement this for the study; further thought (user research) needed, if reasonable at all


Issue and PR template

Per offline discussion we will create templates for opening issues as well as PRs that will help make things a bit more efficient.
For defect issues: Things like common debugging tips and data to include.
For PRs: Description of change, any extra testing recommended, regression risk, etc.
We can propose the templates here so people can comment.

Proposed PR Template:
Description
What does this PR do? If fixing a bug, please describe what the original issue is and how the change resolves it. If applicable, refer to the tracking issue.
Risk level
Low/Medium/High - explain
Testing Validation

[ ] Unit Tests
[ ] Integration Tests
[ ] Extra - Please explain: Stats, Smoke test, ...

Proposed Issue Template:
Description
What issue is being seen? If feature request, please describe desired behavior and what scenario it enables.
Repro Steps (Optional)
If applicable, please include configs, sample requests, etc...
Call Stack (Optional)
Please refer to Envoy Bazel documentation  for more information on gathering a stack.

A few comments:
For the PR template, testing validation section, I would make it clear that we don't expect all testing types in all PRs. I would make it more clear that people should think about different testing types depending on PR risk and to discuss why they did what they did.
For the issue template, can we be more specific, and split into issue requests and bug reports? For issue requests, we should ask for details on the feature being requested, links to standards if applicable, etc. For bug reports, we should be explicit and ask for repro steps, and minimally the config, /cluster, /stats, etc. We should eventually also link to the tool that @htuch is working on. In terms of optional call stacks, if it's a crash, the call stack should be required.

@htuch @dnoe @alyssawilk any comments on ^ ?

Thanks for adding the templates @ccaraman. I like the idea of an issue template. Here's some ideas of things we could ask for:

envoy-static --version
Execution environment (e.g. OS, distribution, etc.)
https://github.com/lyft/envoy/pull/1456 tarball (or logs, stats and other admin endpoints, in addition to the config). TBH, the most useful stuff I've seen so far when chasing bugs on GH is (1) logs gathered at -l trace level (2) stats from the admin endpoint (3) config JSON.

For the PR template, I'm actually a fan of not going too far down the path of a Markdown style template, but prefer to specify an ASCII format with mandatory tags. E.g.
<subsystem>: one line summary.

What does this PR do? If fixing a bug, please describe what the original issue is and how the change resolves it. If applicable, refer to the tracking issue.

risk-level: (low|medium|high)
testing: <explanation of unit and integration tests>
[optional deprecated: <description of what is deprecated>]
[optional release-notes: <description of what to add to next point release notes>]

[optional Fixes #issue]

Or something like that. It's nice to be able to prep the initial PR description in vi and to have some of this information expressed in tags, in particular for automating handling of deprecation and release notes. To do this, it would be good to have some mandatory validation of the commit message.

Also, on risk level, I think it would be good to explain what this means in more detail. Is this basically "risk of crashing/dropping/misdirecting production traffic"? Also, examples of what is expected for each level would be good. My naive guess would be low is most small changes or a new optional filter, medium is localized with heavy testing and high is really complicated things (e.g. flow control, replacing the gRPC transport, etc).

@ccaraman can we actually drop the templates into a public gdoc? I think it will be easier to iterate/comment.

how do you estimate risk level of a feature PT or bug fix PR?
may be have something like "feature w/ config, behavior change,bug fix, etc" ?

@rshriram It is hard to write a guide on how to estimate the risk level. The PR owner should have an idea of how risky the change is or collaborate with someone more familiar to understand the potential impact and go over testing strategies.

@mattklein123 @htuch  Here is the google doc with the template proposals. Please let me know if you can't access/comment on it.

Looks good, added some comments in the doc @ccaraman.

Done!


PWA cache images fetched from other domains

If possible, PWA/service worker cached the images (like Giphy gifs) fetched from other domains

Problem with Giphy:  https://stackoverflow.com/questions/59101707/workbox-cachefirst-network-request-error-with-giphy

Giphy solve in PR #50


Always retrying as the program is running

Hi, as i run the program on my mac, it always showing retrying.
At the beginning i thought it would be the proxy's problem. I tried to change another proxy website and the same problem occurred.
Does anyone know what is the real problem?

I think its not twitter-scraper if you didnt create logger for yourself.

I think its not twitter-scraper if you didnt create logger for yourself.

thanks for reminding me.


Fix nix flake compilation.

Description
Was no longer compiling, attribute gitIgnoreSource didn't exist, it was moved to lib.gitignoreSource.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: It works!

[ ] I updated the CHANGES.md file

Not sure that last checkbox is relevant here.

(Just a note for other maintainers who don't follow xmonad core PRs: https://github.com/xmonad/xmonad/pull/362#issuecomment-1009330181)

It appears as though the flake hasn't been updated on the CI, it's still reading from the old flake - and therefore failing to build. Not sure how you'd fix that, if you have access to the CI's box in any way, then nix flake update inside that box should fix it.

That's just because we haven't merged the PR to xmonad yet, I think.


Make ThreadContext.stashWithOrigin package-private

Description
The ThreadContext class overly exposes many methods as public. Currently, stashWithOrigin is publicly exposed. This PR changes the access modifier on this method to package-private to prevent this internally used method from being exposed publicly.
Related Issues
Related to https://github.com/opensearch-project/OpenSearch/issues/14931
Check List

[X] Functionality includes testing.
[X] API changes companion pull request created, if applicable.
[X] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I noticed that there is a backport label. Wouldn't this be breaking for 2.x?

@kotwanikunal I'm planning to close this PR and open a separate one which introduces a permission for this method similar to https://github.com/opensearch-project/OpenSearch/pull/15016
IMO I'd ultimately like to change the access modifiers for these methods to make these inaccessible outside of the package and I think its possible to do safely, but there are concerns around changing the access modifier.

Closing in favor of https://github.com/opensearch-project/OpenSearch/pull/15039


Authentication fixes

Refactoring the login process to use Auth0 SSO authentication, now used by linuxacademy.com
The branch also includes the download fix from @akopista.

Can this be merged?

This patch works well for me.  Here's another vote for getting it merged, please.

I also confirm that works for me. I think we can merge it ;)

Hello,
I have install the tool with the wiki help page
But now i don't know how to apply this fixe ?
Does someone can help me plz ?
Thanks

That also worked for me.

Hey @progerjkd can you help us? what needs to change to download the files from Linuxacademy.

I am facing below error any ideas where and why it is failing.?
Error message : "Username / Email : ###########
Password:
INFO     Logging in ...
INFO     Retrieving an authentication token. Please wait...
INFO     Authentication token acquired!
INFO     Analyzing ...
INFO     Logged Out!
ERROR    Unknown Exception
ERROR    u'contents'
Traceback (most recent call last):
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/cli.py", line 205, in main
la.analyze()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/linux_academy.py", line 218, in analyze
self._course.assets()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/course.py", line 80, in assets
syllabus = self._parse_syllabus()
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/course.py", line 76, in _parse_syllabus
sp.feed(self._fetch_syllabus())
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/parsers.py", line 62, in feed
super(SyllabusParser, self).feed(clean_html(data))
File "/usr/lib/python2.7/HTMLParser.py", line 117, in feed
self.goahead(0)
File "/usr/lib/python2.7/HTMLParser.py", line 163, in goahead
k = self.parse_endtag(i)
File "/usr/lib/python2.7/HTMLParser.py", line 401, in parse_endtag
self.handle_endtag(elem)
File "build/bdist.linux-x86_64/egg/linuxacademy_dl/parsers.py", line 102, in handle_endtag
self.__buffer['contents'].append(
KeyError: u'contents'
INFO     Please report this issue on https://github.com/vassim/linuxacademy-dl/issues. Make sure you are using the latest version. Be sure to call linuxacademy-dl with the --debug flag and include its complete output !"
And i am using latest version


Global variable changed to lower case in tags

It seems that the rule NormalizeTags (I guess) wants all the tags in lower case. Which is fine, but should not apply to variable names inside tags.
Here is the code I would like to keep:

Here is the code I get after running robotidy:

I will consider it as a bug and fix it - by trying to autodiscover and ignore any possible variables in tags.

Also if the Robocop complains about tags not being lowercase (with variable) it will need separate issue in Robocop repo: https://github.com/MarketSquare/robotframework-robocop/issues . As Robocop tool discover and Robotidy fixes issues.

With the fix, variables are now ignored inside NormalizeTags. I believe they should be formated with RenameVariables transformer - I will open issue for it.


GPG-Pictured-Key Signed Commits. Show Photo?

Gitea version (or commit ref): 1.10.3
Git version: 2.25
Operating system: Ubuntu x64
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Description
Hi, i'm committing with a gpg-picture-hard-coded key . I'm seeing all commits as Verified. Good!!
But i'm not seeing the picture of my signed commits, only sample image. It's normal?

I'm marking this as a feature request - so we can consider extracting the photo from the GPG key to use an avatar in future