Improvements for the single-page documentation.

The single-page documentation is missing content from the multi-page version.
I almost exclusively use the single-page version and thus did not find any information about the project-reports plugin as it is missing.
There are three changes I'd like to suggest:

Add the project-reports doku and any other non-intentionally missing pages to the single-page version
Add some test that makes sure that only intentional pages are left out and not just forgotten
Add and enforce a title convention for plugin chapters like The <name> Plugin

https://docs.gradle.org/7.0/userguide/implementing_gradle_plugins.html and similar pages are also missing

https://docs.gradle.org/current/userguide/build_services.html too

https://docs.gradle.org/6.8.3/userguide/resolution_strategy_tuning.html too

https://docs.gradle.org/7.1/userguide/upgrading_version_7.html too

https://docs.gradle.org/current/userguide/jvm_test_suite_plugin.html too

This also does not only affect the single-page html documentation of course, but also the PDF version of the user guide.

@lkasso How about splitting the PDF and HTML single-page docs then?
I guess for the HTML you would not have that size problem.
For me that would be fine, as I mostly not use the PDF version anyway.
Would be sad if the HTML single-page docs cannot be extended just because a PDF size limit is reached.

@Vampire, this is likely something I will do in the future but it's much lower priority than many other improvements I am planning for the docs.


Cody Dashboard buffer

I hear tell you can see stats like your number of completions, recent CAR, etc. in a Cody Dashboard.
Should probably support this in the Agent, but UI also needs to be built out in the Emacs client.

Would be cool to have links there to logging and debugging buffers, and other resources.


Custom httpx proxy and user_agent for Scraper

Added support for optional usage of proxies and custom user-agents that can be passed in the construction of the Scraper Class. If the options are not provided, the Scraper class will use the random user agent and no proxy.

 What do you mean? I think this version is great and works well.

@cr4kn4x Hi, why close?

@cr4kn4x Hi, why close?

I can reopen again, but I think it won't get merged in near future. Seems like the maintainer is currently inactive

@cr4kn4x Yes the maintainer seems not so active recently. However, your pr is great :heart: and I am very appreciated for your work since the proxy is a must if we don't have large number of accounts and want to scrape in bulk.
Thanks again and I think this proxy function will help lots of users(pls don't get upset/blue just because someone's remark, I back you up :heart: ). :+1:

if we don't have large number of accounts and want to scrape in bulk

unfortunately proxies will not help in this case, as most of Twitter's rate limits are not determined by IP
i appreciate the effort on this PR, it's a great start. however, there are quite a few things to take into consideration. for example, users who want to access the onion site cannot do so, as httpx doesn't support socks5h. i'll come up with a list of requirements.


Support for JSON format

..intended, as shown here: https://github.com/xeals/signal-back/blob/master/cmd/format.go#L69

I think the main reason I stopped working on this is that I couldn't work out what I wanted it to do. CSV is better for analysis and XML is for backups. I can just export the whole thing to JSON in the same way the CSV does, but I'm not set on it.


LotRO - Failed to create surface

Hi,
since the last Update of Lord of the RIngs Online I am unable to start LotRO in DX11 Mode when I use 64bit Client.
With 32bit (DX11) Client or the 64bit Client with D9VK the game starts with no Problem. Its only a problem with the 64bit client and DX11 (dxvk). When I use the wine DX11 implementation the game starts.
The last messages I get are:

info:  DXVK: Read 3902 valid state cache entries
info:  DXVK: Using 4 compiler threads
err:   Failed to create surface
003a:err:ole:CoUninitialize Mismatched CoUninitialize

I use Manjaro (Last Version) with wine 4.16 (Lutris) and DXVK 1.4.1.
Nvidia 1070 Card and a intel CPU (i7 7700)
I testet other DXVK versions and other wine versions. Same error there.
Is it a DXVK Bug or a problem in wine?
wine.log
Greetings

Please try setting dxgi.deferSurfaceCreation = False in dxvk.conf

Thanks for the repley.
I tryed it with setting:

dxgi.deferSurfaceCreation = True in dxvk.conf

The game now starts but I only get a black screen (I can hear the ingame music).
In the Log I see many errors like:

err:   Failed to create surface

Here is the complete log:
wine2.log

I don't know what the problem is, but this cannot be fixed inside DXVK. I suggest sticking to D3D9 in this case.

Hi,
doitsujin, thank you for your help and the tip with the intel GPU.
I disabled the Intel GPU in the Bios and the game now starts without any problem.


undocumented GetResults

I'd like to use the undocumented http://www.zillow.com/search/GetResults.htm which is available in this python lib. It can do fairly complicated searching for houses, based on the params you would enter on the website. I have the start of a parameter builder, and would be happy to PR for it, once it's a bit more cleaned up, if there is interest. As I look at the rest of this API, methods are basically just 1:1 maps of params, but the GetResults params are pretty ugly, so it might make more sense to add a mapping function for it.

Here is what I am currently using to allow for nicer input params, to give an idea of the sort of thing I mean:
function binOption (defaults, options) {
  var out = defaults.toString().split('')
  options.forEach((o, i) => {
    if (o !== undefined) {
      out[i] = o ? 1 : 0
    }
  })
  return out.join('')
}

function zilloSearchParams (options) {
  options.size = options.size || [0, '']
  options.price = options.price || [0, '']
  options.baths = options.baths || [0, '']
  options.lot_size = options.lot_size || [0, '']
  options.year = options.year || [0, '']
  options.bedrooms = options.bedrooms || [0, '']
  options.days = options.days || [0, '']

  const size = [options.size[0] || 0, options.size[1] || ''].join(',')
  const coords = options.coords.map(i => Math.floor(i * 1000000)).join(',')
  const price = [options.price[0] || 0, options.price[1] || ''].join(',')
  const baths = [options.baths[0] || 0, options.baths[1] || ''].join(',')
  const lotSize = [options.lot_size[0] || 0, options.lot_size[1] || ''].join(',')
  const year = [options.year[0] || 0, options.year[1] || ''].join(',')
  const bedrooms = [options.bedrooms[0] || 0, options.bedrooms[1] || ''].join(',')
  const days = options.days || 'any'
  const lt = binOption(
    111101,
    [
      options.byAgent,
      options.byOwner,
      options.foreclosed,
      options.newHome,
      options.openHouse,
      options.comingSoon
    ]
  )
  // "By Agent", "By Owner", "Foreclosures", "New Homes", "Open Houses Only", "Coming Soon."

  const status = binOption(
    111011,
    [
      options.forSale,
      options.makeMeMove,
      options.recentlySold,
      0,
      options.forRent,
      options.foreclosed
    ]
  )
  // Status of home. A 6 digit binary number.
  // input of 111011 means search for all houses
  // (Set to 1 to search "For Sale"), "Make me move", "Recently Sold", (Next bit seems unused), "For Rent", (Set to 1 if you want to search for foreclosure properties)

  const ht = binOption(
    111111,
    [
      options.houses,
      options.condos,
      options.apartments,
      options.manufactured,
      options.lots,
      options.townhomes
    ]
  )
  // Home Type. A 6 digit binary number. 111111 means search for
  // "Houses", "Condos", "Apartments", "Manufactured", "Lots/Land", "Townhomes"

  if (options.foreclosed === undefined) {
    options.foreclosed = 1
  }

  return {
    bd: bedrooms,                        // Bedrooms (number plus) eg input of "1," means 1 bedroom and up
    days: days,                          // Number of days on market. Select "any" for any number of days
    pmf: options.foreclosed ? 1 : 0,     // Search for foreclosed properties (0/1)
    parking: options.parking ? 1 : 0,    // On site-parking (rentals only) (0/1)
    laundry: options.laundry ? 1 : 0,    // In unit laundry (rentals only) (0/1)
    lt: lt,                              // List Type. A 6 digit binary number for filtering by for sale 111111 would mean search for all for sale "By Agent", "By Owner", "Foreclosures", "New Homes", "Open Houses Only", "Coming Soon."
    pf: options.pre_foreclosure ? 1 : 0, // Search for properties in pre-foreclosure (0/1)
    pets: options.pets ? 1 : 0,          // Accepts pets (rentals only) (0/1)
    status: status,                      // Status of home. A 6 digit binary number. input of 111011 means search for all houses (Set to 1 to search "For Sale"), "Make me move", "Recently Sold", (Next bit seems unused), "For Rent", (Set to 1 if you want to search for foreclosure properties)
    pr: price,                           // Price (number plus) eg input of 50000 means 50000 and up
    ba: baths,                           // Bathrooms (number plus)
    ht: ht,                              // Home Type. A 6 digit binary number. 111111 means search for "Houses", "Condos", "Apartments", "Manufactured", "Lots/Land", "Townhomes"
    sort: options.sort || 'days',        // Sort by choice of (days/featured/priced/pricea/lot/built/size/baths/beds/zest/zesta)
    lot: lotSize,                        // Lot size "<min>,<max>"
    yr: year,                            // Year build "<min>,<max>"
    rect: coords,                        // top-left bottom-right lat/long, somma-seperated
    zoom: options.zoom || 11,            // The zoom of the map.
    sf: size,                            // Square feet "<min>,<max>". If either min or max not set just leave blank but keep comma

    spt: 'homes',                        // Seems to be "homes" by default
    pnd: 0,                              // ?? 0 seems to be default
    rt: 6,                               // ?? 6 seems to be default
    ds: 'all',                           // ?? "all" seems to be default
    p: 1,                                // ?? 1 seems to be default
    zso: 0,                              // ?? 0 seems to be default
    red: 0,                              // ?? 0 seems to be default
    pho: '0,',                           // ?? 0 seems to be default
    search: 'maplist',
    mp: ','
  }
}

@konsumer,
How are you?  Yes, the creation of the request url just takes a JS object (hash table) of the params and builds the query string based on those key/value pairs.  What did you have in mind as alternative?  Obviously the GetResults can take a lot of parameters, but It still looks like key/value associations are going to be necessary to build the url.  Thoughts?
Best,
Richard

