Fail arguments parsing upon unknown argument

When using profiler.sh this probably can't happen, but for others writing other frontends, this can catch sad errors like typos in passed arguments, which are currently ignored :( Also, it might catch errors in existing frontends, who knows.
It would be nicer if the returned error included the argument name, but I see that the Error class expects a long-living char * pointer, and does no memory management on its own, so I can't pass a stack buffer (which is deallocated by the point the string is used) or a heap buffer (which will be leaked). Additionally, the Log object is not yet initialized, so I can't use it either.
A static string is still better than nothing, hence I'm proposing this PR as is.
I can maybe replace } else { with #define DEFAULT so it looks nicer? done

Even though the fix looks trivial, I wouldn't call it absolutely harmless.
Ignoring unsupported parameters solved certain compatibility issues, and I used to exploit this before. Recent example: our network runs hundreds Java applications, they update in different time and thus may have different versions of async-profiler agent. After introducing a new sched option, we added it to our profiling system, not caring if the target supports it or not. On newer applications we automatically started collecting profiles broken down by scheduling policy; older application profiles remained as they were.
I admit that my example is probably not very good; I just wanted to say that the proposed change is not really minor - may be, it's better to postpone it until the next major release.
A special problem is the handling of an empty option, which is treated as invalid now. Consider a script that builds a profiler command line like this:
PARAMS="$PARAMS,bar"

It may easily happen that a command line starts with , or contains two commas in a row: foo,,bar. Async-profiler parser should definitely tolerate this.

Hmm, I understand from your example on how it can be exploited easier configuration.
On the other hand - it sounds like the sched example is an opt-in feature that works in best-effort mode, so you are okay with it working where possible. What if you're adding a new argument that's not best-effort? For example, changes the profiling configuration in a non-compatible way with the older configuration; or enabling stability features like --safemode. For those, you want to be absolutely sure that you get what you request.
The empty options case - we can easily make the profiler ignore them, yeah.
All in all, I agree that it's not a minor change and it may break existing frontends which are not prepared. async-profiler is yours and it's up to you if you want it :) I think it's an important robustness improvement for the frontend side. We can definitely wait for the next major release.

Despite (dis)advantages of both approaches, I agree that failing on unknown options is better overall.
Please make sure the parser tolerates extra commas, and I'll definitely accept the change for async-profiler 3.0.

I start working on the next major release, so it's a good time to merge this change.


Resolve some warnings in VS2013

Resolve warning C4100: 'l' : unreferenced formal parameter
Resolve warning C4244: 'return' : conversion from 'lua_Integer' to 'int', possible loss of data

Thanks for the PR. Makes me wonder though if we should instead tell the user to compile Lua with a correctly sized integer depending on whether they are linking it to a 32 or 64 bit executable.


KtLint 0.40.0 has changed mechanism for file path loading, we face errors when .editorconfig is missing

We can't merge #615 now, because file_path is not present in user data. Ktlint now uses editorconfig (EditorConfigLoader.loadPropertiesForFile), we need to workaround it somehow.

Similar bugs are reported in ktlint already: https://github.com/pinterest/ktlint/issues/997

They have solved this on their side, once ktlint 0.41 is out is should be safe for us to upgrade.

For ktlint 0.41, different configuration is required for gradle dependency: https://github.com/pinterest/ktlint/issues/1114

But this configuration also results in the following warning:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.jetbrains.kotlin.com.intellij.util.ReflectionUtil (file:/home/***/.gradle/caches/modules-2/files-2.1/org.jetbrains.kotlin/kotlin-compiler-embeddable/1.4.31/6451ea797cef544e81f507b0d9959cd97ae09c0/kotlin-compiler-embeddable-1.4.31.jar) to method java.util.ResourceBundle.setParent(java.util.ResourceBundle)
WARNING: Please consider reporting this to the maintainers of org.jetbrains.kotlin.com.intellij.util.ReflectionUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

@petertrr what do you think? Are these warnings related to ktlint?

It looks like it is a bug in Kotlin compiler: https://youtrack.jetbrains.com/issue/KT-43520

@dimsuz  this warning indeed originates from ktlint, or even from kotlin compiler. It has been there in previous versions too, and it is indeed a warning, i.e. ktlint still works as expected, until this will be deprecated in JVM.

I see, I'll star it, thanks!


Hackage release <IP_ADDRESS>

I finally got around to uploading this to hackage, here. Since I use this for work, I'm pointing hackage to my fork. (@Lucsanszky, I've invited you to collaborate via github).
Changelog includes:

remove OpenAI.Gym.Prelude - it felt weird to include a prelude for a package of this size (Internal wasn't necessary since there are now only two modules).
haddocks everything
lock in lower bounds (maps to stack lts-8.1 which pins GHC to >= 8.0)
migrate back to newtypes

RE: @Lucsanszky, @catherio

For future reference, because this PR is not getting merged it is now becoming a live feed of any updates to stites/gym-http-api's master (which is being pushed to stackage and hackage)


Depth PointCloud and RGB overlay problem with Stereo_S_U3

Hi,
I've got a problem with the Stereo_S_U3 that is very restrictive for 3d mapping.
If I display those 3 topics into Rviz, they are not stackable :
/camera/depth_registered/points
/camera/depth_registered/image_raw
/camera/rgb/image_rect_color
The PointCloud2 topic is really more accurate that the DepthCloud one, but unfortunately it's the second one that I want to use.
As you can see on this screen shot, the blue data is the DepthCloud image_raw and the red one is the PointCloud2 points:

there are about 10cm bias vertically and 5cm horizontally.
If I take a look to the overlay between those both depth data and the rgb (cropped to 640x400 to be consistent with the depth image resolution), I can see:

an horizontal bias of about 3cm
image size is not consistent

For the depth data it's worst, I can see:

a vertical bias of about 10 cm
Image size is not consistent

Previously I worked with an Astra S and there were no problems.
Any idea to resolve this problem is really welcome.
thanks

This is also something we reported here: #111 but we did not get any answer


form validation in adaptive card

Hi Team,
I am developing a bot in sdk v4 using node js. I have a form in a bot and need to validate the fields like email, phone number etc. Is there a way to do it?

The Microsoft Bot Framework team prefers that how to questions be submitted on Stack Overflow. The official Bot Framework Github repos are the preferred platform for submitting bug fixes and feature requests.  When you post your question on StackOverflow, please elaborate on your issue and be sure to provide code, approaches you have already tried, and possibly screenshots so we can best guide you in the right direction. Specifically, I would add more detail to what you mean by a form in your bot.
Also note, any bugs or feature requests related to the NodeJs SDK should be opened in the BotBuilder-js Repo.


How to debug helm validation failed ?

Hello,
When upgrading a release, I have a failure about validation failing. I would like to get full templates rendered so I can debug this, how to achieve that ?
✗ HelmRelease reconciliation failed: Helm upgrade failed: error validating "": error validating data: [ValidationError(Deployment.spec.template.spec.containers[0].ports[0]): missing required field "containerPort" in io.k8s.api.core.v1.ContainerPort, ValidationError(Deployment.spec.template.spec.containers[0].ports[1]): missing required field "containerPort" in io.k8s.api.core.v1.ContainerPort]

To solve and understand this, I had to execute a helm template manually by using the values from the helmrelease.

Yes I tried this but I got the same error so it was not enough the understand the error.

Thanks but I think we should bundle the debug command inside flux to get the templates errors.


S2C Support for DIO 10  thru DIO 14

Hi Jeff,
I downloaded the latest source for the XBee library. DIO0 thru DIO9 work. However, DIO10 thru DIO14 still don't work. I applied my mods to the source and got them working again. I was really hoping you would think about adding library support for DIOs above 9. I'm not sure how you would do it in the big picture of all the different flavors of XBee. This is what I'm doing to support my S2C version of the XBee.
// INPUT OUTPUT CONFIGURATION

// MODIFIED: InputOutputConfigurationCommand.cs
// Above D9 starts with P0 so I changed the Constructor

public InputOutputConfigurationCommand( InputOutputChannel channel ) :
		 base( (int)channel > 9 ? $"P{(int)channel - 10}" : $"D{(int)channel}" ) {
}

// EXTENDED: InputOutputChannel.cs
// It stopped at Channel 9
public enum InputOutputChannel {
	Channel0 = 0,
	Channel1 = 1,
	Channel2 = 2,
	Channel3 = 3,
	Channel4 = 4,
	Channel5 = 5,
	Channel6 = 6,
	Channel7 = 7,
	Channel8 = 8,
	Channel9 = 9,
	Channel10 = 10,
	Channel11 = 11,
	Channel12 = 12,
	Channel13 = 13,
	Channel14 = 14
}

// EXTENDED: DigitalSampleState.cs
public enum DigitalSampleState : ushort {
	None = 0x0,
	Input0 = 0x1,
	Input1 = 0x2,
	Input2 = 0x4,
	Input3 = 0x8,
	Input4 = 0x10,
	Input5 = 0x20,
	Input6 = 0x40,
	Input7 = 0x80,
	Input8 = 0x100,
	Input9 = 0x200,
	Input10 = 0x400,
	Input11 = 0x800,
	Input12 = 0x1000,
	Input13 = 0x2000,
	Input14 = 0x4000,

	All = Input0 | Input1 | Input2 | Input3 | Input4 | Input5
			| Input6 | Input7 | Input8 | Input9 | Input10 | Input11 | Input12 | Input13 | Input14
}

// EXTENDED: DigitalSampleChannels.cs
public enum DigitalSampleChannels : ushort {
	None = 0x0,
	Input0 = 0x1,
	Input1 = 0x2,
	Input2 = 0x4,
	Input3 = 0x8,
	Input4 = 0x10,
	Input5 = 0x20,
	Input6 = 0x40,
	Input7 = 0x80,
	Input8 = 0x100,
	Input9 = 0x200,
	Input10 = 0x400,
	Input11 = 0x800,
	Input12 = 0x1000,
	Input13 = 0x2000,
	Input14 = 0x4000
}

// ADDED TO: RxIndicatorSampleExtFrame.cs
[SerializeWhen( "DigitalChannels", DigitalSampleChannels.Input13,
	 ConverterType = typeof( BitwiseAndConverter ), ConverterParameter = DigitalSampleChannels.Input13 )]
[SerializeWhen( "DigitalChannels", DigitalSampleChannels.Input14,
	 ConverterType = typeof( BitwiseAndConverter ), ConverterParameter = DigitalSampleChannels.Input14 )]

Anyway, I am using these mods and they do work for the S2C XBee. My favorite was finding out that Configuration Commands above D9 start over with P0. To me, the engineer that came up with that, just wanted to do something really evil ! ! !

Ok, sorry about that. I'll try to get it working this weekend.
On Fri, May 13, 2016 at 7:54 AM -0700, "PaulNoto"<EMAIL_ADDRESS>wrote:
Hi Jeff,
I downloaded the latest source for the XBee library. DIO0 thru DIO9 work. However, DIO10 thru DIO14 still don't work. I applied my mods to the source and got them working again. I was really hoping you would think about adding library support for DIOs above 9. I'm not sure how you would do it in the big picture of all the different flavors of XBee. This is what I'm doing to support my S2C version of the XBee.
// INPUT OUTPUT CONFIGURATION
// MODIFIED: InputOutputConfigurationCommand.cs
// Above D9 starts with P0 so I changed the Constructor
public InputOutputConfigurationCommand( InputOutputChannel channel ) :
base( (int)channel > 9 ? $"P{(int)channel - 10}" : $"D{(int)channel}" ) {
}
// EXTENDED: InputOutputChannel.cs
// It stopped at Channel 9
public enum InputOutputChannel {
Channel0 = 0,
Channel1 = 1,
Channel2 = 2,
Channel3 = 3,
Channel4 = 4,
Channel5 = 5,
Channel6 = 6,
Channel7 = 7,
Channel8 = 8,
Channel9 = 9,
Channel10 = 10,
Channel11 = 11,
Channel12 = 12,
Channel13 = 13,
Channel14 = 14
}
// EXTENDED: DigitalSampleState.cs
public enum DigitalSampleState : ushort {
None = 0x0,
Input0 = 0x1,
Input1 = 0x2,
Input2 = 0x4,
Input3 = 0x8,
Input4 = 0x10,
Input5 = 0x20,
Input6 = 0x40,
Input7 = 0x80,
Input8 = 0x100,
Input9 = 0x200,
Input10 = 0x400,
Input11 = 0x800,
Input12 = 0x1000,
Input13 = 0x2000,
Input14 = 0x4000,
All = Input0 | Input1 | Input2 | Input3 | Input4 | Input5
        | Input6 | Input7 | Input8 | Input9 | Input10 | Input11 | Input12 | Input13 | Input14

}
// EXTENDED: DigitalSampleChannels.cs
public enum DigitalSampleChannels : ushort {
None = 0x0,
Input0 = 0x1,
Input1 = 0x2,
Input2 = 0x4,
Input3 = 0x8,
Input4 = 0x10,
Input5 = 0x20,
Input6 = 0x40,
Input7 = 0x80,
Input8 = 0x100,
Input9 = 0x200,
Input10 = 0x400,
Input11 = 0x800,
Input12 = 0x1000,
Input13 = 0x2000,
Input14 = 0x4000
}
// ADDED TO: RxIndicatorSampleExtFrame.cs
[SerializeWhen( "DigitalChannels", DigitalSampleChannels.Input13,
ConverterType = typeof( BitwiseAndConverter ), ConverterParameter = DigitalSampleChannels.Input13 )]
[SerializeWhen( "DigitalChannels", DigitalSampleChannels.Input14,
ConverterType = typeof( BitwiseAndConverter ), ConverterParameter = DigitalSampleChannels.Input14 )]
Anyway, I am using these mods and they do work for the S2C XBee. My favorite was finding out that Configuration Commands above D9 start over with P0. To me, the engineer that came up with that, just wanted to do something really evil ! ! !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Pushed as part of 3.0.1.  Thanks!

Tested. DIO 0 thru DIO 14 work for Configuration and Change Detection. Thanks Jeff.

Awesome!  No problem
On Sat, May 14, 2016 at 4:55 PM -0700, "PaulNoto"<EMAIL_ADDRESS>wrote:
Tested. DIO 0 thru DIO 14 work for Configuration and Change Detection. Thanks Jeff.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub


Update control radius

Task: task-988024
Link: preview
Update the border radius to 4 px instead of 2 px
Testing

git checkout ellyn/control-radius, npm start
Note the rounded corners on the following pages:
Button: http://localhost:1111/components/button.html
Input: http://localhost:1111/components/input.html
Checkbox: http://localhost:1111/components/checkbox.html

Contributor checklist

[ ] Did you update the description of this pull request with a review link and test steps?
[ ] Did you submit a changeset? Changesets are required for all code changes.
[ ] Does your pull request implement complex UI logic (js/ts)? Consider adding an integration test to test your user flow.
[ ] Does your pull request add a new page to the documentation site? Add your new page for automated accessibility testing in /integration/tests/accessibility.

I'm not sure this is the right conversation for this PR (so I'm approving regardless), but maybe worth having down the road: are rems the right units for border radii? Do border radii really need to scale up and down based on the font size?

