fix: return correct operations from coin and message inputs

Closes https://github.com/FuelLabs/fuels-ts/issues/2763

Related to  https://github.com/FuelLabs/fuels-wallet/pull/1419

Release notes

In this release, we:

Fixed getInputFromAssetId so it returns the correct InputCoin and InputMessage

Summary

At the moment getContractCallOperations is currently returning empty operations on wallets that only have an InputMessage but no InputCoin , this is because getInputFromAssetId function was only returning messageInputs with the 32ZeroBytes hex, which of course excludes any input from a contract call that has a separate assetId such as a predicate call.
Notes
The best way to verify that this fix works is to use this branches build with the fuel-wallet and test it against the connectors. https://github.com/FuelLabs/fuels-wallet/issues/1415 goes into greater detail on how to reproduce the scenarios.
Checklist

[x] I added — tests to prove my changes
[x] I've tested this on https://connectors.fuel.network/ with the fuel-wallet
[ ] I updated — all the necessary docs
[x] I reviewed — the entire PR myself, using the GitHub UI
[ ] I described — all breaking changes and the Migration Guide

I think simulating this with the current infrastructure in fuel-gauge would be cumbersome, launchTestNode only allows for seeding with InputCoins as opposed to InputMessages and not varying flexibility with baseAssetIds
That being said I do think this is an appropriate use case for an e2e test with the testnet but we would need to find a way to mock bridged assets

launchTestNode supports generating messages.
    const testMessage = new TestMessage({ amount: 100 });

    using launched = await launchTestNode({ walletsConfig: { messages: [testMessage] } });

I am unsure if it supports seeding MessageCoin (spendable resource) or Message. If it does not support seeding MessageCoin, we should fix that.

cc @nedsalk

I am unsure if it supports seeding MessageCoin (spendable resource) or Message. If it does not support seeding MessageCoin, we should fix that.

@Torres-ssf It puts messages in the stateConfig.messages array and associates them with the wallets. I tried running getResourcesToSpend on a wallet with test messages while simultaneously excluding all the coins and it fails with not enough coins to fit the target. It's unclear to me how the message would be converted to a MessageCoin.

@Torres-ssf It puts messages in the stateConfig.messages array and associates them with the wallets. I tried running getResourcesToSpend on a wallet with test messages while simultaneously excluding all the coins and it fails with not enough coins to fit the target. It's unclear to me how the message would be converted to a MessageCoin.

@nedsalk I've just validate that it is possible to generate spendable messages. The difference between the 2 is that a MessageCoin has no data:
const spendableMessage = new TestMessage({ amount: 100_000, data: '' });

cc @maschad

const testMessage = new TestMessage({ amount: 100 });

    using launched = await launchTestNode({ walletsConfig: { messages: [testMessage] } });

Thanks for the confirmation @Torres-ssf I was unaware that's how a spendable message could be seeded. I've added a corresponding test in 2a5abeae1  I had to remove the zero amountPerCoin constraint as well so that we could simulate the scenario in the issue.


Simplify and expand CI pipeline

This new layout for CI drastically reduces the amount of boilerplate and makes it easy to expand support to more compilers.

Nice, I'll definitely take a look as soon as I find time

Thanks a lot, this looks much better :+1:


Strange error.

https://i.imgur.com/1QhNEgU.png
After doing the timeline leap, clicking no to check the spread, clicking calculate again, and then scrolling the results makes the program endlessly throw out errors. Not sure of the cause and can't check myself as my Visual Basic trial has expired and I don't intend on cracking it.

Seems it is fixed now, cause i can't replicate.

Closing, in case i'll reopen.


OCPBUGS-26140: Fix formatting in snyk vendor exclusions

Hello! This is a PR to add a snyk file to exclude the vendors folder while scanning the repo. Thanks!

/approve
/lgtm

/retest-required

/retest-required

Azure and GCP OVN tech preview jobs are both failing with this error:
{Operator degraded (AuthStatusHandler_FailedApply): AuthStatusHandlerDegraded: failed to create typed patch object (/cluster; config.openshift.io/v1, Kind=Authentication): .status.oidcClients: field not declared in schema  Operator degraded (AuthStatusHandler_FailedApply): AuthStatusHandlerDegraded: failed to create typed patch object (/cluster; config.openshift.io/v1, Kind=Authentication): .status.oidcClients: field not declared in schema}

That doesn't seem like a failure that's caused by this PR, but maybe it's something broken in the tech preview payload?

/retest-required

/retest-required

/override ci/prow/e2e-azure-ovn-techpreview
No point in running E2E for this PR

/override ci/prow/e2e-gcp-ovn-techpreview


calendar_by

Hi!
notion/collection.py
class CalendarView(CollectionView):
row 310 "query" maybe "query2"?
I tried to parse the calendar and couldn't do it until I changed "query" to "query2" in the file collection.py

Thanks for your comment!
I was having the same issue and this solved it! I honestly don't know why this fixes the issue, any idea?


It is necessary to initialize 'x' to 'undefined'.

See following simplified test
for( var i = 0; i < 10; i++ )
{
    var x = undefined;

    if( i % 2 === 0 )
    {
        x = null;
    }

    if( x === undefined )
    {
        console.log( i );
    }
}

the output is as one can easily guess:
1
3
5
7
9

jshint complains that:
test.js: line 3, col 9, It's not necessary to initialize 'x' to 'undefined'.
However,
for( var i = 0; i < 10; i++ )
{
    var x;

    if( i % 2 === 0 )
    {
        x = null;
    }

    if( x === undefined )
    {
        console.log( i );
    }
}

will produce on V8 (chrome and node)
only
1

As x will not be reset to undefined on every loop interation.
( jshint v2.4.2 )

It's a strange rule.

+1

Explicit initialize to undefined is useless since this is the default javascript behavior.
The problem here is that javascript do not have block scope.
x have to be initialize to undefined at each iteration.
The following code behave like expected and jshint do not complain:
var x;
for( var i = 0; i < 10; i++ )
{
    x = undefined;

    if( i % 2 === 0 )
    {
        x = null;
    }

    if( x === undefined )
    {
        console.log( i );
    }
}

I know, but my first code sample was also good, yet jshint falsely complains.

I know, but my first code sample was also good

This is the mistake: JSHint is identifying bad practices so that you can fix them. If you want to ignore JSHint's advice, tell JSHint that:
/* jshint -W080 */

My first example is NOT bad practice! Its perfect okay code! The only one at fault here is jshint

And -W080 turns of all setting to undefined errors, outside of loops its a good warning, inside its not.

+1 @axkibe
I agree with @axkibe: This is correct javascript code! Doing it or not is just be a mater of taste.
But fixing the code to be JSHint compliant is not a mechanical replacements.
JSHint do not advice. It is developper task to choose how to fix the code. The fact is that for inside loop code, the first idea breaks the code...
I can understand that it is a bad practice because of the default behaviour of variable declaration.
But since JS do not have block scope, it leads to write code a unnatural way, hoisting variable declaration just to avoid JSHint to complain.
It would be nice to also be able to use directives like /* jshint -W080 */ locally to a line and locally to a block of code to silent JSHint on a precise portion of code.
@rwaldron you say "JSHint is identifying bad practices so that you can fix them".
The goal of JSHint is not clear to me. We can read on JSHint web site

"This is JSHint, a tool that helps to detect errors and potential problems"
"JSHint is a program that flags suspicious usage in programs written in JavaScript"
I find all very interesting. But they should be dissociated :
Errors,
Potential problems,
Bad practices.

"JSHint is a program that flags suspicious usage in programs written in JavaScript"

Redeclaring a var binding within the for loop body is a "suspicious usage". If a reader was coming from another language, or was new to programming, this pattern would look like:

a new var binding for each loop turn
the var binding is scoped to the for loop body

Neither thing is true, which is suspicious.

hoisting variable declaration just to avoid JSHint to complain.

What's the point of using a code linting tool if you don't want to follow the guidance it provides?

It would be nice to also be able to use directives like /* jshint -W080 */ locally to a line and locally to a block of code to silent JSHint on a precise portion of code.

Do you mean like this:
/* jshint -W080 */
for (var i = 0; i < 10; i++) {
  var x = undefined;

  if (i % 2 === 0) {
    x = null;
  }

  if (x === undefined) {
    console.log(i);
  }
}
/* jshint +W080 */

Or more like this?
for (var i = 0; i < 10; i++) {
  /* jshint -W080 */
  var x = undefined;
  /* jshint +W080 */
  if (i % 2 === 0) {
    x = null;
  }

  if (x === undefined) {
    console.log(i);
  }
}

https://github.com/jshint/jshint/issues/1484

Having opened this issue a year ago, in the meantime my javascript coding style has anyway developed so I'm manually hoisting variables today. I did this like above when I was relatively new to it coming with many years experience in C++/Java/Lua etc who all do sane variable shadowing.
Anyway the issue with jshint is, it gives the coder the wrong guidance.
It literally says, you do not need to set the variable to undefined here while you have. It would guide you saying like "hey, you know, manually hoisting this would look cleaner"... okay, today I agree, I guess some peopel would argue about it.
But it says "leave that = undefined" away. Which however will break your code. Which back then it did for me, when I wasn't yet aware how variable non-shadowing / hoisting works in Javascript.

BTW:
for (var i = 0; i < 10; i++) {
  var x; x = undefined;

  if (i % 2 === 0) {
    x = null;
  }

  if (x === undefined) {
    console.log(i);
  }
}

Is a nicer looking way to get rid of warning 80 when not wanting to manually hoist x. However to repeat my opinion we shouldn't direct people the wrong way.

Sometimes I need, for the sake of explicitness, to write var a = undefined to make it clear I want it undefined, and I do not forget to initialize it.
I think an unnoticeable difference in script load is much less important that code clearness.

Furthermore:
for (var i=0; i<5; i+=1) {
var x = undefined;
console.log(x);
x = 5;
}
result:
5x undefined
for (var i=0; i<5; i+=1) {
var x;
console.log(x);
x = 5;
}
result:
1x undefined
4x 5
Ok, declaring a variable inside a for loop is a bad practice, but it's NOT signaled by JSHint...

I think this needs to be reopened. The guidance here seems bad. V8 will create a new hidden class each time you add a parameter to an object or class. The recommendation I've heard and read many times is to include any parameters you know you'll need from the start.
supporting example: https://blog.ghaiklor.com/optimizations-tricks-in-v8-d284b6c8b183

