JavaExec task should support executable JAR

Original issue: https://issues.gradle.org/browse/GRADLE-1274
Expected Behavior
Support equivalent of java -jar MyApp.jar. The user should be able to set a property that references the JAR file that defines the main class and classpath in the manifest file.
Current Behavior
Users have to set main = '-jar' as workaround which is not very useable.
Context
From a user:

Just seems that a powerful build tool should support a standard Java invocation mechanism.

Would be nice to see this feature get added as https://github.com/gradle/gradle/pull/2419 is open (although out-of-date right now). Executing fat JAR's and tasks using modules in Java 9 both require nonintuitive workaround that is confusing to read when coming back to a build script.

I updated #2419 so that it can be merged again.

Next month, my pull request will celebrate its first birthday. Could someone please tell me what is keeping it from being merged?

@stovocor I do have a suggestion about your PR and the implementation, but first I would like to address the passive aggressive nature of your birthday comment by reminding you of the Contributor Covenant Code of Conduct. Specifically: Other conduct which could reasonably be considered inappropriate in a professional setting
Now for the actual feedback...
Your portion of the PR checklist has uncompleted items and so that may be a part of why it has not been touched or reviewed. Also I was not able to quickly find any conversation between you and the Gradle team about the implementation plan of your proposed fix. And finally it is likely a very widely used api that is modified in the change set, so while all of the current test suite might be passing it isn't impossible down stream users would not be broken by the change. I would dare say your change set is in violation of the open/closed principle and interface segregation principle. Why not create a subclass of JavaExecTask with a name something like JarExecTask? At that point the change could be proposed as a pull request to gradle directly or published as a stand alone jar for your own usage until you can get a response from one of the project maintainers.

@trevjonez Thank you for the quick response!

but first I would like to address the passive aggressive nature of your birthday comment

I am truly sorry if this comment could be misunderstood as "passive aggressive", because this is not what I wanted to express. It was meant in a funny way while also expressing a bit of frustration because of the lacking progress for unknown reasons.
However, as a potential contributor I feel a bit left alone here. As far as I understand your comment, you do not know exactly why there has been no merge yet, but you provide some suggestions. If someone who could merge my PR decides it is not ready yet, it would be helpful to know why. Modifying the code again by myself without any advice about what is wrong with it, just because the PR gets old, seems not to be the right way.
Regarding the missing PR checklist: My change does not affect more than one public API or 20 files, so as far as I understand it, no link to the design spec is required. That is why I did not check the item. Should I remove it in this case instead?

You might try reaching out directly to a team member on the gradle-community slack group although even that might be futile this week based on this tweet. I am not sure if the checklist has anything to do with the silence, but I know I tend to ignore PR's at work until all the checkboxes get hit and status checks go green, or until someone DM's me to help/look/review.

I just realized that the pull request template has been changed in the meantime, removing the "design spec" item. So I am going to update my PR in the next days, making sure that all items are being checked. Thanks again for the support.

#2419 has been updated again. I performed a rebase against the current master and added documentation for the DSL reference. All items from the checklist are now done, in my opinion the PR is ready to be reviewed.

Hi all,
CC: @bmuschko, @mkobit, @trevjonez
You have probably solved this for yourself by now, I presume, @stovocor. But in case anybody else is interested I've made a JarExec plugin available to the community.
In addition to the core Jar executing capability, if you pass it an optional classpath and the name of a class with a main method, it'll also run plain old non-executable Jars too.
The plugin also adds a super useful jarhelper property to the build. With that helper, you can do things like retrieve resolved artifacts from Gradle's dependency cache. I discussed an example of such a use case in a recent Gradle forums discussion.
Please have a play-around with the plugin and let me know if it foots your bill?

A version of this'll be in 5.6 and works with JavaExec and project.javaexec {}.
If the main class is not defined for the JavaExec and the classpath is a single jar, we will try to execute java with an executable jar.


[Question]:

What is your question?
I'm tired now.

No!  ㅠㅠ


Comparing Arrays does (sometimes) not work. Problem with Objects?

Tested with 0.14.0-alpha.7 I got the following results:
import std.stdio;
import fluent.asserts;
import std.range;
import std.algorithm;

void main() {
	writeln("Edit source/app.d to start your project.");
}

unittest {
	auto a = new Object();
	a.should.equal(a); // Works

	int[] i = [1, 2];
	i.should.equal(i); // Works
	i.should.containOnly(i); // Works

	Object[] l = [new Object(), new Object()];
	l.should.equal(l); // Works
	l.should.containOnly(l); // There is no `object.Object[].object.Object[].containOnly` or `*[].*[].containOnly` registered to the assert operations.

	// Taken from  #85
	auto ll = iota(1,4).map!iota;
	ll.map!array.should.equal([[0], [0,1], [0,1,2]]);       // There is no `int[][].int[][].equal` or `*[][].*[][].equal` registered to the assert operations
	ll.map!array.should.containOnly([[0], [0,1], [0,1,2]]); // There is no `int[][].int[][].containOnly` or `*[][].*[][].containOnly` registered to the assert operations.
}

This doesn't make any sense to me. I think all assertions should pass without error. The last two tests are taken from #85 and should have worked earlier, but not any more.

Hi @skruppy,
I updated the way how the asserts work in 0.14 so this is not a cast problem, because now there are no casts. The problem comes from the fact that the assert operation is not defined for those types. I'll have a look at it.
This happens here: https://github.com/gedaiu/fluent-asserts/blob/master/source/fluentasserts/core/lifecycle.d#L31

I just made another release which enables those operations. Please let me know how it works for you.

Thanks for the quick fix and release :+1: . The MWE seems to work fine with the 0.14.0-alpha.8 release. Testing is currently not so easy because of #97. I will comment/close on this here issue if I worked around the other issues.

I tested a little bit further and I seems the examples above work, but associative arrays won't. They have worked prior version 0.14. See the following MWE (I hope this problem fits into the same bug report):
import std.stdio;
import fluent.asserts;

void main() {}

unittest {
	string[] l = ["1", "2"];
	assert(l == l); // works
	l.should.equal(l); // works

	string[string] al = ["k1": "v1", "k2": "v2"];
	assert(al == al); // works
	al.should.equal(al); // Fails: There is no `string[string].string[string].equal` or `*[string].*[string].equal` registered to the assert operations.
}


Output Tab and  Execution Log no longer exists within the Results tab area.

I am definitely frustrated that the  execution log has now moved to  the right side of the script window.  A couple releases back it was down in the Results tab section.  It is much easier to view when it is horizontal because it holds many columns which  get completely cut off in the  small right window and you have  to expand it covering your script window.
My suggestion is to allow these tabs to be moveable and place it in either location making in user adjustable.

thanks for the suggestion.
Seems to be the duplicate of #12534


fix(schema): Fix examples spec generation for OpenSpec 3

Closes: #1089

/rebase

/rebase

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

v6.13.1
GitHub release

Your semantic-release bot :package::rocket:


Options for TH regexes

Thanks for this awesome package, the TH regexes for type-safe capture groups are excellent!
I migrated from lens-regex-pcre, and noted that it is not possible to customize the Options for a TH regexes (in particular, SubGlobal), in contrast to e.g. _capturesOpt.
Would something like mkRegexQQ or mkRegexTraversalQQ be feasible?

Thanks!  Yeah, that's possible.
Although, "Opt" variants of current QQ functions would not work with SubGlobal.  That's for substitutions only.  Are you trying to do something like this (pseudocode):
myNonAffineTraversal = [_regexOpt SubGlobal|foo (...) baz|]

?

Yeah exactly, I stumbled upon the current traversal being affine.

OK.  I meant to specify SubGlobal is for PCRE2-native substitutions only, since it's free there.
I will open another issue about that since someone on Reddit needed that as well, and prioritize that one, since it's more of a blocker.

1.1.0 is released.  Traversals in the library are now global rather than affine.
I don't want to add mkRegexQQ right now but I may in the future if there is more of a need for it.  I'll keep this ticket open for now in case it becomes an issue again for you or someone else.

Perfect, #5 fixed my use case, but I agree that this can stay open as a low-priority issue. :+1:

Closing due to inactivity.
Final thoughts: A mkRegexQQ helper function that takes arbitrary Options is probably undesirable because of things like callouts[^1].  If a specific Option is desired for TH regexes that cannot be achieved using (?...) syntax in the pattern, let an issue be opened for it.
[^1]: Option may in the future be refactored to distinguish between safe and unsafe options at the type level using GADTs or something, or unsafe options may be removed altogether.


Add atscfg logic for Server Capabilities

Adds parent.config omitting parents which lack server capabilities required by Delivery Services.

[x] This PR is not related to any other Issue

Includes changelog.
Includes unit tests for config generation.
Documentation already exists for Server Capabilities; ATS config generation details are not currently documented.
Which Traffic Control components are affected by this PR?

Traffic Ops
Traffic Ops ORT

What is the best way to verify this PR?
Run unit tests.
Create a DS with Required Capabilities, ensure servers exist which lack those Capabilities, verify parent.config (both at the old TO endpoint, and by atstccfg) don't contain those servers. Verify parent.config DSes with Required Capabilities do contain servers which have the Required Capabilities.
If this is a bug fix, what versions of Traffic Control are affected?
Not a bug fix.
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4452/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4642/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4649/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4651/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4653/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4695/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4696/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4706/


pass variables to load_from_path

alternatively we could try to inject auto variables files into directory.

I don't know that I fully understand what we need to do here .. is this for overrides? does the current converter not pickup the auto files now? do you have an example of how you want to use this upstream in c7n-left?

its so a user can specify the correct variable files on the cli if their not in the root module as autoload, and the interpretation can then use those. ie resolving counts/dynamics/for loops to get an accurate list. lotsa of folks use separate var files per env, and then just pass them through on the cli, in this context we want to enable folks to do that via the c7n-left cli.


It doesn't work

It just constantly output some strange messages.

Thanks for using 1fichier-downloader!
Unfortunately I don't have a machine running macOS for testing available. So either

try change the script such that it works for you or
use another Linux distribution. The script works fine in Debian 12.


Capistrano::SCM::* depends on Capistrano/deploy

