Tag values not visible from Kapacitor Template

Hi,
We are using Template and YAML file to create alerts. Alerts are working fine but messages is missing information related to tags.
Following is the template that I have created
// /usr/share/kapacitor/templates/database_template.tick

var measurement string
var where_filter = lambda: TRUE
var groups = [*]
var field string
var warn lambda
var crit lambda
var period_val duration
var every_val duration
var email_id string
var message_val = '{{.Level}}: {{.TaskName}} at {{ index .Fields "mean" }} - Host: {{ index .Tags "host" }}'
var details_val = 'Alert Level - {{.Level}}
 Alert Name - {{.TaskName}}
 Current Value - {{ index .Fields "mean" }}
Tags - {{.Group}}
Host - {{ index .Tags "host" }}'

stream
    |from()
        .measurement(measurement)
        .where(where_filter)
        .groupBy(groups)
    |window()
        .period(period_val)
        .every(every_val)
    |mean(field)
    |alert()
         .warn(warn)
         .crit(crit)
         .message('{{ index .Tags "host" }}')
         .details(details_val)
         .email(email_id)

Following is the YAML file
# /usr/share/kapacitor/tasks/asm_test_alert.tick
dbrps:
  - { db: "telegraf", rp: "autogen"}
template-id: database_template
vars:
  measurement:
   type: string
   value: AsmSpaceUtilization
  groups:
   type: list
   value:
    - type: string
      value: db_name
    - type: string
      value: dg_name
  period_val:
    type: duration
    value: 10s
  every_val:
    type: duration
    value: 10s
  field:
   type: string
   value : used_pct
  warn:
   type: lambda
   value: "\"mean\" > 40.0"
  crit:
   type: lambda
   value: "\"mean\" > 50.0"
  email_id:
   type: string
   value<EMAIL_ADDRESS>
When I get email alert, I see following in subject
OK: asm_test_alert at 32 - Host:
Its not printing host value when I am using following reference in message - {{ index .Tags "host" }}
Email body shows following
Alert Level - OK
Alert Name - asm_test_alert
Current Value - 32
Tags - db_name=orcl,dg_name=ORCL_DG
Host -
Here also host value is not getting printed.
Am I referring the host tag correctly? or we have a different way to refer host tag?
Same thing works when I directly write TICK script instead of using template.
Regards

Is @advaitvdeo 's comment still correct? Messages can only use tags used in GroupBy() ?

Same issue here, I'm currently able to retrieve tags which triggered the alarm but can not refer to any other tag. Any updates?

@libertyy @mvarge
I am able to access tags in alerts, do you still need help.

@libertyy @mvarge I am able to access tags in alerts, do you still need help.

I need help in this, all my tags related to hostname and environment are being dropped off. I am not very good with TICK stack, if you can help it will be really helpful.


[PowerToys Run] Not detecting certain apps

Environment
Windows build number: 10.0.18363.778
PowerToys version: v0.18.1
PowerToy module for which you are reporting the bug (if applicable): PowerToys Run

Steps to reproduce
Well, it's just there.  Using PowerToys Run, I search for an application (Notion, executable Notion.exe) with or without .exe at the end.  I don't get this as a result when it is not running, and when it is running, it is always the least prioritised, despite matching the exe name.  (note - the window title NEVER contains the word Notion)
Expected behavior
Notion to appear to be launched when it is not, and when it is running it should appear as the first result.
Actual behavior
When I want to launch notion, PT Run doesn't find it at all, whereas Windows Search does.  See Figure 1.
When I want to switch to it, it is consistently the last option.  See Figure 2.
Screenshots
Figure 1:

Figure 2:

hey @supremeicecreme   couple questions:

was this installed before PT was running?
how was Notion installed?
can you share a link so we can test ourselves?

Hello,
I've been using PowerToys for several months now, so yes, it was running at the time Notion was installed.  It was installed via their official installer from their website: https://www.notion.so/

I have the same problem with Notion and some other apps:

MongoDBCompass
FontBase
Postman
VirtualBox

These may be fixed by some of the work we've been focused on for .19 but marking this for .20

@supremeicecreme Do you still see this issue on v0.19.1?

I think this is a duplicate of issue - #3553, where the apps were installed when Powertoys was already running.

I tried all the apps listed here and I was able to find all of them after restarting powertoys.

I tried all the apps listed here and I was able to find all of them after restarting powertoys.

I think we can close this then, as it is a duplicate of #3553.