@jonnylink The issue being discussed here does not concern parameters. Though the reference you cited describes property definition, that is also not relevant to this particular conversation. Perhaps you intended to post your comment elsewhere?

My own additional thoughts:

The statement var x = undefined; is doing two things. The variable declaration, as everybody has already mentioned is virtually hoisted. If you do this manually, then you can focus on the next point.
jsHint has no problems with x = undefined; so manually hoisting the declaration will also solve the problem.
declaring variables on the fly in JavaScript is probably a bad idea, and a hangover from C where it is not so bad. In C, variables have block scope, so it is meaningful to declare a temporary variable as you have done. In JavaScript normal variables do not have block scope, and so their declaration at this point is confusing, and possibly a bit lazy.
Modern JavaScript has the let variable, which is scoped more like the C style variables, and declaring it later has a point.

Just thinking out loud …


schema-ast incorrectly removes the extend keyword when using federation

Describe the bug
Apollo Federation allows subgraphs to extend types in other graphs.  The schema-ast plugin strips the extend keyword off all types when it generates the local combined schema file.  It should keep the extend keyword when extending a type that was not defined else ware in the same subgraph/project.
To Reproduce
Steps to reproduce the behavior:

My GraphQL schema:

scalar Date

schema {
  query: Query
}

type Query {
  me: User!
  user(id: ID!): User
  allUsers: [User]
}

type User {
  id: ID!
  username: String!
  email: String!
  profile: Profile @requires(fields: "id")
}

extend type Profile @key(fields: id) {
  id: ID! @external
}

extend type User {
  alias: String
}

My GraphQL operations:

# n/a

My codegen.yml config file:

generates:
  schema.graphql:
    plugins:
      - schema-ast
    config:
      federation: true
      includeDirectives: true

Expected behavior

Types that are only extended, should keep the extend prefix
Types that are have there original definition locally, should be merged, removing the extend prefix (current behavior)

The above code generated these types.
type User {
  id: ID!
  username: String!
  email: String!
  profile: Profile @requires(fields: "id")
  alias: String
}

type Profile @key(fields: id) {
  id: ID! @external
}

It should have kept the extend prefix on the Profile type, because it is extending a type that is defined in another subgraph
type User {
  id: ID!
  username: String!
  email: String!
  profile: Profile @requires(fields: "id")
  alias: String
}

extend type Profile @key(fields: id) {
  id: ID! @external
}

Environment:

OS: Mac 11.4
NodeJS: 14.17.6

    "@graphql-codegen/add": "^3.1.1",
    "@graphql-codegen/cli": "^2.6.2",
    "@graphql-codegen/introspection": "^2.1.1",
    "@graphql-codegen/typescript": "^2.4.7",
    "@graphql-codegen/typescript-document-nodes": "^2.2.7",
    "@graphql-codegen/typescript-graphql-request": "^4.4.2",
    "@graphql-codegen/typescript-operations": "^2.3.4",
    "@graphql-codegen/typescript-resolvers": "^2.5.4",

Additional context

I have tested the above sample in the live demo
https://www.apollographql.com/docs/federation/entities/#extending-entities

@charles-hill-exelon were you able to solve this? The generated schema still doesn't contain the directives.


Search 6 groundwork - Support (single-valued) projections on fields within nested object fields

https://hibernate.atlassian.net/browse/HSEARCH-3062

@yrodiere thanks for the review.
I think I addressed the comments.
Moreover, I added these 2 extra commits:

HSEARCH-3062 Make Lucene nested paths single valued
HSEARCH-3062 Make ES nested paths single valued

As a followup of:

HSEARCH-3062 Check nested document paths field compatibility

It seems that once we have verified the incompatibilities we don't need a set anymore.
Is there anything else we can improve here?

Merged thanks!


Add a linkedin icon pointing to the linkedin public profile

if linkedin_username is set in _config.yml, a linkedin icon is added
to the footer pointing to the linkedin public profile
https://linkedin.com/in/{{site.linkedin_username}}

note this assume the linkedin account was set a public profile URL as described in https://help.linkedin.com/app/answers/detail/a_id/49315/~/finding-your-public-profile’s-web-address

I was just about to create my own PR for this feature which I just made to my local version and then I found your PR.
@ggouaillardet May I suggest you add the linkedin_username to the _config.yml? Trivial I know, but a few less characters for users to type :smile:

@josephbridgwaterrowe sure i can do that.
do you want me to add it commented
#linkedin_username =

or do you have a linkedin_username in mind ?
currently, @davidtmiller  https://www.linkedin.com/pub/david-miller/55/a3a/55a did not setup a public profile URL, that is why i did not update the _config.yml

Personally I would just add it in blank, I have noticed that blank configuration variables still evaluate to false when rendering the footer layout. For instance, my Facebook username is blank and nothing is shown.
# /_config.yml
twitter_username: jbridgwaterrowe
github_username: josephbridgwaterrowe
linkedin_username: josephbridgwaterrowe
facebook_username:

@ggouaillardet what do you think? Honestly, all of the above work for me :smile:

@josephbridgwaterrowe i added an empty
linkedin_username:

and updated the PR
thanks !

@ggouaillardet Nice, thanks.

I was also about to submit a PR on this.  I encourage the merge of this PR.


is_limiting only applies to REACTANT components

Should this be added to validations.py? If a compound has a non-REACTANT role and is_limiting is defined, raise an error?
I don't think this is necessarily error-worthy if is_limiting = False. Was your intention mostly to clean up the example?

Yes, this is mostly to clean up the example since the editor only outputs is_limiting for REACTANTs and the round-trip tests were failing.


Refactor map functions to use list of values

Overview
This commit changes tsdb.mapFunc to use tsdb.MapInput instead of an iterator. This will make it easier and faster to pass blocks of values from the new storage engine into the engine.

Seems fine, +1 once the debug code is removed.

@otoolep fixed & merged. thanks!


SQL server 2019 support

Docs Build status updates of commit ca20275:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/System-Requirements-for-Microsoft-Dynamics-NAV-2017.md
:white_check_mark:Succeeded

dynamics-nav/System-Requirements-for-Microsoft-Dynamics-NAV.md
:white_check_mark:Succeeded

dynamics-nav/TOC.md
:white_check_mark:Succeeded

dynamics-nav/help-and-support.md
:white_check_mark:Succeeded

dynamics-nav/index.md
:white_check_mark:Succeeded

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


Added more instructions for windows make build command

closes #3453

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


Issue 14768 - Error: index 174762 overflow for static array

https://issues.dlang.org/show_bug.cgi?id=14768
Improve diagnostic message.

Mb, not MiB

