How to load history.yml into highcharts graph?

Hi,
I have a horizontal highcharts bar chart:
horizonbarstack.coffee:

class Dashing.Horizonbarstack extends Dashing.Widget

  ready: ->
    Highcharts.getOptions().colors = Highcharts.map(Highcharts.getOptions().colors, (color) ->
      Highcharts.Color(color).setOpacity(0.82).get "rgba"
    )

  createChart: (series, categories, color) ->
    container = $(@node).find('.horizon-container')
    if $(container)[0]
      @chart = new Highcharts.Chart(

        chart:
          renderTo: $(container)[0]
          type: "bar"
          backgroundColor: null
          marginTop: 0
          marginRight: 15

        title:
          text: ''

        legend:
          enabled: false
          reversed: true
          borderColor: 'white'
          borderWidth: 2

        xAxis:
          categories: categories
          labels:
            enabled: false
          minorGridLineWidth: 0
          minorTickLength: 0
          tickLength: 0

        yAxis:
          gridLineColor: 'gray'
          stackLabels:
            style:
              color: 'white'
            enabled: false
            verticalAlign: 'middle'
            align: 'left'
            #formatter: ->
            #  categories[@x]

          title:
            text: "Total Number of Transactions"

        tooltip:
          formatter: ->
             "Transactions:  " + @y

        plotOptions:
          series:
            stacking: 'normal'
            borderWidth: 0
            pointPadding: 0
            groupPadding: 0
          bar:
            colorByPoint: true
            cursor: "pointer"
            animation:
              duration: 2000
              easing: 'easeOutBounce'
            showInLegend: false
            dataLabels:
              enabled: true
              format: '{x}' + ': {y}'
              color: "white"
              align: 'left'
              verticalAlign: 'middle'
              style:
                fontSize: '17 px'
              connectorColor: "#000000"

        navigation:
          buttonOptions:
            verticalAlign: 'top'
            y: -7
            x: -9
            theme:
              'stroke-width': 1
              stroke: color
              r: 0
              fill: color
              states:
                hover:
                  fill: color
                select:
                  fill: color

        series: series
      )

  onData: (data) ->
    @createChart(data.series, data.categories, data.color)

I'm a bit new to coffeescript/js in general so I apologize if the code doesn't look very nice. :kissing_smiling_eyes: But seeing as the chart only gets drawn when data is sent, I was wondering how to draw the chart upon load if data exists in the history.yml file?
Since the chart doesn't get drawn until it receives data, if the user refreshes the page there's nothing there, so I'd like to be able to load the chart with history.yml until data is received.
history.yml:

instruments: |+
  data: {"series":[{"name":"Instruments","data":[292223,289424,216255,210580,184266,183860,180352,165023,148477,140018]}],"categories":["EEM - 2013-12-21 - 28.0 - PUT","EEM - 2013-12-21 - 35.5 - PUT","SPY - 2013-08-17 - 166.0 - PUT","SPY - 2013-08-17 - 167.0 - PUT","SPY - 2013-08-17 - 165.0 - PUT","JNJ - 2015-01-17 - 55.0 - CALL","SPX - 2013-09-21 - 1300.0 - PUT","SPY - 2013-08-17 - 166.0 - PUT","SPY - 2013-08-17 - 167.0 - PUT","SPY - 2013-08-17 - 166.0 - CALL"],"color":"#2c3e50","id":"instruments","updatedAt":1377504330}

Thanks!

That issue seems old enough to be closed!


Is AudioKit 4.2.3 compatible with Swift 4.0 and Xcode 9.2?

Hi there!
The latest version of Xcode 9.3 introduced some strange compilation errors (not related to AudioKit) in the app I'm working on, which forced us to roll back to Xcode 9.2 and Swift 4.0.
Updating AudioKit 4.1 to 4.2.3 generates the following error for AudioKit:

Module compiled with Swift 4.1 cannot be imported in Swift 4.0.3

I'm wondering if it is possible to use the latest version of AudioKit with Xcode 9.2?
As a side note, using AudioKit 4.1 works well.
Thanks a lot for your kind help!

We only provide compiled binaries built with the current non-beta version of Xcode. As such, 4.2.3 was built with Xcode 9.3 and requires Swift 4.1.
If AK 4.1 works for your needs, then you should probably stick with that as long as it works in Xcode 9.2. Alternatively, you could try to build the framework from source using our provided script.

Thanks! I eventually managed to fix my issue and I'm back to Xcode 9.3 and AudioKit 4.2.3!

@ladislas Curious, how did you get version 4.2.3 working with Xcode 9.3?

@ladislas Oh whoops, for some reason I read your comment as "4.0.3", and was wondering how you got it to build. Never mind!


Connect MongoDB datasource issue on FastReport designer community edition

We are encounter error on FastReport.Community.2021.3.0.zip, could not load file or assembly mongodb.driver.core during select MongoDB data source.
Any one also encounter the same?

This issue should be fixed in https://github.com/FastReports/FastReport/releases/download/v2021.3.21/FastReport.Community.2021.3.21.zip Thanks @KirillKornienko


Refactor the chainntfs package to accommodate future ChainNotifier implementations

This PR refactors the chainntfs package, along with all code within the daemon that interacts with a ChainNotifier. The goal of this PR is to pave the way for future ChainNotifier implementations by setting up a factory abstraction, as well as unifying the current set of tests to exercise all "registered" ChainNotifier interfaces rather than just the BtcdNotifier implementation.
The following is a breakdown of the major changes within this PR:

The existing set of tests has been modified to test against all ChainNotifier implementations the tests are aware of.
In order to add such functionality, the concept of a NotifierDriver has been introduced. A NotifierDriver is a wrapper around a factory function which is capable of create a particular concrete ChainNotifier implementation.
It is now the duty of each sub-package which implements a new ChainNotifier to properly register a corresponding NotifierDriver within a init() function somewhere within the package.
All code which previously directly created a BtcdNotifier now instead use the provided driver API to create a ChainNotifier
Finally, a small bug has been fixed within the htlcSwitch. The struct's quit channel was now properly initialized within its constructor, causing a panic if the Stop() method was called.

LGTM!

I still need to investigate the timeout in the failing test on travis. All the tests pass for me locally with the exception of one which is fixed in #34. The version of rpctest in upstream in the upstream btcd master branch is much more robust than the one present in my fork. I'll get that branch caught up after this and the wallet refactors are in.


Serde Header

This PR introduces SerdeHeader and defines ConfluentHeader as the default implementation. The behavior of Serde is the same, ~one small but important difference is a fixed bug in the encoding of the message index (see the second part of the PR description)~. The changes are mostly just a refactoring and extraction of the header encoding/decoding logic. It makes Serde a bit more flexible and future-proof, since the header format can be switched to a different implementation.
The motivation behind this change is to expose the logic for encoding/decoding the Confluent wire format header. Using this change the caller can now decode the ID manually if needed and fetch the schema from the schema registry.
val, err := serde.DecodeNew(b)
if errors.Is(err, sr.ErrNotRegistered) {
	id, _ := serde.Header().DecodeID(b)

	// fetch schema from schema registry using the ID...

	serde.Register(id, myVal, sr.DecodeFn(func(b []byte, v any) error {
		// decode using the schema ...
	})

	// retry decoding
	val, err = serde.DecodeNew(b)
}

The code above is just an illustration, I'm aware that the code would need to take care of request collapsing.
Related to the discussion in https://github.com/twmb/franz-go/issues/453.

The documentation is worded confusingly and I got tripped up by that the first time I read it. The "reading the array backwards" describes the following sentence itself, not the wire encoding. The following sentence first describes the inner struct -- the 0th index of the inner struct -- and then describes the outer struct -- the 1st index of the outer struct. The paragraph beneath this describes the actual encoding of the array, " So the above array [1, 0] is encoded as the variable length ints 2,1,0 where the first 2 is the length"

Oh you're right, I read it again carefully. I double checked what they do in confluent-kafka-go and seems like the order was correct as it was.
I'll revert this part 👍

Sorry for the delay, finally did a deep review. I don't think I have any comments besides what I've posted now.

Thanks for the thorough review! 👍 I tried to address all comments in one way or another.

This PR LGTM, I say squash and merge it.
This is now missing the ability to set a header but I think per this comment chain: https://github.com/twmb/franz-go/pull/467#discussion_r1231620122 that will be in a follow up PR.

Sounds good to me! I'll take care of the follow-up in a second. I already have a separate branch with the changes, I just had something urgent come up so I didn't get to open the PR, sorry about that.


pkgConf ... does not exist

$ cabal build afterjznqu-mkszchedmv-cemvtpwxvm/afterjznqu-mkszchedmv-cemvtpwxvm.cabal --disable-tests --verbose
...
cabal: Failed to build afterjznqu-mkszchedmv-cemvtpwxvm-<IP_ADDRESS>-inplace. The
failure occurred during the final install step. The exception was:
...dist-newstyle/tmp/package-registration--251564/pkgConf:
openBinaryFile: does not exist (No such file or directory)

This happens for at least a couple packages.

The Setup.hs files were not correct; not needed, fixed: https://github.com/peterbecich/cabal-resolver-issue/commit/5cc82ca635012174c55085864f559206b6ba546a


add pub/sub test for each node to confirm reference system meets the requirements

check to make sure nodes have the right number of pubs / subs so that they conform to the requirements definition.
might also incorporate in this test a frequency check for each topic to make sure they are publishing at the correct rate.

partially implemented in #21

closed with fixes in #30


Analysis: React Accessibility Rules

Microsoft is in the process of creating some React based accessibility tslint rules. We had a small internal brainstorm session. @ipip2005 mentioned in a related issue that he/her was interested as well.
Our current plans are:
Images must have alt-text

reference - https://www.w3.org/TR/WCAG10/wai-pageauth.html#tech-text-equivalent
including image buttons - http://webaim.org/techniques/forms/controls#image_button
including images used as submit buttons -

Labels must be explicitly associated with their controls

reference - https://www.w3.org/TR/WAI-WEBCONTENT-TECHS/#tech-associate-labels
more acceptable examples - http://www.clarissapeterson.com/2012/11/html5-accessibility/
Use title when label not available: https://www.w3.org/TR/2015/NOTE-WCAG20-TECHS-20150226/H65

Describe the purpose of frames

reference: https://www.w3.org/TR/WCAG10/wai-pageauth.html#tech-frame-longdesc

Form buttons must have a descriptive value

reference - http://webaim.org/standards/wcag/checklist
for examples see http://webaim.org/techniques/forms/controls#button

Form inputs must have associated text labels

reference - http://webaim.org/standards/wcag/checklist

Text inputs and TextArea inputs must be labeled

reference - http://webaim.org/techniques/forms/controls#input

Elements with onClick listeners must define role="button"

reference - https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_button_role

Required input elements must define aria-required="true"

reference - http://www.clarissapeterson.com/2012/11/html5-accessibility/

Edit boxes and text areas must include default, place-holding characters

https://www.w3.org/TR/WCAG10/wai-pageauth.html#tech-place-holders
https://www.w3.org/TR/WCAG10-HTML-TECHS/#forms-specific

Images used as "submit" buttons must have text equivalent

https://www.w3.org/TR/WCAG10-HTML-TECHS/#forms-specific

Tables must have summaries

tables should have either a caption or a title: https://www.w3.org/TR/WCAG10-HTML-TECHS/#table-summary-info
tables should have a summary attribute: https://www.w3.org/TR/WCAG10-HTML-TECHS/#table-summary-info
Provide terse substitutes for header labels with the "abbr" attribute on TH

Do not use obsolete html elements/attributes

https://www.w3.org/TR/html5/obsolete.html

Is there anything else you'd like to see?
Any comments?
Thanks in advance!

THere is an es6 related repo here: https://github.com/evcohen/eslint-plugin-jsx-a11y

I'm also writing rules based on this repo. Personally I would prefer to see these rules at first
aria-role, aria-props, aria-proptypes, tabindex-no-positive, role-has-required-aria-props, role-supports-aria-props, img-has-alt
These rules checks a11y fundamentals.
And for me, some rules from https://github.com/evcohen/eslint-plugin-jsx-a11y are not so easy to be used by projects because they will easily break their projects and people might not very likely to fix them. For example
Required input elements must define aria-required="true"
Elements with onClick listeners must define role="button"

I sent an email. It would be great to have these rules in the tslint-microsoft-contrib repo.

Closing this issue because many child issues have been created


Is it a typo?

https://github.com/prdn/pigato/blob/master/lib%2FBroker.js#L81
_.each(self.services, function(service, srvName) {
  if (!service.workers.length) {
    supd = true;
    delete self.services;
  }
});

should delete self.services to be delete self.services[srvName] ?

You're right.
Fixed the issue.
Thank you very much


Unable to locate executable file: 'bash'.

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Command line
Environment

Server - Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
https://dev.azure.com/andreibarbu0946/UiPathGoodPractices
pipeline AndreiBarbuOz.uipath-doc-manipulation
buildId=475

Agent - hosted
Azure Pipelines 10, Agent version: 2.165.2

Issue Description
2020-04-06T14:41:31.4556994Z ##[section]Starting: Build artifact
2020-04-06T14:41:31.4698083Z ==============================================================================
2020-04-06T14:41:31.4698855Z Task         : Command line
2020-04-06T14:41:31.4699464Z Description  : Run a command line script using Bash on Linux and macOS and cmd.exe on Windows
2020-04-06T14:41:31.4700029Z Version      : 2.164.0
2020-04-06T14:41:31.4700473Z Author       : Microsoft Corporation
2020-04-06T14:41:31.4701058Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/command-line
2020-04-06T14:41:31.4701717Z ==============================================================================
2020-04-06T14:41:32.3211213Z Generating script.
2020-04-06T14:41:32.3239876Z Script contents:
2020-04-06T14:41:32.3242191Z c:\my_prog.exe -pack C:__w\1\s\project.json -o C:__w\1\a\ -v 1.0.475
2020-04-06T14:41:32.3251114Z ========================== Starting Command Output ===========================
2020-04-06T14:41:32.3397439Z ##[error]Unable to locate executable file: 'bash'. Please verify either the file path exists or the file can be found within a directory specified by the PATH environment variable. Also verify the file has a valid extension for an executable file.
2020-04-06T14:41:32.3639089Z ##[section]Finishing: Build artifact
[Include task name(s), screenshots and any other relevant details]
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

This is a dupe of https://github.com/microsoft/azure-pipelines-tasks/issues/12537 - lets follow up there as needed


[SPARK-45106][SQL] PercentileCont should check user supplied input

What changes were proposed in this pull request?
Change PercentileCont to explicitly check user-supplied input by calling checkInputDataTypes on the replacement.
Why are the changes needed?
PercentileCont does not currently check the user's input. If the runtime replacement (an instance of Percentile) rejects the user's input, the runtime replacement ends up unresolved.
For example, this query throws an internal error rather than producing a useful error message:
select percentile_cont(b) WITHIN GROUP (ORDER BY a DESC) as x
from (values (12, 0.25), (13, 0.25), (22, 0.25)) as (a, b);

[INTERNAL_ERROR] Cannot resolve the runtime replaceable expression "percentile_cont(a, b)". The replacement is unresolved: "percentile(a, b, 1)".
org.apache.spark.SparkException: [INTERNAL_ERROR] Cannot resolve the runtime replaceable expression "percentile_cont(a, b)". The replacement is unresolved: "percentile(a, b, 1)".
	at org.apache.spark.SparkException$.internalError(SparkException.scala:92)
	at org.apache.spark.SparkException$.internalError(SparkException.scala:96)
	at org.apache.spark.sql.catalyst.analysis.CheckAnalysis.$anonfun$checkAnalysis0$6(CheckAnalysis.scala:313)
	at org.apache.spark.sql.catalyst.analysis.CheckAnalysis.$anonfun$checkAnalysis0$6$adapted(CheckAnalysis.scala:277)
...

With this PR, the above query will produce the following error message:
[DATATYPE_MISMATCH.NON_FOLDABLE_INPUT] Cannot resolve "percentile_cont(a, b)" due to data type mismatch: the input percentage should be a foldable "DOUBLE" expression; however, got "b".; line 1 pos 7;

Does this PR introduce any user-facing change?
No.
How was this patch tested?
New tests.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @MaxGekk

Merged to master/3.5.
There is a conflict from branch-3.4. Could you make a PR for branch-3.4 and 3.3, please, @bersprockets ?


cloudwatch error in Lambda decoder.

With the simple example, I get the following cloudwatch error:
Expecting value: line 1 column 1 (char 0): JSONDecodeError
Traceback (most recent call last):
File "/var/task/notify_slack.py", line 76, in lambda_handler
message = json.loads(event['Records'][0]['Sns']['Message'])
File "/var/lang/lib/python3.6/json/__init__.py", line 354, in loads
return _default_decoder.decode(s)
File "/var/lang/lib/python3.6/json/decoder.py", line 339, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/var/lang/lib/python3.6/json/decoder.py", line 357, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I am trying to send an SNS message via CLI:
aws sns publish --topic-arn --topic-arn <my_sns_topic> --message " This is a test"
upon which I receive a MessageId.
What am I doing wrong here?

This is caused by the SNS event, which is a dictionary, being loaded as json on line 76.
I changed it to
message = event['Records'][0]['Sns']['Message']

This issue has been just fixed in v1.11.0.
Kudos to @ryron01 !


Added incomplete transaction tracing

Closes #1134

This has been incorporated into https://github.com/nanocurrency/raiblocks/pull/1126 with the following changes/notes:

transaction id is initialized with current timestamp to avoid id clashes between runs
rebased on top of the db abstraction merge
the recorder currently exposes a singleton, so it can be accessed from the transaction - this should be refactored later so we can use the recorder in multi-node tests
the tx tracing changes are (partially) documented in https://github.com/cryptocode/raiblocks/wiki/Event-Recorder

tx tracing and backtrace logging are both config options, default off
backtrace logging adds measurable time overhead, only turn on when backtraces are needed (at least until observers are fixed, as they produce large amounts of unnecessary transactions)

@cryptocode Thanks !


Tweaks

And markup changes that were required to implement
those styling tweaks. This commit includes:

update to the subscription form styling
added separator above the subscription form section
update to the tag cloud styling (all tags look similar now)
some other tiny tweaks
TOCBOT now looks like shown here: https://youtu.be/yqtMg7ITJJ8
add tag cloud to archive page template (fixes #13 )

Merged manually, thanks!


Update extraction.md  and Update multimodal.md

grammar correction: "names, dates, address" to "names, dates and addresses".
grammar and tense correction: "retrieving from a text corpus" to "retrieving it from a text corpus", "Check out out Chroma" to "Check out Chroma", "supports single vector store" to "supports a single vector store", "can generate structured output" to "can generate a strucutred ouput", "structure of output" to "structure of the output".

@AryamanJaiswalShorthillsAI , Please fix the conflicts.

@ShorthillsAI  resolved


Changed client.rs example to use client certs and am getting error.

I'm trying to get mTLS client authentication to work.  I changed client.rs in a couple ways.  First, I stole the load_certs and load_private_key functions from examples in the base rustls project.  Then, I changed client.rs from:
    let config = rustls::ClientConfig::builder()
        .with_root_certificates(root_cert_store)
        .with_no_client_auth(); // i guess this was previously the default?

to:
    let config = rustls::ClientConfig::builder()
        .with_root_certificates(root_cert_store)
    .with_client_auth_cert(certs, key);

And initialized certs and keys thus:
    let certs = load_certs("rcfd2_o1.crt");
    let key = load_private_key("rcfd_o1.key");

And with those two changes, I get this error:
➜  tokio-rustls git:(main) ✗ cargo run --example client localhost -p 8770 -c ~/AET/src/aet_demo/ca_certs.pem
   Compiling tokio-rustls v0.26.0 (/Users/eswenson/AET/src/rust/tokio-rustls)
error[E0277]: the trait bound `TlsConnector: From<Arc<Result<ClientConfig, rustls::Error>>>` is not satisfied
  --> examples/client.rs:94:21
   |
94 |     let connector = TlsConnector::from(Arc::new(config));
   |                     ^^^^^^^^^^^^ the trait `From<Arc<Result<ClientConfig, rustls::Error>>>` is not implemented for `TlsConnector`
   |
   = help: the trait `From<Arc<ClientConfig>>` is implemented for `TlsConnector`
   = help: for that trait implementation, expected `ClientConfig`, found `Result<ClientConfig, rustls::Error>`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `tokio-rustls` (example "client") due to 1 previous error
➜  tokio-rustls git:(main) ✗

What is causing this?  I've made no other changes.  Does .with_client_auth_cert(...) work?

Hi @eswenson1,
The issue here is that with_no_client_auth() returns a ClientConfig directly - it has no possibility of error. The function you changed it to, with_client_auth_cert(), returns a Result<ClientConfig, Error>.
You need to decide how to handle the error case, to pull out the ClientConfig that the code that follows expects. If this was a simple example where you don't care about errors, you could use with_client_auth_cert(...).unwrap(), but for a production use-case you'll probably want to handle the error.
Hope that helps

So now the issue is that I'm getting this error:
Custom { kind: InvalidData, error: InvalidCertificate(UnknownIssuer) }

I'm using my own PKI hierarchy, where the rootCA is a self-signed cert.  There is an intermediate (signing) CA, and then the leaf cert.
Should this work?
I read somewhere that there is a bug that if the signing algorithm isn't supported, the UnknownIssuer error is erroneously returned. I'm using the SHA256 hash algorithm in my PKI CA with RSA keys.  Is this supported by rustls (tokyo-rustls)?
Note: openssl s_client works fine against my server and I'm specifying the same trusted CA certs in my RootCertStore as my server is using (and which I use with openssl s_client.

Should this work?

Yes, but you'll need to share more detail to help diagnose the issue. Can you post a complete reproduction?


fix(image): support upload jog images

Currently, answer don't support upload JPG image, it will be blocked at decode stage

before change the IsSupportedImageFile function, it will fail

after changing, it successfully uploaded


default show closed actions list if all actions was closed

as title, that's same with issues and pull list

if (ctx.FormString("state") == "open" && numOpenRuns != 0) || (numClosedRuns == 0  && ctx.FormString("state") != "cloesd") {
    // show open runs
} else {
    // show closed runs
}

Maybe one if is enough?

if (ctx.FormString("state") == "open" && numOpenRuns != 0) || (numClosedRuns == 0  && ctx.FormString("state") != "cloesd") {
    // show open runs
} else {
    // show closed runs
}

Maybe one if is enough?

If ctx.FormString("state") == "open" we should show the open runs, even if there are none

Why not follow the logic of the issue page?
https://github.com/go-gitea/gitea/blob/5e389228f61844c0d59d5bc5974483dae0aa1b8b/routers/web/repo/issue.go#L224-L228

Why not follow the logic of the issue page?
https://github.com/go-gitea/gitea/blob/5e389228f61844c0d59d5bc5974483dae0aa1b8b/routers/web/repo/issue.go#L224-L228

Hasn't read them before, but I think the result is same, isn't it?

Hasn't read them before, but I think the result is same, isn't it?

I'm not sure.

What if state == closed and numOpenRuns == numClosedRuns == 0?

@wolfogre looking good?


The diff coverage is n/a.

:exclamation: Current head a86d58c differs from pull request most recent head 93e405b. Consider uploading reports for the commit 93e405b to get more accurate results

@@            Coverage Diff             @@
##           master    #1245      +/-   ##
==========================================
- Coverage   83.15%   82.90%   -0.25%
==========================================
  Files         300      248      -52
  Lines       13699    11984    -1715
  Branches      660      660
==========================================
- Hits        11391     9935    -1456
+ Misses       2308     2049     -259

Impacted Files
Coverage Δ

...la/org/apache/spark/ml/param/TypedArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/EstimatorParam.scala
12.50% <0.00%> (-75.00%)
:arrow_down:

...org/apache/spark/ml/param/JsonEncodableParam.scala
53.33% <0.00%> (-33.34%)
:arrow_down:

...ain/scala/org/apache/spark/ml/param/MapParam.scala
75.00% <0.00%> (-25.00%)
:arrow_down:

...g/apache/spark/ml/param/PythonWrappableParam.scala
50.00% <0.00%> (-25.00%)
:arrow_down:

...oft/azure/synapse/ml/codegen/GenerationUtils.scala
75.00% <0.00%> (-25.00%)
:arrow_down:

...soft/azure/synapse/ml/cntk/CNTKFunctionParam.scala
11.11% <0.00%> (-22.23%)
:arrow_down:

...crosoft/azure/synapse/ml/io/http/HTTPClients.scala
56.66% <0.00%> (-20.00%)
:arrow_down:

...osoft/azure/synapse/ml/codegen/CodegenConfig.scala
66.66% <0.00%> (-14.82%)
:arrow_down:

...ala/org/apache/spark/ml/param/DataFrameParam.scala
58.33% <0.00%> (-12.51%)
:arrow_down:

... and 64 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 5a921cc...93e405b.


Quest patient registration crash caused by CQF library functionality

Describe the bug
The app crashes after patient registration. However, the patient is still recorded with the correct details eg. age converted to dob (not sure what effect the crash has)
To Reproduce
Steps to reproduce the behavior:

Run Quest and login using the demo account
Click on the Add New Client button in the clients page
Fill in the form with all the patient details
Click save
The app crashes

Expected behavior
The page should close after saving and return to the clients page. The new patient should be immediately available through search or on the correct page number based on alphabetical order
Screenshots
N/A
Smartphone (please complete the following information):

Device: Emulator Google Pixel 2
Android version: Android 11
Build number: [e.g. Settings -> About phone -> Build number]
Version [e.g. 24]

Additional context
Crash logs
2022-01-03 10:27:19.129 184-184/? A/libc: crash_dump helper failed to exec
2022-01-03 10:27:19.862 9590-9590/org.smartregister.fhircore.quest E/AndroidRuntime: FATAL EXCEPTION: main
    Process: org.smartregister.fhircore.quest, PID: 9590
    java.util.NoSuchElementException: Collection contains no element matching the predicate.
        at org.smartregister.fhircore.engine.util.extension.QuestionnaireExtensionKt.cqfLibraryId(QuestionnaireExtension.kt:95)
        at org.smartregister.fhircore.engine.ui.questionnaire.QuestionnaireViewModel$extractAndSaveResources$1.invokeSuspend(QuestionnaireViewModel.kt:192)
        at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
        at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
        at android.os.Handler.handleCallback(Handler.java:938)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:223)
        at android.app.ActivityThread.main(ActivityThread.java:7656)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)
