operator portworx-essentials (24.1.2)

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Are you familiar with our operator pipeline?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Your submission should not

[ ] Add more than one operator bundle per PR
[ ] Modify any operator
[ ] Rename an operator
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/test skip detect-changes

Is there a way to merge this PR? It is a fix for existing portworx-essentials 24.1.2 version where annotations is missing in the portworxoperator.clusterserviceversion.yaml file.

/ok-to-test

/pipeline restart operator-hosted-pipeline

Going to close this now that the related PR is merged and released.


cmd运行报错，报错显示TomlDecodeError

raise TomlDecodeError("Found invalid character in key name: '" +

toml.decoder.TomlDecodeError: Found invalid character in key name: '['. Try quot
ing the key name. (line 1 column 2 char 1)

dupliacte https://github.com/yjqiang/bili2.0/issues/39


Add Emoji Award API

I know there is no test for now, I'll add them as soon as possible. You can review the code whenever. I'll fix as soon as I can.
Fixes: #240

I pushed the new changes.
For the testing what kind of tests are you expecting?
Like the ones in events_test.go or like pipelines_test.go ?

@Lorac This is already looking great! Ultimately @svanharmelen has all the authority here. Regarding tests, I would advise towards the pipelines_test.go approach, events_test.go largely seems to be testing marshaling the types, in the past we've been able to update structs rather quickly.

@svanharmelen Style fixes are in. You can check them out.

LGTM! Thanks again @Lorac!


unable to find option to select app config automatically

While creating a post user has to select app. config
I dont want the user to select the app config. i want to make it default as its confuses some users.
I dont find any option/configuration to select it automatically and skip this step.
Without app config i am getting this error

'NoneType' object has no attribute 'set_author'

@pupattan which version of django CMS / Django are you using?

@pupattan this is currently the behavior of aldryn-apphooks-config, not something djangocms-blog can implement itself
This only happens when you have multiple configurations, right?

i am able to fix this with BLOG_ADMIN_POST_FIELDSET_FILTER
Anyway its a great repo, Helped a lot . Thanks for your work :)

@pupattan thanks :bow: glad it's helpful


Refresh cache more often in GUI while wallet is syncing

This applies to both newly created wallets and opening an existing wallet.
The cache is refreshed every 10 seconds, which means we may have to wait slightly longer to detect that the wallet has finished syncing / checking for new transactions.
We could refresh the cache more often while the wallet is not synced. However, the full information about whether the wallet is syncing is not currently available where the cache refresh interval is set. In the short-term, we could reduce the interval in the simple case that the wallet height is 0 and backend is remote or Electrum server (i.e repeating the logic used elsewhere).
Later on, we can reorganise and reuse the existing code to avoid logic repetition and cover all syncing scenarios.

As we agreed elsewhere, we will try to tackle the short term part for v8 and then leave the longer term solution for a future release.


How to close the connection to the server

Hi @wickning1  - Probably newby question, but is there a proper way to unbind or close the connection to the LDAP server? I wrote a async function that makes some queries to write data to a file, but the program never exits after it's complete.
Thanks!

Currently there's no way to close down the pool: typically you want to keep it running for the life of the process. For a CLI tool you can call node's process.exit() to end your CLI execution and when the process dies that will release all the connections.
In some rare circumstances, you might want to keep your process running after you're completely done using LDAP and you're concerned about leaving open resources. I'll take a look at adding a close() method.

Added and released as 1.3.2.


Can't move to lazy parent route from ErrorComponent

Describe the bug
Can't move to lazy parent route from ErrorComponent. See steps to reproduce
Your Example Website or App
https://stackblitz.com/edit/github-xwen9w-y2yx3p?file=src%2Froutes%2Ffoo%2Fbar.tsx,src%2Froutes%2Ffoo%2Findex.lazy.tsx
/foo/index.lazy.tsx
export const Route = createLazyFileRoute('/foo/')({
  component: () =>
Hello /_others/foo!
,
});

foo/bar.tsx
export const Route = createFileRoute('/foo/bar')({
  beforeLoad: () => {
    throw new Error();
  },
  component: () =>
Hello /_others/foo/bar!
,
});

Steps to Reproduce the Bug or Issue

Click on foobar. The error thrown in beforeLoad. The DefaultErrorRouteComponent is showing
Click on foo. Nothing happend

Expected behavior
Need to move to foo route
Screenshots or Videos

Platform

OS: Windows
Browser: Chrome
Version: 123.0.6312.106

Additional context
No response

Currently I see only one workaround - change /foo route to non-lazy. But it's not suitable for me.

@tannerlinsley Hello! I have tried 1.28.5 and in this version error route component doesn't showing when throw error in beforeLoad. I just see empty page. You can try with example on this ticket, just change the version.
Previous version (1.28.4) works the same as described in the ticket.

Closing because as mentioned, this has been fixed.


(ham)burger problem

As @andyli informed in https://github.com/addshore/wikicrowd/issues/71#issuecomment-1500742867 there is issue with hamburgers.
One thing is that script is checking vegan stuff - I proposed change here: https://github.com/addshore/wikicrowd/pull/82
Second thing is that images marked by Q112156900 (vegan burger) and Q607754 (chicken burger) should not be considered for check. @addshore can you add this?
This two things are (probably) fast to fix. But there is third one. Third one is philosopical - what is hamburger. Because there is https://commons.wikimedia.org/wiki/Category:Hamburgers_by_type milion types of burgers and most of them dont contain ham. And I dont know if there is language difference between US/UK/AUS/other kinds of flavors of English language. It might be that we should turn off hamburger challenge, if we are marking as "hamburger" stuff that dont contain ham.
My edits that are reverted:

https://commons.wikimedia.org/w/index.php?title=File%3ABuffalo_Vurger_-_The_Vurger_Co.%2C_Brighton.jpg&oldid=prev&diff=748002883&diffmode=source
https://commons.wikimedia.org/w/index.php?title=File%3ACrispy_'Chicken'_burger_with_Tater_Tots_-_The_Vurger_Co.%2C_Brighton.jpg&oldid=prev&diff=748002423&diffmode=source
https://commons.wikimedia.org/w/index.php?title=File%3A外賣_雞扒包_黑糖鮮奶_-_添福小食.jpg&oldid=prev&diff=748002342&diffmode=source

Looking at the burger problem, perhaps hamburger should not be used, instead just "burger"
http://www.wikidata.org/entity/Q111981894

It happened again to these files:

https://commons.wikimedia.org/wiki/File:Hot_Chic_-_Patty%26Bun_2024-02-27.jpg
https://commons.wikimedia.org/wiki/File:Fritter_with_rosemary_chips_-_Honest_Burgers_2023-11-27.jpg
https://commons.wikimedia.org/wiki/File:Veggie_Sandwich_with_Cheese_-_Five_Guys_2023-10-21.jpg
https://commons.wikimedia.org/wiki/File:Chicken_cheese_burger_and_Mexican_vegetarian_burger_-_Grubbs.jpg
https://commons.wikimedia.org/wiki/File:Fancy_Veggie_Burger_and_Fries_-_Joe's_Burger_House_2024-07-07.jpg
https://commons.wikimedia.org/wiki/File:T_<EMAIL_ADDRESS><EMAIL_ADDRESS>https://commons.wikimedia.org/wiki/File:Vegan_Hot_Chic_-_Patty%26Bun_2024-04-08.jpg


Implement a Skip Button for Transcription Tasks

Is your feature request related to a problem? Please describe.
Sometimes there are sentences that I'm not comfortable with or I'm not focused enough to transcribe.
Describe the solution you'd like
Having a skip button will allow me to move to the next sentence and ignore it.
Describe alternatives you've considered
Listening and transcribing anyway.
Additional context
Nope, it's pretty simple.

Describe alternatives you've considered
One alternative is to rename the "Not Success" button to "Skip" and add options to the reasons such as "Other/Not Fully Confident."
Additional context
This change will require updates to the frontend only. Implementing a skip button will improve user experience by allowing for a smoother and more efficient transcription process.

I think these are different things, sometimes I disagree with the content or I would like an easier string because I'm not super focused, having the ability to voluntarily skip without specifying why is different than marking the content as problematic for objective reasons.

Transferred to crowd-transcribe repo


Add user response and thinking robot before flowstep callback

In your formless example, flowstepcallback is used to fetch content from server, but in the case where the server is slow, it seems more intuitive to add the user response to the chat list and set the robot to thinking till the success function is called. Could you please let me know if that's possible?
Thank you

@karthikradhakrishnan96 makes sense, I will implement in the coming days.


IContentSecurityPolicyPage pages are not causing an update of the CSP for the page

How to Reproduce:

Go to a page which implements IContentSecurityPolicyPage
View the page on the front end and examine the CSP header
Update the sources assigned to the page
Reload the page on the front end and examine the CSP header

Note that the CSP has not been updated

In page properties set the "Update Modified Date" tick box
Reload the page on the front end and examine the CSP header

Note that the CSP has now been updated

This is because the modified date doesn't update with each content change.  As there is a time element for the cache expiry, I'm going to deprioritize this issue for now.

Pending Release.


Minor clarification in the README

I think the comment reads more clearly this way. I consulted the code comments to make sure I correctly understood what these two functions do.

Coverage remained the same at 100.0% when pulling 2b2a140b732989bb98d87675d104c94af94ebc3a on rgorsuch:master into ebf56d35bba727c68ac77f56f2fcf90b181851aa on fatih:master.

Thanks @rgorsuch


The ase_exploration doesnt fallback to frontier exploration

When the ase_exploration fails the frontier exploration doesnt start automatically there is some issue with the launch files

Hi, you can try to comment out any part of the launch files that starts up frontier exploration, and instead launch it manually to try and work around this. Unfortunately I have stopped maintaining this package and cannot help you further in debugging this.


Could you please add the Khmer corpus?

As above, thank you！

Hi @guangbin79. Thanks for asking. Yes, adding a corpus for Khmer is something that I would like to do.


bug(rich-text) selection that includes link will move link popover to top of selection

Describe the bug
n rich text mode, when selecting a link and moving upwards, the link popover is also moved up vertically to under blocks that aren't links.
To Reproduce

RT mode, type in a few lines of text
Make a link towards the bottom
Start to highlight from bottom to top
Observe: link popover stays visible, and is stuck to the top of the selected area