Uh, I think that's misleading. We mean mebibyte/megabyte (= 1024*1024 bytes), right?
"Mb" is bad, because: "The megabit has the unit symbol Mb or Mbit." (https://en.wikipedia.org/wiki/Megabit)
"MB" is ok, if imprecise, because it's understood that it can mean 10001000 bytes or 10241024 bytes. (https://en.wikipedia.org/wiki/Megabyte)
"MiB" is ok, if uncommon, as it means exactly 1024*1024 bytes. (https://en.wikipedia.org/wiki/Mebibyte)

I stand corrected. Sorry, Kenji!

OK, changed back to MiB.


Charset in accept makes the server ignore the entire media-type requested

Describe the bug
When sending the charset alongside the media-type in the accept-header, the entire header is ignored.
If sending "Accept: application/fhir+xml;charset=UTF-8" I get a json response.
Sending "Accept: application/fhir+xml" or "Accept: application/fhir+xml;q=0.8" I correctly get xml.
According to the standard https://www.iana.org/assignments/media-types/application/fhir+xml it's an accepted parameter for this media-type
FHIR Version?
Stu3/R4/R5 -> all
Data provider?
CosmosDB/SQL Server -> both
To Reproduce
Steps to reproduce the behavior:

Send a request with the header "Accept: application/fhir+xml;charset=UTF-8"

Expected behavior
An xml result
Actual behavior
JSON result

Thanks for reporting the issue!
Are you using the OSS FHIR service or managed FHIR service? Marking it for "UpforGrabs"

We're using the OSS self hosted version

@csteeg - Thanks for the response! The Azure FHIR service supports JSON by default. Currently, XML support is only available as an open-source capability, which makes it a lower priority for our queue at this time.

This seems to be a problem of .net MVC framework?
If I add
SupportedMediaTypes.Add($"{KnownContentTypes.XmlContentType}; charset=utf-8");
SupportedMediaTypes.Add($"{KnownContentTypes.XmlContentType}; charset=utf-16");
to the FhirXmlOutputFormatter constructor, it works.
The CanWriteResult of the base OutputFormatter, calls parsedContentType.IsSubsetOf(supportedMediaType) which in the end calls private bool ContainsAllParameters(MediaTypeParameterParser setParameters) in ReadOnlyMediaTypeHeaderValue
I don't know exactly if IsSubsetOf is really supposed to match the Charset also, but if so the supportedmediatypes should be added for all currently supported mediatypes I guess


Tenant specific versus multi tenant error

[Enter feedback here]
Very nice service, although for my purpose powerbi.min.js in the .Net Core example lacks features to obtain dataset and data, guess the clue lies in "Embed"!
Anyway for Tenant specific .Net core "Embed for your organisation" - "User owns data" lacks instructions for adjusting appsettings.json Domain parameter to match company when not multi tenant.
And in  .net framework adding the parameter in web.conf doesn't work, I am currently trying to find where I can switch from multi to Tenant specific.
Error message thrown:
is not configured as a multi-tenant application. Usage of the /common endpoint is not supported for such applications created after '10/15/2018'.
Experiment so far in web.config (adjusted for privacy!).
<!-- Azure AD application config -->     <add key="Domain" value= "mycompany.com"/>     <add key="TenantId" value="{guid}" />     <add key="clientId" value="{guid}" />
etc.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 90078026-b608-ccf2-f3ef-58021b484f75
Version Independent ID: e51ab568-5610-ff18-c5c0-4d2f27935dfa
Content: Embed content in your Power BI embedded analytics application enabling better embedded BI insights for your organization - Power BI
Content Source: powerbi-docs/developer/embedded/embed-sample-for-your-organization.md
Service: powerbi
Sub-service: powerbi-developer
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Sorted it in web.config use this approach instead:

Question regarding OwinOpenIdConnect.cs line 55 - I changed from "common" to "mycompany.com"
`string tenantSpecificAuthority = tenantCommonAuthority.Replace("common", tenantId);
string tenantSpecificAuthority = tenantCommonAuthority.Replace("mycompany.com", tenantId);`
Is that required, or is it worth using a paramter from web.config?

Thanks for your comment, @JerryMWeeks.
This looks like a support request or product question rather than documentation feedback.
You might find a solution in the Power BI Community. If not, try our support page.


House of the Dead III (D3D8) - Lisa's hair detaches from her head

Software information
The House of the Dead III (PC)
System information

GPU: NVIDIA Geforce RTX 3060 Ti
Driver: NVIDIA 545.29.02 (old), NVIDIA 555.58.02 (current)
Wine version: Wine-GE-Proton8-23 (old), Wine-GE-Proton8-26 (current)
DXVK version: D8VK 1.0 (old), DXVK master 4b0c9c6 (current)

Apitrace file(s)
hod3pc.5.trace.tar.xz
The first stage that the character appears, her hair detaches from her head - almost like it's frozen where it first appears in the shot.
Examples for visual aid, both from Stage 2:

Edit: And though less noticeable, I believe some of the decals of the moss type enemies from Stage 1/Prologue slightly stick out unusually and persist beyond their deaths, similar to Lisa's hair.
NOTE: Though the apitrace is attached with WineD3D as instructed, some facial models are broken under it and I'm not sure if the resulting trace is reproduced perfectly:
(subtitles glitch here happens on D8VK too)

These issues aren't reproduced in dgvoodoo2-through-DXVK.
Note: this issue is re-posted from d8vk#193 (at the advice of @WinterSnowfall), but as can be seen from my last post on that thread at midnight today (1:38 AM @ July 9th, 2024), the issue is still present in the listed DXVK master version post-D8VK merge.

To add some context, the offending section is somewhat into the game and unfortunately the game demo stops right before that part.
@Blisto91 has also determined that the "detached" hair is baked into the trace, as it can be observed even on Windows native while playing it back.
Since it appears to work fine on AMD post-upstreaming (see here), we would need to investigate with a copy of the game if this is perhaps a precision issue on Nvidia (I'm thinking floatEmulation and the like), or an outright bug.

in my case with 2.4 works ok seems a nvidia problem ?

@SeongGino If you have some time could you try a play-through of the problematic area with d3d9.floatEmulation = Strict? If you drop that line in a dxvk.conf file placed near the game's executable it's usually enough. You can verify in the logs that the option is properly parsed and applied; please make sure that is the case.

@SeongGino If you have some time could you try a play-through of the problematic area with d3d9.floatEmulation = Strict? If you drop that line in a dxvk.conf file placed near the game's executable it's usually enough. You can verify in the logs that the option is properly parsed and applied; please make sure that is the case.

Noting the d3d9 in that setting, does that apply to D3D8 too?
I'll investigate before the end of the night. Sounds like a very obscure bug to be vendor specific.

Noting the d3d9 in that setting, does that apply to D3D8 too?

Yes, it does. You can use any of the d3d9 config options with d3d8 as well.

Sounds like a very obscure bug to be vendor specific.

It wouldn't be the first time we've had one like this, if this is what it turns out to be.

Sorry about that, family related emergency.
Unfortunately, no change. Hair still brokey. :(

Unfortunately, no change. Hair still brokey. :(

Alright, thank you for ruling that out at least.

5+ years ago i was poking about this on  NINE + d3d8to9. Non-nvidia path works, but is like 3-4 times slower, so making nvidia path working is desirable.

AFAIR i ended up hacking NINE and maded it work properly by removing instances of R5G6B5 format support. I guess app expect shorter display list or something in particular order there, otherwise gets corrupted and messes up.
Dunno how everything is handled in DXVK, but my suggesting would be in that way... do something with 16-bit depth R5G6B5 format, so that app dont see it... something like that might help this nvidia path.

Thank you for the insight. If that is indeed the case, I'd be happy to just use d3d9.customVendorId = 1002 with the game, assuming that fixes the problem. Keep in mind this is a console port and most likely intended to run around 30 fps anyway.

Well, both ways, this text corruption bug still stays.

I dont remmeber this subtitles glitch... that must be dxvk specific.

I dont remember this subtitles glitch... that must be dxvk specific. But yeah in general, if app is messing something with a formats... then all of these glitches could be just false positive.

Right above, you can see a screenshot from @mrdeathjr28 where those subtitles appear to be fine? We'll try to procure the game and see if we can reproduce any of these things.

Keep in mind this is a console port and most likely intended to run around 30 fps anyway.

The Xbox port of House of the Dead III runs at 60 (which makes sense; HOTD3 in arcades is Chihiro hardware, i.e. a ram doubled Xbox anyways)

Right above, you can see a screenshot from @mrdeathjr28 where those subtitles appear to be fine? We'll try to procure the game and see if we can reproduce any of these things.
That text corruption seems happens only there and last for a second or two....so screenshot could misslead you if you take it a bit before or afterwards.

Right above, you can see a screenshot from @mrdeathjr28 where those subtitles appear to be fine? We'll try to procure the game and see if we can reproduce any of these things.

That text glitch seems happens only there and last for a second or two....so screenshot could misslead you if you take it a bit before or afterwards. And i didnt checked else languages, so maybe glitch is english subtitles specific

For what it's worth, neither the glitch subject of this issue nor that subtitles glitch happens when playing the game with dgVoodoo2 and rendering through DXVK's DX9 path.

Yeah, but dgVoodoo2 uses DX11 or DX12... that is entirely different path.

For what it's worth, neither the glitch subject of this issue nor that subtitles glitch happens when playing the game with dgVoodoo2 and rendering through DXVK's DX9 path.

DXVK's d3d8 is mapped to d3d9, much like d3d8to9 wrapper. As mapped to different path, then they have different bugs potentional. Here bug is either in d3d8 and/or d3d9.

d3d8 is not exactlyd3d8to9 wrapper because have many things proper to dx8 compared d3d8to9 wrapper
however errors still present on 3d8to9 and dxvk
back to error in my case works ok, i finish a gameplay video around 3 hours (2 rescues) and have a glitch but different than posted @SeongGino and appear when you have finish game

There are just so much "bugs" in this, it is not impossible that drivers became borked for this in a way too... somewhere back in history.

DXVK's D3D9 still need buffers optimisations, for SWVPs too, that still run TAD SLOW

SWVP should run fine unless a game does exceptionally weird stuff. It's implemented the same as HWVP except that SWVP supports more vertex constants. To avoid copying 1024 Vec4s for every draw call, we only copy up to the highest constant that gets modified by the application on the CPU and rely on robustness2 to return 0 for out of bounds reads. This approach has generally worked and SWVP stopped being a problem once we landed that, so I'd be surprised if its still slow in this game.
As for the bug, it sounds like it comes down to undocumented locking behavior. Maybe writing data after calling Unlock() or something like that.

@K0bin OK, i spot these optimisations since 2.3.1... as i see for Warrior Kings SWVPs this functioning, but yeah that is only partially. Since yeah i already know more cases of this turning bad, posted you a trace

I don't know what you spot in 2.3.1 but the optimization I was talking about landed years ago.
https://github.com/doitsujin/dxvk/pull/2282

It is this i think "Reworked how dynamic system memory buffers are copied to the GPU in D3D9 games."

About d3d9.customVendorId not functiong for d3d8 apps, that is third bug...

I've just checked with the d3d8-triangle test app, and it is being applied correctly, as determined by the d3d9 customVendorID logic. So are the rest of the fields (customDeviceId, customDeviceDesc). If you spot any particular points where a custom vendor ID isn't retrieved correctly, please open a bug with steps on how we can reproduce it.
Note that the dxvk UI will not report these frontend values, since it takes them from Vulkan directly.

It is this that triggered some SWVP perf i think https://github.com/doitsujin/dxvk/pull/3765

What does "triggered" mean here? Improved or worsened?

Well, latest shader d3d version have 4 fog components, not 1.

No, it does not. The latest shader model which has fixed function fog is SM2_x and as you can see here:
https://learn.microsoft.com/en-us/windows/win32/direct3dhlsl/dx9-graphics-reference-asm-vs-registers-vs-2-x
oFog only has 1 component.

What does "triggered" mean here? Improved or worsened?
Triggered means - even if didnt meant directly to improve it,  it get improved... partially
Partially, means - not in every SWVP app.
oFog only has 1 component.
In NINE we do it 4 anyway, do not hurt. Maybe because newer shader compiler more likes it that way 🤣


fail to build fail_release

Bug Report

What version of TiKV are you using?

bbfedd409b5965c04b9edcb34f0a0907c75d6dd2
make fail_release to reproduce

Seems the direct cause is upgrade raft-engine from 0.3.0 to 0.4.1


Assigning g:gutentags_project_root_finder

I'm looking to assign g:gutentags_project_root_finder to my own function but can't get this to work. I am trying to use let g:gutentags_project_root_finder = function("MyFunction") but this is not allowed since a function reference must start with a capital letter or if it starts with g: the first letter must be a capital letter. Can you adilvise how I see this up correctly ?
I have raised a pull request as a suggested fix. #106
Cheers

You're supposed to set that setting to the name of function, so let g:gutentags_project_root_finder = "MyFunction" should work?

Hi - thanks. Yes - realised the problem was with the fixes in the pull request you merged. With that pull request it works as you detail.
Cheers
Sent from my iPhone
On 27 Nov 2016, at 13:53, Ludovic Chabant <notifications<EMAIL_ADDRESS>wrote:
You're supposed to set that setting to the name of function, so let g:gutentags_project_root_finder = "MyFunction" should work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ludovicchabant/vim-gutentags/issues/100#issuecomment-263104029, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI9yBqrACHEWR12IKNSJgJrcOPSIgcX0ks5rCRrogaJpZM4KsZds.

Cool, thanks.


Fix for #1840, forwards delete operations to SDKs

After a DB operation that was mutated by a beforeSave we'd loop through the new keys in order to set the DB computed values or the original value.
We'd re-inject the __op: Delete object from the data instead of deleting the response key directly.

Closed #1940

[Current coverage][cc-pull] is 91.87%

Merging [#1946][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

2 files (not in diff) in src were modified. more

Misses -2
Hits +2

@@             master      #1946   diff @@
==========================================
  Files            91         91
  Lines          6394       6400     +6
  Methods        1094       1094
  Messages          0          0
  Branches       1341       1341
==========================================
+ Hits           5872       5880     +8
+ Misses          522        520     -2
  Partials          0          0

Powered by Codecov. Last updated by [cd52580...6bd72bc][cc-compare]
[cc-base-branch]: https://codecov.io/gh/ParsePlatform/parse-server/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/ParsePlatform/parse-server/compare/cd525802a63538b48edf6320229017dc3eac4d4e...6bd72bce1ea4cddf3608838023d7fa32da7b418f
[cc-pull]: https://codecov.io/gh/ParsePlatform/parse-server/pull/1946?src=pr

After further investigation, the Parse.com behaves the same way which is troubling...

Will be fixed by: https://github.com/ParsePlatform/Parse-SDK-JS/pull/289

This won't work as is, as the Client SDK's don't handle (at all the Delete operations when coming back from the server...)

Waiting for the JS-SDK that implement the change to be released.

We might be able to test this functionality with 1.9.0-rc1 now.

I don't see it yet

@flovilmart updated the pull request.

@flovilmart updated the pull request.

@flovilmart updated the pull request.

Something in here is causing the tests to consistently fail in postgres. Not sure what, excluding progres is probably fine though.

@drew-gross we may want to add a check on the client SDK version for the support of that feature no?

@flovilmart updated the pull request.

@flovilmart updated the pull request.

@flovilmart updated the pull request.

@flovilmart updated the pull request.

@flovilmart updated the pull request.

fixes #1606

@flovilmart updated the pull request.


[Amazon AWS] Improvements for SQS, DynamoDB and CloudFormation commands

Description

Adds sort descending and consistent reads to DDB queries
Saves the query projection expression in between form renders
Improved toast error handling for SQS, DDB and CFN command actions
Sorts the resources listed for SQS, DDB, CFN commands

Screencast
Error handling with toast

Query descending and persisting projection attributes

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer when you find some time please assist in merging the changes as they have been approved

✅ Approved

@pernielsentikaer sorry I had protection rules on my branches, can you please retry merging?


Uncaught RangeError: Maximum call stack size exceeded

I tried to sort a large list (10,000 elements) in Ellie and I got an exception Uncaught RangeError: Maximum call stack size exceeded:
https://ellie-app.com/3gSGSzwjhZ4a1
According to some members on the Elm forum, the issue might be related to the Debugger, which Ellie is using internally. I'm fairly new to Elm so I'm taking their word and posting the issue here; please let me know if it makes sense to report this in a different repo. Thanks!

I'm getting a similar problem with a much smaller Array. I remember there was a problem in 0.18 with Arrays and I was directed to a third-party, pure-Elm Array implementation as it was a known issue.
That library lives here https://package.elm-lang.org/packages/Skinney/elm-array-exploration/2.0.5

@Pilatch That library has become the official implementation in Elm 0.19.

@svileng your code works for me.
If this was written in Elm 0.18, then that compiler had issues with large List literals (chrome doesn't handle them to well). This has been fixed in 0.19.

@Skinney I've only seen this exception on Ellie, which I thought is on 0.19. Originally I thought this is an issue in the app, but people recommended me to post this in core.
I'm happy to open an issue in the Ellie repo if that's where it should belong?

Ellie should be on 0.19. It could just be that you have a huge data structure. As said, the code doesn't trigger any exceptions for me.

Could you please clarify what do you mean when you say "the code doesn't trigger any exceptions for me" — do you mean that it works okay locally? Or clicking on the sort button on https://ellie-app.com/3gSGSzwjhZ4a1 doesn't produce any exception for you?

Clicking the sort button on ellie doesn't throw any exception. Running Safari 12.

Thanks for clarifying.
For the record, it still happens for me — running Chrome 69.0.3497.100 (MacOS).

There also is an issue with small arrays in elm-test, but that's probably specific to elm-test or node or something. I'll have to get to a simple reproduction of that later. Just making a note here in case anyone already knows about it.

I am able to reproduce the problem in Chrome Version 69.0.3497.100 (Official Build) (64-bit)
//user-images.githubusercontent.com/7709033/46739810-5d169f80-cc6f-11e8-8367-4b146449338e.png

Can confirm that this does not happen locally, using elm reactor.

...And elm-test version 0.19.0-beta9 does just fine until you try to sort 10,000,000 items, at which point I get JavaScript heap out of memory.

It sounds like the root here is that allocating a lot of data can cause problems in certain systems. Given that allocating less is nice in any context, I do not think it makes sense to track this general goal with this particular issue.


UniMiSS+ availability

When will code and weights of UniMiSS+ be available?

Thanks for your asking. We will release them in two weeks.

four weeks passed~

Thanks for your attention. Our server was under maintenance recently and has just been fixed. We will update the code ASAP.

Hi all, we have released UniMiSS+ pre-training code and weights and UniMiSS+ fine-tuning code. Thanks for your attention again.

Hi all, we have released UniMiSS+ pre-training code and weights and UniMiSS+ fine-tuning code. Thanks for your attention again.

whats the main difference between UniMiss+ and UniMiss ? as i cannot find a pdf version of the paper

Hi all, we have released UniMiSS+ pre-training code and weights and UniMiSS+ fine-tuning code. Thanks for your attention again.

whats the main difference between UniMiss+ and UniMiss ? as i cannot find a pdf version of the paper

Please see these links
https://link.springer.com/chapter/10.1007/978-3-031-19803-8_33
https://ieeexplore.ieee.org/stamp/stamp.jsp?tp=&arnumber=10617802


short commands

There is already a short version of generate, why don't we add -n for name and -g for git?
Example:
cargo gen -g https://github.com/user/template.git -n foo

I assume the reason you want this is to type less? Then I'd rather see this:
No --git or -g, but allow passing something
that can we can interpret (a git-parsable URL for now) as a free argument.
Also, omit the -n completely and wait for cargo-generate to prompt you :)

i mostly agree with @killercup - but in the sense of actionable requests from this- @killercup do you mind the -n short for --name? we can punt on the --git sitch til later (open another issue or something...)

-n is fine

Am 31.07.2018 um 17:51 schrieb ashley williams<EMAIL_ADDRESS>i mostly agree with @killercup - but in the sense of actionable requests from this- @killercup do you mind the -n short for --name? we can punt on the --git sitch til later (open another issue or something...)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

closed by #77


scroll right to left

how can i make pages scroll right to left . for supporting arabic language

This is not a feature on our roadmap. Sorry.


S4 - As a user, I want to be able to recover my password in a save way, so that I will not suffer from production down-time due to being locked out of the system in case I have forgotten my password.

Acctepance Criteria

[ ] A user can click on "I forgot my password" on the login page to receive a new, temporary password.
[ ] Upon clicking the button, an e-mail is sent to the provided e-mail address of this user.
[ ] The e-mail contains a link that leads the user to a password reset page.
[ ] The link is valid for a short period of time only.

Development Tasks

[ ] sopra-fs22-group-26/client#55
[ ] sopra-fs22-group-26/server#90
[ ] sopra-fs22-group-26/client#56
[ ] sopra-fs22-group-26/server#92

Should we skip this user story and mention it in the report as a possible extension for the future?

Could be implemented in a further update.


🎮 Game Request
You control a character with a jetpack, in an asteroid field.
You have no weapons...except your jetpack, which spouts flames!

The spacebar control helps people maneuver better.
Point down the features
Controls
Up - Power jetpack
Left - Turn left
Right - Turn right
Spacebar - Turn 180 degrees
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


add yaml deployment implementation for the crane-scheduler module

What type of PR is this?
bugfix
What this PR does / why we need it:
There are two services in the module, namely "crane- Scheduler-Controller "and "crane-scheduler", but only yaml deployment of the "crane-scheduler-controller" module is included in the code. This submission complements the  yaml deployment of crane-scheduler
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

some fields are generated from deployed yaml? better not bring some fields like terminationMessagePath .
can refer to deploy yaml in helm chart repo

Yes.I have made code modifications according to the comments


V3 jobs should be hardened to index.json changes

We have a couple instances in which direct links to V3 resources are passed as arguments to jobs.
For example, Catalog2Lucene takes in a direct link to the cursor of one of our Catalog2Registration containers.
These jobs should fetch these V3 resources from index.json, so that if the containers change, the jobs automatically pull in the correct cursor values.
For example, when we moved from registration1 to registration2 (because of semver2), the V3 monitoring job failed some packages because it was using registration1's cursor while checking registration2.

This should be optional integration. There are cases when you want to run the job before something is mentioned in the index.json.


docs: clarify panic(nil) case since go 1.21 introduced a change

See https://pkg.go.dev/runtime#PanicNilError for details.

coverage: 99.865%. remained the same when pulling 21681b1835f39576cc10d8756a01a714cc06fc26 on doc-panic into 5a65b31607a0aa18043211f9298d5804d8a3a735 on master.


A more permissive license.

With Godot as mit, will be hard to give this engine the focus it deserve.
also is there a way to protect our game scripts/data?

Godot requires your project to compile against it. Without requires MIT since you compile against it

Godot requires your project to compile/link against it. Without MIT it would cause troubles.
Drag[en]gine on the other hand does not require your project to compile/link against it. The license used is thus not a problem to your project. It is only affecting Engine Modules or IGDE Editors since they link against the game engine.
So no, there is no need for a more permissive license since as a game developer the license does not affect you.
Game scripts and data are right now not more protected as being stored in a DELGA file. The reason is simply that many have the misbelief that encrypting or otherwise obfuscating scripts/data would protect their game data. But this is a wrong belief since the key to decrypt files as well as script logic required to de-obfuscate data while loading has to be present in the distributed game files. And if it is present it requires just one guy to filter it out and publish a crack and the entire protection process is void.
That said I did research a bit possible protection methods. In general extend the DELGA file format with encrypting the contained files is most probably the easiest solution to do. The question is only where to hide the decryption key (or in what form). I have toyed with a potential solution for this which in my opinion though would at best fend of script-kiddies and opportunity-rippers but not focused crackers. I'll try to put together a proof of concept to see if it would be enough protection for this group of attackers. But using the IGDE chances are people can get the files out. I mean I recently evaluated some UE games for data protection and albeit this being a AAA engine getting files out of recent games using the UEditor was simple. In the light of this I'm not convinced a strong protection is possible, at best a weak one.
That said if you have a specific protection method in mind I can evaluate it to see if it is viable.

Nice I didn't knew about that.
About the protection, a light one would do, at least for me. Like compiling scripts and a binary model format.

The license was recently modified from L-GPL to MIT, so I think this issue can be closed - unless there are still parts of the project that need to be re-licensed.


No mention of Operations Manager multihoming limitations

As I understand it, there is no way to have a Linux computer use the Log Analytics agent to connect to both a SCOM management group, and a Log Analytics workspace. This cost us many hours of planning in a large customer project because it wasn't clearly explained in the documentation. If you call it out here, it would save people many hours of planning and potential confusion.

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

ID: 012c35ae-69e3-dfe3-6d99-077915f5f555
Version Independent ID: f29da76c-d984-e218-570e-06ac68d2c550
Content: Configure Azure Log Analytics Agent for Hybrid Linux Computer
Content Source: articles/azure-monitor/learn/quick-collect-linux-computer.md
Service: log-analytics
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@bmilbauer Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.
@MGoedtel - FYI

@bmaluijb - If you already have the Log Analytics agent already installed on your Linux server you can configure it to be monitored by both OpsMgr management group and a LA workspace.  Edit the file /etc/opt/omi/conf/omiserver.conf and add this line - httpsport=1270 at the beginning of the file.  This is one of the ports OpsMgr uses to communicate with the server and enables it be discovered as a manually installed agent.  Restart the server after the config file update and if this is a supported distro of Linux with OpsMgr, you can follow the OpsMgr documentation to configure it to authenticate for discovery/monitoring and then discover/monitor the server.  Does this answer your question?  This was documented previously so let me bug this and get it addressed in the meantime.  Thanks. #in-progress.

#please-close as I have updated the documentation with respect to this scenario to help others avoid this situation.  It will be published later this afternoon.  Thanks!


config: fix monitor_interface when not passed in the inventory file | osd:  fix old jewel container activation

Setting monitor_interface in group_vars/all.yml makes the
hostvars[host]['monitor_interface'] non-existing.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1507922
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

This PR is fixing two separate bugs now, want to re-title?

@ktdreyer done.

jenkins test luminous-ansible2.4-centos7_cluster

jenkins test luminous-ansible2.4-docker_cluster


Add function to clear infractions

The problem
In many cases, it's perfectly acceptable to increase infractions monotonically. However, it may be desirable to clear a bad actor's infraction record on certain events before they're jailed. For example, I would like to be able to reset a requestor's infractions on a successful login, since there may be overriding reasons why infractions were made despite the client having the correct password.
This is also, for example, how Windows domain login works to determine account lockout.
Recommended Solution
A new function should be exposed on the Studio called ClearInfraction which allows clearing all infractions for a specific actor/rule combo.

I appreciate this. I'm not going to add this to BadActor currently. The more complexity I add the more it doesn't serve its niche. The more I add the more I fear it destabilizes what and who BadActor serves. It is super lightweight and made to add just enough friction to your web application or API or service, so that Attackers move on.
At this point, we start going down the path of a full cache. And that is when I'd suggest badgerio, or redis, or another KV store.
I hope that makes sense. Again, I appreciate this. Please don't take it personally.

That makes sense. If I wanted something that kept a detailed record of infractions and the ability to mutate them in more specialized ways, I would probably want to keep the data centralized in a separate service. Plus it could easily exacerbate problems in a multi-node architecture where one node has cleared infractions while other nodes still have infraction record for that particular actor.


feat: :sparkles: Added multiple materials from both camps and changed…

… ship textures.

[x] 您的代码是否能够编译通过？
[x] 您是否检查了目前在 pull requests 中是否有与你的贡献功能相同的更改？
[x] 您的贡献是否符合贡献者代码协议？
[x] 您的贡献是否符合开发规则？

Descriptions of this pull request:

No additional description.

Can the color part of the texture be separated in order to produce more color varients?

Can the color part of the texture be separated in order to produce more color varients?

The color part of the textures may be too complex to be seperated. It should get the balance between the texture seperation along with more complex system and the drawing work.

Can the color part of the texture be separated in order to produce more color varients?

The color part of the textures may be too complex to be seperated. It should get the balance between the texture seperation along with more complex system and the drawing work.

Simply adding a mask will work, I suppose. It is not that complex.

Can the color part of the texture be separated in order to produce more color varients?

The color part of the textures may be too complex to be seperated. It should get the balance between the texture seperation along with more complex system and the drawing work.

Simply adding a mask will work, I suppose. It is not that complex.

I'll make it an issue.


Added Profile Page

Fixes Issue
closes #2006

Changes proposed

Screenshots

Note to reviewers

@codervivek5 Can you please review the PR soon and let me know for any more changes to be required

@codervivek5 I have updated a sample avatar

Thankyou for your contribution :)


Webhooks do not fire on threshold violation

Describe the bug
Webhooks are not triggering on threshold violation.
To Reproduce
Steps to reproduce the behavior:

Set up thresholds that will fail on upload, download, or ping
Configure a webhook endpoint and test to confirm working
Enable "Notify on threshold failures"
Run a speedtest
Validate at least one threshold was breached
Inspect log file to see the WARNING log line explaining that no thresholds were found

Expected behavior
Webhook fires on a threshold breach.
Environment (please complete the following information):

LinuxServer.io Docker image
Version 0.19.0

Screenshots
Threshold violation database notification at time of log entry listed below for webhook.

Logs
2024-05-25T00:00:34.460930193Z [2024-05-25 00:00:34] production.WARNING: Failed webhook thresholds not found, won't send notification.

Logs 2024-05-25T00:00:34.460930193Z [2024-05-25 00:00:34] production.WARNING: Failed webhook thresholds not found, won't send notification.

This error is thrown when webhooks are enabled but there are no destinations provided. Make sure you've added at least one webhook url in the notification channels.

There is a destination listed.  Testing the webhook works as expected.

Yeah Gotify is not supported as a destination even thought it "supports" webhooks they need to be in a specific format. The test one just happens to match it.
#40 for supported notification channels.

Yeah Gotify is not supported as a destination even thought it "supports" webhooks they need to be in a specific format. The test one just happens to match it.
#40 for supported notification channels.

Ok.  I will configure my other webhook destination anyway just to confirm.  I didn't check my Gotify logs to see if there were any entries regarding a malformed request.  From reading the code and the log description, it seems like it's not detecting any threshold violations... Hence the issue report.


Restructure more code in preparation for the next release.

Everything looks great to me!

Merged here: 1c22d7b15a42d88040e02431ac3ed82313fe1387


docs: build instructions

Simplified build instructions would be helpful. I did the below, but wasn't sure this was the "right" way.
go build cmd/google-rotation-planner/main.go

It should be go build -o . ./..

I get the following when I execute that:
go build -o . ./..
no Go files in /home/rpocase/git

This is likely just not having my environment setup how go projects want it to be setup, though.


Monospace font for stdout/stderr

It would be nice to have a monospaced font for stdout/stderr.  Crashes that show indicators such as:
module.js:340
    throw err;
    ^

are useless in a non-monospaced font.
A good alternative would be to open a terminal window with stdout/stderr and hide that panel.

Good point.

fixed


[Scala] Serialization exception when a class refers to another class inside a package object

Describe the bug
If you try to serialize a class that refers to something inside a package object, it fails with an error like this:
Exception in thread "main" java.lang.RuntimeException: Create sequential serializer failed,
class: class com.devsisters.ck.benchmarks.Reproducer$SomeClass
	at io.fury.serializer.CodegenSerializer.loadCodegenSerializer(CodegenSerializer.java:48)
	at io.fury.resolver.ClassResolver.lambda$getObjectSerializerClass$2(ClassResolver.java:999)
	at io.fury.builder.JITContext.registerSerializerJITCallback(JITContext.java:132)
	at io.fury.resolver.ClassResolver.getObjectSerializerClass(ClassResolver.java:997)
	at io.fury.resolver.ClassResolver.getSerializerClass(ClassResolver.java:931)
	at io.fury.resolver.ClassResolver.getSerializerClass(ClassResolver.java:830)
	at io.fury.resolver.ClassResolver.createSerializer(ClassResolver.java:1219)
	at io.fury.resolver.ClassResolver.getOrUpdateClassInfo(ClassResolver.java:1151)
	at io.fury.Fury.writeRef(Fury.java:345)
	at io.fury.Fury.write(Fury.java:322)
	at io.fury.Fury.serialize(Fury.java:256)
	at io.fury.Fury.serialize(Fury.java:209)

Expected behavior
There should be no such error.
To Reproduce
import io.fury.Fury

package object O {
  case class Id(value: Int)
}

object Reproducer extends App {

  val fury = Fury
    .builder()
    .withScalaOptimizationEnabled(true)
    .requireClassRegistration(false)
    .withRefTracking(true)
    .build()

  import O._

  case class SomeClass(v: Id)
  val p = SomeClass(Id(1))
  println(fury.deserialize(fury.serialize(p)))
}

Environment (please complete the following information):
Tried with the latest snapshot, Scala 2.13.

By the way, I am interested in replacing Kryo with Fury, so I am testing serialization of a large object we use at work, which is why I find a lot of these issues. Thanks for the quick fixes!

By the way, I am interested in replacing Kryo with Fury, so I am testing serialization of a large object we use at work, which is why I find a lot of these issues. Thanks for the quick fixes!

You are welcome, thank you too for creating those issues. This helps us improve fury scala quickly.


Unresolved paths to some images, e.g static/style/blue/ok_small.png

Some images (e.g. on the page Manage Tools, URL admin/repositories) don't load correctly after launching Galaxy using the  docker image galaxy-stable:19.09. The problems seems to be that e.g.  lib/tool_shed/galaxy_install/grids/admin_toolshed_grids.py hardcodes the blue style by referring to %s/style/blue/ok_small.png where %s is the static URL, i.e. static/. But both galaxy.yml and nginx.conf already map static/styleto  static/style/blue, and the URL thus translates to /galaxy-central/static/style/blue/blue/ok_small.png which does not exist.
I tried fixing this by changing the static-map line in galaxy.yml to read static-map: /static/style=static/style instead of static-map: /static/style=static/style/blue, but that didn't help -- I assume because the Docker image serves static files directly through nginx, not through uWSGI.
Can I fix this through the config somehow or do I need to modify the Dockerfile to change nginx.conf?

Thanks to @jyotipm29 we are back on track :smile:
The new 24.1 image contains a lot of changes and reflects the latest developments in Galaxy. I would like to close this PR, but please feel free to reopen and rebase against the latest version.
Please give it a try:
docker run -p 8080:80 -p 8021:21 -p 4002:4002 --privileged=true -e "GALAXY_DESTINATIONS_DEFAULT=slurm_cluster_docker"   -v /tmp/galaxy-data/:/export/ quay.io/bgruening/galaxy:24.1

... or any other combination. The readme has been updated. Please add any useful tip to it.
For a list of changes, see the Changelog.


Difficulty Rendering Collections through Kotlin Wasm pathway

To Reproduce, load Kotlin's Tutorial 5 (Collections) in the web shell.
In mucking with the code, it was discovered that adding renderOutput to various lifecycle events (e.g. onHandleUpdate, onHandleSync, onInit) did call the rendering function, but did not deterministically trigger collections to render.
This is somewhat of a heisenbug: The collection rendered once every so often -- certainly, not every time.

did not deterministically trigger collections to render

These facts are not in evidence. The data must exist for the Particle to render, so not seeing output doesn't mean it 'did not render'; it's more likely the data never appeared.
I'm not sure I'm the right person to own this bug, since I'm not on the Kotlin team.

I can remove an owner for now. We can revisit the issue more meaningfully post-march.

I did a bit of digging into this today...
I added renderOutput to both the onHandleSync and onHandleUpdate methods. I also logged content in the onRenderOutput method within wasm.ts. I found that onRenderOutput was successfully being called, the model was properly coming through, and the model looked correct.
Not sure where that leaves us, but wanted to add what I'd learned to this issue.


Unnecessary print statement in cupy/core/core.pyx

There seems to be an unnecessary print statement in cupy/core/core.pyx
(https://github.com/pfnet/chainer/blob/master/cupy/core/core.pyx#L1692)
This causes what seems to be debug info to be printed whenever you call cupy.ndarray.repeat()

That's right. I'll remove this line. Thank you!


Would be great to include "--category " command to run selected tests like we have in xamarin test cloud

👍 good feedback. Thank you.


Extension icon greyed out in toolbar - continue

Will this awesome service be restored?

It's grayed out again.

FIREFOX 89.0 FOR WINDOWS

In console you can see that requests error out with 503
GET https://api.shodan.io/dns/resolve?key=MM72AkzHXdHpC8iP65VVEEVrJjp7zkgd&hostnames=account.shodan.io
[HTTP/3 503 Service Unavailable 236ms]

with MM72AkzHXdHpC8iP65VVEEVrJjp7zkgd being the API key used by the authors (see background.js). Error itself states: "Request rate limit reached (1 request/ second). Please wait a second before trying again and slow down your API calls.", so my understanding is that this is an issue with the author's API key and judging by the fact that the repo hasn't been updated in years, I doubt it will get fixed.

@voukau
Spot on excellent analysis --- looks like this add-on is on its way out!

Local workaround I ended up using:

Clone the repo and add your API key on lines 16 and 36 in background.js
Load the add-on to Firefox as a temporary extension via about:debugging

Will this fixed version end up on the addons store? Or until then should I fix this on my own?

I have configured the code to use my API key, but every time i restart FF i need to do it again going to about:debugging. I searched for the problem and all i got was that i need to use a nightly or beta FF in order to permanently use the add-on. Is there any other workaround to this problem?

Hi folks.
I fixed the code and it should work back again as it used to.
@PaulSec Could you have a look at my changes and upload a new version of the add-on?
@portiktamas Please be patient. It will end up on the store eventually.

thanks for your work @romainletendart !
waiting for the store update
Do you have any tips to provide my own Shodan API key ?
I am not familiar with firefox extensions

We had to change the API key due to abuse and we will be releasing a new version of the Firefox add-on that lets you enter your own API key to get around this issue long-term. This add-on uses the old code from our Chrome extension and doesn't have some of the new features (ex. dark mode, domain info). We now have a build process that generates both a Chrome and Firefox extension from the same codebase and you can get the official Shodan add-on for Firefox from the following location:
https://addons.mozilla.org/en-US/firefox/addon/shodan-addon/

Awesome, @achillean do you mind if we redirect users to your version?
We have about ~11k users using this version so far:  https://addons.mozilla.org/fr/firefox/addon/shodan_io/
What are your thoughts you all?

https://addons.mozilla.org/en-US/firefox/addon/shodan-addon/

License is All Rights Reserved --- not open source?

@7h30th3r0n3 That API key was specially created for the browser add-ons and is treated differently by the API. Originally I wanted everybody to be able to use the add-on without having to configure an API key but now that the add-on is used by quite a few people it's become an issue. As a result, we will be adding a settings area for the add-on.
@Hillside502 that's correct, we never open-sourced the Chrome extension so technically the code in this repo should be all rights reserved as well. Note that due to the nature of these types of add-ons you can always look at the code.
@PaulSec I don't know how to transfer/ merge add-ons for the Firefox store. From my perspective, it would be ideal if we had ownership of the FF add-on so we can publish new versions that are in-sync w/ the features of the Chrome version. The new version doesn't yet have a lot of users so that one could be deprecated.

@PaulSec Sry, I missed your first sentence. If it's possible to redirect users to the new page then that would also work!

hey there @achillean, no prob. I sent you a Direct Message through Twitter. Feel free to provide me your e-mail and I will add you as author on the Firefox website to manage it and you will be able to keep all the users too :)


Wrong property used in sampler.ts

" Property 'seedRandom' does not exist on type 'IMath' " -> we need to use "NativeMath.seedRandom(s)" instead of "Math.seedRandom(s)"

@Elli610 Can we close this issue ?


Use make to build binary errors

➜  cloudtty git:(main) ✗ make build
/Users/york/go/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..."
go fmt ./...
go vet ./...
go build -o bin/manager main.go
cannot find module providing package main.go: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
make: *** [build] Error 1

It depends that all the custom files generated by kubebuilder will be removed, right？


perf: compress favicon.png

Pull Request Test Coverage Report for Build 1315

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

Totals

Change from base Build 1314:
0.0%

Covered Lines:
44

Relevant Lines:
44

💛  - Coveralls


Working version

Hi!
I don't know what you're going to do with this package but I made it usable. Feel free to merge it :)
Regards,
Karol

@ksamborski elm-effects is not there for a reason. :)

I know this PR makes no sense now but it has some fixes for bugs that still exists in current javascript part. For example:
err instanceof SecurityError
will not work. There is no such thing as SecurityError.

Separate PR


Use new collective page for displaying events

The events are displayed with an old design. As events are also collectives we can use the new collective page.
The new collective page need to be adapted to handle events, to show date/time and short location info in the header, and display the sections: About, Tickets, Location, Participants, Budget.

Current route structure for events is /:parentCollectiveSlug/events/:eventSlug. However, as per docs:

In Open Collective, every entity is a collective and can be accessed publicly via their unique slug https://opencollective.com/:slug.

I'm thinking it could be good to support both structures, but keeping the default to what is currently implemented as for now?


Do NSG Flow Logs track ESP and AH Protocols

[Enter feedback here]
as per below , ESP and AH Protocols are supported in NSGs , but we are doing some testing and dont see any ESP or AH traffic in the flow logs (but we see corresponding TCP / UDP traffic from same clients)
Would be good to have this added to the FAQ imho
https://learn.microsoft.com/en-us/azure/virtual-network/network-security-groups-overview
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 0ecfbaa6-9625-fef7-b945-f37a71309e64
Version Independent ID: 2368b1fe-4570-18fb-693d-44be6609b080
Content: Azure Network Watcher frequently asked questions (FAQ)
Content Source: articles/network-watcher/frequently-asked-questions.yml
Service: network-watcher
GitHub Login: @halkazwini
Microsoft Alias: halkazwini

@andyjballgit
Thanks for your feedback! We will investigate and update as appropriate.

Seems related to https://github.com/MicrosoftDocs/azure-docs/issues/109459

@andyjballgit
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Hi @andyjballgit
Thank for your feedback!
I'm updating the FAQ to reflect this info. Expect to see it reflected in the doc in the next 12-24 hours.
I'll close this GitHub issue for now, but feel free to comment here as necessary.
#please-close


Fix error output since --version has been replaced with --project-version

Fixes #2485
Bug introduced in #2310 [more precisely, here]

Rebased this from main, so that the recent changes make tests less wonky


Amount of active Slots that host can host is limited to SlotQueue's maxWorkers (default 3)

After merging Proving and Sales state machines in https://github.com/codex-storage/nim-codex/pull/469, the sale state machine's process finishes only after the whole request's duration is passed and a request is paid out (if not taking into consideration the error states and other "edge-cases").
This is a problem because currently, the SlotQueue works in a way that it delegates the Sales process for the chosen Slot to pool of workers (default is 3) and it blocks the worker until the Sales process is finished, which is now much longer then it used (before it was till Slot is filled and then for proving phase it was handed out to the other state machine). So essentially, this limits the number of concurrently hosted slots to the number of workers.

The processed future could be completed at an earlier state. Currently it's completed in sales.onCleanUp, but a different callback could be created, that completes the Future at an earlier state.

Agree, the question is, at what stage? I guess the filled state could be a good place to have it.

Yep, I think Filled would be the most ideal place, as this indicates that the slot processing, both downloading and initial proof generation, as well as filling contract calls, have been completed.


Chunks

Is it possible to generate chunks on all axes?

Yes you can, only the Y axe is missing.
You can modify the WorldManager.cs file  and replace the int3 CalculateChunkCoordinate variable by
private int3 CalculateChunkCoordinate(float3 position) => new int3((int)math.round(position.x / m_chunkDimensions.x), (int)math.round(position.y / m_chunkDimensions.y), (int)math.round(position.z / m_chunkDimensions.z));
Also, line ~209 you can add                 EnqueueChunk(chunkCoordinate + new int3(0, 1, 0), viewerPosition);                 EnqueueChunk(chunkCoordinate - new int3(0, 1, 0), viewerPosition);
for Y axis. It is not optimized in any way, but it works.

Sorry for the late response. What @UnlimitedX said is correct. But especially GPU memory requirements shoot up insanely with this change. All in all the Worldmanager (which is responsible for loading the world around the player) isn't that advanced and sophisticated. You'll probably have to implement your own if you have specific needs.

in addition I needed to add in DualContouring.Process()
if (VertexCount == 0) return Status.Done;


Property with type IList causing exception

Nuget Package
JsonSchema.Net.Generation
Package Version
3.3.1
Operating System
Mac
.Net Target (if relevant, please specify the version in the description)
.Net Framework
A clear and concise description of the bug
I think I am experiencing a variant of this issue, which was fixed by this PR, except with JsonNode.
Locally, I've applied a fix similar to the fix in the above PR for JsonObject, except for JsonNode, and it seems to solve the issue.
Please add test code or steps to reproduce the behavior
Run JsonSchemaBuilderExtensions.FromType on a type with a property that has a type of IList<JsonNode>.
See the following exception:
"System.ArgumentException: An item with the same key has already been added. Key: Item"
What did you expect?
No exception is thrown
Is there any other information you'd like to share regarding this bug?
No response
Code of Conduct

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

Thanks for the report.  I'll work up a test to see if I can replicate this behavior.

Yeah, it looks like the fix is just creating another generator for JsonNode that just generates the true schema (which validates all values).
Unfortunately, I don't have a mechanism for returning a boolean schema.  I'll need to see if I can work that in.  Until then, this will return the equivalen "empty" schema {}.

Scratch that... it just works!  I must have accounted for zero intents means true schema.
{
  "type": "object",
  "properties": {
    "Foo": {
      "type": "array",
      "items": true
    }
  }
}

I should have this out shortly.

Thank you so much for implementing the fix, and for this very useful package!


CA - Test Network Environment For Sending Data To Stage

Describe the bug
Configuration to send data to stage was turned off due to lack of use and needed to be activated
Impact
User could not send date to stage
Expected behavior
Configuration activated and check test environment connection

Ticket Number from Optum "INC27413234"
Subject from email: INC27413234 - CCRS - TACC req - CDC Reportstream unable to send data to stage

Hey team! Please add your planning poker estimate with ZenHub @brick-green @greene858 @jorg3lopez @oslynn @sarahnw @TomNUSDS

Vanessa Burkhart asked a follow-up question on 7/25/2022 asking the following:
The Config was re-enabled, hope this resolves the issue and ability to sent to staging should be back. Please let us know.

Just tested and was able to connect to the CA test location.

@brick-green @BerniXiongA6
We received an email from CA State asking if their connection has been re-established.  Please confirm.

The test connection is up. Can let CA know.

@brick-green I have emailed Vanessa from California that the test connection is up.


fix: updated the read me sheet

:tada: This PR is included in version 0.356.4 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Results not matching with reported ones

I'm trying to replicate the results presented at https://huggingface.co/EleutherAI/gpt-neo-1.3B for Wikitext. However, when I run
python main.py \
    --model gpt2 \
    --model_args pretrained=EleutherAI/gpt-neo-1.3B \
    --tasks wikitext \
    --device 0

I don't get the same results. Am I doing something wrong? Tks!

I also could not get the same results as the one reported (https://huggingface.co/gpt2-large).
I got the below result for gpt2-large:
`

Task
Version
Metric
Value

Stderr

lambada_standard
0
ppl
22.1788
±
0.7741

acc
0.4040
±
0.0068

`

@Yujin-Yujin OpenAI does not report standard Lambada scores, which is why the numbers don’t match. I think we are ultimately not going to allow replication of their numbers, see here for more info and discussion.


Az Module request

Any plan of including the Az Module by default in a future release?

I don't really understand the request, I'm pretty sure Az module is already preinstalled on GitHub agents?


First Webpage Image Larger

First Webpage Image Larger
User Story
AS AN employer
I WANT to view a potential employee's deployed portfolio of work samples
SO THAT I can review samples of their work and assess whether they're a good candidate for an open position
Acceptance Criteria
GIVEN I need to sample a potential employee's previous work
WHEN I am presented with the developer's first application
THEN that application's image should be larger in size than the others

feature/image_tiles


how can set font for all application content and custom library use in it

Thank you for this useful lib. but if use another lib for example dialogs font override not work in that.
it is possible override for them too ?

This will work anywhere there is a view created from the base context of that activity, If a third party lib is instantiating views programmatically then you'll need to apply font programmatically.
You'll need to provide example code/project for a more definitive answer, as I'm not 100% sure what you are asking.


Can bubble popup be used with decluttering card ?

Firstly great work on the bubble card, love how it looks and the functionality is just great.
I wonder if someone knows if it's possible to make use of the bubble popup card along with decluttering card in HA ? - Currently I have buttons on my dashboard which when I hold gives me a custom 'more-info' style popup window, this is automatically generated using decluttering-card as the entity that was pressed is passed as a variable (like so;)
hold_action:
  action: fire-dom-event
  browser_mod:
    service: browser_mod.popup
    data:
      deviceID: this
      content:
        type: custom:decluttering-card
        template: history-timeline-popup
        variables:
          - history-entity: '[[light-entity]]'

I'd really like to switch from using browser_mod.popup to using the bubble popup instead, but I can't figure out if it's possible to have the bubble popup card as a decluttering template and then somehow pass the variable to it.
Obviously what I'm trying to avoid here is having to manually create tons of popups for each entity or room.

Hi and thanks for your feedback! I've just tried and I can confirm that this is not working, decluttering-card modify the code of the pop-up and it's something that I can't fix. I will maybe add a template feature in the future, but only if it's getting too big to maintain for users.


Language ≠ Script

The ‘Language’ dropdown menu contains an inconsistent mix of script and language identifiers. So, for example, ‘Bengali’ which could refer to either the Bengali* language or to the Bengali script, which is also used to write several other languages; ‘Devanagari’ on the other hand refers only to the script, which is used to write numerous languages that exhibit significant orthographic differences that may be reflected in font selection or OpenType Layout behaviour; such orthographic differences are explicitly recognised in the three types of Chinese identified in the list; and ‘Vietnamese’ appears to be the only entry in the ‘Language’ list that unambiguously refers to a language rather than a script.
I would like to suggest that Google Fonts consider moving to a system of two linked menus: one for script and one for language, with the content of the latter depending on the selection of the former. This would enable finer-grain browsing of fonts, and more accurate selection of results, especially if fonts contain a meta table that identifies primary design script or language tags, which could be taken into account in prioritising results. So, for example, almost all Devanagari fonts will support Hindi language reasonably well, but might be particularly tailored to the orthographic conventions of Marathi or Sanskrit (as is the case with fonts in the Tiro Indic collection). Even if fonts do not contain meta tables, it would be a good idea for Google Fonts to tag fonts in a way that prioritises browsing results by script and language, and this would also make it easier to improve specimen language selection. At the moment, for example, all Devanagari fonts show the same Hindi specimen texts, regardless of the orthography targets of the fonts.

Bangla is the preferred endonym for both the language and script.

I just noticed that we now have two linked menus! Fantastic!

I'll fix the Bangla issue.


Implement situation-specific ContractClass parsers

One of the main bottlenecks in contract parsing is the conversion process to match the hash stored in the chain.
We need to allow for the following cases instead:
// Creates the contract from a byte slice and hashes the contents
// without any kind of special preprocessing.
fn from_bytes(buf: &[u8]) -> Result<ContractClass>;

// Creates the contract from a byte slice and hashes the contents
// after converting to the Python serialized JSON format.
fn from_bytes_as_python_json(buf: &[u8]) -> Result<ContractClass>;

// Helper to convert from arbitrary JSON format to Python's one.
fn to_python_json(buf: &[u8]) -> Result<Vec<u8>>;

// Creates the contract while assuming a provided hash to be the
// correct one for this contract. Chaos ensues if it isn't.
// User is responsible for correctness.
fn from_bytes_with_hash_unchecked(buf: &[u8], hash: [u8; 32]) -> Result<ContractClass>;

This way we can minimize overhead in a case-by-case basis.
We would also need to convert the Read and &str versions to wrappers around the bytes based ones.
Please take note that each function takes a slice &[u8] instead of a String to avoid performing a lossy utf-8 conversion.

this is already done in other PRs


Print pad owner

https://jenkins.code.rdkcentral.com/job/github-triggered/25005/ started

https://jenkins.code.rdkcentral.com/job/build-component/60977/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/17530/ started

build-aml-reftv Console Log uploaded to Gist URL: https://gist.github.com/a77295e8d81a274d82324987f02c52cf

build-aml-refboard Console Log uploaded to Gist URL: https://gist.github.com/da31fd460f104d03141c0e8933ff1d25

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132033/

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132034/

https://jenkins.code.rdkcentral.com/job/github-triggered/25029/ started

https://jenkins.code.rdkcentral.com/job/build-component/61011/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/17550/ started

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/29505/ started.

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132112/

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132113/

Smoke tests for ['build-aml-refboard'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132114/

Smoke tests for ['build-brcm-refboard'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132117/

Smoke tests for ['build-aml-reftv'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132115/

Smoke tests for ['build-brcm-va'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132116/

'['build-brcm-va']' Smoke test FAILURE, investigation pending.
Flash BCMREF [SUCCESS]
BCMREF - Video Test [FAILURE]
HLS Video Streaming using aamp-cli [SUCCESS]
DASH Video Streaming using aamp-cli [SUCCESS]
PLAYREADY_DRM Video Streaming using aamp-cli [SUCCESS]
Video Accelerator Home Screen Test [SUCCESS]
Video Accelerator VOD Video Detection Test [SUCCESS]
AAMP CI Tests [FAILURE]
AAMP CI: WebKitBrowser.url : Failed - [about:blank]
AAMP CI: WebApp launch failed
https://jenkins.cmf.code.rdkcentral.com/job/test-bcmref-va-flow/22/

Smoke tests for ['build-brcm-va'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/132132/


Adding Actions to Usage list

I added two of my actions which use actions-toolkit to the usage list in the README.

These are awesome! Thank you for adding them to the list 🎉 ❤ I took a look at the code and I see a couple cases where actions-toolkit can be used more; I'll open some PRs and then merge this in 👍

Just merged in the two PRs you made. Just as a side note, I saw you commented that this made them easier to test, and with that, I'd love to learn how to test actions in general. Is there anywhere specifically you currently know of for documentation on such. I see for shell scripts you could use Bats, but for JS scripts that are placed in actions, what would you recommend looking at?

Sweet!

Is there anywhere specifically you currently know of for documentation on such.

Unfortunately not - the difficulty of testing Actions is one of the main reasons I wanted to write this library. create-an-issue is a good example of a workaround - most of the functionality is outside of the main entrypoint (see IssueCreator), so its easier to write tests for the actual business logic. What it doesn't do is test the entire action, in its Docker container, with all of the context of previous workflows' effects. So - not a solved problem just yet I think!

@JasonEtco Yeah it's a different workflow compared to the applications I usually write so it may take awhile to get my mind wrapped around how to test it. A scenario that may be good to look into is how serverless functions (AWS Lambda, Azure Functions, Google Cloud Functions, etc) are tested given they are kinda similar where they are activated through an event-based system.

Agreed - I think FaaS and Actions are super similar conceptually. Lots of room for tooling there


[Manual Backport 2.x] Updating default compression level for ZStandard Codecs (#8471)

Signed-off-by: Sarthak Aggarwal<EMAIL_ADDRESS>(cherry picked from commit eca5a6cbc4f8454afaebd35e81f4d3c23071d92d)

Description
Backport of #8471
Related Issues
Resolves:
Optimization for #7555
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

@mulugetam - want to bring this to your notice


notice when app engine java components aren't installed

@meltsufin @etanshaul fix #165

$ gcloud components list --format='value(state)' --filter=id:app-engine-java
Your current Cloud SDK version is: 124.0.0
The latest available version is: 125.0.0
name=Not Installed

$ gcloud components list --format='value(state)' --filter=id:app-engine-java

Your current Cloud SDK version is: 119.0.0
The latest available version is: 125.0.0

name=Update Available

Hmm, I should update.

I think you'd want to set --format=json

$ gcloud components list --format='json' --filter=id:app-engine-java
Your current Cloud SDK version is: 124.0.0
The latest available version is: 125.0.0
[
{
"current_version_string": null,
"id": "app-engine-java",
"is_configuration": false,
"is_hidden": false,
"latest_version_string": "1.9.42",
"name": "gcloud app Java Extensions",
"size": 142219489,
"state": {
"name": "Not Installed"
}
}
]
It really shouldn't be saying
Your current Cloud SDK version is: 124.0.0
The latest available version is: 125.0.0

It's only printing that to STDERR; it's very easy to ignore.
On Wed, Sep 7, 2016 at 6:35 PM, Elliotte Rusty Harold <
<EMAIL_ADDRESS>wrote:

$ gcloud components list --format='json' --filter=id:app-engine-java
Your current Cloud SDK version is: 124.0.0
The latest available version is: 125.0.0
[
{
"current_version_string": null,
"id": "app-engine-java",
"is_configuration": false,
"is_hidden": false,
"latest_version_string": "1.9.42",
"name": "gcloud app Java Extensions",
"size": 142219489,
"state": {
"name": "Not Installed"
}
}
]
It really shouldn't be saying
Your current Cloud SDK version is: 124.0.0
The latest available version is: 125.0.0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-plugins-core/pull/206#issuecomment-245440882,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1VgZViWTpb33--gZgqPCvRJvDjYvUAks5qnzwjgaJpZM4J3QGM
.

@elharo are we closing this to start over?

Yes, the version that calls gcloud components shares very little with this branch.

reopening since we may do this


Create Calendar View for Tasks

Task

Create calendar view for tasks to show on a weekly/monthly view

Deliverables

Code related to visualizing tasks in a calendar format

Due 12/7

No longer doing this.


ServiceLoader discoverability

Per discussion in https://github.com/confluentinc/kafka-images/issues/102, we need to ensure the config provider is discoverable using ServiceLoader

I was coming at this from the Kafka side. I see there's always #2 to resolve this.

Workaround: https://github.com/confluentinc/cp-docker-images/issues/828#issuecomment-720533576

@vutkin Version 0.1.0 added the Service Loader configuration. So it should be solved now.

Unfortunately no, it fails with same error. Workaround helps

Yeah, sorry, I overlooked ... 0.1.1 added the ServiceLoader.


启动canbus时，使用cyber_moniter的先后顺序会影响通道的数据类型

问题描述：
启动canbus时，使用cyber_monitor的先后顺序会影响通道的数据类型
截图：
先启动canbus，后使用cyber_monitor查看通道数据的类型

先使用cyber_monitor查看通道，再启动canbus后的数据类型

I have encountered this situation. The reason is that there are 2 nodes that are sending messages. For example, one is in the background and has not been killed.
You can use ps -ef | grep canbus to see if there are multiple processes


How can i use this for geo reversing with google maps as you wrote we can use

Can you explain it better?
Would you to reverse geocoding a location coordinate to an address using google maps apis?

I was asking How can I get reverse geocoding a location coordinate to an address using google maps apis

Use the following method to use Google placemarks:
Location.reverse(location: loc, using: .google, onSuccess: successHandler, onError: errorHandler)

is this method needs to be called on main queue .Because Google mandate call this method on main queue.

Haven't tried that....

Actually I've removed it from current version. I'll update it soon as I can to re-introduce this feature.

Any updates on this one yet? @malcommac

A new version is available on master and supports both reverse geocoding and place autocomplete.
Thanks.


Add feather effect

Fixes #886
Adds feather tool to Effects > Stylize (Considered blur, but I think this is more fitting). This softens the edges of an image without spilling blur out of the image or blurring the contents of the image.
Radius adjusts the strength of the effect, Transparency Threshold is used to determine the "outline" of the object. Requires object be surrounded by transparency to function.
Example with a test image.

Only known issue I am aware of is that the effect will spill over outside selection in preview, however it does not spill over after application. Unsure how to fix this.

One other note is that you can also add tests for the effect in https://github.com/PintaProject/Pinta/blob/master/tests/Pinta.Effects.Tests/EffectsTest.cs

One other note is that you can also add tests for the effect in https://github.com/PintaProject/Pinta/blob/master/tests/Pinta.Effects.Tests/EffectsTest.cs

I can, but because feather replies on transparency, the output is identical to the test input image.

One other note is that you can also add tests for the effect in https://github.com/PintaProject/Pinta/blob/master/tests/Pinta.Effects.Tests/EffectsTest.cs

I can, but because feather replies on transparency, the output is identical to the test input image.

Right, good point :) Feel free to adjust that TestEffect method to let you choose a different input image if that works

Should be good now.

Implemented feedback. Should be thread safe now.
While I did implement ISystemService, I am not very familiar with DI so do correct me if I messed up a part of it.
I said I was avoiding working by-pixel instead of by-border pixel for performance reasons, but I feel like the new code is more responsive. (Guesswork. Haven't measured it.)


function not being started on net6 & v4 functions

I created a function with Kafka trigger and I started facing the issue on Visual Studio :-
This exception was originally thrown at this call stack:
System.Uri.CreateThis(string, bool, System.UriKind, System.UriCreationOptions)
System.Uri.Uri(string)
Grpc.Net.Client.GrpcChannel.ForAddress(string, Grpc.Net.Client.GrpcChannelOptions)
Microsoft.Extensions.DependencyInjection.GrpcServiceCollectionExtensions.AddGrpc.AnonymousMethod__1_1(System.IServiceProvider) in GrpcServiceCollectionExtensions.cs
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitFactory(Microsoft.Extensions.DependencyInjection.ServiceLookup.FactoryCallSite, Microsoft.Extensions.DependencyInjection.ServiceLookup.RuntimeResolverContext)
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor<TArgument, TResult>.VisitCallSiteMain(Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceCallSite, TArgument)
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitCache(Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceCallSite, Microsoft.Extensions.DependencyInjection.ServiceLookup.RuntimeResolverContext, Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope, Microsoft.Extensions.DependencyInjection.ServiceLookup.RuntimeResolverLock)
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitRootCache(Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceCallSite, Microsoft.Extensions.DependencyInjection.ServiceLookup.RuntimeResolverContext)
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor<TArgument, TResult>.VisitCallSite(Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceCallSite, TArgument)
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitConstructor(Microsoft.Extensions.DependencyInjection.ServiceLookup.ConstructorCallSite, Microsoft.Extensions.DependencyInjection.ServiceLookup.RuntimeResolverContext)
...
[Call Stack Truncated]
While trying with Core tools :-
[2022-01-06T20:15:32.195Z] No job functions found. Try making your job classes and methods public. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.).
I tried to find this function in hostbuilder it was not available there, could you suggest if we need to fix this how can we ?
Code available on repository -- https://github.com/shrohilla/dotnet-out-proc
following the documentation -- https://docs.microsoft.com/en-us/azure/azure-functions/dotnet-isolated-process-guide
cc @fabiocav @lpapudippu

@shrohilla how are you running the worker? The exception above indicates a failure parsing the host endpoint, which is automatically passed in by the host when running.

@shrohilla looking at the project in the repo, looks like the project was modified and is missing the SDK reference (that is currently commented out) and using an older version of the worker.
Can you please update to the following?
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.3.0" OutputItemType="Analyzer" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.5.2" />

Without the Worker.Sdk package reference, VS will not recognize the project as a function app (See the below screenshot, function app icon missing). So, the command VS issues for your debug session may not be correct.

Once you add a reference to Microsoft.Azure.Functions.Worker.Sdk, it will be detected as a function app and F5 debugging should work as expected.

Also, consider upgrading to the latest stable version of packages as suggested above.

Closing this as I got confirmation from @shrohilla that this was resolved.

Thanks alot @fabiocav this is resolved.


[Build]: Add makefile rule to build in docker

What this PR does / why we need it:
Which issue this PR fixes:

fixes #
Special notes for your reviewer:
Release note:

@sejust PTAL~


Core: forbid assignments in conditions

Move the WordPress.CodeAnalysis.AssignmentInCondition from the Extra ruleset to the Core ruleset.
As per the proposal in https://make.wordpress.org/core/2019/03/26/coding-standards-updates-for-php-5-6/
Open actions:

[x] Adjust Core PHP handbook to mention this rule in the Clever code section

/cc @pento

Handbook is updated.

Thanks @pento !
I've updated the PR to use the exact text from the handbook in the inline documentation, as well as changed the placement of the rule to match the order of the rules in the handbook.