gp-saml-gui: init at c557c32

Motivation for this change
#103167 gp-saml-gui is a wrapper to authenticate with GlobalProtect when using SAML authorization, which requires a web browser. This is usually used in addition to openconnect.
Things done

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

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

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

This has been updated per recommendations. Thanks!

Please squash the commits together.


L2G feature matrix revision

As a developer I want to revise and refactor some of the logic in the L2G feature matrix generation step
Background
tbd.
Tasks

[ ] Refactor logic for assigning potential target genes to studyLocusIds, to include genes in proximity to all variants in the studyLocus, instead of just the top hit.
[ ] Implement logic to assign potential target genes which have a colocalisation result with a studyLocus, but is out of range from the studyLocus.
[ ] Include additional distance to footprint feature in the L2G model, using the latest VEP variant index.
[ ] Implement logic to save the feature matrix when it is generated.
[ ] Implement logic to merge features across studies which have the same EFO, to address the missingness rate of features in L2G.
[ ] Refactor logic for gold standard Negative selection process, to prioritise keeping GSNs with non-distance features, followed by prioritising the closest genes.
[ ] Implement logic to remove studyLocus where all potential target genes have only distance-based features from the training set.

As a general plan looks good to me. @ireneisdoomed and @DSuveges please have a look to, but we need more granular plan on this. In general, the top priority for us is to reach the level of clean credible set level feature matrix as a first step implementing suggestions above.

@ireneisdoomed which tasks from the list above are going to be implemented in the l2g fm refactoring?

None. The refactoring I am doing is to solve #3252

@xyg123

Refactor logic for assigning potential target genes to studyLocusIds, to include genes in proximity to all variants in the studyLocus, instead of just the top hit.

I guess it should be covered by issue adding tss and footprint based features? https://github.com/opentargets/issues/issues/3258

Implement logic to assign potential target genes which have a colocalisation result with a studyLocus, but is out of range from the studyLocus.

I guess it should be covered by issue adding coloc based features? https://github.com/opentargets/issues/issues/3252

Include additional distance to footprint feature in the L2G model, using the latest VEP variant index.
https://github.com/opentargets/issues/issues/3258

The rest is ok.

We need to re-write this issue next week.

The issue was spllited on several issues.


bug: remove the _getEndAdornment() from NativeAutocomplete.js

Description
remove the _getEndAdornment() from NativeAutocomplete.js as it returns undefined and stops the authentication module.

The part has been commented out and merged.


Using the recommended label syntax for maintainer.

Fixes #1530

@simonpasquier  Here is the correct PR :)

Thanks for the quick fix @tariq1890!


add sigstore verification

add sigstore verification in shield based on sigstore/cosign verification codes
support signatureType: sigstore for keyConfig in IntegrityShield CR
fix a dryrun issue

these codes were merged into dev/refactor branch with this commit https://github.com/IBM/integrity-enforcer/pull/331/commits/064f2bcbec6160d2165c9462d2b721c01c837192 .
close this PR.


fix app_zh_Hans.arb compilation errors

otherwise, compilation fails with:
.dart_tool/flutter_gen/gen_l10n/app_localizations_zh.dart:1056:1: Error: Expected an identifier, but got ','.
Try inserting an identifier before ','.
,
^
.dart_tool/flutter_gen/gen_l10n/app_localizations_zh.dart:1056:1: Error: Expected named argument.
,
^
.dart_tool/flutter_gen/gen_l10n/app_localizations_zh.dart:1053:33: Error: No named parameter with the name '#2'.
    return intl.Intl.pluralLogic(
                                ^^...
/opt/hostedtoolcache/flutter/stable-3.3.10-x64/.pub-cache/hosted/pub.dartlang.org/intl-0.17.0/lib/intl.dart:296:12: Context: Found this candidate, but the arguments don't match.
  static T pluralLogic<T>(num howMany,

(I also updated it at weblate.org, so this PR is probably not needed as it should be automagically marged?)

Yeah, this file is auto-generated, so we better wait for Weblate's pull request.


Bridgestan.jl fails to compile for julia version 1.7.2

When I try using BridgeStan I get the following error:
[ Info: Precompiling BridgeStan [c88b6f0a-829e-4b0b-94b7-f06ab5908f5a]
ERROR: LoadError: syntax: expected assignment after "const"
Stacktrace:
  [1] top-level scope
    @ ~/.julia/packages/BridgeStan/YcHCB/src/model.jl:22
  [2] include(mod::Module, _path::String)
    @ Base ./Base.jl:418
  [3] include(x::String)
    @ BridgeStan ~/.julia/packages/BridgeStan/YcHCB/src/BridgeStan.jl:1
  [4] top-level scope
    @ ~/.julia/packages/BridgeStan/YcHCB/src/BridgeStan.jl:24
  [5] include
    @ ./Base.jl:418 [inlined]
  [6] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt64}}, source::String)
    @ Base ./loading.jl:1318
  [7] top-level scope
    @ none:1
  [8] eval
    @ ./boot.jl:373 [inlined]
  [9] eval(x::Expr)
    @ Base.MainInclude ./client.jl:453
 [10] top-level scope
    @ none:1

removing the const qualifiers in model.jl enables me to at least compile/include BridgeStan.

Ah, just saw that the prerequisites say Julia 1.8, I'm guessing it works for that.

Correct, the const members are a 1.8+ feature. You're as good as anyone to ask (@roualdes and I discussed this briefly when I introduced them) - how conservative should we be on Julia features from newer versions? I have a pretty good sense for a language like Python, but for Julia I don't know if most people update regularly or if there are holdouts on ancient versions.

@nsiccha Are you satisfied with the documented Julia 1.8 requirement?  If so, can we close this issue?

@roualdes

Are you satisfied with the documented Julia 1.8 requirement? If so, can we close this issue?

Sure, that was my oversight.
@WardBrian

how conservative should we be on Julia features from newer versions?

I have no idea. Maybe it makes sense to aim for compatibility with versions which support AdvancedHMC? That's what I use BridgeStan with currently.
I also don't know how easy it is to maintain different Julia versions on one machine, I'd assume not that difficult. It's obviously up to you how you handle this :)
That being said, the benefit of using the 1.8 features appears to be marginal.


Gather support and vectorisation fixes for LLVM code generation

This PR fixes vector code generation, particularly now:

The pointer to the element in the array inside instance struct
id bitcasted to vector after the GEP instruction.
Local variables in the loop are vectorised, and the array
indexing can now be performed with a vector of indices.
Gather instruction support has been added.

The tests for IR and vectorised code execution are created
accordingly.

[x] TODO: add IR tests for gather instruction

I will see why CI is failing with below error:


Addressing the MQTT reconnection bug

Currently the application is not reconnecting to the MQTT Server,

Initial testing of this build looks to be correctly building a reconnection to the upstream MQTT servers.


messaging layer should include receiver metadata such as origination

 Issue by ghaskins
Tuesday Dec 08, 2015 at 18:34 GMT
Originally opened as https://github.com/openblockchain/obc-peer/issues/371

Messages delivered through the framework currently consist solely of a deserialized frame representing on-wire structures (e.g. RecvMsg(msg *pb.OpenchainMessage)).  Anything within the frame should generally be treated as suspect without further validation (such as checking a digital signature).  However, there are use-cases where locally source metadata would be useful: For example, a TLS encrypted channel or a chain-code to chain-code invoke have attributes that are already ascertained by the peer without falling back to a more heavy-handed in-band mechanism such as signature verification.  The proposal is that the messaging layer should include a metadata frame that the peer can use to convey additional information along with a message between elements within itself.  This could minimally include message origination (e.g. peer "X", or chaincode instance "Y") and perhaps a flag to indicate the confidence in the origination assertion (e.g. "verified=true" if the message arrived over TLS or emanated from a chaincode invoke operation).  e.g. RecvMsg(meta *Meta, msg *pb.OpenchainMessage)

 Comment by ghaskins
Tuesday Dec 08, 2015 at 18:34 GMT

@jeffgarratt @binhn @ecb

 Comment by ghaskins
Tuesday Dec 08, 2015 at 22:27 GMT

@jeffgarratt I suspect we are talking about slightly different things (but correct me if I am misinterpreting your response).  While I haven't looked at the message structure of HELLO in any detail, I recognize that any amount of required endpoint identity services may certainly be negotiated here.  However, that isn't the problem as I see it.
The problem is two-fold:

every subsequent message on the stream would likewise need to be validated as authentic based on the identity established in the HELLO (such as part of a TLS session or other Layer 6 service at peer ingress, per datagram consumer point, etc).  This may in fact already be happening today, but...

...the info cannot derive from on-wire data (for security) and it also would ideally carry through with the message (to avoid problems like needing to re-authenticate a signature over and over again at each consumption point, when the protocol permits).