Expected behavior
No link popover visible unless the selection is only on one link. Otherwise only show the link popover when the cursor is on the link.
Screenshots

Desktop (please complete the following information):
Windows 11 Chrome 103
Additional context
https://meta.stackexchange.com/a/380439/51

Popover is currently following the selection $anchor, should be instead following the selection $head.


Apache DevLake - v1.0.1 - Manifest not found error - Unable to install using docker

Search before asking

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

What happened

What do you expect to happen
Unable to install using docker
How to reproduce
Unable to install using docker
Anything else
Unable to install using docker
Version
v1.0.1
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@Dosu - I have downloaded the .env and docker-compose from the location. I see that the paths are right but still i get this error.

using latest worked.


Field values changing types

Hi!
I'm calculating a global dirty value on my form to display the "update" button if any field is modified (so dirty) but it failed on fields with numbers.
On both text or a number inputs, the initial is 100 (int, feeded by an API) and value is "100" (if modifing the field, then rolling back)
Is there a way to cast fields so that my number values stay ints, or anything that would ensure the consistency of types ?
Thanks!

You could potentially use normalize to do your casting. The library itself does not have a strong opinion on type casting.

puuuurrrrr fect.


[KOGITO-1600] - Prepare 0.9.0 Kogito Operator release

Signed-off-by: Ricardo Zanini<EMAIL_ADDRESS>See:
https://issues.redhat.com/browse/KOGITO-1600
Nitpicks/scorecard check for 0.9.0 release.
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

Waiting for https://issues.redhat.com/browse/KOGITO-1601

OperatorHub PR: https://github.com/operator-framework/community-operators/pull/1475


Fix RenderTargetBitmap section header formatting

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


enable testing without client uploads

If you set the env variable LOCAL=local then you can stop the core from requesting images and tests to be uploaded from the client
Change-type: patch
Signed-off-by: Ryan Cooke<EMAIL_ADDRESS>

Should we also perhaps explore testing possibilities where we don't have the flash new images or configure networks, again and again, to speed up the local development process. With core now requesting for images we definitely will be using the image and tests we already have. If there is a way to stop flashing the image as well through the Local env, then I think we should check it out. Flashing takes 6 minutes.

@vipulgupta2048 I like that idea a lot, I think another PR for that would be good. You can actually check for the local env in the test suite to be fair, and then skip steps if you don't want them

This PR was to be able to use the core + worker with fin QA

@balena-ci rebase

@balena-ci I self-certify!

Sounds good to me @rcooke-warwick I think another PR for that would be good. Since flashing and network configuration could be a step that people might want to repeat and not forego.


Integration test against Windows environment

Add ability to integration tests against Windows environment
Integration testing against puppet agents running on Windows is possible and configurable. Export the WINDOWS_DOCKER_HOST envvar to enable the Windows tests. The configurable options are available as envvars WINDOWS_DOCKER_HOST, WINDOWS_DOCKER_CERT_PATH, WINDOWS_DOCKER_TLS_VERIFY, MAIN_HOST_IP. MAIN_HOST_IP is the IP address of the host where the tests are running that should be accessible from containers running in the Windows Docker Daemon.
Add Windows node to integration tests in Jenkins
Adds a Windows 2016, with containers, node to the pipeline using the parallel stages approach for cooperation between Jenkins nodes. This provides a Windows Docker daemon for the integration tests to use

Here's an example of a run where the Windows environment is available https://asciinema.org/a/4QANgWX4IEgU6vOx9Za8TdlAl.
What does this PR do?

What's changed? Why were these changes made?
How should the reviewer approach this PR, especially if manual tests are required?
Are there relevant screenshots you can add to the PR description?

What ticket does this PR close?
Connected to #[relevant GitHub issues, eg 76]
Checklists
Change log

[ ] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[ ] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[ ] This PR does not require updating any documentation

@sgnn7 since @doodlesbykumbi is out today, can you clean up the minor nits (indenting, text suggestions, file location) and put the rest in a follow-up issue? even the file location part can go in a follow-up issue if it's going to put the build at risk of going red - I'd like to get this in and not delay any further.


Add LFS control

Add ability to turn LFS on / off for repo

Implemented added in https://github.com/google/go-github/pull/2429


Example App is broken

The current example app seems not to be working.

I believe you have to call "initialize()" before getting the image stream.

Even after calling initialize(). keeps breaking.

Keep broken for me
E/flutter ( 6392): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: RangeError (index): Index out of range: index should be less than 460852: 460852
E/flutter ( 6392): #0      _Int32ArrayView.[] (dart:typed_data-patch/typed_data_patch.dart:4260:7)
E/flutter ( 6392): #1      new RGBLuminanceSource (package:zxing2/src/rgb_luminance_source.dart:28:25)
E/flutter ( 6392): #2      decode (package:zxing2_example/decode.dart:7:28)
E/flutter ( 6392): #3      _CameraAppState.initState.<anonymous closure>.<anonymous closure> (package:zxing2_example/main.dart:37:23)
E/flutter ( 6392): #4      CameraController.startImageStream.<anonymous closure> (package:camera/src/camera_controller.dart:395:20)
E/flutter ( 6392): #5      _rootRunUnary (dart:async/zone.dart:1362:47)
E/flutter ( 6392): #6      _CustomZone.runUnary (dart:async/zone.dart:1265:19)
E/flutter ( 6392): #7      _CustomZone.runUnaryGuarded (dart:async/zone.dart:1170:7)
E/flutter ( 6392): #8      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
E/flutter ( 6392): #9      _DelayedData.perform (dart:async/stream_impl.dart:591:14)
E/flutter ( 6392): #10     _StreamImplEvents.handleNext (dart:async/stream_impl.dart:706:11)
E/flutter ( 6392): #11     _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:663:7)
E/flutter ( 6392): #12     _rootRun (dart:async/zone.dart:1346:47)
E/flutter ( 6392): #13     _CustomZone.run (dart:async/zone.dart:1258:19)
E/flutter ( 6392): #14     _CustomZone.runGuarded (dart:async/zone.dart:1162:7)
E/flutter ( 6392): #15     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1202:23)
E/flutter ( 6392): #16     _rootRun (dart:async/zone.dart:1354:13)
E/flutter ( 6392): #17     _CustomZone.run (dart:async/zone.dart:1258:19)
E/flutter ( 6392): #18     _CustomZone.runGuarded (dart:async/zone.dart:1162:7)
E/flutter ( 6392): #19     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1202:23)
E/flutter ( 6392): #20     _microtaskLoop (dart:async/schedule_microtask.dart:40:21)
E/flutter ( 6392): #21     _startMicrotaskLoop (dart:async/schedule_microtask.dart:49:5)

I'm using the dart image package to get my luminance and I've had it work great.

in brach xha/fix_example only remove this comments:

Converted fromCameraImage to Image and it worked. But its only work with ResolutionPreset.high

I got it. serialized ImageCamera to work with isolate. and it was beautiful!

https://pub.dev/packages/zxing2/example do not using ZXing at all!! Nice to look it as an example ))


Feature/3_1

commit 1640eb6a152b5e97bf7d87c83785b39416686f28 (HEAD -> feature/3_1, origin/feature/3_1)
Author: Chaim Eliyah<EMAIL_ADDRESS>Date:   Sun Apr 19 23:14:24 2020 -0700

    implement shop by category, resolve #3

commit f786a4110ba4f67624c4eefa723442cf8cf73b13
Author: Chaim Eliyah<EMAIL_ADDRESS>Date:   Sun Apr 19 23:03:31 2020 -0700

    setState method for clickCategory action

commit 5394d85dd57cf978bcdce23d263339dc052ceeed
Author: Chaim Eliyah<EMAIL_ADDRESS>Date:   Sun Apr 19 22:55:55 2020 -0700

    expand category list in fakes

Make sure you understand the concepts here before proceeding. If you need explanations of anything in green, please ask.


After saving a navigation item, it redirects to wrong navigation results

Should be a simple bug, redirection issue.

Fixed with commit 2dfb1fad7396f3038d31f067c5b0bbcd4890c2c0


Precompile better with SnoopPrecompile.jl

Codecov Report

Merging #173 (d469474) into main (42e58bb) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
- Coverage   90.36%   90.28%   -0.08%
==========================================
  Files          24       24
  Lines         944      947       +3
==========================================
+ Hits          853      855       +2
- Misses         91       92       +1

Impacted Files
Coverage Δ

src/ProbNumDiffEq.jl
63.15% <0.00%> (-3.51%)
:arrow_down:

src/diffusions.jl
98.50% <0.00%> (+0.02%)
:arrow_up:

src/initialization/common.jl
96.00% <0.00%> (+0.16%)
:arrow_up:

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


Use categorical dtypes for low cardinality variables?

Following discussion in #473 , it might make sense to convert string and object dtypes to categorical, for low cardinality variables, to save memory and time.

[ ] Benchmark memory saved by categorical dtype

Closing this issue because it has been implemented in TableVectorizer since


rustup-setup.exe panics before install completes, does not configure PATH

Just tried this on Windows 8.1

First of all, "Windows SmartScreen prevented an unrecognized app from running. Running this app might put your PC at risk. Publisher: Unknown Publisher".

After the confirmation Y of Continue? (Y/n) the console window closes very quickly. I noticed some panic message before the window closed, but it disappears too quickly to be read. HKEY_CURRENT_USER/Environment/PATH is not updated, some binaries however appear in %USERNAME%.cargo.

Source.

the console window closes very quickly

rustup-setup.exe not reusing the existing console from which it's run, but creating its own console instance, is probably a problem too.

rustup-setup.exe not reusing the existing console from which it's run, but creating its own console instance, is probably a problem too.

@petrochenkov When I run rustup-setup.exe from a command console or mintty console it does not create a new console. I've filed another issue to deal with that specifically.
And I've changed this issue to be just about the mystery error and closing console. Here's another bug about windows PATH not being configured.

Agh! I reproduced this this morning but no longer can. When I run the restup-setup.exe from exe.rustup.rs it opens a console, waits for confirmation, does the install, then waits for keypress as expected.

Haha, ok, I can reproduce what this is probably seeing by having a compile running while trying to install, making it impossible to delete the bins. What I can probably do is make it so interactive installs on windows display the error, then wait for keypress.


