ruby-shadow required, but can't compile on Ruby 3.0.0

Description
When running Chef on Ubuntu 20.04, Chef throws the following errors when trying to set a user password:
al_users line 27) had an error: Chef::Exceptions::MissingLibrary: You must have ruby-shadow installed for password support!

So I tell Chef to install the ruby-shadow Gem:
chef_gem 'ruby-shadow' do
  action :install
end

and that errs out with:
================================================================================
Error executing action `install` on resource 'chef_gem[ruby-shadow]'
================================================================================

Mixlib::ShellOut::ShellCommandFailed
------------------------------------
Expected process to exit with [0], but received '1'
---- Begin output of /opt/chef-workstation/embedded/bin/gem install ruby-shadow -q --no-document -v "2.5.0" --source=https://rubygems.org ----
STDOUT: Building native extensions. This could take a while...
STDERR: WARNING:  You don't have /root/.chefdk/gem/ruby/3.0.0/bin in your PATH,
          gem executables will not run.
ERROR:  Error installing ruby-shadow:
        ERROR: Failed to build gem native extension.

    current directory: /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0
/opt/chef-workstation/embedded/bin/ruby -I /opt/chef-workstation/embedded/lib/ruby/3.0.0 -r ./siteconf20211201-89170-rv4z38.rb extconf.rb
checking for getspent() in -lshadow... no
checking for getspent()... yes
checking for fgetspent()... yes
checking for setspent()... yes
checking for endspent()... yes
checking for lckpwdf()... yes
checking for ulckpwdf()... yes
checking for sgetspent()... yes
checking for ruby/io.h... yes
creating Makefile

current directory: /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0
make DESTDIR\= clean

current directory: /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0
make DESTDIR\=
compiling shadow/shadow.c
shadow/shadow.c: In function ‘rb_shadow_fgetspent’:
shadow/shadow.c:21:26: error: ‘struct rb_io_t’ has no member named ‘f’; did you mean ‘fd’?
   21 | #define file_ptr(x) (x)->f
      |                          ^
shadow/shadow.c:95:22: note: in expansion of macro ‘file_ptr’
   95 |   entry = fgetspent( file_ptr( (RFILE(file)->fptr) ) );
      |                      ^~~~~~~~
shadow/shadow.c: In function ‘rb_shadow_putspent’:
shadow/shadow.c:21:26: error: ‘struct rb_io_t’ has no member named ‘f’; did you mean ‘fd’?
   21 | #define file_ptr(x) (x)->f
      |                          ^
shadow/shadow.c:154:11: note: in expansion of macro ‘file_ptr’
  154 |   cfile = file_ptr( RFILE(file)->fptr );
      |           ^~~~~~~~
make: *** [Makefile:245: shadow.o] Error 1

make failed, exit code 2

Gem files will remain installed in /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0 for inspection.
Results logged to /root/.chefdk/gem/ruby/3.0.0/extensions/x86_64-linux/3.0.0/ruby-shadow-2.5.0/gem_make.out
---- End output of /opt/chef-workstation/embedded/bin/gem install ruby-shadow -q --no-document -v "2.5.0" --source=https://rubygems.org ----
Ran /opt/chef-workstation/embedded/bin/gem install ruby-shadow -q --no-document -v "2.5.0" --source=https://rubygems.org returned 1```