2022-01-03 10:27:22.317 509-9929/? E/ActivityManager: ANR in org.smartregister.fhircore.quest (org.smartregister.fhircore.quest/org.smartregister.fhircore.engine.ui.questionnaire.QuestionnaireActivity)
    PID: 9590
    Reason: Input dispatching timed out (98187b2 org.smartregister.fhircore.quest/org.smartregister.fhircore.engine.ui.questionnaire.QuestionnaireActivity (server) is not responding. Waited 5003ms for FocusEvent(hasFocus=false))
    Parent: org.smartregister.fhircore.quest/org.smartregister.fhircore.engine.ui.questionnaire.QuestionnaireActivity
    Load: 1.44 / 1.41 / 1.76
    ----- Output from /proc/pressure/memory -----
    some avg10=0.27 avg60=0.18 avg300=0.29 total=14311055
    full avg10=0.01 avg60=0.04 avg300=0.05 total=3901393
    ----- End output from /proc/pressure/memory -----

Crash started happening after merging this commit https://github.com/opensrp/fhircore/commit/a7e511bd4cf2f234dbe9847f4df22981341f0aab on an umerged PR.

This was reported in this PR https://github.com/opensrp/fhircore/pull/909#issuecomment-1003915228

This issue seems to be fixed from another PR as we can no longer reproduce it. We can create a new ticket if it re-occurs


[e2e]: container workflow_dispatch main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11063823635
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Fri Sep 27 03:09:48 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11080579305
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Sat Sep 28 03:12:03 UTC 2024
Tests are passing now. Closing this issue.


Add Update logic for labels and centroids and feed target label values in KMeans fit

Adding Logic to update labels and centroid based on the output of the KMeans Cluster split and also extracting appropriate values from the dataset based on target label to input in the KMeans fit method.
Author: Vishwa Patel

Reference Issues/PRs

What does this implement/fix? Explain your changes.
Any other comments?


ZXGenericMultipleBarcodeReader

This pull request fixes ZXGenericMultipleBarcodeReader which, in the way is implemented on master, always returns nil trying to decode multiple barcodes.
I've used the following code and image to test it:
UIImage *image = [UIImage imageNamed:@"some_barcodes"];

ZXCGImageLuminanceSource *source = [[ZXCGImageLuminanceSource alloc] initWithCGImage:image.CGImage];
ZXBinaryBitmap *bitmap = [ZXBinaryBitmap binaryBitmapWithBinarizer:[ZXHybridBinarizer binarizerWithSource:source]];

ZXDecodeHints *hints = [ZXDecodeHints hints];
hints.tryHarder = YES;

ZXMultiFormatReader *reader = [ZXMultiFormatReader reader];
ZXGenericMultipleBarcodeReader *multiReader = [[ZXGenericMultipleBarcodeReader alloc] initWithDelegate:reader];

NSError *error = nil;
NSArray *results = [multiReader decodeMultiple:bitmap hints:hints error:&error];

for (ZXResult *result in results) {
    NSLog(@"Decoded barcode %@", result.text);
}

The output is as follows:
Decoded barcode http://fr.wikipedia.org/
Decoded barcode<PHONE_NUMBER>
Decoded barcode Hello World!
Decoded barcode 04252614
Decoded barcode<PHONE_NUMBER>843

Works like a charm :+1:
Any plans on getting this merged?

Nice PR, could you maybe write a test for this?

Added unit test @benjohnde

very nice, as I need this for my own project, I hope this is getting merged pretty soon

good job! cheers!

Cool, I've just rebased on top of master.

Looking good so far, I will test it tomorrow. Cheers!

This code seems not working in swift. I am using Zxing through pods and have version 3.0.
Here is my code snippet.
let source: ZXCGImageLuminanceSource = ZXCGImageLuminanceSource(CGImage: image!.CGImage)         let bitmap: ZXBinaryBitmap = ZXBinaryBitmap(binarizer: ZXHybridBinarizer(source:source))         let hints: ZXDecodeHints = ZXDecodeHints.hints() as! ZXDecodeHints         hints.tryHarder = true         let reader: ZXMultiFormatReader = ZXMultiFormatReader.reader() as! ZXMultiFormatReader         let multiReader: ZXGenericMultipleBarcodeReader = ZXGenericMultipleBarcodeReader(delegate: reader)         do{         let results: NSArray = try multiReader.decodeMultiple(bitmap, hints: hints) as NSArray             for result in results {                 NSLog("Decoded barcode %@", result.text!)             }         }catch{             print("ERROR in DECODING")         }

This seems to fail on IOS 9.3 ObjC using the following code, I get the following error message
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'This luminance source does not support cropping.'
*** First throw call stack:
(0x180e6adb0 0x1804cff80 0x1000de6b4 0x1001036e0 0x1000dfd94 0x1000df7fc 0x10002d834 0x1000c0420 0x1005a9a7c 0x1005a9a3c 0x1005af4e4 0x180e20d50 0x180e1ebb8 0x180d48c50 0x182630088 0x186032088 0x1000333a0 0x1808e68b8)
libc++abi.dylib: terminating with uncaught exception of type NSException

This is the code
UIImage *image = [UIImage imageWithCGImage:capture.lastScannedImage];
UIImageWriteToSavedPhotosAlbum(image,nil,nil,nil);

ZXCGImageLuminanceSource *source = [[ZXCGImageLuminanceSource alloc] initWithCGImage:image.CGImage];
ZXBinaryBitmap *bitmap = [ZXBinaryBitmap binaryBitmapWithBinarizer:[ZXHybridBinarizer binarizerWithSource:source]];

ZXDecodeHints *hints = [ZXDecodeHints hints];
hints.tryHarder = YES;

ZXMultiFormatReader *reader = [ZXMultiFormatReader reader];
ZXGenericMultipleBarcodeReader *multiReader = [[ZXGenericMultipleBarcodeReader alloc] initWithDelegate:reader];

