Add Component Matchers for Various Drop Configurations

For #34, #41, and #42, add in the following matchers

Matcher
Mob Drop
Block Drop
Mob Shed
Target Release
Done

minLootingLevel (int)
Yes

1.3.9
Done

maxLootingLevel (int)
Yes

1.3.9
Done

minedWith/killedWith (Item minedWith/killedWith, boolean consume, float damagePer, NBT nbt, boolean fuzzyNbt)
Yes
Yes

1.3.12
Done

itemInOffhand/attackerHasOffhand (Item itemInOffhand/attackerHasOffhand, boolean consumeOffhand, float damageOffhandPer, NBT nbtOffhand, boolean fuzzyNbtOffand)
Yes
Yes

1.3.12
Done

itemInInventory/playerHasInventory (Item itemInInventory/playerHasInventory , boolean consumeInventory, float damageInventoryPer, NBT nbtInventory, boolean fuzzyNbtInventory)
Yes
Yes

1.3.12
Done

inDimension (list: dimension IDs)
Yes
Yes
Yes
1.3.12
Done

inBiome (list: biomes)
Yes
Yes
Yes
1.3.12
Done

inBiomeType (list: types)
Yes
Yes
Yes
1.3.12
Done

minMobHealth (float)

Yes
1.3.12
Done

maxMobHealth (float)

Yes
1.3.12
Done

damageOnShed (float damageOnShed, String sheddingDamageType, boolean sheddingDamageIsUnblockable, boolean sheddingDamageIsAbsolute, boolean sheddingDamageIsCreative)

Yes
1.3.12
Done

minLightLevel (int)
Yes
Yes
Yes
1.3.12
Done

maxLightLevel (int)
Yes
Yes
Yes
1.3.12
Done

targetHasPotionEffect: (potion ID)
Yes

Yes
1.3.12
Done

playerHasPotionEffect: (potion ID) / attackerHasPotionEffect (for mob drops)
Yes
Yes

1.3.12
Done

silkTouch (boolean)

Yes

1.3.12
Done

minedByPlayer (boolean)

Yes

1.3.12
Done

minedByExplosion (boolean)

Yes

1.3.12
Done

minFortuneLevel (int)

Yes

1.3.12
Done

maxFortuneLevel (int)

Yes

1.3.12
Done

minHeight (int)
Yes
Yes
Yes
1.3.12
Done

maxHeight (int)
Yes
Yes
Yes
1.3.12
Done

chance (chance %)
Yes
Yes
Yes
1.3.12
Done

not (any other matchers)
Yes
Yes
Yes
1.3.12
Done

or (any other matchers)
Yes
Yes
Yes
1.3.12
Done

and (any other matchers)
Yes
Yes
Yes
1.3.12
Done

aboveBlock (block with nbt)
Yes
Yes
Yes
1.3.13
Done

aroundMobs (List what, int min, int max, int dist)
Yes
Yes
Yes
1.3.13
Done

aroundBlocks (List what, int min, int max, int dist)
Yes
Yes
Yes
1.3.13
Done

minTimeOfDay (int)
Yes
Yes
Yes
1.3.13
Done

maxTimeOfDay (int)
Yes
Yes
Yes
1.3.13
Done

moonPhase (phase)
Yes
Yes
Yes
1.3.13
Done

insideBlock: (block with nbt)
Yes

Yes
1.3.13
Done

mobHasMainhand (Item mobHasMainhand, NBT mobMainhandNbt, boolean mobMainhandFuzzyNbt)
Yes

Yes
1.3.13
Done

mobHasOffhand (Item mobHasOffhand, NBT mobOffhandNbt, boolean mobOffhandFuzzyNbt)
Yes

Yes
1.3.13
Done

damagePlayer (float damagePlayer, String playerDamageType, boolean playerDamageIsUnblockable, boolean playerDamageIsAbsolute, boolean playerDamageIsCreative)
Yes
Yes

1.3.13
Done

minScore (String scoreboard ID, int min)
Yes
Yes

1.3.13
Done

maxScore (String scoreboard ID, int max)
Yes
Yes

1.3.13
Done

targetRiding

Yes
1.3.14

isRaining
Yes
Yes
Yes
1.3.14

isStorming
Yes
Yes
Yes
1.3.14

Seasons.inSeason (list: season)
Yes
Yes
Yes
1.3.14

Questing.questActive (list: quest ID, status)
Yes
Yes

1.3.14

Questing.hasPoints (list: pool ID, min, max)
Yes
Yes

1.3.14

playerHasAchievement (list: achievement ID)
Yes
Yes

1.3.14

playerHasStatistic (list: statistic ID, min, max)
Yes
Yes

1.3.14

A few things:
(1) toolFortuneLevel should be for mobs as well (Looting).
(2) aroundMobs could use a notAroundMobs complement.
(3) item* should allow detection of nbt/partial-nbt.
(4) As a more generic option than leaderboard checking, allow matching based on http://minecraft.gamepedia.com/Statistics since that is where the leaderboards pull their data from.
(5) Perhaps add achievement checking as well?
How will 'conditions' like consume/damagePer work with the matchers?
Thanks.

So... in a semi-joking manner...

I didn't include looting in there since it was already done and working with mob drops, which, was the only one it'd apply to.
Yes, it could.  Or, you know... {mob: "Cow", min: 0: max: 0, distance: 10} :P
If I must.  :)
You really like giving me work...
See 3, and 4.

(updating the list now)
Regarding consume and damagePer, they'll work the same as they currently do.  Basically, a lot of the mob drop code is already "good to go" with this, I just need to take it out of being mob-drop specific and make it able to be used for mob sheds and block drops without duplicating code, if that makes sense.

#1 Ok.
#2 There is no #2, you just imagined it ;)
#3 Well not must, but it would be rather useful :)
#4 I may or may not use. The problem is a lot of the uses I can see would depend on the change in the value rather than the fixed value. ie, when the set is applied it would snapshot the statistic and then the check would compare (origVal - currVal OP checkVal).
I can see the fixed value being useful in other types of modpacks though.
I actually wouldnt use #5  myself. The minecraft achievement tracker seems to be far too fond of resetting itself. I was just mentioning it for completeness ;-P

Hey, just because you're 40 hours into the map, maybe you haven't opened your inventory yet... <.<
So, for 4... I'd probably end up having to do a generic script condition for that one (which I really should do at some point, but... getting useful data into the scripting system is a bit of a chore.  Plus then I'd have to find out what data people wanted to check the delta of between current and when the set was applied...
It'd be a process, but, it could be done at some point.  Probably not during the first release of it though.  :)

Apparently I shouldn't depend on quick tests between meetings. You can simulate reputation type tracking with the minecraft scoreboard since MC 1.8. The 'reputation' just has to be defined first as a new objective (using 'scoreboard objectives add ...'). BQ already has support for it in the standard pack.
So in short, yes I need scoreboard tracking ;) Thanks.

@Thiana Not sure if you caught it, but I bumped scoreboard tracking up to a 1.3.13 release.  Which is pending on CF right now. :)

I hadn't caught it. I've been working on structures for worldgen.
Thanks :) And the boolean logic is a very nice bonus :)
Question, what are your plans for 1.9.4? Moving to, or updating in parallel with 1.9? Right now it looks like a little more than half my modlist is on 1.9, two still on 1.8.9 and the rest on 1.9.4.

The boolean logic was mostly just me being lazy... because I realized I was going to have to write a lot of "NotSomethingSomethingSomething" classes otherwise. :)
I'd originally planned on just moving forward to 1.9.4 and backporting any major bugs that were found to the 1.9 version, but, I'm going to have to see how things go.  I've been doing a lot of massive internal rework as part of the 1.4.0 version, because my original structure for everything... wasn't the best in order to support the project going forward.  As such, I'm not sure how easy it'll be to even have 1.9 backports.
But!  If it's not too much work, I'll try and keep both versions updated for at least a little while during the transition?

Looks like several more of my mods updated to 1.9.4 overnight. Since CH+BQ are the 'core' and available/will be available for 1.9.4 I'm going to update as well.

For Seasons support, do you also want a matcher for the calculated temperature at the location, or just the current season itself?  I'm assuming both would be good. :)

I only need the current season at the moment since idea is to enhance the effect of 'Blessings' if the season matches that blessing, ie Spring/Summer for the Life deity, Winter for the Spirit/Death deity. I might also tie mob aggression/power to the seasons, ie mobs are more desperate (thus powerful) in non-plentiful seasons. Still debating that part.
I'm actually not sure that the calculated temperature would be that useful? You already implemented inBiomeType which gives us a temperature range option... The current location would just seem to be a duplication of that, unless you want to be really nasty to the player and tell them to find a spot within a biome that is in a particular temperature range. I think that would be a little extreme for anything outside of Blood and Bones ;)


Enhance Test Coverage for @yamada-ui/charts

package: @yamada-ui/charts
Description
@yamada-ui/charts has a test coverage of less than 95%. We will enhance it to above 95%.
Target files

packages/components/charts/src/chart.types.ts

L171

Problem Statement/Justification
Testing is essential to ensure quality when operating a package.
Additional Information
While it's ideal to cover all tests, if there are too many lines to work on, please create a PR within your capacity.
Any deficiencies not covered by the PR will be automatically created as issues.

@hirotomoyamada
The target of this test is type.
Should we add a test for this?

@setodeve @108yen
There is no need to do this.
The only problem with this is that there is no ChartLabel in the source code.

@hirotomoyamada
ChartLabel is used by the user to customize the label.
https://github.com/yamada-ui/yamada-components/pull/445

@108yen
Can I use it on the source code? Otherwise, the test won't pass.

@hirotomoyamada
It's possible, but it doesn't make much sense.
Rather than that, what about adding a test to customize label?


Links in open-feature demo are outdated

Multiple links within https://github.com/open-feature/killercoda/blob/main/openfeature-demo/step1.md do not exist anymore.
i tried to locate them within the playground but did not find them eg hex-color.service.ts

Is this it? https://github.com/open-feature/playground/blob/d659a6962008eff33be797a4a900fcd8f4409646/packages/ui/src/app/demos/fib3r/fib3r-demo.tsx#L157


Applied token gets removed on restarting the plugin

Describe the bug
On applying a token on a Figma component, closing and restarting the plugin the applied token gets un-applied.
To Reproduce
Steps to reproduce the behavior:

Create a Figma component
Apply a token eg. borderRadius
Close the plugin
Restart the plugin
The applied token is removed

Expected behavior
On closing and restarting the plugin, the token should remain applied
Screenshots or Screencasts
If applicable, add screenshots to help explain your problem.
https://user-images.githubusercontent.com/66366384/172600803-95528cfa-54c4-4c7f-b77a-92f2a4768537.mov

