cache ASG InstanceTypes

Summary
Previously, Cluster Autoscaler cached Launch Configurations for ASGs but not Launch Templates; the impacts of this are that any ASG that uses a Launch Template and has 0 size would repeatedly hit the AWS API to get the details of the LT.  If you have enough empty ASGs this can cause API limits to hit.
I refactor the launchConfigurationInstanceTypeCache into a generic instanceTypeExpirationStore, which can hold data for either Launch Configurations or Launch Templates.  At the end of the day, this just maps the ASG id to the instance type associated with that ASG.
Testing done
Still do to
Update the unit tests and add new ones
Manual testing/evaluation in our clusters
First we tested behaviour on the test-mc-c cluster; no changes to scaling behaviour were observed.
Next we tested behaviour on dev-a so as to reduce the impact of making many AWS API calls.  We see that the number of API calls does go down, as desired (this graphs the number of DescribeLaunchTemplateVersion calls CA is making):

At around 13:20, 300 ASGs were added to the cluster.  Using the unpatched version of CA, we make about 3500 DescribeLaunchTemplateVersion calls; at about 14:45, the patched version of CA was inserted into the cluster, which reduced the number of API calls to between 50-100/minute, which is well below our limits.
Other API calls made by CA increase slightly during this time period, but not a concerning amount.  This first graph is DescribeAutoScalingGroups, which are around 25/minute:

The second graph is DescribeTags, which are around 1-300/minute, which is still well below our thresholds for these types of calls:

From a performance standpoint, this change also significantly improves CA performance (by about an order of magnitude):

During the time the new version of CA was running, I observed the cluster scale up and down, so scaling behaviour appears to still be working correctly:

Open questions/Things to check

Does this cause any regressions with Launch Configurations?  (I think as long as our unit tests pass here we'll be OK, but this is maybe something they'll ask about upstream)
Are we confident that if the LaunchTemplate version changes CA will pick it up?

Reviewers
@Ramyak @grampelberg @jiancheung

From a performance standpoint, this change also significantly improves CA performance (by about an order of magnitude):
What is this a graph of?


Aniwave JsonDecodingException

Steps to reproduce

Open aniwave
Click any anime
Episodes don't load and error message appears

Expected behavior
Episodes should load and be streamable
Actual behavior
While trying to load error message appears extension version 14.66
Crash logs
JsonDecodingException: Unexpected JSON token at
offset 23: Expected string literal but 'null' literal was found at path: $.result
Use 'coerceInputValues = true' in 'Json {}' builder to
coerce nulls to default values.
JSON input:
{"status":403,"result":null,"message":"Request is invalid.","messages":["Request is invalid."]}
Aniyomi version
<IP_ADDRESS>
Android version
Android 13
Device
Redmi note 10 pro
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with an official manga extension and this issue can be replicated in the Tachiyomi app, that I should be opening an issue in Tachiyomi's extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Same here. It has been at least a week or two

Wait this isn't the correct repo. Should be https://github.com/aniyomiorg/aniyomi-extensions


Standard Object Text Color Detection

This task involves building a function as part of a text_detection module that will run in the vision pipeline that is capable of processing an image containing at least one ODLC standard object, and detecting the COLOR of a single-character, 1-inch-wide, alphabetic lettering on the Standard Object(s).
The output required for this detection is a color matching one of WHITE, BLACK, GRAY, RED, BLUE, GREEN, YELLOW, PURPLE, BROWN, and ORANGE (as seen in the SUAS Enumeration of Standard Objects).


Correct way to combine multiple meshes?