NSError *error = nil;
NSArray *results = [multiReader decodeMultiple:bitmap hints:hints error:&error];

for (ZXResult *result2 in results) {
    NSLog(@"Decoded barcode %@", result2.text);
}

It fails in the creation of the NSArray *results, I can see the image saved fine to the photoalbum so I know the image is there so I don't understand why it crashes

Not working in Swift! :(
        let source = ZXCGImageLuminanceSource(CGImage: image.CGImage)
        let binarizer = ZXHybridBinarizer.binarizerWithSource(source!) as! ZXBinarizer
        let bitmap = ZXBinaryBitmap.binaryBitmapWithBinarizer(binarizer) as! ZXBinaryBitmap
        let hints = ZXDecodeHints.hints() as! ZXDecodeHints
        hints.tryHarder = true
        let reader = ZXMultiFormatReader.reader() as! ZXReader
        let multi = ZXGenericMultipleBarcodeReader(delegate: reader)
        do {
            let result = try multi!.decodeMultiple(bitmap, hints: hints)
            print(result)
        } catch _ {
            print("Not found!")
        }

it's work but CPU 100+%,  too high

it's work but too slowly.


Loading File Cabinet Folders Loads Forever

Your environment
OS : e.g. iOS MacOS Big Sur
Node.js version :  v16.13.0
suitecloud-cli version : 1.3.1
Terminal/CMD tool : Terminal
java version : java 11.0.12
Describe the bug
For any commands, if you put -i which is the interactive mode, it will run forever that also taking CPU resources. It displays this message " Loading File Cabinet folders" and it loads forever. I tried vsCode extension as well, it has a similar issue. In the Vscode output tab, it outputs the message below
The remote server returned an error:
Read timed out.
To Reproduce
Steps to reproduce the behavior:
suitecloud file:list -i (it doesn't work)
suitecloud file:list --folder (it works)

@faiwudevse Can you try with Node14 LTS, which is recommended for this CLI.
When you do please re-install the CLI again.

I did install node14 LTS (v 14.7.4), and ran this command npm install -g @oracle/suitecloud-cli  to reinstall cli . I still have the same issue in the interactive option when it comes to "Loading File Cabinet folders... ",  that just loads forever(I let it ran on the terminal like 6 hrs until I killed it). Other parts are working if that is not involving loading file cabinet folder.

@faiwudevse We are working on it. Currently this is in testing. We will have an update for you soon.

This should be fixed in latest.
@faiwudevse can you please confirm?

@alii-gator Thanks the latest update work.


Program terminates abnormally for certain frequency threshold

Hi Keren,
For the mushroom data available on uci machine learning repository, we have obtained the graph representation. Each record is now a graph with vertices and edges.
When I run your gspan code for this input with support=0.3 it runs fine.
But when I run it with lesser support say 0.2 it terminates abnormally.
It gives following error:-
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted (core dumped)
I also tried running with the same input, support=0.2 and single thread but it gives same error.
I am attaching the input file I have used.
Can you please take a look in this issue and let me know accordingly?
mushroom_data_train_data.zip
Appreciate your help in advance.

No problem, I would take a look at your data.

Hi Keren,
Thanks a lot :)
On Feb 13, 2016 17:00, "Keren Zhou"<EMAIL_ADDRESS>wrote:

No problem, I would take a look at your data.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-183647107
.

Well, after running the program, I notice that the memory consumption is abnormally high.
Therefore I suspect that under the frequency of 0.2, a considerable number of frequent graphs are generated. I intend to revise the code and verify the place where the memory is released. In the way the program finally could be terminated in spite of long running time.
By the way, 1 year after I wrote the code, I realize plenty of shortcomings in the original version. Maybe I would refractory the whole project later.

Hi Keren,
Thanks a lot for the excellent analysis.
Yeah, memory is the issue.
Please let me know once you fix the issue.
On Feb 14, 2016 07:21, "Keren Zhou"<EMAIL_ADDRESS>wrote:

Well, after running the program, I notice that the memory consumption is
abnormally high.
Therefore I suspect that under the frequency of 0.2, a considerable number
of frequent graphs are generated. I intend to revise the code and verify
the place where the memory is released. In the way the program finally
could be terminated in spite of long running time.
By the way, 1 year after I wrote the code, I realize plenty of
shortcomings in the original version. Maybe I would refractory the whole
project later.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-183791584
.

Hi @parnika26 , have you tried the original version proposed by Yan?
https://www.cs.ucsb.edu/~xyan/software/gSpan.htm
Setting the minimum support as 0.2, the software(gSpan-64) cannot terminate in several minutes, and the memory consumption is extremely high at 8 threads.
Can you run the program and tell me whether it could terminate on your machine?

Hi Keren,
I tried running the gSpan-64 by Yan for the same input with support=0.2.
But the program got killed by the system in few minutes because of high
memory consumption.
Then I tried running gSpan-32 by Yan.
The program is running since 19 hours and has not been terminated yet.
Already 23 GB of data has been generated by the program till now.
But the program is not killed yet.
Please let me know if you get some clues for this issue.
On Mon, Feb 15, 2016 at 2:29 PM, Keren Zhou<EMAIL_ADDRESS>wrote:

Hi @parnika26 https://github.com/parnika26 , have you tried the
original version proposed by Yan?
https://www.cs.ucsb.edu/~xyan/software/gSpan.htm
Setting the minimum support as 0.2, the software(gSpan-64) cannot
terminate in several minutes, and the memory consumption is extremely high
at 8 threads.
Can you run the program and tell me whether it could terminate on your
machine?
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-184120849
.

--
Regards
Parnika Paranjape

When I was writing the program, I had compared  running results of my program with that of gSpan-32 for different data sets. I still remember there's little difference.
Both gSpans (gSpan-32&gSpan-64) by Yan are writing results consecutively onto the disk, so I could check part of the output files to determine where is the problem.

Hi Keren,
I even tried running your code for another data set 'Compound_422' with
support=0.05 and got the same error
terminate called after throwing an instance of 'std::bad_alloc'
what(): std::bad_alloc
Aborted (core dumped)
But when I ran gSpan-32 by Yan for the same input and support=0.05 the
program did terminate successfully and the execution time was 1475.59 sec.
Looking at above analysis, I think the memory consumption is high and the
memory is not getting released at regular intervals in your code. Hence,
the program gets aborted.
Can you please take a look into this issue and let me know accordingly ?
Appreciate your help in advance.
On Tue, Feb 16, 2016 at 1:20 PM, Keren Zhou<EMAIL_ADDRESS>wrote:

When I was writing the program, I had compared running results of my
program with that of gSpan-32 for different data sets. I still remember
there's little difference.
Both gSpans (gSpan-32&gSpan-64) by Yan are writing results consecutively
onto the disk, so I could check part of the output files to determine where
is the problem.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-184563818
.

--
Regards
Parnika Paranjape

Please try the current refine branch. It has following features:

new thread split strategy (not verified). you can changed it to the original version by configuring the Makefile.
add mushroom data
thread terminate logs
reduce memory consumption.

I have tested the Compound_422 data with support=0.05 under 8 threads, and it terminates in 670 sec.
Actually, I think the initial configuration of thread split is not enough to reach a load balance. Latter on we have to push frequent patterns into a queue so that each thread can fetch their tasks from it.

Hi Keren,
Thanks a lot for the help so far.
I tried running the refined code for Compound_422 data with support=0.05
under 4 threads and it successfully terminates.
But when I ran the refined code for mushroom data with support=0.2 under 4
threads I got the same issue after running for some time.
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted (core dumped)
Can you please try to run the code for mushroom data set with support 0.2
or less and let me know if it terminates on your machine?
My machine specifications are as follows:-
RAM:- 4 GB
OS:- 32 bit ubuntu 12.04 LTS
Processor:- Core 2 Duo 3.16 GHz
On Wed, Feb 17, 2016 at 1:32 PM, Keren Zhou<EMAIL_ADDRESS>wrote:

Please try the current refine branch. It has following features:

new thread split strategy (not verified). you can changed it to the
original version by configuring the Makefile.
add mushroom data
thread terminate logs
reduce memory consumption.

I have tested the Compound_422 data with support=0.05 under 8 threads,
and it terminates in 670 sec.
Actually, I think the initial configuration of thread split is not enough
to reach a load balance. Latter on we have to push frequent patterns into a
queue so that each thread can fetch their tasks from it.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-185086615
.

--
Regards
Parnika Paranjape

No problem, but at first I wonder whether your program with gspan-32 running mushroom data has terminated? I remember you said it has run for days?

Hi Keren,
No. The program gspan-32 by Yan did not terminate on its own. I kept it
running for approximately 2 days but it did not terminate.
It generated almost 81 GB of output and program consumed lot of memory due
to which system got very very slow.
And your newer version of code got terminated due to same memory issue
within few minutes for mushroom data at support=0.2 and 4 threads on my
machine.
So, can you please check the behavior of your modified code on your machine
for mushroom data at 0.2 support and let me know the result?
Also, is there any way to deal with this memory issue for big data like
mushroom and lesser threshold?
Appreciate your help in advance.
On Thu, Feb 18, 2016 at 3:22 PM, Keren Zhou<EMAIL_ADDRESS>wrote:

No problem, but at first I wonder whether your program with gspan-32
running mushroom data has terminated? I remember you said it has run for
days?
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-185632594
.

--
Regards
Parnika Paranjape

Thank you for your reply!
Based on your observation, I think my program cannot terminate no matter what optimization have been done. In contrast to Yan's programs, my gspan program writes out results after the termination of whole mining process. In this way the memory consumption will be high until the results are released.
To solve the problem, maybe I can change the method of writing results. But there's still another deal left, my server's disk doesn't have enough space to store more 81 GB data. So I couldn't test it by myself.
With respect to your last question, the memory issue is easy to handle as I mentioned before. However, since the threshold is very low, it must generate a considerable number of patterns and lead to long mining process. Given your case, the problem is caused by the data, not the programs--gspan-32, gspan-64, and my gspan.
Please feel free to let me know if there should be any question. :)

Suddenly I come up with another solution. Maybe there are more efficient graph mining algorithms? Well, I am not specialized in the research area.

Hi Keren,
There is a survey paper based on different graph mining algorithms. So, as
per that paper gspan seems reasonable to me.
Huge data is one of the issues. But we need to work with such data.
So, can you please give a try by modifying the method to write the results
to disk along with graph mining process like Yan's approach instead of
writing them after termination.
I can then test the new changes for mushroom data on my machine.
Appreciate your help in advance.
On Fri, Feb 19, 2016 at 9:38 PM, Keren Zhou<EMAIL_ADDRESS>wrote:

Suddenly I come up with another solution. Maybe there are more efficient
graph mining algorithms? Well, I am not specialized in the research area.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-186279363
.

--
Regards
Parnika Paranjape