Editing the UPGRADING-3.7.md file to clarify that, when adding the following code to the Capfile:
require "capistrano/scm/git"
install_plugin Capistrano::SCM::Git

it should come after require 'capistrano/deploy'.
As reported in Stack Overflow and in an issue.

Thanks for the PR! Could you please edit the PR description so that it is more than just an unmodified version of the PR template? I.e. explain what this PR is about, and reference any related issue numbers.

Looks great! Thanks 🙇


Update property sort order

Updates the property-sort-order rule to ignore non standard CSS properties to allow custom properties as described in #302
Closes #302
Looking for possible feedback on this to see if we think this check should be enabled or disabled by an option flag or whether a whitelist would be preferable to this check.
DCO 1.1 Signed-off-by: Dan Purdy<EMAIL_ADDRESS>

I should also add some extra text to the documentation here.

This looks great :+1:
Thanks!

Going to also add a flag to this to disable this behaviour if needs be. Some might want their 'custom' properties to be ordered.

Torn on whether default value for ignore-custom-properties should be true or false.
On one hand I feel that custom values could be miss-spelt properties and therefore should not be ignored.
On the other a default action of ignoring anything custom would offer ease of use without either side having to take further actions to get this rule working for them.

I think it should remain as false. The default sort order is alphabetical and so it would provide a consistent approach straight out of the box where as it takes a little extra configuration and reading of the documentation to understand why you may not want that.

Fair point. Let's stick with false.


Check notebook runs in Binder

Hey @milanwilliams  - can you please follow this link to the SLTimer README and click on the "Launch Binder" button? That should bring up a Jupyter notebook view of the repository, from where you should be able to click through to your Demo notebook and execute it: can you try this please, and report back on any errors? The idea is to enable anyone on the web to run your notebook and reproduce the notebook's outputs in a live demo.

Hello @drphilmarshall
I received one error message, but I believe it is harmless. It popped up after running the script: %matplotlib inline. Here is the error message:
/home/main/anaconda2/lib/python2.7/site-packages/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
warnings.warn('Matplotlib is building the font cache using fc-list. This may take a moment.')
Other than that, there were no other problems!

Great - I got the same thing, it's just a warning. Thanks!
On Mon, Jun 13, 2016 at 1:50 PM, Milan M. Williams<EMAIL_ADDRESS>
wrote:

Hello @drphilmarshall https://github.com/drphilmarshall
I received one error message, but I believe it is harmless. It popped up
after running the script: %matplotlib inline. Here is the error message:
/home/main/anaconda2/lib/python2.7/site-packages/matplotlib/font_manager.py:273:
UserWarning: Matplotlib is building the font cache using fc-list. This may
take a moment.
warnings.warn('Matplotlib is building the font cache using fc-list. This
may take a moment.')
Other than that, there were no other problems!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DarkEnergyScienceCollaboration/SLTimer/issues/11#issuecomment-225704339,
or mute the thread
https://github.com/notifications/unsubscribe/AArY9y1TBleu9-iOHeUxOS70hijyt9IXks5qLcKkgaJpZM4I0oI4
.


[mle] add delay before orphan announce

mAnnounceDelay is about 80ms by default, seems too short to receive possible parent response for the third parent request before announce.
Here adopts the same kParentRequestReedTimeout delay.

Closing this as the delay design is intentional.


i cant run Geolocator.getCurrentPosition() in background

when i try to getCurrentPosition while application in background or terminated , it returns
' MissingPluginException(No implementation found for method getCurrentPosition on channel flutter.baseflow.com/geolocator)'

Facing the same issue. Have you found any solution for it?

Facing the same issue. Have you found any solution for it?

Duplicate of issue #1041, please have a look at this comment for the fix.

Unhandled Exception: MissingPluginException(No implementation found for method getCurrentPosition on channel flutter.baseflow.com/geolocator)
I am using geolocator package but this type of error coming when app is installed in android device how to solve it


[BUG] seLinux Being Disabled by Default Causing Logging Upgrade to Fail

Rancher Server Setup

Rancher version: 2.6.7-rc6
Installation option (Docker install/Helm Chart): HELM

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Proxy/Cert Details: byo-valid

Information about the Cluster

Kubernetes version: v1.23.7
Cluster Type (Local/Downstream): Downstream Imported AWS RKE2 cluster

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom): admin

Describe the bug
After upgrading rancher from 2.6.6. to 2.6.7-rc6, I upgraded logging to 3.17.7; The rancher-logging app on the installed apps page shows a FAILED tag. the rancher-logging-rke2-journalId-aggregators are in crashloop and after investigation it's because the seLinux value in the values.yaml file is set to false when it should be set to true
To Reproduce

Install previous version of logging in rke2 cluster
configure cluster output
configure cluster flow
verify that logs are flowing to output (eg, elasticsearch)
upgrade logging to latest 100.1.3-up3.17.7 [pointed charts repo to dev-2.6]
click the "Edit YAML" button on the second page of the install
search for global > seLinux > enabled > see it's false instead of true

Result
Upgrade fails; rke2 logging aggregator pods in crashloop
Expected Result
Upgrade succeeds and logs resume flowing to elasticsearch
Screenshots
App Failed State:

Crashloop:

seLinux Set to False:

Deployed After Setting seLinux to True:

@nickwsuse ok, keep me in the loop and we can put this back into Dashboard if @bmdepesa can account for why UI since this sounds like a chart setting.

@gaktive the problem is that the value set during the initial install is not persisted when upgrading. As we understand,  this is not part of the chart itself.

@bmdepesa just so I understand properly, at some point, the seLinux value is changed from the earlier chart default of false to true, and then upon upgrade, despite that value being false in values.yaml in the updated chart, it should remain true?
Based on the linked issue, there's this comment: https://github.com/rancher/rancher/issues/38519#issuecomment-1208364910

Upon upgrading the chart the value was reset to false causing a permission error with the aggregator when it was trying to access a privileged directory.

@catherineluse do you know where this could happen in the UI?