The problem is documented [here](https://github.com/apalmblad/ruby-shadow/issues/26) with a fix [here](https://src.fedoraproject.org/rpms/rubygem-ruby-shadow/blob/rawhide/f/ruby-shadow-2.5.0-cflags.patch). I have submitted [a PR for it](https://github.com/apalmblad/ruby-shadow/pull/27), but here's the problem:
- Chef uses ruby-shadow on the backend on Ubuntu 20.04 for managing user passwords (right? I'm not making this up?)
- The current version of ruby-shadow, 2.5.0, came out in 2015
- The linked patch was written in 2017
- The linked patch still hasn't been merged (and there was no PR for it until I submitted one today)

If Chef is using ruby-shadow on the backend, it would make sense for Chef to also maintain it. Or use something else to handle that functionality.

## Chef Version
<!--- Tell us which version of chef-client you are using (see below for Server+ChefDK bugs). -->
```System Info:
------------
chef_version=17.6.18
platform=ubuntu
platform_version=20.04
ruby=ruby 3.0.2p107 (2021-07-07 revision 0db68f0233) [x86_64-linux]
program_name=/usr/bin/chef-client
executable=/opt/chef-workstation/bin/chef-client```

## Platform Version
Ubuntu 20.04

## Replication Case
<!--- Tell us what steps to take to replicate your problem.  See [How to create a Minimal, Complete, and Verifiable example](https://stackoverflow.com/help/mcve)
for information on how to create a good replication case. -->
cookbook:
```chef_gem 'ruby-shadow' do
  action :install
end```

## Client Output
<!--- The relevant output of the chef-client run or a link to a gist of the entire run, if there is one.

The debug output (chef-client -l debug) may be useful, but please link to a gist, or truncate it. -->

see above

## Stacktrace
<!--- Please include the stacktrace.out output or link to a gist of it, if there is one. -->
```Generated at 2021-12-01 13:07:35 -0700
Mixlib::ShellOut::ShellCommandFailed: chef_gem[ruby-shadow] (baseline::local_users line 10) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
---- Begin output of /opt/chef-workstation/embedded/bin/gem install ruby-shadow -q --no-document -v "2.5.0" --source=https://rubygems.org ----
STDOUT: Building native extensions. This could take a while...
STDERR: WARNING:  You don't have /root/.chefdk/gem/ruby/3.0.0/bin in your PATH,
	  gem executables will not run.
ERROR:  Error installing ruby-shadow:
	ERROR: Failed to build gem native extension.

    current directory: /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0
/opt/chef-workstation/embedded/bin/ruby -I /opt/chef-workstation/embedded/lib/ruby/3.0.0 -r ./siteconf20211201-89170-rv4z38.rb extconf.rb
checking for getspent() in -lshadow... no
checking for getspent()... yes
checking for fgetspent()... yes
checking for setspent()... yes
checking for endspent()... yes
checking for lckpwdf()... yes
checking for ulckpwdf()... yes
checking for sgetspent()... yes
checking for ruby/io.h... yes
creating Makefile

current directory: /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0
make DESTDIR\= clean

current directory: /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0
make DESTDIR\=
compiling shadow/shadow.c
shadow/shadow.c: In function ‘rb_shadow_fgetspent’:
shadow/shadow.c:21:26: error: ‘struct rb_io_t’ has no member named ‘f’; did you mean ‘fd’?
   21 | #define file_ptr(x) (x)->f
      |                          ^
shadow/shadow.c:95:22: note: in expansion of macro ‘file_ptr’
   95 |   entry = fgetspent( file_ptr( (RFILE(file)->fptr) ) );
      |                      ^~~~~~~~
shadow/shadow.c: In function ‘rb_shadow_putspent’:
shadow/shadow.c:21:26: error: ‘struct rb_io_t’ has no member named ‘f’; did you mean ‘fd’?
   21 | #define file_ptr(x) (x)->f
      |                          ^
shadow/shadow.c:154:11: note: in expansion of macro ‘file_ptr’
  154 |   cfile = file_ptr( RFILE(file)->fptr );
      |           ^~~~~~~~
make: *** [Makefile:245: shadow.o] Error 1

make failed, exit code 2

Gem files will remain installed in /root/.chefdk/gem/ruby/3.0.0/gems/ruby-shadow-2.5.0 for inspection.
Results logged to /root/.chefdk/gem/ruby/3.0.0/extensions/x86_64-linux/3.0.0/ruby-shadow-2.5.0/gem_make.out
---- End output of /opt/chef-workstation/embedded/bin/gem install ruby-shadow -q --no-document -v "2.5.0" --source=https://rubygems.org ----
Ran /opt/chef-workstation/embedded/bin/gem install ruby-shadow -q --no-document -v "2.5.0" --source=https://rubygems.org returned 1
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/mixlib-shellout-3.2.5/lib/mixlib/shellout.rb:300:in `invalid!'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/mixlib-shellout-3.2.5/lib/mixlib/shellout.rb:287:in `error!'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/mixlib-shellout-3.2.5/lib/mixlib/shellout/helper.rb:130:in `shell_out_compacted!'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/mixlib-shellout-3.2.5/lib/mixlib/shellout/helper.rb:54:in `shell_out!'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider/package/rubygems.rb:587:in `install_via_gem_command'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider/package/rubygems.rb:555:in `install_package'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider/package.rb:120:in `block (3 levels) in <class:Package>'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider/package.rb:269:in `multipackage_api_adapter'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider/package.rb:119:in `block (2 levels) in <class:Package>'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/mixin/why_run.rb:51:in `add_action'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider.rb:290:in `converge_by'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider/package.rb:118:in `block in <class:Package>'
(eval):2:in `block in action_install'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider.rb:301:in `instance_eval'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider.rb:301:in `compile_and_converge_action'
(eval):2:in `action_install'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/provider.rb:242:in `run_action'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource.rb:600:in `block in run_action'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource.rb:627:in `with_umask'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource.rb:599:in `run_action'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/runner.rb:74:in `run_action'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/runner.rb:108:in `block in run_all_actions'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/runner.rb:108:in `each'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/runner.rb:108:in `run_all_actions'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/runner.rb:132:in `block in converge'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource_collection/resource_list.rb:96:in `block in execute_each_resource'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource_collection/stepable_iterator.rb:114:in `call_iterator_block'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource_collection/stepable_iterator.rb:85:in `step'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource_collection/stepable_iterator.rb:103:in `iterate'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource_collection/stepable_iterator.rb:54:in `each_with_index'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/resource_collection/resource_list.rb:94:in `execute_each_resource'
/opt/chef-workstation/embedded/lib/ruby/3.0.0/forwardable.rb:238:in `execute_each_resource'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/runner.rb:130:in `converge'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/client.rb:686:in `block in converge'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/client.rb:681:in `catch'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/client.rb:681:in `converge'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/client.rb:705:in `converge_and_save'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/client.rb:285:in `run'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/application.rb:305:in `run_with_graceful_exit_option'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/application.rb:281:in `block in run_chef_client'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/local_mode.rb:42:in `with_server_connectivity'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/application.rb:264:in `run_chef_client'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/application/base.rb:352:in `run_application'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-17.6.18/lib/chef/application.rb:67:in `run'
/opt/chef-workstation/embedded/lib/ruby/gems/3.0.0/gems/chef-bin-17.6.18/bin/chef-client:25:in `<top (required)>'
/usr/bin/chef-client:376:in `load'
/usr/bin/chef-client:376:in `<main>'```

Uh, you already worked with the maintainer to fix that yesterday (which was good now we can pull it in properly), but there's nothing to do here.
The chef-workstation package should really be including that but that's needs to be cut against chef/chef-workstation.

Honestly, I wasn't expecting the package maintainer to reply since it had been so long since anything had been done. I'll go file the request in chef-workstation. Thanks!


Is it possible to move items between different versions of the TQVault ,if yes, how?

Yes of course, vault files are fully backwards compatible. Simply point the new version of TQVaultAE to the folder containing your old vault files and they'll simply work!


Input parser is not working correctly for dates

Describe the bug
When typing '1987-05-30', it results on 05/29/1987. Incorrect
When selecting May 30 1987 from the datepicker, it results on 05/30/1987. Correct
When typing 'May 30, 1987', it results on 05/30/1987. Correct
This happens when the days are bigger than 9 only (ie two digit dates)
To Reproduce
Steps to reproduce the behavior:

Go to https://reactdatepicker.com/
Type '1987-05-30'
Press return/enter
See error '05/29/1987'

Expected behavior
Should show '05/30/1987'
Screen Recording
https://github.com/Hacker0x01/react-datepicker/assets/8929/6d64196b-b6e0-4007-b534-f9e98f12adda
Desktop (please complete the following information):

OS: MacOs 14.4.1
Browser Chrome
Version 125.0.6422.78 (Official Build) (arm64)

Additional context
I reproduced in different computers also

It's working for me

I don't know your time zone, but it fails in our CircleCI build with our Capybara tests using PST time zone, and also fails in my local computer with GMT-3 as you see in the video


Fig 49, if want will need .csv file

Allan, do you have a copy of the "HakeAssessHistory.csv"?  I see it is called in the AssessmentHistoryPlot.R code to make Fig 49.  That is if we even want to include that figure...

It is in the data folder. However, you may not be able to get it since our
forks are separate. You would have to merge mine, which I'm not willing to
try out with less than 24 hours left.  Can you wait until tomorrow morning
after Chris merges it all? If not, I can send you the pdf or we can figure
out a way to sync.
On Sun, Feb 7, 2016 at 10:48 PM, aaronmberger<EMAIL_ADDRESS>wrote:

Allan, do you have a copy of the "HakeAssessHistory.csv"? I see it is
called in the AssessmentHistoryPlot.R code to make Fig 49. That is if we
even want to include that figure...
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/76.

I changed the name of the file and fixed an error in the plot legend, so I think it's finished.


export ESM modules with rollup

Alternative approach to ESM modules using Rollup as dev dependency

This PR looks good. I would prefer this over my PR!


:heavy_minus_sign: remove unused dependencies. Closes #272

request
request-promise

request first appeared as part of initial commit but it seems to be
left-over from non commited implementation:
https://git.io/JvgNV
request-promise was added in different PR but it seems without related
code:
https://git.io/JvgN6
/cc @naknode
Thanks!

nice, less is less


Incorrect Version in Release Binary

Thought it was kinda odd, is this intended? I really doubt it is.
Platform: Linux AMD64 target, Windows 10 with WSL 2.3.24, Ubuntu 22.04 LTS Jammy
Test Command (linux-amd64)
curl -L -o chatgpt https://github.com/kardolus/chatgpt-cli/releases/download/v1.7.4/chatgpt-linux-amd64 && chmod +x chatgpt && sudo mv chatgpt /usr/local/bin/ && chatgpt --version

Expected
Something with v1.7.4 in it.
Real Output
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 9580k  100 9580k    0     0  14.2M      0 --:--:-- --:--:-- --:--:-- 14.2M
ChatGPT CLI version v1.7.3 (commit 5121c3563bfb74bf7e35df6ce492d85bf12e1e0d)

If I install with HomeBrew (which I'm thinking might be the better option at this point), the version is correct.
$ brew tap kardolus/chatgpt-cli && brew install chatgpt-cli
$ chatgpt -v
ChatGPT CLI version v1.7.4 (homebrew)

It is the correct version, however, I created the binaries before bumping the release tag. Whoops!
5121c35  - (HEAD, tag: v1.7.4)

Will be more mindful next time!

Fixed this as part of this release: https://github.com/kardolus/chatgpt-cli/releases/tag/v1.7.5


purpose of use of "? super T"

Description
I was very confused about the use of "? super T" as I watched the lecture. Take the case in the screenshot as an example, I think prof expained that anything inside T can be used for testing here. I understand this, like people can use attributes of T inherited from parent class for testing, but I don't see why we need to write "? super T" instead of just "T". Since anything super T is already inside T, T will by its nature inherit any attributes from its parent class/interface, right? Since anything super T is already inside T, then why do you still specify the "? super T" then? Wouldn't just use "T" achieve the same purpose?
Topic:
Use of super in generics
Screenshots

I think it's because if u use  then you can't put any other superclasses when you specify (Predicate pred)
so  would allow a predicate that tests for T and its subclasses (not allowing its superclasses) while <? super T> would allow for a predicate to test for T and its superclasses

<? super T> is the lower-bounded wildcard. For example, when T is a circle, the Predicate<T> can take in Predicate<Circle>, while Predicate<? super T> can take in its supertypes such as things like Predicate<Shape> and Predicate<Object>. I think you may have mixed it up with the upper-bounded wildcard <? extends T> instead? Hope this helps!

 means any class that is more super than T (Classes less specific than T)

 means any class that extends class T (Classes more specific than T)


Magic links do no longer work from<EMAIL_ADDRESS>

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
After upgrading from<EMAIL_ADDRESS>to 2.39.1 or 2.39.2 magic email links no longer work. The emails arrive correctly, but when you click on the link you are not logged in. Reverting to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>makes the links work again.
Expected behavior
Magic email links should just work when you click on them
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: Windows 11
Browser (if applies) Edge 120.0.2210.91
Version of supabase-js: 2.39.2
Version of Node.js: 18.12.1

Additional context
It seems this issue is related to supabase/gotrue-js as when I do not also revert that to 2.58.0 the issue persists.

Just for some more context. A fresh install of 2.39.2 on Mac OS 14 using Safari does indeed work. Do you happen to have an error message after you click on the link. That would be helpful to track the issue down.

@thetechguyseth I didn't see any error messages no. I'll try to see if I can come up with a small reproduction example that I can share. It may take a few days however before I can find some time to do it.

working fine for me, usually the culprit is

@farzd I guess the site URL configuration is not the issue as it has not changed. The links were working previously with the old package versions, then broke after the upgrade and are working fine again when I revert those two packages.

I've spent quite a bit of time now trying to narrow this down. I can reproduce the problem both in my production environment as well as in a local development setting.
Strangely when I slowly remove bits of the application code, the problem at some point disappears and doesn't come back when I revert the changes made. I am starting to suspect this is some kind of weird build order/caching issue in Vite or something.
At this point in time I have not succeeded in isolating it any further. I am closing the issue for now as it is unclear where exactly the problem originates and it cannot be consistently reproduced.


Screenrecord with multiple devices connected

Hi guys!
I have updated scrcpy to 2.7 recently (previously i had quite old version, 2.1 or so, don't remember actually).
And I can't figure out how to record screen when i have several devices connected
Previously I had automated script for this, where I used both -s  key and --record. It worked perfectly fine, but now it just share screen and thats it:
% /usr/local/bin/scrcpy
 -s RZCT40VW82D --no-playback --no-window --record=/Applications/Screen_records/2024-10-21__10-16-05.mp4
scrcpy 2.7 <https://github.com/Genymobile/scrcpy>
INFO: ADB device found:
INFO:     -->   (usb)  RZCT40VW82D                     device  SM_A536B
/usr/local/Cellar/scrcpy/2.7/share/scrcpy/scrcpy-server: 1 file pushed, 0 skipped. 42.8 MB/s (71200 bytes in 0.002s)
[server] INFO: Device: [samsung] samsung SM-A536B (Android 14)
INFO: Renderer: metal
INFO: Texture: 1080x2400
^Z
zsh: suspended  /usr/local/bin/scrcpy
zsh: command not found: -s

Is it a bug or you dropped this ability intentionally?

zsh: suspended  /usr/local/bin/scrcpy zsh: command not found: -s

You have a new line between /usr/local/bin/scrcpy and the remaining of the command, so it executes /usr/local/bin/scrcpy, then -s …, which does not exist.

oh, you're absolutely right! thank you very much


Increase timeout on test_dynamic_timer_clear_then_set_timer

This test is somewhat flaky, and every time it fails it seems like it is because it times out (example). Since the timers being set are all watermark based, this doesn't indicate some sort of bad delay and bumping the timeout shouldn't do any harm.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Rerunning precommit since failure is a different known flake apache_beam.options.pipeline_options_test.PipelineOptionsTest.test_display_data (fixed this morning in #25007) - https://ci-beam.apache.org/job/beam_PreCommit_Python_Commit/26591/testReport/apache_beam.options.pipeline_options_test/PipelineOptionsTest/test_display_data/

Run Python PreCommit


Components on inactive game objects can be initialized twice

Hello
I don't think this is expected behaviour. To reproduce:

Programatically add an inactive game object to the current scene
Notice that all components of the game object that implement ICmpInitializable have their OnInit methods called.
Now activate the game object. All initializable components will be initialized again.

I guess the solution to this, if its not intended behaviour, is to check whether the new game object is active before calling OnActivate in Scene.OnGameObjectAdded. We probably won't make that change locally as we can't be 100% sure what else it will break.

Thanks for reporting this.
I was able to reproduce this behavior and fixed this in my recent commit. There is also a new test to check for this behavior and similar cases.
As for accidentally breaking things, the ICmpInitializable interface is tested quite thoroughly, so we're relatively safe - but in your specific case, I would probably not touch it until after release. Might be not worth it :)


Mention only users who are part of the repo's organization

Hi, I'd like to bring this project up in one of our next project meetings. However, one of the concerns I know we'll have is the fact that we get PRs from the community and the reviewer is usually an expert in that field.  When someone has had several PRs approved they are added to the contributors group and become one of the potential reviewers.
Is it possible to have the mention-bot pull the names from this team rather than just from anyone who touched the file?

This should be pretty easy to hack into the existing code if you are so inclined.
However, I would warn against doing that. This is going to spam the hell out of all the people in that contributor list as they are going to be automatically subscribed to the thread. I would suggest you to try it as is and if it doesn't work, then modify the script to do it.

I'm not sure I understand correctly, but it seems to me that the title of this issue doesn't match the description.  The title contains the word "only", which suggests that this issue is about avoiding mentioning users outside the repository's organization, however that feature is already implemented and configurable via the requiredOrgs configuration parameter.
However the question "Is it possible to have the mention-bot pull the names from this team rather than just from anyone who touched the file?" sounds like it's not about avoiding mentioning users outside the organization, but instead about ensuring that all users within a team are mentioned.  For example, if user A is in the team but was not included via alwaysNotifyForPaths or fallbackNotifyForPaths or findPotentialReviewers, it sounds like you still want user A to be mentioned anyway - is that right?

@aspiers I actually think both you and @vjeux may have misinterpreted what @RandomlyKnighted tried to say. The initial comment is a bit confusing indeed, but after reading it a few times what I understand is that he wants to restrict mentions to just members of the organization (either owners or contributors), since they have an explicit process to onboard frequent contributors from the community into the org. It seems that you guys think he wants to mention all users in the contributors team, but as I see it it's more about using that list as a filter to rule out code authors that aren't on that list. Essentially choosing the potential reviewers using the same method as presently, but picking from a curated pool of trusted reviewers rather than all github users.
I not really fond of that idea though, since I believe encouraging members of the community to become more involved (and eventually taking up more active roles in the project) is a good thing, and ensures that human error, forgetfulness or mere availability lows (in the process of adding people to the manually curated contributors team) don't prevent people to contribute who would otherwise get prompted to do so. Automated processes like MentionBot can be the catalyst to keep projects alive and bustling as people naturally flow in and out of the maintainership role. (As a side-note, I think the Wikipedia community made a good choice in this regard, by considering the administrator role a "barrier removal" more than a "role attribution", since the latter view often leads to calls for more stringent criteria, periodic reconfirmation votes, activity minimums to retain the admin tools, etc.)
If this is ever implemented, I'd strongly suggest it to be an opt-in feature (i.e. off by default).

@waldyrious thanks for the comment, now I understand it better :)

happy to help, and sorry for the long comment (I didn't realize how long it was at the time 😅)

@waldyrious You're probably right, but this would still confirm my assertion that the PR title is completely at odds with the intent of the PR. If the intent is as you describe, then the title should be something like "Mention only users who are part of a team", because a GitHub team would be the most natural way to define the list of users which @RandomlyKnighted refers to as "contributors".  However it would be nice if @RandomlyKnighted could clarify here :-)

Yeah, the opening comment is definitely a little unclear :)

Thanks, yes much better!


New version: LibreWolf.LibreWolf version 119.0-5

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Domain


How to use multiple GPU setup for LightGCN ?

Hey @satsaras , This is notebook where implemented multiple GPU setup for LightGCN. I have mentioned link below this.
Multiple GPU setup for lightGCN
I hope , this setup of code help you out.

@satsaras ,
Here snapshot of code for implementation for multiple GPU.
`import torch
import torch.nn as nn
import torch.nn.functional as F
import time
import sys
assert(len(sys.argv) == 2)
ngpu = int(sys.argv[1])
n_hidden = 128
n_layers = 100
n_repeat = 100
bz = 256
class Model(nn.Module):
def init(self):
super(Model, self).init()
self.fc = nn.Linear(n_hidden, n_hidden)
def forward(self, x):
    for _ in range(n_layers):
        x = self.fc(x)
        x = F.relu(x)
    torch.cuda.synchronize()
    return x

devices = [torch.device('cuda:{}'.format(gpu)) for gpu in range(ngpu)]
inputs = [torch.randn(bz, n_hidden).to(dev) for dev in devices]
model = Model().to(devices[0])
if ngpu > 1:
model = nn.parallel.replicate(model, devices)
start = time.time()
with torch.no_grad():
for _ in range(n_repeat):
if ngpu > 1:
output = nn.parallel.parallel_apply(model, inputs)
else:
output = model(inputs[0])
end = time.time()
print((end - start) / n_repeat * 1000)`


Fix Android Build by removing jniLibs.srcDirs from android/build.gradle

I'm not 100% sure if this is something that affects everyone and should be merged into the actual repo or if it is due to some other libraries we are using, so I'd be cautious before you merge it.

can this be merged and released ? thanks 🙏

@max-zu when testing against react-native-sodium-example (plain react-native app with the only dependency react-native-sodium added) this PR leads to libsdoium.so not being included in the .apk file. Please do some investigation on your project/dependencies and take a look at the example application mentioned. Until the source of the error is found I cannot consider this one as a bug in react-native-sodium package.

@lyubo what version of react-native did you use? I started to experience this issue after i updated RN in my project to 0.64

@lyubo are there any updates regarding this issue ?

@lyubo is it possible to merge this PR ?

@lyubo Could you merge this PR please?

PR was merged but not released


fix: make tag links relative to the root of the page

Previously when we clicked on a tag link, it would append the path to where we currently were, e.g. clicking "tag" on a post in the /posts/ lists, would send us to /posts/tags/tag instead of /tags/tag.
This fixes #7.

I got a bit smarter overnight. 😅
Until now I had my fork checked out in a different folder, but seeing as it's a submodule of my website's Git repo anyway, I simply added my fork as another remote.
git remote add git@github.com:rluetzner/hugo-theme-readable.git
# This is probably not how you do it, but I made it work this way.
git branch fork-branch
git fetch fork fork-branch

This makes it easier to verify the changes using my website and I no longer have to copy code around.

Good find. Tested and functioning. Might want to add the remote tactic to the docs somehow as well, maybe README or in /docs/development.md (would be a new file)

Will do. Personally I think that a separate development.md is best.


ACI:82: Update check your email code validity content from 15 minutes to 2 hours

What?

Update check your email code validity content from 15 minutes to 2 hours
This is to reflect the backend changes in pr

Why?

To be consistent with the change in code validity implemented in the backend
To provide users with accurate content regarding the changes

Related PRs
Backend changes

Hey @ayoshebby. We haven't changed the backend for this (per ACI-76) so closing the PR. The 15 min -> 2 hour change only applies to 'frontend' journeys


Upload should only upload folder contents

🤦

Introduced this bug in my pluto publish mvt stuff when I changed the inputs to s3.upload_folder


build with make file in main path possible?

I usaly build by executeing my own makefile that is in the main path as a separate file called "makefile" it does some special things like converting my svg images automaticaly to pdf and such things so it's not possible to do that with the custom toolchain and defining a latexmk -pdf  command or so.
I wold like to know if its possible to execute that makefile somehow with Atom-LaTeX?
Regards, Novski

It would be the same way you set up a custom toolchain.
You could either set it up per project with a .latexcgf or globally.


Passing follow=True to read_namespaced_pod_log makes it never return

I'm trying to stream logs out of a running pod.
For a pod named 'test' running in the default namespace,
api = client.CoreV1Api()
api.read_namespaced_pod_log("test", "default")

returns the logs so far. But,
api = client.CoreV1Api()
api.read_namespaced_pod_log("test", "default", follow=True)

just never returns. I can't use it with watch.Watch either since that seems to be for a very different purpose.

It looks like I can pass _preload_content=False and get a stream that works - however this is unituitive and doesn't seem documented in the obvious places.

Maybe we can add a follow method to Watch class to set "follow" flag instead of "watch" and stream out strings instead of API objects. but _preload_content need to be documented anyway.

+1 on adding it to Watch (or watch could special case this one?)
On Wed, May 3, 2017 at 11:26 AM, Mehdy Bohlool<EMAIL_ADDRESS>wrote:

Maybe we can add a follow method to Watch class to set "follow" flag
instead of "watch" and stream out strings instead of API objects. but
_preload_content need to be documented anyway.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/client-python/issues/199#issuecomment-298995386,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB23vyGNN2NrMePmgLy81a5qAVLm70Rks5r2Ma9gaJpZM4NIDE6
.

--
Yuvi Panda T
http://yuvi.in/blog

I am just looking for somebody to do this. If nobody took it and I got free time, I will do it.

Watch class can also use inspect.getargspec to see if there is Watch or Follow parameter and then stream the result accordingly.

It would be nice if you can contribute. Here are the things need to be done:

_preload_content need to be documented somewhere
Add a section to Watch class to use inspect.getargspec or some other methods to determine if the passed function has Watch or Follow flag and set the appropriate flag.
Test this on read_namespaced_pod_log to see if it is working. The return type should be set to string if it is not already that (look like read_namespaced_pod_log return type is str so we should be fine there).

Let me know if you need more. I will be happy to review the code when it is ready.

I  added "_preload_content=False" based on the official example , it still has no output.  How can i get the the stream output

Is there any example for make a log following? I can't following the pod log with code
v1.read_namespaced_pod_log(name='test',namespace='default',follow=True,_preload_content=False)
but
kubectl logs -f test
is worked

FYI, read_namespaced_pod_log(self, name, namespace, **kwargs) is the signature of the function, so one cannot really use inspect to see if the function is taking watch or follow.

I made a PR here: https://github.com/kubernetes-client/python-base/pull/93

It works well, but I do not have an idea how can this API support resuming. So if the HTTP connection fails, it seems there is no real way to resume following the log  from where you stopped. The only approach I can see is to require timestamp to be added to every line and then use that?

Yeah for me neither
client.CoreV1Api.read_namespaced_pod_log('my_pod_name', 'default', follow=True, _preload_content=False).read()
nor
client.CoreV1Api.read_namespaced_pod_log('my_pod_name', 'default', follow=True) are working. They both just indefinitely hang or return the logs if the pod is done execution.
@yuvipanda did you do something different from the above?

Ah so what I did initially was close but the temporary fix that worked for me was -
for line in client.CoreV1Api.read_namespaced_pod_log('<pod_name>', 'default', follow=True, _preload_content=False).stream():
    print(line)

@roycaihw Why was this labeled with "help wanted"? There is already a PR for this.

@mitar I was doing a blind conversion from our custom help-needed label to the generic "help wanted" label to make the issues more visible to new contributors: https://github.com/kubernetes-client/python/labels/help-needed. Sorry I didn't look into the status in each issue :)
Thanks for fixing this! I see Yu was already reviewing that PR so I will leave it to him. Please let me know if you need anything from me.

Ah, that makes it clearer. Yes, that PR was reviewed, I have to figure how to make tests though. I looked briefly but it wasn't exactly clear to me.
If anyone here has any spare cycles, it would be helpful if you could help me improving that PR by adding tests. I can add you to my fork repo to contribute if you want.

I am looking through the tests and I may be able to help. Let me work through the watch routines and I'll let you know. @mitar

Watch may not be the correct way to handle live streaming logs.
when 'follow' is set to True, the api would return a HTTPResponse object, and there is no non-blocking api exposed by this object, which means that when client is blocking on reading logs, the connection could only be 'closed' by server (or the client process exited).
For non-blocking reading, I had to write a wrapper to use select.poll(fileno()) to make logs non blocking. (similar to the example for exec which is wrapped by a websocket stream client)
Hope I am not making mistakes here, and i will try to write a PR for this.

@zq-david-wang There are two ways here. One is if you want to handle streaming yourself, but there is also Watch class which you can use to do it for you, in a blocking way. And that one I made a PR for.
So maybe there is something more one could do to improve things here for cases where you do not want to use Watch.

I have added tests to https://github.com/kubernetes-client/python-base/pull/93.
I also opened https://github.com/kubernetes-client/python-base/issues/117. I really think tests should be using proper Kubernetes cluster inside CI and not mocked messages. Because now the author of the code both writes the code and test messages, but there is really not any assurance that test messages align with real Kubernetes messages.

Having trouble using this, am i missing something?
v1 = client.CoreV1Api() w = watch.Watch() for e in w.stream(v1.read_namespaced_pod_log, name=pod, namespace=namespace, follow=True, tail_lines=1, limit_bytes=560, _preload_content=False):    print(e)
Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/usr/local/lib/python3.7/site-packages/kubernetes/watch/watch.py", line 132, in stream     resp = func(*args, **kwargs)   File "/usr/local/lib/python3.7/site-packages/kubernetes/client/apis/core_v1_api.py", line 18538, in read_namespaced_pod_log     (data) = self.read_namespaced_pod_log_with_http_info(name, namespace, **kwargs)   File "/usr/local/lib/python3.7/site-packages/kubernetes/client/apis/core_v1_api.py", line 18576, in read_namespaced_pod_log_with_http_info     " to method read_namespaced_pod_log" % key TypeError: Got an unexpected keyword argument 'watch' to method read_namespaced_pod_log
Posted on stackoverflow if anyone wanted to answer!

Ah so what I did initially was close but the temporary fix that worked for me was -
for line in client.CoreV1Api.read_namespaced_pod_log('<pod_name>', 'default', follow=True, _preload_content=False).stream():
    print(line)

I needed to decode the bytes of the output:
                    for line in v1.read_namespaced_pod_log(pod.metadata.name,'cdist',follow = True, _preload_content=False).stream():
                        print(line.decode('utf-8'))


Multivariate demo reconstruction window error

I tried to test the multivariate tutorial, the reconstruction part runs fine for windows of 100, but when I changed the hyperparameter values to 200 I get an error, I also tried to make the change in the lstm_autoencoder.json and got same error. See both below.
What am I missing?
------Hyper parameter changed to, and the error comes from keras
https://github.com/sintel-dev/Orion/blob/master/tutorials/Multivariate_Input.ipynb
hyperparameters = {
    "mlprimitives.custom.timeseries_preprocessing.rolling_window_sequences#1": {
        "window_size": 200,
        "target_column": 0
    },
    'keras.Sequential.LSTMSeq2Seq#1': {
        'epochs': 1,
        'verbose': True,
        'input_shape': [200, 25],
        'target_shape': [200, 1],
    }
}

---------------- same error when I also changed lstm_autoencoder.json to 200
(orionGPU3) yaser@yaserubuntu:/media/ydrive/time_series/Orion$ python Multivariate_Input.py
/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/sklearn/utils/validation.py:37: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  LARGE_SPARSE_SUPPORTED = LooseVersion(scipy_version) >= '0.14.0'
(10149, 26)
Using TensorFlow backend.
YYprocessing lstm_autoencoder
Exception caught fitting MLBlock keras.Sequential.LSTMSeq2Seq#1
Traceback (most recent call last):
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlblocks/mlpipeline.py", line 549, in _fit_block
    block.fit(**fit_args)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlblocks/mlblock.py", line 302, in fit
    getattr(self.instance, self.fit_method)(**fit_kwargs)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py", line 99, in fit
    self.model = self._build_model(**kwargs)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py", line 54, in _build_model
    model.add(built_layer)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/engine/sequential.py", line 182, in add
    output_tensor = layer(self.outputs[0])
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/engine/base_layer.py", line 506, in __call__
    output_shape = self.compute_output_shape(input_shape)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/layers/core.py", line 399, in compute_output_shape
    input_shape[1:], self.target_shape)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/layers/core.py", line 387, in _fix_unknown_dimension
    raise ValueError(msg)
ValueError: total size of new array must be unchanged
Traceback (most recent call last):
  File "Multivariate_Input.py", line 86, in <module>
    orion.fit(data)
  File "/media/ydrive/time_series/Orion/orion/core.py", line 86, in fit
    self._mlpipeline.fit(data, **kwargs)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlblocks/mlpipeline.py", line 649, in fit
    self._fit_block(block, block_name, context)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlblocks/mlpipeline.py", line 549, in _fit_block
    block.fit(**fit_args)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlblocks/mlblock.py", line 302, in fit
    getattr(self.instance, self.fit_method)(**fit_kwargs)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py", line 99, in fit
    self.model = self._build_model(**kwargs)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/mlprimitives/adapters/keras.py", line 54, in _build_model
    model.add(built_layer)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/engine/sequential.py", line 182, in add
    output_tensor = layer(self.outputs[0])
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/engine/base_layer.py", line 506, in __call__
    output_shape = self.compute_output_shape(input_shape)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/layers/core.py", line 399, in compute_output_shape
    input_shape[1:], self.target_shape)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/keras/layers/core.py", line 387, in _fix_unknown_dimension
    raise ValueError(msg)
ValueError: total size of new array must be unchanged
(orionGPU3) yaser@yaserubuntu:/media/ydrive/time_series/Orion$

Thank you @yaseryacoob for pointing this issue!
There is an incompatibility in building the layers of the autoencoders, working on this right now!

Fixed in PR #300


Can't copy email HTML after adding multi line email footer

I am trying to add this in the footer:
Copyright © {{right_now.year}}  {{location.name}}, All rights reserved.
 Want to change how you receive these emails?
 You can [unsubscribe from this list](https://someurl)

After adding this, there is an error after I try to copy HTML.

If I remove this footer text then it just works fine....

I think it's because of copying the text from web pages or text editors. Because on copy it also copies the styles. I would suggest pasting it to Notepad/Apple Notes and then copying it from there.


Excluding /cdn-cgi/challenge-platform/ not enough to unblock JS challenge

Existing documentation URL(s)
https://developers.cloudflare.com/waf/reference/cloudflare-challenges/#custom-content-security-policy-not-supported
What changes are you suggesting?
Trying to use the current workaround in Safari

If you are setting a CSP using Transform Rules for your entire website, you should exclude URI paths starting with /cdn-cgi/challenge-platform/ in the rule expression to avoid issues with challenges.

I got the following error

EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self' 'unsafe-inline' https://static.cloudflareinsights.com".

After adding 'unsafe-eval' I still got white screen, which I resolved by using worker-src 'self' blob:;.
After doing that, I got

Please unblock challenges.cloudflare.com to proceed.

In sum, to get the challenge page to display in Safari, script-src 'self' 'unsafe-eval' https://challenges.cloudflare.com; frame-src 'self' https://challenges.cloudflare.com; worker-src 'self' blob:; are required on the protected page at the very least.
Note that excluding /cdn-cgi/challenge-platform/ doesn’t work because the above-mentioned CSP directives have to be set on the protected page rather than the challenge page.
Additional information
No response

Hi @nisbet-hubbard
I checked internally and we couldn't identify any issues in the Cloudflare-issued challenge pages (or in pages protected by Cloudflare challenges) from what you described.
The error seems related to a custom CSP you defined, either in your origin server or through Transform Rules.
Here's a list of possible troubleshooting steps:

Make sure you disable any privacy-related browser extensions, since they're not compatible with Cloudflare challenges.
Have you tried other browsers besides Safari? Do you get the same errors?
Can you provide a public example page with the error you described, or can you set up a small self-contained example that shows the issue?
Are you using other Cloudflare products that require their own specific CSP directives?

If you keep having issues, we recommend that you contact Cloudflare Support or reach out to the Cloudflare Community (if you have a Free plan), since this repository only deals with documentation.


Analytics Dashboard Widget Timeout Exceeding before loading data.

In the loading the widgets in APIM-Analytics dashboard-3.1.0, the "Widget.js" level timeout was exceeded. Please refer to the below image.

Affected Product Version:
3.1.0

Fixed with https://github.com/wso2/carbon-dashboards/commit/f8ec4a3d6f842154420c01914886d42334e106fc

This issue needs to be fixed at the base widget level at https://github.com/wso2/carbon-dashboards/blob/master/base-widget/src/Widget.jsx#L189


Export query implementation

I've implemented query exports, as it's something I needed for a project. Just tossing it back to you in case you're interested.
This adds methods z.export.asArray(exportObject[, exportConfig] , callback) and z.export.asStream(exportObject[, exportConfig], callback). They take either a ZOQL string or an export zObject as the first param and an overriding config object for the export watcher (interval, maxAttempts) as an optional second argument. I just fixed and wired up the functions that already existed to monitor an export for completion, download it from the REST API, and delete it when it has been consumed.
I'm not sure if extending z.export sits exactly right with me, as the asArray and asStream methods are obviously not part of the SOAP API. Nevertheless, they're specifically tied to the export zObject type, so arranging them anywhere else seemed needlessly complicated.
There are minimal changes elsewhere; mainly I changed some console.log invocations to use the global logger.

Awesome! Thanks. I'll take a look.

Looks good. I agree that extending z.export does sit a little funky. However the Zuora api is a little funky to begin with. You're welcome to revisit this and change the methods later.

Boy, funky is an understatement. I really appreciate your work on this module though, it saved me a ton of time.
Another thought I had was that it would be preferable for asStream to return a stream instead of passing it to a callback, but I didn't have time to deal with the plumbing that would require. If I have the opportunity, I might tackle that in the future.
Thanks again!


Added code for Flood Fill Graph Problem Statement

Issue-570
https://github.com/smriti596/LearnCPP/tree/issue-570
Proposed Changes

Added code for Flood Fill Problem Statement

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the README.
[x] 👀 I have read the CONTRIBUTING document.

not up to the mark


ex01_publish gets compile error.

Thanks Rémi  for this really this really useful library.  I am trying to run the simple example code
on a arduino and get compiler error
testTelemetry:4: error: 'Telemetry' does not name a type
Telemetry TM;
Any suggestions as to what the problem is?
Thanks.

Hi, thanks!
Have you imported the library into the arduino library folder ?
The instructions are indicated here.
Let me know if that solves your issue

Hi Rémi,
Thanks for the quick response. The library is imported (if I search for Telemetry via Sketch->Include Library->Manager Libaries, I see Telemetry Version unknown INSTALLED). Also I've checked and in my libraries folder is a folder named Telemetry with the 7 .h files and 6 .cpp files.
I noticed that the files Telemetry.h and Telemetry.cpp use Window newline CRLF while the other files use unix LF. Also Windows could not extract from the .zip file giving error "The Compressed Folder ... is invalid. However 7-zip has no problem and in fact I used the IDE Sketch->Include Library->Add .ZIP Library to install it.
I'm using the Aduino IDE version 1.6.12 under Win 7 64bit.  I tried compiling ex01_publish and got the errors attached. ErrorMessage.txt
Also if I select Sketch->Include Library->Telemetry these extra lines are added  to the code:
#include <crc16.h> #include <dictionnary.h> #include <framing.h> #include <Telemetry.h> #include <telemetry_core.h> #include <telemetry_utils.h> #include <transport.h>
but with these the compile fails with the same error.

Allright I found the issue, I simply forgot to update this example! It is using the former interface.
I have updated it, now you don't need to instanciate a Telemetry object the library already does it for you.
See the updated example here.
It should fix your issue, let me know. And thank you for using Telemetry :)
Cheers

Hi Rémi,  No luck :-) I am now getting  errors in transport.h  Error.txt which I think comes from HardwareSerial.h not being correct for my AVR (adafruit feather32u4). I have installed the Adafruit boards from their site, so maybe I need to go to their forum for help. If I tell the IDE to compile for other Arduino boards I get no error. Also if I comment out #include HardwareSerial.h" in transport.h  then I get these errors when I compile for the Arduino boards.
I searched the Adafruit arduino forum and no one seems to have tried your library. Could there be a bug in Adafruits board description? I appreciate any suggestions of what I might ask.
Thanks for your time. Cheers,
PS If this is useful to you: The Adafruit boards that fail as above are all 32u4 with USB Serial support. Adafruit boards like Trinket baased on Attiny85 have no serial support and give this error testTelemetry:8: error: 'Serial' was not declared in this scope Serial.begin(9600); and the Adafruit boards like pro-Trinkets which have Serial but no serial to USB compile with no errors.

I can reproduce this issue, there seems to be an inconsistency with the
arduino build system, because running an arduino example with Serial in it
runs fine. I'll investigate and let you know quickly ;)
Le 18 nov. 2016 5:07 AM, "bbkiwi"<EMAIL_ADDRESS>a écrit :

Hi Rémi, No luck :-) I am now getting errors in transport.h Error.txt
https://github.com/Overdrivr/Telemetry/files/598979/Error.txt which I
think comes from HardwareSerial.h not being correct for my AVR (adafruit
feather32u4). I have installed the Adafruit boards from their site, so
maybe I need to go to their forum for help. If I tell the IDE to compile
for other Arduino boards I get no error. Also if I comment out #include
HardwareSerial.h" in transport.h then I get these errors when I compile for
the Arduino boards.
I searched the Adafruit arduino forum and no one seems to have tried your
library. Could there be a bug in Adafruits board description? I appreciate
any suggestions of what I might ask.
Thanks for your time. Cheers,
PS If this is useful to you: The Adafruit boards that fail as above are
all 32u4 with USB Serial support. Adafruit boards like Trinket baased on
Attiny85 have no serial support and give this error testTelemetry:8:
error: 'Serial' was not declared in this scope Serial.begin(9600); and
the Adafruit boards like pro-Trinkets which have Serial but no serial to
USB compile with no errors.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Overdrivr/Telemetry/issues/60#issuecomment-261444227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPB1Vp1zvpbDU6dE6eUiTeR5YysrpM6ks5q_SSWgaJpZM4K0pGr
.

Adding #include "Arduino.h" to Telemetry.h or transport.h solves the compilation error, I am not sure how it could compile without it in the first place.
For a quick fix, transport.h should look like this
#ifndef TELEMETRY_TRANSPORT_HPP_
#define TELEMETRY_TRANSPORT_HPP_

#include "Arduino.h"
#include "HardwareSerial.h"

int32_t read(uint8_t * buf, uint32_t sizeToRead)
{
// ...

I will make the updates in the repository as well. Please leave this ticket open, I'll close it when it's merged. Thanks
PS: Let me know if all your problems are solved

Hi Rémi, Thanks it is working now! It will be very useful for me. Great work :-) I was mainly interested in using the indexed plots and I have tested it with an example based on your sin plot example. As long as I keep the rate down it works fine. I have found that I must disconnect before trying to upoad from the IDE otherwise I get and error from the IDE. Sometimes when I disconnect the view it gives an error message, but disconnects anyway. I also discovered that I must shut down the plot windows otherwise when too many are open the whole thing bogs down. So yes all my problems (at least concerning Telemetry :-) ) are solved.