To be quite honest, I don't know whether I'd installed the application before or after I installed this version of PowerToys, though I suspect it was after it.
I am now using v0.19.1 and it does show up in the list, the first one in fact.  However, as the window name is not called, and does not contain, the word "Notion", when it is running, it does not show up first in the list (where it is most useful when you want to switch to it from another application.  Instead, it is at the bottom of the list, which I can just press Up and enter, but it shouldn't have to be like that.  Can PT Run be made to prioritise running applications where the name of the EXE running is the same as the one you've typed?  Notion is called "Notion.exe", so this should be simple for stuff like that, so it appears at the top of the list, particularly over the .lnk shortcut to open a new instance of the program?

hi @supremeicecreme , great that it shows up in the latest!
in regards to the second topic you brought up, we've had conversations regarding this in a few different issues:

An option to let running programs be first in the results list -  #3169
Explicitly start new program instance with modifier key - #3884
Rethink integration of window walker into launcher - #3357

WindowWalker, btw, is the app that was integrated into PT and includes the results you see in Run of active windows.
closing this as a duplicate, feel free to open if the other issues don't cover your suggestions!

For those having issues. I find adding the application (for me it's Cmder). Add a short cut to the start menu folder. Then it will appear.

For those having issues. I find adding the application (for me it's Cmder). Add a short cut to the start menu folder. Then it will appear.

I still have this problem with Beekeeper even after adding it to the start menu.
Also, Figma shows up twice, but that's less of a problem.


checking cx_no_check properly

check failure "uncaught exception" is triggered also for exceptions type of cx_no_check.

@oblomov-dev try again, 2.95.25 published


Upgrade Surefire compatibile version from 2.16 to 2.19.1

Surefire API (from Surefire Play! Junit4 Provider point of view) has been changed between these versions (as usual).

New 1.0.0-beta8-SNAPSHOT version of Maven plugin modules and play-app-default-parent module deployed.


Add az cli option to storage/blobs/network-file-system-protocol-support-how-to.md

Add Tabs for powershell and azurecli.
azure cli is probably more common for typical NFSv3 users.

@ne-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @normesta Could you review the proposed changes? It's preferable to check new tab rendering on staging before merging, so we can move this commit to the private repo when you've had a chance to review it. Thanks.

I did a test PR to view the tabs on staging. You have to add an ending --- to end the tabbed section as in line 97 or the content doesn't display correctly. I've added and will merge. Will check on live after publishing...


Long sidebar (with scroolbar) gets cut out bottom in IE

Tested in IE11 on Windows 8.1
When the sidebar is long enough to scroll (like the http://materializecss.com/ sidebar) the bottom part gets cut out of the viewport (showcase link is inaccessible):

The scrollbar also behaves as if the sidenav element is larger than 100% and continues below the viewport. This effect is also present in Chrome, but the bottom-padding of 60px (https://github.com/Dogfalo/materialize/blob/master/bin/materialize.css#L6730) allows the all the contents to be displayed.

I've also noticed that the scroll bar for modal dialogs does not appear in IE (seems to be fine in Chrome).  Scrolling is still possible with the mouse wheel, seems like it's an issue with modal dialog width.

+1 for both issues!
Any known fix for these?

I encountered the same problem. Didn't find this thread back then, so I created #2112

Closed for cleanup purposes. Use https://github.com/Dogfalo/materialize/issues/2112 for comments and discussion instead.


Add note about CSRF_TRUSTED_ORIGINS

With Django 4.0+ setting CSRF_TRUSTED_ORIGINS is necessary. Refer: https://stackoverflow.com/a/70326426/2251364
ScanCodeIO already recommends this setting in the following section: https://scancodeio.readthedocs.io/en/latest/installation.html#run-the-app

@Hritik14 thanks++, please sign of your commit : )


Rename RAM stub to something more useful

The "stub" name doesn't really mean anything. It would be good to have a better / more informative name. Related to #622.

How about "bundle"? "glue"?

"pass-through" might be better, although it doesn't describe why it's there.

I kind of like glue, joiner or bundler? Maybe packing_forcer?


Add pessimistic concurrent control to orchestrator to reduce concurrency issues

Related to https://github.com/NuGet/NuGetGallery/issues/7590.
Validation.SymbolsOrchestrator

Release: 483281
[x] DEV
[x] INT
[x] PROD

Validation.Orchestrator

Release: 483284
[x] DEV
[x] INT
[x] PROD

Confirmed proper lease observation and retry on two different validation sets.

After enabling the leasing and re-enabling, queue-back, our validation times are back to goodness.


Searching with multiple values

The following produces an HTTP 500 error with no error details. It would be very helpful if this could work as expected. Being able to make requests like this would save a lot of traffic since otherwise, we'd have to make a separate request for every deal we want to fetch.
<?php

$stageIds = [1, 2, 3];
$deals    = $baseClient->deals->all(['stage_id' => $stageIds ]);

Similarly:
<?php

$dealIds = [1, 2, 3];
$deals    = $baseClient->deals->all(['id' => $dealIds ]);

Variation (only returns the first ID if found):
<?php

$dealIds = [1, 2, 3];
$deals    = $baseClient->deals->get( $dealIds );

Hi @KOGI,
According to https://developers.getbase.com/, filtering by multiple stage_ids is not currently supported.
In your last example example, you're trying to use get method to fetch few deals, but it's used for retrieving single object. You can use all with ids param instead.


Create collection and resource under virtual to allow access to policies

@caphrim007
Issues:
Fixes #800
Problem:
The virtual resource currently does not have SDK access to its policies.
Therefore we cannot perform CRUD operations on these policies. We need
to add the collection for Policies_s and the resource Policies.
Analysis:
I had to refactor the policy _create method to make the logic read
better from a developer's viewpoint. We changed some of the underlying
implementation, but we preserved existing functionality. We may have
loosened restrictions for behaviour against older tmos versions. And in
this respect, we have broken backwards compatibility.
Tests:
New tests for virtual, and refactored tests for policy all pass

@caphrim007: wait until @jlongstaf gives his go-ahead on a few tests he's running before you look at this

Also @caphrim007: this breaks backwards compatibility because I changed source code and broke a test, then I had to go fix up the test to get it to work. This is our tell-tale sign that we have broken backwards compatibility. We just need to be aware of this when we make our next release, which will probably be tomorrow.

@caphrim007, @pjbreaux : these changes work fine for me. Please feel free to merge.

@pjbreaux so is this text, "but we preserved existing functionality", in the description not true?

Tis a lie! I tried to preserve the old functionality, but couldn't, because the old way kind of sucked.


Git and Github for Researchers in Snack form

Experience using Git and Github use among the research community can be surprisingly mixed.  A broadly relevant introduction that can be worked through quickly, provides a high level understanding of the benefits of integrating Git/GitHub into one’s work, and is relevant for all expertise levels would be valuable.  While not comprehensive, this Snack would serve as an on-ramp to funnel people into learning more via resources on the web now or future DaSL courses.
Many labs have a real need for a basic “introduction to Git and Github”.  They need a very quick walk through of:

creating an account on GitHub,
sending that GitHub username to SciComp to get added to the Fred Hutch organization,
Creating a repository on the web
installing GitHub Desktop (and hint at GitKraken)
cloning that repo to their computer
make a commit to the readme, push to GitHub
provide example of the pattern of: clone - branch - commit - push - pull request Git flow
Provide overview as to why this is useful for code management during the research process and links to other resources to learn more including how to collaborate (forking,  etiquette, basic licensing, etc).

Do we want certification and/quiz abilities? Or are we more thinking just interactive activities but no need to have proof they've completed the "snacks"?

naa.  Let's have people just enjoy the snacks and use that as a way to get people to take the Real Classes where there might be a certification, etc.

Here's a "snack table" https://github.com/fhdsl/dasl-snack-github


Add go and cloning to auto merge of dependabot

What this PR does / why we need it

The action for performing the automatic merge of some updates was
failing due to the environment not being fully set up. This adds
installing go and cloning the repo so the ensure-deps step can run.

Hmm, I didn't know we wanted to automatically merge dependabot PRs.
Will we get into problems where dependabot is attempting to re-base a PR after a previous one has merged creating conflicts? Or will the gh CLI tool prevent us from creating those clashes.

Yeah, that's a valid concern @jpmcb. I was worried at first too, but since we're restricting this to only "hack/runner/webhook/go.**", I think the risk impact is pretty low. David and I had chatted about it and it would be easy enough to revert something if we find an issue.
Definitely would not want to do this repo-wide.

Ahhhh gotcha gotcha - that makes sense. I missed the detail where this was only running on - "hack/runner/webhook/go.**"
Thanks for the clarification! 🙏🏼


Fix: Outputs not displaying

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Please also include any acceptance criteria if you have any.

Outputs should now display on the outputs page
Link to the issues for bug recreation steps

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce
the testing if necessary.
Before PR's can be merged they will need to be tested by QA and approved where
applicable. To flag the change to QA assign @XGovFormBuilder/qa as one of the reviewers.

[ ] Create any output and check if it displays

Checklist:

[ ] My changes do not introduce any new linting errors
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have rebased onto main and there are no code conflicts
[ ] I have checked deployments are working in all environments

I appreciate this is one of those pull-requests where the lines of code are probably less than the lines of sentences that would be needed to explain what the change is doing.
But as someone who isn't as familiar as others with this project, a few descriptive words to explain what this is fixing would be super helpful 😁

@RichardBray there should now be a link to the related issue in the description

Thanks for closing my issue peeps! 💖


Steve API Fix - At-scale testing

rancher/rancher#37046

Followed the validation procedure outlined by Andrew in the comments on this issue: https://github.com/rancher/rancher/issues/36954
Compared audit logs between 2.5.12 and 2.6.5 on Small at-scale RKE1 clusters. The 2.6.5 cluster had significantly less 404s overall and significantly less duplicates as well.


AndroidX Support

Can this library be updated to support AndroidX. Project fails to build on newer versions of react-native.

you can try the beta version

Currently using 2.9.1-alpha.3 and getting the following errors:
> Task :react-native-agora:compileReleaseJavaWithJavac FAILED /Users/sinthu/Documents/git/rally-mobile/node_modules/react-native-agora/android/src/main/java/com/syan/agora/AgoraVideoView.java:10: error: package android.support.annotation does not exist import android.support.annotation.RequiresApi;                                  ^ /Users/sinthu/Documents/git/rally-mobile/node_modules/react-native-agora/android/src/main/java/com/syan/agora/AgoraManager.java:4: error: package android.support.v7.widget does not exist import android.support.v7.widget.LinearLayoutCompat;                                 ^ /Users/sinthu/Documents/git/rally-mobile/node_modules/react-native-agora/android/src/main/java/com/syan/agora/AgoraModule.java:4: error: package android.support.annotation does not exist import android.support.annotation.Nullable;                                  ^ /Users/sinthu/Documents/git/rally-mobile/node_modules/react-native-agora/android/src/main/java/com/syan/agora/AgoraModule.java:2718: error: cannot find symbol                            @Nullable WritableMap params) {                             ^   symbol:   class Nullable   location: class AgoraModule 4 errors
I am able to patch the following imports with the androidx equivalent and its working. However, I would like to see this formally addressed.

@sin2 have your try the beta version?

Updated to 3.0.0-beta.4 and its working. Thanks!


Clozes stand out too much

vs normal:

It would be nice if they could blend in just a tiny bit more, similar to the normal look! Thanks so much for making this theme <3

@rayvin185 Thanks for reporting! I will look into this!

@rayvin185 In the meantime, the Minimal Clozes plugin seems to alleviate many of the cloze issues.

(mostly the dark themes seem to have better mileage for the workaround.)

@rayvin185 I recently pushed a change to fix the cloze contrast.
I’ll be marking this issue as closed for now. Feel free to reopen or give feedback on the update(v1.2.7).


Update Composer dependencies (2022-12-24-00-11)

Visual regression test passed!
View the visual regression test report


stable/airflow minor fixes

Fix various small issues with the stable/airflow chart.
Signed-off-by: Nathan J. Mehl<EMAIL_ADDRESS>Is this a new chart
No
What this PR does / why we need it:

Install variables before running initdb; custom connections
may require variables to be defined.

Install connections before running initdb; models may
depend on custom db connections.

Fix URI-style connections: --conn_type and --conn_uri are
not compatible flags.

Fix the default "variables:" value in values.yaml: this should
be a string with a JSON object inside, not a yaml map, and
this caused confusing warning output from coalesce.go.

Which issue this PR fixes
N/A
Special notes for your reviewer:
None.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign gsemet

@gsemet to the white courtesy phone :)

Looks great! Thanks a lot !
/lgtm


Added FirmwareManagement cucumber project

Added FirmwareManagement cucumber project

All is well.

All is well.

This pull request makes the build fail.

This pull request makes the build fail.

This pull request makes the build fail.


Suggestion: support a directory selection in Open File dialog

Harold,
is it possible to make a directory selection working not only via drag-n-drop, but also by directly selecting it in Open File dialog? From what I heard, it should work in the majority of browsers, including Chrome...
Also - just in case the information might be in handy - I've actually overridden the handling of the selection to include the information about the relative path of each of the selected files.

Hi @alexandis,
to allow directory selection in the open file dialog, just add the webkitdirectory attribute to your <input type="file" /> element. Hope this helps.

When I use this attribute - there is a system message, offering me to upload files somewhere (when I agree - nothing happens). It is not an expected behavior for me, saying honestly. What I would like to have instead - selecting folder acted in the same way as if I just "drag-n-drop"-ed this folder, so drop event would raised eventually...

I've tried webkitdirectory again with ngModelChange - at least it process ALL the files, even if there are thousands of them. But - this standard dialog does not allow me selecting several folders... :( so it is not so flexible as drag-n-drop, unfortunately.
Anyway, I think I can close the original bug, because - with specific limitations - the directory selection DOES work.


Linux is  in /tmp no /home/{user}/tmp

Well,  when the install is global in linux.
var temp = require('temp');
console.log(require('os').tmpDir, temp.dir);

Print
/tmp /home/{user}/tmp

This should be fixed. What version of node are you on? Seems to work on 0.10, and 0.12.

Tested on fresh Linux instances as root, with the library installed globally, and Node v0.12.4, v0.11.16, and v0.10.38 without being able to reproduce (in all cases temp.dir was /tmp). Please reopen with specific linux and node versions (and any other steps to reproduce) if you are still encountering this issue.


Style(repo): Introducing markdown file lintting

Description

Using our new Remark Config for linting our Markdown files.
Additional context

https://github.com/lmc-eu/code-quality-tools/tree/main/packages/remark-config
Issue reference

coverage: 78.435%. remained the same
when pulling 2dc90329805e51bee6ad58327c3465cbc1526ac1 on style/markdown-linting
into 7ba942ca24b4e85ff1eeb447f94150aa97dfc7f3 on main.


PP-391: Decision language checking

Before checking out branch

You can skip this step if you already have decisions on your local environment
Checkout develop then run make ahjo-migrations

To test

Checkout branch, run make drush-cim drush-cr
Login to the container with make shell, then run: drush ap:update --logic=language --limit=100 -v
This should update decisions and set the "Record language checked" field to TRUE whenever the record data is loaded

Getting followin error:
`In DrushArgvInput.php line 227:
[Symfony\Component\Console\Exception\RuntimeException]
The "--logic" option does not exist.
Exception trace:
at /app/vendor/drush/drush/src/Symfony/DrushArgvInput.php:227
Drush\Symfony\DrushArgvInput->addLongOption() at /app/vendor/drush/drush/src/Symfony/DrushArgvInput.php:161
Drush\Symfony\DrushArgvInput->parseLongOption() at /app/vendor/drush/drush/src/Symfony/DrushArgvInput.php:87
Drush\Symfony\DrushArgvInput->parse() at /app/vendor/symfony/console/Input/Input.php:55
Symfony\Component\Console\Input\Input->bind() at /app/vendor/symfony/console/Command/Command.php:214
Symfony\Component\Console\Command\Command->run() at /app/vendor/symfony/console/Application.php:1027
Symfony\Component\Console\Application->doRunCommand() at /app/vendor/symfony/console/Application.php:273
Symfony\Component\Console\Application->doRun() at /app/vendor/symfony/console/Application.php:149
Symfony\Component\Console\Application->run() at /app/vendor/drush/drush/src/Runtime/Runtime.php:118
Drush\Runtime\Runtime->doRun() at /app/vendor/drush/drush/src/Runtime/Runtime.php:48
Drush\Runtime\Runtime->run() at /app/vendor/drush/drush/drush.php:72
require() at /app/vendor/drush/drush/drush:4`

Getting following error:
In DrushArgvInput.php line 227:

  [Symfony\Component\Console\Exception\RuntimeException]
  The "--logic" option does not exist.

I had the wrong command in the test instructions, should be fixed now.


Kotlinx Coroutines

Adding support for coroutines would be nice. First of all, the new KItem class contains a list of callbacks, with coroutines and flows KItem would look
  public sealed class ClickEvent
  public data class LeftClick(val player: Player, val hand: Player.Hand): ClickEvent()
  public data class RightClick(val player: Player, val hand: Player.Hand): ClickEvent()

  public class KItem(material: Material, amount: Byte = 1, damage: Int = 0): ItemStack(material, amount, damage) {
      private val _events = MutableSharedFlow<ClickEvent>()

      public val clicks = _events.asSharedFlow()
      public val leftClicks get() = clicks.filterIsInstance<LeftClick>()
      public val rightClick get() = clicks.filterIsInstance<RightClick>()

      override fun onLeftClick(player: Player, hand: Player.Hand) {
          GlobalScope.launch { _events.emit(LeftClick(player, hand)) } // in most cases
      }

      override fun onRightClick(player: Player, hand: Player.Hand) {
          GlobalScope.launch { _events.emit(RightClick(player, hand)) }
      }
  }

from the call site, it would look like
  item.leftClicks.collect { (player, hand) ->
     // do something
  }

Flow API allows doing a lot of stuff as rxjava does. https://kotlinlang.org/docs/reference/coroutines/flow.html

Wow! I did not know about the Flow API -- will take this into consideration. I'll see what I can do :+1:

Actually, it won't work sadly -- Minestom calls it via the override.

Strange, this example works perfectly well. Are you adding suspend modifier to the function? We can't add suspend modifier because it changes the function signature, so that's why there is GlobalScope.launch. In our case, runBlocking can be used, because we emit the value and immediately return from the function.
`
public sealed class ClickEvent
public data class LeftClick(val player: Player, val hand: Player.Hand): ClickEvent()
public data class RightClick(val player: Player, val hand: Player.Hand): ClickEvent()
public class KItem(material: Material, amount: Byte = 1, damage: Int = 0): ItemStack(material, amount, damage) {
private val _events = MutableSharedFlow()
  public val clicks = _events.asSharedFlow()
  public val leftClicks get() = clicks.filterIsInstance<LeftClick>()
  public val rightClick get() = clicks.filterIsInstance<RightClick>()

  override fun onLeftClick(player: Player, hand: Player.Hand): Unit = runBlocking {
      _events.emit(LeftClick(player, hand))
  }

  override fun onRightClick(player: Player, hand: Player.Hand): Unit = runBlocking {
      _events.emit(RightClick(player, hand))
  }

}
`

How would we add left/right clicks?

if we have a click event we can just emit that click event to the _events. On the other side, if someone wants to listen to the events it can be done as
val item = KItem(Material.STICK)

item.leftClicks
    .onEach { println("${it.player.name} used left click") }
    .launch(GlobalScope)

Yeah, left/right click triggering already happens natively through Minestom -- that does seem like an interesting way to add them.


Problem with java 6 version (4.2.1-java6-release): incompatible class version (1.7)

Hi,
I have a maven enforcer rule on my project so that I can only use Java 6 compatible maven dependencies.
I imported your simple-java-mail:4.2.1-java6-release, I got this output from maven:
[INFO] --- maven-enforcer-plugin:1.4.1:enforce (enforce-bytecode-version) @ thinpoc-ejb ---
[INFO] Restricted to JDK 1.6 yet org.simplejavamail:simple-java-mail:jar:4.2.1-java6-release:compile contains org/simplejavamail/outlookmessageparser/model/OutlookAttachment.class targeted to JDK 1.7
[WARNING] Rule 0: org.apache.maven.plugins.enforcer.EnforceBytecodeVersion failed with message:
Found Banned Dependency: org.simplejavamail:simple-java-mail:jar:4.2.1-java6-release
Use 'mvn dependency:tree' to locate the source of the banned dependencies.

Seems that org.simplejavamail:outlook-message-parser is compiled with Java 7.
My maven enforcer rule, for reference:
<plugin>
				<groupId>org.apache.maven.plugins</groupId>
				<artifactId>maven-enforcer-plugin</artifactId>
				<version>1.4.1</version>
				<executions>
					<execution>
						<id>enforce-bytecode-version</id>
						<goals>
							<goal>enforce</goal>
						</goals>
						<configuration>
							<rules>
								<enforceBytecodeVersion>
									<maxJdkVersion>1.6</maxJdkVersion>
									<!--<excludes>-->
										<!--<exclude>org.mindrot:jbcrypt</exclude>-->
									<!--</excludes>-->
								</enforceBytecodeVersion>
							</rules>
							<fail>true</fail>
						</configuration>
					</execution>
				</executions>
				<dependencies>
					<dependency>
						<groupId>org.codehaus.mojo</groupId>
						<artifactId>extra-enforcer-rules</artifactId>
						<version>1.0-beta-5</version>
					</dependency>
				</dependencies>
			</plugin>

Please fix,
Thanks!

Oh hmm, I see. Thanks for the report, I'll try to have it fixed this weekend.

I tried to use the enforcer plugin in the project as well, but I'm not able to get it complaining about the dependencies. Not sure what I'm doing wrong. I'll keep on it though.

I also tried to recompile your project with Java 6 target level, but I did not understand how did you manage java 6 compatibility since I came accross some classes that only available from Java 7 like StandardCharsets and Phaser. Do you know which tag/commit is the last which is java 6 compatible, or could you please explain how do you guarantee java 6 compatibility if these classes are present in the code, even if I checkout the 4.2.1 tag?

Honestly, I'm taking a quite naive approach in hindsight; I simply set the target compiler version to 1.6, but that doesn't give any guarantees about the dependencies, which I package along.
This approach worked while Simple Java Mail had fewer versions, but it won't work anymore. I'm not sure I can still support Java6 now. Do you have suggestions?

Thanks for your time and your fast response!
Unfortunately I don't as you are already using classes from JDK 7.
I think it would be better to state somewhere that java 6 is not really supported from now so that no one may fall into the same pitfall, maybe removing java-6 from the release name would make sense.

I'll see what it takes to make a reduced java6 version. Otherwise I will deprecate the java6 releases and stick to java7.

Sounds good, thank you for your effort!

I've released customised 4.2.3-java6-release with a customised outlook-message-parser 1.1.16-java6-release. This is the last java6 release I will do, as it is simply too much manual labor to create a limited second edition.
I've removed the Phaser completely which has the following consequences:

If authenticated proxy is used, the bridging proxy server will not be shut down automatically (and might not run the second time)
If mails are sent in async mode, the connection pool will not be shut down anymore by itself

This means your server/application might not stop properly due to lingering processes. To be completely safe, only send emails in sync mode (used by default) and don't use authenticated proxy config.


Replace JSHint with ESLint

Might be good to add https://github.com/DockYard/eslint-plugin-ember-suave ?

@marcoow yeah, that makes sense in general, but IMHO not in this repo since we don't have any actual Ember code in here.

@marcoow yeah, that makes sense in general, but IMHO not in this repo since we don't have any actual Ember code in here.

Right, good point


Latest commit of pom.xml breaks war generation

clean package war:war no longer works as of the latest commit.  Instead, a solitary ROOT.jar is in its place.  clean package war:exploded jetty:run works fine still.

Somehow, the clover integration option was ticked on the CI server.  Unticking it fixed the issue.


Redis High Availability

Is your feature request related to a problem? Please describe.
Currently we do not natively support Redis in high availability mode for online serving.

Redis is a single point of failure
Redis cannot currently scale out more than one node, leading to resource utilization problems (disk, cpu).

Additionally, there is a problem with the current implementation of the Redis client in Apache Beam where it establishes too many connections to Redis.
Describe the solution you'd like
The goal is to add support for HA Redis as part of Feast. This would require

Users should be able to configure the addresses to these stores as part of their serving deployment. This may necessitate an update the Redis store in the store.proto (or the creation of a new type of store like RedisCluster.
Refactoring the Feast Redis IO in the Apache Beam ingestion code to support Redis clusters.
Refactoring the Feast Serving client to support Redis clusters.
Ensuring that the Redis clients (both for cluster and non-cluster mode) do not create an excessive amount of connections. Ideally each dataflow or beam node should be sharing a single connection pool between all bundles.

I would take this opportunity to close #247 and switch to:
https://github.com/lettuce-io/lettuce-core

I would take this opportunity to close #247 and switch to:
https://github.com/lettuce-io/lettuce-core

Hell yea. Adding this to the solution above.


AddN cannot handle shapes [x,y] + [1,y] or [x,1]

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.13.0
Custom code
Yes
OS platform and distribution
Windows 11, CentOs8, Ubuntu
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
I found this behaviour when calculating hessians of my custom TensorFlow function. For performance, I manipulate some tensors with unique and gather. This pops up for hessians only so I can't think of how to manipulate my code to avoid AddN.
Surely, AddN should support the same operations as Add?
Output:
`2.13.0 Also repeated in 2.16.0-dev20231020
Shapes:
X=  [9 10]
Y[o,:] =  [1 10]
X + Y  [9 10]
tf.add_n([x,y]) fails!
ValueError: Dimension 0 in both shapes must be equal, but are 9 and 1. Shapes are [9,10] and [1,10].
	From merging shape 0 with other shapes. for '{{node AddN}} = AddN[N=2, T=DT_FLOAT](x, y)' with input shapes: [9,10], [1,10].`

Standalone code to reproduce the issue
import tensorflow as tf
o=tf.newaxis

nel=tf.range(1,10,1,tf.float32)
nph=tf.range(5,15,1,tf.float32)
nel_2D=tf.repeat(nel[:,o],tf.shape(nph)[0],axis=1)

@tf.function
def add_N(x,y):
 return tf.add_n([x,y])
@tf.function
def add_v2(x,y):
 return x+y

tf.print('Shapes:\nX= ',tf.shape(nel_2D), '\nY[o,:] = ',tf.shape(nph[o,:]))

tf.print('X + Y ',tf.shape(add_v2(nel_2D,nph[o,:])))

tf.print(' tf.add_n([x,y]) fails!')

add_N(nel_2D,nph[o,:])

Relevant log output
No response

@josh0-jrg,
Thank you for reporting the issue. We are currently investigating the issue, and I kindly request some time to thoroughly analyze the problem in order to offer a resolution. Thank you!

Directly from the documentation:

All inputs in the list must have the same shape. This op does not broadcast its inputs. If you need broadcasting, use tf.math.add (or the + operator) instead.

You can explicitly broadcast your inputs using something like tf.broadcast_to.


add deviation

Some devices report last-switchover-reason as SYSTEM_INITIATED instead of USER_INITIATED for gNOI.SwitchControlProcessor.

Pull Request Functional Test Report for #1823 / 7294883f83eb0b92e6b62f4edd44b3eeae53685d
Virtual Devices

Device
Test
Test Documentation
Job

Arista cEOS

TE-8.2: Supervisor Failure

Cisco 8000E

TE-8.2: Supervisor Failure

Cisco XRd

TE-8.2: Supervisor Failure

Juniper cPTX

TE-8.2: Supervisor Failure

Nokia SR Linux

TE-8.2: Supervisor Failure

Openconfig Lemming

TE-8.2: Supervisor Failure

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 47.655%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1341

Relevant Lines:
2814

💛  - Coveralls


[react-i18n] Fixes time zone handling for humanized date style

Description
This PR fixes an issue with the way that formatDate handles time zones when using the Humanize date style.
Currently, it works as expected for any dates that are not "yesterday", "today", and "tomorrow". When the date is between yesterday and tomorrow, it uses the time zone to generate the time (e.g. "11:00 AM") but it does not use the time zone when generating the yesterday/today/tomorrow label. From the nature of the bug, it looks like it falls back to the user's local time zone.
For example: Given a timestamp of "Sept 9, 2020, 2:00 am, UTC" and a user located in EST, formatDate (with UTC as the time zone) will currently humanize the timestamp as "Yesterday at 2:00 am", since the date in the user's time zone is Sept 8th. This PR changes this behaviour so it humanizes the timestamp as "Today at 2:00 am", since the date in the supplied time zone is Sept 9th.
Also, the test changes are a bit... verbose and could probably be cleaned up. I left the existing tests intact because I wanted to ensure that this didn't introduce any regressions. The drawback is that the "today" / "tomorrow" / "yesterday" tests now all have "when a locale is specified" and "when a locale is not specified" counterparts, which feels a bit awkward.
cc @Shopify/web-foundations
Type of change

[x]  Patch: Bug (non-breaking change which fixes an issue)
[ ]  Minor: New feature (non-breaking change which adds functionality)
[ ]  Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

@alexandcote now < date should be a time-zone-agnostic comparison that just compares in the internal representations (unix timestamp). Where are you suggesting we do the TZ manipulation?


Enforce identical type constraints

Fixes #6210 and #6211.
Before class/interface merging, only interfaces needed to be checked for differing type parameter lists. Now classes and interfaces all need to be checked. Refactor the check into its own function as well.

:+1:


Add option to scan a list of files

Can you please take a look and approve the PR https://github.com/LibrePhotos/librephotos/pull/626.
The goal is to be able to scan individual newly added files instead of always scanning the full library. Thanks!

Thanks again for implementing this :)


add fb graph API calls

for now just forget about my git merges, it's damn annoying, but auto-merge isn't possible as some of the templated iOS files and my files are different (so git conflicts etc). I could make a final version based on the repo templates if needed later on.


Support on customized transformations

Background
In JAX, transformations can be customized for user defined functions, for example customizing the JVP rule. This can be achieved via two ways

wrap your custom function as a primitive, and register jvp rules for the primitive.
use the decorator @jax.custom_jvp.

This is very useful because when you need a function with a customized gradient, for example, giving a discrete function a continuous gradient relaxation, it can be done by the register mechanism, the upper level constructs, e.g. jax.grad(f) are kept the same not matter what the jvp of f is.
Density Functional Theory
In order to perform elegant DFT, we would like to keep the high level program like the following
import autofd.operators as o
import dft.energy as e

def wave_ansatz(param, r):
  ...

def energy(crystal, param, occ):
  psi = o.partial(wave_ansatz, args=(param,), argnums=(0,))

  def psi_to_rho(psi):
    rho = o.compose(
      lambda v: jnp.sum(occ * jnp.real(jnp.conj(v) * v)), psi
    )
    return rho

  def energy_functional(psi):
    # use psi to build rho, in a functionally differentiable way.
    rho = psi_to_rho(psi)
    # compute the total energy using the functionals
    etot = (
      e.kinetic(psi) +
      e.hartree(rho) +
      e.external(rho, crystal) +
      jax_xc.energy.lda_x(rho)
    )
    return etot

# gradient descent to optimize crystal, param, occ etc.
jax.grad(energy)(crystal, param, occ)

The problem with the elegant code.
One difficulty to achieve this elegance is the efficiency, if we write ugly code, we may optimize all the above energy calculations by hand deriving each terms and implement the derived formula. For example, kinetic energy of planewave is simply summation over the norm of G and k grids. Because applying laplace operator to planewaves only adds a constant factor, and that the psi is normalized and there's no need to compute the <psi|psi> because it is always 1. However, all this level of details are not available to autofd, resulting in an inefficient implementation when we follow the above code for DFT.
Why we want the elegancy?
Then why don't we just implement the derived formula than following the functional form? It is not just for the elegancy, but also for extensibility. For example, one thing that we often do is to linearize the energy at the current value of rho, and use it as an effective potential for computing energy bands for cystals. Which gives me headache already if we were to re-derive the math for constructing fock matrix, and implement them as FFT. However, if we could support the above syntax, we could have an easier way around.
def band_structure(crystal, fixed_rho):

  def potential_energy(rho):
    return (
      e.hartree(rho) +
      e.external(rho, crystal) +
      jax_xc.energy.lda_x(rho)
    )

  _, eff_energy = jax.linearize(potential_energy, (fixed_rho,))

  def energy_under_veff(param, k):
    psi = o.partial(wave_ansatz, args=(param,), argnums=(0,))
    return e.kinetic(psi) + eff_energy(psi_to_rho(psi))

  bands = {}
  for k in k_vectors:
    bands[k] = jnp.eigh(jax.hessian(energy_under_veff)(param, k))[0]
  return bands

How to keep elegancy and efficiency?
Compiler is the way! Let's write elegant code and enjoy the cleanness of various tasks in DFT while relying on some compiling process convert the code to high performance. The optimizations in the compiler takes lots of rules here and there, therefore, we want to enable autofd to support custom rules when the user knows a more efficient implementation.
Custom rules for any operators
In JAX, we can customize the rules for the transpose operator, jvp operator etc. A straightforward extension is to support customized rules for all operators.
Again with kinetic energy as an example, say we have build a primitive for computing kinetic energies of wave functions.
# A custom primitive for kinetic energy
kinetic_p = core.Primitive("kinetic")

def kinetic(psi):
  kinetic_p.bind(psi)

def kinetic_impl(psi):
  # general implementation

We can customize it for a specific wave function
@autofd.custom_operator_rule
def psi(r):
  ...

@psi.def_operator_rule(kinetic_p)
def psi_kinetic_rule(r):
  ...

Mixing different rules
One difficult question is how can we customize many different rules and how do they interfere with each other. Can we retain the custom kinetic rule when we first apply the JVP rule on psi? I need to study further before having an idea for this. (To be continued)

@sail-sg/ai-for-science


Test ignored if @BeforeMethod in base class fails for another test class

TestNG Version
latest (TestNG 6.15.0-SNAPSHOT)
Expected behavior
test2 is not ignored, because setUp for test2 successfully finished
Actual behavior
test2 ignored
Is the issue reproductible on runner?
IntelliJ
Test case sample
public class BaseClass {

    @BeforeMethod(alwaysRun = true)
    public void setUp(Method method) {
        if (method.getName().contains("test1"))
            Assert.assertEquals(2, 1);
    }

}
public class MyTest3 extends BaseClass {

    @Test
    public void test1() {
       System.out.print("test1");
    }

}
public class MyTest4 extends BaseClass {

    @Test
    public void test2() {
        System.out.print("test2");
    }

}

Thanks for the report.
@krmahadevan Do you have free time to have a look at it?

@juherr - I will take a look at this.


Roll Engine from 64e659991025 to 073263e39d1d (1 revision)

https://github.com/flutter/engine/compare/64e659991025...073263e39d1d
2020-10-27<EMAIL_ADDRESS>Roll Skia from 5bbbb49f1da0 to 7737a5bd2510 (13 revisions) (flutter/engine#22140)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


OpenVPN: Enhance configuration - Compression

This PR deals with OpenVPN compression configurations for both servers and clients. The official documentation suggests using the option --compress instead of the deprecated --comp-lzo. It allows to use the LZ4/LZ4 v2 algorithms in addition to the LZO. Besides, in view of the VORACLE attack, the stub/stub v2 algorithms are made available.
Before:

After:

This is a part of the changes requested in #2048.

according to https://community.openvpn.net/openvpn/wiki/DeprecatedOptions  --compress will be deprecated as of 2.5, --comp-lzo as of 2.4, but both are still available. I don't mind supporting --compress as well, but we shouldn't drop features without a clear migration unless absolutely necessary. An option could be to keep the old compress options and start explaining which option to choose, although the general idea from the authors seem to be to stay away from both...

according to https://community.openvpn.net/openvpn/wiki/DeprecatedOptions  --compress will be deprecated as of 2.5, --comp-lzo as of 2.4, but both are still available. I don't mind supporting --compress as well, but we shouldn't drop features without a clear migration unless absolutely necessary. An option could be to keep the old compress options and start explaining which option to choose, although the general idea from the authors seem to be to stay away from both...

OpenVPN compression still has its use cases. To me, retaining the ability to decide on a case by case basis is important. I like the idea of adding these and briefly explaining which to choose in the GUI.

@AdSchellevis Thank you for your feedback. You are right that the newer option will be deprecated as of 2.5. Modified this PR to keep both options.

@vnxme although the translators might not like the texts being replaced, it's probably a good idea to name the option explicit here since it can be a bit confusing. I'll pull it into master, thanks!


Variable declaration garbage collected and destroyed after loop in recursive function call

Description
After the implementation of variable declaration using the var keyword, calling a function recursively destroy the variables used within a loop.
Expected behavior and actual behavior.
Function call should be successful when called recursively within it body and only temporary variable should be destroyed. Even when variables are destroyed it should recreate on another call
Steps to reproduce the problem.
Onene()

block Onene()
	var Onet = [1,2,3,4]
	var on
	for on in Onet
		display "Done with : " + on + nl

	end
	Onene()

resolved in the commit https://github.com/simple-lang/simple/commit/516af14b05c4dd5b8bd98dc1ae6d5990792c4ff8


Update TileMap when its TileSet changes

Make TileMap monitor its TileSet for changes and emit a signal when the TileSet changes. This makes the editor update and show the updated version of the TileSet.
Can be tested using the demo project in #17274 and following the steps described there.
Fixes #17274

Thanks!

Cherry picked into 3.0.3


req.route returns array, how to get active route?

Hello,
im trying to get the active route from req.route, (when array is returned)
example:
app.get(['/index', '/product/:id'], function(req,res) {
   console.log(req.route);
   res.json({error: false})
});

expected output:
Route {
  path:
   [ '/index',
     '/product/:id'
 ],
  stack:
   [ Layer {
       handle: [Function],
       name: '<anonymous>',
       params: undefined,
       path: undefined,
       keys: [],
       regexp: /^\/?$/i,
       method: 'get' } ],
  methods: { get: true } }

how can i get the active route,
if request url was: /product/15 i want to know that the route is /product/:id

Under the hood, there is only one route, not two. You have two different paths to the single route. You can always declare them separately. Express does not actually know which path of a multi path route the URL was matched against currently.

@dougwilson
i know i can declare them separately, but actually i can't because of the way the application built,
so when /product/:id hit the express use regex to know what route to execute,
so is there any way to do the same thing buy using req.path or req.url to extract the route from the array?

Both paths are combined into a single regular expression by the path-to-regexp library,  which is why Express does not know which patch marched, as there is only one regexp to match against for that route

This is the one regular expression that Express is matching your route against: /(?:^\/index\/?$|^\/product\/(?:([^\/]+?))\/?$)/i

@dougwilson
so the problem is how to extract matched url and match it with the array?

so the problem is how to extract matched url and match it with the array?

That's what I'm saying: it's not currently possible to do.

@dougwilson
ok, thank you,
Anyway, please keep the case open, if anyone knows how to do it

https://gitter.im/expressjs/express is the place for support and discussions.

i found the solution
// req.path => /product/123
// the route => /product/:id
var result = req.path.match(/^\/product\/(?:([^\/]+?))\/?$/i)

if(result != null) {
// matched
}


Update README.md

I have updated this file

Closing this due to inactivity, @abdullah818 pls reopen if needed.


Fresh Pull isnt working with subreddits

earching for tickers in shortsqueeze...
Populating results for shortsqueeze...
Traceback (most recent call last):
  File "tasks_to_run.py", line 94, in <module>
    scrape_reddit_stocks.main()
  File "/root/3_aurora----Production/Stocksera/scheduled_tasks/reddit/stocks/scrape_trending_posts.py", line 25, in main
    results_df = get_financial_stats(results_df, minimum_volume[index], minimum_mkt_cap[index], allow_threading)
  File "/root/3_aurora----Production/Stocksera/scheduled_tasks/reddit/stocks/AutoDD.py", line 258, in get_financial_stats
    quick_stats_df = get_quick_stats(ticker_list, min_vol, min_mkt_cap, threads)
  File "/root/3_aurora----Production/Stocksera/scheduled_tasks/reddit/stocks/AutoDD.py", line 278, in get_quick_stats
    unprocessed_df = download_quick_stats(ticker_list, quick_stats, threads)
  File "/root/3_aurora----Production/Stocksera/scheduled_tasks/reddit/stocks/fast_yahoo.py", line 118, in download_quick_stats
    for retrieved_stats_dict in response_list:
TypeError: 'NoneType' object is not iterable

I cannot solve this error and strangly all I need to do is remove the subreddit section of the reddit scrape.

you have corrected the issue if anyone who fetches this uses docker.


Conflict Bug

v4.0.0-rc.3-b168
https://pastebin.com/3Gr3iYdA
It happened by error. I think the problem is that I was modifying a quest with the UI but my quests.yml was opened and when closed it, I clicked on save so there is a conflict.
I restarted the plugin (delete and re uploaded files). I don't know how I can fix it. Please help me. l can send the quests.yml file too.

just updated to last build (v4.0.0-rc.3-b178)
similar error : https://pastebin.com/secQgNFu

Ok, so i patched a lot of things but now there is a new error. The plugin is loading but when i click on an npc, it begins spamming this error :
https://pastebin.com/ystbsGe3

when i click on an npc while the console is spammed. it shows an other error :
https://pastebin.com/DYUrXcbA

i downgraded to 177 and I didn't have any issues since then.


bad attribute formatting

cc @IwanKaramazow
Input:
type reactClass = ReasonReact.reactClass [@@ocaml.deprecated "ReactRe.reactClass got moved under ReasonReact.reactClass. Please codemod all the references of ReactRe.reactClass to it."];

Output:
type reactClass = ReasonReact.reactClass

  [@@ocaml.deprecated
    "ReactRe.reactClass got moved under ReasonReact.reactClass. Please codemod all the references of ReactRe.reactClass to it."
  ];

Let's see how this one is formatted with @let-def's changes.

Should I submit a fix? What branch has let-def's changes?

This branch has the most recent changes: https://github.com/let-def/reason/tree/es6_fun-3/ .
Changes are quite profound and while parser is in a good state, the printer implements the language but the formatting is not so pretty yet. Maybe I can open a separate PR which details the changes in syntax so that we can discuss formatting (it is not clear to me how things should be formatted :), there is a large aesthetic part to talk about).

<EMAIL_ADDRESS>  "ReactRe.reactClass got moved under ReasonReact.reactClass. Please codemod all the references of ReactRe.reactClass to it."
]
type reactClass = ReasonReact.reactClass;

Looks good!


New Model Elements [API]

Additional, standard new ones:

label
type
code

See  #125 for the CT and Model work.

Also consider adding the other standard elements to the classes: bcName is not changed to name yet for example and we might want to add description together with name to all classes. What are the pros and cons?


Add metadata.hcl and updated README.md files for all images

This PR does not include changes to monopod. It only adds a metadata.hcl file for each image to track things like the URL of a project, end of life data, a short description for use on Academy and images console etc.
There is also an README.md for each image based on the provided metadata, and rendered from a template in monopod.
The monopod changes to render things will be in a separate PR since it needs everyone to switch to using it.

Love this! We should make this part of monopod scaffold too, by creating a metadata.hcl.tmpl in TEMPLATE

Glad you thought of that too, I added it over in #1889: monopod/pkg/commands/templates/metadata.hcl.tpl

So close here, but testing with autodocs seems to result in no readme body content at all, so marking this as a draft until I can figure out why the content is getting eaten.


Hide tooltip from the lane/card

Hi @rcdexta @dapi Is there a way we can get rid of this default tooltip from lane and cards(which display the lane title always)??

I don't see any optional flag to hide/show this tooltip. Is there a way to hide this??

@rcdexta Any idea about this?

@rcdexta I think the tooltip should not overlap the card...

Do we have a fix for this issue? The tooltip should only be displayed on the Vertical Section Title and not on complete Section.

Hi!
I think best way is make you own Card.js


Line numbering breaks the linking

Issue
When adding :lineno: to code-blocks and literalinclude's this breaks the linking and all the links are absent with the compiler giving Could not resolve and Could not match a code example to HTML errors respectively.
I am sorry, I would like to elaborate, but I cannot understand, how to submit minimal reproducible example. Please tell me, how I can be more helpful in any way.

Thanks for submitting the issue! Your description was to the point and I think gives most of the information to debug, so don't worry 👍 What is meant by a minimal reproducible example would be a minimal but complete project that when built with Sphinx would produce the error. So in this case most likely just an index.rst and a simple config.py. For small stuff, just including it in the issue is entirely fine, like so:
# index.rst
Example
*******

.. code-block:: python
    :linenos:

    import numpy

# conf.py
extensions = ["sphinx-codeautolink"]

Thank you for the detailed answer!
Attached here is my minimal_reproducible_sample.zip. Sorry that it took a week to do it, I will respond more quick now.

Hey! Sorry for dropping this for the longest time. I've now pushed a fix, expect this to be released in the coming days once I've had a look at the other issues. I'll leave this open in the meantime and close it on release so you'll get a notification. In the meantime you can git-install to try it out if you want to!

Finally, on the 1st anniversary of this ticket, the new version has been published and this can be closed! :P


Sandbox does not recover properly from errors, leading to busy issues

If you go to integration_test/exqlite/all_test.exs and comment out the line for:
Code.require_file "#{ecto}/integration_test/cases/interval.exs", __DIR__

and run the test suite, the first test fails due to some reason, and then the rest of the tests fail with "Database busy". Which means the failed test is not rolling back its transaction correctly...
I believe db_connection has a :disconnect return status that I noticed some of the other adapters take use of when hitting errors. I think we need to use that for some of the errors we cannot recover from, so that the entire connection is closed and the database does not stay in an unreachable state until the entire application is restarted!
To be clear, this may not be scoped only to the sandbox. This probably represents an error that could occur in normal "production" application use.

Yea I was running into this issue as well on my end of things. And do believe it could affect the production application use.

@kevinlang do you have a reliable way to reproduce this error? We used to before we were able to get the returning clause functioning. I just want to make sure when we flip some of these :error's to :disconnect that we have a test that reliably bombs.

Uncommenting the type.exs and running the integration tests reproduces it for me.

Yea this is a hard one to track down. If we bubble back any :disconnect some other errors occur.


CUPS prints number of copies squared

I'm not exactly sure what's causing this, but I'm assuming copies is getting processed twice somewhere. My setup is:
printers
...
raw CUPS print server (to handle windows as well as cups clients)
...
clients (both windows and cups)
Windows clients print just fine. For instance if I ask for 2 copies I get 2 copies. CUPS clients print copies squared. So when I ask for 2 copies I actually get 4.

P.S. By saving the job at the raw server I can see that the input is a (gzipped) PCL document containing 4 pages. So the problem happens somewhere before the job gets to the raw server.

What version of CUPS?
What OS/Distribution?

Cups 2.1.4 on gentoo linux.

Good morning Mickael,
I know this issue is close, but we are experiencing on Ubuntu 18.04 with CUPS 2.2.7 this strange problem.
So I post a comment here, hoping this is the right place.
We use Ubuntu 18.04 clients to print to raw queues on a Ubuntu 18.04 server.
We do this for print accounting and control in our application.
We do many many tests, on Ubuntu 18.04, Kubuntu 18.04, Ubuntu 19.04, and to finish, on Debian 9 (with CUPS 2.2.1)
For our testing, we create a printer called MYREMOTEPRINTER .
This is the name of the raw queue on server and, to keep it simple, the name of the printer for the clients.
Each time we print a document (from Firefox, evince, and many others applications), asking 2 ou more copies, we get 2^2, or 3^3 and so on copies.
Same behavior if we print from command line ("lp -d MYREMOTEPRINTER -n 2 MyDocument.PDF").
A document of 3 pages, 2 copies, will generate 3 * 2^2 pages = 12 pages instead of 6.
And if you want to print just a unique page, 10 copies, you will get 100 pages !
This problem doesn't exist on Ubuntu 14.04 with CUPS 1.7.2
The same problem has been opened on Launchpad :
https://bugs.launchpad.net/ubuntu/+source/cups/+bug/1785644
We discovered also that if we declare the remote printer as :
lpd://REMOTESERVER/MYREMOTEPRINTER
instead of
ipp://REMOTESERVER/printers/MYREMOTEPRINTER
(and declare cups-ldp in inetd.conf, of course) problem is not present.
My question : why bug #5015 has been closed ?
Is it really solved ?
This doesn't seem just an Ubuntu bug (Debian has same problem, I can test on Fedora too).
How to explain that changing the protocol used to transmit data from clients to server (from IPP to LPD) seems to solve the problem ?
Do you think this could be solved in a next release ?
A backport or patch could be available for current releases ?
Best regards
Marco from France

@msaesis The copies-squared problem is always caused by a misconfiguration - when the Client runs local filters it produces copies, but then the server sees the print file from the client as plain old PostScript and extracts the number of copies from the PostScript headers, leading to double copies.
You need to use the IPP Everywhere driver on the clients (or raw queues, which are deprecated but have a similar effect) so that all of the driver processing happens on the server.

Thank you for your answer.
But, this kind of configuration works fine on CUPS 1.7.2 and Ubuntu 14.04 (which use that version of CUPS) just reached EOL in April 2019.
So, my questions are :

no more compatibility with old configurations ?
(I know that raw is deprecated, but I need it for the moment)
why changing transport protocol between clients and server from IPP to LPD makes it to work  again in CUPS 2.2.7 ?
how to mix Windows clients with native drivers and driverless clients on Linux without creating more queues, for instance ?

I haven't still studied all possibilities of "IPP driverless" or "IPP Everywhere" model, but it seems to me that direct communication between client and printer is necessary, so how to have a print server to control and count user printings ?
And what about compatibility ?
I haven't seen many manufacturers (but HP) speaking about "IPP Everywhere" compatibility in their technical specifications (Kyocera, Toshiba, ...)
Thanks for your help and your advice
Marco

@msaesis Using LPD flags the job as raw on the server. And in the case of Windows clients you will always have problems because most drivers expect to be talking directly to the printer and have never been tested over IPP.
As for what changed, I have no idea. CUPS is just part of the printing system on Linux and filter and driver changes can often break things in unpredictable ways - that's one of the reasons we have been pushing so hard for IPP Everywhere/AirPrint support (98+% of all printers support AirPrint, about 10% explicitly support IPP Everywhere, more are coming!)


[Dependency Update] Upgrade cuDNN & NCCL

Description
Since the CI have upgraded to use cuDNN 7.5.1 (#14950) , we can upgrade the CUDA 9.0/9.2/10.0 with latest cuDNN 7.5.1 & NCCL 2.4.2
@perdasilva please check it
Checklist
Run three models ResNet50 with ImageNet & LSTM with PTB & MLP with MNIST
Performance shown below
Environment: P3.16xlarge Deep Learning Base AMI
Codebase: commit 1540a84f1eca937235c51b507ea716c614f40805
I also applied the #14837 PR change
The unit of thoughput is samples/per second
Each throughput is calcuated by average of 5 runs
ResNet
model: Resnet50
dataset: Imagenet
number of gpu: 8
epochs: 3 (only to test throughput)
preprocess command: sudo pip install gluoncv==0.2.0b20180625
command: python mxnet_benchmark/train_imagenet.py --use-rec --batch-size 128 --dtype float32 —num-data-workers 40 —num-epochs 3 —gpus 0,1,2,3,4,5,6,7 --lr 0.05 --last-gamma —mode symbolic —model resnet50_v1b —rec-train /home/ubuntu/data/train-passthrough.rec —rec-train-idx /home/ubuntu/data/train-passthrough.idx —rec-val /home/ubuntu/data/val-passthrough.rec —rec-val-idx /home/ubuntu/data/val-passthrough.idx
github repo: https://github.com/rahul003/deep-learning-benchmark-mirror.git*

Throughput Tables
cuDNN 7.5.1/NCCL 2.4.2
cuDNN 7.3.1/NCCL 2.3.4
Perforamnce Difference

CUDA 10
2831.54405
2821.9832
0.339%

CUDA 9.2
2832.36803
2843.28968
-0.384%

CUDA 9.0
2815.83939
2851.92915
-1.265%

**There is another performance regression with --batch-size 256 --dtype float16 --mode hybrid, please find more details on #14838
LSTM
model: LSTM
dataset: PTB(Penn Treebank)
number of gpu: 1
epochs: 10
command:
python2 benchmark_driver.py --framework mxnet --task-name mkl_lstm_ptb_symbolic --num-gpus 1 --epochs 10 --metrics-suffix test --kvstore local
python word_language_model/lstm_bucketing.py —num-hidden 650 —num-embed 650 —gpus 0 --epochs 10 --kv-store local

Throughput Tables
cuDNN 7.5.1/NCCL 2.4.2
cuDNN 7.3.1/NCCL 2.3.4
Perforamnce Difference

CUDA 10
847.98222
868.28966
-2.339%

CUDA 9.2
1005.25185
1051.06692
-4.359%

CUDA 9.0
1002.59081
1028.46962
-1.265%

The CUDA 10 have a performance regression issue, please see #14725 to find more details.
MLP
model: 3 dense layers with num_hidden=64 and relu as activation
dataset: MNIST
number of gpu: 1
epochs: 10
command:
python2 benchmark_runner.py —framework mxnet —metrics-policy mlp —task-name mlp —metrics-suffix test —num-gpus 1 —command-to-execute 'python3 mlp.py' —data-set mnist

Throughput Tables
cuDNN 7.5.1/NCCL 2.4.2
cuDNN 7.3.1/NCCL 2.3.4
Perforamnce Difference

CUDA 10
4192.20685
4094.76838
2.38%

CUDA 9.2
4212.68214
4280.69164
-1.589%

CUDA 9.0
4232.10159
4273.43268
-0.967%

Comments
@szha @lanking520 @eric-haibin-lin @perdasilva

@mxnet-label-bot add[CI, pr-awaiting-merge]


system icon

This should be the GUIDE icon and should say NWB GUIDE

Note this is Mac-specific; Windows shows the proper icon on the toolbar


Upload & Download

Summary

Fixed uploads on Android so they are not blocking and multiple uploads can be performed at the same time.
Added download with progress indicator on Android and it saves the file to the specified path.

Pending

iOS download with progress and saving the file
JS implementation of the download method

@avinashlng1080 mentioning you just if you want to get familiar with this library

@migbot ready for review.


Warning: unknown command \Join

I see this whenever I load Makie:
┌ Warning: unknown command \Join
└ @ MathTeXEngine ~/.julia/packages/MathTeXEngine/ZP0gS/src/parser/commands_registration.jl:48

As a user, I can't act on this (I don't think?), and I didn't import MathTeXEngine (Makie did), so I'm not really in a position to do anything about it. I think if this a real problem, it should be a hard-error so Makie's CI fails and they don't make a release declaring compatability with this version of MathTeXEngine, and if it's not a real problem, then it should emit a @debug log or such so it doesn't show up every session for downstream users.

That's a small problem I somehow let slipped through. It is solved on the lastest version of MathTeXEngine.jl, but a bit of work is needed for compat with Makie.
Once I am done with https://github.com/JuliaPlots/Makie.jl/pull/1785 and it is merged, it will go away.

If it's just a small non-breaking fix, you could fix it on a branch going off the last minor version and tag it as a patch release. Then it wouldn't have to wait for Makie merging the minor version.

Fixed by https://github.com/JuliaPlots/Makie.jl/pull/1952


Disable i128 integers on Emscripten targets

Emscripten doesn't currently support i128 and will fail with the following error if they're used:
LLVM ERROR: Instruction not yet supported for integer types larger than 64 bits

Thanks! I published this in 1.0.74.


Add "azure-load-balancer-mode" to document

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
I could find no document and release note about "service.beta.kubernetes.io/azure-load-balancer-mode" annotation.
https://github.com/kubernetes/kubernetes/blob/release-1.10/pkg/cloudprovider/providers/azure/azure_loadbalancer.go#L52
Proposed Solution:
Add a document about "service.beta.kubernetes.io/azure-load-balancer-mode" annotation.
Page to Update:
https://kubernetes.io/docs/concepts/services-networking/service/#type-loadbalancer

/remove-lifecycle stale

/assign @feiskyer


Web: Cursor jumps to the start, if TextField (with Autofill) changes size or transform

Description

If an TextField with the autofill parameter (this is important) gets an error from a StreamBuilder / FutureBuilder the cursor goes to the start of the TextField and it isn't possible to type anything new.
Steps to Reproduce

Code to reproduce
import 'dart:math';

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  Future<int> throwExecptionAfter3Seconds() async {
    await Future.delayed(const Duration(seconds: 3));
    throw Random().nextInt(1000);
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Demo',
      home: Material(
        child: Padding(
          padding: const EdgeInsets.all(32),
          child: Center(
            child: FutureBuilder<void>(
              future: throwExecptionAfter3Seconds(),
              builder: (context, snapshot) {
                return TextField(
                  autofillHints: [AutofillHints.email], // Bug works only with this line!
                  decoration:
                      InputDecoration(errorText: snapshot.error?.toString()),
                );
              },
            ),
          ),
        ),
      ),
    );
  }
}