Hi there,
I'm looking to use the ADDITION operation with multiple meshes. I see in your example: https://gkjohnson.github.io/three-bvh-csg/examples/bundle/simple.html this has the output I expect.
The example combines 2 meshes; a sphere and a cube.
What would be the correct process for combining say 5 meshes (let's name them A B C D E)?
Is there a way of combining all 5 in 1 in a single command?
Or do you have to do (((A+B) + C) + D) + E?
Or is ( (A+B) + (C+D) ) + E more efficient?
I'm just trying to understand the best way to approach this problem when the number of meshes grows to a few thousand.
Many thanks,
John

Hello! Performance is still in progress and there are still some implementation quirks affecting speed but that aside I have not benchmarked or done any notional computation complexity comparisons for the types of operational hierarchy differences your asking. My gut says there's value to keeping the operations hierarchy shallow rather than deep but I can't be sure.
If you'd like to contribute a benchmark (see benchmark folder) or some kind of computational breakdown to figure this out and contribute it back it would be good to have!


remove unwanted variable

Signed-off-by: slayer321<EMAIL_ADDRESS>related to bugbash
https://lift.sonatype.com/result/longhorn/longhorn-manager/01FHW2JKTWJ3Z4RA5MMT2A53WS?t=CustomTool "Staticcheck"|SA4006

@slayer321 are you willing to address this https://github.com/longhorn/longhorn-manager/pull/1092#discussion_r732400289?

yes, I have already done that. You can check my last commit

b.Requests[corev1.ResourceCPU]

But I still see the error did not check https://github.com/longhorn/longhorn-manager/blob/23234846129c26afedecd2fd45654af779286928/controller/controller_test.go#L312-L314

Closing for long inactivity. @slayer321 feel free to continue this PR later on. Thank you.


Keybind to cycle between different display modes

Currently the only way to change display modes is to open up the mod's settings menu and change it through there.
I think it would be useful to have a keybind to cycle between all of the different modes so, for example, I can bring up the overworld/nether view when I need it without having to pause the game.

At the moment, there is a client command to select the display mode, "/coordinates mode ", but I will definitely add this.

I'm trying to move the coordinates display to the top right corner instead of the left, but the /coordinates movehud command doesn't work and neither does the key bind.


UCSF Library: Marion Baldur Sulzberger papers [#28012]

Collection details

Collection Registry URL: https://registry.cdlib.org/admin/library_collection/collection/28012/change/
Endpoint: Nuxeo
Preview on -stage: https://calisphere-stage.cdlib.org/collections/28012/

Communications

Freshdesk thread URL

https://help.oac.cdlib.org/a/tickets/118590
https://help.oac.cdlib.org/a/tickets/119705

Request type
[Select all that apply]

[x] New collection harvest
[ ] Re-harvest whole collection (on request)
[ ] Re-harvest metadata only (on request)
[ ] Scheduled recurring harvests
[ ] Delete whole collection (remove from publication)
[ ] Item record removals (remove from publication)
[ ] OAC METS replacement
[ ] Redirects needed
[ ] Other

Notes:
Harvesting request details
[copy/paste from webform request]

Notified Lisa N. on 2024-08-12 (https://help.oac.cdlib.org/a/tickets/118590)

On Hold, per Lisa N.


Option to Hide Battery Icon from Menu Bar for a Cleaner Interface

Is your feature request related to a problem? Please describe.
I often find the menu bar on my device cluttered, especially with icons that I don't frequently use. The "battery" application icon is one such example. Its constant presence in the menu bar adds to the clutter without providing necessary information at a glance, as I don't often need to check my battery status.
Describe the solution you'd like
I would like an option in the "battery" application settings that allows users to hide its icon from the menu bar. This feature should be easily toggleable, enabling users who wish to monitor their battery status closely to keep the icon visible, while others can choose to remove it for a cleaner look.
Describe alternatives you've considered
An alternative could be an auto-hide feature where the icon only appears when the battery is low or in a critical state. Another possibility is integrating the battery status into another menu icon or notification center, reducing the need for a separate battery icon.
Additional context
This change aims to enhance user experience by reducing menu bar clutter, catering to users who prefer a minimalistic interface. The flexibility to show or hide the icon according to personal preference would be greatly appreciated.

Not to disregard this suggestion, but you can just "Quit" the app from the dropdown and that'll remove the menu bar icon. The battery limit is, however, still applied normally, since I think the app is just a GUI extension to a CLI backend.
However, any time you want to make a change, you would have to re-open the app.

You are right, thanks! @ArnavVarshney


feat: added examples in dice game step 14

Checklist:

[x ] I have read and followed the contribution guidelines.
[x ] I have read and followed the how to open a pull request guide.
[ x] My pull request targets the main branch of freeCodeCamp.
[ x] I have tested these changes either locally on my machine, or GitPod.

Closes #56674

Changes have been committed.


(QENG-4204) Add AlwaysBeScheduling hypervisor

This commit adds a new hypervisor called 'abs' that will support
generating configurations necessary for working with CI.next and the
AlwaysBeScheduling service.
Currently, the behavior of the abs hypervisor is basically the same as
the vmpooler hypervisor. The only difference is it doesn't fixup the
nodes to support old PE versions.
Having a built-in hypervisor for ABS alleviates the need for users to
manually override the hypervisor key on each node.

@puppetlabs/cinext for review

This should probably come with documentation explaining what "ABS" is, otherwise it looks pretty good to me.

@waynr We don't really say much about the hypervisors as it is, but I added mention of it in the README (with a link to the private repo) and some docstrings.


Create feature versioning mechanism to handle impact of elemental-operator changes in the UI

With the changes coming from https://github.com/rancher/elemental-operator/issues/691, a mechanism will need to be created to handle feature versioning in the UI dependent on the elemental-operator version.
Two immediate areas of impact are:

Raw disk images can only be built using the UI starting from elemental-operator v1.5.x
The machine registration defaults in latest UI RC are only supported for v1.5.x and onwards. Basically the snapshotter related defaults. On previous versions the defaults should not include the snapshotter field.

I just found this issue https://github.com/rancher/elemental-ui/issues/175

The issue mentioned above is fixed so all good to me.


flexget history crashes when piping to less or redirecting to a file

Expected behaviour:
print history when piping output to less or redirecting to a file
Actual behaviour:
crashes with:
Could not start manager: 'ascii' codec can't encode character u'\u2013' in position 47: ordinal not in range(128)
Steps to reproduce:
flexget history | less
or
flexget history > history.txt
Additional information:

Flexget Version: 2.0.38
Python Version: 2.7.11
Installation method: git
OS and version: OS X

Maybe try to upgrade.
I test on my setup with version 2.1.1 and don't face this issue.

I've been constantly upgrading to newest versions and am now on 2.1.1 and still getting the same error. flexget history | less no longer gives a stack trace, it just stops producing output, so if you scroll to end you see the history output just cuts short. But if you redirect the output you can still see the problem exists:
$ flexget history > /tmp/flex.out
Could not start manager: 'ascii' codec can't encode character u'\u2013' in position 47: ordinal not in range(128)

This has something to do with system encoding, It was asked before.
Note that you can use the API for this as well:
curl -X GET --header "Accept: application/json" "http://host:5050/api/history/?page=1&max=50" < /tmp/flex.out

and the encoding issue has been fixed in other commands in the past. the api only works if you run the daemon and many of us don't. i only run flexget via cron or the command line.

This works for me:
$ git diff flexget/logger.py
diff --git a/flexget/logger.py b/flexget/logger.py
index fc6524b..626c948 100644
--- a/flexget/logger.py
+++ b/flexget/logger.py
@@ -114,7 +114,8 @@ def console(text, *args, **kwargs):
    Accepts arguments like the `print` function does.
    """
    if not isinstance(text, str):
-        text = str(text).encode(io_encoding, 'replace')
+        text = str(text)
+    text = text.encode(io_encoding, 'replace')
    kwargs['file'] = getattr(local_context, 'output', sys.stdout)
    print(text, *args, **kwargs)
    kwargs['file'].flush()  # flush to make sure the output is printed right away

Cool, wanna push a pr?

why was this labeled "Pending information"? ok, i'll answer the question- no i don't wanna push a pr, cause i only have access via phone these days. i gave you a 2-line patch, how is that insufficient?

This actually has been fixed on next branch with dcce2b58a58fcec8996dded65a9225e0d428f35d
Will mark this as resolved when we merge that branch.

Fixed with https://github.com/Flexget/Flexget/commit/dcce2b58a58fcec8996dded65a9225e0d428f35d

@grr can you confirm it's fixed?

yes, mostly. it doesn't crash now, but the new ascii colorization causes visual artifacts when piping through less. it should do something like this instead:
if sys.stdout.isatty(): colorize()

That's a good idea, we'll see about adding it

This issue is happening again. Tried with flexget history | less and flexget series list all | less. #1407 is the same issue.


OutOfMemoryError since upgrading to version 0.5.0

We had some issues with OutOfMemoryErrors since upgrading to version 0.5.0.
We were running version 0.4.1 for the last 6 months without issues and upgraded to 0.5.0 with the same volume of events, same configuration thresholds and same machine resources.
After this upgrade we registered OOM errors and the application did not recover from them. We restarted the containers a few times but always ending up with the same result.
We fixed this by moving to larger instances types, but with the same volume of events as before this was not expected.
More info:
sinkLzoBufferByteThreshold=500000000
sinkLzoBufferRecordThreshold=2000000
sinkLzoBufferTimeThreshold=3600000

@JorgeAlmeida There are two things to take into consideration:

Before 0.5.0, OOMs were swallowed by the KCL and now they're bubbling so it's most likely that you had those before but they weren't crashing the application.
If you still have the logs for 0.4.1 I invite you to look for "Error writing raw stream to output stream" log messages which were indicating ooms.
amazon-kinesis-connectors, the library we're using to read from Kinesis, has no backpressure which means that if, for some reason, we accumulate delay sending events to s3, the buffer will keep on growing until an oom is reached.


tool_root_dir in the inference script

Thanks for your excellent jobs.
When I run the inference script inference_chatgpt_pipeline_virtual.sh  , I notice that there is no folder called toolenv/tools.
python toolbench/inference/qa_pipeline_multithread.py \
    --tool_root_dir toolenv/tools

Which folder should I set it to?

Thanks for your response. :-)


feat: update js-sdk to rc5

run npm audit fix
update to js-sdk rc5
now that we support the new namespaces, revent back to using the original example

Fixes #20

@mhmd-azeez bumped your permission level up! Sorry for the delay.

@chrisdickinson thanks!


Add support for parsing comments

This pull request adds parsing of single line and multiline comments. The comments are not returned in the AST but rather ignored. We may add support for returning comments in the future but that seems quite a bit more involved.
Related to https://github.com/rqlite/sql/issues/18

Ah, I missed this -- looks like a superset of #18 which I just merged?

I mean #17.

Ah, yeah, I didn't see that one. I can refactor to just add multiline.

I'm good, small conflict.

I pushed up the change. I ended up using my version so the scanner could still return the COMMENT token and then it's ignored in the parser.

Thanks @benbjohnson -- merged.


Typo in T2* Ramsey Characterization document

Summary
Typo fixed in T2* Ramsey Characterization document
fixes #1495

 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.

@Brijeshthummar02 Please comment after you have signed the CLA and we can get this merged.

@wshanks signed the CLA aswell as resolved as requested.

Thanks @Brijeshthummar02!


InvalidRegistration on iOs with GCM

Hi, I'm using 1.7.1 version of plugin, compiled at https://build.phonegap.com.
When I send notifications to iOs devices, I get InvalidRegistration error from GCM. I saw this issue as closed, but what should I do to make plugin work?
All details of code here: http://stackoverflow.com/questions/38164686/phonegap-push-notification-plugin-with-ios

I have the same issue :( i have the registration event but when i use the push on POSTMAN it said InvalidRegistration


Conditional branching for waiver on Join Us page

My contributions to this PR:

renamed component to JoinUsConditionalRoute

component checks "localStorage" to determine if waiver has been accepted
improved styling of component
fixed issue with Firebase logEvent working (this has not been tested, I'm not sure how to test this in local dev environment)

relocated Saturday map route to bottom of "Join Us" page (after waiver accepted)
adjusted ScrollToTop component to not use smooth scroll


Unable to access LocalLifecycleOwner

Describe the bug
java.lang.IllegalStateException: CompositionLocal LocalLifecycleOwner not present
	at androidx.lifecycle.compose.LocalLifecycleOwnerKt$LocalLifecycleOwner$1.invoke(LocalLifecycleOwner.kt:26)
	at androidx.lifecycle.compose.LocalLifecycleOwnerKt$LocalLifecycleOwner$1.invoke(LocalLifecycleOwner.kt:25)
	at kotlin.SynchronizedLazyImpl.getValue(LazyJVM.kt:74)
	at androidx.compose.runtime.LazyValueHolder.getCurrent(ValueHolders.kt:29)
	at androidx.compose.runtime.LazyValueHolder.getValue(ValueHolders.kt:31)
	at androidx.compose.runtime.CompositionLocalMapKt.read(CompositionLocalMap.kt:90)
	at androidx.compose.runtime.ComposerImpl.consume(Composer.kt:2135)
        .....

Affected platforms

iOS
Android

Versions

Kotlin version*: 2.0.0-RC2
Compose Multiplatform version*: 1.6.10-beta03
Device (model or simulator for iOS issues): Google Pixel 3A

Expected behavior

Lifecycle version*: 2.8.0-beta02

That version of the lifecycle library doesn't look valid.

You must use androidx.compose.ui.platform.LocalLifecycleOwner on Compose 1.6.
This was moved to another module (androidx.lifecycle.compose.LocalLifecycleOwner) recently in AOSP, but it won't work until Compose 1.7

I get this error, when trying to use LifecycleStartEffect from org.jetbrains.androidx.lifecycle:lifecycle-runtime-compose:2.8.0. Seems to me like the current release is simply broken.
I had to manually provide androidx.compose.ui.platform.LocalLifecycleOwner.current to LifecycleStartEffect to avoid the error
LifecycleStartEffect(Unit, lifecycleOwner = androidx.compose.ui.platform.LocalLifecycleOwner.current) {
    // ...
}

@kodebach if you're talking about Android, it's tracked on Google side here - https://issuetracker.google.com/issues/336842920

Thanks for the info. I was testing on Android, but assumed it would affect all platforms since LifecycleStartEffect is defined in common code.

No, only Android is affected - multiplatform 1.6.10 uses a new one under the hood

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Replaced rawgit.com urls with combinatronics.com urls.

Combinatronics.com is a drop in replacement for rawgit.com which is EOL.

Thanks, but I'd rather use jsdelivr.


Bump minor version to 1.4.0

Bump addressable from 2.7.0 to 2.8.0
Add Microsoft OAuth provider enum

Codecov Report

Merging #117 (0f3e97d) into master (cc03fff) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #117   +/-   ##
=======================================
  Coverage   98.26%   98.26%
=======================================
  Files          28       28
  Lines         576      576
=======================================
  Hits          566      566
  Misses         10       10

Continue to review full report at Codecov.

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


Andrs-Betancourt_CU-36ag55e

Redefines Root Endpoint as Check-Health. Returning a JSON object to see if the server is accepting requests.

Task linked: CU-36ag55e Create Root Endpoint to Check the Health of the Backend


Wrong businessDiff result on a specific case

const moment = require('moment-business-days')
const days = moment('2021-02-03T01:59:00-05:00').businessDiff(moment('2021-02-02T00:00:00-05:00'))
console.log(days)

Result is 2 instead of 1.
Is this a bug or have I misunderstood how businessDiff() works?

It was fixed in 052a4a7c1413e7418e42b4d931857abc3903c2b1 but it hasn't been released yet. I'll ping @kalmecak

Confirmed. It works on latest code.


Makes Dalvik Code in source and sinks readable

Sometimes, sink names and source names in issues can take up the form of
Dalvik Codes which are hard to read. Adds code to make use of the
existing Davlik to Human Readable converter used for readables apply to
sink and source names.
Signed-off-by: Abishek V Ashok<EMAIL_ADDRESS>Fixes: https://github.com/MLH-Fellowship/sapp/issues/6
Test plan:

Download the zip of a SAPP db containing the MT run to help you reproduce the issue and the source code of what MT was run on
Extract both zip files and run the following command to get started  looking into this in SAPP:

python3 -m sapp.cli server --source-directory repo/ --debug

After the frontend has also started in debug mode (npm run-script start), go to localhost:3000
See that sink and source names don't use Dalvik code anymore.

Before this change:

After this change:

Obsolete as per internal discussions


Ensure setter appears before getter

This PR ensures that all getter/setter pairs first declare the setter and then the getter. Otherwise, the TypeScript compiler tsc will output .d.ts files with the getter before the docs for the property. This is mainly for readability.
All getter/setter pairs have been moved up to directly follow the constructor.
It also properly documents the instance properties of all math classes, which in turn fixes many TS declaration types across the API.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

I'm a little concerned that longer term rules like this will become a pain/burden if we're not automatically enforcing them (setters before getters and always after constructor). And tbh I'm a little uneasy having documentation requirements dictate code structure/layout.

@slimbuck So we already enable this rule:
https://eslint.org/docs/rules/grouped-accessor-pairs
But I'm going to tighten is up by also enabling this option:
https://eslint.org/docs/rules/grouped-accessor-pairs#setbeforeget
So we can indeed enforce this.
As for placing getters/setters directly after the constructor, that's just a consistency thing I think we should agree on as a team. Now that we have embraced ES6 classes everywhere, our class layout should be part of our coding standards. When you're working across the codebase, it's quite jarring to sometimes find getters/setters and the top and sometimes and the bottom and sometimes dotted around a class.


bst_height.cpp : Add height of binary search tree algorithm #266

This adds  bst_height.cpp file  i.e  "Height of Binary Search Tree" in C++.
Closes #266

@orichamaru : Thanks for Contributing


Scared Squid Boss!

A Boss which leaves an aoe in response to a trigger such as too many heroes being nearby or as its HP falls below certain thresholds. No auto attack.
Phases:
90% HP: Leaves first aoe
60% HP: Second aoe
50% HP: Begins leaving aoe's in response to triggers (too many heroes being nearby) and gains armor and magic resist to prolong the interesting part of the fight. Would also require a visual indicatior (change in boss colour) so as to inform the player of the change in boss activity
30% HP: Third aoe
10% HP: Reduces vision of players in the arena down to a minimum and applies a miss chance debuff
or
10% HP: The boss leaves aoe's more rapidly and without trigger prerequisites
Maybe both?
aoe has to deal damage but other effects could be slows, miss chance, darkening of player screen, reduced turn rate.

First of all - thank you for your suggestion :)
I like the initial mechanic of this: Smoke screens and hard to catch , hence prolonging the boss_battle for a chance for the enemy to strike.
But I have some questions:
1)Would the boss move around a lot (aside from the AoE/trigger)? If yes , then how fast (slow/medium/fast)?
2)For which ~ tier do you think this boss fits? Or rather , how tanky should this boss be?
3)You said this boss has no auto-attack. But then what does this boss do for the 100%-->50% HP part? Just stand in place and take the hits?
4)At 10%: if you want to limit vision "to a minimum" , then what do you see as a "minimum"? (I know it's for the balance/coding teams to decide , but I'm curious of your opinion)
Also , does the boss still keep running away? Cause if the players vision is too small and they walk in a group , then the boss might get a bit too frustrating and time-consuming.
5)Have you thought of a boss_item for this boss yet? It's not a "must have" in a [boss mechanic suggestion] , but it could help us save some time (or create something better ,  out of inspiration)
Overall: I think the base mechanic is a good thing (as I said in the beginning), but , as you may know, one of our requirements is for the mechanics to be obvious  (so that the players quickly figure out how to beat the boss_fight) and interesting at the same time. In my opinion you made it interesting , but it might get the more complicated the more near the end of the boss_fight.

Thanks for the advice @VoidsKeeper . Here is my attempt to answer all your questions
1- In my head I imagined this boss as something you have to chase down and runs away from heroes at a fast pace but runs out of breath after running for a time, allowing heroes to catch up with him and deal damage.
2 - This boss is weak in concept and would require massive buffs to the damage and aoe of his aoe to make him a viable late game boss. I suggest it as an early boss, probably tier 1 perhaps tier 2.
3 - For some reason I added that as an edit after the initial creation of the boss. It doesn't really make sense without proposing another method of directly that fits with the boss being "scared". However an auto attack doesn't seem right. An ability, perhaps a swipe behind the boss, would be more in sync with the boss concept.
4 - Sorry for the vague wording there. The vision radius needs to be small so my thought was anywhere between 300 and 600 aoe depending on the size of the arena. 300 is very small so would probably be inappropriate in most cases so I was thinking 450 for a standard sized arena.
5 - An item that leaves an aoe similar to the boss' could be appropriate for any role any could be upgraded or combined with other miss chance items to enhance the effect. Maybe a Squid Ball 🦑 which can be combined with different items to make it suitable for different roles: Squid staff, Squid sword, Squid pants.
Now that i'm looking at this. I probs should have just edited the original ey.

Any updates on this one? I remember our talk about it!
Please update the post with corrections and reformat it according to #37 for clarity.


eo-core failed to parse plugin descriptor

While trying to compile this project, the following error occurs:
C:\javaworkspace\eo>mvn compile
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] eo-parent
[INFO] eo-compiler
[INFO] eo-maven-plugin
[INFO] eo-archetype
[INFO] eo-core
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building eo-parent 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-enforcer-plugin:1.4.1:enforce (enforce-maven) @ eo-parent ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create (jcabi-build-number) @ eo-parent ---
[INFO] ShortRevision tag detected. The value is '7'.
[INFO] Executing: cmd.exe /X /C "git rev-parse --verify --short=7 HEAD"
[INFO] Working directory: C:\javaworkspace\eo
[INFO] Storing buildNumber: 231d9f3 at timestamp: 2018-12-11T12:47:54
[INFO] Storing buildScmBranch: master
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create-timestamp (jcabi-build-number) @ eo-parent ---
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building eo-compiler 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-enforcer-plugin:1.4.1:enforce (enforce-maven) @ eo-compiler ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create (jcabi-build-number) @ eo-compiler ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create-timestamp (jcabi-build-number) @ eo-compiler ---
[INFO]
[INFO] --- antlr4-maven-plugin:4.5.3:antlr4 (jcabi-antlr4) @ eo-compiler ---
[INFO] No grammars to process
[INFO] ANTLR 4: Processing source directory C:\javaworkspace\eo\eo-compiler\src\main\antlr4
[INFO]
[INFO] --- maven-resources-plugin:3.0.1:resources (default-resources) @ eo-compiler ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO]
[INFO] --- maven-compiler-plugin:3.6.0:compile (default-compile) @ eo-compiler ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 40 source files to C:\javaworkspace\eo\eo-compiler\target\classes
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building eo-maven-plugin 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-enforcer-plugin:1.4.1:enforce (enforce-maven) @ eo-maven-plugin ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create (jcabi-build-number) @ eo-maven-plugin ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create-timestamp (jcabi-build-number) @ eo-maven-plugin ---
[INFO]
[INFO] --- maven-resources-plugin:3.0.1:resources (default-resources) @ eo-maven-plugin ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory C:\javaworkspace\eo\eo-maven-plugin\src\main\resources
[INFO]
[INFO] --- maven-compiler-plugin:3.6.0:compile (default-compile) @ eo-maven-plugin ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 2 source files to C:\javaworkspace\eo\eo-maven-plugin\target\classes
[INFO]
[INFO] --- maven-compiler-plugin:3.6.0:compile (compile-helpmojo) @ eo-maven-plugin ---
[INFO] Nothing to compile - all classes are up to date
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building eo-archetype 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- maven-enforcer-plugin:1.4.1:enforce (enforce-maven) @ eo-archetype ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create (jcabi-build-number) @ eo-archetype ---
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create-timestamp (jcabi-build-number) @ eo-archetype ---
[INFO]
[INFO] --- maven-resources-plugin:3.0.1:resources (default-resources) @ eo-archetype ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 5 resources
[INFO]
[INFO] --- maven-compiler-plugin:3.6.0:compile (default-compile) @ eo-archetype ---
[INFO] No sources to compile
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building eo-core 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] eo-parent .......................................... SUCCESS [  9.765 s]
[INFO] eo-compiler ........................................ SUCCESS [ 20.982 s]
[INFO] eo-maven-plugin .................................... SUCCESS [  9.501 s]
[INFO] eo-archetype ....................................... SUCCESS [  0.125 s]
[INFO] eo-core ............................................ FAILURE [  0.031 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 45.302 s
[INFO] Finished at: 2018-12-11T12:48:25
[INFO] Final Memory: 26M/628M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to parse plugin descriptor for org.eolang:eo-maven-plugin:1.0-SNAPSHOT (C:\javaworkspace\eo\eo-maven-plugin\target\classes): No plugin descriptor found at META-INF/maven/plugin.xml -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginDescriptorParsingException
C:\javaworkspace\eo>

@yegor256/z please, pay attention to this issue

@simonjenga/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256/z everybody who has role DEV is banned at #153; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@simonjenga this is not valid anymore

@simonjenga this is not valid anymore

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@4a07447a-25ae-4dc9-b0d1-e99cf6d3ad97, thread: pool-1155-thread-1
com.zerocracy.radars.github.GhProject[69] com.zerocracy.radars.github.GhProjectNotFoundException: Github project for 'cqfn/eo' repo was not found

0.55: Issue: cqfn/eo#153, Comment: 453548545

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@4a07447a-25ae-4dc9-b0d1-e99cf6d3ad97, thread: pool-1155-thread-1
com.zerocracy.radars.github.GhProject[69] com.zerocracy.radars.github.GhProjectNotFoundException: Github project for 'cqfn/eo' repo was not found

0.55: Issue: cqfn/eo#153, Comment: 453548545

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@4a07447a-25ae-4dc9-b0d1-e99cf6d3ad97, thread: pool-1158-thread-1
com.zerocracy.radars.github.GhProject[69] com.zerocracy.radars.github.GhProjectNotFoundException: Github project for 'cqfn/eo' repo was not found

0.55: Issue: cqfn/eo#153, Comment: 453552045

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@4a07447a-25ae-4dc9-b0d1-e99cf6d3ad97, thread: pool-1158-thread-1
com.zerocracy.radars.github.GhProject[69] com.zerocracy.radars.github.GhProjectNotFoundException: Github project for 'cqfn/eo' repo was not found

0.55: Issue: cqfn/eo#153, Comment: 453552045

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1


Remove version: Jackett.Jackett version 0.20.2489

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.2489/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Error return code "rc=-1" when updating Arduino IDE

Hi Dears,
since a couple of years I started working with ESP8266 (EPS-01 in particular).
I have a MQTT based application running without issues, using Arduino IDE to code and compile
I needed to reinstall Arduino IDE and when recompiling my code MQTT cannot connect (although ping is reacheable and NTP client is working ok for ESP board).
The error/return code I get is the following "failed, rc=-1".
So my question is: what cases pusubclient library drops this error code (I know it means MQTT_DISCONNECTED -1)?  Any well-known case?
Any hint will be welcome! Thanks!

Hello. Are you trying example sketches?


Smtp

This fixes an old todo in the code where the HELO EHLO is supposed to be followed by the FQDN of the client, the code instead sent the recipient.   Probably worked fine on a same server but depends on what rules they put in place in postfix for example.
I did not cache or save the result in the PR, seems like that value would not change so that may be worth considering since to validate FQDN requires some network calls which could? be slow? What do you think?

I fear this can create a situation where this might get stuck on a bad value, that might also get rejected. I'd rather have an automatically generated setting that I can override, rather than something like this.

nods ... I had a few concerns as well, as it certainly could create an incorrect value.  TBH I'm surprised it worked at all prior to this ... it probably had issues if you were using an external SMTP service, or any SMTP receiver that was setup correctly.
We could call the function to seed an initial 'client' value in the ACP which of course can then be changed and saved.  Just need to add a new input field and txt string.


[AIRFLOW-2524]Add SageMaker Batch Inference

Make sure you have checked all steps below.
JIRA

[X] My PR addresses the following Airflow JIRA issues and references them in the PR title.

https://issues.apache.org/jira/browse/AIRFLOW-2524

Description

[X] Here are some details about my PR, including screenshots of any UI changes:

This PR allows user to start a Amazon SageMaker Transform job (Batch Inference) using the SageMakerCreateTransformJobOperator
User can also check the progress(state) of the transform job through the SageMakerTranformSensor

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

tests/contrib/hooks/test_sagemaker_hook.py
tests/contrib/operators/test_sagemaker_create_transform_job_operator.py
tests/contrib/sensors/test_sagemaker_transform_sensor.py

Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[X] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[X] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3767 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3767      +/-   ##
==========================================
- Coverage   77.67%   77.66%   -0.01%
==========================================
  Files         204      204
  Lines       15849    15849
==========================================
- Hits        12310    12309       -1
- Misses       3539     3540       +1

Impacted Files
Coverage Δ

airflow/models.py
88.74% <0%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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

It looks like the tests are failing:

======================================================================
24) ERROR: test_create_transform_job (tests.contrib.hooks.test_sagemaker_hook.TestSageMakerHook)
----------------------------------------------------------------------
   Traceback (most recent call last):
    .tox/py27-backend_mysql/lib/python2.7/site-packages/mock/mock.py line 1305 in patched
      return func(*args, **keywargs)
    tests/contrib/hooks/test_sagemaker_hook.py line 439 in test_create_transform_job
      wait_for_completion=False)
    airflow/contrib/hooks/sagemaker_hook.py line 258 in create_transform_job
      ['TransformInput']['DataSource']['S3Uri'])
   KeyError: 'S3Uri'
======================================================================
25) ERROR: test_create_transform_job_db_config (tests.contrib.hooks.test_sagemaker_hook.TestSageMakerHook)
----------------------------------------------------------------------
   Traceback (most recent call last):
    .tox/py27-backend_mysql/lib/python2.7/site-packages/mock/mock.py line 1305 in patched
      return func(*args, **keywargs)
    tests/contrib/hooks/test_sagemaker_hook.py line 454 in test_create_transform_job_db_config
      create_transform_params, wait_for_completion=False)
    airflow/contrib/hooks/sagemaker_hook.py line 258 in create_transform_job
      ['TransformInput']['DataSource']['S3Uri'])
   KeyError: 'S3Uri'

@troychen728 can you shed a light on the error @schipiga is encountering?

Can you rebase onto master as well? :)

Hi @Fokko and @schipiga
Sorry for late updates. @troychen728 is unavailable for development recently. I will target to update this PR sometime next week.

@Fokko
Thanks for your review
Now @troychen728 and I finished training, tuning, batch inference for sagemaker in Airflow. The only remaining part is endpoint inference. I currently aim at sending the PR next week for that.


Multiple versions installed when upgraded - Community edition

my environment running ArangoDB
I'm using ArangoDB version:

[x] 3.3

Mode:

[x] Single-Server

Storage-Engine:

[x] rocksdb

On this operating system:

[x] Windows, version: 10

this is an installation-related issue:
I started using ArangoDB community edition on my windows 10. Started with 3.3.3 , when new versions for bug fixes I upgraded from the link that is provided and running installer. Instead of updating version  of current ArangoDB , it creates new standalone application on windows. This is same for community edition as well.
Please see below screenshot.

I can switch to different versions on web interface just by starting arangod.exe (Arango Server) for that instance. I can see multiple file locations as below

C:\Program Files\ArangoDB3 3.X.X\etc\arangodb3
C:\Users\Prateek\AppData\Local\ArangoDB3-client 3.X.X

It gets confusing to decide where my database files are associated with.
Expected behavior : Need to upgrade current version , not have multiple versions.

The Windows installer creates a new folder for the binaries and other server-related files each time in C:\Program Files\ArangoDB 3 x.x.x, which I think is intended - you can't replace files while they are on use under Windows (this is different in Linux systems).
The data files however are shared by default (it's the install for all users and as service option). They reside in C:\ProgramData\ArangoDB. Foxx services are in C:\ProgramData\ArangoDB-apps.
After installation of a bug fix release, you can remove the binaries from the old version via the control panel > programs and apps using the uninstaller. Don't tick the option to remove databases and apps though!

Hi @pmahend1
Was Simran's message useful for you? Is there anything else we can do to support you in this ticket?
Best,

@OmarAyo I have already done uninstalling older versions safely.
Raised bug as it was suggested on Stackoverflow question
It would be nice to have feature to upgrade existing software by prompting closure of running application.

This should be solved starting from 3.4.0-RC.3
Please could you check 3.4.0-RC.3 when it is released?
Thanks,

@sleto-it Sure. will check and update.

I could see DB being migrated or something being shown on command prompt while upgrade software was running .
I think its somewhat fixed. I can still see multiple versions on my desktop.

I can run the old ArangoDB server excutable but it wont start old server, one thing new version is locking DB files so that old version arangod.exe wont run.
I can run old arangosh.exe but good thing it points to new db server.
C:\Program Files\ArangoDB3 3.3.10\usr\bin>arangod.exe
2018-10-26T13:57:22Z [572] INFO {authentication} Jwt secret not specified, generating...
2018-10-26T13:57:22Z [572] INFO ArangoDB 3.3.10 [win64] 64bit, using build tags/v3.3.10-0-g902a980389, VPack 0.1.30, RocksDB 5.6.0, ICU 58.1, V8 5.7.492.77, OpenSSL 1.0.2a 19 Mar 2015
2018-10-26T13:57:22Z [572] INFO using storage engine mmfiles
2018-10-26T13:57:22Z [572] INFO {cluster} Starting up with role SINGLE
2018-10-26T13:57:22Z [572] INFO {authentication} Authentication is turned on (system only)
2018-10-26T13:57:22Z [572] FATAL failed to remove an abandoned lockfile in the database directory, please check the file permissions of the lockfile 'C:\ProgramData\ArangoDB\LOCK': system error

C:\Program Files\ArangoDB3 3.3.10\usr\bin>arangosh
Please specify a password:

                                       _
  __ _ _ __ __ _ _ __   __ _  ___  ___| |__
 / _` | '__/ _` | '_ \ / _` |/ _ \/ __| '_ \
| (_| | | | (_| | | | | (_| | (_) \__ \ | | |
 \__,_|_|  \__,_|_| |_|\__, |\___/|___/_| |_|
                       |___/

arangosh (ArangoDB 3.3.10 [win64] 64bit, using build tags/v3.3.10-0-g902a980389, VPack 0.1.30, RocksDB 5.6.0, ICU 58.1, V8 5.7.492.77, OpenSSL 1.0.2a 19 Mar 2015)
Copyright (c) ArangoDB GmbH

Pretty printing values.
Connected to ArangoDB 'http+tcp://<IP_ADDRESS>:8529' version: 3.4.0-rc.3 [server], database: '_system', username: 'root'

Please note that a new bugfix version '3.3.19' is available
Type 'tutorial' for a tutorial or 'help' to see common examples
<IP_ADDRESS>:8529@_system>
<IP_ADDRESS>:8529@_system> exit

Uf wiederluege! σåìΦºü∩╝ü Na shledanou! Auf Wiedersehen! Bye Bye! Adiau! ┬íHasta luego! ╬ò╬╣╧é ╧ä╬┐ ╬╡╧Ç╬▒╬╜╬╣╬┤╬╡╬»╬╜!
╫£╫ö╫¬╫¿╫É╫ò╫¬! Arrivederci! Tot ziens! Adj├╢! Au revoir! πüòπéêπüåπü¬πéë ╨ö╨╛ ╤ü╨▓╨╕╨┤╨░╨╜╨╕╤Å! At├⌐ Breve! ╪«╪»╪º╪¡╪º┘ü╪╕! Uz redz─ô┼íanos!

But what if I run arangod.exe from admin privileges?
C:\Program Files\ArangoDB3 3.3.10\usr\bin>arangod.exe
2018-10-26T14:06:17Z [11424] INFO {authentication} Jwt secret not specified, generating...
2018-10-26T14:06:17Z [11424] INFO ArangoDB 3.3.10 [win64] 64bit, using build tags/v3.3.10-0-g902a980389, VPack 0.1.30, RocksDB 5.6.0, ICU 58.1, V8 5.7.492.77, OpenSSL 1.0.2a 19 Mar 2015
2018-10-26T14:06:17Z [11424] INFO using storage engine mmfiles
2018-10-26T14:06:17Z [11424] INFO {cluster} Starting up with role SINGLE
2018-10-26T14:06:17Z [11424] INFO {authentication} Authentication is turned on (system only)
2018-10-26T14:06:17Z [11424] INFO running WAL recovery (1 logfiles)
2018-10-26T14:06:17Z [11424] INFO replaying WAL logfile 'C:\ProgramData\ArangoDB\journals\logfile-13518334.db' (1 of 1)
2018-10-26T14:06:17Z [11424] INFO WAL recovery finished successfully
2018-10-26T14:06:18Z [11424] ERROR In database "Foo": Database directory version (30400) is higher than current version (30310).
2018-10-26T14:06:18Z [11424] ERROR In database "Foo": It seems like you are running ArangoDB on a database directory that was created with a newer version of ArangoDB. Maybe this is what you wanted but it is not supported by ArangoDB.
2018-10-26T14:06:18Z [11424] ERROR In database "Fighters": Database directory version (30400) is higher than current version (30310).2018-10-26T14:06:18Z [11424] ERROR In database "Fighters": It seems like you are running ArangoDB on a database directory that was created with a newer version of ArangoDB. Maybe this is what you wanted but it is not supported by ArangoDB.
2018-10-26T14:06:18Z [11424] ERROR In database "Nirvana": Database directory version (30400) is higher than current version (30310).
2018-10-26T14:06:18Z [11424] ERROR In database "Nirvana": It seems like you are running ArangoDB on a database directory that was created with a newer version of ArangoDB. Maybe this is what you wanted but it is not supported by ArangoDB.
2018-10-26T14:06:18Z [11424] ERROR In database "_system": Database directory version (30400) is higher than current version (30310).
2018-10-26T14:06:18Z [11424] ERROR In database "_system": It seems like you are running ArangoDB on a database directory that was created with a newer version of ArangoDB. Maybe this is what you wanted but it is not supported by ArangoDB.
2018-10-26T14:06:18Z [11424] INFO using endpoint 'http+tcp://<IP_ADDRESS>:8529' for non-encrypted requests
2018-10-26T14:06:20Z [11424] INFO Please note that a new bugfix version '3.3.19' is available
2018-10-26T14:06:20Z [11424] INFO ArangoDB (version 3.3.10 [win64]) is ready for business. Have fun!

then I was able to connect via 3.3.10 (old version) with warning.

@pmahend1 How I believe it is expected to work is, that if you manually start the uninstaller and remove the ArangoDB instance, then the start menu entries should get removed along with the binaries. But this requires 3.4+. The uninstallation of older versions should remove the binaries as expected, but leaves the start menu entries behind.
If you overwrite an existing installation with new binaries using the installer, then most but not all files should get overwritten, but you will end up with start menu and app entries for both versions.

I did have only v3.3.10 when installing 3.4.
I uninstalled v3.3.10 now , I will carefully uninstall 3.4 and install 3.3.19 and then install 3.4 again.
Wondering why it should work only with immediate previous one version 3.3.19 .
It gets confusing . So I guess best solution at the moment is to uninstall any existing software without deleting database files and install latest one with importing/retaining old database files.

Uninstalling while keeping the database files and then installing the new version is the cleanest option in my opinion, yes.
Otherwise we would need to make the installer run the uninstaller of the previous version automatically in the background I suppose. But there are quite a few additional stumbling blocks, so not sure if we can rework all that for RC4.

Yeah. that would require to force close all db executables.
To not mess up existing db; users should still be informed to close the db server and instances.I am not sure how copying database files while it still running is reliable. There are possibilities to corrupt old db files if done while installation ?


formatDocument

It can use the PHPParser's pretty printer

Method 'textDocument/formatting' needs as response TextEdit array. PHPParser's pretty printer returns always formatted string and simplest solution is to create one TextEdit to replace whole document. If you don't plan to implement it in near future I can try to do this :) or maybe you have different idea how utilize PHPParser.

Yes, I think that is the easiest solution. We currently also sync all textdocuments with full content, it doesn't seem to have significant perf impact, but it reduces complexity. We can adjust the implementation later.
Using PHPParser is of course not required for this, we could look at other formatting tools for PHP. But we already have the AST, so it is easier to just use the formatter, and the pretty printer comes included.
I'm super short on free time atm so yeah I would be more than happy if you implement it :)

I was looking for alternative but I didn't found anything actively developed/maintained. For now PHPParser's pretty printer is easy to use but is not customizable. I think it will be easy to replace it with different formatting library if something will came up in the future.

We could customize it by subclassing it. Or doing a PR to support some options :)
But I would like to get basic features out as quick as possible, customization can be added later.


Questions about deep prompt per layer

Hi, I have a question about deep prompt.
I understand that deep prompts are implemented through past_key_values in model.
Then how can I see the actual prompt weights per layer?
I mean, the shape of prompt is (prefix_len, config.num_hidden_layers * 2 * config.hidden_size) if without trans.
And the shape of past_key_values for input is [2, batch_size, n_head, prefix_len, n_embd] per each layer. I believe that the first '2' corresponds key and value for attention mechanism.
Here I want to obtain [prefix_len, config.hidden_size] vector just like embedding vector of prompt-tuning v1.
Do you have any idea for this?
Thanks : )

@eunjiinkim Hi,
Thanks for your interest in P-Tuning v2. I think the code you are looking for lies in here. And it is transformed into  [2, batch_size, n_head, prefix_len, n_embd] at here.
Do these codes solve your problems? Feel free to ask if you still have other questions.

I think the attention mechanism is interesting regarding p-tuning v2's behavior. I notice that for many attention heads actually prefix tokens do not change anything in the attention computation. For other heads, it is a rather valuable research problem to see what are prompt tokens actually doing in p-tuning v2.

Thanks for your comment!
Actually, I'm trying to investigate the prefix tokens in terms of not only attention weights but also the relationship with model own embedding.
So I'd like to get the vectors whose shape is like [prefx_len, hidden_size] in order to calculate similarity with model parameters.
As you said, past_key_values are splitted into [2, batch_size, n_head, prefix_len, n_embd].
So, do you mean that I can only see past_key_values separated by key and value?
Here, hidden_size is n_head*n_embd, but '2' is a key issue in my case because these two vectors have their own weights.
Should I just see key and value vectors separately or can I multiply key and value vectors?

@eunjiinkim Hi,
I now understand your question, that you want the prompt embedding, instead of its corresponding key and value, right?
In p-tuning (v1), the key and value are derived from using attention heads' linear projection matrices to transform prompt embeddings. However, for the consideration of optimization, in p-tuning v2 the keys and values are separate from the very beginning; if not, it still converges but may require another set of hyper-parameters according to my experience.
Existing p-tuning v2 implementation doesn't allow such features. To achieve the goal, it might take you some efforts to modify the huggingface transformer source code (e.g., for roberta, see this line) to only pass [1, batch_size, n_head, prefix_len, n_embd] as past_key_value, and project it into corresponding keys and values for concatenation, such as (without testing, not sure if my code really works):
key_layer = torch.cat([self.key(past_key_value[0]), key_layer], dim=2)
value_layer = torch.cat([self.value(past_key_value[0]), value_layer], dim=2)

and in this case after training converges, I think the [prefix_len, hidden_size] is exactly what you need.

@Xiao9905
HI,
thanks for your effort. It really helped me.
I've changed some codes in soft prompt and modeling_gpt2.py.
For soft prompt, I removed *2 (which was a problem for me)
self.new_size =self.config.num_hidden_layers * self.config.hidden_size
and removed split(2) in past_key_values input.
        past_key_values = past_key_values.permute([2, 0, 3, 1, 4]).to(self.device)
For gpt2 modeling,  I modified past_key and past_value just like you've suggested in Roberta.
        if layer_past is not None:
            past_key, past_value = layer_past, layer_past
            key = torch.cat((past_key, key), dim=-2)
            value = torch.cat((past_value, value), dim=-2)

I think it works and gives quite similar result with original size prompt-v2 in my experiment.
Thank you SO much. Now I can do what I want. :)

That's cool. Looking forward to your research!


Stop generating report for deleted package.json

Hello,
On the Github of my package, I am receiving dependabot alert for a deleted package.json.
You can see in the screenshot that no folder /client exist in the https://github.com/trollepierre/recontact_travel_blog project. (It has been renamed in /front several months ago)
This is kind of frustrated, and I don't know how to prevent that.
Feel free to ask more precision.

Package manage/ecosystem
Javascript
Manifest contents prior to update
https://github.com/trollepierre/recontact_travel_blog/network/alerts

Updated dependency

What you expected to see, versus what you actually saw

Images of the diff or a link to the PR, issue or logs

@trollepierre looks like the Dependency Graph had these files cached. I've re-detected your manifest files and the client files are no longer showing up so will close this out. Let me know if you have any further issues.


[1.3.0-RC4] ignores missing plugins

steps
create new project
add non-existing plugin to plugins.sbt
start sbt
problem
sbt starts up as if nothing is wrong
expectation
You expect a ResolveException like with sbt 1.3.0-RC3 or any version before that
found this issue while trying to reproduce #4852, you can find more context and a reproducer project at this comment

hmm, I tried with an other project and could not reproduce, removed the ./target folder and the problem went away


Unable to load docker config

I've forked the main branch of aries-vcr.
I got the error while building.
The error is same as:

How to fix it?

@mjour, I just pulled the latest code and did a ./manage build, and it succeeded.
I'm using Docker Desktop on Windows 11
$ docker version
Client:
 Cloud integration: v1.0.29
 Version:           20.10.22
 API version:       1.41
 Go version:        go1.18.9
 Git commit:        3a2c30b
 Built:             Thu Dec 15 22:36:18 2022
 OS/Arch:           windows/amd64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.16.3 (96739)
 Engine:
  Version:          20.10.22
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.18.9
  Git commit:       42c8b31
  Built:            Thu Dec 15 22:26:14 2022
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.14
  GitCommit:        9ba4b250366a5ddde94bb7c9d1def331423aa323
 runc:
  Version:          1.1.4
  GitCommit:        v1.1.4-0-g5fd4c4d
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Looks like your environment is having issues pulling registry.access.redhat.com/ubi9/python-39:latest.  Try running a docker pull registry.access.redhat.com/ubi9/python-39:latest to see if you can pull the image itself.

I've just download latest code and there is an different error.

I'll look into that error

@mjour if you pull the latest this should be fixed

@mjour if you pull the latest this should be fixed

I will retry it.

The latest codes does work well.


Cannot process request when trying to view players

Any player that I click on I receive an error that it could not process. I've double and triple checked all the settings and they look to be fine but that's the only error I receive.

"
Notice:  A non well formed numeric value encountered in C:\xampp\htdocs\WUA\classes\class.Player.inc.php on line 111
{"BANEXPIRY":"01/01/1970 01:00:00","BANNED":"0","BANREASON":"","CREATIONDATE":"05/28/2017 06:10:52","CHEATED":"0","CHEATREASON":"","EMAIL":"Eveswafford@test.com","INVENTORYID":"1069938454274","IPADDRESS":"/<IP_ADDRESS>","KINGDOM":"4","LASTLOGOUT":"01/01/1970 01:00:00","MONEY":"0 irons","MUTED":"0","MUTETIMES":"0","MUTEEXPIRY":"01/01/1970 01:00:00","MUTEREASON":"","NAME":"Eveswafford","PLAYINGTIME":"1 day and 4 hrs","POWER":"0","WURMID":"453023444736","image":"avatar_e_120.png","success":true}"
The issue appears to be the way the LASTLOGOUT is being written from the server.

Ok found the reason why it's having issues.
You have the
if ($user["LASTLOGOUT"] != null)
{
$user["LASTLOGOUT"] = date("m/d/Y H:i:s", $user["LASTLOGOUT"] / 1000);
}
on lines 99-102 AND 109-112


Fix unexpected error when an empty set is returned

Closes #122
Signed-off-by: Manuel Carmona<EMAIL_ADDRESS>

@ajnavarro ping


[MOD] Change comment/thread font size

Is this request to port an existing userscript or mod, or create a new one?
Link to the source script if applicable.
Describe the solution you'd like
Add the ability to adjust font sizes for threads and comments independent of the page's font size.
Additional context
For some users, the font size for comments/threads is too small, and increasing the font size impacts how the page is laid out on mobile. This comment is where this idea is from: https://kbin.social/m/kbinMeta/t/183928/-/comment/741140

One thing we should start looking to do is consolidating features so that we don't have 1,000s of mods that, e.g., hide one element. It seems like this would fit well into a mod that controls a variety of different onscreen elements, and you can granularly adjust the ones you want. Or turn it off altogether.
Author: you
Dynamically changes the font size of elements

Headers [dropdown]
Comments [dropdown]
Post description [dropdown]
Usernames [dropdown]

etc

I was just thinking about that, I think it's a good idea. Speaking of organizing things, it might be nice to sort some of the stuff from the general page into more specific pages so the options are a little more spread out. I think adding a few extra pages could be good once we add some more mods, like "Navigation" for example

Yes, although we can't do that too much after a certain point, as it is liable to be confusing going forward, once people get accustomed to certain mods being in a certain place

Closing as duplicate of https://github.com/aclist/kbin-kes/pull/171


ibmcloud: Included Init Contianers along with nginx pod

Added initcontianers along with nginx pods in order to add users manually
Fixes:  #1450

Why would the dockerhub's nginx image - arguably the most testes oci image - require an init container?


How to extract the waveform from a file?

The problem is that this package requires a certain format of the samples you provide to the waveform. The problem is that in your example, you use audiowaveform package to extract the samples. But if we want to use pure flutter and extract these samples from IOS phones for example it's not possible.
Is there a way to extract the samples once we have a .wav file on disk on the phone?

any solutions?


MAC Address

doese the mac address  of the remotes realy need to be provided befor compile time?

Actually this could even be dynamic in most cases (e.g. ESP8266).
Generally I would be rather moving in a generic proxy direction, that removes the need of explicitly registering each sensor on the receiver(hub/proxy) side.
At the moment I only would be able to hack this via e.g. mqtt or bluetooth_proxy side and that would only affect HA, not the esphome side.

Hi! Yes, i agree that it would be better to avoid, in a real production environment, fixed MQTT topics and BLE mac addresses. I thing that it would be great, for MQTT topic, automatic suffixing with the device MAC address (like mDNS name), and getting BLE mac addresses from MQTT topic or from flash NVS that could be read in runtime. Is that possible in your opinion? It would be more flexible.


Live lot should be ArtsyPurple in lost list view on iPad

I think this is also a side-effect of https://github.com/artsy/eigen/issues/1651 - I've not seen this anywhere in-app. Not closing but moving to QA for validation.

Issue moved to artsy/auctions #186 via ZenHub


[12] Add generated data timeseries notebook

Closes #12.
This PR:

adds functions for generating synthetic research funding and private investment data (iss_forecasting/analysis/utils/time_series.py)
adds a notebook using the generated time series data to plot charts, test for stationarity, cross correlation plots and granger causality (iss_forecasting/analysis/tech_category_level/research_funding_and_investment_generated_data.py)

There was an issue where investment_raised_total was being found to granger cause research_funding_total. This was due to using a random seed in the function iss_forecasting.analysis.utils.time_series.generate_ts

At first, I was surprised by the results in the granger tests where for example in the yearly data, the tests would suggest that research funding granger causes investment at lag 1 (which was expected) but then also at lags 2 and 3.
This is due to how the granger tests work. When testing lag 2, it includes lag 1 and lag 2. When testing at lag 10 it is including lag 1-lag 10. The results make sense as the important lag is still included when testing the later lags.

Thank you, Jack - I will aim to take a look at this by end of Thursday.
On Wed, 9 Feb 2022 at 13:07, Jack Rasala @.***> wrote:

@Jack-0-0 https://github.com/Jack-0-0 requested your review on: #13
https://github.com/nestauk/iss_forecasting/pull/13 [12] Add generated
data timeseries notebook.
—
Reply to this email directly, view it on GitHub
https://github.com/nestauk/iss_forecasting/pull/13#event-6034223774, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHDWGHKZPMA7ZL2ZPO4HJV3U2JRKNANCNFSM5NSHYA3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.Message ID:
@.***>

--
58 Victoria Embankment
London
EC4Y 0DS
			www.nesta.org.uk

http://www.nesta.org.uk/ |					Twitter http://www.twitter.com/nesta_uk
|				 				LinkedIn
http://www.linkedin.com/groups?gid=1868227&goback=.gdr_1274367066783_1
|				 				Facebook
http://www.facebook.com/pages/NESTA/116788428355432?v=wall&ref=sgm
If
you no longer want to receive emails from Nesta, send an email to
@.*** @.>. This email and any
attachments are confidential and may be 				subject to legal privilege. Any
use, copying or disclosure other 				than by the intended recipient is
unauthorised. If you have 				received this message in error, please notify
the sender 				immediately or by email to				 				@.
@.***> and delete this message and any
copies from your computer and network. The views expressed in 				this
email are those of the author and do not necessarily 				reflect the views
of Nesta. Nesta is a company limited by 				guarantee and registered in
England and Wales with company 				number 7706036 and charity number
1144091. Registered as a 				charity in Scotland number SC042833.
Registered office: 58 Victoria Embankment, London, EC4Y 0DS.
--

Thank you @Jack-0-0 - I've left a few small comments, but otherwise looks good to merge!
Happy to see that the measures behave pretty much as expected. Perhaps would be interesting to also inspect time series with some underlying trend (not just constant/noisy baseline) but can leave that for some other time to experiment with.

Thank you @Jack-0-0 - I've left a few small comments, but otherwise looks good to merge!
Happy to see that the measures behave pretty much as expected. Perhaps would be interesting to also inspect time series with some underlying trend (not just constant/noisy baseline) but can leave that for some other time to experiment with.

Thank you for the review @beingkk. I've updated generate_ts's docstring and added a random seed.


--lineseparator option to use whatever separator the file is already using

in my team, some people use git core.autocrlf=true and others use input, which means robotidy will change the line endings unnecessarily. it would be nice if there was an option to make it use whatever line endings the file already has

Currently we have three (fixed) options: native, windows and linux. To implement your idea we would need to implement some sort of checker for separator type before running transformations (and we could call --lineseparator detect or --lineseparator keep). Is it what you have in mind?
What if line separator are not consistent? Easiest would be just use the first line separator. We could also try to count and pick most popular one (with small hit to performance but nothing noticeable).

i'm not sure what the best way is, maybe see how other formatters handle this issue? for example prettier: https://prettier.io/docs/en/options.html#end-of-line

"auto" - Maintain existing line endings (mixed values within one file are normalised by looking at what’s used after the first line)

"The other are doing this" is deciding factor ;) We can go with 'auto' and just check for first line ending. Basically if linesep is set to auto and we're saving the model to disk here:
https://github.com/MarketSquare/robotframework-tidy/blob/d9a7cd6c26f97a927a8bc9171f04a71202520961/robotidy/app.py#L98
then we could check the source for line endinds and use it instead. Here usage of 'newlines' attribute:
https://stackoverflow.com/questions/2798627/how-can-i-detect-dos-line-breaks-in-a-file

I can do it - or leave if for you if you want to create PR.

thanks! sorry i didn't see your comment


Do not throw an exception when port is not open

... otherwise the program terminates without a result.
like so...
❯ python3.9 portscanner_async.py
Traceback (most recent call last):
  File "/home/jugmac00/Projects/cpython-book-samples/33/portscanner_async.py", line 31, in <module>
    results = asyncio.run(scan(80, 100, host))
  File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/home/jugmac00/Projects/cpython-book-samples/33/portscanner_async.py", line 24, in scan
    await asyncio.gather(*tasks)
  File "/home/jugmac00/Projects/cpython-book-samples/33/portscanner_async.py", line 10, in check_port
    r, w = await asyncio.wait_for(future, timeout=timeout)
  File "/usr/lib/python3.9/asyncio/tasks.py", line 476, in wait_for
    return fut.result()
  File "/usr/lib/python3.9/asyncio/streams.py", line 52, in open_connection
    transport, _ = await loop.create_connection(
  File "/usr/lib/python3.9/asyncio/base_events.py", line 1064, in create_connection
    raise OSError('Multiple exceptions: {}'.format(
OSError: Multiple exceptions: [Errno 111] Connect call failed ('<IP_ADDRESS>', 81), [Errno 101] Network is unreachable

Zweiter :-)

huh. must be a linux thing. thanks


No requirements file makes compatibility issues challenging to debug.

Closing this issue because it does not provide enough information on how to reproduce. If you report back with the specific compatibility issue you're having, we can re-open.


fix for #3427. Added test case.

@lvca can this be merged?

It's impossible to see what you changed in ORecordLazyList.java. Is it a formatting problem?

I think so. I have just removed an unused import.

Ok, merging anche checking it right after.

Thanks, as always ;-)

@lvca my 2c, in case you are using the github web interface to see the diff, add ?w=1 at the end of the URL to ignore whitespace in the comparison, using that, i can clearly see that there was only 1 line change to remove the unused import.

?w=1 works, thanks!


Fixed the Deploy to Azure button

Fixes #
Fixed the Deploy to Azure button

Closing PR as a new Deploy button has been introduced.


HTTP endpoints call from c#

To make the code portable, it makes more sense posting to the Endpoints. Any chance of giving a complete example under such a scenario? I've been battling with it for the last 8 hours and cannot get my JSON to be serialised to the required format

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

ID: 8f57e416-a37f-e719-4a3e-58a4e6390818
Version Independent ID: dbb3821b-5793-b41b-9601-559b00e4096a
Content: C# Quickstart for Azure Cognitive Services, Text Analytics API
Content Source: articles/cognitive-services/text-analytics/quickstarts/csharp.md
Service: cognitive-services
Product: unspecified
GitHub Login: @LuisCabrer
Microsoft Alias: luisca

@marvoh Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@LuisCabrer Hi, can you please check the issue and update the document as necessary? Thanks a lot!

HI @marvoh,
If you are asking how to use C# to post to endpoints using http calls rather than the SDK, you can follow the guidance here to properly form an http request using .NET.
You can shape the request and response depending on the endpoint based on these instructions.
Thanks!

Hi @marvoh,
If you use Visual Studio to create a Console Application. You can replace the entire Program.cs file with the following code that calls the HTTP endpoint using C# and it will work. You will need to provide your own API key, ensure you are using the same region, and alter which endpoint you are using, but it should work great!.
using System;
using System.Net.Http.Headers;
using System.Text;
using System.Net.Http;
using System.Collections.Generic;

namespace TextAnalyticsCSharpCore
{
    struct Document
    {
        public string id;
        public string text;
    }

    class Program
    {
        // **********************************************
        // *** Update or verify the following values. ***
        // **********************************************

        // Replace the accessKey string value with your valid access key.
        const string accessKey = "enter key here";

        // Replace or verify the region.
        //
        // You must use the same region in your REST API call as you used to obtain your access keys.
        // For example, if you obtained your access keys from the westus region, replace
        // "westcentralus" in the URI below with "westus".
        //
        // NOTE: Free trial access keys are generated in the westcentralus region, so if you are using
        // a free trial access key, you should not need to change this region.
        const string uriBase = "https://westus.api.cognitive.microsoft.com/text/analytics/v2.0/languages";

        static void Main()
        {
            Console.WriteLine("Getting the language for a record");

            List<Document> documents = new List<Document>();
            documents.Add(new Document() { id = "1", text = "This is a document written in English." });
            documents.Add(new Document() { id = "2", text = "Este es un document escrito en Español." });
            documents.Add(new Document() { id = "3", text = "这是一个用中文写的文件" });

            GetLanguage(documents);

            Console.WriteLine("\nPlease wait a moment for the results to appear. Then, press Enter to exit...\n");
            Console.ReadLine();
        }

        /// <summary>
        /// Queries the language for a set of document and outputs the information to the console.
        /// </summary>
        static async void GetLanguage(List<Document> documents)
        {
            var client = new HttpClient();

            // Request headers
            client.DefaultRequestHeaders.Add("Ocp-Apim-Subscription-Key", accessKey);

            HttpResponseMessage response;

            // Compose request.
            string body = "";
            foreach (Document doc in documents)
            {
                if (!string.IsNullOrEmpty(body))
                {
                    body = body + ",";
                }

                body = body + "{ \"id\":\"" + doc.id + "\",  \"text\": \"" + doc.text + "\"   }";
            }

            body = "{  \"documents\": [" + body + "] }";

            // Request body
            byte[] byteData = Encoding.UTF8.GetBytes(body);

            using (var content = new ByteArrayContent(byteData))
            {
                content.Headers.ContentType = new MediaTypeHeaderValue("application/json");
                response = await client.PostAsync(uriBase, content);
            }

            // Get the JSON response
            string result = await response.Content.ReadAsStringAsync();

            Console.OutputEncoding = System.Text.Encoding.UTF8;
            Console.WriteLine("\nResponse:\n");
            Console.WriteLine(JsonPrettyPrint(result));
        }

        /// <summary>
        /// Formats the given JSON string by adding line breaks and indents.
        /// </summary>
        /// <param name="json">The raw JSON string to format.</param>
        /// <returns>The formatted JSON string.</returns>
        static string JsonPrettyPrint(string json)
        {
            if (string.IsNullOrEmpty(json))
                return string.Empty;

            json = json.Replace(Environment.NewLine, "").Replace("\t", "");

            StringBuilder sb = new StringBuilder();
            bool quote = false;
            bool ignore = false;
            int offset = 0;
            int indentLength = 3;

            foreach (char ch in json)
            {
                switch (ch)
                {
                    case '"':
                        if (!ignore) quote = !quote;
                        break;
                    case '\'':
                        if (quote) ignore = !ignore;
                        break;
                }

                if (quote)
                    sb.Append(ch);
                else
                {
                    switch (ch)
                    {
                        case '{':
                        case '[':
                            sb.Append(ch);
                            sb.Append(Environment.NewLine);
                            sb.Append(new string(' ', ++offset * indentLength));
                            break;
                        case '}':
                        case ']':
                            sb.Append(Environment.NewLine);
                            sb.Append(new string(' ', --offset * indentLength));
                            sb.Append(ch);
                            break;
                        case ',':
                            sb.Append(ch);
                            sb.Append(Environment.NewLine);
                            sb.Append(new string(' ', offset * indentLength));
                            break;
                        case ':':
                            sb.Append(ch);
                            sb.Append(' ');
                            break;
                        default:
                            if (ch != ' ') sb.Append(ch);
                            break;
                    }
                }
            }

            return sb.ToString().Trim();
        }

    }
}

#close 8985

close 8985

#sign-off

#please-close

Thank you for this, I understand we are at different timezones by the time you responded I had already got it to work, hopefully someone else will find it useful.
Thank you for the effort.


Text-mining or text mining

I see no hyphen in the title but a hyphen in the abstract. 🙂

going with "text mining" without hyphen


Add 'citadel_' prefix to controller monitoring package metrics

Now that we are using OpenCensus metrics, there are no auto-prepended prefixes on metric names. This PR adds the citadel_ prefix to controller metrics.
/cc @pitlv2109
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[X] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

/lgtm
Thanks for the change.
Be sure to update your other PR as well.

@Monkeyanator: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-presubmit-tests-master
e03e66cdccc5d5ebc43ccb483ffae8c6cc44f2fe
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/hold
The Citadel Grafana dashboard expected the citadel_secret_controller_ prefix on all these metrics. Prometheus metrics were exported as namespace_subsystem_metricname-- do we want to 1) change the dashboard to just have citadel_ prefix, or 2) change the exported metric prefixes back to the fully qualified citadel_secret_controller prefix?

Do you need to put the whole metric names in grafana dashboard json file? Like in citadel-dashboard.json, there are citadel_server metrics and there are citadel_secret_controller metrics. If so, I think for each metric, just add the prefix from the previous namespace + subsystem, then make the change in the dashboard accordingly?

Changed the metrics back to have their original citadel_secret_controller prefixes. :+1:

@Monkeyanator: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

istio-pilot-multicluster-e2e-master
036b3777bdd2223e2213b83e0abc63038e0e89df
link
/test istio-pilot-multicluster-e2e-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Text folding from gutter is broken

It looks like set-mark is triggered when I fold text with arrows in gutter.
The effect of this, is that almost any other action unfolds the text back.

If I use cursor movement keys, the text is selected... after the unfold, with unfolded text as initial selection.
If I click anywere with mouse, cursor jump there as expected, but again - with just folded text being unfolded.

To keep expected behaviour (not to trigger set-mark) I have to fold using keybinding, or to press ESC after folding from the gutter.

Link to the same issue on atom/atom: https://github.com/atom/atom/issues/10991

Please try the latest version.


add AppleScript

I had to adjust the strokes, but otherwise I think it looks fine. Hope I didn't forget anything.

Published 1.8.0 containing Applescript. Thanks! Nice icon 😃


Use ForbiddenFilter before LogoutFilter

Removes access to localhost:8080/logout with header x-forbidden: true.
See response: curl -H "x-forbidden: true" localhost:8080/logout
Loved your presentation at JavaLand 😊

Hi David!
I'm glad you like the talk 🎉
You are absolutely correct, this should come before any content-generating filter. I'll blend this in the correct commit, so that the repo keeps the structure of "one commit per concept". Thanks!

This is now in main through 8f83076. Note that I regularly force-push to this repo, so the commit sha will change from time to time.
Thanks!


Docs? And how to get Get-AutomationPSCredential running in the Az Module?

@schittli commented on Tue Mar 17 2020
Hello
Because the Online Azure Editors (e.g. the PowerShell Script Editor in Azure Automation Runbook) is a shame (it does not offer much more than a syntax highlighting), we're required to use the desktop PowerShell with Connect-AzAccount and the Az Module to be productive and not spend days and use try & error to get simple scripts running.
This Page here is called "Docs".
Why does it not help to get started, so that we can use Get-AutomationPSCredential in desktop PowerShell?
Thanks a lot, kind regards,
Thomas

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: 2f830218-a56b-6a7e-d6e7-7736896d3f91
Version Independent ID: ae33d704-d0b7-d478-8602-cc787f706d72
Content: Anmeldeinformationsobjekte in Azure Automation
Content Source: articles/automation/shared-resources/credentials.md
Service: automation
Sub-service: shared-capabilities
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@srvbpigh commented on Tue Mar 17 2020
Hello, @schittli
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@schittli Thank you for your feedback. To further understand your feedback and route it accordingly so that respective team can evaluate the feedback please share some additional information.
Is the feedback to enhance the document by including the steps to use Get-AutomationPSCredential on desktop version powershell ?
Requesting @MGoedtel to further review this thread . Thanks

Good evening @BharathNimmala-MSFT
thank you for your answer!
The background is that in Azure security should be a very important aspect.
This includes having the right cmdlet's available to handle the credentials.
When dealing with Azure, one quickly realizes that one needs the Get-AutomationPSCredential command. Unfortunately, this command is not part of the Az module, but of some probably foreign module that was developed for the horrible, outdated and unmaintained IDE PowerShell ISE.
So it makes no sense to have core security commands like Get-AutomationPSCredential in some foreign PowerShell modules that expect to use outdated development environments.
This immediately raises the question whether Microsoft really takes security seriously. Has a code review ever been done for Get-AutomationPSCredential? Apparently the code hasn't been updated for a long time and we can hardly believe that it is bug-free, because Microsoft hasn't been able to release decently tested software for a long time - just read the press of Windows updates for the last 2 years.
Therefore, the Docs should at least explain how to safely install all the major cmdlets - especially all the security-related cmdlets.

I've escalated your feedback to the Automation PM's and ask that they directly respond to your feedback. #in-progress

We've made a number of recent changes to the content to make it clear about using internal versus external cmdlets with Automation runbooks and this should address your feedback.  We also no longer reference the PowerShell ISE add-on for editing/testing runbooks outside of the Runbook authoring experience in the portal.  Our recommendation continues to be that you manage editing/managing runbook logic from the portal.  #please-close


Do not intercept errors from the custom error service

This change is

Coverage remained the same at 43.721% when pulling 8de75c22a30c12d8f30021792a6606f10897ea73 on aledbf:do-not-intercept-errors into 7d4cd1ba27e664c1289fdf00b92a5f4fbbc20acd on kubernetes:master.


Update deprecated Dockerfile commands

These commands in the current Dockerfile for node installation are deprecated and a large warning appears when docker is building the image.
https://github.com/RaythaHQ/raytha/blob/5e8af6111fa4c385573c024710fb2bcb98772b35/Dockerfile#L6
Should be replaced with:
RUN apt update
RUN apt install -y curl ca-certificates curl gnupg
RUN mkdir -p /etc/apt/keyrings
RUN curl -fsSL https://deb.nodesource.com/gpgkey/nodesource-repo.gpg.key | gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg
RUN echo "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_16.x nodistro main" | tee /etc/apt/sources.list.d/nodesource.list
RUN apt update
RUN apt install -y nodejs

Will be released with v1.1.4.
Thank you


Docs.MS Release Updates for azure-digitaltwins-core

Update docs metadata and targeting for release of azure-digitaltwins-core

Docs Build status updates of commit c0afc78:
:clock10: Full build: the average full build time is 97 min(s) 49 sec(s), this is based on the last 823 build(s) for this repository.

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

File
Status
Preview URL
Details

legacy/docs-ref-autogen/overview/EventHubs/Client.yml
:warning:Warning
View (azure-java-legacy)
Details

docs-ref-services/digitaltwins-core-readme-pre.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

package.json
:white_check_mark:Succeeded

legacy/docs-ref-autogen/overview/EventHubs/Client.yml

[Warning-DuplicateUids] Uid(azure.java.sdk.landingpage.services.eventhub.Client) has already been defined in docs-ref-services/messaging-eventhubs-readme.md.

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


Add QCFractal snowflake test

Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Add test for running jobs through QCFractal.
[x] Update CI conda env

Status

[ ] Ready to go

3.10 is broken I think?