OK, I will update the corresponding feature, but please wait for days. My holiday is over and I have to return to work.

Hi Keren,
Sure, I will wait.
Please let me know once you done with the changes.
Thank you so much for being so much helpful :)
On Mon, Feb 22, 2016 at 7:08 AM, Keren Zhou<EMAIL_ADDRESS>wrote:

OK, I will update the corresponding feature, but please wait for days. My
holiday is over and I have to return to work.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-186965186
.

--
Regards
Parnika Paranjape

Hi Keren,
Just a gentle reminder.
Any update on the corresponding feature that we discussed in previous mails?
Appreciate your help in advance.
On Mon, Feb 22, 2016 at 11:34 AM, parnika paranjape <
<EMAIL_ADDRESS>wrote:

Hi Keren,
Sure, I will wait.
Please let me know once you done with the changes.
Thank you so much for being so much helpful :)
On Mon, Feb 22, 2016 at 7:08 AM, Keren Zhou<EMAIL_ADDRESS>wrote:

OK, I will update the corresponding feature, but please wait for days. My
holiday is over and I have to return to work.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-186965186
.

--
Regards
Parnika Paranjape

--
Regards
Parnika Paranjape

Sorry, I am still busy dealing with assignments offered by my mentor. I must finish them first, or he will not be satisfied.
Therefore, I am so sorry that I can't guarantee when could I update the corresponding feature.

Hi Keren,
Thanks for the update.
Sorry for the trouble but I intend to use your code for graph mining in my
work.
So, if you could give the desired modifications then it will be really
helpful in my work.
Can you please try to modify the corresponding feature as soon as you done
with your assignments?
Appreciate your help in advance.
On Thu, Mar 3, 2016 at 6:35 AM, Keren Zhou<EMAIL_ADDRESS>wrote:

Sorry, I am still busy dealing with assignments offered by my mentor. I
must finish them first, or he will not be satisfied.
Therefore, I am so sorry that I can't guarantee when could I update the
corresponding feature.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-191521995
.

--
Regards
Parnika Paranjape

Thank's OK, I am trying to find a balance between tasks given by the mentor and my own activities.

Hi Keren,
Sure, no problem.
Please let me know when you are done with the corresponding feature.
Thanks a lot again.
On Mar 4, 2016 5:34 AM, "Keren Zhou"<EMAIL_ADDRESS>wrote:

Thank's OK, I am trying to find a balance between tasks given by the
mentor and my own activities.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-192028879
.

Hi Keren,
How are you and your work?
Just a gentle reminder.
Did you get any chance to look into the issue and modify the code?
Please let me know since, my work has got stuck on this issue.
I understand that you have lot of work but can you please try to look into
the issue as soon as possible?
Appreciate your help in advance.
On Fri, Mar 4, 2016 at 7:59 AM, parnika paranjape <
<EMAIL_ADDRESS>wrote:

Hi Keren,
Sure, no problem.
Please let me know when you are done with the corresponding feature.
Thanks a lot again.
On Mar 4, 2016 5:34 AM, "Keren Zhou"<EMAIL_ADDRESS>wrote:

Thank's OK, I am trying to find a balance between tasks given by the
mentor and my own activities.
—
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-192028879
.

--
Regards
Parnika Paranjape

I would try my best, but can you tell me your deadline?

@parnika26 see develop branch. In this version, I set up a number to denote the maximum number of patterns remain in the memory. After the frequent patterns exceeding the number, it holds a global lock, and writes patterns to the disk.
By the way, I am really unsure whether the result is correct or not without any test.

Hi Keren,
Sorry for the late response.
I was not well for a long period so, replying late.
I tested your developed branch code.
As compared to previous version, current version consumes less memory.
But sometimes the code gives split thread error  'split thread error! size
1336, thread_size 4' for certain support values. I got this error when I
ran the code for given data set at support >= 0.2.
When support < 0.2, the code does not give this error.
Do you know the cause for this error and how to fix it?
Please find attached the data set.
On Mon, Mar 21, 2016 at 9:00 AM, Keren Zhou<EMAIL_ADDRESS>wrote:

@parnika26 https://github.com/parnika26 see develop branch. In this
version, I set up a number to denote the maximum number of patterns remain
in the memory. After the frequent patterns exceeding the number, it holds a
global lock, and writes patterns to the disk.
By the way, I am really unsure whether the result is correct or not
without any test.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Jokeren/DataMining-gSpan/issues/13#issuecomment-199097561

--
Regards
Parnika Paranjape

@parnika26 , according to the code, this is because there are less than 4 initial set that has support >= 0.2.
For example, suppose we have 'A', 'B', 'C', 'D' as the initial code, but the number of all codes start with 'D' is less than 0.2 * total. In this way we only start with 'A', 'B' and 'C'.
Hence, if we set support < 0.2, all codes satisfy the condition. Or we could set thread_num less than 4.

Hi Keren,
Thanks a lot for the quick response.
I will then try to tun with less support or less number of threads.
Thanks once again for your support so far :)
On Tue, Apr 19, 2016 at 8:26 PM, Keren Zhou<EMAIL_ADDRESS>wrote:

@parnika26 https://github.com/parnika26 , according to the code, this
is because there are less than 4 initial set that has support >= 0.2.
For example, suppose we have 'A', 'B', 'C', 'D' as the initial code, but
the number of all codes start with 'D' is less than 0.2 * total. In this
way we only start with 'A', 'B' and 'C'.
Hence, if we set support < 0.2, all codes satisfy the condition. Or we
could set thread_num less than 4.
—
You are receiving this because you were mentioned.

I will keep on maintaining this project. :)


Log Display Filter

I'd like to be able to search and filter log output directly in the UI. A simple text match with ?* wildcards would suffice but, ideally, it would support regex.

Thanks, this is a good suggestion that's been requested in chat a few times.
For now, we'd recommend you use the output to standard output command-line or API and then you can either write to a file or pipe into grep as you need to


Feature request: Velocity Support

Is your feature request related to a problem? Please describe.
Velocity support is a growing proxy platform separate from BungeeCord that is gaining popularity and has already seen adoption from some large networks like TheArchon. Some server owners (including myself) have had interest in moving in the past to Velocity but were deterred by a lack of plugin support, creating a rather large barrier to entry. However, I have recently seen more and more plugins adopt support for Velocity and I'm looking forward to having the option to move to Velocity in the (hopefully-near) future. Adding support for Velocity (while it is an albeit small platform compared to BungeeCord) could definitely prove beneficial to many server owners. Assuming Velocity keeps growing at the rate it is, it isn't that unlikely that there will be more server owners interested in moving to Velocity that could be held back by a lack of plugin support.
Describe the solution you'd like
I'd like for velocity support to eventually be added to NetworkInterceptor.
Describe alternatives you've considered
There is always the alternative of staying with BungeeCord or simply running a Velocity network without NetworkInterceptor on the proxy and only on the Spigot backends but these both have drawbacks. Velocity has a number of benefits over BungeeCord that for the sake of brevity I won't list but I'll link to them here which makes sticking with BungeeCord a drawback in and of itself. Running a network where the proxy doesn't have NetworkInterceptor has the obvious drawback of not having NetworkInterceptor, this is a really useful plugin and there aren't any good alternatives that I've been able to find for Velocity.
Additional context
It's definitely worth noting that the numbers Velocity has compared to BungeeCord are not very high and while I do urge you to compare the growth of both proxies as well as their servers-to-players ratios, I understand if this is held at lower priority.
I should also mention that I am interested in (and am actively) learning more about how Velocity differs from BungeeCord and I'd be happy to share what I find to help the process along- I believe this plugin should be available for everyone, regardless of platform.

Velocity is outside the current scope of the project. Given that NetworkInterceptor is open-source, there are several possible paths forward:

Under the Unlicense, anyone is free to fork the project and extend it to include Velocity.
We would be amendable to a well-designed, coded and tested PR submission, with the goal of incorporating the code.
We would consider doing the work, given a substantial contribution to fund the design, development and testing.

We would consider doing the work, given a substantial contribution to fund the design, development and testing.

Of monetary nature, correct?

Please open a SpigotMC PM to discuss. We flooded the Bungee feature request with too much chatter.

Implemented in version 3.1.2


🛑  0.15.2 TypeError: interceptors.createInterceptor is not a function

It looks like a version 0.15.2 of this package was released about 4 hours ago.
After re-installing node_modules where I use msw I noticed this package was updated after my build and all of my unit and integration tests broke with the following error:
 FAIL  src/Routes/Dashboard/Dashboard.test.tsx [ src/Routes/Dashboard/Dashboard.test.tsx ]