Using at least the example of a consenter module, it currently has no peer flow discrimination (IIUC).  It only receives (on-wire derived) datagrams via the RecvMsg() interface where any ability to tie it back to an underlying flow is lost and therefore has to assume every field in the message is suspect.
Further, there are also cases where it may be advantageous to have messages generated internally (such as modeling a chaincode-to-chaincode "invoke" as an injected message, or to implement "distributed oracles" such as timer expiry, physical delivery confirmation, etc.  In these cases, digital signatures may be a poor mechanism because the generation and verification would happen within the same process context just to work around the lack of a trust context.
There are certain assertions that a peer can make for a given flow with certain properties (such as messages arriving over TLS or emanating from chaincode/distributed-oracles) that would be useful to downstream recipients within the same context.
In light of this, what I am proposing would be useful is to have the concept of metadata that is attached to a given message that is internally trusted because it's generated in-context rather than passed in on the wire.  This metadata could be modeled as fields within an existing message payload definition and enforced by ensuring that we overwrite those fields on ingress from the wire.  However, that seems a little risky for a few reasons.  I personally think it would be cleaner to implement a separate structure outside of the externally visible message schema, though I would settle for in-band payload over not having the feature at all.
I think it might be cleaner to think of it as (simplified)
type Message struct {
from string
verified bool
payload *pb.OpenchainMessage
}
..RecvMsg(msg *Message) {}
As an example use-case, envision a contract that accepts funds from two counter-parties.  Using standard "bitcoin account" semantics, its easy to understand that the cryptographic identities of the two parties could be codified into the contract by means of properties passed at instantiation.  These parameters could help enact the appropriate rights and restrictions w.r.t. interacting with the contract such that the contract could verify digital signatures for requests that come in.
Now suppose I want to build a composite contract, one in which one or more of the counterparties might be other contracts themselves (basically a chaincode "invoke").  Without considerations such as as above, it would be difficult for me to express this contact such that I could enforce counterparty identity.  If I expect a digital signature, I would need the counter-party contract to posses a private key and an ability to generate a signature (not an easy prospect when the ledger is at least partly public).  If I disable identity verification, anyone could inject a message that impersonates the counterparty contract.  However, if I have a trust-context, I could simply code the contract to only accept the "from" field from a particular identity, and know that the VP is enforcing the authenticity of the metadata.

 Comment by jeffgarratt
Tuesday Dec 08, 2015 at 21:01 GMT

the current system includes a HelloMessage structure in the HELLO message exchanged between peers which contains a PeerEndpoint structure.  This structure is intended to be identifier for the opposite end of the communication stream.  I think the important thing to remember is that the FRAME (ie msg) exists within a pre-existing context of the established stream (through HELLO protocol).  I agree that additional information may be desired and encoded within the individual message within the context of the established stream, which is why the payload is opaque ([] byte), and intended be defined by the message TYPE implementer.  @binhn @ghaskins @ecb


[Fix] CountedMap::set now takes Counter into account

This potentially needs a migration for all the pallets using CountedMap and set, but I'm not sure how actionable that is. At the very least all the future occurrences will be fixed.
Fixes https://github.com/paritytech/substrate/issues/13212
Integration guide
Check if you ever used CountedStorageMap::set. In that case it is necessary to call initialize_counter since otherwise the stored count and the actual number of elements will differ. There is no migration provided but can be done in https://github.com/paritytech/substrate/issues/13218.

🙏 Did you also check the other modification funtions like try_mutate_exists and mutate_exists?

Yeah, there were already tests for those. It seems like only set was affected, 0 tests in both StorageMap and CountedMap.

bot merge

This potentially needs a migration for all the pallets using CountedMap and set, but I'm not sure how actionable that is. At the very least all the future occurrences will be fixed. Fixes #13212

Would be good if you write a more proper description here about how/when a parachain/solo-chain team needs to do a migration.

Would be good if you write a more proper description here about how/when a parachain/solo-chain team needs to do a migration.

Done in https://github.com/paritytech/substrate/issues/13218. Basically anybody using set method on a CountedMap before this PR has been merged would need to write a migration that calls initialize_counter on this map.

I updated the MR description to mention it.
The point is that para-chain teams are going to look at this MR and not understand what changes for them, so we need to give some details about migrations et al.

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:
https://forum.polkadot.network/t/polkadot-release-analysis-v0-9-38/2122/1

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:
https://forum.polkadot.network/t/substrate-developer-newsletter-february-2023-notable-updates-for-frame-devs-new-deep-dives-released-calling-builders-to-apply-to-rfps-and-bounties/2137/1


Delete examples/gemini/python/docs-agent directory

Description of the change

Motivation

Type of change
Choose one: (Bug fix | Feature request | Documentation | Other)
Checklist

[ ] I have performed a self-review of my code.
[ ] I have added detailed comments to my code where applicable.
[ ] I have verified that my change does not break existing code.
[ ] My PR is based on the latest changes of the main branch (if unsure, please run git pull --rebase upstream main).
[ ] I am familiar with the Google Style Guide for the language I have coded in.
[ ] I have read through the Contributing Guide and signed the Contributor License Agreement.

I think this PR may have been an error?


fix: Update Teradata rtrim() to account for tabs and other whitespace

Closes #1272
This updates the default RTRIM(column, " ") for Teradata so that it also accounts for tabs, newlines, and other whitespace characters.

/gcbrun

Hi,
I suggest adding this to our test suite as follows:
Change this line to
,(3,3,3,3,3
,12345678901234567890,1234567890123456789012345,123.3,123456.3,12345678.3
,'Hello DVT','C ','Hello DVT' || _latin '20090B0A0D0C'XCV
,DATE'1970-01-03',TIMESTAMP'1970-01-03 00:00:03'
,CAST('1970-01-03 00:00:03.000-03:00' AS TIMESTAMP(3) WITH TIME ZONE)

This confirms that RTRIM on the Teradata end strips the white space characters at the end before comparison with BQ. Change this line to
,(2,2,2,2,2
 ,NUMERIC '12345678901234567890'
 ,BIGNUMERIC '1234567890123456789012345'
 ,NUMERIC '123.22',123456.2,12345678.2
 ,'Hello DVT','B ','Hello DVT \t\v\n\r\f''
 ,DATE '1970-01-02',DATETIME '1970-01-02 00:00:02'
 ,TIMESTAMP '1970-01-02 00:00:02-02:00')

This confirms that RTRIM on BigQuery removes the whitespace characters prior to comparison with other engines. We could do this for other engines - but this is a good start for Teradata and BigQuery.
Thanks.
Sundar Mudupalli

Remember that dvt_core_types is used for column validation testing (and perhaps comparison fields too), we need to be sure we don't break those tests. Therefore we would need to add the extra characters across all engines because they all get compared to BigQuery, which is no big deal, just pointing it out.

We already have a table in TD for testing CHAR PKs with --trim-string-pks. I can add the tabs in that table so it won't disturb the core_types table.

Neil - you are right. In column validation, we use the string length for min, max and sum, so it can create an issue - especially if the dvt_core_types for other databases are not updated. Between Teradata and BQ, the min/max/sum may remain the same - though I had not considered that when I first proposed this.
Thank you.
Sundar Mudupalli

@sundar-mudupalli-work I added extra tabs and whitespace to our existing char_id table which tests the use of -tsp and CHAR column validation. Before this fix, the tabs would have broken the hash validation and marked the rows as failures.


Error downloading from twist moe

Below image show the settings I have done

This error came up
[INFO] : Twist URL detected...
[INFO] : Collecting download links...
[ERROR] : Expecting value: line 1 column 1 (char 0)
[INFO] : Retrying to collect download links...
[ERROR] : Expecting value: line 1 column 1 (char 0)
[ERROR] : Failed to retrieve download links!

anyone have any idea how to resolve this??

Maybe because they used source "https://twistcdn.bunny.sh" and not "air-cdn.twist.moe" as they updated the servers a long time ago.

Anyway i found a better alternatives, so thanks for the reply man
On Fri, Mar 5, 2021, 1:19 AM Ikel12<EMAIL_ADDRESS>wrote:

Maybe because they used source "https://twistcdn.bunny.sh" and not
"air-cdn.twist.moe" as they updated the servers a long time ago.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Oshan96/monkey-dl/issues/49#issuecomment-790784580,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APABW7NBQBSXNM6NXFW7VODTB66KPANCNFSM4XMFVOCA
.

Anyway i found a better alternatives, so thanks for the reply man

care to share?

https://github.com/anime-dl/anime-downloader


CORS issue

Question Checklist

[x] Updated Amplitude.js to the latest version
[x] I read the docs
[x] I searched for existing GitHub issues

Question Subject
Getting around the CORS issue
Question Description
Having trouble with a CORS header response. I am using an api that works just fine outside of Amplitude. I can create an audio element that uses an external source from the same api with no issue. Using the same link within Amplitude, I get the following error in my console:
Access to audio at 'https://hearthis.at/...' from origin 'https://...' has been blocked by CORS policy: The 'Access-Control-Allow-Origin' header contains multiple values '*, *', but only one is allowed.
I have ensured that the "live" flag is set. Checking in the network tab in DevTools shows that the header from the source does have "," which, again, works fine outside of Amplitude. Kind of at a loss. I'd love to use the plugin, but this limitation is a little strange. I could allow uploading larger files if I have to, I just get more out of the api for hearthis.at (automatic waveform data, duration, etc)

Closing due to inactivity. Could be related to #317


fix(FocusManager): removes all stories except ColumnWithRows

Description
This PR does the following:

removes all stories except for ColumnWithRows
adds two controls: direction and wrapSelected

References
LUI-766
Testing

pull branch
yarn start
go to Focus Manager => Column with Rows story
test added controls to make sure expected behavior from controls work

Automation
The tests for Focus Manager will probably need to be updated since we removed a few stories and and added controls to the one existing story
Checklist

[x] all commented code has been removed
[x] any new console issues have been resolved
[x] code linter and formatter has been run
[x] test coverage meets repo requirements
[x] PR name matches the expected semantic-commit syntax

Sanity Tests Execution Failed.

:tada: This PR is included in version @lightningjs/ui-components-v2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Ye glm script

@ye-zhi
I think you can try to merge and work on the tests /travis as well.
Make sure to have the tests for your functions and locate them in the good directories (functions or script)


CSharp generator - check for null response content.

Checks if the response._Content is null before attempting to read the stream. Fixes #1291

I'll release today or tomorrow, ok?

Good stuff, cheers.


NPM Download

Are you guys listed in the NPM download site?

Do you mean here? https://www.npmjs.com/package/apexcharts


feat: add support to show logo for oidc provider

This PR will add following changes:

Add support to show logo for oidc provider.
RIght now only one logo is added but we can add more logos just by adding the svg and adding the entry in oidcLogoMap.

Export Container component so user can build custom screens which follow same styling as sdk components.

Call ListCurrentUserGroups and GetOrganizationsByCurrentUser apis as ListUserGroups(userId) and GetOrganizationsByUser(userId) returns 503.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 40.99%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4338

Relevant Lines:
10583

💛  - Coveralls


extension support for loading images using headers

Some manga websites require headers in order to load images

Fixed will be available in the next release


Release lucet, provide a crate

I'm trying to build lucet using the nix package manager, using the method described here. Using git dependencies is somewhat problematic and it would be much easier to fetch lucet as a crate -- or at least to avoid submodules.
Could lucet be released on crates.io?

Yes, we'd like to do this, we just never got around to it. Thanks for the reminder!

Hi Nicolas!
Good news!
https://crates.io/search?q=lucet
You can now find the Lucet crates crates.io  🎉

Hi Frank, this is awesome. Thanks!

@nmattia Could you build lucet using Nix? If so, can you share the Nix expression? Thanks a lot

@antoinerg yes I managed using naersk.
If you need to build lucet itself, here's how to do it: https://github.com/nmattia/naersk/blob/2d57092902c39dae5934bdbe48e1907961080009/test.nix#L58-L74
If you just require it as a dependency this should work:
naersk.buildPackage ./path/to/code
  { buildInputs = [ cmake python3 ]; }


Update list users

This would merge the updated new layouts for the employees and organizations. The majority of these changes revolve around the html and any code necessary to get the bare minimum to compile was taken in order to fulfill the request on a basic ui level.

Closing in favor of #128 , as this PR's content was merged to save review time.


Fix mistaken use of dollar sign in cddl files

Description
At the moment , cddl generate that we use for cddl roundtrips does NOT generate values for these types that we prefixed with $.
The types for which values ARE generated correctly that are prefixed with $ (eg: $hash28, $vkey), are also defined in the same way, prefixed by dollar.
It's not very clear from the spec that this should be the case, but running cddl generate makes it  clear that it's simply ignoring $required_signers,  $treasury_withdrawals_action and $new_committee when defined in the way they are currently in the cddl.

Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
versioning process.
[ ] The version bounds in .cabal files for all affected packages are updated. If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
[ ] All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages. New section is never added with the code changes. (See RELEASING.md)
[x] Code is formatted with fourmolu (use scripts/fourmolize.sh)
[x] Cabal files are formatted (use scripts/cabal-format.sh)
[x] hie.yaml has been updated (use scripts/gen-hie.sh)
[ ] Self-reviewed the diff

If you want to test this yourself, do this:

Create a file a.cddl with this content:

required_signers = nonempty_set<$addr_keyhash>
addr_keyhash          = $hash28
$hash28 /= bytes .size 28
nonempty_set<a> = [+ a]

run:  nix-shell -p  cddl
run: cddl a.cddl generate 1
observe the error
change first line in a.cddl to required_signers = nonempty_set<addr_keyhash>
run: cddl a.cddl generate 1

The reason we don't get an error in tests is  because it's optional this in transaction_body:
 , ? 14 : required_signers

Or for GovAction:
gov_action =
  [
   new_committee
  //  treasury_withdrawals_action
  ]

set<a> = [a]
$hash28 /= bytes .size 28
$hash32 /= bytes .size 32
reward_account = bytes
coin = uint
unit_interval = #6.30([uint, uint])

epoch = uint

addr_keyhash           = $hash28
scripthash            = $hash28

credential =
  [  0, addr_keyhash
  // 1, scripthash
  ]
committee_cold_credential = credential
committee = [{ committee_cold_credential => epoch }, unit_interval]

gov_action_id =
  [ transaction_id   : $hash32
  , gov_action_index : uint
  ]

new_committee = (4, gov_action_id / null, set<committee_cold_credential>, committee)
treasury_withdrawals_action = (2, { reward_account => coin })

With  the current version in master, which is:
new_committee = (4, gov_action_id / null, set<$committee_cold_credential>, committee)
treasury_withdrawals_action = (2, ${ reward_account => coin })

cddl b.cddl generate throws an error.
If we remove the $ as my PR does, then values are generated as expected

wow, great catch!


Provide based docker images in docker hub

These scripts are nice, specially the ones to create custom domains on top of the base images, but it would be even simpler if these images were created and uploaded by you directly to docker hub or your registry of choice.

These images cannot be uploaded to the Docker Hub because they require Oracle proprietary software which requires the acceptance of the Oracle license terms prior to download/use.
We are investigating our own Oracle Docker Registry but this has no ETA.


presentation

Add detailed example of composition and how they transform the string
negatives

More useful if the audience learn something or I learn something
adjust beta parameter so that late stage programs get added
no need to go over the code
presentation: show people things such that they will learn; or at least understand and might have interesting feedback;


Standards

We need to add uniform code standards to ensure everything is consistent.

Done in #6 .


Fixed cors issue

I had cors issues and thought I'd push my fix.
How were you guys able to use piston without dealing with cors issues?

Piston has CORS disabled for a reason - The API allows for you to modify the system (installing packages) from it, so it shouldn't be directly called from the browser.
Instead it is encouraged to proxy the execute requests through from some sort of backend, like is done with public piston, and is why we enable CORS on it.

How about configurable from environment and by default disabled?
It is likely a common use case to run piston on a different port to the web server so it would be useful.
const cors = require('cors');

function getCorsOrigin() {
	const corsOriginFromEnv = process.env.PISTON_CORS_ORIGINS ? process.env.PISTON_CORS_ORIGINS.trim() : 'false'
	return (corsOriginFromEnv && corsOriginFromEnv.toLowerCase() === 'false') ? false : [corsOriginFromEnv.split(",")]
}

const corsOptions = {
    origin: getCorsOrigin(),
    methods:
        "GET,HEAD,PUT,PATCH,POST,DELETE",
    allowedHeaders:
        "Access-Control-Allow-Headers,Access-Control-Allow-Origin,Access-Control-Request-Method,Access-Control-Request-Headers,Origin,Cache-Control,Content-Type,X-Token,X-Refresh-Token",
    credentials: true,
    preflightContinue: false,
    optionsSuccessStatus: 204
};
router.use(cors(corsOptions))


Change config

Also, I have the suggestion to improve inflating views in fragments - in case that we never use all params from method onCreateView we can create base fragment with default implementation and reduce unused params, like:
Old:
override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
    return inflater.inflate(R.layout.fragment_search_with_categories, container, false)
}

New:
override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
    int layoutId = getLayoutId()
    return inflater.inflate(layoutId, container, false)
}

