*: fill in WatchResponse.Header

Related to https://github.com/coreos/etcd/issues/3848.

/cc @xiang90 @heyitsanthony  Please take a look if I am doing this right.
I tested it and worked:
[Watch] Started! (endpoint: localhost:1278)
resp.Header: cluster_id:1816128046471434258 member_id:12892637578336944577 revision:100 raft_term:2
watcher created: id 00000000
resp.Header: cluster_id:1816128046471434258 member_id:12892637578336944577 revision:100 raft_term:2
PUT: mHSJEEiKRs bMuKkkepZJ
resp.Header: cluster_id:1816128046471434258 member_id:12892637578336944577 revision:100 raft_term:2
PUT: mHSJEEiKRs HCoHlYldXk

Just fixed method naming (remove Get).

@heyitsanthony Just fixed as in your feedback. PTAL. Thanks.

overall lgtm, but @xiang90 is more familiar with this code

@xiang90 Just fixed. Please take a look again. Thanks!

@xiang90 Just fixed. PTAL. Think we should delete revision from header?
resp.Header: cluster_id:8073902162752198467 member_id:14246209564175586634 raft_term:2
[revision] 0
watcher created: id 00000000

resp.Header: cluster_id:8073902162752198467 member_id:14246209564175586634 raft_term:2
[revision] 101
PUT: lkQOkGZJiY uhaxRpWjOj

resp.Header: cluster_id:8073902162752198467 member_id:14246209564175586634 raft_term:2
[revision] 102
PUT: lkQOkGZJiY DKPPGvKcXO

I think the header revision is important for watching-- e.g., do a range request, see nothing is there, then watch starting at the header rev for updates.

Yeah will fix. I misunderstood. Thanks,
On Wed, Jan 6, 2016 at 11:10 AM, Anthony Romano<EMAIL_ADDRESS>wrote:

I think the header revision is important for watching-- e.g., do a range
request, see nothing is there, then watch starting at the header rev for
updates.
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/etcd/pull/4138#issuecomment-169423506.

--
Sincerely,
Gyu-Ho Lee

@xiang90 @heyitsanthony Just fixed based on the feedback. PTAL.
Thanks!

@xiang90 Just fixed some test errors. Thanks!

To double check I ran the watch server and it worked:
[Watch] Started! (endpoint: localhost:1278)

resp.Header: cluster_id:12892637578336944577 member_id:1816128046471434258 revision:100 raft_term:2
[revision] 100
watcher created: id 00000000

resp.Header: cluster_id:12892637578336944577 member_id:1816128046471434258 revision:101 raft_term:2
[revision] 101
PUT: WFcqPvPKjr eJFcJpNhFG

resp.Header: cluster_id:12892637578336944577 member_id:1816128046471434258 revision:102 raft_term:2
[revision] 102
PUT: WFcqPvPKjr vWqLKzZlUh

@xiang90 I added verbose flag. Sample output:
./etcdctlv3 --endpoint="localhost:1178" watch --verbose
watch a
[revision: 7]
watcher created: id 00000000
[revision: 8]
PUT: a ddd
[revision: 9]
PUT: a ddddfdf

LGTM

Thanks, will merge after CI passes.


PHP 8.1 deprecations

Currently deprac seems to output some PHP 8.1 deprecations:
> @php vendor/bin/deptrac
PHP Deprecated:  Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 14

Deprecated: Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 14
PHP Deprecated:  Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 19

Deprecated: Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 19
PHP Deprecated:  Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 14

Deprecated: Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 14
PHP Deprecated:  Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 19

Deprecated: Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///Users/test/Documents/Projects/int3grate.localhost/vendor/qossmic/deptrac-shim/deptrac/.box/src/RequirementCollection.php on line 19

I checked the box project and it seems to be fixed there: https://github.com/box-project/box/blob/3.16.0/requirement-checker/src/RequirementCollection.php Seems like deptrac was created with an older version of the box project?

@dbrumann thank you!


Typo in Discussions GraphQL API docs

What article on docs.github.com is affected?
Using the GraphQL API for Discussions, added in 87699fd119211756b05dabfee535830af92d1082.
What part(s) of the article would you like to see updated?
Right now it says:

Get a discussion. Returns null if discussion with the specified ID exists.

While I believe it should say:

Get a discussion. Returns null if discussion with the specified ID does not exist.

@radekmie Thank you so much for opening an issue 💖 You're right! That should say Get a discussion. Returns null if discussion with the specified ID does not exist.
You or anyone else is welcome to open a PR to make that fix!


Export workflow is different when exporting via Admin UI vs programatically

Describe the bug
We define two ways to export data

Using the Admin UI integration
Programmatically by using Resource.export()

If you use both of these methods you will find that the results you end up with are different.  Note that the Admin UI will give you a file download as output, whereas export() will give you a tablib Dataset object.  However, only the Admin method will call Dataset.export_data() (which in turn means that Dataset.export() gets called).
I believe this should be corrected so that the export data is created from the same Tablib representation.  For example, this means that security fixes such as #1546 do not get applied when calling export() programmatically.
Note the fix to this will have to ensure that existing implementations are not disrupted.

Also note that you can export via form action (e.g. for categories)

Closing this as not an issue.  The workflow is different but I think this is correct behaviour.

If calling via the Admin UI, you are choosing a file format to export to, hence tablib export can be called on the appropriate format (eg. xlsx, csv etc)

If clients calling export() directly then they are receiving a raw dataset, and then can call tablib's export method themselves, and apply any escaping args as appropriate.

Here is the workflow:
admin.py
export_action()
  (ExportMixin) get_export_data()
    (BaseExportMixin) get_data_for_export()
      (Resource) export()
  (Format) export_data()
    (Dataset) export()

Programmatic export calls only
(Resource) export()

The form action export (for categories) has the same workflow as Export button export in the Admin UI.


Wrong spelling (german version) "Verkaufskänale"

Used Shopware Version: <IP_ADDRESS>
Description:
In the German version of the configuration of the language pack, under Einstellungen > Erweiterung > Sprachpaket is a wrong spelling in the name of the tab Verkaufskanäle (see screenshot).
Environment:
Shopware <IP_ADDRESS>
Language Pack: 2.2.0
[screenshot-localhost-2022.07.19-11_32_51.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1658244816_62d6ced0badd6_1658244812_62d6ceccb38b6_screenshot-localhost-2022.07.19-11_32_51.png]

[public] Shopware automated response: This issue is linked to the internal issue: LAN-110
[created from LAN-110, comment 514149]


This is not supported videojs version5

its doesn't work with video-js.5/video.min.js, how can be worked with video-js.5/video.min.js
Thanks,
Pon

I confirm, it does not work with videojs 5.

Did you guys get any solution for this.?

See #86


Image should stretch with in Post and should be responsive

https://wordpress.org/support/topic/image-in-post-width/

https://www.dropbox.com/s/d3ba45pa2hki16o/Image-Width.jpg?dl=0


0.7.6 is not fully published

Hi!
The latest version for @pintora/standalone is 0.7.5 whereas it is 0.7.6 for other packages, it seems there's a CI pipeline which failed.

Thank you for the issue , but it's alright because they are not always bumped together.  The latest version of standalone is 0.7.5.
I will figure out some other way to avoid this confusion, though.


Update README.md

Chore : Added JS Challenger Interview HandBook in Readme.md File

@DhanushNehru can you review it and merge my pr if follows guideline

Thanks for your contribution @YashkShrivas4491


TVPaint: Added init file for worker to triggers missing sound file dialog

Brief description
TVPaint worker will open TVPaint init file which has missing sound file so all missing sound files can be ignored at the start of worker.
Description
This dialog can be triggered randomly at any time on worker which is "not nice" as someone has to connect to worker machine and check "Ingore missing dependency" and hit "ok". Because of that the dialog can be shown at the start of worker so for rest of running process are missing sound files ignored.
Testing notes:

Run TVPaint worker
Dialog showing missing sound file should show at the start

We have the issue on occasion as well when rendering on Deadline.
Have you found any validations we could maybe check before sending it off to render?
Also am I right in thinking that the following is the workflow for getting around this?

open a temp TVP file that has missing sounds dependencies
force close the TVP file
now all TVP files opened after this will ignore the missing sound dependencies

Have you found any validations we could maybe check before sending it off to render?

Only way that could work is to remove all audio references or in theory could work to read the file as binary, find loaded audio, remove it and store the binary content back :) (not tested).

Also am I right in thinking that the following is the workflow for getting around this?

Yes, that's the goal.


Request body is not allowed for the 'GET' method

During the test execution, the following output is logged:
Request body is not allowed for the 'GET' method

I believe this is due to the fact that Vegeta will not accept a nil value for the byte array during an HTTP Request and the OCM SDK flags this as a potential issue when using GET requests.
We should fix this upstream to reduce the noise. I believe the best place to fix this is in Vegeta, itself.

There is already an issue opened for that, thanks to @nimrodshn
https://github.com/tsenart/vegeta/issues/556

Opened a PR in vegeta
https://github.com/tsenart/vegeta/pull/564


FIX whole page reload when press enter

fix #1024
This is my first contribution to an open source project. Please guide me if I were wrong.

That's a good fix, does this occur only in the InterestModal, or is this something we might need in other modals/forms as well?

As far as I tested, I found page reloading issue when pressed enter only in Interest Modal. But it would be a great user experience if the user can just press enter after filling all the required forms instead of clicking the button.

Let's do it for all the modals, it would be a great help :) Thank you!

OK, I will update my pr soon thanks for the suggestion <3

Thank you so much for the contribution, will make sure it goes out on the next update! 🔥

Thank you so much too for creating this great app. I've learned a lot by reading the source code. I'll try to make more contributions in the future.


Caching buildtrees only works with BuildElement (cannot enter a shell after a failed build)

See original issue on GitLab
In GitLab by [Gitlab user @abderrahimk] on Feb 15, 2020, 14:16
Summary
When I try to enter a shell on a build failure, I get an error instead.
Steps to reproduce