add test for empty customHost

Please provide a description of this PR:

FYI https://github.com/istio/istio/pull/47126

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
276abe011af193d141f71f259684055bbf9c437a
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes

@ramaraochavali


It looks like X-Mailer can no longer be changed?

Was this feature removed?  In earlier versions of Nodemailer "X-Mailer" header could be changed or removed.

use xMailer: false option to disable this header


Fix deal creation with hash as argument

Fix a bug introduced in 78112f2 where it was impossible to create a deal
by just passing a hash to create. sanitize method was expecting
passed argument to respond to value which in case of hash was failing.

Other than this, looks got to me.


wrong param name for strictssl configuration

There is a typo in the name of the parameter for configuring strictssl. The option name is usestrictssl, but the used variable is option.conn_usestrictssl. This leads to the behavior, that the option --usestrictssl false is ignored.

Sorry for the late reply. I'll merge your pull request!


tvg-logo uses random local address instead of server IP

Please first read the documentation. Problems affecting the client, e.g. Plex or Emby should be reported there in the forum.
Describe the bug
xteve is randomly selecting an interface to use for tvg-logo
Server (please complete the following information):

OS Linux Ubuntu
Arch Intel 64
xTeVe Version and Build 2.2.0 Build: 0200

To Reproduce
Steps to reproduce the behavior:

Use xteve
Look at m3u

Expected behavior
The correct server IP should be selected, or the interface should be selectable via configuration
Screenshots
#EXTINF:0 channelID="x-ID.57" tvg-chno="305" tvg-name="SBS PopDesi" tvg-id="305" tvg-logo="http://<IP_ADDRESS>:34400/images/771d895da337f257b1a690df227a877c.png" group-title="FTA",SBS PopDesi
http://broadcast.example.com/stream/771d895da337f257b1a690df227a877c
#EXTINF:0 channelID="x-ID.58" tvg-chno="306" tvg-name="SBS Chill" tvg-id="306" tvg-logo="http://<IP_ADDRESS>:34400/images/251973bc29a99699d26fa4eb7d8b349e.png" group-title="FTA",SBS Chill
http://broadcast.example.com/stream/2452c05838b36ca2a82617d7d446b9df
#EXTINF:0 channelID="x-ID.59" tvg-chno="307" tvg-name="SBS PopAsia" tvg-id="307" tvg-logo="http://<IP_ADDRESS>:34400/images/02b046550afa6897c74b7b8169a63621.png" group-title="FTA",SBS PopAsia
http://broadcast.example.com/stream/463caa533fd8df97214a31f237a626de

Desktop (please complete the following information):

OS: N/A
Browser N/A
Version N/A

Additional context
Add any other context about the problem here.

I saw something similar once when installed on a Windows PC running Docker. It wanted to grab the WSL network address for some reason. I had to shut down all other NICs then reinstall.

Did you ever figure out a fix for this?  Same thing happened to me.  I believe it happens after the scheduled update.  I'm not sure how to fix, but the workaround temporarily is to go to the playlist source and update through the web interface.

Unfortunately no.

Did you ever figure out a fix for this? Same thing happened to me. I believe it happens after the scheduled update. I'm not sure how to fix, but the workaround temporarily is to go to the playlist source and update through the web interface.

There's a fork that allows you to enter the host IP. This prevents xTeVe from grabbing other network adapters. The dev reached out to me after a post on another forum. I've been running it for months now with no issue.
https://github.com/SCP002

There's a fork that allows you to enter the host IP. This prevents xTeVe from grabbing other network adapters. The dev reached out to me after a post on another forum. I've been running it for months now with no issue.

Can you enter the host name (rather than host IP)?

There's a fork that allows you to enter the host IP. This prevents xTeVe from grabbing other network adapters. The dev reached out to me after a post on another forum. I've been running it for months now with no issue.

@dbowles1975 Can you enter the host name (rather than host IP)?

Nope.

Yeah - that's been my biggest frustration - lack of support for SSL (via a reverse proxy) and correct configuration (or honouring) of a hostname


[CT-3034] [Bug] relative path in DBT_PROFILES_DIR fails

Is this a new bug in dbt-core?

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

Current Behavior
I set globally the environment variable DBT_PROFILES_DIR to ~/.dbt which breaks the execution:

But it works when I set the absolute path e.g. DBT_PROFILES_DIR=/root/.dbt.
Expected Behavior
I would expect that dbt resolves the relative path in environment variable DBT_PROFILES_DIR.
Steps To Reproduce

run export DBT_PROFILES_DIR=~/.dbt
run dbt run

Relevant log output
No response
Environment
- OS:Debian 11
- Python: 3.9.2
- dbt: 1.6.1

Which database adapter are you using with dbt?
other (mention it in "Additional Context")
Additional Context

dbt-databricks 1.6.1

Note: Found an issue with my test and adjusted the issue accordingly. Relative paths seem to not working anymore in 1.6.1. Before I used dbt version 1.4.6 where this worked out fine.

@leo-schick It looks like you're running dbt inside a Databricks notebook/workflow(?). I don't see this error while running dbt-core locally, including more recent versions. I'm wondering if this error is specific to running against DBFS? We've seen some other issues recently that fall into this category:

https://github.com/dbt-labs/dbt-core/issues/8608
https://github.com/dbt-labs/dbt-core/issues/7428

$ export DBT_PROFILES_DIR=~/.dbt
$ dbt run --profile sandbox-databricks
10:39:39  Running with dbt=1.6.1
10:39:40  Registered adapter: databricks=1.6.3
10:39:40  Found 1 model, 1 operation, 1 test, 0 sources, 0 exposures, 0 metrics, 686 macros, 0 groups, 0 semantic models
...

@jtcohen6 Yes this is running inside a Databricks Notebook

We're going to close this issue, and document it (along with other issues labeled dbfs) as a limitation of running dbt-core inside a Databricks notebook.

@jtcohen6 it's not related to Databricks - dbt simply doesn't support tilde for neither --project-dir nor --profiles-dir:
Error: Invalid value for '--project-dir': Path '~/.local' does not exist.
Error: Invalid value for '--profiles-dir': Path '~/.local' does not exist.

The path of course exists. Tested with dbt-core 1.7.2.


EC2 image syncing is very slow

Description:
EC2 image syncing is so slow in my local dev environment that it eventually times out the neo4j session. It eventually timed out for me after 6 minutes.
I believe this is because it's fetching every single AMI in every region, in each account, and that's hundreds of thousands of images. The call to describe images should probably use a filter, and should likely default to Owners=['self']
To Reproduce:
Sync with ec2:images enabled.
Logs:
{"asctime": "2021-06-17 06:48:11,603", "name": "cartography.sync", "levelname": "ERROR", "message": "Unhandled exception during sync stage 'aws'", "exc_info": "Traceback (most recent call last):\n  File \"/home/cartography/cartography/sync.py\", line 73, in run\n    stage_func(neo4j_session, config)\n  File \"/home/cartography/cartography/util.py\", line 59, in timed\n    result = method(*args, **kwargs)\n  File \"/home/cartography/cartography/intel/aws/__init__.py\", line 201, in start_aws_ingestion\n    requested_syncs,\n  File \"/home/cartography/cartography/intel/aws/__init__.py\", line 139, in _sync_multiple_accounts\n    aws_requested_syncs=aws_requested_syncs,  # Could be replaced later with per-account requested syncs\n  File \"/home/cartography/cartography/intel/aws/__init__.py\", line 57, in _sync_one_account\n    RESOURCE_FUNCTIONS[func_name](**sync_args)\n  File \"/home/cartography/cartography/util.py\", line 59, in timed\n    result = method(*args, **kwargs)\n  File \"/home/cartography/cartography/intel/aws/ec2/images.py\", line 82, in sync_ec2_images\n    load_images(neo4j_session, data, region, current_aws_account_id, update_tag)\n  File \"/home/cartography/cartography/util.py\", line 59, in timed\n    result = method(*args, **kwargs)\n  File \"/home/cartography/cartography/intel/aws/ec2/images.py\", line 61, in load_images\n    update_tag=update_tag,\n  File \"/usr/local/lib/python3.7/site-packages/neo4j/__init__.py\", line 503, in run\n    self._connection.fetch()\n  File \"/usr/local/lib/python3.7/site-packages/neobolt/direct.py\", line 419, in fetch\n    return self._fetch()\n  File \"/usr/local/lib/python3.7/site-packages/neobolt/direct.py\", line 440, in _fetch\n    self._receive()\n  File \"/usr/local/lib/python3.7/site-packages/neobolt/direct.py\", line 487, in _receive\n    raise self.Error(message)\nneobolt.exceptions.ServiceUnavailable: Failed to read from defunct connection Address(host='neo4j', port=7687) (Address(host='<IP_ADDRESS>', port=7687))"}
{"asctime": "2021-06-17 06:48:12,056", "name": "__main__", "levelname": "ERROR", "message": "Failed to sync", "exc_info": "Traceback (most recent call last):\n  File \"custom_sync.py\", line 271, in main\n    cartography.sync.run_with_config(sync, config)\n  File \"/home/cartography/cartography/sync.py\", line 150, in run_with_config\n    return sync.run(neo4j_driver, config)\n  File \"/home/cartography/cartography/sync.py\", line 73, in run\n    stage_func(neo4j_session, config)\n  File \"/home/cartography/cartography/util.py\", line 59, in timed\n    result = method(*args, **kwargs)\n  File \"/home/cartography/cartography/intel/aws/__init__.py\", line 201, in start_aws_ingestion\n    requested_syncs,\n  File \"/home/cartography/cartography/intel/aws/__init__.py\", line 139, in _sync_multiple_accounts\n    aws_requested_syncs=aws_requested_syncs,  # Could be replaced later with per-account requested syncs\n  File \"/home/cartography/cartography/intel/aws/__init__.py\", line 57, in _sync_one_account\n    RESOURCE_FUNCTIONS[func_name](**sync_args)\n  File \"/home/cartography/cartography/util.py\", line 59, in timed\n    result = method(*args, **kwargs)\n  File \"/home/cartography/cartography/intel/aws/ec2/images.py\", line 82, in sync_ec2_images\n    load_images(neo4j_session, data, region, current_aws_account_id, update_tag)\n  File \"/home/cartography/cartography/util.py\", line 59, in timed\n    result = method(*args, **kwargs)\n  File \"/home/cartography/cartography/intel/aws/ec2/images.py\", line 61, in load_images\n    update_tag=update_tag,\n  File \"/usr/local/lib/python3.7/site-packages/neo4j/__init__.py\", line 503, in run\n    self._connection.fetch()\n  File \"/usr/local/lib/python3.7/site-packages/neobolt/direct.py\", line 419, in fetch\n    return self._fetch()\n  File \"/usr/local/lib/python3.7/site-packages/neobolt/direct.py\", line 440, in _fetch\n    self._receive()\n  File \"/usr/local/lib/python3.7/site-packages/neobolt/direct.py\", line 487, in _receive\n    raise self.Error(message)\nneobolt.exceptions.ServiceUnavailable: Failed to read from defunct connection Address(host='neo4j', port=7687) (Address(host='<IP_ADDRESS>', port=7687))"}

Please complete the following information::

Cartography release version or commit hash [e.g. 0.12.0 or 95e8e11913e2a44a4d4682506d8364a638ceac69]

0.38.0

Ah, makes sense. Thanks for the PR.
Side note: can you describe how the session timeout happens?

I'm not really sure how the timeout happens. I'm guessing the issue is just that it's taking longer than the max connection time for the neo4j connection?


Add Border and Hover Effect to "Contact Our Customer Support Team" Section

The "Contact Our Customer Support Team" section currently lacks a border and hover effect, which would improve the visual appearance and interactivity. To enhance the user experience, we need to add:
A visible border around the "Contact Our Customer Support Team" form/section.
A hover effect that changes the border color and possibly adds a subtle animation to draw attention when the user interacts with the section.
I would like to work on this..
Please assign me this issue...

@swaraj-das  Please assign this work to me. I'll solve this in a short time.

@swaraj-das  Can you please add level label

I would like to contribute on this as i am good full stack developer. please assign me that issue.

I can closing this issue because this issue is merged


New Crowdin updates

This PR was created via the Crowdin GitHub integration, but we're transitioning to the GitHub action in #564.


Player: taking damage and stagger

Could this be done within a single "hurt" animation and some function calls disabling specific stuff on the nodes? If so it can pretty much be tackled together with #22 quite trivially. Lemme know if this is a good approach

Taking damage is +/- already in place. It's a little complex at first glance as it's decoupled from the entities that can take damage. Check HitBox and Stats on the player scene. Damage sources, that can hit a HitBox, can be any DamageSource (an Area2D with DamageSource.gd attached).
Stagger should be a state on the Player that you enter upon taking damage. It can mainly be an animation.


Debugging the plugin

Hi
I have a large intranet and did all modification as explained in readme.md, but i haven't this plugin running. The max that i found in the Apache log error was:
[Tue Oct 13 14:23:38 2015] [error] [client <IP_ADDRESS>] PHP Warning:  Invalid argument supplied for foreach() in /var/www/html/piwik/plugins/IntranetGeoIP/IntranetGeoIP.php on line 118, referer: http://www-dtpnet/content/webdatafone

Some suggestion?

@filhocf my tip is, that your data file is wrong.
But please replace this file, with this one: https://github.com/ThaDafinser/Piwik-IntranetGeoIP/blob/ThaDafinser-patch-1/IntranetGeoIP.php
And post the result here (it should be an array)

@filhocf feedback?

Sorry by my delay, but i think that i'm doing some mistake.
I revised my IntranetGeoIP.data.php and can't find a error there.
And in the /var/log/apache2/error.log haven't more problem.
Is necessary run some script to get a result in the map?
2015-10-15 9:35 GMT-03:00 Martin Keckeis<EMAIL_ADDRESS>
@filhocf https://github.com/filhocf feedback?
—
Reply to this email directly or view it on GitHub
https://github.com/ThaDafinser/Piwik-IntranetGeoIP/issues/17#issuecomment-148372429
.

@filhocf du you used the file i posted? Then piwik should log the error (depending on your settings).
Alternative for debugging is doing var_dump() directly and activate debug modus.
See those FAQs
http://piwik.org/faq/troubleshooting/faq_115/
http://developer.piwik.org/api-reference/tracking-api#debugging-the-tracker
When i have time i may create a testpage for this.

Hi There,
I think somethings has changed for Piwik 2.16 as I don't get it working.
First I downloaded the GeoIP (Php) and copied the file over to my website. There I renamed it to C:\inetpub\wwwroot\piwik\misc\GeoIPCity.dat
I then copied over the IntranetGeoIp plugin to the plugins section.
Both plugins are enabeled
I also enabeld the logging (debug) in the config.ini.php.
piwik.zip
Can you please tell me what I'm doing wrong here
Thanks for reading and the development

Do you wanna track internet or intranet sites

Hi There,
I want to track Intranet sites...
I adapted the example data in such a way that whenever I do something on
the monitored site, I should see a special adress for me.
Thanks for reacting
2016-02-19 15:54 GMT+01:00 Martin Keckeis<EMAIL_ADDRESS>
Do you wanna track internet or intranet sites
—
Reply to this email directly or view it on GitHub
https://github.com/ThaDafinser/Piwik-IntranetGeoIP/issues/17#issuecomment-186244373
.

Hi,
It seems that I found a solution..
I forgot to mention I was working with Windows  where you need to add php support but also download the php_geoip.dll and put it in the c:/php/ext dll and of course adapt the php.ini file add the following "extension=c:\php\ext\php_geoip.dll"
Thanks for reading
Guy Forssman