I'm not sure this is the right conversation for this PR (so I'm approving regardless), but maybe worth having down the road: are rems the right units for border radii? Do border radii really need to scale up and down based on the font size?

It looks like Fluent's card radius do change between different sizes, but the change is not based on the font size.


changing healthbars length

Describe your problem here.
Is there a way to change the healthbars length going into a song using lua? im just wondering as i need it for a mod
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
i have only added characters.

setProperty('healthBar.scale.x', 2)

do you possibly know how to make the health go past its original health? (like make it balanced with the healthbar)

do you possibly know how to make the health go past its original health? (like make it balanced with the healthbar)

Nope

do you possibly know how to make the health go past its original health? (like make it balanced with the healthbar)

What do you mean with that, because you can use setProperty('healthBar.scale.x', 1), or you can use:
if(getProperty('health') > value) then
  setProperty('health', maxHealth)
end

(Note, if you use the second code, replace value by the maximum health, and replace maxHealth by the maximum health


babs-init raises exception when there is no ses- folders in a non sub- folder.

When attempting to run babs-init, it determined that a folder in my BIDS directory 'code' was a subject and raised an exception:

Checking whether each input dataset is a zipped or unzipped dataset...
input dataset '1080_wasabi' is considered as an unzipped dataset.
Performing sanity check for any unzipped input dataset...
babs-init failed! Below is the error message:
Traceback (most recent call last):
File "/dartfs-hpc/rc/home/j/f003z4j/miniconda3/lib/python3.9/site-packages/babs/cli.py", line 234, in babs_init_main
babs_proj.babs_bootstrap(input_ds,
File "/dartfs-hpc/rc/home/j/f003z4j/miniconda3/lib/python3.9/site-packages/babs/babs.py", line 400, in babs_bootstrap
check_validity_unzipped_input_dataset(input_ds, self.type_session)
File "/dartfs-hpc/rc/home/j/f003z4j/miniconda3/lib/python3.9/site-packages/babs/utils.py", line 95, in check_validity_unzipped_input_dataset
raise Exception(
Exception: In input dataset #1 '1080_wasabi', there is no ses-* folder in subject folder 'code'!

Hi Michael! What's that code folder within your BIDS dataset used for? My understanding is that code should not be located within your BIDS dataset. Could you please remove this code folder?

@zhao-cy I think it's fairly common to have code directories in BIDS, along with other directories like source_data. We should have BABS only look for sub-directories directly under the BIDS root

Ah sorry about this! Yes that's indeed a bug in BABS - thanks for letting us know, Michael! I'll fix it asap.
Note to myself: see function check_validity_unzipped_input_dataset() in utils.py. Seems this issue is only in validating the input datasets, but not when listing the subjects/sessions for analysis.


Add Asciify

Adds asciify feature, closes #343

apologies for the delayed review, am super busy rn but will try and look at this at some point!

No problem, I'm still working on improving the tests with unittest mock

yeah this is a known issue, just duplicate the strings for now


feat(sbb-tooltip): introduce sbb-tooltip component

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #1025
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

See Review Guidelines for more information what is checked during review process.
Changes
Changes in this pull request:

implement tooltip component

Browsers
I tested the build on the following browsers:

[x] Firefox Desktop
[x] Chrome Desktop
[x] Edge Desktop
[x] Safari Desktop
[x] Chrome Mobile
[x] Safari Mobile

Screen readers
I tested the build on the following browsers:

[x] JAWS Firefox Desktop
[x] JAWS Chrome Desktop
[x] NVDA Firefox Desktop
[x] NVDA Chrome Desktop
[x] VoiceOver Safari Desktop
[x] VoiceOver Chrome Desktop
[x] VoiceOver Safari Mobile
[x] Android Accessibility Suite Chrome Mobile

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

ux review:

[ ] issue-1: https://www.figma.com/file/9r6xSfNmEfCFxl1yFYedrj?node-id=13858:467257#294511986
[ ] issue-2: https://www.figma.com/file/9r6xSfNmEfCFxl1yFYedrj?node-id=13858:469623#294512739
[ ] issue-3: the animation seems a little wild to me. can we make it a little more decent, like the menu? or like this: https://tympanus.net/Development/TooltipStylesInspiration/ (first example)

@dauriamarco perfetto

Codecov Report

Merging #1425 (f85435a) into master (ab917b0) will increase coverage by 3.01%.
The diff coverage is 65.19%.

@@            Coverage Diff             @@
##           master    #1425      +/-   ##
==========================================
+ Coverage   54.85%   57.86%   +3.01%
==========================================
  Files          49       66      +17
  Lines        1659     2447     +788
  Branches      406      654     +248
==========================================
+ Hits          910     1416     +506
- Misses        671      938     +267
- Partials       78       93      +15

Impacted Files
Coverage Δ

src/components/sbb-clock/sbb-clock.tsx
0.00% <0.00%> (ø)

src/components/sbb-logo/sbb-logo.tsx
0.00% <0.00%> (ø)

src/components/sbb-menu/sbb-menu.tsx
26.49% <ø> (ø)

...ents/sbb-pearl-chain-time/sbb-pearl-chain-time.tsx
100.00% <ø> (ø)

...in-vertical-item/sbb-pearl-chain-vertical-item.tsx
93.75% <ø> (ø)

...-pearl-chain-vertical/sbb-pearl-chain-vertical.tsx
100.00% <ø> (ø)

src/components/sbb-pearl-chain/sbb-pearl-chain.tsx
67.74% <ø> (-3.69%)
:arrow_down:

src/components/sbb-section/sbb-section.tsx
85.00% <ø> (ø)

src/components/sbb-signet/sbb-signet.tsx
0.00% <ø> (ø)

...mponents/sbb-slot-component/sbb-slot-component.tsx
100.00% <ø> (ø)

... and 47 more

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


Android support?

It's c++ so it should work with JNI on Android? But I see no mention about it?

Yes,it should work,but jni interfaces are lot of work, There're a Java client already, it's the better choice

Thanks @melode11


Conformance: Training tests

This is the issue in KF project board tracking conformance tests.

@james-jwu We already have issues for Training Operator: https://github.com/kubeflow/training-operator/issues/1695 and Katib: https://github.com/kubeflow/katib/issues/2044. Should we re-use them to the project board ?

I will close the 2 issues I created. I have already linked your issues to the project board.


Convert catalog table to list

Lists offer more flexibility with display options.
https://material.io/design/components/lists.html
https://material-ui.com/components/lists/
Also, this would allow us to use react-window, which is a much smaller library than react-virtualized.

Before screenshot:

Will post After when ready.


Test on windows, macOS

Add a matrix strategy, macOS, windows testing.

Closing while I fix windows tests.


Clean up uninitialized dangling shards

metastore::open_shards succeeds
ingester::init_shards fails
uninitialized shards are not cleaned up

Closed via #5029.


Support hosting headscale with an HTTP path prefix

Add support for hosting headscale endpoints with a HTTP path prefix. This is a feature requested by several developers, mainly to support running headscale behind a reverse proxy where there are other backends which headscale should not conflict with.
Unfortunately, through testing the tailscale client implementation has some strange behavior when adding a path to --login-server. It respects this prefix for certain endpoints, but does not for /derp, /ts2021, and /bootstrap-dns.
Additionally, the admin API at /api/v1 breaks if you host it with a path prefix as the paths are hardcoded into the generated code from swagger.
These considerations were applied to this implementation so that only the supports paths will use the path prefix, and documentation was updated to clarify this.
Closes #1126

[x] read the CONTRIBUTING guidelines
[x] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[x] updated documentation if needed
[x] updated CHANGELOG.md

Hi @kev-the-dev, unfortunately we cannot merge this.
Having the API half-split in methods that work on a subpath and other than don't would add a lot of confusion and issues.
Reverse proxies are a source of user issues in Headscale, and this would add on top :(
--
Happy to have a chat on project priorities over DM in Discord :)


Updated log_poisson_loss() function

result is updated in the log_poisson_loss(targets, log_input, compute_full_loss=False, name=None) function. For input values too large, error message is displayed instead of returning NaN.

@Aravind-11
Line 98
result = math_ops.exp(log_input) - log_input * target
It should be targets instead of target.

@Aravind-11 Any update on this PR? Please. Thanks!

I'm not sure how to add unit test. Can someone help?

@Aravind-11, @mihaimaruseac Any update on this PR? Please. Thanks!

@Aravind-11, @mihaimaruseac Any update on this PR? Please. Thanks!

@Aravind-11, @mihaimaruseac Any update on this PR? Please. Thanks!

@Aravind-11, @mihaimaruseac Any update on this PR? Please. Thanks!

Error message is the way to go.
On Tue, 29 Mar 2022 at 7:18 PM, Surya Prakash Mishra <
@.***> wrote:

@.**** commented on this pull request.
In tensorflow/python/ops/nn_impl.py
https://github.com/tensorflow/tensorflow/pull/51765#discussion_r837499362
:

if(!result.is_nan()):

 raise ValueError(

     "Input contains NaN, infinity or a value too large for dtype('float64') (%s vs %s)" %

     (log_input.get_shape(), targets.get_shape()))

@Aravind-11 https://github.com/Aravind-11 can you please answer to
this. Should it return NaN or an error message is a better way to go?
TIA
—
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/51765#discussion_r837499362,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDCEOVSLAXUBJAXPV6DSSTVCMC2ZANCNFSM5DD3FICQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

See https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/nn_test.py

Also, please change PR title, https://cbea.ms/git-commit/

Sure!
On Wed, 30 Mar 2022 at 1:30 AM, Mihai Maruseac @.***>
wrote:

Also, please change PR title, https://cbea.ms/git-commit/
—
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/51765#issuecomment-1082318461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDCEOW5G36ITHPRGD3S7JDVCNONZANCNFSM5DD3FICQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Aravind-11 Any update on this PR? Please. Thank you!

@Aravind-11 Any update on this PR? Please. Thank you!


Julia: reinit at latest

Make build pass

Motivation for this change
Close #93333 and Fix https://github.com/NixOS/nixpkgs/issues/91930 .
Things done
Using ofborg to test the builds.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build julia

Build timeout ... I guess there's no choice other then failing locally.

So the plan is not to have LTS?

So the plan is not to have LTS?

I think maybe removing it was unintentional? @doronbehar said elsewhere:

I'm experimenting a more broad change to the situation at #98043 . There, only the derivations julia_lts and julia will be left, and hopefully they will build at some point, with most of our deps used instead of upstream's.

I worded my question based on the assumption that between words and code, the latter is the authoritative source — maybe I have just underestimated the WIP part, of course.
(But it is a question either way, not a statement, of course)

In any case, I think that julia 1.0 ought to be included, and I would also suggest that it be called julia_10 or julia_10_lts rather than julia_lts, as people may wish to continue using it even after a new LTS release has been declared, and ideally one package will not upgrade to another without a human explicitly changing a derivation.  Says the blog post on Julia's Release Process:

Fortunately, even [a new LTS branch] does not force people using Julia 1.0.x to upgrade: they can keep using the last 1.0.x version and packages that are compatible with it. At that point it will be the most stable, thoroughly tested, patched version of Julia in existence, so it will be safe to keep using indefinitely if one doesn't need newer features.

Oh please folks, I didn't intent to discard the LTS, it's just that it's expression wasn't ready yet and I wanted to get CI happy until 1.5.1 was working, though now it doesn't matter as I'm testing the build locally.

Also, I wouldn't lend high hopes in the work here. I'm getting weird "file not found" errors during build, when trying to use as many external deps as possible. I'm using direnv and lorri and this .envrc and shell.nix put in where the tarball was extracted:
# .envrc
eval "$(lorri direnv)"
PATH=$PATH:$CMAKE_BIN
out=$PWD/nix-out
# for the "out command":
PATH=$PATH:$PWD/nix-out-cmd-bin

Where the file $PWD/nix-out-cmd-bin/out has:
#!/bin/sh
echo $out

And shell.nix has:
with import /home/doron/repos/nixpkgs {};

mkShell {
  inherit (julia) buildInputs nativeBuildInputs makeFlags;
  CMAKE_BIN="${cmake}/bin";
}

I'm launching make with eval make ${makeFlags[@]}. The persistent error is:
cat: /home/doron/downloads/julia-1.5.1/usr/manifest/suitesparse-wrapper: No such file or directory
WARNING: using mismatched version for :
  want

Trying building with their suitesparse or ours, and using their blas and lapack didn't help.

I fixed 1.0.4 over at https://github.com/Lucus16/nixpkgs/tree/fix-julia
I don't know how you want to fit it in so I'll leave that to you.

@Lucus16 It would be great to have a working expression for 1.0.4 in this PR. You can open a PR against my branch pkg/julia/latest which this PR originates from.

@doronbehar Here you go: https://github.com/doronbehar/nixpkgs/pull/1

idk if this is helpful or not, but the build does succeed using the derivation from 1.3 (lightly modified), but 22 tests Fail (although 38530970 tests pass :): https://gist.github.com/tbenst/495998f0b7e1bfbac1c75ac2155ac528

I've incorporated Guix' method for Julia where the patches are applied on our derivations via overrides, and USE_SYSTEM_ variables are set to 1. I got the sha256 of all patches and outdated versions Julia supposedly needs, but I haven't give it a full try. The overrided gmp, libuv and libunwind build, but I haven't checked if llvm builds, and if Julia builds with them.

@doronbehar Let me know if you need a hand. I would really like to have Julia back in nixpkgs.

@doronbehar Let me know if you need a hand. I would really like to have Julia back in nixpkgs.

You can help by try and building it as is. I got lazy to do it on my notebook laptop, as it needs building a custom version of llvm_7. If the llvm build succeeds, I'd also appreciate if you'd setup cachix and share the llvm build with me.

I've tried this today rebased on top of 5d0e2dedd5594cdf3ebbda8bc61310e227235a9c, and got the following build error:

01:38:51|~/config|master⚡*
λ nix-env -f /home/matklad/projects/nixpkgs -iA julia
installing 'julia-1.5.2'
these derivations will be built:
  /nix/store/w8z1i4130m7jhqnllzgpfbpbspq91k87-julia-1.5.2.drv
building '/nix/store/w8z1i4130m7jhqnllzgpfbpbspq91k87-julia-1.5.2.drv'...
unpacking sources
unpacking source archive /nix/store/01m42pwxzzd188jbbixvjz2ywpf6hsbq-julia-1.5.2-full.tar.gz
source root is julia-1.5.2
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file julia-1.5.2/deps/srccache/objconv.zip
patching sources
configuring
no configure script, doing nothing
building
build flags: SHELL=/nix/store/k8p54jg8ipvnfz435mayf5bnqhw4qqap-bash-4.4-p23/bin/bash USE_BINARYBUILDER=0 PREFIX=\$\(out\) prefix=\$\(out\) USE_SYSTEM_LIBUNWIND=1 USE_SYSTEM_PCRE=1 PCRE_CONFIG=/nix/store/vcxly22i1csi2xbjyjvsn6vk19cf5lnw-pcre2-10.35-dev/bin/pcre2-config PCRE_INCL_PATH=/nix/store/vcxly22i1csi2xbjyjvsn6vk19cf5lnw-pcre2-10.35-dev/include/pcre2.h USE_SYSTEM_DSFMT=0 USE_SYSTEM_BLAS=1 USE_SYSTEM_LAPACK=1 USE_SYSTEM_GMP=1 USE_SYSTEM_LIBUV=1 USE_SYSTEM_LLVM=1 USE_SYSTEM_MPFR=1 USE_SYSTEM_SUITESPARSE=1 USE_SYSTEM_UTF8PROC=1 USE_SYSTEM_MBEDTLS=1 USE_SYSTEM_LIBSSH2=1 USE_SYSTEM_CURL=1 USE_SYSTEM_LIBGIT2=1 USE_SYSTEM_PATCHELF=1 USE_SYSTEM_ZLIB=1 USE_SYSTEM_P7ZIP=1 USE_SYSTEM_OPENLIBM=1
Warning: git information unavailable; versioning information limited
Warning: git information unavailable; versioning information limited
Warning: git information unavailable; versioning information limited
rm: cannot remove 'scratch/dsfmt-2.2.3/': No such file or directory
patching file dSFMT.h
Hunk #2 succeeded at 181 (offset 2 lines).
Hunk #3 succeeded at 202 (offset 2 lines).
Hunk #4 succeeded at 220 (offset 2 lines).
Hunk #5 succeeded at 244 (offset 2 lines).
Hunk #6 succeeded at 265 (offset 2 lines).
Hunk #7 succeeded at 284 (offset 2 lines).
Hunk #8 succeeded at 295 (offset 2 lines).
Hunk #9 succeeded at 307 (offset 2 lines).
Hunk #10 succeeded at 318 (offset 2 lines).
Hunk #11 succeeded at 330 (offset 2 lines).
Hunk #12 succeeded at 341 (offset 2 lines).
Hunk #13 succeeded at 353 (offset 2 lines).
Hunk #14 succeeded at 376 (offset 2 lines).
Hunk #15 succeeded at 391 (offset 2 lines).
Hunk #16 succeeded at 407 (offset 2 lines).
Hunk #17 succeeded at 423 (offset 2 lines).
Hunk #18 succeeded at 439 (offset 2 lines).
Hunk #19 succeeded at 449 (offset 2 lines).
Hunk #20 succeeded at 459 (offset 2 lines).
Hunk #21 succeeded at 470 (offset 2 lines).
Hunk #22 succeeded at 483 (offset 2 lines).
Hunk #23 succeeded at 497 (offset 2 lines).
Hunk #24 succeeded at 511 (offset 2 lines).
Hunk #25 succeeded at 520 (offset 2 lines).
Hunk #26 succeeded at 529 (offset 2 lines).
Hunk #27 succeeded at 539 (offset 2 lines).
Hunk #28 succeeded at 549 (offset 2 lines).
Hunk #29 succeeded at 559 (offset 2 lines).
Hunk #30 succeeded at 569 (offset 2 lines).
Hunk #31 succeeded at 579 (offset 2 lines).
Hunk #32 succeeded at 592 (offset 2 lines).
Hunk #33 succeeded at 605 (offset 2 lines).
Hunk #34 succeeded at 618 (offset 2 lines).
Hunk #35 succeeded at 630 (offset 2 lines).
patching file test.c
patching file dSFMT.c
cat: /build/julia-1.5.2/usr/manifest/suitesparse-wrapper: No such file or directory
WARNING: using mismatched version for :
  want
Creating usr/etc/julia/startup.jl
Copying in usr/share/man/man1/julia.1
/build/julia-1.5.2/contrib/install.sh 755 /build/julia-1.5.2/contrib/julia-config.jl /build/julia-1.5.2/usr/share/julia/
PATH="/nix/store/1krkymlcn5yvg71camz6nckrhssqz5f8-gnum4-1.4.18/bin:/nix/store/z23xkm52b6jdpb098sv9h63rxi8zhnwj-which-2.21/bin:/nix/store/38vxlxrvg3yji3jms44qn94lxdysbj5j-perl-5.32.0/bin:/nix/store/rxjkdqgziqgy71yj7k6j65by8dzp6z84-gfortran-wrapper-9.3.0/bin:/nix/store/h770niqkf08ksgm0n6a271kc21w95b13-binutils-wrapper-2.31.1/bin:/nix/store/hiwz81i1g3fn3p0acjs042a4h5fri6dh-binutils-2.31.1/bin:/nix/store/kbysa4xpmn0013bvplw3fl2i1yydifjc-glibc-2.32-bin/bin:/nix/store/7fdm1ij6f7xwjmhpici1bh3xh08n0sp7-coreutils-8.31/bin:/nix/store/346skv0d24rqnf4npknbp9h5bs14j8zy-python3-3.8.6/bin:/nix/store/r9djcvv9hh6cma006jxlcz3xljz7zck0-cmake-3.18.2/bin:/nix/store/qza56yd923g39sbnn6gn5xh3awrz4lfl-patchelf-0.12/bin:/nix/store/8madhhcxhxlf0rrc5hdsk65l206r3cc6-gcc-wrapper-9.3.0/bin:/nix/store/shs8kax43dk69fh94g5w4bc1ra3nv2j2-gcc-9.3.0/bin:/nix/store/kbysa4xpmn0013bvplw3fl2i1yydifjc-glibc-2.32-bin/bin:/nix/store/7fdm1ij6f7xwjmhpici1bh3xh08n0sp7-coreutils-8.31/bin:/nix/store/vcxly22i1csi2xbjyjvsn6vk19cf5lnw-pcre2-10.35-dev/bin:/nix/store/m2qy16xk9cn5yprqm3a09ik8cxyv3igr-pcre2-10.35-bin/bin:/nix/store/xwa3yfa0zxy5zs914pq9wa3rsj310yn9-mbedtls-2.16.3/bin:/nix/store/bnqwzn3ywfxq2kn4l4s2c68jnbx2ilbg-curl-7.72.0-dev/bin:/nix/store/19k826d182q6a3k2ply1wrsb2knimsz4-nghttp2-1.41.0-bin/bin:/nix/store/8ry6c2r2dl5v9v45dvk7mgxh6416ckn7-libkrb5-1.18-dev/bin:/nix/store/vl9drphppxnxjwkfkr7sijh4d0gvcg3n-libkrb5-1.18/bin:/nix/store/wp81dnxgrpi4lgy20r82kfg9nwrmypkm-openssl-1.1.1g-bin/bin:/nix/store/vcvcpdn0bspcl722qkwp2s72wws9gw7s-curl-7.72.0-bin/bin:/nix/store/qza56yd923g39sbnn6gn5xh3awrz4lfl-patchelf-0.12/bin:/nix/store/jw0gpz7s9y8jbxp7h36pr7sj72bssm45-p7zip-17.01/bin:/nix/store/jqvbvnyl82c0zdiwl12spr2qh9hs1im6-llvm-8.0.1/bin:/nix/store/3gxpdac17sbrxmc7iv3dfsw5pskdllrz-ncurses-6.2-dev/bin:/nix/store/jxk39sgwpfnsfb00dsff5p96mxaj6m33-ncurses-6.2/bin:/nix/store/8ard3kjhyhqbp3ylxa7djfhndqfybbi8-xz-5.2.5-bin/bin:/nix/store/7fdm1ij6f7xwjmhpici1bh3xh08n0sp7-coreutils-8.31/bin:/nix/store/ny7vshs45d5k193ql3b51dak0hjwwqc2-findutils-4.7.0/bin:/nix/store/1avv8qnga55v6kyh9bvv72pf1nwhivv7-diffutils-3.7/bin:/nix/store/98an38bw0fwk6a5mickblc0h07jf00vz-gnused-4.8/bin:/nix/store/4crjhnvd8yhk063r7pwxpw6kd8aq0rbv-gnugrep-3.5/bin:/nix/store/ycf3h904fbai22by992pm9a0iwnwyxhp-gawk-5.1.0/bin:/nix/store/3kg5y8qb1fmq7sv77lj7xy5vldx2m9za-gnutar-1.32/bin:/nix/store/qxybvmiq4xksmzy26ivc0hd61c94bln0-gzip-1.10/bin:/nix/store/71w61r13p1pvmgz16106k0ps0k7xpb6m-bzip2-<IP_ADDRESS>.1-bin/bin:/nix/store/vc851g6wnyyykb5jxgbal3whi5495xia-gnumake-4.3/bin:/nix/store/k8p54jg8ipvnfz435mayf5bnqhw4qqap-bash-4.4-p23/bin:/nix/store/p7g6l88dl48fjx41qb628sh72vpv1v0h-patch-2.7.6/bin:/nix/store/8ard3kjhyhqbp3ylxa7djfhndqfybbi8-xz-5.2.5-bin/bin:/build/julia-1.5.2/usr/tools" PATCHELF="patchelf" FC="gfortran -m64" /build/julia-1.5.2/contrib/fixup-libgfortran.sh --verbose /build/julia-1.5.2/usr/lib
Discovered traces of libgfortran within
Got SONAMES of
Warning: git information unavailable; versioning information limited
    PERL base/pcre_h.jl
    PERL base/errno_h.jl
    PERL base/build_h.jl.phony
    PERL base/file_constants.jl
make[1]: *** No rule to make target '/build/julia-1.5.2/usr/include/uv/errno.h', needed by 'uv_constants.jl'.  Stop.
make: *** [Makefile:66: julia-base] Error 2
builder for '/nix/store/w8z1i4130m7jhqnllzgpfbpbspq91k87-julia-1.5.2.drv' failed with exit code 2
error: build of '/nix/store/w8z1i4130m7jhqnllzgpfbpbspq91k87-julia-1.5.2.drv' failed

01:38:56|~/config|master⚡*

Just as feedback, I can successfully build this locally on my machine. I do not really know how to upload anything to cachix but if someone walks me through the steps, then I am happy to do that.

@doronbehar I tried building the current merge-head and got the same error as @matklad. I followed the basic setup instructions for cachix and then did a
nix-build . -A julia.buildInputs | cachix push r-burns

I haven't used cachix before but hopefully I did that right and it can be of some assistance

I followed the basic setup instructions for cachix and then did a
nix-build . -A julia.buildInputs | cachix push r-burns

I haven't used cachix before but hopefully I did that right and it can be of some assistance

Luckily that product is excellently built (:green_heart: @domenkozar ) and I think you did it correctly, so thanks for effort. However I can't verify I got your llvm build. Unfortunately even if it would have worked I don't know how would I go about and fix the issue described in @matklad's comment. I'm also not as motivated as (I think) you are or as I was a while ago.. Sorry.

The last push is just a rebase and a merge conflict fix.

Closed in favor of https://github.com/NixOS/nixpkgs/pull/101933 .

Oh but there's @Lucus16 commit I forgot here which fixes 10.. I will open a new PR.


Add cyclical learning rate

Add cyclical learning rate as reported by Leslie N Smith. Cyclical learning rates for training neural networks. arXiv preprint arXiv:1506.01186v3, 2016.
See this example to understand what a cyclical learning rate is about
The function can be easily coupled to others like tf.train.exponential
...
cosine_lr = tf.train.cosine_cyclical(starter_learning_rate, global_step,
                                          decay_steps, end_learning_rate, T)
cosine_exp_lr = tf.train.exponential_decay(cosine_lr,global_step,300,0.7,staircase=False)

For the a better comprehension, see this example  of how the snippet above looks like in practice.

@wicke, didn't find a good reviewer for this change based on the change history, do u mind reroute this if you know anyone worked on this?

I'm happy that another contributor proposed another cyclical learning rate on PR #20785 based on the same reference. I just want to make it clear that the cyclical learning rate I've coded is based on a cosine curve, not linear as proposed by #20785

Nagging Reviewer @martinwicke: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Ping @martinwicke for review.

Have you compared the experiment results with cosine_decay_restarts and https://github.com/tensorflow/tensorflow/pull/20785?
https://github.com/tensorflow/tensorflow/blob/483913a32ef0807791a9b5edfb9628cd66c786a2/tensorflow/python/training/learning_rate_decay.py#L710

Nagging Reviewer @martinwicke: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Nope.
But as I mentioned before, the cyclical learning rate I've coded is based on a cosine curve, not linear as proposed by #20785

Nagging Reviewer @martinwicke: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @martinwicke: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 29 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @martinwicke: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 45 days with no activity and the awaiting review label has been applied.

The image linked in the opening post shows SGDR and it is reproducible with cosine_decay_restarts (https://github.com/tensorflow/tensorflow/pull/11749).

The image linked in the opening post shows SGDR and it is reproducible with cosine_decay_restarts (#11749).

I agree. There is no need to duplicate a function. I haven't seen this before. My bad =P . I'm gonna close this PR.


Refine the README.md

I think we'd better demonstrate the purpose of this new keras-contrib repository in the README.md. From my point of view, the purpose is to:

provide an official keras extension repository for the keras community, which contains newest layers, activations, loss functions, optimizers, and so on.
keep the original keras repository clean, only the most valuable functions will be finally merged into the original keras repository.

Any suggestions?

@patyork


Fix Maternity Smart Answer Error

There's a problem with the calculator when the last SMP payment is after the last working day
of the month. This was spotted for June 2018, but we think it would happen whenever this
scenario arises.
Previously the code iterated the SMP payments according to the last working day of each month
but it did not tackle the scenario where the final SMP payment is outside of the last working
day of leave.
The new code iterates through the last working day of each month while the last working day of
the month is less than the final SMP pay date. This ensures that all payments are made. In this
scenario an extra month is added to which the last payment will be made.

 (5) Maternity smart answer outcome error


Broken link in README.md of charts folder

Checking the content of data-space-connector/charts/README.md, I found that the link to the description of the Minimal Viable Dataspace is linked to here when maybe it should be here

Could you please provide a PR with the fix?

I might. I didn't do it before because it was unclear if this was the correct link. PR created #35


can it do super resolution?

Can VAR do super resolution like GigaGan super resolution for example. Gigagan is the most impressive super resolution algorithm till now.
And if yes would you be able to add support for it Later next month or so?

VAR supports zero-shot super resolution. Although it might not rival the GigaGan upsampler, we're planning to release a demo for testing in the coming days. Stay tuned for updates!

Hi keyu,
I'm Bingyue's friend and I'm very impressed with this work!
I have a question regarding large size image with super high resolution.
First, let me try to understand the fundamental logic. Correct me if I'm wrong.

The basic idea is to establish a self-supervised learning mechanism. In VAR, we follow the process as
raw img
-> embedding f
-> forward: (r_K -> ... -> r_1)
-> backward: (r_1 -> ... -> r_K)
-> recovered  embedding f^
-> reconstructed img

i.e. from fine to coarse and then inversely from coarse to fine

The learning is based on the probabilistic generative model for the conditional generation probabilities:
P(r_k | r_(k-1), ..., r_1)  for k = 1, ..., K, with r_0 as a pre-defined start (, i.e. guidence).

Based on this understanding, regarding a large size image with super high resolution, we can set the dimension of the the embedding vector f to be higher for more representation capability.
Considering the main-stream techniques like the one in the paper “Scalable Diffusion Models with Transformers”, one technique is to “patchify” the raw image into patches (i.e. tokens) and then find the “best” embedding of each patch by a transformer-architecture based learning. When each token embedding is decoded back into a “predicted” patch, then all the “predicted” patches can be re-organized together to recover the whole image.
Now, the QUESTION is: can we also do the “patchify” and then apply the fine→coarse→fine process to each patch and then reorganize the “predicted” patches to recover the whole image?
Not quite sure which of the two method is better. I mean
a) setting dimension of the the embedding vector f to be higher for more representation capability
b) patchifing the raw image into patches, working on each patch, and the piecing together the “predicted” patches
One intuition for the “patchify” in method b) is that there could be some un-smoothed piecing-together when the computation for the optimization process is not enough yet. Note that breaking a whole image into pieces actually destroys the spatial connection information of the pieces. The method a) does not need to deal with the problem of piecing-together because the embedding is regarding the whole image.
Best,
Xugang Ye

@judywxy1122 Thank you for your kind words! The question is a bit detailed; let me give it some thought and I'll get back to you shortly.


Build Context issue

hi first of all thanks for your amazing job I'm  currently using a splash screen as my home: and navigating from there to my homepage and I'm using a custom app bar class so I'm getting null pointer error because my home: is my splash screen I managed to set my home: to my main home page and send the context to app bar class for drawer opening but this means that I have to lose my splashScreen and I cant use this drawer in my other screens as well any suggestion sorry for taking your time thanks

I had the same issue, I resolve this issue using this.
use -> scaffoldKey.currentState.context instead of context
pass globalkey to the custom app bar
SlideDrawer.of(scaffoldKey.currentState!.context)!.toggle();


Can not edit days in lifecycle configuration

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

What happened:
I could not edit the days when I add lifecycle rule for an object.

What you expected to happen:
allow user to edit the days.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Dashboard (release/branch) version:v0.5.3
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@yuejuSDS Please take a look

@jimccfun Please take a look.

@wisererik can you please check the RC2 and confirm the issue status?


bench: Add benchmark for rollback

Signed-off-by: MyonKeminta<EMAIL_ADDRESS>
What have you changed? (mandatory)
This PR adds benchmarks for MvccTxn::rollback.
It's divided two three different cases:

rollback_prewrote: Prewrite happens before the rollback
rollback_conflict: Another transaction's Prewrite happens before the rollback
rollback_non_prewrote: The rollback happens before prewrite

What are the type of the changes? (mandatory)

Improvement (change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
By CI
Does this PR affect documentation (docs) update? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

/run-integration-tests

@breeswish Do you have any idea about the failing test? Although it's not related to this PR.

The test itself was broken and has been fixed in pingcap/tidb-test#773.
/run-integration-common-test

@breeswish PTAL, thanks!

mvcc_rollback_prewrote/BTree_KL64_VL64                                                                                [71/1830]
                        time:   [10.225 us 10.708 us 11.218 us]
Found 12 outliers among 100 measurements (12.00%)
  3 (3.00%) high mild
  9 (9.00%) high severe
mvcc_rollback_prewrote/BTree_KL64_VL65
                        time:   [11.002 us 11.091 us 11.181 us]
Found 9 outliers among 100 measurements (9.00%)
  4 (4.00%) low mild
  4 (4.00%) high mild
  1 (1.00%) high severe

mvcc_rollback_conflict/BTree_KL64_VL64
                        time:   [13.135 us 13.226 us 13.319 us]
Found 6 outliers among 100 measurements (6.00%)
  1 (1.00%) low mild
  5 (5.00%) high mild
mvcc_rollback_conflict/BTree_KL64_VL65
                        time:   [13.257 us 13.861 us 14.551 us]
Found 17 outliers among 100 measurements (17.00%)
  3 (3.00%) low mild
  1 (1.00%) high mild
  13 (13.00%) high severe

mvcc_rollback_non_prewrote/BTree_KL64_VL64
                        time:   [11.307 us 11.798 us 12.334 us]
Found 16 outliers among 100 measurements (16.00%)
  5 (5.00%) high mild
  11 (11.00%) high severe
mvcc_rollback_non_prewrote/BTree_KL64_VL65
                        time:   [11.051 us 11.475 us 11.980 us]
Found 14 outliers among 100 measurements (14.00%)
  1 (1.00%) high mild
  13 (13.00%) high severe

mvcc_rollback_prewrote/Rocks_KL64_VL64
                        time:   [35.643 us 37.843 us 40.116 us]
mvcc_rollback_prewrote/Rocks_KL64_VL65
                        time:   [35.081 us 37.399 us 39.843 us]
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

mvcc_rollback_conflict/Rocks_KL64_VL64
                        time:   [52.123 us 54.932 us 57.835 us]
mvcc_rollback_conflict/Rocks_KL64_VL65
                        time:   [48.584 us 50.529 us 52.538 us]
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

mvcc_rollback_non_prewrote/Rocks_KL64_VL64
                        time:   [46.432 us 48.767 us 51.157 us]
mvcc_rollback_non_prewrote/Rocks_KL64_VL65
                        time:   [44.486 us 47.163 us 50.265 us]

txn_rollback_prewrote/BTree_KL64_VL64
                        time:   [24.663 us 25.006 us 25.323 us]
txn_rollback_prewrote/BTree_KL64_VL65
                        time:   [27.438 us 27.688 us 27.931 us]
Found 4 outliers among 100 measurements (4.00%)
  2 (2.00%) low mild
  2 (2.00%) high mild

txn_rollback_conflict/BTree_KL64_VL64
                        time:   [30.000 us 32.053 us 34.725 us]
Found 9 outliers among 100 measurements (9.00%)
  1 (1.00%) low mild
  8 (8.00%) high severe
txn_rollback_conflict/BTree_KL64_VL65
                        time:   [30.093 us 30.332 us 30.562 us]
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) low mild

txn_rollback_non_prewrote/BTree_KL64_VL64
                        time:   [29.220 us 30.477 us 32.041 us]
Found 17 outliers among 100 measurements (17.00%)
  8 (8.00%) high mild
  9 (9.00%) high severe
txn_rollback_non_prewrote/BTree_KL64_VL65
                        time:   [27.890 us 28.497 us 29.210 us]
Found 3 outliers among 100 measurements (3.00%)
  1 (1.00%) high mild
  2 (2.00%) high severe

txn_rollback_prewrote/Rocks_KL64_VL64
                        time:   [381.48 us 394.98 us 407.82 us]
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe
txn_rollback_prewrote/Rocks_KL64_VL65
                        time:   [374.66 us 391.05 us 408.22 us]
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

txn_rollback_conflict/Rocks_KL64_VL64
                        time:   [382.33 us 397.98 us 414.41 us]
Found 4 outliers among 100 measurements (4.00%)
  4 (4.00%) high mild
txn_rollback_conflict/Rocks_KL64_VL65
                        time:   [371.65 us 383.53 us 395.70 us]
Found 6 outliers among 100 measurements (6.00%)
  2 (2.00%) low mild
  3 (3.00%) high mild
  1 (1.00%) high severe

txn_rollback_non_prewrote/Rocks_KL64_VL64
                        time:   [465.25 us 483.34 us 502.85 us]
Found 12 outliers among 100 measurements (12.00%)
  4 (4.00%) high mild
  8 (8.00%) high severe
txn_rollback_non_prewrote/Rocks_KL64_VL65
                        time:   [416.96 us 437.67 us 460.22 us]
Found 9 outliers among 100 measurements (9.00%)
  2 (2.00%) high mild
  7 (7.00%) high severe

Please fix the conflicts.

@AndreMouche @breeswish PTAL again thanks!
Git diff works terrible in this PR, which makes it hard to resolve conflicts... Hope this will not cause confusions to you.

/run-integration-tests


Revert "Week 1 branch"

Reverts nxdavar/my-portfolio#2

Why are you reverting this? You are removing your own work?


Always run post cache action

Why?
I'm mainly doing this to decrease cycle time as I'm starting to use this in our
megarepo. The scenario is, I'm trying to convert our backend test over, they're
failing but dependencies are not being cached. Dependency install take something
like 50 minutes.
It's my understanding that failed builds (e.g. failing on test run) should still
have their dependencies cached.

Did you merge ahead of CI via Admin, or do we not have protected branches setup correctly?

@pbrisbin I forced merge, hope that's okay. The test step stalled out for all commits.

Oh sure, it's definitely OK, we just usually comment about it since it's unusual.


Order Back Button linked to Order instead of HomePage

PR #167
I've created a link back from the back button of track order to order page. Please assign this issue to me.

Hey @gauriket  Thanks for fixing this !! If you liked our project kindly leave a star to the repo for us and lets connect on GitHub !!


[Request] Can this tool support sound transfer?

As title, if this tool able to pass the sound of the system? Maybe it needs something like a virtual sound card installed if the machine has no equipment.
And thanks for this amazing tool!

Hello,
It does not support sound, if possible, I will implement it in a future version.
Regards


Added click event type support

Types from docs, i left doing type for event for later as it is humongous,
https://visjs.github.io/vis-network/docs/network/
Also I've set items type as unknown because I couldn't get values which are in docs, i am not sure what labelId is

There's a few issues with the types as is. I'll post a change with slightly better typing.
Mostly we should be using intersection types to combine things instead of unions, and I set it up so we don't have to set up our list of unsupported ones directly either.


Not detecting module definition installed with NPM

Hi. I'm trying Typescript 2.0 and the new way of importing definitions with npm install @types/.... Everything works fine with the typescript cli but the Sublime plugin is showing an error in the error list (Cannot find module...). Not supported yet?

I didn't read the doc carefully. I had to specify manually the version of typescript. My bad...


Feature Request: reverse columns and rows in cluster view

My elasticsearch clusters normally have a lot of more indices than cluster nodes. It would be extremely helpful if the cluster nodes would be columns instead of rows, and the indices would be rows instead of columns. Maybe making it switchable is even the best, because I can imagine that the current view can be better in some cases.

@mheese hey, there are 2 issues here:
1- Rendering a long list of indices x shards can make the view quite slow. Having them paginated somehow makes this a little "snappier".
2- Usually, people have a lot more indices than nodes. Still, it's normal that people have more than say, 7 nodes(which is the number of index columns I can currently fit on my display size). If rows x columns were reversed, that means that for a given index, I might not see where its shards are all at once(I have 10 data nodes).
On the other hand... what's your use case for looking at all indices at once? Are you actually interested on the shard allocation, or just having a list with all of them? I have on my pipeline to have a indices view(such as the nodes view which is already done), but would be more of index x index stats(number of documents, size...).


Add option to pass multer options to Multipart uploads

Feature Request
Problem
Currently, @TypedFormData.Body() uses multer to read and store file uploads but it doesn't support passing options to the multer singleton.
I am trying to use DiskStorage to store uploads but multer uses MemoryStorage by default.
Additionally, the type of files within @TypedFormData.Body() is File, which hides a lot of the information in Express.Multer.File types and it's not supported in Node versions prior two 20.
Request

Add option to be able to pass multer options to multipart uploads
Use Express.Multer.File type instead of File for files (see https://expressjs.com/en/resources/middleware/multer.html#api)

Thanks!

As I also have to consider the fastify case,  supporting express-multer options are not easy problem.
Will you suggest or send a PR that can make both express and fastify compatible?

Hi @samchon
I found this package https://www.npmjs.com/package/fastify-multer, which is a port of express-multer. It supports the same options and it outputs the same output types.
I think using this library would simplify it a bit.
Let me know what you think

Seems good. What about challenge a PR in here code?
https://github.com/samchon/nestia/blob/master/packages/core/src/decorators/TypedFormData.ts

@samchon what do you think about the PR?

I looked your PR. Changing dependent library of @TypedFormData() in fastify requires major level update.
As I'm not developing the multipart/form-data type in the backend development, so I can't sure that fastify-multer is enough valuable for processing the major version level update. I hope many other users' opinions of this issue, but as you know, nestia users are not such a lot.
Therefore, consider the aspect and tell me your opinion about this thing. If you still think that the major version update is valuable, I'll re-check it when typia v7 and nestia v4 major updates.

I think this would be a great addition since the multipart uploads are a big part in some projects that heavily depend on multimedia files. The other solution that would not require a breaking change is to have different options in the @TypedFormData decorator for fastify and express. I believe though, that the unification simplifies things

Owner

hey, any estimated time on this issue? we are keeping nestia and nestjs generator just for this issue, I'd be really nice to have this feature to fully migrate to nestia

Completed in v4 major update.


Utkarsh pro/patterns/mesh core support

Description
This PR adds support for:

Registering all workloads/traits specified in their respective directories.
Moved to in memory provider
Common handler for all istio core resources

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Thanks, @utkarsh-pro. There are a few lint checks to attend to here.


Feature: Select folders to display in folder list

Expected behaviour
I'd like to select the folders that I want to see in the folders list (and so, exclude other folders from my list).
Actual behaviour
You can select groups like "All folders", "1st class folders", ...
Steps to reproduce

Go to "Account settings"/"Folders" or just "Folders" when you're in an account.

Environment
K-9 Mail version: 5.105
Android version: 4.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP
Sorry if this feature is already there, then I just did not find it yet. I searched your issues wit the query "folders".

As you’ve seen it by yourself, this is the “folder class” feature. You can set each folder display class and then adjust what classes are displayed.

Wow...great! I misunderstood the "class" label, of course it is what you say. I thought it is the level of the folder in the folders tree.
An awesome work you do - thank you so much.


dyld: Library not loaded: /usr/local/opt/bdw-gc/lib/libgc.2.dylib when launching binary

I installed the newest version of crystal, 0.24.2_2, via Homebrew on Mac OS X High Sierra. This bug will trigger with any binary program as soon as you launch the program:
Program:
puts 1

Terminal commands:
crystal build program.cr
./program

Error:
dyld: Library not loaded: /usr/local/opt/bdw-gc/lib/libgc.2.dylib
  Referenced from: {location of file}/./program
  Reason: image not found
Abort trap: 6

The problem is that the dylib file is called libgc.1.dylib instead of libgc.2.dylib, but I'm not sure how to fix that, and simply renaming the dylib file doesn't seem like a good option.

@Sija The solution you gave works, but I think I should still keep the issue open so that a more permanent solution arises.

have you made an issue for the homebrew guys? It seems like a packaging issue to me.

Did you intend to close this yet?


Can Scout use @extend to include Bootstrap elements

I've seen some examples of folks doing things like @extend .btn in Sass.
This doesn't seem to work in Scout.
Can this be done?

Scout is using the 2012 version of Sass. It can do anything that version can.

You can try this with Scout-App 2 (2016 version).
Closing this ticket. Feel free to open a new one if the issue is present in the new version of Scout.


[r2.11-rocm-enhanced] Switch to ROCm 5.4.0

retest Ubuntu-GPU-multi please

retest Ubuntu-GPU-single please

retest Ubuntu-sanity please

retest Ubuntu-CPU please

retest Ubuntu-GPU-multi please


Change inline configuration format

Prepare for https://github.com/fregante/ghat/issues/9
- # DO NOT EDIT BLOW, USE: npx ghat fregante/ghatemplates/node --exclude jobs.Test
+ # FILE GENERATED WITH: npx ghat
+ # SOURCE: https://github.com/fregante/ghatemplates
+ # OPTIONS: {"source":"fregante/ghatemplates/node","exclude":["jobs.Test"]}

Advantages:

The command to run is now just npx ghat
There's a full repo URL (clickable for RGH users)
No cross-platform issues when creating/parsing argv from a string

Disadvantages:

A bit longer
Can no longer be copy-pasted as a whole across repositories 😕

Change:
- # FILE GENERATED WITH: npx ghat
+ # FILE GENERATED WITH: npx ghat fregante/ghatemplates/node
  # SOURCE: https://github.com/fregante/ghatemplates
- # OPTIONS: {"source":"fregante/ghatemplates/node","exclude":["jobs.Test"]}
+ # OPTIONS: {"exclude":["jobs.Test"]}

Advantages over the previous version:

It can still be partially copy-pasted across repositories
The OPTIONS line only appears if there were options passed


Kotlin runtime initial structure

Introduce new apollo-runtime-kotlin module with initial structure:

ApolloClient similar to JVM version, a facade for user interface
RequestExecutor is a business logic layer responsible for query execution. Allows to create a chain / pipeline of several executors. Each executor observes, modifies, and potentially short-circuits GraphQL operation requests. NetworkExecutor is a final / terminal request executor in the chain, responsible for network execution via configured NetworkTransport
NetworkTransport is an abstraction for networking. Apollo is going to provide several NetworkTransport implementations: Http(JVM/iOS) and WebSocket(JVM/iOS)
DispatchersContext provides configuration for coroutine dispatchers, that will be put into ExecutionContext

See ApolloClientTest as an idea how everything connected.
Here is the rough idea how normalized cache will be integrated:

it's going to be completely isolated from runtime
it will copy ApolloClient with injected executor into ApolloClient#executors to provide cache functionality via extension function fun ApolloClient.withCache(<some config>) -> ApolloClient
it will add own execution context into Apollo#executionContext to provide default configuration such as default cache policy
it will provide extension functions for ApolloQueryCall that will make copy of RealApolloCall with and injected request execution context (in such way we can provide request cache policy that will override default one) via fun ApolloQueryCall.cachePolicy(CachePolicy.CacheFirst) -> ApolloQueryCall
cache executor will handle all cache logic by intercepting operation execution before NetworkExecutor

Part of #2222

CI seems to be unhappy with some changes affecting kmp-android-app

I can't seem to compile the example when isolated from the project. Is the kotlin runtime available on jCenter? Gradle cant resolve it. The kotlin multiplatform docs link to the example but I can't get it to compile.

@RoryKelly it's not released yet but you shuold be able to use the snapshots.
See https://github.com/apollographql/apollo-android#releases for more details.

@martinbonnin works great! it looks like there's no normalized cache (yet). Is this correct should I be following #2222 for updates?

@RoryKelly yup, that's a great ticket to follow. There are a lot of moving parts at the moment. If you have ideas how to structure the cache or features you'd like to have, feel free to comment down there.


docs: fix typo in Flightcontrol deployment option #970

Corrected Flightcontrol spelling
❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

It is failing on Vercel because it is not authorized, does this depend on me? @pi0


chore: bdd-gh-1587012988 to 0.0.1

chore: Promote bdd-gh-1587012988 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


inbound port 8050 or 8060

the documentation says 8050 (TCP) inbound
however the diagram says 8060 inbound

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

ID: 567118bc-05db-cb9d-d14f-faf835da048a
Version Independent ID: 9aaa1635-bb8c-07b4-ebc5-55baeb98149c
Content: Security considerations in Azure Data Factory
Content Source: articles/data-factory/data-movement-security-considerations.md
Service: data-factory
GitHub Login: @nabhishek
Microsoft Alias: abnarain

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Thank you for bringing the inconsistency to our attention @aaronhughesdelt .  I will assign the issue to the author.
@nabhishek , could you please correct this port number inconsistency?
The inconsistency is in the Firewalls section.  "8060" does not appear in the document text, but does in one of the images, as shown below.

@MartinJaffer-MSFT Thanks for bringing this up. Fixed it.

#please-close

doesn't look this this has been updated there still ref to 8050 throughout the doc
FAQ - What are the port requirements for the self-hosted integration runtime to work? - "Open inbound port 8050"
On-premises data store credentials - Ports used when encrypting linked service on self-hosted integration runtime - "By default, PowerShell uses port 8050" - image underneath this text

@nabhishek  I do see both 8050 and 8060 in the text, in different portions of the same document.  Did you mean to leave the other portions as 8050, or should I go and change all 8050 -> 8060 ?

@MartinJaffer-MSFT Please update all 8050 -> 8060. The current default is 8060 and can be modified by the customer if needed.

@nabhishek  sorry I missed your reply.  I have made the text changes to the document, but I can't figure out how to change the screenshot.  I have provided new images in the pull request.  If you could insert them for me when you review, that would be great.

Thanks for the feedback! This article has been refreshed to address the issue. We will now proceed to close this thread.
(Please open a new issue if you have follow up questions.)
#please-close


[BUG] New QRcodeID generated even if the QRCodeID already exists

Describe the bug
The db generates new Id sometimes even if there already exists an instance of the qrCode in the db
To Reproduce
Scan the qrCode without location and delete the qrCode and try to scan it again with enabling location
Expected behavior
It should just add the qrCode to existing document

Additional context
N/A

Couldn't replicate this, can you do a screen recording?

N/A

Relevant forum discussion here


Content type definition names translation

I have some clients who prefers to have their admin completely translated in french. So, when I create custom content types in the content definitions, I will normally name my content types in english to avoid accents in code and/or database. Workaround on this would be to make those translatable by using T().
@Html.ActionLink(T("Create New {0}", T(Model.DisplayName)).Text, "Create", new {area = "Contents", id = Model.Name})

Here is an example of what I'm doing in the admin views to make it translatable. Is it something that could be added in core ? It is sure that you need then to add those in a .po file wich is normal since those content type names are case by case for each websites we do. But also, it could provide a way to translate all those content type names in the admin pretty easily. Maybe, we could have a .po file defined just for these and a UI to edit them. Just some thoughts.

I have made a branch on my side wich include changes for that.
https://github.com/Skrypt/Orchard/tree/content-types-translation
Tell me if you want me to send a pull request.

duplicate of #6750


Added Klokurier Links

In order to promote unia.xyz and in order to keep authority over distributed links, Klokurier decided to use unia.xyz

Ich hab vergessen die links einzutragen D:


(PYL-R0201) No use of self

Description
The method doesn't use its bound instance. Decorate this method with @staticmethod decorator, so that Python does not have to instantiate a bound method for every instance of this class thereby saving memory and computation. Read more about staticmethods here.
Occurrences
There is 1 occurrence of this issue in the repository.
See all occurrences on DeepSource → deepsource.io/gh/open-bacen/bancocentralbrasil/issue/PYL-R0201/occurrences/

Fixed https://github.com/open-bacen/bancocentralbrasil/commit/e030bc4a36135151e49351660eb4cc3c5d3b2552


UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128)

I used bot yesterday，it‘s work ,and 4 hours ago , i git pull it . then it's not work
OS: UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128)
2016-08-08 09:57:01,546 [sentry.errors.uncaught] [ERROR] [u'ServerSideAccessForbiddenException: Seems your IP Address is banned or something else went badly wrong...', u'  File "pokecli.py", line 502, in ', u'  File "pokecli.py", line 131, in main', u'  File "pokecli.py", line 140, in report_summary', u'  File "pokemongo_bot/metrics.py", line 76, in capture_stats', u'  File "pokemongo_bot/api_wrapper.py", line 106, in call', u'  File "pokemongo_bot/api_wrapper.py", line 63, in _call', u'  File "pgoapi/pgoapi.py", line 220, in call', u'  File "pgoapi/rpc_api.py", line 129, in request', u'  File "pgoapi/rpc_api.py", line 332, in _parse_main_response']
os: kali-rolling debian
Git Commit: dee28d9fb1a973b95019bc07fb1e206872bd986d
Python Version:Python 2.7.12