default configuration of buildstream
try to build an element that fails
enter s at the prompt

What is the current bug behavior?
Build failure on element: vm/repo.bst

Choose one of the following options:
  (c)ontinue  - Continue queueing jobs as much as possible
  (q)uit      - Exit after all ongoing jobs complete
  (t)erminate - Terminate any ongoing jobs and exit
  (r)etry     - Retry this job
  (l)og       - View the full log file
  (s)hell     - Drop into a shell in the failed build sandbox

Pressing ^C will terminate jobs and exit

Choice: [continue]: s

Dropping into an interactive shell in the failed build sandbox

Error while attempting to create interactive shell: Buildtree is not cached locally or in available remotes

What is the expected correct behavior?
a shell into the failed build.
Other relevant information

BuildStream version affected: 0591b0c8b038aa9f054b295748f3423384711d50

In GitLab by [Gitlab user @abderrahimk] on Feb 15, 2020, 14:16
changed title from {-c-}annot enter a shell after a failed build to {+C+}annot enter a shell after a failed build

In GitLab by [Gitlab user @abderrahimk] on Feb 15, 2020, 14:16
changed the description

In GitLab by [Gitlab user @abderrahimk] on Jul 4, 2020, 18:59
So there are actually two bugs:

Buildstream assumes the "build tree" is in %{build-root}, which is only true for BuildElement https://gitlab.com/BuildStream/buildstream/-/blob/5daa22ae6f60568849254274fdbf57bde5b919d8/src/buildstream/element.py#L1651
Buildstream silently fails if it can't find %{build-root}, so this goes unnoticed until the buildtree is actually needed https://gitlab.com/BuildStream/buildstream/-/blob/5daa22ae6f60568849254274fdbf57bde5b919d8/src/buildstream/element.py#L1656

BuildElement marks both %{build-root} and %{install-root}, and sets the output directory to %{install-root}. What should be done to be generic enough is to collect the directories that the plugin has marked using Sandbox.mark_directory() with the exception of the output directory (set using Sandbox.set_output_directory(), which is the artifact directory).
Since buildstream now tries to reconstruct the sandbox of the failed build rather than persisting it, this is a regression compared to 1.x.

In GitLab by [Gitlab user @abderrahimk] on Jul 4, 2020, 19:00
changed title from Ca{-nnot enter a shell after a failed build-} to Ca{+ching buildtrees only works with BuildElement (cannot enter a shell after a failed build)+}

I think this can be easily solved with some policy though:

If you think your element should have a build tree that gets cached, then structure your element such that what you want to be cached is under %{build-root} (of course the element implementation can also set %{build-root})

Don't have the expectation that all elements under the sun will cache their build trees, instead exit more gracefully with an informative error explaining that the element in question does not provide support for this (then we'd get hopefully the same error when trying to bst shell into a filter or compose element).

This is tangentially related to #1001

I don't agree with this analysis. Any element that runs stuff in a sandbox should be able to drop into the sandbox to debug failures. I see two ways to go about this:

design a way to cache build trees for all elements with a sandbox
prefer the actual sandbox, and only fall back to reconstructing it from the build trees if it's not locally available

I think we should do both, but either one will fix the most common use case.

Visited this in our meeting, this needs to be revisited in order to determine if this can be fixed without breaking cache keys.


[Release-1.26] - etcd snapshot ConfigMap controller excessive sync requeuing

Backport fix for etcd snapshot ConfigMap controller excessive sync requeuing

#9241

Unable to replicate the issue with k3s or rke2 on older versions.
Checked and confirmed that the error is not observed on the current RC v1.26.14-rc4+k3s1 validation


Broken SQLite API: PRAGMA index_info returns SQL_PARSE_ERROR

alembic 1.13.1
SQLAlchemy 2.0.25
sqlalchemy-libsql 0.1.0

I encountered this when trying to run alembic revision --autogenerate on my turso instance, it raised the following error:
sqlalchemy.exc.DatabaseError: (sqlite3.DatabaseError) SQL_PARSE_ERROR: SQL string could not be parsed: unsupported statement: PRAGMA main.index_info(sqlite_autoindex_userhistory_1);
[SQL: PRAGMA main.index_info(sqlite_autoindex_userhistory_1);]

I also tried running it directly, the same error was raised:
import sqlalchemy
engine = sqlalchemy.create_engine(...)
with engine.connect() as conn:
    conn.execute(sqlachemy.text("PRAGMA main.index_info('sqlite_autoindex_userhistory_1');"))

tried both with and without the main. prefix.
This syntax is part of the SQLite API
The equivalent SELECT * FROM pragma_index_info('sqlite_autoindex_userhistory_1'); works, but this is not what used by alembic.

I have the same problem

@MarinPostma @psarna Is this a limitation in the libSQL server parser?

@ishefi @jedafeza This is a bug in the libSQL server parser, pull request #918 ought to fix it.


Can't get copilot chat to open in Visual studio code

Version: 1.82.2 (user setup)
Commit: abd2f3db4bdb28f9e95536dfa84d8479f1eb312d
Date: 2023-09-14T05:55:25.390Z
Electron: 25.8.1
ElectronBuildId: 23779380
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
it shows release version installed but when I clcik on the extension it takes me to the details page rather than opening the chat window

Does regular copilot work for you? And what do you mean can't get it to open, do you not see a chat icon?

Hi Logan,
Thanks for replying.
When I click on the copilot extension or the copilot chat extension it just
takes me to the details window.
I made sure I have copilot activated and I just got email saying copilot
chat is available to all users.
VS studio code is running latest version.
Julian
On Wed, Sep 20, 2023 at 2:34 PM Logan Ramos @.***>
wrote:

Does regular copilot work for you? And what do you mean can't get it to
open, do you not see a chat icon?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/407#issuecomment-1728388391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7WLYDJEWJBQHCJ7EBJ7DCTX3NHMTANCNFSM6AAAAAA5APT3CY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Julian Hardaker CRS, ABR
Voted Post Independent Local's Choice Best Realtor 2014, 2015, 2016 and
2017!
The Best Way Home Real Estate
Office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
@.***
skype: julian.hardaker
http://www.TheBestWayHome.com http://www.thebestwayhome.com/
http://goog_508099208/
Watch a short video about The Best Way Home Team
https://www.youtube.com/watch?v=0wuPgohOVng
http://www.thebestwayhome.com/
IMPORTANT NOTICE: Never trust wiring instructions sent via email. Cyber
criminals are hacking email accounts and sending emails with fake wiring
instructions. These emails are convincing and sophisticated. Always
independently confirm wiring instructions in person or via a telephone call
to a trusted and verified phone number. Never wire money without
double-checking that the wiring instructions are correct.

Can you try signing out and back into vscode via the account menu in the bottom left?

I did that and it did get me to authorize visual studio code. That worked
but i still get taken to the details window when I click on the extension -
image attached.
On Wed, Sep 20, 2023 at 2:55 PM Logan Ramos @.***>
wrote:

Can you try signing out and back into vscode via the account menu in the
bottom left?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/407#issuecomment-1728412426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7WLYDO6H24GWT6AGPKML5LX3NJ3JANCNFSM6AAAAAA5APT3CY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Julian Hardaker CRS, ABR
Voted Post Independent Local's Choice Best Realtor 2014, 2015, 2016 and
2017!
The Best Way Home Real Estate
Office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
@.***
skype: julian.hardaker
http://www.TheBestWayHome.com http://www.thebestwayhome.com/
http://goog_508099208/
Watch a short video about The Best Way Home Team
https://www.youtube.com/watch?v=0wuPgohOVng
http://www.thebestwayhome.com/
IMPORTANT NOTICE: Never trust wiring instructions sent via email. Cyber
criminals are hacking email accounts and sending emails with fake wiring
instructions. These emails are convincing and sophisticated. Always
independently confirm wiring instructions in person or via a telephone call
to a trusted and verified phone number. Never wire money without
double-checking that the wiring instructions are correct.
www.TheBestWayHome.com

You need to attach the image via github.com as email strips it

Hi Logan,
I am embarrassed to say I did find the chat button on the left window!!
Julian
On Wed, Sep 20, 2023 at 3:38 PM Logan Ramos @.***>
wrote:

You need to attach the image via github.com as email strips it
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/407#issuecomment-1728461665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7WLYDOML3S56RRZ5H2QPNDX3NO53ANCNFSM6AAAAAA5APT3CY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Julian Hardaker CRS, ABR
Voted Post Independent Local's Choice Best Realtor 2014, 2015, 2016 and
2017!
The Best Way Home Real Estate
Office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
@.***
skype: julian.hardaker
http://www.TheBestWayHome.com http://www.thebestwayhome.com/
http://goog_508099208/
Watch a short video about The Best Way Home Team
https://www.youtube.com/watch?v=0wuPgohOVng
http://www.thebestwayhome.com/
IMPORTANT NOTICE: Never trust wiring instructions sent via email. Cyber
criminals are hacking email accounts and sending emails with fake wiring
instructions. These emails are convincing and sophisticated. Always
independently confirm wiring instructions in person or via a telephone call
to a trusted and verified phone number. Never wire money without
double-checking that the wiring instructions are correct.

🎉


is therepossible to place circle around marker in osm like google map Circle class and how to place number counter inside marker dynamically example needed

