[question]Does envoy support socket timeout ?

Like the connect_timeout in cluster,does envoy support socket timeout?Such as socket read timeout or socket write timeout?

Envoy supports many different types of timeouts. See here: https://www.envoyproxy.io/docs/envoy/latest/faq/configuration/timeouts

I have seen this doc before,But it does not content the socket read timeout or socket write timeout.

If you are referring to SO_RCVTIMEO or SO_SNDTIMEO values than Envoy is not using them, because it is not using blocking I/O.

@yanavlasov Maybe use libevent EV_TIMEOUT in file_event could work?

I do not EV_TIMEOUT applies to to reading or writing from sockets. This is used to create time based event in libevent. Perhaps a better approach is for you to describe the problem you are trying to solve so someone can suggest a solution.

here is my circumstance, Such as thrift proxy , I want to set thrift request timeout in envoy. Surely I can set thrift request timeout in my downstream application code,but I want envoy can work with request timeout if the downstream forget.

This is something that may need to be implemented in the thrift extension if it does not exist already. Adding @zuercher @rgs1  as owners.

cc: @fishcakez


The engine plan file is generated on an incompatible device, expecting compute 7.0 got compute 8.6, please rebuild.

I use V100，please provide the origin model ，thank you

Hi, the baseline model is pretrained by Tianwei, you may contact him to get access to the ckpt<EMAIL_ADDRESS>.

@hufangjian  Hi, I work around this problem to use the trtexec to rebuild the engine from your nvidia environment. e.g.:
$ trtexec --onnx=pfe_baseline32000.onnx --workspace=16384 --saveEngine=pfe_baseline_fp16.engine
$ trtexec --onnx=rpn_baseline.onnx --workspace=16384 --saveEngine=rpn_baseline_fp16.engine.
b.t.w. the path(like model path) in the source code is hard coded, you need to make some modification.

On the other hand, you could set the bool value params.load_engine = false in file samplecenterpoint.cpp


2024-07-03 — 1 new commit(s) made to reference.md in Google Transit's repo

Commit Nº 1
AUTHOR × DATE

Kayla Firestack ( firestack )
July 2, 2024   @   01:59 PM ETC (05:59 PM UTC)