this should fix it: https://github.com/PokemonGoF/PokemonGo-Bot/issues/2909

@designer2k2  no ,not fixed

They banned additional IPs, the one your using is in that list.
Run the bot from somewhere else ...


Clock does not recognise fmt-10.0.0 format specifier

Version 10.0.0 of fmt introduced support for padding seconds, minutes and hours (so e.g. producing output like  1 rather than 01). I have tested that this works on my system by compiling and running the following C++ program:
#include <fmt/chrono.h>

int main() {
    auto t = tm();
    t.tm_year = 2010 - 1900;
    t.tm_mon = 7;
    t.tm_mday = 4;
    t.tm_hour = 1;
    t.tm_min = 15;
    t.tm_sec = 58;
    fmt::print("{:%_I}", t);
    return 0;
}

As expected, this prints  1 (with an initial space).
However, when I try using the same format string with waybar, it simply produces the literal text %_I:

My clock configuration is as follows:
    "clock": {
        "tooltip-format": "<tt>{calendar}</tt>",
        "format": "{:%_I}",
        "calendar": {
            "mode": "month",
            "on-scroll": 1,
            "weeks-pos": "left",
            "format": {
                // omitted for brevity
            }
        },
        "actions":  {
                // omitted for brevity
        }
    },

Note that the format string is exactly the same as that used in the C++ program above.
For reference, I am using EndeavourOS (so basically Arch Linux), with waybar-0.9.20-2 and fmt-10.0.0-1. My compositor is Sway, though I presume that makes no difference.

