feat: 0.2.0 migration script for connections

Adds migration script for the changes introduced by the out of band and did exchange pr. See the markdown file for a description of the changes.

Codecov Report

Merging #773 (b1428a7) into main (ad12360) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #773      +/-   ##
==========================================
+ Coverage   87.51%   87.67%   +0.15%
==========================================
  Files         430      432       +2
  Lines       10561    10698     +137
  Branches     1827     1840      +13
==========================================
+ Hits         9242     9379     +137
  Misses       1256     1256
  Partials       63       63

Impacted Files
Coverage Δ

packages/core/src/agent/MessageReceiver.ts
81.30% <ø> (-0.84%)
:arrow_down:

packages/core/src/agent/TransportService.ts
100.00% <ø> (ø)

...modules/connections/repository/ConnectionRecord.ts
95.65% <ø> (-0.10%)
:arrow_down:

packages/core/src/storage/BaseRecord.ts
86.95% <ø> (ø)

packages/core/src/agent/MessageSender.ts
85.79% <100.00%> (-0.16%)
:arrow_down:

...ore/src/modules/connections/DidExchangeProtocol.ts
86.81% <100.00%> (ø)

.../modules/connections/services/ConnectionService.ts
86.38% <100.00%> (+0.16%)
:arrow_up:

...ages/core/src/modules/dids/repository/DidRecord.ts
100.00% <100.00%> (ø)

.../modules/dids/repository/didRecordMetadataTypes.ts
100.00% <100.00%> (ø)

...re/src/modules/oob/messages/OutOfBandInvitation.ts
95.00% <100.00%> (+0.47%)
:arrow_up:

... and 3 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 ad12360...b1428a7. Read the comment docs.


Change the SingleRows.next method to retain error and timeout values

Codecov Report

Merging #35 (1372fda) into master (b8ab640) will decrease coverage by 0.08%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   81.97%   81.90%   -0.08%
==========================================
  Files          12       12
  Lines        2164     2166       +2
==========================================
  Hits         1774     1774
- Misses        390      392       +2

Flag
Coverage Δ