[C++][Parquet] Minor: moving EncodedStats by default rather than copying

Describe the enhancement requested
Moving EncodedStats because it holds two std::string. This could benifit for non-SSO optimized data for FLBA/String statistics ( It seems to be useless for SSO optimized data?)
Component(s)
C++, Parquet

Issue resolved by pull request 41727
https://github.com/apache/arrow/pull/41727


compatibility test occasionally fails on jdk9

com.netflix.spectator.api.CompatibilityTest > check FAILED
    org.junit.ComparisonFailure at CompatibilityTest.java:202
295 tests completed, 1 failed
:spectator-api:test FAILED

Usually works if build is run again.

com.netflix.spectator.api.CompatibilityTest > check STANDARD_ERROR
    SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
    SLF4J: Defaulting to no-operation (NOP) logger implementation
    SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
    [Measurement(gauge,1234567890,49.0)] != [Measurement(long-timer:a=b:statistic=activeTasks,1234567890,3.0)]


fix: allow non system:master users to create/update rbac

What type of PR is this?
/kind bug

What this PR does / why we need it:
When users who is not in system:master group try to create or update rbac resources, there is a panic in type assertion, which  converts *unstructured.Unstructured into *rbac.Role/ClusterRole/RoleBinding/ClusterRoleBinding.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

NONE

/assign


Feature request: List.LinkItem should extends Link (preact-router)

When navigating using
Link

<List.LinkItem href="/somePath">Link</List.LinkItem>

The whole app is rendered again.
When navigating using
<Link href="/somePath">Link</Link>

<List.Item onClick={() => route('/somePath')}>Link</List.Item>

Routing works as expected for a SPA
Would it be possible to either extends the Link component from preact-router or provide the same behavior when navigating, as well as the activeClassName for List.LinkItem as well as the other link components if there are others  ?

Should this really be included in this repo though? Preact can also be used with other routers, and this package doesn't depend on preact-router, so I think it might be better to just use onClick or extend the component yourself?

I agree with the separation concern.
However, because the List.LinkItem is semantically a link it should support navigation in a SPA, or not exist at all if it doesn't bring anything more than an List.Item.
I proposed to either directly re-implement preact-router Link behavior (not a fan of this idea)
or
Make this package depend on preact-router, but as you said, there may be people using other routers so it would result in bigger bundle for them.
The problem with the onClick technique is that's it's not good for SEO and it adds a listener where a simple href attribute solve both problem. (Seems that Link from preact-router is using the onClick method to work in SPA, but keep the href attribute to allow SEO).
Maybe the proposition should be to remove this sub-component if either of the propositions above sounds like separation concern breaking.

And wouldn't something like:
class PreactRouterLinkItem {
	_onClick(e) {
		// route + prevent(e)
	}

	render({ children, href }) {
		return <List.LinkItem href={href} onClick={this._onClick}>{children}</List.LinkItem>;
	}
}

do the trick?

Yep that's almost exactly what I ended up with after reading preact-router implementation.

Right, but shouldn't that be enough then? You can fairly easily plug in any router you want without the library depending on it. Maybe we could add this example to the documentation, but I don't quite think that this belongs in this package.

I kinda agree with @filoozom because more prople use react-router than preact-router.

The problem with the onClick technique is that's it's not good for SEO and it adds a listener where a simple href attribute solve both problem.

The best part with this is you can put and href and prevent event's default and still call
route(event.tartget.getAttribute('href'))


Set arrow-body-style to 0

arrow-body-style is a confusing and troublesome rule, especially for Promise chains. We are setting the rule to 0. It is the developer's choice whether or not to use braces when surrounding arrow-bodes.
@jhurliman

LGTM. This will be a minor version bump I think since it's loosening the linter.


[vcpkg] Improve vcpkg::Files::Filesystem error handling

thank you so much ))
"DB" can now be edited


Improve info in case of tracing activities

Description:
While conducting debug activities you often enable trace log level, and you want to check the actual requests and response between client and server (i.e., #1802).
This PR:

adds info about request and response bodies when trace log level is enabled.
reduces the default log level to info instead of trace

Related issue(s):
N/A
Notes for reviewer:
N/A
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

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

Comparison is base (63953ee) 77.73% compared to head (53bdad4) 77.75%.

:exclamation: Current head 53bdad4 differs from pull request most recent head 748d373. Consider uploading reports for the commit 748d373 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1817      +/-   ##
==========================================
+ Coverage   77.73%   77.75%   +0.02%
==========================================
  Files          40       40
  Lines        3023     3026       +3
  Branches      603      603
==========================================
+ Hits         2350     2353       +3
  Misses        489      489
  Partials      184      184

Files
Coverage Δ

packages/server/src/koaJsonRpc/index.ts
68.22% <100.00%> (+0.60%)
:arrow_up:

packages/server/src/server.ts
69.51% <100.00%> (+0.12%)
:arrow_up:

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

PR #3133 Updates the configuration doc to list the default LOG_LEVEL for now.


Weird Segfault.

I noticed a weird segfault that appeared in my code. Here is a small reproduction:
This segfaults when ran, yet
Def Push-op as (Let Op; Push Op);
Let Commands be List ("+" "-" "%" "*")
Let Error be False;
List ();
For command in Commands
    Case "+" (Push-op)
    Case "-" (Push-op)
    Case "*" (Push-op)
    else (Set Bad-command; Set Error to True; Prints ("Error: Unknown Command " Bad-command "\n"));
Do If Not Error
    (Set Commands; For command in Commands Print)
    (Print "");
Clear;

This code works,
Def Push-op as (Let Op; Push Op);
Let Commands be List ("+" "-" "%" "*");
Let Error be False;
List ();
For command in Commands
    Case "+" (Push-op)
    Case "-" (Push-op)
    Case "*" (Push-op)
    else (Set Error to True; Put "Error: Unknown Command "; Print);
If Not Error
    (Set Commands; For command in Commands Print)
    (Print "");
Clear;

and the only difference seems to be this code which causes the segfault
else (Set Bad-command; Set Error to True; Prints ("Error: Unknown Command " Bad-command "\n"));

as opposed to this code which works.
else (Set Error to True; Put "Error: Unknown Command "; Print);

Further information: Prints doesn't segfault by itself, based on this reproduction I made that works,
Let Bar be  "Bar"; Prints ("hielo " Bar "\n");

seems to be segfaulting at compile time

should be fixed in 86d2fdb


Issue maintenance permissions for all repositories

This is an issue around the maintenance of issues of adding a label and maintaining the assignee for the issue.  It does not appear that anyone can add a label or change the assignee if they are not a committer on the repository.  This is a very restrictive rule.  Can this be changed?
When adding this issue I cannot assign to anyone or add a label on this either.

Even after adding I cannot assign or put a label on.

That is by design; generally those individuals who are committers would be the ones triaging issues.
One way I do see communities automate this process is through the use of a CODEOWNERS file, which can automatically request reviews from the committers responsible for areas of the code base. Learn more about it at https://docs.github.com/en/free-pro-team@latest/github/creating-cloning-and-archiving-repositories/about-code-owners/.
In addition, using issue templates and pull request templates can be used for automatically applying labels based on the issue coming in. We use this extensively for OMP for this purpose, as well as to provide a template for issues and pull requests. Learn more at https://docs.github.com/en/free-pro-team@latest/github/building-a-strong-community/about-issue-and-pull-request-templates
Let me know if this helps address the concern!

Andrea will assign herself to this issue to research links provided by John

Committer responsibilities established.

Having discussed at TSC, this meeting is considered closed.


Interactive Rebase Editor not opened but plain text file when called from terminal

Calling "Rebase current branch onto Commit" as "interactive" opens the "Interactive Rebase Editor" just fine. The command executed under the hood in the terminal is git -c sequence.editor="code --wait --reuse-window" rebase --interactive e2db9b3b250adc217665d30640c16f2fdfe774c8
However, pasting the same command in the terminal again does not open that editor, it just opens VSCode as a plain text file!
The same problem applies when running git rebase -i from terminal, just a plain text file is opened in VSCode, no Interactive Editor.

GitLens Version: 11.3.0
Git Version: Run git --version 2.25.1

VSCode Version: Run code --version 1.54.3
OS Version: Ubuntu 20.04.2 LTS

Steps to Reproduce:

set up git to use VScode as editor

[core]
	editor = code --wait

run git rebase -i ...
vscode opens git-rebase-todo as plain text file

Errors seen:
[2021-03-24 09:02:22:953] [/home/..../.git/rebase-merge] git rev-parse --show-toplevel • FAILED 8 ms

Error: Command failed: /usr/bin/git -c core.quotepath=false -c color.ui=false rev-parse --show-toplevel
fatal: this operation must be run in a work tree

Is this under WSL or are you on Ubuntu natively?

Can you please verify this fix in tomorrow's insiders edition?
You can install the insiders edition from here. Be sure to disable/uninstall the stable version of GitLens first.

Is this under WSL or are you on Ubuntu natively?

native Ubuntu


Implement ERC20 indexing status endpoint

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

20 of 20 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 88.65%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
5975

Relevant Lines:
6740

💛  - Coveralls


GO:0031508   pericentric heterochromatin formation

single inheritance
GO:0031508   pericentric heterochromatin formation
has only one none-regulation child
GO:0140727   siRNA-dependent pericentric heterochromatin formation
are there other mechanisms of pericentromeric heterochromatin assembly, or do they all involve siRNA-dependent
@colinlog
(for pombe it is only siRNA-dependent)

There are non.RNA dependent mechanisms of pericentric heterochromatin formation, see for eg PMID:14612388
So we decided to

[ ] remove parent ''small ncRNA-mediated heterochromatin formation'' from 'pericentric heterochromatin formation' ''
[ ] add direct asserted superclass ''small ncRNA-mediated heterochromatin formation' to GO:0140727 siRNA-dependent pericentric heterochromatin formation

Are you sure about this?

There are non.RNA dependent mechanisms of pericentric heterochromatin formation, see for eg PMID:14612388

"These results are consistent with a model in which Lsh is recruited by intact heterochromatin structure and then assists in maintaining heterochromatin organization by establishing CpG methylation patterns."
I remember when we  originall discussed  this, we concluded that establishment (aka formation) was always via siRNA mechanisms.
this paper (PMID:14612388) is describing  non-RNA dependent mechanisms for maintaining pericentric heterochromatin, (i.e after it is formed). Do you have any other papers supporting a non RNAi-based mechanism for pericentromeric heterochromatin formation?

We obsoleted 'heterochromatin maintenance' - the paper states "These results are consistent with a model in which Lsh is recruited by intact heterochromatin structure and then assists in maintaining heterochromatin organization by establishing CpG methylation patterns" - so heterochromatin formation still seems OK for this.
Do you suggest hat we restore heterochromatin maintenance ? Then would that be RNA-independent?

Actually, if
[RNAi-dependent heterochromatin assembly.]stays under
pericentric heterochromatin formation
this is probably OK.


⚠ Bofrost.fr has degraded performance

In 0f69f6d, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1932 ms

Resolved: Bofrost.fr performance has improved in 4be1f36.


Update Roadmap.md

Overview
Changed the path to the vNext docs, since the branch was renamed :D
Changes

Fixes: All the things

changed base to htk_development as we do not take changes directly into master :)

Then best get it merged to dev and then master quickly.  As it's master that is broken!


Fix errant capitalisation in DNS TXT response

Rebases #627.

@rktjmp Are you able to sign the updated Salesforce CLA?


Support self-signed certificates with Elasticsearch

@aliostad asked about self-signed certificates with ES. I mentioned if this becomes common practice, we could consider supporting it. Looking at spark, it does seem like common practice, as they even have a configuration item for it.
https://www.elastic.co/guide/en/elasticsearch/hadoop/current/configuration.html#_ssl
Please upvote if you are interested in this cc @openzipkin/elasticsearch

Hi, thank you!
Problem was that we had a valid cert but the hostname was not matching and there is absolutely no way to sort this out.
I spent hours and days (and I am glad because I learnt so much) on it and imported the cert (by creating my own dockerfile and adding steps to do it) only to find out the error is hostname not verified.
I wish we could have an option to disable hostname verification. This is what I was getting and I could not find no non-code solution. There was a solution by running -javaagent:blah.jar but seems it replaces spring's agent.

👍

Related to #1833?

https://github.com/openzipkin/zipkin/tree/master/zipkin-server#using-a-custom-key-store-or-trust-store-ssl

I wish we could have an option to disable hostname verification.

I still do not see that there is a option to disable hostname verification. This could be a useful thing.


Tests for FluentSiteTreeExtension::Link()

Migrated from old 3 branch, along with various tests and cleanup to the (awesome) tests that are already there. :)
For @robbieaverill to review

PR for scrutinizer fix at https://github.com/tractorcow/silverstripe-fluent/pull/295


Invariant Violation: _registerComponent(...): Target container is not a DOM element.

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Trying to mount a component for unit testing and getting error. Has this come up for anyone before?
    Invariant Violation: _registerComponent(...): Target container is not a DOM element.

    at invariant (node_modules/fbjs/lib/invariant.js:42:15)

Code
...
import { FormattedMessage } from 'react-intl'

class TestMenu extends PureComponent {
  render() {
    return (

           <FormattedMessage
               defaultMessage={'HELLO'} />

    )
  }
}

export default TestMenu

test.jsx
describe.only('<Test />', () => {
  let testMenu

  beforeEach(() => {
    testMenu = mount(<TestMenu {...props}/>)
  })

  it('Should render the Formatted Component with 'Hello'', () => {
    expect(testMenu.find('span').text()).to.equal('Hello')
  })

Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
"babel-jest": "^21.2.0",
"chai": "^4.1.1",
"jest": "^21.2.1",
 "webpack": "^3.4.1",

Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions. Thank you :)


pull request for e2e test at 01/20/2017 14:45:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


I want to work on a libary that works with the newest version of snapchat.

I notice this repo is still using really old endpoints such as /bq/login.
I want to create a new libary to use with the newest version of snapchat, currently <IP_ADDRESS>.
This includes all the new endpoints and a way to avoid detection.
I have both the newest endpoints and the means needed to avoid detection.
Would you like to work with me on this? Please leave a message below with your skype ID.

I would like this. My account got locked rather quickly. (I'm not a dev here, but rather a user.)
Could you just fork it and fix what you can, Snowlav? Or make a pull request.

Is there a disclosure online on the new API like Gibsec's on the old one?

I am afraid not, we must make one ourselves
2015-02-22 5:51 GMT+01:00 Anastasis Germanidis<EMAIL_ADDRESS>
Is there a disclosure online on the new API like Gibsec's
http://gibsonsec.org/snapchat/fulldisclosure/ on the old one?
—
Reply to this email directly or view it on GitHub
https://github.com/agermanidis/SnapchatBot/issues/6#issuecomment-75419682
.

I think there's a lot of people that would appreciate a new library.
If you do host a page with documentation such as Gibsec, could you also post simple explanations on how Snapchat communicated with the API? I understand it's through POST requests and then parsing the reply, but is the actual snap data (blob is it called?) stored in the reply?

Using the new API is definitely worthwhile (my IP has been blocked a bunch of times too) but I'm not sure I have the time to work on it in the immediate future.
It'd be awesome if you decide to work on it @Snowlav -- you can either fork this library or make a lower-level API client library that SnapchatBot could use instead of pysnap.
Cheers,
-A

I've just been doing some research and I believe it's possible to manipulate the chat on Snapchat with loq.

+1

Maybe route the connections through Tor to avoid IP's getting blocked? I'll probably make a PR with that now that I think about it.

Evan it's not needed now that I have found a way to avoid detection :)
2015-02-23 17:34 GMT+01:00 Evan Hopkins<EMAIL_ADDRESS>
Maybe route the connections through Tor to avoid IP's getting blocked?
I'll probably make a PR with that now that I think about it.
—
Reply to this email directly or view it on GitHub
https://github.com/agermanidis/SnapchatBot/issues/6#issuecomment-75577770
.

@Snowlav you're working on something ? :)

Yes and no.
I want to build a new library but my coding skills aren't great.
What I can do however is reverse engineer and that besides a lot of
networking is how I found out some exploits and how to avoid detection.
I got in touch with some clever minds. And if any of you want to contribute
to building a library along with me and those people, again, feel free to
leave your skype and I will contact you :)
On Feb 24, 2015 12:27 AM, "N07070"<EMAIL_ADDRESS>wrote:

@Snowlav https://github.com/Snowlav you're working on something ? :)
—
Reply to this email directly or view it on GitHub
https://github.com/agermanidis/SnapchatBot/issues/6#issuecomment-75660935
.

Even if you could even make a document with the new endpoints, that would be super helpful too!

/loq/gae_server_list
/loq/ping
/loq/all_updates
/loq/conversation
/loq/conversations
/loq/double_post
/loq/send
/loq/retry
/loq/login
/loq/register_username
/loq/friend_search
/loq/clear_conversation
/loq/clear_feed
/loq/conversation_post_messages
/loq/conversation_auth_token
/loq/local_story
/loq/friend_hide
/loq/update_user
/bq/delete_story
/bq/friend
/bq/get_captcha
/bq/bests
/bq/story_blob
/bq/post_story
/bq/retry_post_story
/bq/set_num_best_friends
/bq/upload_chat_media
/bq/solve_captcha
/bq/update_feature_settings
/bq/update_stories
/bq/chat_typing
/bq/update_snaps
/bq/user_exists
/bq/phone_verify
/bq/story_thumbnail
/bq/chat_media
/ph/device
/ph/find_friends
/ph/blob
/ph/settings
/ph/upload
/ph/logout
these are most new ones
2015-02-24 1:29 GMT+01:00 Anastasis Germanidis<EMAIL_ADDRESS>
Even if you could even make a document with the new endpoints, that would
be super helpful too!
—
Reply to this email directly or view it on GitHub
https://github.com/agermanidis/SnapchatBot/issues/6#issuecomment-75672232
.

I'm down to work on it. Skype: ehopki

Do you have Telegram ?

To anyone interested, I don't think Snapchat sends chat messages through the API as you would've expected; the endpoint when interacting with the endpoint is "chat_typing". It looks something like this:
{
    "timestamp" : timestamp,
    "recipient_usernames" : [users],
    "username" : "username",
    "req_token": token,
    "features_map" : data
}

What does datacontain ?

@robert bq_typing is doing exactly what it says, it only indicates that you
are typing, it's not holding mac / payload.
2015-02-26 0:34 GMT+01:00 N07070<EMAIL_ADDRESS>
What does datacontain ?
—
Reply to this email directly or view it on GitHub
https://github.com/agermanidis/SnapchatBot/issues/6#issuecomment-76085364
.

Yeah I understand that, I'm just saying that chat_typing is the only endpoint I've seen so far when interacting with the chat. It's used to give a heads up notification that someone is typing (to the recipient) - when sending messages there is no interaction with the API.

Then you missed another endpoint completely :)
/loq/conversation_post_messages is called twice after /bq/chat_typing and
is triggered by the button, to post a message.
(if you are talking about the latest version though, this might be
different, I haven't checked the latest version yet)
But the endpoint /loq/conversation_post_messages should be working with
this repo, and an updated user-agent.
2015-02-26 18:01 GMT+01:00 Robert<EMAIL_ADDRESS>
Yeah I understand that, I'm just saying that chat_typing is the only
endpoint I've seen so far when interacting with the chat. It's used to give
a heads up notification that someone is typing (to the recipient) - when
sending messages there is no interaction with the API.
—
Reply to this email directly or view it on GitHub
https://github.com/agermanidis/SnapchatBot/issues/6#issuecomment-76217951
.

Ahhh, I believe I am using the latest version. I haven't seen conversation_post_messages at all and I've thoroughly tested it.
I do also agree that this library needs a bit of updating (implementation of /loq/ and it shouldn't require pysnap, it should directly implement it (with the appropriate credits in README.md).

I could see them moving to XMPP or a similar protocol (either over raw TCP or over WebSockets) for chat as it's more efficient than the constant polling of an HTTP endpoint.

@agermanidis you might be correct. I've captured some TCP packets and it seems that the application is communicating via TCP. However, this could or could not be the chat conversations - it might be something else.
Here's an extract:
CONNECT feelinsonice-hrd.appspot.com:443 HTTP/1.1
Host: feelinsonice-hrd.appspot.com
Proxy-Connection: Keep-Alive
User-Agent: Snapchat/<IP_ADDRESS> (Nexus 5; Android 5.0.1#1602158#21; gzip)

HTTP/1.1 200 Connection Established
FiddlerGateway: Direct
StartTime: 18:43:17.838
Connection: close

@krissrex, I'm just starting to implement this library, can you explain a bit what you mean by setting the timeout from 5-90 randomly? Is it per request? or overall, adding a delay?

@ObjectiveTruth I may be wrong however how I interpreted his edit is that he changed the default timeout to 90 for his bot, instead of 5, which allowed his bot to not get blocked.
I am interested in contributing to more API research and also the development of the updated library.

@Sh1eld, I just noticed its a constant and someone seems to have made the edit already by setting it by default to 15. Thanks for the quick reply!

@ObjectiveTruth You can either change the default for yourself or you can do bot.listen(timeout=90) (I think) as shown in the reflectorbot example.

Big up to you guys !

The timeout just delays how often the bot runs a check (get snaps and monitor friend activity).
If you guys are interested in working on a library leave your Reddit or Skype usernames.

I'm not a python expert, but looking over the code, looks like there needs to be more entropy to the calls.
The instant fire back is a pretty dead give away of a bot, same with the sleep(timeout) regularity is pretty suspicious too.
My 2 cents..

@Rob-- @ObjectiveTruth We should add a more natural way of getting snaps. Maybe even try to simulate the function of the apps.
I think more lag would be me stealth.

The new Snapchat implements a couple of new API endpoints that need to be called, this library doesn't implement them. If these new endpoints aren't called then Snapchat probably gets suspicious.

+1 for new API end points support

I've pretty much finished implementation of the the loq endpoints - I'll look into the newer ones. I've also added the ability to register an account.
The library isn't entirely finished but it should do the job, https://github.com/Rob--/SnapWrap

Would you fuse it with the snapchatBot lib ?

check out mgp25's Snap-Api on here

also, https://github.com/JorgenPhi/php-snapchat/wiki/API-v2-Research


Indentation being destroyed by moving multiple lines of code via Alt+Arrow Keys

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T17:16:27.618Z)
OS Version: Linux x64 4.2.0-42-generic
Extensions:

Extension
Author (truncated)
Version

html-css-class-completion
Zig
1.8.1

togglehs
bbe
0.1.2

tortoise-svn-for-vscode
cds
0.1.1

prototxt
dmi
0.0.6

jupyter
don
1.1.3

python
don
0.7.0

vscode-html-css
ecm
0.1.8

vscode-great-icons
emm
2.1.15

code-python-isort
fre
0.0.3

cppcheck
mat
0.2.1

vscode-clang
mit
0.2.2

cpptools
ms-
0.12.3

java
red
0.8.0

vscode-icons
rob
7.12.0

cmake
twx
0.0.17

cmake-tools
vec
0.10.2

debug
web
0.21.2

clang-format
xav
1.5.0

(1 theme extensions excluded)

Steps to Reproduce:

Select multiple lines of code
Move them using Alt+Arrow keys
Every line of code that will be crossed by the moving lines will change its indentation.

Reproduces without extensions: Yes

Do you mind sharing the code snippet to help better troubleshoot?


feat: add automatic storage_reserved calculation for block type disks

Which issue(s) this PR fixes:

Issue # https://github.com/longhorn/longhorn/issues/9871 https://github.com/longhorn/longhorn/discussions/9797
What this PR does / why we need it:
This PR addresses a bug where the Storage Reserved value is incorrectly set to 0 during automatic disk registration when using Engine v2 block type.
Special notes for your reviewer:
To retrieve the total capacity of the block device, the implementation uses unix.Syscall.
If there is a more efficient or reliable approach to gather block device capacity, please provide your suggestions or feedback.
Additional documentation or context

To retrieve the total capacity of the block device, the implementation uses unix.Syscall.
If there is a more efficient or reliable approach to gather block device capacity, please provide your suggestions or feedback.

With v2, you should have these information in the bdevMap, like for example here: https://github.com/longhorn/longhorn-spdk-engine/blob/main/pkg/spdk/engine.go#L662

To retrieve the total capacity of the block device, the implementation uses unix.Syscall.
If there is a more efficient or reliable approach to gather block device capacity, please provide your suggestions or feedback.

With v2, you should have these information in the bdevMap, like for example here: https://github.com/longhorn/longhorn-spdk-engine/blob/main/pkg/spdk/engine.go#L662

Thank you for the review.
This code runs during the initial disk registration phase, before the lvstore is configured. Therefore, I didn’t use the getDiskStat function and focused on simply collecting the disk capacity at this stage.
Let me know if further adjustments are needed.

Hello @jangseon-ryu is the PR ready?

Hello @jangseon-ryu is the PR ready?

Yes, it has been completed. https://github.com/longhorn/longhorn-manager/pull/3294/commits/e820258837641855ca479cefbcad3c0a07930253

@jangseon-ryu Thanks for your contribution! Great work!


SPR-52 Deployment Workflows

Description
This PR includes the following proposed change(s):

Added deployment script that builds, tags, pushes, deploys, etc.
Added deployment script
Added deploymentconfig file for API

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change
[ ] Dependencies added/removed

Aspect(s) of Project Affected

[ ] Frontend
[ ] API
[ ] Database
[x] Workflows
[ ] Documentation

Best Practices Checklist

[x] I have performed a self-review of my own code
[x] My code follows the Airbnb React Style Guidelines
[ ] Documentation has been updated to reflect my changes
[ ] New and existing tests pass locally with my changes

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Testing in this PR...

Going to change the triggers to allow testing here, please don't merge yet.


⚠ SceneTime has degraded performance

In 6606509, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1835 ms

Resolved: SceneTime performance has improved in 50b77d2 after 10 minutes.


restart-required: command fails after an exporter.yml change

This command fails only with the first element of the list (exporter in our case)
Test Scenario:

add a comment to vars/exporter/exporter.yml
commit change
execute tdp restart-required --dry

Execution Error:
2022-10-04 15:38:50,591 - DEBUG - tdp.<module> - Logger initialized
Traceback (most recent call last):
  File "/home/diode-ben-aissa01/dev/tdp-lib/tdp/core/runner/operation_runner.py", line 30, in __next__
    operation_log = next(self._operations)
StopIteration

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/diode-ben-aissa01/dev/tdp-lib/venv/bin/tdp", line 5, in <module>
    tdp()
  File "/home/diode-ben-aissa01/dev/tdp-lib/venv/lib/python3.6/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/home/diode-ben-aissa01/dev/tdp-lib/venv/lib/python3.6/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/home/diode-ben-aissa01/dev/tdp-lib/venv/lib/python3.6/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/diode-ben-aissa01/dev/tdp-lib/venv/lib/python3.6/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/diode-ben-aissa01/dev/tdp-lib/venv/lib/python3.6/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/home/diode-ben-aissa01/dev/tdp-lib/tdp/cli/commands/restart_required.py", line 103, in restart_required
    for operation in operation_iterator:
  File "/home/diode-ben-aissa01/dev/tdp-lib/tdp/core/runner/operation_runner.py", line 34, in __next__
    self.fill_deployment_log_at_end()
  File "/home/diode-ben-aissa01/dev/tdp-lib/tdp/core/runner/operation_runner.py", line 43, in fill_deployment_log_at_end
    state if state else self.deployment_log.operations[-1].state
IndexError: list index out of range

https://github.com/TOSIT-IO/tdp-lib/pull/221 should have fixed this issue, are you up to date ?

#221 should have fixed this issue, are you up to date ?

yes

Ok, this is actually an edge case, where all actions returned by restart-required are noops. I will upload a fix


Remove the encodeCache.

Fixes https://github.com/tetratelabs/wabin/issues/9.

Thanks @cube2222!

No problem, thanks for accepting the PR!


fix: Remove overriding executionDeadline with maxDurationDeadline. Fixes #13044

Fixes #13044
Motivation
I've noticed that the maxDurationDeadline calculation process is unusual and this value is overriding executionDeadline.
For example, with the following workflow, it waits for 120 seconds and exits with error code 10:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: retry-sample-
spec:
  activeDeadlineSeconds: 3600
  entrypoint: retry-example
  templates:
    - name: retry-example
      retryStrategy:
        limit: 2
        backoff:
          duration: 10s
          factor: 2
          maxDuration: 3m
      container:
        image: alpine
        command: [sh, -c]
        args: ["sleep 120; exit 10"]

But, when the first attempt fails, the second node's container has a deadline within 1 minute, and it is killed by its wait container.
The latest Argo workflow follows this timeline (not exact times, for easier understanding):
0s: firstChildNode started (deadline = 3600s)
120s: firstChildNode finished and exited with an error
  (deadline = firstChildNode.StartedAt + backoff.maxDuration = 180s)
130s: Waited for 10 seconds(backoff duration) & secondChildNode started (deadline: 50 seconds left)
180s: Deadline exceeded, wait container kills main container

Modifications

The maxDurationDeadline was calculated by adding maxDuration to firstChildNode's startTime.

Changed to calculate maxDurationDeadline with lastChildNode's finishedTime.

executionDeadline was overridden by maxDurationDeadline.

Removed this override.

Verifications
I've tested with the above workflow, and it is working as expected.

I'll close this PR (updating documentation about maxDuration will be enough.)

Superseded by #13068 docs PR


Shape mismatch in MJX's put_data function

Intro
Hi!
I am a masters student at Vrije Universiteit Amsterdam, I use MuJoCo for my research on Modular Robotics.
My setup
Currently, Mujoco - trying to convert existing Evolutionary Computing for modular robotics framework Revolve2 (https://github.com/ci-group/revolve2) from Mujoco to MJX 3.2.3
What's happening? What did you expect?
I am following the provided tutorial on https://colab.research.google.com/github/google-deepmind/mujoco/blob/main/mjx/tutorial.ipynb#scrollTo=Jtz7j1PDOnw5 and convert currently working code running the mujoco.mj_step to run using MJX.
I get an error related efc_J:
Steps for reproduction

checkout https://github.com/rustam-e/revolve2/tree/develop
dependency installation - https://ci-group.github.io/revolve2/installation/index.html
run the experiment with python examples/4_example_experiment_setups/4c_robot_bodybrain_ea-mjx/main.py

the only differences from a functional examples/4_example_experiment_setups/4c_robot_bodybrain_ea/main.py can be found in https://github.com/rustam-e/revolve2/commit/30bfc7fda58704366196c3a80be56341de6ce8cf
Minimal model for reproduction
the issue is consistent with all of the experiements we ran
Code required for reproduction
running any of the examples by changing
from:
from revolve2.simulators.mujoco_simulator import LocalSimulator
to:
from revolve2.simulators.mjx_simulator import LocalSimulator
would throw consistently the same error like:
ValueError: could not broadcast input array from shape (0,10) into shape (4,10)
the only diference across examples being the second dimension
Confirmations

[X] I searched the latest documentation thoroughly before posting.
[X] I searched previous Issues and Discussions, I am certain this has not been raised before.

The issue seems to be due to the fact the the Mujoco did not instantiate efc_J property of the model, but running the mj_step once before the put_data call seems to have resolved the issue

It's good that you found the solution, but put_data should either handle this situation (being called before any arena allocations) gracefully or, if that's not possible, give the user an informative error.
That, I think, is a bug. WDYT @erikfrey ?


Handling height

I found using react-socks quite straightforward, however, the single biggest thing which is holding it back is that, as far as I can tell, it does not handle height as well.
Example:
Mobile breakpoint works like a charm but as soon as I put my phone in landscape, Tablet breakpoint gets triggered. I can get by with some CSS hacks.
In time I'd like to contribute and maybe implement height handling as well.
Cheers.

@admapop Thanks for the report and I am glad you like the lib and want to contribute. We have a few open issues, feel free to go through them and see if you could pick something up.
Now, to answer the issue, I think landscape mode is supposed to trigger a bigger breakpoint as breakpoints are purely based on width. I quite don't understand what you mean by height handling. Do you want to elaborate that a bit?
AFAIK, this is how even css breakpoints behave. Pl CMIIW.

@flexdinesh Sorry I've only noticed this now.
Height handling in the sense that nowadays you can have a phone that when in landscape the width is equal to that of an iPad in Portrait and what ends up happening is that the tablet breakpoint gets triggered when in fact it should still be the mobile layout.

It's a compelling case. Breakpoints were meant to be width driven and not device driven but having a height driven breakpoint would benefit a lot of use cases.
I think we can try and figure out a way to add height driven breakpoints and add it as an option rather than restricting based on portrait/landscape mode.
What ya'll think?

@rehman-00001 I'll have a play around but I think that will do it!

Honestly I'd say that the preferred solution would be to mimic how CSS handles such cases.
CSS allows you to setup media queries with:

orientation: landscape/portrait (https://developer.mozilla.org/en-US/docs/Web/CSS/@media/orientation)
aspect-ratio: min-aspect-ratio: x/y (https://developer.mozilla.org/en-US/docs/Web/CSS/@media/aspect-ratio)

Another solution when you need more control over what the user might actually see is to create media queries that gives you information about orientation, resolution and aspect ratio. Previously this has been combined with using CSS media queries to set body:before with content that indicates whatever breakpoint we should look for. An (quite old) example is here: https://gist.github.com/prbaron/6460f566673ff0282494
And adding those CSS breakoint-declarations to above the fold inline CSS should be usable with great control of more than simple widhts.
This also allows for using orientation and aspect ratio instead if widths and heights, that might end up being a lot of devices.


It is possible to make a "skill tree".

Hi, is it possible to right click and drag an image. Make a kind of "skill tree" and make lines that connect each image. The idea is to click on some images and it will release it, and a line will connect to it.

You can get the mouse state using the procedures, and then check for dragging with global variables.


RTSP Source Adapter doesn't support streams with b-frames

Investigate the problem. Fix if possible.

This can be fixed by an element that stores buffers in a queue and them sets them DTS calculated from PTS.

@tomskikh please fix it.


Externalize "/" commands

It would be excellent if instead of having a big switch in the middle of app.js we could have a commands.js file that contains all the commands in their own objects.  Command objects should have the function they will run, a description of what they do (to be later used  in /help, and preferably a list of built-in aliases for each command.

It doesn't cover everything here, but we can at least get all of this out of app.js

It seems good enough for now.  I'll make /help a separate issue.


Integration with Wordpress REST API

Hello,
I've tried to find a way to integrate this template with Wordpress and the best I've found is https://github.com/EvanAgee/vuejs-wordpress-theme-starter
However that starter theme is (slightly?) different from your template I mean there is a webpack.mix.js from Laravel, a /src/bootstrap.js and /src/firebase.config.js and so on - things nothing to do with this template.
I've tried to config manually a fresh webpack template via http://vuejs-templates.github.io/webpack/backend.html but it seems the doc is out of sync with the code.
The doc says config/index.js should look like:
...
dev: {
    port: 8080,
    proxyTable: {}
  }

but I have:
...

    // Paths
    assetsSubDirectory: 'static',
    assetsPublicPath: '/',
    proxyTable: {},

    // Various Dev Server settings
    host: 'localhost', // can be overwritten by process.env.HOST
    port: 8080, // can be overwritten by process.env.HOST, if port is in use, a free one will be determined
    autoOpenBrowser: false,
    errorOverlay: true,
    notifyOnErrors: true,
    poll: false, // https://webpack.js.org/configuration/dev-server/#devserver-watchoptions-

Basically I need two things: a dev environment which should work on http://localhost/sitename and the code deployed live to an URL like http://example.com
Thank you very much

The config file still contains all of the options mentioned, it just received a few additional ones in recent updates. Yes, we have to add those to the docs, but the additional options don'T keep you from following the advice on that page.
I will leave this open as a question for a few days, but you will probably be better of asking people on forum.vuejs.org or chat.vuejs.org

Ok thanks, I've did it but can't found a usable answer. I'll wait then for the doc update.

I'll wait then for the doc update.

Just to be clear: That update won't help you much with your task - the new options I added and will document are just convenience stuff for how the development server works. All of them are totally irrelevant with regards to integrating with another backend like WP REST.

Ok I see. Then I'll find a way on my own to make the integration.


document run command opts and args in README

I think it's supposed to be something like this.
kubeztl -zConfig ./ziti-id.json -service "kube apiserver service" get pods

Reminder for issue resolver to check the run command usage in https://github.com/openziti/ziti-doc/blob/main/docusaurus/blog/zitification/kubernetes/index.md after it's documented

https://github.com/openziti-test-kitchen/kubeztl#the-big-finale---zitified-kubectl


IllegalAccessError when I add the library

Hi,
First, thanks for all, I love your library and it's working good with a new project :)
When I add the library in an old project, after the compilation I have this error :

FATAL EXCEPTION: IntentService[GCM] Process: my.app, PID: 7667 java.lang.IllegalAccessError: Method 'void android.support.v4.content.ContextCompat.()' is inaccessible to class 'com.google.android.gms.iid.zzd' (declaration of 'com.google.android.gms.iid.zzd' appears in /data/app/my.app-1/base.apk:classes2.dex) at com.google.android.gms.iid.zzd.zzdL(Unknown Source) at com.google.android.gms.iid.zzd.(Unknown Source) at com.google.android.gms.iid.zzd.(Unknown Source) at com.google.android.gms.iid.InstanceID.zza(Unknown Source) at com.google.android.gms.iid.InstanceID.getInstance(Unknown Source) at gcm.RegistrationIntentService.onHandleIntent(RegistrationIntentService.java:54) at android.app.IntentService$ServiceHandler.handleMessage(IntentService.java:67) at android.os.Handler.dispatchMessage(Handler.java:102) at android.os.Looper.loop(Looper.java:154) at android.os.HandlerThread.run(HandlerThread.java:61)

This is the line of the problem :
InstanceID instanceID = InstanceID.getInstance(this);
I have look the reason of this error and it seem it's because I use a version too old of my play-services.
compile 'com.google.android.gms:play-services:8.3.0'
But it's working without adding the lib wisefy so how can I isolate this problem because I don't want to update my library ?

Oh!  Interesting...yes, there is a conflict that coming from compile 'com.android.support:appcompat-v7:25.3.1'  in WiseFy.  I don't think we actually need the support libraries for anything at this time, so I'm okay with removing it for now until it's actually in use.
I'll push an update, but an alternative would be to exclude the version of appcompat-v7 that WiseFy is bringing in.

Thanks a lot !
How can I exclude the version of appcompat-v7 that WiseFy is bringing in ?

This should do it:
compile ('com.isupatches:wisefy:1.0.9') {
      exclude module: 'appcompat-v7'
}

Thanks for reporting everything :)

Thanks it's working :)

Should be okay in 1.0.10 as well


fix(mantine): export TableContext so it can be used on tests

Proposed Changes

Exports TableContext so it can be used on tests on admin ui.

Potential Breaking Changes
No
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

What are you trying to achieve in your test? I don't think it is necessary to export the context since it is never used directly in the admin

I have a set of components (that renderes inside of the tbale) that uses the useTable hook so there would be a need to have the context to do proper unit tests on those components.

I would recommend that you test your full table instead of testing your components in isolation. It will be less dependent on implementation details and more stable. Feel free to DM me or the guild if you need guidance
If you haven't read this awesome blog post from Gaël I strongly recommend it. There's a section about mocking as little as possible


Replacing How To Code Data series with The HtDP book

Is it possible to replace the how to code data series on edx be replaced with the book HtDP , since most of the content gets blocked with the paywall.

I've opened an RFC on the How to Code classes. Your input would be valuable! Comment here: https://github.com/ossu/computer-science/issues/588


docs: update how to use swc plugin in rspack rule

Summary
update how to use swc plugin in rspack rule. experimental should be used in jsc

Checklist

[ ] Tests updated (or not required).
[x] Documentation updated (or not required).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

LGTM, thanks👍🏻


Dethrone Erik

The last matching pattern in the CODEOWNERS file takes precedence and is currently blocking #936

bors merge


TESTING: Items for discussion at GOC LA meeting

Add items here: http://wiki.geneontology.org/index.php/2016_Los_Angeles_GOC_Meeting_Agenda

Only for testing. Please dismiss.


Edit Jasmine grunt rule to include code coverage.

Edited Rule.jasmineSpec so that it measures code coverage by default using grunt-template-jasmine-istanbul.
The updated rule maintains the same directory layout assumption as before*, but now users can also pass in a list of source files instead if they'd like.
*e.g. the dir layout assumption for Rule.jasmineSpec 'example_module' would be:

code to be tested is all of build/example_module/**/*.js, excluding build/example_module/**/*spec.js
tests are build/example_module/**/*spec.js
Now the code coverage would be summarized at build/coverage/example_module/index.html
Allowing users to specify source files will let them customize the files needed to run tests, e.g. mocks needed for testing.

ran grunt test

You'll also need to bump the package version, so uproxy-networking and uproxy can depend on this change.

This was merged into https://github.com/uProxy/uproxy-lib/pull/118
Closing.


Added abstract and updated description

jason, we dont need abstract in the template description.

:+1:

CircleCI is failing due to OverLimit error.


[WIP] Sponge API 7 support

This port will currently only work with SpongeForge 1.12.2 using API 7. Port still needs some work but works for what I need in GriefDefender. I also had to swap from kotlin to groovy for the sponge folder as I'm more familiar with it.

While working on v4 #113, I have made supporting multiple platforms a lot easier, so if you're up to adding sponge support to it then feel free to. ;p


Deployment

Deployment
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
GIVEN a prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can access my website from any browser


Kafka's SkimpyOffsetMap takes up much of the heap space

68% of the heap seems to be taken up by the byte buffer allocated for SkimpyOffsetMap. The content seems to be all zero.

The allocation seems to be here https://github.com/apache/kafka/blob/02ebfc6108c86d8f7c74396f440ae5336e0bcae2/core/src/main/scala/kafka/log/LogCleaner.scala#L353

Most of the other byte buffers are for MemoryRecords, which is expected


merge existing config when committing

Fixes #1141 by merging parent config with any -run supplied config when committing containers to images. If no -run config is given, the committed image inherits the parent's entire config. Any keys given with -run will overwrite the corresponding keys' values from the parent.

For future reference: this was deprecated and removed from the documentation less than a month later: https://github.com/docker/docker/commit/168f8aba74d9c2996acec6fe1b93a2301523e305

@mhsmith Not exactly. The commit --run option was deprecated, so that no longer has any impact on this feature. But the more important use case always was and still is the inheritance of the source container's config into the image resulting from the commit.
That did not happen prior to this change.


fix(security): Remove ENABLE_REGISTRY_ACL in compose

Remove use of feature flag ENABLE_REGISTRY_ACL in compose as it is no longer used

Closes: #89
Signed-off-by: Jim Wang<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/master/.github/CONTRIBUTING.md.
What is the current behavior?

There is left-over flag ENABLE_REGISTRY_ACL in compose and should be removed as edgex-go PR is going to remove it.
Issue Number: #89
What is the new behavior?
No flag ENABLE_REGISTRY_ACL used any more.
Does this PR introduce a breaking change?

[ ] Yes
[x ] No

New Imports
Are there any new imports or modules? If so, what are they used for and why?
Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information
This PR should merge together with edgex-go PR https://github.com/edgexfoundry/edgex-go/pull/3461

recheck

recheck

recheck


Add Linux aarch64 and ppc64le builds

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This should be good to go. It looks like the failing build had a timeout issues unrelated to this package. If you re-run the failing tests it should be just fine.


Action on tool delete

@bgruening @hmenager @joncison @matuskalas @piotrgithub1
Hi all,
This issue is regarding tool deletion from the ecosystem, especially pull requests coming from outside (e.g. from bio.tools or from forks).
It could be the case that in our content file structure we have something like:
data/tool_id/tool_id.biotools.json where this is the only file. If we delete this file then the whole folder goes as well, if this is intended behaviour it's fine but we just have to agree on what happens on delete.
I was thinking of, istead of deleting the file immediately, especially when changes are coming from bio.tools perhaps we can mark tools as deleted by renaming the bio.tools files to something like .deleted.tool_id.biotools.json. In this way the folder doesn't go away.
I think we can code and handle both cases, it's just that we have to agree on what we want the functionality to be. What are your opinions on this? Please tag other people.

I agree with @piotrgithub1 suggestion that we rename the entry rather than delete it, but keep the .deleted part at the end, i.e. tool_id.biotools.json.deleted
In this way the file won't have a .json extension, but a .deleted extension which will separate it from regular .json files.


query required

hi Kim
Could you do a query for
present during, absent during in the same modificationannotation
for
https://github.com/pombase/curation/issues/2489
cheers

ignore, did with grep


Allow validators to keep state during validation

From https://github.com/pydantic/pydantic-core/pull/430#issue-1619909885
Currently we do a first pass in strict mode for unions and then a pass in lax mode. If we moved forward with something like #430 we'd be adding a third pass.
To avoid the cost of re-doing work we already did, we should let validators pass some sort of "state" or "token" up to their caller. The caller can then call back into them with that thing and they can resume from where they left off.
For example, a list validator might know that that it made it up to the 100th item in strict mode, so it can just start off from the 100th item in lax mode.
A string validator might know that strict mode and lax mode are the same as far as it's concerned so if the thing it's validating failed the first pass in strict mode it doesn't need to try again in lax mode, it can just fail fast.
One way to approach this would be to have:
trait Valdiator<ValStateTokenType> {
  type ValStateToken;
  fn validate(..., state: Option<ValStateTokenType>) -> (Option<ValStateTokenType>, ValResult<...>);
}

And then validators that don't use this can be:
impl Validator for SomeStatelessValidator {
    type ValStateToken = ();
    ...
};

Or:
struct ValidatedIndex { idx: usize, strict: bool };

struct ListValidator {};

impl Validato for ListValidator {
    type ValStateToken = ValidatedIndex;
    ...
};

Unfortunately this won't be as simple as adding a generic. Since we're using a big old enum with enum_dispatch the trait can't have any generics or associated types. I think the only thing we can do here is add another level of indirection:
BuildValidator.build() -> Validator.prepare() -> StatefulValidator.validate()

or
BuildValidator.build() -> Validator.copy() -> Validator.validate()

Validators that don't need any state could just return Self from prepare()/copy()


Support highlighting of lambda params?

Support highlighting of this:
[1, 2, 3].map { |a| a + 1 }

The a in |a| should be highlighted as a parameter.

It depends of theme, my extension already have parameter support.
Currently I'm using
https://marketplace.visualstudio.com/items?itemName=faustinoaq.theme-olivine-comments
to
get good crystal highlighting.
[image: Inline image 1]
Notice the blue a in |a|
On Sat, Jul 29, 2017 at 12:36 AM, Hanyuan Li<EMAIL_ADDRESS>wrote:

Maybe support highlighting of this:
[1, 2, 3].map { |a| a + 1 }
The a in |a| should be highlighted as a parameter.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/faustinoaq/vscode-crystal-lang/issues/8, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AC7Nx6Fee4hIpr1BgwZmLnRY0_hzIEQgks5sSsTcgaJpZM4OnQ5X
.

Notice the blue a in |a|

Shouldn't it be highlighted in the same way as a parameter, though?

Unless I change this line to Ruby, VSCode themes does't recognize the crystal delimiter.
"begin": "(?<=\\{|do|\\{\\s|do\\s)(\\|)",
"captures": {
  "1": {
    "name": "punctuation.separator.variable.crystal"
  }
},

@Qwerp-Derp I found the issue the Crystal syntax .tmLanguage don't have any rules to parameters with types. As far I can I changed the endCapture to get all parameters using:
"end": "\\)\\s*$|\\)\\s*:",

Also I found a lot of dirty code when I converted .tmLanguage to .json
I'll do a PR fixing this issue.
Internally related to #7

Fixed on #9


Remove incorrect Message bound on writeAvros

R: @jkff
CC: @nevillelyh

Thanks. This requires a little fixup in FormatPayloadUsingCoder - I'll do that myself and merge.

@jkff Thanks! Just noticed that as well. Cheers.


o365 ADFS phishlet TLS error

Hey!
I'm trying to create a lure using o365 phishlet with uncommented lines for ADFS, but when the initial o365 tries to redirect to adfs site I get the following error:
WARN: Cannot read TLS response from mitm'd server local error: tls: no renegotiation
and a 404 in the browser.. any idea what could be the issue?

Hello. Yes, we can help you. Contact us at<EMAIL_ADDRESS>

O365 ADFS fixed thanks to sir gretzky @Mrgretzky on icq


Primitives discussion

I'd like to get some thoughts from those who might be interested about the 'primitives' that should be supported out of the box - these are the data types that every game should have. For example, MUSH/MUX use 'Player', 'Thing', 'Room', and 'Exit' - with players and things being pretty damn similar. MUDs have items, NPCs, players, rooms, etc. Some MUD drivers also have 'blueprint items' which are interacted with slightly differently than regular items, and so on.
Obviously, as a fresh engine, we have the power to completely rethink this. I don't want to put more datatypes than we need to in CarbonMU core since part of the focus of this project is to build something modular - something like 'weapon' is too game-style-specific IMO - but I've also been building exclusively MUSH/MUX/MUSE games for 20 years, so I could use some input on what other core primitives we should support.
It feels to me like the traditional four (Player/Thing/Room/Exit) are a lock, but I'm open to disagreement, and also any new ideas. Just comment.

Thanks to everyone who weighed in on this discussion. I'm back working on advancing CarbonMU again and am turning my attentions to data modeling.
After quite a bit of thought, I've come up with a couple decisions, some of which contradict previous information and some of which align well. Consider these an open RFC if you're interested in weighing in, but I'm feeling fairly good about these so far.

I'm abandoning my previous goal to have a pluggable database back-end, and going to standardize on MongoDB. Yes, yes, I know, but it makes a lot more sense to me to be able to leverage all of the features of a proper database system without having to re-create them, like querying and indexing and so on. I'm not wild about having to introduce an external dependency, but I did want to always support MongoDB as the "robust option".
If you're wondering why Mongo and not something else, the main reasons are: a) I have prior experience modeling MUish data in both schemaed SQL and MongoDB, and have found a schemaless structure to be ideal, b) I've successfully used MongoDB for MU* data and had a pleasant experience, c)  installing and administrating MongoDB is quite straightforward.
Conceptually, there will be two large categories of data: Tangible things and non.
Tangible things will be things that are represented in a game-world room, things players can see in rooms and interact with using 'core' commands - rooms, players, things and exits will be the primary classes here, although I'm sure there will be others, or more properly, behaviors that can be attached to them. They will share a certain amount of functionality, so still looking into the best way to implement that.
Non-tangible things are things that are not in rooms, like channels, bboards, etc. There won't be much if any common functionality between independent classes, so the game's data model won't impose much if anything here.

:thumbsup: for picking one thing to standardize on.  Don't try to be All The Things.

OK, the basic primitive classes along with their relations to each other were implemented in 048a10f271dd4b0ce28d87e21e28b3579d64ccc4, so closing this discussion issue out.


Virustotal flags TaskbarX as 1/61 as malicious

https://www.virustotal.com/gui/file/ab192d20ed1ff6a88fef856b1a0dcf997e5022437c09a686c28356b43dc5a3e2/detection
could this be a false positiv from the vendor or real?

I got the same messages
https://www.virustotal.com/gui/file/ab192d20ed1ff6a88fef856b1a0dcf997e5022437c09a686c28356b43dc5a3e2
https://www.filescan.io/uploads/65a41b953c61cd5b42507132
https://www.virustotal.com/gui/file/d29ef3e35aaeeee67ff3757c7383e32e1618efe0c93c82d9205b52fba7a40f70


Integrated terminal input buffer-like echo on Ubuntu 22.04 arm64

Type: Bug
Integrated terminal display entered characters grouped by 6.
For example string 123456 is displayed when I input symbol "6".
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:23.865Z)
OS version: Linux arm64 4.9.337-33
Modes:

System Info

Item
Value

CPUs
Cortex-A53 (6 x 1908)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
2, 2, 2

Memory (System)
3.62GB (0.80GB free)

Process Argv
--unity-launch --crash-reporter-id 598067e8-a786-411f-8a85-7315b4a8b93f

Screen Reader
no

VM
0%

DESKTOP_SESSION
mate

XDG_CURRENT_DESKTOP
MATE

XDG_SESSION_DESKTOP
mate

XDG_SESSION_TYPE
x11

Extensions: none
A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263:30880072
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
89544117:30613380
2i9eh265:30646982
showlangstatbar:30737416
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
welcomedialog:30887143
pythonnosmt12:30797651
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30869155
dsvsc013:30795093
dsvsc014:30804076
dsvsc015:30845448
pythontestfixt:30871694
pythonregdiag2:30871582
pyreplss1:30886140
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
accentitlementsc:30887149
dsvsc016:30886110
dsvsc017:30886112
dsvsc018cf:30886115
aa_t_chat:30882232

That's how it looks
https://github.com/microsoft/vscode/assets/14954512/48189ce0-c4e4-463a-9d1e-2061df04ff9a

Thanks for creating this issue! To help narrow it down, please take the time to follow below steps. Thanks a lot and happy coding 🙏
Is the issue caused by an Extension?
Stop VS Code and from the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If it still reproduces, jump to the next chapter.
If you see it is an issue with the extension, please run the command Start Extension Bisect and follow the instructions to find the extension that is causing this issue.

Please report the issue to the extension causing this.
Is the issue caused by VS Code Core?
If the issue is not caused by an extension, it is maybe caused by your configuration. Try to run code --disable-extensions --user-data-dir <directory> where <directory> is an empty folder. This will ensure Code is starting with a fresh data directory, e.g. no specific settings and without any extensions running.
If it still reproduces, it was maybe fixed already in our nightly release. Please download from https://code.visualstudio.com/insiders/ and see if you can reproduce the issue.
Finally, to help us narrow down what change caused the issue, please run the following command to find the build. It will start previous releases of VS Code insiders to see where the regression was introduced:

npx --yes vscode-bisect@latest
follow the instructions until you found the offending build
report back the commit range in this issue

Thanks!
/label info-needed

Thanks for quick response!
Before to register the bug I have tested insider version and uninstalled all extensions so code --disable-extensions did not help.  The code --disable-extensions --user-data-dir <empty directory> is useless too.
I have no resources to install node.js so I did bisect manually and discovered that package code-insiders_1.81.0-1690979803_arm64.deb has not this bug but next code-insiders_1.82.0-1693988300_arm64.deb has.
I hope this info will help to fix the bug.
Kind regards

@rzhao271 any idea why this would be the case?

Try insider's or changing terminal.integrated.gpuAcceleration. Also does it occur if you change shells?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

Try insider's or changing terminal.integrated.gpuAcceleration. Also does it occur if you change shells?

@meganrogge terminal.integrated.gpuAcceleration value off or canvas solved the issue with my GPU software.
Thanks a lot!


Removing TO-DO.txt and README.txt

Summary:
Removing unused text files.
Intended Effect:
TO-DO.txt is online in our wiki here: https://github.com/stan-dev/stan/wiki/To-Do-List
I've updated the online one (minimally).
README.txt is out of date and I don't think it's necessary. If anyone thinks we need to have a version of this around, we can add it back in.
How to Verify:
See the diff in github.
Side Effects:
None.
Documentation:
Just removing out-of-date doc.
Reviewer Suggestions:
@bob-carpenter

Just FYI, there's a mention of README.md in the manual, but that file was left alone. I removed README.txt.

It seems odd not to have a README with the distribution, but it's OK by me.

This doesn't really need a merge build, does it?

We still have the README.md.
And... no, we don't.
On Tue, Mar 24, 2015 at 7:23 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

This doesn't really need a merge build, does it?
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1406#issuecomment-85738281.

The alternative is to include a README.txt. What do you want in it? The old
one hadn't been touched since v1.3.0.
On Tue, Mar 24, 2015 at 7:24 PM, Daniel Lee<EMAIL_ADDRESS>wrote:

We still have the README.md.
And... no, we don't.
On Tue, Mar 24, 2015 at 7:23 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

This doesn't really need a merge build, does it?
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1406#issuecomment-85738281.

Oh, right --- the .md actually goes out with the distribution.
I keep thinking of it as something that shows up on the GitHub
page only.
Presumably nobody's going to be confused as to what a .md file
is and it's human readable as text.
So unless others have comments, I suggest just merging.

Bob

On Mar 25, 2015, at 10:27 AM, Daniel Lee<EMAIL_ADDRESS>wrote:
The alternative is to include a README.txt. What do you want in it? The old
one hadn't been touched since v1.3.0.
On Tue, Mar 24, 2015 at 7:24 PM, Daniel Lee<EMAIL_ADDRESS>wrote:

We still have the README.md.
And... no, we don't.
On Tue, Mar 24, 2015 at 7:23 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

This doesn't really need a merge build, does it?
—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1406#issuecomment-85738281.

—
Reply to this email directly or view it on GitHub.


Update deprecated torch.qr in glow.py example

torch.qr is deprecated for a long time and is being removed by https://github.com/pytorch/pytorch/pull/70989.
This PR makes the example compatible with new and old PyTorch versions.

The test errors seem unrelated - "OSError: Could not connect to socket".


Add Doc struct to libXML

This will add access to some document specific attributes such as version, encoding, standalone, ids/refs, and charset to libXML.

Looks good, thanks. But unless we use those properties in some way I don't see a reason to add just this change.

No Problems,  Would it be beneficial to bundle this with some of the other functionality i was building out?  Such abilities as add_child, add_sibling, new_doc, new_node, etc..?

Sure, I guess added functionality should be similar to that of Nokogiri.

OK thanks, I'm familiar enough with Nokogiri. Go ahead and close this out and I'll see what I can do.

If you want we can keep this open and you can continue to push to this branch with --amend or with different commits if you provide different features.

Sounds Good.  It'll be a little while till I can revisit this, but I can add new feature commits if you don't mind keeping it open.


Can we create a table in a folder using CslCommandGenerator.GenerateTableCreateCommand

Hi I am trying to use CslCommandGenerator.GenerateTableCreateCommand
Here is my code:
var tableName = "ASMAlertFeedback";
           using (var kustoClient = KustoClientFactory.CreateCslAdminProvider(kcsb))
           {
               var command =
                   CslCommandGenerator.GenerateTableCreateCommand(
                       tableName,
                       new[]
                       {
               Tuple.Create("TimeStamp", "System.DateTime"),
               Tuple.Create("isAlertUseful", "System.String"),
               Tuple.Create("RoutingID", "System.String"),
               Tuple.Create("CorrelationId", "System.String"),
               Tuple.Create("ServiceOid", "System.String"),
               Tuple.Create("Namespace", "System.String"),
               Tuple.Create("SubscriptionID", "System.String"),
               Tuple.Create("ReportingIdentity","System.String"),
               Tuple.Create("CustomerFeedback","System.String"),
               Tuple.Create("IncidentID","System.String"),
                       });
               kustoClient.ExecuteControlCommand(KustoDbName, command);
           }

           using (var kustoClient = KustoClientFactory.CreateCslAdminProvider(kcsb))
           {
               DateTime TimeStamp = DateTime.UtcNow;

               try
               {
                   var command = string.Format(".ingest inline into table {0} [{1},{2},{3},{4},{5},{6},{7},{8},{9},{10}]", tableName, TimeStamp, isAlertUseful, routingID, CorrelationId, serviceOid, Namespace, subscriptionID, ReportingIdentity, Feedback, IncidentID);
                   kustoClient.ExecuteControlCommand(KustoDbName, command);
               }
               catch (Exception ex)
               {
                   logger.LogError($"Error = {ex.ToString()} , IncidentID={IncidentID} and Subscription={subscriptionID} and ReportingIdentity={ReportingIdentity}");
                   string error = @" Feedback service failure! Please contact<EMAIL_ADDRESS>                   return Content(error, "text/plain");

               }
           }

However, this will create the table in the database(it is correctly doing so)..But i have a folder under /Tables//TableName i.e. ASMProcessedDB/Tables/ASMAlertFeedback/AsmAlertFeedback
will this code be able to create in the specific folder if the table doesn't exist..if yes, how to?
Or if it does exist will it be able to find the table and update it if I establish the correct connection to the table? I believe the connection string will still stay the same.
Also, is there any logic to check if the table exists or not then only create the table?

Just to clarify, the folder attribute of a table is simply a hint to UI tools (like Kusto Explorer, and Kusto Web Explorer), suggesting them to display the table under a specific folder, for the user's convenience. There's no folder entity in Kusto.
CslCommandGenerator provides multiple methods to create tables, some of them receive a string folder attribute.