TypeError: interceptors.createInterceptor is not a function
 ❯ Proxy.setupServer node_modules/.pnpm/msw@0.40.2_typescript@4.6.4/node_modules/msw/node/lib/index.js:8191:42
    8189|         }
    8190|         let resolvedOptions = {};
    8191|         const interceptor = interceptors.createInterceptor({
       |                                          ^
    8192|             modules: interceptors$1,
    8193|             resolver(request) {
 ❯ src/test/server.ts:5:15
 ❯ async src/test/setup.tsx:26:31

I tried manually installing version 0.15.1 and setting the resolution to 0.15.1 of this package, and my error was resolved.
It seems like there's a breaking change in 0.15.2.
If anyone else stumbles on this, downgrading this package to 0.15.1 fixed it for me.

@kettanaito v0.15.2 did a lot of changes to the codebase and public APIs. Should it be released as v0.16.0 instead?
ref: https://github.com/mswjs/interceptors/compare/v0.15.1...v0.15.2

Thanks for reporting this. Looks like I haven't named the refactor commit so it'd bump minor version. I will address this on Monday. For now, pin the dep to 0.15.1.

Okay, so I've got a chance to get to this sooner.

Re-published the 0.15.1 state as 0.15.3.
Will release the actual next version as 0.16.0.

https://semver.org/ ~ breaking changes should use a major version bump

@philBrown if you read the spec carefully enough, you will notice that breaking changes in pre-1.0 releases should, in fact, bump the minor version. Hence the pre-1.0 with the ability to iterate faster.


Fix issue #114 - Fail to find the image element in a DICOM file with private Sequence tags

The PR fix and prevent the behavior of the parser "trying to detect and parse private sequences as they not necessarily follow the DICOM standard" causing the parser state to be corrupt.

Coverage increased (+0.07%) to 51.991% when pulling efec99bc686fa228b2fab3bdf19b2443e1728fa5 on issue-114 into 374ad3d20a92bd2a7ec56d90dfce2fb2815fee13 on master.

@Zaid-Safadi Do you have any sample DICOM or test cases we can incorporate around this PR?

This PR should not be accepted as it introduces non standards conforming behavior and the design of this library is to be strictly standards compliant out of the box.  As Yagni points out, the standard is very clear about how to handle this situation and that is how it was implemented.  To keep with this design intention and support this use case, it would have to be activated through an opt in flag that is off by default.

Hey @yagni , unfortunately I can't share the sample I have. I can't get the file anonymized because of the private elements that are needed for this.

@Zaid-Safadi Here is my 2cent on the issue. Looking at your patch, here is my understanding:

You want to parse the entire DICOM DataSet (including Private attributes, and in this case a Private sequence)
The cornerstone library properly implement CP-246 for Sequence with VR=UN (and decode nested DataElement using Implicit Little Endian Transfer Syntax)

Now in your case you seem to have found a case where the Private Sequence is declared with VR=UN, but the encoding of the nested Data Element is Explicit Little Endian. If this is the case, this is illegal (per DICOM standard).
To convince yourself (since you cannot share your DICOM DataSet), here are two ways to check:
[Let's consider that the private sequence described above is at tag XXXX,YYYYY]
Solution 1:

Use dcmdump to check it can decode the private sequence,
Using dcmdump you should see the nested attributes underneath tag XXXX,YYYYY
If you cannot see the attributes underneath tag XXXX,YYYYY, this mean you found a private sequence that is missing in your dicom.dic file (on my system: /usr/share/libdcmtk14/dicom.dic). If so update this file so that the tag is properly declared as Sequence.
Start item (1) again, now you should see the nested attributes underneath tag XXXX,YYYYY
Finally. If the above does not work, but dcmdump using --disable-cp246 does work for you, there you know you are dealing with an illegal DICOM encoded DataSet.

Solution 2:

Make sure gdcmdump can dump attributes nested underneath tag XXXX,YYYYY, if not solution #2 wont work.
Use gdcmconv to convert your original DICOM DataSet into another one. Eg:
gdcmconv original.dcm clean.dcm from your UNIX/cmd prompt.
Check the two files are identical. Eg: md5sum original.dcm clean.dcm should be enough.
If they are not identical and suddenly dcmdump can now handle the private sequence, this means that the bug may be related to invalid CP-246 handling on your DataSet.

Pay attention above gdcmdump != dcmdump.
In any case you should be able to reproduce the bug with a public DICOM dataset containing Sequence, there is nothing special about Private Tag in the DICOM standard (at least for the encoding of the Data Element).

@malaterre I’m afraid your understanding of the issue is not accurate.
Simply, the parser is trying to parse a private element in an implicit VR DS because it wrongly assumes it’s a sequence when it may or may not be.
According to the standard the parser should just skip over the element all together which what this fix does.

@Zaid-Safadi! I didn't realize you had merge rights. Thanks for wrapping up this issue :pray:


Config.json sample

Hi
Where can we find a sample of a config.json file to provide ?
Thanks

Thanks for your report. The config.json is created from docker itself, you basically mount the created json of docker -v /home/<user>/.docker/config.json:/config.json. For watchtower it is commonly for storing the credentials of docker login so that watchtower could update private containers.

Ok thank you, didn't get it like that. I thought it was a specific file from watchtower


Implementation of DNCP fragmentation in dncp*.[ch] + unit test

N/A - it was removed from draft -08.


Implement the add() function

Please describe the feature you want this project to offer
Complete the add() function in app.py
Describe the solution you'd like
Take number of values the user wants to enter. Now take these values from the user and return the sum of all these values.
Describe alternatives you've considered (Optional)
A clear and concise description of any alternative solutions or features you've considered.
Additional context (Optional)
Add any other context or screenshots about the feature request here.

PR #6 merged


Validate gas costs for tests that contains SSTORE refunds

See the original issue: https://github.com/gakonst/foundry/issues/812
The story after the REVM port is better, but still somewhat unvalidated:

Currently there is still a small discrepancy of 42 gas that is unaccounted for between the trace of the root call and what REVM reports as the actual used gas after the call. This only happens when the test contains refunds which can only happen with SSTORE.
We need a sure-fire way to validate which number is correct and adjust the other one accordingly. An idea is to implement unit tests for the executor that test against the test cases in EIP-3529 as these cases have clearly defined gas costs. Note that the costs in the table assumes that all slots are hot, but our executor always starts with slots cold.
We have to assume that the number reported by REVM is truth, so the fault most likely lies in the tracer inspector.

Latest numbers dropped and it seems that we are still a little bit off: https://github.com/mds1/convex-shutdown-simulation/pull/16
If we add the instrinsic to Foundry's result it gives us 22,583,004 which is 2,995 gas higher than the 22,580,009 that Tenderly, Ganache and Hardhat agree on

Would be good to re-evaluate this now that we have isolation mode (--isolate)

Tentatively marking as resolved by isolation mode


Make sure forno will transition to Cel2 cleanly

See https://clabsco.slack.com/archives/C0705GMH2A1/p1721935701800609.

Right now, the plan is to just leave all ODIS Alfajores nodes pointing at https://alfajores-forno.celo-testnet.org/ under the assumption that url will cleanly redirect to the L2 network.


UpdateCheckRun failed: Invalid request. Only 65535 characters are allowed

When I try to run this action I get this error:
{
   "severity":"ERROR",
   "time":"2022-06-14T04:04:49.473460382Z",
   "logging.googleapis.com/sourceLocation":{
      "file":"/home/runner/work/reviewdog/reviewdog/doghouse/server/github_checker.go",
      "line":"45",
      "function":"github.com/reviewdog/reviewdog/doghouse/server.(*checkerGitHubClient).UpdateCheckRun"
   },
   "message":"UpdateCheckRun failed: {\"message\":\"Invalid request.\\n\\nOnly 65535 characters are allowed; 95964 were supplied.\",\"documentation_url\":\"https://docs.github.com/rest/reference/checks#update-a-check-run\"}"
}

Perhaps my workflows have too many errors? :) Can the report be limited to 65535 characters?

@haya14busa I don't have access to close issues in this repo. Commenting here since I use this action often. This issue has been resolved by https://github.com/reviewdog/reviewdog/pull/1264 so it can be closed now. Thank you.

Thanks 💯
I also updated permission of this repo so that action-maintainer can maintain this repo.


datasetId field not set in read group set objects returned by searchReadGroupSets

I'm using data from the server unit tests as the server's data.  The files are arranged like so:
dataset1/
    reads/
        1kg-low-coverage/
            ... BAM files ...
        wgBam/
            ... BAM files ...
    variants/

Assumption: each subdirectory in reads corresponds to a read group set.
I create a search request with datasetId = dataset1 and the other fields unset (default values).
Expected:
The call to searchReadGroupSets will return a SearchReadGroupSetsResponse with its readGroupSets field containing two read group sets, each with its datasetId field equal to "dataset1".
Actual:
The call to searchReadGroupSets returns a SearchReadGroupSetsResponse with its readGroupSets field containing two read group sets.  Both have datasetId equal to null.

Verified fixed in #559.


xdg-desktop-portal: 1.14.4 -> 1.14.5

Description of changes
https://github.com/flatpak/xdg-desktop-portal/commit/750b9e5d6d7cdf451bb4f9f4fc97682fafa1a364
Fixes Pipewire RT priority in flatpaks.
The test seems to be flaky on my end?
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

It failed in a different way for me locally. Oh well.


Smacknet label configuration

Related-to: IOTOS-771 IOTOS-807 IOTOS-808.
Provide network configuration to netlabel  for Smack.
Signed-off-by: Alejandro Joya<EMAIL_ADDRESS>

Please add header boilerplate (license terms and copyright statements) to the C file and script.

Would you consider converting smacknet also to a separate stand-alone recipe, like netvalue is done now? The meta-security-framework layer might be the right place for it.

I have built the smacknet service as stand alone and  added the license appropriate license headers

This starts to look good. However, you don't need to patch your own patches, like adding and then removing smacknet from smack recipe. Just create a suitable patch set and force push it to the same branch from which you made your PR.


Sync stable/7.0 with master

Thank you for contributing to openstack/fuel-plugin-plumgrid!
openstack/fuel-plugin-plumgrid uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


[incubator-kie-issues#1367] The context function should throw an error when providing objects with the same keys

Closes https://github.com/apache/incubator-kie-issues/issues/1367
The context function should throw an error when providing objects with the same keys

@baldimir @gitgabrio Ready to be reviewed


option la() not allowed r(198);

Dear Daniel,
I am incurring in the following error message:
betterbarci price mpg, over(foreign) n by(temp) bar format(%9.2f)
option la() not allowed
r(198);
Would you know how to fix this? I use Stata 15
Thanks!
Ernest

Hi Ernest,
I happen to face the same problem. Did you find an effective solution to it? I would love to hear.
Thanks!
Manuel

In case you are still wondering. The problem with the "la()" option happens because of the Stata version you are using. I am currently using Stata 15.
I managed to "solve" the issue by going to the ado file and dropping the `la' local from line 196.
Hope that works for you too.


Allow inlining types in services signature instead of reference to $OpenApiTs

Description
Hi,
There is nothing wrong with using $OpenApiTs per se but it doesn't work well with both vscode/monaco signature hints and doc generator. An example of this is the doc generated by jsr:

I'd much rather have the types inlined and would prefer to have an option to inline the types directly:
$OpenApiTs["/w/{workspace}/flows/create"]["post"]["req"])

vs
{ workspace: string, requestBody: { ... } }

Yep. Duplicate of https://github.com/hey-api/openapi-ts/issues/410


clarify release support

This update is in response to requests for clarification on expectations for versioning of a gNSI server.

@dplore @marcushines
I was actually just about to post a similar item but would like to draw attention to protobuf/API versioning in general as various umbrella projects under OpenConfig are inconsistent and propose a more cohesive approach to this
gNSI

IDL specifies a package name which contains a version (v1 today) - this affects the URI in the HTTP header
IDL leverages gNOI types to encode a semver option (1.2.0 today)
all gNSI services import a common version so any changes need to increment the global version

gNOI

All distinct gNOI services specify their own package name - no versioning in the package
All distinct gNOI services encode a unique semver option from gNOI types
No common versioning

gNMI

No versioning in the package name
Redefinition of the semver option (vs. reuse from gNOI types)

gRIBI

No versioning whatsoever but a directory structure (v1) that started to think about versioning

Now, some request for comments here as there are multiple approaches (some of which are more friendly to transition paths than others)

Does gNSI need to encode the version in the package name?  We now have 2 versions at play here - the semver in version.proto and as soon as it breaches 2.0.0 means that the package really should be bumped to v2
Does gNSI need to be globally versioned?  A change in one API signals a change in all
To the above point, if that is the case then why don't we make all proto IDL part of the same package (gnsi) w/ the service blocks making the differentiation?
If gNSI is globally versioned then what is the justification vs. say gNOI? (And it could very well be that it is all or nothing from an implementation standpoint)
Should types.proto be more generic, removed from gNOI and reused for all sub-projects rather than create the dependence -> gNOI (or complete redefinition in gNMI)
Thoughts on consistency across all OpenConfig projects implementing gRPC/GPB APIs (all projects must be versioned)

Also note, that it appears the release numbering is now also misaligned w/ the proto versioning (release 1.2.1 vs. proto 1.2.0)
https://github.com/openconfig/gnsi/releases/tag/v1.2.1
IMO a release that fixes something like stub regen should be a patch build off 1.2.0 (e.g. 1.2.0-20230605.00)

Thanks for the comments!
A version name is needed in the package so a client can support multiple major versions simultaneously.  This means the packages need to have different names.   Clients are expected to carry this burden so a given gNSI server will only need to support a single version.  In a large network, there will be many devices with multiple vendors and some rollout strategy that results in different g* services versions running on different devices.  So the client needs to take on this burden.
It's a good idea to consider using a semver build annotation if code regeneration is updated and the proto file is not changed.  But probably we should endeavor to update the proto and generated code together in one release so we don't need this extra annotation.
I'll put together a bigger proposal to make the versioning strategy consistent across the g* services and share in the near future.

A version name is needed in the package so a client can support multiple major versions simultaneously. This means the packages need to have different names. Clients are expected to carry this burden so a given gNSI server will only need to support a single version. In a large network, there will be many devices with multiple vendors and some rollout strategy that results in different g* services versions running on different devices. So the client needs to take on this burden.

Having the version in the package name is imo a good thing but one that has not been seen in any OpenConfig API work to date (common in many other gRPC/HTTP APIs).  Essentially when the package name is altered, you can consider this an entirely new specification for the API + endpoint.  In theory, you could not even consider the previous version however a protocol should be put in place of what is actually allowed/disallowed between say v1 -> v2 of a set of service specifications (e.g. being mindful of a transition path for the client + server)

It's a good idea to consider using a semver build annotation if code regeneration is updated and the proto file is not changed. But probably we should endeavor to update the proto and generated code together in one release so we don't need this extra annotation.

Agreed and this applies directly to my last comment where the release was updated to 1.2.1 diverging the proto IDL/specifications

I'll put together a bigger proposal to make the versioning strategy consistent across the g* services and share in the near future.

Thx - a few considerations worth noting

Within a backwards compatible major version, abide by protobuf language rules for compatibility
The package version is only uprev'd on major versions v1 -> v2
The semantic version within the IDL (or set of IDL under a common package) continues as-is.  (e.g. a change in the IDL from 1.2.0 -> 1.3.0 -> 1.3.1 all iterates the version extension but the package name remains at v1 throughout) - The published releases follow this scheme + any build annotations (to accommodate non-API changes)
For the semantic version extension, consider one "version" extension option that is leveraged for all APIs vs. each g* project redefining and/or needing their own imports (gNMI, gNOI + gNSI all reimplement the same concept today)
Where a set of services is to share a common version, consider making these all part of the same package.  For instance, if gNOI continues to be "per service" versioned under the umbrella project, that is fine and disaggregated in that one implementation can support a subset at a time easier.  For gNSI, since the goal is that all services come in tandem and are mandatory dependencies, all import and use a common version then I see no reason why these need separate package names (they could continue to just implement separate service blocks and share a common package/version)


Format more code in jsx pre tags

See #420 and #398. It should probably format more code in pre tags instead of turning off formatting in them.

I just stumbled upon this issue, and I do not understand why <pre> needs to be special cased at all. It looks like this has been introduced in https://github.com/dprint/dprint-plugin-typescript/issues/398, but the two snippets of code in that issue ("input" and "unexpected output") have exactly the same runtime semantics. Whitespace inside JSX is not significant (all whitespace and newlines are collapsed to a single space), regardless of which tag contains it.
You can see in https://babeljs.io/repl#?code_lz=DwBwTgpgfAUABHYBjA9gEwnJAbAhgZ3wDlcBbCAXgCI8A7AcwFdd6IBaACwBdTsrYECAGTYuAbhBD64-IOGix-ELlpTxAFQ4BLfHB1x8KcnC4QAHlwB0I8QHolKtWNny7k6c4TBbqDLG_g0M4woJACiL6YOATEZJQ0Kkws7Ny8_C5wNhJOmQoOqh6a-vqGxqYW1gr2ygXiuW5Ost6R_raBsEA&sourceType=module (<- enable the React preset in the left sidebar) that the output code for the two cases is exactly the same.


Verify Webhook with Umlauts/Special Characters

Describe the bug
Dear team,
we use the SDK to verify webhooks and it works for 99% of the cases.
For the failing ones we noticed that the customer have special characters e.g. in the customer name (e.g Ü Ä Ö for German customers or Å/å) and the validation fails.
We added a unit test for these payloads and its reproducible - but we cant share it because it contains customer data.
How can we validate in these cases?
Thanks,
Max
Steps to reproduce
Added the following test with a vailing JSON payload.
Also tested it with a valid one to ensure that the code works.
Expected behavior
Webhook validation successful
Code snippets
/** @test */
        public function it_verifies_a_signature2(): void
        {
            $json = '<PAYLOAD WITH SPECIAL CHARACTERS>';

            $request = Psr17FactoryDiscovery::findRequestFactory()
                ->createRequest('POST', '/paddle/notifications')
                ->withHeader(
                    PaddleSignature::HEADER,
                    '<header>',
                )
                ->withBody(
                    Psr17FactoryDiscovery::findStreamFactory()->createStream($json),
                );

            $secrets = [
                // The correct key
                new Secret('<KEY>'),
            ];

            self::assertTrue((new Verifier(null))->verify($request, ...$secrets));
        }

### PHP version

PHP 8.2.7

### SDK version

latest

### API version

version 1

### Additional context

I can share the json payload / key via email if needed: firstname@lastname dot it

Hi @MaxMelcher thank you for taking the time to report this, we can definitely improve our test coverage here to ensure this package is handling this correctly.
I've taken some time to add a couple of test cases that I hope covers your use case here which I've added in PR #49. Would you be open to take a look and letting us know if this does indeed cover your usage?
With the tests passing in the PR it does prompt a few possible questions that maybe you could elaborate on for your usage.

How is the request object you provide to the verifier constructed?
Are you using any framework that is handling the request object first?
Is there any possible middleware running potentially mutating the request in anyway?

I'll also note that while testing this I noticed a strict difference between escaped characters and their printed counterparts, e.g. Ü would not verify the same as \u00dc which is how the raw request is received in my testing.
I'd really like to figure out the point of failure here so we can make this process as streamlined and robust as possible

Hi @mikeymike,
thanks for this - we might not have access to the raw request.
We use Azure LogicApp as receiver of the notification. This allows us to easily replay notifications and decouple the from our application.
Paddle --> LogicApp --> our Api that verifies the webhook payload.
The payload is decoded when the logicapp receives the paddle request - we have done this also in the with the previous paddle API that needed the public key to verify. With the old API we did not experience this issue with hundreds of transactions.
Not sure what we can do on our side. Any recommendations?
Thanks,
Max


Update Audio Data Loader to Support Mixture Dataset

Pulls in the New Mixture Features Into Audio Space!
Tested that this fixes the previous epoching errors in the whisper_tiny config.

Weights & Biases Link for the test training run on the Tutorial Docs!
https://wandb.ai/i18nlp/levanter/runs/rwaft6sv?nw=nwuserheld

also, huge relief that it actually works for audio!


Support mixed windows/linux by default via scheduling to linux

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

Fixes #1187

Special notes for your reviewer:
Release note:
support for running on heterogenous clusters (linux tests only)

@vladimirvivien in principal is this ok ? if so I can test it this wknd to make sure it works

rebased, can we get this into the next release :) ?

rebased, can we get this into the next release :) ?


npm_config_target_arch=arm64 install x64 on a x64 server

Issue Summary
when I run (on x64 server):
npm_config_target_platform=linux npm_config_target_arch=arm64 yarn add<EMAIL_ADDRESS>file node_modules/sqlite3/lib/binding/napi-v6-linux-musl-arm64/node_sqlite3.node

it returns:
node_modules/sqlite3/lib/binding/napi-v6-linux-musl-arm64/node_sqlite3.node: ELF 64-bit LSB shared object, ARM aarch64, version 1 (SYSV), dynamically linked, with debug_info, not stripped
when doing the same with v5.1.7:
npm_config_target_platform=linux npm_config_target_arch=arm64 yarn add<EMAIL_ADDRESS>file node_modules/sqlite3/build/Release/node_sqlite3.node

it returns:
node_modules/sqlite3/build/Release/node_sqlite3.node: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, with debug_info, not stripped
so v5.1.7 installs the x64 binary even if I say I want the arm64 binary
Relevant logs or output
see above
Version
v5.1.7
Node.js Version
v21.5.0
How did you install the library?
npm_config_target_platform=linux npm_config_target_arch=arm64 yarn add sqlite3@version

It seems we had the same issue with 5.1.7 - in our case it was with an Electron app and apparently it resulted in an x64 version being built instead of arm64. Reverting to 5.1.6 fixed the issue: https://github.com/laurent22/joplin/issues/9922
That's not great anyway to have such major bug in a patch release. Are the developers even aware of the issue?

npm uninstall sqlite3 --force
npm install<EMAIL_ADDRESS>--build-from-source --target_arch=arm64 --fallback-to-build --force
Reverting to 5.1.6 fixed


fix changelong and version

fixes for https://github.com/Azure/azure-sdk-for-python/pull/35203

/check-enforcer override


[devops] Fix failed analysis does not fail build

https://github.com/agnesoft/adev/runs/4680129661?check_suite_focus=true

https://github.com/agnesoft/adev/runs/4680129661?check_suite_focus=true


workaround for issue #12

workaround for issue #12 along with an added test that demonstrates the issue.
also added a stack.yaml build descriptor and a .gitignore.
Regards,
Thierry

Looks good, thanks for debugging!


Only one server as subuser

Hello,
unfortunately a user can only be a sub user at one server.
If i add user X as subuser to Server 1 and after that to Server 2.
He got perms to control Server 2 but at Server 1 it looks like:

And if i click on it:

The permissions entry in the MySQL Database for the user looks like:
{"XXX":{"key":"XXX","perms":["console.commands","console.power","files.view","files.edit","files.save"],"perms_gsd":["s:console","s:query","s:power","s:files","s:files:get","s:files:put"]}}
Could that be fixed? ;)

Already fixed in 0.8.0, probably related to https://github.com/PufferPanel/PufferPanel/issues/397

Perfect! When will 0.8.0 be available?

Eventually. Still having it tested by some other developers first. Its getting there though.

That is what I like to hear @DaneEveritt


kafka异步sink  clickhouse 404 问题

Search before asking

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

What happened
2022-10-11 13:55:45 | ERROR | ClickHouse-writer-callback-executor-1 | com.streamxhub.streamx.flink.connector.clickhouse.internal.ClickHouseWriterTask | [StreamX] Error ClickHouseSink executing callback, params =
{ user: default, password: ******, hosts: http://<IP_ADDRESS>:8123 }
, StatusCode = 404
2022-10-11 13:55:45 | WARN  | ClickHouse-writer-callback-executor-1 | com.streamxhub.streamx.flink.connector.clickhouse.internal.ClickHouseWriterTask | [StreamX] Next attempt to send data to ClickHouse, table = icc_test.BoardContNPMCycleTime2, buffer size = 10, current attempt num = 1, max attempt num = 1, response = NettyResponse {
statusCode=404
headers=
Date: Tue, 11 Oct 2022 05:51:10 GMT
Connection: Keep-Alive
Content-Type: text/plain; charset=UTF-8
X-ClickHouse-Server-Display-Name: icc-bigdata-db
Transfer-Encoding: chunked
X-ClickHouse-Exception-Code: 47
Keep-Alive: timeout=3
X-ClickHouse-Summary: {"read_rows":"0","read_bytes":"0","written_rows":"0","written_bytes":"0","total_rows_to_read":"0","result_rows":"0","result_bytes":"0"}
body=
Code: 47. DB::Exception: Missing columns: 'S17B' while processing query: 'S17B', required columns: 'S17B' 'S17B': While executing ValuesBlockInputFormat. (UNKNOWN_IDENTIFIER) (version <IP_ADDRESS> (official build))
StreamPark Version
v1.2.3
Java Version
1.8
Flink Version
v1.4.0
Scala Version of Flink
2.11
Error Exception
2022-10-11 13:55:45 | ERROR | ClickHouse-writer-callback-executor-1 | com.streamxhub.streamx.flink.connector.clickhouse.internal.ClickHouseWriterTask | [StreamX] Error ClickHouseSink executing callback, params =
{ user: default, password: ******, hosts: http://<IP_ADDRESS>:8123 }
, StatusCode = 404
2022-10-11 13:55:45 | WARN  | ClickHouse-writer-callback-executor-1 | com.streamxhub.streamx.flink.connector.clickhouse.internal.ClickHouseWriterTask | [StreamX] Next attempt to send data to ClickHouse, table = icc_test.BoardContNPMCycleTime2, buffer size = 10, current attempt num = 1, max attempt num = 1, response = NettyResponse {
	statusCode=404
	headers=
		Date: Tue, 11 Oct 2022 05:51:10 GMT
		Connection: Keep-Alive
		Content-Type: text/plain; charset=UTF-8
		X-ClickHouse-Server-Display-Name: icc-bigdata-db
		Transfer-Encoding: chunked
		X-ClickHouse-Exception-Code: 47
		Keep-Alive: timeout=3
		X-ClickHouse-Summary: {"read_rows":"0","read_bytes":"0","written_rows":"0","written_bytes":"0","total_rows_to_read":"0","result_rows":"0","result_bytes":"0"}
	body=
Code: 47. DB::Exception: Missing columns: 'S17B' while processing query: 'S17B', required columns: 'S17B' 'S17B': While executing ValuesBlockInputFormat. (UNKNOWN_IDENTIFIER) (version <IP_ADDRESS> (official build))

Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

please use english and describe your problem clearly


'/streamlit_keycloak/frontend/public/bootstrap.min.css.map': No such file or directory

Thank you for this component.
After solving the local development integration with keycloak with
"Realm Settings" -> "Security Defenses" -> "Content-Security-Policy" to
frame-src 'self'; frame-ancestors 'self' http://localhost:8501 localhost; object-src 'none';

I am able to use this.
For now the demo code is working, but somehow I am receiving a 404 for a file not being there.
streamlit                 1.26.0
streamlit-keycloak        1.1.1
2023-08-30 21:37:25.489 ComponentRequestHandler: GET [..]/.venv/lib/python3.11/site-packages/streamlit_keycloak/frontend/public/bootstrap.min.css.map read error
Traceback (most recent call last):
  File "[..]/.venv/lib/python3.11/site-packages/streamlit/web/server/component_request_handler.py", line 55, in get
    with open(abspath, "rb") as file:
         ^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: '[..]/.venv/lib/python3.11/site-packages/streamlit_keycloak/frontend/public/bootstrap.min.css.map'

Thanks  for the quick response.
It seems to work but throws an exception for each login increasing the bill from Sentry ;-)

Got the same issue here. Prevents my streamlit page from loading.

Same for me, what the status here?

Same for me too! Any help on this issue?

This should be easily fixable on my end by including the sourcemap as well. Does this crash your app or is it just a warning?

Same issue for me but the crashes my app.

Same problem here


Generate files from another package

Hi, I am pretty new to this project but found it very interesting!
I am working on building a hexagonal architecture structure in Go, and one of the thing I am doing is separating the ports (interface) from their corresponding concrete implementations adapters.
I was looking to compose my adapter with the weaver.Implements[T], where T is the port (interface, abstraction) which reside on another package.
There is this limitation now, of requiring the interface in the same package as the concrete implementation (when using the weaver CLI). For me that goes against the coupling and segregation concerns I want to enforce with this type of architecture.
I was wondering if that limitation be removed in the future.
Thanks

Hi @jpearll! Thanks so much for checking out the project. You're totally correct that right now we require a component interface and component implementation to be in the same package. We elaborate on this more in #98. I'll close this issue as a duplicate of #98, if that's okay with you. We'll definitely re-examine the limitation and see if it can be removed. Feel free to re-open if you have any other questions :)


[TIR][Bug] The buffer map, parameters and body do not agree on buffer variable identity.

Function parameters in TIR are incorrectly maintained when lowering. Even though there is no reason these should be different we don't maintain the variable identity between the function parameters, the buffer keys, buffer values and function body.
import tvm

output_shape = input_shape = (10, )
x = tvm.te.placeholder(input_shape, name="x")
y = tvm.te.placeholder(input_shape, name="y")
compute_output = tvm.te.compute(output_shape, lambda i: x[i] + y[i])
schedule = tvm.te.create_schedule([compute_output.op])
lowered = tvm.lower(schedule, [x, y, compute_output], simple_mode=False)
kernel = lowered["main"]
params = kernel.params
buffer_map_keys, buffer_map_values = zip(*kernel.buffer_map.items())
buffer_map_values = [buf.data for buf in buffer_map_values]

for p, k, v in zip(params, buffer_map_keys, buffer_map_values):
    print(f"param_match_key {p.same_as(k)}")
    print(f"value_match_key {k.same_as(v)}")
    print(f"param_match_value {p.same_as(v)}")

print(f"param_match_key {params[0].same_as(buffer_map_keys[0])}")
print(f"value_match_key {buffer_map_keys[0].same_as(buffer_map_values[0])}")
print(f"param_match_value {params[0].same_as(buffer_map_values[0])}")
print(f"params match in body {params[0].same_as(kernel.body.body.value.a.buffer_var)}")
print(f"key match in body {buffer_map_keys[0].same_as(kernel.body.body.value.a.buffer_var)}")
print(f"value match in body {buffer_map_values[0].same_as(kernel.body.body.value.a.buffer_var)}")

cc @Hzfengsy @junrushao1994 @tqchen

It seems that params are always the same as keys, but unequal with buffer.data, while the buffer.data is used in the body. Right?

Yeah it looks like the params in the PrimFunc signature are not the variables used in the function body.
pf = lowered["main"]
# `addr1` is the variable in the function signature
addr1 = pf.params[0].handle
# `addr2` is the variable contained in the corresponding buffer
addr2 = pf.buffer_map[pf.params[0]].data.handle
# `addr3` is the variable used in the function body
addr3 = pf.body.body.value.a.buffer_var.handle

# we can see addr2==addr3, but addr1 != addr2
print(addr1)
print(addr2)
print(addr3)

Is this or a bug or intentional? like, to avoid cyclic dependency? @tqchen

This is an expected behavior. See explaination of the buffer_map signature here https://github.com/apache/tvm/blob/main/include/tvm/tir/function.h#L57
Specifically, in the low level code, note there is a difference between the two things:

The DLTensor* pointer that points to the DLTensor* structure
The actual data pointer in the DLTensor structure that points to a data field.

The tvmscript format illustrates this fact more clearly
@tvm.script.tir
def fun(a: ty.handle, b: ty.handle):
    Abuf = tir.match_buffer((3,), a, dtype="float32")
    Bbuf = tir.match_buffer((3,), b, dtype="float32")
    A_buffer_var: ty.Pointer("float32") = Abuf.data
    B_buffer_var: ty.Pointer("float32") = Bbuf.data
    B_buffer_var[0] = A_buffer_var[0] + 1

See also a C version that illustrate the type signature)
void func(DLTensor* a, DLTensor* b) {
    A_buffer_var = (float*)a.data;
    B_buffer_var = (float*)b.data;
    B_buffer_var[0] = A_buffer_var[0] + 1;
}

In the low level IR, the buffer vars corresponds to the data field(A_buffer_var and B_buffer_var). While the parameter arguments corresponds to the pointer to the DLTensor* field. Their relation are maintained via the match_buffer statements, and translates to the buffer_map field in the function signature.
This behavior is intended because it is important to distinguish the following three things in the IR:

T0: in-memory pointer to the DLTensor*

T1: abstract buffer(Abuf, Bbuf) that only exists during compile time, but will be staged to the individual elements

T2: The pointer to the data field (A_buffer_var and B_buffer_var)

T0 is needed because we need to faithfully represent the argument being passed to a function.

T1 is needed, and cannot be T0 because in certainly cases like GPU, we cannot afford to pass a DLTensor* pointer directly to the kernel, but need to construct a buffer in an abstracted way, which get stagged to the individual fields

T2 is needed because this is the data pointer that we directly operates on.

Given this is more of a development question, we can also followup on the https://discuss.tvm.apache.org/


PSR HTTP standards

Hi, I wrote a wrapper around this library ... maybe someone is interested in it, because there are some basic HTTP (PSR) interfaces implemented: WORK IN PROCESS (maybe I can also use the Encode / Decoding of this lib?!)

PSR-3: Logger Interface
PSR-7: HTTP Message Interface
PSR-17: HTTP Factory Interface
PSR-18: HTTP Client Interface

https://packagist.org/packages/voku/httpful

Hi, Thank you for the suggestion. There was another ticket (#461) asking about implementing PSR-7's RequestInterface and ResponseInterface.


EventPublishingServiceDiscovery destroy check duplicated

In the org.apache.dubbo.registry.client.EventPublishingServiceDiscovery.destroy():
    public void destroy() {

        assertDestroyed(DESTROY_ACTION); // 内部调用了isDestroyed方法,如果已经destroy则抛出异常

        if (isDestroyed()) { // 此处再次进行判断,如果已经destroy,则打印日志
            if (logger.isWarnEnabled()) {
                logger.warn("It's ignored to stop current ServiceDiscovery, because it has been stopped.");
            }
            return;
        }

Can this place be optimized? If necessary, I can submit a PR.

Do it with confidence.

@hengyoush
Are you still paying attention to this issue?