Compilation Issues

Thank you for the work.
The repository doesn't compile,
Initially I got an error saying IKDTree.h not found.
I disable tests in CmakeLists and it was gone .
But after that I have a strange issue
/usr/bin/ld: CMakeFiles/featsense.dir/src/featsense/mapping.cpp.o: in function apply_registration(boost::shared_ptr<pcl::PointCloud<pcl::PointXYZI> >&, boost::shared_ptr<pcl::PointCloud<pcl::PointXYZI> >&, float)': mapping.cpp:(.text._Z18apply_registrationRN5boost10shared_ptrIN3pcl10PointCloudINS1_9PointXYZIEEEEES6_f[_Z18apply_registrationRN5boost10shared_ptrIN3pcl10PointCloudINS1_9PointXYZIEEEEES6_f]+0xcc): undefined reference to fast_gicp::FastVGICPCuda<pcl::PointXYZI, pcl::PointXYZI>::setInputTarget(boost::shared_ptr<pcl::PointCloudpcl::PointXYZI const> const&)'
Could you please help, I see that fastgicp is included in catkin_libraries in the CMakeLists but still dosen't work

Probably a good idea to disable building tests by default.
Please tell me more about your build environment.

Did you compile fast_gicp with CUDA support?
OS?
ROS Version?

Thank you for the quick reply.
I am using ros noetic on ubuntu 20.04.
I have cuda 11.08. as and gcc 9.4.0.

which version of fast_gicp did you use?

I used the master branch

Ok, I made the mistake of not adding the correct fast_gicp version as a submodule as well. Could you try to build the repo with a different version of fast_gicp?
For example, commit aff82ee0778178b8942e2d4d31855c0d8421ac3b is from July 22. This work was completed in Nov/22. Please try an older commit like this one:
cd fast_gicp
git checkout aff82ee0778178b8942e2d4d31855c0d8421ac3b
<compile with CUDA enabled>

Please start from a clean workspace to minimize the change other issues arising.

For me it was working with fast_gicp commit "ee1fdbae7dd86284b52a06141358c89982d67850" from Feb 2023:
cd fast_gicp
git checkout ee1fdbae7dd86284b52a06141358c89982d67850
<compile with CUDA enabled>

@sumanthrao1997 : It would be interesting to know from which version on it starts to fail compiling.
Greetings
Alex