unittests
81.90% <0.00%> (-0.08%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lib/rows/single.lua
88.46% <0.00%> (-7.38%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Keychain-swift doesn't compile

I recently added keychain-swift using Cocoapods to my project, and it doesn't compile. It has several errors in the main "KeychainSwift.swift" file.
Some of the errors are:
Offending Code: lastResultCode = SecItemDelete(query as CFDictionary)
Error Message: [String:Any] is not convertible to 'CFDictionary'; did you mean to use 'as!' to downcast?
Offending Code: let data = Data(bytes: bytes)
Error Message: Use of undeclared type 'Data'
Offending Code: if let value = value.data(using: String.Encoding.utf8) {
Error Message: Value of type 'String' has no member 'data'
Offending Code: @discardableResult
Error Message: Unknown attribute 'discardableResult'
Offending Code: lastResultCode = withUnsafeMutablePointer(to: &result) {
Error Message: Extraneous argument label 'to:' in call
Offending Code: public func getData(_ key: String) -> Data? {
Error Message: Use of unresolved identifier 'Data'
You're also getting a ton of warnings for this:
Error Message: Extraneous '_' in parameter: 'value' has no keyword argument name

Hello @jpcguy89, thanks for the feedback. The recent version of KeychainSwift is written in Swift 3.0. if you are using a previous version of Swift you can setup an older version of the library using these instructions
https://github.com/marketplacer/keychain-swift/wiki/Legacy-Swift-versions
Let me know if it helps.

Do you still have this issue, @jpcguy89?

Closing the ticket, feel free to open if the issue still exists.


[Bug] allow http repository in cosign check

Kyverno Version
1.12
Kubernetes Version
1.27
Kubernetes Platform
EKS
Description
i want to use an http repository to check signed image but get an error with this content that server gave http respone to https client
Steps to reproduce

i dont know

Expected behavior
i
Screenshots
No response
Kyverno logs
No response
Slack discussion
No response
Troubleshooting

[X] I have read and followed the documentation AND the troubleshooting guide.
[X] I have searched other issues in this repository and mine is not recorded.

Hi @saeedmohassel - we will need as many details as possible to help you troubleshoot the issue. Any logs/errors/policy manifests that you can share?

cc @vishal-chdhry


Non ptp epoch restriction

Resolves #80

When matching trodes timestamps from position tracking and rec file, first restrict the rec trodes timestamps to those corresponding to a rec system clock time within the epoch
Enables accurate mapping when trodes timestamps are not uniques across epochs, but are unique within the epoch
Gives minor speed and efficiency improvements
Added documentation for get_position_timestamps()

Confirmed that these changes resolve conversion with non-ptp, non-unique trodes timestamps conversions


TEI-serializer event export

currently events have no TEI-representation. Would be nice…

Feel free to provide an example how an event could look like in tei


Windows requires byte prepended to data

Also, as a separate note, I had to deploy on Windows and to make it work I had to send an additional 0 to the devices. Then everything was working; without it the lights were ignoring any commands. I have no idea why it works with the 0.
And I had 2 different devices, a Luxafor ORB and a Kuando, and both behaved the same.
if "WINDOWS" in platform.system().upper():

    def patched_write_strategy(instance: lightslib.Light, data: bytes) -> int:
        data = bytes([0]) + data
        return instance.device.write(data)

    lightslib.Light.write_strategy = patched_write_strategy

Originally posted by @aponoran in https://github.com/JnyJny/busylight/issues/416#issuecomment-2501453875

@aponoran  I created a new issue to track the Windows extra byte problem.
As I mentioned in #416, we need to preserve the write_strategy property which encodes which write method a device needs: write or send_feature_report.  I mentioned making the windows check in the __bytes__ magic method but I think it makes more sense to check for windows in the method: busylight.lights.light.Light.update. I'm going to commit an update in a second with a terrible first pass check for Windows.

Possible fix for #87 and #56.

I tested on Windows with the above commit and everything works fine! Used a Luxafor ORB and a Kuando Omega.

This fix is now available in version 0.33.0. Again, thanks for your help!


alternative simple notifier

I wanted to take a fairly different approach to harbormaster integration.  Specifically:

Don't rely on a local arc install
Use staging repositories for handoff: https://secure.phabricator.com/book/phabricator/article/harbormaster/  This allows just checking out refs/tags/${GIT_TAG} using existing plugins & paramters.

Having two "phabricator jenkins plugins" seems like an obviously bad state of affairs, so I wanted to show what I had so far and see if was trending towards a mergable direction.  I tried to not touch any of the existing stuff except to make it optional.  This creates some annoying global duplication in config, but I migration appeared very complicated (I don't have a whole lot of experience with jenkins plugin development).  Longer term (like after Phabricator does something with it) I think generic integration wtih code and line coverage would be an obviously good path to go down, and this plugin seems well poised to do that.
It that sounds reasonable let me know what needs to be cleaned up.  I'm also usualy in #phabricator if this is totally confusing and needs more context.
Known issues:

This produces 2 Phabricator sections in the global config page.  I don't know what jelly magic is needed to fix that.

This is great, and is something I'm tracking at the source tree in https://github.com/uber/phabricator-jenkins-plugin/issues/21
I noticed that the staging support recently landed, and was going to see if I could implement it in my free time.
Would you mind closing this here and re-opening against https://github.com/uber/phabricator-jenkins-plugin so we can track all issues / code review there?

And yes, I have a branch where I started integrating Harbormaster lint/unit/coverage results since I saw that merged into phab upstream a couple days ago

Okey dokey.  You might want to put that in the README, as far as I know the 'normal' thing for jenkins plugins is issues in jira and pull requests to the jenkinsci repo.


datapath: clarify comment for EncryptNode

EncryptNode is no longer limited to IPsec. And in fact it currently is only supported for Wireguard.

/test


Can the hub be imported as a managed cluster?

Describe the bug
I am using the following script to import one managed cluster -> https://github.com/open-cluster-management-io/ocm/tree/main/solutions/setup-dev-environment
However I also want to import the hub as a managed cluster too (like self-managed). And I modified the script to make the hub join and try and accept the CSR for it from the hub. However I get the following error about not finding the CSR for hub
Expected behavior
I expect the hub to be also imported as a managed cluster

`+ echo 'Accept join of hub,cluster1'
Accept join of hub,cluster1

KUBECONFIG=/tmp/hub.yaml
IS_KIND_ENV=true
clusteradm accept --context kind-hub --clusters managed,hub --skip-approve-check
Starting approve csrs for the cluster managed
CSR managed-c592l approved
CSR managed-t9mlg approved
set hubAcceptsClient to true for managed cluster managed

Your managed cluster managed has joined the Hub successfully. Visit https://open-cluster-management.io/scenarios or https://github.com/open-cluster-management-io/OCM/tree/main/solutions for next steps.
Error: [managedclusters.cluster.open-cluster-management.io "hub" not found, no csr is approved yet for cluster hub]
make: *** [mco-kind-env] Error 1
coquadro@coquadro-mac multicluster-observability-operator % kubeclt get managedclusters -A
zsh: command not found: kubeclt
coquadro@coquadro-mac multicluster-observability-operator % kubectl get managedclusters -A
NAME      HUB ACCEPTED   MANAGED CLUSTER URLS             JOINED   AVAILABLE   AGE
managed   true           https://hub-control-plane:6443   True     True        7m2s
coquadro@coquadro-mac multicluster-observability-operator % `

Hi @coleenquadros Yes, the hub can be imported as a managed cluster.
Can you paste your commands for init the hub and join the cluster? if you are using a kind cluster, remember to add the --force-internal-endpoint-lookup for the clusteradm join command. Here is the doc for your reference.

HI @zhujian7
Here is the function
`
deploy_hub_and_managed_cluster() {
cd $(dirname ${BASH_SOURCE})
set -e
hub=${CLUSTER1:-hub}
c1=${CLUSTER1:-managed}
hubctx="kind-${hub}"
c1ctx="kind-${c1}"
echo "Initialize the ocm hub cluster\n"
clusteradm init --wait --context ${hubctx}
joincmd=$(clusteradm get token --context ${hubctx} | grep clusteradm)
echo "Join hub to hub\n"
$(echo ${joincmd} --force-internal-endpoint-lookup --wait --context ${hubctx} | sed "s/<cluster_name>/$c1/g")
KUBECONFIG=/tmp/managed.yaml IS_KIND_ENV=true
echo "Join cluster1 to hub\n"
$(echo ${joincmd} --force-internal-endpoint-lookup --wait --context ${c1ctx} | sed "s/<cluster_name>/$c1/g")
echo "Accept join of hub,cluster1"
KUBECONFIG=/tmp/hub.yaml IS_KIND_ENV=true
clusteradm accept --context ${hubctx} --clusters ${c1},${hub}  --skip-approve-check
kubectl get managedclusters --all-namespaces --context ${hubctx}
}
`

I think I found the issue @zhujian7
I duplicated managed clusters command. It worked.
 echo "Join hub to hub\n"
  $(echo ${joincmd} --force-internal-endpoint-lookup --wait --context ${hubctx} | sed "s/<cluster_name>/$c1/g")

I will close the issue. Thank you


Reflection: conditionally build the target library

There is a build of the reflection library for the tools which we want to build
if we are building the tools.  The target library should only be built if we are
building the standard library.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @gottesmm

@swift-ci please test


Cannot compile sql query for constants

If I want to push initial work into sql for this query:
P = scan(public:adhoc:TwitterK);
PP = [from P emit 3 as three];
store(PP, Test);

it fails
Traceback (most recent call last):
  File "/base/data/home/runtimes/python27/python27_lib/versions/third_party/webapp2-2.5.2/webapp2.py", line 570, in dispatch
    return method(*args, **kwargs)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/myria_web_main.py", line 517, in post
    self.get()
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/myria_web_main.py", line 495, in get
    push_sql=push_sql)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/myria_web_main.py", line 124, in get_physical_plan
    multiway_join=multiway_join, push_sql=push_sql)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/myria_web_main.py", line 110, in get_plan
    push_sql=push_sql)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/myrial/interpreter.py", line 419, in get_physical_plan
    return self.__get_physical_plan_for__(target_phys_algebra, **kwargs)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/myrial/interpreter.py", line 408, in __get_physical_plan_for__
    return optimize(logical_plan, **kwargs)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/compile.py", line 64, in optimize
    return optimize_by_rules(expr, target.opt_rules(**kwargs))
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/compile.py", line 53, in optimize_by_rules
    expr = recursiverule(expr)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/compile.py", line 44, in recursiverule
    newe = rule(e)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/rules.py", line 19, in __call__
    return self.fire(expr)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/language/myrialang.py", line 1230, in fire
    sql_plan = cat.get_sql(expr)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/language/sql/catalog.py", line 181, in get_sql
    return self._get_unary_sql(plan)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/language/sql/catalog.py", line 131, in _get_unary_sql
    input = self.get_sql(plan.input).alias("input")
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/language/sql/catalog.py", line 181, in get_sql
    return self._get_unary_sql(plan)
  File "/base/data/home/apps/s~myria-web/1.380988297246986780/raco/language/sql/catalog.py", line 141, in _get_unary_sql
    for (name, e) in plan.emitters]
AttributeError: 'int' object has no attribute 'label'

confirmed.
fix aspect 1: make sure the push-into-sql failure detection catches this error. it will not be pushed down, but it should fall back to the in-myria implementation.
fix aspect 2: make it push down correctly.


There should be priority labels added to issues in the repository

I am adding lots of issues to this repository yet many of them are small UI stuff or low priority. Adding a priority label to different issues may help sort through them.

Is there a way for me to add labels to issues I open or only @duke7553  have access to do that?

I'm pretty sure that I only have access to that. Sorry. If you think an issue is critical or something, please let me know in the issue report and I'll evaluate that claim.

Closing this now, as I will add the priority labels to the issues.

@yaichenbaum There is now a way to flag a bug as high/critical priority.


Remove GetTypeProvider

Supports https://github.com/microsoft/typespec/issues/3805

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.


Request: change button markup

Hey there. Roto is working great, but I have a feature request regarding the buttons.
Not sure if this is where I'm supposed to post this, haven't used github before.
Can you please make button detection work purely via class names ("prev"/"next"), not on position (at the moment they must be after the UL tag) or type (so DIV's, A's, etc can be used, and not just BUTTONs).
This example would put a foward (<) and backward (>) button on either side of a dynamically-resized UL:

<

>

123

456

789

... whereas with the current system it's a fair bit more complicated (the UL block won't dynamically resize to fill only the space remaining on the line if floating objects are drawn after it).
Also (forgive me if this sounds a bit silly), but can you please use object arrays (c[0], c[1], c[2] etc) instead of manual names (c0, c1, c2 etc) in the demo code (as per the example above)?
Thanks!

I am aware that it is also possible to specify the buttons via ID, but it would be nice to have them done automagically by inserting them anywhere inside the carousel outer container.
NB: For this all to work you'd also need to specify an offset for all LI elements via a class definition (as opposed to offsetting just the UL element). In the 5-min testing hack I made on your script, however, I just stuck the UL inside an overflow:hidden container div.

Hi there,
OK, I see the problem you have. This is caused by the new flexibility to have block-level elements other than ULs be rotoed  -- the script now presumes the first child of the container is the "rotoFrame", rather than looking for a UL.
I've worked around this by adding an optional classname, "rotoFrame", which you can apply to the element you want to move within the container. You can see this in action in the new demo page. That allows you to put the buttons anywhere within the container you want.
Can I ask why you want me to use array notation in the element ids?

Just on the demo, looks nicer (additional elements will be auto incrementing by design, lends itself well to even the simplest AJAX implementations). Probably not even necessary.

Cheers, will try out the rotoFrame def momentarily. Any plans to apply a positioning offset to the LI elements instead of the UL container? Though this may be problematic to people using the aforementioned new flexibility on elements without containers, or with pre-specified offsets (without some additional workaround code).

On 24 Oct 2011, at 13:49, Neftaly Hernandez wrote:

Cheers, will try out the rotoFrame def momentarily. Any plans to apply a positioning offset to the LI elements instead of the UL container? Though this may be problematic to people using the aforementioned new flexibility on elements without containers, or with pre-specified offsets (without some additional workaround code).

Not really -- I had looked at it, but you'd be setting potentially quite a few offsets rather than just one, so there would most likely be performance implications, as well as implications for the complexity of the code ...
R

Even if you're instead changing a CSS class rule applicable those elements (position:relative or a +/- margin)? I'd imagine the resultant positioning updates would be heavily optimized by the browser.
Sucks that negative padding doesn't work =/

Well, it might be OK in a modern browser, using CSS transforms, but falling back to jQuery.animate would most definitely be slower. I don't think the benefits justify that, or the increase in code complexity.

Bummer, thanks anyway.


Feedback

I used this tool for testing my web.
It was so helpful for me .
Even a normal php developer having no knowledge in automation can use it .
Keep up good work guys .

Hi @lakinmohapatra thanks for your feedback! You're welcome.
Also want to share with you a list of some new tools for test automation with headless Chrome -
https://medium.com/@kensoh/chromeless-chrominator-chromy-navalia-lambdium-ghostjs-autogcd-ef34bcd26907
I've also updated that list to the awesome test automation list (not affiliated with them) but find that list very useful to keep track of various testing tools. They also have a list for PHP tools there :smile:
https://github.com/atinfo/awesome-test-automation

Closing the issue for now, thanks for your feedback @lakinmohapatra !


Fix grammatical error for 'Account Information' translation

Hi,
While developing a Magento site using your language pack I noticed a grammatical error in the translation for 'Account Information' (in Dutch, separate words that combine into a noun should always be concatenated), and thought you might want to know about it.
Best,
Sjoerd

@sderoij can you solve the merge conflicts please?

@sderoij I have copied your change to the nl_NL.csv file.


Add Overwrite Support For github_repository_file

Fixes https://github.com/terraform-providers/terraform-provider-github/issues/438
/cc @techdragon @shoekstra @cornfeedhobo

Thanks for raising discussion on the update case.  I had not given it too much thought.
After thinking about this a bit, I recommend we rebrand overwrite to overwrite_on_create.
We can then assume that whenever the update code path is executed, we will always be overwriting a file, which matches the existing implementation.
To further safeguard, we can add the ForceNew attribute to overwrite_on_create so that if configuration changes, we can start fresh with a new file.  What do you think @anGie44 ?

q

Is there any news on when this might be reviewed/merged?

I recommend we rebrand overwrite to overwrite_on_create.
I like this idea, suits the intended behavior 👍 my only hesitation would be adding the ForceNew attribute since the field is intended to be used only at create time and it might be more of a pain point to force recreation for users

Tests are looking good.  The GHES ones are failing as the infrastructure is down at the moment.


Add Snappy Driver Installer Origin

Note that unlike many apps that claim themselves to be a driver installer, SDIO is actually legit.

@Ash258 Thanks again, fixes applied.


Show extended values of names

Make sense to handle names with a similar component of hashes but different logic for ellipsing string.
Consider the fully responsive approach which will use whitespace better.

Possible solution https://jsfiddle.net/maxim_mazurok/oujctpz8/56/

It was impossible to achieve with given tools. Better to drive it manually


use annotations proxy log level

The proxy config map should use the annotation value of vertica.com/client-proxy-log-level. This pr is for updating the logic by replacing the hardcoded value.

Looks good! Minor: can you add a webhook rule to check that the log level is one of these : TRACE|1=DEBUG|2=INFO|3=WARN|4=FATAL|5=NONE?

Looks good! Minor: can you add a webhook rule to check that the log level is one of these : TRACE|1=DEBUG|2=INFO|3=WARN|4=FATAL|5=NONE?

Sure, I will add one in the webhook PR


Doc update for 3rd party quick start

Proposing some changes to https://github.com/highlightjs/highlight.js/blob/master/extra/3RD_PARTY_QUICK_START.md
I worked out the process while working on https://github.com/highlightjs/highlightjs-curl, so that repo and this doc are in alignment, as well as the proposed template repo https://github.com/jf990/highlightjs-language-template
based on a review here there are a few final changes to complete, but wanted to get some feedback before finishing.
Changes
Update https://github.com/highlightjs/highlight.js/blob/master/extra/3RD_PARTY_QUICK_START.md to improve the doc.
Checklist

[ ] Added markup tests, or they don't apply here because... (doc update)
[ ] Updated the changelog at CHANGES.md TBD

updated and tested inside extra as a cloned repo and updated assuming we are going to move forward with the template repo. @joshgoebel @allejo suggestions applied.

Under testing we should mention ONLY_EXTRA env which will more tightly scope the test to only running against languages inside extra rather than all the core languages.  This only works for markup tests, but those are the most common that need to be run over and over.
ONLY_EXTRA=true npm run test-markup

@joshgoebel suggestions are up. thanks!


Replication fails on first update after I restart orientdb cluster 2.2.0-beta

Replication fails on first update after I've restarted orientdb cluster.  All updates afterwards works fine.  The first update is lost.  I tested by updating via the http orientdb server.
Using orientdb-community-2.2.0-beta in Linux.

Please could you post the file distributed-config.json under your database directory?

{"@type":"d","@version":0,"version":3,"servers":{"@type":"d","@version":0,"nysobase0023":"master","nysobase0024":"replica","nysobase0025":"replica"},"clusters":{"@type":"d","
@version":0,"internal":{"@type":"d","@version":0},"index":{"@type":"d","@version":0},"*":{"@type":"d","@version":0,"replication":true,"servers":["nysobase0023","nysobase0024"
,"nysobase0025","<NEW_NODE>"]}},"replication":true,"resyncEvery":15,"autoDeploy":true,"hotAlignment":false,"executionMode":"asynchronous","readQuorum":1,"writeQuorum":1,"fail
ureAvailableNodesLessQuorum":false,"readYourWrites":true}