abstract fun getLayoutId(): @LayoutRes Int


fixed a bunch of nits

tweaked names a bit
replaced context() with wrap() to propagate cancellation
fixed gossipnet msg size limits
added Debug for the public type

Great, changing BatchStoreState::last from a SyncBatch to a BatchNumber makes a whole lot more sense then unconditionally pushing all the L2 blocks to gossip peers after each L1 batch whether they need it or not 👍 We discussed this with @brunoffranca and @RomanBrodetski on standups but none of us were sure whether it's intentional, although the fact that the .proto file said it was a "QC" made it suspicious that it was a copy paste error from the similar block related artifacts.
Perhaps the title can be updated to reflect this change as it will also have to be fixed on the era side, ie. a bit more significant than a nit.


TypeError: Missing required parameter 'digestmod'

Hi! I keep receiving the following error while running ldapdomaindump, do you have any idea what might be going wrong?
ldapdomaindump $target -u 'BLACKFIELD.local\support' -p '#00^BlackKnight'
[*] Connecting to host...
[*] Binding to host
Traceback (most recent call last):
  File "/usr/bin/ldapdomaindump", line 3, in <module>
    ldapdomaindump.main()
  File "/usr/lib/python3/dist-packages/ldapdomaindump/__init__.py", line 912, in main
    if not c.bind():
  File "/usr/lib/python3/dist-packages/ldap3/core/connection.py", line 563, in bind
    response = self.do_ntlm_bind(controls)
  File "/usr/lib/python3/dist-packages/ldap3/core/connection.py", line 1302, in do_ntlm_bind
    request = bind_operation(self.version, 'SICILY_RESPONSE_NTLM', ntlm_client, result['server_creds'])
  File "/usr/lib/python3/dist-packages/ldap3/operation/bind.py", line 81, in bind_operation
    server_creds = name.create_authenticate_message()
  File "/usr/lib/python3/dist-packages/ldap3/utils/ntlm.py", line 379, in create_authenticate_message
    nt_challenge_response = self.compute_nt_response()
  File "/usr/lib/python3/dist-packages/ldap3/utils/ntlm.py", line 485, in compute_nt_response
    response_key_nt = self.ntowf_v2()
  File "/usr/lib/python3/dist-packages/ldap3/utils/ntlm.py", line 497, in ntowf_v2
    return hmac.new(password_digest, (self.user_name.upper() + self.user_domain).encode('utf-16-le')).digest()
  File "/usr/lib/python3.8/hmac.py", line 153, in new
    return HMAC(key, msg, digestmod)
  File "/usr/lib/python3.8/hmac.py", line 51, in __init__
    raise TypeError("Missing required parameter 'digestmod'.")
TypeError: Missing required parameter 'digestmod'.

Make sure you remove all current versions of ldap3 and install the latest, that one should work fine with python 3.8

Thank you! Can confirm it works.


Processing restructure

Description
Changes proposed in this pull request:

Restructure processing to make it to support easier dynamic configuration for different handlers

Related issue(s)

https://github.com/kyma-project/api-gateway/issues/95

This looks good in my opinion, maybe just adding some comments about what the interfaces and their function do would help a little bit with readability, but otherwise 👍


[hotfix][test] Remove unused waitUntilJobInitializationFinished of Te…

…stUtils

What is the purpose of the change
Remove unused waitUntilJobInitializationFinished

@flinkbot run azure


Add new SIF 2.4.0 release, tidy previous SIF release notes

Description / Motivation

Added new release download SIF 2.4.0. Why: It was recently added to dev.sitecore.net. Is needed with forthcoming SXP 10.4 release. Needed to do a dry run release test, prior to the big SXP 10.4 release coming in late April!
Minor tidying of release notes for previous SIF 2.x releases. Why: Some got messed up a little when porting from dev.sitecore.net. Some were already messy in dev.sitecore.net.

How Has This Been Tested?

Viewed documentation results in the Notepad++ markdown viewer panel.
Did page by page comparisons between my new .md file and page in dev.sitecore.net

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] Documentation update (non-breaking change; modified files are limited to the /data directory or other markdown files)

Checklist:

[X] I have read the Contributing guide.
[X] My code/comments/docs fully adhere to the Code of Conduct.
[ ] My change is a code change.
[X] My change is a documentation change and there are NO other updates required.
[ ] My change has new or updated images which are stored in the /public/images folder that need to be migrated to Sitecore DAM

Deployment authorized, should appear on Vercel in a few minutes

Deployment for latest change is now authorized to push to Vercel preview environment.

Marcus has reviewed and approved the latest PR


GetItems Recursive

Looks like in GoogleDriveAdaptor: line 850. Need to prepend $dirname to $id. Otherwise recursive lookup not working. E.g. currently it's just passing $id of the obj to next getItems read.
Please advise, otherwise will fork.
Thanks, nice work!

Hi, I am having the same issue.
I replace $id with $result['path'] to fix the issue.
Thank you.

@wiljonlim Yes! You are right. so I'll fix it. Thanks!

@tjhorlacher , @wiljonlim Thanks! 👍


[AP-007] don't render columns without a display order

Summary:
We don't want to render all fields automatically -- only those that have
order defined in the struct tags.

Current dependencies on/for this PR:

main

PR #46   👈

This comment was auto-generated by Graphite.


Allow --format to use different delim in table format

- What I did
This fix is an attempt to address https://github.com/docker/docker/pull/28213#issuecomment-273840405
Currently when specify table format with table --format "table {{.ID}}...", the delimiter in the header section of the table is always "\t". That is actually different from the content of the table as the delimiter could be anything (or even contatenated with ., for example):
$ docker service ps web --format 'table {{.Name}}.{{.ID}}' --no-trunc

NAME                ID
web.1.inyhxhvjcijl0hdbu8lgrwwh7
 \_ web.1.p9m4kx2srjqmfms4igam0uqlb

- How I did it
This fix is an attampt to address the skewness of the table when delimiter is not "\t".
The basic idea is that, when header consists of table key, the header section will be redendered the same way as content section. A map mapping each placeholder name to the HEADER entry name is used for the context of the header.
- How to verify it
Unit tests have been updated and added to cover the changes.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)
This fix is related to #28313.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

/cc @thaJeztah

I think this is causing tests to fail on master:
20:54:19 cli/command/formatter/task.go:71: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:79: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:84: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:101: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:106: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:111: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:119: c.AddHeader undefined (type *taskContext has no field or method AddHeader)
20:54:19 cli/command/formatter/task.go:132: c.AddHeader undefined (type *taskContext has no field or method AddHeader)

The error is likely caused by overlapping merge from different PRs. Let me take a look and fix it.

sounds right. I have most of the fix, but I'm seeing the header as <no value>

@yongtang https://github.com/docker/docker/pull/28213 did indeed

https://gist.github.com/dnephin/4f35a12f131f70f1d5bf04b1bd8b6cb8 seems to be most of it, but one test is failing because of whitespace. I'm not sure if the test needs to be fixed, or the code.

Created #31533 for the fix.