Fixed in 107


Non linear load to energy consumption behavior

Hi, I just checked the SPECpower_ssj2008 dataset and saw that they contain much more than just the min and max energy consumption. So I wonder if there is there a specific reason why you assume a linear load to energy consumption behavior. Wouldn't it be better to perform a approximation over all available load levels (10% steps)?
full SPECpower_ssj2008 dataset
SPECpower_ssj2008 form

Thanks for your question. You can find an explanation of the methodology at https://www.cloudcarbonfootprint.org/docs/methodology#compute
If you have further questions, I suggest you post them over in https://groups.google.com/g/cloud-carbon-footprint sp someone who worked on the original methodology will be able to respond.


Bad bash syntax highlighting with multi line strings and single quotes

Does this issue occur when all extensions are disabled?: Yes

Metadata
Version: 1.85.1
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:47:11.635Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.15.0-91-generic snap
Bug
Initial variant

Code:
#!/bin/bash

cat << \
────────────────────────────────────────────────────────────────────────────────
oh no's
/* this comment and everything
 * after has wrong colors
 */
────────────────────────────────────────────────────────────────────────────────

function foo() {
    echo "Function foo"
}

foo

As you can see, even here in Github everything after the oh no's has wrong colors.
Other variant

Code:
#!/bin/bash

cat << \
EOL
oh no's
/* this comment and everything
 * after has wrong colors
 */
EOL

function foo() {
    echo "Function foo"
}

foo

Working example
Without single quote
This works in VSCode but not on Github.

Code:
#!/bin/bash

cat << \
EOL
oh nos
/* this comment and everything
 * after has wrong colors
 */
EOL

function foo() {
    echo "Function foo"
}

foo

Without line break after <<
This works in VSCode and on Github.

Code:
#!/bin/bash

cat <<EOL
oh no's
/* this comment and everything
 * after has wrong colors
 */
EOL

function foo() {
    echo "Function foo"
}

foo

Another workaround
Just escape the single quote.

Code:
#!/bin/bash

cat << \
────────────────────────────────────────────────────────────────────────────────
oh no\'s
/* this comment and everything
 * after has wrong colors
 */
────────────────────────────────────────────────────────────────────────────────

function foo() {
    echo "Function foo"
}

foo

Thank you for all the details! I've copied the issue to the repo that we pull our shell script syntax highlighting grammar from: https://github.com/jeff-hykin/better-shell-syntax/issues/73


Help Understanding / Confusing project references

Hello, I really appreciate this project. It's helped tremendously in my understanding of Flutter. That said, I had a fair amount of trouble getting my head wrapped around it because not only am I new to Flutter but the organization of the project really made me stretch as I was figuring out how things fit together. I'd like to propose a few changes to make it easier for others. I've already made most of these in my fork but I'm happy to open a PR if it's not too invasive. Here are my proposed changes:

All folder names under the example directory should match the name of the library in the pubspec.yml
All libraries in the example directory that are 'front-end' libraries (those that expose a UI via a main.dart) should start with flutter_ and be named with the pattern flutter_<implementation>.
Supporting libraries should start with the same name as the UI package unless they are shared across multiple projects, in which case they should start with todos_
Front-end packages should not take a reference on other front end packages
Project exports should all have a main export file. For front-end libraries that file is main.dart. For the rest it is <library_name>.dart.
Project imports from the same library should use a relative reference so that it is clear the import originates from the package you are already in. The Dart style guide regarding package imports should be followed.
The projects that handle repository behavior should begin with todos_repository_

When you do these things, you wind up with an example directory that looks like this:

Great! Since we're moving out of the example directory I should be able to do clean PRs into the new location. By convention the projects would move to lib/src so combining these ideas here's how I think it would shape up in terms of PRs:

Consolidate what is currently in lib/src into a todos_app_base library. (As a side benefit we should be able to remove the duplicate uuid and localization files later on with more ease.)
Move all of the non bloc projects to lib/src
Move the bloc projects. I'm a little fuzzy on this one so let me know.

If I have it right the directory structure would ultimately look like this:
* lib/src/bloc
* lib/src/bloc/base
* lib/src/bloc/base/standard ?  ( formerly example/blocs )
* lib/src/bloc/base/simple      ( formerly example/simple_blocs )
* lib/src/bloc/mobile
* lib/src/bloc/web
* lib/src/flutter_built_redux
* lib/src/flutter_firestore_redux
* lib/src/flutter_inherited_widget
* lib/src/flutter_mvc
* lib/src/flutter_mvi
* lib/src/flutter_mvi_base
* lib/src/flutter_mvu
* lib/src/flutter_redurx
* lib/src/flutter_redux
* lib/src/flutter_scoped_model
* lib/src/flutter_vanilla
* lib/src/integration_tests
* lib/src/todos_app_base
* lib/src/todos_repository_base
* lib/src/todos_repository_firebase
* lib/src/todos_repository_firebase_rtdb
* lib/src/todos_repository_simple

Since this is a reasonably large change it might make sense to start a new branch off of master so that I can PR against that instead and once everything is settle you could merge it up? I'll PR against master to begin with and look for feedback.
Thanks again!

Move all of the non bloc projects to lib/src

I was thinking of moving everything to the root directory for two reasons: First, I figured that would make them easier to find. Second, like the example directory, I think putting the code in lib/src would make more sense if all of those belonged to the same app or project, but they're all independent pieces.
Thoughts?

Move the bloc projects. I'm a little fuzzy on this one so let me know.

Let's keep bloc and simple_bloc separate for now, since they're meant to be two separate projects! So we'd have:
/
  -vanilla
  -scoped_model
  -bloc
    -core
    -mobile
    -web
  -simple_bloc
    -core
    -mobile
    -web

Since this is a reasonably large change it might make sense to start a new branch off of master so that I can PR against that instead and once everything is settled you could merge it up?

Yep, sounds great! Thanks again for all the help and raising the issue :)

I agree that lib/src is probably not ideal since you're only supposed to put private library code in there, but putting each sample into the project root doesn't feel right either. According to the docs it looks like putting them in lib may be the right choice since that folder is intended to be public? So then we would have:
/
  -lib
    -bloc
      -core
      -mobile
      -web
    -flutter_scoped_model
    -flutter_vanilla
    -flutter_<implementation>
    -simple_bloc
      -core
      -mobile
      -web
    -todos_app_base
    -todos_repository_base
    -todos_repository_<implementation>

Does that work?
I noticed we're using the terms core and base for sort-of the same thing. We probably want to pick one or the other and stick with it for consistency, I'm indifferent as to which so lmk if you have a preference.
I enjoy the collaborative PR approach so I'll send you my first draft pretty quick.

Yep -- definitely put stuff inside the lib directory when it comes to packages!
However, in my head, I don't think of the root of this project as a Dart package (it is currently, but that's a bit of a mistake :P). I see the root as a place that holds some Dart packages (some of them are Flutter apps, others provide supporting code), and each of those packages have their own lib directory.
You can see a similar approach on official Dart projects like json_serializable, where the Repo contains a few Dart packages at the root of the project, and each package has the lib directory within.

I noticed we're using the terms core and base for sort-of the same thing.

Good call. Let's go with core -- I've seen it used more commonly in other examples.

That's reasonable, I'll go ahead and update my first PR to reflect this feedback. 👍

Closed by #104, #110, #111, #112, #118


Haskell override clean up

Ping @peti, specifically for the first point on jailbreak-cabal:

jailbreak-cabal override isn't needed, shared executables are disabled
by default (or is this precautionary?)
Removed and added some infinite recursion overrides, moved them to
configuration-nix.nix
Unbreak zstd and attparsec-varword
mysql override is already in configuration-nix.nix
Move some overrides for disabling tests due to internet connectivity
to configuration-nix.nix

Motivation for this change
Things done

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

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

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

I merged the heap fix directly into master as 45d727057958d911453b7401482157c874d2e95a.

https://github.com/NixOS/nixpkgs/commit/6f19c997d243a8c5f359e2456fb31479f2886905 merged to haskell-updates.


Added conan io package manager

Please note, the autoupdate block is not entirely correct. The actual version should replace . with _ in the url strings. Could someone help with this?

ScoopInstaller/Main#83


tile-config-generator doesn't seem to understand tildas (~) in file paths

Describe the bug
$ tile-config-generator generate --pivotal-file-path=~/Downloads/pivotal-container-service-1.3.0-build.38.pivotal --base-directory=~/tmp/src/pks-tile-config-generator --do-not-include-product-version
error: open ~/Downloads/pivotal-container-service-1.3.0-build.38.pivotal: no such file or directory
$ ls -l ~/Downloads/pivotal-container-service-1.3.0-build.38.pivotal
-rw-r--r--@ 1 user  staff<PHONE_NUMBER> Jan 25 10:31 /Users/user/Downloads/pivotal-container-service-1.3.0-build.38.pivotal

Expected behavior
I was hoping that the binary would understand the meaning of tilda (~) in file paths provide via command line switches
Version info (please complete the following information):

tile-config-generator: 0.0.14
tile pivotal-container-service
tile-version 1.3.0

Work around was to expand the ~, replacing it with the absolute path to the tile.

The functionality of this tool has been moved into the officially supported om tooling.  https://github.com/pivotalservices/tile-config-generator/blob/master/README.md#deprecation-notice Please re-open this issue in that repository as that will get prioritize by team focusing on the future development.


Refactor Server customize shutdown and signals to support embedding

This change currently aims to minimize the code diff. Still need to test for my use case.

Fixes #492


Release v2.30.0 ?

The dist/ folder of the v2.0 branch has v2.27.0 as the latest version of the javascript available in the repo
However your v2 guide located here:
https://developers.braintreepayments.com/start/hello-client/javascript/v2
includes the following reference to Braintree.js v2.30.0:
<script src="https://js.braintreegateway.com/js/braintree-2.30.0.min.js"></script>

Why is the v2.0 branch not showing a newer version?

Never mind!
Found it here -> https://github.com/braintree/braintree-web-bower/tree/2.x

:+1:


Tabular: Updated type() checks to isinstance() checks

Issue #, if available:
Description of changes:
Tabular: Updated type() checks to isinstance() checks
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-323-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-323/1/index.html


BEAM-13439 Type annotation for ptransform_fn

Because the return type of the ptransform_fn is not specified, certain typecheckers will assume the type signature of functions decorated with it remain unchanged. However, ptransform_fn specifically adjusts the type signature such that the function no longer takes a pcoll.
This can be corrected by changing the the definition of ptransform_fn from:
def ptransform_fn(fn):

to:
def ptransform_fn(fn) -> Callable[..., _PTransformFnPTransform]:

This allows the typechecker to understand the return type of a function decorated with ptransform_fn will instead be a _PTransformFnPTransform type.
Additionally, for Python 3.10, ParamSpec could be used to preserve the signature of the internal function as well.
from typing import ParamSpec, TypeVar, Concatenate

P = ParamSpec("P")
T = TypeVar("T")
R = TypeVar("R")

def ptransform_fn(fn: Callable[Concatenate[PCollection[T], P], R]) -> Callable[[P], _PTransformFnPTransform]: ...

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@robertwb Can you review this please? Kenneth Knowles mentioned you may be a good reviewer for this change.

@tvalentyn - Would you be able to take a look at this change? Or find another person to review?

@robertwb does this change affect Beam's type inference machinery at all ?

@tvalentyn - Would you be able to take a look at this change? Or find another person to review?

Thanks for the bump. Also cc: @ryanthompson591.

Our type checking tool now kicked in:
08:06:01 apache_beam/transforms/ptransform.py:1005: error: syntax error in type comment '(Callable) -> (...) -> _PTransformFnPTransform'  [syntax]
08:06:01 Found 1 error in 1 file (checked 818 source files)
08:06:02 error: mypy exited with status 2

Interesting. Pyright has no issues with this. I'll adjust so that mypy works as well.

@tvalentyn : Should be fixed now.

Thank you!


Unprocessible Entity when providing time offs with a date

https://github.com/giantswarm/personio-go/blob/ffeb4be5cb48d7a10c02bee8acad21d7f353bceb/v1/personio.go#L446
This line outputs the date for the URL in RFC3339 however the API only accepts YYYY-MM-DD format (2006-01-02) otherwise a 422 error is returned.

Thank you for this bug report and the proposed fix.
Just some minor polishing and that pro forma form and I will merge it. See PR.

The PR resolves this issue, closing.
@surdaft Thank you for your contribution and patience!


Tests profiling

Hi!
I can run node --cpu-prof out/test/runTest.js to profile test runner, but I can't figure out how to profile my extension. My extension is running slow only under tests, so I would like to understand what is going on. Any tips would be appreciated.

You can use the inspector api to trigger a profile run. Though it could be interesting to have built-in support for it.


GUVNOR-2431: Redesign Copy, Save, Rename, Restore and Delete Popups (away from FormStylePopup)

This PR updates the usage of PopUps in order to call the new PopUps API (that implements the MVP pattern using Errai UI).
See:

https://issues.jboss.org/browse/GUVNOR-2431
https://github.com/uberfire/uberfire/pull/443

Code formatting please!

@manstis Thanks! :)

Thanks @karreiro +1 to merge (with others when green).
@pefernan What to review (since this PR is on Form Modeller)?


There is no need to change innerHTML to insert values from the store there - it is enough to change innerText

What docs page needs to be fixed?

\docs\tutorials\fundamentals\part-1-overview.md
\docs\tutorials\fundamentals\part-5-ui-and-react.md
\examples\counter-vanilla\index.html

What is the problem?
Store stores values - not html, it makes no sense to change html and for security purposes
What should be changed to fix the problem?
valueEl.innerHTML = state.value.toString()
should be changed to
valueEl.innerText = state.value.toString()

Appreciate the feedback, but it's really not worth changing at this point :)  Besides, the example is trying to show the basic Redux + UI interaction sequence, not the intricacies of DOM APIs.

But this is also semantically incorrect and inculcates unsafe methods of DOM modification for beginners.
However, you are the owner of the repository - your decision is decisive :)


[3.5] Assets can import fine, but will crash godot when zipped.

Godot version
v<IP_ADDRESS>.rc.custom_build[338114d47]
System information
windows 11 pro, ver 10.0.22621 Build 22621
Issue description
export.zip
The folder can be imported without problem, but will crash godot once zipped.
Steps to reproduce
It's compiled with https://github.com/funexpected/godot-flash-module.  Just create an empty project and import the zip file, it crashes.
Minimal reproduction project
proj.zip

https://drive.google.com/file/d/1hDim51mCZuW-2JGfq6R0ok4MFeIf0N3n/view?usp=sharing This is compiled godot file.

Can you replicate this without the module?

Can you replicate this without the module? We do not support third party modules and if the error does not occur without the module it's likely not a fault in the engine, and should be reported to the developers of the module
The module seems to use zip files, so the cause is most likely the module

Solved, the module uses a zip format that's different from 7-zip or bandizip.

In the future, please don't make issue reports over a module here but on the module itself, we have no ability to fix a third party module


Unable to connect Robot ROS1 free_fleet client and PC ROS2 free_fleet server using Pocket Wifi.

Operating system and version:
PC: Ubuntu20.04, galactic robot: UbuntuMATE20.04, noetic

How to install RMF
I installed RMF following the guide in the link below.
https://github.com/open-rmf/free_fleet

Description of the bug
I started free_fleet_client on my robot's PC and free_fleet_server on my PC according to barebones example, but I can't connect.
I have confirmed the ping connection and port connection between the robot and the PC.
Both the robot and PC set ROS_DOMAIN_ID=0 and opened ports 7400~8000, which would be used by DDS, to check the connection.
Pocket Wifi is used as a common network.
Steps to reproduce the bug
PC: ROS_DOMAIN_ID=0, open ports 7400~8000 and start ff_examples_ros2 fake_server.launch.xml
　Robot: ROS_DOMAIN_ID=0, open ports 7400-8000 and run ff_examples_ros1 fake_client.launch.
　No connection.
Expected behavior
The free_fleet_client running on the robot is registered in the log of free_fleet_sever.
Actual behavior
No log is registered and no communication is established.
Additional information
Network settings were confirmed at the following link.
http://wiki.ros.org/ja/ROS/NetworkSetup
　The following link was used as a reference for opening the port.
https://docs.ros.org/en/foxy/Concepts/About-Domain-ID.html
　A Raspberry Pi 4 Model B is used as the robot PC.

Hello there! Unfortunately I am unable to recreate this issue without the same hardware at hand. Some questions before I look further into it,

Does this setup work when these 2 machines are on a different wifi?
Does this setup work when these 2 machines are on a different pocket wifi?
Does basic pub sub using ROS 1 or ROS 2 across this same network work? This will give us more information whether it is something that needs tweaking on the network level, cyclonedds or free fleet.
If the above works, could you try your setup using basic pub sub on cyclonedds, https://github.com/eclipse-cyclonedds/cyclonedds/tree/master/examples?

Some things to note,

since one of the machines is using ROS 1, setting ROS_DOMAIN_ID will not do anything, that is only useful for communications between ROS 2 machines.

I will be removing the bug label as this is not related to free_fleet's implementation or our usage of cyclonedds, but rather the requirement of using a Pocket Wifi instead.

Thanks for the reply.
I was able to confirm the connection by inactivating the firewalls on the robot PC and ROS2PC.


Fixed Cannot read property 'toString' of null at CoinbaseOrderEntryGateway.sendOrder.

sorry found a bug at startup IF tbp and pDiv have too small values, THEN the calculated quote have 0 size. Thats OK, but is not ok to calculate the price of empty orders and try to send them.
Now while avoiding to calculate the price, corrupted orders are not send anymore.

Thank you!


QScroller issuing console.log statements

QScroller seems to be issuing console.log statements when scrolling the scroller.
I was using the console in the inspector for something else when I suddenly started getting log statements when scroll my QScroller
Mainly stating "update position" .. but also "no selected item found ..."

hmmm, thanks for this. I'll look into it.

Will be fixed in next release.


Slow select by address query