Yes, in that case it is a UI issue. There is UI code that attempts to take all the non-default values from the existing chart version, then pre-apply them to the values.yaml when you go to upgrade the chart, because it assumes you would want to persist all the changes you made before. (https://github.com/rancher/dashboard/blob/master/shell/pages/c/_cluster/apps/charts/install.vue#L242)

The logic for how this works in the UI is pretty generic though. I wonder if the formatting or structure of the values.yaml changed between chart versions. If so, the UI might not be able to accurately apply the same value from the old values.yaml into the new values.yaml.

Thanks for the all the clarification. Transferring back to dashboard, though we need to assess if we can quickly get this in for 2.6.7.


Fix imports of generated go proto bindings

Fixes
#39744
Summary
After running the following in accordance with the installation instructions for go:
$ go get -d github.com/tensorflow/tensorflow/tensorflow/go
$ go generate github.com/tensorflow/tensorflow/tensorflow/go/op
$ go test github.com/tensorflow/tensorflow/tensorflow/go

The first command generates error message unrelated to this PR, a fix for which is ready to pull. That error message is relatively low impact and should not impact the functionality of the library after the go generate command is run. However, the third command generates the following error:
$ go test github.com/tensorflow/tensorflow/tensorflow/go
../../../go/src/github.com/tensorflow/tensorflow/tensorflow/go/saved_model.go:25:2: cannot find package "github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto" in any of:
	/home/abls/go/src/github.com/tensorflow/tensorflow/tensorflow/go/vendor/github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto (vendor tree)
	/usr/local/go/src/github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto (from $GOROOT)
	/home/abls/go/src/github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto (from $GOPATH)

This is caused by a bad import that seemed to be from an accident during a refactor. The problematic import that occurs in several files is:
import corepb "github.com/tensorflow/tensorflow/tensorflow/go/core/protobuf/for_core_protos_go_proto"

This line is supposed to point to the generated files, however the generated files are actually placed in github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto due to the following line in several proto files in tensorflow/core/protobuf read by protoc during generation:
option go_package = "github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto";

The fix is simply to correct the import lines to point to the actual location of the files.

Needed to change this a bit to match the BUILD rule, which is required for bazel compatibility:
https://github.com/tensorflow/tensorflow/commit/e5e495db7bee77cd0fd5dda3b06bd743cbcf1ef8

Needed to change this a bit to match the BUILD rule, which is required for bazel compatibility:
e5e495d

@jhseu Any idea when it will be in a release?

Someone submitted a fix and no one seems to have reviewed this...just awful
Thanks for your effort, I will make an installation of your branch <3


[py2py3] RuntimeError with unsupported type in Configuration.py

Impact of the bug
WMCore
Describe the bug
The Configuration module seems not to like the bytes complex data type in python3 unit tests, see full traceback in [1]
I believe there is some planned python3 work to be done in the Configuration module, which might be exactly what is reported in this issue.
How to reproduce it
nosetests test/python/WMCore_t/WMSpec_t/WMWorkload_t.py

Expected behavior
The Configuration module should be able to deal with strings in a consistent way between python2 and python3, including this possible byte string vs unicode string.
Additional context and error message
[1]
======================================================================
ERROR: _testBlockCloseSettings_
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dmwm/wmcore_unittest/WMCore/test/python/WMCore_t/WMSpec_t/WMWorkload_t.py", line 2031, in testBlockCloseSettings
    testWorkload = self.makeTestWorkload()[0]
  File "/home/dmwm/wmcore_unittest/WMCore/test/python/WMCore_t/WMSpec_t/WMWorkload_t.py", line 136, in makeTestWorkload
    harvestTaskCMSSWHelper.setDataProcessingConfig("pp", "dqmHarvesting",
  File "/home/dmwm/wmcore_unittest/WMCore/src/python/WMCore/WMSpec/Steps/Templates/CMSSW.py", line 159, in setDataProcessingConfig
    self.data.application.configuration.pickledarguments = pickle.dumps(args)
  File "/home/dmwm/wmcore_unittest/WMCore/src/python/WMCore/Configuration.py", line 146, in __setattr__
    self._complexTypeCheck(name, value)
  File "/home/dmwm/wmcore_unittest/WMCore/src/python/WMCore/Configuration.py", line 124, in _complexTypeCheck
    raise RuntimeError(msg)
RuntimeError: Not supported type in sequence:<class 'bytes'>
for name: pickledarguments and value: b'\x80\x04\x95X\x00\x00\x00\x00\x00\x00\x00}\x94(\x8c\tglobalTag\x94\x8c\x05Bogus\x94\x8c\x0bdatasetName\x94\x8c /bogusPrimary/bogusProcessed/DQM\x94\x8c\trunNumber\x94K\x00u.'
Added to WMAgent Configuration.Use ConfigurationEx to skip checks on config params

@mapellidario Dario, if this is exactly what you have been working on lately, then you might be the best person to work on this. Otherwise, I'd assign it to @goughes to make sure he also gets some of these ;)

I would prefer to take this, since I already have #9974 opened a long time ago and seen some ways that Configuration can silently fail on us.
However, this is likely going to require some team-wide discussion.

Actually, I rebased #9974 on top of master and it fixes this issue!

Sicen we merged #9974, we may need to close this one, too. On dmwm/WMCore - master the unittest
nosetests test/python/WMCore_t/WMSpec_t/WMWorkload_t.py:WMWorkloadTest.testBlockCloseSettings

is not failing for me. I still have 3 unittests failing in nosetests test/python/WMCore_t/WMSpec_t/WMWorkload_t.py, but i think that they were already there. @amaltaro what do you think?

If those 3 unit tests fail with the same reason as reported in this issue, then this would be an opportunity to have a PR fixing/closing it :-D Otherwise we just close it through the recently merged #9974 PR.

The test file test/python/WMCore_t/WMSpec_t/WMWorkload_t.py is related to slice 17 #10329. It' s already in the list of test files that I should check for slice 17, which is available here, and as you can see in #10422 there is some work we have to do before getting to slice 17 :sweat_smile:
So, if this is not a problem for you, in order not to open an issue for every failing unittest, I would stick to my schedule, postpone the work on this remaining 3 failing unittests, and close this issue because the specific unittest that was mentioned here has been fixed.

Anyway, two of the failing unittests are related to the usual assertItemsEquals -> assertCountEqual, while one is related to pickle and strings (ugh :sob: ), problems for future us!
[dmwm@70c3f941ccaf WMCore]$ nosetests test/python/WMCore_t/WMSpec_t/WMWorkload_t.py
....E........E.........E.........
======================================================================
ERROR: test persistency
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dmwm/wmcore_unittest/WMCore/test/python/WMCore_t/WMSpec_t/WMWorkload_t.py", line 294, in testC
    workload.save(self.persistFile)
  File "/home/dmwm/wmcore_unittest/WMCore/src/python/WMCore/WMSpec/Persistency.py", line 49, in save
    pickle.dump(self.data, handle)
TypeError: write() argument must be str, not bytes

======================================================================
ERROR: Test Overrides setter and getter methods
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dmwm/wmcore_unittest/WMCore/test/python/WMCore_t/WMSpec_t/WMWorkload_t.py", line 2112, in testOverrides
    self.assertItemsEqual(overrideArgs, result)
AttributeError: 'WMWorkloadTest' object has no attribute 'assertItemsEqual'

======================================================================
ERROR: _testUpdatingLFNAndDatasetStepChain_
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dmwm/wmcore_unittest/WMCore/test/python/WMCore_t/WMSpec_t/WMWorkload_t.py", line 871, in testUpdatingLFNAndDatasetStepChain
    self.assertItemsEqual(testWorkload.getAcquisitionEra(), acqEra)
AttributeError: 'WMWorkloadTest' object has no attribute 'assertItemsEqual'

----------------------------------------------------------------------
Ran 33 tests in 0.169s

FAILED (errors=3)
[dmwm@70c3f941ccaf WMCore]$

Thanks for this explanation, Dario. Yes, I agree.
This issue has been fixed with #9974, thus closing it.


[main] Upgrade to latest dependencies

After eventing 1.11 released -creydr
/cc knative/func-approvers
/assign knative/func-approvers
Produced by: knative-sandbox/knobots/actions/update-deps

@lance do we need this before release cut?

@lance do we need this before release cut?

Yes we do

@lance @matejvasek I've rerun the hack's codegen script. It seemed to fix go.sum diff reported by one of the checks. Hopefully it will go through now.
We should be able to cut the release branch afterwards.
/approve
/lgtm


sensitivity question

If I run the same model twice the sensitivities should be the same both times right?

yes.

as I thought, thanks. Might have seen that not be true over the weekend. will dig into that next week.


Add screenshots of components

Currently, the only way to know the styles look is to grab this crate and try it out. It'd be great if there were some examples of how each component looked like how Angular Material does.
It would be great if there were demos like that too but that would require having a website built with these styles which is a lot of work, especially considering that this project in its early stages. So including screenshots in docs on docs.rs or in this repo would be a welcome improvement.

It is right that there is not a published website at the moment which it is planned in the future however you can see the visual examples running the documentation page, you can find how do it here

@hamza1311 from today there is already a website where you can find all the visual examples, thanks for you suggestion. This is the link https://yewstyles.spielrs.tech . I close this issuce


Back button must be pressed twice to leave PDP

Type: minor
Describe the bug
When navigating to a product the URL updates twice and requires two back-button presses to leave. The interim page has an incorrect title.
To Reproduce
Steps to reproduce the behavior:

Start 3.1.0 production build
Click on a product from homepage
Press the back button
Press the back button again

Expected behavior
Only one back-button press is required to navigate back to the homepage after viewing product details.
Screenshots

Desktop (please complete the following information):

OS: Arch Linux
Browser: Ungoogled Chromium
Version: Version 84.0.4147.89 (Official Build) Arch Linux (64-bit)

Additional context
Not a 3.2.0 regression but an existing issue.

The Routing probably works fine, but what happens is that first we route to the product page, and next set the url to point directly to that variant: https://github.com/reactioncommerce/example-storefront/blob/f8dbd9e6a4800b3b3d87d193bfae88f989b35b90/components/ProductDetail/ProductDetail.js#L77
In v3.1 we used Router.pushRoute with { replace: true }, from next-routes, now we use  the built inRouter.replace and not Router.push.
I would have expected both implementations to not create another entry in the session history, as described here:
https://nextjs.org/docs/api-reference/next/router#routerreplace
I wonder what's wrong her and if this is a nextjs bug.

I noticed this issue as well on mobile specifically on the latest reaction commerce 3.8. If you are on product page and swipe to go page to previous page it just shows same product page again until you do it again finally actually taking you back to previous page.


Remove empty space between variables and punctuation

Rationale / Begründung
After some Django commands, an empty space is generated which separates the output from the following punctuation.
Source:

Output:

This behaviour was observed after {% render_*** ... %} and {% endif %}.
Affected
Users, Managers
Minimal functionality
If possible, the white space should not be inserted.
References / Verweise
https://rdmocat.aip.de/projects/13/views/8/

Hi @Zack-83, is it only after render_set_value_inline_list

No, also after render_value.


Allowing ingress from Kubernetes API via service selector doesn't work

Expected Behavior
I'd expect the following policy to allow ingress from the Kubernetes API to a mutating webhook service:
apiVersion: crd.projectcalico.org/v1
kind: NetworkPolicy
metadata:
  name: allow-webhook-ingress
  namespace: webhook
spec:
  selector: app = 'webhook'
  types:
    - Ingress
  ingress:
    - action: Allow
      protocol: TCP
      source:
        services:
          name: kubernetes
          namespace: default
      destination:
        ports:
          - 9443

Current Behavior
Requests from the Kubernetes API keep getting blocked.
Steps to Reproduce (for bugs)

Apply the above policy to a mutating webhook service
Trigger a request to the webhook service

Your Environment

Calico version: 3.21.6
Orchestrator version (e.g. kubernetes, mesos, rkt): Kubernetes 1.23.5
Operating System and version: AKS

@Tolsto I'd expect that to work as well, unless your Kubernetes API endpoints are behind some sort of NAT or in general do not match the endpoints listed in kubectl get endpointslices.

apiVersion: crd.projectcalico.org/v1

I'd be remiss if I didn't call out this issue as well: https://github.com/projectcalico/calico/issues/6412

❯ kubectl get endpointslices
NAME         ADDRESSTYPE   PORTS   ENDPOINTS     AGE
kubernetes   IPv4          443     <IP_ADDRESS>   38d

Output is what I'd expect. It's a vanilla AKS setup with Calico.

Ah, I suspect the problem is that on AKS the API server(s) is hidden behind a load balancer, so outbound traffic is sent to <IP_ADDRESS>, but inbound traffic from the API server(s) arrive on different addresses. Might be worth it to see if you can catch one of the packets as it arrives and see what it's source address is.

@Tolsto did you happen to get a chance to test @caseydavenport 's theory?

Closing for inactivity.  Please reopen if needed.


additional modifications on to_moltype()

added additional tests and wrapped up dev

please address #286 as part of this, it will simplify the code


Improvements to create-svelte

change the initial message: remove the "stop, aplha" note and replace it with a "beware, beta" note
add eslint setup option
add prettier setup option
add link to the svelte adders project by @babichjacob
introduce type checks to the project

nice


Implement A2HS feature

Implement add to home screen (A2HS) for the webapp
please check

Try to add the app to the home screen on your mobile. If you find any  bugs, add a task or issue for it.
As of now it is working well on android. Due to the limitations from ios it is hard to use web bluetooth.
Yet to check when device isn't connected to internet

[ ] app functionality
[ ] Data acquisition


Cmd wasm dump

This change is

Review status: 0 of 4 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


FEAT: Region heatmap

Provide ability for regions color to be adjusted based on value, like a heatmap. Include the ability to customise the heatmap scale.

This feature has been added as part of #7

This feature is merged to main


[SILOptimizer] Use BitfieldRef instead of invalidating SSAPrunedLiveness.

SSAPrunedLiveness::invalidate is used as though it reset the state of the instance it is called on.  Clients then reuse the instance with the expectation that it has been reset.  But since it has not been reset, this results in unexpected liveness results.
rdar://108627366

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test source compatibility


Support user provided Exasol docker-db version

Situation
ETAJ should use the user provided Exasol docker version. When running CI or locally, we usually set the docker-db versions (e.g, as environment variable). It would help if the ETAJ uses this version of docker-db.
For example the one set by the com.exasol.dockerdb.image parameter:
-Dcom.exasol.dockerdb.image=${{ matrix.docker_db_version }}

Acceptance Criteria

Uses user provided docker-db version

exasol-test-setup-abstraction-java delegates to exasol-testcontainers which in turn uses system property com.exasol.dockerdb.image. So setting this system property will allow the user to define the Exasol DB version to use.
Ci build uses this mechanism to test with Exasol 7.1 and 8.
We also use this in https://github.com/exasol/exasol-test-setup-abstraction-server/blob/main/go-client/server.go#L24-L32
Closing as already implemented.


Grade School should return a sorted Array with tuples instead of a dictionary

For the test func testSortSchool()  we want to return an object that is sorted by keys and values of the dictionary. It seems like the test is working too hard and may not even be right, since the keys in a Swift Dictionary are not ordered.
Instead of
 let expected = [             3 : ["Kyle"],             4 : [ "Christopher", "Jennifer" ],             6 : [ "Kareem"]         ]
We should return
let expected = [ (3,["Kyle"]), (4,["Christopher", "Jennifer"] ),     (6, ["Kareem"])]
Also tuples are the swift way to return groups of multiple values.
I am still new to swift :) but I was at WWDC and showed my code to some devs at the swift lab. This was their suggestion.

Seems reasonable.  There are certainly a lack of Tuples in the current Swift exercises

No objection from me. Go for it!

OK.

I love tuples but I think current the implementation using the swift's associative container seems more appropriate. https://en.wikipedia.org/wiki/Associative_array
If a  [(key,value)] is required to be sorted, you could start by extending the Dictionary Type followed by in place sort or similar.
extension Dictionary{

    func toKeyValuePairArray() -> Array<(Key, Value)>{
        return Array(zip(self.keys, self.values))
}

Probably the most suited container for this would be a NSOrderedSet
https://developer.apple.com/library/prerelease/ios/documentation/Foundation/Reference/NSOrderedSet_Class/index.html


Pipelines: PublishBuildArtifacts -> PublishPipelineArtifact

The PublishPipelineArtifact task is significantly faster than the PublishBuildArtifacts task.
Docs for reference: https://learn.microsoft.com/en-us/azure/devops/pipelines/artifacts/pipeline-artifacts
Exact perf numbers TBD. Will edit the description once I have them.

/azp run

Our pipelines are on OneBranch which automatically injects PublishPipelineArtifact tasks for us on a dedicated folder.

@kythant I'm still seeing for pipelines like this one: https://microsoft.visualstudio.com/ProjectReunion/_build/results?buildId=96849120&view=logs&j=6eacd493-a123-5df2-c54d-e0f4ac9b7cf6&t=f9e13036-6881-585b-8291-749bca2d5d7f
That the build artifacts task is being used:

cc @kythant to review


Fix The EliminateSortUnderSubquery rule and the EliminateOrderByConstant rule share the same ruleType #31403

cherry pick from #31403

run buildall


Zig test - run single-thread (only)

Run the fibonacci sample C in comparison to Zig test . A discrepancy in runtime between the zig test and fib-c is perceived.
Note: This comparison occurs if you put the same values into the fib(n) function. The zig test has smaller numbers to better pass the CI test.
Reference

https://github.com/ziglang/zig/issues/15953

Although zig test run sequentially, it only displays the output after finishing all calculations. Different from the sample application.


[th] Wrong spelling

Hi, using the demo and in the program, the function convert number 1 to Thai text wrong, both with NUMBERTEXT() and MONEYTEXT().
Example: 11,111.00
Current conversion (wrong) : หนึ่งหมื่น เอ็ด พัน เอ็ด ร้อยสิบเอ็ด
Correct conversion should be : หนึ่งหมื่น หนึ่ง พัน หนึ่ง ร้อยสิบเอ็ด
Problem:
The function converts every "1" after the first digit to "เอ็ด" except "10" digit.
Why it's wrong:
"1" in Thai number can be "หนึ่ง" or "เอ็ด" but most of the time it will be "หนึ่ง"
Please see here for reference and more info.

I looked around the source and found this commit  that tried to fix this exact issue. I don't know if it's fixed before but currently the issue still persists.

Okay, the issue was fixed in Feb 19. This issue should be closed now.
Just FYI, however, the LibreOffice extension and the release were still using Oct 18 code. Anyone not building from latest code will experience this issue.


Cannot overwrite a text without deleting it

Is there anyway to turn the mask overwrite option on like IMask library?
https://imask.js.org/guide.html
thanks

Hi @lamnouar ,
Have a read https://github.com/RobinHerbots/Inputmask#insertmode


fix: fix varbinary type error

Which issue does this PR close?
Closes #637
Rationale for this change
Fix the problem of table creation failure mentioned in issue.
What changes are included in this PR?

Fix SqlDataType convert error.
Add create table test with VARBINARY type.

Are there any user-facing changes?
User could create table with VARBINARY type normally.
How does this change test
Pass integration tests.

Codecov Report

Merging #653 (fa61a6b) into main (9e7733b) will increase coverage by 0.08%.
 Consider uploading reports for the commit 9ccdcf8 to get more accurate results

@@            Coverage Diff             @@
##             main     #653      +/-   ##
==========================================
+ Coverage   66.84%   66.92%   +0.08%
==========================================
  Files         283      283
  Lines       44006    43962      -44
==========================================
+ Hits        29414    29422       +8
+ Misses      14592    14540      -52

Impacted Files
Coverage Δ

analytic_engine/src/compaction/scheduler.rs
68.09% <ø> (ø)

analytic_engine/src/instance/flush_compaction.rs
66.41% <0.00%> (-0.22%)
:arrow_down:

analytic_engine/src/sampler.rs
99.33% <0.00%> (ø)

analytic_engine/src/sst/file.rs
76.08% <ø> (-0.09%)
:arrow_down:

analytic_engine/src/table_options.rs
64.12% <ø> (ø)

catalog_impls/src/table_based.rs
83.58% <0.00%> (ø)

catalog_impls/src/volatile.rs
0.00% <0.00%> (ø)

cluster/src/config.rs
77.77% <ø> (ø)

cluster/src/shard_tables_cache.rs
0.00% <0.00%> (ø)

common_types/src/datum.rs
50.69% <0.00%> (ø)

... and 109 more

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


ignite-cli is unable to create new app in windows

Describe the bug
Does ignite-cli only works in linux and mac os ?
PS E:\react-native> npx ignite-cli@latest new PizzaApp13 --yes

                                               .;;,.    .clll.
                                              .clll,    ,llll.           ...
                                               .,;,.    ;lllc'.,.   .';cllllc,
     .',,.                           ...      .'',,. .:lllllllll. .;lllc:;clll,
    .llll,                    .,...;cllll:.   ,llll, .llllllc,,'  clll:'.,cllc.
    ,llll'       ......;::.  'llcclllllllll'  ,llll,   .llll.    ,lllllllll:'
    ,llll.    .;clllllclll.  'llllc,..,llll,  :llll.   'llll.    ,lllc,''..
    ;llll.   :llllc:cllllc.  ,llll.   .llll, .lllll.   ,lllc...  ,lll:...';:.
   .llllc.  ;lll;.   :lll,   ;lll;    'llll, .llllc.   ,llllcll' .:lllllllll,
   .llll,  'lll:    .clll,   :lll,    ,llll. 'llll,    'lllllcc'   ';::::;'.
   .llll,  ,lll:    .llll'  .llll,    ;llll. ,llll,     .....             .',..
   ,llll.  'llll:,,:cllll.  .lllc.    .''..  ... ....',;         ..',,:::;'..
   ,llll.   'clllllllllll.     .....        ..,::cllllc.   .',:clllc;,'.
   :llll.     ',;,'.;lllc...,;:clll;  ..',:cllllllllll:,;clllllc;,'.
   llll:        ...'cllllllllllllll::cllllllc;,',cllllllllc;'..
   llll,   ..,clllllllll::,,.,lllllllc:,'...    :lllllc,..
   c:;,.  .cllllc:;:llll.    ,ll:;,..          ;ll:,..
          :lll,.  .clll,     ..               .,..
          :lll:,,cllll,
          .cllllllllc'
           .'cllll:.

    █ Creating PizzaApp13 using Ignite 8.10.1
    █ Powered by  ∞ Infinite Red  (https://infinite.red)
    █ Package Manager: npm
    █ Bundle identifier: com.pizzaapp13
    █ Path: E:\react-native\PizzaApp13
    ────────────────────────────────────────────────────

    🖨  3D-printing a new React Native app
    🎨 Getting those last few details perfect
    🧶 Installing npm dependencies (wow these are heavy)
C:\Users\chankruze\AppData\Local\npm-cache\_npx\bec2839915ba8fc3\node_modules\gluegun\build\index.js:15
    throw up;
    ^

Error: Command failed with exit code 1: which unix2dos
'which' is not recognized as an internal or external command,
operable program or batch file.
    at makeError (C:\Users\chankruze\AppData\Local\npm-cache\_npx\bec2839915ba8fc3\node_modules\execa\lib\error.js:60:11)
    at handlePromise (C:\Users\chankruze\AppData\Local\npm-cache\_npx\bec2839915ba8fc3\node_modules\execa\index.js:118:26)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  shortMessage: 'Command failed with exit code 1: which unix2dos',
  command: 'which unix2dos',
  escapedCommand: 'which unix2dos',
  exitCode: 1,
  signal: undefined,
  signalDescription: undefined,
  stdout: '',
  stderr: "'which' is not recognized as an internal or external command,\r\n" +
    'operable program or batch file.',
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}

Node.js v20.9.0
PS E:\react-native>

Ignite version
8.10.1
Additional info
PS E:\react-native> npx ignite-cli doctor
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
System
  platform           win32
  arch               x64
  cpu                12 cores       Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  directory          react-native   E:\react-native

JavaScript
  node               20.9.0       D:\nodejs\node.EXE
  npm                10.1.0       D:\nodejs\npm.CMD
  yarn               -            not installed
  pnpm               -            not installed
  expo               -            not installed

Ignite
  ignite-cli         8.10.1       C:\Users\chankruze\AppData\Local\npm-cache\_npx\e31027f3785124a8\node_modules\.bin\ignite.CMD
  ignite src         build        C:\Users\chankruze\AppData\Local\npm-cache\_npx\e31027f3785124a8\node_modules\ignite-cli\build

Android
  java               19.0.2       D:\jdk-19.0.2\bin\java.EXE
  android home       -            D:\Android\Sdk

Tools
  git                git version 2.36.1.windows.1   D:\Git\cmd\git.EXE

Hi, can you try npx ignite-cli@next?

This is fixed in npx ignite-cli@next


Add documentation for implementing PRPL

It would be great to add some documentation for how to implement PRPL with this template.
There are some existing docs for async components and lazy loading in the Vue and VueRouter docs. But I'm not sure if thats all thats required, or if I also need to set up the web worker to preload etc.

While there are always opportunities to manually tweak settings, this template is PRP-friendly by default, with lazy-loading left as an exercise for the developer.

Push is handled via <link rel="preload'> (HTTP/2 push is out of scope for this project.)
Render is obviously being handled by Vue.
Precache is handled by registering a the service worker that's generated in the production builds.
Lazy-loading has been brought up in the past (https://github.com/vuejs-templates/pwa/pull/7, https://github.com/vuejs-templates/pwa/pull/69) and the decision was reached that it wouldn't be possible to ship a default configuration that was optimal for all use cases.


Added bower.json

Simplify the installation with bower

No, see https://github.com/Leaflet/Leaflet.heat/issues/16#issuecomment-155834094


Handling for non-json response

Describe the change
Since the original error response is included in the body field of openai.RequestError, even when the response is not in JSON format, it would be more appropriate to return openai.RequestError directly, rather than returning an errorString. This approach ensures that the original error details are preserved within the body field.
Describe your solution
Removed the condition that checked if the response was in a non-JSON format and returned an errorString with the original error response. Now, the original error response will be directly included in the body field of openai.RequestError.

@sashabaranov Can you please look into this?

Thank you for the PR!

Thanks @sashabaranov. Can you please review this again?


#0: upsample fix for undefined behavior on multi-devices run.

Ticket
Link to Github Issue
Problem description
Provide context for the problem.
What's changed
Describe the approach used to solve the problem.
Summarize the changes made and its impact.
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Merging it..


Question

I dl 0.7, where I can find the .json with all my pal ?
With dump button in previous version, the app create a .json
Is it possible to do it with new version ?
And edit .json, then save by clicking on "Save Change To File"

Looks like no, new version can not export/import pals, it can only make a copy of pal in the same save. No more moving pals to other saves.

Yes, im waiting for owner answer, to know if he is gonna add again or not

Thx u !


How to use additional community plugins

Hi Ole, thank you very much for the wonderful work, I really appreciate your effort!
I am trying to set up my digital garden, and I found that the plugin "obsidian-latex" (https://github.com/xldenis/obsidian-latex) does not work by default. Is there a way to use additional plugins in connection with "obsidian-digital-garden"?

Unfortunately there is no way for users to configure arbitrary plugins to work together with the digital garden plugin. Some plugins is however possible to support by adding it as a feature. I can look into if it's easy to add support for this. Do you have an example of what you expected to work? The plugin currently supports mathjax, but I guess you use some notation not included in mathjax?

I see. The plugin I am referring to is "obsidian-latex" which is used to be able to have LaTeX macros directly in Obsidian. However, I realized I can use a text-expander (like Espanso) to obtain what I need without using the plugin. Therefore,  I think there is no rush to look into this specific plugin so that you may focus on other more pressing issues ;-) Thank you very much for the help!

I recently stumbled upon this amazing math-related plugin: mathpad. It allows, among other things, to plot functions and embed the result in a note. I think it would be amazing to be able to use this plugin in connection with Digitalgarden.

That looks interesting! I will look into it when I have the time. I think that would be a really cool addition. Thanks for the suggestion!


Hyperliquid - Failed to submit market order due to "Order has zero size."

Describe the bug
for monitoring purpose
Ran tests on Hyperliquid connector for couple of days then review logs. Observed that there are instances of failed to submit market orders when connector is trying to place a close order using v2 strategy
2024-01-19 16:25:58,509 - 615066 - hummingbot.smart_components.executors.position_executor.position_executor - INFO - Placing close order --> Filled amount: 0 | TP Partial execution: 0
2024-01-19 16:25:59,702 - 615066 - hummingbot.connector.derivative.hyperliquid_perpetual.hyperliquid_perpetual_derivative.HyperliquidPerpetualDerivative - NETWORK - Error submitting sell MARKET order to Hyperliquid_perpetual for 0.00 SOL-USD 84.8780000000000001136868377216160297393798828125.
Traceback (most recent call last):
  File "/home/test01/development/hummingbot/connector/exchange_py_base.py", line 452, in _create_order
    await self._place_order_and_process_update(order=order, **kwargs,)
  File "/home/test01/development/hummingbot/connector/exchange_py_base.py", line 469, in _place_order_and_process_update
    exchange_order_id, update_timestamp = await self._place_order(
  File "/home/test01/development/hummingbot/connector/derivative/hyperliquid_perpetual/hyperliquid_perpetual_derivative.py", line 393, in _place_order
    raise IOError(f"Error submitting order {order_id}: {o_order_result['error']}")
OSError: Error submitting order 0xc88b1c9d64aa088b41388e2a24ad8d98: Order has zero size.
2024-01-19 16:25:59,714 - 615066 - hummingbot.connector.derivative.hyperliquid_perpetual.hyperliquid_perpetual_derivative.HyperliquidPerpetualDerivative - WARNING - Failed to submit SELL order to Hyperliquid_perpetual. Check API key and network connection.
2024-01-19 16:25:59,719 - 615066 - hummingbot.strategy.script_strategy_base - INFO - Creating SOL-USD sell order: price: NaN amount: 0.
2024-01-19 16:25:59,720 - 615066 - hummingbot.smart_components.executors.position_executor.position_executor - INFO - Placing close order --> Filled amount: 0 | TP Partial execution: 0
2024-01-19 16:25:59,720 - 615066 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "order_id": "0xc88b1c9d64aa088b41388e2a24ad8d98", "order_type": "OrderType.MARKET", "event_name": "MarketOrderFailureEvent", "event_source": "hyperliquid_perpetual"}
2024-01-19 16:25:59,720 - 615066 - hummingbot.connector.client_order_tracker - INFO - Order 0xc88b1c9d64aa088b41388e2a24ad8d98 has failed. Order Update: OrderUpdate(trading_pair='SOL-USD', update_timestamp=1705681559.0, new_state=<OrderState.FAILED: 6>, client_order_id='0xc88b1c9d64aa088b41388e2a24ad8d98', exchange_order_id=None, misc_updates=None)

Steps to reproduce

Add hyperliquid_perpetual and use v2 strategy
On this issue, we are using this script
Run the bot and observe behavior when placing close orders

Release version
dev-1.24.0
Type of installation
Source
Attach required files
logs_hyperliquid_a.log
logs_hyperliquid_a.log.2024-01-19.log
logs_hyperliquid_a.log.2024-01-20.log

I see that the strategy sends a market order with a price of NaN and a amount of 0, but the exchange's market order does not allow the amount to be 0.
I think it's normal

Added max retries on position executor for this type of behavior


Actions log invocations is empty

Describe the bug
When the fragments debug mode is enabled the produced data contains empty invocations array:
"invocations": []

To Reproduce
Steps to reproduce the behavior:

Debug the AllActionsDebugDataTest.
Check the HTTP response body.

Expected behavior
Invocations should contain action log details.

As we've established, it is expected in cases when the action is not a behaviour, or does not provide logs. All cases present in the use case have been covered.

@tomaszmichalak can we close this now?


🐛 | Spotify error - unable to contact server.

Spotify errors with unable to contact server.
installed phoniebox onto buster with the one line script.
checked the status of mopidy. Error shown.

It seems I get logged in, and play lists are found etc.
I expected Spotify to connect with my credentials.
Further information that might help
I can post more details if necessary. I'm certain my credentials are correct. If no one else has this error, then it must be on my side.

WiFi issue. Works now.


update Random Number wiki page

include information about the (new) RNG counter

https://github.com/carothersc/ROSS/wiki/Random-Numbers has been updated to reflect the new functionality.


get_bounding_box returning numbers magnitudes off

For a gds cell with a max element at ~ 2000, cell.get_bounding_box() returns: [[-0.285  0.465]
[ 1.04   3.635]].

@McCrearyD Could you provide a minimal example showing the problem? I cannot replicate what you describe...

Load this gds file into gdspy and on the cell, call "get_bounding_box()". This will return seemingly random floating point numbers.

@McCrearyD The GDSII file is loaded correctly.
What you're seeing are coordinates expressed in micrometers (default for GDSII files), and you probably wanted nanometers.
Please take a look at units in GDSII and GdsLibrary for more information.
In short, you can get load this library in nanometers by using:
import gdspy

gds = gdspy.GdsLibrary(infile="test.gds", unit=1e-9, precision=1e-12, units='convert')
c = gds.top_level()[0]
print(c.get_bounding_box())

Good call, I was unaware gdspy automatically changed it's units.
Thank you.


Write 3.0 changelog

But I would expect something like:

Rewrote whole system
List of neat features of new system
Any content changes, like new chapters and interactives.

details to come...

3.0.0

Release date: TBC
Downloads: Source downloads are available on GitHub

Changelog:

Rebuild the Field Guide to use a django system (see the github milestone here):

Modernise navigation features
Introduce search functionality for English chapters
Improve glossary features

Improve chapters and chapter content:

Chapters are now split up into sections
General content, grammar and spelling fixes
Introduce new chapter sections:

User experience by Hayley van Waas for the Human Computer Interaction chapter.
General purpose compression by Hayley van Waas for the Coding - Compression chapter.

Improve interactives:

Update and modernise (in some cases rewrite from scratch) existing interactives.
Introduce interactive thumbnail generator
Introduce many 'uninteractives' – allowing image text to be translated.
Introduce new interactives:

Rename 2d-arrow-manipulations to 2d-shape-manipulations
Alarm Timer
Algorithm Timer
Binary Cards
Braille Alphabet
City Trip
Dictionary Compression
Dot combinations
LZSS compression
LZW Compression
Pixel Grid

Remove now-obsolete interactives:

MD5-hash
ncea-guide-selector
ziv-lempel-coding

Redesign homepage
Update README
Update contributors page
Improve licencing structure to make it easier to find and navigate on GitHub
Introduce initial German and Spanish translations (WIP)


Switch btp logins ?

Hello @qmacro  - Thanks for the blog post and the Friday's session. I was wondering as to how we can switch the btp logins between different trial accounts when the config file holds a default value. I tried to use --sso with the login and then it defaults the subdomain ( of course ) which is not good to switch. Then I wondered if I can have multiple config files named as per my convenience and the btp cli gives me the option to choose one of the files to load the credentials. I guess that is not there yet.
Any thoughts ?

Hey @srinivas2210 thanks for the great question. In fact, I'd been wondering about this too. There's nothing that is directly supported yet in the btp CLI itself, but I have a home-grown solution that I use; perhaps it's worth sharing that on an upcoming live stream - what do you think? If you want a quick look beforehand, have a look in the btp directory in my dotfiles: https://github.com/qmacro/dotfiles/tree/main/scripts/btp. Hope that helps for now!


Troubles with using/filling/understanding the template and template instructions

Hi @malmans2,
I open up this issue to collect feedback from evaluators on JN-related stuff. Could you possibly conduct anyone facing trouble with the template and instructions here? Inserting a disclaimer on the home page of the repo or something similar?

All set. I've added a note in the README, in the template instructions, and in the template notebook.

Probably for me to do: we need to make it more clear that the 'graphical abstract' (first image before the methods) should have a caption (whether embedded in the image, a formal figure caption, or text below an inserted image).

Hi, that looks almost correct (see example below, you need to keep the 'attachment:', and don't need to manually add the figure numbering), and the height is not fixed so please vary it to work for your image (I should make that clear in the template also). Thanks!
---
height: 500px
---
The figure shows a sample  of the ECMWF monthly temperature bias for Western North America as a function of the starting month and of the valid month of the forecast. More details on how to interpret the results are available below in the notebook.

Okay got it!
Thank you!


Allow backtrace information to be optional

There appears to be no way to disable backtrace information embedding into a program. There are some cases where bundling filenames and other information needed for backtraces is not desired. Two cases I can think of immediately is wanting to provide as little information as possible in proprietary products. The other case is where you need the final binary to be as small as possible.
There is an old set of patches that added this that seem to be stale now.. Maybe they can be used as a starting point? https://github.com/japaric/std-with-cargo And there was also some discussion about this on the rust-internals IRC, but nothing seemed to come out of it: https://botbot.me/mozilla/rust-internals/2015-08-20/?msg=47709853&page=4

:+1: Always bothered me.

What backtrace information? We use the symbol names or debug info, just like C and C++.
If you're compiling an executable in release mode, all you have to do is strip symbols (the same as a proprietary C or C++ project would have to do).

What backtrace information? We use the symbol names or debug info, just like C and C++.
If you're compiling an executable in release mode, all you have to do is strip symbols (the same as a proprietary C or C++ project would have to do).

It would make sense to provide a way to compile it without the symbol names & debug info. Futhermore, panics will show file name and line number (though this is not rustc, but libstd).

@Ticki To compile without debug info: remove -g. Cargo.toml has a way to turn it off even for debug mode, while release mode doesn't come with debug info.
Compiling without symbol names doesn't make sense: they're certainly mandatory for libraries, and executables cannot drop symbol names until linking has finished.
EDIT: gcc appears to have an -s option for stripping the symbol names after linking. I guess we could add that?

To compile without debug info: remove -g. Cargo.toml has a way to turn it off even for debug mode, while release mode doesn't come with debug info.

That'll still show line number and file name when panicking.

gcc appears to have an -s option for stripping the symbol names after linking. I guess we could add that?

Yeah, that was what I thought of. It -s often comes very handy.

Panics are not "backtrace information", they're file!() and line!() macros - having a way to obscure those would be useful, but you'd have to recompile the entire libstd in that mode if you want to get rid of all of them (they're just string literals as far as the post-expansion compiler is concerned).

Also, if you want a solution now for the "strip symbols part", you can use strip instead of having it built-in like gcc -s.

Panics are not "backtrace information", they're file!() and line!() macros - having a way to obscure those would be useful, but you'd have to recompile the entire libstd in that mode if you want to get rid of all of them

Yes, you're right. They're not backtrace information, they're debug information. These could be removed in release mode (by using #[cfg(build = "debug")] in libstd).

Update: The crates in rust-lang/rust has been Cargo-ified.
I have a lightly tested fork of rust-lang/rust that makes the backtraces (RUST_BACKTRACE / libbacktrace) optional via a Cargo feature (only usable with the newer "rustbuild" build system). But even if that lands upstream, it won't be super easy to use because (a) our binary releases would still ship with backtraces enabled, (b) even if you are OK with bootstrapping rustc along with the std crate, rustbuild doesn't expose (AFAIK cc @alexcrichton) an easy way to customize the Cargo features of the std crate and (c) trying to directly build the std crate using Cargo (as in cd src/libstd && cargo build) doesn't work for some combination of Cargo features like enabling jemalloc. Also (c) is annoying because you have to package the Cargo output into a sysroot before you can use it.
TL;DR To implement this we have to decide how to expose this (optional) feature to end users.

@japaric https://github.com/rust-lang/rfcs/pull/1133 while not completely solving the problems you mention is a big step in that direction.

rustbuild doesn't expose an easy way to customize the Cargo features of the std crate

Actually, there is a config.toml that rustbuild can use. I've send a PR implementing optional backtraces and exposing them via that config.toml: rust-lang/rust#35051

What's the current status of this?

There's one more good reason for this, that hasn't been mentioned yet. I want to disable the message because re-running the panicking binary is not always a good thing to suggest to end-users.

You can use a panic hook to disable that message, right?

Panic hook won't remove the unwanted data from the executable.


Error 400 occurred stating "updateOrgBadRequest &{Code:400...Please enter valid domain address." Post user tries to create new signing entity name from Contributor console DEV

Error 400 occurred stating "updateOrgBadRequest &{Code:400...Please enter valid domain address." Post user tries to create new signing entity name from Contributor console DEV

https://images.zenhubusercontent.com/194341141/33f7b1fe-5142-4201-a8db-3024f4b22989/recording__47.mp4
Error Message Received:
[PUT /v1/orgs/{salesforceID}][400] updateOrgBadRequest &{Code:400 Message:organization-service - service.UpdateOrganization: UpdateOrganization.Queryx: insert response err: FIELD_CUSTOM_VALIDATION_EXCEPTION: Please enter valid domain address. XRequestID:c8c0b7e0-6559-11eb-8d2a-216bc2325840}
Payload
{"companyName":"Google Inc","companyWebsite":"google.com","signingEntityName":"Google India","userEmail":"maan_vaidya@rediffmail.com","createdBy":"937b80ca-e4fe-4486-b563-945d4f4baff7"}
End Point:
https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/user/937b80ca-e4fe-4486-b563-945d4f4baff7/company

This error might not be relevant anymore with the latest changes because it was happening when we were adding a new signing entity (which was calling the upda method on org service). I think we need to retest this after the signing entity removal changes are in place.

This flow is no more applicable as signing entity changes have been reverted and system is back to old behavior.
Moreover when  Amol and Manish tried create org today, they didn't face this error. So, closed this issue.


Add unit tests

Closes #5

For the integration test, we could use a Docker image with FSL installed, I guess? I am not sure if we want to go to the effort of getting that to pass when our goal is to drop FSL entirely.

I'd run the integration test locally until we completely drop FSL. I don't think the docker is worth the effort.

Local checks sound good to me.

Right now we cannot write any other unit tests without using FSL.
Shall we merge this and add new unit tests as we replace FSL with Python code?

Can you add the other outputs from MELODIC so we can test the feature functions? The melodic_FTmix, melodic_mix, and 4D thresholded component image (which I think AROMA creates from the individual 3D thresholded component images) should be sufficient.

Can you add the other outputs from MELODIC so we can test the feature functions? The melodic_FTmix, melodic_mix, and 4D thresholded component image (which I think AROMA creates from the individual 3D thresholded component images) should be sufficient.

I think I'm only missing the 4D thresholded component image. I'll add it now.

Ultimately, I'd like to move the test data into a subfolder of tests/, with its own tests-specific utils.py containing a get_test_data_path() function, but that doesn't have to go in this PR if you don't want to do it now.

Ultimately, I'd like to move the test data into a subfolder of tests/, with its own tests-specific utils.py containing a get_test_data_path() function, but that doesn't have to go in this PR if you don't want to do it now.

What would you like to store there? The original workflow results?

What would you like to store there? The original workflow results?

Exactly. Plus any other data we plan to test against, but which isn't useful for the package as a whole. At the moment that's just the original workflow results, but we could add more stuff there over time.

Will add it to this PR before I switch off for the day.


app crash OSH__SHM_APP_MAPP_ERR

Required information
Operating system:
E.g. Ubuntu 18.04 LTS
Compiler version:
E.g. GCC 7.4.0
Observed result or behaviour:
A clear and precise description of the observed result.
Expected result or behaviour:
What do you expect to happen?
/home/worker/sgf/iceoryx/iceoryx_utils/source/posix_wrapper/shared_memory_object/shared_memory.cpp:149 { bool iox::posix::SharedMemory::open(int, mode_t, uint64_t) }  :::  [ 2 ]  No such file or directory
/home/worker/sgf/iceoryx/iceoryx_utils/source/posix_wrapper/shared_memory_object/shared_memory.cpp:149 { bool iox::posix::SharedMemory::open(int, mode_t, uint64_t) }  :::  [ 2 ]  No such file or directory
Shared Memory does not exist.
Unable to create shared memory with the following properties [ name = /iceoryx_mgmt, access mode = AccessMode::READ_WRITE, ownership = OwnerShip::OPEN_EXISTING, mode = 0000, sizeInBytes = 59111944 ]
Unable to create SharedMemoryObject since we could not acquire a SharedMemory resource
Unable to create a shared memory object with the following properties [ name = /iceoryx_mgmt, sizeInBytes = 59111944, access mode = AccessMode::READ_WRITE, ownership = OwnerShip::OPEN_EXISTING, baseAddressHint = 0, permissions = 0000 ]
2022-10-27 10:43:29.818 [ Error ]: ICEORYX error! POSH__SHM_APP_MAPP_ERR

@sgf201 Thanks for the report. The bug looks like the issue you already reported here: #1761 and I think they may have the same origin.
From the first look it seems that iox-roudi was started and then something deleted the shared memory resource /dev/shm/iceoryx_mgmt which we require for the zero copy inter process communication. Is it possible that you call ipcrm -all after iox-roudi was started?

@sgf201 Closing this as duplicate of #1761. Feel free to re-open if the issue persists.


Support for installing packages from environment.yml files

Problem
At the moment it can be confusing that packages must be specified explicitly in a XeusPythonEnv.packages list.
Folks familiar with Binder might expect dependencies installed in a build environment to also be available in the kernel.
The fact that there could be a mismatch between the frontend extension and the package in the kernel is the issue (for example with widgets).
This is already mentioned in the documentation in the case of the Pyodide kernel: https://jupyterlite.readthedocs.io/en/latest/howto/configure/simple_extensions.html#the-case-of-jupyter-widgets-and-custom-renderers
But also applies for the jupyterlite-xeus-python kernel.
Proposed Solution
For a site deployer, having a single environment.yml would make things more convenient.
It's not clear yet how this should be handle and whether the empack step in the xeus-python addon should accept a environment.yml as is.
The main issue will probably be packages required for the build (for example jupyterlab or jupyter-server to index content) don't need to be installed in the kernel.
Additional context
cc @nthiery since we discussed this use case over a video call

Thanks for summarizing this item of our discussion!
Just to reformulate from the user's perspective: it's a simple natural mental
model to have two clearly separated environments:

the environment used to build the site
the execution environment for interactive use of the material on the site;
That's the usual model when using e.g. jupyter-book.

On the other hand, the separation between front-end/server/kernel gets
technical, and is best left as implementation detail that the user need not
worry about if at all possible.
Of course, there may be cases where the build environment is a superset of
the execution environment, e.g. when the build involves executing notebooks.


use new Content Block update type when publishing

Once the work in this PR [1] is done to introduce
the new type we should be able to start using it
in Whitehall - we should not see any functionality change yet, but the new update types should be
being logged
[1] https://github.com/alphagov/publishing-api/pull/2904
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

@Harriethw Is this ready for review now? (Barring the conflicts, natch)

@pezholio yep it should be, just trying to run locally again to double check... having a bit of issue with Docker this morning


Error occured at: src/parser.c - read_log - 2669

When I want to use goaccess to analyse my log file, the following error occurs. How to solve this problem?
goaccess <access.log> -o  real-time-html
GoAccess - version 1.2 - Dec 21 2017 06:35:41
Config file: /usr/local/etc/goaccess.conf
Fatal error has occurred
Error occured at: src/parser.c - read_log - 2669
Unable to open the specified log file. No such file or directory

Sounds like it can't find your access log.
Have you made sure you have the right path to your access log and that you have the right permission to read the log?


1.9.1 -> 1.10.1 regression: "index_scheduler: internal: decoding failed."

Hello there,
I recently upgraded my 1.9.1 instance on Ubuntu 24.04 to 1.10.1, and all of a sudden the instance started to complain that something wrong occurred.
What I exactly did:

Ensured nothing was about to be written to the instance
Copied data dir to a safe place
Created a dump through the REST API
Stopped the instance
Upgraded
Started the instance manually by importing the dump
Once done, stopped the temporary instance, then asked systemd to launch 1.10.1.
So far so good. REST API was OK, search was OK and so on.

Then I ran my usual background task to import fresh data, but after a few minutes, the server started to return 500 errors, I couldn't even list indexes.
Here's what a service meilisearch status said:
● meilisearch.service - Meilisearch
     Loaded: loaded (/etc/systemd/system/meilisearch.service; enabled; preset: enabled)
     Active: active (running) since Wed 2024-09-11 05:52:25 UTC; 4s ago
   Main PID: 608372 (meilisearch)
      Tasks: 9 (limit: 2320)
     Memory: 6.4M (peak: 6.7M)
        CPU: 21ms
     CGroup: /system.slice/meilisearch.service
             └─608372 /usr/bin/meilisearch --config-file-path /etc/meilisearch.toml

Sep 11 05:52:26 meilisearch[608372]: 2024-09-11T05:52:26.164716Z ERROR index_scheduler: internal: decoding failed.
Sep 11 05:52:26 meilisearch[608372]: 2024-09-11T05:52:26.456139Z ERROR HTTP request{method=POST host="xxx.xxx.xxx.xxx>
Sep 11 05:52:26 meilisearch[608372]: 2024-09-11T05:52:26.456339Z  INFO HTTP request{method=POST host="xxx.xxx.xxx.xxx>
Sep 11 05:52:27 meilisearch[608372]: 2024-09-11T05:52:27.165206Z ERROR index_scheduler: internal: decoding failed.
Sep 11 05:52:28 meilisearch[608372]: 2024-09-11T05:52:28.165595Z ERROR index_scheduler: internal: decoding failed.
Sep 11 05:52:28 meilisearch[608372]: 2024-09-11T05:52:28.665288Z ERROR HTTP request{method=POST host="xxxxxxxx.xyz>
Sep 11 05:52:28 meilisearch[608372]: 2024-09-11T05:52:28.665864Z  INFO HTTP request{method=POST host="xxxxxxxx.xyz>
Sep 11 05:52:28 meilisearch[608372]: 2024-09-11T05:52:28.671386Z ERROR HTTP request{method=POST host="xxxxxxxx.xyz>
Sep 11 05:52:28 meilisearch[608372]: 2024-09-11T05:52:28.671538Z  INFO HTTP request{method=POST host="xxxxxxxx.xyz>
Sep 11 05:52:29 meilisearch[608372]: 2024-09-11T05:52:29.165880Z ERROR index_scheduler: internal: decoding failed.

I did the whole process a second time without having more luck, so I reverted to 1.9.1. I ran my ETL to import fresh data and everything worked as expected.
Unfortunately, this is sensitive data that I cannot share, so I could not help find a bug by providing a dataset. But is there some places I should look on my machine that could help find clues or more details about why the new instance crashed that way?
Thank you,
Ben

Hello @bpolaszek 👋

I did the whole process a second time without having more luck, so I reverted to 1.9.1. I ran my ETL to import fresh data and everything worked as expected.

I am sorry to hear that. So, at least you have a working version of Meilisearch and didn't lose any data.

Unfortunately, this is sensitive data that I cannot share, so I could not help find a bug by providing a dataset. But is there some places I should look on my machine that could help find clues or more details about why the new instance crashed that way?

That's very unfortunate because it seems to be a runtime issue. It doesn't seem to come from the environment or the key-value store but straight from Meilisearch. I would like to know where you got Meilisearch. Have you compiled it by yourself? Used the official Docker image?
Have a nice day 🌞

Hi @Kerollmops,
Thank you for taking some time to answer me.
Looks like "past me" did dirty things such as snapshot importing routines, and "new me" just forgot about that 🙄
Except a snapshot cannot be imported from a previous Meilisearch version. This led to Meilisearch crash and I didn't get context to figure out why.
I retried earlier yesterday and everything worked perfectly.
Sorry for wasting your time 😓


URL rewrite does not happen for minified CSS files

Are you sure this is not a problem with your configuration? (ask on the
Google Group) Yes

Minify commit/version: 2.1.5
PHP version: 5.3.10

What steps will reproduce the problem?
1. Include min.css file in groupsConfig or as individual file.

Expected output:
URLrewrite should happen for "<filename>.min.css" files too.
Or atleast there should be some option to allow the URLrewrite for
"<filename>.min.css" files.

Actual output:
URLrewrite is not happening for minified file.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 23 Jul 2012 at 2:16

Fixed in 335800947b31

Original comment by<EMAIL_ADDRESS>on 30 Sep 2012 at 10:46

Changed state: Verified


Use immutable state for data processing

This should help avoid bugs when state is mistakenly reused and mutated later.
Examples of useful libraries:
https://github.com/immutable-js/immutable-js
https://github.com/kitten/extendable-immutable
https://github.com/immerjs/immer

Fixed in 9c1cd27


Perf timestamps

Reminder that commits are nicely split if someone wants to review this.

Huge thanks for figuring out a good strategy to avoid inconsistencies.
Also the general timer capabilities are super interesting! (Timeline? :smiley: )


mv: fix the output of an error message

adjust a GNU test

Should make tests/mv/dup-source.sh pass

Congrats! The gnu test tests/mv/dup-source is no longer failing!

Great :)


deepmerge dependency causes a warning

[1] Warning: ...\node_modules\ngx-translate-multi-http-loader\dist\multi-http-loader.js depends on 'deepmerge'. CommonJS or AMD dependencies can cause optimization bailouts.

You should check this page of the build section from the angular documentation: https://angular.io/guide/build#configuring-commonjs-dependencies, to allow "lodash" as a commonJS dependency.

I know that I can allow this dependency.
The documentation says this :

Is any update on that? I do not know if could be possible to replace deepmerge lib by a esm one. Thanks!

@ebussieres, @acuevas91  Done on the latest v9.1.0 version!
Since npm v^8.19.0 does now  auto import peerDependencies, I didn't added in the README.md file, but if you wish you could also run an npm i deepmerge-ts in your project.
@ojacquemart FYI, the library isn't using loadash since it's way to heavy, but deepmerge-ts, which should be faster :)
Let me know if it works the way it should.


Chronocube app

Hi, I had my app on the list of open source projects, but I'd love to have it on the apps list (it's still open source tho).
Description: Chronocube is a webapp (also available as a cross-platform desktop app, thanks to Electron) that lets you time your Rubik's Cube solves and organize them, scramble the cube, save the scrambles, export/import solves... All with a minimal and beautiful UI.
URL: You can check the homepage, the source code, the webapp and the desktop apps.
Notes: Although I included the link and icon in the commit, you can also download a full-size icon

Yeah, definitely!


Can't import Bestiary - Troglodyte

Can't import statblock:
Troglodyte      CR 1
XP 400
Male troglodyte (Pathfinder RPG Bestiary 267)
CE Medium humanoid (reptilian)
Init -1; Senses darkvision 90 ft.; Perception +0
Aura stench (30 ft., DC 13, 10 rounds)
Defense
AC 15, touch 9, flat-footed 15 (-1 Dex, +6 natural)
hp 13 (2d8+4)
Fort +7, Ref -1, Will +0
Offense
Speed 30 ft.
Melee club +2 (1d6+1) or
bite -3 (1d4), 2 claws -3 (1d4)
Ranged javelin +0 (1d6+1)
Statistics
Str 12, Dex 9, Con 14, Int 8, Wis 11, Cha 11
Base Atk +1; CMB +2; CMD 11
Feats Great Fortitude
Skills Stealth +5 (+9 in rocky areas); Racial Modifiers +4 Stealth, +4 Stealth in rocky areas
Languages Draconic
Other Gear club, javelin (3)
Ecology
Environment any underground
Organization solitary, pair, clutch (3-6), squad (7-12 plus 1 cleric or druid of 3rd level and 1-2 monitor lizards), or band (20-80 plus 20% noncombatants, 1 chieftain of 3rd-6th level, 1-3 clerics or druids of 3rd-6th level, and 3-13 monitor lizards)
Treasure npc gear (club, 3 javelins, other treasure)
Special Abilities
Darkvision (90 feet) You can see in the dark (black and white only).
Stench (30 ft., 10 rounds, DC 13) Living creatures in aura's range are sickened for listed duration (Fort neg).

I got this statblock from Hero Lab.  You might benefit from retesting with aonprd.com.  Sometimes Hero Lab truncates some of the descriptions.
I used the latest Pathfinder system and your latest module to test this.


Feature/third task corrections

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1038

Relevant Lines:
1038

💛  - Coveralls


Everybody, who can do the app on the shelf?

Everybody, who can do the app on the shelf? Like this https://itunes.apple.com/cn/app/id1293167709? Mt=8
Https://itunes.apple.com/cn/app/id1317244973? Mt=8

This appears to be spam.


add support for isArrayAccessible

Merged as: https://github.com/phpstan/phpstan-webmozart-assert/commit/c5009369a492219d3fb0cc6828aeca283fe3baaf
Thank you.


Add ParallelAxis tick Format

i think this feature will be so usefull
if we want to display two diffrent units in same time ( F° and C°  for example)

Sounds interesting indeed but I don't see how you would implement it.
There would be a callback on the second axis to convert the values ?
Could you give us a code example about how it would look like ?

<Plot {...DEFAULT_PLOT_CONFIG}>

<ParallelAxis   id= 'x' label='Fahrenheit'  tickFormat ={(x)=>(x*9/5 + 32)} />


unit tests fails but works fine on BTP / on prem: TypeError: Cannot read properties of undefined (reading 'set')

After downporting a unit test seems to fail with the following error:
TypeError: Cannot read properties of undefined (reading 'set')

The unit tests work fine on the BTP ABAP Environment / on premise
https://github.com/dominikpanzer/cacamber-BDD-for-ABAP/actions/runs/4992812508/jobs/8941031900

fixed in https://github.com/abaplint/transpiler/pull/1223 as part of the transpiler project

ok.  new error - a new issue or did I do something wrong?
https://github.com/dominikpanzer/cacamber-BDD-for-ABAP/actions/runs/4993039575/jobs/8941582995

I've opened that error as https://github.com/open-abap/open-abap-core/issues/674

I had a look at it, but I have no idea how to solve that myself. doesn't seem to be possible without knowledge of the kernel-functions?

I'll fix it, sometime..
close this issue and move to https://github.com/open-abap/open-abap-core/issues/674 ?

sure


DBZ-1831 Support MongoDB Oplog operations as config

https://issues.redhat.com/browse/DBZ-1831
This PR supersedes https://github.com/debezium/debezium/pull/1294 and is based on the work that is in PR https://github.com/debezium/debezium/pull/1316.
@gunnarmorling Would it make more sense for me to cherry-pick the final commit into PR #1294 or is it sufficient that we merely rebase this once PR #1294 has been merged to master?

Ok, I took the liberty and abandoned best practices for once by force-pushing to this branch just the commit this is about, as #1294 has been merged to master. I hope it's ok.

Ok, pushed another commit. Awaiting CI.

@blcksrx Apllied, thanks!


Please clarify queries related to acct.proto file

Could you please clarify below queries related to acct.proto file. Thank you.

As per the proto file, ACK will be sent for a group of accounting records.  Since, the connection is over a grpc channel that ensures delivery of the messages, what is the necessity for sending these ACKs?

In addition to the ACK sent for a group of accounting records, there is also an ACK sent at least every 60 seconds and the connection gets reset if an ACK is not received for 120 seconds. What is the rationale behind sending this keepalive ACK since there is already ACKs at lower layers (grpc/tcp)?

When a record request is received with last message received timestamp, all the messages after the timestamp need to be sent out. This happens when the connection is alive and not during connection initiation. Are there any scenarios where collector asks for the past timestamp because there are chances of retransmission of same packets by accounting service?

When a record request is received with zero time, all the messages in the system need to be transmitted. What is the number of messages that we are looking to store in the system?

Is it expected to store the messages in the system even after the message is successfully sent out?

For accounting push, when the config gets added should the accounting service send a dummy record with just the timestamp to the collector. What will be the response from the collector for this dummy message with timestamp?

Required functionality could be achieved with accounting pull alone and there seems to be no necessity for accounting push. For accounting push, accounting service needs to do a dial out and collector IP/port config, TLS certificate params has to be added under accounting service config and accounting service has to read the config and initiate a dial out connection which might be unnecessary.

I think that this is resolved and another can be opened if there are remaining questions.

closing.


Bring back Quarkus native test in PR

As the Quarkus native checks are now stable, I'd propose to bring them back into the normal PR execution instead of running them on request (via label) or nightly. The time to execute on latest GH runners machine is about 1 hour, in line with the rest of the test. Any objection?

+1


[Feature]: enable pwa

How the project can be improved?
Good to have web and mobile apps
What are the acceptance criteria?

[ ]

Additional information
No response
Code of Conduct

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

Hello,
would you please explain more? thanks

hello sir, I mean to say, vuestorefront has a feature of pwa to take the advantage of service worker caching, push notifications, responsive design etc.
Please add these feacutres too in a vue-prestashop frontend

of course it supports.
Vue Storefront uses @nuxtjs/pwa module which adds support for PWA functionalities:
https://docs.vuestorefront.io/v2/getting-started/theme.html
Vue Storefront is mainly a NuxtJs application, that we can benefit from all Nuxtjs features.
You can also read more about the PWA:
https://pwa.nuxtjs.org/

Thanks sir,

@ashishkpaul welcome.


Notification successfully received in Teams , yet GitAction step fails

Hi @neonidian ,
Firstly thanks for your GitHub Action utility, I found it to be the most functional of the Teams notification utilities I have looked at recently.
I am encountering a bit of a strange problem. The utility is successfully and correctly publishing to my desired Teams channel, but the step is failing with the following errors::

But the message is successfully sent and appears where I expect it within my MS Teams channel:

GitAction snippet:
- name: Notify Teams channel of the test results
  uses: neonidian/teams-notify-build-status@v4
  with:
    # Can send to multiple channels in MS Teams
    webhookUrl: |
      ${{ secrets.TEAMS_INCOMING_WEBHOOK_URL }}
    # Specify a title header
    title: Test Result Summary for REDACTED on the REDACTED test environment
    titleBackgroundColor: ${{ env.STATUS_COLOR }}
    status: ${{ env.STATUS_COLOR }}
    message: >-
      💡 Total: ${{ env.TOTAL }} | ✅ Passed: ${{ env.PASS }} | ❌ Failed: ${{ env.FAIL }} | ⏭ Skipped: ${{ env.SKIP }}
  env: # Enable actor labels and buttons using environment variables
    SHOULD_DISPLAY_ACTOR_LABEL: true
    SHOULD_DISPLAY_VIEW_RUN_BUTTON: true
    SHOULD_DISPLAY_VIEW_COMMIT_BUTTON: true

For the moment, I have simply added the following to my build step, which stops the entire job from failing: continue-on-error: true

Thanks for using this action and reporting this issue. A successful response from Teams server should have the text "1" but null is returned. I suspect this could be because of this bug that Microsoft has acknowledged and are working on a fix.
My guess would be that though Teams does render it in the UI, there is something going wrong with the server not responding with 1 due to this bug

Thanks for the info @neonidian hopefully they fix this at some point!

I will monitor the bug report. I don't think this issue needs to stay open.


FormatException (FormatException: Неожиданный символ (в символе 1) ‰PNG ^ )

I bought the fluxstore program, connects via API to the Wordpress site. The program works, but on the emulator it gives such an error.
FormatException (FormatException: Unexpected character (at character 1) ‰PNG ^ )
I deleted the entire image on the site but it didn't help. Deleted the flutter project and reopened it. There are many solutions on the Internet, but I could not change my code in any way, I'm afraid to break the whole application. The program works on the web, but when I open it on the emulator, it gives the following error.
I deleted the entire image on the site but it didn't help. Deleted the flutter project and reopened it. There are many solutions on the Internet, but I could not change my code in any way, I'm afraid to break the whole application. The program works on the web, but when I open it on the emulator, it gives the following error

@ravanmusayev2020
The goal of this Github repo is not to give assistance on personal code or the whole package/plugin project, but to improve Flutter framework/ Flutter 1st plugin/package reporting bugs or feature requests.
It seems your question is best suited for support channels like StackOverflow, so please ask the same there for solutions if any.
If you are proposing a change, please use the appropriate new issue template and fill out the requested details so that we can address this issue properly.
Closing for now.


Refactoring: Update Field definition

[x] rename label into name

[x] introduce second type parameter

[ ] add getter and setters

Fix included in #372