[SPARK-12443][SQL] encoderFor should support Decimal

JIRA: https://issues.apache.org/jira/browse/SPARK-12443
constructorFor will call dataTypeFor to determine if a type is ObjectType or not. If there is not case for Decimal, it will be recognized as ObjectType and causes the bug.

Test build #48073 has started for PR 10399 at commit 6afb1a4.

Test build #48073 has finished for PR 10399 at commit 6afb1a4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48073/
Test PASSed.

Merged build finished. Test PASSed.

cc @cloud-fan @marmbrus @davies

Test build #48102 has started for PR 10399 at commit 2875a5f.

LGTM, pending tests.

Test build #48102 has finished for PR 10399 at commit 2875a5f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48102/
Test PASSed.

Merged build finished. Test PASSed.

Decimal is a private class.  We should not expose it to users until we have audited the API.

Unfortunately, Decimal is not private and we may leak it to users already. cc @rxin

@rxin Could you check this? Thanks.

ping @rxin

Test build #49110 has started for PR 10399 at commit ed90585.

Jenkins, retest this please.

Test build #49123 has started for PR 10399 at commit ed90585.

Test build #49110 has finished for PR 10399 at commit ed90585.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49110/
Test FAILed.

Merged build finished. Test FAILed.

Test build #49123 has finished for PR 10399 at commit ed90585.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49123/
Test FAILed.

retest this please.

Test build #49145 has started for PR 10399 at commit ed90585.

Test build #49145 has finished for PR 10399 at commit ed90585.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49145/
Test PASSed.

@rxin @JoshRosen Please see if this patch is ok for you. Thanks.

ping @rxin @JoshRosen

I don't think we should make it easier to use decimal until we review its API.  It was intended to be internal.

Let me close this.

Can you re-open this and bring it up-to-date?

@rxin ok.

Test build #53733 has started for PR 10399 at commit c29c04d.

Test build #53733 has finished for PR 10399 at commit c29c04d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53733/
Test PASSed.

@rxin tests passed. Anymore change I should make to this?

Should we also handle Decimal inside external row? i.e. improve RowEncoder.extractorsFor to accept Decimal also.

Yea that's a good idea.

@cloud-fan @rxin Yeah, but when we do construction, we will generate a Java BigDecimal back to external row. Because constructorFor is binded DecimalType to that external type.

That's fine. We just want to support the inbound.

Because externalDataTypeFor is already binded DecimalType to java.math.BigDecimal. Changing RowEncoder.extractorsFor can't make RowEncoder support Decimal in external row.
In order to support this, we need to change Invoke.
@rxin @cloud-fan Please review this and let me know if it is good. Thanks!

Test build #53760 has started for PR 10399 at commit 5e41f6d.

Maybe I missed something, can't we simply add a Decimal.apply(d: Decimal): Decimal = d to fix it?

I did originally have the same idea. However, it can't. Because for each field in a external row, Invoke(inputObject, method, externalDataTypeFor(f.dataType) will be called to get the value from row and use correct type. Here externalDataTypeFor will return ObjectType(classOf[java.math.BigDecimal])  for DecimalType.  We can't bind DecimalType to two different types. In other words, for each data type, we assume the corresponding value is in specified object type.

Test build #53760 has finished for PR 10399 at commit 5e41f6d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53760/
Test FAILed.

Merged build finished. Test FAILed.

We can't bind DecimalType to two different types
Agree, but we can work around it by using Object as the external type for DecimalType, and add comments to explain it can be java decimal or Decimal. Then we just need to add Decimal.apply(v: Any), what do you think?

@cloud-fan Sounds reasonable. Let me try it. Thanks.

Test build #53772 has started for PR 10399 at commit e1ac393.

Test build #53772 has finished for PR 10399 at commit e1ac393.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53772/
Test FAILed.

Test build #53779 has started for PR 10399 at commit e4a153c.

Test build #53779 has finished for PR 10399 at commit e4a153c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53779/
Test PASSed.

Test build #53844 has started for PR 10399 at commit 9781676.

LGTM except one document comment, thanks for working on it!

Test build #53859 has started for PR 10399 at commit de22b50.

LGTM pending tests

@cloud-fan @marmbrus @rxin Thanks for reviewing this!

Test build #53859 has finished for PR 10399 at commit de22b50.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53859/
Test FAILed.

Merged build finished. Test FAILed.

Test build #53844 has finished for PR 10399 at commit 9781676.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53844/
Test FAILed.

the test is passed locally.

retest this please.

Test build #53885 has started for PR 10399 at commit de22b50.

Test build #53885 has finished for PR 10399 at commit de22b50.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53885/
Test PASSed.

cc @marmbrus for final sign off.

ping @marmbrus please let me know if this is ok for you now, thanks.

Test build #54128 has started for PR 10399 at commit ea36aeb.

Test build #54128 has finished for PR 10399 at commit ea36aeb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54128/
Test PASSed.

Thanks, merging to master.


3.5: StreamPeerTCP silently drops data passed to put_data if the connection has not yet occurred

Godot version
3.5.stable
System information
Issue was found with a Windows 10 system, unknown details. Test script was tested on Ubuntu 22.04.1 LTS.
Issue description
I encountered request freezing in a project using a custom TCP-based protocol.
Once I worked out where the freezing was occurring, I decided to double-check the Godot code.
I can't make a "full" reproduction project because such a project would need to introduce a network delay of some sort somehow but I have given a script which uses example.com as a source of network delay.
The logic problem is at https://github.com/godotengine/godot/blob/3.5/core/io/stream_peer_tcp.cpp#L102 - presently https://github.com/godotengine/godot/blob/a627cdafc53bd9d986471fcaceac7e51213a7a68/core/io/stream_peer_tcp.cpp#L102
Along with at https://github.com/godotengine/godot/blob/a627cdafc53bd9d986471fcaceac7e51213a7a68/core/io/stream_peer_tcp.cpp#L271
If the status is STATUS_CONNECTING and remains STATUS_CONNECTING when p_block is true, a silent discard of data will occur - write reports this as OK, 0 bytes written, but put_data does not check for this.
Steps to reproduce
The below script will attempt to connect to example.com and send an HTTP request.
Importantly, it does this via put_data without confirming the connection has occurred yet, which causes silent discard.
The program then thus freezes.
extends Label

func _ready():
	var client = StreamPeerTCP.new()
	_log("doing listen")
	# this is assumed to take more time than it takes to run the code
	_check(client.connect_to_host("example.com", 80))
	var status = client.get_status()
	_log("client state: " + str(status) + " expected " + str(StreamPeerTCP.STATUS_CONNECTING))
	if status != StreamPeerTCP.STATUS_CONNECTING:
		_log("-- SYSTEM CONFIGURATION NOT WITHIN EXPECTED PARAMETERS FOR TEST, BUG WILL NOT TRIGGER --")
	# make some bytes so we can lose them in a tragic accident
	var msg = "GET / HTTP/1.1\r\nHost: example.com\r\n\r\n"
	var msg_b = msg.to_utf8()
	_log("sending data")
	_check(client.put_data(msg_b))
	_log("intentionally failing client data get to avoid Nagle/etc. shenanigans")
	client.get_partial_data(1) # intentionally fail this
	_log("retrieving data")
	# now that the data has suitably been lost:
	# try to get all data until client close
	var total = PoolByteArray()
	while true:
		var dat = client.get_partial_data(1)
		var err = dat[0]
		if err != OK:
			break
		total.append_array(dat[1])
	_log("got " + str(len(total)) + " bytes, text: " + total.get_string_from_ascii())
	client.disconnect_from_host()

func _log(txt: String):
	print(txt)
	text += txt + "\n"

func _check(err: int):
	_log("> " + str(err))

Minimal reproduction project
No response

oddly, the master branch do not have these lines
https://github.com/godotengine/godot/blob/200b9cde88da485659a6376266af8e9bf7be186f/core/io/stream_peer_tcp.cpp#L141-L147
while a git blame points the exact same commit in both branches
https://github.com/godotengine/godot/blame/200b9cde88da485659a6376266af8e9bf7be186f/core/io/stream_peer_tcp.cpp#L144
https://github.com/godotengine/godot/blame/a627cdafc53bd9d986471fcaceac7e51213a7a68/core/io/stream_peer_tcp.cpp#L102


[e2e] Add a persistent-seeder script

and associated SC_ENV, that successfully runs in a Docker container.

Progress toward #1548 .
Instuctions

cd packages/e2e-tests
docker build -t dapp .
docker run -it dapp /bin/bash to get an interactive shell inside the Docker container
yarn persistent-seeder to launch web3torrent & dappeteer , upload a file and make a deposit

TODO

[x] cleanup
[ ] deployment (separate PR)
[ ] use this as launchpad for enabling dappeteer in CI (separate PR)

Could you add notes to the production dashboard for this too?

We'll also likely want to set up automated deploying inside CircleCI


[SPARK-7601][SQL] Support Insert into JDBC Datasource

Supported InsertableRelation for JDBC Datasource JDBCRelation.
Example usage:
sqlContext.sql(
s"""
|CREATE TEMPORARY TABLE testram1
|USING org.apache.spark.sql.jdbc
|OPTIONS (url '$url', dbtable 'testram1', user 'xx', password 'xx', driver 'com.h2.Driver')
""".stripMargin.replaceAll("\n", " "))
sqlContext.sql("insert into table testram1 select * from testsrc")
sqlContext.sql("insert overwrite table testram1 select * from testsrc")

Can one of the admins verify this patch?

Jenkins, test this please.

Merged build triggered.

Merged build started.

Test build #32641 has started for   PR 6121 at commit f3fb5f1.

Test build #32641 has finished for   PR 6121 at commit f3fb5f1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32641/
Test PASSed.

Ah, I forgot we never linked these together.  Thanks for doing this!  Merged to master and 1.4