Unfortunately I was unable to get it running with your suggestion, I have the same error as below.
could you please let me know the PCL version you are using.
/usr/bin/ld: warning: libpcl_filters.so.1.13, needed by /usr/local/lib/libpcl_segmentation.so, may conflict with libpcl_filters.so.1.10
/usr/bin/ld: CMakeFiles/featsense.dir/src/featsense/mapping.cpp.o: in function apply_registration(boost::shared_ptr<pcl::PointCloud<pcl::PointXYZI> >&, boost::shared_ptr<pcl::PointCloud<pcl::PointXYZI> >&, float)': mapping.cpp:(.text._Z18apply_registrationRN5boost10shared_ptrIN3pcl10PointCloudINS1_9PointXYZIEEEEES6_f[_Z18apply_registrationRN5boost10shared_ptrIN3pcl10PointCloudINS1_9PointXYZIEEEEES6_f]+0xcc): undefined reference to fast_gicp::FastVGICPCuda<pcl::PointXYZI, pcl::PointXYZI>::setInputTarget(boost::shared_ptr<pcl::PointCloudpcl::PointXYZI const> const&)'
/usr/bin/ld: mapping.cpp:(.text._Z18apply_registrationRN5boost10shared_ptrIN3pcl10PointCloudINS1_9PointXYZIEEEEES6_f[_Z18apply_registrationRN5boost10shared_ptrIN3pcl10PointCloudINS1_9PointXYZIEEEEES6_f]+0x107): undefined reference to `fast_gicp::FastVGICPCuda<pcl::PointXYZI, pcl::PointXYZI>::setInputSource(boost::shared_ptr<pcl::PointCloudpcl::PointXYZI const> const&)'
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/featsense.dir/build.make:666: /home/s/fnt_ws/devel/.private/warpsense/lib/warpsense/featsense] Error 1
make[1]: *** [CMakeFiles/Makefile2:2985: CMakeFiles/featsense.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

Just now, I redid the steps we explained and at least for my system it is compiling. With the fast_gicp commit I wrote before.
Now we have to figure out what's different to your system. At first, your compiler says your two PCL versions "may" conflict. I have pcl from standard sources installed at /lib/x86_64-linux-gnu/. You have a second PCL installed at /usr/local/lib/. In general, it should not be a problem to have two versions of a software installed, but try to remove the "/usr/local/lib" one. I assume you installed it by source using cmake somewhere. If you are lucky you can just call make uninstall in the build folder of your PCL source directory.
But my feeling says the main reason for failure lies in fast_gicp.  Have you tried removing the "build" "devel" and "log" folder before recompiling?
My system:

Ubuntu 20.04.6 LTS
NVIDIA GeForce RTX 2070 SUPER + Driver 470.223.02 + CUDA 11.4
AMD Ryzen 7 3700x
warpsense commit "2bcd5fe14182378e03092e973e64d2c7e73c4e10"

commented out "add_subdirectory(test)" in CMakeLists.txt

fast_gicp commit "ee1fdbae7dd86284b52a06141358c89982d67850"
PCL: Version 1.10.0 installed via apt (maybe by installing ROS?) to folder /lib/x86_64-linux-gnu/

Thank you @aock for the detailed reply. That was the issue with two pcl libraries,  I uninstalled the locally installed one and it works. Thank you for the amazing support

also @aock and @juliangaal  there is no need to git check out different commits. it works with fast_gicp latest commit and warpsense latest commit as well.

Sounds great. We should update the docs to mention the PCL conflicts.


Genkai Tokki: Seven Pirates [PCSG00882]

App summary

App name: 限界凸旗 セブンパイレーツ
App serial: PCSG00882
App version: 1.00

Vita3K summary

Version: v0.1.8
Build number: 3242
Commit hash: https://github.com/vita3k/vita3k/commit/32b607cb
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with no issues
Screenshots:

Log
vita3k.log
Recommended labels

Playable

suggesting ingame+ tag
version 0.2.0 3571
combat mode is completely glitched, just barely after the OPs first 2 screenshots

Can anyone tell me how to fix the part on vita3k android with the characters missing? You can see who is talking or see them move around at all.


Plugin should have an associated Citation object

This would be in addition to (or replacing) citation_text, and could contain things like DOI, BibTex formatted citation, and action-specific citations.

I really like the idea of adding a way for the citations. I'm unsure which are your goals with this, but in Qiita we have a similar approach, and in order to avoid multiple problems such that duplication of effort or sync problems, we decided to just store the DOI, as with that information the user can download the bibtex formatted citation, given that the official pages already have mechanisms to provide this information, and other information. Just wanted to throw this in there, since it may simplify the design of this part.

Please note currently reported citation and help URLs are not working, since all plugins are under qiime2 and not qiime2-plugins on Github
Usage: qiime feature-table [OPTIONS] COMMAND [ARGS]...
Plugin website: https://github.com/qiime2-plugins/q2-feature-table
Getting user support: No user support information available. See plugin website:
https://github.com/qiime2-plugins/q2-feature-table
Citing this plugin: No citation available. Cite plugin website:
https://github.com/qiime2-plugins/q2-feature-table

@mdeleeuw this has been fixed upstream and the fix will be available in the next release. Note that the incorrect URL only applies to the q2-feature-table plugin.


Add redis policy and policy assignment (and required bump to v3.98 of the terraform provider)

Description of your changes
This pull request adds the azurerm_redis_cache_access_policy and azurerm_redis_cache_access_policy_assignment. In order to get those objects created, a bump to 3.98 of the provider was also required. Due to https://github.com/hashicorp/terraform-provider-azurerm/pull/24798 I had to recreate apis/storage/v1beta1/zz_generated.resolvers.go as the share_name and storage_account_name fields were removed in favor of storage_shared_id. No other changes have been done for the rest of the 3.98 upgraded objects .
As with other pull-requests, some files got the “ to `` change before any change was made, this is the first commit in this pull-request.

Fixes #700
I have:

[X] Read and followed Crossplane's contribution process.
[X] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
I still can't get make run to work locally, so no testing have been done.

Hi @knutejoh, many thanks for your efforts in this PR 🙏
As far as I understand from your explanation, we need to bump the native tf provider to add these resources. Native provider bump PRs need to be implemented and tested on their own in order to deal with breaking API changes.
Roughly for a tf provider upgrade with the new architecture:

Version update in Makefile
Preparation of the Underlying provider fork
Then we can update go.mod and open PR
Test all resources that have breaking API changes

Unfortunately, community members are not currently authorized to prepare the Underlying provider fork.
Therefore, it would be better to close this PR and open a native provider bump request/issue if it does not exist. Once this is implemented we would welcome your contribution/PR adding these two resources.

I agree, I'll close this.


Net::HTTP Auto Instrumentation generates spans for OTEL Exporter Requests

Description of the bug
It appears as though Net::HTTP Instrumentation v0.19.2 is generating spans when combined with OTLP Exporter v0.20.5.

Share details about your runtime
Operating system details: Linux, Ubuntu 20.04 LTS
RUBY_ENGINE: "ruby"
RUBY_VERSION: "2.6.8"
RUBY_DESCRIPTION: "ruby 2.6.8p205 (2021-07-07 revision 67951) [x86_64-linux]"
Share a simplified reproduction if possible
I will update this issue with a minimal case to reproduce this issue as soon as possible.

I've been having trouble trying to produce a minimal code sample that reproduces this problem.
I'm going to give this another try tomorrow before the SIG meeting.

Per our conversation in the SIG meeting, it may be a result of our application overriding the sampler configuration.
@fbogsany pointed out that the untraced helper is scoped to the "current" span however it is not applied to any additional child spans if the sampler is set to always_on.
https://github.com/open-telemetry/opentelemetry-ruby/blob/911a72979569e6898e764b1e0cee63a01de5bacf/common/lib/opentelemetry/common/utilities.rb#L75

See here for the samplers. Anything other than a ParentBased sampler will ignore the unsampled/non-recording parent span that untraced creates. For a ParentBased sampler, the local_parent_not_sampled sampler will be consulted, which defaults to ALWAYS_OFF: https://github.com/open-telemetry/opentelemetry-ruby/blob/911a72979569e6898e764b1e0cee63a01de5bacf/sdk/lib/opentelemetry/sdk/trace/samplers.rb#L72
The above holds true for anything not wrapped in a ParentBased sampler, including the trace-id-ratio-based sampler.
The more I reason through this, the more I feel our current approach to "un-tracing" is lousy - we should rethink this IMO (much as I dislike the implicit context-based approach).


[1.19-beta] Block decorations don't remeasure if content changes

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
First of all congratulations on the awesome improvements in the latest Atom releases, that's a great leap forward 💯
Though it looks like a bug in the new editor rendering rewrite (#13880) slipped through regarding block decoration rendering.
The height of a block decorations doesn't get recalculated anymore if the content changes.
Steps to Reproduce

Place cursor in the middle of a file
Execute the following code in the console and observe odd rendering:

const item = document.createElement("span")
const editor = atom.workspace.getActiveTextEditor()
const cursorRow = editor.getCursorBufferPosition().row
const marker = editor.markBufferPosition([cursorRow, 0])
editor.decorateMarker(marker, { type: "block", item: item })
setTimeout(() => {
  item.innerText = "[block decoration]\n[block decoration]\n[block decoration]\n[block decoration]"
}, 100)

Resize window to see layout recalculation and correct rendering

Expected behavior (Atom 1.18):

Actual behavior (Atom 1.19 and master):

Reproduces how often: 100%
Versions
macOS
Atom    : 1.19.0-beta0
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0

and
Atom    : 1.20.0-dev-ccdae0fc8
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0

Additional Information
This bug is especially critical for Hydrogen since we render live results from a running kernel inside a block decoration, that can change dimensions over time.

/cc: @nathansobo @as-cii


[NEW ALGORITHM] Stack Permutation

Name:
[NEW ALGORITHM]  Given two arrays, both of unique elements. One represents the input queue and the other represents the output queue. Our task is to check if the given output is possible through stack permutation.

About:
Application of stack data structure.

Sample Input and Output:
Input: arr1[] = [ 1, 2, 3 ] , arr2[] = [ 2, 1, 3 ]
Output: YES
Explanation:
push 1 from input to stack
push 2 from input to stack
pop 2 from stack to output
pop 1 from stack to output
push 3 from input to stack
pop 3 from stack to output
Input: arr1[] = [ 1, 2, 3 ] , arr2[] = [ 3, 1, 2 ]
Output: Not Possible

Labels:
new algorithm, gssoc-ext, hacktoberfest, level1

Assignees:

[x] Contributor in GSSoC-ext
[x] Want to work on it

Additional Information:
Please assign me this, I want to work on this.

Please assign this issue to me.

@ArijitMandal123 assigned

@pankaj-bind I have created the issue first please assign it to me.

@shuvojitss assigned


Bug: No java runtime present, requesting install

Using SDKMAN on Catalina yields a strange result with some old applications such as Talend:
/Users/ed/Development/TOS_DI-20170623_1246-V6.4.1/TOS_DI-macosx-cocoa.app/Contents/MacOS/TOS_DI-macosx-cocoa
No Java runtime present, requesting install.

but the jdk is clearly available:
sdk home java 8.0.202-zulu
/Users/ed/.sdkman/candidates/java/8.0.202-zulu
○ echo $JAVA_HOME
/Users/ed/.sdkman/candidates/java/current

I'm having the same or very similar issue on archlinux with oh-my-zsh.
Multiple java versions are installed, but none of them is available.
I think the problem is JAVA_HOME is incorrectly set at some point?
❯ sdk install java

Downloading: java 11.0.7.hs-adpt

In progress...

############################################################################################################################################################################################################# 100.0%############################################################################################################################################################################################################# 100.0%

Repackaging Java 11.0.7.hs-adpt...

Done repackaging...

Installing: java 11.0.7.hs-adpt
Done installing!

Do you want java 11.0.7.hs-adpt to be set as default? (Y/n): y

Setting java 11.0.7.hs-adpt as default.
❯ which java
java not found
❯ echo $JAVA_HOME
/home/agilob/.sdkman/candidates/java/current
❯ cd $JAVA_HOME/bin
❯ ls
jaotc  jarsigner  javac    javap  jconsole  jdeprscan  jfr    jimage  jjs    jmap  jps         jshell  jstat   keytool  rmic  rmiregistry  unpack200
jar    java       javadoc  jcmd   jdb       jdeps      jhsdb  jinfo   jlink  jmod  jrunscript  jstack  jstatd  pack200  rmid  serialver
❯ cd
❯ java
zsh: command not found: java
❯ sdk use java 11.0.7.hs-adpt

Using java version 11.0.7.hs-adpt in this shell.
❯ java
Usage: java [options] <mainclass> [args...]
...
JAVA FOUND NOW
...

Open another shell
❯ java
zsh: command not found: java
❯ sdk default java 11.0.7.hs-adpt

Default java version set to 11.0.7.hs-adpt
❯ java
zsh: command not found: java
❯ sdk use java 11.0.7.hs-adpt

Using java version 11.0.7.hs-adpt in this shell.
❯ java
Usage: java [options] <mainclass> [args...]
           (to execute a class)
   or  java [options] -jar <jarfile> [args...]
JAVA FOUND again

It looks like sdk default doesnt set correct path leading to /java/@current
New shell
❯ sdk default java 11.0.7.hs-adpt

Default java version set to 11.0.7.hs-adpt
❯ java
zsh: command not found: java
❯ which java
java not found
❯ echo $JAVA_HOME
/home/agilob/.sdkman/candidates/java/current

New shell:
❯ sdk use java 11.0.7.hs-adpt

Using java version 11.0.7.hs-adpt in this shell.
❯ java
Usage: java [options] <mainclass> [args...]

❯ echo  $JAVA_HOME
/home/agilob/.sdkman/candidates/java/11.0.7.hs-adpt

❯ sdk version

SDKMAN 5.8.2+493

I can reproduce it with 14.0.1.hs-adpt 11.0.7.hs-adpt and 8.0.252-open

I'm can reproduce it on another computer with archlinux.

@agilob I've just tried to replicate this but can't. I'm also on arch linux with oh-my-zsh. I then tried running up a fresh ubuntu docker container, once again it all works fine.
Could you please check that you don't have any special options set in your .zshrc file? Also, can you confirm that you are not using a login shell?

Could you please check that you don't have any special options set in your .zshrc file?


Progress additional text at speed section incorrect work

Hi! We have an issue regarding adding additional symbol if width of the text lower that it was before.
For example:
I have next template
const pbTpl = `{{ (cycle . "←" "↖" "↑" "↗" "→" "↘" "↓" "↙" ) }} {{speed . "%s samples/s"}}`

This template is used in the next code:
pbPrefix := fmt.Sprintf(`{{ green "VM worker %d:" }}`, i)
bar = barpool.AddWithTemplate(pbPrefix+pbTpl, 0)

where
// AddWithTemplate adds bar with the given template
// to the global pool
func AddWithTemplate(format string, total int) *pb.ProgressBar {
	tpl := getTemplate(format)
	bar := pb.ProgressBarTemplate(tpl).New(total)
	Add(bar)
	return bar
}

When I start the tool it shows the correct text like
VM worker 0: → 3233 samples/s

but when when the number of characters decreases, then an additional character appears at the end of the line
VM worker 0: → 323 samples/ss

VM worker 0: → 32 samples/sss

Is it possible to fix this behavior?
Related issue you can find here: https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4555

v3.1.3 created


Other similar project

Hello,
I'm a user of another similar tool named delta: https://github.com/dandavison/delta
I just discovered your project, and as I didn't see it in the acknowledged tools, I was wondering if you knew about it (:
Maybe you can take inspiration from delta too if you want some of its featuresin this project!

Thanks, yes I heard about it recently after I was nearly done implementing this. It's definitely very impressive and more feature rich. It looks like they implement a complete pager, which would be a lot more work, but will keep an eye out for features that I can support here easily


Create Admin page

Front-end

Acceptance Criteria:
For User component (Minh):
[ ] Pull up the data from existing users
[ ] Be able to update the specific user information
[ ] Basic UI design


JPEG quality level in Image Resizer

Description of the new feature / enhancement
Please add JPEG quality level for each size preset like on attached screenshot. Having a simple field for manual percentage input instead of a slider would be nice too.  This will expand the capabilities and versatility of the program. For other types of images (tif, gif, png, etc...) there is no particular need for individual quality adjustments. Sorry for my English.

Scenario when this would be used?
For Image resizer...
Supporting information
No response

Wouldn't there need to be quality options for every single option? Why just JPEG? /needinfo

Because quality, which is directly related to volume (in kb), is just as important as size (pixels). There are different tasks that require different parameters of these two options (size and quality).  Why just JPEG? I don't mind having this extra option in other formats, but JPG/jpeg is the most common and compact format for displaying raster graphics on the web. In other words, it is very important for web designers. The other two formats (GIF & PNG) are also important for web design, but this requires more complex quality settings and is hardly advisable to apply this in this program, which is more focused on convenient, batch editing. Sorry for my English. Thanks!

Because quality, which is directly related to volume (in kb), is just as important as size (pixels). There are different tasks that require different parameters of these two options (size and quality).  Why just JPEG? I don't mind having this extra option in other formats, but JPG/jpeg is the most common and compact format for displaying raster graphics on the web. In other words, it is very important for web designers. The other two formats (GIF & PNG) are also important for web design, but this requires more complex quality settings and is hardly advisable to apply this in this program, which is more focused on convenient, batch editing. Sorry for my English. Thanks!
From: Joseph Finney
Sent: Thursday, November 9, 2023 6:45 AM
To: microsoft/PowerToys
Cc: arts-md ; Author
Subject: Re: [microsoft/PowerToys] JPEG quality level in Image Resizer (Issue #29571)
Wouldn't there need to be quality options for every single option? Why just JPEG? /needinfo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

/dup https://github.com/microsoft/PowerToys/issues/7374


Expose CLI API programmatically

I would like to avoid installing the CLI manually and would prefer to use it from a build.rs file. This way I can define it as a build dependency and have it in my Cargo.lock. It would also be one less command to have to install manually.
The current implementation only exposes the Opt struct and run function. Would it be possible to expose more functions?

What functions do you need to run from the CLI?
You can setup and run migrations manually: https://docs.rs/sqlx/0.4.2/sqlx/migrate/struct.Migrator.html
Or embed them to run on startup: https://docs.rs/sqlx/0.4.2/sqlx/macro.migrate.html
Creating and dropping databases can be done in SQL, that's all sqlx db [create, drop] do.

I am interested in the prepare command.
I am using the offline feature. I have a directory with my SQL scripts to handle schema migrations (I do not use the SQLx migration tool).
My goal is that anyone should be able to compile the project, even if they don't have any configured database.
Only those who edit the SQL files should need a database. I would like to detect changes to the SQL scripts and refresh the sqlx-data.json files when a change is found.
Using the prepare function programmatically currently requires me to use env::set_var and env::set_current_dir before calling run from my build.rs.

You'll probably be interested in #941 then as it aims to make cargo sqlx prepare redundant by not requiring a merging step. That should be part of the 0.5 release targeted for Q1 of next year.

Thank you for the link, I will follow this MR then.
I still think that it would be nice if any feature provided by the CLI was also available through an API.

I still think that it would be nice if any feature provided by the CLI was also available through an API.

From what I understand – apart from prepare (which is actually mostly impossible to provide an API for in the library) – everything should be possible with the API of SQLx.
Most of sqlx-cli is minuscule, see the implementation of the database sub-command: https://github.com/launchbadge/sqlx/blob/master/sqlx-cli/src/database.rs

I confirm that everything is possible with the API. The wording in the issue may be overly general when I was mostly interested in prepare. I was able to use it in my code using the current API and it seems that there PRs to make this subcommand less relevant.
You may close this issue if you feel that it is not relevant.


Fix format of errors returned by API (and more)

Add localAgentLauncher to the list of healtchecks ignored by the leader
Fix exception mapper returning plaintext response rather than JSON
Improve error message in Java client in case of bad server response
Delete file added by mistake

Coverage decreased (-0.03%) to 92.533% when pulling b6eefe2bef9781e96df70b7eb0119c4783df616a on mprimi:GENIE-502-api-json-exceptions into 8b24cd1f672d4512d205480843fff2a9ce21721d on Netflix:master.


Better error handling when loading a dead Storylines product

Better error handling of an invalid Storylines product would be useful.
The as-is: loading a bunk Storylines ID just leads to an infinite loading spinner. Libraries download but otherwise nothing happens. The user might be expecting something to load, unaware that they're accessing a bad URL.
Example: https://environmental-maps.canada.ca/RAMP-Storylines/index-ca-en.html#/en/asdf
It would be great if we could simply display a broken link, a message that the Storylines product couldn't be loaded, something to that affect.

Previous efforts on this may have been made and lost?


RPA Manager release notes 2024-06/1

Related PR https://github.com/mulesoft/docs-release-notes/pull/8494/files


[BUG] Snowflake connector not partition aware

Type of Connector
Certified Connector
Name of Connector
Snowflake
Describe the bug
When retrieving more than 1k records of data the connector only returns the data contained within the first partition created by the Snowflake API. The rest of the records in subsequent partitions are not returned and ignored by the connector.
Is this a security bug?
No, this is not a security bug
What is the severity of this bug?
Severity 1 - Connector is broken or there's a very serious issue
To Reproduce
Create an instance of the connector in Power Automate
Query a Snowflake table with more than 1k records
Count records returned to Power Automate and Power Apps
Records will be less than total contained in Snowflake table
Expected behavior
the connector should traverse all partitions that are created by the Snowflake API so that the records returned match the number of records contained in the table.
Environment summary
Power Apps
Power Automate
Snowflake
Additional context
No additional context

Hi, just bumped into this problem as well. You can workaround this by using the "Check the Status and Get Results" action, which allows you to define the partition number you want the results from (indexed from zero).
I ended up creating a do until loop, to iterate over partition count. Unfortunately this is a lot slower than fetching all the results within one statement, but I guess this is what we have to live with for time being 😕 Hopefully Snowflake is able to fix this asap...
And also - I needed to set a default role for myself in Snowflake end as the "Check the Status and Get Results" action does not provide a way to send the role 🤪

@TerhoAntila I am facing the same issue, thanks for your comment. I am doing the same loop but i am facing some issues due to the "append to array" action in power automate doesn't work with nested arrays, how did you solve it?; I am trying one solution flattening the array using select or even parse Jon, but I need to do an interaction for each item of the array, and that is extremely slow

I was able to also resolve the issue using a loop. here's what worked for me

First execute a Snowflake query that returns the record count of your query using Submit SQL Statement Operation
Submit your query to retrieve the records using Submit SQL Statement Operation
Store the results (data) attribute from step 2 in a string variable
Split the string from step 3 into an Array variable by ',]'
Count the number of records in the array from step 4
Create a condition to determine if the array record count = total expected record count you got in step 1
If it is less then kick off a Do Until that takes the statement handle property from step 2 and executes a Check the Status and Get Results Operation to retrieve the next record set from the Snowflake API Queue
This should run until all records are processed, but beware i have seen extremely slow performance in Power Apps with record sets >10k records. Also note, i did not use the Convert results set rows from array to objects operation. i just grab the data from the Execute sql statements and Check status ops and append them to a string variable. i then return the string to Power Apps and process it into a collection using ForAll and Concats. Hope that helps.

Hello @jesusfraire
I am also facing the same issue in loading the records from Snowflake that are more than 1k, but am not expert in power automate so hard to understand the steps you have mentioned so it would be really great full if you can help me by providing the screen shots!

Hello @jesusfraire or @GabrielNova24
I am also facing the same issue in loading the records from Snowflake that are more than 1k, but am not expert in power automate so hard to understand the steps you have mentioned so it would be really great full if you can help me by providing the screen shots for the steps from 3 as I am struggling to resolve this, please!
Thanks in advance!

Please @jesusfraire or @GabrielNova24, could you hell us to understand a little bit more about this loop?I'm particularly stuck on stage 7.

Hello @Edpobj
I am actually stuck at 4th step, I did split the string using the same formulae to append it in array. throwing error like, "The input value is of type 'Array' which cannot be appended to the variable 'Array' of type 'Array'. The action type 'Append Array Variable' only supports values of types 'Float, Integer, String, Boolean, Object'." is there any workaround for this?

Hi all for Step 3 it is simply assigning the returned data attribute from the Submit Snowflake Query task to a string variable. recordsString = @{outputs('Submit SQL Statement for Execution')?['body/data']}
You can then iterate over this object to count the number of records present in the results and determine if it is equal to or less than the total expected records calculated in step 1
snowflakeResultsArray = @{split(variables('recordsString '),'],')}
returnedRecordCount=@{add(length(variables('snowflakeResultsArray')),1)}
If expectedRecordCount > returnedRecordCount
Do until returnedRecordCount=expectedRecordCount
Check Status and Get Results (Task)
Append to String
recordsString = @{outputs('Check_the_Status_and_Get_Results')?['body/data']}
Update Record Count
returnedRecordCount= @{length(split(variables('htmlString'),'],'))}
Update an index variable (use this to access the next Snowflake data partition in the Do While

@jesusfraire Thanks for your response I am getting the data in string format in PowerApps but converting this string into a collection seems to be bit complex.
for example I am getting the data in this format [["1","test","Result"], ["2", 'test2", "result2"]]........I have tried forall and split functions but data is not getting formatted into the collection in expected way. I wonder how did you achieve this. If possible could you please write the powerapps formulae to convert this ?

@jesusfraire Could you include a screenshot of your flow?
I'm having some trouble understanding how you have set up the "Do until" part of the flow, and how you index the result to the variable.


[svelte-table] Table component errors on render in sveltekit's ssr

Describe the bug
svelte-kit, most popular framework to work with svelte, fails to server-side render the components of svelte-table.
Your minimal, reproducible example
https://github.com/AdaptingAFM/svelte-table-on-sveltekit
Steps to reproduce

Clone the repo
Init the project with your package manager
Start the server with the dev command
Load the index page, which should be located at localhost:3000

Expected behavior
Out-of-the-box compatibility with svelte-kit, if not possible, consider adding a script to https://github.com/svelte-add/svelte-add
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
Arch Linux
react-table version
8.0.0-alpha.87
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

@Mokshit06 any ideas here?

@tannerlinsley  This error is due to our current build script compiling the <Placeholder /> component in svelte-table as a client component, but svelte expecting different interface for server. I'll either update the build script to compile it for server as well, or just add the precompiled file to source.

Has this been solved in the new beta?

yep this has been solved in #3980


github: add workflow to run the unit tests

There are already a couple of unit tests implemented in the project but they aren't executed on the current build workflow.
Let's extend the build workflow to all run the unit tests afterwards.

I'm already working on that workflow.


Today the script stopped working

Description
Nothing happens after loading the sg page
Steps to Reproduce

Go to "https://www.steamgifts.com/"
See error on console

Console Errors
http://prntscr.com/pwjm9v
http://prntscr.com/pwjooe
System (please complete the following information):

ESGST Version: [Userscript v8.5.9(Tampermonkey)]
Browser + Version: [Chrome v78.0.3904.97]

Does the console say anything like "Upgrading storage..."?

yep

To what version?

http://prntscr.com/pwjwfx

please tell me how can I update these files?

Or should it itself be updated? (unfortunately I have to leave), I checked for updates and they are not there, the script still does not work

I haven't released a new version, it's just in the development stage. If you want, you can test it from here to see if it works: https://github.com/rafaelgssa/esgst/releases/tag/v8.5.10-dev.1

I try it and have this error on console
http://prntscr.com/pwlv1o
now without "Upgrading storage..."
http://prntscr.com/pwlvf2

Thank you, that helps, will try to fix soon.

No problem, it’s in my interests to help you so that i can continue to use the script in the future)

I am also having this same problem

Do you have information or timelines when you can or fix this problem?

Probably in the next 3-6 hours.

Thanks for the answer, I look forward to, I hope the error disappears

Hi, can you try again with this dev version: https://github.com/rafaelgssa/esgst/releases/tag/v8.5.10-dev.2

Hi there, its worked for me


Add AEROSPACE_EMPTY_WORKSPACES env var to exec-on-workspace-change

Adding AEROSPACE_EMPTY_WORKSPACES to the exec-on-workspace-change event will make it possible to dynamically show/hide or visually indicate which workspace has no windows currently in it in sketchybar or similar softwares.
Example from the AeroSpace Goodness section in the readme:
aerospace.toml
exec-on-workspace-change = ['/bin/bash', '-c',
    '/opt/homebrew/opt/sketchybar/bin/sketchybar --trigger aerospace_workspace_change FOCUSED_WORKSPACE=$AEROSPACE_FOCUSED_WORKSPACE EMPTY_WORKSPACES=$AEROSPACE_EMPTY_WORKSPACES'
]

.config/sketchybar/plugins/aerospace.sh
#!/usr/bin/env bash

# make sure it's executable with:
# chmod +x ~/.config/sketchybar/plugins/aerospace.sh

#!/usr/bin/env bash

# make sure it's executable with:
# chmod +x ~/.config/sketchybar/plugins/aerospace.sh

if echo "$EMPTY_WORKSPACES" | grep -q "$1"; then
	LABEL_COLOR="$RED" # marks empty workspaces in red
else
	LABEL_COLOR="$WHITE" # marks workspaces with windows in white
fi

if [ "$1" = "$FOCUSED_WORKSPACE" ]; then
    sketchybar --set $NAME background.drawing=on label.color="$LABEL_COLOR"
else
    sketchybar --set $NAME background.drawing=off label.color="$LABEL_COLOR"
fi

Currently, if adding EMPTY_WORKSPACES=$(aerospace list-workspaces --monitor focused --empty) to ~/.config/sketchybar/plugins/aerospace.sh it is slow, because it computes it for every workspace on workspace change,  which is not ideal and the performance drop is visible. I could not find other way to do it dynamically so it updates if the workspace becomes empty/popuplated. I've red to readmes many times, looked at other configs in github for a way to do it with no luck.
Checklist

[x] I've searched for duplicates. My search queries were (please don't forget to search in closed issues too):

--empty
EMPTY_WORKSPACES
"show empty workspaces"

[x] I've read the documentation https://nikitabobko.github.io/AeroSpace/guide & https://nikitabobko.github.io/AeroSpace/commands
[x] I've searched in the documentation. My search queries were:

empty
empty_workspaces
show empty

aerospace CLI client version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201
AeroSpace.app server version: 0.14.1-Beta 5861e85b65cb7dc914a0a68027c56e2bc90510fd

Mergning this issue with #351


New Makefile does not work on OSX

The new make file is not even trying to compile the libraries of GRSISort. It is also missing a
PLATFORM = $(shell uname) at the start. I'm not sure why it won't find the libraries. Perhaps we are using functions that don't exist on OSX?

It works better if I remove the -link 2 from the library name list. Is this important to keep? I think I'm still battling with a few things...

It is not, I added a few things to force cleaner code on my system.
On Aug 25, 2015 12:35 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

It works better if I remove the -link 2 from the library name list. Is
this important to keep? I think I'm still battling with a few things...
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134710686
.

I'm stuck here
make: *** No rule to make target /Users/rdunlop/Programs/GRSISort/include/==>', needed by .build/libraries/GROOT/GROOTDict.cxx'.  Stop.
any guesses?

Did you make clean before you pulled?
On Aug 25, 2015 12:38 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

I'm stuck here
make: *** No rule to make target/Users/rdunlop/Programs/GRSISort/include/==>',
needed by .build/libraries/GROOT/GROOTDict.cxx'. Stop.
any guesses?
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134712185
.

yes. And many other libraries "compiled". I can't find the .so files, but I can find .o and .d files.

Hmm. I didn't have a Mac to try this on and Vinzenz's wasn't available
yesterday.
Is it compiling them and just not linking?
On Aug 25, 2015 12:40 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

yes. And many other libraries "compiled". I can't find the .so files, but
I can find .o and .d files.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134713384
.

Maybe, does it link at the end?

Yes. And the .so libs would be in the libraries directory
On Aug 25, 2015 12:44 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

Maybe, does it link at the end?
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134715189
.

I think there might be a sneaky newline somewhere in the makefile.

I also changed LIBRARY_DIRS   := $(shell find libraries/ -type d 2> /dev/null) to LIBRARY_DIRS   := $(shell find libraries/* -type d 2> /dev/null) because it kept finding the libraries directory and tried to compile it...

I think you might need the line
LINKFLAGS_SUFFIX += -dynamiclib -undefined dynamic_lookup -single_module -Wl,-install_name,'@executable_path/../libraries/$$@'
in the makefile (for darwin only), at least that was in our old makefile. I can't really test right now, because I'm using root6 on my mac and that just complicates things even further. I'm switching back to root5 and will try to get it to compile.

so far I found that in the LIBRARY_DIRS line you need to remove the '-links 2 ' part (still works on linux), and in the dict_header_files line you need to change 'head $(1) -n 1 ...' to 'head -n 1 $(1) ...'
It's still not compiling for me, I get a lot of "Undefined symbols for architecture x86_64:" warnings, but maybe it works for you, @r3dunlop

Ok I’ll look into it later. I also did a find on libraries/* so that it didn’t try to compile liblibraries.so

On Aug 25, 2015, at 6:15 PM, Vinzenz Bildstein<EMAIL_ADDRESS>wrote:
so far I found that in the LIBRARY_DIRS line you need to remove the '-links 2 ' part (still works on linux), and in the dict_header_files line you need to change 'head $(1) -n 1 ...' to 'head -n 1 $(1) ...'
It's still not compiling for me, I get a lot of "Undefined symbols for architecture x86_64:" warnings, but maybe it works for you, @r3dunlop https://github.com/r3dunlop
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134757774.

I think I found the solution, I'm just waiting for it to finish compiling on my mac, then test it on the linux machines here and in Guelph.

Right now it compiles for me, but I can only run it from within the libraries directory, because the shared-libraries are missing their paths.

This also breaks the compilation of the users directory. This can be easily fixed by moving the users directory into the libraries directory. The other option would be to leave it where it is and change the makefile.

How does it break the user directory? What makes it different than the util
directory?
On Aug 25, 2015 6:57 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

This also breaks the compilation of the users directory. This can be
easily fixed by moving the users directory into the libraries directory.
The other option would be to leave it where it is and change the makefile.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134781270
.

I was hoping for there to be classes that could be modified and compiled in to the code that inherits from classes which are treated in the main code. These classes are got ignored so would be dependent on the user. It's a bit of a stretch, and can probably be taken out as no one will probably use it like that anyway and changes will likely be wanted by all.

On Aug 25, 2015, at 8:59 PM, pcbend<EMAIL_ADDRESS>wrote:
How does it break the user directory? What makes it different than the util
directory?
On Aug 25, 2015 6:57 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

This also breaks the compilation of the users directory. This can be
easily fixed by moving the users directory into the libraries directory.
The other option would be to leave it where it is and change the makefile.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-134781270
.

—
Reply to this email directly or view it on GitHub.

Once someone can confirm this works on their mac. and someone else can confirm it works on Linux we can close this.

Apart from the warnings (-pthread and "unsupported compiler"), the compiling works fine on my mac, and grsisort runs without a problem (though I don't have any midas file on my laptop to test it on).
The issue right now is that the program quits with a "double free corruption" on linux. I don't understand why that is, the makefile is essentially unchanged when using it under linux (most changes I made are only for Darwin, the others didn't change anything). Maybe someone can try and checkout the old version of the makefile on linux and see that compiling it with that solves the issue? Then I could try and figure out what the difference is.

The -pthread error can be removed with -Qunused-warnings, or something along those lines, during compilation.

On Aug 26, 2015, at 10:59 AM, Vinzenz Bildstein<EMAIL_ADDRESS>wrote:
Apart from the warnings (-pthread and "unsupported compiler"), the compiling works fine on my mac, and grsisort runs without a problem (though I don't have any midas file on my laptop to test it on).
The issue right now is that the program quits with a "double free corruption" on linux. I don't understand why that is, the makefile is essentially unchanged when using it under linux (most changes I made are only for Darwin, the others didn't change anything). Maybe someone can try and checkout the old version of the makefile on linux and see that compiling it with that solves the issue? Then I could try and figure out what the difference is.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135050464.

So there is one other obstacle for people to be aware of. If you had previously compiled a version with CSM, and then did not make clean before updating to a non CSM version, the dictionaries will survive beyond a make clean. This means if you get an error about the CSM dictionary, the cure is to delete the CSM dictionary by hand.

Also, be aware that we might have to go through and change some of the names of libraries being loaded in the code.
Many libraries used to drop the T, ie libGRSIFormat.so is no libTGRSIFormat.so.

This makefile has caused a lot of problems with how we have the code written. I can't clone some of the classes I was using leading to me to think some dictionaries aren't linking properly. It's also very hard to diagnose what the problems are. This is on a linux machine as well. I will likely move back to the old version until some of this gets sorted out because I can't do any work right now.

double free corruption is solved with pull f91f076.
I can not reproduce the cloning problem on grsmid01.  What classes are causing you issues?

TGainmatch can't be cloned on newest version but can be on last version.

On Aug 27, 2015, at 12:15 PM, pcbend<EMAIL_ADDRESS>wrote:
double free corruption is solved with pull f91f076.
I can not reproduce the cloning problem on grsmid01. What classes are causing you issues?
—
Reply to this email directly or view it on GitHub.

grifuser's GRSISort seg faults when going from FragmentTree->AnalysisTree. Pulled again after @pcbend's commit this morning, resorted fragment file, still seg faults. Investigating more...

TGainMatch did not clone because the pointer fhist was left undefined.
Init it to zero fixes the problem.  I have fixed this on the grifuser
account and pushed it to the GRSIUser git thing, but do not have its
password to make a pull request to GRIFFINCollaboration.
I spent some time on this new make file, I believe it will make better code
catching errors like this one.  As far as I can tell, all linking is being
handled correctly.
On Thu, Aug 27, 2015 at 12:42 PM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

TGainmatch can't be cloned on newest version but can be on last version.

On Aug 27, 2015, at 12:15 PM, pcbend<EMAIL_ADDRESS>wrote:
double free corruption is solved with pull f91f076.
I can not reproduce the cloning problem on grsmid01. What classes are
causing you issues?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135488336
.

Hi Jenna, can you give me a runnumber and calfile location?
On Thu, Aug 27, 2015 at 1:05 PM, SmithJK<EMAIL_ADDRESS>wrote:

grifuser's GRSISort seg faults when going from FragmentTree->AnalysisTree.
Pulled again after @pcbend https://github.com/pcbend's commit this
morning, resorted fragment file, still seg faults. Investigating more...
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135494185
.

You can pull request as yourself. Anyone can request a pull of anyone's fork to anyone's fork.

On Aug 27, 2015, at 1:31 PM, pcbend<EMAIL_ADDRESS>wrote:
Hi Jenna, can you give me a runnumber and calfile location?
On Thu, Aug 27, 2015 at 1:05 PM, SmithJK<EMAIL_ADDRESS>wrote:

grifuser's GRSISort seg faults when going from FragmentTree->AnalysisTree.
Pulled again after @pcbend https://github.com/pcbend's commit this
morning, resorted fragment file, still seg faults. Investigating more...
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135494185
.

—
Reply to this email directly or view it on GitHub.

I've been running
grsisort -a fragment02340_001.root 47Ca_walk_corrected.cal 47Caruninfo.info
in /data3/griffin/47K as grifuser@grsmid01
fragment02340_001.root was freshly extracted from MIDAS file today.

Ah, I didn't know that.  As it is now I'm not near a computer.  If you can
get from the grsiuser github account that would be great, if not I'll try
when I am back in front of a computer later this afternoon.
On Aug 27, 2015 1:33 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

You can pull request as yourself. Anyone can request a pull of anyone's
fork to anyone's fork.

On Aug 27, 2015, at 1:31 PM, pcbend<EMAIL_ADDRESS>wrote:
Hi Jenna, can you give me a runnumber and calfile location?
On Thu, Aug 27, 2015 at 1:05 PM, SmithJK<EMAIL_ADDRESS>wrote:

grifuser's GRSISort seg faults when going from
FragmentTree->AnalysisTree.
Pulled again after @pcbend https://github.com/pcbend's commit this
morning, resorted fragment file, still seg faults. Investigating
more...
—
Reply to this email directly or view it on GitHub
<
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135494185>

.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135501360
.

Does it fail with no info file?

On Aug 27, 2015, at 1:39 PM, SmithJK<EMAIL_ADDRESS>wrote:
I've been running
grsisort -a fragment02340_001.root 47Ca_walk_corrected.cal 47Caruninfo.info
in /data3/griffin/47K as grifuser@grsmid01
fragment02340_001.root was freshly extracted from MIDAS file today.
—
Reply to this email directly or view it on GitHub.

Just to clarify, @pcbend, git from GRSIUser instead of GRIFFINCollaboration?
@r3dunlop, will try that next.

Also, @jhenderson88 reported a similar fail when using LeanGRSI with new TIGRESS data.

With GRIFFINCollaboration's LeanGRSI, yes, it seg faults when I don't include the runinfo file. But I did include the run info file when sorting the fragment.

Last week I changed the streamer of run info and did not update the version. This caused me problems at some point. I updated it a few weeks ago. So it should work now I think.

On Aug 27, 2015, at 1:45 PM, SmithJK<EMAIL_ADDRESS>wrote:
Just to clarify, @pcbend https://github.com/pcbend, git from GRSIUser instead of GRIFFINCollaboration?
@r3dunlop https://github.com/r3dunlop, will try that next.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135504402.

The problem is quite obvious though. The printed Run Info looks like garbage.

...just understood @pcbend's comment. Too many conversations at once.

RunInfo as grifuser looks fine.

Did you remove the CSM Dictionary? It might still be there causing problems.

In TGRSIAnalysis/TCSM? No dictionary file:
-rw-r--r-- 1 grifuser users 5.9K Aug 24 12:54 TCSMHit.cxx
-rw-r--r-- 1 grifuser users  27K Aug 24 13:53 TCSM.cxx
-rw-r--r-- 1 grifuser users  333 Aug 24 14:05 LinkDef.h
-rw-r--r-- 1 grifuser users 3.9K Aug 25 16:30 TCSMData.cxx

I'm sorting old data - do I need some kind of PPG thing?

The PPG is supposed to be ignored if it does not exist.

On Aug 27, 2015, at 1:57 PM, SmithJK<EMAIL_ADDRESS>wrote:
I'm sorting old data - do I need some kind of PPG thing?
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135507192.

Looks like it's failing in SortFragmentTreeByTimeStamp(), at the line int major_max = fCurrentFragTree->GetMaximum("TimeStampHigh");

I'm snooping around on your computer right now and I don't want to screw up stuff for you, so can you print out the pointer of the fCurrentFragTree right before that line? I can't think of a reason why it would change, but I don't know what it would seg fault otherwise.

Another thing to be aware of is that this is mulitthreaded, so it might not be that line, the seg fault could be in the other thread.

I am seg faulting at Guelph as well.

fCurrentFragTree: 0x1a43dd0

It's unlikely that it's that line. The frag tree would have to be deleted between indexing and getting the max. There is something else going on. Maybe it has to do with the changes to the write Q? any idea how old this problem is?

Yesterday I was using whatever version of LeanGRSI @VinzenzBildstein left me on the grifuser account on Tuesday and it worked. This morning was the first time I saw the problem and when I checked the last build time for grsisort, it was 2:15 Pacific yesterday.

Maybe the TDetector update happened? The TDetector worked for me before the new makefile.

I think my update yesterday did include the TDetector update.

Can you point me to a runnumber and a calfile where  this is happening?
On 8/27/15, SmithJK<EMAIL_ADDRESS>wrote:

I think my update yesterday did include the TDetector update.

Reply to this email directly or view it on GitHub:
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135517195

scratch that, i see your old responce.  a lot has happened this afternoon.
On 8/27/15, Peter<EMAIL_ADDRESS>wrote:

Can you point me to a runnumber and a calfile where  this is happening?
On 8/27/15, SmithJK<EMAIL_ADDRESS>wrote:

I think my update yesterday did include the TDetector update.

Reply to this email directly or view it on GitHub:
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135517195

It's happening in every run. Cal file or no cal file.

So I can confirm the memory to the tree still exists as I can print out the name of the tree. But the GetMaximum call is indeed the culprit. I can make this call in the interpreter. Is the dictionary not linking properly? Putting a bad string in the get maximum should return 0, not seg fault.

I can also type all of these lines in one-by-one and it works fine in the interpreter.

Ok, It works again.
It was an issue with changing the eventqueue to a singleton and an
infinite loop.  all is good.
I fixed this with grifuser on grsmid01.  I pushed to the GRSIUser
github but once again do not know how or have the password to create a
pull request upstream.
On 8/27/15, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I can also type all of these lines in one-by-one and it works fine in the
interpreter.

Reply to this email directly or view it on GitHub:
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135522564

I’ll do it again, but just go to the GRSIUser github page, and then do it like you would on your profile. Its nice because it will keep the commit as GRSIUser, but it will say you are doing the pull request. This makes it easier to figure out who did what.

On Aug 27, 2015, at 2:59 PM, pcbend<EMAIL_ADDRESS>wrote:
Ok, It works again.
It was an issue with changing the eventqueue to a singleton and an
infinite loop. all is good.
I fixed this with grifuser on grsmid01. I pushed to the GRSIUser
github but once again do not know how or have the password to create a
pull request upstream.
On 8/27/15, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I can also type all of these lines in one-by-one and it works fine in the
interpreter.

Reply to this email directly or view it on GitHub:
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135522564
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135523943.

Don't pull to the Collaboration yet. I put in a lot of diagnostic printf
statements that should be removed first.
On Thu, Aug 27, 2015 at 12:01 PM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I’ll do it again, but just go to the GRSIUser github page, and then do it
like you would on your profile. Its nice because it will keep the commit as
GRSIUser, but it will say you are doing the pull request. This makes it
easier to figure out who did what.

On Aug 27, 2015, at 2:59 PM, pcbend<EMAIL_ADDRESS>wrote:
Ok, It works again.
It was an issue with changing the eventqueue to a singleton and an
infinite loop. all is good.
I fixed this with grifuser on grsmid01. I pushed to the GRSIUser
github but once again do not know how or have the password to create a
pull request upstream.
On 8/27/15, Ryan Dunlop<EMAIL_ADDRESS>wrote:

I can also type all of these lines in one-by-one and it works fine in
the
interpreter.

Reply to this email directly or view it on GitHub:

https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135522564

—
Reply to this email directly or view it on GitHub <
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135523943
.

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-135524223
.

Oops. Too late.

I just learnt that.. Update and push :p

I went to the washroom and when I came back, my computer looked like the matrix.

Merged #461. Take the blue pill and pull the most recent update.

So it seems that the issues with the makefile have been solved. So can we close this issue?

Also, be aware that we might have to go through and change some of the names of libraries being loaded in the code.
Many libraries used to drop the T, ie libGRSIFormat.so is no libTGRSIFormat.so.

This was a problem in grsi-config. Now fixed in pull request #473.

That’s ok though. I would prefer all the libraries to have had a T in them in the first place.

On Sep 4, 2015, at 1:05 PM, SmithJK<EMAIL_ADDRESS>wrote:
Also, be aware that we might have to go through and change some of the names of libraries being loaded in the code.
Many libraries used to drop the T, ie libGRSIFormat.so is no libTGRSIFormat.so.
This was a problem in grsi-config. Now fixed in pull request #473 https://github.com/GRIFFINCollaboration/GRSISort/pull/473.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/453#issuecomment-137794168.

Has this been entirely fixed?

I haven't had problems for a while.

On Sep 10, 2015, at 3:59 PM, SmithJK<EMAIL_ADDRESS>wrote:
Has this been entirely fixed?
—
Reply to this email directly or view it on GitHub.

I haven't really used grsisort on my laptop, so I can't say. @r3dunlop, did you sort files and do fits on your laptop? If so, I think that we can close this issue.

Yes. I almost constantly go back and forward between Linux and OS X. I'm happy closing this.

On Sep 10, 2015, at 5:44 PM, Vinzenz Bildstein<EMAIL_ADDRESS>wrote:
I haven't really used grsisort on my laptop, so I can't say. @r3dunlop, did you sort files and do fits on your laptop? If so, I think that we can close this issue.
—
Reply to this email directly or view it on GitHub.


Add Escalator/conveying preset

Closes #5350

Thanks @mchlp !  I renamed some of the files, but otherwise merged as-is.


Can't signin with MFA

Describe the bug
My tenet requires MFA and I use MS authenticator on my phone
To Reproduce

Go to 'https://developer.microsoft.com/en-us/graph/graph-explorer'
Click on 'Sign in to Graph Explorer'
Enter user name , password ang the MFA
no error but my ID not showing. I am still ""displayName": "Megan Bowen","

Expected behavior
My graph call should be for my tenant
Screenshots
Desktop (please complete the following information):

OS: [Windows 10]
Browser [chrome]
Version [latest]

AB#9660

Hi @Ofer-Gal. Are you still experiencing this issue?

Yes!
a Logon shows, I choose my production ID but never get the authenticator to ask for approval
Graph-Explorer shows error: "Authentication failed - interaction required"
Is there a new version I can test?

Okay @Ofer-Gal . Please attach screenshots of the steps you are taking to sign in up to the error message.

Logon

Then

It does not even ask for password.

Does the issue persist when you try signing in with a different account or a different browser?
You can try that just to confirm that the issue is not with your account.

Different browsers / Computers etc..
I got all the way to the authenticator but Graph explorer still not connected

This appears to be an issue with your tenant. Re-registering MFA should solve this.
If you are the admin, log in to your tenant on https://aad.portal.azure.com/.

Then click on Users. That should load all users including your admin user profile. Click on the user profile with the MFA issue.

Then click on authentication methods

Here, you can revoke the MFA sessions and try to sign in to Graph Explorer to confirm that you can still sign in, then you can click on 'Require re-register MFA'. Then go back to Graph Explorer and try to sign in again. Follow the prompts which will help you re-register MFA.

If the admin user is the one with the MFA issue, try revoking all sessions then sign in to GE again. If this doesn't resolve the issue, create a new support request on 'Troubleshooting + Support' on your aad portal.

My admin is asking what "revoking all sessions" means? where do You do it?
Thanks

After signing in to https://aad.portal.azure.com/, your admin should click on Users on the left tab :

Then click on your user profile that is experiencing the MFA issue: In my case, I clicked on 'Evans Test User'

Then click on 'authentication methods':

At the top of the tab, you can see 'Require re-register MFA' and 'Revoke MFA sessions'.
Revoking MFA sessions will sign you out of all your accounts and you'll have to sign in afresh. 'Require Re-register MFA' makes it so that when you signs in next time, you'll be requested to set up a new MFA authentication method. Either of those should solve the issue

Thank you


fix(readme): add the postgres user

:tada: This PR is included in version 1.49.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hydration error in certain website pages

Describe the bug
A hydration error is occuring when launching the website in local. It occurs in the following components:

Badge
BarChart
Bleed
Container
ContextualMenu
Flex
Grid
Image
Paragraph
Sidenav
Typography

To Reproduce

Clone the repo
Go into /website
Run npm install
Run npm run dev
Go to any of the listed container pages and click refresh or F5
You should be seeing the error

Expected behaviour
No error should appear in any component page.
Screenshots

Additional context
More information about the error possible causes

A solution is ready. It will be added within #1945.


Namespace directives

Your directives should contain names spaces. I just spent a week troubleshooting an issue where I had a directive with an attribute called "collapse" and when I ran it through my grunt build process it was not working. Eventually i figured out that if i removed angular-ui from my project it worked again so I believe there is a conflict between angular-ui's directive "collapse" and an attribute of my directive called "collapse".
If angular-ui had namespaces for it's directives like "ui-collapse", it is much less likely to have conflicts.

Duplicate of #3396


GROUP BY clause will not properly rendered if there is a function applied column expression

Environment details

OS type and version: Windows 10 Pro, macOS Ventura 13.4
Python version: 3.8.10
pip version: 23.1.2
sqlalchemy-bigquery version: 1.6.1

Steps to reproduce

Compose an SQL statement using SQLAlchemy Core or ORM which includes a GROUP BY clause with an expression timestamp_trunc(`some_column`, hour)
Render it (or execute it and confirm how it was rendered by examining SQLAlchemy's log)

Code example
Below is an example script to reproduce the symptom. Please refer to the first comment block in the file for detail.
"""An example script to reproduce the bug.

Executing this script outputs (partially folded for ease of reading):

    sqlalchemy:          1.4.49
    sqlalchemy-bigquery: 1.6.1
    SQLite----------------------------------------
    SELECT timestamp_trunc(sensor.time, hour) AS time,
           max(sensor.value) AS max_1
    FROM sensor
    GROUP BY timestamp_trunc(sensor.time, hour)
    BigQuery----------------------------------------
    SELECT timestamp_trunc(`sensor`.`time`, hour) AS `time`,
           max(`sensor`.`value`) AS `max_1`
    FROM `sensor`
    GROUP BY `timestamp_trunc_1`

In the final line, there is a reference to a non-existing colunn
of which name is `timestamp_trunc_1`. The line should be rendered as:

    GROUP BY timestamp_trunc(`sensor`.`time`, hour)

...like the rendering result for SQLite, written above.
"""
import sqlalchemy as sa
import sqlalchemy.dialects.sqlite as sqlite
import sqlalchemy_bigquery as bigquery

print("sqlalchemy:         ", sa.__version__)
print("sqlalchemy-bigquery:", bigquery.__version__)

engine = sa.create_engine("sqlite://")
metadata = sa.MetaData()

sensor_table = sa.Table(
    "sensor",
    metadata,
    sa.Column("time", sa.TIMESTAMP),
    sa.Column("value", sa.String),
)

stmt = sa.select(
    sa.func.timestamp_trunc(sensor_table.c.time, sa.text("hour")).label("time"),
    sa.func.max(sensor_table.c.value),
).group_by(
    sa.func.timestamp_trunc(sensor_table.c.time, sa.text("hour")),
)

print("SQLite" + "-" * 40)
print(
    stmt.compile(
        dialect=sqlite.dialect(),
        compile_kwargs={"literal_binds": True},
    ),
)

print("BigQuery" + "-" * 40)
print(
    stmt.compile(
        dialect=bigquery.dialect(),
        compile_kwargs={"literal_binds": True},
    ),
)

Stack trace
N/A (no exception will be raised unless you send it to BigQuery)

Would love to know the cause of this! I'm getting the same error when using ROLLUP inside the groupby clause:
Example code:
import sqlalchemy as sa
from sqlalchemy_bigquery import BigQueryDialect

        query = (
            sa.select(sql_projections)
            .select_from(model_expression)
            .where(sa.and_(*filter_expressions))
            .group_by(func.rollup(*groupby_expressions)) // <-- relevant rollup expression
            .having(sa.and_(*having_expressions))
        )

        query_str = str(
            query.compile(dialect=BigQueryDialect())
        )

I've been working around it by using sa.text but would appreciate some insight into why this is happening.
Output Query:
SELECT timestamp_trunc(`created_date`, YEAR) AS `created_date`,
       `agency` AS `agency`,
       `city` AS `city`,
       count(*) AS `row_count`,
FROM `quickstart.nyc_311`
GROUP BY `rollup_1` // <-- error here, this should be ROLLUP(1, 2, ..) as it is in other dialects

Working on implementing group by, rollup and cube clauses as part of another customer issue, which should include a fix for this

@sgryjp @liutmelody This fix is now a part of branch development-build-1.11.0.dev3. I will be setting up a prerelease soon on PyPI as well for this development release, the reason being that this branch is based off our SQLAlchemy 1.4-2.0+ migration work. Please try it out and let me know if it solves your problem

@kiraksi Hi, thank you for fixing this issue! I confirmed that the problem was fixed by your commit 5cfc28089baa6106cf30f9efb268231792da9251.
I also confirmed that the branch development-build-v1.11.0.dev3 does reproduce the problem. It seems that the commit bd38a5e14145ca77ec462e7cf4e9a989f2eb1fb3 somehow gets the bug back again. Could you check if there is anything suspicious? Thank you in advance.

(.venv) PS> git log --oneline development-build-v1.11.0.dev3                                                                           87a75dc (HEAD, origin/development-build-v1.11.0.dev3, development-build-v1.11.0.dev3) create development build 1.11.0.dev3
0c882b9 test commit
033d329 reformat logic
5154814 added test case
bd38a5e fixed render label as label assignment
68afc39 test basic implementation of group_by_clause and visit_label
ece7f1f removed unnecessary clause function changes, edited tests
e82f5dd test commit to run kokooro tests
310cfa7 Merge branch 'development-build-v1.11.0.dev2' into grouping-rollup-cube
0c4cf07 (tag: v1.11.0.dev2) create development release 1.11.0.dev2
5cfc280 feat: grouping sets, rollup and cube compatibility
...(log continues)...

(.venv) PS> pip install --force-reinstall git+https://github.com/googleapis/python-bigquery-sqlalchemy.git@bd38a5e14145ca77ec462e7cf4e9a989f2eb1fb3 >$null; python .\issue879.py
  Running command git clone --filter=blob:none --quiet https://github.com/googleapis/python-bigquery-sqlalchemy.git 'C:\Users\sgryjp\AppData\Local\Temp\pip-req-build-d9h_kbnv'
  Running command git rev-parse -q --verify 'sha^bd38a5e14145ca77ec462e7cf4e9a989f2eb1fb3'
  Running command git fetch -q https://github.com/googleapis/python-bigquery-sqlalchemy.git bd38a5e14145ca77ec462e7cf4e9a989f2eb1fb3
  Running command git checkout -q bd38a5e14145ca77ec462e7cf4e9a989f2eb1fb3
sqlalchemy:          2.0.27
sqlalchemy-bigquery: 1.11.0.dev2
SQLite----------------------------------------
SELECT timestamp_trunc(sensor.time, hour) AS time, max(sensor.value) AS max_1
FROM sensor GROUP BY timestamp_trunc(sensor.time, hour)
BigQuery----------------------------------------
SELECT timestamp_trunc(`sensor`.`time`, hour) AS `time`, max(`sensor`.`value`) AS `max_1`
FROM `sensor` GROUP BY `timestamp_trunc_1`

(.venv) PS> pip install --force-reinstall git+https://github.com/googleapis/python-bigquery-sqlalchemy.git@68afc3959d988b77a244699e7d5e4f39bd233917 >$null; python .\issue879.py
  Running command git clone --filter=blob:none --quiet https://github.com/googleapis/python-bigquery-sqlalchemy.git 'C:\Users\sgryjp\AppData\Local\Temp\pip-req-build-hywz32py'
  Running command git rev-parse -q --verify 'sha^68afc3959d988b77a244699e7d5e4f39bd233917'
  Running command git fetch -q https://github.com/googleapis/python-bigquery-sqlalchemy.git 68afc3959d988b77a244699e7d5e4f39bd233917
  Running command git checkout -q 68afc3959d988b77a244699e7d5e4f39bd233917
sqlalchemy:          2.0.27
sqlalchemy-bigquery: 1.11.0.dev2
SQLite----------------------------------------
SELECT timestamp_trunc(sensor.time, hour) AS time, max(sensor.value) AS max_1
FROM sensor GROUP BY timestamp_trunc(sensor.time, hour)
BigQuery----------------------------------------
SELECT timestamp_trunc(`sensor`.`time`, hour) AS `time`, max(`sensor`.`value`) AS `max_1`
FROM `sensor` GROUP BY timestamp_trunc(`sensor`.`time`, hour)

@sgryjp This is odd, the previous commit you mentioned was a fix that introduced other issues, I created a new fix that is now on development-build-v1.11.0.dev3, I even confirmed the fix by adding various test cases. Can you confirm again whether development-build-v1.11.0.dev3 does not fix your issue with the new commits there? Or create an example I can run as a test based off the code you are getting this error on? Please let me know.

@kiraksi Thank you for response!
Unfortunately, I confirmed that neither of tag v1.11.0.dev3 nor HEAD of the branch development-build-v1.11.0.dev3 does not fix my issue. Running the "code example" in the first comment results the same as below:

$ pip install --force-reinstall<EMAIL_ADDRESS>python issue879.py
...(omitted)...
sqlalchemy:          2.0.27
sqlalchemy-bigquery: 1.11.0.dev3
SQLite----------------------------------------
SELECT timestamp_trunc(sensor.time, hour) AS time, max(sensor.value) AS max_1
FROM sensor GROUP BY timestamp_trunc(sensor.time, hour)
BigQuery----------------------------------------
SELECT timestamp_trunc(`sensor`.`time`, hour) AS `time`, max(`sensor`.`value`) AS `max_1`
FROM `sensor` GROUP BY `timestamp_trunc_1`


Julia doesn't boot with "optimisationLevel" variable

I recently bumped into a problem where whenever I tried to boot Julia within Atom I would get the following error:
ERROR: unknown option `-2`

And Julia would never finish booting. I tracked the problem to this line. Namely
as.push "-O#{optimisationLevel}"

I assume this tries to start Julia with a line something like julia -O2 (which reproduces the error). Removing #{optimisationLevel} solves the problem for me though!
After all that rambling I guess my question is, is this the correct syntax for starting Julia? It doesn't work for me and I haven't seen it before.

It is, but only for julia 0.5 onwards. I think we have a 0.4 branch, so you could either use that our stay in the latest tagged version.

Ah ok. Thanks for clearing that up :)

We should probably just avoid the flag if the level is 2 anyway.

Good idea. ;)

Execution is 90% :)


[Event Request]Application “Tests-Workflow”, 134174 "WF Demo Sales BOrder Approvals"

[Event Request]Application “Tests-Workflow”, 134174 "WF Demo Sales BOrder Approvals"
Could you please insert the OnTestInitialize Event at the beginning of Initialize procedure:

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


edit Open vSwitch license info so that GitHub recognizes it

Hello! 👋
(CCing @dankohn who has requested this work so that the license will appear correctly in https://landscape.cncf.io/selected=open-v-switch)
GitHub uses a library called Licensee to identify a project's license
type. It shows this information in the status bar and via the API if it
can unambiguously identify the license.
This commit creates a LICENSE file that stores the full text of the
Apache license. It also removes the COPYING file and transfers its
contents to a new "License" section in the README.
Collectively, these changes allow Licensee to successfully identify the
license type of Open vSwitch's codebase as Apache.

Thanks.  I made some updates and submitted the updated version to our developer mailing list for review (which is the more common way that we handle submissions for OVS).  You should be CCed so that you continue to receieve updates.

Thanks @blp. I'll follow the thread on your mailing list.


Use PSRule for security audits instead of template analyser

template analyser raises a number of false positives, also this approach gives us the ability to control which features and settings we want to validate.

There are 6 issues to resolve for the app service template https://github.com/Azure-Samples/azure-search-openai-demo-java/security/code-scanning?query=pr%3A91+is%3Aopen+tool%3Apsrule+


[Interpreter / Calyx Comp] Fix the reset signals

Small PR which closes #584 and (hopefully) fixes some discrepancies with the reset signal behavior in the verilog primitives.

Seems like this also has changes for sqrt. Was that intentional?

That is correct, square root also has a reset port

Cool; all looks good to me on a quick look.


Backend initialization failed - ModuleNotFoundError: No module named 'flask'

Describe the bug
The AI inference backend failed to initialize.
Please try restarting the application.
If the problem persists, you can check the Details for additional information about the error.
from flask import Flask, jsonify, request, Response, stream_with_context
ModuleNotFoundError: No module named 'flask'
Screenshots :

Log :

Environment :

OS: Win11
GPU: Arc A770LE 16GB
CPU: i7 14700
Version: v1.22.1-b

Python 3.12
Pip list :

flask 3.0.3 > checked
pip 24.3.1 > checked

Broken python environment during installation. You can follow these steps to recover:

Open a terminal under <InstallDir>\resources\service\
Execute ..\env\python.exe -m pip install -r .\requirements-arc.txt


fix: some fuzzy class names will also be incorrectly matched (close #484)

This pull request fixes the problems described in #484.
Added the catchPrefix method to dynamically generate the corresponding regular expression with prefix when matching numbers or fraction, etc., so that absolute precision can be achieved.

Codecov Report

Merging #486 (4576839) into main (d827f96) will increase coverage by 0.11%.
The diff coverage is 93.86%.

@@            Coverage Diff             @@
##             main     #486      +/-   ##
==========================================
+ Coverage   87.60%   87.71%   +0.11%
==========================================
  Files          57       57
  Lines        4842     4967     +125
  Branches     1039     1000      -39
==========================================
+ Hits         4242     4357     +115
- Misses        337      350      +13
+ Partials      263      260       -3

Impacted Files
Coverage Δ

src/lib/utilities/handler.ts
67.57% <66.66%> (-2.48%)
:arrow_down:

src/lib/utilities/dynamic.ts
95.59% <96.61%> (+1.30%)
:arrow_up:

src/lib/extract.ts
83.72% <100.00%> (-0.37%)
:arrow_down:

Continue to review full report at Codecov.

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

Unfortunately, when I was about to finish writing this solution, I found a better one.
We can split the utility at the very beginning when we match to the key, and then match based on the remaining part, which is much more efficient. So that we don't have to do a regular expression match every time.
This solution will be implemented in the next version, and the branch will not be merged for now.

Temporary revert this fix due to the regression #543 #536 #542


Display the autocomplete dropdown upwards if it is at the bottom of the browser window

Hi,
I use the typeahead directive in a popover. If the popover is placed at the bottom of the window, the dropdown should be displayed at the top of the text input.

Our drop downs have the same behavior as the current bootstrap 3 drop downs. Reopen if you'd like to submit a pull request.


Add single quote to list of characters escaped

interpolation.yml states that only four characters are escaped: & " < >.  It should also include the single quote '.  (It should probably also explicitly state whether an implementation can escape additional characters and still remain spec-compliant.)
Not escaping the single quote means <tag attr='{{var}}'> is unsafe, in spite of being semantically equivalent to <tag attr="{{val}}"> which is safe.  That's confusing and a recipe for exploits!
Such a change should largely be compatible with mustache-in-the-wild: a number of mustache implementations /already/ escape the single quote (at least in some cases), so any shop which uses mustache in multiple languages is unlikely to be relying on the non-escaped-single-quote:
Mustache.js: escapes single quotes.
mustache (ruby): uses CGI. CGI doesn't escape single quotes in Ruby 1.9.x, but does in Ruby 2.0.
pystache: Single quotes escaped as of Python 3.2.
mustache.go: Escapes single quotes (as '!).

This is still an omission as of 2023. I welcome a pull request to address this.
(My implementation, Wontache, escapes single quotes as well.)

I think this should be the default, but will require a major version change. Let's tag things with backwards compatibility breaks that we'd like to do anyway?

I would say "bugfix". Bug fixes are always breaking in a way, but we don't bump the major version for them, because the thing that stops working should not have worked in the first place.
I would be more hesitant with this stance if most implementations didn't escape single quotes yet. Given that the practice is already widespread, however, I feel this is just a fix for an omission.


Solve #11 - Update PTO use cases columns names

Closes #11
Description
In this PR:

the column used to fetch the user's names for the PTO use cases was changed from "Person" to "Description".
the support email date format was updated to use %d instead of %e (reference)
the tests were updated

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
581

Relevant Lines:
581

💛  - Coveralls


Automated cherry pick of #2483: fix: zstack reset passworkd with username

Cherry pick of #2483 on release/2.10.0.
#2483: fix: zstack reset passworkd with username

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@d9946f7). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #2486   +/-   ##
================================================
  Coverage                  ?   7.52%
================================================
  Files                     ?     410
  Lines                     ?   63973
  Branches                  ?       0
================================================
  Hits                      ?    4814
  Misses                    ?   58748
  Partials                  ?     411

Flag
Coverage Δ

#aFlag
7.52% <0%> (?)

Impacted Files
Coverage Δ

pkg/util/esxi/virtualmachine.go
0% <0%> (ø)

pkg/util/ucloud/instance.go
0% <0%> (ø)

pkg/util/aws/instance.go
0% <0%> (ø)

pkg/compute/guestdrivers/managedvirtual.go
0% <0%> (ø)

Continue to review full report at Codecov.

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

/approve
/lgtm


How to compile code to run on OS (Centos8 or  others ) ?

The main dependency is clang11 and foundationdb cli. Or you can try compile code with docker https://github.com/ByConity/ByConity/blob/master/docker/debian/dev-env/README.md


Is This Still Maintained?

Just Wondering

No, unfortunately not. I haven't used it for a long time. However, it seems like some people still use it.
Do you have any problems?

there are a few issues, im making a fork to keep it maintained i'll make a pull request soon.


Windows builds failing

Windows builds seem to fail after d366c169874e266b82a517249ae2479f3c7a0387

terminal-size downgrade result
https://github.com/dvvvvvv/lsd/actions/runs/81436854
terminal-size windows test result
https://github.com/dvvvvvv/terminal-size/actions/runs/81444672
This seems to be a problem caused by terminal_size crate.
terminal_size crate affects the behavior of lsd in core.rs: 34,35.
#[cfg(target_os = "windows")]
let tty_available = terminal_size().is_some(); // terminal_size allows us to know if the stdout is a tty or not.

This is a syntax related to whether not to display the icon when there is no terminal. And as you can see from the link I wrote above, It appears that the github action has a console buffer of size 1,1. However, looking at the code, this seems to be a bug with terminal_size crate.
Anyway, the test shows the following error message.
thread 'test_list_almost_all_populated_directory' panicked at 'Unexpected stdout, failed var.is_match(one
two
$)
└── var as str:  one
 two

This shows that pattern matching has failed because of the icon.
There are several solutions.

downgrade terminal-size crate.
change args in tests.

Well, I guess technically 1,1 passes for some().
Since we cannot really control flags, I think we can force classic in the tests.
I guess we should also have a test with --icons always and --color always in there too.

I suspect that the latest version of terminal-size will work completely opposite to 0.1.8.  So first of all, it would be a good idea to check if it works normally in Windows. If it doesn't work as I thought, lowering the version would be a better option.  If not, it would be better to do it according to @meain

I checked it. terminal-size 0.1.11 doesn't seem to work as I thought.
I'll leave the results for each version below.
master build with terminal-size 0.1.8:
PS C:\Users\smlim2\src\lsd> .\target\release\lsd.exe C:\Users\smlim2\AppData\
 Local   LocalLow   Roaming

master build with terminal-size 0.1.11:
PS C:\Users\smlim2\src\lsd> .\target\release\lsd.exe C:\Users\smlim2\AppData\
Local
LocalLow
Roaming

It would be better to downgrade the terminal-sized version until the terminal-sized behavior is properly modified.
Apart from the version downgrade, it would be a good idea to fix the launch options as well. With that test method, it is impossible to properly test the display of the icon.
Therefore, it would be correct to run a separate test with the icon display option when testing for the icon became possible.

@dvvvvvv I am not exactly sure how this crate works. I am still a bit confused as to what might be going on. Feel free to correct me.
From what I understand, if the console buffer size returns anything other than not available I think terminal-size crate should return that a tty is available. Maybe for windows GitHub actions used to report terminal as not available but now started to report as 1,1 or something?
Also, the GitHub action seems to fail saying that there is are icon, but from the example you have given it seems to say that there are no icons with 0.1.11.

@meain Sorry.  I wrote more complicated than necessary.
What I wanted to show was that the terminal-size crate showed completely different results depending on the version.
This seems to be a bug in version 0.1.11 if it comes from a simple conditional error.
In conclusion, lsd compiled with terminal-size crate 0.1.11 behaves like this on Windows:
When the icon should show => not showing
When the icon should not be visible => showing
So at this point, it would make sense to drop the dependency version.

Looks like this is taken care of https://github.com/eminence/terminal-size/pull/19

I've pushed an upgraded terminal-size in its own PR:  #361


truncateDeep() throws with objects not inheriting hasOwnProperty()

https://github.com/bugsnag/bugsnag-js/blob/67753db6e65ad96fc4a726dc76d2c76fe195aa97/src/bugsnag.js#L761
This line is not safe and breaks history.pushState and history.replaceState when state contains object(s) that do not inherit hasOwnProperty (e.g. were created with Object.create(null)).
Observed behavior
TypeError: e.hasOwnProperty is not a function

Steps to reproduce

Have auto breadcrumbs enabled and call history.pushState with a state containing object(s) that do not not inherit hasOwnProperty.

var plainObject = Object.create(null);
plainObject.dummyProperty = true;
var state = {myObject: plainObject};
history.pushState(state, 'foo', 'bar.html');

Version
3.x

Hey @brianreavis, thanks for the bug report. I'll get this looked into.

Thanks @bengourley!

Hey @brianreavis, v3.3.1 was just released containing the fix for this. If you are using the CDN link, the latest version will be fetched automatically and should just work™. Otherwise, please manually upgrade and let me know how you get on. Cheers!


Loading schedule on every artisan command run

Laravel Version: Any
PHP Version: Any

Description:
Hello! Why does Laravel run \App\Console\Kernel::schedule in its constructor. Why do we need to have some schedule on every artisan call? I think it's a bit overhead for non-schedule commands.
A similar issue was closed by its creator https://github.com/laravel/framework/issues/24165

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Brotli OptimizeHuffmanForRle

Zopfli has the function OptimizeHuffmanForRle which is used to improve the compression of huffman trees. Brotli has an improved, but compatible version.
How to copy this function in zopfli?

I tried with libzopfli – works well. Size improvement is minimal, and about 2% speed improvement.
Apply this patch after replacement of OptimizeHuffmanForRle with OptimizeHuffmanCountsForRle.
diff -Naur zopfli/Makefile brotli/Makefile
--- zopfli/Makefile
+++ brotli/Makefile
@@ -1,7 +1,7 @@
 CC = gcc
 CXX = g++

-CFLAGS = -W -Wall -Wextra -ansi -pedantic -lm -O2
+CFLAGS = -W -Wall -Wextra -ansi -pedantic -lm -O2 -std=c99
 CXXFLAGS = -W -Wall -Wextra -ansi -pedantic -O2

 ZOPFLILIB_SRC = src/zopfli/blocksplitter.c src/zopfli/cache.c\
diff -Naur zopfli/src/zopfli/deflate.c brotli/src/zopfli/deflate.c
--- zopfli/src/zopfli/deflate.c
+++ brotli/src/zopfli/deflate.c
@@ -396,13 +396,13 @@
 compression, especially its rle-part will be more likely to compress this data
 more efficiently. length containts the size of the histogram.
 */
-bool OptimizeHuffmanCountsForRle(size_t length, uint32_t* counts) {
+unsigned OptimizeHuffmanForRle(size_t length, size_t* counts) {
   size_t nonzero_count = 0;
   size_t stride;
   size_t limit;
   size_t sum;
   const size_t streak_limit = 1240;
-  uint8_t* good_for_rle;
+  unsigned char* good_for_rle;
   // Let's make the Huffman code more compatible with rle encoding.
   size_t i;
   for (i = 0; i < length; i++) {
@@ -422,7 +422,7 @@
   // Now counts[0..length - 1] does not have trailing zeros.
   {
     size_t nonzeros = 0;
-    uint32_t smallest_nonzero = 1 << 30;
+    size_t smallest_nonzero = 1 << 30;
     for (i = 0; i < length; ++i) {
       if (counts[i] != 0) {
         ++nonzeros;
@@ -451,7 +451,7 @@
   }
   // 2) Let's mark all population counts that already can be encoded
   // with an rle code.
-  good_for_rle = (uint8_t*)calloc(length, 1);
+  good_for_rle = (unsigned char*)calloc(length, 1);
   if (good_for_rle == NULL) {
     return 0;
   }
@@ -459,7 +459,7 @@
     // Let's not spoil any of the existing good rle codes.
     // Mark any seq of 0's that is longer as 5 as a good_for_rle.
     // Mark any seq of non-0's that is longer as 7 as a good_for_rle.
-    uint32_t symbol = counts[0];
+    size_t symbol = counts[0];
     size_t step = 0;
     for (i = 0; i <= length; ++i) {
       if (i == length || counts[i] != symbol) {
@@ -502,7 +502,7 @@
         for (k = 0; k < stride; ++k) {
           // We don't want to change value at counts[i],
           // that is already belonging to the next stride. Thus - 1.
-          counts[i - k - 1] = static_cast<uint32_t>(count);
+          counts[i - k - 1] = count;
         }
       }
       stride = 0;

Good afternoon!
I thank for the help. You couldn't send the ready zopfli version with a patch. At me it is impossible to apply it.

@lorents17 You may test https://github.com/MegaByte/zopfli/tree/brotli_huffman

thank you very much!


Make e2e shell tests work standalone

Simply incorporate some of the boilerplate from hack/e2e.go into the scripts in hack/e2e-suite.
Use environment variables with default values to allow overrides in kubectl command line and to use a versioned package root.
Tested:

Ran go run hack/e2e.go -test on a test cluster.
Ran the test cases individually.
Ran hack/e2e-suite/goe2e.sh -t Pods to confirm it takes arguments.
Also fixed cluster/test-network.sh (which should be more and more irrelevant.)

Next step is to refactor hack/e2e.go replacing runBash and friends with a straight exec.Command run. The other commands (-up, -build, -down etc.) will also become standalone internal scripts.
As I expected, cluster/test-network.sh was a mistake and is now starting to become a maintenance burden.
@satnam6502 @zmerlynn

@brendanburns

LGTM. If we moving ahead with the shell scripts I'd complain but this is fine for the graveyard.
Please don't wrecking-ball further into hack/e2e.go until I've got my PR in.

(I'll merge during core hours.)


"Bestaudio" should be ignored if there's no audio streams

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS> (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
When watching a video through mpv, my default quality is bestvideo[height<=?480]+bestaudio/best. This works fine in most websites, but some don't have separate audio streams and so fail the bestaudio check, moving over to the best available quality. This affects peer.tube instances, for example.
As any site that lacks a separate audio stream will have it included in the video itself, perhaps the bestaudio format could be ignored if no audio streams are found? I realize this can be fixed just by using bestvideo[height<=?480]+bestaudio/bestvideo[height<=?480]/best instead and have already done so in my mpv.conf, but it's not the most intuitive thing and a less experienced user might trip over.
Verbose log
No response

There is probably a simpler way to achieve what you want. But can't really say without example URL and verbose logs

But no, yt-dlp will not be made to "ignore" bestaudio if you asked it to download that format!

Surely bestvideo only matches streams with acodec=='none', so bestvideo[height<=?480]+bestaudio/bestvideo[height<=?480]/best will give you in order of preference:

a merger of the "best" video-only and audio-only streams where the height of the video, if known, is not more than 480 pixels, or if none
a video-only stream where the height of the video, if known, is not more than 480 pixels, or if none
the "best" combined video and audio format, if there is one.

As reading TFM would show, yt-dlp understands bestvideo* to mean the best video format including formats that also have audio streams.


Incorrectly assumes a browser environment, Headers not found

When importing QdrantClient and instantiating a local client connection, I receive the following error:
$ node src/index.mjs
file:///home/duane/tmp/try-qdrant/node_modules/.pnpm/@qdrant+js-client-rest@1.1.6_typescript@5.0.4/node_modules/@qdrant/js-client-rest/dist/esm/qdrant-client.js:36
        const headers = new Headers();
                        ^

ReferenceError: Headers is not defined
    at new QdrantClient (file:///home/duane/tmp/try-qdrant/node_modules/.pnpm/@qdrant+js-client-rest@1.1.6_typescript@5.0.4/node_modules/@qdrant/js-client-rest/dist/esm/qdrant-client.js:36:25)

I can work around this via:
import fetch, { Headers } from 'node-fetch';

global.fetch = fetch;
global.Headers = Headers;

import { QdrantClient } from "@qdrant/js-client-rest";

const client = new QdrantClient({ url: "http://<IP_ADDRESS>:6333" });

However, it would be ideal if QdrantClient does not assume ambient Headers exist.

The required engine is Node 18, which ships with native fetch and thus the Headers constructor. You should have gotten a warning after npm install-ing the dependency. In any case, it will work with Node 16 if you polyfill it, as you just did.


Bugsnag gem crashes when used with an older mongo gem (1.12)

Trying to use bugsnag gem together with mongo 1.12 raises the following error during application boot time: uninitialized constant Mongo::Monitoring (NameError).
Goal
To make sure Bugsnag can be used with applications that are stuck with an older mongo version, even if Mongo integration cannot be used in that case.
Changeset
I added a check to make sure Mongo::Monitoring class is defined before Bugsnag tries to use it. Pretty simple.
Testing
Tested manually.

Hey @isabanin, thanks for the PR! We'll look at reviewing/merging this when priorities allow.


how can search contacts by country ?

how can i search contact only from particular country ?

It is not possible with the api.

http://stackoverflow.com/questions/23375418/search-skype-for-users-by-country


fix: seed method upload or insert

Description
Resolve issue #279, so that users can revert to pre-1.4.2 behaviour and avoid need for extra IAM permissions
Models used to test - Optional
Tests are in the change  -  I was unable to find a quick way of verifying that the insert vs. upload method had been used but I checked debug lots (a log message indicates which strategy) and query logs to confirm that the correct number of rows were queryable
Checklist

[/] You followed contributing section
[/] You kept your Pull Request small and focused on a single feature or bug fix.
[/] You added unit testing when necessary
[/] You added functional testing when necessary

Confirmed can toggle between the two behaviours by setting the seed_by_insert flag in the appropriate connection profile in the project where I was having the problem upgrading, this will let us update past 1.4.2 :heavy_check_mark:


Discussion on the parallelism of cooperative_group

Hello, I have a confusion about the code implementation.
We use the GPU to take advantage of its parallelism to speed up the find or insert process.
Take the find operation as an example, cooperative_group is used in implementation. A key is searched in parallel using 16 threads, and 16 keys are actually searched serially in the tile.
So I think using tiles to find a key is a waste of GPU parallel resources: in fact, the process of finding a key by a single thread will not be very slow, and when we use 16 threads to find a key, only 1 thread is used, and the other 15 threads doing the same thing.
Therefore, from my point of view, I think that if each thread is designed to look up different keys, this can make full use of the parallel characteristics of the GPU and better speed up the lookup operation. What do you think?
Looking forward to your reply, thank you in advance!

Hi @ninixing, It is better to think about the memory access patterns to understand our design decision here. To fully utilize the DRAM bandwidth, we have to read data from global memory in a coalesced manner; and in the case of $b=16$ and 8-byte key-value pair, that means we are reading an entire cache line from the DRAM every time we read a bucket. Optimizing memory access patterns is the primary motivation for using the per-thread assignment per-tile processing approach.
Please also see my comment and the discussion in this issue.
I hope this clarifies things. Let us know if you have more questions!

Yes, your understanding is correct. I should add that for insertion, step 3 becomes:

16 threads compare the sentinel key to the key in their slot. A sentinel key indicates that the slot is empty, and by making that comparison, we find the offset of the next empty location in the bucket. Once we find the empty location, one of the 16 threads atomically sets (i.e., insert) the new key-value pair.

If the atomic operation fails, we try again. If the bucket is full, then we follow a probing scheme to determine the next bucket we should attempt insertion in. The probing discussion is a longer one, but I can just refer you to specific sections in the paper if you're interested.


[DEM-NNN] version check for Qiskit in setup

Adjustment for compatible qiskit versions

Coverage remained the same at 100.0% when pulling 4e14f99974c14670ed8c94bcc704db87e1147917 on feature/DEM-NNN/versioncheck_qiskit into 0bfb6d869fc16f40ee5faa8e16a0bec217e75dfd on dev.