MESSAGE
fix(gtfs/schedule/spec): add missing `Foreign` specifiers (#479)
It seems like these new fields were added but were not consistant with the rest of the document, this fixes those type fields to match with other foreign ID fields.

FILES CHANGED

https://github.com/google/transit/commit/95178e45f8b3b32b417fa10c744359e355d7dd4e

https://github.com/google/transit/commit/95178e45f8b3b32b417fa10c744359e355d7dd4e

@emmambd this is an editorial change, no effect on the validator


Use route intercepting for modals

Using route intercepting for modals instead of having components added everywhere the modal needs to be open is probably better. It'll also help to have a slightly better architecture.

(I've already tried implementing this before making everything a component but it crashed without any useful logs and couldn't find anything on Google)

Will be done creating a ModalsContext with a handler for it. It has several benefits over route intercepting, plus route intercepting doesn't allow exit animations nor overlapping modals (quite bad).


Fix scaling issue in landscape mode on tablet

Color Picker dialog is shown on a tablet (Samsung) device in landscape mode with
colorPicker.show(childFragmentManager, "color_picker")
Expected result: dialog is scaled to fit screen height (width of a screen, height of a media)
obtained result: color picker dialog not fit into media height, and no option to get access to dialog's view to scale programmatically.
See attached screenshot/

@a5-android Hello!
fixed in version 1.0.3 #4


Bug: If an error occurs after clicking the send comment button, the comment text will still be cleared, just as if it was sent successfully

Fixed by f4d8748 commit


Data not updated

Hi,
The level-3 COVID data pulled today (8/4) using R is not updated. The data for 8/3 and 8/2 are the same.
Thanks.

Fixed. And added an automated test to spot similar bugs in the future.
Many thanks!


Error removing login item while using su otheruser

General troubleshooting steps

[x] I have retried my command with --force and the issue is still present.
[x] I have checked the instructions for reporting bugs.

[x] None of the templates was appropriate for my issue, or I’m not sure.

[ ] I ran brew update-reset && brew update and retried my command.
[x] I ran brew doctor, fixed as many issues as possible and retried my command.
[x] I understand that if I ignore these instructions, my issue may be closed without review.

Description of issue
I've installed brew as user "admin" which has "Admin" permissions. I always run brew commands while logged in as user "david" (a user which has "standard" permissions), by switching to admin in the terminal by issuing su admin. This always worked, but now gives errors when removing casks because apparently the "other users" login item can't be removed:
su admin
brew cask remove bettertouchtool

this gets stuck on:

==> Removing login item BetterTouchTool

If i run:
su -l admin
brew cask remove bettertouchtool

i get:
==> Failed command:
/usr/bin/osascript -e tell application "System Events" to delete every login item whose name is "BetterTouchTool"

Command that failed
brew cask remove --force bettertouchtool

Output of command with --force --verbose --debug
$ brew cask remove --force --verbose --debug bettertouchtool
==> Uninstalling Cask bettertouchtool
==> Uninstalling Cask bettertouchtool
Warning: Accessibility access cannot be disabled automatically on this version of macOS.
See System Preferences to disable it manually.
==> Un-installing artifacts
==> 3 artifact/s defined
#<SortedSet:0x007f82db1d1b80>
==> Un-installing artifact of class Hbc::Artifact::Uninstall
==> Running uninstall process for bettertouchtool; your password may be necessary
==> Removing login item BetterTouchTool
==> Executing: ["/usr/bin/osascript", "-e", "tell application \"System Events\" to delete every login item whose name is \"BetterTouchTool\""]
==> 36:91:
==> execution error: An error of type -610 has occurred. (-610)
Error: Command failed to execute!

==> Failed command:
/usr/bin/osascript -e tell application "System Events" to delete every login item whose name is "BetterTouchTool"

==> Standard Output of failed command:

==> Standard Error of failed command:
36:91: execution error: An error of type -610 has occurred. (-610)

==> Exit status of failed command:
#<Process::Status: pid 6900 exit 1>

/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:71:in `assert_success'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:36:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:14:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:18:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:160:in `block in uninstall_login_item'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:158:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:158:in `uninstall_login_item'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:58:in `block in dispatch_uninstall_directives'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:55:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:55:in `dispatch_uninstall_directives'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/uninstall.rb:7:in `uninstall_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:403:in `block in uninstall_artifacts'
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/set.rb:674:in `each'
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/set.rb:674:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:400:in `uninstall_artifacts'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:353:in `uninstall'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/uninstall.rb:22:in `block in run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/uninstall.rb:12:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/uninstall.rb:12:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/abstract_command.rb:35:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:98:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:168:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:132:in `run'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:101:in `<main>'
Error: Kernel.exit
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:173:in `exit'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:173:in `rescue in run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:156:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:132:in `run'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:101:in `<main>'

Output of brew cask doctor
$ brew cask doctor
==> Homebrew-Cask Version
Homebrew-Cask 1.6.3-13-gaf578f5
caskroom/homebrew-cask (git revision bcdcf; last commit 2018-05-10)
==> macOS
10.13.4
==> SIP
Enabled
==> Java
N/A
==> Homebrew-Cask Install Location
<NONE>
==> Homebrew-Cask Staging Location
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads
~/Library/Caches/Homebrew/Cask (3 files, 29.5MB)
==> Homebrew-Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask (3959 casks)
/usr/local/Homebrew/Library/Taps/domt4/homebrew-chromium (1 cask)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-fonts (1161 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions (177 casks)
==> Contents of $LOAD_PATH
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/Library/Ruby/Gems/2.3.0/gems/did_you_mean-1.0.0/lib
/Library/Ruby/Gems/2.3.0/gems/json-2.1.0/lib
/Library/Ruby/Gems/2.3.0/extensions/universal-darwin-17/2.3.0/json-2.1.0
/Library/Ruby/Gems/2.3.0/gems/psych-2.2.4/lib
/Library/Ruby/Gems/2.3.0/extensions/universal-darwin-17/2.3.0/psych-2.2.4
/Library/Ruby/Site/2.3.0
/Library/Ruby/Site/2.3.0/x86_64-darwin17
/Library/Ruby/Site/2.3.0/universal-darwin17
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/vendor_ruby/2.3.0
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/vendor_ruby/2.3.0/x86_64-darwin17
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/vendor_ruby/2.3.0/universal-darwin17
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/x86_64-darwin17
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/universal-darwin17
==> Environment Variables
HOMEBREW_CASK_OPTS="--appdir=/Applications --prefpanedir=/Library/PreferencePanes"
LC_ALL="en_US.UTF-8"
PATH="/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/Homebrew/Library/Homebrew/shims/scm"
SHELL="/bin/bash"

Output of brew tap
$ brew tap
buo/cask-upgrade
caskroom/cask
caskroom/fonts
caskroom/versions
discoteq/discoteq
homebrew/completions
homebrew/core
homebrew/fuse
homebrew/php
homebrew/services

Problems with AppleScript on multi-user machines is an open issue: https://github.com/caskroom/homebrew-cask/issues/19076

This always worked, but now gives errors

No idea as to why it's stopped working, sorry.


Add Prometheus and Grafana support

This PR adds:

Prometheus, Grafana support
Basic Nodejs dashboard
Docker compose support and configuration

Merging this PR will be delayed.
After discussing with @Didi about providing just a small Nodejs dashboard, can help more tech savvy users, but overall is something not very useful outside core development of sentinel itself.

The PR will be concluded with the inclusion of data metrics rails (what and how metrics collected).
Rails should be extendable, to include later metrics.
Get access to metrics data without using grafana. (targeting a more broad integration in general)
Provide a custom grafana dashboard based on that set of metrics.

merging this even without application specific metrics data yet fed, but without making it the default docker compose file.


Fix for #265; message source frames don't update (Indigo)

Several plugins were storing the source frames of messages received when
they first received a message but never updating them, so subsequent
messages in different frames would be rendered incorrectly.

Per our discussion at lunch on Friday, this needs a more sophisticated fix for any plugin that keeps a buffer of messages.

I've updated all of the plugins that store buffers of messages so that they store the source frames for each message individually and transform them appropriately, so that should fix the issue.

I made one comment inline about size datatypes. Otherwise, LGTM. I vote to merge this, then soak-test for a while before we cut another Debian release.


Receipt Info Not Updated in Transactions Table

so for this issue, receipt table was deleted. from now, receipt API will not update any part of the database. Rather it's just going to extract information. In the UI the automatically extracted info will be filled, then just like manual transaction the rest of the information will be filled by the user. Then add transaction API will be called.


Can't add 2nd hyperhdr instance

 Have searched through the current issues and none represent my issue
Version of the custom_component
v0.0.7
Configuration
I have two instances of HyperHDR, one in the living room and one in my bedroom. The first one, in the living room, was added to the integration fine. But after putting together another hyperhdr instance in the bedroom I cannot add this 2nd setup to the integration.

Add your logs here.

Describe the bug
A clear and concise description of what the bug is.
When I try and add a 2nd hyperhdr instance in HA, via Settings / Devices & Services / HyperHDR / Add Entry and then I provide it with the IP address of the new setup and port 19444, on clicking submit, I get the message "Service is already configured"
So it doesn't look like the current integration can handle more than one instance.
Debug log

Add your logs here.

I'm not sure what could be causing your issue.
I tried setting up a second instance and it worked as expected.

I will try removing the integration and adding it again.


GHW Operation Canceled:  The runner has received a shutdown signal.

The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled.
I believe this happens due to AWS ec2 instance runner issue.
https://github.com/starburstdata/stargate/actions/runs/3385449725/jobs/5623601208#step:19:2024

Related: https://github.com/starburstdata/stargate/issues/4696


Changed #ifndef WIN32 to #ifdef POSIX to better identify POSIX specif…

Minor changes to positively identify POSIX code rather then assuming that not WIN32 = POSIX

This breaks builds on linux because the POSIX macro is not predefined. #12


YouTube Live URLs fail canPlay check

Current Behavior
YouTube's live URL format (ex: https://youtube.com/live/AbCdE1Fgh2i) fails the pattern check if you use ReactPlayer.canPlay().
Expected Behavior
ReactPlayer.canPlay("https://youtube.com/live/AbCdE1Fgh2i") should return true
Steps to Reproduce
Pass "https://youtube.com/live/AbCdE1Fgh2i" into ReactPlayer.canPlay(url)
Environment

URL attempting to play: Any YouTube video containing live instead of watch in the URL. This is not my video but an example https://youtube.com/live/WrFhI8AxpOc
Browser: Chrome Version 108.0.5359.98
Operating system: macOS Monterey

Other Information
The product I work on allows users to embed a video into their instance. We use ReactPlayer.canPlay(url) to validate a form field. The form field is showing a validation error anytime someone uses a /live URL.

canPlay returns false because that URL cannot be played – youtube.com/live URLs are not supported. I’ll see how easy it would be to add support for them.

Should I open a new issue for this bug or it's the same logics?
// The 2 urls target the same video
console.log(ReactPlayer.canPlay('https://youtube.com/w/lthffObMcw0')); // false
console.log(ReactPlayer.canPlay('https://www.youtube.com/watch?v=lthffObMcw0')); // true

Just ran into this issue aswell. It would be nice to have youtube.com/live URL support, if it isn't too hard to implement.


JS injected CSS causes slider container overgrowth.

If you insert an img into a slider and set object-fit to cover; its container will get en extra bottom "space". Looks like it's caused by this line in the tiny-slider.js:
$ && (M || ht ? (Fi(zt, "#" + le + " > .tns-item", "font-size:" + h.getComputedStyle(rt[0]).fontSize + ";",
Demo link/slider setting:   https://codepen.io/anon/pen/pQRKzz
Tiny-slider version: 2.8.8
Browser name && version:  All major desktop and mobile browsers
OS name && version:   Windows / Android

Hey, I didn't see you set object-fit: cover anywhere in the demo

Sorry, looks like codepen lost it. https://codepen.io/anon/pen/pQRKzz

I think this issue is similar with a > img which create a little bit space beneath the img, so
.tns-item img {
  vertical-align: bottom;
  /* or display:block; */
}

will fix this issue.


RangeSlider Thumbs Out of Sync With Tick Values

Describe the bug
The RangeSlider thumbs don't appear to line up with the tick lines as expected. This is even without manipulating the thumbs themselves. I have a suspicion the problem gets worse the higher the values thumb values are. My min max range is 0 - 749, but I can see the problem start in the mid low 10s.
Steps to reproduce
I made a simple app to showcase the issue. There really isn't much to it. Assuming I didn't mess something up, the problem is fairly apparent. This is another bug along similar lines. His issues seems to be with losing sync over time, whereas what I'm reporting is a desync with the ticks starting with values declared in WPF before any manipulation. His screenshots are nice though so I'm linking for reference.

In the screen shots you can see that the slider (the lower control) is aligned with the ticks where both of the thumbs on the RangeSlider are off tick.
Here is a link to the git project I took my screens from. Let me know if there is any issues grabbing or running it. I had a co worker verify but you never know.
Thanks for all your hard work!
Environment
MahApps.Metro version: v2.4.5
Windows build number: Win10 1909 [Version 10.0.18363.836]
Visual Studio: 2019 16.6.0
Target Framework: .Net Core 3.1 (v4.5.2, v4.7.2)

Hi @terrordoll
I think this can be controlled by the MinRangeWidth-Property. If you set this to 0 the result looks like:

I hope this helps
Happy coding
Tim

Hi @terrordoll
I think this can be controlled by the MinRangeWidth-Property. If you set this to 0 the result looks like:

I hope this helps
Happy coding
Tim

I think you may be misunderstanding me. The issue isn't with minwidth, it's that explicitly setting the the slider upper or lower values don't coincide to the tick marks. If I set tick interval to 25 and then in xaml set the upper value to 100, it doesn't line up with a tick as expected. This behavior is the same if we drag.

@terrordoll
Changing MinRangeWidth has an influence, please test it in your App:

MinRangeWidth = 30

MinRangeWidth = 0
I think the issue is here, that the slider thumb cannot be centered on the ticks as this would make it undraggable if UpperValue=LowerValue
Am I right?
Thanks
Tim

Hrmmm. I see what you are saying and perhaps this is an issue of perception.
My expectation was that the RangeSlider thumbs would center on the line. This is both consistent with the standard slider and visually accurate with the tick marks. For the sliders to be off tick, regardless of the reason, is to induce inaccuracy. If there is agreement on this, then the issue is really with the logic that governs dragging or binding order of operation.
For example, I would expect binding both thumbs to the same value would result in either the upper or the lower thumb being set at the correct location and the other being offset by minimum value (and potentially 1/2 the width of the thumb, I'm not clear on the math). What I wouldn't expect is both of them being visually "off".
But honestly I'm not even sold that is what's happening. Set the lower value to 0 and the upper to 25, with a interval of 25. You can see that the thumb is slightly to the right of the selected value. Now drag the thumb down the line and you can watch it drift a few pixels or more at the end of the slider.
You can see upper value is a few pixels to the right-

Further down the line and it's crept to the left a pixel or two-

All the way down and it's centered-

Lower thumb has the reverse behavior-

My assertion is there is some bug causing drift and additionally I think the default behavior of the control is confusing. At least to me.

Hi @terrordoll
Below please find a Style which should suite your needs.  I wanted to say thanks for the effort regardless of whether or not it's a solution for me. I appreciate it.


Split export results into pages

This would be useful to include the results in LLMs that have a maximum token context window.

Fixed with #7.


⚠ Bofrost.de has degraded performance

In 9b22231, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1593 ms

Resolved: Bofrost.de performance has improved in 3979547 after 7 minutes.


Local variable inside an if block inherits the type of the previous block.

When a condition is true before assigning the local variable:
if x = "abc"
  pp! x          # => "abc"
  pp! typeof(x)  # => String
end

if true && (x = 1.1)
  pp! x          # => 1.1
  pp! typeof(x)  # => (Float64 | String)
end

if (x = 1.1) && true
  pp! x          # => 1.1
  pp! typeof(x)  # => Float64
end

https://play.crystal-lang.org/#/r/9j6y

This issue can be decomposed in two parts.
First part, when a variable is assigned in an if’s condition, the variable is available outside the if block.
if x = 123
  pp! x # => 123
end
pp! x   # => 123

I don’t know if it’s a feature, but I expected the variable to be isolated inside the if block, like for the try block:
123.try do |x|
  pp! x # => 123
end
pp! x   # => Error: undefined local variable or method 'x' for top-level

Second part, when a variable already exists and it is reassigned in an if’s condition, sometime the type is overwritten, sometime merged.
Overwritten when the assignment is made in the first condition:
x = "abc" # => String
if (x = 123) && true
  pp! typeof(x) # => Int32
end
pp! typeof(x) # => Int32

And merged when the assignment is preceded by a true condition:
x = "abc" # => String
if true && (x = 123)
  pp! typeof(x) # => (Int32 | String)
end
pp! typeof(x) # => (Int32 | String)

Same here, I expected the variable to be perfectly isolated like in a try block:
x = "abc" # => String
123.try do |x|
  pp! typeof(x) # => Int32
end
pp! typeof(x) # => String

I think all of this is expected behavior.
First, Crystal doesn't have block scoping; scopes of variables are limited only to the function entirely.
Second, yes, Crystal will make a union out of a variable if it could have different types in different conditions. Then there is a special case that, if it's 100% certain the assignment will overwrite anything prior, then prior types can also be thrown away.
Then you can still argue that if true && (x = 1.1) is "100% certain" but the optimizations just don't go that far and probably aren't intended to.

A variable's scope is the entire method, or a block. Never an if, while, until or unless.
That said, this is a bug: x on the second if should not have mixed types.

Thank you, oprypin and asterite, for your explanations.
I like the behavior of the try block, is it possible to pass several variables to it at the same time? Something like that:
{@x, @y, @z}.try_all do |x, y, z|
  # here x, y, z are not nil
end

Which would be equivalent to:
@x.try do |x|
  @y.try do |y|
    @z.try do |z|
      # here x, y, z are not nil
    end
  end
end

Thanks @Blacksmoke16, I closed this issue because during the last two years I have never encountered this problem again. And the title and the description are unclear so even if someone has the same problem I doubt that he will be able to find this issue via the search.


Question: How to use handler outside of the list

Hi,
Is there a work around to set a element outside the list as the handler to drag elements with in a table. Can we pass in a dom reference to the handler?

+1

You may achieve this by using absolute positioned elements.

Of course, that would be a way, but I don't think that's what the OP was referring to.
For example, if you're using Sortable to create a drag'n'drop builder, it would be helpful if you could specify any DOM element as a handler.
This way, you wouldn't need to inject a handler for Sortable to use, 'inside' every module of the template that you're using Sortable on ("within list items").

Just like what cosmin said, I'm looking for a way sortable can bind handler that is not within the list/table.
E.g. If i want to make table rows sortable but would like to have the handler not inside the rows. Would be useful if we can set a dom reference to the handler (which is not required to be inside the list/table). This will make the sortable plugin more flexible

It impossible to make when using Native DnD.


Possibly revamp/hone the validation/checking that goes on at time of randomization

This always comes up: The fact that site staff may have to re-enter / verify the stratification factors at the exact time of randomization. It is imperative that mistakes not be made at this stage, so coordinators/investigators are always asking about double-checking/validation/confirmation.
I wanted to get an issue on the books to keep this in mind when we pick back up with Slice.
From a related email thread:

I was going to bring this particular issue up with Remo again – i.e. either 1) having the stratification factors pulled from elsewhere (i.e. other Designs/Sheets) or 2) having each individual factor be sort of “double checked” against entry that has occurred previously.
Right now there is a rudimentary system in place where the Randomization cannot occur in Slice if a single criteria (i.e. variable on another Design) is not met (i.e. has the value specified). Maybe this could be embellished a bit to make #2 above more robust.

Perhaps allow each stratification factor to be linked to variable, whose value would be pulled for a given subject at the time of randomization.

In discussing, I think we could do these similar to calculated variables, so that you'd provide a formula along with the stratification factor so that you could essentially compute a value from multiple variables. We'll have to figure out how to handle the subject having multiple (or missing) values for certain variables though.

Reference the "Child Information Worksheet" in the PATS project in production.

ciw_age_years represents the child's age in years.
ciw_age_cat_for_randomization is the calculated field below it, using this calculation:
(ciw_age_years > 2 && ciw_age_years < 6) ? 1 :  (ciw_age_years >= 6 && ciw_age_years < 10) ? 2 : -9

This calculated field should then be referenced from the "Age Category" randomization stratification factor, where the "1" and "2" values will align:

If the factor is tied to another variable (or just the calculation itself), then this stratification factor could only be set in this fashion (i.e. can't be set manually). In cases where the calculation was not resolving into a "1" or a "2", we would need, in some manner, to alert the user as to why randomization is being thwarted at this point (e.g. some reference to the calculation and/or Child Information Worksheet variables needing to be filled in).


Compatibility with Rust 1.33

Change-type: patch
Signed-off-by: Zahari Petkov<EMAIL_ADDRESS>

@balena-ci I self-certify!


Adf.ly broken

http://adf.ly/1DX0XD
1º = Decrypted: 3ace91a5644ca7h|tps://github.com/matteocontrini/node-b{passer/7c1fe1be82ea5799
2º = Decrypted: 1d31a6ffbfa457h|tps://github.com/matteocontrini/node-b{passer/5:5f7290498c8a90
3º = Decrypted: ba6:cbb43ab12ah|tps://github.com/matteocontrini/node-b{passer/&c4b19f8de545315

Hi, I'm sorry but I currently don't have time for fixing this. Although the new algorithm for decoding the link has been already extracted in other open source projects.


how to debug a torch op?

I had developed a GPT-NEOx torch op. but when I load the .so lib file and execute the python code, it throws an exception without any additional message or traceback.
#python /project/deployment/FTGeneration.py
[INFO] WARNING: Have initialized the process group
Segmentation fault (core dumped)

so I want to ask your way to debug torch op when you develop yours. excuse me for I m not familiar with c++. thx.

Add debug/log message into the pytorch op and trace code by code.

Add debug/log message into the pytorch op and trace code by code.

oh, I see. I'll do the same. thx so much.


Allow to add a custom opener to func_map

Feature description
Hi,
it seems for me (i'm fresh in Lua) that the set of allowed keys of func_map is restricted to action_funcref from keymap.
I'd like to attach a custom opener that does an action similar to tab drop (see links below).
Describe the solution you'd like
Please allow to attach a function (as the opener) that get the path, (and, if provided) line and column to allow make a customized jumping.
Additional context
https://github.com/neovim/neovim/issues/14567
https://github.com/przepompownia/vim-arctgx/blob/master/plugin/tab.vim#L11

What's different between drop and tab drop? If they are the same, I can add a drop action.

Oh, my bad. tab drop will open a new tab for target buffer if the buffer is not displayed but drop will switch to target buffer in the current window.

Maybe tab drop could be present by default in action_funcref but it is not sufficient for me - I am not use tab drop because of the bug mentioned above (that exist in traditional Vim too).

I have added three new jump commands. All of them will close qf window, you can wrap the command by yourself.
FYI: https://github.com/kevinhwang91/nvim-bqf/issues/39

I see that
lua require('bqf.qfwin.handler').open(true, 'TabDrop')

works for me when I call it manually from the cmdline, but filetype autocommand
nmap <buffer> <CR> <Cmd>lua require('bqf.qfwin.handler').open(true, 'TabDrop')<CR>

seems to be overwritten by <Cmd>lua require('bqf.qfwin.handler').open(false)<CR>
How to attach such custom funcref? Why not to allow overriding action_funcref from config (if it's currently impossible)?

I see that
lua require('bqf.qfwin.handler').open(true, 'TabDrop')

works for me when I call it manually from the cmdline, but filetype autocommand

No TabDrop but tab drop

nmap <buffer> <CR> <Cmd>lua require('bqf.qfwin.handler').open(true, 'TabDrop')<CR>

seems to be overwritten by <Cmd>lua require('bqf.qfwin.handler').open(false)<CR>
How to attach such custom funcref? Why not to allow overriding action_funcref from config (if it's currently impossible)?

You can, RTFM

No TabDrop but tab drop

tab drop  causes unwanted side effects (see links in the initial message) so I need to use such custom replacement.

FYI: #39

require('bqf').setup({
  auto_enable = true,
  func_map = {
    open = '',
  }
})

allows to disable nmap <CR> defined by the plugin so now I can use the mapping mentioned above.


Not checking for ESP_FAIL in AsyncCanDriver::transmit

For non-blocking implementations of the embedded-hal the ESP_FAIL error is treated as WouldBlock, shouldn't the same be done for the async function implementation?
pub async fn transmit(&self, frame: &Frame) -> Result<(), EspError> {
    loop {
        let res = self.driver.borrow().transmit(frame, delay::NON_BLOCK);

        match res {
            Ok(()) => return Ok(()),
            Err(e) if e.code() != ESP_ERR_TIMEOUT => return Err(e),
            _ => (),
        }

        WRITE_NOTIFICATION.wait().await;
    }
}

No idea without digging (again) into the ESP IDF C source code of the TWAI driver...

@leonardodanelutti The way I read the code (I might be wrong) is that ESP_FAIL would only be returned if you are creating the driver without a TX queue (i.e. tx_queue_len == 0)
Can you confirm if you are doing that?

@ivmarkov I actually didn't get that error, I was looking at the async implementation for the CanDriver and it seemed strange that the ESP_FAIL error wasn't also checked. Yes, I think you are right, as it is also written in the documentation, ESP_FAIL is returned only if the TX queue is disabled and another message is transmitting at that moment. When that error occurs, I don't think the function should return, right?

Yes I'll patch the driver shortly.

https://github.com/esp-rs/esp-idf-hal/commit/f958b77bf094f1dc42ea12ebacc651ed1a244221


CBAPI-2681: Guide for Watchlists in CBC SDK

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Tests have been added that prove the fix is effective or that the feature works.
[ ] New and existing tests pass locally with the changes.
[x] Code follows the style guidelines of this project (PEP8, clean code, linter).
[x] A self-review of the code has been done.

What is the ticket or issue number?

https://jira.carbonblack.local/browse/CBAPI-2681
Pull Request Description

Added a guide to the "Guides and Resources" section of the documentation covering watchlists, feeds, reports, and IOCs.
Does this introduce a breaking change?

[ ] Yes
[x] No

How Has This Been Tested?

Documentation only.

Done.


GL texture size checks and unsigned int samplers

I don't think we are clear to publish the patched version of GL crate but otherwise the changes are good to go!
@homu r+

:pushpin: Commit 0cfe0b5 has been approved by kvark

:zap: Test exempted - status


Ss 165 update documentation in user guides

Status

[ ] Ready
[ ] Draft
[ ] Hold

Description
Describe your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to include a link to that issue.
Types of changes
What types of changes does your code introduce to STACKn?

[ ] Hotfix (fixing a critical bug in production)
[ ] Bugfix
[ ] New feature
[ ] Documentation update

Checklist
If you're unsure about any of the items below, don't hesitate to ask. We're here to help!
This is simply a reminder of what we are going to look for before merging your code.

[ ] This pull request is against dc-develop branch (not applicable for hotfixes)
[ ] I have included a link to the issue on GitHub or JIRA (if any)
[ ] I have included migration files (if there are changes to the model classes)
[ ] I have read the CONTRIBUTING doc
[ ] I have included tests to complement my changes
[ ] I have updated the related documentation (if necessary)
[ ] I have updated the release notes (docs/releasenotes.md)
[ ] I have added a reviewer for this pull request
[ ] I have added myself as an author for this pull request

Further comments
Anything else you think we should know before merging your code!

From my point of view serve guides are in general could be made more interesting, but everything is fine if we are just doing minor restructuring


Add assembly ("memory-safe") to the inline assembly blocks

See https://docs.soliditylang.org/en/v0.8.17/assembly.html#memory-safety for information on the tag
If via-ir is used, the compiler may lift variables from stack to memory. This is only correct if all assembly block uses memory correctly, i.e., only use memory from [0, 0x40) (scratch space) and allocating memory starting from the current free memory pointer.

From experiments, this has no improvement on performance, so it is mostly for via-ir to be able to avoid stack too deep.
Need to think through on this.
Some snippets abuse 0x40 and 0x60 temporarily.
Some snippets allocate memory after populating it.
Not certain about how will the compiler handle such cases.

From experiments, this has no improvement on performance

Yes, this is true. If the original code doesn't need lifting to memory, then there wouldn't be any difference.
The is more important when the library is used in contracts, and they need this memory lifting. It would be blocked by having Solady as a dependency.

Some snippets abuse 0x40 and 0x60 temporarily.

Strictly speaking, this is abusing memory and can potentially lead to undefined behaviour. But in practice, I think it's unlikely these will have an effect.

Some snippets allocate memory after populating it.

This should be fine. Assuming the allocation follows the usual rules.
Maybe to give you some idea of what's going on: usually, the free memory pointer points to 0x80. If there are, say 3 variables that needs to be lifted into memory, instead of starting the free memory pointer at 0x100, the compiler instead starts at 0x80 + 3 * 0x20. The three words starting from 0x80 until 0xa0 are then statically used to store these lifted variables. If you happened to use these slots in your inline assembly block, then it can lead to undefined behaviour.
In short, even if you mark a small subset of assembly blocks as "memory-safe", it's still useful for contracts using those functions. @Vectorized


Create GH action for automated docker builds on releases

As we discussed, this is a first draft of publishing docker images on releases.
cc @williamFalcon @Borda

@Borda the docket image that will be created depends on the dockerfile we will provide here

@Borda the docket image that will be created depends on the dockerfile we will provide here

just don't see the Dockerfile or command/path in the action...

@Borda I added a build matrix as well as build arguments for the docker file and it's location along with your other requested changes. All paths and arguments for the docker file are based on #1569

@williamFalcon @justusschock I think it was not complete..

will continue in #1613


Homebridge  Turns Off When I Exit Terminal

Everything works great when I start home bridge.
I think hit CNTRL + Z to exit the process and then hit "bg" to send it to background (still works at this point) and exit out of terminal.
Home bridge immediately stops and kills the process. Any ideas?

I use screen.  This enables a little more control over the job than the bg, fg tools.

That gave me an idea. I logged into VNC and opened up terminal and ran this script and left it on. Instead of logging in via my iPad or computer that I shut down this keeps it on always.

Please don't get me wrong but this isn't the place to learn linux basics. You should start by learning some basics. Here are some useful links:
https://github.com/nfarina/homebridge/wiki/Running-HomeBridge-on-a-Raspberry-Pi#running-homebridge-on-bootup-initd
https://github.com/fhd/init-script-template


Basic String.prototype.matchAll and friends implementation

This seems to work in my testing, although unfortunately enabling the tc39 tests for this turn out to require additional code changes, and they:

don't work because tests use non supported syntax/features
some of them are wrong as the version used is kind of old and apparently the spec/tests changed  in between :sob:

I have used the fact that I can run the tests (and a newer version of them) in k6 and have done so, but still, some fail. So I will need to look into those first.
Also, this definitely will need a "fast path" as the current performance isn't up to snuff :( :
BenchmarkRegexpMatchCache-8<PHONE_NUMBER> ns/op         1410097 B/op      22338 allocs/op
BenchmarkRegexpMatchAll-8<PHONE_NUMBER> ns/op         2077386 B/op      31859 allocs/op

I still welcome any feedback even before this becomes a non draft

This seems to work in my testing, although unfortunately enabling the tc39 tests for this turn out to require additional code changes, and they:

don't work because tests use non supported syntax/features
some of them are wrong as the version used is kind of old and apparently the spec/tests changed  in between :sob:

I have used the fact that I can run the tests (and a newer version of them) in k6 and have done so, but still, some fail. So I will need to look into those first.
Also, this definitely will need a "fast path" as the current performance isn't up to snuff :( :
BenchmarkRegexpMatchCache-8<PHONE_NUMBER> ns/op         1410097 B/op      22338 allocs/op
BenchmarkRegexpMatchAll-8<PHONE_NUMBER> ns/op         2077386 B/op      31859 allocs/op

I still welcome any feedback even before this becomes a non draft

I fixed all tc39 tests as of e29bb81, but the ones that use
var a = {
  get [Symbol.match]() {
  ...
  }
}

or
var a = {
   [Symbol.match]: function() {
   ....
   }
}

two of which I've backported and included in the test, I guess I can do all 4 if you want.

I fixed all tc39 tests as of e29bb81, but the ones that use
var a = {
  get [Symbol.match]() {
  ...
  }
}

or
var a = {
   [Symbol.match]: function() {
   ....
   }
}

two of which I've backported and included in the test, I guess I can do all 4 if you want.

Hi. I've had a quick look, there is nothing clearly wrong about the approach. I haven't looked at the specs close enough to see which optimisations (if any) are possible, but I would accept the PR without optimisations, as having any support is better than not having it and polyfilled implementations are unlikely to offer a better performance. I'll have a closer look at the code when the PR is ready.

Hi. I've had a quick look, there is nothing clearly wrong about the approach. I haven't looked at the specs close enough to see which optimisations (if any) are possible, but I would accept the PR without optimisations, as having any support is better than not having it and polyfilled implementations are unlikely to offer a better performance. I'll have a closer look at the code when the PR is ready.

Final benchmarks from here:
name                time/op
RegexpMatch-8        147µs ± 8%
RegexpMatchCache-8  3.96ms ± 8%
RegexpMatchAll-8    5.51ms ± 9%

name                alloc/op
RegexpMatch-8       33.3kB ± 0%
RegexpMatchCache-8  1.41MB ± 0%
RegexpMatchAll-8    2.08MB ± 0%

name                allocs/op
RegexpMatch-8          668 ± 0%
RegexpMatchCache-8   22.3k ± 0%
RegexpMatchAll-8     31.9k ± 0%

So just calling exec is still much faster, but my tries at optimizations showed that I can only really do something in the calls for next and only after I have checked that absolutely nothing has changed which I expect won't really be that much faster so :shrug:
I ran this script with k6 before and after the changes in https://github.com/loadimpact/k6/pull/1811 (which basically drop corejs and update goja to the code here, the previous code isn't what is currently in goja's master, but I don't think it matters enough I am comparing the corejs Polyfill). The script is :
var body = "someString Taht we care about \r\n".repeat(1000)

exports.default =  function() {
    for (var i= 0; i < 1000; i++) {
        for (var v of body.matchAll(/care about/g)) {}
    }
}

Where the default function execution took 4.73s(no polyfill) vs 5.01s with polyfill. Some variation exist but is within 100ms so I think we can call it good :D

Final benchmarks from here:
name                time/op
RegexpMatch-8        147µs ± 8%
RegexpMatchCache-8  3.96ms ± 8%
RegexpMatchAll-8    5.51ms ± 9%

name                alloc/op
RegexpMatch-8       33.3kB ± 0%
RegexpMatchCache-8  1.41MB ± 0%
RegexpMatchAll-8    2.08MB ± 0%

name                allocs/op
RegexpMatch-8          668 ± 0%
RegexpMatchCache-8   22.3k ± 0%
RegexpMatchAll-8     31.9k ± 0%

So just calling exec is still much faster, but my tries at optimizations showed that I can only really do something in the calls for next and only after I have checked that absolutely nothing has changed which I expect won't really be that much faster so :shrug:
I ran this script with k6 before and after the changes in https://github.com/loadimpact/k6/pull/1811 (which basically drop corejs and update goja to the code here, the previous code isn't what is currently in goja's master, but I don't think it matters enough I am comparing the corejs Polyfill). The script is :
var body = "someString Taht we care about \r\n".repeat(1000)

exports.default =  function() {
    for (var i= 0; i < 1000; i++) {
        for (var v of body.matchAll(/care about/g)) {}
    }
}

Where the default function execution took 4.73s(no polyfill) vs 5.01s with polyfill. Some variation exist but is within 100ms so I think we can call it good :D


fix pydatavec for python 3

... and python2 install problems

@maxpumperla is it simple to add the chaining mechanism that Philip mentioned here? https://stackoverflow.com/c/konduit/questions/11

@ShamsUlAzeem buddy, I already did that a week ago :D


Feat/shadcn folder

@rogeruiz originally I was going to do this, then I convinced myself not to do it, then I actually ended up doing it 😭  My bad, but I do like this organization. Apologies for the tailwind changes, they are non-functional changes so those can be ignored, just were changed due to my plugin. Bulk of this is basically moving src/components/ui to src/components/common/Shadcn/ui.

@jvmontes I understand you've been pushing for shadcn/ui to be considered an outside library or framework, but it's not. It's a headless UI. Once this code was copied into our codebase it's now owned by us. There's no good way to bring updates from shadcn/ui into our codebase as that's not how it really works. I don't think we need to move things around. In fact, I think we can remove a lot code in shadcn/ui Sidebar since we own it now and we don't need a super-robust sidebar for our application.
Check out the docs for how robust this thing is, https://ui.shadcn.com/docs/components/sidebar
Here's the information about how shadcn/ui is not a libary, https://ui.shadcn.com/docs

This is NOT a component library. It's a collection of re-usable components that you can copy and paste into your apps.
What do you mean by not a component library?
I mean you do not install it as a dependency. It is not available or distributed via npm.
Pick the components you need. Copy and paste the code into your project and customize to your needs. The code is yours.
Use this as a reference to build your own component libraries.

That all sounds reasonable to me. I’m open to suggestions on how to
structure things. If you think we should keep things as they are, that
makes sense to me.
I personally just didn’t wanna have to go through the code we had and
remove stuff.
Jorge Viramontes
jorgeviramontes.com http://www.jorgeviramontes.com/
On Mon, Nov 18, 2024 at 7:18 PM Roger Steve Ruiz @.***>
wrote:

@jvmontes https://github.com/jvmontes I understand you've been pushing
for shadcn/ui to be considered an outside library or framework, but it's
not. It's a headless UI. Once this code was copied into our codebase it's
now owned by us. There's no good way to bring updates from shadcn/ui into
our codebase as that's not how it really works. I don't think we need to
move things around. In fact, I think we can remove a lot code in shadcn/ui
Sidebar since we own it now and we don't need a super-robust sidebar for
our application.
Check out the docs for how robust this thing is,
https://ui.shadcn.com/docs/components/sidebar
Here's the information about how shadcn/ui is not a libary,
https://ui.shadcn.com/docs
This is NOT a component library. It's a collection of re-usable components
that you can copy and paste into your apps.
What do you mean by not a component library?
I mean you do not install it as a dependency. It is not available or
distributed via npm.
Pick the components you need. Copy and paste the code into your project
and customize to your needs. The code is yours.
Use this as a reference to build your own component libraries.
—
Reply to this email directly, view it on GitHub
https://github.com/Alacran-Studio/lpdd/pull/38#issuecomment-2484507036,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS4NRZMGZ4MPF27QAVKEWD2BKGVJAVCNFSM6AAAAABR5PAD22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBUGUYDOMBTGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Intermittent very long page loads (sometimes with timeout)

I have not seen this recently. This might be due to the site being cached on my computers, but also did not find it when user testing on fresh computers. I suggest we close it.


Name parameter New-DbaDatabaseSnapshot should be name of snapshot and not suffix

Is this:

[ ] feature
[X] bug:

Steps to Reproduce

Run Command:

New-DbaDatabaseSnapshot -SqlInstance server -Database Northwind -Name Northwind_dbss

Received snapshot named "NorthwindNorthwind_dbss"

Expected Results

Run Command:

New-DbaDatabaseSnapshot -SqlInstance server -Database Northwind -Name Northwind_dbss

Receive snapshot named "Northwind_dbss"

As it is called "Name" it should be the exact name of the snapshot being created. If a user wants to append the something to the name of the snapshot a parameter like "Suffix" or "Append" should be provided.

it's really neither, more of a "template" .
If it's accepted as a solution I can work in a -Name that takes the exact string passed in OR a -NameTemplate that takes the interpolation as it is right now

Template does not sound right to me. Whether you want to use the math term (yeah I googled it :expressionless: ) you are appending the value to the basename of the snapshot.
I would say offer both the -Name and a -NameSuffix, which is more self-explanatory than NameTemplate. I'll leave that to @potatoqualitee .

you can pass a -Name 'fool_{0}_snap' to have the {0} placeholder to be picked up from the databasename...
nonetheless in any other case is a suffix.

I working on template system for file and path parameters for Backup-DbaDatabase, may we move template handling to internal function?

dunno if it's needed.... if you want to support flexibility, a template string is plenty. The "problem" here is just naming parameters properly, not the functionality itself

@potatoqualitee : ideas on the parameter naming here ?

I like shawn's suggestions


:bug:fix firefox can't set height smaller than content on flex item

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

带有flex-grow:1的container元素的高度在子节点height较大时直接采用子节点高度，而不是根据父容器剩余高度适应


Feature/civ 2002

Before creating a pull request make sure that:

[X] commit messages are meaningful and follow good commit message guidelines
[X] README and other documentation has been updated / added (if needed)
[X] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-2002
Change description
Make changes to civil-general-applications and civil-general-app-ccd-definition for respondent details
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

merging this with a failure in smoke test since it requires GA CCD changes. IT has been proven by switching branches smoke test passes


Update README.md

I'd like to create a branch like this from develop and then merge it with the sar and the updated readme into develop and main

redundant


Feature Request: Add support for client-only usage

I want to use this for parsing a properties file from the client side to JSON and then upload only the parsed, validated results. Is this all under the scope of this project?
The primary change would be to support File Blob as input and remove a required dependency for node:fs or expose the parser function only.

Hi @ajitzero - which version are you currently using?
In the package's latest version, getProperties takes as input either a string or a buffer. There is no dependency on node:fs.
Since your use case is within a browser, you can convert your Blob to a string, and it should work seamlessly.
Please let me know if I misunderstood your question.

you can convert your Blob to a string, and it should work seamlessly.

I see. The example from the README implied this is for server-side usage only (with possibly a file upload pattern). Let me try this and confirm. I'll update and close this as necessary.

I apologize for any confusion caused by the documentation. Could you please provide more details about the specific part that gives the impression of being limited to server-side usage?
We recently made updates to our documentation due to a previous version containing two different sets of APIs. One set had a dependency on node:fs, while the other did not. Explaining this distinction became quite complex, which is why we decided to remove the node:fs dependency in the recent updates.

I'm going to close this due to innactivity. @ajitzero  please le me know if you have suggestions on how to improve the documentation or need anything else.


fix: typo in the docs

Correct the typo that is from min_destance to min_distance

Nice, thanks for fixing this.


v1.2.1 - Directory Structure Updates

reconfigure directories to decrease nesting and increase project cohesiveness

Must've gotten lucky with the timing of that scratch org creation - 1 minute for full creation + deploy + 190 tests run + code coverage uploaded. 😅


lil list of the 7 quests that aren't able to be completed atm and whats wrong with each one (updated)

War effort - quest randomly became pick up able today however arthur in ione isnt able to be talked to (didnt try the other npcs in the same step) (also he randomly moved 5 blocks earlier today from wheres hes supposed to be to off the side of the platform in ione (he wasnt able to be talked to before or after moving))
Witch Hunt - Starting npc is there and gives u their default voice line about the witch's village but no quest
(Edit#idek - Mindless traveler chapter 3 -starts at the start point for mindless traveller 4  but seems to work soo)
Trials of a Magma Storm - getting teleported down into the lava room where volcanion and heatran are fighting does not start the fight with volcanion, despite the quest updating to defeat volcanion
(Edit#2 -Betrayer's Oubliette was fixed 6 hours before i made this so that ones doable now xD)
(Edit #2312412: iron maiden was fixed the day when i made this so i was able to do it and it unlocked these 2 quests which are both also broken xD)
Much needed supplies - mason doesnt talk to u when u go to take him the supplies (in bastion)
as iron sharpens iron - also takes u to mason who still refuses to talk to u xD
also just to add to the wtf when i went to try hand in the "as iron sharpens iron" supplies to mason the supplies from the "much needed supplies" quest disappeared from my inventory... but didnt update the quest... so ya thats odd
(also merciless isnt able to be done atm so idk about that one (because it requires war effort))
managed to complete all the other quests so i assume they are fine
Hope this helps <3
(Edit#53341 - Remember to Exorcise - u can complete this one but its still broken in a very annoying way, after the boss fight against the possessed guy ur supposed to be dragged into a fight vs the dusknoir possessing him, but u get 3rd person glitched consistently every time u beat him (where u remain in ur battle view but in third person at the same time, when u look around ur character looks around too but cant move, can be fixd by relogging). this wouldnt be an issue as u can normally just relog; but the quest resets u to before the boss fight if u log off or leave the cave, so u cant actually finish the quest without using a workaround to fix the issue. the easiest ways to workaround it are to either stand behind the boss in the exact spot where the dusknoir will spawn, so that when ur third person glitched u can still send out ur poke (it falls at ur feet rather than shooting where u aim when ur in this glitch) and it triggers the fight with the boss, or to fight it from wherever then get someone to stand inside the dusknoir, and tp to them whilst glitched, then throw out ur poke to start the fight)

bumping :P

#271 #270 #269 #268 #267 split this up into individual posts

most are fixed so ill close and leave the individual posts for unfixed up


stm32h7: add SMPS PWR option for STM32H7X7

Summary
The dual core STM32H747 / STM32H757 there is an additional option to select SMPS rather than LDO as the power selection.
This commit adds this option to the STM32H747 config and the stm32h7x7xx source.
Impact
I don't expect any breakage as the default is without SMPS and not changed.
Testing
None yet. Will backport and test later.

Backported and tested on STM32H747 successfully.


Draft of instructions for setting up Mattermost server using TLS

Thanks CS! I made some minor tweaks, +1

Do not merge until https://github.com/mattermost/platform/pull/4119 goes in

@crspeller can you please rebase this onto the v3.5-documentation branch so we can merge before release?

Thanks! Tweaked and rebased in https://github.com/mattermost/docs/pull/608


Missing field "createdDateTime" when retrieving pages of a site

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Linux
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Describe the bug / error
When retrieving news items of a site using the Microsoft Graph API the field "createdDateTime" is missing. The url we're using is:
https://graph.microsoft.com/beta/sites/{siteId}/pages?expand=webparts&$filter=promotionKind eq 'newsPost'
This returns all news items successfully, but without the specified field.****
Steps to reproduce

determine the siteId of the site
call the MS Graph URL with the siteId in it
see that the field is missing (attached an example of what is returned, all personal info is replaced by "x").
SharePoint-export.txt

Expected behavior
Following the documentation on https://learn.microsoft.com/nl-nl/graph/api/resources/baseitem?view=graph-rest-beta we would expect to see the field "createdDateTime" in the results.

Also confirming the same problem, pulling newsPost content with the promotionKind parameter, does not provide the createdDateTime value in the API response.

@VesaJuvonen @sangle7. Is there any update on this regarding the API? I appreicate the API endpoint for SharePoint pages is still BETA.
Looking at the update video in June 2023: https://devblogs.microsoft.com/microsoft365dev/microsoft-graph-for-sharepoint-pages-public-preview-update/, the examples used also show the createdDateTime property missing from the responses as well.
Will this be fixed?

@jamesmacwhite Hi, thanks for reporting this. Will take a look and get back to you soon

@sangle7 Thank you! @roelplieger originally reported it. I just found this GitHub issue when I also came across the same problem when using the SharePoint pages BETA API with news posts.

The fix has been checked in and should be deployed in ~3 weeks

@sangle7 Thank you, this is great news! Look forward to testing it out when deployed.

This appears to be now deployed and being returned in the API!


MSL: Fix dref lod workaround on combined texture/samplers

Example broken shader (compile with --msl --msl-version 20300 --msl-sample-dref-lod-array-as-grad): shader-fs-090d3829caba1bbc.spv.zip

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


Gulp task does not terminate

If I run the my task via gulp it does not terminate (both on clean runs or test failures).
The last log is

[10:13:39] gulp-angular-protractor - Webdriver standalone server is stopped

and in case of test failures:
{ [Error: protractor exited with code 1]
  message: 'protractor exited with code 1',
  showStack: false,
  showProperties: true,
  plugin: 'gulp-angular-protractor',
  __safety: { toString: [Function: bound ] } }

This is my gulp task:
gulp.task('test', function (done) {
    gulp.src([])
        .pipe(gulpProtractorAngular({
            configFile: 'test/conf.js',
            autoStartStopServer: true
        })).on('error', function (error) {
            console.error(error)
        }).on('end', done);
});

Hi
Could check if you have not a similar issue link this: https://github.com/rochejul/gulp-angular-protractor/issues/33#issuecomment-281473188
Because the plugin stop the selenium server, not your application server
Regards

Yes I saw issue #33, but I don't have another task dependency. The server needs to run for the tests and the tests are executed standalone with gulp test (e.g. from a terminal).
BTW thanks for the fast repsonse :)

The application server is always running ? Or is launched with gulp ?
What is your process to run your functional tests ?
Because if you saw "[10:13:39] gulp-angular-protractor - Webdriver standalone server is stopped", that means we have killed the selenium server
And if you saw:
{ [Error: protractor exited with code 1]
message: 'protractor exited with code 1',
showStack: false,
showProperties: true,
plugin: 'gulp-angular-protractor',
__safety: { toString: [Function: bound ] } }
This is the error output from seleninum to explain the error (basically, it said "error code 1", so this is no really helpfull, but it means the tests are failed)
Could you give me a sample of your configuration to reproduce your issue ? And to check your gulp version, plugin version, node / npm version, etc ...
Many thanks

Currently I am integrating the tests in our build chain, but at the moment the server is started from the IDE and I run the tests manually from commandline when the application is already running (in the future this should be integrated with a gradle task which starts the server first and then the tests via npm test which in turn runs gulp test).
Versions:

node : v8.2.1
npm: 5.3.0
gulp: 3.9.1
gulp-angular-protractor: 1.0.0

And this is my protractor conf:
let os = require('os');
let path = require('path');

let Jasmine2HtmlReporter = require('protractor-jasmine2-html-reporter');

exports.config = {
    seleniumAddress: 'http://localhost:4444/wd/hub',
    multiCapabilities: [{
        browserName: 'chrome',
        chromeOptions: {
            args: [
                '--disable-blink-features=BlockCredentialedSubresources', // needed for authentication
                '--headless', '--disable-gpu', '--window-size=1920x1040'
            ]
        }
    }],
    suites: {
        client: 'client/*.spec.js',
        widgets: 'widgets/*.spec.js',
        full: '**/*.spec.js'
    },
    params: {
        auth: {
            user: 'user',
            password: 'password'
        }
    },
    onPrepare: function () {
        // load page here to authenticate user
        browser.get(`http://${browser.params.auth.user}:${browser.params.auth.password}@localhost:8080/#/`);

        return browser.getCapabilities().then(capability => {
            let date = (new Date()).toISOString().replace(':', '.').slice(0, 16);
            let browserName = capability.get('browserName');
            jasmine.getEnv().addReporter(new Jasmine2HtmlReporter({
                savePath: path.resolve(os.tmpdir(), 'htmlReports', `${browserName}_${date}`)
            }));
        });
    }
};

Many thanks!

Hummm have you tried to run the examples from the plugin ? You can find the examples here: https://github.com/rochejul/gulp-angular-protractor/tree/master/examples
If it works on your side, could you create a sample project to illustrate your issue ? With similar package.json, protractor configuration and some similar tests ?
Many thanks

So after intensive testing I can tell you, that the problem is the protractor version: if protractor 4.x is used the gulp task terminates normally, with protractor 5.x it hangs.

Interesting
What kind of action can I bring ? To restrict the protractor dependency ?
Have you notice a specific event with protractor 5 ?
I think I have to investiguate to find why on protractor 5, it cannot terminate.
I will rename your issue name to specify that issue
Regards

Hm good question. Protractor is not even a dependency of this plugin. At the moment I am fine with restricting the protractor dep in my own project, but maybe a hint in the Readme with a link to this issue would be a good idea.
What do you mean with specific event?

I've been trying to work out what's causing this, but all I have to add so far is that selenium isn't terminated – I put in a process.exit() like so:
  return gulp
    .src(…)
    .pipe(protractor({ … }))
    .on('end', function () {
      setTimeout(() => process.exit(), 2500);
    });

After the node process ends, the selenium server is still there, listening on its port and blocking further runs.

Hmm, further to that, if I manually kill the selenium process in another tab, the gulp process exits cleanly.
[14:06:58] Finished 'test-e2e' after 6.04 s
# I run `killall -9 java` in another terminal here
[14:07:01] gulp-angular-protractor - Webdriver standalone server will be closed```

Hi @Drarok
I think there is a new step onto the protractor selenium server to hang the end of the execution (I remember a similar issue for older versions).
I hope to find some time on the next week to look on it.
Regards

Thanks, waiting for some good news.  I have been manually killing the process since we decoupled the Protractor and Webdriver from the framework.

Ok, it seems we are faced to an issue around the selenium server: https://github.com/angular/webdriver-manager/issues/199
So protractor added this new issue: https://github.com/angular/webdriver-manager/blob/master/lib/cmds/shutdown.ts#L35 because their associated "webdriver shutdow" command is inefficient now
I try to find a workaround
Regards

Okkkkkkkkkkkkkkkkkkkkkayyyyyyy
So, it seems this is not clear for the Selenium team since 3.X if we really could stop the server or not (arguments for security issues).
Maybe I should use instead something like that (to stop the webdriver-manager instance):
child.kill(); // because it was raised with spawn

I have to check this
Regards

Was fixed in version 1.1.0.
There is an example folder "examples/example-06" to illustrate
Regards


Youtube stream has no sound

Just noticed that when I try ti stream youtube videos, VLC comes up with no sound. How to fix?
UPD: it doesn't even stream, it just shows a static image, a snap captured from the stream.

@rip222 Livestreamer has been abandoned. Please try our fork of Livestreamer at http://github.com/streamlink/streamlink


Auto-Tag failure flag

After attempting to Auto-Tag a file and failing, set a flag for the file to notify the user can go back and use the Auto-Identify feature on it.

I forgot there's a "Remove archives after successful tagging" option.


SNAPSHOT dependency does not get resolved while compiling a dependent project after publishing depended-on jar to remote repository when both operations are on the same box.

Build chain
I am using scala 2.10.3 with sbt 0.13
The scenario
I have a module M that my service S depends on.
S and M are not under the same umbrella.  \
The M is part of a multi module project, while S is a standalone project.
S has the snapshot version of the M added as a dependency.

I made some changes to the M
I published the changes to my remote repo
I started a clean build of S on the same
Sbt is unable to pickup the latest snapshot of M. In fact, the build fails.

The error
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  ::          UNRESOLVED DEPENDENCIES         ::
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  :: com.something#someart_2.10;0.1-SNAPSHOT: not found
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::

Observation
The ivydata xml file for M contains the following line after performing the publish operation.
resolver=sbt-chain
Removing this line allows the dependency to be picked up

I am closing this due to inactivity, but if this is still relevant let us know with more details (reproduction steps etc).


More appropriate icon for manage access

I think it be helpful from a UI perspective to have a more relevant icon for "sharing the resource"

I have done the modification and this is what it looks like:

@dtarb @mauriel133 what do you think? Should I make a PR?

Much better icon. Good work, Jeff.
On Thu, Dec 8, 2016 at 10:22 AM Jeff Sadler<EMAIL_ADDRESS>wrote:

I have done the modification and this is what it looks like:
[image: image]
https://cloud.githubusercontent.com/assets/6943441/21020066/e4ad7990-bd40-11e6-89ac-c422420e36b7.png
@dtarb https://github.com/dtarb @mauriel133
https://github.com/mauriel133 what do you think? Should I make a PR?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hydroshare/hydroshare/issues/1699#issuecomment-265799277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADT89MiskpDesLSIX4egv4IIw-kOIWaXks5rGDzBgaJpZM4LIEdl
.

I am OK with both the old and the new one.  If folks want to change I have no objection.

I like this icon as well.

@jsadler2 Are you planning to implement this change?

This has been pulled into develop


Multiremote with wdio-sauce-service

Hi,
I'm using wdio-sauce-service with remote and it works great with single instance tests, whoever with multiremote shows jobs in saucelabs where we have two browers running in parallel. When the job completes the status in saucelabs is unknown  when Passed or Failed would have been expected.  Does this work with multiremote, any example config (if relevant?)
Thanks :+1:
Environment
wdio-sauce-service 0.4.7
wdio 4.8.0
sauce connect is used via HA setup so no sauceConnect used via the plugin

@adrian-brown we don't have support for multiremote. Happy to see this being implemented though.

Ah cool thanks for letting me know.

would that also resolve the experience in saucelabs where many multiremote tests use the same running sauce VM at the moment causing a timeout eventually (I set 30 mins). Locally I get new instances of 2 chrome instances per multiremote test.  I run the tests from wdio runner so don’t see how I can force new browser sessions for multiremote tests. Or is there a way?

Any chance this could be taken up, or is this a very isolated problem not shared by the masses?  Using sauce in this way would be a big win vs running on an isolated container

@adrian-brown feel free to provide a PR


Edit "Hidden" string in English

change "Startup position: Hidden" to "Startup Position: System Tray"

finally figured out the power of branching


smplayer-with-smtube: Add version <IP_ADDRESS>-smtube21.10.0

supersedes https://github.com/ScoopInstaller/Extras/pull/8770

closes https://github.com/ScoopInstaller/Extras/pull/8734

See comment on the SMTube PR for details.

The package is modified based on extras/smplayer.

Settings persist fine but $dir\mpv\yt-dlp.exe is not persisted, meaning it has to be redownloaded after an update

My concern is that if yt-dlp gets outdated, SMTube may not work properly.
I tried to put an old version (ver 2022.01.21) of yt-dlp into the folder, but SMTube does not prompt user to update it.
By the way, yt-dlp is updated about once per month (release page)

Okay that seems fair, if they add an update check in the future then we can maybe persist it.


[Lorem Ipsum] Add options to Paste to active app and then copy to clipboard

Description
Adds feature to paste to active app and then copy to clipboard.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

closes https://github.com/raycast/extensions/issues/13945

Hello @pernielsentikaer , can you please help review/merge this?


Incomplete cleanup / Cost trap

The workshop and CDK itself lack information about how to clean up correctly and which cost may occur.
As stated in #126 and ignored for years in https://github.com/aws/aws-cdk/issues/986, the user is left with a DynamoDB table, S3 bucket, CloudWatch logs and CloudFormation stack after finishing the workshop, without knowing about their existence.
This is a dark pattern (some may call it worse) as the user acts in good faith and in compliance with the cleanup section, yet will be left in an unclean state for redeployment and could be charged for resources he thought were destroyed.
I suggest the following steps:

add a clear warning about potentially occurring costs (upfront, users may not complete the workshop)
add description how/where to find an overview about all things which will/might create charges (e.g. in AWS Console)
extend the code examples about information on the RemovalPolicy options
implement and include the cdk bootstrap --destroy command

Hey @DC4JG :wave: Thanks for bringing this up.
I agree that this could be confusing for new users and should be improved.

add a clear warning about potentially occurring costs (upfront, users may not complete the workshop)
add description how/where to find an overview about all things which will/might create charges (e.g. in AWS Console)

These would be a great first issue for anyone looking to contribute.

extend the code examples about information on the RemovalPolicy options
implement and include the cdk bootstrap --destroy command

These action items I believe could be their own issues/PRs and could be implemented after the others.
I'm marking this as p1 which means it has been prioritized as important, although please keep in mind that we do have a large number of issues at the moment. It may be some time before we are able to solve this particular issue. We use +1s to help us prioritize our work, and as always we are happy to take contributions if anyone is interested to pick this up and submit a PR.


Swallows characters when replacing multiple occurences on a single line

Minimal example
# test.ts
# Usage: node --loader ts-node/esm src/steps/test.ts
import { Readable } from "node:stream";
import { pipeline } from "node:stream/promises";
import replace from "stream-replace-string";

async function main() {
  await pipeline(
    Readable.from(
      (function* () {
        yield "One two one\n";
        yield "One two one\n";
        yield "One two one two\n";
        yield "One two one two\n";
      })()
    ),
    replace("two", "three"),
    process.stdout
  );
}
main();

Expected output
One three one
One three one
One three one three
One three one three

Actual output
One three one
One three one
One three one threeOne three one three

I (or someone else) should probably add tests. I think that the bug might not be dependent on the line, but the chunk. In your example, each line is in its own chunk. I'm not sure, so adding tests would help. @danmichaelo can you add tests? If not then I can add tests.

Actually I will add tests but I might prefer if someone else fixes the bug.


[Don't Merge] Make TaskBuilder fill DiscoveryInfo with IpAddress' discovery information

Make TaskBuilder fill DiscoveryInfo with IpAddress' discovery information
Note: this depends on mv/discovery_info

Deprecated by #2745


UX: HMW help people understand markers better

As a user I want to get hints on existing markers what they mean and how they are used so that I can use them correctly.
Description
When people see markers in the text they might not know what they mean, e.g. \ms.
Design a way to give hints e.g. on hovering over the marker or showing the backslash menu. Also design a way to link or show the usfm description from there.
observed in TC learning session

What is a "TC learning session"?
I would note that while this is a good idea should people be in the space of dealing with markers, the general plan in P10S is to allow users to get the results they want without having to learn anything about USFM or its markers.

tc = translation consultant
Yes, I agree that we want to have people use it without markers as much as possible. I assume there will be situations where they still have to check the markers (e.g. when the visual difference of representation is not clear enough) and others that are used to it, might want to look at markers - still they do not know all of them as we observed.


#364 beautify openapi

@Vany I do not have a clear reason why the coveralls lowered the test output. This issue though has been resolved, I will therefore close this pull request. Sorry I wasn't able to help you on this one. Can you let me know what interests you and I can help to create the right issues for you. Thanks Rob


msolve: 0.6.1 -> 0.6.3

Description of changes
https://github.com/algebraic-solving/msolve/releases/tag/v0.6.3
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Dear lord, how long does the test/diff/diff_reals_dim0-swapvar.sh test take?
Builds & runs successfully on macos 12 x86_64 & nixos x86_64 with doCheck = false - don't have the time to spend on the tests.

Looks like https://github.com/algebraic-solving/msolve/issues/110. Should be fixed in the new release.


This, I think, correctly flattens the last dim during load, but needs more testing and error control

Closes #8

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 91.91%. Comparing base (f8389e6) to head (7f95517).

Files
Patch %
Lines

src/nested_ragged_tensors/ragged_numpy.py
95.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   91.61%   91.91%   +0.30%
==========================================
  Files           1        1
  Lines         322      334      +12
==========================================
+ Hits          295      307      +12
  Misses         27       27

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

A better version of this is in the flatten function in #32


DEV: do.py, adoption of pkg pydevtool (removed non SciPy specific code).

Part of gh-15489:  unify all developer CLI tools
Some clean up of non-scipy specific code include in the first version...
Also bumped doit dependency version.
Note you must update installed packages:
conda env update --file environment_meson.yml

LGTM.
One small tweak, updating the REQUIREMENTS section under do.py with the updated package info would be helpful.
Thanks, @schettino72

updating the REQUIREMENTS section under do.py

I removed the version from there and directed users to enviroment_meson.yml

In it goes - thanks again @schettino72!

This works nicely in practice, I think it will help educate users about what is going on under the hood:
💻  ninja -C /home/rgommers/code/scipy/build
...
💻  meson install -C build
...


Lesson "A3 : Reflected Cross-Site Scripting" does not work on latest Chrome versions

FYI the latest versions of Google Chrome (tested on version 48.0.2564.97) do not allow scripts in the query string. Here's the error shown in the console:
The XSS Auditor refused to execute a script in 'http://<IP_ADDRESS>:8899/lucky.php?name=%3Cscript%3Ealert%28%22toto%22%29%3C%2Fscript%3E' because its source code was found within the request. The auditor was enabled as the server sent neither an 'X-XSS-Protection' nor 'Content-Security-Policy' header.

This is a good thing, but might be worth putting a warning on the lesson :-)

This project seems abandoned, anyway...


[Bug] not_null constraint only has effect when specified at the column level

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When applying constraints to a contracted model, the not_null constraint appears to not be evaluated, UNLESS it's applied at the column level.
Reproduction case:
-- in models/test_constraints_application.sql
{{
  config(
    materialized = 'table'
    )
}}

select 1 as id, 'foo' as bar union all
select 1 as id, 'foo' as bar union all
select 1 as id, null as bar

# schema.yml
models:
  - name: test_constraints_application
    config:
      contract:
        enforced: true
    constraints:
      - type: unique
        columns: [id]
      - type: not_null # has no effect
        columns: [bar]
    columns:
      - name: id
        data_type: int
      - name: bar
        data_type: varchar(3)
        constraints:
          - type: not_null # works as expected

Expected Behavior
I expected the constraint to apply in both of the cases commented in the .yml above. I tried to reproduce this using Databricks, but was unable to do so (i.e. it works as expected in both cases). So it appears this problem may be specific to dbt-snowflake.
Steps To Reproduce

Use version 1.8 of dbt
Copy the examples from the Current Behavior section to your project
Comment / uncomment the model - level and column - level contraints
dbt run -s test_contraints_application

Relevant log output
No response
Environment
- OS: dbt Cloud
- Python: dbt Cloud
- dbt-core: dbt Cloud snowflake adapter keep on latest version
- dbt-snowflake: dbt Cloud snowflake adapter keep on latest version

Additional Context
No response

@matt-winkler I believe this was an intentional choice, because data warehouses only support not null as a column-level constraint. I'm not seeing that said explicitly in #6750 (when we were deciding on the spec), but I'm reading it between the lines :)
From Postgres documentation:

A not-null constraint is always written as a column constraint.

From Snowflake's documentation:

It can be specified only as an inline constraint within the column definition.

On Databricks (docs), adding/removing not null constraints happens via alter table ... alter column, rather than alter table ... add|drop constraint (for check constraints).


Switch Expert axis to avoid unnecessary copy for  layout change

Before: https://screenshot.googleplex.com/5LhbKL58gBwAGUr
After: https://screenshot.googleplex.com/BFziQHuGGDuAjhN
0.5% to 0.8% improvement after borrowing sharding from paxml.

LGTM! Please wait for @RissyRan approval as well

Need code owner's approval. Thank you @gobbleturk.


Fixup for last PR: Actually remove the version

I think this is better since there should be no need to specify a version at all.
Ref: https://issues.redhat.com/browse/EC-86

I clicked merge just a little too fast on #1172.

I'll merge at 4:59:59 :grin:

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

Project coverage is 100.00%. Comparing base (f57a22e) to head (47d3527).

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #1173   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          117       117
  Lines         6141      6141
=========================================
  Hits          6141      6141


onLoadStart event is returning page url instead of page title

Bug description:
As per documentation, onLoadStart returns a SyntheticEvent wrapping a nativeEvent with these properties:
canGoBack
canGoForward
loading
target
title
url
But the title returned is the page url (which is already returned in the url field), not the page title.

Expected behavior:
the method should return the webpage title
Screenshots/Videos:

Environment:

OS: Windows
OS version: 11
react-native version: 0.69.6
react-native-webview version: 11.23.0

I think the documentation just needs to be updated. It makes sense that the WebView wouldn't know the page title before loading the page... though I'm somewhat curious if there's any difference between platforms. You only tested on Windows?

I tested it while using an Android phone and Expo on Windows.
Also onLoadProgress has a similar behaviour: when it fires the title value is empty, then it gets filled with the page url and finally with the proper page title.
I think it would be better if the title value stays empty until available, to fill it with the url value and then change it seem like a behaviour prone to generate issues.

I think it would be better if the title value stays empty until available

This is why I said I was curious if there is a difference between platforms. react-native-webview does not have logic to return the URL instead of the page title, so if that's the behavior you are seeing then that's how the native WebView on the platform you're testing with behaves. It's possible that some platforms do return empty strings... I'm not sure.


Performance

Is it ok if performance is almost twice as bad? I've fine-tuned my shaders to perform at about 75fps, then I've tried to use this addon, found the RT and shader before the UI, enabled effects, everything is fine, but the performance is at ~45fps.

Is it ok if performance is almost twice as bad? I've fine-tuned my shaders to perform at about 75fps, then I've tried to use this addon, found the RT and shader before the UI, enabled effects, everything is fine, but the performance is at ~45fps.

do not use regrade ultimate. this was the solution for me

Ok, I'll try that, thank you.

Ok, I'll try that, thank you.

you may still lose 5% fps but it is manageable

is actually happening the same as soon as i enable rt....
using it without rest is okay ,but as soon as im using i get 30 fps from 60


feat: on nix, add chalklike color-coded messages

does what it says in the title: adds color to the nix log messages similar to our chalk messages in js scripts

@ckerr would you mind adding some screenshots?

@codebytere sure. This is from the automate-depot-tools-and-yarn branch, but the colors are the same. Suggestions / improvements welcomed.


Fix query plan css

This PR fixes #12176

Pull Request Test Coverage Report for Build 245217881

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

Totals

Change from base Build 245133184:
0.0%

Covered Lines:
19010

Relevant Lines:
44457

💛  - Coveralls


How to use the Vicuna model?

The Vicuna model will generate some unrelated output, so how to control the max_lrngth in :
model.generate(inputs.input_ids.cuda(),   max_length=??)

Hi @sev777 , as long as you use in-context learning with a few demonstrations in the prompt, the output should follow the demonstrations' format in most cases. See the prompts we used in our experiments.
For max_length, you may just use something long enough (e.g., max_length=300 for CoT cases) and then extract the actual prediction from the output.

Hi @sev777 , as long as you use in-context learning with a few demonstrations in the prompt, the output should follow the demonstrations' format in most cases. See the prompts we used in our experiments. For max_length, you may just use something long enough (e.g., max_length=300 for CoT cases) and then extract the actual prediction from the output.

Thank you for the reply.
But when I use the model from： https://huggingface.co/lmsys/vicuna-7b-v1.3，the generate code is as follow：
     inputs = tokenizer(cur_prompt, return_tensors="pt")     generate_ids = model.generate(inputs.input_ids.cuda(),                                   max_length=inputs.input_ids.shape[1]+100)     res=tokenizer.batch_decode(generate_ids[:,inputs.input_ids.shape[1]:], skip_special_tokens=True,  clean_up_tokenization_spaces=False)[0]
The cur_prompt is same as the task_prompt in run_mello.ipynb.
But the res is : ##รভ secretaryகष [unused327] answer :รভ secretaryகष  ...
Should I use Vicuna-chat?
Thanks again！


Discussion: Editor and Documentation Accessibility

Howdy,
This issue is designed to open up a discussion about the accessibility in GraphiQL.  Currently at Vrbo we have a team member who is blind and since most tooling is not accessible first, it makes it difficult for them to both learn and utilize GraphQL to its fullest.
I wrote the title as Editor and Documentation because I think each could have their own challenges as we look at what is needed.
Editor:
I did a quick test of a few screen readers.  For the most part all of them get stuck in the editor with no way out.  This could also be due to my lack of knowledge around screen readers so warrants more investigation.
Documentation:
It was hard to test this too much because it tended to toss me back into the editor.  I think GraphQL lends itself nicely to make documentation a wonderful experience.  Having good descriptors on moving from parent to child I think will make navigating through a breeze.
Im a novice when it comes to accessibility but hope to help out and my team member is up for testing out any features.

thanks @cmoody for reporting! I'll be sure to check it out in greater depth this weekend.

It looks like Codemirror 6 would introduce some improvements for the editor:
https://github.com/codemirror/CodeMirror/issues/4604
However, its not certain when that will come out, and we will be switching to Monaco in a few months anyways

@cmoody, accessibility is important for GraphiQL to be inclusive. I appreciate that you file this issue. Are you / your coworker aware of how to approach this problem? I recall that there are some linters to help a11y compatibility.
Anything that would help us have a more structured approach would be appreciated!

in my experience, the linters can only do so much. for example in react, we can lint the static prop names of components but not values before theyd been cast. So a linter like a11y is great to catch things statically, but then you need to do runtime checks in CI, there are tools we can use with or alongside e2e suites, and thatll help us keep track of many more issues. So I would say first step is an end to end suite, and adding the linter, and then adding a more comprehensive check against, say, our netlify@next that maybe even gates against accessibility debt introduced in a PR. I architected something like this for the new medicaid/medicare patient enrollment forms when i was consulting for CMS in baltimore. 508 had just synced up with WCAG2 at that point, so im quite familiar with using that as a standard.

Here's a really cool example, wish we had this then!
https://github.com/cypress-io/cypress-example-recipes/tree/master/examples/blogs__a11y
Again, this is just the first step, of catching and documenting these issues, the greatest effort will be in resolving them.

@cmoody hey there! so we have released a version that should have some substantial accessibility improvements, big thanks to @kasperisager for making such a huge first-time contribution for this! If your colleague has a chance to review that would be splendid. In the time being, I'll set up the automated suite above to prevent accessibility debt in the future.

I won't close this ticket until we can ensure that they and other folks who use non-sight based browsers are able to use all the GraphiQL features.  Unfortunately the remaining issues from my cursory audits, after the spike, seem to be built into codemirror 5


Implement User Authentication

This resolves #38 because that was caused by an issue in a library that is no longer being used.


Show Preview with STC as Custom Widget raises TypeError

Run wxGlade
Add a Frame
Add a Custom widget with instance class set to stc.StyledTextCtrl
Select the widget's Code tab
In the Extra (import) section, enter: import wx.stc as stc
Save the design as a .wxg file
Select the Frame in the tree control
Click "Show Preview"

An error similar to this is displayed:
ERROR   : None
An unexpected error occurred!

Exception type:      <class 'TypeError'>
Exception details:   DC.DrawRectangle(): arguments did not match any overloaded call:
  overload 1: argument 1 has unexpected type 'float'
  overload 2: argument 1 has unexpected type 'float'
  overload 3: argument 1 has unexpected type 'float'

Application stack traceback:
  File "/home/richardt/Sources/applications/GUI Builders/wxGlade/issues/_wxglade_509896714.py", line 39, in self_window_1_on_paint

Date and time:       2022-09-26T09:07:54.437643
Python version:      3.10.6
wxPython version:    4.2.0
wxWidgets platform:  __WXGTK__
wxGlade version:     1.0.4

If you generate and run the code for the design, it runs without errors.

Thanks for reporting. Things should work now again also with Python 3.10.

revision 1.0.5 about to be released


polymc: Use portable configuration

Reverts #674.
By principle, Scoop should use portable versions if possible. Otherwise, they should be in the Nonportable bucket. This has already been applied to the other official buckets (https://github.com/ScoopInstaller/Extras/pull/7573), the games bucket should follow too.

[x] I have read the Contributing Guide.

A migration script would keep current users happy (https://github.com/Calinou/scoop-games/pull/674#issuecomment-1207192740).

@Calinou All good. Run the migration script to migrate your config to Scoop:

Ping @dada513 @DioEgizio @Pato05 @txtsd

polymc-qt5 should be adapted to this too

polymc-qt5 should be adapted to this too

@DioEgizio I'm assuming the persist files are the same in the legacy version, right? They seem to be missing in the manifest.

polymc-qt5 should be adapted to this too

@DioEgizio I'm assuming the persist files are the same in the legacy version, right? They seem to be missing in that manifest.

Right now polymc-qt5 uses the not portable build, but the files stay in %APPDATA%/PolyMC too, so the migration script should be the same

Right now polymc-qt5 uses the not portable build, but the files stay in %APPDATA%/PolyMC too, so the migration script should be the same

Are you sure? It has been using the portable version from the start, so I don't think a migration script is needed for that. I'm just concerned about the lack of persisted files in that manifest.

After some thought, I would prefer using the non-portable config version because if PolyMC is installed globally, you would need admin permissions to write to files every time.

@sitiom maybe keeping it non-portable for global installs?
Shouldn't be hard since I saw that PolyMC enables portable mode when there's a "portable.txt" file in its root directory.
It would be better keeping all data in a predictable directory to make uninstallation easier, instead of everything scrambled in the file system.

Yes, the actual difference between portable and not portable configurations is managed by the portable.txt file; if it's there is portable, if it's not there it's not portable

@sitiom maybe keeping it non-portable for global installs?

It would be better keeping all data in a predictable directory to make uninstallation easier, instead of everything scrambled in the file system.

If you make the global installation non-portable and the local portable, that would be scrambled, wouldn't it?

@sitiom maybe keeping it non-portable for global installs?

It would be better keeping all data in a predictable directory to make uninstallation easier, instead of everything scrambled in the file system.

If you make the global installation non-portable and the local portable, that would be scrambled, wouldn't it?

yes but most users would install locally anyways, and you could add a note for that, to warn the user

this would resolve the problem for local installs

It's not worth the inconsistencies that it will bring between the local and global installations. The current setup works well, config files are located at %appdata%\PolyMC regardless of whether it is local or global.

I think it's better to keep polymc non-portable in this case.


Inparallel to fake hal to hal e2e

Will take a look at this after the dependent PR is landed (this just seems to add an e2e test).


[Release Radar Request] Dependabot Changelog Helper v2.0.0

Open Source Project name
Dependabot Changelog Helper
What is your project?
A GitHub Action to help you easily manage dependabot package updates by adding them to your changelog!
Built around the Keep-a-Changelog format, this action will look for an entry line for an updated package and either

Add it if not found (including adding the ### Dependencies and ## [] sections!)
Update it if the package has been upgraded after an initial entry was written

Version
2.0.0
Date
November 26
Description of breaking changes
newVersionLineNumber is removed. Instead, if a version is not found, this action now looks for an unreleased version. If neither version can be found, the action will error. Read more in the README.md
GitHub Repo
https://github.com/dangoslen/dependabot-changelog-helper
Website
N/A
Link to changelog
https://github.com/dangoslen/dependabot-changelog-helper/blob/main/CHANGELOG.md
Social media
N/A
Anything else to add?
N/A

Congrats on being featured in the November 2022 Release Radar 🥳

Read the blog here
Check out our GitHub Tweet