Feat/ruby

adds dockerfile and entrypoints for ruby

This has been superseded by the chore/ruby_cleanup to pull in your changes and integrate with the testing setup


[Docs]: Update Documentation References from PNPM to NPM Migration

what's wrong in the documentation?
We recently migrated our project from PNPM to NPM as the package manager. As a result, various references to PNPM need to be updated across our documentation files, including the README and CONTRIBUTING files.
Add screenshots
No response
Code of Conduct

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

Hi @shyamtawli, I will update this file. Can you assign this task to me?


Platformio-atom-ide-terminal is not working properly

Hi, the platformio-atom-ide-terminal is not working. I wrote python and it was saying:
python : The term 'python' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

python

  + CategoryInfo          : ObjectNotFound: (python:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

Read https://github.com/platformio/platformio-atom-ide-terminal/issues/543
If uninstalling and reinstalling does fix it, oh well.

@ivankravets please close the OP is no longer existing


nr_diagram

Unable to run nr.diagram with latest release 0.13.
[root@salt-master-3004 /]# salt nrp1 nr.diagram
nrp1:
'nr.diagram' is not available.
ERROR: Minions returned with non-zero exit code
[root@salt-master-3004 /]# salt-run nr.diagram L2 v3d
'nr.diagram' is not available.

Hello, please refer to sample usage section on how to call it - https://salt-nornir.readthedocs.io/en/latest/Nornir Runner Module.html#nr-diagram - in essence, need to use salt-run not salt app to call it as nr.diagram is a runner module not execution module.

Let me know if any further help required, closing for now.


ARROW-4651: [Flight] Use URIs instead of host/port pair

I haven't changed the client/server construction interfaces to really take advantage of this. I would rather follow up with creating builders instead, as proposed, as that would be cleaner than special-casing the logic in the current constructors.

Does the change in format/ need to be voted on the mailing list?

Hmm, I suppose we should, even if Flight is unstable for now.

Does the change in format/ need to be voted on the mailing list?

I think it should be. Specifically, what is the URI format support for now. That was my main ask in general on this ticket: what do we want to support in the URI? Let's define it specifically.

Ok, I will put up a proposal on the mailing list. Thanks for the comments!

The protocol change proposal was formally accepted on the Arrow-dev mailing list. Now this PR needs to be rebased (and conflicts fixed) before it gets reviewed. @lihalite, do you have time for this? Otherwise, I can take up.

@pitrou sorry about that, I will rebase & clean up things later today (just got back from vacation).

I've now rebased this.

Codecov Report

Merging #4047 into master will increase coverage by 0.15%.
The diff coverage is 90.62%.

@@            Coverage Diff             @@
##           master    #4047      +/-   ##
==========================================
+ Coverage   88.03%   88.19%   +0.15%
==========================================
  Files         758      637     -121
  Lines       92803    70246   -22557
  Branches     1251     1252       +1
==========================================
- Hits        81701    61952   -19749
+ Misses      10989     8173    -2816
- Partials      113      121       +8

Impacted Files
Coverage Δ

cpp/src/arrow/flight/flight-test.cc
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/util/uri.cc
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/flight/types.h
100% <100%> (ø)
:arrow_up:

cpp/src/arrow/flight/server.cc
67.33% <100%> (-17.25%)
:arrow_down:

cpp/src/arrow/flight/test-util.cc
82.08% <100%> (-5.22%)
:arrow_down:

cpp/src/arrow/flight/internal.cc
79.82% <100%> (-10.78%)
:arrow_down:

cpp/src/arrow/flight/client.cc
62.63% <68.75%> (-29.96%)
:arrow_down:

cpp/src/arrow/flight/types.cc
88.09% <93.33%> (-1.56%)
:arrow_down:

go/arrow/math/int64_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_avx2_amd64.go
0% <0%> (-100%)
:arrow_down:

... and 278 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 e32c0b3...e97e577. Read the comment docs.

I intended to change the C++ server API. I think I rebased out the change on accident as I was also trying to implement the builder APIs at the same time, but now I'd rather do that in a follow up PR.
Thanks for the review, I'll get to fixing these issues!

While I'm at it, might as well add the builder APIs here, and make TLS-enabled services possible. In C++/Python, I did not go all the way to a Java-style builder - the changes there are much more invasive, and I don't think it's worth it until we fully settle on supporting another transport. (And even then, I think it could be done within the current APIs, or with minimal changes to them.)

Rebased, tests pass.

Thanks for the feedback! I've updated things.
gRPC supports binding to port 0 for a free port, we just need a way to report that back to the API user.

Does this need reviewing on the Java side?

Thanks @pitrou. For the Java side, perhaps @jacques-n could take a look again?

Updated to fix an inadvertent API breakage. (Ticket is quite useless if you can't read its value...)

There are actually lots of little things I'm noticing here now that I'm trying to test internally, so please hold off while I fix things over the next couple days...apologies for the trouble.

Ok, this should be ready now; fixed some inadvertent API breakages.

It looks like this is near merge-readiness. @jacques-n can you review/sign off on the Java changes?

Rebased with master as well.

I think Jacques is traveling right now so it may be a little time before we can get a go-ahead from the Java side. Would my review on the C++/Python side be helpful?

I'd appreciate any feedback! On the Java side, I'm not in a particular rush to get this merged, and I can keep up with master, I'd just like to make sure I get everything in for 0.14.

Cool, BTW I'm guessing on a somewhat longer release timeline than usual for 0.14 to give various in-flight efforts to sort themselves out, e.g. toward end of June / beginning July

Please make sure to run rebase any java CLs and re-run CI to make sure javadoc's are in place.

@jacques-n I think you are the last approver on this PR

Looks good to me. Thanks for pulling this together @lihalite!

thanks @lihalite! It might have already been discussed here, but what is the testing strategy for TLS-enabled Flight going to be? Unless I missed it is doesn't seem that this is tested now, can we open a JIRA?

It's not currently tested, JIRA: https://jira.apache.org/jira/browse/ARROW-5397
We'll need some way to generate certs/keys.

The easiest thing to do is to store self-signed certs and the corresponding private key in the repo.
Preferable along with the script to regenerate them when the expiry date is reached.


New UI/UX DESIGN needed MAJOR UI/UX changes required and missing images

Describe the bug
UI is BAD and not aligned properly and missing images
Expected behavior
UI is not good at all major changes required
Screenshots

in above image image is missing
Example  Fill x { [x] }  into  bracket

[x ] I have checked the existing issues *
[ x] I have read the Contributing Guidelines *
[x ] I have starred the repository *
[x ] Working under Gssoc24*

@Venkatknight2295 show me demo

hey @HimanshuNarware if you have any figma design i can make the ui components the ui is not great and as we focus on the internship junction (marketing side) we need to improve ui.
please assign this issue to me and also provide the figma files if you have.
I am excited to work on this issue and work with you guys.

Hey @HimanshuNarware If You Please Tell Us We Want To Work On This Issue Or Not You Also Added Labels.

can you assign this isssue to me i worked on it to design better ui
As I learned mern full stack development

under gssoc'24

@HimanshuNarware Please assign this issue to me under GSSoC24


[Bug?] Wrongly paste when enter "p" into insert node

Hi, I encountered a problem, when I press "p" in insert node, neovim paste something for me and returns into normal mode, but what I try to do is to just put a literal p there. I have not seen any discussion yet(or just I couldn't find it), how to solve it? Is there a way to disable paste in visual select mode? Or it is a bug? If not, how to insert literal p properly into beginning of insert node?

keymap("v", "p", '"_dP', opts)
This is culprit.
I use a keymap by others, if you too have this line, comment it out.

keymap("v", "p", '"_dP', opts)
This is culprit. I use a keymap by others, if you also have this line, comment it out.

The mentioned keymap is pretty useful still. Here is kind of a workaround hack to mitigate this:
vim.keymap.set("v", "p", function()
  if require("luasnip").in_snippet() then
    vim.api.nvim_feedkeys("p", "n", false)
  else
    vim.api.nvim_feedkeys('"_dP', "n", false)
  end
end, opts)


Enable CA1827: Do not use Count() or LongCount() when Any() can be used

https://docs.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1827

PowerShell-CI-windows
PSTests.Sequential.CommandPredictionTests.PredictInput`
The collection was expected to contain a single element, but it was empty.
at PSTests.Sequential.CommandPredictionTests.PredictInput() in D:\a\1\s\test\xUnit\csharp\test_Prediction.cs:line 115

@iSazonov Why have you approved, there are failing tests!

@iSazonov Could you please restart tests.


Add example argument to gradient calculation

Some models need access to the original example to get gradients, so adding this backwards-compatible change to support those models.

I ran the tcav_test after updating the test to ensure it still passes. Have not run a full TCAV with this change but it should have no effect on existing TCAV uses.


[filetests]: add arithmetics to zk-runtest

Here I will add cranelift tests to our zk-filetest pipeline and try to find if something is broken

Aaaaaaaaaaand, here we are, I copied all tests from arithmetic.clif to zk-arithmetic.clif, except lower bit types which are not related to zk. And, it works, nothing failed on side of test infra, only known bug about signed division (and we can see here how test results are printed in case of fail, thanks for it)

Aaaaaaaaaand, first benefit from new test infra. Our i32 bnot doesn't work. Or we knew this before?? TODO: filetests don't have a lot of tests for bitops. Should we copy wasm spectest here?

Signed right shift is not implemented yet, so didn't add tests for it


Update Grant-CsTeamsAppPermissionPolicy.md

Note block added replacing the bold Note. These changes fit Microsoft's style guide.
https://learn.microsoft.com/en-us/powershell/scripting/developer/cmdlet/strongly-encouraged-development-guidelines?view=powershell-7.3

Learn Build status updates of commit 51167b9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Grant-CsTeamsAppPermissionPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Hi @unicornpancakes, thank you for your contribution.
In cmdlet’s articles the [!NOTE] (also [!IMPORTANT] & [!CAUTION]) should not be used because it can’t be rendered later by the Get-Help cmdlet in the console window.

@unicornpancakes Gentle reminder. Thanks

Ack, thank you!


Allow ripgrep search provider to work for files backed by file scheme, but not file scheme themselves (vscode-userdata)

Fixes #115434

cc @roblourens, the code works but the location isn't ideal. Currently residing in /api to make use of the existing RipgrepSearchProvider which happens tailored to remote stack, but of course it isn't part of api and doesn't have anything to do with remote. Curious if you have any suggestions here, if not I'll keep looking.

I don't get it, how do you end up with a folder query for the userData scheme? When I try this, I see the code-snippets file under file: (but the issue reproes)


Ollama

see here

@Anindyadeep can you check if they support llama2/mistral? Otherwise let's close the issue

Yes we need to put this in medium priority, since it is an engine

@Anindyadeep can you check if they support llama2/mistral? Otherwise let's close the issue

Yes we need to put this in medium priority, since it is an engine

isn't ollama practically a wrapper around llama.cpp though?


Feature/dt 1424 add blacklist filter

Accepts a blacklist as a comma separated string and removes blacklisted values from the set list
Removed scrapwood.sc from the project

@moltude technically I don't believe you can't have two optional inputs from the command line, but since we're going to change it anyway, I think it's okay for now.


leaderlayer attach and detach multiple times in one frame causes a crash

import 'package:flutter/gestures.dart';
import 'package:flutter/material.dart';
import 'package:flutter/rendering.dart';
import 'package:flutter/gestures.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: Text('flutter'),
        ),
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: const <Widget>[
              MyHomePage(),
            ],
          ),
        ),
      ),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key}) : super(key: key);
  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  bool flag = true;
  LayerLink link = LayerLink();
  void _incrementCounter() {
    setState(() {
      flag = !flag;
    });
  }

  @override
  Widget build(BuildContext context) {
    return _CompositedTransformTarget(
        link: flag? link : null,
        child: _CompositedTransformTarget(
          link: flag? null : link,
          child: Text.rich(
              TextSpan(text: 'abc', recognizer: TapGestureRecognizer()..onTap = (){
                _incrementCounter();
              }),
          ),
        )
    );
  }
}

class _CompositedTransformTarget extends SingleChildRenderObjectWidget {
  const _CompositedTransformTarget({
    Key? key,
    this.link,
    Widget? child,
  }) : super(key: key, child: child);
  final LayerLink? link;

  @override
  _RenderLeaderLayer createRenderObject(BuildContext context) {
    return _RenderLeaderLayer(
      link: link,
    );
  }

  @override
  void updateRenderObject(BuildContext context, _RenderLeaderLayer renderObject) {
    renderObject.link = link;
  }
}

class _RenderLeaderLayer extends RenderProxyBox {
  _RenderLeaderLayer({
    LayerLink? link,
    RenderBox? child,
  }) : _link = link,
        super(child);

  LayerLink? get link => _link;
  LayerLink? _link;
  set link(LayerLink? value) {
    if (_link == value) {
      return;
    }
    _link = value;
    markNeedsPaint();
  }

  @override
  bool get isRepaintBoundary => true;

  @override
  void paint(PaintingContext context, Offset offset) {
    super.paint(context, offset);
    if (_link != null) {
      context.pushLayer(LeaderLayer(link: _link!, offset: offset),(_, __){}, Offset.zero);
    }
  }
}

Click the abc causes the crash.
What happened was due to the both renderobject of _CompositedTransformTarget got marked paint dirty in the same frame. Our current flushpaint paints the child first before the parent. When child paints, the layer is attached to the child. During the parent paints, it removes the child layer which cause the layer to detach from the child first and then paint, which then attach the layer back to the child.
══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY ╞═════════════════════════════════════════════════════════
The following assertion was thrown during a scheduler callback:
Assertion failed:
file:///Users/chtai/git/flutter/packages/flutter/lib/src/rendering/layer.dart:2130:12
_leader != null
is not true

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49  throw_
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 29:3    assertFailed
packages/flutter/src/rendering/layer.dart 2130:20                             [_unregisterLeader]
packages/flutter/src/rendering/layer.dart 2239:11                             detach
packages/flutter/src/foundation/node.dart 153:12                              dropChild
packages/flutter/src/rendering/layer.dart 395:11                              dropChild
packages/flutter/src/rendering/layer.dart 1082:7                              removeAllChildren
packages/flutter/src/rendering/object.dart 132:17                             _repaintCompositedChild
packages/flutter/src/rendering/object.dart 100:5                              repaintCompositedChild
packages/flutter/src/rendering/object.dart 995:29                             flushPaint
packages/flutter/src/rendering/binding.dart 499:19                            drawFrame
packages/flutter/src/widgets/binding.dart 883:13                              drawFrame
packages/flutter/src/rendering/binding.dart 363:5                             [_handlePersistentFrameCallback]
packages/flutter/src/scheduler/binding.dart 1144:15                           [_invokeFrameCallback]
packages/flutter/src/scheduler/binding.dart 1081:9                            handleDrawFrame
packages/flutter/src/scheduler/binding.dart 995:5                             [_handleDrawFrame]
lib/_engine/engine/platform_dispatcher.dart 1011:13                           invoke
lib/_engine/engine/platform_dispatcher.dart 159:5                             invokeOnDrawFrame
lib/_engine/engine/initialization.dart 128:45                                 <fn>
════════════════════════════════════════════════════════════════════════════════════════════════════
Another exception was thrown: Assertion failed: file:///Users/chtai/git/flutter/packages/flutter/lib/src/rendering/layer.dart:2159:16

cc @goderbauer


travis builds on osx are broken

example here: https://travis-ci.org/fireeye/flare-floss/jobs/398879200
these are blocking PR checks.

closed via #294


Error while deploying appx file.

I tried to install the appx file of HelloBlinky on a raspberry pi 3 and and am getting this error. Any help will be appreciated.
Failure reason: Windows cannot install package Blinky_<IP_ADDRESS>_arm__1w720vyc4ccym because this package depends on a framework that could not be found. Provide the framework "Microsoft.NET.CoreRuntime.1.0" published by "CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US", with neutral or ARM processor architecture and minimum version 1.0.23819.0, along with this package to install. The frameworks with name "Microsoft.NET.CoreRuntime.1.0" currently installed are: {}. Failure text: Package failed updates, dependency or conflict validation. (0x80073cf3)

How are you trying to deploy the APPX file?  Are you including the dependency APPX files?
As an example, from the Device Portal, to deploy HelloBlinky, you should build the APPX and then enter:

Enter App Package = HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Blinky_<IP_ADDRESS>_arm_Debug.appx
Enter Certificate = HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Blinky_<IP_ADDRESS>_arm_Debug.cer
Click Add Dependency 2x

HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Dependencies\ARM\Microsoft.NET.CoreRuntime.1.0.appx
HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Dependencies\ARM\Microsoft.VCLibs.ARM.Debug.14.00.appx

Click Go button

Thanks for the help. Your solution worked.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Brian Fjeldstad<EMAIL_ADDRESS>Sent: Tuesday, February 14, 2017 5:10:57 PM
To: ms-iot/samples
Cc: Derrick Agyemang; Author
Subject: Re: [ms-iot/samples] Error while deploying appx file. (#410)
How are you trying to deploy the APPX file? Are you including the dependency APPX files?
As an example, from the Device Portal, to deploy HelloBlinky, you should build the APPX and then enter:

Enter App Package = HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Blinky_<IP_ADDRESS>_arm_Debug.appx
Enter Certificate = HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Blinky_<IP_ADDRESS>_arm_Debug.cer
Click Add Dependency 2x
*   HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Dependencies\ARM\Microsoft.NET.CoreRuntime.1.0.appx
*   HelloBlinky\CS\AppPackages\Blinky_<IP_ADDRESS>_Debug_Test\Dependencies\ARM\Microsoft.VCLibs.ARM.Debug.14.00.appx
Click Go button

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ms-iot/samples/issues/410#issuecomment-279770980, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AO38RO5eTf9tEHxXRLBHvQb3lexxGaGtks5rceAhgaJpZM4MAR7R.

Closing based on our confirmation


Z-index: Fix wrong z-index order in themeing doc ( #6707)

Description
I've noticed an inconsistency in z-index ordering between doc pages mudtheme-zindex and mudblazor-z-index's. As specified in #6707, it looks like the first one is correct while the second is not, so I fixed the second one.
Fixes: https://github.com/MudBlazor/MudBlazor/issues/6707
How Has This Been Tested?
I deployed the server project and I've browsed to the documentation page I've modified, and I've seen that modifications had been applied successfully.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

@henon merging this, the change is simple and looks correct.


Add additional test coverage

Proposed changes
Added additional tests to increase code coverage
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (improve a current implementation without adding a new feature or fixing a bug)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Build (changes that affect the build system)
[ ] Docs (documentation only changes)
[x] Test (adding missing tests or fixing existing tests)
[ ] Other... Please describe:

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING documentation
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
Helps complete #10

Codecov Report

Merging #11 into master will increase coverage by 1.7%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #11     +/-   ##
=========================================
+ Coverage   84.69%   86.39%   +1.7%
=========================================
  Files          40       40
  Lines         882      882
=========================================
+ Hits          747      762     +15
+ Misses        135      120     -15

Impacted Files
Coverage Δ

ArkEcosystem.Crypto/Configuration/Fee.cs
100% <0%> (+17.64%)
:arrow_up:

ArkEcosystem.Crypto/Configuration/Network.cs
100% <0%> (+27.27%)
:arrow_up:

ArkEcosystem.Crypto/Networks/Mainnet.cs
100% <0%> (+100%)
:arrow_up:

Continue to review full report at Codecov.

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


Issue with k8s.io/docs/tasks/access-application-cluster/web-ui-dashboard/

"e.g. frontends" should be "e.g., front ends"

I'm not sure this is something we should spend much time on, actually. There are 70 occurrences of the term "frontend" in the English-language documentation, versus 15 occurrences of either "front end" or "front-end":
jaypipes@udb42383a8e5155:~/src/k8s.io/website/content/en$ ack -l frontend | wc -l
70
jaypipes@udb42383a8e5155:~/src/k8s.io/website/content/en$ ack -l "front[ -]end" | wc -l
15

While it might be correct English to use the term "front end" versus "frontend", I see both used interchangeably as accepted expressions referring to the website/design side of a platform.
https://www.sitepoint.com/spelling-frontend-front-end-front-etc/ has a good overview of the use of both terms.

While it might be correct English to use the term "front end" versus
"frontend",
I see both used interchangeably as accepted expressions referring to the
website/design side of a platform.

I think you missed my point.
"Front end" is the noun form. "Front-end" and/or "frontend" is the
adjective form.
(It's somewhat like "back up" vs. "backup" (or maybe "back-up")--the verb
form vs. the noun-or-adjective form. You don't "backup" data; you back up
data; then then you have a backup or have a backup copy. If there's any
doubt, note that you have backed up data, and you have not "backupped"
data.) )
(

https://www.sitepoint.com/spelling-frontend-front-end-front-etc/ has a
good overview of the use of both terms.

The content of that page doesn't seem to reflect any awareness of the fact
that which part of speech (noun vs. adjective) is involved affects which
form of the word/phrase is correct, so I wouldn't recommend using it as a
primary reference. (It may very well be good for whether the adjective form
is "front-end" or "frontend" or either.)
Side note: "Front end" (and presumably "front-end" too) isn't a new
term--it was used for the tuner part of TVs decades ago.))
When the text doesn't use the correct form, it's less readable, because the
reader has to correct for the false grammatical signals sent by the
incorrect form.
(Given that correct forms are like:
"X connects to the front end."
and:
"X connects to the front-end port."
(or maybe:
"X connects to the frontend port."
then writing
"X connects to the front-end."
hinders readability, because it sounds like another word(s) is coming (to
complete the noun phrase begun by the adjective "front-end"), but then
suddenly the sentence ends, and the reader's mind has to back up and start
interpreting "front-end" as "front end".
Sure, readers can handle that, but why not use standard English and make
documentation easier to read?)
Daniel
On Wed, Aug 28, 2019 at 10:32 PM Jay Pipes<EMAIL_ADDRESS>wrote:

I'm not sure this is something we should spend much time on, actually.
There are 70 occurrences of the term "frontend" in the English-language
documentation, versus 15 occurrences of either "front end" or "front-end":
jaypipes@udb42383a8e5155:~/src/k8s.io/website/content/en$ ack -l frontend | wc -l
70
jaypipes@udb42383a8e5155:~/src/k8s.io/website/content/en$ ack -l "front[ -]end" | wc -l
15
While it might be correct English to use the term "front end" versus
"frontend", I see both used interchangeably as accepted expressions
referring to the website/design side of a platform.
https://www.sitepoint.com/spelling-frontend-front-end-front-etc/ has a
good overview of the use of both terms.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/16128?email_source=notifications&email_token=ACIDHZJZSZGMAFNS7Z4P6E3QG4YMHA5CNFSM4IRYLLB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5NA6MY#issuecomment-525995827,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACIDHZI7XLKA42PJSWK7MG3QG4YMHANCNFSM4IRYLLBQ
.

@dsbos While I appreciate your detailed response to Jay's comment, please remember that the Kubernetes documentation was not always written by native English speakers, much less native English speakers who also happen to be professional Technical Writers.
If you feel strongly about the misuse of frontend, please submit a pull request with corrections.
Contribute to Kubernetes docs
Kubernetes Documentation Style Guide
Kubernetes and Cloud Native Computing Foundation Code of Conduct

/priority awaiting-evidence

/triage awaiting-evidence

I'll get it right yet
/priority awaiting-more-evidence


Subtract output values from offset when inscription spent as fee

Fixes #1841.
Indexes will need rebuilding.

@raphjaph this looks good to me - should prioritize for review.

I have found this bug too, it made a big confusion to my current data. hope prioritize for review.

hello guys,  when to merge this commit and fix history data? I think this bug is very fatal

I think this change is correct. In addition to a test, can you please describe what the bug is, why this fixes it, and the observable change in behavior? There was a lot of discussion in #1841, but I think it also discusses a different issue, so it's a bit confused. If there's already a different issue describing this, you can also link that issue.

The bug is #1841.
https://github.com/casey/ord/issues/1841#issuecomment-1483950070 explains the bug and why this fixes it.
The observable change in behavior is that ord tracks inscriptions correctly when they are spent as change rather than tracking them incorrectly.
I don't understand the confusion here. Sorry if I'm being slow.

I think the bug is very well explained! I just added the changes you made to the test in a new test so it is very clear what the bug was. This is ready for you to have a look @casey!


Build redis info_hash list from MySQL database

We'll need to switch docker cloud to using a persistent volume, but this should be fairly reasonable. Just query a specific table listing all of the validated info_hashes and create the redis sets based on this data.

We need a schema something like this:
id autoincrement
info_hash varchar(<some length)> not null
created_by varchar() nullable=True
created_by_ip varchar() NOT NULL


netconf driver integration

[x] Issue: #138
[x] List of related PRs. : None
[x] Project builds without any errors.
[x] Unit Tests updated (or created where needed) and all pass.
[x] You have ran manual functional “smoke” test (does product as a whole still work?).
[x] User Story meets the acceptance criteria - and passes. acceptance criteria should be understood at outset.
[x] There is a PR for update of internal documentation for the affected Project(s) - README etc.
[x] Description of the changes: #138

Builds failing but I'm happy this is a Travis/Python 3.7 issue. Builds are fine for 3.8 and 3.9.


Add step for disconnecting

Before running the command "adb usb" I've found that first running the command "adb disconnected <ip>:5555" is required to be run.
If this "adb disconnect" command isn't run then the following error is displayed after running the command "adb usb": 'error: more than one device/emulator'

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

ID: 0fe8d6a6-7a42-810a-d5d4-e3bd162ae09a
Version Independent ID: c7851826-27c9-9126-cb46-9300c57c440b
Content: Set Up Device for Development - Xamarin
Content Source: docs/android/get-started/installation/set-up-device-for-development.md
Product: xamarin
Technology: xamarin-android
GitHub Login: @conceptdev
Microsoft Alias: crdun

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Added useful tip with local addresses

Added useful tip with local addresses not to run ifconfig every time to remember your local IP.

I do really want to accept your work @smileart but things have diverged too much. If you'd like to resubmit I would very much appreciate it, but I understand if you're not interested.


Scrape Character Attribute Data

There doesn't seem to be a nice pre-existing dataset to use.
However, there are plenty of websites that provide tables with character attribute data (e.g. https://ultimateframedata.com/stats or https://kuroganehammer.com/Ultimate/Attributes).
I'll have to do a little bit of data scraping.

Implemented with https://github.com/J99thoms/Super-Smash-Dashboard/commit/d888bc3b1875ebf0c0cdef25ab7d2f3ab0c1bd7c.


Docker can't find my images/containers

On Ubuntu 14.04, I had faced the "docker won't start after upgrading to 1.7" issue, so I had downgraded to 1.6.2 and it worked fine.
Today, after a reboot, docker runs but cannot find any images or containers. The files still exist on disk, so it's not a hw failure.
# docker info
Containers: 0
Images: 0
Storage Driver: aufs
 Root Dir: /datadrive/dockerstore/aufs
 Backing Filesystem: extfs
 Dirs: 0
 Dirperm1 Supported: true
Execution Driver: native-0.2
Kernel Version: 3.16.0-29-generic
Operating System: Ubuntu 14.04.2 LTS
CPUs: 1
Total Memory: 3.362 GiB
Name: eru
ID: STRT:7755:73RU:BJ6W:3OKG:OUP4:6DGY:Z7O5:QPWF:HYKY:FC5W:KITI
WARNING: No swap limit support

The files/meta are on the devicemapper directory and the aufs one is empty.
# ls /datadrive/dockerstore/
aufs  containers  devicemapper  execdriver  graph  init  linkgraph.db  repositories-aufs  repositories-devicemapper  tmp  trust  vfs  volumes

# du -sk /datadrive/dockerstore/
23762420	/datadrive/dockerstore/

# du -sk /datadrive/dockerstore/aufs/
16	/datadrive/dockerstore/aufs/

# du -sk /datadrive/dockerstore/devicemapper/
21634000	/datadrive/dockerstore/devicemapper/

# cat /etc/default/docker
DOCKER_OPTS="-g /datadrive/dockerstore"

I tried apt-get install linux-image-extra-$(uname -r)  as per this discussion and rebooted, same problem. :sob:
What can I do?

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Added --storage-driver=devicemapper on the DOCKER_OPTS and it worked. :grinning:

I had faced the "docker won't start after upgrading to 1.7"

I suspect that the problem you were having, is that devicemapper on your system does not have "udev sync" support. Running without udev sync is known to cause problems and, in some cases, can lead to data corruption.
For that reason, docker 1.7.0 (deliberately) refuses to start if udev sync is not supported, but you can override that behaviour using the dm.override_udev_sync_check storage-opt.
For more information see: https://github.com/docker/docker/pull/11412, https://github.com/docker/docker/pull/14415 and the issue described in https://github.com/docker/docker/issues/4036
So your best options are:

Switch to the AUFS storage driver (but you will have to save / load your images to migrate them to the new storage driver)
Build a dynamically linked version of docker: https://github.com/docker/docker/issues/4036#issuecomment-111174341. At this moment, we don't provide dynamically linked binaries for download.
override the driver to use (--storage-driver=devicemapper) and set --storage-opt dm.override_udev_sync_check=true. Use this at your own risk, because this configuration is known to cause problems, as described in https://github.com/docker/docker/issues/4036

@thaJeztah Thank you! I've overriden the driver and will now make backups and migrate to a clean vm with 1.7.

@rotatingJazz great! I know it's a hassle, but wanted to give you the right information so that you understand what's the cause / reason behind this!
If you have ideas on improving this (e.g., by better documentation around this), we welcome pull requests! :smile:


uhh skid?

bro forgor to give credits to Laze

fr

Skuld precedes Laze in age; you can verify this by referencing the publication date on https://www.trellix.com/about/newsroom/stories/research/skuld-the-infostealer-that-speaks-golang/. Although it's no longer on your GitHub profile, you're aware of the timeline.
Regarding the "skid", I don't see where I "skidded" your project: it's true I recently collected wallet paths from diverse repositories, including Laze, to improve my stealer, but crediting individuals for paths isn't a common practice on GitHub repositories.
Should I attribute you for these paths?  I'm open to doing so if you prefer.

Just because you use the name of an older grabber does not mean it is the same.

In fact, this account you are currently using for GitHub was not even created until months after that article was posted.

And your statement does not justify well enough that "your" program has exact same lines as Laze. As well as a very blatant copy of design, which since then code is the same, can be inferred you are just copy and pasting.

blud didnt even make the exodus injection 😭
you cant just justify your blatant skidding by giving credits 😭
you didnt do the hard work of actually reversing exodus

I want to clarify that I didn't copy Laze and I am the creator of Skuld.
I hadn't intended to release Skuld originally. The decision was driven by feeling pressured after Laze was released. I hurriedly published Skuld to avoid any accusations of copying Laze.
I opted to create a new account for the release. I did not want Skuld to get stars only because my old alias deathined got mentionned on many articles.
Regarding the Exodus injection, I provided due credits when using it, as many individuals acknowledge when utilizing someone else's code. Therefore, attributing blame to me in this context wouldn't be fair.


Simulator.py

Description of changes:
Modified the BaseLocalSimulator class to support circuits with non-contiguous qubit indices.
Removed validation for contiguous qubit indices in the _validate_operation_qubits method.
Added logic in the run_openqasm method to detect non-contiguous qubit indices and remap them to contiguous indices if necessary.
Updated the _create_results_obj method to use the used_qubits for measurements if they are non-contiguous.
Implemented the _map_to_contiguous_qubits method to map non-contiguous qubits to contiguous qubits.
Testing done:
Tested the modified CustomLocalSimulator class with circuits containing non-contiguous qubit indices.
Verified that the simulator produces correct results for circuits with both contiguous and non-contiguous qubit indices.
Merge Checklist
General
I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have updated any necessary documentation, including READMEs and API docs (if appropriate)
Tests
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have checked that my tests are not configured for a specific region or account (if appropriate)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks for the contribution! Please make the changes to existing files so they can be used when LocalSimulator is instantiated. As well, please add unit tests to cover all of your changes.

Can this be closed? Looks like you're already working on the issue in #259.


Extension

Refactor into an extension so it can be pulled into any collector distribution:

Add start/stop calls to start and stop the underlying pipeline components
Use go embed to embed static files and cfg metadata (had to move them under the go code source tree for go embed directives to work)
Use a new main.go based off the otel contrib main.go as an example, with this new extension loaded
Add an example of use with explicit call out to use localhost:8000 as 8888 is reserved for prometheus metrics
move from log.Logger to zap.Logger to satisfy requirements from otel collector libs
update to 0.68.0 API, remove deprecations.
Move from internal/otto to otto as we're now publishing this package for inclusion
Important: There is a limitation in the way the extension runs in that it cannot know what receivers, processors and exporters are registered. Specifically what receivers are registered. So I added explicit fields to the config to allow for that. That might not be the right idea and we could ask for a better API for the extension to tap into registered components.

Thanks for the PR but we ended up retiring the Otto functionality as a POC and creating a new app in its place.


Players create light clouds

@erikbuck Can I leave this command in the code?
Issue #33 and issue#34 were created on June 19 to let players create clouds anywhere and may build their homes on clouds
issue#42 was created on June 21 that block the user to build on clouds!
@w1158mxs left a preview on June 22 in issue#42 that will use the key it was mentioned in issue#33 and issue#34 to build clouds and will let players build their homes on clouds!
while issue#80 used keys
@w1158mxs uses another key to build clouds
@see README.md to know how to play it

Requested in wrong project!


Unresolved P/Invoke method build warning for UWP using msal

We've received the following build warning for our UWP app which uses Microsoft.Identity.Client 4.22.0 and Microsoft.Identity.Client.Extensions.Msal 2.16.5.
The app and authentication appears to be working ok, but I wondered if this is a problem, or there is something we need to do to fix the warning?
##[warning]C:\Users\VssAdministrator\.nuget\packages\microsoft.net.native.compiler\2.2.9-rel-29512-01\tools\Microsoft.NetNative.targets(805,5): Warning :       MCG : warning MCG0007: Unresolved P/Invoke method 'libsecret-1.so.0!secret_schema_new' for method 'System.IntPtr Microsoft.Identity.Client.Extensions.Msal.Libsecret.secret_schema_new(System.String, System.Int32, System.String, System.Int32, System.String, System.Int32, System.IntPtr)'. Calling this method would throw exception at runtime. Please make sure the P/Invoke either points to a Windows API allowed in UWP applications, or a native DLL that is part of the package. If for some reason your P/Invoke does not satisfy those requirements, please use [DllImport(ExactSpelling=true) to indicate that you understand the implications of using non-UWP APIs.
We receive about 10 similar warnings for different P/Invoke methods related to the package - is the possible runtime exception something we can fix?

This library calls into KeyChain and KeyRing on Mac / Linux. Those methods will not be called on Windows.
That said, this library wasn't designed for use with UWP, because on UWP, MSAL will store the token cache in a DPAPI protected file on its own. You do not need to customize the cache.


add entities to get_metadata

I'm sitting in @mih's kitchen and he made me add this at knifepoint

Tests are failing on travis, but it's because the conda installer appears to be having serious issues on linux at the moment. Everything's passing on both OS X and Windows, so I'm merging.

... he made me add this at knifepoint

At least he didn't make you drink all his liqueur, did he? ;)

He's working on that as I type this.


Viewstyle 's borderLeftcolor  or other pozitionColor attribute is never work?

I set those color,but no none worked!

to additonal descript ,  only add the  borderStyle attri ,can cause this problem!

Seems to be the same bug as https://github.com/facebook/react-native/issues/3159
What version of react-native are you using? Is it on iOS or Android?
Can you please provide a more specific description and say exactly what you expected to happen, in comparison to what actually happened?

What I'm trying to do
What's happening
[!What I'm trying to do]
(http://i.imgur.com/9ZVk72f.png)
What I'm trying to do
[!What I'm trying to do]
(http://i.imgur.com/37WOZLF.jpg)
Code
const styles = StyleSheet.create({
  titleSection: {
    flex: 1,
    borderBottomWidth: 1,
    borderBottomColor: 'black',
    borderStyle: 'dotted',
    flexDirection: 'row'

  },

 <View style={styles.titleSection}>
          <FAIcon name={this.state.collapsed ? "chevron-circle-right" : "chevron-circle-down"} style={styles.titleIcon} size={20} />
          <TouchableOpacity onPress={() => this._collapser()}>
            <Text style={styles.title}>{this.props.titleName}</Text>
          </TouchableOpacity>
        </View>

Running on react-native run-ios

i am using version 3.2 ,i am sorry the latest vesion is worked .  thank your excellent work @ericvicenti @Gacnt

It's not working for me, latest version.

u can try it on facebook rn website . @Gacnt

@Gacnt, does it work when borderRadius is set? Do you get any warnings?

Here is a rnplayground
https://rnplay.org/apps/dMM6qA with my exact issue, uncomment line 30 in that playground and watch the border disappear.

yeah ,it is not work on rnplay.
here is sourcecode
` <View style={{ width: 230,
height: 230, backgroundColor: '#fff',  borderBottomColor: '#f00',
borderBottomWidth: 4, borderStyle:'dashed',}}>

test border color.

   </View>`

any update?


fix: allow regex

This fixed the allow regex to allow - characters in the username.

const body = "/allow set-multiplier @ubiquibtyaddd true"
const regex = /^/allow set-(\S+)\s@(\S+)\s(true|false)$/;
body.match(regex)
lol it works.

const body = "/allow set-multiplier @ubiquibtyaddd true" const regex = /^/allow set-(\S+)\s@(\S+)\s(true|false)$/; body.match(regex)
lol it works.

Yes!

ok I am doing an admin action for both PRs then.


quic: Bump maximum history length, make configurable

@timwoj / @ckreibich - would like to get that into 6.1. This is the change in the spicy-quic repo that in a second step would be cherry-picked into the Zeek tree after rewriting (or so my approach currently). Mind approving?
@dopheide-esnet shared a breakdown of the history field and there were many variations capped at 10 characters.


Expose Memory Usage Metrics

Is your feature request related to a problem? Please describe.
It is currently impossible to get memory usage metrics from grpc. Sometimes grpc hits its internal memory limit and cancels calls with ENHANCE_YOUR_CALM. If the application code knows the grpc memory usage then it can adapt to these memory limits and modify its behavior.
Furthermore it would be nice to be able to build dashboard on the grpc memory usage using these metrics.
Describe the solution you'd like
An api that would expose how much memory the grpc server is using and what is the limit for the grpc server. This would allow both the application code to adapt to grpc nearing its memory limit and also allow users to build metrics to understand and keep track of the grpc server's memory usage over time.

@yash620 This is something we cannot do practically because it's not easy to track all memory consumption of gRPC. Therefore you might need to consider using low-level functions to measure the system resource. e.g. monitoring the memory usage of the server process.
@donnadionne PTAL (since you're an owner of resource-quota.)

The server is oftentimes an object in the application program. So it would be hard to distinguish memory usage between the application program and the sever if we are only monitoring system metrics. If all memory consumption can't be exposed then exposing whatever is accounted for in resource_quota would be helpful as well.
I did try raising the max memory usage of the server to be larger than the system limits and monitor the system memory usage and respond within the application appropriately based on what the system memory metrics said. But I don't think this is an efficient usage of memory though because unless the grpc server detects that its resource_quota is about to be exceeded it doesn't attempt to non-destructively reclaim memory.


wails & web: update sign tx screen

We are removing line break and useless spaces in Sign tx screen for wails and web frontend.

Codecov Report
Patch and project coverage have no change.

Comparison is base (442b9e6) 42.01% compared to head (ecdd8ed) 42.01%.

: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.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #505   +/-   ##
=======================================
  Coverage   42.01%   42.01%
=======================================
  Files          15       15
  Lines        1071     1071
=======================================
  Hits          450      450
  Misses        575      575
  Partials       46       46

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Feature Request: Autostart

Please add a function that attempts to autostart the RPyc service as that would enable a lot of really fun automation without having to first open the program manually.

I think you can launch Binja from the command line, and when the plugin loads just start the service?

I’ve completely forgotten why I submitted this, but i still want it :)

Done, thanks for the suggestion! It is a useful feature. Let me know how it goes


Make kwctl inspect & kwctl annotate understand spec.backgroundAudit in metadata.yml

Acceptance criteria

[ ] kwctl inspect and kwctl annotate should accept a new backgroundAudit field in metadata.yml, analogous to mutating. The field is set to true by default
[ ] Provide unit & e2e tests.

As discussing in a call.  @viccuad is already working on this issue while he works in the #331. Therefore, I'm assigning him to this issue as well.


SOLR-15957: Add information to Prometheus Exporter startup logging

https://issues.apache.org/jira/browse/SOLR-15957
New startup logs look like:
INFO  - 2022-01-27 15:55:39.177; org.apache.solr.prometheus.exporter.SolrExporter; Starting Solr Prometheus Exporting on port 8989
INFO  - 2022-01-27 15:55:39.197; org.apache.solr.prometheus.collector.SchedulerMetricsCollector; Beginning metrics collection
INFO  - 2022-01-27 15:55:39.289; org.apache.solr.prometheus.exporter.SolrExporter; Solr Prometheus Exporter is running. Collecting metrics for Solr Node: http://solr-host:8983/solr

That is currently the way that it works. I guess it could accept multiple in the future.


Javascript error when running app

Hey
This project seem like just the thing I'm looking for, but I really need the GUI app to work.
After having quite a bit of trouble getting node.js and electron installed I finally managed to build the app. However, when I try and run the .exe I get a javascript error
A JavaScript error occurred in the main process

Uncaught Exception
Error: Cannot find module 'menubar'
    at Function.Module._resolverFilename (module.js:336:15)
    at Function.Module._load (module.js:286:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:7)
    at Object.<anonymous>
(C:\...\Elodie-win32-x64\resources\app\app\module...:15)
    at Module._compile (module.js:434:26)
    at Object.Module._extensions..js (module.js:452:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)

Do you have an idea what is causing this, and how to fix it?
Is it a bug with the windows desktop app or am I doing something wrong?

Hello,
You have to install menubar module:
npm install menubar

You can get all of the dependencies by running npm i from the root of the repository. That will include menubar.
The menubar app is still in alpha so please reopen if you still have issues.


Poll Time for HLS Manifest

Have you read the FAQ and checked for duplicate open issues?
yes
Is your feature request related to a problem? Please describe.

yes.  While earlier specs say to use the target duration tag to poll for manifest updates, later specs suggest the last segment duration as the poll time. Sometimes target duration is just a set length 1 or 2 seconds larger than the set segment length. So a manifest that has variable segment length which is very common in ad insertion workflows can sometimes have buffer starvation near the live edge. This should provide more robust support for hls ssai workflows. This is a spec that is followed commonly in commercial players like bitmovin, theoplayer, and OS hlsjs.
Describe the solution you'd like
use last downloaded segment duration as the poll time for the next manifest update. A variable poll time.
Basically later official spec versions describe that approach so it is spec approved.
When a client loads a Playlist file for the first time or reloads a
Playlist file and finds that it has changed since the last time it
was loaded, the client MUST wait for at least the duration of the
last segment in the Playlist before attempting to reload the Playlist
file again, measured from the last time the client began loading the
Playlist file.
https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis#section-6.3.4
Describe alternatives you've considered
n/a
Additional context
n/a

In order to do this we have to change how we internally update playlists. Right now they are all updated at the same time, with this change they should be updated separately because the video and audio segments do not have to last the same time, for example.


Account mutations

Codecov Report

Merging #11 into master will increase coverage by 4.56%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   85.29%   89.85%   +4.56%
==========================================
  Files           3        6       +3
  Lines          34       69      +35
  Branches        4        6       +2
==========================================
+ Hits           29       62      +33
- Misses          5        7       +2

Impacted Files
Coverage Δ

src/types/currency.ts
100.00% <ø> (ø)

src/models/account.ts
93.54% <100.00%> (ø)

src/models/baseModel.ts
100.00% <0.00%> (ø)

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 882ce4d...4b44bbe. Read the comment docs.


Added Edgar, Knack and Corgibytes

Updated previous pull request by adding link to company profile directly from README.md

Thanks for that! :)


Add STT engine based on Kaldi GStreamer server

Add an STT engine based on https://github.com/alumae/kaldi-gstreamer-server which offers an HTTP interface (very similar to GoogleSTT) and a WebSocket interface.

@Kaljurand  Can you test the implementation?

Thanks for the quick implementation!
I got it working after changing the headers-line to:
headers = {'Content-Type': 'audio/x-raw-int; rate=%s' % frame_rate}

I didn't test all the error condition handling though.
(Also, had to comment out "import mad" in client/plugin.py because this dependency was not installed as part of the requirements.)

In case you want to test it with an online server then you can use the URL http://bark.phon.ioc.ee/english/speech-api/v1/recognize. It's meant to be a demo (e.g. the recognition models are not very accurate), i.e. it should not be used as the default setting.
Ideally the plugin should be based on the WebSocket interface. This would allow you to stream the audio to the server already while the user is speaking and also start to immediately process the transcription. This would make the whole interaction snappier. I guess some of the other STT plugins would profit from such a streaming mode as well. (But that's a separate issue.)

Ideally the plugin should be based on the WebSocket interface. This would allow you to stream the audio to the server already while the user is speaking and also start to immediately process the transcription. This would make the whole interaction snappier. I guess some of the other STT plugins would profit from such a streaming mode as well. (But that's a separate issue.)

That'd need some major changes to the way recording and transcription is handled right now. It's a good idea, but this needs some thinking.


Play vod manifest created from live (similar to closed issue?)

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
3.1.2 / 3.3.0-pre
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?#
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Chrome / Win
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

Awaiting permission from customer to send URI
What configuration are you using?  What is the output of player.getConfiguration()?

Default
What did you do?
Tried to play a manifest from a customer (Manifest URI will be supplied when I can get permission from customer)
What did you expect to happen?
Content plays
What actually happened?
We have a customer with a manifest which won't play and it seems similar to issue https://github.com/google/shaka-player/issues/1884
Can you please clarify the outcome of this issue?

initially it was tagged as "bad content"
later this was removed, and it was tagged as "bug"
Later the issue was closed, but I couldn't see a resolution.

Our customer's manifest is vod, created from live.
It has a mediaPresentationDuration of 30 minutes, but its only period has a start time of 51 years (time since 1970)
During parsing, an assertion fails in createTimeline because the periodDuration is negative.
It goes on to try to acquire segments at t=30 minutes, even though the first segment is t=51 years (due to clampSeekToDuration_)
I'd really appreciate it if you could let us know the resolution of 1884, so we can get back to our customer to let them know whether their content is at fault.
Incidentally I also tried this content in dash.js, and saw the debug:
[704][StreamController] Waiting for<PHONE_NUMBER> ms before playback can start

https://dvb.org/wp-content/uploads/2020/02/A168r3_MPEG-DASH-Profile-for-Transport-of-ISO-BMFF-Based-DVB-Services_ts_103-285-v140_June_2021.pdf clause 10.9.7 indicates how a player should be able to handle such cases of converting live to vod,  in this case the first period may have a start time that is not zero as mentioned above, so i think it is important to test this behaviour well in shaka player aswell and some test content (both live and vod would be helpful).

From the symptoms and the similar issue in dash.js, I would suggest that there's likely something wrong with the way the content is converted from live to VOD.  But without a manifest URI, there's no way to confirm.


JSON in data-* attributes

What is this feature about (expected vs actual behaviour)?
Minification is breaking JSON placed in data-* attributes. Actually, it doesn't "break" the JSON part, it replaces single quotes with double quotes.

Becomes:

Unfortunalety, you can't use single quotes in JSON. But you can use single quotes in HTML.
How can I reproduce it?
<?php

require_once 'vendor/autoload.php';

use voku\helper\HtmlMin;

$htmlMin = new HtmlMin();

ob_start();
?>

<?php
$html = ob_get_clean();

echo $htmlMin->minify($html);

Does it take minutes, hours or days to fix?
I have no idea, I didn't dig into code.

Thanks for the good bug report. :+1: The issue is fixed in version 3.1.7.


fix bug: EXGAE  EXAT wrong expire time

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.dengxiaochao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you.


Setting "currencyItems" in vtt.cfg

I realize that I'm shouting into the void here, but just in case someone happens to know the answer, or the mod author returns from hiatus, here is my question:
In vtt.cfg, there's an option to set "currencyItems" - the default is "minecraft:emerald", but does this actually affect anything? I've tried setting it to a handful of other items, modded or vanilla, but I haven't noticed any changes taking place when I reload the game.
Thanks for your time! :)

I too was looking for an answer for this. As far as I could tell, I found no difference when adjusting it.


Add Table.create and Index type

Working on multiple branches at once got to be a mess, especially when I wanted to use the two things together. So here we are. This PR exists for code review and to remind me that I'm doing this if I somehow forget

Regarding adding a get_indexes or get_columns method to DDLSession
In order to guarantee that we can produce Index and Column objects, we'd have to make it possible to get a Table object from a database schema
Which I guess means DDLSession.get_table is gonna have to be a thing

In order for DDLSession.get_table to exist, we would need a way to get all the bound tables for a DatabaseInterface; we don't want to go through the probably expensive process of creating a Table from a database schema if we've already created it.
Would a dictionary of table names pointing to Table objects on the Databaseinterface be appropriate? It would be required, then, for Table.drop as well as DDLSession.drop_table to update that dictionary, which makes things messier.

Well, TableMetadata has a tables attribute which is a name -> value mapping of table objects.
However, I'm not too keen on that, because the state of a table in the app and the state of a table in the database can differ.

Oh, right, of course. So should we just produce owner-less index and column objects? Maybe with a table_name attribute?

That could work, yes.

Should PrimaryKey be changed to a subclass of Index, or is that not an appropriate representation of how things work>
I'd probably want to add the primary key index in Table._calculate_primary_key, regardless, since that seems like an appropriate place to do it

No; primary keys are not always indexed.

👌

As far as I can tell through some quick research, all three of the database drivers we currently support enforce mandatory creation of indexes on primary keys. So, then, I assume it's okay at this point to always create an index corresponding to the primary key, even if the primary key itself is not an index?
If you want, we could add enforces_primary_key_indexing as a dialect property

Yes, it'd be okay to have an auto-created index, but the primary key object itself should not be a index subclass.

Gotcha, thanks.

Ready 4 review whenever


Prepare commands to build and install operator for PR checks

The goal of this issue is to create commands to build and install operator for test suite setup for PR checks

Duplicate , as this is covered in (https://github.com/project-codeflare/codeflare-operator/pull/136/)


Fix cp overwriting identical files

If cp is passed a destination that is location of a source file, in effect, copying a file to itself, the source file will become empty. This prevents this situation form happening by verifying that the destination is a different location than the source.
Fixes #678

Same bug from #673 is affecting the windows tests here. Let's see if we can implement a fix.

Codecov Report

Merging #679 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #679      +/-   ##
==========================================
+ Coverage   94.32%   94.33%   +0.01%
==========================================
  Files          33       33
  Lines        1180     1183       +3
==========================================
+ Hits         1113     1116       +3
  Misses         67       67

Impacted Files
Coverage Δ

src/cp.js
90.35% <100%> (+0.26%)
:white_check_mark:

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 b6b2cd8...311ed4d. Read the comment docs.


Creating an object in js lost all built-in function links

When is js I create a new Object :
new MyObject(); which extends Sprite2D, the object is well created, but the functions _process, _ready, _draw and all the built-in functions are not called.
It seems to only happen when I try to create an object which is not initially linked to the scene. I mean, I didn't put the script on an object on my scene in the editor.
I am using Godot 4.0.

just this.set_process(true)


Add gitfile to ig

The gitignore will not ignore .git and itself. However, IMO they should be ignored instead of letting users to add them to global settings.
Close #370

yeah,you are right,.git shouldn't be included


EditInPlace not removing focus when clicking outside of component

Package
Carbon for IBM Products
Description
I created an original issue https://github.com/carbon-design-system/ibm-products/issues/6050, and currently it is delivered to "@carbon/ibm-products": "2.52.0-rc.0".. It kind of works but not fully fixed.
If you click on the input, and without making any changes (dont enter any input), then click away, the focus is not removed.
If you do make a change, like add or remove a letter and then click away, the focus is removed.. so it's an improvement but not fully fixed.
Note, this may or may not be a duplicate of https://github.com/carbon-design-system/ibm-products/issues/6106.. I saw this, but wasn't clear if it was the same as what I found.. And so i'm opening this one with additional details.. I am reproducing the issue using the 2.52.0-rc.0 build, which includes 6050 fix, but found 2 scenarios that focus is not removed and documenting the steps below.
Component(s) impacted
EditInPlace
Browser
No response
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
2.52.0-rc.0
Suggested Severity
Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.
Product/offering
IBM Data Replication
CodeSandbox or Stackblitz example
https://stackblitz.com/github/carbon-design-system/ibm-products/tree/main/examples/carbon-for-ibm-products/EditInPlace
Steps to reproduce the issue (if applicable)
I can produce this issue in my project, but i can also reproduce it from the stackblitz EditInPlace sample..
I went to https://stackblitz.com/github/carbon-design-system/ibm-products/tree/main/examples/carbon-for-ibm-products/EditInPlace, stopped the server in the terminal, ran npm install<EMAIL_ADDRESS>to install the prerelease build that includes fix for the https://github.com/carbon-design-system/ibm-products/issues/6050 issue... (note that the PR that fixed this is https://github.com/carbon-design-system/ibm-products/pull/6146) and is listed as included in the ibm-product's releases page..
Using the stackblitz code sample example above:
I started with abc in the input

Step 1: click on the input to gain focus

Step 2: WITHOUT typing anything (dont add or remove input character), click away.. it remains focused but it's really not focused anymore..

Another scenario not working..
Step 1: Start with abc, and you click and focus on the input..

Step 2: Type d (so results in abcd)

Step 3: Click the X (cancel) button.. it reverts it back to abc (as expected), and then click away from the input.. the component still stays focused

if you do type something, like add a letter d (so it results in abcd), then click away, it saves and removes focus as expected..
If you remove a letter, like remove c (so it results in ab), then click away, it also saves and removes focus as expected..
So it seems like if you focus on the component, and at the point of clicking away (try to remove focus, or focus on another component), if this component's value is the same (either you changed it or you clicked cancel to revert your changes), it doesnt remove focus..
Release date (if applicable)
Release date Nov 7
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current issues for duplicate issues

This issue is observed when onBlur prop is not passed. We are working on it.


[WIP] Simplify HasMirrorTransform computation

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 0.10.999-cibuild0020729-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

I don't see any public use for HasMirrorTransform

IMO very important for debug.
also i do not think it is the job of the renderer to calculate such things, he is not supposed to do an calculation of an element according to his parent, but to process a finite value of the element.

@danielmayost

FlowDirection, not HasMirrorTransform I think is more important in the public API. All controls are written for LTR by default. So if FlowDirection is RTL you know it had a mirror transform applied. Do I miss something?
I fully agree that mirror transforms shouldn't be done in the renderer at all. That is layout related calculations that are done at a level higher than a renderer. A renderer is literally just drawing primitives in my opinion of course with some transforms. However, not layout related transforms.

Do I miss something?

amm, yes, HasMirrorTransform does not mean the Control are shown as mirror, but apply MirrorTransform on himself. if a parent is RTL and his child is LTR so both will get MirrorTransform, because the child shouldn't appear as mirror, otherwise, the child appear as his parent which is mirror presentation.
Panel    - FlowDirection=RTL - HasMirrorTransform=True
^
Button - FlowDirection=LTR - HasMirrorTransform=True

@danielmayost I see. I have to agree with @Gillibald then that this property shouldn't be in the public API. It is an implementation detail. The real property is FlowDirection.
In my opiniom: If this property is ever needed in the future it can be added as a non-breaking change. Just switch internal modifier to public.

@robloo i agree.
It just helped me a lot to debug issues in FlowDirection, but anyone who wants to will probably be able to do it independently.

You can test this PR using the following package version. 0.10.999-cibuild0020796-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


Flag errors are not ExitCoder

my urfave/cli version is
v2.2.0
Checklist

[x] Are you running the latest v2 release? The list of releases is here.
[x] Did you check the manual for your release? The v2 manual is here
[x] Did you perform a search about this problem? Here's the Github guide about searching.

Dependency Management

[x] My project is using go modules.
[x] My project is using vendoring.
[ ] My project is automatically downloading the latest version.
[ ] I am unsure of what my dependency management setup is.

Describe the bug
If you mark a flag as "required" for instance and it is missing, then the error that is bubbled up to the root cli.App.Run(..) call does not satisfy the required interfaces for HandleExitCoder.
To reproduce

Have a required flag
Have a panic after the call to HandleExitCoder
Call the CLI without it

Observed behavior
The panic is called
Expected behavior
HandleExitCoder calls os.Exit(..)
Additional context
It should be possible to define the exit code for a specific flag (or for the application in general) if there are flag errors. Be them required, or not defined as a flag.

Makes sense! I would be happy to review a PR from anyone that fixes this 🙏

Would you prefer it to be a global setting for the whole App or Command or one specific to a flag?

I'm not quite sure I'm understanding where a setting is coming into play here? I think my expectation is that the existing required flags error satisfies that HandleExitCoder interface.

Yes but what should that exit code be? Should it be static and unchanging? I think that it should be able to be set by users of this library.

Ah, gotcha! I'm of the opinion that we should either:

exit 0 with a global setting for configuring that
return when hitting required flags errors, in which a way that an explicit os.Exit isn't required

I'd be interested in more opinions from @urfave/cli though!

I think returning an ExitCoder should be part of it.
Another option (I like this one because it is the most customization while still having a fast path for global setting):

Have a hierarchy of exit codes: use app setting or, if set, command setting or, if set, flag setting

Though, now that I have looked more deeply at the code, there is a slight problem with having the flags define their own exit codes. What happens if multiple flags produce errors (currently only required).

Should the first win?
Should the last win?
Should the highest value win?
Should the lowest non-zero win? (What about negative values?)

I have looked at some what other programs do (such as posix exit). It seems to be undefined if you try and use a negative number, or if you try and use a number outside of [0, 255] since that is all that wait_pid will return. All this is to say that the last option (lowest wins) should probably be never considered.

Nice investigative work ✨

bump

@urfave/admins Any thoughts on this ?

This should be fixed as part of #1662


Reworking directory structure has broken imports

Changed main.py to __main__.py in order to be able to run the python package directly. Fixed imports, as they were using a local installation of the multi_stat_analysis package from testpypi.


com.microsoft.kiota.serialization.ParseNodeFactoryRegistry throws RuntimeException if service returns status code 503 with unexpected content type

Situation and background:
The Microsoft Graph API had some issues some days ago, where some calls to the API would return HTTP status code 503, with some html error message.
During this time, our use of the Graph API SDK, which in turn uses Kiota, lead to several application level crashes, because of uncaught RuntimeExceptions was propagated up the call stack from Kiota.
My observations, based on my understanding of the code:
The source of the exception, ParseNodeFactoryRegistry  method getParseNode will attempt to find a parser for an expected content type.
In the case of Graph API, we expect JSON, so we only set up factories to handle that.
When the Graph API has an issue/downtime/whathaveyou, the method finds no factory to handle the response, and throws a RuntimeException directly.
Straight forward, I feel theres' two possible solutions.

Make getParseNode throw ApiException, which is a bit safer to try/catch than a pure "catch all" RuntimeException
Add a special case content type text/html factory that handles special error page responses, which might be in scope only for the graph api sdk.

With limited knowledge of how easy a custom factory is as an end user of MS Graph SDK, we feel forced to stop catching ApiException, and start catching RuntimeExceptions instead, hoping that no other runtime exceptions gets thrown and swallowed by our logic.

link to the method that throws the exception: https://github.com/microsoft/kiota-java/blob/0cd6b6d8a3263c6cc2d1d1fe6b2358c054da46e0/components/abstractions/src/main/java/com/microsoft/kiota/serialization/ParseNodeFactoryRegistry.java#L56C8-L57C97
Stacktrace from our log:
java.lang.RuntimeException: Content type text/html does not have a factory to be parsed
at com.microsoft.kiota.serialization.ParseNodeFactoryRegistry.getParseNode(ParseNodeFactoryRegistry.java:56)
at com.microsoft.kiota.http.OkHttpRequestAdapter.getRootParseNode(OkHttpRequestAdapter.java:582)
at com.microsoft.kiota.http.OkHttpRequestAdapter.throwIfFailedResponse(OkHttpRequestAdapter.java:649)
at com.microsoft.kiota.http.OkHttpRequestAdapter.send(OkHttpRequestAdapter.java:280)
at com.microsoft.graph.users.item.events.item.EventItemRequestBuilder.get(EventItemRequestBuilder.java:176)
at com.microsoft.graph.users.item.events.item.EventItemRequestBuilder.get(EventItemRequestBuilder.java:176)
.. internal code folllows here,  graphClient.me().events().byEventId(id).get(rc -> addStandardOutlookRequestHeaders(rc.headers));

Hi @etcoyvindf
Thank you for using the SDK and for reaching out.
Could you please provide which endpoint and operation you are using? The service should NOT be replying with html pages and we'll want to report that to the service team. If you could also include client request ids and timestamps, it'd be great.
Since Api exception derives from runtime exception, it's unchecked, which means there would be no impact to the method prototype.
That class was originally designed to represent errors coming back from the service itself, and one could argue this is a client exception not being able to parse the result due to lack of support for a given format. What would you think about adding a "ClientException" similar to the API exception?

@baywet Wondering whether the new ClientException type should be the default wrapper type we expose for all internal client-related exceptions. We still throw a bunch of IllegalArgumentExceptions and RuntimeExceptions in different layers of the Kiota code

For the runtime exceptions probably, for the argument validation, most likely not. Since it's a standard practice to use those exception types.


fix(cli): re-enable parallel scripts

What:
Fixes an issue with parallel scripts
Why:
PR #29 broke the following case:
p-s -p lint,test

How:
This commit fixes it by getting the scripts from the args first and seeing if there are any scripts available to run. If not, then we show the help message.
CC: @giladgo, @abhishekisnot, @rowanoulton, @tleunen, @jisaacks, @nkbt
Could one of you give this the green light and press the green button please?

LGTM

:shipit:

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

No coverage report found for master at 3c77bd1.

Powered by Codecov. Last updated by [3c77bd1...85fe58f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/kentcdodds/p-s/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/kentcdodds/p-s/compare/3c77bd1183317762216c110b0af8184a3af500e1...85fe58f22ecd833293716177f19bf68f3499d581?src=pr
[cc-pull]: https://codecov.io/gh/kentcdodds/p-s/pull/30?src=pr

Ok, I'm pretty certain this'll work now :-)

Sweet! The build is finally green!
Pinging again: @giladgo, @abhishekisnot, @rowanoulton, @tleunen, @jisaacks, @nkbt
Could one of you review this and merge if it looks good?

LGTM ;)

Nice.
Just got greenkeeper updates. All builds are green :)

Thanks for the update @nkbt!


Control state of the access to the HTTP

Fixes #140

[x] Add docs to README
[x] Check for thread-safe issues

Could I review it once the documentation is written on the README? Not easy to review and understand how it's supposed to work without it 😇

bors merge


fix: helm formatting and brackets

Fix helm formatting and brackets
Summary of Changes
Recent changes causing those errors, so I fixed them:
Error: parse error at (amundsen/templates/deployment-frontend.yaml:84): "-"
Error: parse error at (amundsen/templates/deployment-frontend.yaml:87): "-"
Error: parse error at (amundsen/templates/deployment-frontend.yaml:87): unexpected "}" in end

Tests
I didn't modify anything. Changing only helm formatting and after those changes helm was successfully deployed on my side.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

It was duplicated in #1391 and already merged.


Update requirements and systemd stuff

Can you also add systemctl daemon-reload to the %post and %postun scripts?

I don't think we need it for %postun:
[root@fermicloud032 ~]# rpm --eval %{systemd_postun}

systemctl daemon-reload >/dev/null 2>&1 || :

[root@fermicloud032 ~]# rpm --eval %{systemd_postun_with_restart}

systemctl daemon-reload >/dev/null 2>&1 || :
if [ $1 -ge 1 ] ; then
        # Package upgrade, not uninstall
        systemctl try-restart  >/dev/null 2>&1 || :
fi


Add HighResMIP and PMIP for FGOALS-f3-L

Add HighResMIP and PMIP for FGOALS-f3-L
@yongqiangyu
Fix #808

@yongqiangyu thanks for the update, your changes are now in place, take a peek at CMIP6_source_id.html and let me know of any further tweaks required


diff headers give error

When patching with https://github.com/maximeh/buildroot/blob/0400322f2d4e28faa98e31815921f54106aeb2e6/package/openssl/openssl-004-musl-termios.patch, it give the next error:
/home/piranna/Proyectos/NodeOS/node_modules/diff/lib/patch/parse.js:44
        throw new Error('Unknown line ' + (i + 1) + ' ' + JSON.stringify(diffs
              ^
Error: Unknown line 1 "http://rt.openssl.org/Ticket/Display.html?id=3123"
    at parseIndex (/home/piranna/Proyectos/NodeOS/node_modules/diff/lib/patch/parse.js:44:15)
    at Object.parsePatch (/home/piranna/Proyectos/NodeOS/node_modules/diff/lib/patch/parse.js:120:5)
    at applyPatch (/home/piranna/Proyectos/NodeOS/node_modules/diff/lib/patch/apply.js:13:22)
    at /home/piranna/Proyectos/NodeOS/node_modules/nodeos-barebones/scripts/preinstall:56:30
    at fs.js:334:14
    at FSReqWrap.oncomplete (fs.js:95:15)

Seems that after adding support for multiple indexes this got broken and now the first comment lines of the patch don't get ignored.

I've checked it and the error was introduced on version 2.0.0

I've relaxed the parser to ignore unexpected lines unless in strict mode.

Released in 2.1.2

Cool, thank you! :-)


Cache client partition results to save the repeated sort() calls

For more information, see #286

I pushed a new change that uses an array rather than a map.  I think not returning a copy does not let people modify the result, but it also saves on memory allocations.  I think this behavior should be ok(?)  Not sure why someone would depend upon that.

also saves on memory allocations

Yes it does, which is a good point.

Not sure why someone would depend upon that.

Neither am I (we don't, as far as I am aware) but I've learned to expect to be surprised :)

I've mentioned a few style nits, but I don't think there's anything really wrong with this as it stands.
@Shopify/kafka for an additional review please?

This looks good to me; just some comments about naming to make the intent clearer.

I've pushed one more version based upon the comments.  If there are more thoughts on names, it's ok if you want to pull my branch and make them before pushing upstream.

This looks great now. Thanks for your contribution!


Codecombat container doesn't start

Hello!
Could you please kindly check the Docker installation guide?
I try to run the container using Docker Compose, but it fails after start
Here is the log:
https://pastebin.com/3z085661
Thank you in advance!
Best regards,
Alex

I'm having a similar problem, please help us, because EVERYTHING I try fails.