Hi @bradrn , As far as I know, the waybar now doesn't support fmt 10 version. But I might be wrong. Anyway I'll check it, once have a time. Thank you

waybar now doesn't support fmt 10 version

Insofar as I’m aware, it shouldn’t matter, since the call to fmt::print (or whichever function is used by waybar) should be the same either way. It certainly works on my machine with fmt-10.0.0 — it’s just that the new format specifiers introduced by that version don’t seem to work.

waybar now doesn't support fmt 10 version

Insofar as I’m aware, it shouldn’t matter, since the call to fmt::print (or whichever function is used by waybar) should be the same either way. It certainly works on my machine with fmt-10.0.0 — it’s just that the new format specifiers introduced by that version don’t seem to work.

Clock module uses the exact fmt library. So.. it's definitely a strange case when you are able to use fmt10 ... but clock component shows unexpected results.. .OK, let me check...

I probably should have mentioned that other format strings show the expected results, e.g.:

This is with {:%a %Od %b,\t%I:%M %p}.

I'm currently experiencing issues with waybar on arch.
Initially after system update, waybar stopped working, because of a missing library fmt9, I had fmt10.
I build fmt9 from source and the problem was I was missing fmt10. I installed both at the same time, I'm stuck on this error message and waybar build crashing on be from the clock test module.
after install from AUR:
waybar: symbol lookup error: waybar: undefined symbol: _ZN6spdlog7details7log_msgC1ENS_10source_locEN3fmt2v917basic_string_viewIcEENS_5level10level_enumES6_
When trying to build:
/usr/bin/ld: clock.cpp:(.text+0x3881): undefined reference to fmt::v9::detail::assert_fail(char const*, int, char const*)'`
there is like 1000 of these, each containing fmt::v9, but when I have only it installed it errors the same.
Sorry if this is cryptic/incomplete, I'm not that familiar with cpp or it's debugging.

Probably related to this: Since my last system update (waybar and fmt were both updated) the decimal points are printed, when just giving the format specifier %S.

@DigitalDragon64 Same, created an issue for it here: https://github.com/Alexays/Waybar/issues/2378.

Hi @bradrn , I found your conversation fmtlib/fmt#3516
Finally I built waybar with dependency fmt = 10.0.0. There issue here: test/chrono-test.cc
format:
EXPECT_EQ(fmt::format("{:%_OI,%_OH,%_OM,%_OS}", tm), " 1, 1, 2, 3");
EXPECT_EQ(fmt::format("{:%-OI,%-OH,%-OM,%-OS}", tm), "1,1,2,3");
doesn't work. In the same time I see that both merge requets were approved and now are part of the vmt:10 version according release note. See fmt 10 RN - row Improved validation of format specifiers for std::chrono::duration (https://github.com/fmtlib/fmt/issues/3219, https://github.com/fmtlib/fmt/pull/3232). Thanks [@ShawnZhong (Shawn Zhong)](https://github.com/ShawnZhong).
So actually looks like need to raise an issue to fmt upstream.

So it actually looks like it needs an issue to be raised to fmt upstream.

Except that exactly the same format string works when I directly use my version of fmt (see the test program above), but doesn’t work when I use it with waybar. So this can’t be an issue with fmt.