I feel like we could just add it as-is and document all the obtuse parameters, or use the parameter builder function above and have nicer options. When I have some time, I'll PR and set it all up.

Is there interest in adding the above code for mapping params? I forgot about this, but I am still happy to make a PR for it.

@konsumer yes please ;)

@konsumer notes after some trial and error on my end:
The rect query params in this format:
bottomLeft.long,bottomLeft.lat,topRight.long,topRight.lat
Given two LatLong markers, I developed some code to fix them into zillow's eight-digit-no-decimal format:
``
let rect: string = "";
let tokenLength: number = 0;
for (const char of ${bottomLeft.longitude},${bottomLeft.latitude},${     topRight.longitude   },${topRight.latitude}) {
if (char === ".") {
continue;
} else if (char === "-") {
tokenLength -= 1;
} else if (char === ",") {
tokenLength = -1;
} else if (tokenLength === 8) {
continue;
}
rect += char;
tokenLength += 1;
}

https://www.carlyzach.com/functions/playground/0a31a2ffadc563a8c491ad59ecde49cc?language=javascript

I'm not using this lib anymore, but feel free to use above code to get you started.

When and if #20 gets merged I'll do that. I figured out some other options and was able to start using it on https://house-rank.carlyzach.com.
See https://github.com/blaskovicz/house-rank-api/commit/bf7f1634bf0e1b8c1dc2bbfa66f08cb137cb5006#diff-648bbbd558888fd428674ecdc24e44aeR172


Assertion error on buffer_id for yr_rules_load_stream

I just recently upgraded to 4.0.2 from 3.11.0, and without any other changes I am getting a new assertion error that I wasn't previously getting when loading compiled Yara rules from memory with yr_rules_load_stream with "large" Yara rulesets.
Assertion failed: buffer_id < arena->num_buffers, file ......\libyara\arena.c, line 390
Example code:
YR_STREAM yr_stream_disk;
yr_stream_disk.user_data = &stream_disk;
yr_stream_disk.read = (YR_STREAM_READ_FUNC)read_yara_stream;
result = yr_rules_load_stream(&yr_stream_disk, &rules_disk);

stream_disk is a custom struct that gets passed to my read function read_yara_stream below:
size_t read_yara_stream(void *buffer, size_t size, size_t count, void *user_data) {
	char *buf = (char *)buffer;
	STREAM *stream = (STREAM *)user_data;

	size_t n;
	for (n = 0; n < count; n++) {
		if (size) {
			memcpy(buf, stream->_ptr, size);
			stream->_ptr += size;
			stream->_cnt -= size;
			buf += size;
		}
	}
	return n;
}

Based on my testing I can only reproduce this error when I am loading in rulesets that are larger than roughly ~180KB (non-compiled) and/or ~570KB compiled.  This could still be user error on my part, but it seems odd that I can run these compiled rulesets (and some much larger) with the Yara binary, but then those same rulesets loading dynamically through yr_rules_load_stream error out with this assertion error once the rulesets get too large.

I may have traced it back to a potential root cause, but I'm unsure if this is a bug or not...
When using the following two test rulesets (separate rule files):
`rule failing {
strings:
$s1 = "testing"
$s2 = "this"
condition:
    $s1 or $s2

}
`
`rule passing {
strings:
$s1 = "this"
condition:
    $s1

}`
I am originally compiling these rules on Ubuntu 20.04 with yara-python 4.0.2 and then running the compiled rules with the yara32.exe binary (also v4.0.2) on a Windows machine.  The "passing" rule does not appear to have any issues, but the "failing" rule (presumably due to the multiple conditions?) fails on all of the files that pass to the second condition with "error code 4" (ERROR_COULD_NOT_MAP_FILE).

I would like to understand the issue better, I have a few questions:

How do you compile the rules that you load with yr_rules_load_stream?
The rules are compiled using YARA 4.0.2?
The rules are compiled in the same machine that later loads them?

So it is very possible that these two issues I described above are unrelated, but I encountered both of them during my initial testing as described below.
First, to answer your questions:

The rules are compiled using yara-python 4.0.2 on an Ubuntu 20.04.1 host
Correct, yara-python 4.0.2 on the compiling host (Ubuntu 20.04.1) and 4.0.2 on the Windows host as well
No, they are two different hosts

The first error (assertion error on rule loading, described in my first comment) is the initial issue I ran into, however during my testing to attempt to narrow down the focus on why I was encountering this error, I started to notice the second issue of getting consistent ERROR_COULD_NOT_MAP_FILE "error code 4" messages.
test.yar
rule failing {
    strings:
        $s1 = "testing"
        $s2 = "this"

    condition:
        $s1 or $s2
}

Compilation on the Linux host
>>> import yara
>>> yara.__version__
'4.0.2'
>>> rules = yara.compile('test.yar')
>>> rules.save('test.yarc')

Execution on Windows host (with matching files in current directory)
yara32.exe -C test.yarc .
I'm hoping that this potential issue with cross-platform compilation is the root cause of both of these problems.

Ok, I see that you are compiling the rules with yara32.exe, assuming that your Linux host is a 64-bits system (which is probably the case) the issue may be related with compiling with a 32-bits YARA and loading them with a 64-bits. Maybe some regression was introduced in YARA 4.x

I am actually doing the opposite: compiling on the Linux host in 64-bit and then running them on the Windows host with the 32-bit binary.  Switching to the 64-bit binary on the Windows host did actually fix that issue for 64-bit -> 64-bit, however, when I compile on the Linux host with 32-bit Yara I am still encountering the same error when executing with the 32-bit binary on Windows.

I did some tests in a Linux host, rules compiled with a 32-bits YARA loaded correctly with the 64-bits YARA and the other way around. I'll try to test it in Windows host.

Fixed in 9fb4b8d560731fb78a9908be12ff54c03e185a09


Add color variations

@pivilartisant New dark blue color is problematic for me. Can you check with @LucasF243 ? 🤔

@pivilartisant New dark blue color is problematic for me. Can you check with @LucasF243 ? 🤔

I checked with him and he validated !


[HeroSystem6eHeroic] Improved readability style update. Minor bug fix.

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Version 3.30
Updates:
•	Style update to the Gear and Skills pages to reduce visual clutter and improve readability.
•	Updated README screenshots.
Fixes:
•	Corrected missing name-title separator for characteristic rolls that don't use degree of success.

Character Sheet Info Roll20 Internal Use only.


Launch worker with dumb-init

Launch worker with dumb-init in order to avoid zombie processes[1]
A detailed explaination of the issue: http://engineeringblog.yelp.com/2016/01/dumb-init-an-init-for-docker.html
[1]http://pastebin.com/zccs39JS

Up!


Runelite Lagging on game PC

Hello guys ,
Runelite is lagging somehow.
i5 8600k coffee lake processor
GTX 1060 6GB Graphic card.
Game motherboard etc etc....
Can anyone help me fix this problem?

https://github.com/runelite/runelite/wiki/Troubleshooting-problems-with-the-client past that guide we cannot help you more with this


boards/arm/efr32xg24_dk2601b: enable semailbox

This PR enables the Secure Element Mailbox node for the efr32xg24_dk2601b and uses it as an entropy source.

Hi, I've removed the DNM label because this PR is no longer modifying the west.yml file. I'm not sure why the bot hasn't triggered itself automatically when these changes were dropped, and I've seen it being active in other PRs in the meantime).


Add basic user interface to select configurations and populate json view

Available  mock features.

Create new config
Load config.
Delete config.
Reset Config.
Json Editor View.

Completed


License

What license is applicable for the code in this repo?
I would guess as it's all ASM that would be included in a finalized game it would be MIT, BSD-3-clause, MAYBE CC0, or something similar, but I do not see a license in the repo so I wanted to ask.
Thank you!

Yup; it's missing. I'll review the details of the different licenses as soon as possible to put the license that best suits the purpose of MSXlib.
The idea is that everyone can use any routine the way they want: copy into their own projects, use MSXlib directly, just as a base for their own libraries... So I'll go for a very permissive license.
Thanks for reporting!

I finally set the BSD-3-Clause license (one of your suggestions). Thanks!

No problem! That's my personal choice these days as well :)
Thank you for your work on this lib as well! Reading the assembly has helped me understand the Z80 quite a bit I think


incorrect values displayed in AES Flight Information

Reported by oxmoxtatze on discord
Reported AES output:

Actual flight data:

This was introduced as part of cleanInteger() in the v0.6.8 update.


Doesn't change HandleLidSwitch event

I use arch + bspwm, I have this in my bspwmrc:
killall -q powerkit; powerkit&

I ran powerkit --configure and configured it so that when I close the lid on AC power nothing happens, yet if I close the lid on AC power it goes to sleep. This is on a fresh arch install(I only installed basic utilities, and a WM). I use optimus-manager if that changes anything.

#44


K14130: Reduce RackAwarenesssTest to unit Test

While working on KAFKA-13877, I feel it's an overkill to introduce the whole test class as an integration test, since all we need is to just test the assignor itself which could be a unit test. Running this suite with 9+ instances takes long time and is still vulnerable to all kinds of timing based flakiness. A better choice is to reduce it as a unit test, similar to HighAvailabilityStreamsPartitionAssignorTest that just test the behavior of the assignor itself, rather than creating many instances hence depend on various timing bombs to not explode.
Since we mock everything, there's no flakiness anymore. Plus we greatly reduced the test runtime (on my local machine, the old integration takes about 35 secs to run the whole suite, while the new one take 20ms on average).
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ping @vvcephei for a quick review.

I'm merging in this PR now since it includes a hotfix on [StreamTaskTest.java] to turn the build to green.


remove pool size setting instead of sync.Pool runtime put-get interface;

We found a situation in which your solution proved to be useful.
But limiting the size of the connection pool would be cumbersome for us.
Therefore, we have replaced this setting with sync.Pool.

Thank you for the pull request!
I have checked the content and there is a change to use EncodedConn, so I will prepare an interface that can use EncodedConn.
As for the pool size limit, let me think about it. Even with the existing implementation, there should be no limit on the number of active connections, but on the number of idle connections.
Also, using sync.Pool seems to make it difficult to control the release of *nats.Conn.