@Forsskieken for this plugin you dont need the GeoIp extension.
Just a correct IntranetGeoIP.data.php inside the config folder.


Combining VSCode Vim plugin with VSCode Easymotion

Greetings! I recently met this extension and I like the idea of it and appreciate the effort behind it.
I came up with an issue that would be nice if you could give it some of your time to see if there's any way to combine this two plugins as it would be the perfect efficiency combo.
There's already someone who thought the same as me in this opened issue in VSCode plugin repo and that I already commented about the behavior between this two plugins.
I've checked your plugin code but I'm no familiar with VSCode plugin programming so I don't know if this issue has something to do with your plugin behavior or the vscode vim plugin behavior so I apologize if it's not this plugin 'responsability', even if that's the case I would like to know what do you think about it. thanks.

Hey, thanks!  Glad you like the extension!
So I think there may be a bit of confusion here due to the naming of this extension.  EasyMotion was originally a plugin for Vim.  VSCodeVim has an implementation of EasyMotion as it works in Vim, which you can enable with the setting "vim.easymotion": true as detailed in the VSCodeVim readme.  The issue that you linked seems to be referring to that.  If you're already using VSCodeVim, I'd recommend trying that out rather than this extension as it is more in line with how Vim works.  I actually wrote this extension to mimic that plugin in a way that made sense to me, as I'm pretty bad at Vim style text editing but really enjoyed being able to jump around like that.
Afaik it would not be possible to make this extension compatible with VSCodeVim without directly integrating this code into VSCodeVim as both take control of VS Code's type event.  I don't have any plans to do so, but if you really wanted to, you could fork VSCodeVim and try to do that integration yourself.  However, I really think the best option for you would be to try out the EasyMotion plugin that is built into VSCodeVim.


Use gpt-3.5-turbo-instruct

How can I update the model from text-davinci-edit-001 to gpt-3.5-turbo-instruct?
I have already modified:
MODEL_AUTOCOMPLETE_ID := "gpt-3.5-turbo-instruct"
url := "https://api.openai.com/v1/completions"

but I am not getting any response.

Hi @boromyr, thanks for opening the issue. According to OpenAI docs, "Most models that support the legacy Completions endpoint will be shut off on January 4th, 2024".
I will change both edit/auto-complete functions to use the chat endpoints in the future.

I tried version 0.3 but it doesn't work with gpt-3.5-turbo-instruct.
Also, line 4 should be #SingleInstance force,
line 87 should be: body.messages := [{"role": "user", "content": UserInput . CutText}] otherwise without UserInput no instruction will be sent.
Version 2 of cJson.ahk has been released, already tested with your code and it works.

Thanks for your feedback and the heads up!

I tried version 0.3 but it doesn't work with gpt-3.5-turbo-instruct.

Sorry, I initially misunderstood. I thought that gpt-3.5-turbo-instruct was going to be deprecated, but now I understood that it is running. It does not work because it uses a different API schema from gpt-3.5-turbo. Do you get better experience using gpt-3.5-turbo-instruct rather than gpt-3.5-turbo? If so, I will support it in the future.

Also, line 4 should be #SingleInstance force,

According to AHK docs, if force is omitted, it already defaults to Force.

line 87 should be: body.messages := [{"role": "user", "content": UserInput . CutText}] otherwise without UserInput no instruction will be sent.

You are right. Thanks!

Version 2 of cJson.ahk has been released, already tested with your code and it works.

Thanks for the information. As soon as I update the code to AHK 2.0, I will also update the third-party libraries.

gpt-3.5-turbo-instruct has several advantages explained better here https://blog.nextideatech.com/openai-gpt-3-5-turbo-instruct/, including more coherent output and not being a chat, for example to translate selected text without getting as output Certainly, here is the translated text: translated text.
cJson also works with version 2 with GPT3-AHK v1.

gpt-3.5-turbo-instruct is now compatible with the newer OpenAI scheme, so you only need to change the model in L64 of GPT3-AHK.ahk: https://github.com/htadashi/GPT3-AHK/blob/f9995797cf7e7cc060c65b50eab1f78545851b7c/GPT3-AHK.ahk#L64


Fix already_declared attribute for enums and structs

Hi! I encounted an issue while working with the crate while using already_declared:
When generating the rust function wrappers for an extern swift function with an already_declared struct/enum the generated code used a locally defined struct/enum with this patch we add super to that function signature.
An example of code that was previously not working:
#[swift_bridge::bridge]
mod ffi_i {
  enum Foo {
    Bar.
  }
}

use ffi_i::Foo;

#[swift_bridge::bridge]
mod ffi {
  #[swift_bridge(already_declared)]
  enum Foo {}

  extern "Rust" {}
  extern "Swift" {
    fn baz(a: Foo);
  }
}

Previously the genrated function would be something like:
mod ffi {
  fn baz(a: Foo) {
    // ...
  }
}

now it's
mod ffi {
  fn baz(a: super::Foo) {
    // ...
  }
}

I'm not sure if it fixes all edge cases but it's working for me locally.
Let me know if I there's something to change to get this merged or if I was just not using already_declared correctly 😅

Thanks for your first contribution!
Just need to add tests and then we can land this.
Can add an extern "Swift" function in here
https://github.com/chinedufn/swift-bridge/blob/72e1759e0504e3a5a32587605c98e410f12ea9e7/crates/swift-integration-tests/src/enum_attributes/already_declared.rs#L13-L21
And in here
https://github.com/chinedufn/swift-bridge/blob/72e1759e0504e3a5a32587605c98e410f12ea9e7/crates/swift-integration-tests/src/struct_attributes/already_declared.rs#L14-L24
And then you can add an extern "Swift" function to this codegen test
https://github.com/chinedufn/swift-bridge/blob/2bffcb08c1f0ca8bbcba9c78c5ae28a9cddfb632/crates/swift-bridge-ir/src/codegen/codegen_tests/already_declared_attribute_codegen_tests.rs#L149-L177
and this one
https://github.com/chinedufn/swift-bridge/blob/2bffcb08c1f0ca8bbcba9c78c5ae28a9cddfb632/crates/swift-bridge-ir/src/codegen/codegen_tests/already_declared_attribute_codegen_tests.rs#L102-L123
In both of the tests you can use the ExpectedRustTokens::ContainsManyAndDoesNotContainMany to confirm that we generate code for the extern Swift function that uses super::TypeName

Thanks for the review and detailed guidance!
Added the tests :)

Yeah CI is currently failing due to https://github.com/chinedufn/swift-bridge/pull/225#issuecomment-1552361044
Test looks good, I'll land this. Thanks!


Problems when subscribing to flash store and hovering over  tag where load function uses redirect.

Issue Description
I am currently using sveltekit-flash-message to show a toast after redirecting from a protected page to a public page. To achieve this, I followed the example in the readme by subscribing to the flash store, which triggers my toast each time the redirect happens. However, I encountered an issue where the toast sometimes triggers even when I redirect to a page where the flash-message redirect is not used. I have checked for the undefined case, but the problem persists.
Investigation
After some investigation, I found a way to reproduce the error. The issue occurs when I hover over the protected button, where the flash-message redirect occurs on load. Then, if I click another link, the flash-message gets set again, and the toast re-triggers.
Reproduction Steps
To reproduce this issue, follow these steps:

Go to the public page.
Hover over the protected button (where the flash-message redirect happens on load).
Go to the homepage.

Then the toast is unnecessarily shown.
Try it yourself
Go and run my example, just pnpm i and pnpm dev to run it
https://github.com/hansaskov/sveltekit-flash-messages-toasts-error

Since Svelte preloads the links, this is expected. You can modify that behavior with these options: https://kit.svelte.dev/docs/link-options

Thank you. You are correct that changing the data-sveltekit-preload-data attribute to be tap instead of hover does fix this issue.
Do you think it is possible to not set the flash-message if the preload page is not navigated to?

Not unless there is a way to detect if a link is preloading through the RequestEvent, and I don't think there is.

I have figured out a workaround, that i believe makes sense. The idea is to store the url pathname we expect to navigate to within the flash-message, then only trigger the Toast if the expected url pathname matches the pages pathname.
Example
Start by updating the flash message to include the expected pathname we navigate to.
// app.d.ts
declare namespace App {
	interface PageData {
		flash?: {
			type: 'success' | 'error';
			message: string;
			pathname: string;  // Update here
		};
	}
}

When we try to navigate to the /private page, we redirected to the homepage with the message of "Message from /private" as well as the pathname of the page we expect to redirect to.
// src/routes/private/+page.server.ts
export const load: PageServerLoad = async (event) => {
	const message = { type: 'error', message: 'Message from /private', pathname: '/' } as const;
	throw redirect(303, message.pathname, message, event);
};

In the layout page we subscribe to the flash store, which will trigger each time the flash-message is updated. The code will make an early return if $flash is undefined or if the pathname of the current page is not the same as the expected pathname from the flash-message
// layout.svelte
import { getFlash } from 'sveltekit-flash-message/client';
import { page } from '$app/stores';

const flash = getFlash(page);