@bradrn, Waybar uses date features like date::zoned_time. According cpp reference now it included into C++20 standard. I've just checked: fmt doesn't work with date::zoned_time. In the same time pure sys_time timpoint works well with fmt.
I'll create an issue to fmt to extend glibc extension on the whole chrono library including TZ API.

Ah, that makes sense! Thanks for looking into this @LukashonakV.


[BUG] Dependencies as currently declared bring in JUnit for production classpath

Describe the bug
I tried adding jcefmaven to my dependencies and found that I now have junit on the production classpath.
To Reproduce
Steps to reproduce the behavior:

Make a new Gradle project
Add implementation("me.friwi:jcefmaven:<IP_ADDRESS>")
Build the project and you will see junit being fetched.

Expected behavior
JUnit should only be on the classpath for running tests.
Environment (please complete the following information):

OS: Windows (but probably irrelevant)
Architecture: amd64 (but probably irrelevant)
Version: <IP_ADDRESS>

Seems to be json-simple bringing it in, so basically it looks like you just switch to a normal JSON parser and everything is good? Pulling in random JSON parsers isn't good security anyway.

json-simple is in no way a random json parser. It is practically the goto standard together with gson.

@hakanai So I checked and apparently I was wrong. Bringing json-simple on classpath indeed includes an outdated version of junit that is vulnerable to exploits, so I switched over to gson now. It also seems like json-simple really is not a standard json parser anymore.
The change will be included in all builds after <IP_ADDRESS>. Once my PR for JCEF gets accepted I will issue a new version.
Thanks for your feedback, I did not notice so far! :)