Run code.
Type something before the error (from FutureBuilder) is thrown --> error will be thrown after 3 seconds.
See that cursor at the beginning and typing isn't possible anymore.

Flutter doctor

Flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.21.0-10.0.pre.55, on Mac OS X 10.15.5 19F101,
    locale en-DE)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.47.3)

[✓] Connected device (4 available)
    ! Error: iPhone von Roland is not connected. Xcode will continue when iPhone
      von Roland is connected. (code -13)

• No issues found!

cc: @nturgut @LongCatIsLooong

I couldn't reproduce the bug on iOS or Android. I followed the steps and the cursor stayed at the end of the text, and the text field remained responsive, after the error text showed up. I'm on Channel master, 1.21.0-10.0.pre.70.

Can't reproduce on Android or Windows desktop, didn't test MacOS desktop.
iOS:
If I place the cursor at the beginning, can't type at all anymore, moving cursor doesn't help.
Steps used for web

flutter build web
cd build/web
http-server (Node.js package)

Windows web:

Chrome: Cursor jumps to beginning, can't type anymore. Can type again after repositioning cursor.
Firefox: Cursor doesn't move, can type. If I place the cursor at the beginning, can't type at all anymore, moving cursor doesn't help.
Edge: Weird double cursor issue and field gets cleared randomly but this doesn't repro.

MacOS web:

Chrome: Same as windows
Firefox: Same as windows. Also cursor gets more and more offset from the end of the text the more I type?
Safari: Cursor doesn't move, can't type. Can type again after repositioning cursor. Same issue with cursor offset.

In any case, the behavior is very buggy here. Marking as reproducible on latest master 1.21.0-10.0.pre.70.

flutter doctor -v
[√] Flutter (Channel master, 1.21.0-10.0.pre.70, on Microsoft Windows [Version 10.0.19041.450], locale en-US)
    • Flutter version 1.21.0-10.0.pre.70 at C:\Development\flutter_master
    • Framework revision 0629030a05 (5 hours ago), 2020-08-13 00:01:03 -0400
    • Engine revision 7571e7c380
    • Dart version 2.10.0 (build 2.10.0-11.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\marku\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.6.30320.27
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.0)
    • Android Studio at C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (5 available)
    • SM G950F (mobile)           • ce12171c51cc001c03 • android-arm64  • Android 9 (API 28)
    • sdk gphone x86 arm (mobile) • emulator-5554      • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)           • windows            • windows-x64    • Microsoft Windows [Version 10.0.19041.450]
    • Web Server (web)            • web-server         • web-javascript • Flutter Tools
    • Chrome (web)                • chrome             • web-javascript • Google Chrome 84.0.4147.125

• No issues found!

Dartpad seems to produce the same results https://dartpad.dev/61afe9770fdfc4334eb4aa31e016d889

@AndroidNils  can you share an example code piece?
@markusaksli-nc unfortunately Dart Pad is behind. It is best to test Autofill related features on the master channel.

I forgot to mention this: This is an only Flutter web issue!
@nturgut There is already an example in the description. Again:
import 'dart:math';

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  Future<int> throwExecptionAfter3Seconds() async {
    await Future.delayed(const Duration(seconds: 3));
    throw Random().nextInt(1000);
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Demo',
      home: Material(
        child: Padding(
          padding: const EdgeInsets.all(32),
          child: Center(
            child: FutureBuilder<void>(
              future: throwExecptionAfter3Seconds(),
              builder: (context, snapshot) {
                return TextField(
                  autofillHints: [AutofillHints.email], // Bug works only with this line!
                  decoration:
                      InputDecoration(errorText: snapshot.error?.toString()),
                );
              },
            ),
          ),
        ),
      ),
    );
  }
}

I'm looking at this example here are a few things:

On web only having email field won't be a good design for most browsers. The information will be filled for sure. However, Chrome for example recognizes the form after some fields are all filled and save the information. (This is not related to web, it is how browsers are designed.)
Why there is an exception thrown in 30 seconds? If you can explain with a real life situation?