Please note: issues related to or caused by osmbonuspack will be closed with a
suggestion to open an issue at https://github.com/MKergall/osmbonuspack
(feel free to delete whatever doesn't apply)
Issue Type
[ ] Question
[ ] Bug
[ ] Improvement
[ ] Build system related
[ ] Performance
[ ] Documentation
Description and/or steps/code to reproduce the problem
Environment
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
5.6.5

there's no built in way at the moment. It's possible to create a polygon around a marker by calculating the points of a circle. This has been asked recently

Just added some code to make plotting circle's simple. feel free to use in your own code. it will be added in v6

You already had Polyline.pointsAsCircle(center, radius) utility method.

@MKergall doh! fixed

Hi i found one bug that , In marker class, I wanna to rotate marker .when i try  to rotate marker like this sample code
public void tmpMarker() {
    markerTmp = new Marker(mMapView);
    markerTmp.setTitle("test");
    markerTmp.setPosition(geoPointTmp);
    markerTmp.setAnchor(Marker.ANCHOR_BOTTOM, Marker.ANCHOR_CENTER);

    markerTmp.setIcon(ContextCompat.getDrawable(getActivity(), R.drawable.direction_arrow));
    mMapView.getOverlays().add(markerTmp);

    mMapView.invalidate();
    iMapController.animateTo(mgeoPoint);
    final Handler handler = new Handler();
    final Runnable runnable = new Runnable() {
        @Override
        public void run() {

            markerTmp.setRotation(rotateMarker);
            markerTmp.

//                mMapView.invalidate();
rotateMarker = rotateMarker + 30.0f;
if (rotateMarker >= 360f) {
rotateMarker = 0.0f;
}
handler.postDelayed(this, 2000);
        }
    };
    handler.postDelayed(runnable, 2000);

}

i got the marker is moving little bit and rotate its like , rounding with small circle radius.. I wanna to rotate marker not moving even single bit just rotation..


Update deploy-master.md

Added missing line continuation backslash in /etc/systemd/system/kubelet.service file listing.

Thanks for catching this!


Refactored metadata linting

The existing user-facign API did not change, we still have the CLI commands we're used to, tmt [test|plan|story] lint and tmt lint. internally, things are not different, with linters split into categories, automagical discovery, formalized internal APIs, and new CLI look (--list-checks, --failed-only, --enable-checks & brand new output).
$ tmt story lint /spec/plans/import
/spec/plans/import
pass C000 fmf node passes schema validation
pass C001 summary key is set
pass C002 summary is reasonably long
pass S001 correct key are used
fail S002 story is required

@happz while running it manually, this looks weird:
warn C001 summary key is missing, it is useful for quick inspection
skip C002 summary key is missing, it is useful for quick inspection

@happz also we will need to add some documentation about this, describing all the available checks?

@happz also we will need to add some documentation about this, describing all the available checks?

@thrix --list-checks should help:
$ tmt lint --list-checks
Linters available for tests
C000: fmf node should pass the schema validation
C001: summary key should be set, it is very useful for quick inspection
C002: summary should not exceed 50 characters
T001: all keys are known
T002: test script must be defined
T003: test directory path must be absolute
T004: test directory path must exist
T005: relevancy has been obsoleted by adjust
T006: coverage has been obsoleted by link
T007: manual test path is not an actual path
T008: manual test should be valid markdown

Linters available for plans
C000: fmf node should pass the schema validation
C001: summary key should be set, it is very useful for quick inspection
C002: summary should not exceed 50 characters
P001: all keys are known
P002: execute step must be defined with "how"
P003: execute step methods must be known
P004: discover step methods must be known
P005: remote fmf ids must be valid

Linters available for stories
C000: fmf node should pass the schema validation
C001: summary key should be set, it is very useful for quick inspection
C002: summary should not exceed 50 characters
S001: all keys are known
S002: story key must be defined
$

How about --disable-checks CHECK-ID ?
These changes look great overall. Very nice usability improvement.

Could '--outcome-only' be improved? I did tmt lint --outcome-only warn and
/stories/features/prepare/all

/stories/features/prepare/minimal
warn C002 summary should not exceed 50 characters

/stories/features/prepare/repository

Technically it works as expected - Shows only specified outcome. However each object name is printed as well so it is hard to read. Could this be changed into 'print object name only it there is some check output as well? Similar to '--failed-only' does

Sure, makes sense. Done in ec6f8a8387fd31a41a5196f977c0989ab79ec6d2.

How about --disable-checks CHECK-ID ?
These changes look great overall. Very nice usability improvement.

Done in eb6ea5621fb454b870eea17748c76cbe76689ffe

A note from #2008, should add a test condition for when the fmf root is in a different path. Maybe that should be made a test fixture/conftest.py so that it can be added to all test.

A test for #2006 added in fa2520b0

What about using a different color for the skip label? Currently it can be easily confused with pass.

Green/Gray/Red/Yellow for pass/skip/fail/warning? These should be standard

When an invalid key is present, the problem is reported twice:
warn C000 key "requires" not recognized by schema, and does not match "^extra-" pattern
fail T001 unknown key "requires" is used

Is this intentional? When two unknown keys are provided, duplication is gone:
warn C000 fmf node failed schema validation
warn C001 summary key is missing
fail T001 unknown key "hm" is used
fail T001 unknown key "requires" is used

Not sure why.

Green/Gray/Red/Yellow for pass/skip/fail/warning? These should be standard

Sounds good to me. Maybe blue could be also suitable for skip as an "info" color.

Green/Gray/Red/Yellow for pass/skip/fail/warning? These should be standard

Sounds good to me. Maybe blue could be also suitable for skip as an "info" color.

Changed to white in 5b356c5a as Click does not offer gray. Let me know whether it should be blue instead...

When an invalid key is present, the problem is reported twice:
warn C000 key "requires" not recognized by schema, and does not match "^extra-" pattern
fail T001 unknown key "requires" is used

Is this intentional? When two unknown keys are provided, duplication is gone:
warn C000 fmf node failed schema validation
warn C001 summary key is missing
fail T001 unknown key "hm" is used
fail T001 unknown key "requires" is used

Not sure why.

I do know why, fixed in e2169ff1.

Changed to white in 5b356c5 as Click does not offer gray. Let me know whether it should be blue instead...

Thanks. Just tried with my light terminal and the skip tags are hardly readable :( Let's go with blue instead.

Changed to white in 5b356c5 as Click does not offer gray. Let me know whether it should be blue instead...

Thanks. Just tried with my light terminal and the skip tags are hardly readable :( Let's go with blue instead.

Changed to blue in 8fc1e89c

/packit test

The only failure is /tests/finish/ansible error on fetching the centos:7 image. Happened multiple time recently. Filed as #2063.


More info for missing pair

@GregoryFaust ,
I'm trying to track down a problem with grouping reads.  The error I get:
samblaster: Can't find first and/or second of pair in sam block of length 1 for id: xxxxx
It would be exceptionally helpful if the error reported the chromosome and position of the read so they could be quickly pulled for examination.
Thank you.

Closed in release 0.1.23

@GregoryFaust
Dear author,
I have the same issue here. But what do you mean by "Closed in release 0.1.23"?
I am using 0.1.23 right now. And I am using "samtools sort -n" command to sort bwa aligned reads in a SAM file.
What do you think of it? Thanks in advance!

By "Closed in 0.1.23" I mean that I included the requested additional information into the error message for missing mate, namely the chromosome and position of the read that produced the error.
Yes, "samtools sort -n -O SAM" or "sambamba sort -n -u -o /dev/stdout | sambamba view -f sam -h" will work to sort output from bwa or any other paired-end read aligner in a way that samblaster can use.

@vyx-greg-faust Thanks for your explanation and instructions.


Legal BMP file cannot be opened

ImageInput::open fails for this fil
gracehopper.zip
e

Aha!  Solution here: https://github.com/OpenImageIO/oiio/pull/3375
It was an "undocumented" header configuration that PhotoShop is happy to write despite not being described in MSDN.

@DougRogers Is that bmp file ok to check into our testsuite? It doesn't seem to have any attached license or ownership information, though I assume that the likeness of her is from a government photo and thus probably in the public domain.

Should be fine.  The image is posted here, albeit in a different format.
https://commons.wikimedia.org/wiki/File:Grace_Hopper.jpg

I think that bmp is poster here:
https://chromium.googlesource.com/chromium/src/+/HEAD/third_party/blink/renderer/platform/testing/data/


chore: merge main back to alpha

:tada: This PR is included in version 1.15.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


After downloading a repository with LFS files as an archive, unnecessary git-hooks created and as a result trying to create a PR results in an error 500

Description
After downloading a repository with LFS files as an archive, unnecessary git-hooks created and as a result trying to create a PR results in an error 500
Steps for representation:

Import or clone this repo
Download the repository as an archive via the Web UI
Create a PR with any modified lfs file
Close the PR and then create a new one just like it
Get error 500 and broken PR

git lfs env
git-lfs/3.0.2 (GitHub; windows amd64; go 1.17.2)
git version 2.35.1.windows.2

Logs
2022/09/12 14:35:04 ...ervices/pull/pull.go:477:pushToBaseRepoHelper() [E] [631f1967-13] Unable to push PR head for sergey-cs/representation-for-bug-with-download-archive#4 (89:sergey-cs/representation-for-bug-with-download-archive:refs/pull/4/head) due to Error: exit status 1 - batch request: missing protocol: ""
        error: failed to push some refs to 'HIDDEN\\sergey-cs\representation-for-bug-with-download-archive.git'

        /source/services/pull/pull.go:477 (0x1c099ee)
        /source/services/pull/pull.go:426 (0x1c050f2)
        /source/services/pull/pull.go:71 (0x1c050f3)
        /source/routers/web/repo/pull.go:1220 (0x21a05d2)
        /source/modules/web/wrap_convert.go:47 (0x1fae9f6)
        /source/modules/web/wrap.go:41 (0x1facc29)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:442 (0x1792bf5)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/middleware/get_head.go:37 (0x2014ea4)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/context/context.go:802 (0x1b7989a)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:71 (0x1790a2c)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:314 (0x17923db)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:442 (0x1792bf5)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/routers/web/base.go:174 (0x21e3151)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/gitea.com/go-chi/session@v0.0.0-20211218221615-e3605d8b28b8/session.go:257 (0x1584ddd)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:110 (0x1fadb48)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:71 (0x1790a2c)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:314 (0x17923db)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:442 (0x1792bf5)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/routers/common/middleware.go:79 (0x201a4a2)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/routing/logger_manager.go:123 (0x1fa894f)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/middleware/strip.go:30 (0x2017c18)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/chi-middleware/proxy@v1.1.1/middleware.go:37 (0x201451e)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/routers/common/middleware.go:32 (0x201a2f2)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:88 (0x17909e1)
        /source/modules/web/route.go:200 (0x1fac06d)
        /usr/local/go/src/net/http/server.go:2916 (0x8914da)
        /usr/local/go/src/net/http/server.go:1966 (0x88c996)
        /usr/local/go/src/runtime/asm_amd64.s:1571 (0x3adf80)

2022/09/12 14:35:04 ...ers/web/repo/pull.go:1244:CompareAndPullRequestPost() [E] NewPullRequest: Push: sergey-cs/representation-for-bug-with-download-archive:branch-with-changed-lfs-file sergey-cs/representation-for-bug-with-download-archive:refs/pull/4/head exit status 1 - batch request: missing protocol: ""
        error: failed to push some refs to 'HIDDEN\sergey-cs\representation-for-bug-with-download-archive.git'

        /source/routers/web/repo/pull.go:1244 (0x21a09b3)
        /source/modules/web/wrap_convert.go:47 (0x1fae9f6)
        /source/modules/web/wrap.go:41 (0x1facc29)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:442 (0x1792bf5)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/middleware/get_head.go:37 (0x2014ea4)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:63 (0x1fad0af)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/context/context.go:802 (0x1b7989a)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:71 (0x1790a2c)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:314 (0x17923db)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:442 (0x1792bf5)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/routers/web/base.go:174 (0x21e3151)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/gitea.com/go-chi/session@v0.0.0-20211218221615-e3605d8b28b8/session.go:257 (0x1584ddd)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/wrap.go:110 (0x1fadb48)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:71 (0x1790a2c)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:314 (0x17923db)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:442 (0x1792bf5)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/routers/common/middleware.go:79 (0x201a4a2)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/modules/web/routing/logger_manager.go:123 (0x1fa894f)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/middleware/strip.go:30 (0x2017c18)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/chi-middleware/proxy@v1.1.1/middleware.go:37 (0x201451e)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /source/routers/common/middleware.go:32 (0x201a2f2)
        /usr/local/go/src/net/http/server.go:2084 (0x88deee)
        /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.7/mux.go:88 (0x17909e1)
        /source/modules/web/route.go:200 (0x1fac06d)
        /usr/local/go/src/net/http/server.go:2916 (0x8914da)
        /usr/local/go/src/net/http/server.go:1966 (0x88c996)
        /usr/local/go/src/runtime/asm_amd64.s:1571 (0x3adf80)

Workaround
At the moment of downloading the archive via Web UI, unnecessary hooks created in the git folder. If you remove them, the problem will be fixed

Also, to prevent the same problem, repeatedly disable the ability to download the repository as an archive from the web ui using config the repository.DISABLE_DOWNLOAD_SOURCE_ARCHIVES=true
The problem also persists in 1.17.2
Gitea Version
1.17.1
Can you reproduce the bug on the Gitea demo site?
No

I believe this is the same as https://github.com/go-gitea/gitea/issues/19810 . I have a plan to fix, posted as a comment on that issue today.

I actually can't reproduce it on vanilla Gitea, but here's a branch, maybe someone can see if it fixes it for them. https://github.com/go-gitea/gitea/compare/main...philip-peterson:gitea:peterson/poc-archive-patch?expand=1

@philip-peterson, Are you moving LFS files to the gitea repository?

After downloading the archive do you get unnecessary git-hooks?

This issue can be closed along with #19810.


Replace react-icons with lucide-react icons (not from single library but from unocsss preset icons, lucide section)

Se crea PR: https://github.com/Afordin/hackafor-2/pull/28


A question about the code

Hi Wei, can you tell me why the ce losses are multiplied by  0.00001?
cls_loss_0 = 0.00001 * cross_entropy_losses(im_cls_prob_0, labels.type(im_cls_prob_0.dtype))

For different MIL Learners, their localizers output instance localization scores respectively.  For Localization Completeness, we add discrepancy loss among these instance localization scores.  But this may hurt the instance localization ability of some localizers, meanwhile  deteriorate the detection performance and image classification ability of MIL Learners.  So we add cls_loss_0 = cross_entropy_losses(im_cls_prob_0, labels.type(im_cls_prob_0.dtype)).  However, we find that cls_loss_0and cls_loss_1 could enforce the similiraty of localization scores of different MIL Learners which decreases the Localization Completeness and increases the Localization Redundancy. So the ce losses are multiplied by 0.00001 (0.0001 is also valided in our experiment which show comparable performance with 0.00001). Then cls_loss_0 = 0.00001 * cross_entropy_losses(im_cls_prob_0, labels.type(im_cls_prob_0.dtype)).

Thanks very much！
One more question. Is the implementation of PCL same as the official one?
How could we test the standard PCL via this repo?

Our implementation of PCL is almost the same as the official one PCL,  expect that our codes are more concise and  less complicated. Besides, the method of getting the cluster centers is a little different.  There are two ways of getting the cluster centers as  mentioned in paper PCL:

Finding proposal cluster centers using the highest scoring proposal
Finding proposal cluster centers using graphs of top ranking proposals

And in our implementation of PCL, we just choose the first method that selecting the highest scoring proposals as the proposal cluster centers.
And for testing the standard PCL via this repo, we may modify codes for getting proposal cluster centers. For example, now in D-MIL, the code is instane_selector or get_highest_score_proposals. We can replace it with code borrowed from this link PCL.

If needed, I can upload another branch for testing the standard PCL method.

Sorry to bother you, why did you report the wrong model? Is this reference a folder built by yourself? After using many methods, I still can't solve the error. If possible, can you reply?


Added new classes to return the edges of polyhedra

Description
The new class polyhedron.edges returns the a list of the edges of a polyhedron as vertex-index pairs, similar to the current polyhedron.faces class. The class polyhedron.get_edge_vectors returns a list of edges as vectors in 3D space.
Types of Changes

[ ] Documentation update
[ ] Bug fix
[x] New feature

Checklist:

[x] I am familiar with the Contributing Guidelines.
[x] I agree with the terms of the Contributor Agreement.
[x] The changes introduced by this pull request are covered by existing or newly introduced tests.
[x] I have updated the changelog and the credits.

Looks like CircleCI is failing because of inaccuracies in the dodecahedron's stored data. I feel that we should wait for #177 to be merged before merging this branch (#171), so that the small fixes currently in test_edges can be reverted before this branch goes live. This will result in a cleaner merge.

@vyasr @bdice
From reading the discussions it seems that the final solution would be to return a numpy array of sorted pairs? After that docs should be updated accordingly? Is this correct?

@DomFijan Yeah, I think there were a few proposals above. One was to return a set of tuple of int. Another was to return a NumPy array of the same data. I think I lean towards a NumPy array, and just document that the data is in sorted order with i < j.
What I mean concretely is this:
from functools import cached_property  # Requires Python 3.8 or newer, which is fine

class Polyhedron:
    @cached_property
    def edges(self):
        """:class:`numpy.ndarray`: Get the polyhedron's edges.

        Results returned as vertex index pairs,  with each edge of the polyhedron
        included exactly once.  Edge (i,j) pairs are ordered by vertex index with i<j.
        """
        all_edge_pairs = np.array([[i, j] for face in self.faces for i, j in zip(face, np.roll(face, -1))])
        all_edge_pairs.sort(axis=1)
        ij_pairs = np.unique(all_edge_pairs, axis=0)
        # Make the edge data read-only so that the cached property of this instance cannot be edited
        ij_pairs.flags.writeable = False
        return ij_pairs

I haven't tested this explicitly but I suspect you can tidy that up with pre-commit and it should be good to go. The best part about using a NumPy array as the return value is that you can use it to index the vertices. (You can't do this as easily if you return a set of tuples of integers.) Then you can compute the edge vectors with one line:
edge_vectors = vertices[edges[:, 1]] - vertices[edges[:, 0]]

I advocated against a set because we want to preserve order, so the options would be a numpy array or a list. Between those two I would lean towards a numpy array, yeah.

@DomFijan Yeah, I think there were a few proposals above. One was to return a set of tuple of int. Another was to return a NumPy array of the same data. I think I lean towards a NumPy array, and just document that the data is in sorted order with i < j.
What I mean concretely is this:
from functools import cached_property  # Requires Python 3.8 or newer, which is fine

class Polyhedron:
    @cached_property
    def edges(self):
        """:class:`numpy.ndarray`: Get the polyhedron's edges.

        Results returned as vertex index pairs,  with each edge of the polyhedron
        included exactly once.  Edge (i,j) pairs are ordered by vertex index with i<j.
        """
        all_edge_pairs = np.array([[i, j] for face in self.faces for i, j in zip(face, np.roll(face, -1))])
        all_edge_pairs.sort(axis=1)
        ij_pairs = np.unique(all_edge_pairs, axis=0)
        # Make the edge data read-only so that the cached property of this instance cannot be edited
        ij_pairs.flags.writeable = False
        return ij_pairs

I haven't tested this explicitly but I suspect you can tidy that up with pre-commit and it should be good to go. The best part about using a NumPy array as the return value is that you can use it to index the vertices. (You can't do this as easily if you return a set of tuples of integers.) Then you can compute the edge vectors with one line:
edge_vectors = vertices[edges[:, 1]] - vertices[edges[:, 0]]

This is clean code, but is ~50% slower than the current method. However, the sort that np.unique performs does give you the right answer. The following code adapts this to improve the method's performance while also cleaning up my previous code:
``python

from functools import cached_property  # Requires Python 3.8 or newer, which is fine
class Polyhedron:
@cached_property
def edges(self):
""":class:numpy.ndarray: Get the polyhedron's edges.
    Results returned as vertex index pairs,  with each edge of the polyhedron
    included exactly once.  Edge (i,j) pairs are ordered by vertex index with i<j.
    """
    ij_pairs = np.array([[i, j] for face in self.faces for i, j in zip(face, np.roll(face, -1)) if i < j])
    sorted_indices = np.lexsort(ij_pairs.T[::-1])
    sorted_ij_pairs = ij_pairs[sorted_indices]
    sorted_ij_pairs.flags.writeable = False
    return sorted_ij_pairs

And, as you mentioned - the edge_vectors code has been

@bdice, any chance you can take a look at this before the end of Wednesday next week if you have the time? If not, I can jump in so we can finish this PR. There are a couple more tasks that depend on this that we want to start moving along. :)