2021/11/01 20:24:23 /build/crud/crud_transaction.go:255 SLOW SQL >= 250ms
[42080.194ms[] [rows:25[]
SELECT
"transactions"."from_address",
"transactions"."to_address",
"transactions"."value",
"transactions"."block_timestamp",
"transactions"."hash",
"transactions"."block_number",
"transactions"."transaction_fee",
"transactions"."receipt_status",
"transactions"."type",
"transactions"."method",
"transactions"."value_decimal"
FROM "transactions"
WHERE type = 'transaction'
AND
(from_address = 'hx54f7853dc6481b670caf69c5a27c7c8fe5be8269' OR to_address = 'hx54f7853dc6481b670caf69c5a27c7c8fe5be8269')
ORDER BY block_number desc LIMIT 25

Ran the planner on the table as is, and it's coming back as relatively slow (~ 40 seconds, which we knew).
Tossed the index it's using (transaction_idx_block_number) into the prewarm cache to see if that helped. Didn't do much.
Ran vacuum on whole transactions db and it seemed mostly clean (I think now that we're not using it for counts, so it's not super full of dead tuples).
Same deal after vacuum, so I'm not sure where to go now. Maybe we can rebuild the index or something? That seems a bit drastic though.

I'm tweaking the performance specs of the volume to see if we can improve things a bit now that we're only using one instance. I'm thinking some things might be getting jammed up while we're doing read/writes.

As I'm going through experimenting with things, I'm starting to see that this might be a really bad query to begin with. Even with the index scans, if we're using a completely un-warm instance: it takes >20 mins to plan the query. If we add the index to prewarm, it doesn't get appreciably faster (since I cancelled it before it finished). Now I'm prewarming the whole transactions table and I'm expecting it to be about the 40 seconds we were looking at before.
Are there any other ways we could write this query? @pranav925

Did some more experimenting with rewriting the query.
Running the base query without limits changes the plan from a backwards index scan to a heap bitmap scan (which is super fast). The only downside is now we have all the data, which is potentially pretty huge.
Thought maybe I'd be able to trick the planner into letting me return the main filtered data and then run the limit on that, but it was too smart for me.
CTE (no faster):
WITH a (from_address, to_address, value, block_timestamp, hash, block_number, transaction_fee, receipt_status, type, method, value_decimal) AS (
    SELECT
        "transactions"."from_address",
        "transactions"."to_address",
        "transactions"."value",
        "transactions"."block_timestamp",
        "transactions"."hash",
        "transactions"."block_number",
        "transactions"."transaction_fee",
        "transactions"."receipt_status",
        "transactions"."type",
        "transactions"."method",
        "transactions"."value_decimal"
    FROM "transactions"
    WHERE type = 'transaction'
    AND (from_address = 'hx54f7853dc6481b670caf69c5a27c7c8fe5be8269' OR to_address = 'hx54f7853dc6481b670caf69c5a27c7c8fe5be8269')
) SELECT * FROM a;

Subquery (~3x slower):
SELECT *
    FROM (
             SELECT from_address,
                    to_address,
                    value,
                    block_timestamp,
                    hash,
                    block_number,
                    transaction_fee,
                    receipt_status,
                    type,
                    method,
                    value_decimal
             FROM transactions
             WHERE type = 'transaction'
               AND (from_address = 'hx54f7853dc6481b670caf69c5a27c7c8fe5be8269' OR
                    to_address = 'hx54f7853dc6481b670caf69c5a27c7c8fe5be8269')
         ) AS a
ORDER BY a.block_number desc LIMIT 25;

Failing us getting this business running faster through various planner tricks, do we have any options for caching?

I think I have a resolution that should work. Earlier comment about CTEs wasn't quite accurate for PG >12. Needed to add MATERIALIZED in there, so this works v good.
WITH a (from_address, to_address, value, block_timestamp, hash, block_number, transaction_fee, receipt_status, type, method, value_decimal) AS MATERIALIZED (
    SELECT
        "transactions"."from_address",
        "transactions"."to_address",
        "transactions"."value",
        "transactions"."block_timestamp",
        "transactions"."hash",
        "transactions"."block_number",
        "transactions"."transaction_fee",
        "transactions"."receipt_status",
        "transactions"."type",
        "transactions"."method",
        "transactions"."value_decimal"
    FROM "transactions"
    WHERE type = 'transaction'
    AND (from_address = 'cx0000000000000000000000000000000000000001' OR to_address = 'cx0000000000000000000000000000000000000001')
) SELECT * FROM a ORDER BY a.block_number desc LIMIT 25;


Connect acetolactate synthase GO:0003984 to valine biosynthesis GO0009099

I'm trying to find genes associated with valine production. I started with term GO:0003984, which identifies acetolactate synthase, which is one of the early steps in this pathway. It doesn't look like this term is connected to valine biosynthesis GO:0009099. I would expect that GO:0003984 would be a child of GO:0009099.
Thanks,
Dan

I'll make a has_part GO:0003984 to GO:0009099
But then should I also add while I'm at it
ketol-acid reductoisomerase activity
dihydroxy-acid dehydratase activity
L-valine transaminase activity
The last two activities also involved in other pathways, hence the has_part is a better choice
Taking info on the other steps from
https://biocyc.org/META/new-image?object=VALSYN-PWY

Thanks. When do you think the change will show up in the change log? https://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0003984

@ukemi can you look at what I plan to do with this?

Hi @hdrabkin,  This makes sense, but it won't show up in annotations. So to get the entire list of genes, either they will have to be annotated to valine biosynthetic process or a search will need to be done on all the enzyme activities that are necessary parts.

Yes, I'm assuming @danolson will want to do this

Is there something you need be to do? I don't understand @hdrabkin 's last comment. (also, FYI, @DanOlson is not me, despite the similar name).

No; this is going in as we speak. I assumed you would want to get back genes annotated to the various genes involved in the pathway; so they would need annotations first. However, if you are just 'browsing the ontology', everything is good.

done; (and changed your id in my comment; sorry).
CLosing

@hdrabkin or @ukemi, can you send me a link to show me where the changes were made? On the 0009099 page (https://www.ebi.ac.uk/QuickGO/term/GO:0009099), the last update was 2016-11-28. On the 0003984 page (https://www.ebi.ac.uk/QuickGO/term/GO:0003984), the last update was 2015-04-24.
Thanks,
Dan

You can see it in the ontology file which can be found here
http://purl.obolibrary.org/obo/go.obo (3-31-18 version)
[Term]
id: GO:0009099
name: valine biosynthetic process
namespace: biological_process
def: "The chemical reactions and pathways resulting in the formation of valine, 2-amino-3-methylbutanoic acid."  L-valine transaminase activity


nitrokey-fido2-firmware: init at 2.4.1

Description of changes
This is the firmware for the Nitrokey FIDO2 key, the packaging of which was continued during the Summer of Nix 2024 with sponsorship by NLNET.
This is not an executable, but a binary firmware that can be flashed onto such keys.  According to a discourse thread, inclusion of such artifacts in nixpkgs shouldn't be an issue.

cc @amerinor01 @jleightcap @imadnyc @RCoeurjoly @fricklerhandwerk
Related PRs:

#337966
#337959
#337956
#337954
#337951

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@sbruder I can't believe we didn't include you earlier! Could you message me @dre:imad.nyc on matrix or email<EMAIL_ADDRESS>for the rest of the groups info? I can't seem to find it anywhere on the internet.

Hey, I initially packaged the nitrokey firmware as part of Summer of Nix 2022. It’s great to see it now being included into nixpkgs. Could you please include me as a co-author of the commits of this and the related PRs that are based on the previous work? That would be great to indicate the provenance of the code, as there is nothing right now that links the new PRs to the initial work from 2022. However, I don’t have the time to continue working on this, so I can’t be a maintainer for the packages.

Thanks for you work! Done, you are included as a co-author. As @imadnyc said, if there was someone else involved, please let us know so that we can include them.


Running AppiumForMac with logged out user

I'm able to run tests against Mac desktop app via AppiumForMac from the same machine, or from a remote machine as long as the user is logged in.
If the user is logged out (I log-in, start an ssh daemon, log-out) and then try to run the test - it will fail because AppiumForMac will not find the app (it will never become the front app).
Did anyone find any work-arounds for this? Is it even possible to run it with the user being logged-out (or it has to be logged in at all times)?
Thanks

Probably log-in session is necessary to get Window service on macOS.
Or some process could work if the parent process was launchd

The issue in my case was that the screensaver was running.
I tried a bunch of things, like:
    caffeinate -u -t 2

    osascript -e 'tell application \"System Events\" to key code 123'

    osascript -e 'tell application "System Events" to get name of application processes whose frontmost is true and visible is true'

    pmset schedule wake "$(date -v+7S '+%m/%d/%Y %T')"

but none of them did work in my case. What did work was simulating a click and spacebar keypress using cliclick like ./cliclick c:1,1 kp:space (you need to build cliclick on your own).


Is compression result cached?

Is the compression done on the response for every request, or done once then cached and that cached response sent back instead?
Such as if I am serving static client files, will a pre-compressed response be sent back or will the files be compressed on demand?

The compression is done on every response (as applicable to which compressions methods are suppored for that specific client) and are not cached.


[Feature request]: --version switch

What do you need?
As of this writing the fabric --help doesn't display the current version it is running, and there is no --version switch to give this information so the user will know what version they have.

Hi
would like to pick up this enhancement


image list and detail page issue

[x] 1. The state of the image on the list page, but unknown on the detail page

[x] 2. Sort by the state on the list page could make the page work inappropriately, something like the menubar does not pop up by clicking the ellipsis

[x] 3. Select one of the items, proceed clone as form, users are not able to add more than one label or annotation

[ ] 4. Is it necessary to suppress the create a virtual machine  on the menu bar if the image state is not healthy?

[x] 5. This functionality does work

remove the protips hint from the dialog.

Validated all except the item 4th


Overwriting packaged installation with cutting edge file breaks TagUI (casperjs not found)

On some computers, the default behavior when overwriting an existing folder is actually deleting folders which are not present in the source but present in the target folder. Thus, overwriting a local packaged installation that works, with the cutting edge version, may break the program.

above commit changes wording to below so that folders in existing src folder won't get deleted.

Tip - to try cutting edge version with the latest features, download master.zip to overwrite an existing packaged installation (be sure to manually select and move the folders & files inside master.zip's tagui/src folder to replace your existing tagui/src folder, otherwise some OSes will delete existing target folders that are missing from source folder)

Sorry for asking, but which OS / Software does delete folders when unpacking an archive into an existing folder?

I can't really replicate, but I've encountered this before on Windows 10. Just today, my colleagues encountered this on their macOS, which is kinda strange.....

Oh.. I don't want to automatically create packaged installation that have cutting edge code.
Will prefer a gap of a couple of months so that pro users can test out the new features first by choosing to manually overwrite their local installation. Then i publish that into a new packaged release.


Changed directory separators to use path.sep instead of '/'.

This should fix #39. I tried to change as little as possible.

Published as 2.3.5

Thanks a lot for doing this. :-)


[APIM-3.X.X] - "Scopes" element is not added to the swagger 2.0 "securityDefinitions" object

Description:
In the default Pizzashack API in wso2am-3.X.X or if we try to upload the swagger 2.0 file to create an API in publisher, the securityDefinitions element is added to the swagger file. But this does not include the scopes element.
Original Swagger 2.0 definition

Auto updated swagger 2.0 definition after uploading the original definition to APIM-3.X.X.

Even if the swagger 2.0 definition having securityDefinition without scopes is uploaded to the publisher, the API got created successfully. There aren't any errors in the carbon console or in the publisher swagger editor. But when we enabled the API definition auditing in 42crunch, we were able to observe this issue and it is a critical error.

Steps to reproduce:
Please make sure to enable the 42crunch API definition auditing.

Deploy the Pizzashack API in the publisher which is in swagger 2.0 format.
The API got deployed successfully and the securityDefinitions does not contain scopes.
Downloaded the swagger definition of Pizzashack API.
Uploaded the downloaded Pizzashack API definition.
Navigate to Publisher --> API --> API Definition --> AUDIT API
You will be able to see the audit report

Affected Product Version:
wso2am-3.0.0
wso2am-3.1.0
wso2am-3.2.0
Proposed Solution
In OpenAPI 3.X.X the scopes element is added as an empty object, "scopes: {}", by default. If the same approach can be used for swagger 2.0, this issue can be solved.

Hi guys,
I'm a part of the team that reported this issue.
I am told this issue affects all APIs that are created in the Publisher, not just the PizzaShack. Please make sure to check that as well, thank you!

Closing as per the public fix has been added via: https://github.com/wso2/carbon-apimgt/pull/9927


[Impeller] Implement an impeller::Contents subclass allows manipulating one or more input images into a single output image.

This class is meant to be abstract and only handles setting up intermediate render passes.

cc @bdero


MH-12795 REST docs don't respect @Produces annotation on class level

Found this while testing PR #155 :-(
See https://opencast.jira.com/browse/MH-12795 for details

Do you have an example of how to best test this?

@lkiesow The easiest way to test this is to build PR #155, start Opencast and have a look at the "Response formats" in the REST API Docs: You won't find application/json, application/v1.0.0+json in the Base API anymore, for example. Then applied this patch and the response formats will be there again.


Command not working

appsody deploy --generate-only command does not work inspite of taking the latest Appsody

What error are you seeing with appsody? Please provide output detail and the platform which you are using (e.g. MacOS, Windows, Linux).


drop table policy

trino:default> drop table hive.yiutto.ts_20230805;
Query 20230803_074307_00041_4k4d2 failed: Access Denied: Cannot drop table yiutto.ts_20230805

why not drop table,  but ranger -> acess - result -> allowed

I am closing this for now. Feel free to comment or reopen later


Update rising thermal bubble example in README

X-ref: #449

I can't think of an (easy) way to test this stuff.
One possibility is to run the example in CI, and then somehow auto-paste the example code into README.md (at the same time the documentation is created?) Not sure its worth the effort but its obviously a problem when the README is out of date.

Yeah could probably test it through some sophisticated text grepping or regex to pull the code out but I think it we keep it down to 2 very short examples, we can just be more diligent about updating the README.


Investigate usage of com.unity.roslyn for source generation in Unity 2020

Unity Technologies provides the com.unity.roslyn package for source generation in Unity 2020. It should be investigated whether it could be used as a dependency to provide source generation for Unity 2020. All source generation features would be available to everyone if support for Unity 2019 is dropped. I believe it to be the right thing to do, as source generation would be a good replacement for reflection.

com.unity.roslyn allow code generation to be used in Unity 2020. It is a preview package, but I believe it to be worth it.


Update README.md (a small typo)

Corrected a minor typo: "Where are are going" -> "Where are we going"

@a-kapoor - thanks for catching this.   👍


Change the device class of the smart sensor from humidity to moisture

Fixes https://github.com/py-smart-gardena/hass-gardena-smart-system/issues/213

Thx for the PR, I merge it. !

Sorry, this is actually not working at all. Humidity was fine to me, why the change? Anyway, moisture is not exposed by the integration.
@grm Could you revert this PR please?
@DNier Any idea what could be wrong? The new entity shows up, but does not contain a valid state. It's not fed by the sensor

The py-smart-gardena lib does not return "soil_moisture" but still "soil_humidity". Either change it back or adapt the library and push a new version:
https://github.com/py-smart-gardena/py-smart-gardena/blob/master/src/gardena/devices/soil_sensor.py


Allow the user to specify multiple models

I will allow the user to use multiple models from groq AI. The user will either be able to specify one specific model, or generate multiple files using each model.

closed with 32650c4


Introduce CLI Unit Tests in the CI/CD pipeline

Introduce CLI Unit Tests covering all possible options in the package.json via node test and add the unit tests into CI/CD pipeline.

Introduced Unit Tests with multiple options to the CI/CD for node versions 15.x and 16.x
https://github.com/jadiagaurang/minify-all-cli/actions/workflows/node.js.yml


Inconsistent table parsing behavior

I've parsed the following PDF consisting of 2 tables:
ENI_2018_p12-13.pdf
The two tables have the same schema and include financial metrics per country & per continent ("EUROPE", etc.) and for the whole company "Eni Group" (bottom of 2nd table).
Here's the parsing results exported to PDF:
ENI_2018_p12-13_parsed.pdf
# Code snippet of parsing task
json_objs = LlamaParse(verbose=True).get_json_result(document_path)

dfs = []
for page in json_objs[0]["pages"]:
  for item in page["items"]:
    if item['type'] == 'table':
      df = pd.DataFrame(item["rows"][1:], columns=item["rows"][0])
      dfs.append(df)

A few observations:

First column "Full year 2018" was renamed to "Country"
Continent rows in table 1 ("EUROPE", "AFRICA") were removed, BUT not in table 2 ("ASIA AND OCEANIA", "AMERICAS" still there)
"Eni Group" row in table 2 is still there
Column "Total revenues" was renamed to "Total revenues (€ thousand)", but only on this column, and only in Table 1

My questions:

Do you have a kind of semantic/understanding layer in your parsing?
Could you explain the delta in parsing between table 1 and table 2

As a user, I'm not sure what to expect. For example, on one side, I love it that you renamed "Full year 2018" to "Country" as this new name seems closer to what the column really is about. However, in that case, I believe I would expect a consistent behavior in what is parsed and how it's done (e.g. keep or remove continents?) and also perhaps be given a way to adjust the parsing behavior.
Thanks!

Hi! Sorry for the delay and thanks for your comments and questions.
While your table is quite standard, LlamaParse needs to be working on a wide variety of tables. Our pipeline use different methods to ensure we get the best quality but discrepancies can still appear.
We recently release our latest API version and it should fix this one.


.createOrder method always has no price value

I'm submitting a ...
[x] bug report
[ ] question about the decisions made in the repository
Code
class LimitOrder extends BaseOrder {
  constructor(api) {
    super(api);
  }

  create(side, amount, params) {
    this.side = side;

    this.postOnly = params.postOnly;

    this.status = states.SUBMITTED;
    this.emitStatus();

    this.createOrder(price, amount);
  }

  createOrder(price, amount) {
    this.amount = this.api.roundAmount(amount);
    this.price = this.api.roundPrice(price);

File Path
exchange/orders/limit.js
Action taken (what you did)
while .createOrder method has been called, price always has no value. Because price never been declared inner .create function.

Ahhh yea I should have NOT pushed the limit order to github, apologies.
Context: Gekko Broker is a library to do advanced (and normal) orders, however only the sticky order is actually working and hooked up right now.
Mostly because I am super busy and (for now) the sticky order is the only order type used by Gekko (for now).
Feel free to propose a PR that does one of the following:

fixes the current limit order (note: the base "order" class is a bit of a mess)
adds a comment in the code noting that it's not working.
removes the code alltogether.

Else I will do 3. at some point (until it's fixed).

I went for option 2 for now, so that anyone else can easily pick up 1 (without having to look to far for the current code).


bitfinex import, gekko live: Unable to stitch datasets.

I'm submitting a ...
[ x] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
I have use the last git version of branch develop of gekko.
I have unable candleWriter.
I have start an import of more than one day and directly start gekko live like this on bitfinex:
node gekko --config config.js --import && node gekko --config config.js
The historySize is on 780 minutes so 13*60.
Expected result (what you hoped would happen)
Than gekko live use local data for the warmup and stitch data.
Actual result (unexpected outcome)
When gekko live start it is not unable to stitch datasets :(
2018-04-01 10:00:02 (DEBUG): Available local data:
2018-04-01 10:00:02 (DEBUG): from: 780 minutes ago
2018-04-01 10:00:02 (DEBUG): to: 2 minutes ago
2018-04-01 10:00:02 (INFO): Usable local data available, trying to match with exchange data.. 2018-04-01 10:00:02 (DEBUG): Fetching exchange data since 17 minutes ago
2018-04-01 10:00:02 (DEBUG): Available exchange data:
2018-04-01 10:00:02 (DEBUG): from: 0 minutes ago
2018-04-01 10:00:02 (DEBUG): to: 0 minutes ago
2018-04-01 10:00:02 (DEBUG): Unable to stitch datasets.
2018-04-01 10:00:02 (INFO): Not seeding locally available data to the trading method.
2018-04-01 10:00:02 (INFO): The exchange does not return enough data. 0 minutes are still missing.
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
When i do the same things on gdax it s work with no problem but on bitfinex it s not.
Something is really wrong somewhere.
I have try a lot of different things to make the import and start gekko live but each time same result on bitfinex: Unable to stitch datasets.

In your config, what are the candleSize and historySize settings?

Check this PR:
https://github.com/askmike/gekko/pull/2095
https://github.com/askmike/gekko/pull/2112
For me bitfinex is working without problems on USD-FUN and USD-SNG

The PR #2095 doesn't have anything to do with this issue unless @yepla is using postgresql, which he's probably not.


docs: update calico hyperkube instructions

Missed a few things in the AWS guide when we updated the Calico process.

lgtm


A dynamic tts approach

Is your feature request related to a problem? Please describe.
I noticed that current tts model is manually set on admin panel. But as a bilingual user, I would like the tts to be able to generate speech based on the output language. e.g.(I ask chat how to say Apple in Chinese, i would like to hear ‘apple’ pronounced in Chinese.
Describe the solution you'd like
online: google language detection
local: https://github.com/pemistahl/lingua-py
Describe alternatives you've considered
User should select primary and secondary language, to make language detection process easier
Additional context
I can start work on this feature cuz it would be helpfull to me :)

Related - https://github.com/open-webui/open-webui/issues/2256


Replicate PowerBI dashboard for smartspim (but for every modality)

https://app.powerbi.com/groups/me/reports/0d08e142-bdb3-492c-8bbc-698b088b62a2/ReportSection?ctid=32669cd6-737f-4b39-8bdd-d6951120d3fc&experience=power-bi&bookmarkGuid=2d70a5e0-1a75-42b1-8c87-e6341383e08f

Talk to Camilo about putting the QC json in place
Video??
SmartSPIM specific dashboard

https://codeocean.allenneuraldynamics.org/capsule/2846012/tree
Code Ocean
https://codeocean.allenneuraldynamics.org/capsule/8412427/tree
Code Ocean


About APPID

Hello,
I'm the technical staff of CoolKit eWeLink, how did you get our APPID and APP Secret?

it is a shame that you make this inquiry as a coolkit staff.
Better worry about providing better products as my 4CH Pro R2 had problems after a firmware update and no one gave me an answer.

Better worry about providing better products as my 4CH Pro R2 had problems after a firmware update and no one gave me an answer.
Go ahead Alexx 👏👏

Regarding this issue, please contact Sonoff customer service staff directly
URL: https://itead.freshdesk.com/support/home
Email<EMAIL_ADDRESS>Best regards.

Hi AlexIT
I am totally new to the HA environment. How do I use a Sonoff PIR sensor
and botton?
Cat seem to get it right?
Knd regards

@coolkit-carl Hi. Somewhere in similar projects.


SNMP trap not using the correct engine ID

Description
When sending a SNMP trap (using the SNMP plugin) the engine ID is different in every call. Additionally when "forcing" it on the securityMap for the SNMP constructor it doesn't seem to be correctly converted.
Steps
Steps to reproduce the behavior:

Execute the tests/autoTestAll.SNMP.js, function testSNMP3ClientTrap
Using WireShark or similar examine the packet sent to the target
Verify the msgAuthoritativeEngineID sent
Restart OpenAF and re-execute steps 1 to 3 and verify the msgAuthoritativeEngineID is different and not equal to the engineID provided.

Expected behavior
The expected behaviour would be for the msgAuthoritativeEngineIDs to equal between executions when the engineID is provided. Additionally the msgAuthoritativeEngineIDs should be based on the provided engineID.

Included in stable build 20210721


[CSL-16] Documentation for developers

@gromak

Provide per-module and per-exported-function documentation with correspondence between the implementation and the paper. (Customer: Galois, Chepurnoy's Team)
For infrastructural code, provide reasoning behind picking given approach and plans regarding further work. (Customer: WellTyped, Chepurnoy's Team)
Emphasize threading model. (Customer: WellTyped)
Emphasize how time-warp is used and how we want to improve it. (Customer: WellTyped)
Write a guide to pos-prototype for newcomers (Customer: IOHK, Serokell)

@shersh
Some warnings are left. And some automatically generated functions (like lenses) are non documented. But it's minor. Overall we have almost 100% ''Haddock'' coverage.


Current CMake build documentation leads to undefined references

OS: Ubuntu 19.04
CMake: 3.15
Page: https://www.glfw.org/docs/latest/build_guide.html
Documentation states to use the OPENGL_gl_LIBRARY cache variable. However, this appears to be null (leads to undefined references during compilation) when using CMake 3.15 on Ubuntu 19.04. Using the OPENGL_LIBRARIES result variable stated here in the cmake.org documentation instead does work.
Doesn't work (current documentation) target_link_libraries(myapp ${OPENGL_gl_LIBRARY})
Works target_link_libraries(myapp ${OPENGL_LIBRARIES})
I believe this will likely be an issue on other distributions aswell.

Thank you, will fix!


add workflow to verify swagger generation

This PR adds a workflow that checks if code generation has been executed (Swagger installation is derived from https://goswagger.io/install.html to conform Github actions)
swagger-gen.sh now determines whether to run in a docker container based on the absence of the swagger binary on the host machine. If the swagger binary exists, it will be used instead.

I see you're switching from docker to apt-installed executable to generate code from swagger. What's the reason behind it? Docker comes prebundled with Github Workflow, so this should work out of the box. The advantage of using a go-swagger docker image is, that the version is always identical on workstations and Github pipelines alike. This would ensure identical code generation.


Support Paginator async filtering?

Example from discussion on discord:
async def get_filtered_next_page(
    paginator: Paginator,
    filter_callback: Callable,
    n_results: int = 10,
    max_requests: int = 10,
) -> AsyncIterator:
    """Get at most `n_results` filtered results from the paginator, in at most `max_requests`
    actual requests.
    """
    n_filtered_results = 0
    start_page = paginator.kwargs['page']

    async for result in paginator:
        # Yield the next result if it passes the filter
        if await filter_callback(result):
            yield result
            n_filtered_results += 1

        # Stop if we've either reached the needed number of results, or we've made too many requests
        reached_max_requests = (paginator.kwargs['page'] - start_page) == max_requests
        if n_filtered_results == n_results or reached_max_requests:
            break

This example would still work, but the only part I don't like about it is that you have to know about some internal attributes (paginator.kwargs['page']) which might change at some point. I could probably make that easier to do. Maybe a separate Paginator.filter() method or something?


Show intermediate progress when running benchmarks

Closes https://github.com/mlcommons/mobile_app_open/issues/348

Screenshots


Exclude jsr311 lib from swagger jersey2 library

swagger-jersey2-jaxrs brings jsr311 library which has conflicting
APIs with jersey2 we are using.

Current coverage is 64.28% (diff: 100%)

Merging #819 into master will decrease coverage by <.01%

@@             master       #819   diff @@
==========================================
  Files           723        723
  Lines         33997      33997
  Methods           0          0
  Messages          0          0
  Branches       5195       5195
==========================================
- Hits          21857      21854     -3
- Misses        10608      10610     +2
- Partials       1532       1533     +1

Powered by Codecov. Last update 70a172d...c08c85e


how to put "Authorization: Bearer {token}" in authorization header?

Tesla.build_client([
      {Tesla.Middleware.Headers, [ {"user-agent", "my-app"}, {"Authorization", "Bearer #{access_token}"} ] },
      {Tesla.Middleware.BaseUrl, base_url},
    ])

I tried this but it seems not working.
how can I do this?

This code looks good. How do you check if it’s working or not?
On 11 Jul 2018, 20:11 +0200, Tae Hwan Kim<EMAIL_ADDRESS>wrote:

Tesla.build_client([
{Tesla.Middleware.Headers, [ {"user-agent", "my-app"}, {"Authorization", "Bearer #{access_token}"} ] },
{Tesla.Middleware.BaseUrl, base_url},
])
I tried this but it seems not working.
how can I do this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes it works!
other things cause the problem. Sorry!


Command to restart container

Firstly, thanks for the amazing tool, it works brilliantly for us.
Is your feature request related to a problem?
We use devspace with Ruby on Rails. Most of the time, the hot code reload works brilliantly and the onUpload.restartContainer is set to false. However, when working in rails, it is sometimes necessary to restart the application (such as when changing an initializer).
Restarting the container would achieve this as well, so our current solution involves changing restartContainer to true, cancelling and then restarting devspace dev before changing the code to trigger the reload, before setting restartContainer back to false.
Which solution do you suggest?
A CLI command to perform the exact action that restartContainer performs when the code changes.

/kind feature

Thanks for opening this issue. That's an interesting case where you want auto-restart disabled but on-demand restarts as user-friendly and quickly as possible. With the current capabilities, you could run devspace dev -d but that won't always work because the deployment may not have changed. Another option is devspace dev -b which forces a new tag to be built and therefore, it should always restart the containers. However, I get that image building may take a while and is not as user-friend. I actually have the same issue with another application as well but I always did the image rebuilding. But not every team has images which are as cache-optimized as the one I'm working with in this case.
An idea that just came to my mind to make manual restarts easy to trigger: How about a hotkey like pressing "r" or pressing  or something like that which tells devspace to run trigger the hot-reload (aka kill the process pid2 process to that the restart helper kicks in and restarts the application again). We could print a message about this capability before starting the log stream and then just reading the stdin while doing log streaming.
@tom-acceleratecompliance What do you think about this idea?
@FabianKramm Any thoughts on this?

Hi @LukasGentele - thanks for taking a look at this!
I had attempted to use devspace dev -d/-b but ran into the issue you mentioned around how long the container takes to rebuild. I don't know the internals of devspace, but would something like devspace dev -r be possible? Expected behaviour being "Start devspace while immediately restarting the container as though restartContainer was set to true and a code change had occured"
This feels very user friendly to me, kill devspace dev with a control+c, start again with devspace dev -r.
One workaround we did find if we're doing work which requires server restarts frequently is to change the container yaml to just sit and wait:
command:
  - tail
args:
  - "-f"
  - "/dev/null"

Then we run devspace enter and start the server normally with puma or rails s. We can just kill and restart the server as needed in that context as required.
To any rails devs reading this, this approach has the added advantage of allowing byebug/pry to work nicely.

@tom-acceleratecompliance the flag should be even easier to do.
I know a few teams who use DevSpace using rails. A common pattern is what you described. Instead of opening the terminal in a second window, you can also tell DevSpace to open the terminal instead of opening the log stream. Take a look at dev.interactive for the relevant config options

Implemented with v5.1.0


the hw of mask img is different with the inout img?

why

input

Mask height and weight should be the same as in image. Resize it inside workflow.


fix(kafka): DescribeGroup shouldn't return an error when encounter a non-existent groupId

PR Description
Type of change

[x] Bug fix

Checklist

I have run format.sh under script
I have commented my code, particularly in hard-to-understand areas
New and existing unit tests pass locally with my changes

Split node checking & log modification into other (two) PRs? I am not sure.

Split node checking & log modification into other (two) PRs? I am not sure.

Or you can merge #1811 first, then I will fix lookup and other part later


Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

Version Control, collaboration.


Add the missing cmake file for parallel stl

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (161f9fc) 91.43% compared to head (1629f1d) 91.43%.
 Consider uploading reports for the commit 8f5cff2 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2618   +/-   ##
========================================
  Coverage    91.43%   91.43%
========================================
  Files          460      460
  Lines        17356    17356
========================================
  Hits         15870    15870
  Misses        1486     1486

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


upgrade vulnerable lib

https://github.com/open-cluster-management/backlog/issues/15962
Signed-off-by: Marco<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/retest


[Bug Report] Disabled actionbars 6-8 are showing

Version:
DragonflightUI v0.11.5
Game Version:
Cataclysm
Issue:
Upon login actionbars 6-8 are showing even when disabled.

Hey,
I found some blizzard spaghetti code which updates the whole button when show/hide keybind/macro text in some circumstances; should be fixed with f996087d3df8c049ecec2de1420c278eb598f094


Get team from App CR annotation

Towards https://github.com/giantswarm/giantswarm/issues/16061
Check the App CR for the team annotation before checking the AppCatalogEntry CR. This lets the user override the value in Chart.yaml.
kg get app app-exporter-unique -o yaml | k neat
apiVersion: application.giantswarm.io/v1alpha1
kind: App
metadata:
  annotations:
    application.giantswarm.io/team: robin
...
  name: app-exporter-unique
  namespace: giantswarm
spec:
  catalog: control-plane-test-catalog
  kubeConfig:
    inCluster: true
  name: app-exporter
  namespace: giantswarm
  version: 0.3.0-0382a48359522e3221ac1c78c2003609baacc2be

curl -s http://localhost:8000/metrics | grep app-exporter
app_operator_app_info{app="app-exporter",catalog="control-plane-test-catalog",name="app-exporter-unique",namespace="giantswarm",status="deployed",team="robin",version="0.3.0-0382a48359522e3221ac1c78c2003609baacc2be"} 1

Checklist

[x] Update changelog in CHANGELOG.md.

Tested on gremlin.


Support Go Modules, updated providers.json

Hey,
thanks for the library. I already added it to two projects in the last weeks ;)
This PR contains a few small changes:

I added the files for Go Modules support
Some cosmetic changes to the README
Updated the providers.json to the current state

Best,
Felix

Hi Felix, thank you for the MR, I merged it.


Not enough clarity on the direction of a service endpoint

In your first sentences, it seems like the endpoint is outgoing, connecting to external services, which is what I thought a service connection was for, after your list of properties, the implication seems to be that its now incoming a way for extensions to acquire information... which is it? Also, the title of this document is Service Endpoints in Azure DevOps, but there is just a few sentences on the actual topic of Service Endpoints, this is a topic on the AZ-400, and should have a lot more depth and not just an example on creating an extension.

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

ID: 90db6e4c-6240-fe2e-3043-0a7d186b9ca7
Version Independent ID: a62ec500-3330-e28a-8019-118a3b9e261f
Content: Service Endpoints - Azure DevOps
Content Source: docs/extend/develop/service-endpoints.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @chcomley
Microsoft Alias: chcomley

@kelsosharp Thank you for the feedback, assigning this to the author for review.

Hi @kelsosharp, Please accept my apology for losing sight of this issue. There is a more recent issue for this article, however, which I've added your feedback to so we can be sure it's properly addressed. I'm closing this issue, so please reference this issue and you can @mention me if you have any further questions or concerns.
We're working on a complete overhaul of this area and are in the beginning phases of that work now.
Thank you!


Check JDK11 and riscvcross roles on RHEL7

Saw an issue with this new role on RHEL7 when I tried it this week. Also JDK11 doesn't appear to have been installed on there and we should have it available.

Running playbook on a RHEL7.5 machine now 👍


Design history page

This task is aimed at designing and implementing the history page, which lists all user transactions, including donations and fund distributions
Fixes #62

I've deployed this to review1.


add jammy and x64 labels for self-hosted runners

Description
Adds the jammy and x64 labels to the self-hosted runner workflows to ensure they don't break as we add more architectures and Ubuntu OS versions.

/canonical/self-hosted-runners/run-workflows 5e2ca9b3d1d88657ad1531c9b05c79a8b03cb79b


fixed URI construction when the server URI included a specific path (http://localhost:1337/parse)

this may be needed by setting up a server to use parse on a specific path as follows:
var express = require('express');
var ParseServer = require('parse-server').ParseServer;

var config = require('./config.js');

var app = express();
var api = new ParseServer(config.parse);

// Serve the Parse API on the /parse URL prefix
app.use('/parse', api);

app.listen(1337, function() {
  console.log('parse-server-example running on port 1337.');
});

Using the current parse-server (v2.2.13) and parse .Net client  SDK (v1.7.0) you can use the following client code.
ParseClient.initialize(new ParseClient.Configuration {
    ApplicationId = "YOUR_APP_ID",
    Server = "http://YOUR_PARSE_SERVER/parse/" // trailing slash is important
  });

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@thewizster Is correct. The 'proper' way is to simply add a trailing slash to the end of your URI. Without it, the Uri constructor just uses the RFC behavior for constructing a relative path, as should be expected.

@thewizster thank for your prev comment, I forgot about it as it has been a busy month


ValueError: Protocol message Document has no "type" field.

Hello,
I've got an issue since I've passed my project to google-cloud-language 2.0.0.
I tried on the Cloud Shell Protocol offered by the official documentation. This bug appears everytime.
Environment details

OS type and version: MacOS - Catalina
Python version: Python 3.7.0
pip version: pip 20.2.4
google-cloud-language version: 2.0.0

pip freeze
asgiref==3.2.10
cachetools==4.0.0
certifi==2019.11.28
chardet==3.0.4
colorama==0.4.4
cygrpc==1.0.4.post6
google-api-core==1.23.0
google-api-python-client==1.12.4
google-auth==1.22.1
google-auth-httplib2==0.0.4
google-cloud==0.34.0
google-cloud-language==2.0.0
googleapis-common-protos==1.6.0
grpcio==1.31.0
grpcio-tools==1.23.0
idna==2.8
libcst==0.3.13
oauth2client==4.1.3
packaging==20.4
pkginfo==1.6.0
proto-plus==1.11.0
protobuf==3.13.0
Steps to reproduce

I use the sample code of the library here : samples/v1/language_syntax_text.py
python3 samples/v1/language_syntax_text.py

Code example
def gcl_analyze_syntax(sentence):
    client = language_v1.LanguageServiceClient()
    type_ = language_v1.Document.Type.PLAIN_TEXT
    language = "fr"
    document = {"content": sentence, "type": type_, "language": language}
    encoding_type = language_v1.EncodingType.UTF8
    tokens = client.analyze_syntax(request = {'document': document, 'encoding_type': encoding_type}).tokens
    return tokens

or
def sample_analyze_syntax(text_content):
    """
    Analyzing Syntax in a String
    Args:
      text_content The text content to analyze
    """

    client = language_v1.LanguageServiceClient()

    # text_content = 'This is a short sentence.'

    # Available types: PLAIN_TEXT, HTML
    type_ = language_v1.Document.Type.PLAIN_TEXT

    # Optional. If not specified, the language is automatically detected.
    # For list of supported languages:
    # https://cloud.google.com/natural-language/docs/languages
    language = "en"
    document = {"content": text_content, "type": type_, "language": language}

    # Available values: NONE, UTF8, UTF16, UTF32
    encoding_type = language_v1.EncodingType.UTF8

    response = client.analyze_syntax(request = {'document': document, 'encoding_type': encoding_type})

Stack trace
Traceback (most recent call last):
  File "samples/v1/language_syntax_text.py", line 110, in <module>
    main()
  File "samples/v1/language_syntax_text.py", line 106, in main
    sample_analyze_syntax(args.text_content)
  File "samples/v1/language_syntax_text.py", line 54, in sample_analyze_syntax
    response = client.analyze_syntax(request = {'document': document, 'encoding_type': encoding_type})
  File "/usr/local/lib/python3.7/dist-packages/google/cloud/language_v1/services/language_service/client.py", line 533, in analyze_syntax
    request = language_service.AnalyzeSyntaxRequest(request)
  File "/usr/local/lib/python3.7/dist-packages/proto/message.py", line 461, in __init__
    pb_value = marshal.to_proto(pb_type, value)
  File "/usr/local/lib/python3.7/dist-packages/proto/marshal/marshal.py", line 205, in to_proto
    pb_value = rule.to_proto(value)
  File "/usr/local/lib/python3.7/dist-packages/proto/marshal/rules/message.py", line 32, in to_proto
    return self._descriptor(**value)
ValueError: Protocol message Document has no "type" field.

Thanks,
Arthur

I am having the same problem. Let me know if you salved this somehow. Thanks!

i have the same question. And when i downgrade my google.cloud.language to 1.3.0, python still went wrong saying "do not have the attribute document"

Okay thank you @busunkim96.
For those who didn't have understand, here's the code fixed.
def gcl_analyze_syntax(sentence):
    client = language_v1.LanguageServiceClient()
    type_ = language_v1.Document.Type.PLAIN_TEXT
    language = "fr"
    document = {"content": sentence, **_"type_": type__**, "language": language}
    encoding_type = language_v1.EncodingType.UTF8
    tokens = client.analyze_syntax(request = {'document': document, 'encoding_type': encoding_type}).tokens
    return tokens

The samples and RST files need to be updated with this change.

Fixed in #49 .


Add support of 128 bit keys to 3DES CMAC

I asked this question awhile ago
https://github.com/ARMmbed/mbedtls/issues/4195
I checked the handling of 128 bit keys in mbedTLS part of OP-TEE and it turned out that the fix is actually trivial.
I'm about to add the same (almost) code to OP-TEE and I think that it might also make sense to add it to the original mbedTLS.
With this tiny change one can use
cipher_info = mbedtls_cipher_info_from_type(MBEDTLS_CIPHER_DES_EDE_ECB);
ret = mbedtls_cipher_cmac_starts(&ctx, mac_des3_cmac_vect6_key_128, 128);

with 128 bit key and get the same result as with
cipher_info = mbedtls_cipher_info_from_type(MBEDTLS_CIPHER_DES_EDE3_ECB);
ret = mbedtls_cipher_cmac_starts(&ctx, mac_des3_cmac_vect6_key_192, 192);

without having to copy K1 to K3 (extending 128 bit key to 192 bits)
Simple test run
====================================
3DES CMAC test using 128 bit key and MBEDTLS_CIPHER_DES_EDE_ECB:
====================================
Key (128 bits):
61 c4 ec 20 70 e9 bf 2a
ec 02 d0 b0 e9 9e 8f 01

Input data:
ff bd 75 61 93 91 88 bc
Output:
55 a0 4c 23 d5 b4 3e 49
====================================
3DES CMAC test using 192 bit key and MBEDTLS_CIPHER_DES_EDE3_ECB:
====================================
Key (192 bits):
61 c4 ec 20 70 e9 bf 2a
ec 02 d0 b0 e9 9e 8f 01
61 c4 ec 20 70 e9 bf 2a

Input data:
ff bd 75 61 93 91 88 bc
Output:
55 a0 4c 23 d5 b4 3e 49

Please could you add some tests. It will also need a DCO (sign-off, as per https://github.com/ARMmbed/mbedtls/blob/development/CONTRIBUTING.md), and a Changelog entry. Thanks!

Thanks for this - however given that this has not updated for some time, and that 3DES is approaching end-of-life (see e.g. https://www.cryptomathic.com/news-events/blog/3des-is-officially-being-retired ), it probably doesn't make sense to add any new 3DES functionality to Mbed TLS.


Merge pull request #1 from MicrosoftDocs/master

Update from Original

@rojasja,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Final VMS removal

This is dependent on #151 being merged first.
Checklist
Thank you for your contribution. Please answer the following:

[X] I acknowledge that I am authorized to submit this code under
the terms of the Apache License
[ ] All new public APIs are documented
[ ] All new public functions have tests
[ ] All new public functions have fuzzing tests

Rebased, fixed merge conflicts, and pushed.

Huzzah!  Consigned to dustbin of history.  Thanks folks!


I found that changing the broker's palin_ Acl. yaml has no effect, but changing proxy's plain_ Acl.yaml can take effect, accessed through proxy in the k8s environment

I found that changing the broker's palin_ Acl. yaml has no effect, but changing proxy's plain_ Acl.yaml can take effect, accessed through proxy in the k8s environment

Proxy and broker have their own ACL configuration file. If accessed through proxy, the ACL configuration of proxy takes effect for the client.


FacebookAI/xlm-roberta-large-finetuned-conll03-english

Hey
Does this work with FacebookAI/xlm-roberta-large-finetuned-conll03-english too? And where can I find base_vocabulary.json?

Hi @AiTester950,
Sorry, I think it doesn't.
FacebookAI/xlm-roberta-large-finetuned-conll03-english is a finetuned XLM-RoBERTa, which means it uses XLMRobertaTokenizer, which uses SentencePiece. RobertaTokenizer uses byte-level Byte-Pair-Encoding, which is what's implemented in this repo.


keystone: also build shared library

Motivation for this change
the library was also installed in a weird prefix.
Things done

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

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

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


Configurator web interface is slow to load

The web interface for the configurator takes about 20 seconds to load for me (didnt time, but it's slow)
Any idea what is causing this?
I'm using Firefox on Windows 10.

This is something that is known to be an ESP32 thing. I'm trying out some alternatives to the WebServer library I'm using now. My experience is that the more requests you send to the ESP32 the slower it gets.

09bc3dc1379095a75623b51246f3c00995af1fde Makes the switch to ESPAsyncWebServer. Although this requires the user to install two additional libraries, I think the speed gained by this is very much worth it. If you experience any speed/reliability issues, feel free to re-open.


Add note in vagrant docs about sudo

can we not bake this into the scripts?
I'm also wondering a couple of things... Should we -

Lock on minimum vagrant version https://docs.vagrantup.com/v2/vagrantfile/vagrant_version.html (i think most of us have been testing on 1.7.2?)
Add an if around override.vm.provision :hosts to test if the vagrant-hosts plugin is installed
Forward the ports for the web interfaces in the Vagrantfile (incase the vagrant-hosts plugin is disabled?)

Was going to bake it into the scripts, but wasnt sure. I'll add a minimum vagrant version of 1.7, as thats when things changed in the config.ssh. Will also add the hosts test as well

lets do min version and the if wrapper for now then and just a docs fix

Add check for vagrant and if wrapper. Didnt do port forwarding as we have the hosts on private network

Code here LGTM, docs look a bit suspect.

Aren't we already checking for if the plugin is installed, so isn't the check for the plugin during the rest of the vagrant file redundant?
throw error if vagrant-hosts not installed
unless Vagrant.has_plugin?("vagrant-hosts")
raise "vagrant-hosts plugin not installed"
end


Obstacle Avoidance Replan Error

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
Hello, we are testing the autoware stack in the simulation with our map. Most things run out fine, except one problem: as shown in the video, the ego vehicle tried to yield to a background vehicle at a roundabout by performing a path replan but somehow triggered an error. We have looked into the issue and tuned some parameters in the avoidance files but the error persists. I have attached a demo video and the raw log output from autoware. Please take a look, any fix or explanation is appreciated!
log.txt
https://github.com/autowarefoundation/autoware.universe/assets/74877573/05a34b56-9f42-4318-b1d1-9c7374ca74ce
Expected behavior
Autoware vehicle successfully avoids the background vehicle.
Actual behavior
Autoware vehicle reported planning and control error during yielding.
Steps to reproduce
For confidential reasons we cannot provide the full map of our testing environment. If needed, we are willing to provide a partial map to help reproduce the issue! Thank you!
Versions
No response
Possible causes
No response
Additional context
No response

Excellent! After applying the changes, the situation has been largely improved.
Could you please explain the changes and why the previous design was not working?


Optimize Window monoid a bit

Rather than switching inside the monoid, switch on creation so the JIT can optimize better.

Codecov Report

Merging #652 into develop will increase coverage by 0.05%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #652      +/-   ##
===========================================
+ Coverage    82.93%   82.98%   +0.05%
===========================================
  Files          109      109
  Lines         5209     5232      +23
  Branches       316      323       +7
===========================================
+ Hits          4320     4342      +22
- Misses         889      890       +1

Impacted Files
Coverage Δ

...e/src/main/scala/com/twitter/algebird/Window.scala
96.29% <100%> (+22.38%)
:arrow_up:

...src/main/scala/com/twitter/algebird/Priority.scala
16.66% <0%> (-8.34%)
:arrow_down:

...main/scala/com/twitter/algebird/monad/Reader.scala
52.94% <0%> (-5.89%)
:arrow_down:

.../main/scala/com/twitter/algebird/Applicative.scala
61.29% <0%> (-3.23%)
:arrow_down:

.../main/scala/com/twitter/algebird/Successible.scala
91.66% <0%> (ø)
:arrow_up:

.../main/scala/com/twitter/algebird/HyperLogLog.scala
92.8% <0%> (+0.39%)
:arrow_up:

.../main/scala/com/twitter/algebird/BloomFilter.scala
95.27% <0%> (+0.42%)
:arrow_up:

...c/main/scala/com/twitter/algebird/MapAlgebra.scala
77.88% <0%> (+0.96%)
:arrow_up:

.../main/scala/com/twitter/algebird/Approximate.scala
90.47% <0%> (+1.58%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


⚠ Bofrost.es has degraded performance

In 3a2b2e8, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1535 ms

Resolved: Bofrost.es performance has improved in ef8b898 after 9 minutes.


Create add organization functionality by connecting the form to the API  Work Products

Overview
Create add organization functionality by connecting the form to the API  Work Products
Action Items
We want to refine the tag generator page to help project owners find and generate tags to add to their repository so that they can be included in the Civic Tech Index. This is a key functionality that will help to include more projects and organizations in the index.

[ ] Connecting the form to the API

Resources/Instructions
Figma Design Document

Existing site

@kliu28 do you still have availability to finish this?

@Olivia-Chiong unfortunately no--someone else can pick this up instead :)

Needs form validation. All fields are returning errors at current iteration.


chore(readability): Made code more readable

Description of changes:
While reviewing the VPC helper code looking to improve the architecture, decided the design is actually decent but it was hidden by hard to read code.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 94d6e66fe53e1cfa1dbea341dcb70510608b4d30
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


estimated fee - add to send and convert screen

What would you like to be added:

Add estimated fee info into send and convert screen only when amount input is filled

Why is this needed:

To inform user of transaction fee

Refer to test case here: #946


#3628 add support for IsRepeatable not being specified on directives

Add support for IsRepeatable not being specified in GraphQL schemas

Directive.IsRepeatable becomes nullable
When Directive.IsRepeatable is null, code assumes that the the directive is not repeatable

Closes #3628

/AzurePipelines run

Will be included into 11.3.0-preview.5


[CELEBORN-1351] Introduce SSLFactory and enable TLS support

What changes were proposed in this pull request?
Add SSLFactory, and wire up TLS support with rest of Celeborn to enable secure over the wire communication.
Why are the changes needed?
Add support for TLS to secure wire communication.
This is the last PR to add basic support for TLS.
There will be a follow up for CELEBORN-1356 and documentation ofcourse !
Does this PR introduce any user-facing change?
Yes, completes basic support for TLS in Celeborn.
How was this patch tested?
Existing tests, augmented with additional unit tests.

The test failure does not look related to this PR (and the same tests passed here):
- celeborn spark integration test - pushdata timeout will add to pushExcludedWorkers *** FAILED ***

+CC @otterc, @FMX, @RexXiong, @SteNicholas, @waitinfuture

@mridulm, did you update any document? There are unnecessary changes of document in this pull request. Please check the changes of document.

Oh no, this is some weird artifact of pulling from my other branch - let me revert those, sorry about that !

To answer query about which modules are relevant, all modules can be secured using TLS.
At a minimum, given shared secret is negotiated when LifecycleManager registers with Master, it is advisible to secure rpc calls - towards that end, we will split the rpc calls within the application (driver/executors) and from client to server's (see CELEBORN-1356), so that users can independently configure those.

Codecov Report
Attention: Patch coverage is 72.83237% with 47 lines in your changes are missing coverage. Please review.

Project coverage is 40.12%. Comparing base (6153ba4) to head (d80bacd).

Files
Patch %
Lines

...apache/celeborn/common/network/ssl/SSLFactory.java
73.28%
17 Missing and 14 partials :warning:

...ache/celeborn/common/network/TransportContext.java
72.23%
8 Missing and 2 partials :warning:

.../common/network/client/TransportClientFactory.java
77.78%
3 Missing and 1 partial :warning:

...apache/celeborn/common/rpc/netty/NettyRpcEnv.scala
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2438      +/-   ##
==========================================
+ Coverage   39.24%   40.12%   +0.88%
==========================================
  Files         217      218       +1
  Lines       13432    13600     +168
  Branches     1173     1195      +22
==========================================
+ Hits         5270     5455     +185
+ Misses       7863     7831      -32
- Partials      299      314      +15

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

@SteNicholas, please do let me know if I can help clarify better ... hopefully the details above help.

I will merge it tomorrow if it is still open.

LGTM. Merging to main(v0.5.0).

Thanks for the reviews @SteNicholas, @otterc !


Ability to pass arguments to the threadpool init.

https://github.com/ImagicTheCat/love-threadpool/commit/5845f86fbecfa6af72c0513b5ed5f47fea03ff24


Add cute robot animation

Issue

close #380

Overview (Required)

Add cute droid robot animation at end of  feedlist in Home.
Is this cute? If you have better animation ideas, please let me know!

Links

Screenshot

Before
After

Thank you! It feels good enough! This is my taste, but what about anime that bounces up a little? 😄

Thank you! It feels good enough! This is my taste, but what about anime that bounces up a little? 😄

Thank you for your idea! I fixed animation and I updated After Screenshot movie. Please confirm 🙏


callable predicates for filter_by and group_by

Proposal to add callable predicates to ShapeList's filter_by and group_by.

filter_by accepts an arbitrary Callable[[Shape], bool] type in additon  to Axis and GeomType.
group_by accepts an arbitrary Callable[[Shape], K] (where K is the grouping key) type in addition to Axis and SortBy.
additionally group_by returns a GroupBy object instead of a list. this object is still subscriptable by index, but also allows to retrieve the groups by key or by shape.
internally the old argument types are converted to callable, this should simplify the code and facilitate future additions.

The PR is not merge-ready (I haven't updated the docstrings and there may be a few typing tweaks to do) but here it is for discussion.

Interestingly a single test failed on only one of the 6 runners. @gumyr is that a thing known to happen, or do you think I broke the code in an non-obvious way?

I've seen a single test fail like that a few times.  I'm used to this in very large real-time systems which are difficult to make fully deterministic but to see it here is a little worrisome. I haven't really spent any time investigating but at first glance I didn't see a pattern.

to see it here is a little worrisome. I haven't really spent any time investigating but at first glance I didn't see a pattern.

A few commits later the single seemingly unrelated test (test_regular_polygon_matches_polar) that failed the first time only on 3.10, windows is now passing. Should we blame cosmic rays at the github datacenter and move on? (unless more intermittent failures pop up in the future)


📝 Documentation: Use a more universally appealing description

Bug Report Checklist

[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Current description:

Template TypeScript Node.js package with all the CI bells & whistles I commonly use. ✨

That's not great because it's not super descriptive. "I commonly use" isn't particularly convincing.
I'm thinking a replacement like maybe one of...

A TypeScript template with all the juicy tooling you could ever want.
A quickstart-friendly TypeScript package template with high grade formatting, linting, and automatic releases.

Additional Info
...it's not yet clear to me what the best next description would be?

another chat GPT suggestion: "Save time and reduce errors in your TypeScript projects with this battle-tested package template, complete with essential tools for formatting, linting, testing, and seamless releases."

I'm going with:

A TypeScript template with comprehensive formatting, linting, releases, testing, and other great tools built-in.

✨
Thanks for the help!


Fix logging race condition

Motivation

One of our unit tests has been failing due to a race condition bug in our logging code.

Modifications
Change summary
replaced wait() with wait_for(), which establishes a maximum duration for the thread to unblock itself if the condition variable never gets notified.
Testing

tested via code tracing and lots of test runs, will need additional time to completely verify fix.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

the only question is how long to set the rtime argument for

@RogerZhongAWS I have not been able to reproduce the flakey test on my local machine, although I have observed it on ci. Can you try adding an additional step to the CI build to repeat the test (as shown below) with and without this change so that we can have confidence that your change resolves the issue?
./test/test-aws-iot-device-client --gtest_filter=LogQueueTest.notifyAllOnShutdown --gtest_repeat=100