Ah, I see the issue is not the exception or the errors. The issue is the location is changing, framework triggers the size/transform message.
I'm renaming the issue.

@nturgut Example: You have a login form, where a BLoC validates the data. If the user enters a wrong character, the BLoC will throw an error (via a StreamBuilder), like "Invalid character", "Invalid mail", "Password is too short", "Password is no correct". If this happens, the cursor jumps to the start and the user can not enter the correct data.

@nturgut Thank you for the quick fix! 💙


Support new h5py has a "locking" argument

Closes #3541
I tested  src/silx/io/test/test_h5py_utils.py against different h5py/libhdf5 on Ubuntu
h5py: 2.8.0, hdf5: 1.10.4
h5py: 2.9.0, hdf5: 1.10.4
h5py: 2.10.0, hdf5: 1.10.4
h5py: 2.10.0, hdf5: 1.10.6 CONDA
h5py: 3.1.0, hdf5: 1.12.0
h5py: 3.2.0, hdf5: 1.12.0
h5py: 3.3.0, hdf5: 1.12.0
h5py: 3.3.0, hdf5: 1.10.6 CONDA
h5py: 3.4.0, hdf5: 1.12.1 REMARK
h5py: 3.5.0, hdf5: 1.12.1

and on Windows
h5py: 2.10.0, hdf5: 1.10.5
h5py: 3.1.0, hdf5: 1.12.0
h5py: 3.2.0, hdf5: 1.12.0
h5py: 3.3.0, hdf5: 1.12.0
h5py: 3.4.0, hdf5: 1.12.1 REMARK
h5py: 3.5.0, hdf5: 1.12.1