@bdice thanks for the prompt reaction :)


Add Morse (Telegram API) sample project

Hi @macielti
Thank you for your PR.
Can you please update the README.md to specify which library are you testing?


feat(tangle-dapp): Add liquid staking unstaking card

Summary of changes
Provide a detailed description of proposed changes.

➕ Ability to redeem (schedule unstake) minted tokens on the Parachain.
➕ Added hook to calculate exchange rate for minting and redeeming for any given token/LST.
➕ Created hook for fetching ongoing time units, as well as basic logic for estimating unlock times. This will be completed in my future PRs.
➕ Mint/redeem fees are also calculated and displayed.
➕ Wallet balance for the selected token is now fetched from the parachain and shown. It also shows a skeleton loader when loading.
🖌 Implemented designs of unstake card, unstaking requests table, unstake confirmation modal, cancel unstake modal, and more.
🔧 Added missing check required when redeeming: Check that all delegations aren't occupied, otherwise the redeem extrinsic will fail.
🔧 Refactored some utility functions, fixing bugs and making the code clearer (ex. formatBn no longer uses confusing regex).
🐛 Fixed bug with the formatBn function which was causing certain values like 0.0001 to be incorrectly formatted as 0.1.

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] apps/zk-explorer
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes #2336

Screen Recording
If possible provide a screen recording of proposed change.
liquid-staking-unstake.webm