@dnephin Ah yes the test needs to be changed. The original test uses delim " " (whitespace). Under the old way of header process, that will caused the content to be rendered with delim " " (whitespace) and the header to be rendered with \t (tab).
This PR (#30733) actually fixes the issue so the header/content delim should be consistent now.

Thanks @dnephin @mlaventure and sorry for the inconvenience caused.


Fix docs version, reformat and add explanations

Description
fixes #1365 by fetching tags prior to pip install . when building docs (as @jni suspected).
Adds the "explanations" folder to the sphinx docs, and moves the "developers" and "explanations" folders into docs/source with the rest of the docs.
I had to reformat number of things in the markdown, so a couple reminders when writing markdown that we want to use in docs:

There should only be a single top level heading (# Title) in a document
Don't use **bold** text as a heading, use #, ##, ### instead

Also, it's worth pointing out that the linking strategy used in our roadmap (e.g. [mission](MISSION_AND_VALUES.md#our-mission)) works fine when viewing in github... but most of the links are broken in the sphinx docs.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

Docs render great for me - see https://napari.org/docs/explanations/rendering.html


Add a hint feature so that the user won't get frustrated when stuck.

A solver library (without source code yet) can be made available. Would it be worthwhile to integrate it?

You mean using a library that's only available as a binary blob? I don't think all people would trust that...
In general, having a hint would indeed be appreciated.


Why doesn't this also have a way to GENERATE rss?

Thanks for the feedback. FeedReader mainly parses documents. Generating feeds is a different story and would require some significant effort to add it to the current solution.


TypeScript 5.x isn’t picking up types correctly

Bug
Getting the following error with<EMAIL_ADDRESS>and TypeScript 5:

Could not find a declaration file for module 'vitest-fetch-mock'. '/Users/drew/Sites/drwpow/openapi-fetch/node_modules/.pnpm/vitest-fetch-mock<EMAIL_ADDRESS>implicitly has an 'any' type.
  Try `npm i --save-dev @types/vitest-fetch-mock` if it exists or add a new declaration (.d.ts) file containing `declare module 'vitest-fetch-mock';`

Solution
I’ve opened #11 that should resolve the issue. But see the question there on what types/test.ts is doing. I’m not sure how that’s affected.

I guess I don't quite understand why types no longer works for you.  I don't see any mention in the breaking change release notes that seems related.
Maybe you could try running typescript with the --traceResolution flag to help find out what's happening?

I noticed this package is still on TypeScript 4.6. 4.7 introduced a breaking change in how "types" works so that it must appear in the package imports (docs). Or, because there are other implications of providing imports (such as all non-specified files are no longer importable), the simpler solution is to merely omit types altogether and rely on the shadowing behavior of .d.ts as my PR did. That will be less disruptive to users than providing imports.

What are your tsconfig.json settings @drwpow? Is your moduleResolution set to Node16 or NodeNext, and does your package.json have "type": "module"?
I'm guessing this has to do with the moduleResolution property in tsconfig.json and not TypeScript 5.0. If you have "moduleResolution": "Node16" in tsconfig.json and "type": "module" in package.json, TypeScript will use ESM resolution rules for import paths. One of those rules is that under ESM, you cannot do directory imports, which might be why "types": "types" doesn't work, but "types": "types/index.d.ts" would.
If moduleResolution is set to Node, you wouldn't see this error because TypeScript will treat all imports as CommonJS, which allows for directory imports.
Docs on this: https://www.typescriptlang.org/docs/handbook/esm-node.html

What are your tsconfig.json settings @drwpow? Is your moduleResolution set to Node16 or NodeNext, and does your package.json have "type": "module"?

"moduleResolution": "NodeNext" (and "module": "NodeNext"), and yes package.json has "type": "module"—I’m using ESM

I'm guessing this has to do with the moduleResolution property in tsconfig.json and not TypeScript 5.0 … But I think this should simply be resolved in vitest-fetch-mock's package.json by simply setting "types": "types/index.d.ts".

It’s actually 4.7 that made the breaking change. And they discourage setting types in the top-level now in favor of exports, which this package doesn’t use. And I’d be wary of setting that because when adding exports then all files within the package can’t be imported unless explicitly declared in that object (or if a ".*": "./*" glob key is added).
…or as an alternative, you can just remove the "types" key and TypeScript will just associate every .js file with the similarly-named .d.ts file as it’s done for a long time, and the 4.6 vs 4.7 breaking change behavior is sidestepped altogether. Which my PR did 🙂
Either approach will work, but the behavior is very easy to recreate—just update TypeScript in this project >= 4.7, and consume in a project >= 4.7. Perhaps ESM is also the triggering factor for that behavior, ESM as the official module format of JavaScript is now table-stakes for packages to support.


Retiarii Experiment running on remote training service

I am trying to configure a Retiarri experiment to run remotely. However, I am having a hard time understanding how to configure the experiment. I was wondering if there are any examples of how to specify the host, ssh key path, etc.? Thanks

It's same as HPO experiment. You should probably figure out how to configure an HPO experiment with Experiment API, and use that experience for remote.
We will note that there's a request for remote example in Retiarii.

I also found this example from the documentation
experiment = Experiment('remote')
experiment.config.trial_command = 'python3 trial.py'
experiment.config.machines.append(RemoteMachineConfig(ip=..., user_name=...))
...
experiment.run(8080)
however, when I try to use this example I get the error - Traceback (most recent call last):
File "D:\machine_learning\LinuxRemote_Windows\NAS_V2.py", line 335, in
experiment.config.machineList.append(RemoteMachineConfig(ip='<IP_ADDRESS>', user_name='lofgren', ssh_key_file='D:\machine_learning.ssh\id_rsa'))
AttributeError: 'NoneType' object has no attribute 'machineList'
Thanks.

Here is what I have so far... but I cannot figure out how/where to specify the machine list
if name == 'main':
base_model = Net()
search_strategy = strategy.GridSearch()
model_evaluator = FunctionalEvaluator(evaluate_model)

exp = RetiariiExperiment(base_model, model_evaluator, [], search_strategy)
exp_config = RetiariiExeConfig()
exp_config.experiment_name = 'Remote - Trial 1'
exp_config.trial_concurrency = 1
exp_config.nni_manager_ip = '<IP_ADDRESS>'
exp_config.trial_gpu_number = 1
exp.run(exp_config, 8082)

any help would be appreciated.

Have tried several different things... but it appears I am stuck. Any help or guidance would be appreciated. thanks,

I apologize, I must have modified my code before adding it to the comment. I have done what you have included and still does not fix the problem.

I found that there is, actually, an example here:
https://github.com/microsoft/nni/blob/master/test/retiarii_test/cgo_mnasnet/test.py

Thank you.. this was very helpful.
When I run the experiment I am now getting a 500 server error (shown below) and I am trying to figure out where I am going wrong. Any help would be appreciated.
[2022-05-10 10:27:20] INFO (numexpr.utils/MainThread) NumExpr defaulting to 8 threads.
[2022-05-10 10:27:26] INFO (nni.experiment/MainThread) Creating experiment, Experiment ID: t2xhugv8
[2022-05-10 10:27:26] INFO (nni.experiment/MainThread) Connecting IPC pipe...
[2022-05-10 10:27:27] INFO (nni.experiment/MainThread) Starting web server...
[2022-05-10 10:27:29] INFO (nni.experiment/MainThread) Setting up...
[2022-05-10 10:27:33] ERROR (nni.experiment.rest/MainThread) rest request POST http://localhost:8081/api/v1/nni/experiment failed: 500 {"error":"Too many files and directories (2001 already scanned) in D:\, please check if i[2022-05-10 10:27:33] ERROR (nni.experiment/MainThread) Create experiment failed
Traceback (most recent call last):
File "d:\machine_learning\LinuxRemote_Windows\NAS_Remote.py", line 358, in
exp.run(exp_config, 8081)
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\retiarii\experiment\pytorch.py", line 309, in run
self.start(port, debug)
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\retiarii\experiment\pytorch.py", line 258, in start
self._proc, self._pipe = launcher.start_experiment_retiarii(self.id, self.config, port, debug)
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\experiment\launcher.py", line 184, in start_experiment_retiarii
raise e
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\experiment\launcher.py", line 173, in start_experiment_retiarii
rest.post(port, '/experiment', config.json())
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\experiment\rest.py", line 38, in post
return request('post', port, api, data, prefix=prefix)
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\experiment\rest.py", line 29, in request
resp.raise_for_status()
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\requests\models.py", line 960, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http://localhost:8081/api/v1/nni/experiment

Please be aware that there is an ignore file you can set to avoid too many files being uploaded.
Doc: https://nni.readthedocs.io/en/stable/experiment/training_service/remote.html

Got it. that fixed that issue... now I am getting another error that I can't seem to understand.
Traceback (most recent call last):
File "/usr/local/stow/python/amd64_linux26/python-3.9/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/local/stow/python/amd64_linux26/python-3.9/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/retiarii/trial_entry.py", line 28, in
engine.trial_execute_graph()
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/retiarii/execution/python.py", line 57, in trial_execute_graph
graph_data = PythonGraphData.load(receive_trial_parameters())
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/retiarii/integration_api.py", line 39, in receive_trial_parameters
params = nni.get_next_parameter()
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/trial.py", line 40, in get_next_parameter
_params = platform.get_next_parameter()
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/runtime/platform/local.py", line 56, in get_next_parameter
params = load(fp=params_file)
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/common/serializer.py", line 352, in load
return json_tricks.load(fp, obj_pairs_hooks=hooks, **json_tricks_kwargs)
File "/home/lofgren/.local/lib/python3.9/site-packages/json_tricks/nonp.py", line 266, in load
return loads(string, preserve_order=preserve_order, ignore_comments=ignore_comments, decompression=decompression,
File "/home/lofgren/.local/lib/python3.9/site-packages/json_tricks/nonp.py", line 236, in loads
return json_loads(string, object_pairs_hook=hook, **jsonkwargs)
File "/usr/local/stow/python/amd64_linux26/python-3.9/lib/python3.9/json/init.py", line 359, in loads
return cls(**kw).decode(s)
File "/usr/local/stow/python/amd64_linux26/python-3.9/lib/python3.9/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/local/stow/python/amd64_linux26/python-3.9/lib/python3.9/json/decoder.py", line 353, in raw_decode
obj, end = self.scan_once(s, idx)
File "/home/lofgren/.local/lib/python3.9/site-packages/json_tricks/decoders.py", line 44, in call
map = hook(map, properties=self.properties)
File "/home/lofgren/.local/lib/python3.9/site-packages/json_tricks/utils.py", line 66, in wrapper
return encoder(*args, **{k: v for k, v in kwargs.items() if k in names})
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/common/serializer.py", line 550, in _json_tricks_func_or_cls_decode
return import_cls_or_func_from_hybrid_name(s)
File "/home/lofgren/.local/lib/python3.9/site-packages/nni/common/serializer.py", line 531, in import_cls_or_func_from_hybrid_name
return cloudpickle.loads(b)
File "/home/lofgren/.local/lib/python3.9/site-packages/cloudpickle/cloudpickle.py", line 679, in subimport
import(name)
ModuleNotFoundError: No module named 'torch.cpu'
I am able to run other NNI experiments on this machine so I know that I have pytorch installed...

Well, I succesfully fixed the previous problem by updating pytorch.
However,  now when I start an experiment it will run the first trial succesfully but I get an error after the first trial is complete:
[2022-05-10 16:24:48] INFO (root/MainThread) Successfully update searchSpace.
Exception in thread Thread-3:
Traceback (most recent call last):
File "C:\Users\u0736288.conda\envs\spyder\lib\threading.py", line 973, in _bootstrap_inner
self.run()
File "C:\Users\u0736288.conda\envs\spyder\lib\threading.py", line 910, in run
self._target(*self._args, **self._kwargs)
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\runtime\msg_dispatcher_base.py", line 51, in run
command, data = receive()
File "C:\Users\u0736288.conda\envs\spyder\lib\site-packages\nni\runtime\protocol.py", line 61, in receive
header = _in_file.read(16)
OSError: [Errno 22] Invalid argument
any ideas?

We've seen such reports of pipe error on Windows, e.g., #4766. Unfortunately, there has been no clue over what could have happened. The problem can't be reproduced on POSIX platform.

Thanks for the feedback. Switched over to a linux system that seems to correct the problem. But I am now running into this error...
[libprotobuf FATAL google/protobuf/stubs/common.cc87] This program was compiled against version 3.9.2 of the Protocol Buffer runtime library, which is not compatible with the installed version (3.17.3). Contact the program author for an update. If you compiled the program yourself, make sure that your headers are from the same version of Protocol Buffers as your link-time library. (Version verification failed in "bazel-out/k8-opt/bin/tensorflow/core/framework/tensor_shape.pb.cc".)
any ideas on what is causing this??

Maybe uninstall tensorflow could help?

uninstalling tensorflow and changes versions of different packages solved the problem.
thanks.


Set results count as custom dimension 5

This will enable us to see the corresponding result count for a query in Google Analytics
Trello https://trello.com/c/lqm0JpMn

Search page examples to sanity check:

http://finder-frontend-pr-1634.herokuapp.com/search/all
http://finder-frontend-pr-1634.herokuapp.com/search/research-and-statistics
http://finder-frontend-pr-1634.herokuapp.com/search/news-and-communications?parent=%2Feducation&topic=c58fdadd-7743-46d6-9629-90bb3ccc4ef0
http://finder-frontend-pr-1634.herokuapp.com/get-ready-brexit-check/questions
http://finder-frontend-pr-1634.herokuapp.com/drug-device-alerts
http://finder-frontend-pr-1634.herokuapp.com/find-eu-exit-guidance-business
http://finder-frontend-pr-1634.herokuapp.com/find-eu-exit-guidance-business?keywords=eori&order=relevance
http://finder-frontend-pr-1634.herokuapp.com/find-eu-exit-guidance-business?sector_business_area[]=aerospace&order=topic
http://finder-frontend-pr-1634.herokuapp.com/uk-nationals-living-eu
http://finder-frontend-pr-1634.herokuapp.com/prepare-business-uk-leaving-eu

Other finders

is sends the data when search is re-performed on the page, but

it doesn't fire when search is performed elsewhere (homepage, /search) and you land here for results
dimension value is "X result(s)", spoke to the analyst and number is expected


improvement Support to add is_used_in_rewrite property when create materialized view

Proposed changes
Add  is_used_in_rewrite property when create mv. Default true;
If is_used_in_rewrite is false which means the mv would not partion in query rewrite.
Such as mv def is as following:
        CREATE MATERIALIZED VIEW mv1
        BUILD IMMEDIATE REFRESH AUTO ON MANUAL
        DISTRIBUTED BY RANDOM BUCKETS 2
        PROPERTIES (
        'replication_num' = '1',
        'is_used_in_rewrite' = 'false'
        )
        AS
        SELECT k1, k2, count(*) from t1 group by k1, k2;

if we run query as following, this mv would not partition in the query rewrite:
SELECT k2, count(*) from t1 group by k2;

run buildall

run buildall

is_used_in_rewrite -> use_for_rewrite

run buildall

run buildall

run buildall

run buildall

run buildall


ChurnMode refactored

churn: code refactoring

Let's wait with this PR until we decide how Genesis will be integrated.

Hasn't Genesis been integrated yet?

Hasn't Genesis been integrated yet?

On the top level, no - 9.3 became 10.0-pre and there wasn't time to do this. On the other hand, the integration work left to do is quite small once @coot typed-protocols new-api is merged into consensus as the Genesis integration PR is on top of it, and a few other PR's in node/api/cli.


Tint Image

Is there an easy way to set the TintColor of the ImageView showing the Image?

ImageView and tintColor:
theImageView.image = [theImageView.image imageWithRenderingMode:UIImageRenderingModeAlwaysTemplate];
[theImageView setTintColor:[UIColor redColor]];

Sounds fine for me :)

Great :)
But could you please make a new release, as I'm using cocoapds :)


Handling slashes in directory names and file names

I have to handle ZIP files in which the directory names and file names may or may not contain slash characters. I have no access to the source of the ZIP files and can only assume that they were generated from a non-filesystem (maybe a database).
As an (made up) example at the root of the ZIP file a directory movies/books may contain a file The Hobbit/Der Hobbit.txt. The example would be stored in the ZIP file as two entries:

movies/books (directory flag true)
movies/books/The Hobbit/Der Hobbit.txt (directory flag false)

This would normally result in a file Der Hobbit.txt nested 3 levels deep in directories when extracted by zip4j (or any other major ZIP software as far as I know) as the slash is interpreted as directory separator.
I wrote an implementation that uses not only the slashes in the entry names but the directory flag on ZIP entries as well to reconstruct the original directory structure as a tree. I can then use the tree to extract the data into a filesystem with the trouble-slashes encoded or somewhere else where slashes are supported in file/directory names.

Long story short: I uploaded the implementation just in case you deem it relevant for zip4j. I'm not sure myself if it fits into the scope of your library because the use case seems quite special. But if you think it does I would be happy to help adjusting and integrating it into zip4j.

This is an interesting use case. I will analyse your code, and consider the implications of integrating such feature (which arguably is slightly deviating from zip specs) in zip4j and update this thread. I appreciate that you provided a well documented code to look at.

After some analysis into your code and the requirement in general, I feel that this cannot be part of the zip4j functionality, and this will open up some more uncertain behaviour on different os. In general, the creators of the zip file should be aware of the complications of having slashes in file names. This leads to unexpected behaviour on different OS's. I will be closing this issue as I feel that this is a rather unique request and I am afraid that integrating this into zip4j might lead to more problems than solutions.

After encountering some more "specialties" in the ZIP files I have to read, I have to agree that handling such non standard-compliant cases is better kept separate. I updated the repository with a better approach and restructured some things but the general point remains.
I thank you though for taking the time to look into it, it is very much appreciated!


Support Datetime function LOCAL TIME

Motivation

To support Datetime function LOCAL TIME in Kotlin-JDSL.

Modifications

Created JPQL model and serializer for LOCAL TIME.
Created Dsl, Expressions, and Serializer tests for LOCAL TIME.

Commit Convention Rule

Please commit your modification based by conventional commit:

Commit type
Description

feat
New Feature

fix
Fix bug

docs
Documentation only changed

ci
Change CI configuration

refactor
Not a bug fix or add feature, just refactoring code

test
Add Test case or fix wrong test case

style
Only change the code style(ex. white-space, formatting)

chore
It refers to minor tasks such as library version upgrade, typo correction, etc.

If you want to add some more commit type please describe it on the Pull Request

Result

The Datetime function LOCAL TIME is created.

Closes

Issue #603 · Support Datetime function LOCAL TIME

 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.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (755c8d5) 91.97% compared to head (9982500) 91.96%.

Files
Patch %
Lines

...er/jpql/serializer/impl/JpqlLocalTimeSerializer.kt
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #636      +/-   ##
==========================================
- Coverage   91.97%   91.96%   -0.02%
==========================================
  Files         297      299       +2
  Lines        3215     3222       +7
  Branches      195      195
==========================================
+ Hits         2957     2963       +6
- Misses        204      205       +1
  Partials       54       54

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

Please sign our contributor license agreement.

Please change the base branch from main to develop.


Create azure-container-webapp.yml5

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

@Magnus6803 Let's discuss this on Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)


[Bug]: waitForNetworkIdle timeout due to missing response

Minimal, reproducible example
Was not able to reproduce this with minimal example, but hope you can help me anyway.

Error string
Timeout exceeded while waiting for event
Bug behavior

[X] Flaky
[ ] PDF

Background
I have a test that uses the waitForNetworkIdle function, but some specific requests are sometimes treated as still running even though they are already done.
await page.waitForNetworkIdle({idleTime: 100, timeout: 120000});

Here I added some additional code to debug and as you see the requests are already done.
To check which requests are still active I added a try catch to the waitForNetworkIdle function:

And here you can see that the request in numRequestsInProgress function still has no response

The strange is that when listening on requestfinished(which shall also be used by puppeteer) you can see that the request already returned about 2 minutes before the timeout of waitForNetworkIdle function containing a response with ok status.
I'm not sure if this could be the reason but the requests this happens all have status code 204 (No content).
Expectation
If request are done they do not block waitForNetworkIdle function
Reality
Sometimes already done requests still block waitForNetworkIdle function
Puppeteer configuration file (if used)
No response
Puppeteer version
20.3.0
Node version
19.4.0
Package manager
pnpm
Package manager version
9.2.0
Operating system
Windows

Without a reproducible example it's difficult to help. Perhaps there is a browser bug that misses CDP events for certain requests.

I might have a related issue @OrKoN
After upgrading Puppeteer to latest version i have chrome processes hanging in my htop list.
The example is here:
export async function createPDF(
	html: string,
	htmlPath?: string,
	writePath?: PathLike
) {
	const browser = await puppeteer.launch({
		headless: 'new',
	});
	const page = await browser.newPage();

	if (htmlPath) {
		await page.goto(htmlPath, { waitUntil: 'networkidle0' });
	} else {
		await page.setContent(html, { waitUntil: 'networkidle0' });
	}

	const pdf = await page.pdf({
		format: 'a4',
		printBackground: true,
		displayHeaderFooter: true,
		headerTemplate: ' ',
		footerTemplate: `

					/

		`,
		margin: { bottom: '20mm', left: '20mm', right: '20mm', top: 0 },
	});

	await page.close();
	await browser.close();
	if (writePath) {
		writeFileSync(writePath, pdf);
	}
	return pdf;
}

@ me if i need to open a new Bug report

@SK-FComputer please open a new issue and please provide a complete example that can be run independently

@OrKoN at the moment I'm not able to give a local reproducible project but I have a suggestion:
Would it be usefull to have another option for waitForNetworkIdle function which provides an array of URLs which puppeteer will not wait for? (or a domain and don't wait for all requests under this domain)
In our case, the blocking requests have no impact on the test or the end user and only capture metrics.
I know it's more of a workaround, but at the moment I have no other idea what I could do.

@OrKoN does this work together with waitForNetworkIdle function to affect the numRequestsInProgress/inFlightRequestsCount(as it is called in new versions) or is it more of a replacement?


Upscaling image in (1,2) ratio?

Hi, first of all, thank you for the code!
I have to upscale image in (1,2) ratio, height is same and width increases twice. Could anyone please guide on what is the best practice to do it with existing model. As far as I understand, self.pixel_shuffle = nn.PixelShuffle(up_scale) is a proportional layer, receiving only int in an argument.

you could add a down sample layer in height dimension.


Add last_data_time to issue_time metric

Will support M2M's PEARs.

Added comparison of issue time to last data time in https://github.com/ktindiana/sphinxval/commit/d1b797fba969c8e9880e218151ef40ea0aa7a908


[QUESTION] 我使用mips-linux-gnu-gcc编译连接后报错：undefined reference to '__assert_fail'

[ 90%] Building C object samples/common/CMakeFiles/samplescommon.dir/file_input_stream.c.o
[ 90%] Building C object samples/common/CMakeFiles/samplescommon.dir/file_io.c.o
[ 91%] Building C object samples/common/CMakeFiles/samplescommon.dir/g711_file_loader.c.o
[ 92%] Building C object samples/common/CMakeFiles/samplescommon.dir/h264_file_loader.c.o
[ 92%] Linking C static library ../../lib/libsamplescommon.a
[ 92%] Built target samplescommon
[ 93%] Building C object samples/kvsapp/CMakeFiles/kvsappcli.dir/kvsappcli.c.o
[ 93%] Building C object samples/kvsapp/CMakeFiles/kvsappcli.dir/option_configuration.c.o
[ 94%] Linking C executable ../../bin/kvsappcli
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(JPEG_Encoder.c.o): In function ConfigureChannel': JPEG_Encoder.c:(.text+0x68): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(JPEG_Encoder.c.o): In function AddSection.part.0': JPEG_Encoder.c:(.text.unlikely+0x20): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(CoreManager.c.o): In function SetClockCommand': CoreManager.c:(.text+0x2d0): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(CoreManager.c.o): In function IsEnc1AlreadyRunning': CoreManager.c:(.text+0x438): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(CoreManager.c.o): In function IsEnc2AlreadyRunning': CoreManager.c:(.text+0x490): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(CoreManager.c.o):CoreManager.c:(.text+0x9e4): more undefined references to __assert_fail' follow /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(imp_encoder.c.o): In function on_encoder_group_data_update':
imp_encoder.c:(.text+0x10fc): undefined reference to __xstat' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(imp_encoder.c.o): In function channel_encoder_init':
imp_encoder.c:(.text+0x3220): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(imp_encoder.c.o): In function update_one_frmstrm':
imp_encoder.c:(.text+0x3b04): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(framesource_tseries.c.o): In function on_framesource_group_data_update':
framesource_tseries.c:(.text+0x1ad8): undefined reference to __xstat' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(osd.c.o): In function OSD_Draw_Layer_Cover_Pic':
osd.c:(.text+0x13fc): undefined reference to __xstat' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(ivs_base_move.c.o): In function imp_base_move_process':
ivs_base_move.c:(.text+0x894): undefined reference to __xstat' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(MergeBaseMove.c.o): In function MergeBaseMove':
MergeBaseMove.c:(.text+0x212c): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(BufferAPI.c.o): In function AL_Buffer_Destroy':
BufferAPI.c:(.text+0x384): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(BufferAPI.c.o): In function AL_Buffer_Unref':
BufferAPI.c:(.text+0x6fc): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(lib_rtos.c.o): In function Rtos_CreateMutex':
lib_rtos.c:(.text+0x1a4): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(lib_rtos.c.o): In function Rtos_GetMutex':
lib_rtos.c:(.text+0x224): undefined reference to __assert_fail' /home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(lib_rtos.c.o):lib_rtos.c:(.text+0x26c): more undefined references to __assert_fail' follow
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(pmem_method.c.o): In function get_pmem_size': pmem_method.c:(.text+0x98): undefined reference to __isoc99_sscanf'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(DevicePool.c.o): In function AL_DevicePool_Open': DevicePool.c:(.text+0xe4): undefined reference to __strdup'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(DevicePool.c.o): In function AL_DevicePool_Close': DevicePool.c:(.text+0x25c): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(BoardMips.c.o): In function AL_sDmaAllocator_SetExtraMemory': BoardMips.c:(.text+0x100): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(AllocatorDefault.c.o): In function AL_sWrapperAllocator_GetPhysicalAddr': AllocatorDefault.c:(.text+0xd0): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(BufferStreamMeta.c.o): In function AL_StreamMetaData_ChangeSection': BufferStreamMeta.c:(.text+0x258): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(BufferStreamMeta.c.o): In function AL_StreamMetaData_SetSectionFlags': BufferStreamMeta.c:(.text+0x2b8): undefined reference to __assert_fail'
/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/libraries/amazon/amazon-kinesis-video-streams-media-interface/3rdparty/T31/lib/glibc/libimp.a(BufferStreamMeta.c.o):BufferStreamMeta.c:(.text+0x2ec): more undefined references to `__assert_fail' follow
collect2: error: ld returned 1 exit status
gmake[2]: *** [samples/kvsapp/CMakeFiles/kvsappcli.dir/build.make:124: bin/kvsappcli] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:575: samples/kvsapp/CMakeFiles/kvsappcli.dir/all] Error 2
gmake: *** [Makefile:136: all] Error 2
root@parallels-Parallels-Virtual-Platform:/home/parallels/Desktop/tmp_transfer/amazon-kinesis-video-streams-producer-embedded-c/build#

使用 mips-linux-uclibc-gnu-gcc 编译没有报错，但是在build/bin 里面只生成了 kvsappcli 可执行文件而没有生成文档里描述的  “kvsappcli-ingenic-t31” 和 “kvsappcli-ingenic-t31-static”。这是为什么？

这个问题已经解决了


Gradle plugin should allow declaring a minimum runtime target

Expected Behavior

The expected behavior is twofold:

Plugin loading considers the minimum runtime target on older Gradle version and provides a meaningful error message.
The resolution of a plugin from the repository should make his best effort to resolve a plugin version that is compatible with the plugin's minimum runtime target.

Current Behavior

The latest version of a plugin will always be chosen which may cause cryptic failures when the plugin uses newer APIs.
Context

This is a common feedback from plugin authors.
Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

There's a bunch of different things we might want to do with the declaration:

Gradle plugin plugin provides the Gradle API for the target version at compile time.
Gradle plugin plugin by convention runs functional tests against target version and current version (for example)
Show this information in the plugin portal to humans.
Suppress deprecation warnings for deprecations that have been added since that target version.
...

The point is that there are really a bunch of different pieces here:

Add some way to declare the target version
Use this information in a bunch of different, and independent, places to do something useful.

I don't think we need to implement both items in the description in order to add value, it would bes ok to implement just one of them. For example, it would be ok as a first step to have only new Gradle versions honour the metadata, and older Gradle versions can continue to attempt to load the plugin and fail in weird ways.


Automatically create a queue/topic when publishing or subscribing to a queue/topic

Tried with MB
When we publish or subscribe to a queue/topic using a client like jmeter, the queue/topic will be automatically created in the MB.
But with ballerina, the queue/topic should be already created manually prior to publishing or subscribing to the queue/topic.
Expected
Create a queue/topic automatically when publishing or subscribing to a queue/topic or come up with a native method to create a queue/topic.

@dilinisg - Is this similar to the https://github.com/ballerinalang/ballerina/issues/2044 ? In that case, when subscribing to queue/topic, it automatically creates the queue/topic, but fails when publishing. Can you confirm that behaviour?

Yes it is same. My bad. It works for subscribing to topics and queues. Hence, closing the issue


Graph view elements unreadable in dark theme

Storage Explorer Version: 1.2.0
Platform/OS Version: Windows 10 x64 (1803)
Architecture: ia32
Steps to Reproduce:

Choose dark theme
Run a query against a Cosmos DB graph database

Expected Experience:

Vertex labels should show in a light font color so they can be read against the black background
Edit pencil icons on Properties & Sources should show up in a light color
Progress indicator circles should be light so they can be seen

Actual Experience:

Vertex labels are shown in a dark font color and cannot be read against the black background
Edit pencil icons can't be seen
Progress indicators are black so only the gray edges of the circles are visible

Thanks for the feedback @LarsKemmann. We will fix this.

Azure Storage Explorer support for Azure Cosmos DB has been discontinued as of v1.24.0. We recommend users to use Azure Cosmos DB’s Data Explorer, which can be found in Azure Portal or as a stand-alone version in http://cosmos.azure.com/ for the best Azure Cosmos DB experience. You can also find additional Azure Cosmos DB tooling here: Azure Cosmos DB Tools


Show currently offline monitors at the top of the status page in 'offline' group

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
Currently, the status page isn't glanceable for offline monitors. I have a lot of monitors and it requires me to scroll through the list to see which monitors are offline.
✔ Solution
I would find it useful if offline monitors could be shown at the top of the status page in an 'Offline' group.
This also allows displaying the status page on a TV for keeping track of offline monitors.
Optionally, make this behaviour configurable.
❓ Alternatives
No response
📝 Additional Context
Slightly similar as https://github.com/louislam/uptime-kuma/issues/1165, https://github.com/louislam/uptime-kuma/issues/1921 and #1585

Realized in #1585

Realized in #1585

Not 100% what this issue describes (but is related) since that issue is for the dashboard. I would like an overview of offline monitors on the statuspage.

Any news on this subject? It would be of great help to many.

@Zandor300
v1.23.0 will include #3312
=> I think this issue is resolved and can be closed

@Zandor300 v1.23.0 will include #3312 => I think this issue is resolved and can be closed

@CommanderStorm
PR https://github.com/louislam/uptime-kuma/pull/3312 does not implement this feature request. The PR implements filtering on the dashboard but my feature request is for the statuspage.

Fair enough, sorry

Is there any update on this?

No PR has been raised to add sorting abilities to the status page.
Our contribution guide can be found here: https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md
From a design decision, storing such state in the url such as the one currently is being attempted in https://github.com/louislam/uptime-kuma/pull/4496


Support for --delete-offsets for consumer group topic

Problem Description
Explaining the problem with an example:
ConsumerGroupA has topicA, and after few days we move the topicA to ConsumerGroupB due to some use case. Since the retention is infinite for this use cases the ConsumerGroupA for topicA stays active. The lag dashboards keep showing the lag for this dead topic group.
Since other topics are still using ConsumerGroupA we cannot delete the ConsumerGroupA.
We just want to delete the topicA from ConsumerGroupA.
Problems:

The alerts set on the lags result in false alerts due to this. As the alerts come for the dead topic/consumerGroup.
The lag dashboards becomes noisy due to this.

Based on this stackoverflow answer, Kafka  2.4.0+ has support to delete individual consumer group id offsets from a topic which should solve this problem.
./kafka-consumer-groups.sh \
  --bootstrap-server kafka.{env}.elinvar.tech:9092 \
  --delete-offsets \
  --group {my-group} \
  --topic topicName0

KAFKA KIP for --delete-offset
This has a great detail about this problem.
https://cwiki.apache.org/confluence/display/KAFKA/KIP-496%3A+Administrative+API+to+delete+consumer+offsets
Support request
Is it possible to add support for --delete-offsets in Sarama?

Available in v1.30.0 and newer since https://github.com/IBM/sarama/pull/2006 merged


[integrations] Opt out of sending validation messages and add necessary fields

Previously we had omitted the synced and last_updated fields, but these are necessary for SignalForm’s internal state.
This also changes the behavior of SignalForm’s integration creation to skip validation/test messages (which create PagerDuty pages otherwise) per the documentation at https://developers.signalfx.com/v2/reference#integration-1.
r? @sjung-stripe

lgtm


@event expression is not handled correctly

Vue.js version
1.0.21
Reproduction Link
http://codepen.io/pkawiak/pen/grezPW?editors=1111
Steps to reproduce
Click model.log and model.log() buttons and observe console. Replace model by clicking Replace Model button and repeat clicking model.log and model.log() buttons.
What is Expected?

Click model.log(): Model 1 message appears in console.
Click model.log: Model 1 message appears in console.
Click Replace Model: model is replaced.
Click model.log(): Model 2 message appears in the console.
Click model.log: Model 2 message appears in the console.

What is actually happening?

Click model.log(): Model 1 message appears in console.
Click model.log: Model 1 message appears in console.
Click Replace Model: model is replaced.
Click model.log(): Model 2 message appears in the console.
Click model.log: Model 1 message appears in the console.

hmm strange, even
this.$set("model.log",m2.log)

doesn't change the v-on:press='model.log' binding to the m2.log


Add Bobby's autonomous modes with a few edits

We need to test this code with the rangeFilter.reset() removed and the old AutomaticShootCommand class

Will be tested tomorrow after this is merged.


Send DidBecomeActiveNotification when OOM enabled

:scroll: Description
The OOM code of the Cocoa SDK also relies on the
SentryHybridSdkDidBecomeActive to work properly.
If a user has enableAutoSessionTracking disabled wrong
OOM events would be created.
:bulb: Motivation and Context
Came up when investigating https://github.com/getsentry/sentry-cocoa/issues/1852.
:green_heart: How did you test it?
Hard to test as there are no unit tests.
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[x] I updated the docs if needed
[ ] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #905 (dadf253) into main (7cf74d7) will increase coverage by 5.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #905      +/-   ##
==========================================
+ Coverage   89.77%   95.55%   +5.78%
==========================================
  Files         104        2     -102
  Lines        3207       45    -3162
==========================================
- Hits         2879       43    -2836
+ Misses        328        2     -326

Impacted Files
Coverage Δ

dio/lib/src/sentry_transformer.dart

.../lib/src/http_client/sentry_http_client_error.dart

dart/lib/src/sentry_tracer_finish_status.dart

dart/lib/src/client_reports/discard_reason.dart

dart/lib/src/protocol/sentry_exception.dart

dart/lib/src/sentry_exception_factory.dart

dart/lib/src/transport/rate_limit_parser.dart

dart/lib/src/protocol/sentry_event.dart

dart/lib/src/platform/platform.dart

...art/lib/src/enricher/enricher_event_processor.dart

... and 92 more

Continue to review full report at Codecov.

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

Is this related to the huge amount of OOMs on iOS?

Nvm, you even linked it :) Sorry

@kuhnroyal, only if you have enableAutoSessionTracking disabled.

I checked but this is enabled by default and I have not changed it.


exporter is not exporting metrics...

Hi there
I'm facing an issue with the exporter, but I'm not really sure whether it is an exporter issue... or actually an issue with either OTC or my permissions. I hope the explanation below is clear enough:

I managed to get the exporter up and running in an openshift environment
From this openshift environment I can access multiple tenants on OTC
Whilst I can get the exporter to expose metrics on tenant A, I can not get it to export metrics for tenant B
for tenant B, when I look at the logs of the exporter, it is full of lines containing '...(get_ecs_mapping): Created name entry for ECS...' so ECS mapping works.
But there are no metrics being logged which explains why they are not exposed to prometheus

Any ideas what could be the cause? The users I use to access Tenant A and B have the same rights in OTC. Both are members of a dedicated 'monitoring' user group that has the following policy for the projects in question: "Guest, CES Administrator".
Thanks :-)

Hi,
I doubt that the problem is related to your IAM permissions, it should work just with BASE->Tenant Guest permission, I don't even have here the CES Administrator role added to the user group. And as you already stated, on the other tenant with the same permissions it works.
Unfortunately, there is an OTC CES API limitation that allows only 140 requests per minute and the API does not support bulk requests (the exporter has to do one request against the API per metric that it tries to get). This is already addressed with the OTC team and I hope that they release soon such a feature to make bulk requests against the CES API. Taking this into consideration if you are trying to get more the 140 metrics (for example, 10 metrics per ECS multiplied by 14 ECS would already reach the maximum allowed by OTC) you will probably face some issues (like some metrics being exposed by the exporter and some missing)
For further analysis please do the following test:
Run a curl against the exporter to see if it is really not exposing any metric:
curl http://localhost:8000
If you are using the container provided you can easily install curl with:
apk add curl
Please also inform around how many resources you are monitoring on tenant A and tenant B. This will help me to simulate your problem.

Hi Tiago
Sorry for the late response!
As I'm using your exporter in an Openshift Cluster it is a bit more tricky, as in that specific setup I can not simply install curl on the pod. But i could build a new docker image with curl installed (if necessary).
Regarding the amount of resources in tenant A and B and main or sub-projects:

in tenant A (working) all instances are created in the main project and there are a total of 22 ECS instances with ~50 disks attached to them
in tenant B (not working) all instances we're interested in are in a sub-project (PROJECT_ID variable is set to the sub-project id) and there are a total of 38 ECS instances (but with ~270 disks attached to the ecs instances)

It seems you reached an API requests limit from OTC. I already tried to contact the support team asking to implement some kind of bulk request. However, without any positive response.
If you are able to contact them too, try to ask them too. Maybe with more people requesting for it, they might implement such functionality.
About the limitation:  https://github.com/tiagoReichert/otc-cloudeye-prometheus-exporter#known-limitations

Hey,
sorry to revive this now, but I'm running into the exact same issue.
When Using the Main Project, the ECS metrics get exported correctly, but in the Sub-Project, I get no metrics at all.
I understand the API limit, but it should still get metrics up until it reaches that limit.
From logs it seems like the "get_metric_value" function is not triggered at all, when using a sub project.
All I get is:
(get_ecs_mapping) Created name entry for ECS xxx successfully (get_ecs_mapping) Created name entry for ECS xxx successfully
(get_ecs_mapping) Created name entry for ECS xxx successfully (get_ecs_mapping) Created name entry for ECS xxx successfully
(save_config_file): Configuration file successfully saved`
So there 0 metrics exported, even though all Instances are recognized correctly.
When using the main project ID, the get_metric_value function is triggered.
Did anyone figure out what causes this issue?


Update PR template with more specific wording

This PR just adds some more explicit wording for new module contributions into the PR template. As a newcomer, I found the current wording at first a bit confusing, it was clear once I understood which files were meant.
This PR should help other newcomers to not run into any confusion and be more productive.

Thanks for your feedback and contribution, @gvenzl !

You are very welcome!


Converting synchronized block to RW lock.

@sumwale @kneeraj @suranjan please review the following snippet specifically. We can discuss about the race condition getting introduced with this change. IMO, this is negligible.
  protected void processMessage(DistributionMessage msg) {
    boolean isNew = false;
    InternalDistributedMember m = msg.getSender();
    boolean shunned = false;

    // First grab the lock: check the sender against our stabilized view.
    latestViewLock.readLock().lock();
    try {
      if (isShunnedNoSync(m)) {
        if (msg instanceof StartupMessage) {
          endShun(m);
        }
        else {
          // fix for bug 41538 - sick alert listener causes deadlock
          // due to view lock being held during messaging
          shunned = true;
        }
      } // isShunned

      if (!shunned) {
        isNew = !latestView.contains(m) && !surpriseMembers.containsKey(m);

        // If it's a new sender, wait our turn, generate the event
        if (isNew) {
>>>>>>>>> unlocking the readLock so that addSurpriseMember can acquire writeLock >>>>>>>>>>>>>
          latestViewLock.readLock().unlock(); // synchronized
          shunned = !addSurpriseMember(m, getStubForMember(m));
>>>>>>>>>>>>>>>>>>>>>>
        } // isNew
      }

      // Latch the view before we unlock
    } finally {
>>>>>>>>>> avoiding extra readLock.unlock if (isNew == true) >>>>>>>>>>>>
      if (!isNew) { // note: if(isNew) then we release read and acquire writeLock inside addSurpriseMember.
        latestViewLock.readLock().unlock(); // synchronized
      }
>>>>>>>>>>>>>>>>>>>>>>
    }

    if (shunned) { // bug #41538 - shun notification must be outside synchronization to avoid hanging
      warnShun(m);
      if (VERBOSE_VIEWS || logger.fineEnabled()) {
        logger.info(LocalizedStrings.DEBUG, "Membership: Ignoring message from shunned member <"
            + m +">:" + msg);
      }
      throw new MemberShunnedException(getStubForMember(m));
    }

We should also have Bruce look at this. Please send him an email with link to this PR.

Soubhik, what happened to this and what was the response of Bruce? This is fine by me. Lets get this in if possible.

@soubhik-c Any update on this one from Bruce? See comments and lets close out these long pending ones for 0.71

Comment from Bruce:

This pull request doesn't match any source base I'm familiar with. GFE 8.2 holds a writeLock while dispatching the message so there is no contention with addSurpriseMember.  Geode does the same.
I think it is a worthy effort to rework the 8.2/Geode locking in the MembershipManager to increase concurrency.  Maybe you want to look into contributing to Geode?

@suranjan Can you look through the code of GFE 8.2 to see how it is handled? The main problem area is the synchronized() in processMessage which the Geode code has removed (and retained only for shunned members).


Retry the fallback mix format if closest returned format is bad:

fixes #284

thanks for this, looks good to merge, but I want to do a bit of manual testing first


Release/3.1.1

This is a patch release that fixes a bug in session management. It can crash the app at startup due to an incorrect migration of session data from the older versions of the tracker.
CHANGELOG
Bug fixes:

Fix object cannot be nil (key: userId) when initialising Snowplow tracker (#675)

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Add Endpoint for Capital Gains Yield

Are you adding a new feature?

[x] YES
[ ] NO

Are you enhancing the old feature?

[ ] YES
[x] NO

Is your feature request related to a problem?

[ ] YES
[x] NO

Describe your feature/enhancement.

I would like to add a feature that would calculate the Capital Gains Yield on a stock.
The capital gains yield will equal a company's total stock return if a company does not pay dividends.
The formula for the capital gains yield is used to calculate the return on a stock based solely on the appreciation of the stock.
It can be calculated by using the following formula: (P1 - P0) / P0.
P0 : Initial stock price where as P1: Stock price after first period.
Can you add the feature by yourself?

[x] YES
[ ] NO

Additional Information

I would like to work on this issue if its seen as a good one under GSSOC'23.

Hello @ighoshsubho could you add a level to this issue. I am asking this to gain points in the leader board of GSSOC'23 once I make the PR. Thanks.

Closing as done!


Headings have faux 'Francois One' bold

Take a look at the site on Mac Firefox - the headings are smeared almost into illegibility. It's bad in other browsers, too - the vertical strokes are much wider than the horizontal ones.
Since Francois One does not have a bold face, and you want bold for the other fonts in the stack, you could add a @font-face rule just for Francois One that uses the real font for bold styling instead of a fake bold:
@font-face {
font-family: 'Francois One';
font-style: normal;
font-weight: bold;
src: local('Francois One Regular'), local('FrancoisOne-Regular'), url('http://themes.googleusercontent.com/static/fonts/francoisone/v6/bYbkq2nU2TSx4SwFbz5sCKRDOzjiPcYnFooOUGCOsRk.woff') format('woff');
}

Haven't been able to reproduce. Could you post a screenshot?

Screenshot here for anyone wishing to confirm: http://imgur.com/S35fv (thanks @astearns!)


Update virtual-scroll docs to use let syntax

Change #item to let item syntax for the VirtualScroll component. This changed in beta7 due to the angular2 update to RC1. See https://github.com/driftyco/ionic/blob/2.0/CHANGELOG.md#angular-update-to-200-rc1
Ionic Version: 2.0.0-beta.8

Thank you!


Unable to create new album

Whenever I try to create a new album it throws an error.

Hi, apologize for the delay in response.
To help you trouble shoot, if you are using Chrome, can you open developer tools and let me know what error message you see in the console tab? https://developers.google.com/web/tools/chrome-devtools/

Given long timeframe since the original post, i'll close this issue for now. feel free to reopen if you are still experiencing the error


CAMEL-14514: Improve performance of Camel Google Pub/Sub

Use google-cloud-pubsub library
Asynchronous pull
Cache producers
https://issues.apache.org/jira/browse/CAMEL-14514

@oscerd Thanks for your input. I'll make the requested changes tonight.

Can you please rebase?

I also noticed issues with the example project for pub/sub and a dependency conflict on the bom. Already working on fixing these. When those are done I'll rebase.

@alvinkwekel let us know when you think this PR is good for review and potential merge.

I think it's ready for a further review.
The new Google library and setup of the component improves the performance drastically.
But due to the different library some functions of the Camel component like explicitly configuring the connection factory is no longer there. Also the CamelGooglePubsub.AckDeadline and CamelGooglePubsub.MsgAckId headers are no longer available. I'd like to hear what your thoughts are on that.

Can you please run the component build with sourcecheck profile enabled and fix the reported Warnings? Thanks.

So without the connection factory how do you specify which cloud to use? Or well there is only one the big google cloud ?

I think the endpoint can be configured for use with the local running Pub/Sub Emulator. I'll investigate this further.

So without the connection factory how do you specify which cloud to use? Or well there is only one the big google cloud ?

I now have a draft implementation for setting a custom endpoint when using a local Pub/Sub emulator. Needs a bit of extra work...

Thanks @alvinkwekel for keeping us up to date on your progress. Looking forward to have all work done so we can get this to run faster, thanks for the hard work so far.

My open points are done now. Looking forward to your review.

Implemented a synchronous pull in addition to the async pull
Integration test with testcontainers instead of process-exec-maven-plugin
Various improvements

Very good the guava change in parent/pom requires a change otherwise the other comments are minior/cosmetic

I'll merge this today or tomorrow at most.

merging this before I will port over my other work