No manual tests done on Mac.
REMARK: skips test_modes_multi_process due to  "Versions of libhdf5 and h5py use incompatible file locking behavior"

Works on Linux but not on Mac and Win:
https://gitlab.esrf.fr/silx/bob/silx/-/pipelines/59323
Not sure yet why.

It seems that disabling the locking for reading while writing does not work on windows with libhdf5 1.12.1 or libhdf5 1.10.7 (i.e. new locking behavior):
>>> import h5py
>>> f = h5py.File("test.h5", mode="w")

>>> import h5py
>>> f=h5py.File("test.h5", locking=False)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\projects\silx\.virtualenvs\py38\lib\site-packages\h5py\_hl\files.py", line 507, in __init__
    fid = make_fid(name, mode, userblock_size, fapl, fcpl, swmr=swmr)
  File "C:\projects\silx\.virtualenvs\py38\lib\site-packages\h5py\_hl\files.py", line 220, in make_fid
    fid = h5f.open(name, flags, fapl=fapl)
  File "h5py\_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py\_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "h5py\h5f.pyx", line 106, in h5py.h5f.open
PermissionError: [Errno 13] Unable to open file (file read failed: time = Fri Nov 12 20:01:17 2021
, filename = 'test.h5', file descriptor = 3, errno = 13, error message = 'Permission denied', buf =<PHONE_NUMBER>24DCD0, total read size = 8, bytes this sub-read = 8, bytes actually read =<PHONE_NUMBER>3709551615, offset = 0)

CI passes: https://gitlab.esrf.fr/silx/bob/silx/-/pipelines/59331

@t20100 @vasole Disabling file locking is less and less supported with increasing HDF5 version.
Concurrent access by disabling file locking is not supported in these cases:

mode != "r": causes file corruption
SWMR: does not apply
libver > v108: does not work
windows and HDF5_HAS_LOCKING_ARGUMENT: does not work

After this MR if you do File(filename, mode="r") in the last two cases you will get a ValueError saying

File locking is mandatory for HDF5 readers on Windows (only libhdf5 < 1.12.1 or libhdf5 < 1.10.7 support disabling file locking)
File locking is mandatory for HDF5 readers with lower version bound v112 (only v108 supports disabling file locking)

In those cases you need to explicitly call File(filename, mode="r", locking=True).
The reason I raise ValueError instead of silently setting locking = True is because such a reader could block a writer from opening the file and therefore block an ongoing experiment.
Here is the complete implementation of the locking behavior:
def _hdf5_file_locking(mode="r", locking=None, swmr=None, libver=None, **_):
    """Concurrent access by disabling file locking is not supported
    in these cases:

        * mode != "r": causes file corruption
        * SWMR: does not work
        * libver > v108: does not work
        * windows and HDF5_HAS_LOCKING_ARGUMENT: does not work

    :param str or None mode: read-only by default
    :param bool or None locking: by default it is disabled for `mode='r'`
                                 and `swmr=False` and enabled for all
                                 other modes.
    :param bool or None swmr: try both modes when `mode='r'` and `swmr=None`
    :param None or str or tuple libver:
    :returns bool:
    """
    if mode != "r":
        if not locking and locking is not None:
            raise ValueError("File locking is mandatory for HDF5 writers")
        return True
    if swmr:
        if not locking and locking is not None:
            raise ValueError("File locking is mandatory for SWMR HDF5 readers")
        return True
    if IS_WINDOWS and HDF5_HAS_LOCKING_ARGUMENT:
        if not locking:
            raise ValueError(
                "File locking is mandatory for HDF5 readers on Windows (only libhdf5 < 1.12.1 or libhdf5 < 1.10.7 support disabling file locking)"
            )
        return True
    low, _ = _effective_hdf5_libver_bounds(libver)
    if low != "v108":
        if not locking:
            raise ValueError(
                f"File locking is mandatory for HDF5 readers with lower version bound {low} (only v108 supports disabling file locking)"
            )
        return True
    return locking is True

I will replace the last two value errors with warnings.

Ok I made a better locking behavior:
def _hdf5_file_locking(mode="r", locking=None, swmr=None, libver=None, **_):
    """Concurrent access by disabling file locking is not supported
    in these cases:

        * mode != "r": causes file corruption
        * SWMR: does not work
        * libver > v108 and file already locked: does not work
        * windows and HDF5_HAS_LOCKING_ARGUMENT and file already locked: does not work

    :param str or None mode: read-only by default
    :param bool or None locking: by default it is disabled for `mode='r'`
                                 and `swmr=False` and enabled for all
                                 other modes.
    :param bool or None swmr: try both modes when `mode='r'` and `swmr=None`
    :param None or str or tuple libver:
    :returns bool:
    """
    if locking is None:
        locking = bool(mode != "r" or swmr)
    if not locking:
        if mode != "r":
            raise ValueError("Locking is mandatory for HDF5 writing")
        if swmr:
            raise ValueError("Locking is mandatory for HDF5 SWMR mode")
        if IS_WINDOWS and HDF5_HAS_LOCKING_ARGUMENT:
            _logger.warning(
                "Non-locking readers will fail when a writer has already locked the HDF5 file (this restriction applies to libhdf5 >= 1.12.1 or libhdf5 >= 1.10.7 on Windows)"
            )
        low, _ = _effective_hdf5_libver_bounds(libver)
        if low != "v108":
            _logger.warning(
                "Non-locking readers will fail when a writer has already locked the HDF5 file (this restriction applies to libver >= v108)"
            )
    return locking