You have writeQuorum to 1, but in order to have consistency you should have the majority, so 2.

I've update writeQuorum to 2.  Deleted distributed-config.json.  Restarted.  Confirmed writeQuorum is now 2.  The same issue.  First update to database is not being replicated.

Do you have any log of the failure of first update in any of the 3 servers?

There is nothing logged.  I've tried upping log verbose from INFO to DEBUG, but nothing is logged.

How can I replicate this problem? Do you experiment the same problem with an empty db or just with yours?

Any database on cluster.  After I stop and restart all orientdbs in cluster.  I update any field for a record in database, for example GratefulDeadConcerts with the web interface.  The changed field doesn't show on a select on the other servers.

This has been fixed in v2.2.0-rc1 and v2.2.0-SNAPSHOT.


paket pack succeeds even when there're missing files

For file type of paket.template, even when some files specified at file section cannot be located, paket pack would still succeed without reporting any error/warning. Is there a switch that can let it fail in case it cannot locate one or more specified files?
Thanks!

mhm. I think it should fail in that case.

fixed in latest alpha


Exception when generating JSON format on jira plugins

Hi,
I am trying to use maven owasp dependency checker on my jira plugin custom devs. I am facing an issue when generating a report in JSON format.
A mvn -X command gives the following exception:

[DEBUG] Malformed JSON?
com.google.gson.stream.MalformedJsonException: Invalid escape sequence at line 72784 column 61 path $.dependencies[272].identifiers[0].name
at com.google.gson.stream.JsonReader.syntaxError (JsonReader.java:1568)
at com.google.gson.stream.JsonReader.readEscapeCharacter (JsonReader.java:1559)
at com.google.gson.stream.JsonReader.nextQuotedValue (JsonReader.java:1014)
at com.google.gson.stream.JsonReader.nextString (JsonReader.java:815)
at org.owasp.dependencycheck.reporting.ReportGenerator.prettyPrint (ReportGenerator.java:453)
at org.owasp.dependencycheck.reporting.ReportGenerator.pretifyJson (ReportGenerator.java:404)

See full log at https://gist.github.com/fsergot/609c242ca3f294b10c8f8afbe41a1e43
Note this works for other formats (tested with HTML & XML) so just the JSON fails.

Is there any chance you could share the generated JSON report that is generated? The debug error is stating that the originally output file could not be read in and pretty-printed. So somehow the original template is generating malformed JSON. However, without an example I would be at loss as how to fix the issue.


Corrupt sass code

Does CssComb.js work with SASS?
I've tried to use it with SASS, and it's corrupted code. See screenshot of git diff.

Why?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/5739085-corrupt-sass-code?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).

I have similar experiences with same kind of input and output

Before:
.block
  +media(lap)
    color: green

  +media(palm)
    color: red

  color: black

  +last
  +hide-text

After:
.block
color: black

  +media(lap)
    color: green

  +media(palm)
    color: red

  +last
  +hide-text

There are two spaces in line before +last.
Config:
{ "sort-order": [

["color"], ["$include"]

]}

Run in command line: node lib/cli.js include-specified-2.sass
"version": "3.1.5" (current dev branch)
"csscomb-core": "3.0.0-4"
"gonzales-pe": "3.0.0-29"

@hudochenkov, green test for your issue, fixed in dev: https://github.com/csscomb/csscomb.js/blob/dev/test/options/sort-order-sass/test.js#L101

@kalashnikovisme, green test for your issue, fixed in dev: https://github.com/csscomb/csscomb.js/blob/dev/test/options/sort-order-sass/test.js#L108


Change the base image from java to openjdk

The Java image was deprecated in favor of the OpenJDK and it will stop being maintained at the end of the year (2016-12-31). This change addresses the issue and shouldn't affect any image as the Java image is running OpenJDK.

Thanks!


Add download option

In Random Anime Profile , check it here

Add a button to save the image
Add a loder to whole card when data is getting fetched up and showing up on the webpage

Hey @swapnilsparsh please assign me this task under IWOC'23. I have already made changes in the file.


RECONSUME_LATER only retry once

I trace the MQClientAPIImpl.consumerSendMessageBack function, and found it works correctly in client, when renturning ConsumeConcurrentlyStatus.RECONSUME_LATER. But the message do not enter %RETRY%ConsumerGroup topic. Sometimes the message enter the retry topic once, but it just disappear even when the client throw RECONSUME_LATER again.
The problem cluster version is 4.3.1. Another cluster having almost same configs and version do not have the problem.
So, I can not find a clue.

I found the broker in the problem cluster has broken consumequeue/SCHEDULE_TOPIC_XXXX strucure, how can I fix it?

The current ScheduleMessageOffset_4 is invalidly larger then the max ScheduleMessageOffset_4, After I sent enough msg to this delay level, the problem gone.

@lastboy1228 could you provide more information about this issue, you can describe it in Chinese.

@lastboy1228 could you provide more information about this issue, you can describe it in Chinese.

用“mqadmin brokerStatus”命令，看到各个scheduleMessageOffset_X里，“已处理offset” 大于 “已接收的offset”。（可能是转生产时，拷贝了测试集群的store/consumequeue文件夹）
在我贴的图里，scheduleMessageOffset_4的已处理记录是6012，已接收记录是395；第二次消费失败的消息，进入scheduleMessageOffset_4后就不再被broker处理和投递了。
我手动生成大量各个延迟级别的消息，使得“已处理offset” 小于 “已接收的offset”后，新的延迟消息就开始被正常投递了。问题已解决


[Feature]: TimePicker Component

Describe the feature
I really like the UI component library and hope to add a TimePicker component to select or input the set time (hour, minute, second) .For example, like this：

Additional information

[ ] I intend to submit a PR for this feature.
[ ] I have already implemented and/or tested this feature.

We have TimeField component implemented, and seeing that this is a very custom component, I'll create it as an example like these components


Remove bgCacheReg.done

The API is intended for Service Workers and most of the time will expect that the download/upload completes a significant time after it's started. In that case, most consumers will be implementing event listeners to deal with success/failure.
If event listeners are going to be the expected use case for notification on status, I'd suggest you remove the "done" promise just to keep the API clean.

I wanted .done for page-based code, but you can replicate this using SW and postMessage.
I'll remove this, but we should look into ways of exposing progress so pages can show UI. Happy for that to be v2.

Done, although I ended up recreating it in most of my examples, see backgroundFetchDone https://github.com/jakearchibald/background-fetch/blob/master/README.md


Allow update without upper bound on iterations.

@jluttine, let me know if you'd like PRs in larger chunks (rather than one PR per change).
Anyway, sometimes it's useful to run the algorithm until it converges without knowing an upper bound on the number of iterations. Passing repeat=None will run until the convergence test is satisfied.

One PR per change is convenient for me. Thanks for the PR, this was a good idea!


Missing the triggering test cases in the checked out version

I encountered an issue with the Defects4J framework while attempting to retrieve and verify triggering test cases for several project versions. Specifically, some of the triggering test cases listed by Defects4J do not appear to exist in the project's source code after checkout.
Steps to Reproduce:
Check out CLI version 16b:
defects4j checkout -p Cli -v 16b -w ./tmp

Export triggering test cases:
../defects4j/framework/bin/defects4j export -p tests.trigger -o output.txt -w ./tmp

Inspect the output.txt file for triggering test cases. The following were returned:
org.apache.commons.cli2.bug.BugCLI123Test::testMultipleChildOptions
org.apache.commons.cli2.bug.BugCLI123Test::testParentOptionAndChildOption
org.apache.commons.cli2.bug.BugCLI123Test::testSingleChildOption
org.apache.commons.cli2.commandline.DefaultingCommandLineTest::testGetOptions_Order
org.apache.commons.cli2.commandline.PreferencesCommandLineTest::testGetOptions_Order
org.apache.commons.cli2.commandline.PropertiesCommandLineTest::testGetOptions_Order
org.apache.commons.cli2.commandline.WriteableCommandLineImplTest::testGetOptions_Order

Manually search the project directory for the testGetOptions_Order test cases in the specified classes.
Observed Behavior:
The following test cases do not exist in the checked-out version upon manual inspection:
org.apache.commons.cli2.commandline.DefaultingCommandLineTest::testGetOptions_Order
org.apache.commons.cli2.commandline.PreferencesCommandLineTest::testGetOptions_Order
org.apache.commons.cli2.commandline.PropertiesCommandLineTest::testGetOptions_Order
org.apache.commons.cli2.commandline.WriteableCommandLineImplTest::testGetOptions_Order

The remaining test cases (BugCLI123Test) were found as expected. Similar issues were observed in the following project versions: Cli 16, 22, 27, 30, Codec 1, Lang 8, 9, 10, 54, 57, Math 6, 12, 22, 29, 41, 43, 69, 101, 102, and Mockito 1, 4.
Expected Behavior:
All triggering test cases listed by Defects4J should exist in the project after checkout.
Environment:
Defects4J version: 2.0.1
Operating system: ubuntu 20.04

Hi @Md-Siam07,
The test classes in the commandline directory extend CommandLineTestCase. The testGetOptions_Order test method is defined in that super class.
If you want to resolve an executed test method to the test class that defines it (as opposed to the class on which it was called), then you may need to analyze super classes as well.
Hope this helps.
Best,
René

Hi @rjust,
Thank you for your reply. In such case, wouldn't getting the triggering test cases as org.apache.commons.cli2.CommandLineTest::testGetOptions_Order be more straightforward?

The output is correct in that these are the test classes on which the testGetOptions_Order methods are invoked. The behavior of testGetOptions_Order may depend on the runtime type of the test class.
Additionally, if a base class (say Base) is abstract and implements a common test method (say testInherited), then outputting Base::testInherited would be incorrect: this test method cannot be invoked on the given test class, and the output is insufficient to determine the actual subclass on which the method needs to be invoked by JUnit.
What you are asking for is an analysis that determines in which class a particular test method is implemented. This is different from Defects4J's dynamic analysis that outputs the triggering test cases, which if executed via JUnit reliably trigger the bug.
We would welcome a contribution with an automated analysis that, given a triggering test class and method, outputs the class that implements that method. You may be able to derive that information from (verbose) logging or a simple analysis (static or dynamic via reflection) that starts from the invoked test class and looks for the test method; if not found, it will continue with the direct superclass.

Thank you for clarifying, @rjust.


Adding panic test for unset environment variable

Purpose:
Part of onboarding exercise: https://swift-nav.atlassian.net/browse/DOL-160, please feel free to reject this PR
Technical Detail:
By setting and unsetting environment variable for the "WORKSPACE" key, it's able to test whether the unwrapping of test_data_dir() would panic when unit testing

Could add some test cases for missing S3StorageClass uploads (right now only tests Standard and RRS, could generalize into a S3StorageClass test or something) if needing some ideas...
https://github.com/swift-nav/esthri/blob/c0267742665e6ea890d8e96af3e14837b29a30f5/crates/esthri/tests/integration/upload_test.rs#L119-L163

Thanks for the idea, I will close this PR and work on your suggestions
On Tue, Sep 27, 2022 at 4:27 PM adrian-kong @.***>
wrote:

Could add some test cases for missing S3StorageClass uploads (right now
only tests Standard and RRS, could generalize into a S3StorageClass test or
something) if needing some ideas...
https://github.com/swift-nav/esthri/blob/c0267742665e6ea890d8e96af3e14837b29a30f5/crates/esthri/tests/integration/upload_test.rs#L119-L163
—
Reply to this email directly, view it on GitHub
https://github.com/swift-nav/esthri/pull/363#issuecomment-1259037461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZIWP6BJ6V6R7HXQWNSX2G3WAKHVLANCNFSM6AAAAAAQWN46CQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Reject duplicate keys on the notifier

Signed-off-by: Burcu Dogan<EMAIL_ADDRESS>

Do you think we should return an error from that method since a duplicate id is more of a user error than the send on closed channel?

OK, I am happy to break the APIs and one. The call site of (*Notifier).Add is commented out in the supervisor -- there are no changes out of the scope of the chanotify package.

LGTM


Session/tunnel leaks

Consumers should reasonably expect dropping a tunnel to cause it to get closed, and dropping the session and all tunnels tied to it to close the session. This doesn't seem to be happening.
Investigation should start at the muxado layer and work its way up until the culprit is found. Entirely possible that I messed it up at the very bottom level :stuck_out_tongue:

Resolved by #62


Cleaned up project organization

I moved all the projects into a more standard src folder. This helps keep things organized.

lgtm, I'm merging this pr.


Pre-Release Marketing Site Feedback

http://trackthetrolley.com/2/
Work in progress, but thoughts on the overall concept-direction?
@agautsc

I like the background video/design but I think it's a little too shaky. I think you might be able to stabilize the video digitally without it looking awful. And I don't really like how the scrolling is hijacked to make it 'smoother' or whatever.
But other than that I think it will look great once the content is filled in!

Good feedback. Thanks Adam
On Jul 6, 2015, at 12:49 PM, Adam Hodges<EMAIL_ADDRESS>wrote:

I like the background video/design but I think it's a little too shaky. I think you might be able to stabilize the video digitally without it looking awful. And I don't really like how the scrolling is hijacked to make it 'smoother' or whatever.
But other than that I think it will look great once the content is filled in!
—
Reply to this email directly or view it on GitHub.


[Feature] Compatible with upper and lower case when use hivesql queries paimon external table

Search before asking

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

Motivation

use sparksql wirte data into a paimon table(has upper case fields) by filesystem-catalog
use hivesql to create a paimon external table
use hivesql select upperCaseField(or uppercasefield) from paimon external table can work

Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I can fix it


testNamePattern typeahead polish

This is a ticket to keep track of all the polish needed for the testNamePattern typeahead.
Open questions:

What should happen when a it is already being filtered by the testFilePattern?
Do we want to update how we show each "row" of the results?
Should we use regex or fuzzy search?
How can we make sure that we show the correct number of matching tests?

cc: @cpojer @zouxuoz

My thoughts:

We should probably allow to make it so the two filters work together and potentially have a way to show that in the UI. Since we now collapse the Usage panel, we could probably print a line above that to show which filters are applied and add a way to clear them. cc @thymikee.
Maybe. You are the expert.
Fuzzy search. There is an issue for that somewhere on this tracker, I believe, and a proposal for a library. Question: should we change the file name pattern to be fuzzy as well? (probably, yes).

For the first one: we might need to amend the list of test results to add the testPath so that we can filter over them.

We should probably allow to make it so the two filters work together and potentially have a way to show that in the UI. Since we now collapse the Usage panel, we could probably print a line above that to show which filters are applied and add a way to clear them.

I'm all for that. Filters should be persistent and we need to show it to the user. They should also get cleared on Esc or Backspace (one by one?) I think.

Or c, for clear? :D

Where you guys thinking something like this for surfacing the active filters?

I think that sounds good. Is there any way to collapse both patterns and the headline into a single line and still make it useful?
"Active Filters filename: jest-cli.*watch, testname: foobar"
?

Sounds like a good idea, I'll give that a shot

Here is a quick mockup of how it could look, but I feel that the menu is starting to feel huge... I wonder if we should rethink how we show this menu

yeah, we probably should update it, although with @thymikee's work it is at least collapsing. I think your proposal is a good start.

Closing this in favor of #2595


Update index.md

The exclusion of contractions, and the addition of these extra commas, will further benefit the clarity and rhythm of the above text

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


update plugin to new API with ActivityAware

with flutterEmbedding v2. registrar->getActivity() returned null.
https://flutter.dev/docs/development/packages-and-plugins/plugin-api-migration

@mix1009 Thanks for your help :D


Auth0 - jQuery CDN needs to be replaced

Describe your problem and - if possible - how to reproduce it
Can't sign in after update,while I can sign in fcc forum as well using my account.
Add a Link to the page with the problem
Tell us about your browser and operating system

Browser Name: chrome /edge
Browser Version:
Operating System:

If possible, add a screenshot here

hi @Wood-Cube can you please send me an email mrugesh at freecodecamp dot org with all of the below details:

Screenshots of the Chrome DevTools (Press F12 on Windows/Linux OR Cmd + Options + I on macOS) and check the console tab.

The general region where you are located.

Exact steps that you were trying to follow when you get this error.

Closed by https://github.com/freeCodeCamp/auth0-templates/pull/4

@Wood-Cube this should have been resolved now. Can you please try again?

@raisedadead  非常感谢

You are welcome @Wood-Cube


Make an editable, autocomplete text box with select2

I want to build an autocomplete text box by using select2, which users can also insert a new value (some thing like html5 datalist).
To do this:

I made editable select as suggested at http://stackoverflow.com/questions/2141357/editable-select-element, by putting the input box on the select box.
Convert the select to select2
Disable the select 2 search filed.
???

