Add ExportCommand

Yan Han and I have decided that we want to do black box testing for export and import. Therefore, the tests are not in this pull request.

Codecov Report

Merging #102 (00f323c) into master (08e0e91) will decrease coverage by 1.09%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #102      +/-   ##
============================================
- Coverage     70.52%   69.43%   -1.10%
  Complexity      582      582
============================================
  Files            96       98       +2
  Lines          2005     2038      +33
  Branches        246      249       +3
============================================
+ Hits           1414     1415       +1
- Misses          504      536      +32
  Partials         87       87

Impacted Files
Coverage Δ

...address/logic/commands/schedule/ExportCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
66.66% <0.00%> (-2.90%)
:arrow_down:

...ess/logic/parser/schedule/ExportCommandParser.java
0.00% <0.00%> (ø)

...in/java/seedu/address/model/schedule/Schedule.java
42.85% <ø> (ø)

...a/seedu/address/model/schedule/RecurFrequency.java
68.96% <0.00%> (-1.41%)
:arrow_down:

...n/java/seedu/address/storage/JsonAdaptedEvent.java
100.00% <0.00%> (ø)

.../java/seedu/address/storage/JsonAdaptedPerson.java
86.56% <0.00%> (ø)

...ava/seedu/address/storage/JsonAdaptedSchedule.java
100.00% <0.00%> (ø)

...u/address/storage/JsonSerializableAddressBook.java
100.00% <0.00%> (ø)

.../seedu/address/storage/JsonAddressBookStorage.java
100.00% <0.00%> (+12.50%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM!


Fixes the auto-size text bug.

Fixes #1602.

There is still an issue, because the superview must be responsible for clearing the previous frame.


style: 格式化代码

General

✏ Mark the necessary items without changing the structure of the PR template.

[ ] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[x] My code follows the style guidelines of this project
[x] Is the code format correct
[x] Is the git submission information standard?
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

我执行了 lint:eslint lint:prettier lint:stylelint 命令来格式化, 因为目前有些代码样式不符合规范


Add TfsCmdlets 2.0.0-beta.14+2095.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


improve flow for loggin out & directly logging in again

Fixed by changing a setting to redirect correctly to app.jodal.nl


fix some typo in Makefile

Coverage remained the same at 50.704% when pulling 8c42d0a75deba05461c0902aa1b0155261592dda on yhua123:dev into a05a9c1afd7a2eadfe5f6df8a2ab9da2d4019d27 on vmware:dev.


Xcode Cloud build failed for Flutter with Command PhaseScriptExecution failed with a nonzero exit code

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Integration of Xcode Cloud with Flutter iOS project
Actual results
Command PhaseScriptExecution failed with a nonzero exit code
Logs
Run custom shell script 'Run Script'
/Users/mtpc-236/Development/flutter/packages/flutter_tools/bin/xcode_backend.sh: No such file or directory
Error: Command PhaseScriptExecution failed with a nonzero exit code
Flutter Doctor output
[✓] Flutter (Channel stable, 3.7.11, on macOS 13.4 22F66 darwin-arm64, locale en-IN)
• Flutter version 3.7.11 on channel stable at /Users/mtpc-236/Development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision f72efea43c (3 months ago), 2023-04-11 11:57:21 -0700
• Engine revision 1a65d409c7
• Dart version 2.19.6
• DevTools version 2.20.1
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/mtpc-236/Library/Android/sdk
• Platform android-33, build-tools 31.0.0
• ANDROID_HOME = /Users/mtpc-236/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14C18
• CocoaPods version 1.12.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
[✓] Connected device (3 available)
• iPhone 14 Pro Max (mobile) • 72391A1B-1621-49AB-BB39-3FA82FC48799 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)
• macOS (desktop)            • macos                                • darwin-arm64   • macOS 13.4 22F66 darwin-arm64
• Chrome (web)               • chrome                               • web-javascript • Google Chrome 114.0.5735.198
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Hi @VikramZala93. Thanks for filing this issue. Please can you provide more information on how this gets produced:

Does this happen with a newly created flutter project or an existing one?
If possible, can you provide a step-by-step process to reproducing this.

Also, I see you are on an older version of flutter, can you upgrade to the latest versions to confirm this issue still persists? You can also try out possible solutions on this Stack Overflow Thread if it works in your case.

Hi @VikramZala93. From your recent explanation, do you mean it builds fine locally and the issue only occurs when pushed??

Yes, exactly. I am able to build and upload the TestFlight in local environment. Issue is occurring when I pushed the same for Xcode Cloud.

@VikramZala93 Xcode Cloud is heavily targeted at "pure" Apple projects. Flutter apps require some additional setup before they can be built, e.g. you need to install the flutter tool, and as far as I know, you have to manually script your Xcode Cloud workflow to do it.

I have integrated setup the manual script for Xcode Cloud to install the flutter tool. Kindly review the ci_post_clone.sh for your reference. Thank you so much for your support and time!

@VikramZala93 I think the problem is that you export PATH, but it doesn't have any effect if the script is not sourced. So the PATH is modified only during ci_post_clone.sh, but not afterwards when Xcode builds the app.

@bartekpacia Could you please suggest me about how to modify the path after Xcode builds the app?
I have tried by changing the "FLUTTER_ROOT" path in Xcode as cloning path showing during the Flutter install using Git but it is showing the same error.
Kindly review the following screenshots and suggest the needful. Thanks!

You could modify the scripts in Build Phases to accomplish that.
For example, in your ci_post_clone.sh:
flutter_root=$(readlink -f $(dirname $(which flutter))/../)
echo $flutter_root > ~/.flutter_root

and then alter the contents of your Build Phases with:
if [ -z "$FlUTTER_ROOT" ]; then
  FLUTTER_ROOT=$(cat ~/.flutter_root)
fi

/bin/sh "$FLUTTER_ROOT/packages/flutter_tools/bin/xcode_backend.sh" build

I didn't test that myself, so it might require modifications, but that's a general gist.

@bartekpacia @dam-ease  I have modified the ci_post_clone.sh with above mentioned changes but it is still showing the same error.
It is not updating the "FLUTTER_ROOT" path during the Flutter install using Git.
Please review the following screenshots and suggest the needful.
Thank you so much for your time and support!

Hi @VikramZala93. Pending when this gets figured out, can you try the solution on this StackOverflow Thread if it works in your case?

@dam-ease I have tried all the 3 solutions mentioned in StackOverflow Thread but, nothing worked for me. Kindly suggest the needful. Thanks!

Hi @VikramZala93. Thanks for your patience.

Can you confirm that Debug.xcconfig and Release.xcconfig aren't missing #include "Generated.xcconfig"?

If you're using any dependencies with pods, this is what Release.xcconfig should look like:
#include "Pods/Target Support Files/Pods-Runner/Pods-Runner.release.xcconfig"
#include "Generated.xcconfig"

And Debug.xcconfig
#include "Pods/Target Support Files/Pods-Runner/Pods-Runner.debug.xcconfig"
#include "Generated.xcconfig"

This is a similar/related issue https://github.com/flutter/flutter/issues/56085, underlying comments seems to have steps to set it up properly

@dam-ease I have confirmed that "#include "Generated.xcconfig" is there in Debug.xcconfig and Release.xcconfig file. kindly review the screenshots for the same.
I have also reviewed and confirmed that https://github.com/flutter/flutter/issues/56085 steps are set it up properly in my code.
Kindly suggest the needful. Thank you for your support and time!

Hi @VikramZala93. Thanks for confirming and for your patience. I'm trying to reproduce your issue on my end as well and can't seem to reproduce and can't seem to point out where the issue is on your end considering you pointed out you are following all steps. Can you create a new flutter project and follow up on the steps as highlighted in the docs to confirm if this issue still reproduces on your end?

@dam-ease I have to integrate the Xcode Cloud for my current live iOS project (i.e. in production environment) and I have confirmed and followed the steps that mentioned in the docs Kindly suggest the needful. Thank you for your support and time!

Hi @VikramZala93. Thanks for your response. Here is a sample repo that works https://github.com/huycozy/test_xcode_cloud. Can you try this out??

Any updates here?
I'm using flutter 3.13 and yet facing same issue for ios build. It wasn't there in previous version for me.
Solutions tried

pod clean and re-install
flutter clean and flutter pub get
xcode clean


Zero padding corsika output file

Fixes #646

Yes, I know it doesn't exactly solve the problem, I mentioned it in #723. I quote my explanation below.

OK, I realise now why Raul used the env variables. It is to support the simulation of a range of runs on the same machine (or output to the same directory).