The warning are there because the exception you will get from HDF5 in those cases is cryptic.
@t20100 Now you can review it.

LGTM.
I just spotted one list typo, enable_file_locking now being deprecated.

There is a failing test after refactoring. I'll fix it.


👂 Feedback - Adding a Service Principal to deploy ALZ-Bicep. -- Pre-Requisites to Wiki

Question/Feedback
Adding a Service Principal to deploy ALZ-Bicep. PR will be created to add Deploying ALZ-Bicep Pre Requisites
Possible Answers/Solutions?
Grant Access to the Service Principal
PowerShell:
#API calls have migrated from AAD Graph to Microsoft Graph
Install-Module Microsoft.Graph -AllowClobber -Force
Connect-Graph -Scopes "User.Read","Application.Read.All"
$spn = (Get-MgServicePrincipal -Filter "DisplayName eq '$ServicePrincipalName'")
New-AzRoleAssignment -Scope '/' -RoleDefinitionName 'Owner' -ObjectId $spn.Id

#Connect/Use Service Principal
$SecuredPassword = ConvertTo-SecureString -String $secretValue -AsPlainText -Force
$Credential = New-Object -TypeName System.Management.Automation.PSCredential ($ApplicationId, $SecuredPassword)
Connect-AzAccount -ServicePrincipal -TenantId $TenantId -Credential $Credential

@FallenHoot are you planning to create a PR for this? Just so we know 👍
Let us know 👍

Hey @FallenHoot, Just checking in to see if you want to contribute for this issue?
If we don't hear from you in the next 2 days, we will put on the backlog for ourselves to resolve👍

The ALZ Bicep team will pick this up

I try to setup the serviceprincipal permission, but it seems there is no clear documentation what we have to add.
I'm getting "AuthorizationFailed" error in Azure DevOps pipeline

I try to setup the serviceprincipal permission, but it seems there is no clear documentation what we have to add.
I'm getting "AuthorizationFailed" error in Azure DevOps pipeline

Hey @pherbel, do the docs here help at all? https://github.com/Azure/ALZ-Bicep/wiki/DeploymentFlow#deployment-identity

Finnally I could solve it.
I create service principal through Azure DevOps Service connection UI and after that I added 'Owner' role for Root Management Group in the Azure Portal.
But it seems in the portal you can't really add Owner for the Root ('/'),  It will add to the first group after the root.
This was helpful.
https://github.com/Azure/Enterprise-Scale/blob/main/docs/EnterpriseScale-Setup-azure.md
You really should use this to add for the root:
$spn = (Get-AzADServicePrincipal -DisplayName $spndisplayname).id
New-AzRoleAssignment -Scope '/' -RoleDefinitionName 'Owner' -ObjectId $spn

Glad you got it solved. We will get our docs updated here.
@shaunjacob see above. Need to make the docs clear on how to create the correctly scoped SPN

@shaunjacob any update on this one buddy?


[chore] Fix malformed copyright headers

Summary
While doing some other work I noticed that some of the templates had the header messed up - probably a side effect of when this was done: https://github.com/facebook/react-native/commit/8bd3edec88148d0ab1f225d2119435681fbbba33
So this is a simple search&replace to fix that.
Changelog

[Internal] [Fixes] - fix malformed copyright headers
Test Plan

N/A

closing in favour of: https://github.com/facebook/react-native/pull/32840


CLI: ACL for port is registering to local user instead of domain user

From Hai Liu:
It looks like my ‘netsh http show urlacl url=http://+:7071’ is locked to [machine_name]\hailiu
After I delete the urlacl and added redmond\hailiu to urlacl, then I can run ‘func host’ or ‘func run’ under non-admin cmd
Looks like there is a bug on the first run ‘func host’ under admin that is provisioning the wrong user (local user vs domain user)?

@ahmelsayed Please review https://github.com/Azure/Azure-Functions/issues/87 and see if it looks like the same root cause. It looks like it based on my reading of the thread.


python311Packages.slack-sdk: 3.26.1 -> 3.26.2

Diff: https://github.com/slackapi/python-slack-sdk/compare/refs/tags/v3.26.1...v3.26.2
Changelog: https://github.com/slackapi/python-slack-sdk/releases/tag/v3.26.2
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 279807 run on x86_64-linux 1

2 packages failed to build:

ntfy
ntfy.dist

8 packages built:

python310Packages.slack-bolt
python310Packages.slack-bolt.dist
python310Packages.slack-sdk
python310Packages.slack-sdk.dist
python311Packages.slack-bolt
python311Packages.slack-bolt.dist
python311Packages.slack-sdk
python311Packages.slack-sdk.dist

Result of nixpkgs-review pr 279807 run on x86_64-linux 1

10 packages built:

ntfy
ntfy.dist
python310Packages.slack-bolt
python310Packages.slack-bolt.dist
python310Packages.slack-sdk
python310Packages.slack-sdk.dist
python311Packages.slack-bolt
python311Packages.slack-bolt.dist
python311Packages.slack-sdk
python311Packages.slack-sdk.dist


fix(cmd): stop generators as last to avoid deadlock on exit

As jobs unconditionally read from generator's channel there can be
a situation when generator is closed and will not produce
any new values while some jobs still wait on empty channel deadlocking
and preventing process exit.

As jobs unconditionally read from generator's channel there can be a situation when generator is closed and will not produce any new values while some jobs still wait on empty channel deadlocking and preventing process exit.

Thanks for PR, it is better to make .Get return nil when context is Canceled. Please take a look at https://github.com/scylladb/gemini/pull/372

As jobs unconditionally read from generator's channel there can be a situation when generator is closed and will not produce any new values while some jobs still wait on empty channel deadlocking and preventing process exit.

Thanks for PR, it is better to make .Get return nil when context is Canceled. Please take a look at #372

Why is it better? You're adding select cost to each request sent.

As jobs unconditionally read from generator's channel there can be a situation when generator is closed and will not produce any new values while some jobs still wait on empty channel deadlocking and preventing process exit.

Thanks for PR, it is better to make .Get return nil when context is Canceled. Please take a look at #372

Why is it better? You're adding select cost to each request sent.

It is better because it does not create more intertwining between contexts as result you do not care when to close generators context. But your idea is even better - we need to drop context from the partition and close channels instead when generator cycle is ended.

Technically they have no relation to each other because those are 2 different contexts - not connected. I agree that doing some dance around closing channels could be better in principle but it is much more risky patch - as per sending to closed channel case, and it'd introduce separate closing mechanism to already existing one which would be confusing.
I think it's common pattern that dependency is created first and destructed last, which is the case here. Not ideal though as it's a weird pattern that generator has it's own waitgroup which nobody waits for.

As jobs unconditionally read from generator's channel there can be a situation when generator is closed and will not produce any new values while some jobs still wait on empty channel deadlocking and preventing process exit.

Thanks for PR, it is better to make .Get return nil when context is Canceled. Please take a look at #372

Why is it better? You're adding select cost to each request sent.

It is better because it does not create more intertwining between contexts as result you do not care when to close generators context. But your idea is even better - we need to drop context from the partition and close channels instead when generator cycle is ended.

Technically they have no relation to each other because those are 2 different contexts - not connected. I agree that doing some dance around closing channels could be better in principle but it is much more risky patch - as per sending to closed channel case, and it'd introduce separate closing mechanism to already existing one which would be confusing.
I think it's common pattern that dependency is created first and destructed last, which is the case here. Not ideal though as it's a weird pattern that generator has it's own waitgroup which nobody waits for.

I would rather prefer to make it straigh than further entangle logic even if it requires more effort.

That'd be best, but ending some things by context cancellation and some things by channel closing or default select I'd not say it's straight.

As jobs unconditionally read from generator's channel there can be a situation when generator is closed and will not produce any new values while some jobs still wait on empty channel deadlocking and preventing process exit.

Thanks for PR, it is better to make .Get return nil when context is Canceled. Please take a look at #372

Why is it better? You're adding select cost to each request sent.

It is better because it does not create more intertwining between contexts as result you do not care when to close generators context. But your idea is even better - we need to drop context from the partition and close channels instead when generator cycle is ended.

Technically they have no relation to each other because those are 2 different contexts - not connected. I agree that doing some dance around closing channels could be better in principle but it is much more risky patch - as per sending to closed channel case, and it'd introduce separate closing mechanism to already existing one which would be confusing.
I think it's common pattern that dependency is created first and destructed last, which is the case here. Not ideal though as it's a weird pattern that generator has it's own waitgroup which nobody waits for.

I would rather prefer to make it straigh than further entangle logic even if it requires more effort.

That'd be best, but ending some things by context cancellation and some things by channel closing or default select I'd not say it's straight.

Agreed, but channel closing is not a way to cancel something it is a necessary precedure of closing generation cycle.
That is exactly why I would not want to propagate context cancelation as means of sending singal to stop anything.
I would want to move all that logic into stop.Flag and relay only on it.

Addressed at #372


[Model Control Plane] link cached artifacts

Describe changes
I fixed the linking of cached artifacts to Model Versions in the Control Plane to achieve consistency between cached and non-cached runs of pipelines dealing with Models in the Control Plane.
The gist of this bugfix is in _link_cached_artifacts_to_model_version function. Might be not the nicest one, though.
Ignore changes in tests/integration/functional/steps/test_model_config.py - This is just a port of stability improvement for non-related tests from another branch.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[x] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Ah, there was actually an implementation bug... In cached linkage, I modified the artifact_config object, which changed the step annotation as a side effect. I doubt that it could impact real cases, but now it is fixed in eb16111


Dev: does not allow changing accounts after sign out

Also seen in dev now.

Also reproed locally.


Error in graphhopper for android

hi , I'm using graphhopper for android , before I run this project in android studio and every thing was ok. but now I run that project and that have error ,that error is :
" chweighting [fastest|car] is not equal to [loaded [FASTEST|car]"
what should I do?

We changed a bit the format. This is nothing you should mess with, just make sure you do not change the GraphHopper version or reimport the area.

I didn't change Graphhopper version :(
where can I download 0.5 ?

Thanks a lot, I used 0.5 , and it worked correct


Add type definitions to fix typescript project

Added type definitions to "autoHide" and "forceVisible" to fix erros on use it on typescript projects

Thanks!