Now I want to open select 2 when users type the text box by calling select2("open") and pass every keyup to select2 search field which has the .select2-search__field
The code is as below (the events of opening and passing the keys to input is not working. It is because I could not find correct selectors :( )
    //Convert all selects
    $('select').each(function (index) {
        $(this).select2({
            minimumResultsForSearch: Infinity
        });

        if ($(this).closest("div").hasClass("select-editable")) {
            console.log('asd');
           //When user clicks on input open the select2
            $(this).next().next().click(function () {
                //Open the select2
                $(this).parent().prev().prev().select2("open");
            });
            //when user types pass the keys to select2
            $(this).next().next().keyup(function (e) {
                //Copy value from text filed into hidden search box and trigger it
                $('.select2-search__field').val($(this).val());
                $('.select2-search__field').trigger('keyup');
            });
        }

    })

My forms has lots of selects and I must find input and select 2 options dynamically.
fiddle: http://jsfiddle.net/zmf91sLf/

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


Can't parse 'extended' PIC format

Hi, for example:
text = """
       01  WORK-BOOK.
        10 TAX-RATE PIC S9(13)V9(2) SIGN LEADING SEPARATE.
        10  AMOUNT        PIC X(2).
        10  AMOUNT2       PIC S9(4)V9(2).
"""

gets parsed but:
text = """
       01  WORK-BOOK.
        10 TAX-RATE PIC S9(13)V9(2) SIGN LEADING SEPARATE.
        10  AMOUNT        PIC XX.
        10  AMOUNT2       PIC S9(4)V9(2).
"""

results:
Traceback (most recent call last):
File "XX\pyparsing\core.py", line 1134, in parse_string
raise exc.with_traceback(None)
pyparsing.exceptions.ParseException: Expected end of text, found '10'  (at char 90), (line:4, col:9)

X(2) should be supported. Can you post the full copybook snippet?
Regarding XX it is not supported. Will add.

X(2) is parsed correctly in first message.
What goes wrong is:
 02 LINE 1 COL 2 AMOUNT PIC X(2).

02 LINE 1 COL 2 AMOUNT PIC X(2).
This format is new to me. Can you open a new issue and elaborate what is the "LINE 1 COL 2" clause?

Can this be closed?

02 LINE 1 COL 2 AMOUNT PIC X(2).
This format is new to me. Can you open a new issue and elaborate what is the "LINE 1 COL 2" clause?

Didn't readed that. I'll close that and open a new one. Thanks.


feat: Issue Credentials V2 (W3C/JSON-LD) with Commits Squashed

Codecov Report

Merging #1092 (e94c554) into main (ef20f1e) will increase coverage by 0.06%.
The diff coverage is 84.32%.

@@            Coverage Diff             @@
##             main    #1092      +/-   ##
==========================================
+ Coverage   88.34%   88.40%   +0.06%
==========================================
  Files         701      706       +5
  Lines       16110    16341     +231
  Branches     2597     2640      +43
==========================================
+ Hits        14232    14447     +215
- Misses       1871     1887      +16
  Partials        7        7

Impacted Files
Coverage Δ

...ges/core/src/modules/credentials/CredentialsApi.ts
86.91% <ø> (ø)

...les/credentials/formats/CredentialFormatService.ts
100.00% <ø> (ø)

...ntials/formats/indy/IndyCredentialFormatService.ts
87.95% <0.00%> (+0.52%)
:arrow_up:

...s/protocol/v2/handlers/V2IssueCredentialHandler.ts
96.00% <ø> (ø)

...s/protocol/v2/handlers/V2OfferCredentialHandler.ts
67.74% <ø> (ø)

...protocol/v2/handlers/V2RequestCredentialHandler.ts
96.42% <0.00%> (ø)

...ckages/core/src/modules/dids/domain/DidDocument.ts
96.47% <ø> (ø)

...e/src/modules/proofs/formats/ProofFormatService.ts
100.00% <ø> (ø)

...credentials/formats/jsonld/JsonLdOptionsRFC0593.ts
57.14% <57.14%> (ø)

...ls/formats/jsonld/JsonLdCredentialFormatService.ts
85.88% <85.88%> (ø)

... and 17 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@NB-MikeRichardson seems like tests are still failing. Can't merge before those are resolved...

The failing test only seems to fail in 18.x. I have run this test on my local machine for all versions of node, and it runs fine except in 18.x. It is related to the Bbs signatures it seems.
How does one go about fixing this test?

@NB-MikeRichardson seems like tests are still failing. Can't merge before those are resolved...

It's the problem with BBS libraries. How can we avoid running them in node 18?

@NB-MikeRichardson seems like tests are still failing. Can't merge before those are resolved...

It's the problem with BBS libraries. How can we avoid running them in node 18?

Strangely enough I can get it running (sometimes) on my local machine. It seems to be inconsistent, do we know what the problem is with BBS libraries in 18.x?

@NB-MikeRichardson for all tests that include BBS code you should use describeSkipNode17And18 instead of describe at the top of the test file. This will skip the tests in Node 17 and 18


output csv format causes error importing into neo4j

while loading monarchkg_v2024-09-12/results/kg_nodes.csv into a neo4j db (version 5.24.2) i received this error:
// Import nodes from kg_nodes.csv in batches
             CALL {
                 LOAD CSV WITH HEADERS FROM 'file:///kg_nodes.csv' AS row
                 WITH row.id AS id, row.type AS type, apoc.convert.fromJsonMap(row.properties) AS properties
                 CREATE (n:Entity)
                 SET n.id = id,
                     n.type = type,
                     n += properties
             } IN TRANSACTIONS;
At /var/lib/neo4j/import/kg_nodes.csv @ position 26116004 -  there's a field starting with a quote and whereas it ends that quote there seems to be characters in that field after that ending quote. That isn't supported. This is what I read: '"f'

the offending quotes are in the line containing character position 26116004:
"Xenbase:XB-GENE-29084259","biolink:Gene","{""name"":""LOC100498474"",""xref"":""UniProtKB:A0A8J0T6Y3|NCBIGene:100498474|RefSeq:KAE8578778|Xenbase:XB-GENE-29084259|Xenbase:XB-GENEPAGE-29084258"",""in_taxon"":""NCBITaxon:8364"",""in_taxon_label"":""Xenopus tropicalis"",""provided_by"":""alliance_gene_nodes"",""synonym"":""LOC100498474|retinal cone rhodopsin-sensitive cGMP 3\"",5\""-cyclic phosphodiesterase subunit gamma"",""full_name"":""retinal cone rhodopsin-sensitive cGMP 3\"",5\""-cyclic phosphodiesterase subunit gamma"",""symbol"":""LOC100498474"",""type"":""SO:0000704""}"

there is a comma in the word 3,5-cyclic in the full name string ""full_name"":""retinal cone rhodopsin-sensitive cGMP 3"",5""-cyclic phosphodiesterase subunit gamma"" that is being read as a table delimiter because the code is erroneously  inserting an escape sequence to include the ,.
this appears to break the json syntax but it might just be a neo4j thing, i haven't tried to import the csv file anywhere else.

Thanks for isolating the problem! The escaped quote character before the comma indeed looks suspicious. I'll try to load the CSV file in dask and Neo4j to reproduce the issue. Could you please let me know which version you are using with neo4j --version or cypher-shell --version. The full import script would also help if you want to share it.

Check 1: Upstream from the CSV export, the entry in the SQLite database for Xenbase:XB-GENE-29084259 seems to be fine. The JSON string in the properties column is parsed correctly by DB Browser for SQLite. The escaped quote character is required to keep the value string intact.

Check 2: Python's csv module, which was also used to export the data in the first place, can read kg_nodes.csv without an issue and the suspicious entry seems also to be decoded correctly.

Check 3: LibreOffice Calc is able to load the relevant portion of the CSV file without a problem. The subset was generated with head kg_nodes.csv -n 70000 | tail -n 10000 > kg_nodes_subset.csv. This was necessary because the full CSV file contains too many lines for the program.

Preliminary conclusion: It seems to be a problem with Neo4j's CSV loader. The CSV export in kgw uses Python's built-in csv module with writer = csv.writer(f, dialect="excel", quoting=csv.QUOTE_ALL), which was suggested somewhere on the web to have high compatibility. Perhaps another parameterization would result in a difference in how the quotes are set in the file, potentially so that Neo4j would also be able to load it.

I was able to reproduce the error with a fresh installation of neo4j 4.4.38 with apoc from https://github.com/neo4j-contrib/neo4j-apoc-procedures/releases/download/<IP_ADDRESS>/apoc-<IP_ADDRESS>-core.jar.

Further observation: The slightly modified writer = csv.writer(f, dialect="excel", quoting=csv.QUOTE_ALL, escapechar='\\') results in a CSV file that Neo4j can load. When reading the file with Python's csv module, the raw data can be loaded as well, but the property column is different and using json.loads(nprop) on it leads to a decoding error due to having \\" instead of \"  within a quoted value, which leads the string to terminate too early.

I've tried a few other parameterizations of Python's csv module when exporting a CSV file and found one that seems compatible with Neo4j's parser, but unfortunately not with other programs such as LibreOffice or the csv module's default reader. Here's a simple converter script that takes a CSV file generated by kgw and converts it to the form that Neo4j seems to read accurately:
import csv
import os

def convert_csv(source, target):
    with open(source, 'r') as f1, open(target, 'w') as f2:
        reader = csv.reader(f1)
        writer = csv.writer(f2, dialect="excel", quoting=csv.QUOTE_ALL, doublequote=False, escapechar='\\')
        for row in reader:
            writer.writerow(row)

# Convert the CSV containing the nodes
convert_csv(
    os.path.join("monarchkg_v2024-09-12", "results", "kg_nodes.csv"),
    os.path.join("monarchkg_v2024-09-12", "results", "kg_nodes_for_neo4j.csv")
)

# Convert the CSV containing the edges
convert_csv(
    os.path.join("monarchkg_v2024-09-12", "results", "kg_edges.csv"),
    os.path.join("monarchkg_v2024-09-12", "results", "kg_edges_for_neo4j.csv")
)

I think the following is a proper way to load the nodes from the new CSV file in Neo4j, but couldn't figure out an efficient and correct way to load the edges yet:
CALL {
  LOAD CSV WITH HEADERS FROM 'file:///kg_nodes_for_neo4j.csv' AS row
  WITH row.id AS id,
       row.type AS type,
       apoc.convert.fromJsonMap(row.properties) AS properties
  CALL apoc.create.node([type], {id: id})
  YIELD node
  SET node += properties
} IN TRANSACTIONS;

Alternatively, one can load the GraphML file produced by kgw directly into Neo4j, but I don't know how to ensure that GraphML node types become Neo4j node labels:
CALL apoc.import.graphml("file:///var/lib/neo4j/import/kg.graphml", {readLabels: true, useTypes: true})


Fixed a bug when the URL doesn't have a path component

Added validation to check that the user passed in URL has a path component.
e.g. http://www.example.com/path
The path is needed to build the Git Repository on the appliance.
Links [Optional]

https://bugzilla.redhat.com/show_bug.cgi?id=1386254

Steps for Testing/QA
In the Automate -> Import/Export UI Screen for the URL try the following combinations

abc                                                                                should fail
http://www.example.com                                            should also fail
https://github.com/mkanoor/SimpleDomain              should work

@eclarizio
Please review

:+1: LGTM

Euwe BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1393023

Euwe Backport details:
$ git log -1
commit 85b508cbf642428317cc5e125f3398031baaf57c
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Fri Nov 11 08:41:18 2016 -0500

    Merge pull request #12566 from mkanoor/bugzilla_1393023_round2

    Fixed a bug when the URL doesn't have a path component
    (cherry picked from commit 365ff11285cca7c970567d71f9f30ec2881a20cc)

    https://bugzilla.redhat.com/show_bug.cgi?id=1393023


Node tableHead not parsed correctly

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions
<EMAIL_ADDRESS>Link to runnable example
No response
Steps to reproduce
The following code should produce a tableHead node:
fromMarkdown(text, {
        extensions: [gfm()],
        mdastExtensions: [gfmFromMarkdown()],
    });