Code Checklist
Please be sure to add .stories documentation if any additions are made to libs/webb-ui-components.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

@yurixander I got overflow UI when using Arc browser on Liquid Staking page

@vutuanlinh2k2 I'm not seeing the UI overflow problem on my end, also the UI on your screenshot seems to be from previous code... do you happen to be on the most up to date version of this branch?

@AtelyPham Seems like a tiny bug regarding table headers was introduced in #2436:

@AtelyPham Seems like a tiny bug regarding table headers was introduced in #2436:

Can you check again on the develop branch? I don't see the error on the live development site.

@AtelyPham Check here: https://deploy-preview-2436--tangle-dapp.netlify.app/nomination, for some reason the dev branch preview doesn't show the problem, but that link (the deploy preview of #2436) does. The problem also occurred after merging those changes for me.


virtical divider for navs?

any chance we can get a vertical dividers for navbars?

@nh916 this is a feature request, not an issue.


inner query filters applied to the outer query

when using a filter (using jinja template) inside an inner query and use add ( remove_filter=True ) as a second parameter to the filter_values the filter still applied to the outer query not to the right place
the Shipment__Type filter should be applied under the created at filter
but after viewing the query it's applied to the outer query . . . .

superset version 1.4.1
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

I have the same problem, I applied remove_filter=True constraint, and the filters still remain in the outside query.

This problem still exists in 2.0 :/ — I'll try to dig into this issue this week to find the root cause.
As a workaround I create a column that contains the filter value on the fly and move the original column to another one.

I believe there's a feature flag that needs to be enabled for the remove_filter=True constraint to work. Unfortunately, I'm not able to modify feature flags since I'm using Preset, but the workaround seems to be to rename the column in a subquery.

Is this true?  Which feature flag?  I'm having the same issue with both Filter Charts and Native Dashboard Filters.

Perfect!  It works now both with filter charts and the new native filters.  Thanks.
I was poking around after I posted this.  If anyone is interested, the line is here:
https://github.com/apache/superset/blob/master/superset/connectors/sqla/models.py#L1505

If the filter contains a single value, such as upc = 'xxx', does it support it? Or is it necessary to use 'in'? upc in ({{ "'" + "','".join(filter_values('upc')) + "'" }})

It sounds like this is likely fixed by now, and is pretty out of date if not. If people are still encountering this in current versions (3.x) please open a new Issue with updated context or a PR to address the problem. Thanks!


Switch to k3s or k3d

Kind does not run without privileged docker, which means that internal GR folks cannot use localdev.
I instead suggest switching to either https://k3d.io/v5.6.0/ or https://github.com/k3s-io/k3s.
Using k3s to force us to generate the manifests / helm charts to setup anyways. I think that's an acceptable barrier to put up, and brings the dev setup closer to the prod setup, but we would have to figure out how to get the binaries into a k3s pod as you build them (a single pod with a volume?)
k3d would be easier to setup but wouldn't move us closer to a prod dev setup.

Let me know your preference for k3d or k3s. I would vote for k3s

Kind has a rootless option: https://kind.sigs.k8s.io/docs/user/rootless/

Let's not do this work without putting some thought into it first. Kind does have a rootless mode but there were issues getting it to work at GR, tho this may not be an insurmountable problem.

@severinson - can you share details on the GR issues, and is it something we can help with? The diverse ways in which people set up development environments is a constant source of friction.


[FLINK-12910][python][tests] Fix the Python catalog test failure

What is the purpose of the change
This pull request fixes the python catalog test failure.
Brief change log

fixes the python catalog test failure

Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@flinkbot approve all
Merging


Mismatch SignatureValue compared to samltool

xml-crypto is not generating the correct value for the SignatureValue for the signature. I've verified by feeding the same XML repeatedly to xml-crypto and there's a mismatch in the output.
The SignatureValue value from samltool works with SP but the one xml-crypto generates doesn't.
DigestValue is correct and matches the one from samltool though.
return new Promise(function (resolve, reject) {

    let SignedXml = require('xml-crypto').SignedXml;
    var sig = new SignedXml();

    sig.signingKey = require('fs').readFileSync(self.privateKey);
    sig.canonicalizationAlgorithm = "http://www.w3.org/2001/10/xml-exc-c14n#";
    sig.signatureAlgorithm = "http://www.w3.org/2000/09/xmldsig#rsa-sha1";

    sig.addReference("//*[local-name()='Response']",
        [
            "http://www.w3.org/2000/09/xmldsig#enveloped-signature",
            "http://www.w3.org/2001/10/xml-exc-c14n#"
        ],
        "http://www.w3.org/2000/09/xmldsig#sha1"
    );

    sig.keyInfoProvider = new (function () {
        this.getKeyInfo = function () {
            return `<ds:X509Data><ds:X509Certificate>${self.getPublicKey()}</ds:X509Certificate></ds:X509Data>`
        }
    });

    sig.computeSignature(xml, {
        prefix: 'ds',
        location: { reference: "//*[local-name(.)='Issuer']", action: "after" } // place the signature after the Issuer
    });

    resolve(sig.getSignedXml());
});

xml signature
<ds:Signature xmlns:ds="http://www.w3.org/2000/09/xmldsig#">
    <ds:SignedInfo>
        <ds:CanonicalizationMethod Algorithm="http://www.w3.org/2001/10/xml-exc-c14n#"/>
        <ds:SignatureMethod Algorithm="http://www.w3.org/2000/09/xmldsig#rsa-sha1"/>
        <ds:Reference URI="#pfx238f2615-2267-6ee6-efa9-24dd82d85764">
            <ds:Transforms>
                <ds:Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature"/>
                <ds:Transform Algorithm="http://www.w3.org/2001/10/xml-exc-c14n#"/>
            </ds:Transforms>
            <ds:DigestMethod Algorithm="http://www.w3.org/2000/09/xmldsig#sha1"/>
            <ds:DigestValue>6okYcbSa3y18LSbOBJTMMWN7/Rs=</ds:DigestValue>
        </ds:Reference>
    </ds:SignedInfo>
    <ds:SignatureValue>
        ULNmpuKwXwHwqUMybbwnupC/y9WZohGBx9Y.....NQVwpNC+sBc6etPhSq5BVw==
    </ds:SignatureValue>
    <ds:KeyInfo>
        <ds:X509Data>
            <ds:X509Certificate>
                MIIDuDCCAqACCQD.......Zsb3JpZGExDuJ/M7+/P+enkq9Vx0thQHUaI+ndr1gpZE=
            </ds:X509Certificate>
        </ds:X509Data>
    </ds:KeyInfo>
</ds:Signature>

@sweetodev , this certainly sounds like something we'd like to address. Could you please create a PR with a test suite that replicates the problem, including notes about how you signed things over at SAML Tool? That would really help everyone in getting to the bottom of this as you aren't the first person to report having difficulty getting signatures to match.


Migrate codebase to pmd 7

Summary
With version 7, pmd will introduce a lot of changes in its apis. So it could be a good idea to work on migrating arch4u rules to these new apis.
The migration guide can be found here : https://pmd.github.io/pmd/pmd_userdocs_migrating_to_pmd7.html

Work started in https://github.com/dgroup/arch4u-pmd/tree/168-wip-pmd-7.x


Evaluation of random agent dies

Run evaluation on random agent
./scripts/evaluate_train.sh

2020-02-16 01:06:18,857 [INFO] robothor_challenge - Agent action: MoveBack
2020-02-16 01:06:18,870 [INFO] robothor_challenge - Agent action: RotateRight
2020-02-16 01:06:18,883 [INFO] robothor_challenge - Agent action: Stop
2020-02-16 01:06:18,898 [INFO] robothor_challenge - Task Start id:420 scene:FloorPlan_Train1_2 target_object:Apple|+01.98|+00.77|-01.75 initial_position:{'x': 3.0, 'y': 0.910344243, 'z': -1.75} rotation:270
2020-02-16 01:06:19,135 [INFO] robothor_challenge - Agent action: Stop
Traceback (most recent call last):
File "example_agent.py", line 17, in
r.inference()
File "/app/robothor_challenge/init.py", line 86, in inference
episode_result['success'] = simobj['visible']
TypeError: 'NoneType' object is not subscriptable

System info: Fresh GCP instance n1-standard-4 (4 vCPUs, 15 GB memory) 1 x NVIDIA Tesla P4, Ubuntu 18; nvidia drivers and nvidia-docker installed.

Updated train/val.json files have been pushed to the repo which don't cause this error.


Updated README

changelog:

updated README.md with new arguments -v, -st, -nf and -nl
translation of the help text in docs/README_RU.md

I do not know why so, but it turns out in one line.
countfiles -v countfiles --version


extra self.subscription in tf2 tutorial

Perhaps I'm missing something but I don't see the reason for the extra "self.subscription" in the tutorial
for a python tf2 broadcaster, see below:
    self.subscription = self.create_subscription(
        Pose,
        f'/{self.turtlename}/pose',
        self.handle_turtle_pose,
        1)
    self.subscription     <<<<<<<<<< Why is this here?

def handle_turtle_pose(self, msg):

The code is copied from here: https://github.com/ros/geometry_tutorials/blob/37420516b9aa4feb3d6d2f7566643dea2fd9fa7a/turtle_tf2_py/turtle_tf2_py/turtle_tf2_broadcaster.py
Looks like it's a simple mistake from when the tutorial was created: https://github.com/ros/geometry_tutorials/pull/34/files#diff-1cdaa036d32a0780aa1f61086f52ea49448669e9efabd29d66e6c1ab78a09a25
We could remove that line in the tutorial and in the original code.

It still runs fine.  I was just wondering if there’s more to it. Cheers.

It still runs fine. I was just wondering if there’s more to it. Cheers.

Yeah, there is a simple reason behind this line actually. It is used to suppress the "unused variable" warning from displaying.
You can find the same line on Writing a simple publisher and subscriber (Python) tutorial.

I only edited that part in the documentation in this PR as updating the geometry_tutorials package will require additional unnecessary work


infer or extrac args and opts type from program

I like to move my action handlers into a separate file and it would be nice if I could somehow infer the arguments and options types that are generated by just passing in an exported instance of the new Command() object. I see that I can pass in my own generic to the constructor but that kind of defeats the purpose of this library. Is there currently a way to do this?

This is more a proof of concept than a specific answer, but hopefully answers your question.
// cli.ts
import { Command } from '@commander-js/extra-typings';

export const example = new Command()
    .argument('<filename>')
    .option('-p, --port');

import { example } from './cli';
import { Command } from '@commander-js/extra-typings'

type ActionArgs<T> = T extends Command<infer Args, infer Opts>
    ? [...Args, Opts, T]
    : never;

function myAction(...args : ActionArgs<typeof example >) {
    const [ filename, options, cmd ] = args;
}

type MyActionArgType = ActionArgs<typeof example >;
function myAction2(filename: MyActionArgType[0], options: MyActionArgType[1], cmd: MyActionArgType[2]) {
}

I suspect the type inference could also be done with something like Parameters<typeof example['action']> but I didn't work out how to use the resulting type.

Ah ha! Got there with the utility types too:
import { example } from './cli';

type ActionMethod = typeof example['action'];
type ActionCallback = Parameters<ActionMethod>[0];
type ActionArgs3 = Parameters<ActionCallback>;

So I was able to get that to work as long as I exported the variable where I instantiated the Command constructor, however it seems to not work well when I want to do sub commands. Here is my setup
import { Command } from "@commander-js/extra-typings"

export const genCommand = new Command("gen")

genCommand
  .description("Code generation tools")
  .command("component")
  .description("Generate a new component")
  .argument("[name]", "Name of the new component")
  .option("-d, --dir <dir>", "Directory to create the component in")
  .action((name, options) => {
    console.log("component gen")
    console.log("name", name)
    console.log(options)
  })

genCommand.command("dialog").action(() => {
  console.log("dialog gen")
})

In this case the dialog will probably end up having the same arguments and options but there will definitely be other sub commands I'll want to add that will have different arguments or options.
I'm not fully understanding how I would properly get a separate type for each sub command given the setup you showed, I'm sure there is something there but I'm not well versed in typescript enough to know the right direction.

Subcommands are just commands too. So you can export them to get at their types, just like with the program.
import { Command } from "@commander-js/extra-typings"

export const genCommand = new Command("gen")
  .description("Code generation tools");

export const componentCommand = genCommand.command("component")
  .description("Generate a new component")
  .argument("[name]", "Name of the new component")
  .option("-d, --dir <dir>", "Directory to create the component in")
  .action((name, options) => {
    // ...
  })

export const dialogCommand = genCommand.command("dialog")
  .action(() => {
    console.log("dialog gen")
  })

Ah that actually makes sense, I will attempt this setup, once I figure out my build issues with another package 🙄

An answer was provided, and no further activity in a month. Closing this as resolved.
Feel free to open a new issue if it comes up again, with new information and renewed interest.


Histogram-based MI estimation

This PR adds the simplest approach to the MI estimation. We estimate discrete $p(x, y)$ by binning and use the formula
$$I(X; Y) \approx \sum_{(x, y) \in \mathcal X \times \mathcal Y } p(x, y) \log \frac{p(x, y)}{p(x)p(y)}.$$
As we bin each dimension of $\mathcal X$ into $n_X$ bins and each dimension of $\mathcal Y$ into $n_Y$ bins, we need to have
$$n_X^{\dim \mathcal X} \cdot n_Y^{\dim \mathcal Y} \ge 2^{\dim \mathcal X + \dim \mathcal Y}$$
bins – this quickly becomes too large both in terms of time and memory requirements.
Note that the current implementation uses np.histogramdd and NumPy arrays won't be able to handle the case $\dim \mathcal X + \dim \mathcal Y > 32$. (Anyway, even for 2 bins per dimension this would require dozens of gigabytes of RAM).

How do you handle the number of bins? There also exist some heuristics like computing a penalized MI score, where add a penalty term, which is dependent on the number of bins. After fixing the bins in such a way, the penalty can be removed again to estimate MI.

How do you handle the number of bins? There also exist some heuristics like computing a penalized MI score, where add a penalty term, which is dependent on the number of bins. After fixing the bins in such a way, the penalty can be removed again to estimate MI.

Currently it's a hyperparameter chosen by the user... :smiling_face_with_tear:
Could you recommend some papers/blogposts describing this approach?

For example, here in Sec. 4.1 we describe a regularization based on the Minimum Description Length principle. There are two possible codings for this "Stochastic Complexity", which are described here. These scores have a very similar motivation to marginal likelihood in the Bayes literature. In the limit, they converge to a BIC score, where the number of parameters (in our case) relates to the number of bins. This paper uses this asymptotic variant.


Remove dligs from Old Hungarian font because make spelling problems p…

…repare ab letter to correct form
On branch remove-oldhun-dlig
Changes to be committed:
modified:   src/NotoSansOldHungarian/Noto Sans Old Hungarian GSUB.txt
modified:   src/NotoSansOldHungarian/NotoSansOldHungarian.glyphs

@googlebot I fixed it.

@marekjez86 @google-admin
See Issue #215


[BUG] Security Management: Cannot Add users to Field Security Profile

Issue
Cannot Add users to Field Security Profile
To Reproduce

When in Field Security Management
I select the user, then click the "Add to this field security profile" button,
The pane closes with no message and the user is not added.

Expected behavior
User should be added to profile
Desktop (please complete the following information):

OS: Windows 10
Edge (also tried with Chrome)
Version 110.0.1587.56 (Official build) (64-bit)

Additional context
Can remove users from the profile no problem, but unable to add them via the app.

Acknowledge. Let me check on this amd get back.

@rich835 - I am not able to replicate this, I would like to connect with you and see if we can work on this together. Can you reach out, post a message with a good time to connect on https://www.linkedin.com/in/ravi-chada/.

@rich835 - I have a new zip file uploaded and ready for you. Please check the latest download. This should fix the issue.


Update ChargePoint tests to use new constructor with dependency injection.

Once all old constructors no longer are used in other areas of the code, then they can be removed, ensuring the new constructor is used exclusively.
Also, the tests have been split up to use multiple test fixtures that cater to the kind of functionality being tested to make it easier to read and understand for future maintenance.
This work also had the nice side effect of eliminating some flakiness resulting from the fixture creating a TestChargePoint object and some tests creating a separate ChargePoint, as when both were doing cleanup, it sometimes resulted in segfaults related to database connections.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have made corresponding changes to the documentation
[x] If OCPP 2.0.1: I have updated the OCPP 2.0.1 status document
[x] I read the contribution documentation and made sure that my changes meet its requirements

No, this PR is not based off of the test fixture renames work. I broke up the test fixture into smaller, easier to understand fixtures, and I named those using the new naming convention. This entire PR is its own refactoring unrelated to the test rename refactoring PR.


Adding AdditionalLabels

What this PR does:
Adding the possibility to people to add Labels and  the all objects that will be created by the operator.
Which issue(s) this PR fixes:
Fixes #235
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

You're off to a good start. There are some other objects that the operator like the superuser secret and endpoints for the additional seeds service. There might be more, but those are the ones I know of immediately.
Are you going to add corresponding functionality for annotations?

I think for now I will only add the labels for the first step. I can do the annotations later if you want? If you want both of them now I can also work on getting both.

I will be adding the missing test tomorrow.

@jsanda @burmanm could you do another round of review?
For the annotation, do you mind I do that in another merge request?

@jsanda @burmanm do you still have sometime to review this one 🥺

Last week was a really short week for me due to the Thanksgiving holiday, and I just didn't have time to get back to it. I will do another review today or tomorrow.

My apologies for the delayed review. The changes look good. There is another place you need to update for the default super user secret. See here.

Does it make sense to use the AddOperatorLabels there also?

Does it make sense to use the AddOperatorLabels there also?

yeah, I think so

Does it make sense to use the AddOperatorLabels there also?

yeah, I think so

Done

Nice thank you ;)


New prod registry

Resolves # . (This is applicable only if this pull request relates to a GitHub issue, delete the line otherwise)
Notes:
*
*
*

Pipeline is green but another jenkins built it
https://sandbox-build.platform.hmcts.net/blue/organizations/jenkins/HMCTS_Sandbox_Pipeline_Test%2Fcnp-jenkins-library/detail/PR-501/17/pipeline


7/01 Publish for Anton and Allen

Docs Build status updates of commit c9aad04:
:warning: Validation status: warnings

File
Status
Preview URL
Details

rd/index.yml
:warning:Warning

Details

rd/map.yml
:warning:Warning

Details

rd/allen-oneill.yml
:white_check_mark:Succeeded

rd/anton-boyko.yml
:white_check_mark:Succeeded

rd/media/profiles/allen-oneill.jpg
:white_check_mark:Succeeded

rd/media/profiles/anton-boyko.jpg
:white_check_mark:Succeeded

rd/toc.yml
:white_check_mark:Succeeded

rd/index.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/about).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/map).

rd/map.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/about).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/map).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


WIP: try different approach to JSX

This is an attempt at a more modular approach to JSX.

I'm most curious for thoughts on desired output.  Since JSX can include embedded JS you get into this vicious cycle where now you really need to jump back into javascript sublanguage again... so you end up with code like:
<BrowserRouter>
        <div>
          {/* comment */}
          { test "}" }

Is it really critical/necessary to flag all of these as javascript, or should they really be something more like subst?
Then you also have comments inside tags (which seems to be valid and used):
box = (
)

Which results in code like:
box="yes" // comment
``

But technically the `//comment` is Javascript code, not HTML... so should it really also be in `javascript` or `subst` block?

I think the answers start to depend a lot on whether you think of this as a back and forth between JS and HTML or rather one larger meta-language such as "JSX".  In the latter case for example, I don't think the comment needs to be flagged as JS... though I do wonder in that case if `jsx` wouldn't be a better over-arching class than `html`.

@egor-rogov @allejo

But technically the //comment is Javascript code, not HTML... so should it really also be in javascript or subst block?
I think the answers start to depend a lot on whether you think of this as a back and forth between JS and HTML or rather one larger meta-language such as "JSX". In the latter case for example, I don't think the comment needs to be flagged as JS

I don't think it's our goal to mark different parts of code with appropriate languages. Our goal is to highlight comments as comments, no matter which language they belong to.
At the first glance the PR looks complicated. Registering of an internal language, a module in lib... I don't like the idea very much.
Btw, please remind me, what's wrong with the current implementation?

Btw, please remind me, what's wrong with the current implementation?

There is no good way to properly handle {{ javascript.code() }} or // comments inside HTML in the current implementation since neither of these things are valid HTML.
Hence needing an internal JSX variant of HTML.

I don't think it's our goal to mark different parts of code with appropriate languages.

subLanguage has always done that... are you suggesting that it should not?

At the first glance the PR looks complicated. Registering of an internal language, a module in lib... I don't like the idea very much.

Would you prefer a run-time require vs the module?  In this case there is no point at all of the "_jsx" language without Javascript, so registering and linking at run-time seems wrong. And splitting the sublanguage out into it's own module is just good organization - so I'm not completely sure what you're objecting to WRT that.

Registering of an internal language

Why? Does the idea of internal languages bother you in general?  Would it bother you less if they were hidden and not in the listLanguages namespace?

Also, the reason it's so dynamic is because eventually this can be used inside Javascript or Typescript, so you won't just have a single "_jsx" grammar. You'd have variants also: _jsx_javascript, _jsx_typescript, etc... since the JSX grammar needs a way to embed snippets of the parent grammar. So each variant would have a different internal sublanguage value with {{ }} blocks.

Although I just had another thought that I need to look into.

Picking out HTML tags singly (one by one) inside of JS is another [much simpler] idea... But that is also a much "dumber" solution, in that it can't identify blocks of HTML - and it's also broken in that it would incorrectly highlight JS code (or keywords) inside of HTML blocks. Ex:
var x = (
class is not a keyword here
);

I'm not sure how people are using HLJS to highlight JSX code but if it was me I'd want a different background/style applied to xml blocks inside Javascript... they really should be rendered more like "string" or heretic something.
So it seems we need to detect actual blocks of HTML (which is how this has always worked in the past) not just individual HTML tags here and there.

You'd have variants also: _jsx_javascript, _jsx_typescript

What other variants are there? Also, wouldn't this be jsx and then tsx?

I'm not sure how people are using HLJS to highlight JSX code but if it was me I'd want a different background/style applied to xml blocks inside Javascript... they really should be rendered more like "string" or heretic something.

While the React website uses Prism, can't we use the React website as the basis of how people [would] use highlight.js to highlight JSX? https://reactjs.org/tutorial/tutorial.html#what-is-react

So it seems we absolutely need to detect actual blocks of HTML (which is how this has always worked in the past) not just individual HTML tags here and there.

Excuse the ignorance, when you say "blocks of HTML" do you mean matching the
 with the
 in your example and consider that a "block?"

What other variants are there? Also, wouldn't this be jsx and then tsx?

I'm not sure, maybe only those two.  The names don't matter they are just internal identifiers.  If the class/sublangauge names matter then we can encode pretty names and then we can discuss what those might be... but it's possible they might both actually be the same (xml).

While the React website uses Prism, can't we use the React website as the basis of how people [would] use highlight.js to highlight JSX? https://reactjs.org/tutorial/tutorial.html#what-is-react

It's one point of data, true, but see my larger point about why we need to identify WHOLE blocks in any case - regardless of how we colorize the background.

Excuse the ignorance, when you say "blocks of HTML" do you mean matching the  with the  in your example and consider that a "block?"

Yes.  We need to know "html starts here" and "html ends here" or else we have no idea whether we are highlighting JS or HTML.

Actually, perhaps this could build on the work in https://github.com/highlightjs/highlight.js/pull/2261 and then we have a tiny HTML parser... so that we're actually MATCHING beginning and end tags... I think JSX is always supposed to have a single wrapping tag in any case, is it not?
You still have the case issue, but that can be solved for a small case like this with just double specifying the case in the HTML matching rules.

I had another look into this. Inasmuch I can't propose something better/simpler, I'm not against it.

Registering of an internal language

Why? Does the idea of internal languages bother you in general? Would it bother you less if they were hidden and not in the listLanguages namespace?
Would a single internal: true flag perhaps suffice?

My concern here is that an internal language can potentially cause some hard-to-troubleshot problem, like name collision or something. That's why I'd prefer either to make _jsx a rightful grammar or to encapsulate it in another namespace.

My concern here is that an internal language can potentially cause some hard-to-troubleshot problem, like name collision or something. That's why I'd prefer either to make _jsx a rightful grammar or to encapsulate it in another namespace.

Except JSX isn't a grammar, it's only a sub-grammar.  You don't think _ is already a namespace? Would __jsx make you any more comfortable? :-)  The name will be auto generated for the language so conflicts isn't really a concern I have... Ie you'd have __jsx_javascript and __jsx_typescript, etc.

The name will be auto generated for the language so conflicts isn't really a concern I have... Ie you'd have __jsx_javascript and __jsx_typescript, etc.

How's that?
  hljs.registerLanguage("_jsx", JSX_LANGUAGE({containers}));

A conflict isn't going to happen and if it does, it's the users problem - since these would be in the languages list they would be visible and avoidable.

It's not visible when you're looking into src/languages catalog...
But yes, underscore is like a namespace, so maybe we'll just need a word or two in developer's docs about this naming convention, and that's it.

How's that?

That's just how it would work internally - to prevent conflicts.

It's not visible when you're looking into src/languages catalog...

No, because it's not a true grammar. :-)

But yes, underscore is like a namespace, so maybe we'll just need a word or two in developer's docs about this naming convention, and that's it.

I agree only to the extent that we should make it easy for people to filter them out (if need be), say if they wanted to say build a dropdown or something - so to that end a small mention in the docs might make sense.  I still think I might scrap this whole approach in favor of the new callbacks though and just match the pair of HTML tags with callbacks... this was an experiment.  We'll see.
Right now I'm focused on other things and waiting for the website/server changes we need so we can do a proper actual v10 release.  So this stuff is likely pushed until 10.1 anyways.

Hi @joshgoebel, can I ask why this was closed? I'd be happy to take this PR over and fix any remaining issues. Unless you want to take a different approach entirely, also happy to do that.

Wish I had left some notes, LOL.   Please see the newly created https://github.com/highlightjs/highlight.js/issues/2998

I'd suggest reading #2998 and the 3 other threads fully and then if you think you have some unique insights we'd be happy to hear them. :)


Setting a Schema for DBUnit for Oracle

If a schema name is available for the connection it will be automatically set for dbunit. With that the DBUnit session will be bound to operate only on the specified schema and will not try to read the whole database

:+1:

CI failure is related to my wrong configuration of Sonar for external PRs.


Train on multiple GPUs on a single host EC2 instance

Do I need to provide any additional arguments to run train.py on multiple GPUs? I see there are some flags for distributed training -- but I'm just focussing on running on multiple GPUs in a single host machine.
Currently, it's training on a single GPU even though the host has multiple GPUs.

Hi @ronanki
I havent updated the code to support multi-gpu training yet.
I will add the multi GPU support ASAP.

Thank you very much.
It worked after making a small change in distributed.py following this issue:
param._execution_engine.queue_callback(allreduce_params)

replaced with
Variable._execution_engine.queue_callback(allreduce_params)


Add display users

Description
Added page to display all saved users
API update
Rout update
Nav updates
Minor test update
Screenshot(s) (if applicable, e.g. style updates)

PR Readiness Checklist

General

[x] Verified all changes are related to feature described above (no scope creep)

Branch Stuff

[x] Checked target and base branch (should be feature branch merging INTO main)
[x] Merged latest main into feature branch (so it has latest code)
[x] Resolved any conflicts between feature branch and main

Test Stuff