Sorry it took me so long to reply.
I just added a method for Encoded Connections GetEncoded/PutEncoded
See here for usage https://github.com/octu0/nats-pool/blob/900bfece3476bd9f730a3a23a77b49f65860351a/pool_test.go#L477-L532
About the pool size, I wonder if you could try ZeroSize Pool
It's not really a pool because it was originally designed to behave like leaky buffer, but you can use it without worrying about the size.
Here's how it's usage https://github.com/octu0/nats-pool/blob/900bfece3476bd9f730a3a23a77b49f65860351a/pool_test.go#L534-L570


Feature complete : mergeback 2d_coords

Master ticket for completing current work on 2d_coords (part of ORCA data support effort).
Rationale :

in the end, the 2d_coords branch will eventually be ok to merge back
in development, we will want to merge in PRs that don't meet our usual standards

incomplete implementation
known problems
temporary limitations
documentation needs fixing
temporary implementation awaiting other outstanding work for a better solution

in these cases, all the outstanding work needs to be recorded in issues marked with the '2d_coords' label.
broken tests can skipped during development using "@skip_2d" #3099
once all open issues+prs labelled 2d_coords are closed (in theory),
everything is up to standard + we can merge back.

but note especially : #3101, #3102
in practice, it will still want a bit of an overall review

Done in #3153.


Is it possible to disable the right click (contextmenu)

Hello, first of all thank you for this great project!
Second, is there a way to configure drawflow to not display the close button on right click on a node? I just want to display the normal browser right button context menu.
Thank you very much and kind regards,
Serafeim

Hello @spapas
Thanks!
For view normal context menu and not display close button.
   editor.contextmenu = (e) => {
            editor.drag = false;
            editor.drag_point = false;
            editor.connection = false;
            editor.ele_selected = null;
            editor.editor_selected = false;
            editor.dispatch('contextmenu', e);
          }

          editor.start();

Add code before start.
View other options: https://github.com/jerosoler/Drawflow/issues/80 https://github.com/jerosoler/Drawflow/issues/47 https://github.com/jerosoler/Drawflow/issues/187

Thank you so much for the quick and perfect solution and the recommendations! I am greateful!
Kind regards,
Serafeim


Documentation unclear tf.batch_to_space

URL(s) with the issue:
https://www.tensorflow.org/api_docs/python/tf/batch_to_space
Description of issue (what needs changing):

Example needs to be added.
crops part in Args: section is very difficult to understand. Needs to be better formatted (put into code).

Clear description:
Documentation is unclear, no example is shown. Also, there is one extremely long paragraph with code in between in the crops section under args.
Parameters defined
Are all parameters defined and formatted correctly?
=> Not well formatted.
Returns defined
Are return values defined?
=> Yes
Raises listed and defined
Are the errors defined?
=> No
Usage example
Is there a usage example?
=> No
Submit a pull request?
=> No

@nikochiko Do you want to work on improving this section? I could not understand from the opened issue if you have only reported an issue or also proposed to work with it? In later case, I can join you in re-formatting the doc for this topic.

@copperwiring Sure, I would like to work on it. But I will have to look into its technical details before that. If you have any resources, it would be great if you share them. I'll try to learn something about it from code in the meanwhile.

@oanush Please assign this to me and @copperwiring . Will open a PR soon.

@nikochiko Feel free to raise a PR and reference this issue. Thanks!

@ymodak this issue be used for Google Code-in as a Documentation and research challenge?

@dubesar I had started working on the formatting part of the issue and was planning to make a PR till tomorrow. The job is mostly to

Sync the docs formatting with its r1 version.
Make the documentation easier to understand, editing it and adding "Examples:" and "Raises:" sections.
I can make some minor edits with the language to make it more suitable as a Google Code-In challenge.
@ymodak , @dubesar ?

@nikochiko Yeah that would be great if some part of this issue is available for Google Code-in

@dubesar See the referenced PR and a comment on it. I have made some additional changes and made the task more clear.

PR merged. Closing issue.


Use joi to run clientside form validations

https://www.npmjs.com/package/joi

Joi had too many dependencies. Used validator instead.


EBS volumes generated for instances should receive tags

My team is using a mixture of hashicorp/terraform-aws-consul, hashicorp/terraform-aws-vault, and our own modules for all other types of instances. We noticed that EBS volumes generated for Consul and Vault instances do not get any tags, so they show up in the UI as having no name. Consequently, it is more difficult to determine at a glance if they are legitimate or actively-used volumes.
For our own modules, we use aws_launch_template rather than aws_launch_configuration, and we put tag_specifications blocks within them like so:
  tag_specifications {
    resource_type = "instance"

    tags {
      Name = "${var....}"
    }
  }

  tag_specifications {
    resource_type = "volume"

    tags {
      Name = "${var....}"
    }
  }

The Terraform docs suggest that the tag_specifications feature is exclusive to aws_launch_template although I hope that you may be able to determine some way to get the same thing in aws_launch_configuration as well.

There's no way in Terraform to tag EBS volumes in an ASG with a launch configuration. The only options are:

Move to launch templates. This would be a significant, backwards incompatible change.
Use a script (e.g., User Data) to tag the EBS Volumes while the server is booting.


Create JBrowse images in gene pages

Probably for @scottcain

Related to this: https://github.com/GMOD/jbrowse/issues/772


Add support to run random graph partition on Linux instead of SageMaker

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing in favor of #764


Extension not working with VS Code insiders 1.56.0 2021-04-22 build

On Today's VS Code

With current extension (Rolling back to an earlier version of the extension doesn't help)

Press F1 and choose NET Interactive: Open notebook results in an error: the full text is Command '.NET Interactive: Open notebook' resulted in an error (command 'dotnet-interactive.openNotebook' not found)

Open a notebook  and the message appears on the status bar to say we are working via the Jupyter server

Try to change Kernel - the list of kernels is

On another machine where VS code has not been updated I see the expected kernel - it was on this machine before VS code updated.

We recommend to use VSCode stable as Insiders has a faster update cycle and the Notebook apis are quickly evolving. the PR #1286 is addressing this cycle of updates but there will likely e some more breaking changes happening in our build for VSCode Insiders

Those are cell languages, have you tried bringing up the kernel selection option using command palette (CTRL+SHIFT+P) ?
you should see an option like this

Then go ahead and you should see some options including .NET Interactive

@colombod   It's the select kernel list

if I click the cell language a selection box flashes and vanishes.

We recommend to use VSCode stable as Insiders

Listen... Hear that...? it's the sound of my palm hitting my face over and over...   When did .NET interactive stop needing insiders ? :-) ?


[Feature Request] Github repo support in commit messages

In case you would manage code in github repo's and work items in Azure Devops, link both can be done using '**AB#**workitem' in your commits. Would be good if the 'add to commit message' action could detect in the (remote) branch is linked to a github repo and based on this add 'AB#workitem' instead of '#workitem' to the commit message.
Reference:
https://learn.microsoft.com/en-us/azure/devops/boards/github/link-to-from-github?view=azure-devops#use-ab-to-link-from-github-to-azure-boards-work-items

@fvet Interesting idea, thanks for bringing it up 👍


Builds from Unity freeze in HMD

I am exporting a working scene from unity, but when I run it it appears on the headset then just freezes (on screen if I have direct mode ticked in my vr cams in project then I can still see the app running and even the head tracking working, but in the headset just a frozen still). I am having to use core snapshot 0.6.1320 because if I go to the newer ones (tried many) I always get back to cannot create render manager. Finding the right combination of unity, osvr core seems almost impossible compared to back when it used to just work!..

I dont know why but this has stopped happening, I could use lower res than hd to stop it but now it isnt happening at all.

Just wanted to voice that, after a fresh Windows install, I had this happen too!
Started up a build in Direct Mode and the screen was frozen. Dropped the resolution and it works fine. Went back up and it started up frozen again.
Will report back if it also magically goes away after some time.

yes I still observe this in "good" or higher quality around 50% of the time.

Yeah, I'm running into the issue again, in both Direct and Extended mode, even when setting the app at a lower resolution. Happens even with the new package build provided in #208.

Playing around with the quality settings, it looks like it may be related to V-sync.
@amiga Can you try disabling V-sync to see if that gets rid of the problem for you as well?  (Edit->Project Settings->Quality; go through each of the Levels and set V Sync Count to "Don't Sync")
I seem to remember reading somewhere that it was recommended to turn V Sync off for VR (possibly OSVR specifically) anyway.

Correct, V-sync should be set to "Don't Sync"

@amiga any updates on this issue? are you using HDK2 or another HMD?
Regarding you original post, there shouldn't be any compatibility issues with OSVR-Core and OSVR-Unity. If Rendermanager fails to create, it's more likely a driver, hardware, firmware, or configuration issue.
Not sure if this helps but FWIW, my workflow is to install the latest OSVR SDK from here, and then download the latest Unity package from here, replacing the one in my existing Unity project by closing the project, opening it again, and importing the package before hitting play. If RM fails to create in Unity, I run the RenderManager example programs included with the SDK to see if it's just a Unity issue or not. Usually if there's an issue, the RM example programs won't work either, then it's a matter of troubleshooting the issues mentioned above. If the RM examples do work and RM still fails in Unity, I test a fresh import of the plugin in a new project.
Definitely make sure your firmware is updated if you are using HDK, there are some known issues with direct mode with older firmware.

HDK2 with newest firmware (hdk2svr-2.00.hex) I am still getting it in good quality or above, I am sticking to "fastest" to avoid it. To be honest I have not tried turning off vsync yet, I need to give that a go.

Confirmed that vsync causes a freeze.


Why this project named Karma?

Why this project named Karma?

https://github.com/karma-runner/karma/issues/376#issuecomment-14577083

thanks maksimr

@kimmking no problems!


Improve support for directories/files in current path...

This works...
fsql "SELECT * FROM ./_src WHERE name LIKE %.js"

this does not...
fsql "SELECT * FROM _src WHERE name LIKE %.js"

Thanks for reporting!
Should be resolved as of d2f0ce05c716a7c178d61bd4d69b0d1e1c08f241, let me know if it's still not working.

Going to keep this open until confirmation that it's fine.

Working... :-)