Queries will show country with the city that is queried.

If you were to look up the temperature by entering "london", then the response would not tell you what London it was checking (The one in England or the one in Ontario).
Also, if you entered a series of valid locations ("london paris berlin"), it would tell you a single temperature value, but you wouldn't be able to tell which city it comes from.
This PR should change the location that is returned to return both a city name and a country, so that the location is made clear to the user.
Please comment if you have feedback about this PR, or if you spot an error. Thanks!

Thanks for the PR @mishakuzma. This is very helpful to the user as there are multiple city names across the globe. Merging and closing.


ENOENT: no such file or directory, open '...../views/pages/layouts/main.handlebars'

Is having the layouts directory mandatory?
Is having the main.handlebars file mandaotry?
If so - the documentation does not state they MUST exist in order for the engine to work.
Here is what I have tried that resulted in the quoted error message:
...
app.set( 'views', __dirname + '/views/pages/' );
...
app.get('/', function( req, res ) { res.render( 'index' ) } );

The file '/views/pages/index.handlebars' does exist.
I realize it may be a noob problem but still it is either the module which doesn't support simplest cases or the documentation which fails to communicate what is required.

All new development is on the repo express-handlebars/express-handlebars.
you can set the defaultLayout option to null if you do not want a layout.

Thank you, that helped. I updated the readme file in the repo you mentioned to give better information on opting out from using layouts. I think this issue can be closed now.