[x] Added unit tests to cover modified / additional code

GitHub Stuff (on the right 👉)

[x] Updated labels
[x] Linked issue to which this PR relates
[x] Added assignee
[x] Added at least one (1) reviewer

Still got conflicts in App, looks like

Okay, think I finally fixed that.


remove 'a' remove 'hidden'

Maybe I don't understand the term hidden

Hi @Anderman, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Merged in b7146ec450364b6604cc90a9561182e9a24b8caf


feat: add in request options to python

I wanted to go with a TypedDict here to start that pattern, however in python 3.7 (which we support), there's no easy way to not include keys within the dict (e.g. here you'd always need to include every key: {timeout_in_seconds: None, additional_headers: None, additional_query_parameters: None, additional_body_parameters: None, }) which felt like poor UX.
This can be done better with NotRequired over Optional, but this requires python >= 3.8 to leverage typing_extensions (or typing in >= 3.10)

I think reasonable to bump the dep to python >= 3.8, dont think we should use base model

@dsinghvi would rather not introduce a breaking change in this PR

https://github.com/merge-api/merge-python-client/pull/69 example of a user trying to upgrade us to python 3.8 so that we aren't locking the pydantic version either

merge-api/merge-python-client#69 example of a user trying to upgrade us to python 3.8 so that we aren't locking the pydantic version either

Addressed by https://github.com/fern-api/fern/pull/2967

@dsinghvi would rather not introduce a breaking change in this PR

We made some questionable decisions around forcing the use of pydantic.BaseModel in a ton of places and this continues that trajectory, rather than improving the client. I'd rather have an untyped requestOptions if we want to not make a breaking change and can bump the dep in the next release

Updated the PR to use TypedDicts


Is slice supported as params?

With a query like this:
...
AND sku NOT IN (@SKU)

I'm getting this error:
sql: converting argument with name "SKU" type: unsupported type []string, a slice of string

So, the question is, the string slice is supported by the driver?

Arrays are not supported by SQL server.
On Fri, Dec 14, 2018, 06:22 Diego Bernardes<EMAIL_ADDRESS>wrote:

With a query like this:
...AND sku NOT IN (@SKU)
I'm getting this error:
sql: converting argument with name "SKU" type: unsupported type []string, a slice of string
So, the question is, the string slice is supported by the driver?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/denisenkom/go-mssqldb/issues/432, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuFsXOx7KBAZOOxbs-TgJZud4M-fwIhks5u47RBgaJpZM4ZTnjj
.

But the thing is, I don't want to use it like Postgres where it have native array support.
With other golang sql drivers, when you pass a slice it make a list of elements like this:
skus := []string{"1", "2", "3"}

And a query like this:
AND sku NOT IN (@SKU)

Would generate:
AND sku NOT IN ('1', '2', '3')

@diegobernardes I'm glad that's something you want to do. I do similar things all the time (but not how to describe).
SQL Server Parameters don't work like that. The SQL Query text and each parameters are sent to the server as-is, without any inspection or modification. Thus this would involve parsing T-SQL accurately. This driver used to attempt to do this before database/sql supported named parameters, but it was buggy and hard to do correctly.
So, one thing I do to get around this, especially if I have a number of IDs I want to match (like you describe), is to template the SQL and join the IDs together prior to sending the SQL to the driver. Do escape your inputs and tend to use N'' esp if using nvarchar. Or if that sounds too scary, you can have your template expand a list of identifiers NOT IN (@SKU_1, @SKU_2, ...) and then generate the parameters separately you append to the prams.
Lastly, another way to do this is to join the strings together with a dilimitar then define a table valued function (using CLR) to split the string of values.
select
    p.*
from
    product p
where
    sku not in (select list.Item from dbo.Split(@SKU, N'|') list)
;

This is the most convenient assuming you are joining either numbers or an identifier that is known to never contain a certain character.
Hope this helps.

Thanks @kardianos, helped alot. It's the first time using mssql, sorry my ignorance.


advanced find

Is your feature request related to a problem? Please describe.
How can a tenant find one email amongst millions with this code?
The Veeam Explorer for MS Exhchange/Sharepoint/OneDrive have an 'advanced find' to help find exactly whats required.
Could similar functionality be added to this code?
Would API changes also be required?
Additional context

Thanks Niels.
Would you mind telling me how would eDiscovery be implemented?
Could it be done using the current API or is it on the API roadmap?

It can be done already (see the RESTful API user guide) so it should be possible to have the same options as the GUI.

The API docs linked only cover searching based on from and to addresses ("query": "from: user_X to: user_Y").
Would the more sophisticated search options available in 'advanced find' also be supported? If so, what format should be used?

They are supported. The documentation should be updated soon with all the options but you can use things like cc, bcc, sent, received,... I don't have the full list myself yet though.

Sorry for jumping in on this thread but can you tell me what the full list of search options are? As of now, I'm searching by 'to', 'from' and 'subject', e.g. (("query": "from: user_X to: user_Y subject: XXXX")) but I would like to search by date, e.g. ("query": "from: user_X to: user_Y subject: XXX sent: <>") also. Is this possible?

U should be able to use sent and received for these. I'm not sure if multiple are supported or a maximum of 2 though.
Veeam's documentation should have the full list upon v5 release.

Can you tell me when v5 is due for release?

Can you tell me when v5 is due for release?

@tar565 still working on testing everything and fixing minor bugs. Due to all the restore scenario's (basic and MFA options), this takes some time. I hope to publish the code later this week or next week.

@tar565 still working on testing everything and fixing minor bugs. Due to all the restore scenario's (basic and MFA options), this takes some time. I hope to publish the code later this week or next week.


Update AWS S3/EC2 sync operations (#1290)

This PR will update/create the CI/CD workflow for syncing s3 and updating versions.gridlabd.us. This is a targeted CI/CD update that should be considered separately from the mass image generation and upload PR, as it deals with a much more specific issue and a significantly different potential process, which should not delay the other higher priority PR.
Current Issues
TODO
Code Changes

[x] AWS S3 sync on cloud folders

[x] code.gridlabd.us
[x] docs.gridlabd.us
[x] geodata.gridlabd.us
[x] install.gridlabd.us
[x] status.gridlabd.us
[x] tutorials.gridlabd.us
[x] version.gridlabd.us
[x] www.gridlabd.us

[ ] Upgrade versions.gridlabd.us EC2 instance if needed

Bug Fixes
TODO
Documentation Changes
TODO
Test and Validation notes
TODO

Per @Thistleman, "failing post to s3 curl command in github actions is correct behavior. It is a 400 error as updates are only allowed for the develop and master branches."


About ItemGuard lifetime

Since the inner object of Pool is an Arc, lifetime is not needed.
By removing it, it will simplify usage when someone wants to send ItemGuard<T> via channel.

@oblique that'd be a nice improvement; would you want to file a patch for that?

Sure. I opened a PR.


fix: let user's sent message has markdown style

The user's sent message (which has a "message-request" class name), is currently in "raw text", no styling at all.
Here let it has markdown style.
Showcase:

At least, this really ease my eyes when talking about my long long code block with ChatGPT.
I don't know if there is any edge cases that would cause problem if user's sent message has markdown style.
How do you think?

User input may pose an XSS risk.

Oh, thanks for reminding this. Sure is a serious problem.


Nonstop fireworks when player is down and leaves server

When a player gets downed and leaves the sever fireworks shootout nonstop until sever is restarted.

Fixed in the latest version. Waiting for curse to approve


Fix/zero division

Status
PRODUCTION / DEVELOPMENT
Description
A few sentences describing the overall goals of the pull request's commits.
Related PRs
List related PRs against other branches:

branch
PR

other_pr_production
link

other_pr_master
link

Todos

[ ] Tests
[ ] Documentation

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
roslaunch pkg_A executable_A

The car should move.

https://github.com/CPFL/Autoware/issues/734

How can we actually test this change to prove that the bug has now been fixed?


feat: add function to copy and transpose scalars into device memory (PROOF-642)

Rationale for this change
Add a function that will copy and transpose scalar exponents into device memory.
This will be used as part of the bucket method.
What changes are included in this PR?

Add a new component that creates an array of transposed scalars

Are these changes tested?
Yes.

:tada: This PR is included in version 1.34.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


GH1591: Add ChocolateyDownload alias

Add alias for choco download. Mentionend in the XML comments functionality which requires Chocolatey busines or licensed editions.
Fixes #1591.

@gep13 This should be ready for another review

@pascalberger this looks good to me.
I "think" the plan just now is to complete the 0.20.0 release, which is a major chunk of work, and then start pulling in other new functionality.

@gep13 Any chance to get this in now that 0.20.0 is released? Anything I should do (rebase, etc)?

@pascalberger LGTM, I'll do a final review post CI finishes and then merge if all's ok.

@pascalberger  your changes have been merged, thanks for your contribution 👍


merge package from arlac77/npm-package-template-esm-only

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   94.09%   94.09%
=======================================
  Files           6        6
  Lines         220      220
  Branches       37       37
=======================================
  Hits          207      207
  Misses         13       13

Impacted Files
Coverage Δ

src/database.mjs
82.05% <0%> (-2.57%)
:arrow_down:

src/category.mjs
93.87% <0%> (+1.02%)
:arrow_up:

Continue to review full report at Codecov.

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


audioplayer  setSourceUrl() not found

log:
arm_release_ver of this libmali is 'g2p0-01eac0', rk_so_ver is '7'.
flutter: The Dart VM service is listening on http://<IP_ADDRESS>:41859/8my5k4tgU9Q=/
flutter: ---------------------
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: MissingPluginException(No implementation found for method setSourceUrl on channel xyz.luan/audioplayers)
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:294:7)

I add the dependencies audioplayers: ^1.1.1 in pubspec.yaml
it can play audio on the web,but can not found setSourceUrl on linux device

I don't know which plugin you are using, but it seems that there is a problem with the implementation of the plugin. Plugin implementation is not taken care of in this repository. Please report to the plugin you used.

ok