add sui-node binary paths used in the past

Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Building here: https://github.com/MystenLabs/sui-operations/actions/runs/8180752536/job/22369391144


chore: upgrade faker-js latest

Upgrade faker to latest version
Allow generateHashKey to use arbirary long seed string

Alright after some discussion we decided to:

Import directly the definitions from faker to not break determinism with older versions
Maybe in the future completely delete faker as we only use it to re-export now

@justinvdm I also had to write a custom script for esbuild to build recursively the sources inside src/locales/...
If you're happy with it we can merge and release the new version on npm, we should not have any breaking changes.

Thank you for doing this @avallete! Looks great, happy for it to land 👍


Support array interchange protocols

Investigate how we can support the contour mark in vega-lite from the altair side. See https://github.com/vega/vega-lite/issues/6043#issuecomment-1567460160 for a PoC.
Array-based packages in python have adopted dlpack as array API standard through the .__dlpack__ interface. Potential support through pyarrow might be realized through https://github.com/apache/arrow/issues/33984.
It would be great if we can use facet/row/column by axis/dimension in the array or tensor.

See also: https://github.com/loco-philippe/ntv-numpy which is a new json representation dataformat and standard for multidimensional array data.

See also https://github.com/data-apis/array-api-compat which is a compatibility layer for common array libraries to support the Array API.

@mattijn I know there is more to it than this, but is this able to help at all?

https://github.com/vega/altair/pull/3501

It wouldn't cover the alt.Chart(data=...) part, but maybe there are other places you could throw some arrays at?
Also, pinging @MarcoGorelli as I've seen these:

https://github.com/narwhals-dev/narwhals/blob/ad5616a2c6488c5cb1c5a6dcef71ac00a8c6d65a/narwhals/_polars/series.py#L96
https://narwhals-dev.github.io/narwhals/api-reference/series/#narwhals.series.Series.__arrow_c_stream__


Add Num+ and Num- to key events

Hey Phaser 3 team,
I have noticed that Numpad+ and Numpad- keys are missing from Phaser 3. I am not sure, but maybe adding them to this class here would be sufficient:
https://github.com/photonstorm/phaser/blob/v3.20.0/src/input/keyboard/keys/KeyCodes.js
The reason is that I would like to override ctrl + Num+ and Num- for a zooming functionality of a mathematical software.
Thanks :)

I've added them in and they'll be part of the 3.21 release.
For any other missing keys, you can just specify the keycode yourself when creating a Key object. You can get a list of keycodes from here: https://www.cambiaresearch.com/articles/15/javascript-char-codes-key-codes


Python versions

Codecov Report

Merging #315 (59ffd3c) into master (9e9d5ec) will increase coverage by 1.50%.
The diff coverage is n/a.

:exclamation: Current head 59ffd3c differs from pull request most recent head e81eebd. Consider uploading reports for the commit e81eebd to get more accurate results

@@            Coverage Diff             @@
##           master     #315      +/-   ##
==========================================
+ Coverage   75.37%   76.87%   +1.50%
==========================================
  Files          40       40
  Lines         532      532
  Branches      106      106
==========================================
+ Hits          401      409       +8
  Misses        115      115
+ Partials       16        8       -8

Impacted Files
Coverage Δ

health_check/db/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/cache/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/storage/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/contrib/redis/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/contrib/celery/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/contrib/rabbitmq/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/contrib/celery_ping/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

health_check/contrib/s3boto_storage/__init__.py
100.00% <0.00%> (+33.33%)
:arrow_up:

Continue to review full report at Codecov.

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


[Bug] Cannot find 'image' field view in any of the regular locations.

Bug report
What I did
class CarTypeController extends CrudController
{
    use \Backpack\CRUD\app\Http\Controllers\Operations\ListOperation;
    use \Backpack\CRUD\app\Http\Controllers\Operations\CreateOperation;

    public function setup()
    {
        $this->crud->setModel("App\Models\CarType");
        $this->crud->setRoute("admin/CarTypes");
        $this->crud->setEntityNameStrings('Car Types', 'Car Types');
    }

    protected function setupListOperation()
    {
        $this->crud->setFromDb();

        $this->crud->setColumnDetails('image',[
            'label' => "image,
            'name' => "image",
            'type' => 'image'
        ]);
    }

    protected function setupCreateOperation()
    {
        $this->crud->setValidation(CreateRequest::class);
        $this->crud->addfield([
            'label' => "Image",
            'name' => "image",
            'type' => 'image'
        ]);
    }
}

What I expected to happen
to find image upload filed in create form
What happened