| Column 1 | Column 2 | Column 3 |
| -------- | -------- | -------- |
| foo      | bar      | blob     |
| baz      | qux      | trust    |
| quux     | quuz     | glob     |

Expected behavior
fromMarkdown(text, {
        extensions: [gfm()],
        mdastExtensions: [gfmFromMarkdown()],
    });

### Actual behavior

TableRow is returned instead.
```json
{
    "type": "table",
    "align": [
        null,
        null,
        null
    ],
    "children": [
        {
            "type": "tableRow",
            "children": [
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "Column 1"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "Column 2"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "Column 3"
                        }
                    ]
                }
            ]
        },
        {
            "type": "tableRow",
            "children": [
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "foo"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "bar"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "blob"
                        }
                    ]
                }
            ]
        },
        {
            "type": "tableRow",
            "children": [
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "baz"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "qux"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "trust"
                        }
                    ]
                }
            ]
        },
        {
            "type": "tableRow",
            "children": [
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "quux"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "quuz"
                        }
                    ]
                },
                {
                    "type": "tableCell",
                    "children": [
                        {
                            "type": "text",
                            "value": "glob"
                        }
                    ]
                }
            ]
        }
    ]
}

Runtime
Node v16
Package manager
npm v7
OS
macOS
Build and bundle tools
Vite

Welcome @diervo! 👋
Sorry you ran into some confusion.
A few things:

Mdast is a markdown syntax tree, not HTML, there is no tableHead node type https://github.com/syntax-tree/mdast-util-gfm-table/tree/abf8c36f37e363194f994cc650a83ce1926c83f9?tab=readme-ov-file#nodes
This package does not make the syntax tree https://github.com/syntax-tree/mdast-util-gfm-table does

I suspect your question is an XY problem, you probably want to do something with a remark plugin?
If so consider opening a discussion with some context on what you are trying to solve https://github.com/orgs/remarkjs/discussions/categories/q-a

Hi @ChristianMurphy,
I understand a little bit about syntax trees, as I have worked quite a lot in the past with @wooorm while he was building micromark and the "formal specification" of markdown :)
The reason I filed the issue was mostly to get the team's take on whether having a new node of type tableHead to disambiguate between a regular tableRow and this new tableHead would be something that the team is sympathetic about.
Seems that micromark is already disambiguating and tokenizing it correctly, so seemed like a pretty straight-forward change in the AST generation
There are a lot of precedents of this disambiguation on similar "ast-like abstractions" (ex. ProseMirror), and it will allow for the user to configure whether or not they want to assume that the first row is a header or not.

Thanks for the clarification @diervo!
I'm open to the idea of a new node type.
That's a pretty significant change, which would touch a lot of repos.
I'd lean a bit towards it being an RFC https://github.com/unifiedjs/rfcs
To queue it up as a possibility for the next major release.

It will be hard for me to find the time, not necessarily for the RFC but to help push it through the whole process (I been in enough standardization comittes to know that it takes a village... xD)
But knowing that you are open to it, I will do my best to eventually find some time.
FWIW I think this change could be non-breaking if we default to the current behavior and add some options on top, but certainly it will require some non-trivial debate.

This change would go into mdast. And then be pulled in everywhere else.
There is indeed no tableHeader in mdast, which is intentional.
It existed years ago, and was removed: https://github.com/remarkjs/remark/commit/593bb82ada925f65a37cd7e54c5a8a1895b17813. The main discussion then happened here: https://github.com/syntax-tree/mdast/issues/6. These links are more for historical purposes, it’s years later and I’m sure I changed my mind on many things since. But some of that I still agree with.
I think ASTs should be simple. It should be hard to make mistakes in ASTs. With different names for these node types, it’s easier to accidentaly inject a tableHeader as the 2nd row. Or tableRow in the 1st. Or remove that 1st header row. What then?
But differentiating currently if needed, seems clear enough: index == 0.
Finally, as the ASTs and projects grow older, I think it’s sensical to be more stable.
What use cases does this really improve?


Evaluate Account Linking interface

We are evaluating the Webnative account linking interface and seeking feedback from developers.
Existing interface
Account linking in Webnative creates event emitters that listen for account linking events.
An accountProducer listens for a PIN from an accountConsumer and delegates the account when a user verifies the PINs match on both devices.
// The producer should already have an active session
const producer = await program.auth.accountProducer(program.session.username)

// The producer receives a challenge PIN from the consumer
producer.on("challenge", challenge => {
  // Either show `challenge.pin` or have the user input a PIN and check if they're equal.
  if (userInput === challenge.pin) challenge.confirmPin()
  else challenge.rejectPin()
})

// The producer reports whether a user approved or rejected
producer.on("link", ({ approved }) => {
  if (approved) console.log("Linked device successfully")
})

The accountConsumer listens for similar events:
// The consumer should not have an active session
const consumer = await program.auth.accountConsumer(username)

// The consumer generates a PIN and sends it to the producer
consumer.on("challenge", ({ pin }) => {
  // Display the PIN
  showPinOnUI(pin)
})

// The consumer receives an approval or rejection message from the producer
consumer.on("link", async ({ approved, username }) => {
  if (approved) {
    console.log(`Successfully authenticated as ${username}`)
    session = await program.auth.session()
  }
})

Alternative interface
An alternative interface could move the account linking to program.accountLinking and change the way it works.
A device with an existing session calls program.accountLinking.listen and listens for a PIN from a device that would like to be linked. It delegates the account when a user verifies the PINs match on both devices.
// On a device with an active session
program.accountLinking.listen()

// This device receives a PIN from the device that would like to be linked
program.accountLinking.on("challenge", (pin) => {
  // Either show `challenge.pin` or have the user input a PIN and check if they're equal.
  if (userInput === pin) program.accountLinking.confirmPin()
})

// Report whether the user approved or rejected
program.accountLinking.on("link", ({ approved }) => {
  if (approved) console.log("Linked device successfully")
})

A device that would like to be linked calls program.accountLinking.link and listens for similar messages.
// On a device without a session
program.accountLinking.link(username)

// This device generates a PIN and sends it to the device with an active session
program.accountLinking.on("challenge", (pin) => {
  // Display the PIN
  showPinInUI(pin)
})

// Handle an approval or rejection message from the device with an active session
program.accountLinking.on("link", ({ approved, username }) => {
  if (approved) {
    console.log(`Successfully authenticated as ${username}`)
    session = await program.auth.session()
  }
})

👍 I like this potential change because the method names seem a little more semantically accurate. I'm not thinking to myself "what is a consumer vs producer".


4: Identity dijit needs 3.x reference

I know this will be fixed soon but if you want styling for the identity manager to login, you'll need this stylesheet from 3.x:
<!DOCTYPE html>

  <meta charset="utf-8">

  <meta http-equiv="X-UA-Compatible" content="IE=edge">

  <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
  <meta name="mobile-web-app-capable" content="yes">
  <meta name="apple-mobile-web-app-capable" content="yes">
  <meta name="apple-mobile-web-app-status-bar-style" content="default">

  <link rel="stylesheet" href="//js.arcgis.com/4.0/esri/css/main.css">

  <link rel="stylesheet" href="css/boilerplate.css">

  <link rel="stylesheet" href="https://js.arcgis.com/3.17/esri/themes/calcite/dijit/calcite.css">

Maybe we should just use a dojo theme instead of mixing 3x and 4x. For now at least.
I know that calcite theme looks better :<

I'd hate to bring in a whole theme for just this. I think it's fine for now as 4.1 will have this fixed soon anyways. Do you know if there's a way to just grab the identity manager calcite css or is it al one big heap?

closing as this is being addressed in the API.


[Regarding URL rewriting]

Hi Tsepo, your tutorial is great, I just have this question here. Once you deploy your project, why could my project does not have any input fields? Have you used URL rewriting but have not documented inside your tutorial?

@brickfreelayer I did not use any URL rewriting in my tutorial. If you integrate JSF into your Spring Boot application, you lose the extensionless URLs as one would normally get with using Spring's controllers.
There's third party libraries that you can use to rewrite your URLs if you do not like the .xhtml extensions online. Personally, there isn't one I have used before.
Another options is, you can write a request dispatcher that can map your extensionless URLs to XHTML extensions.

Thanks for your response. I'm new on the integration and I think your tutorial here is a good "HelloWorld" one. This path is not a beaten one, so thanks for your help!


About train.py

Looking forward to your train.py file!Eager to learn how you trained this model!
Please upload it as soon as possible, please!

Hello, thank you for your message.
The team are investigating on some reproductibility issues. We aim to have the final code (including the train.py) + pre-print available by 14th of April.
Thanks for your patience.


[ui] waitForUrl() failure does not fail test if not matched

as reported here: https://stackoverflow.com/q/61528804/143475

Hi,
I created that question on Stackoverflow.
So, I followed your guide: https://github.com/intuit/karate/wiki/Developer-Guide
But I wasn't able to have successfull tests during build.

karate-core module -> is failing on ScriptTest.testBigDecimalsInJson:1442
[INFO] Running com.intuit.karate.ScriptTest
22:22:34.894 [main] DEBUG com.intuit.karate.ScriptTest - result: {"foo":"12345"}
22:22:34.981 [main] DEBUG com.intuit.karate.ScriptTest - expected net.minidev.json.parser.ParseException: Unexpected End Of File position 11: null
22:22:35.240 [main] DEBUG com.intuit.karate.ScriptTest - result: <foo bar="5">5</foo>
[ERROR] Tests run: 133, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.918 s <<< FAILURE! - in com.intuit.karate.ScriptTest
[ERROR] testBigDecimalsInJson(com.intuit.karate.ScriptTest)  Time elapsed: 0.006 s  <<< FAILURE! java.lang.AssertionError       at com.intuit.karate.ScriptTest.testBigDecimalsInJson(ScriptTest.java:1442)
karate-junit4 module -> is failing on syntax.feature:531
[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.26 s <<< FAILURE! - in com.intuit.karate.junit4.syntax.SyntaxTest
[ERROR] [1:6] syntax examples([syntax] karate syntax demo)  Time elapsed: 0.234 s  <<< ERROR!
com.intuit.karate.exception.KarateException: syntax.feature:531 - path: $.profit, actual: -1002.2000000000000, expected: -1002.2, reason: not equal (big decimal : primitive)

So, I skipped tests, but it seems that the problem with assertion is still there.

Hi,
I created that question on Stackoverflow.
So, I followed your guide: https://github.com/intuit/karate/wiki/Developer-Guide
But I wasn't able to have successfull tests during build.

karate-core module -> is failing on ScriptTest.testBigDecimalsInJson:1442
[INFO] Running com.intuit.karate.ScriptTest
22:22:34.894 [main] DEBUG com.intuit.karate.ScriptTest - result: {"foo":"12345"}
22:22:34.981 [main] DEBUG com.intuit.karate.ScriptTest - expected net.minidev.json.parser.ParseException: Unexpected End Of File position 11: null
22:22:35.240 [main] DEBUG com.intuit.karate.ScriptTest - result: <foo bar="5">5</foo>
[ERROR] Tests run: 133, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.918 s <<< FAILURE! - in com.intuit.karate.ScriptTest
[ERROR] testBigDecimalsInJson(com.intuit.karate.ScriptTest)  Time elapsed: 0.006 s  <<< FAILURE! java.lang.AssertionError       at com.intuit.karate.ScriptTest.testBigDecimalsInJson(ScriptTest.java:1442)
karate-junit4 module -> is failing on syntax.feature:531
[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.26 s <<< FAILURE! - in com.intuit.karate.junit4.syntax.SyntaxTest
[ERROR] [1:6] syntax examples([syntax] karate syntax demo)  Time elapsed: 0.234 s  <<< ERROR!
com.intuit.karate.exception.KarateException: syntax.feature:531 - path: $.profit, actual: -1002.2000000000000, expected: -1002.2, reason: not equal (big decimal : primitive)

Now the assertion is working as expected.

@marekmasek yes for some reason those tests depend on the OS / JDK version combo, they run fine on our CI. thanks for confirming

Hello Peter. I have the same problem in my project.
waitForUrl() test passed through waitForUrl() with incorrect url and went down to code
Example:
* waitForUrl('incorrect_url')
Log:
13:43:39.230 [nioEventLoopGroup-2-1] DEBUG c.intuit.karate.driver.DriverOptions - << {"method":"Runtime.executionContextCreated","params":{"context":{"id":4,"origin":"https://cdn.rutarget.ru","name":"","auxData":{"isDefault":true,"type":"default","frameId":"3DC1D1C787EF3E360FE0A38E6A9031CF"}}}}
13:43:39.850 [nioEventLoopGroup-2-1] DEBUG c.intuit.karate.driver.DriverOptions - << {"method":"Page.frameStoppedLoading","params":{"frameId":"3DC1D1C787EF3E360FE0A38E6A9031CF"}}
13:43:39.871 [ForkJoinPool-1-worker-1] DEBUG c.intuit.karate.driver.DriverOptions - waitForUrl - retry #1
13:43:40.366 [nioEventLoopGroup-2-1] DEBUG c.intuit.karate.driver.DriverOptions - << {"method":"Page.loadEventFired","params":{"timestamp":16963.198358}}
13:43:40.370 [nioEventLoopGroup-2-1] DEBUG c.intuit.karate.driver.DriverOptions - << {"method":"Page.frameStoppedLoading","params":{"frameId":"F07DAE1C48C85F0E050AD8DAC958EC7C"}}
13:43:42.872 [ForkJoinPool-1-worker-1] DEBUG c.intuit.karate.driver.DriverOptions - waitForUrl - retry #2
13:43:45.873 [ForkJoinPool-1-worker-1] DEBUG c.intuit.karate.driver.DriverOptions - waitForUrl - retry #3
13:43:48.874 [ForkJoinPool-1-worker-1] WARN  c.intuit.karate.driver.DriverOptions - failed after 3 retries and 9003 milliseconds

@RakhmedovRS have you read the comments above ? it will be more helpful. if you can follow this process and confirm this is fixed: https://github.com/intuit/karate/wiki/Developer-Guide

Yes I read it. I forgot to add information to my comment. I run my project on macOS Catalina v.10.15.4 and I don't have such problems. But my teammates who use MS Window have problems.
From you comment I see that I need to download your project and check is everything works fine or not.
What additional information can I provide to help to fix this bug?

@RakhmedovRS yes if you can follow the developer guide, you can confirm if it is fixed or not. else we will release an RC2 version next week so you can wait for that.

I walked through your instruction. And during running one test failed.
log:
failed features:
com.intuit.karate.multi-scenario-fail: multi-scenario-fail.feature:7 - assert evaluated to false: a != 1
com.intuit.karate.no-scenario-name: [3.3:19] no-scenario-name.feature:13 - assert evaluated to false: 3 != 3
[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.771 s - in com.intuit.karate.RunnerTest
[INFO] Running com.intuit.karate.ConfigTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.002 s - in com.intuit.karate.ConfigTest
[INFO] Running com.intuit.karate.XmlUtilsTest
16:37:56.575 [main] DEBUG com.intuit.karate.XmlUtilsTest - map: {env:Envelope={={env:Header=null, env:Body={={QueryUsageBalanceResponse={_={Balance=null, Result={Success=null, Error={Category=DAT, Code=DAT_USAGE_1003, Description=Invalid Request: Invalid Input criteria: No asset found for license/eoc<PHONE_NUMBER>71198/855939)., Source=SIEBEL}}}, @={xmlns=http://www.intuit.com/iep/ServiceUsage/IntuitServiceUsageABO/V1}}}, @={xmlns=http://www.intuit.com/iep/ServiceUsage/IntuitServiceUsageABO/V1}}}, @={xmlns:S=http://schemas.xmlsoap.org/soap/envelope/, xmlns:env=http://schemas.xmlsoap.org/soap/envelope/}}}
[ERROR] Tests run: 22, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.044 s <<< FAILURE! - in com.intuit.karate.XmlUtilsTest
org.junit.ComparisonFailure:
expected:<...goo>

[]> but was:<...goo>

[
]>
at com.intuit.karate.XmlUtilsTest.testPrettyPrint(XmlUtilsTest.java:226)
[INFO] Running com.intuit.karate.http.HttpUtilsTest
[INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.003 s - in com.intuit.karate.http.HttpUtilsTest
[INFO] Running com.intuit.karate.http.HttpRequestBuilderTest
[INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 s - in com.intuit.karate.http.HttpRequestBuilderTest
[INFO] Running com.intuit.karate.http.HttpClientTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.002 s - in com.intuit.karate.http.HttpClientTest
[INFO] Running com.intuit.karate.ScriptTest
16:37:56.777 [main] DEBUG com.intuit.karate.ScriptTest - result: {"foo":"12345"}
16:37:56.826 [main] DEBUG com.intuit.karate.ScriptTest - expected net.minidev.json.parser.ParseException: Unexpected End Of File position 11: null
16:37:57.032 [main] DEBUG com.intuit.karate.ScriptTest - result: 5
[INFO] Tests run: 133, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.749 s - in com.intuit.karate.ScriptTest
[INFO] Running com.intuit.karate.JsonTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 s - in com.intuit.karate.JsonTest
[INFO] Running com.intuit.karate.ScriptValueTest
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 s - in com.intuit.karate.ScriptValueTest
[INFO] Running com.intuit.karate.RunnerOptionsTest
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.011 s - in com.intuit.karate.RunnerOptionsTest
[INFO]
[INFO] Results:
[INFO]
[ERROR] Failures:

[]> but was:<...goo>

[
]>
[INFO]
[ERROR] Tests run: 299, Failures: 1, Errors: 0, Skipped: 0
[INFO]

@ptrthomas all tests passed, everything is ok.
macOS Catalina v.10.15.4
java version "1.8.0_201"
Java(TM) SE Runtime Environment (build 1.8.0_201-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.201-b09, mixed mode)

@RakhmedovRS thanks for confirming that waitForUrl is now working fine with the latest code

@ptrthomas could you tell me when you'll release 0.9.7?

@RakhmedovRS no. you'll get 0.9.6.RC3 next week

I run my application several time with 0.9.6.RC3.
Now everything works well.

@RakhmedovRS thank you for confirming 👍

@ptrthomas Hello again. During 2 weeks of working with 0.9.6.RC3 I noticed performance reduction. Has somebody reported about such problems with 0.9.6.RC3?

@RakhmedovRS no

@RakhmedovRS if this is really an issue, please follow this process so we can replicate: https://github.com/intuit/karate/tree/develop/examples/ui-test

0.9.6 released


Don't mask out build env vars for cross-ruby

This was introduced in commit a7c113e580c1565000166f9967da82ff2ecdaeaa to avoid interference with host build. However it prohibits setting important flags for cross-build.
Also don't force 'LDFLAGS=-pipe -s' in configure, which doesn't work with clang compiler targeting MacOS. It can now be set externally.
Fixes #178

OK.
Users should use rake-compiler carefully.


Issue with facebook

Hello,
I have a issue to connect facebook channel to the bot.
System Information

SDK Language: .Net
SDK Version:  v3.9.0
Development Environment:  Visual Studio 2015 ; deployed on azure as a web application

Bot Info

Bot name: Mon Chere Bot
Bot handle: cherebot
App ID: 0c7e0624-1318-4015-afc4-ffd244ca9dae
Connected channels: Skype, Facebook

Issue Description
I have a issue to connect facebook channel.
First, the bot is working, I can speak to it through the botframework-emulator, the web chat and skype
Second, I already test a bot whitch was generated automatically with "bot service" from Azure and I succeed connecting it to facebook
So, very strange, I try to debug with Visual Studio, and when it tries to send a simple answer (see code below), it raises an exception : System.UnauthorizedAccessException
message :
Authorization for Microsoft App ID 0c7e0624-1318-4015-afc4-ffd244ca9dae failed with status code Forbidden and reason phrase 'Forbidden'
stacktrace :

Authorization for Microsoft App ID 0c7e0624-1318-4015-afc4-ffd244ca9dae failed with status code Forbidden and reason phrase 'Forbidden'    at Microsoft.Bot.Connector.JwtTokenRefresher.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Microsoft.Bot.Connector.Conversations.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Bot.Connector.ConversationsExtensions.d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Bot_Application1.MessagesController.d__1.MoveNext()
Code Example
(simple answer code)
ConnectorClient connector = new ConnectorClient(new Uri(activity.ServiceUrl));
Activity reply = activity.CreateReply(reponse);
await connector.Conversations.ReplyToActivityAsync(reply);
and I added in the code before :
Microsoft.Bot.Connector.MicrosoftAppCredentials.TrustServiceUrl(@"https://facebook.botframework.com", DateTime.MaxValue);
Steps to Reproduce the Issue

speak to the bot

Expected Behavior
working
Actual Results
Throw an exception

I recreate everything and now it's working...


return(foo) => return foo

I edited all files in seed/challenges with the regex s/return(\(.\{-}\));/return \1;/g, which effectively removes the parentheses around the returned value, replacing the first with a space.
I read the git diff and everything looks fine, but I didn't test it all, because that would be taking masochism to a whole other level. So, it has my seal of approval, which, fair warning, has failed in the past; do with this as you please. :mag: :smoking:
Fix #2784

Nice Work as always @ahstro!


have code for decision code but the while loop is not working on server.js line 73 .

Description
Please include a summary of the change and a link to which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change status

[ x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x ] Test A the do while loop on server js is not being invoked or running correctly. spent a long time
trying to get it to work. Jamie helped a bit as well. please can someone else (Sneha maybe) have
a look
[ ] Test B

Checklist:

[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

this is all done and tested that it's working . also added auth to some of the routes to make sure they are logged in


Create Technical Documentation for Anatomize.py

We should include an overview of, step-by-step, how Anatomizer.py works. Including:

loading metadata (parser, templates, etc)
polling logs
post-processing

Completed:

https://github.com/magneticstain/Inquisition/wiki/Installing-Inquisition-Suite
https://github.com/magneticstain/Inquisition/wiki/Inquisition-User-Guide
https://github.com/magneticstain/Inquisition/wiki/Anatomize.py-Technical-Manual


Add test for end of line comments on module include lines

Merging because low risk, just a test case.


Add space to AM/PM string when we define twelve hours true

Now we have: 06:00AM, and i suggest add a space: 06:00 AM.

You can just add it directly in the translate function from the options.


how to prevent app launching if user copies url from copy share url(from instagram)

i want to let user to download in background with notification
current code takes the user to the app and  over there asks to download or share link ..

Thanks @altaf2892 I'll try to implement it, who knows can :)


Error fires on connect every time

Socket.EVENT_CONNECT_ERROR fires on every single connection attempt, regardless of whether the attempt was successful.
The following is the stacktrace produced:
Quobject.EngineIoClientDotNet.Client.EngineIOException: xhr poll error ---> System.AggregateException: One or more errors occurred. ---> System.ArgumentNullException: Value cannot be null.
Parameter name: s
at System.IO.StringReader..ctor(String s)
at Newtonsoft.Json.Linq.JObject.Parse(String json, JsonLoadSettings settings)
at Quobject.EngineIoClientDotNet.Client.Socket.OnPacket(Packet packet)
at Quobject.EngineIoClientDotNet.ComponentEmitter.Emitter.Emit(String eventString, Object[] args)
at Quobject.EngineIoClientDotNet.Client.Transports.Polling.DecodePayloadCallback.Call(Packet packet, Int32 index, Int32 total)
at Quobject.EngineIoClientDotNet.Parser.Packet.DecodePayload(String data, IDecodePayloadCallback callback)
at Quobject.EngineIoClientDotNet.Client.Transports.Polling._onData(Object data)
at Quobject.EngineIoClientDotNet.ComponentEmitter.Emitter.Emit(String eventString, Object[] args)
at Quobject.EngineIoClientDotNet.Client.Transports.PollingXHR.XHRRequest.OnData(String data)
at Quobject.EngineIoClientDotNet.Client.Transports.PollingXHR.XHRRequest.b__7_0()
at System.Threading.Tasks.Task.Execute()
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at Quobject.EngineIoClientDotNet.Client.Transports.PollingXHR.XHRRequest.Create()
--- End of inner exception stack trace ---

I have the same bug, I don't know if this issues are correlated but someone with the Java Socket.io client is having the same issue Here.
Tried to force Upgrade to true but nothing seems to change.
@SeanLatimer Do you find a workaround or a fix?


NPM release GitHub action

I recommend https://github.com/changesets/changesets, we (The Guild) use it a lot, it comes with a changelog generation and new releases are done by merging a Pull Request (created by changesets).
Example pull request with a patch update:
https://github.com/kamilkisiela/graphql-hive/pull/399
Example pull request that releases a new version:
https://github.com/kamilkisiela/graphql-hive/pull/400

@kamilkisiela thanks for the suggestion!
This is definitely a good idea to use such a workflow if (when) we will have a higher volume of merged PRs over time.
For now, I was thinking of something as simple as NPM publish triggered by tag creation with some manual changelog which should be not hard to compose.

I've added a simple NPM publish workflow.


use webpack-dev-server 2.7.1 to fix npm start

https://github.com/webpack/webpack-dev-server/issues/1101#issue-258321001

I was running into this issue across several projects; thanks for the fix!


nixos/plantuml-server: fix

Description of changes
PlantUML Server was failing since Jetty 12.x upgrade.
Indeed, PlantUML Server does not support Jetty versions higher than 12.x: it has introduced major breaking changes, see

https://github.com/jetty/jetty.project/releases/tag/jetty-12.0.0
https://eclipse.dev/jetty/documentation/jetty-12/programming-guide/index.html#pg-migration-11-to-12

I added a NixOS test to be notified sooner if this NixOS module breaks again.
Also done:

Added jetty_11, jetty_12
Aliased jetty to jetty_12
Removed with lib; notation from plantuml-server module
Removed allowPlantumlInclude option (ALLOW_PLANTUML_INCLUDE has been removed upstream)
Updated jetty_12 to 12.0.3
Added hardening options to plantuml-server SystemD service
Added myself as maintainer

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[x] Fits CONTRIBUTING.md.

Since 11.x.nix and 12.x.nix are largely identical, I think it's worth factoring out the common code:
# 11.x.nix
{ lib, stdenv, fetchurl }:

import ./common.nix {
    inherit lib stdenv fetchurl;
    version = "11.0.18";
    src = ...;
}

# common.nix

{ lib, stdenv, fetchurl, version, src }:

stdenv.mkDerivation rec {
    inherit version src;
    ...
}

Hello @emmanuelrosa, thanks for the review!
Indeed, I factored the common code in the 11.x.nix and 12.x.nix files into a common.nix file.

Super cool refactoring!


add --host-port for serve

add host_port arg to the serve_backend

$ ilab model serve --help
  --host-port TEXT            Host and port to serve on. [default:
                              <IP_ADDRESS>:8000; config: 'serve.host_port']

Seems some requests 2604 1823  for this arg, so update it. And sorry for the original one 1446 that missed the previous git env. please check this one, thanks.
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Functional tests have been added, if necessary.
[ ] E2E Workflow tests have been added, if necessary.

How about the simpler --bind?

How about the simpler --bind?

Agreed, or --listen.  When I read --host-port I assumed the argument was a port number, not a host:port specification.
Is the port number required or is there a default?  If there's a default, then --bind or --listen makes more sense.

@dcantrell there is a default: https://github.com/instructlab/instructlab/blob/main/src/instructlab/configuration.py#L224, it's "<IP_ADDRESS>:8000"

OK, that's good.  So thinking about it even more, I would say --listen or --listen-on is the more friendly way to name the option.  --bind assumes the user has knowledge of the BSD sockets API.  And while many of us do, it's kind of jargon specific to the sockets API.

I am ok with --listen, if more people agreed, I would like to change. hi team, please feel free to share.

--listen is fine with me :+1:

It would be good to follow precedent on the arg name.  Are there good examples of using --listen for this use case out in the wild?
My first reaction to --listen is it sounds like a verb rather than an argument accepting host:port.  --listen-on doesn't have the verb issue.  But again, would be good to follow precendent.

If we go with listen do we change the name of the underlying configuration variable host_port too? With the way the PR is written right now, the the default value for host_port in serve() is the value of ctx.obj.config.serve.host_port.
After a quick glance at the code changes in src/clickext.py would need to be made for --listen to take the value of _serve.host_port by default.

If we change the config we would want to make it backward compatible and still support the old param.
My vote would be:

Accept the config name of host_port as precedent (the name isn't ideal but it's workable)
Be consistent with the name in the arg and use --host-port

Because:

The confusion created over a rename and the maintenance of the backwards compatible support doesn't seem worth it
unless we can point to a well formed precedent on a different naming choice

From some quick searching, the most common methodology seems to be to separate --host and --port into two different params.  Sometimes the port param is called host_port which is what makes our current option a little confusing (at least for me).  The help text, docs, and examples mitigate that enough for me to not feel like it's a big issue.

Having host and port split apart is a convention I've seen used before too. What would the backwards compatibility look like if we remove host_port from the config and split it apart in the config and had the flags discussed here?

Sorry, based on the suggestion（good examples of using） again, do some research
gunicorn --bind <IP_ADDRESS>:8000 myapp:app
uvicorn app:app --host <IP_ADDRESS> --port 8000
flask run --host=<IP_ADDRESS> --port=8000

Seems more common for --host and --port, actually, my first time is used --host and --port, changed to host-port due to easy to get the config from the suggestion.
Also, from the code, looks like separate , together for the operation, e.g. get the config host_port, and use function to separate it. If used separate options, maybe easier.

lol, if really separate host_port, seems more to change..

pick one?
ilab model serve --host-port <IP_ADDRESS>:8000

ilab model serve --host <IP_ADDRESS> --port 8000

ilab model serve --bind <IP_ADDRESS>:8000

ilab model serve --listen <IP_ADDRESS>:8000

ilab model serve --listen-on <IP_ADDRESS>:8000

hi, I updated with --host and --port

@alimaredia could you please take a look?


Email to DataPalooza Registrants

To be sent via Eventbrite with logistics information

scheduled Wednesday afternoon message to be sent out at 6:00 pm

scheduled Friday morning message to be sent out at 9:00 am