Script randomly skips waves

Before opening an issue:

[x] I tested the latest release
[x] I looked at other issues (even the closed ones)
[x] I read the Troubleshooting Guide
[x] I'll fill this template properly

Your issue will get closed/locked by the bot in case proper info isn't provided
If you open unnecessary/dumb issues, you'll be rewarded with appropriate tags.

Describe the bug
A few days ago I switched to a new node to farm and created a new script, but this script does not what its supposed to do.
Most of the time the script for wave 3 gets activated on wave 2, sometimes the script even executes the steps for wave 2 on the second enemy on the first wave, followed by the steps for wave 3 on the third one (still wave 1). One time this was happening I think I noticed FGA trying to tap the skill for wave 2 right after it pressed the attack button for wave 1 (before the cards even popped up).
I previously tried to find a solution on the FGO reddit to this problem, but nobody could help me solve it.
Script: > i h g1 d ,#, f 5 ,#, g1 i h k1 c 4 ,#,
Party: Summer Scathach, Yu Meiren, Waver
Node: Fallen Babylon
Server: NA
Link to reddit: Here
Screenshots
x
Video
x
Device Info (please fill at-least 'Device model'):

Device model: [Honor 8]
Screen size: [1920x1080]
Android Version: [Android 8 (?)]
RAM: [4GB]

There's a Stage Counter SImilarity option in More-options/Fine-tune.
Try reducing that to 75%.


juliaup windows exe download

where can I get juliaup windows .exe instead of downloading it from Microsoft Store?

On Windows the only supported distribution at the moment is via the Windows Store. I don't plan to distribute individual binaries at the moment as that would essentially defeat the entire auto-update idea. What is the use-case you are trying to solve?

I was thinking of removing bloatware and default Microsoft apps from Windows. Block all sorts of Microsoft telemetry that will also kill Microsoft store and outlook using WPD.app, policies and win registry
I was wondering if I can get JuliaUp via some 3rd party(non-Microsoft) place

I personally don't use the windows store for anything. So I had to literally figure it out just for juliaup. I would much rather just download it from julialang.org and update it infrequently.
I do agree that the windows store is more convenient now that I have it set up.

Closing as dup of #343

@ViralBShah I think winget install julia -s msstore should work on all Windows versions now, out-of-the box without the need to interact with the store. We could also remove the non-store version of Julia from winget and at that point the command winget install julia would do the trick.
Another option is that once we make this the default we have a link that directly opens the store on the correct app page. So someone would go to julialang.org/install, click one link and then click "Install" and that would be it. I think that would work, but not entirely sure.

I think winget install julia -s msstore should work on all Windows versions now, out-of-the box without the need to interact with the store.

I tried it, but…
> winget install julia -s msstore
Found Julia [9NJNWW8PVKMN] Version Unknown
This package is provided through Microsoft Store. winget may need to acquire the package from Microsoft Store on behalf of the current user.
[snip]
Verifying/Requesting package acquisition...
Verifying/Requesting package acquisition failed: no store account found

Yes, this is on a non-linked system.

You know I did get it through the winget install julia -s msstore approach - but I thought that meant that I was actually installing through the windows store and it used something I had stored on my computer for the Windows Store login or something.


Make installable via Scoop

Installing this using Scoop seems like the most logical choice. (I can pick it up if you don't want to create a manifest yourself. I'm just creating the issue to know why this is not in Scoop, since it really seems like a weird oversight.)

I did consider this, but it felt 'weird' submitting it myself. Feel free to do it and let me know if you need help with the manifest, I have prior experience with those.

I actually think that not submitting it is weirder, and it led me to believe there is no such tool available (since I tried scoop search search and it didn't produce anything useful). I have submitted https://github.com/ScoopInstaller/Main/pull/1703

For now I have added the manifest to the repo so it can be installed with scoop by pointing to it: https://github.com/shilangyu/scoop-search/commit/78108c69a2bcae3953b0a586ffaec7daaeeab3be
Will remove it once it is officially on scoop

Thank you for your help, I have updated the README to point to the official scoop manifest :)

Thank you for your help, I have updated the README to point to the official scoop manifest :)


V2.1.9 is not pushed to GitHub

Version 2.1.9 is not released to GitHub. this affected a non working Homebrew installation.
Homebrew tries to install 2.1.9 which is not released yet.

Oops I forgot to do that, I will definitely get it pushed today

https://github.com/willdoescode/nat/releases/tag/v2.1.9


Convert from 16bit to 24bit fail

Hello
I try to  convert 16bit wav to 24bit wav use this crate,
the convert file can be generated as 24bit wav file.
but there is no sound  when I play the generated wav file.
and I try to convert 32bit to 24bit ,it is a big noise wav file.
Is there some wrong??
the code:
use std::path::Path;
use std::{fs, i16, io};

use hound::{SampleFormat, WavSpec, WavSpecEx, WavWriter};

extern crate hound;

fn main() {
    let path: &Path = "chn_1.wav".as_ref();
    let mut reader = hound::WavReader::open(path).unwrap();
    let spec = reader.spec();
    println!("{:?}", spec);

    let samples: Vec<i16> = reader.samples().map(|s| s.unwrap()).collect();

    // println!("{:?}",samples);

    let new_path: &Path = "chn_1_resample.wav".as_ref();

    // let mut buffer = io::Cursor::new(Vec::new());
    let file: fs::File = fs::File::create(new_path).unwrap();
    let mut buf_writer = io::BufWriter::new(file);
    let spec = WavSpecEx {
        spec: WavSpec {
            channels: 2,
            sample_rate: 48000,
            bits_per_sample: 24,
            sample_format: SampleFormat::Int,
        },
        bytes_per_sample: 3,
    };

    let mut writer = WavWriter::new_with_spec_ex(&mut buf_writer, spec).unwrap();
    for sample in samples {
        let _ = writer.write_sample(sample);
    }

    writer.finalize().unwrap();

}

Your code takes the 16 bit samples and reinterprets them as 24 bit samples without a shift. This means that a wave of maximum amplitude in 16 bits, would be at 1/256 of the maximum amplitude in 24 bits, making it sound silent. If you left-shift by 8 bits before writing the samples, that should resolve your issue. Be careful to widen the i16 sample to i32 before doing the shift.
Similar for 32 to 24 bits, you'd have to right shift by 8 bits and explicitly discard the additional information.

i got it,thanks


Update mockito-scala to 1.17.27

About this PR
📦 Updates org.mockito:mockito-scala from 1.17.14 to 1.17.27
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.mockito", artifactId = "mockito-scala" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.mockito", artifactId = "mockito-scala" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #29.

Superseded by #34.


third_party\eigen3\unsupported\Eigen\CXX11

when testing tensorflow.dll in vs2015. but  vs2015 can not recongnize "third_party\eigen3\unsupported\Eigen\CXX11\Tensor".
I succeeded in compiling tf1.4 on windows 10+vs2015+cuda8.0+cudnn5.1.  tensorflow.lib and tensorflow.dll  are there.
the error log:
E:\tensorflow2\third_party\eigen3\unsupported/Eigen/CXX11/Tensor(1): fatal error C1014: 包含文件太多: 深度 = 1024
My win32 console program does nothing ,only include some files .src code as follow:
#include
#include
#include
#include "tensorflow/core/framework/tensor.h"
//#include "tensorflow/core/framework/graph.pb.h"
//#include "tensorflow/core/public/session.h"

fatal error C1014: 包含文件太多: 深度 = 1024
fatal error C1014:too many files:depth =1024

I found the error is caused by wrong path.  now  everything is ok.

How did you fix that?

@luyuesheng

@cocoakang
E:\tensorflow
E:\tensorflow\tensorflow\contrib\cmake\build\nsync\src\nsync\public
E:\tensorflow\tensorflow\contrib\cmake\build
E:\tensorflow\tensorflow\contrib\cmake\build\eigen\src\eigen
E:\tensorflow\tensorflow\contrib\cmake\build\protobuf\src\protobuf\src

Sure, don't use this path: E:\tensorflow\third_party\eigen3


QuicrClient: Support Message Handling for MoQ Mesages

This work requires few changes
1: Mapping between MOQ identifiers ( track-alias, subscribe-id, full trackname) to QUICR identifiers ( Namespace, Name)
2. Support handlers for MoQ Messages
3. Handle subscriptions ( for publishers) forwarded
4. URI convertor integration

I believe this is covered by #136  and #161


fAgentTimelineAnalytics CQD fields description and note have conflicting information - which is correct?

Hi team,
As in the title, on this page the fAgentTimelineAnalytics CQD fields description and note have ambiguously worded or conflicting information on the definition of "Call Count" in this table.
The field description says
Call Count | Whole number | Summarize: SumNumber of calls presented and answered by agent
Yet the note (in purple) under it says
Every call queue call presented to an agent is counted on this report, even if it wasn't answered by the agent.
I am customising this report for my client, so I need to know what description of the call count is correct - ie does it include or exclude calls not answered by the agents.
Thank you very much.
Sophia

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

ID: 628345c6-f5f9-fe8e-bfff-faea00a8a25c

Version Independent ID: 30f2d051-5af5-0063-7099-38b5b08ec1e4

Content: Auto Attendant & Call Queue Historical Report - Microsoft Teams

Content Source: Teams/aa-cq-cqd-historical-reports.md

Service: msteams

GitHub Login: @CarolynRowe

Microsoft Alias: crowe

Hello @datafishnz an update to the article is coming soon, thanks for your feedback!

Article has been updated in https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/commit/34ea59feefe5c7ae082e754156d469f8f47a4e3d
Thanks
@scanum