Cannot find 'image' field view in any of the regular locations.
And image.blade.php is not found in fields folder
What I've already tried to fix it
??
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 7.4.27 (cli) (built: Dec 14 2021 19:52:13) ( ZTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
LARAVEL VERSION:
v8.83.1@bddba117f8bce2f3c9875ca1ca375a96350d0f4d
BACKPACK VERSION:
5.0.7@bf44be0ab24d2254bc266609ee697136f1a8bc4a

Hi @0x004d4m ,
This is because the image field is a PRO field. You need to purchase and install backpack/pro in order to use it.
I've opened an issue so that we show clearer error message when a dev tries to do that - see https://github.com/Laravel-Backpack/CRUD/issues/4196. I'm not happy with the solution, but... is seems like it's something we gotta do, I imagine it was very frustrating to see this and have no idea how to fix it.
Cheers!

ok thanks for clarifying that.


storage: add local indexer scheduler

What problem does this PR solve?
Issue Number: ref #9032
Problem Summary:
What is changed and how it works?
Extract LocalIndexerScheduler from https://github.com/tidbcloud/tiflash-cse/pull/203 to reduce the difficulty of code review. Meanwhile, LocalIndexerScheduler::Task::dmfile_ids is changed to LocalIndexerScheduler::Task::page_ids, which means we use page ids to identify each task, so we can add task for ColumnFileTiny later.
storage: add local indexer scheduler

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

LocalIndexerScheduler::Task::dmfile_ids is changed to LocalIndexerScheduler::Task::page_ids

Maybe need some better design (or names), since after logical split the left and right segment will hold the same dmfile_id but different page_id.

Rename to:
struct DMFileID
{
    DMFileID() = default;
    explicit DMFileID(PageIdU64 page_id_)
        : page_id(page_id_)
    {}
    PageIdU64 page_id;
};
// The page id of the ColumnFileTiny.
struct ColumnFileTinyID
{
    ColumnFileTinyID() = default;
    explicit ColumnFileTinyID(PageIdU64 page_id_)
        : page_id(page_id_)
    {}
    PageIdU64 page_id;
};
using FileID = std::variant<DMFileID, ColumnFileTinyID>;

PTAL~

LocalIndexerScheduler::Task::dmfile_ids is changed to LocalIndexerScheduler::Task::page_ids

Maybe need some better design (or names), since after logical split the left and right segment will hold the same dmfile_id but different page_id.

rename to
    // The file id of the DMFile.
    struct DMFileID
    {
        explicit DMFileID(PageIdU64 id_)
            : id(id_)
        {}
        PageIdU64 id;
    };
    // The page id of the ColumnFileTiny.
    struct ColumnFileTinyID
    {
        explicit ColumnFileTinyID(PageIdU64 id_)
            : id(id_)
        {}
        PageIdU64 id;
    };
    using FileID = std::variant<DMFileID, ColumnFileTinyID>;

/test pull-integration-test


Circle lab w/unit tests for review

I resolved conflicts locally, but Pull Request still says 'Can’t automatically merge. Don’t worry, you can still create the pull request.'

Sweet! "This branch has no conflicts with the base branch" - worked

Looks like no conflicts -- yeah!


Can't apply themes

Hi there,
when I'm tryinf to execute npm install -g mtslack
I have the following error:
npm ERR! Error while executing:
npm ERR! /usr/bin/git ls-remote -h -t<EMAIL_ADDRESS>npm ERR! Host key verification failed.

OS: Ubuntu 18.04.4 LTS
Version of Slack: Snap 4.4.2 64-bit

what version of mtslack ?

mtslack

7.3.1

Thanks, I used a fork of a package that did not work in Windows, but it probably worked on my machine because it's my package, never tried from other machines...
Try to reinstall it, it should install v7.4.0, and tell me if that fixes the problem

Thanks for the answer, version v7.4.0 solved my problem


S3 should not connect to cloud metadata server for instance region to get bucket region

Currently, with #58 fix, we always talk to aws metadata server to get initial region to get actual bucket region, this assumption won't apply when user are initiating commands from local mac.
Fix would be:

If bucket object is initiated from inside aws instance, we query the metadata server
If bucket object is initiating from local mac, we guess the partition from dummy region from each partition

Duplicating #113


Prototype of fixing numeric truncation

We'd want tests for BigQueryBigNumeric as well, and I strongly
suspect the implementation should be improved too... but it's a
straw man for discussion.

Urgh - I need to rebase this onto the new gax-v4....


SOAP computation hangs when calc is not None in ASE trajectory

Seems like there's a bug that hangs computations of SOAP descriptors when the input ASE trajectory snapshots have calculators assigned.
Tried with dscribe 1.2.1.
for mol in mol_traj:
        mol.calc=XTB(method="GFN2-xTB")
        ... # some calculations
        mol.calc=None # without this line SOAP calculation hangs...

soap = SOAP(
            species=["H", "C", "O"],
            periodic=False,
            rcut=5.0,
            sigma=0.5,
            nmax=5,
            lmax=5,
            average="outer",
            crossover=True,
            dtype='float64
            )

pos = [0,1,4,5]

derivatives, descriptors = soap.derivatives(
            mol_traj,
            positions=[pos] * len(mol_traj),
            n_jobs=10,
            # method="analytical"
            )

That is quite interesting... DScribe is not directly using the calculator objects for anything, so I'm not sure where this is coming from. DScribe could always unassign the calc-property from the systems, but that does not seem right either if the user is not expecting this. Making a copy is an option but it may be quite expensive to do. Needs some thinking, but the workaround of unassigning the calc-property should do before there is a better fix.

Thanks! I'll also let know if this problem will persist in the future.


How to modify the backgroundColor of the Button when disabled ,

How to modify the backgroundColor of the Button when disabled ,

Check out Issue #152. I'll close out this issue if adding a disableStyle will solve this for you.
Let me know and we can move this discussion over there.

@q351941406 looks like this has been added and will be released with 0.9.4 today!

This feature is available in the latest release of RNE.


Line Number and Position Tracking

Excuse me, I want to track the line number of the code in the grammar analysis（yacc）, but modifying your code according to the official document can not find the current number information in the result. What should I do?
Thank you very much for your help！！！！！！
def p_expression(p):
     'expression : expression PLUS expression'
     p.lineno(1)        # Line number of the left expression
     p.lineno(2)        # line number of the PLUS operator
     p.lineno(3)        # line number of the right expression
     ...
     start,end = p.linespan(3)    # Start,end lines of the right expression
     starti,endi = p.lexspan(3)   # Start,end positions of right expression

I want to know the line numbers of all methods, but I don't know how to modify your code.

I am not sure. Did you enable tracking of line numbers as explained here? I'm not sure if that's necessary but could be worth a shot.

I am not sure. Did you enable tracking of line numbers as explained here? I'm not sure if that's necessary but could be worth a shot.

thans for u help,the problem shas been resolved


Provisioner deletes lambda and updates version rather than update latest alias

It seems like it would be better to keep older versions of the lambda. You could create a new version of the lambda, and then update the latest alias. This would make development easier and make deployment easier.

This was developed a while ago. I'll take a look again. It sounds like the behavior may have been part of an early design choice that no longer applies. Thanks!

Looks like this was done to make updates easier since I don't have to worry about previous versions. Looking into it.

@QuinnCiccoretti I think this does it but I'm out of time to test today. Any chance you could kick the tires on this branch and let me know if it works as expected?

@cea2aj


"Implement default members" on trait methods defined by macro generates invalid code

Using version: fd109fb58 2021-05-10 stable
Code:
macro_rules! define_fn {
    ($name:ident, $ty:ident) => {
        define_fn!($name, $ty <>);
    };

    ($name:ident, $ty:ident < $($param:ident),* >) => {
        fn $name<$($param),*>(
            &mut self,
            _param: &mut $ty <$($param),*>,
        ) -> Self
        where
            $($param: Clone),*
        {
            self.clone()
        }
    };
}
trait SomeTrait: Clone {
    fn explicit_fn(&mut self) -> Self {
        self.clone()
    }

    define_fn!(macro_defined_fn, String);

    define_fn!(macro_defined_parameterized_fn, Vec<u8>);
}

#[derive(Clone)]
struct Foo;

impl SomeTrait for Foo {
// <- "implement default members" here
}

The assist leads to the following invalid code:
impl SomeTrait for Foo {
    fn explicit_fn(&mut self) -> Self {
        crate.clone()
    }

    fnmacro_defined_fn<>(&mutself,_param: &mutString<>,)->Selfwhere{crate.clone()}

    fnmacro_defined_parameterized_fn<u8>(&mutself,_param: &mutVec<u8>,)->Selfwhereu8:Clone{crate.clone()}
}

More specifically:

it loses all the whitespace and concatenates everything together
it leaves empty angle brackets and an empty where clause there (not sure if that is valid rust code?)
no idea where that crate.clone() comes from?

Found this while implementing this trait here:
https://github.com/getsentry/relay/blob/802953b0df8228b7d2810ab73e5c11eca527b630/relay-general/src/processor/traits.rs#L11-L32

What happens if you first expand the macro, then trigger the assist? This is probably more than a single issue.

It's two known issues: 1. whitespace missing in macro expansions, 2. self in self.clone() being 'qualified' to crate. We have issues for both, let me find them...

is #6588, 2. is #7499.


NUI-561 hybrid cli: standalone and aio plugin

For NUI-561.

standalone cli name: adobe-asset-compute
documented all 3 install options: aio, devDependency, global standalone
also list with useful description in aio command list:
"Develop and test Adobe Asset Compute workers"
support -h in addition to --help in standalone mode

Explaining the various options from the readme:
This can be installed & used in three ways:

aio plugin
local devDependency
global standalone cli

Install as aio plugin
This requires aio-cli.
aio plugins:install @adobe/aio-cli-plugin-asset-compute

It provides the asset-compute command topic. You can run e.g. the test command using:
aio asset-compute test-worker

Install as local devDependency
Install into your devDependencies:
npm install --save-dev @adobe/aio-cli-plugin-asset-compute

Then use in npm scripts in your package.json, for example for tests:
   "scripts": {
       "test": "adobe-asset-compute test-worker"
   }

Use npx if you want to manually run it. Inside your project run for example:
npx adobe-asset-compute run-worker

Install as global standalone cli
Install globally using:
npm install -g @adobe/aio-cli-plugin-asset-compute

This will add the adobe-asset-compute cli to your system. Run using:
adobe-asset-compute test-worker

This looks good! I verified help output between various ways of running.
🔥🐝 adobe-asset-compute --help
Tool to develop and test Adobe Asset Compute workers

VERSION
  @adobe/aio-cli-plugin-asset-compute/1.0.3 darwin-x64 node-v10.16.1

USAGE
  $ adobe-asset-compute [COMMAND]

COMMANDS
  run-worker   Run worker from local project using Docker
  test-worker  Run tests from local project

🔥🐝 aio asset-compute --help
Develop and test Adobe Asset Compute workers

USAGE
  $ aio asset-compute

OPTIONS
  -v, --verbose  Verbose output
  --version      Show version

COMMANDS
  asset-compute:run-worker   Run worker from local project using Docker
  asset-compute:test-worker  Run tests from local project

Also, slight improvement: In standalone mode, maybe add something that displays the version, in case people report bugs (you know, similar to ènvinfo`). The standalone-cli version would be the bare minimum to start investigating! (But that can be done in another PR too).

@tmathern

Error: ENOENT: no such file or directory, open 'manifest.yml'

I actually deliberately had that error that way, since I thought it would explain the problem very well (you have to run it in a dir with a manifest.yml). There is an existing unit test that checks for something with "manifest.yml" in the error message. In any case, this is not something this PR touches, so if you have a suggestion for a more helpful message, let's do it in a separate PR/issue.

tests fail on Windows

Let's address this in a separate PR, since this one doesn't assume to fix any Windows issues.

maybe add something that displays the version

It's right in the help output:
Tool to develop and test Adobe Asset Compute workers

VERSION
  @adobe/aio-cli-plugin-asset-compute/1.0.3 darwin-x64 node-v13.12.0

USAGE
  $ adobe-asset-compute [COMMAND]

COMMANDS
  run-worker   Run worker from local project using Docker
  test-worker  Run tests from local project

I would prefer to not create a full blown cli. Also this is still an experiment...

Ttoally missed the version info in the help output O_O.
Otherwise LGTM.


Fix unterminated quoted string in bootstrapping script.

A recent commit introduced an unquoted string, breaking the bootstrapping shell script.

Thanks for the quick PR!


Evict executions on unresponsive executors (by default)

See discussion in https://github.com/azkaban/azkaban/pull/1661#issuecomment-431197754.
However, I don't think the problem is in the application design. If an executor instance is removed, the deployment strategy should make sure that the corresponding executor entry is deleted from the DB. Or then there's some actual bug that I haven't seen before.
Any way, enabling evicting again for now. Users (like me) can set azkaban.evictUnresponsiveExecutions=false to keep the previous behaviour.
There is actually one nice-to-have improvement to be done before stopping evictions by default again: automatic refreshing of removed executors from the DB. It was blocked by some preliminary work before, but now this is available to be built upon: https://github.com/azkaban/azkaban/pull/1975#discussion_r225306559.

@HappyRay here it is

@HappyRay is this still urgent?

@HappyRay ref
https://github.com/azkaban/azkaban/pull/1661#issuecomment-431197754, I thought you were in a need of this change. Did you find a way to live with the current code maybe?

We decided to try an alternative fix first #2007
The deployment is taking place. So far so good. But it takes a while for the symptoms to show up.
This change will be our backup plan.

@HappyRay this should provide an even better alternative: https://github.com/azkaban/azkaban/pull/2016. I wouldn't want to see additional delay in executor shutdown. Indeed https://github.com/azkaban/azkaban/pull/2016 makes it totally redundant.

@chengren311 does the #2007 slow down the normal shutdown process too?


Adding a Github Action for greeting

Adding a Github Action for greeting

Hey, @arpit456jain assigning to you.


Implement set of SCPs to protect overlay elements from modification

Describe the issue
No technical controls (Service Control Policies) exist to protect overlay elements (parts of the landing zone that CPF adds on top of the landing zone). We want to implement a set of technical controls as identified by #784
Additional context

This is about protecting team members' access by not accidentally deleting components they need

Definition of done

Controls implemented and tested with layer automation tools (LZ1)
Tags for Client VPN overlay layers have been created

Moving to blocked, the tag based approach complete but untested. I have added a dependency on #787 so that we can test in LZ1 before deploying on LZ2.

Part of the 1.5 upgrade, blocked until then

Waiting for 1.5

getting stale, questionable relevance...


How to disable autofocus?

Hi, how can I disable autofocus after clone element? Not sure what to do with “focusableElement“.
Thank you for the great plugin.

Set focusableElement: false seem working fine.


Reactivate toolkit perception checks and parameterizaton equivalence tests

Automate the following into Travis:

@j-wags 's molecule-loading equivalence tests: https://github.com/openforcefield/openforcefield/blob/master/examples/deprecated/rdkit_openeye_comparison/compare_rdk_oe_mol_loading.ipynb
@swang 's  OE+RDK parameter application test: https://github.com/openforcefield/openforcefield/blob/swang/examples/forcefield_modification/RDKitComparison.ipynb

We don't expect them to produce identical results 100% of the time (I mean, it would be nice, but it's likely that significant modifications to RDKit would be needed to get it matching OETK for all molecules). But we do want to have some continuously-updated measure of the toolkit's current performance, loud alarm bells in the case that we make changes that drop this number, and the ability to expand the test set to new relevant chemistry.
Relevant: The FreeSolv and AlkEthOH tests DO check parameter application, but those datasets are only available in mol2 format, so they're only performed using the OpenEye backend. A really good test would be to carry these tests out using RDKit as well (Maybe we could convert the datasets to SDF using OpenBabel, though that does add another layer of potential bugginess)

rdkit does have mol2 readers, but only supports one molecule per file. However, one message in the mailing list suggested to go through the Mol2 block and splitting into individual chunks manually (see attachment here). Is there any other feature we are missing in the parser?

Because there is no standard mol2 spec, RDKit only supports reading Corina-format mol2 files, while OpenEye supports Tripos-format mol2. I think the lack of a well-defined standard is a major impediment to widely-supported mol2 parsers.

Relevant: The FreeSolv and AlkEthOH tests DO check parameter application, but those datasets are only available in mol2 format, so they're only performed using the OpenEye backend.

If this is an issue, we can convert these to SDF with the OpenEye toolkit and make those datasets available instead.

Agree these are high priority, and we should just document any discrepancies and then, long-term, work on getting the discrepancies tracked down and fixed. But separately.
And yes, we should just convert the mol2 files we're using to sdf so we can cross-compare with both toolkits. Avoid the RDKit mol2 parser as it has many issues.

Linking to another discussion on mol2 in a similar OSS toolkit: https://github.com/Acellera/htmd/issues/847


Attribute Error :  'module' object has no attribute 'table'

I am trying to use the query instead of model. But it throws the following error,
Query : db.table('users').select('id').get()
AttributeError: 'module' object has no attribute 'table'

Strange. What's the exact code that triggers the error (how is the DatabaseManager initialized?)

Sorry, issue on my side. I rectified.


⚠ Bofrost.at has degraded performance

In 5e4e921, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2727 ms

Resolved: Bofrost.at performance has improved in 13b8dc4 after 49 minutes.


AEM 6.2 problems

Hi, the extension seams to have problem syncing with the newest version of Adobe Experiance Manager 6.2. Anyone tried this with the latest version? 6.2 came out this year, and the plugin have not had an release sine December last year.

@Dankojo I am using the lastet version. Syncing text files, html, js, css, jsp ... works fine. I am having trouble with .content.xml files though. I did not start investigating, yet.

Hi @neslob
Have a look to https://docs.adobe.com/docs/en/crx/2-3/how_to/how_to_use_the_vlttool.html

The VLT Sync feature only supports simple files and folders but detects special vault serialized files (.content.xml, dialog.xml, etc) and ignores them silently. Thus it is possible to use vault sync on a default vlt checkout.

You can still right clic on those files and "Export to server" manually

thanks @josebercianowhitbread
The problem is that once I save a .content.xml file. Autosync kicks in and breaks my sync.
I have to manually

close brackets
force quit a brackets-node process (running @100%)
start brackets

also having issues syncing with 6.2. syncing never starts

Thanks for the answers, but there must be a better way to work with
front-end development and AEM. The process with brackets and it crashing
not syncing all files and manually have to sync files is so time consuming.
On Thu, 9 Mar 2017, 19:21 Saulo Venancio<EMAIL_ADDRESS>wrote:

also having issues syncing with 6.2. syncing never starts
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Adobe-Marketing-Cloud/aem-brackets-extension/issues/78#issuecomment-285435556,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACwdOP9itTefpKLae1Py7maNuOY3clHMks5rkEMzgaJpZM4KdIxU
.

I'm using Brackets 1.7 and AEM 6.1 and having the same issue. An entire client library wasn't syncing to AEM until I would manually export the content.xml file to server.


[Feature Request] send errors to a quickfixlist

I love this plugin but I wish that there was a way to handle errors, something like vim-dispatch.

This is something that I wanted to implement, I do not know how to do it, but if you can help me I would be grateful, you could venture with PR.But I'll try to do it

I was working on implementing this using the terminalopen() function but it seem there is a but in the way it handles stderr. So I will open an issue on the neovim repo and when it will be fixed we can work on having error handling.

Opened an issue on neovim repo. I will give update on changes.

Thank you, I would really appreciate a pr for this


cub::TilePrefixCallbackOp::WarpReduce doesn't permit ptx arch specialization

cub::TilePrefixCallbackOp should also accept ptx arch as its template argument, to specialise WarpReduce<T> (https://github.com/NVlabs/cub/blob/1.5.5/cub/agent/single_pass_scan_operators.cuh#L658)

Fixed in the above commit for v1.6.3


Confusing parts of VisibleTimeRange API

The TimeBoundary requires passing in a TimeInt.

In our set_time_seconds APIs, we allow floating point seconds values.
At the moment if you mistakenly pass a float in seconds, it's converted to an int implicitly (and incorrectly)
Proposal: make this require a rwarg: seq, seconds, or nanos

API doesn't make it clear you can't specify a time range more than one

Because the input takes the form of a list, it's not clear that you can't do:

    time_ranges=[
        rrb.VisibleTimeRange(
            "time",
            start=rrb.TimeRangeBoundary.absolute(0),
            end=rrb.TimeRangeBoundary.absolute(10),
        ),
        rrb.VisibleTimeRange(
            "time",
            start=rrb.TimeRangeBoundary.absolute(20),
            end=rrb.TimeRangeBoundary.absolute(30),
        ),
    ]

as a user I might expect this to give me the union of the two ranges. Instead I believe it's simply first-value-wins for the timeline key.

At a minimum we should probably warn on duplicate values here.

nothing to add to the first point and I think it's quite important to improve this as suggested.

API doesn't make it clear you can't specify a time range more than one

it is documented that it's a first value wins situation /// If a timeline is listed twice, the first entry will be used..
We could look into making this also take a dictionary but that gets tricky to map out to the existing data structures. I'm not convinced yet this is that much of a problem to justify a lot more time spent on it.

it is documented that it's a first value wins situation

Not in the doc visible in the editor:

This other thing I was thinking we could do would just be scan them and print a warning.


The LLVM optimization option O3 on Triton side cause some GEMM test case fail.

The LLVM optimization option O3 on Triton side cause some GEMM test case fail because of the in-correct result.
Some part of the output results are set to 0 instead the expected value while the other part are correct.
The test case is test_op|[16-16-16-1-1-2-None-None-None-False-False-float16-float16-True-True-None-None|] of test_matmul.py
If the LLVM optimization option O0 is used, the issue doesn't happen.

@chengjunlu this is likely caused by the InstCombine pass in LLVM. I discovered this for a different variation of the workload: https://github.com/intel/intel-xpu-backend-for-triton/issues/381#issuecomment-1919271813.

Same as #381, blocked by a problem in the IGC compiler.

This issue duplicates

#381

I'd like to keep this open until we can verify that the -O3 IGC problem is fixed.

I cannot reproduce this issue with the latest Triton on XPU.
Not sure which changes fixed this issue. But I think we can close this issue for now since it is not reproduced.

Reopen the issue. I just found I didn't enable the DPAS correctly in previous test.

Has double confirm the test case passed on the latest Triton with the DPAS instruction. This issue cannot be reproduced.
Let's close this issue.


Fix incorrect check which always yields true

I might be overlooking something trivial here, but I can't see any way where length is not a number?
It's initialized as 0, so when you pipe an HTTP stream to it, it'll still be 0, and it'll just fall back and not try to read the length from the headers. Surely this isn't how it was supposed to be? :-)

Hate to "bump", but any thoughts on this?

Don't mean to rush you but any chance of getting this addressed soon? ;)


Dropped oauth table

refs https://github.com/TryGhost/Toolbox/issues/309

we're removing the OAuth prototype and the table was never used, so we
should be good to drop it
this commit adds a migration to drop the table

@allouis Yep, there's a bunch of cleanup but the table is disconnnected from the rest of the implementation so we should be able to get this in first


process.env.NODE_ENV is undefined

Version
3.0.0-beta.5
Reproduction link
https://codepen.io/anon/pen/yKBxvQ
Steps to reproduce
1 .vue.config.js
module.exports = {
baseUrl: process.env.NODE_ENV === 'production' ? "{{.urlPrefix}}" : "debug",
}
2 .run vue-cli-service build --mode production in command line
What is expected?
run vue-cli-service build --mode production in command line ; process.env.NODE_ENV === 'production'
What is actually happening?
process.env.NODE_ENV === undefined

Um...seems to be a bug, user options is loaded before NODE_ENV initialized.

We might have an architectural issue here. The build command (which is setting the production mode) is implemented as a plugin (like all other commands), and plugins receive the loaded config in their arguments - so the config has to be loaded before the build command can be run.
This might not be a simple change this line type of bugfix.

As a workaround, I remember putting NODE_ENV=production in front of the existing build command

@LinusBorg , Yes, I managed to find a clean fix and realized it's not that simple...Actually we can pass all user arguments in to Service when calling its constructor. However commands like serve and build have their own default  arguments, which can not simply get during Service initialize stage.

I'm using a similar way as @xvaara but match the process.env.npm_lifecycle_script to get the mode value, and use Dotenv to load the .env.[mode] file at the vue.config.js 's beginning.


Autocomplete folder- and filenames

It's not possible to tab-autocomplete filenames nor foldernames. It really slows you down when doing cd, mv, cp etc.

Sorry I didn't see another issue was already opened.


Encryption with 256-bit key

If pdf is encrypted with 256-bit key and empty owner-password, it can be viewed with Preview app (Mac) without password. Adobe Reader asks for password.
--encrypt "secret" "" 256 --
If pdf is encrypted with 128-bit keys and empty owner-password, Preview app asks for password to view.
--encrypt "secret" "" 128 --use-aes=y --
(Version: 6.0.0)

This is not a bug with qpdf. This behavior is actually specified by the PDF specification. In the PDF spec version 1.7, it's algorithm 3.3, step 1. In the ISO standard, it's item a of algorithm 3 in section <IP_ADDRESS>. It states that if the owner password is empty, the user password should be used in its place. This algorithm applies to 128-bit and 40-bit encryption. In those cases, the file is encrypted with a key generated by the user password, and the user password is encrypted using the owner password.
For 256-bit encryption, the user and owner passwords can be used independently to decode an encryption key used for the whole file, and there is no special case around empty owner passwords.
What qpdf and many other readers do is to attempt an empty password if the file is encrypted and no password is given. Perhaps Adobe Reader doesn't attempt the empty string as an owner password.

Thank you for the explanation.  Unfortunately, I suspect many users of qpdf do not realize that encrypting a pdf using AES-256 encryption with only a user password (and a blank or null owner password) will create very insecure PDF's (unlike PDF's encrypted using 128-bit AES or RC4).  How can such PDF's be secure when many PDF readers including Apple's Preview will open it up, decrypt it and display it without asking for a password!
I recommend that qpdf generate a warning message when encrypting PDF's using AES256 whenever the owner password is blank or null informing users that the resulting encrypted PDF is not very secure since the owner password has not been specified.
Since this is closed, I will open up a new feature request.

Thank you for the explanation.  Unfortunately, I suspect many users of qpdf do not realize that encrypting a pdf using AES-256 encryption with only a user password (and a blank or null owner password) will create very insecure PDF's (unlike PDF's encrypted using 128-bit AES or RC4).  How can such PDF's be secure when many PDF readers including Apple's Preview will open it up, decrypt it and display it without asking for a password!
I recommend that qpdf generate a warning message when encrypting PDF's using AES256 whenever the owner password is blank or null informing users that the resulting encrypted PDF is not very secure since the owner password has not been specified.
Since this is closed, I will open up a new feature request.

Please note that it appears that one can create an encrypted PDF using AES256 encryption with only specifying a user password using other tools (like the free PDFEncrypt for Windows) that will NOT open in Apple's Preview without specifying the user password.
And if you open such a PDF up using Adobe Reader (or use pdfinfo from poppler), one can verify that only the user password is set but not the owner password (or permissions password).
I really do not know how tools like PDFEncrypt create encrypted PDF's using AES256.  But it appears to be a more secure way than the way that qpdf currently employs.  You may want to see how others do it.
In my view, if the official PDF spec is brain dead, it probably does not make sense to follow it!

Please note that it appears that one can create an encrypted PDF using AES256 encryption with only specifying a user password using other tools (like the free PDFEncrypt for Windows) that will NOT open in Apple's Preview without specifying the user password.
And if you open such a PDF up using Adobe Reader (or use pdfinfo from poppler), one can verify that only the user password is set but not the owner password (or permissions password).
I really do not know how tools like PDFEncrypt create encrypted PDF's using AES256.  But it appears to be a more secure way than the way that qpdf currently employs.  You may want to see how others do it.
In my view, if the official PDF spec is brain dead, it probably does not make sense to follow it!

It looks like PDFEncrypt sets the owner password to the same value as the user password when encrypting a PDF using AES256 whenever the owner password is not specified.  As a result, one must enter a password whenever one tries to open such a PDF using any PDF Reader (including Apple's Preview).
I also find it interesting that Adobe Reader shows that the user password is set BUT not the owner password (which is not technically true) for a PDF encrypted in this way.
I tried encrypting a PDF using qpdf with AES-256 encryption when both the user and owner password are set to the same non-empty value.  Adobe Reader also shows that the user password is set BUT not the owner password (which is not technically true).
I recommend that qpdf set the owner password to the same value as the user password whenever the user fails to provide an owner password (or if the owner password is empty) when qpdf is encrypting a PDF using AES-256.  Otherwise, the encrypted PDF's will NOT be secure at all since they can be opened up in various PDF Readers including Apple's Preview.

It looks like PDFEncrypt sets the owner password to the same value as the user password when encrypting a PDF using AES256 whenever the owner password is not specified.  As a result, one must enter a password whenever one tries to open such a PDF using any PDF Reader (including Apple's Preview).
I also find it interesting that Adobe Reader shows that the user password is set BUT not the owner password (which is not technically true) for a PDF encrypted in this way.
I tried encrypting a PDF using qpdf with AES-256 encryption when both the user and owner password are set to the same non-empty value.  Adobe Reader also shows that the user password is set BUT not the owner password (which is not technically true).
I recommend that qpdf set the owner password to the same value as the user password whenever the user fails to provide an owner password (or if the owner password is empty) when qpdf is encrypting a PDF using AES-256.  Otherwise, the encrypted PDF's will NOT be secure at all since they can be opened up in various PDF Readers including Apple's Preview.

@RobK88 did you read Encryption? The last paragraph explains your problem.

@RobK88 did you read Encryption? The last paragraph explains your problem.

Thanks.  Yes, the docs do explain that one can create a PDF with a blank or empty password using qpdf.  No doubt qpdf is a very powerful tool.
But my point is trying to protect the average user.  If one creates an encrypted PDF using 128bit AES encryption with a user password and an empty owner password, the resulting encrypted PDF is secure (as long as you used a good password).  All the PDF readers (including Apple's Preview) will require a password before decrypting and opening the PDF.  But the minute you try to encrypt the same PDF using AES256 encryption, the resulting PDF will be encrypted BUT not secure at all if the owner password is left blank.  Some PDF Readers like Apple's Preview can open it up without requiring a password at all!  This is NOT obvious.  I found out about this quirk by mistake.
The developer, Jay,  is just following the crazy PDF spec.  But to protect the average user, there should be at least a WARNING when one tries to encrypt a PDF using AES256 when the owner password is left empty.  (e.g.  "The encrypted PDF is NOT secure when owner password is empty".).  Better yet, prevent the user from even creating such an encrypted PDF using AES256 unless the user specifies an over-ride flag on the command line (e.g. insecure-pdf=y).
This issue only applies to PDF's encrypted using AES256 encryption due to the way the PDF spec is written.

Thanks.  Yes, the docs do explain that one can create a PDF with a blank or empty password using qpdf.  No doubt qpdf is a very powerful tool.
But my point is trying to protect the average user.  If one creates an encrypted PDF using 128bit AES encryption with a user password and an empty owner password, the resulting encrypted PDF is secure (as long as you used a good password).  All the PDF readers (including Apple's Preview) will require a password before decrypting and opening the PDF.  But the minute you try to encrypt the same PDF using AES256 encryption, the resulting PDF will be encrypted BUT not secure at all if the owner password is left blank.  Some PDF Readers like Apple's Preview can open it up without requiring a password at all!  This is NOT obvious.  I found out about this quirk by mistake.
The developer, Jay,  is just following the crazy PDF spec.  But to protect the average user, there should be at least a WARNING when one tries to encrypt a PDF using AES256 when the owner password is left empty.  (e.g.  "The encrypted PDF is NOT secure when owner password is empty".).  Better yet, prevent the user from even creating such an encrypted PDF using AES256 unless the user specifies an over-ride flag on the command line (e.g. insecure-pdf=y).
This issue only applies to PDF's encrypted using AES256 encryption due to the way the PDF spec is written.


CI fails in x86 windows github runners

Downloading uv 0.5.1 (x86_64-pc-windows-msvc)
Installing to C:\Users\runneradmin\.local\bin
  uv.exe
  uvx.exe
everything's installed!

To add C:\Users\runneradmin\.local\bin to your PATH, either restart your system or run:

    set Path=C:\Users\runneradmin\.local\bin;%Path%   (cmd)
    $env:Path = "C:\Users\runneradmin\.local\bin;$env:Path"   (powershell)
Error: Unable to locate executable file: uv. Please verify either the file path exists or the file can be found within a directory specified by the PATH environment variable. Also verify the file has a valid extension for an executable file.```

It's fixed here #99 check this one https://github.com/yezz123/authx/actions/runs/11806947315/job/32892644653?pr=698


Run evaluation on PP4AV with same approach as evaluation results in the paper

Hi,
I have read the research paper, and evaluated license & face model on PP4AV dataset and didn't get matching results, AP/AR for face models not even close to 40%. I was using https://github.com/rafaelpadilla/review_object_detection_metrics for the evaluation.
I did not find a way to evaluate custom model using your filtering approach for smaller bounding boxes. Is there a script that is usable to run against PP4AV GT detections in format (YOLO or COCO) and get results AP/AR in the same way you have shown in the research paper to produce results that can be compared.
Thank you for research and creating the first step towards a good benchmarking dataset for anonymization in the traffic environment.

Thanks @Zeleni9 for your interesting.
You can find the script to filter smaller bboxes here:
https://github.com/khaclinh/pp4av/blob/main/tools/data_processing/filter_box_threshold.py
In script will work with YOLO data format.
In this repo, you can find several scripts for evaluation. We provide the script to evaluate performance with COCO format:
https://github.com/khaclinh/pp4av/blob/main/evaluations/baseline_evals/main.py
If your model predicted results in YOLO format, we provide script to convert it into COCO format for evaluation:
https://github.com/khaclinh/pp4av/blob/main/tools/data_processing/format_transform.py
Finally, we also provide script (in Matlab) to generate PR curves as WIDER-FACE format.

You can find how to organize the ground truth and prediction make compute it here:
https://github.com/khaclinh/pp4av/tree/main/evaluations/widerface_evals
I hope these guide could be useful to you.
If you have any error or issue while executing script, please feel free to share it with me.
Thanks and best,

@Zeleni9
Do you make your predicted result in YOLO format?

Thank you for fast response and great explanation, yea I currently writing it in YOLO format but in absolute values. I have pp4av in ground truth:
plate 401 728 490 752
plate 792 630 826 639
and detections:
face 0.5021612048149109 1154 584 1170 605
plate 0.3836093842983246 707 654 774 669


[Kubernetes] No configmap created with leader_candidate: true if namespaces are not defined

**Output of the info page **
kubectl exec -it  dd-agent-3dmzk /etc/init.d/datadog-agent status --namespace=kube-system                                                                                 ✔  2714  15:16:37
Warning: Known bug in Linux Kernel 3.18+ causes 'status' to fail.
Calling 'info', instead...
2017-09-13 14:16:49,337 | WARNING | dd.collector | utils.service_discovery.config(config.py:31) | No configuration backend provided for service discovery. Only auto config templates will be used.
====================
Collector (v 5.17.0)
====================

  Status date: 2017-09-13 14:16:31 (18s ago)
  Pid: 24
  Platform: Linux-4.4.78-k8s-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/collector.log

  Clocks
  ======

    NTP offset: -0.0044 s
    System UTC time: 2017-09-13 14:16:49.472537

  Paths
  =====

    conf.d: /etc/dd-agent/conf.d
    checks.d: /opt/datadog-agent/agent/checks.d

  Hostnames
  =========

    ec2-hostname: ip-172-20-56-149.eu-west-1.compute.internal
    local-ipv4: <IP_ADDRESS>
    socket-hostname: dd-agent-3dmzk
    hostname: i-03f748ace24133fb6
    local-hostname: ip-172-20-56-149.eu-west-1.compute.internal
    instance-id: i-03f748ace24133fb6
    socket-fqdn: dd-agent-3dmzk

  Checks
  ======

    ntp (5.17.0)
    ------------
      - Collected 0 metrics, 0 events & 0 service checks

    disk (5.17.0)
    -------------
      - instance #0 [OK]
      - Collected 32 metrics, 0 events & 0 service checks

    kubernetes (5.17.0)
    -------------------
      - instance #0 [OK]
      - Collected 133 metrics, 0 events & 3 service checks

    docker_daemon (5.17.0)
    ----------------------
      - instance #0 [OK]
      - Collected 90 metrics, 0 events & 1 service check

  Emitters
  ========

    - http_emitter [OK]

2017-09-13 14:16:51,531 | WARNING | dd.dogstatsd | utils.service_discovery.config(config.py:31) | No configuration backend provided for service discovery. Only auto config templates will be used.
====================
Dogstatsd (v 5.17.0)
====================

  Status date: 2017-09-13 14:16:44 (6s ago)
  Pid: 21
  Platform: Linux-4.4.78-k8s-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/dogstatsd.log

  Flush count: 152
  Packet Count: 1732
  Packets per second: 1.0
  Metric count: 10
  Event count: 0
  Service check count: 0

====================
Forwarder (v 5.17.0)
====================

  Status date: 2017-09-13 14:16:51 (1s ago)
  Pid: 17
  Platform: Linux-4.4.78-k8s-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/forwarder.log

  Queue Size: 1618 bytes
  Queue Length: 1
  Flush Count: 502
  Transactions received: 387
  Transactions flushed: 386
  Transactions rejected: 0
  API Key Status: API Key is valid

======================
Trace Agent (v 5.17.0)
======================

  Pid: 16
  Uptime: 1559 seconds
  Mem alloc: 845744 bytes

  Hostname: dd-agent-3dmzk
  Receiver: <IP_ADDRESS>:8126
  API Endpoint: https://trace.agent.datadoghq.com

  Bytes sent (1 min): 0
  Traces sent (1 min): 0
  Stats sent (1 min): 0

Additional environment details (Operating System, Cloud provider, etc):
Debian GNU/Linux 8 (jessie)   4.4.78-k8s, Kubernetes 1.7.4, AWS
Steps to reproduce the issue:

Deploying the following datadog daemonset to the k8s cluster:

apiVersion: extensions/v1beta1
kind: DaemonSet
metadata:
  name: dd-agent
  namespace: kube-system
spec:
  updateStrategy:
    type: RollingUpdate
  template:
    metadata:
      labels:
        app: dd-agent
      name: dd-agent
    spec:
      containers:
      - image: datadog/docker-dd-agent:latest
        imagePullPolicy: Always
        name: dd-agent
        ports:
          - containerPort: 8125
            name: dogstatsdport
            protocol: UDP
        env:
          - name: API_KEY
            value: ********
          - name: KUBERNETES
            value: "yes"
          - name: SD_BACKEND
            value: docker
          - name: KUBERNETES_LEADER_CANDIDATE
            value: "True"
          - name: KUBERNETES_LEADER_LEASE_DURATION
            value: "60"
          - name: KUBERNETES_NAMESPACE_NAME_REGEX
            value: "'.*'"
        resources:
          requests:
            memory: "128Mi"
            cpu: "100m"
          limits:
            memory: "512Mi"
            cpu: "250m"
        volumeMounts:
          - name: dockersocket
            mountPath: /var/run/docker.sock
          - name: procdir
            mountPath: /host/proc
            readOnly: true
          - name: cgroups
            mountPath: /host/sys/fs/cgroup
            readOnly: true
      volumes:
        - hostPath:
            path: /var/run/docker.sock
          name: dockersocket
        - hostPath:
            path: /proc
          name: procdir
        - hostPath:
            path: /sys/fs/cgroup
          name: cgroups

Wait 5 minutes, then

kubectl get configmaps --namespace kube-system

Describe the results you received:
No  datadog-leader-elector configmap present, no kubernetes events showing up in Datadog events
Describe the results you expected:
NAMESPACE     NAME                                 DATA      AGE
kube-system   datadog-leader-elector               0         5m
Kubernetes events starts to show up in Datadog
Additional information you deem important (e.g. issue happens only occasionally):
If I override the /etc/dd-config/conf.d/kubernetes.yaml.example in the datadog/docker-dd-agent:latest config
Dockerfile:
FROM datadog/docker-dd-agent:latest

COPY ./kubernetes.yaml.example /etc/dd-agent/conf.d/kubernetes.yaml.example

kubernetes.yaml.example diff:
--- /Users/lonodi/dev/work/kubernetes.yaml.example.orig	Wed Sep 13 15:26:03 2017
+++ /Users/lonodi/dev/work/kubernetes.yaml.example	Wed Sep 13 14:46:03 2017
@@ -113,8 +113,8 @@
   # The namespaces for which events should be collected.
   # If not modified, the 'default' namespace will be used.
   #
-  # namespaces:
-  #  - default
+   namespaces:
+    - default

   # The regexp used to select namespaces for which events should be collected.
   # The matched namespaces will be added to the "namespaces" list.

And recreate the Daemonset, it start to work, Configmap is created and events are showing up for all namespaces.

Hi @lorantonodi ,
To diagnose this issue we will need a debug flare and your modified kubernetes.yaml.
Could you please open a support ticket with<EMAIL_ADDRESS>and send them a debug flare? This will allow us to see if there are any permission issues going on.
Regards

Hey @xvello,
Sure thing.

@lorantonodi or @xvello
How did this issue get resolved? I am seeing the same thing in the latest version of dd-agent

@Boojs this was fixed by moving the agent to a monitoring namespace, and amending the serviceaccount / clusterrolebindings / daemonset to reference this namespace. The kube-system namespace is reserved for kubernetes and users creating objects there is not recommended.
Our helm chart supports running in a custom namespace when deployed with the --namespace option.


Update vpn_customer_gateway.html.markdown

To give a hint of the notation of the DH Group.

LGTM! Thanks @snooops!


FileActivatedEventArgs should have an optional NeighboringFilesSortOrder property

Proposal: FileActivatedEventArgs should have a NeighboringFilesSortOrder property
Summary
The FileActivatedEventArgs class should have a property NeighboringFilesSortOrder which has a type equivalent to Windows.Storage.Search.SortEntry, and it should be nullable since it's optional. This would be set by the launcher (e.g. File Explorer) when launching an app with a single file, in order to suggest an initial ordering of neighboring files. For example if a file is double-clicked in File Explorer, then File Explorer will provide the file path along with current sort order of the directory (maybe only if it is not in the default name-ascending order).
Rationale

In UWP the FileActivatedEventArgs class has a NeighboringFilesQuery property to access the neighboring files. There are many problems with this, in addition to the fact that it suffers from the slow speed of creating StorageFiles. For example, it seems to use the indexer, so it is unreliable and frequently misses all or some files in the directory. It's also awkward to save it for later use etc. However, one good thing about it is that when provided by File Explorer, it uses the current sort order of the folder, which is intuitively what the user expects. (I have seen a user sort photos by date in File Explorer then open the first one by double-click and rely on the fact that the app loads the photos in the same order). Currently I don't think there is a way of maintaining this behavior without the neighboring files query. Hence the solution of adding a NeighboringFilesSortOrder property.

Scope

Capability
Priority

This proposal will allow developers to (mostly) maintain the ordering functionality offered by UWP's NeighboringFilesQuery without having to use NeighboringFilesQuery  (assuming the necessary file access permissions)
Must

This proposal will allow the launcher to signify the intent of loading neighboring files, either by the presence of NeighboringFilesSortOrder or an additional boolean property.
Could

This proposal will change file access permissions
Won't

Important Notes
This would require cooperation of the launching app - most importantly, File Explorer would probably need to be updated to provide this property.
There is another discussion to be had on file access permissions (that's another reason why NeighboringFilesQuery exists, and I think a lot of us would like to be able to work around that problem) but it should happen separately.
Open Questions
Should there be another property bool IncludeNeighboringFiles or similar, to suggest that neighboring files should be loaded (just as a hint to the app being launched)? This could be inferred from the presence of NeighboringFilesSortOrder, but it might be best to allow that to be null to indicate no preference in the sort order, but still allow the indication that neighboring files should be loaded.
Should NeighboringFilesSortOrder actually be of type IVectorView<SortEntry>, i.e. a list, which covers the most general case of ordering? This would match QueryOptions.SortOrder.

Hi @benstevens48  - my apologies for the delay in replying. Thanks for the suggestion. I've added it to the backlog for consideration for a future release. Internal ID for reference is 37595633.


update dev release to 3

Issue #, if available:
Description of changes:
continuation of https://github.com/aws/eks-distro/pull/713
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/override kubernetes-1-22-presubmit
Ran it already here, so this is just incrementing the number: #713