flash.subscribe(($flash) => {Do you think it is possible to not set the flash-message if the preload page is not navigated to?
@ciscoheat
Owner
[ciscoheat](https://github.com/ciscoheat) commented [Jul 16, 2023](https://github.com/ciscoheat/sveltekit-flash-message/issues/17#issuecomment-1637075752)

Not unless there is a way to detect if a link is preloading through the [RequestEvent](https://kit.svelte.dev/docs/types#public-types-requestevent), and I don't think there is.
@hansaskov
	if (!$flash) return;
	if ($flash.pathname != $page.url.pathname) return;

	errorToast.message = $flash.message;
	toastStore.trigger(errorToast);

});;

You might have noticed that a small bug is still present with the above implementation. When hovering over the private page, then manually navigating to the homepage, the toast is incorrectly displayed. This can be fixed by only triggering the toast if the navigation was done using redirects. To implement this we can create the isGotoNavigated variable and subscribe it to the after navigate store to be true when the navigation was of type goto. Then return early if the navigation is not a redirect.
import { getFlash } from 'sveltekit-flash-message/client';
import { page } from '$app/stores';
import { afterNavigate } from '$app/navigation';

const flash = getFlash(page);

let isGotoNavigated = false;
afterNavigate(({type}) => {
	isGotoNavigated = ['goto'].includes(type as string); // is true when the navigation was a redirect
})

flash.subscribe(($flash) => {
	if (!$flash) return;
	if ($flash.pathname != $page.url.pathname) return;
	if (!isGotoNavigated) return;

	errorToast.message = $flash.message;
	toastStore.trigger(errorToast);
});

I would really love to see this implemented directly into the library as i believe it represents what most people expect when using the it for the first time.
I would like to hear your feedback on this, and if there are some downsides to this implementation. I also would like to contribute myself and put it under a settings flag. But that will be in a couple of days

I'd like to keep things simple, so introducing an extra variable into the flash message isn't feasible as a requirement. Testing the type in afterNavigate isn't really optimal either since the cookie has already been set during the preload.
The only way to keep things simple would be to prevent the redirect (and hence the cookie) during the preload, but if it's not possible to detect preload in the load function, I'd rather add a note about data-sveltekit-preload-data in the docs.

That is totally understandable, preventing the redirect is the most elegant solution. And i think you are correct that differentiating between preloaded and non preloaded functions is non trivial.
I'm closing this issue, as it is intended behavior that can be mentioned in the docs.

Thanks for understanding, I'll reopen this until I've fixed the docs!


Shared IP Issue.

How does this gem manage to throttle shared ip?
Suppose an app has login form with rate limitation of 5 requests :-
`
throttle('logins/ip', :limit => 5, :period => 20.seconds) do |req|
.....
end
`
Take a scenario where users are accessing the app via shared network (workplace/office) and some users (>5) say 10 tries to access the login form same time.
Now as they are sharing same IP (external) therefore this will cross the rate limit.
How to tackle this down?

Hi @kathuriaranjan:
I don't know of good, general way to distinguish many users sharing the same IP address.
I choose my throttle limits such that even if there are many users sharing the IP, legitimate use does not reach the throttle limit. Only clear abuse triggers the throttle.
(Also note that it's unlikely all 10 users would try to log in at the same time. It would take coordination for 10 people to all log in during the a 20-second time period.)


Create 4-Median-of-Two-Sorted-Arrays.cs

File(s) Modified:4-Median-of-Two-Sorted-Arrays.cs
Language(s) Used: csharp
Submission URL: https://leetcode.com/submissions/detail/796821008/

Thanks, @agnihotriketan!


This app is great!

Thanks for making this awesome app.
Would you be willing to add it to https://github.com/electron/electron-apps? I'm sure more developers would appreciate knowing about it. :)

Done. ✅

🍰


User Authentication with Django

Issue solved: #5
Features added:

Signup and Login Page using Bootstrap (Redirect to Login and Signup links added respectively)

User Signup using Username, Password and Email (error message popup if user already exists with entered username)

User Login using Username and Password (error message popup if invalid credentials are entered)

User logout

Issue solved: #5
Features added:

Signup and Login Page using Bootstrap (Redirect to Login and Signup links added respectively)
User Signup using Username, Password and Email (error message popup if user already exists with entered username)
User Login using Username and Password (error message popup if invalid credentials are entered)

Hey @SubhradeepSS , can you please share the screenshots here so that I can review easily?

@diyajaiswal11 I have added some screenshots of login,signup page(including error messages popping) and also home page having the logout option in navbar.

@diyajaiswal11 I have added some screenshots of login,signup page(including error messages popping) and also home page having the logout option in navbar.

Good job !! @SubhradeepSS :)


Please review the Ionicons font-face definition

I'm also experiencing the issue reported at https://github.com/driftyco/ionicons/issues/144, looked into the @font-face declaration and was slightly confused.
We're using Ionicons with our Ionic Framework app, and I feel like the font-face declaration is counter-productive in this use case (which, to my understanding, is the primary use case of the font).
First of all, I agree with @mowcixo, I don't see how the query on the URL has any benefit when being used in an app bundle. And even if the behavior on Windows Phone 8 is a bug in mobile IE 11, if removing the query from the src attribute allows the font to be used on the platform, why not consider changing it? After all, the current definition even includes #iefix.
Second, the src attribute seems unnecessarily verbose. It is my understanding that .eot is included on the web for IE8 compatibility, something that should have no relevance in the Ionic framework.
I guess, the .ttf enables support Android <4.4, so it is probably required, but should be ordered below the .woff IMHO.
I don't see the need for the .svg, as all browsers that support .svg fonts, also support WOFF. To my understanding, this also holds true on the web.
Please keep in mind that all font variants are usually packaged into the app bundle and contribute to the overall size. The .svg and .eot weigh 380 KB which might not be required in the package.
For testing purposes, I slimmed down the font-face to:
@font-face {
  font-family: "Ionicons";
  src: url("../fonts/ionicons.woff") format("woff"), url("../fonts/ionicons.ttf") format("truetype");
  font-weight: normal;
  font-style: normal; }

Which worked great on BlackBerry 10 and Windows Phone 8 (yes, I do realize that I tested only on platforms you don't support).
I'd be interested to hear your thoughts on this.

I don't understand why this hasn't gotten any response :( Because this is such a pain for us, we're now maintaining https://github.com/hartwig-cordova/ionic-bower.
The fix itself is in https://github.com/hartwig-cordova/ionic-bower/tree/fix/fonts. We publish -wp8 tags with the fix applied. We use https://github.com/hartwig-cordova/ionic-bower/tree/v1.0.0-beta.14-wp8 ourselves.

Hey there, so ionicons have a double life - as the icons for ionic and an all purpose font-icon for the web. That impacts how we've setup ionicons

version query string: This forces the browser to load the most recent icons and not a cached version when they get updated.
Again, since ionicons can be used for the web, and not just compiled apps, they have to support the full gamut of browsers.

https://developer.mozilla.org/en-US/docs/Web/CSS/@font-face
@mowcixo idea of a hook to remove the unnecessary files to save on file size in interesting. @mowcixo would you be able to open an issue for this on the cli repo?
For now, we are going to keep the fonts the way the are as we're close to 1.0.

Will resist post 1.0

@mhartington I understand that Ionicons can also be used standalone, but I don't see why the integration in the Ionic Framework can't be adjusted since there is a separate file for this and the SCSS from Ionicons isn't used in the Framework.

Besides, I have not tested in IE for desktop, but I suppose it fails on it
too.... It's not a target for Ionicons to work on IE ?
2015-02-16 14:38 GMT+01:00 Oliver Salzburg<EMAIL_ADDRESS>
@mhartington https://github.com/mhartington I understand that Ionicons
can also be used standalone, but I don't see why the integration in the
Ionic Framework can't be adjusted since there is a separate file
https://github.com/driftyco/ionic/blob/master/scss/ionicons/_ionicons-font.scss
for this and the SCSS from Ionicons isn't used in the Framework.
--
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ionic/issues/2599#issuecomment-74509127.

--
Moisés Gramary Barbosa
<EMAIL_ADDRESS>0034 687800099
One bright morning when my work is over, Man will fly away home.

@mowcixo Works fine on desktop IE11

Yes, you are right, and in 8, 9 and 10 too. I'll see if I can make an
assembled version of my hook and propose a pull request on Ionic CLI.
2015-02-16 14:48 GMT+01:00 Oliver Salzburg<EMAIL_ADDRESS>
@mowcixo https://github.com/mowcixo Works fine on desktop IE11
--
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ionic/issues/2599#issuecomment-74510259.

--
Moisés Gramary Barbosa
<EMAIL_ADDRESS>0034 687800099
One bright morning when my work is over, Man will fly away home.

Hi all,
i understand that ionicons is an independant project with its own life. But, like @oliversalzburg, I don't undestand why the ionicons integration supports all the not-needed features for mobile targets (EOT, SVG). As of today (ionic 1.3.1), ionicons is still bundled with all fonts flavors.
In ionic.scss, instead of :
@import
  // Ionicons
  "ionicons/ionicons.scss",

we could have :
@import
  // Ionicons
  "ionicons/ionicons-variables";
  "ionicons-font-for-ionic";
  "ionicons/ionicons-icons";

And in ionicons-font-for-ionic, inject the code from @oliversalzburg :
@font-face {
  font-family: $ionicons-font-family;
  src: url("#{$ionicons-font-path}/ionicons.woff") format("woff"),
  url("#{$ionicons-font-path}/ionicons.ttf") format("truetype");
  font-weight: normal;
  font-style: normal;
}

.ion {
  display: inline-block;
  font-family: $ionicons-font-family;
  speak: none;
  font-style: normal;
  font-weight: normal;
  font-variant: normal;
  text-transform: none;
  text-rendering: auto;
  line-height: 1;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
}


Modified typos in several code blocks for plots

Changed some typos that were present in the code cells for plots of the HMC outputs. These code cells were also re-run.

@CamDavidsonPilon One last typo to be fixed in Chapter 2.

thanks!


TESTING Gha codebuild shim

Issue # (if available):
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing in favor of actual pr


Add instances for ShortByteString

failure with bytestring<0.10.4

Good catch, do you want me to raise the lower bound or use CPP? bytestring-0.10.4 was released in 2013 so a lower bound seems pretty reasonable.

No, I want to support GHC-7.4. Please use CPP to guard instances.

additions to previous: this is testing package, I try to have as wide bounds as possible, so it can be used in all situations where one would expect to use QuickCheck (which also has support back to 7.4 IIRC)

Alright, I’ve added the CPP.

Thanks!


Backport "HBASE-26788 Disable Checks API callback from test results in PRs" to branch-1

Signed-off-by: Sean Busbey<EMAIL_ADDRESS>

I'm not sure if this file is used from base-branch on a PR or not, so just in case...

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
3m  1s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-1 Compile Tests _

+0 :ok:
mvndep
0m 25s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  6s
Maven dependency ordering for patch

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

-1 :x:
whitespace
0m  0s
The patch 1 line(s) with tabs.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

4m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4159/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4159

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux d75cdac78e5f 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-4159/out/precommit/personality/provided.sh

git revision
branch-1 / f712a4ceb5

whitespace
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4159/1/artifact/out/whitespace-tabs.txt

Max. process+thread count
30 (vs. ulimit of 10000)

modules
C:  U:

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4159/1/console

versions
git=2.17.1 maven=3.6.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-1 Compile Tests _

+0 :ok:
mvndep
0m 25s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  6s
Maven dependency ordering for patch

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

2m 14s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4159/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4159

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 62e0c04051e0 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-4159/out/precommit/personality/provided.sh

git revision
branch-1 / f712a4ceb5

Max. process+thread count
30 (vs. ulimit of 10000)

modules
C:  U:

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4159/2/console

versions
git=2.17.1 maven=3.6.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.