I then thought of a way to implementing the zero padding even when using these env variables (copying from Konrad's implementation in the latest run script), but realised it is not really necessary. We do not plan to simulate a range of runs sequentially or using the batch farm in this way. It is functionality we do not need and can also be implemented in the future without relying on the internal system of CORSIKA to advance the run numbers.

So, once again, I suggest to accept this simplified solution and then remove the entire option of simulating a range of runs (or we retire the old production script in general). We can do that in a separate PR.

Yes, I know it doesn't exactly solve the problem, I mentioned it in #723. I quote my explanation below.

OK, I realise now why Raul used the env variables. It is to support the simulation of a range of runs on the same machine (or output to the same directory).

I then thought of a way to implementing the zero padding even when using these env variables (copying from Konrad's implementation in the latest run script), but realised it is not really necessary. We do not plan to simulate a range of runs sequentially or using the batch farm in this way. It is functionality we do not need and can also be implemented in the future without relying on the internal system of CORSIKA to advance the run numbers.

So, once again, I suggest to accept this simplified solution and then remove the entire option of simulating a range of runs (or we retire the old production script in general). We can do that in a separate PR.

I think we need to discuss in person. We have not decided to discontinue the tool, which I am against, and for me it does not make sense to implement something that does not really fix the issue.

But removing this functionality of simulating a range of runs does not discontinue the tool or break the possibility of simulating CORSIKA and/or sim_telarray separately. It just removes the functionality of simulating a few runs sequentially.
This type of functionality is simply not needed (why wouldn't we run in parallel instead?). When running on the batch farm it might be useful, but we can achieve the same behaviour by copying the grid behaviour and just submitting multiple jobs with changing run numbers instead of using the internal CORSIKA (IACT actually) counting mechanism.

Your proposal here is changing the name of the .zst output file of CORSIKA to include the zero padding but you are giving the very same name of the file to every output file (run) corsika_run000001_gamma_za020deg_azm000deg_North_TestLayout_test-production.zst and that is just not correct for the current implementation.
Independent of that, we still need to correctly retrieve the runs in simtel_runner one by one and that is still not possible because simtel_runner writes into simtel-data/North/gamma/data/ the .zst output file based on the name of the CORSIKA .zst output file it received. If the name is the same,  simtel_runner is overwriting at each step its output file into simtel-data/North/gamma/data/. In the end, we have a single output file named run000001_gamma_za020deg_azm000deg_North_TestLayout_test-production.simtel.zst.
If you want to insist in this PR, I suggest you to implement a full working example, i.e., running the production tool with --showers-only and then with --array-only. I would only agree that it works if a number of simtel output .zst files are produced and if the output is not the same for every run. And again, I point you to the solution in #723 in which this is already possible!

OK, let me remove the option of simulating multiple runs with one call in this PR and then the example you mentioned will work. I will not include simulating multiple runs because it is not necessary functionality.
The solution in #723 goes against the DPPS decision for file names, so it cannot be accepted.

OK, further discussions made me look for another option and I found a fairly simple (but "hacky" one). I just set a place holder in the input template file and then replace it with sed. This keeps the functionality of running on a range of runs and the zero padding. It's ugly, but it works.
@VictorBarbosaMartins could you please test and say if it works for you (and if you are not horrified by the hack)?
@GernotMaier since this is really a hack, I wouldn't mind getting also your opinion, to make sure everyone's on board with it.

I don't mind that it is a bit uglier now, as it seems to be working. With this, the old production tool is again functional and has a zero padding (did not think this would be so important.). I tested locally as I suggested before and, therefore, I am approving.
As a side note, the tests are still not sufficient to detect whenever the tool gets broken again. So, I will wait for us to merge #726 before implementing better tests.

Fine with me - and I am not making any comments about the super-long run numbers planned...

Fine with me - and I am not making any comments about the super-long run numbers planned...

Well, it's either you limit to a 100,000, which is too low (we already simulate 40,000 proton runs in each production) or to 1 million. I vote for the latter.
Merging, thanks for the quick review!


DatabaseCleaner cleans my development database instead of test

Hi.
DatabaseCleaner cleans my development database instead of test
I have tried all possible configurations.
And I still have this problem.
database_cleaner 1.6.2
Rails: 5.1.5
Ruby 2.5
Gemfile:

group :development, :test do
  gem "byebug", platforms: [:mri, :mingw, :x64_mingw]
  gem "factory_bot_rails", "~> 4.8", ">= 4.8.2"
  gem "figaro"
  gem "faker", "~> 1.8", ">= 1.8.4"
end

group :development do
  gem "web-console", ">= 3.3.0"
  gem "listen", ">= 3.0.5", "< 3.2"
  gem "spring"
  gem "spring-watcher-listen", "~> 2.0.0"
  gem "pry-rails", "~> 0.3.6"
  gem "better_errors"
  gem "binding_of_caller"
end

group :test do
  gem "capybara"
  gem "selenium-webdriver"
  gem "simplecov", "~> 0.15.1", :require => false
  gem "shoulda-matchers", "~> 3.1", ">= 3.1.2"
  gem "database_cleaner"
  gem "rspec-rails", "~> 3.7", ">= 3.7.2"
end

in rails_helper i have forced env to test:
ENV['RAILS_ENV'] = "test"
DatabaseCLeaner conf file ( i have tried all posible combinations for configuration:
RSpec.configure do |config|

  config.before(:suite) do
    DatabaseCleaner.clean_with(:truncation)
  end
  config.before(:each) do
    DatabaseCleaner.strategy = :transaction
  end
  config.before(:each, js: true) do
    DatabaseCleaner.strategy = :truncation
  end
  config.before(:each) do
    DatabaseCleaner.start
  end
  config.after(:each) do
    DatabaseCleaner.clean
  end

end

From tests when i print ENV i get TEST env but each time after test my development database is cleaned and test database is not used:
p ENV["RAILS_ENV"] #=> "test"
database config file have default configuration for test:
test:
  <<: *default
  database: appname_test
  username: <%= ENV["DATABASE_USER"] %>
  password: <%= ENV["DATABASE_PASSWORD"] %>
  reconnect: true

Any idea where to look to fix this bug ?
**after executing the rspec command all the data in my development database is deleted **
Thx

Set your environment from the shell, not from Ruby.
export RAILS_ENV=test

If you set it from inside a Ruby file, then Rails.env may have already been set.

I'm not sure if you have read what I have written.
If I check the environment from the test, the environment i can confirm that the env is TEST not dev, even though databasecleaner uses the development database
code inside some test:
p ENV["RAILS_ENV"] #=> "test"
how you cans see everything is correct.
I also want to emphasize that database_cleaner is present only in the test environment:
group :test do
  gem "capybara"
  gem "selenium-webdriver"
  gem "simplecov", "~> 0.15.1", :require => false
  gem "shoulda-matchers", "~> 3.1", ">= 3.1.2"
  gem "database_cleaner"
  gem "rspec-rails", "~> 3.7", ">= 3.7.2"
end

There was some bug, this problem was present in different devices.
Test Env was configured correctly, from tests i was geting TEST env but database cleaner was using development database instead of test.
For now i fixed it adding ENV['RAILS_ENV'] = 'test'  to  spec_helper.rb ( i had it only in rails_helper.rb but it seems that it is not enough for database_cleaner )
in my case database cleaner is using test db only if i add ENV['RAILS_ENV'] = 'test'  in both files (rails_helper.rb and spec_helper.rb)

@RTJ Thanks for sharing! I'm glad it's solved.

@RTJ @etagwerker Having the exact same issue! wondering how does your spec_helper.rb and rails_helper.rb files looks like, as mine starts with:
rails_helper.rb:
require 'spec_helper'
ENV['RAILS_ENV'] ||= 'test'

So, why the need to add it to spec_helper.rb? And more importantly, if ENV is set after it's required in rails_helper.rb, why is it not working?
My hunch is that this can really be a bug. Thoughts?

Having the same issue with Rails 6 - using v1.8.5.
Adding "ENV['RAILS_ENV'] ||= 'test'" to my rails_helper file didn't fix it for me.
Any idea ?

https://github.com/DatabaseCleaner/database_cleaner/issues/668#issuecomment-780117915
↑ My this comment means the same thing, https://github.com/DatabaseCleaner/database_cleaner/commit/77dece1e145899231a57b0fead8a20f6e43d3d0a made Rails.env.test? return false, so development DB will be clean up after running rspec.

@dfabreguette

Having the same issue with Rails 6 - using v1.8.5. Adding "ENV['RAILS_ENV'] ||= 'test'" to my rails_helper file didn't fix it for me. Any idea ?

Perhaps when you run your RSpec, you are not specifying the RAILS_ENV variable. Currently, Rails interprets this as being in development, which is why setting ENV['RAILS_ENV'] ||= 'test' is ineffective. Just change this to ENV['RAILS_ENV'] = 'test', and it should function correctly.


Simplify annotation handling in tutorial

Codecov Report

Merging #314 (6021f77) into master (7fe9eaa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #314   +/-   ##
=======================================
  Coverage   97.10%   97.10%
=======================================
  Files          48       48
  Lines        1314     1314
  Branches      178      178
=======================================
  Hits         1276     1276
  Misses         14       14
  Partials       24       24

Continue to review full report at Codecov.

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


Batch upload polishing

An umbrella issue, feel free to add subtasks as individual comments.

Duplicates #2676.


Fix #81 - Vertical scrollbar overlays header

#81
I'm not too sure if this is something you think needs fixing but it was causing us a little grief by obscuring the sort arrow in the right hand columns header.
Feels a little better keeping the scrollbar inside the body of the table to my eye. Let me know what you think.

Hey @mikestead Thanks for the PR!
@jbonta @wlis What do you think about changing the scroll bar to not overlap the header?

yeah, I think this makes sense. Does the header's shadow still cover the scrollbar? Do you have a screenshot? cc @wlis

@jbonta good point, the scroll bar track is sitting on top of the header shadow which looks off.
The Scrollbar z-index defaults to 99 in the component, but it can be passed a different z-index prop to use. The shadow z-index is defined in the css file. I can bump that up to 100 (see before / after images below) but not sure that's a great fix.

Thanks for the PR! It's been merged in, with the adjustment to the z-index (100).


[Godot 4] Menu scene title not displayed correctly

Expected
Actual

This was fixed somewhere along the way.


Changes to SAML settings to support WS-Federation/ Active Directory

I would like to find out if python-saml library works out of the box without any changes with the Microsoft STS(Security Token Service).
I propose the following patch to this library which uses additional flags to the SAML settings file.
Pull Request: https://github.com/onelogin/python-saml/pull/129. Kindly review this diff. Also if there is a simpler/better way of doing it please let us know.
Thanks & Regards,
Sandeep

I already closed the PR 126 where I commented to you that python-saml is able to work out of the box, with the right settings with ADFS.


Fix StandardErrorPath logging to stdout.log

The StandardErrorPath was set to stdout.log it should be set to
something else like stderr.log.

looks good, thanks.


Can you get Periphery's actual output using this plugin?

All I see output is a "Summary of Results", which is not much use.

Hey @dpt-roche, You can by using shared lane context Actions.lane_context[SharedValues::PERIPHERY_RESULTS]... Here is an example of how we use it:
desc 'Analyses unused code and posts statistics'
private_lane :post_unused_code do
  results = Actions.lane_context[SharedValues::PERIPHERY_RESULTS]
  UI.user_error!("You must run 'periphery' action before this lane") if results.nil?

  # Periphery currently reports unused, redundantPublicAccessibility, redundantProtocol and redundantConformance results.
  # We only want to track unused references so filter and group by 'kind' (i.e class, variable etc) for more detailed stats
  stats = results
    .filter { |result| result.hints.include?("unused") }
    .group_by(&:kind)
    .map { |kind, results| [kind, results.length] }
    .to_h

  # Also add the total count
  stats['total'] = stats.values.inject(0, :+)

  # Track in InfluxDB
  influxdb(table_name: 'global_ios_unused_code', values: stats) if ENV['CP_TRACK_METRICS']
end

I didn't expect anybody else to be using the plugin so I hadn't gotten around to documenting this in the README yet 😄 PRs welcome 🙂

Right now, the value assigned to that property is just an array but from #1, I do want to change this in the future.
The idea would be that its a class and with that, we could add extra methods to provide more convenience in filtering/querying the data rather than having to do it manually like the snippet above.
Since there is no release tag, please keep an eye out for changes with this 🙂

Thanks so much for that. It's really helped.


Replace bit.ly URL with aka.ms variant

The following snippet contains a bit.ly URL which is uncommon given MS's common practice of relying on the aka.ms shortner.
https://github.com/MicrosoftDocs/windows-itpro-docs/blob/758f7a98a5ca788cfa82024e7d5dfb87e3a1245a/windows/security/threat-protection/windows-defender-application-control/wdac-wizard.md?plain=1#L37

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

ID: 054ba21b-1dd4-f891-74f4-950b6dbba31e
Version Independent ID: 8837a26c-2bdc-fcfa-e1d9-928460ccd2b6
Content: Windows Defender Application Control Wizard - Windows security
Content Source: windows/security/threat-protection/windows-defender-application-control/wdac-wizard.md
Product: m365-security
Technology: windows-sec
GitHub Login: @jgeurten
Microsoft Alias: dansimp

Thank you @0xThiebaut for letting us know about this issue. Thank you @RAJU2529 for fixing it. Changes have been merged and will render live later today. Thank you for helping us improve our documentation!


Release prebuilt libiconv

Hi,
I am trying to reproduce
https://medium.com/@zw3rk/a-haskell-cross-compiler-for-android-8e297cb74e8a
But had a really hard time carrying out the ./build-libiconv step on my 2018 macbook.
Is it possible to release prebuilt versions of libiconv similar to what you mentioned for ghc here?
http://hackage.mobilehaskell.org/
That would be really appreciated.
Ni

I’m afraid I don’t have any time to build them. But I’ll keep this issue open until I have.

Look forward to it.
Ni
On Wed, Jul 24, 2019, 8:59 PM Moritz Angermann<EMAIL_ADDRESS>wrote:

I’m afraid I don’t have any time to build them. But I’ll keep this issue
open until I have.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zw3rk/ghc-build-scripts/issues/3?email_source=notifications&email_token=ABSZNYYFFIFXHX45IOY5N33QBEQKBA5CNFSM4IGMGWZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2YITEQ#issuecomment-514886034,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABSZNYYZG2JDX7CZQGPWVC3QBEQKBANCNFSM4IGMGWZQ
.


deferred close on prepared statement in query method

Added deferred close to prepared statement in query method to prevent same issue as fixed in #130

thank you

This seems broken.
https://travis-ci.org/mattn/go-oci8/builds/189749424

I'll revert this.


"Auth session missing!" when getting user server side with supabase.auth.getUser()

Version
@nuxtjs/supabase: 1.2.2
nuxt: 3.12.3
Reproduction Link
https://stackblitz.com/edit/nuxt-starter-v3dd6h
Problem
Our Nuxt web application is experiencing a critical issue affecting user experience since few months. After approximately one hour of inactivity on a tab, users receive an "Unauthorized" error when attempting to perform an action on the webapp.
Problem details

Our Nitro server uses supabase.auth.getUser() before executing the code of each route to authenticate the user. (By the way, I don't use await serverSupabaseUser(event) because it uses getSession() and this is not recommended by Supabase on the server side)
After a period of inactivity from the user on our webapp, this verification fails on the server side, causing the "Unauthorized" error.
On the client side, the user still appears to be logged in and can perform actions involving database operations (insert, select, update).
The backend, however, no longer recognizes the user as authenticated. The session does not exist anymore in Supabase auth.session table.
Even if the user wants to logout via a button that is using supabase.auth.signOut(), it fails with the error:

{
  code: "session_not_found",
  message: "Session from session_id claim in JWT does not exist"
}

which leaves my users unable to do anything, they can no longer perform actions or log themselves out, I'm starting to get a lot of complaints about this.
To try to debug, I have currently followed these steps:

I read about how Supabase sessions work, I found that sessions are stored in the auth.sessions table
I've compared the cookies on the front side with those received on the back side, and they're exactly the same.
I compared the cookies of a functioning session and the cookies of a session that returns the message "Auth session missing!" when I use supabase.auth.getUser() on the server side. They both contain a sb-access-token and a sb-refresh-token.
By pasting both sb-access-tokens into https://jwt.io/ to see their content, they both contain the same payload, only the session_id key changes (and of course exp (expiration time) and iat (issued at), no problem on that side, the exp was further away than iat)
When I search for the session_id of the working connection in the auth.sessions table, it exists, and conversely, when I search for the one that comes from the cookie that returns "Auth session missing!", it doesn't exist in the auth.sessions table.
Then I tried to change the "Access token (JWT) expiry time" in the Supabase dashboard, and set the JWT expiry to 604800 (one week) instead of 3600 (one hour), but I'm still disconnected after one hour.

Which leads me to think that there's a problem with the session refresh, something must be wrong and the session isn't properly refreshed.
Steps to reproduce
Not easy to reproduce because you have to wait for a period of inactivity in the tab containing the webapp where you are logged in. When I'm developing new features, I regularly get the bug I'm describing here. I come back to the tab and want to perform an action, but I get "Unauthorized" (which is the error returned from the API route when the user is not found).

Login to the webapp
Don't touch the webapp tab for a while
Return to the webapp, and perform actions: front-end actions work, but on back-end user is not considered logged in, and his session no longer exists.

What is Expected?
The refresh session must be reliable and a user can stay connected for two weeks if he wants to.
What is actually happening?
The user is randomly disconnected after one hour, his session is lost and non-existent in Supabase's auth.sessions table.

Thank you for the work you have done with this module.

Can you have a try with the latest release? https://github.com/nuxt-modules/supabase/releases/tag/v1.3.1
You can open it back if it's not fixed.

Can you have a try with the latest release? https://github.com/nuxt-modules/supabase/releases/tag/v1.3.1
You can open it back if it's not fixed.

@larbish On NPM the module is still in 1.2.2, I can't install the latest version for testing because it's not available (yet).
But thank you for all your hard work !

Should be good now!

Should be good now!

@larbish
I've just installed it and I get an error message when I launch Nuxt,
I've even tried deleting the package.lock and the node_modules but I still get this message.
Uncaught SyntaxError: The requested module '/_nuxt/@fs/Users/quentin/Code/my-app/node_modules/@supabase/postgrest-js/dist/cjs/index.js?v=64b02254' does not provide an export named 'default' (at wrapper.mjs:1:8)

Thats point to:

Indeed, I've seen this error which is not happening in the playground... I fixed it in 1.3.2! Sorry, I hope it's ok now.

@larbish Hello, I've done a lot of tests and I still have the problem of the session not refreshing in my app,
I think we can reopen this issue.
I'm asking a lot of questions all over the Supabase ecosystem to understand and solve this problem.
In your own projects that use this lib, if you don't touch the project tab for 1 day and you come back to it, are you still connected or not?

@felixgabler Thanks for your work on the new version which uses ssr!
Sorry to ping you here but I have a small question, I've been trying for a few weeks to solve a problem which is that my users session are deleted from auth.session at some point by something. I've gone through all the docs, read a lot of issues to find out what's going on, but I haven't found the reason yet.
I wanted to ask you if in your project the session refresh works correctly and if you can, for example, close the browser for a few days then come back to your app and still be connected? (In Supabase's Github one of the devs of their Auth lib told me that there was normally no problem with this, but since I've been using this module for +6 months I've never managed to stay connected for several days).
Or more simply, can you be logged in on a tab, not touch it for a day, come back to it and still be logged in?
I used debug: true in the config to see what was going on, and I can see the logs of auth which refreshes regularly, except that when I paste the access token into jwt.io and take the session_id to look for it in the auth.session table there's nothing....
How is it possible to still be connected to the front end, being able to perform actions on the DB, but without a valid session!?
GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.106Z #_autoRefreshTokenTick() begin
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.106Z #_acquireLock begin 0
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.107Z #_acquireLock lock acquired for storage key sb-ywqgmvpallcihcdmurdx-auth-token
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.107Z #_useSession begin
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.107Z #__loadSession() begin
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #getSession() session from storage {access_token: 'eyJhbGciOiJIUzI1NiIsImtpZCI6InhtWHpNWjBtUUVVbjR5V3…xzZX0.8VvSxuea56oynR7L4WdInS4MLSM0hCi3Y4UtIoZILYU', token_type: 'bearer', expires_in: 604800, expires_at:<PHONE_NUMBER>, refresh_token: 'L_HAIMidKYmWcIQ8lUo8zQ', …}
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #__loadSession() session has not expired expires_at<PHONE_NUMBER>
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #__loadSession() end
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #_autoRefreshTokenTick() access token expires in 11378 ticks, a tick lasts 30000ms, refresh threshold is 3 ticks
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #_useSession end
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #_autoRefreshTokenTick() end
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #_acquireLock lock released for storage key sb-ywqgmvpallcihcdmurdx-auth-token
GoTrueClient.ts:266 GoTrueClient@0 (2.64.2) 2024-07-08T17:54:50.108Z #_acquireLock end

I still don't understand where this could be coming from, after a lot of researchs it seems that there are many of people in this situation but no one has found a solution...
If you have any advice or info I'd be grateful,
Thanks again for your work!

Hi! As far as I know, it should not be possible to be logged out in the frontend and still being able to perform restricted DB actions. The DB will receive your authentication credentials from the JWT. I'm sorry if this is no real new insight.
As for the issue where you get logged out after a while, I don't think this happens for us. We haven't received complaints at least and I've stayed logged in over weekends.
I'm hoping things will become more stable with the 1.0.0 release of @supabase/ssr

@felixgabler Thank you much for your reply,
I have the impression that since a few days they have changed something on their side because I have really much less log Unauthorized coming from my users than before while I have strictly changed nothing since I updated this module which includes the SSR.....
I hope as you say that with the release of v1.0 it will be completely stable.
Thanks again for all your hard work!


Access blocked: This app’s request is invalid

I face this error after running on localhost:

Most likely an error in the Google credential. Make sure to add localhost as an approved url in the Google credential.
Check https://developers.google.com/identity/protocols/oauth2/web-server#creatingcred


[Linalg to gemmini] add convert linalg to gemmini pass.

add performance test and batch_matmul pass.

I have added performance tests and linalg.batch_matmul op.

About Performance Testing
Three files are referenced here, tiled_matmul_ws.c, tiled_matmul_ws_At.c and conv.c.


1.0.4 patch train updates

@JunTaoLuo the Localization repo is already upgraded to VS 2017 in the commit https://github.com/aspnet/Localization/commit/d1ee393c72030b05f23f763c6c0152ff94527633, `project.json' is no longer available, could you please react to these changes

Now I notice that these changes will be in rel/1.0.2 branch 😄

ping @pranavkm


add: Support for rejectOnEmpty

Current coverage is 95.90% (diff: 100%)

Merging #10 into master will increase coverage by 0.28%

@@             master        #10   diff @@
==========================================
  Files             1          1
  Lines           160        171    +11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            153        164    +11
  Misses            7          7
  Partials          0          0

Powered by Codecov. Last update 01f0130...03bb26f

v1.4.0


Better pusher support

on_payload_receive for all
on_streamer_live

App\\Events\\StreamerIsLive
Example payload:
{
  livestream: {
    id: 5274636,
    channel_id: 3612810,
    session_title: '✈ WARTHUNDER✈CHINA TREE GRIND!!!!',
    source: null,
    created_at: '2023-06-22T05:50:32.000000Z'
  }
}


Question : does parameter interpolation support function ?

Is this possible to use some kind of functions in parameters interpolation ?
Simple things like tolower or replace are supported or doest it have to be done in bash or something else ?

There isn't any function support in interpolation. Currently any edits you need to make to the contents needs to happen in the command / args / script of the step.

@sbwsg thanks.
one more question, how can i retrieve the branch from a PipelineResource ? is something like this possible $(inputs.resources.clone.branch) ?

You can use $(inputs.resources.clone.revision) i think - this will be the branch / tag / commit SHA or ref that was cloned.

I will have to check how to get the branch information from prow, not related to tekton.
Thanks

@eddycharly having a way to apply simple functions to parameters would be really handy, so I think we should keep this as a feature in the backlog. @skaegi is this something you started looking into?

I was about to close it but ok, let's keep it opened if it serves the backlog purpose.


[Bug]: Desktop version, the data source is invalid and the link is not grayed out

ODC version
ODC4.2.1
OB version
Oceanbase4.1.0
What happened?
Desktop version, the data source is invalid and the link is not grayed out

What did you expect to happen?
Desktop version, the data source is invalid and the link grayes out
How can we reproduce it (as minimally and precisely as possible)?
Desktop version, the data source is invalid and the link is not grayed out
Anything else we need to know?
No response
Cloud
No response

pass


Support per-language overrides for gitlens.codeLens.scopes & gitlens.codeLens.symbolScopes

My Factorio Mod Debug extension adds language support for various Factorio files, but GitLens's default config for the changelog is a bit silly, since I classified the groups as Classes.
I currently set
{
          "language": "factorio-changelog",
          "scopes": [
            "document",
            "custom"
          ],
          "symbolScopes": [
            "Namespace"
          ]
        }

in my settings.json under gitlens.codeLens.scopesByLanguage but this is clunky and I end up suggesting the same to every user too. I tried to apply these settings as a language defaults in my configurationDefaults contribution, but it does not appear to allow setting your settings. What is the intended way for me, as a language provider, to offer default gitlens config for my languages?

In the next release you will be able to set per-language overrides for gitlens.codeLens.scopes & gitlens.codeLens.symbolScopes


Positioning of JSDoc comments

As commented here, block comments are often not where you'd expect them. Please read the specific issue for details. However, it refers to the official CoffeeScript compiler output.
In CoffeeSense, the input is somewhat modified, which is why in the linked case, the following would be enough for in-IDE type hints:
#
###*
# @param a {string}
###
method1 = (a) ->

#
###*
# @param b {string}
###
method2 = (b) ->

Note the extra # on top of the blocks. In many cases, you can also write ``###* ... ### which is even shorter but fails compilation inside objects
This is treated by tsserver as

/**
 * @param a {string}
 */

let method1 = function(a) {};

/**
 * @param b {string}
 */
let method2 = function(b) {};

Maybe there are better ways of achieving proper block comment positioning? Or even better, something like the extra # should be inserted automatically so the user does not have to care about it anymore. This is somewhat annoying as it requires rewriting source maps (similar to how variable declaration rewriting already works), but it's possible.
I want more feedback from smart people on this, so until then, this stays as is

There is one more issue when dealing with classes:
even when writing (note the fat arrow):
#
###*
# @param b {string}
###
method2 :  (b) =>
  console.log b

the output js results in
constructor: ->
  /**
  * @param b {string}
  */
  this.method2 = this.method2.bind(this);
  ...

within the constructor, and within the class body:
method2(b){
  console.log(b)
}

without any doc. So b is still treated as any ;(

@robert-boulanger nice catch! but is there any real reason to use fat arrows for class method definition?

A real world scenario would be (and is) if an instance of such a class is for example, pushed into an reactive array in Vue3.
As soon as a methodof this  instance will be accessed through the Vue component the reactive array is.a member of, Vue creates a proxy out of our instance which causes that this gets lost unless the method is bound to the instance. Means we need a fat arrow.

A real world scenario would be (and is) if an instance of such a class is for example, pushed into an reactive array in Vue3. As soon as a methodof this instance will be accessed through the Vue component the reactive array is.a member of, Vue creates a proxy out of our instance which causes that this gets lost unless the method is bound to the instance. Means we need a fat arrow.

I tried to reproduce what you just described:
Vue 3 SFC Playground example
but I cannot see the class instance's this to get lost (method() can still access the class prop).
Another Vue-related issue I could think of would be a Vue 2 (old-school options api style) component, in which the Vue this gets lost inside the method:
Vue 2 SFC Playground example
here, this.vueProp inside Item::method is not reachable because the class instance's this takes precedence. But you could easily fix this by saving this to a var beforehand, as you surely know anyway. Having a class method invoke side effects is a bad idea anyway, in the latter example, method() should really be part of Vue methods:, not the class
In any case, I don't think the JSDoc comment block positioning issue can be reasonably fixed in extension code :-/ But it should  be with the above linked CoffeeScript issue. I'll add a comment there

Since the positioning of block comments was why this revert happened https://github.com/phil294/coffeesense/commit/0a147d137dae1abeb06f512d4c0de804716df8e2 and it seems block comments are incorrectly positioned anyway... would it make sense to restore this?
It seems like it would fix a lot of my TypeScript issues since the 'empty comment' hack still works with it.

@STRd6 edemaine's branch, which was in use for a short period before being reverted again as you noticed, translates the following
#
###* @type {abc} ###
a = 1
a = '1'
b = 2

into
/** @type {abc} */
var a, b;
a = 1;
a = '1';
b = 2;

which is of course bananas, so it can't be used (yet) and the lsp still uses the baked in workaround which translates into
/** @type {abc} */
let a = 1;
a = '1';
let b = 2;

as expected. (debug with VSCode command CoffeeSense: Show generated JavaScript for current file)
So yes, the comment blocks are always broken without the leading empty comment hack #, but with it, only the latter version is correct.
Hope I understood you correctly.
Either way, I'd be interested in seeing your concrete TS issues that you are seeing because of the current state of the extension, if you are willing to share.

I see what you mean...
It looks like without block comments edemaine's branch works great with destructuring but doesn't work with block comments.
{sep} = require "path"

->
  sep

// Current implementation
var sep

({sep} = require("path"));

(function() {
  return sep; // Later reference inside of a function causes TypeScript to be unable to infer type.
});

// edemaine's branch
var {sep} = require("path"); // TypeScript can figure this out automatically

(function() {
  return sep;
});

I tried compiling with the block comments in various places and found that destructuring and block comments really don't get along either.
For the example you gave here is what I got when running edemaine's branch:
/** @type {abc} */
var a;

a = 1;

a = '1';

var b = 2;

It seems almost reasonable but splitting the assignment onto a separate line isn't great.
I'll try and get a local dev version of CoffeeSense running with the CoffeeScript branch and see what happens.

Time to work on my branch again I guess... Sorry, this fell off my radar for a while and it's taking a while to get back to.

I guess both logics could be combined too, if it's worth the effort

I now did that and it seems to work fine. Idk why I didn't do this from the start. @STRd6 your {sep} example should now work fine with v1.10.0, see also this test. This gets transformed by ede branch already to var { abc } ... even though there is a block comment before it. Not without the something_else = 1 part though, but in that case, the extension transforms it. So I don't exactly know why, but everything works now (?)
.. as long as you do the # prefixing nonsense of course

@phil294 I've been trying out v1.10.0 and it's very good so far! I think most of the remaining issues have to be fixed in coffeescript's comment handling.
Thank you for your work!

Adding some links to CoffeeScript issues for reference:
https://github.com/jashkenas/coffeescript/issues/5417
https://github.com/jashkenas/coffeescript/issues/5415
Someday I may build up the courage to tackle these in a CoffeeScript PR.

The ```###* ... *###` syntax is a lot easier to remember, works great, thank you!

I'm wondering if there's value in adding a vscode command to insert a coffeesense / coffescript jsdoc comment stub. I keep forgetting the correct syntax and always come back to this github issue to remember.

@surjikal I think it depends on where you use a comment block - sometimes ``### works, sometimes #\n###, sometimes a lone ###.
I haven't found a one-fits-all solution yet unfortunately (both in front of variables and object properties), which is why the extension can't do it for you. But maybe there is. If you have found one and worked with it for a while and it works reliably, please let me know! Else we'll just have to wait for someone to tackle the CoffeeScript compiler enhancements.

As things have gotten stale around here, I have had another look into this from the extension's point of view and fixed it there now. CoffeeSense now automatically transforms all of your
### something ###

into
#
### something ###

, also multi-line comment blocks, so you don't have to do that or the ``### thing anymore when you're working with ###* @... ### kind of JSDoc elements. I've noticed some of you do that, so you might want to update your code now! (unless you're also working with an external pipeline of course)

Thanks for the hard work, it's a really great extension!

Hm.. for some reason, the
###*
@typedef {foo} bar
###

has stopped working recently. Using the backticks make it work though:
``###*
...
###

Looking at the compiled source produced by coffeesense, without the backticks, the comments are being stripped out. I'll post back here if I find a workaround.

Hi @surjikal, that's strange as I haven't pushed an update to CoffeeSense in a long time. So I guess you found a bug that has been there all along. If you can narrow this down to a smallest possible reproducable single file snippet, maybe it'll be well fixable.


Allow Symfony v5

Hi @OwlyCode
I plan to use this bundle inside a Symfony v5 app, that is why I opened this PR (draft for now)
Also, do you think we can create a https://github.com/symfony/recipes-contrib ?
TODO:

create a pre release tag so we can test it
discuss the recipe creation

improve the doc if the recipe is created

Thank you

Friendly ping @OwlyCode

Hello and sorry for the long delay. I've currently very little time to maintain this library. I would even consider giving it away if someone is interested because it deserves better.
This PR is great, could you have a look at why the tests are failing? Thanks!

@OwlyCode thanks for the reply, of course i will have a look at the tests
The bundle is "quite simple" in its job, there is not so much things it deserves more imho that what is already here :)
I think just making it Symfony v5 ready is enough, even if not "maintained" anymore
I may ping you back when this PR is finished/ready 👍

@OwlyCode maybe PHP v5.4/v5.5 are no more relevant here? or with an other branch/tag?
tests are green otherwise

Hi,
@OwlyCode I'm open to follow your work on the lib if you agree.
Regarding this PR, I agree with @94noni, we should do a new major version and update the composer dependencies.

Hello @Deamon I don't know how I managed that but your message went unnoticed. Are you still ok to take on the lib?

Hi @OwlyCode, I still agree to take the lib.
To be honest, it might moved it to my company's namespace, because I might not be alone to work on it.
Is it fine for you ?

👋🏻 ping @OwlyCode @Deamon :)
is there new things related to this PR?
Or should I fork myself
Many thanks

Hello, any news on this PR? Or any alternative?
This is blocking our project to upgrade to Symfony 5.
Failing tests are unrelated. It happens because old PHP version image are no more available to download.

@Deamon I initiated the transfer, you should be able to take over now. Have you got a packagist account so I can add you on it too?

@Deamon I initiated the transfer, you should be able to take over now. Have you got a packagist account so I can add you on it too?

Nevermind, I found you on packagist. Let me know if you are able to migrate the repository from there too.

@Deamon I initiated the transfer, you should be able to take over now. Have you got a packagist account so I can add you on it too?

Thanks @OwlyCode,
I don't know how long it take to be notified for the transfert by Github but I haven't recieved anything yet.
I see the package in packagist.

@Deamon will you then merge this PR and tag a new release?
thank you

@OwlyCode I finally received the transfert email, but after the link was expired. Could'd you reiterate the opération ?
sorry for the inconvenience.

Closing as I now use an internal/private fork
@Deamon please feel free to report back when you take over this projet/PR for symfony v5 :)
Thank you

Repo at https://github.com/94noni/NoniHtml2pdfBundle

Well, I thought I would be able to manage this but I havn't seen those few months.
@94noni do you want to take part as a collaborator to this repo ?

@Deamon no i prefer to continue on my repo
The readme and composer give explaination and credit to original author
Thanks

I can transfer the ownership to you, would make sense if you continue to work with the bundle.

Hum for now there is nothing more i want/need to add on it
For me it is ok to keep like this :)

Ok, feel free to come back and ask for it if needed.


Pass curl options to Guzzle?

How can I pass curl options directly to Guzzle?
I'm running into a problem with the PayPal Express gateway and the solution that I'm finding online means passing options directly to curl, like
curl_setopt($ch, CURLOPT_SSLVERSION, 3);
curl_setopt($ch, CURLOPT_SSL_CIPHER_LIST, 'SSLv3');

I've tried passing it in the parameters array of the purchase method, but no luck:
$options = [
        'amount'        => $amount,
        'currency'      => $currency,
        'card'          => $this->card(),
        'returnURL'     => $this->returnURL,
        'cancelURL'     => $this->cancelURL,
        'curl'          => [
            CURLOPT_SSLVERSION => 3,
            CURLOPT_SSL_CIPHER_LIST => 'SSLv3'
        ]
    ];

Seems to have been answered here: http://stackoverflow.com/questions/35415273/how-to-pass-curl-options-through-omnipay-to-guzzle

An answer was given, but not a correct answer.
And I came to the official OmniPay issues to specifically find out if there was a way to pass options, which was only answered there by hacking a core file (/Message/AbstractRequest.php). Are you confirming there is nothing built into OmniPay to allow passing options on to curl?

I will repeat my earlier statement that setting the SSL version to SSLv3 will not work. You need to set it to TLS v1.2 which is what PayPal require. This should automatically be handled by libcurl. I have verified it myself on libcurl v7.44 on CentOS 6.
Correct, although you need to try changing the gateway code to explicitly request SSLv3, not the core code. This is an issue with the omnipay-paypal gateway not omnipay core.  Each gateway plugin handles its own communication via guzzle or some other mechanism to the gateway itself.
The correct place to request support for omnipay issues is stack overflow, as per the README files.

Per my example, I was not just setting SSL to v3, but was also setting the cipherlist, though I think I placed the wrong value in the ciperlist value above, so thanks. But my understanding is that setting the cipher list will force the minor version of the TLS being used.
Didn't realize there wasn't a core method to interact with Guzzle. Must have been confused with my years working on OpenGateway. Sorry about that.
to be fair - the readme also says you can bring more detailed questions here and, since I thought this might end up being a feature request. Also, I've always thought that issue lists on GitHub were more direct access to the developers, while StackOverflow was community support. My apologies for mis-understanding. Will head over there and make a comment so future people visiting will know the solution, and to accept the answer, then, I suppose.

thanks for your help.


[Bug]: Page properties non-CTA icons have too many cursor icon types

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Cursor icons (point, grab and arrow) needs to be defined even for the boundary of icons that don't support a CTA (Visible, home and disabled home page delete icon). See attached video to see how

Steps To Reproduce

Go to Page Properties and hover around the icons present for each page and observe

Environment
Release
Version
Cloud

Closing as page setting moved to property pane


[Commutiy]: Prompts Fixed for ZERO_SHOT_REACT React Agent Type in create_sql_agent function

Description: The correct Prompts for ZERO_SHOT_REACT were not being used in the create_sql_agent function.  They were not using the specific SQL_PREFIX and SQL_SUFFIX prompts if client does not provide any prompts. This is fixed.
Issue: #23585

@ccurme

@ccurme

@ccurme


Cache is bypassed when v8r is invoked using npx (and not installed locally)

The default cache path for flat-cache is node_modules/flat-cache/.cache. That's fine if you npm install but if you don't have it installed locally
npx filename.json
npx filename.json

doesn't use the cache second time round because npx installs a clean copy of the package each time. This isn't a huge deal because if you care about speed, you won't want to install a new copy every time you run it (which takes longer than the validation itself). Equally this could be solved by setting cacheDir to somewhere outside node_modules.

changed in 0.5.0

changed in 0.5.0


Training convergence

My model training appears to be poor in covering to decreased validation loss, while the validation accuracy gets to 0.75-0.8 after just a couple of epochs, and then just bouncing around that range without improving.

Is that reasonable? Assuming it's not, what could explain that? Patches not capturing the segmentation classes appropriately or having poor resolution (I'm using 1024 pixel patches which are then scaled to 320 pixel patches as the required input size in the multi-class segmentation tutorial), or alternatively having the architecture/loss defined not well?

Are you reducing the learning rate after each epoch in which the validation loss does not decrease?

I thought it's being done by the model in the example discussed here, but I don't see it. I just see LR = 0.0001. How do I make it change the learning rate when the loss doesn't decrease?

By default, ReduceLRonPlateau starts reducing after 10 epochs of no continued decrease. I would change the default parameters and also change verbose to 1 so you can see when and what it is being reduced to. This might help.
What percentage of the data is going to training and validation, also, are you using augmentation?

Are there any parameters you recommend starting with or parameter ranges to experiment with?
I'm current using the following and it doesn't converge well.
BATCH_SIZE = 5
LR = 1e-3
EPOCHS = 100
keras.callbacks.ReduceLROnPlateau(monitor='val_loss', factor=0.2, patience=5, min_lr=1e-7, verbose = 1)
Thanks

The reason for the OOM error from the other post is likely due to your batch size being 5 instead of 1. Also, there is some merit in using batch sizes as powers of 2.
How many images do you have to use?


Remove some peer dependencies

Using peer dependencies with our build tooling is unneeded because the tools are intended only for use in our repo, and peer dependencies are a hassle for downstream consumers.
Note that we have legitimate peer dependencies declared in the azure-client and Property DDS packages. Those dependencies are not addressed in this PR. See #7913 for more details.

@heliocliu @curtisman @skylerjokiel @ChumpChief I made this change originally just so build-tools wouldn't have the rushstack eslint config as a peer dep, because it wasn't needed and caused problems. But then I figured the rationale applied to these other uses within our common/tools packages. I don't think peer deps get us anything in these cases, so I removed them all.


X-Forwarded-Proto always set to https

Right now X-Forwarded-Proto is always set to https even though Aptible will listen on http.

@cancan101: How are you reproducing this? X-Forwarded-Proto should only be set in the server that's listening on 443, not the one listening on 80.

Issue would be if I have an nginx service that received on traffic on http / 80 but then connects to upstream on https. Assuming the upstream is a private VHOST, it should be able to trust the X-Forwarded-Proto set by nginx. Right now I think that X-Forwarded-Proto will be set to https even though the user connected originally to the nginx on http.


Add python API documentation

This is already there under compatibility. Seems this section is missing in the docs website.

@supunkamburugamuve does this get resolved with the new docs?


Exit code should match test results

Got a failure here: https://github.com/apache/camel-kamelets/pull/55/checks?check_run_id=2094300651
But the test ended with success (i.e. I think it exited with 0). Using YAKS 0.3.0.

Hm, this used to work properly. Let me see why this is broken


HIVE-21198 Introduce a database object reference class

The amount of possible refactors is a lot bigger then comfortable in 1 issue, so I'm reducing the scope for this and opened HIVE-21630 as an umbrella ticket.

This got meged as 44697f0aa766a8329a599572013e9ba0a6750637. Thank you for the review, I'll include these in the followup.


Reminder: Clean out the dead branches of code and make a new repository for 2015!

Please and thank you (we have new members from comp sci and comp engineering who are very interested in our code base).

We're going to continue using the same repository, just tagged with a new version number release.

You could change the repo name without much breakage.
On Tue, May 26, 2015, 8:06 PM Ahmed Samara<EMAIL_ADDRESS>wrote:

We're going to continue using the same repository, just tagged with a new
version number release.
—
Reply to this email directly or view it on GitHub
https://github.com/IEEERobotics/bot2014/issues/363#issuecomment-105701270
.

I've changed the repo name to 'bot' unless anyone has a suggestion for a better name,

+1 to bot (original name)
On Fri, Jun 12, 2015, 3:06 PM Ahmed Samara<EMAIL_ADDRESS>wrote:

Closed #363 https://github.com/IEEERobotics/bot/issues/363.
—
Reply to this email directly or view it on GitHub
https://github.com/IEEERobotics/bot/issues/363#event-329870008.

We still need to clean the dead branches, there are so many that its tough to tell which ones are useful

I could help closing the branches I started. If Ahmed does some of his we would be in pretty good shape.

Closed all defunct branches.


Allowed multiple ignored watch patterns

Allows for multiple ignore patterns while watching as is allowed here.

@Buzzec thanks for the PR. looks good. just will validate the CI is not broken and i'll merge it.

@Buzzec i'll fix the tests. thanks a lot for the PR


chore(protocol): Re-organizes Modules and Errors

Description
Removes public visibility from modules so crate docs don't show all the pub use mod::* types as re-exported.
Also moves error types from utils into a new errors module.

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#261 👈
main

This comment was automatically generated by st.


⚠ Bofrost.at has degraded performance

In 39f9013, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 3109 ms

Resolved: Bofrost.at performance has improved in 12798b3.


Added SMT query hashes to hints to skip unchanged queries.

@nikswamy: the basic functionality is in place. Whenever the hashes match, it reports Query-stats ... succeeded (with hint) ... statistics={fstar_cache_hit=1}. It will also read old hints and then update them with hashes when --record_hints is on. It will use hashes whenever --use_hints is on, but I suppose it would be a good idea to add a command-line parameter to "opt in".

Hi Christoph,
Many thanks for this feature.
In its current form, the PR is very hard to review since it contains many auto-generated files (.ml, .hints etc.)
Do you think you could submit another version of the PR with just the changes that actually need to be reviewed?
As for timing: I would wait for the ongoing Everest merge to stabilize and then try to update our projects and all hints to use this feature.
A question: will it possible to re-use old hint files with this feature turned off (i.e., just --use_hints with old hint files). Or, are we forced to upgrade all hint files as soon as this feature lands?
Thanks,
Nik

Agreed, it's a bit of a mess at the moment. I checked in new snapshot files because the extraction that worked perfectly fine on my machine didn't go through the CI for reasons that weren't obvious from the log files. I may have been using a different version of OCaml or some such thing.
Feature-wise, yes, old hint files will be read in exactly the same way as before when --use_hints is on. They will only be upgraded with the extra hash field when --record_hints is on. I also added --use_hint_hashes so the by default the behavior is exactly the same as before. Something's still wrong with that flag though, so don't merge just yet.

We've completed the Everest merge of last week. So, I'd be happy to have this feature completed and merged in as soon as you're done.

Working on this, should be done by the end of today!

Features are there, but the OCaml extraction still won't work on the CI machine and I'm not getting any useful error messages. Apparently, the snapshot code extracts different code from the same sources when run on the CI machine, but so far I wasn't able to determine why.

Nik, there's a dropdown in the Github UI that allows you to pick which commits to display :) Just exclude the snapshot commit.

I'm excited about this feature :) IIUC, it's essentially an implementation of https://github.com/FStarLang/FStar/issues/1140, right?
How well will this work with the interactive mode? Are the caches constructed as we go, so that reprocessing the same query will now be much faster? That would be particularly nice.

We have this bit in the interactive mode:
  if FStar.Options.record_hints() || FStar.Options.use_hints() then
    FStar.SMTEncoding.Solver.with_hints_db (List.hd (Options.file_list ()))
      (fun () -> go init_st)
  else
    go init_st

Does it need to be updated?

I second Clement's excitement about this feature. If this could be made to work well in interactive, it would be a big productivity boost. But F* generates fresh identifiers on the fly, so as for now a query issued a second time will differ syntactically (mostly alpha-renaming, I believe) from the first time it was ran.

But F* generates fresh identifiers on the fly, so as for now a query issued a second time will differ syntactically (mostly alpha-renaming, I believe) from the first time it was ran.

Good point.  I don't expect this would be so hard to fix, though — probably just an issue of rolling back the gensyms when we step back in the interactive mode

@cpitclaudel: I think that part should be fine, we still need to load the hints, regardless of whether we use or ignore the hashes.
@s-zanella: Absolutely! There are some issues to think about, e.g. variable renaming, ordering, incrementality, etc, but I'm sure we can come up with something that's useful in interactive mode as well.

Just pushed a few improvements. This version takes less memory (smt2-string only generated once), but it's still not quite as fast as I'd like it to be. Ideas for optimizations are most welcome!

Can you rebase and move the snapshot to the end? Or is there a way on github to exclude a snapshot that's in the middle of the history?

I don't think so, that would break the history. I can force-push after I hack the history locally, but I wouldn't want to do that after comments have been added (I think they would just disappear). The changes in fd15e83 are very small and local though.

Regarding this question from @cpitclaudel

We have this bit in the interactive mode:

  if FStar.Options.record_hints() || FStar.Options.use_hints() then
    FStar.SMTEncoding.Solver.with_hints_db (List.hd (Options.file_list ()))
      (fun () -> go init_st)
  else
    go init_st

Does it need to be updated?

I don't think so. The new feature is only enabled under FStar.Options.use_hints() && FStar.Options.use_hint_hashes(). So, this code should be ok as is.

@nikswamy thanks for adding comments, I totally neglected that part of the job!


cluster-repo is missing OWNERS file

Hi guys,
it seems that cluster-repo is missing OWNERS file, so accepting PR isn't working as expected (I cannot approve PR that was created by me with my-jx-bot)
Or is OWNERS file missing be design?

I'm pretty sure this is fixed by know


[cmake] add support for Conan, a C++ package manager

It would be a nice feature to be friendly with Conan.
It allows to propagate easily flags for:

C++ language standard
ABI (ABI incompatibility between GCC version)
runtime libraries with Visual Studio
and so on...

@bryceschober is kindly managing a repository to host the conan recipe for CppUTest.
While trying to set up a CI on AppVeyor, he met a lot of link issues related to runtime incompatibility (https://github.com/bryceschober/conan-cpputest/issues/5).
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.14.26428\bin\HostX86\x86\link.exe /ERRORREPORT:QUEUE /OUT:"C:\projects\conan-cpputest\test_package\build\39d1509292314c21074b4137bae955683dd6acf9\bin\example.exe" /INCREMENTAL:NO /NOLOGO /LIBPATH:C:/Users/appveyor/.conan/data/CppUTest/master/bryceschober/testing/package/ea63b9b23f301d1bee2b08a4f9d864eda3d538a3/lib /LIBPATH:C:/Users/appveyor/.conan/data/CppUTest/master/bryceschober/testing/package/ea63b9b23f301d1bee2b08a4f9d864eda3d538a3/lib/Release /LIBPATH:"C:\Tools\vcpkg\installed\x86-windows\lib" /LIBPATH:"C:\Tools\vcpkg\installed\x86-windows\lib\manual-link" CppUTest.lib CppUTestExt.lib winmm.lib kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib "C:\Tools\vcpkg\installed\x86-windows\lib\*.lib" /MANIFEST /MANIFESTUAC:"level='asInvoker' uiAccess='false'" /manifest:embed /PDB:"C:/projects/conan-cpputest/test_package/build/39d1509292314c21074b4137bae955683dd6acf9/bin/example.pdb" /SUBSYSTEM:CONSOLE /TLBID:1 /DYNAMICBASE /NXCOMPAT /IMPLIB:"C:/projects/conan-cpputest/test_package/build/39d1509292314c21074b4137bae955683dd6acf9/lib/example.lib" /MACHINE:X86 /SAFESEH  /machine:X86 example.dir\Release\example.obj
     3>CppUTest.lib(Utest.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MT_StaticRelease' in example.obj [C:\projects\conan-cpputest\test_package\build\39d1509292314c21074b4137bae955683dd6acf9\example.vcxproj]
     3>CppUTest.lib(MemoryLeakWarningPlugin.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MT_StaticRelease' in example.obj [C:\projects\conan-cpputest\test_package\build\39d1509292314c21074b4137bae955683dd6acf9\example.vcxproj]

It appeared that conan set up correctly the flags in CONAN_LINK_RUNTIME but they were ignored by CMakeLists.txt from CppUTest. While building the test package (with CMakeLists.txt using conan_basic_setup()), it sets up the good flags but then the example object file conflicts at the link time with CppUTest.lib.
This pull request would solve the above issue.

Coverage remained the same at 99.874% when pulling bfc278a70c267b809518d4a1ec071065f49b72e3 on matlo607:feature-conan-support into 77c10f9da108c530a1140d619beb1853189336ff on cpputest:master.


Fix URL-concatenation in HttpRepositoryRef

This should not blindly concatenate provided URL and /schemas/. Also we can add preliminary validation of uri as not all URIs can be correct endpoint.

What should the behavior be instead?

We have few options:

Force absence of trailing / in resolver-config schema
Failing with meaningful error message if trailing slash is there
Removing trailing slash if it is present: if url.enswith('/'): url.rstrip('/') else url

I'd like to go with third option.

Sounds good

@chuwy  - Is removing the trailing slash for url at HttpRepositoryRef.scala the only change ?

@saj1th actual logic should also check if trailing slash is already present. If not - add it. Otherwise - yes.

@chuwy
did you mean if already present remove, and not present - do nothing.
More like
val urlNorm =
  if (url.endsWith("/")) url.substring(0, url.length - 1) else url

Looks like a trivial change - but happy to help :)

That's right. Trivial change, significant UX improvement.
I'd just change url.substring(0, url.length - 1) to url.dropRight(1) as we don't want to mess with indexes too much.

👍

This was incorporated into #113


Bug report

the youtubetogether command when using it says that the discord does not have permissions when I have set all the permissions

That's not the bot's permissions.
That's the server or your id problem as probably its not registered for youtube together that is in beta now.
Follow these steps:
Step 1: Open YouTube and select the video you want to watch with friends.
Step 2: Open Discord and join a channel to start streaming the video.
Step 3: Then look for 'Screen' and click on it.
Step 4: Then select the window that you want to watch with your friends.
Step 5: Make sure that the 'Sound' toggle button is switched on.
Step 6: Then click 'Go Live'.
Step 7: The video is available to watch. Now, send an Invite to your friends to watch the video together.

@BestGamersH  be smart its a error in the code


Fix for Issue #71: Repeated modifies clause crash.

This is a fix for Issue #71 on github, which solves a boogie crash that occurs when an inlined procedure has multiple modifies clauses for the same variable. This fix silently ignores the repeated clause.

I will merge this in since it is likely not the cause of the one failing regression. I will revert it if it is.


Install for users only, fix user environment variables

https://github.com/uiuing/goplus_installer-online/commit/444d3b85e5cb1a4bddfbb57e6198030403687713


updated DSAdmin page for Docker

Added Instructions to run DSAdmin in Docker Setup.

Note!!
Merge after PR#20425 in Oppia gets merged.

@seanlip Updated with required changed!!

Thanks! I will merge once other PR gets merged.
On Sat, Jun 8, 2024 at 8:33 PM Sean Lip @.***> wrote:

@.**** approved this pull request.
Thanks, LGTM! Merge whenever you like :)
—
Reply to this email directly, view it on GitHub
https://github.com/oppia/oppia-web-developer-docs/pull/330#pullrequestreview-2105865256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWL2L6J6436K6XEAWNPE4ALZGMMMLAVCNFSM6AAAAABJAA5GN2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMBVHA3DKMRVGY
.
You are receiving this because you were assigned.Message ID:
@.***>

@jayam04 This has been open for 2 weeks, are we planning to merge it?

@jayam04 Ah -- no, I think we should get the other PR merged so that this page is correct and we don't confuse contributors. What's blocking the other PR, can we get that in soon?

@seanlip, this PR can be merged now.

@U8NWXD Can you please commit this PR.


Offline transactions: remove the ability to repeat transaction

Offline transactions: remove the ability to repeat transaction from "transaction details" screen. SBBS addres is inserted not address

Checked


ARROW-6836: [Format] add a custom_metadata:[KeyValue] field to the Footer table in File.fbs

This is ARROW-6836 ... a separate issue (ARROW-6837) pertains to accessing the new field.  Unsure whether these are separate issues and/or pull-requests.  In case they are separate, here is the format one.

I think it's OK for this to be separate as long as we follow up quickly with implementations in C++ and Java

Did we have a vote for this already? do we need one?

@wesm thoughts?

@emkornfield I am OK with merging this without a vote since it makes this table consistent with other metadata containers where we have custom_metadata. Sound good?


delete a computer from mouse without borders

Description of the new feature / enhancement
Delete a computer from the computer list from mouse without borders
Scenario when this would be used?
right click on computer, delete
Supporting information
if one of the computer is no more used, it remains in the list and as it can't be reached, there is a lot of notification
I would like to delete it easily wihtout uninstalling powertoys on all computers

/dup #30897


No SMS to my mobile when I tried connect using RDP Gateway

I tried review all my configuration the my Azure account, NPS and RD gateway adn active diretory twice  , when I tried connect by RDP the PC , everthing works fine , but to put 2 authentication security using RDP Gateway  erver settings I receive this error:
Anyone can help with this ?
Follow the error
[Content]
Remote Desktop can’t connect to the remote computer "<IP_ADDRESS>" for one of these reasons:

Your user account is not authorized to access the RD Gateway "SERVER.LABDEMO.com"
Your computer is not authorized to access the RD Gateway "SERVER.LABDEMO.com"
You are using an incompatible authentication method (for example, the RD Gateway might be expecting a smart card but you provided a password)

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

ID: 61df48bb-a963-b1f0-8f64-b55d0bd6904c
Version Independent ID: 677de59f-a222-8905-11c0-a3ac619d8b56
Content: Remote Desktop Gateway integration with Azure MFA NPS extension
Content Source: articles/active-directory/authentication/howto-mfa-nps-extension-rdg.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@franciscorenault
Thanks for your feedback! We will investigate and update as appropriate.

Text message and verification codes are not supported at this time. The code for RDP was written in the early 1990s and has not been updated to support two-way text message authentication. The supported methods are cell phone number, landline number, and an application on a mobile device.
If you still have these errors when using the supported authentication methods, please check the steps in the troubleshooting guides that addresses these errors. https://blogs.technet.microsoft.com/networking/2010/01/14/remote-desktop-gateway-client-fails-authentication-with-your-user-account-is-not-authorized-to-access-the-rd-gateway/
https://support.microsoft.com/en-us/help/2578133/the-remote-desktop-gateway-service-incorrectly-blocks-a-user-account-w

We will now proceed to close this thread. If you have further questions feel free to tag me in the comments and I will gladly continue the discussion.

all this link I saw before , please I googled and didnt find any answer , Someone really had this issue before , I need hell on this issue

We will now proceed to close this thread. If you have further questions feel free to tag me in the comments and I will gladly continue the discussion.

all these links I saw before , please... I googled and didnt find any answer , Someone really had this issue before ? , I need help on this issue


Autogenerate dataset name field at publication

Note, will need to convert the actual Dataset.data_types values to the corresponding descriptions in this yaml: https://github.com/hubmapconsortium/search-api/blob/test-release/src/search-schema/data/definitions/enums/assay_types.yaml

This is related to https://github.com/hubmapconsortium/entity-api/issues/152


support JDK15 Sealed Classes

https://openjdk.java.net/jeps/360

https://github.com/openjdk/jdk/commit/d42bfef8a44673aa9ae993d3f5bad954add5f6c9
getPermittedSubclasses does not contains JDK15? 🤔 since JDK16? 🤔


[Not for Merge] Disable ReactPerf and Update Systrace React Integration

This is a work in progress, not for merge yet.
We will likely disable ReactPerf in Fiber because it's not clear how to make it work correctly, and whether the heuristics it used even make sense in Fiber.
Instead, we are now hooking into User Timing API API if one is available in the environment, to report the React measurements. In this PR, I applied an RN sync manually (which is necessary to test this change until we land Fiber support), and implement the necessary User Timing API subset on top of Systrace in the second commit. The corresponding React PR is at https://github.com/facebook/react/pull/9071.

First commit: React sync + enabling Fiber (irrelevant to this PR) https://github.com/facebook/react-native/commit/666a00498ec4705f785d69b35bfac8c9a944d7c1

Second commit: Replace React debugging code with a User Timing polyfill for Fiber https://github.com/facebook/react-native/commit/4fb86f7d5f3f9be76fec7d59de9808a0404d1821

The result is more or less equivalent to the Systrace React integration we had before, although it also displays some Fiber-specific features (such as a separate commit phase). It also doesn't rely on private React APIs and is consistent with what we use for the browser, so we shouldn't be breaking this integration again in the future.
One unfortunate problem here is that there is no way to re-label Systrace measurements after they happened. In the browser, we are able to rename measurements to include warning messages (since by the time the measurement ends, we know if user did some bad pattern). However, Systrace can only include the label when starting the measurements, so the warnings are not exposed.

cc @bvaughn — this is for me to pick up after you do the next sync

@gaearon FYI landed the 16.0.0-alpha.4 sync yesterday

What's the status of this? Is there currently any way to profile JS Thread events? I'm really struggling to pinpoint performance bottlenecks with current tooling 😞

@sdeleon28 For what it's worth, the new DevTools profiler plug-in will support the next React Native release.

@bvaughn that's useful info. Thanks!
Still doesn't change the fact that I have zero tools to identify the bottleneck I'm currently facing, tho :(
I'm going to give console.time and console.profile a try. Less than ideal, but this is my last resort - Suffocation / No breathing / Don't give a BLEEP if I cut my arm, bleeding


Setting value prop immediately after component initialization may not work

Expected behavior:
CodeMirror should always render the value of value prop.
Bug
Under rare circumstances, the value of value may not match what is rendered.
For example, the final value that is rendered in CodeMirror should be "second", but can sometimes be "first".
<script>
let value = "first";
onMount(() => {
    value = "second"
})
</script>

<CodeMirror bind:value />

This bug can be solved by not returning on this line, so that editor state is always updated when value changes.
This bug may be caused by this line, if view and value change at the same tick, the second change of value may be marked as the first change and update is skipped.

Hello,
Sorry for my late response.
I'm not sure we can fully disable the debounce behavior because when value change, the entire context need to be redrawn.
An option could be to add an option to disable this behavior when you have full control on your performances.
What do you think?

Hello @Lifulifu,
Thank you for reporting this issue.
We added a new option nodebounce in v1.2.0.
You can now do:
<CodeMirror bind:value nodebounce />


Skip metadata when downloading files list in admin, therefore make improve speed significantly

Task linked: QF-2936 [Admin] Speed up list file by skipping the metadata (sha256)


getting this exception with new version 3.2.0

Compiling lib/main.dart for the Web...
Target dart2js failed: Exception: /root/.pub-cache/hosted/pub.dartlang.org/infinite_scroll_pagination-3.2.0/lib/src/ui/paged_sliver_builder.dart:254:33:
Error: Method 'addPostFrameCallback' cannot be called on 'WidgetsBinding?' because it is potentially null.

'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('/sdks/flutter/packages/flutter/lib/src/widgets/binding.dart').
WidgetsBinding.instance.addPostFrameCallback((_) {
^^^^^^^^^^^^^^^^^^^^
Error: Compilation failed.

Same here

Just accept the pull request please

Thank you for the lib. Same issue? I revert to 3.1.0 and the problem's gone.
../../FlutterProjects/flutter/.pub-cache/hosted/pub.dartlang.org/infinite_scroll_pagination-3.2.0/lib/src/ui/paged_sliver_builder.dart:254:33: Error: Method 'addPostFrameCallback' cannot be called on 'WidgetsBinding?' because it is potentially null.

'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../FlutterProjects/flutter/packages/flutter/lib/src/widgets/binding.dart').
Try calling using ?. instead.
WidgetsBinding.instance.addPostFrameCallback((_) {
^^^^^^^^^^^^^^^^^^^^

FAILURE: Build failed with an exception.

in flutter 3.0, WidgetsBinding.instance is no longer nullable.
I assume, upgrading to flutter 3 should resolve your issue.

I've solved this error putting a null-check on /.pub-cache/hosted/pub.dartlang.org/infinite_scroll_pagination-3.2.0/lib/src/ui/paged_sliver_builder.dart to don't execute function if the instance is null...
On the line 254 where it's written:

WidgetsBinding.instance.addPostFrameCallback((_) {

change it to:

WidgetsBinding.instance?.addPostFrameCallback((_) {.

version 3.2.0 of this package has specifically removed that so that it removes a warning in flutter version 3.0.
please upgrade to flutter version 3.0. if you do not plan on upgrading to version 3.0, then there is no reason to use version 3.2.0 of this package. you can downgrade to version 3.1.0.


Add has() and unique() functions

Closes #14

Any plans to tag a new release after this?

@Petah yes I really should. I'll be dropping <7.1 support with the next release, which will open up a ton of opportunities around type hinting, and being able to drop a lot of the reflection now that I can spread into a new.

@jkoudys when do you plan to release?

 I don't think anyone on an ancient WordPress install running on some forsaken 5.4 server somewhere is going to use immutable.php .


HDDS-2541. CI builds should use merged code state instead of the forked branch

What changes were proposed in this pull request?
As of now the github actions based CI runs uses the branch of the PR which is the forked repo most of the time.
It would be better to force a rebase/merge (without push) before the builds to test the possible state after the merge not before.
For example if a PR branch uses elek/hadoop-ozone:HDDS-1234 and request a merge to apache/hadoop-ozone:master then the build should download the HDDS-1234 from elek/hadoop-ozone AND rebase/merge to the apache/hadoop-ozone before the build.
This merge is temporary just for the build/checks (no push at all).
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2541
How was this patch tested?
This is under testing with the current PR. With the following CI runs the rebase should be visible from the build output.

Seems to be working https://github.com/apache/hadoop-ozone/runs/309786238 has successful rebase.
In the mean-time the master has bean cleaned up: now I am re-pushing the same change with the same parent to this branch. It should be green if it's properly rebased.
(This pr includes dcfe5f34 but not 29e85bbf from the master. Both of them are required for a green build.)

Seems to be supported out of the box for pull requests. Closing while investigate.


Small datatype changes to get tests working on spark

This is a(n):

[x] under-the-hood change

closing as i've fixed these in the 0.6.5 refactor. I promise we're going to get to a cleaner spot (I hope!)


Storage Permissions tasks is missing from guide

Hi, seems that the description for tasks to grant app permissions for storage is missing from guide now. Only the image after "The app configuration policy for Defender for Endpoint autogranting the storage permission is now assigned to the selected user group.", but above does not talk about need to create this policy.

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

ID: 60db9275-9265-ebca-a145-3a3eede2353a
Version Independent ID: 9d19ab3c-eb32-4211-14d9-559998589eb3
Content: Deploy Microsoft Defender for Endpoint on Android with Microsoft Intune
Content Source: microsoft-365/security/defender-endpoint/android-intune.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@msbemba

@Kazzan Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9141
Once the author reviews and approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @Kazzan Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Confirmation behavior Replace does not work

I have copied from lua/cmp/config/default.lua and modified:
  confirmation = {
    default_behavior = cmp.ConfirmBehavior.Replace,
  },

It worked previously (see #1115) but currently it works like for Insert. Using git bisect I found that it's broken from 49c4050e.

I can confirm that I am seeing the same behavior

https://github.com/hrsh7th/nvim-cmp/pull/1148 fixes your problem?

Yes, it fixes confirmation, thanks. Please respect it in select_next_item and select_prev_item too.


Make the Intl polyfill optional

formatToParts landed in the Chrome beta and newish builds of Firefox. That completes the story for them in terms of polyfilless support. Just need to figure out the best way to build this to optionally have it available.
Note that the tests don't pass b/c there are some simple differences in Intl strings, like where commas go, that sort of thing. For now we'll always use the polyfill for tests, even though their Intl strings are objectively worse, because it lets us keep the test code the same across platforms.

Going to wait until formatToParts lands in Node (should be next month), then rip out the explicit use of Intl and remove it from the dependencies. It'll be up to the user to inject it.

Folding into the todo list


273 ability to mark vet concerns and incidents as resolved

Change Summary

Added a button to allow vet concerns and incidents to be resolved.
Changed query to only display unresolved vet concerns and incidents

Change Form

[x] The pull request title has an issue number
[x] The change works by "Smoke testing" or quick testing
[ ] The change has tests
[ ] The change has documentation

@scratchclaggy

Everything looks good to me, but I'm having trouble testing it. I'm guessing you've confirmed it works as exprected? Also some conflicts to clear up here before the final approval.

You're right, I just tested it on the preview and it doesn't seem to be working. I think it has something to do with having a query on multiple fields, which requires indexing? I've fixed it by moving the filtering for unresolved statuses to the client side, let me know if this would be an issue.


Test against Python 3.6.

Ping @pinkerton

Amazing, thanks for updating this! Apologies for the delay.

You're welcome, no worries about the delay.


Two Pointers: Determine if a triplet exists in an array which sums up to a target in Python

Given an array of integers, nums, and an integer value, target,
determine if there are any three integers in nums whose sum equals the target.
Return TRUE if three such integers are found in the array. Otherwise, return FALSE.
Sample Input : 3, 7, 1, 2, 8, 4, 5 Target : 18
Output : True
Sample Input : 0 -1 1 Target : 2
Output : False

I would like to work on this issue.

I would like to work on this issue.

Sure, I've assigned it to you


compose: Extracting "compose_validate.js".

This commit first moves the compose.validate() function out
with the functions that are needed by it. Then one by one
checked for which function is now not needed in compose.js.
This moves all validation related functions out of "compose.js"
to "compose_validate.js".

Testing plan:
Tested this with tools/lint and tools/test-js-with-node

@showell @timabbott FYI

I tested the coverage.
Before and after

You’ve changed the order of the extracted functions relative to each other. Was that intentional? If not, it would be better to leave them in the same order so that this is easier to review with tools like git show --color-moved.

Yes thanks to notify this @andersk
I have updated the PR keeping the functions in the same order.

I don't understand the purpose of the compose_announce module.  I'd have expected those functions to just be moved to compose_validate.js in the original commit, just like you did in the third commit.  And I believe with that change, you could squash this and just have a single commit that moves a bunch of code without modification to compose_validate.js (with very few changes to individual lines).
Most of what's messy about the first commit right now is trying to access properties that were left behind in compose.js.

@timabbott @showell I have updated and squashed the commits. This now don't have any cyclic dependency where compose.js is used in compose_validate.js.
Also all the functions are as it is copied, changing the places where they were used and changing the function for resetting user_acknowledged_all_everyone to explicitly set the values.

Merged, thanks @Signior-X!  I think this version is a lot cleaner than the earlier ones.


Harmonize toolchains with OpenBLAS (GCC for OpenMP, Xcode's Clang for CUDA)

Removes dependency on clang-omp in the case of Mac OS X

Build triggered. sha1 is merged.

Build started sha1 is merged.

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/131/

Build finished. No test results found.


What is the difference between the standard implementation?

What is the difference between the standard implementation and material-calendarview?
the latest support library(platform's implementation) now have a full material look and feel

We made this library to have a Material calendar widget that worked on 4.0+. So this library is to function, at least, as a design back port. It's also meant to be more customizable than the platform's widget android.widget.CalendarView. Even for targeting 5.0 and up, the date picker was updated to Material design, but not platform's calendar widget. It still looks Holo.


guestfs: run libguestfs container with no-root user

Enable to run the libguestfs-tools container with a nonroot user.
Signed-off-by: Alice Frosi<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Enable to run libguestfs-tools pod to run as noroot user

/retest

question: can this cause compatibility issues, for example a cluster with an older version of guestfs will not work with a newer virtctl?

hi @maya-r you are of course correct. We can actually have 2 cases:

virtctl is newer than the libguestfs image
the libguestfs image is newer then the virtctl command

The case 2 isn't a problem because the older version of virtctl command run as user 0 that is able to access the less privileged files. The first case is problematic as the no-root user cannot access the appliance :/. The best option I can think of is to add an option where the user can set the uid of the container and if the libguestfs image is older, they can still run the image by setting the uid=0. WDYT?

I tested the new changes with an old image and setting --uid=0 and keeping the paths consistent makes the new changes backwork compatible with old images:
$ virtctl guestfs pvc-fs --uid 0 --image  quay.io/kubevirt/libguestfs-tools@sha256:b4784466dac3aa95f9670b1919f30179d74897347ff8dd28f1e9b10261ba5f1a
bash-4.4# libguestfs-test-tool
[...]
===== TEST FINISHED OK =====
bash-4.4# guestfish -a disk.img run
bash-4.4# echo $?
0

/retest

@xpivarc sorry I somehow missed the notifications of your comments. I integrated what you suggested and also using the mode instead of using nobody user.

/retest

hi @xpivarc when you have a bit of time, could you please have again a look to this PR? Thx

/approve

hi @maya-r you are of course correct. We can actually have 2 cases:
1. virtctl is newer than the libguestfs image

2. the libguestfs image is newer then the virtctl command

The case 2 isn't a problem because the older version of virtctl command run as user 0 that is able to access the less privileged files. The first case is problematic as the no-root user cannot access the appliance :/. The best option I can think of is to add an option where the user can set the uid of the container and if the libguestfs image is older, they can still run the image by setting the uid=0. WDYT?
EDIT: the paths need also to stay compatible

/approve
/hold
@davidvossel Any concerns from your side?

@davidvossel kind ping. Do you have any concerns with this PR?

/hold cancel

/cherry-pick release-0.53

/cherry-pick release-0.44

/cherry-pick release-0.44
Mistake :)
/cherry-pick release-0.49

/retest


[feat] Is it possible to run it with non-square kernels?

It is needed to make some small changes in kan_convolutional/convolution.py, changing kernel_side:int for a kernel shape. You can make a PR and we'll check it out


(FACT-1900) Only check for sysnative if we are running under Wow64

Previously, facter would check for the existence of
%sysroot%\sysnative and assume it was the correct system32 directory
if it exists. This is incorrect if that directory has somehow been
created as a normal directory in the normal filesystem.
Instead, we should just check for whether we are running under WOW64,
and spit out the correct directory based on that query.

I'm totally relying on appveyor to build/test this on Windows, so let's see what happens.

I guess I could also just hardcode system32 here, since we no longer support 32-bit agents on 64-bit Windows? But I figure if anyone is building this code themselves I don't wanna just break the semantics of this.

CLA signed by all contributors.

Also, should this be targeted at 3.11.x? Seems like a bugfix.

@ekinanp ah, yup. I had based it on 3.11.x, just targeted the PR at the wrong branch. I've also pushed up a fix for the lints


add @munkm slides and repo links

I've added links to the rendered slides in the README.md (rendered with reveal.js). If it'd also be helpful to add a pdf copy of the slides in a folder of this repo let me know and I'll add them!

It's up to you! Also: