Feature/composer auth

@axllent I’ve run into another issue; some of our apps use private composer packages hosted by our GitLab instance (docs).
Running this command on those apps results in this error:
  [Composer\Downloader\TransportException]
  The "https://git.my-gitlab-server/api/v4/group/242/-/packages/composer/packages.json" file could not be downloaded (HTTP/2 404 ):
  {"message":"404 Group Not Found"}

(Arguably GitLab should respond with 401 rather than 404 but ¯\_(ツ)_/¯)
In other pipelines (phpunit tests, for example), I handle this by running this script step:
phpunit:
  script:
    - composer config http-basic.git.my-gitlab-server ___token___ "$PERSONAL_ACCESS_TOKEN"
    - composer install
    - phpunit # etc.

I tried this first, but the docker image doesn’t have composer installed so it failed.
For now I’ve built a “prep” job that creates and caches an auth.json, and this works fine:
cache:
  paths:
    - auth.json

composer-mr:prep:
  stage: composer_mr_prep
  script:
    - composer config http-basic.git.my-gitlab-server ___token___ "$COMPOSER_MR_TOKEN"

composer-mr:
  stage: composer_mr
  script:
    - gitlabci-composer-update-mr username email@email branch

This PR adds two new variables used by composer config to set auth credentials.
Again, I’m not a Go developer, so I’m not quite sure I have the syntax correct, but does this idea make sense to you and seem like the best way to accomplish this?
Thanks again!

Uggg…I just realized this commits the auth.json into the repo.

@macbookandrew Thanks for the contribution again! A couple of things which will probably affect how / if this progresses:

Composer is actually installed, however given that some projects use composer 1 and others 2, they are named composer-1 and composer-2. I could alias composer to composer-2, however the whole point of naming them this was was to correctly define both so the wrong one wouldn't be used, and allow for composer-3 etc in the future. Does that fix the issue for you (without having to modify the code for this) - ie: can you then do what you need to by using the composer-2 binary?
The composer.json & composer.lock are unfortunately not the only files that should necessarily be checked in on a composer update. Sometimes composer modules will add necessary configuration files within projects etc (I'm specifically thinking of our most common use-case which involved the Silverstripe framework / CMS). Rather than limit to just the files we think may be checked in, isn't the better approach for you to use .gitignore in your project to specifically exclude files generated from your process such as auth.json?

Ah, yes—composer 2 config… works perfectly
Makes sense, and I did after submitting this update my job to add auth.json to the .gitignore but didn’t love the process.

I’ll open a new PR with some suggested updates to the docs to help other people with privately-hosted packages.
Thanks!

Awesome, thanks! I'm not against making changes, but not when it can be avoided :)


Hyperlink not working

this "ADD YOUR PROJECT" is not working. i checked the code. hyperlink reference in correct, but still its not functional

Please have a look here https://github.com/girlscript/GirlScript-Summer-of-Code/pull/2

Resolved in #2 , so closing this


Best practice

To differentiate Windows, Linux and OSX build in CMake.
Do you prefer :

The regular CMake variable MSVC, UNIX etc ...
The vcpk variable from triplet configuration : VCPKG_CMAKE_SYSTEM_NAME (Windows, Linux, Darwin)

Note: to use VCPKG_CMAKE_SYSTEM_NAME we need to inject it in portfile.cmake

Currently, VCPKG_CMAKE_SYSTEM_NAME have 6 values:
VCPKG_CMAKE_SYSTEM_NAME    TOOLCHAIN

NOT DEFINED                Windows
WindowsStore               Windows(UWP)
Linux                      Linux
Android                    Android
Darwin                     Osx
FreeBSD                    Freebsd

@Cheney-W I asked this question because VCPKG_CMAKE_SYSTEM_NAME was accessible in portfile but not in CMakeLists.
for example my port for libuv have to inject it in vcpkg_configure_cmake : https://github.com/microsoft/vcpkg/pull/3437/commits/a64f5186d369bdf3201d005215a95b3df8640bf4#diff-2518762e404a460b0163cf80475362afR20
currently most of port use CMake variable MSVC, UNIX
Note: btw, I don't know if it's still the case.


Prepare for release 1.24

@ahocevar there are CORS issues with the Boundless WMS layer used in some examples. Could you please see if the server can be fixed? Thanks a lot.
epsg-4326.html:1 Access to Image at 'http://demo.boundlessgeo.com/geoserver/wms?SERVICE=WMS&VERSION=1.3.0&REQUES…W_DR&WIDTH=256&HEIGHT=256&CRS=EPSG%3A4326&STYLES=&BBOX=-90%2C-180%2C90%2C0' from origin 'http://localhost:4000' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:4000' is therefore not allowed access. The response had HTTP status code 503.

@gberaudo I have made our admins aware of the issue. I cannot promise when it will be fixed though.

Great, thanks @ahocevar.

@fredj, done and reworked the examples.


WIP: Updated linting to run in related test sequence per #2344

Modifying the way that molecule runs linting steps per #2344

molecule lint will now only run the main yamllint against the role
molecule verify and molecule converge will now run the linting specified in their respective sections of the molecule.yml file

Thanks! Shared some thoughts in https://github.com/ansible/molecule/issues/2344#issuecomment-544603305.

Dropped in favour of https://github.com/ansible/molecule/pull/2447


Move-DbaDatabaseFile - Documentation changes for review

Ran through the comments and documentation, changed a bunch of grammar and phrasing to shorten and hopefully simplify some of the explanations. Altered -InputFile fundamentally because I think the explanation may have been copy pasted and not changed. Changed capitalization and period use to be more consistent.

This is a PR on top of @ClaudioESSilva's submission, I believe we would either need to do a few commits on top of this to finish up.

@ClaudioESSilva this appears to be a PR for merging into your development branch, suggest you take responsibility for merging it.

hey @ClaudioESSilva how's this coming?

I will try to do something this week but I can't promise :weary:
As soon as I can contine on this I shout!

np i did it ;)


[core] Fix TypeError when setting user agent string in React Native

Platform.constants was introduced React Native version v0.63 or later. This PR adds a check before accessing the property.
Packages impacted by this PR
@azure/core-rest-pipeline
Issues associated with this PR
#26609

API change check
API changes are not detected in this pull request.


updated in-cluster monitoring service and servicemonitor labels

/lgtm after test passes

/unhold

/close and created new PR https://github.com/openshift/addon-operator/pull/182

/unclose


test playbook wrong yaml

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/retest


Uninstall Defender for Endpoint MacOS

Can you provide uninstall documentation for MacOS?

@seliyyah Please indicate which document you are referring and what improvements you suggest. Here, we are limited to documentation fixes and improvements.

I was just referring to any documentation around uninstalling Defender for Endpoint on MacOS.  I haven’t been able to find official documentation, so technically it’s not a documentation improvement request.  Apologies for any confusion.  Thank you.

@seliyyah Considering the nature of the issue, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


refactor(fee): transaction info creator may fail

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#506  👈
#505
#504
#503
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nimrod-starkware and the rest of your teammates on  Graphite


adding-integrations/managing-listings-on-github-marketplace/adding-we…

…bhooks-for-a-github-marketplace-listing/

Why:

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.


Rename "user" to "username" in authentication requests

Currently the post request contains:
{
    "**user**": "username",
    "remoteAddress": "<IP_ADDRESS>:1234",
    "sessionId": "A base64 SSH session ID",
    "passwordBase64": "Base 64 password"
}

{
  "**username**":"ssh username",
  "sessionId": "ssh session ID"
}

I would suggest to either use username or user, to avoid confusion.

Your change has been added. The authentication requests (password and pubkey) now send an additional username field next to the user field. The user field will be removed in a later release so all users are urged to update their authentication server implementation.


fix support for macOS and Linux & add EAP-MD5 CHAP method

Tested on macOS 10.12.1 and Ubuntu 16.04.
Fix #9
Now able to choose between old and new CHAP method. See README.md.

The same as #13 , please check #13 's code review.

@zonyitoo It's strange that I can't see the comments of that pull request though I can see the number is 6.

Please check it again, I have submitted just now.


Update platform list with sles-15-X86-64

Description
.expeditor/release.omnibus.yml already has
  sles-12-x86_64:
    - sles-12-x86_64
    - sles-15-x86_64

but .buildkite/build-test-omnibus.sh needs the container_platforms and esoteric_platforms to be updated for those builders to be enabled.
Related Issue

Types of changes

[ ] 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 change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Ref build log : https://buildkite.com/chef/chef-chef-main-validate-adhoc/builds/755 - Passed

https://github.com/chef/chef/blob/99619374d31f2466cad7597dd3abd121f0d93af7/.buildkite/build-test-omnibus.sh#L14
Here you need to add: debian-10-arm:debian-10-arm and debian-11-arm:debian-10-arm

Looks like this still needs to be updated?

https://github.com/chef/chef/blob/99619374d31f2466cad7597dd3abd121f0d93af7/.buildkite/build-test-omnibus.sh#L14
Here you need to add: debian-10-arm:debian-10-arm and debian-11-arm:debian-10-arm

Looks like this still needs to be updated?

We have  PR :https://github.com/chef/chef/pull/14005  for debian support.

https://github.com/chef/chef/blob/99619374d31f2466cad7597dd3abd121f0d93af7/.buildkite/build-test-omnibus.sh#L14
Here you need to add: debian-10-arm:debian-10-arm and debian-11-arm:debian-10-arm

Looks like this still needs to be updated?

We have PR :#14005 for debian support.

But it has a conflict because the change is on the same line, so need a rebase first before we can merge.

Duplicate of https://github.com/chef/chef/pull/14044


mpi initialization fix

baselines.common.tf_util.initialize must now be called by all MPI workers.

you're right, this isn't a problem if using the version of baselines in requirements.txt.


Adds JWT Auth mechanmism using GitHub Oauth

Protects API methods for:

adding and delete repos
viewing, creating and deleting Helm releases

The JWT is stored in an httpOnly cookie and is only valid for 2 hours. A second cookie, accessible via JS, is used in the UI to determine if the user is logged in or not.

Codecov Report

Merging #361 into master will decrease coverage by 2.62%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #361      +/-   ##
==========================================
- Coverage   87.67%   85.05%   -2.63%
==========================================
  Files          18       19       +1
  Lines         844      870      +26
==========================================
  Hits          740      740
- Misses         68       94      +26
  Partials       36       36

Impacted Files
Coverage Δ

src/api/middleware/authgate.go
0% <0%> (ø)

Continue to review full report at Codecov.

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


Format code as monospace (and highlighted) in editor window

Is your feature request related to a problem? Please describe.
When typing code into the editor, it's always frustrating that it's not monospace and doesn't have syntax highlighting. Syntax highlighting is less important than it being monospace, but both are annoying.
Describe the solution you'd like
It'd be really useful if when a line was indented by four spaces or surrounded by fences, it could be converted to a monospace font and possibly also get some kind of syntax highlighting.
Describe alternatives you've considered
I looked for ways to add my own CSS to do this, but I really can't seem to understand how you manage to style text inside a textarea. I'd be happy to make a PR for this if you'd point me to the right place to put my CSS.

You can style cm-comment to get monospace working.
.cm-comment {
    font-family: monospace;
}

This isn't perfect, however. As spaces inside code blocks are not monospaced. As far as I can see there's no (easy) way to get that working in this version of CodeMirror.

Yes, it would be really useful for programmers or anybody who frequently puts code blocks in their markdown.


[Demo review] MFA enabling process - Show MFA enabling options

Show disabled options when MFA is not enabled, so admin is confident on what they are enabling. See CMS Admin turns on 2FA: https://projects.invisionapp.com/dsm/silver-stripe/silver-stripe/folder/components/5c68f96ffb26bb0018525d1a

Probably replaced with https://github.com/silverstripe/silverstripe-mfa/issues/144

Agreed, lets close in favour of that for now


Question

How will I be able to install this module on windows? I'm sorry if this is a dumb question.

Hey, no worries. I haven't gotten around putting it on pypi yet, so you have to do it manually: Clone or download the project such that pygame_textinput.py is right next to the file from which you want to import it. Then simply import pygame_textinput like in the example. Hope this helps.

Thanks a lot for explaining it to me! I would suggest you mention this in your readme file. A lot of people, all over the internet, have run into this problem and beginners like me just don't know how to proceed. Thanks again for your help!

Hm, I think PyPI is the better option. I'll get around to it soon. Thanks for the hint!


[0.13.7] Scaladoc of AutoPlugin needs reformatting

Just look at it: http://www.scala-sbt.org/0.13.7/api/index.html#sbt.AutoPlugin
Its comment doesn't follow Scaladoc conventions.

PR to fix this welcome.  Or a link to the conventions :)


dotnet 5.0 traces have large percentage of unresolved modules in CPU stacks view

I'm seeing a huge percentage of exclusive samples that are not resolved to a module:

I'm trying to profile the grpc client/server benchmark on windows with .net 5. Is this a known issue on .net 5, or something specific to this scenario?

After attaching the debugger, I think CodeAddressesInModule is 0 for most of the dotnet dlls in the trace. Are there some extra events I need to be collecting that I don't have?

Oops, the first trace didn't have ClrEvents on which explains the widespread missing symbols. Same problem, but less severe with the following trace:

PerfViewData.zip

@kayle, this is most common when either rundown doesn't happen (e.g. you don't get events that describe JIT events), or the trace wasn't merged and was either copied off box or one or more binaries changed.  I don't see a log with the trace - do you happen to have it handy?  I'm curious if merge succeeded.

Sure, here's the log/symbols for the rest of the zip file (was too big to upload as an attachment before). I see errors that it couldn't find crossgen.exe. Maybe perfview could download it at runtime (https://github.com/dotnet/installer/issues/3438)?
PerfViewData.log.txt
PerfViewData.etl.zip

Looks like merge succeeded.  The crossgen errors should not be an issue - in most cases, you don't need to use crossgen anymore - this is something that was required in previous version, but our diagnostics story has gotten better.
I don't see anything obviously wrong at this point.  Does this repro if you set the verbosity to verbose instead of informational?  Also, does it repro if you don't specify a focus process?

Yes, it repros with -ClrEventLevel:Verbose and without a focus process.
The repro is not difficult:

git clone https://github.com/grpc/grpc-dotnet
start server with dotnet run -c Release -p .\perf\benchmarkapps\GrpcAspNetCoreServer\ --protocol h2c
start client with dotnet run -c Release -p .\perf\benchmarkapps\GrpcClient\ -- -u http://localhost:5000 -c 10 --streams 50 -s unary -p h2c --grpcClientType grpcnetclient

I've tried collecting traces with the GUI or commands similar to PerfViewCollect.exe collect -TplEvents:None  -ClrEventLevel:Informational -ClrEvents:Default-Threading-ThreadTransfer -BufferSize:1024 -FocusProcess:GrpcClient.exe
The extra flags are to try to avoid some of these common events which affect overall CPU utilization:

System.Diagnostics.Eventing.FrameworkEventSource/ThreadPoolDequeueWork
Microsoft-Diagnostics-DiagnosticSource/Activity1Start/Start
Microsoft-Windows-DotNETRuntime/ThreadPoolWorkerThread/Wait
Microsoft-Windows-DotNETRuntime/ThreadPool/IODequeue

@brianrob After setting fold% to 0, it looks like the ?!? samples could be inlined calls or some other runtime overhead:
For example, this trivial IsCompleted method has 16 inclusive samples that call ?!?

                   /// <summary>Gets whether the <see cref="ConfiguredValueTaskAwaitable{TResult}"/> has completed.</summary>
                   public bool IsCompleted
                   {
                       [MethodImpl(MethodImplOptions.AggressiveInlining)]
 16.0 |                get => _value.IsCompleted;
                   }

and this method has 13 inclusive samples that call into ?!?

 13.0 |        public static StringComparer OrdinalIgnoreCase => OrdinalIgnoreCaseComparer.Instance;

Why would perfview not map these samples to the correct module? Is more context needed for inlined samples?


Basiccontext

Incorporate the sources of basiccontext
Remove the unnecessary 'overflow: hidden' which can cause reflow
Also use the minimized version of lazysizes

I would be in favor to put this not in script and style but in another folder like deps/scripts and deps/style to separate the code of the front from the dependencies.

@ildyria No problem. I asked a while back where to put them but, given that I didn't get a response, I figured I would force the issue by submitting a PR. Clearly it worked :wink:.
@d7415 Thanks! I figured there would be no problem given that they came from the same person but it's good to know that you checked the licenses...


enhanced mouse tracking mode (1006) is set improperly when exiting tcell

Fixed to properly reset enhanced mouse tracking mode (1006) when calling enableMouse(0).
related #477
Behavior before modification:
If I quit the tcell application with the mouse enabled and then start vim, the cursor doesn't move when I click the mouse.
The symptom was confirmed by _demos/mouse.go in the linux environment on Chromebook and Windows.

So the problem is that VIM and tcell are fighting over mouse control...

What need to do is figure out if we can query the mouse state when starting up, and restore it to what it was.
Unconditionally not clearing the mouse flags is unacceptable.
(To be honest, I think the problem here is VIM's fault -- when returning from a subshell, it should reenable the mouse behavior it wants, rather than making assumptions that the child process didn't mess with it.  This is done for other settings like terminfo flags, and the mouse handling needs to be treated in the precise same way.)

I found the specification of control sequnces of terminal app in chromebook.
https://chromium.googlesource.com/apps/libapps/+/master/hterm/doc/ControlSequences.md

@gdamore
I also checked the source of putty (https://git.tartarus.org/simon/putty.git) and found that putty also completely ignores 1003. Also, I didn't see any particular reason why 1006 had to be enabled when mouse events were disabled.

I'm going to take a close look at this.  My fear is that there is some impedance mismatch here in the intended behavior and the actual... and what happens in the terminals.
It is not surprising to me that some of these flags are ignored by terminals... the design actually depends upon it -- they should enable what they can, and ignore the rest.
The other thing is that my commentary earlier may also have been misunderstanding the context where this occurs.

@gdamore  thank you very much!


Fix #25: missing icon override on the Build Time Trend page

Could probably use a selector like the one this commit adds to override the icons properly everywhere?
[src$='/images/16x16/blue.png'], [src$='/images/16x16/blue_anime.gif'] {
  .icon(@size-sm);
  ...
}

This also applies to #20 I guess

It won't fix #20 cause Matrix uses 24x24 icons instead of 16x16.


Add "is" and "has" operators

is will just be syntax sugar for Object.js (with a polyfill)
foo is bar

has will be JS in but reversed order
foo has "property"

TODO: Any good reason to switch to has instead of JS standard in?

What is the argument against implementing both? I understand wanting to support the standard diction of in, but find has "reads" better.

@rdenmark This is purely a "style vs familiarity" battle.
foo is bar vs Object.is(foo, bar)
foo has "prop" vs "prop" in foo
Both things are worth it in the long run, I think, but make Squiggle larger. I think there's probably enough value here. Especially if Squiggle polyfills Object.is, since Safari and IE don't currently support it.

Another thought I just had: switching to "has" over "in" makes it so "in" is only used for one thing in the language (declaration scope like let x = 1 in x), making it so this confusing thing won't parse:
let x = "k" in y in x

Two completely different "kinds" of in right next to each other... yuck.


Flutter Secure Storage Mock Throws type 'Null' is not a subtype of type 'Future<String?>'

Hello, could someone suggest me an idea how to solve this?
When im trying to mock flutter secure storage i get this error " type 'Null' is not a subtype of type 'Future<String?> "
-This is my storage
import 'package:flutter_secure_storage/flutter_secure_storage.dart';
abstract class IStorage {
Future saveData(String value, String key);
Future getId(String key);
}
class MyStorage implements IStorage {
MyStorage(this._storage);
final FlutterSecureStorage _storage;
@override
Future getId(String key) async {
return await _storage.read(key: key) ?? 'No data founded';
}
@override
Future saveData(String value, String key) async {
await _storage.write(key: key, value: value);
}
}
-My Service
import 'dart:convert';
import 'package:dio/dio.dart';
import 'package:fixed/shared/storage.dart';
import 'package:flutter/cupertino.dart';
class ApiService {
final String _baseURL = 'https://jsonplaceholder.typicode.com/';
Dio _dio = Dio(BaseOptions());
final IStorage _storage;
ApiService(this._storage, {Dio? dio}) {
_dio = dio ?? Dio();
_dio.options.baseUrl = _baseURL;
}
Future get() async {
String id = await _storage.getId('id');
final response = await _dio.get('${_baseURL}posts/$id');
debugPrint(json.encode(response.data));
return json.encode(response.data);
}
}
My Test File
import 'package:dio/dio.dart';
import 'package:fixed/api/api_service.dart';
import 'package:fixed/shared/storage.dart';
import 'package:fixed/store/store1.dart';
import 'package:flutter_secure_storage/flutter_secure_storage.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:http_mock_adapter/http_mock_adapter.dart';
import 'package:mocktail/mocktail.dart';
class MockStorage extends Mock implements IStorage {}
class MockStorage2 extends Mock implements MyStorage {}
class MockFlutterSecureStorage extends Mock implements FlutterSecureStorage {}
void main() {
final dio = Dio();
final dioAdapter = DioAdapter(dio: dio, matcher: const UrlRequestMatcher());
late ApiService service;
late MockFlutterSecureStorage mockSecureStorage;
setUp(() {
mockStorage = MockStorage();
mockSecureStorage = MockFlutterSecureStorage();
dio.httpClientAdapter = dioAdapter;
service = ApiService(dio: dio, MyStorage(mockSecureStorage));
});
void mockData() {
dioAdapter.onGet('https://jsonplaceholder.typicode.com/posts/1', (request) {
return request.reply(200, {'data': 'sucessfull'});
});
}
void mockGetId() {
when(() => mockStorage.getId('id'))
.thenAnswer((invocation) => Future.value('1'));
}
group('Test Service', () {
test('Test endpoint', () async {
//IT THROW ERROR
when(() => mockSecureStorage.read(key: 'íd')).thenAnswer((_) async => '');
final data = await service.get();
expect(data, equals('{"data":"sucessfull"}'));
verify(() => mockSecureStorage.read(key: 'id')).called(1);
});
});
}

Hey @DiegoVega19, please format your code first and insert the whole error if you can. 🙏  https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax

i know it's been a year but i have the same problem here and for the sake of brevity i'll show only the write function.. and the weird thing is that when i test the saveLocalData isolating the storage.write as you'll see below the test passes but inside my authenticateCrewMember it does not..
my AuthProvider uses the StorageService as a dependency and DI is made with get_it on app load
abstract class StorageService {

  Future<bool> write({
    required String key,
    required String value,
    bool deleteAll = false,
  });
}

class StorageServiceImpl implements StorageService {
  StorageServiceImpl(this._storage);

  final FlutterSecureStorage? _storage;

  @override
  Future<bool> write({
    required String key,
    required String value,
    bool deleteAll = false,
  }) async {
    try {
      await _storage?.write(
        key: key,
        value: value,
        aOptions: getStorageAndroidOptions(),
      );
      return true;
    } on PlatformException catch (e, s) {
      log('[PLATFORM-EXCEPTION] - STORAGE-SERVICE:WRITE: $e, $s');
      if (deleteAll) {
        await _storage?.deleteAll();
      }
      return false;
    } catch (e, s) {
      log('[ERROR] - STORAGE-SERVICE:WRITE: $e, $s');
      return false;
    }
  }
}

Auth Provider
class AuthProvider with ChangeNotifier {
  AuthProvider({
    required this.storageService,
    required this.onboardService,
  });
  final StorageService storageService;
  final OnboardService onboardService;

  CrewModel? _crew;
  CrewModel? get crew => _crew;

  bool get isAuth {
    return _crew != null && _crew!.qrcodePayload.isNotEmpty;
  }

  Future<bool> authenticateCrewMember({required String qrCodePayload}) async {
    log('AUTHENTICATING CREW MEMBER...');
    try {

      CrewModel? crew = await onboardService.authenticateCrewMember(
        qrCodePayload: qrCodePayload,
      );
      if (crew == null) {
        throw AppException('O QRCode apresentado não retornou dados.');
      }
      _crew = crew.copyWith(qrcodePayload: qrCodePayload);

      await saveLocalData(_crew!);

      log('[RESPONSE:PRD] - AUTH-CREW-MEMBER: _crew: $_crew');
      notifyListeners();

      return true;
    } catch (e, s) {
      log('[ERROR:PRD] - AUTH-CREW-MEMBER - $e, $s');
      throw AppException('$e');
    }
  }

  Future<void> saveLocalData(CrewModel crew) async {
    log('saving data');
    try {
      await storageService.write(
        key: STORAGE_CREW_AUTH_KEY,
        value: crew.toJson(),
      );
      log('data saved');
    } catch (e, s) {
      log('save error: $e, $s');
    }
  }
}

AuthProviderTest
class MockStorageService extends Mock implements StorageService {}
class MockOnboardService extends Mock implements OnboardService {}

void main() {
  late StorageService storageService;
  late OnboardService onboardService;

  late AuthProvider authProvider;

  setUp(() {
    storageService = MockStorageService();
    onboardService = MockOnboardService();

    authProvider = AuthProvider(
      storageService: storageService,
      onboardService: onboardService,
    );
  });

  tearDown(() {
    resetMocktailState();
  });

  group('AuthProvider.saveLocalData -', () {
    *** this test passes ***

    test('should call saveLocalData and return void when successful', () async {
      final tModel = CrewModel.empty();
      when(() => storageService.write(
            key: STORAGE_CREW_AUTH_KEY,
            value: tModel.toJson(),
          )).thenAnswer((_) => Future.value(true));

      await authProvider.saveLocalData(tModel);
      verify(() => storageService.write(
            key: STORAGE_CREW_AUTH_KEY,
            value: tModel.toJson(),
          )).called(1);
    });
  });

  group('AuthProvider.authenticateCrewMember -', () {
    test(
        'should call authenticateCrewMember and return a CrewModel when successful',
        () async {
      const qrCodePayload = 'xxx';
      final tModel = CrewModel.empty();

      when(() => onboardService.authenticateCrewMember(
              qrCodePayload: qrCodePayload))
          .thenAnswer((_) => Future.value(tModel));

     *** this throws the error ***
      when(() => storageService.write(
            key: STORAGE_CREW_AUTH_KEY,
            value: tModel.toJson(),
          )).thenAnswer((_) => Future.value(true));

      final result = await authProvider.authenticateCrewMember(
          qrCodePayload: qrCodePayload);

      expect(result, true);
      expect(authProvider.crew, isNotNull);
      expect(authProvider.crew?.qrcodePayload, qrCodePayload);

      verify(() =>
              mobipixOnboardService.getQRCodeType(qrCodePayload: qrCodePayload))
          .called(1);
      verify(() => mobipixOnboardService.authenticateCrewMember(
          qrCodePayload: qrCodePayload)).called(1);

      verifyNoMoreInteractions(mobipixOnboardService);

      *** this test does not pass because it is not called ***
      verify(() => storageService.write(
            key: STORAGE_CREW_AUTH_KEY,
            value: tModel.toJson(),
          )).called(1);
      verifyNoMoreInteractions(storageService);
    });
  });
}

Thats the error when I run debug in the test
type 'Null' is not a subtype of type 'Future<bool>', #0      MockStorageService.write (file:///Users/XXX/Dev/YYY/my-app/test/provider/auth.provider_test.dart:11:7)
      #1      AuthProvider.saveLocalData (package:my-app/provider/auth.provider.dart:148:28)
      #2      AuthProvider.authenticateCrewMember (package:my-app/provider/auth.provider.dart:60:13)
      <asynchronous suspension>
      #3      main.<anonymous closure>.<anonymous closure> (file:///Users/XXX/Dev/YYY/my-app/test/provider/auth.provider_test.dart:109:22)
      <asynchronous suspension>
      #4      Declarer.test.<anonymous closure>.<anonymous closure> (package:test_api/src/backend/declarer.dart:215:9)
      <asynchronous suspension>
      #5      Declarer.test.<anonymous closure> (package:test_api/src/backend/declarer.dart:213:7)
      <asynchronous suspension>
      #6      Invoker._waitForOutstandingCallbacks.<anonymous closure> (package:test_api/src/backend/invoker.dart:258:9)
      <asynchronous suspension>

any help will be very much appreciated..
cheers

forgot:
Flutter doctor

[✓] Flutter (Channel stable, 3.16.0, on macOS 13.2.1 22D68 darwin-arm64, locale en-BR)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.87.2)
[✓] Connected device (3 available)
[✓] Network resources

flutter_secure_storage: ^9.0.0
mocktail: ^1.0.3

If anyone is still having this issue can you please provide a link to a minimal reproduction sample? it's much easier for me to help if I'm able to reproduce the issue locally, thanks!

If anyone is still having this issue can you please provide a link to a minimal reproduction sample? it's much easier for me to help if I'm able to reproduce the issue locally, thanks!

Here is a minimal reproducible example. Please take a look at it: https://github.com/felangel/mocktail/issues/252


Include  RedisBloom/t-digest-c as part of implementations in Other Languages

This PR makes reference to t-digest-c as part of RedisBloom.

I saw the tweet go by.
THere is LogHistogram in the t-digest package which has similar properties to HdrHistogram but is more attuned to low accuracy/small size applications (5% accuracy, for example).


Bump dependencies to Hibernate 5.6

May we change the versioning schema to match the Hibernate base version (5.6) we compiled against?

Best wishes for 2022 @markusheiden

I don't see any changes in the Hibernate API thus last version of hybernate-hydrate should be compatible with Hibernate 5.6 and vice versa. Doesn't it?

I didn't check that. But can you guarantee that? For me, it feels better to run against the same version as it has been compiled against. I experienced no concrete problems though, so there is no need to do that currently.

I think we could rely on unit test to test again may Hibernate versions. With Maven profile and GitHub actions. I'll create an issue.

@arey First of all: Best wishes for 2022. Sorry, I overlooked your comment, while being in a hurry like always...
Theoretically, we need a build against every (supported) main Hibernate version. Otherwise, when upgrading the build to a newer Hibernate version and doing some changes we may use an API that is not present in older Hibernate versions without getting test failures. But as already said: That's just theoretical.


Features/staff page

#86

For Aesthetic Purposes

Make the margin underneath "Members of the .... Board" larger.

Make the margin above the footnotes larger.


Add validation for chatGPT imports, stopping any breaking issues with imports of incompatible chats

Tested on 1000 chats from 3 separate people...
chatGPT imports might use external tools like Dall-E or there also might be corrupted chats that were interrupted several times from users.
To deal with this, I basically added a new validation function to make sure the chat imports look good and a new try catch block for any bad imports. In the case that they do not, we just won't include them.

Sorry about the multiple commits. Hard to get all the ways chatgpt might structure their chat conversations with all their newer features:(

No worries, this is heroic work you're doing. I think a lot of people are going to really love this feature once it's merged.

LGTM, Thanks!


chore(main): release 0.17.1

:robot: I have created a release beep boop
0.17.1 (2024-09-17)
Bug Fixes

fix not properly pass args and kwargs when cache disabled (539a501)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.17.1
:sunflower:


devsite 3 | Landing pages

[ ] Landing pages for site SEO that target specific keywords

legacy due to #94


Update translations for SS4.

Fix unit-tests for SS 4.1
Add a classname-map config.

Codecov Report

Merging #627 into master will decrease coverage by 0.01%.
The diff coverage is 25.49%.

@@             Coverage Diff              @@
##             master     #627      +/-   ##
============================================
- Coverage      46.5%   46.49%   -0.02%
  Complexity     1582     1582
============================================
  Files           119      119
  Lines          4608     4609       +1
============================================
  Hits           2143     2143
- Misses         2465     2466       +1

Impacted Files
Coverage Δ
Complexity Δ

src/Model/Modifiers/Shipping/Weight.php
0% <0%> (ø)
13 <0> (ø)
:arrow_down:

src/Forms/OrderActionsForm.php
81.48% <0%> (ø)
31 <0> (ø)
:arrow_down:

src/Checkout/Step/Address.php
0% <0%> (ø)
11 <0> (ø)
:arrow_down:

src/Page/CartPageController.php
0% <0%> (ø)
4 <0> (ø)
:arrow_down:

src/Checkout/Step/ContactDetails.php
0% <0%> (ø)
7 <0> (ø)
:arrow_down:

src/Reports/SideReport/HeavyProducts.php
0% <0%> (ø)
5 <0> (ø)
:arrow_down:

src/Extension/ProductVariationsExtension.php
53.98% <0%> (ø)
28 <0> (ø)
:arrow_down:

src/Forms/ShopAccountForm.php
0% <0%> (ø)
11 <0> (ø)
:arrow_down:

src/Model/Modifiers/Tax/Base.php
0% <0%> (ø)
2 <0> (ø)
:arrow_down:

src/Checkout/CheckoutFieldFactory.php
0% <0%> (ø)
18 <0> (ø)
:arrow_down:

... and 31 more

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 9c8f729...bf9d017. Read the comment docs.


Add MILO to mc.tokenlist.json

Name: MiloInu
Symbol: MILO
Contract: 0xe98E1DD1541F604e47dB990336f687e9c6FA56D1
Chain: Avalanche
ChainId: 43113
@pecio222

43113 is Fuji, you sure?

Hi,
No MILO is Avalanche C-Chain (43114)
Do I have to do another pull request ?
On Sat, Dec 7, 2024 at 3:31 AM Pecio @.***> wrote:

43113 is Fuji, you sure?
—
Reply to this email directly, view it on GitHub
https://github.com/traderjoe-xyz/joe-tokenlists/pull/1623#issuecomment-2525028895,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFMVK2EY2NYFFHFEMIDHG4L2EKW6PAVCNFSM6AAAAABTF6ILFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRVGAZDQOBZGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fixed branding inconsistency between logo and site name.

Title and Issue number

Title: Fixed branding inconsistency between logo and site name.
Issue No.: #11
Code Stack: Frontend (Navbar component)
Description

Updated the logo to reflect the brand name "FitFlex" instead of "FitFlux."
Ensured the logo and site name are consistent across all pages.
Improved the visibility and size of the logo, as per the latest design guidelines.
Removed the unnecessary gray highlight around the logo for a cleaner look.

Video (mandatory)
https://github.com/user-attachments/assets/c78131db-c7d2-490c-a5f0-75f4ca7ba58b
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
[X] I have gone through the contributing.md file before contributing.

Additional context (Mandatory)
This PR ensures that the logo now matches the brand name "FitFlex" for consistent branding.
Are you contributing under any Open-source program?

[✔] I'm a GSSOC-EXT contributor.
[✔] I'm a HACKTOBERFEST contributor.

@jeevan10017 why these checks are failing, could you please tell me??

@SCR01 invalid. We do not want to change the logo.


Dev

Description
Please include a summary of the change and a link to which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change

[ ] 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

Change status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] 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
[ ] There are no merge conflicts

I have no idea what to check here man.
I think image upload for new projects was finished.

Loading the project shows several warnings as well.
Compiled with warnings.

./src/components/SearchPageSearchBar/SortDropDown.js
  Line 11:  'Link' is defined but never used                no-unused-vars
  Line 15:   'logoutURL' is assigned a value but never used  no-unused-vars

./src/components/_LandingPage/LandingPage.js
  Line 177:  img elements must have an alt prop, either with meaningful text, or an empty string
for decorative images  jsx-a11y/alt-text

./src/components/_ProjectPage/EditProject.js
  Line 228:  No duplicate props allowed  react/jsx-no-duplicate-props

./src/components/_ProjectPage/Post/EditPost.js
  Line 259:  'ImgContainer1' is assigned a value but never used  no-unused-vars
  Line 266:  'Img' is assigned a value but never used            no-unused-vars

./src/components/ProjectTile/ProjectCard.js
  Line 31:  Unnecessarily computed property ['@media (max-width: 1000px)'] found  no-useless-comp
uted-key
  Line 37:  Unnecessarily computed property ['@media (max-width: 500px)'] found   no-useless-comp
uted-key

./src/components/ReviewCard/ReviewCard.js
  Line 33:  Unnecessarily computed property ['@media (max-width: 500px)'] found  no-useless-compu
ted-key

./src/components/_ProjectPage/Project.js
  Line 38:  No duplicate props allowed  react/jsx-no-duplicate-props

./src/components/_AccountPage/_ReviewList/ReviewList.js
  Line 3:  'Link' is defined but never used  no-unused-vars

./src/components/_ProjectPage/ProjectPage.js
  Line 3:  'Route' is defined but never used  no-unused-vars

./src/components/_AccountPage/_ProjectList/ProjectList.js
  Line 5:  'Nav' is defined but never used  no-unused-vars

./src/components/_AccountPage/AccountSideBar.js
  Line 5:  'MenuDrawer' is defined but never used  no-unused-vars

./src/components/SearchPageSearchBar/SearchPageSearchBar.js
  Line 6:   'Dropdown' is defined but never used              no-unused-vars
  Line 7:   'DropdownToggle' is defined but never used        no-unused-vars
  Line 8:   'DropdownMenu' is defined but never used          no-unused-vars
  Line 9:   'DropdownItem' is defined but never used          no-unused-vars
  Line 10:  'UncontrolledDropdown' is defined but never used  no-unused-vars
  Line 12:  'ClickAwayListener' is defined but never used     no-unused-vars
  Line 13:  'Grow' is defined but never used                  no-unused-vars
  Line 14:  'Paper' is defined but never used                 no-unused-vars
  Line 15:  'Popper' is defined but never used                no-unused-vars
  Line 16:  'MenuItem' is defined but never used              no-unused-vars
  Line 17:  'MenuList' is defined but never used              no-unused-vars
  Line 38:  'SelectStyle' is assigned a value but never used  no-unused-vars

./src/components/MakerTile/MakerCard.js
  Line 7:   'CardHeader' is defined but never used                                no-unused-vars
  Line 12:  'Avatar' is defined but never used                                    no-unused-vars
  Line 35:  Unnecessarily computed property ['@media (max-width: 1000px)'] found  no-useless-comp
uted-key
  Line 40:  Unnecessarily computed property ['@media (max-width: 500px)'] found   no-useless-comp
uted-key

./src/components/ReviewerTile/ReviewerCard.js
  Line 7:   'CardHeader' is defined but never used                                no-unused-vars
  Line 27:  Unnecessarily computed property ['@media (max-width: 1000px)'] found  no-useless-comp
uted-key
  Line 32:  Unnecessarily computed property ['@media (max-width: 500px)'] found   no-useless-comp
uted-key

./src/components/_ProjectPage/NewProject.js
  Line 88:   'ProjectImageUpload' is assigned a value but never used  no-unused-vars
  Line 104:  'ProfileHeader' is assigned a value but never used       no-unused-vars

./src/index.js
  Line 9:  'logger' is defined but never used  no-unused-vars


Update tailwind.config.ts

export default causing error #1498.  converted to module exports
Closes #
✅ Checklist

[ ] I have followed every step in the contributing guide (updated 2022-10-06).
[ ] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
[Short description of what has changed]

Screenshots
[Screenshots]
💯

This is the tailwind config for the documentation site. I doubt it's causing problems with scaffolded apps as it doesn't get included in those.

This is fixed by reinstalling packages to update the peer dependency jiti, see issue for more information


install: Drop SELinux-disabled warning

We have e2e tests for this today. On a related topic, unfortunately the way bootc-image-builder sets up the container it runs bootc in it doesn't mount selinuxfs, so we see SELinux as disabled when it's not.
We should fix that, but it also avoids user confusion to drop the warning here.
While we're here, change things so we more consistently log the state computed.
Closes: https://github.com/containers/bootc/issues/419

Orthogonal(ish) to this, shoudl we enable seinuxfs in our bib containers? It sounds like a good correctness fix?

I think so yes


Update Docs - Android build from source instructions

Update the NDK version required from r10e to r16
Related PRs
This is to support facebook/react-native#19536

Sorry we missed this one, it seems like this was already updated since.

I, perhaps, should have closed this one after I closed #19536 😅 - Thanks!


Silence logger

This was logging a message for every class in the stack for servers stacked in an instance of Rack::Cascade, even in API classes which didn't respond to the request

+1, this interferes with our log gathering processes too, we added an explicit logger middleware to log things in our special format

merged, thanks for the pull. It'd be nice to add an optional simple request logging middleware to Grape

I would merge this to master as well, this was added in https://github.com/intridea/grape/commit/63fdf7494923b180fc452ec13c92964fc268145f which I think was after the previous release.


Update drive-content-managers.yaml

Add myself as member for Kubeflow docs drive.

/lgtm
/approve


Added option for max allowed selection

This is already added in a newer version with the better approach.


Update RemotingErrorIdStrings.resx

Please don't accept this....
PR Summary

PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Change is not breaking
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

Please don't accept this....

Close this PR as requested :)


PdfMiner forces wrong behaviour in WordsToLine Module

Summary
PdfMiner introduces Character object in the chain making lines to paragraph module to have wrong behaviour.
Steps To Reproduce
Steps to reproduce the behavior:

Select PdfMiner
Upload foo.pdf

Expected behavior
Table words should be embedded into lines
Actual behavior
Table words are not embedded into lines
foo.pdf

Resolved in PR #122


Using say() with files_info method posts multiple messages

Hello!
As I mention in the title, I'm using the files_info method to get the name, link and content of uploaded files in a channels, and then using the say() method to post this information in a channel.
Whenever I do this, however, the bots sends multiple messages instead of just one, non-stopping, to the point that I have to kill it so it stops sending messages.
Why could this be?
This is the code as example:
@slack_app.event("file_shared")

def handle_file_shared(body, logger, say):
logger.info(body)
file_id = body["event"]["file_id"]

file_name = f"""{slack_app.client.files_info(
    token=os.environ.get("SLACK_BOT_TOKEN"), file=file_id
 )["file"]["name"]}"""

file_link = f"""{slack_app.client.files_info(
    token=os.environ.get("SLACK_BOT_TOKEN"), file=file_id
 )["file"]["permalink"]}"""

file_content = f"""{slack_app.client.files_info(
    token=os.environ.get("SLACK_BOT_TOKEN"), file=file_id
 )["content"]}"""

say(
    text=f"name: {file_name}, link: {file_link}, content: {file_content}",
    channel="1234",
)

@lautaropacellaDR - Sounds like an infinite loop. Is it possible that your say() is triggering another event that is causing say() to be called again?

@lautaropacellaDR As @srajiang mentioned above, the message posted by say() method contains a file URL. Doing so can cause the infinite loop as the message shares the same file again. To avoid this, you can check if the user ID in the payload (or context.user_id) is not your app's bot user's ID before posting the message. I hope this was helpful to you.

Ohh I see, so the link also counts as a file. Thanks a lot @seratch!


Study NimBro's RoboCup 2023 Object Detection system

I'll be studying the Object Detection system that NimBro used in RoboCup 2023, and turned public after the competition. Therefore, I'll be checking the possibility to improve our system based on theirs.
https://github.com/JanNogga/athome23_detection

Their dataset download link is broken. I've sent an email to the rep owner and I'm waiting for an answer/update.


Is ar-augmented-image sample currently working?

Is ar-augmented-image sample currently working?
I tried it on my device but video.png and rabbit.png are empty files and video url seems to be not working anymore.
If I want to implement augmented image, should I use sceneform-android instead?

Same comment.
I wanted to check whether the code works by copy/pasting one of the .glb files from the other examples and changing the code of the "qrcode.png" to call this .glb file instead and it worked. This basically means they need to fix this empty files and put that URL on back again.

Hi @tjsearsz, but can you see the video in Ar?

@ThomasGorisse hi, thank you for constantly updating. Could you please provide feedback to this? Have a great day.

Hello @AnByronV
I haven't tried with videos yet. However, I see that the URLs are back again, so I believe it should work.
The only thing missing part of this example is to add the images. In the meantime, what you could do is to import one random image into the project and then change the name of that image in the code that is being used for the video.

I was able to get it work by overwriting the images with some of my own. In general it's working but it is not updating the position of the glb model when I rotate the marker the 3D object (spoons) are sometimes rotated but mostly in random directions. I thought that I could place a virtual image exactly above the marker but that is impossible if it doesn't track the right orientation and coordinates of the augmentedImage.
Also sometimes the spoons seems to loose their anchor and keep moving with the camera at least on one axes.

@PixelsteinLangenselbold  Glad to know that you were able to make it work :smiley:

In general it's working but it is not updating the position of the glb model when I rotate the marker the 3D object (spoons) are sometimes rotated but mostly in random directions

What you are saying is that when you rotate the market the spoons (3D Object) still keep the same position they were initially detected right? That happens to me. Maybe there is a missing flag/configuration?

Yes right, it finds the image and places the object "somewhere"

@Kang-Dongyoung , @tjsearsz and @PixelsteinLangenselbold Could you share some screen records?

Hi @ThomasGorisse , I am attaching the images of my result.

target image to detect

result
Things I've done to make this work

replace empty image with my own
replace video url
replace empty plane.glb that VideoNode.kt is referring to with non-empty plane.glb file from sceneform augmented image sample

Problem

image detecting and video playing works but video does not align properly with target image in terms of size, orientation.


Secure Speech Service Key/Region

Motivation
Ensure sensitive credentials cannot be exposed. We are currently returning the Azure Speech Key from the /api/config endpoint in plain text https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/blob/ede0d6d93abd38c1b7b071ebf96ad18a3098b457/code/app.py#L32-L40
How would you feel if this feature request was implemented?

Requirements

Stop returning the value of the Azure Speech Key from the /api/config endpoint. Consider using SecretStr from pydantic https://docs.pydantic.dev/2.6/examples/secrets/ to return a masked version of the secret
Ensure that the speech service from the frontend is still functional (it calls the /api/config endpoint) #501
Stretch: Improve this endpoint to return the full config with all secrets masked

Tasks
To be filled in by the engineer picking up the issue

[ ] Implement https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/samples/js/browser/README.md#token-exchange-process
[ ] #683

The solution now has KeyVault - should this be utilised?

Looks like we are now using Key Vault. I think this is more that the key is returned in plain text from the /api/config end point 😬https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/blob/ede0d6d93abd38c1b7b071ebf96ad18a3098b457/code/app.py#L37
I'll reword the issue to be clearer.

https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/samples/js/browser/README.md#token-exchange-process can be relevant for this task

After discussing with the team, it has been decided to remove the /api/config endpoint entirely, as there is on requirement for the endpoint, and to have the endpoint while also masking secrets is non-trivial.
If such a requirement arises, the endpoint may be added back to the application.


Error on Line -1

I am importing this in my Gradle:
compile ('com.crazyhitty.chdev.ks:rss-manager:0.75'){
exclude module: 'stax'
exclude module: 'stax-api'
exclude module: 'xpp3'
}
and these in my activity:
import com.crazyhitty.chdev.ks.rssmanager.Channel;
import com.crazyhitty.chdev.ks.rssmanager.RSS;
import com.crazyhitty.chdev.ks.rssmanager.RssReader;
I'm developing an Android app, and two different activities need to be able to parse RSS Feeds. One of these sections works perfectly, parsing this feed just fine, while the other right here, gives me a strange error.

I am implementing these two exactly the same, aside from which feed they are to parse. The first is able to get all the information I need and display and manipulate it perfectly, while the second fails in rssFeedsLoaded(List rssList). If I type anything as much as ''rssList.getChannel();'', it gives me the Attribute 'base' error and fails, returning to the previous activity.
Any more information needed can be provided if needed. The source code for this app is public on my profile, and these are the two activities that use this RSS reader.
Working Activity:
https://github.com/rileyclauss/PHSMobile/blob/master/PHSApp/app/src/main/java/com/phsapp/phsapp/athleticsActivity.java
Broken Activity:
https://github.com/rileyclauss/PHSMobile/blob/master/PHSApp/app/src/main/java/com/phsapp/phsapp/newsActivity.java
Email me at<EMAIL_ADDRESS>

After further testing, I've found that the issue is the source I'm grabbing from.  The penn source does not work and I believe it is the fault of the site admins. I'm not sure what's causing the error, but it seems to only occur on this feed.


Fix positional args.

Hi gang,
This code is currently broken due to this error:
Syntax error in module 'logdna_cloudwatch': non-default argument follows default argument (logdna_cloudwatch.py, line 64)

which was introduced in this PR.
This PR should fix it, but it might be worth adding either linting or some simple unit tests to find issues like this.

Thanks, @lextoumbourou, I just wanted to start fixing this - you did it before me.


nil pointer dereference in batch handling

We've seen multiple panics in production starting with following stack trace:
 runtime error: invalid memory address or nil pointer dereference
 stack trace 65536 bytes
 goroutine 1630159 [running]:
panic(0xcda0e0, 0xc420016030)
	/usr/local/go/src/runtime/panic.go:458 +0x243
cloud.google.com/go/datastore.(*Iterator).nextBatch(0xc420499dd0, 0xe05a7f, 0xb)
	src/cloud.google.com/go/datastore/query.go:668 +0x132
cloud.google.com/go/datastore.(*Iterator).next(0xc420499dd0, 0xc420dba0e0, 0x197, 0xc66720, 0xc421251040)
	src/cloud.google.com/go/datastore/query.go:621 +0x45
cloud.google.com/go/datastore.(*Client).GetAll(0xc4203def40, 0x7faf99486140, 0xc4210d7a40, 0xc420e6c270, 0xc3d8c0, 0xc420dba0e0, 0xc41ff922f8, 0xc4221d0cb8, 0x41288e, 0x40f07f, ...)
	src/cloud.google.com/go/datastore/query.go:498 +0x102
improbable.io/lib/db/gds.(*client).GetAll(0xc420368270, 0x7faf99486140, 0xc4210d7a40, 0xc420e6c1a0, 0xc3d8c0, 0xc420dba0e0, 0x4106bf, 0xd09bc0, 0xd321a0, 0x413000, ...)

As of yet the reproduction case is not clear. The GetAll was on ~3500 entities.

Can you paste the commit SHA for your cloud.google.com/go pkg which generated this panic?
Will be hard to debug otherwise. The line numbers in your stack trace don't match master.

We have glcoud.google.com/go vendored at commit 30b36ef. The line creating the panic is https://github.com/GoogleCloudPlatform/google-cloud-go/blob/30b36efdf5c81a1d3f9fac38e1496c29e8adc420/datastore/query.go#L668

Thanks.
Seems probable that the returned resp can have a nil Batch and we're not accounting for it.
@jba can you take a look?

That seems to be the only explanation, but the docs say that it can't happen.
If you can reproduce, that would be great.

I just consulted with the Google Datastore team, and they don't see how Batch can be nil. This may be a bug in our client, but I can't see how. Without more information or a repro I don't think we'll make progress. You can try adding some log statements to your vendored client. Printing t.req and resp with %#v may tell us something.

It looks like this may be a bug in a gRPC client interceptor we use for retrying GDS RPCs. I am upstreaming a fix for the gRPC middleware in question. I'll close this issue for now, thank you for your patience.


build: include cls_rbd in rook's dependencies

I would like to find a better solution for this. but this is fine for now.


nextrelease benchmark

Run various benchmarks which can lead to quantifiable data established every release, in this case, establish and record various benchmarks for the v0.14.0 "jack-o'-lantern" release (https://github.com/lily-seabreeze/sappho/milestone/1). The desired effect is having a base to compare optimizations against, or any refactoring at all.
Open to discussion; hoping to hear approaches. A few I can think of:

Use something like pycallgraph for a single frame, or many frames averaged out (I think it may do this automagically)
Benchmarks through demo.py
Use tests to determine benchmarks

@txanatan is this something that'd interest you?


Chapter 16 exercise 9 solution

Since a solution to chapter 16 exercise 9 is currently not in this repository,
would an owner of this repo approve a PR which copies my solution (taking out the Colab specific parts) into this repository?

That would be absolutely awesome, thanks a lot @DCtheTall !
Could you please add an intro or outro paragraph to give yourself proper credit? Something like this:
"Big thanks to Dylan Cutler who created the solution to this exercise."
👍


Fix bug in ipsec when tunnel isolation is disabled

Fix #6022 that will add multiple phase 2 IPs to the same interface when tunnel isolation is disabled.
@AdSchellevis I believe that as $tunnels is not defined anywhere on this function, using $configured_intf it's able to append to the ['networks'] array which configures multiple IPs on the ipsec interface when tunnel isolation is disabled. This will imitate pfsense behavior.

@trunet nice catch! thanks! looks like a regression in https://github.com/opnsense/core/commit/77743cfc091cfe3c3cfdb4af97252b4dbb7ab73e


CCMenuAdvanced inertial scrolling

While scrolling if you move your finger and than just release your finger of the menu if should scroll a bit more - smooth scrolling
Like a UITableView

Reverted changes back. Inertial scrolling doesn't work in horizontal mode & is laggish.
Probably some additional calculations for velocity is needed during moving finger.
Moving this issue futher from 0.2 milestone.


Convert to use new approach for tests on single device

@Mekahou   You can review and merge when you are ready.  If things are unclear then lets update to make this code as clean as possible.
I don't believe this should break any interfaces, but it is worth doublchecking a few things.  I tested it on the GPU so don't worry about that.


Stackblur produces incorrect image

See https://github.com/TheQwertiest/foo_spider_monkey_panel/issues/57#issuecomment-470099440

@Ottodix, should be fixed in the latest build, once it's ready


Broken Links Fixed

PR Description 📜
Fixed all the broken links
Also fixed the background theme
and fixed styles for full site
Fixes #740

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for review.

## Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

resolve conflicts

@ayush-t02 Done


Don't give slash in annotations and multi-line directives special meaning

(e.g. don't treat / as start of regex)
Note that the ABNF in this branch doesn't include the @{exclude-min}, @{exclude-max}, @{default} annotations, so some merges will need to be done before yanking the ABNF into the new draft.

Coverage increased (+0.9%) to 97.587% when pulling 35229ebadea07ee8e5e06ecc453484e36412dcd5 on codalogic:slash-in-annotations into 2e55b0eef2e2a65a5ee7b7a8743f829846663447 on arineng:master.

Coverage increased (+0.9%) to 97.587% when pulling 35229ebadea07ee8e5e06ecc453484e36412dcd5 on codalogic:slash-in-annotations into 2e55b0eef2e2a65a5ee7b7a8743f829846663447 on arineng:master.

Coverage increased (+0.9%) to 97.587% when pulling 35229ebadea07ee8e5e06ecc453484e36412dcd5 on codalogic:slash-in-annotations into 2e55b0eef2e2a65a5ee7b7a8743f829846663447 on arineng:master.


Bring in container_memory_reservation config

The default config value is described in here.

@antonbabenko I have just tested the config update from default 128 to 256 for my atlantis cluster, it works good.

So I think we are good to bring this config back to the ECS task definition.

relates to #33


Mandrel integration tests are failing with quarkus main due to new warnings

Mandrel integration tests are failing with quarkus main due to https://github.com/quarkusio/quarkus/issues/21698
See https://github.com/graalvm/mandrel/runs/4319402886?check_suite_focus=true

Closed by https://github.com/quarkusio/quarkus/pull/21709


Symfony 5 support

Cant install with Symfony 5, "symfony/asset": "5.0.*", throws error

We are sorry but this package is actually not compatible with Symfony 5. README file lists Symfony versions.


Deprecate LOCAL2 in SphericalCoordinates

🦟 Bug fix
Summary
cc: @peci
With the introduction of https://github.com/gazebosim/gz-math/pull/616. We will be deprecating LOCAL2 as a work around in favor or LOCAL. As part of this change, its probably a good idea to update gz-msgs as well.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

there is one gcc deprecation warning left: https://build.osrfoundation.org/job/gz_msgs-ci-pr_any-noble-amd64/7/gcc/

Yeah, I'm not exactly sure how to solve this as the warning is coming from the generated protobuf. It seems like the Proto compiler reuses the highest value. There are several options. Perhaps I could introduce a DO_NOT_USE in the highest proto?

This change in https://github.com/gazebosim/gz-math/pull/616 triggered a regression in gz-transport14-debbuilder:
Reference build:

gz-transport14-debbuilder-jammy#512
gz-transport14-debbuilder-noble#513

Log output:
/usr/lib/x86_64-linux-gnu/pkgconfig/../../../include/gz/msgs11/gz/msgs/convert/SphericalCoordinates.hh:80:54: warning: ‘gz::math::v8::SphericalCoordinates::LOCAL2’ is deprecated [-Wdeprecated-declarations]
   80 |   return math::SphericalCoordinates::CoordinateType::LOCAL2;
      |                                                      ^~~~~~
/usr/lib/x86_64-linux-gnu/pkgconfig/../../../include/gz/math8/gz/math/SphericalCoordinates.hh:82:15: note: declared here
   82 |               LOCAL2 GZ_LOCAL2_DEPRECATED = 5
      |               ^~~~~~
autopkgtest [11:18:31]: @@@@@@@@@@@@@@@@@@@@ summary
build                FAIL stderr: In file included from /usr/lib/x86_64-linux-gnu/pkgconfig/../../../include/gz/msgs11/gz/msgs/Utility.hh:27,
autopkgtest [11:18:31]: Binaries: resetting testbed apt configuration

@arjo129  can you please take a look?

@Crola1702 Those errors should be fixed by this PR.

Thanks @peci1! Will use that PR.


Unable to map site path to UNC

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 4.9.4
Ruby: 2.0.0p648
Distribution: RHEL7
Module version: 3.0.0

How to reproduce (e.g Puppet code you use)
Adding a share path to a UNC location
iis_site {$websitename:
name        => $websitename,
path        => $sitepath,
port        => $port,
ip          => $ipaddress,
host_header => $websitename,
app_pool    => $applicationpool,
}
where $sitepath is defined in yaml as  sitepath: '\foo.com\bar\aaa\bb\ccc\ddd'
What are you seeing
Validate method failed for class path: File paths must be fully qualified, not sitepath: '\foo.com\bar\aaa\bb\ccc\ddd'
What behaviour did you expect instead
For it to map the UNC path to the sitepath
Output log
Any additional information you'd like to impart
In the iis_site.rb there is this line
raise("File paths must be fully qualified, not '#{value}'") unless value =~ %r{.:(/|\)} || value =~ %r{//[^/]+/[^/]+}
I'm not sure if the // regex is correct, should it be \ (\\) ?

Hi. I'm going to close this Issue. We deprecated the module and the new version is available here: https://forge.puppet.com/puppetlabs/iis


fuzzy: handle cases where no term is in range on rearrange()

Also fix an example
Signed-off-by: Bruno Dilly<EMAIL_ADDRESS>

LGTM

thanks, merged


full images

is there a way we can get frames the full size instead of it being resized smaller, and also is there a way we can input our own videos for it to be extracted

It's not resizing the videos in the script, maybe the download is.
In regards to adding local videos, yes I've pushed a new commit so you can pull in a directory containing 1 or more videos and it will perform extraction for  frames and faces
https://github.com/hassan-sd/tiktok-face-scraper/commit/3654a7f29452e442a4411669e60d6d6c408339b4


CVE-2007-2379 CPE meaning

https://nvd.nist.gov/vuln/detail/CVE-2007-2379
This used to have cpe:2.3:a:jquery:jquery:*:*:*:*:*:*:*:* but was changed to cpe:2.3:a:jquery:jquery:-:*:*:*:*:*:*:* in 2021.
What is the - supposed to mean? If it's still supposed to match all versions, then this is a False Negative.

@OrangeDog See NIST CPE Naming documentation for more info.
The meaning of hyphen is signalled in section <IP_ADDRESS>.1:

The logical value NA SHALL bind to a single hyphen.

So it stands for: version Not applicable/Not used

@aikebah so it should apply to all versions then?

It should then be reported for all versions (but I'm not sure whether that actually happens, because a hyphen SHOULD only be used for something that doesn't have a version). As jquery does use versions the 'known affected software configurations' in NVD should be corrected.


ProfileContext does not seem to work on NMT?

TF version R1.5, Cuda 9.1, Cudnn 7.0.4 NMT download from March 9
note: exact same problem occurred with earlier download of NMT
this has been duplicated on V100s and P100s
add the call to ProfileContext to train.py just before the training loop
and indent all the following lines by 2 spaces
this should profile the first 100 steps
with tf.contrib.tfprof.ProfileContext('/home/levinth/train_dir'
, trace_steps=range(600, 699),
dump_steps=[699]
                                            ) as pctx:

while global_step < num_train_steps:
  ### Run a step ###
  start_time = time.time()
  try:
    step_result = loaded_train_model.train(train_sess)
    hparams.epoch_step += 1

down to:
Done training
loaded_train_model.saver.save(
train_sess,
this generates a lot of errors when invoked as:
python -m nmt.nmt --src=de --tgt=en --hparams_path=nmt/standard_hparams/wmt16_gnmt_4_layer_small.json --out_dir=/home/levinth/nmt_temp/deen_gnmt --vocab_prefix=/home/levinth/WMT/vocab.bpe.32000 --train_prefix=/home/levinth/WMT/train.tok.clean.bpe.32000 --dev_prefix=/home/levinth/WMT/newstest2013.tok.bpe.32000 --test_prefix=/home/levinth/WMT/newstest2015.tok.bpe.32000 > deen_gnmt_log_256_cud2_mar9_prof.txt 2>&1
the file wmt16_gnmt_4_layer_small.json is the same as the default but with the hidden size set to 256
the log contains:
created infer model with fresh parameters, time 0.12s
Start step 0, lr 1, Mon Mar 19 12:53:10 2018
Init train iterator, skipping 0 elements
2018-03-19 12:53:18.052126: I tensorflow/stream_executor/dso_loader.cc:147] successfully opened CUDA library libcupti.so.9.1 locally
Node dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/multi_rnn_cell/cell_0_attention/cell_0_attention/attention/dropout/random_uniform incompatible shapes: Shapes (?, 512) and () are not compatible.
each of these of errors occurs 39 times (in various orders)
Node dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/multi_rnn_cell/cell_0_attention/cell_0_attention/attention/dropout/random_uniform incompatible shapes: Shapes (?, 512) and () are not compatible.
Node dynamic_seq2seq/encoder/bidirectional_rnn/fw/fw/while/dropout/random_uniform incompatible shapes: Shapes (?, 256) and () are not compatible.
Node dynamic_seq2seq/encoder/bidirectional_rnn/bw/bw/while/dropout/random_uniform incompatible shapes: Shapes (?, 256) and () are not compatible.
Node dynamic_seq2seq/encoder/rnn/while/dropout/random_uniform incompatible shapes: Shapes (?, 512) and () are not compatible.
a profile gets created but appears to only have data from a single step
tfprof> op -select micros,occurrence,float_ops -order_by micros -step 1000
Options -step=1000 not found.
Available steps: 11
any thoughts on how to get this to work?

the lines in large font like
, trace_steps=range(600, 699),
dump_steps=[699]
are commented out with pound signs..hence the font change
I should have deleted them
note trying to switch the range from the first 100 steps  to steps 600-699 made things MUCH worse


findspam.py: obfuscated_words: prevent FP on contraction of "who are"

https://metasmoke.erwaysoftware.com/search?utf8=✓&why_is_regex=1&why=Obfuscated+word+in+\w%2B+-+"who're" is massively FP, and the hits where it's not FP are coincidental, and usually also trigger other rules

I'm sorry about not reviewing this. I got swamped by some stuff in real life and completely forgot about the review notification.
It looks good. I'm glad you merged it.


Diagonal Movement calculated wrong

Hello,
even though i enabled the diagonal movement rule to be euclidean

it doesnt calculate it correct and just takes 5ft. for every diagonal move.

I tried to install the PF2 Integration in hopes i get the correct measurement now but after installing the module i dont see any difference and dont even see the module in my module management nor in the Drag Ruler configurations.

Is there a way to fix this? We use diagonal movement quite frequently and it has to be correct for us otherwise we cant use this module.
Thanks in advance

The Pf2 module only works in Pathfinder 2e, as the name suggests. This is the reason wh it's not listed in the module management.
Do you have the Terrain Ruler module installed? If so, try if deactivating that module solves the issue and please report the results.

Yeah, disabling the terrain ruler did the trick. Now it measures perfectly fine.
But when i apply difficult terrain via Enhanced Terrain Layer it doesnt apply the difficult terrain to your drag ruler so we have to count it manually again and...Is there a way to work around this?

Terrain Ruler currently doesn't support euclidian measurements. Since this isn't a bug in Drag Ruler itself I'm going to closr this issue. However if you would like to see support for that in Terrain Ruler, feel free to open an issue in the Terrain Ruler repo.
Until Terrain Ruler supports euclidian distances a possible workaroud would be using the 5/10/5 grid rule, which is a pretty good approximation of euclidian.


Rename folder asakatsu_reports

Rename the folder asakatsu_reports to reposts.

@elucian Thank you for this Elucian! I merged the PR so please close the issue.


diesel does not connection to postgres database with sslmode required

Setup
Versions

Rust: stable 11 sept 2022
Diesel: 2.0.0
Database: postgres
Operating System mac m1

Feature Flags

diesel:
["postgres","numeric","chrono"]

Problem Description
diesel-cli runs migrations successfully with sslmode=require attached to database url
but when performing diesel_insert in program, it says error performing TLS handshake: no TLS implementation configured"
What are you trying to accomplish?
Connect to postgres database with sslmode required.
What is the expected output?
Successful database insert
What is the actual output?
error performing TLS handshake: no TLS implementation configured"
Are you seeing any additional errors?
N/A
Steps to reproduce
Setup postgres 14 database with encryption only connection.
Create insert query using diesel with sslmode required

Checklist

[x] I have already looked over the issue tracker  and the disussion forum  for similar possible closed issues.

https://github.com/sgrif/pq-sys/issues/25

[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[x] This issue can be reproduced without requiring a third party crate

Also adding allow to sslmode url query results in
UnableToSendCommand, "invalid connection string: invalid value for option sslmode

This sounds like an issue with your environment. Can you verify that psql postgres://your/db_url works for you. If not that indicates that your database url does just not match your setup.


TypeScript Extension Conversion

Convert the extension from PS ---> TypeScript to run on both Windows and Ubuntu build agents.

Hello @ejohn20, Can you help me to complete this feature ? I converted the script has you can see in my fork. To get the VSIX working on Azure DevOps, I need to run the following script but I'm not sure how to integrate that in your CI/CD.
$env:BUILD_VERSION="<IP_ADDRESS>"
$env:BUILD_ENVIRONMENT="Dev"

Remove-Item -Path D:\code\alaincroisetiere\azuredevops\node_modules -Force -Recurse
Remove-Item -Path D:\code\alaincroisetiere\azuredevops\src\Tasks\dependency-check-build-task\node_modules -Force -Recurse

cd D:\code\alaincroisetiere\azuredevops\src\Tasks\dependency-check-build-task\
npm install
tsc
Remove-Item D:\code\alaincroisetiere\azuredevops\src\Tasks\dependency-check-build-task\node_modules\*\.github -Force -Recurse

cd D:\code\alaincroisetiere\azuredevops\
npm install

.\build\set-installer-version.ps1
npm run package-dev

Amazing! I wasn't aware that anyone was going to pick this one up! :)
Can you email / DM me your Azure DevOps org name? I will share the "dev" version of the extension with you and make the above modifications to see if we can get this working in dev.

This is the same as my github username, alaincroisetiere.

@alaincroisetiere The OWASP Dependency Check - DEV extension has been shared with you. I'm updating the build definition to include the commands you mentioned above.

The build pipeline handles the new TS version's build / release just fine. I bumped the new version to 6.* so folks relying on the PS version can opt in to the TS version rather than it being auto-installed.
Version <IP_ADDRESS> published in dev: https://marketplace.visualstudio.com/items?itemName=dependency-check.dependencycheck-dev.
Feel free to start testing, etc. and let me know if you need to make adjustments. It will probably be a few days before I have time to vet both agents and test the new fields, etc.

The compiled dependency-check-build-task.js is missing into vsix. Is build.sh executed in the CI/CD ?

Thanks for the heads up. Missed these commands in the build pipeline:
cd .\src\Tasks\dependency-check-build-task\
npm install
npm run build
Remove-Item .\node_modules\*\.github -Force -Recurse

I have added them and a new version <IP_ADDRESS> published to the marketplace. Can you take a look and verify it's packaged correctly.
To make this easier going forward, I'm going to convert the Azure Pipeline to a YAML driven workflow so it's in this repo as well. Will take a few days to get around to it.

My first run with the dev extension didn't make it very far:
2021-02-17T14:16:57.0268914Z Starting Dependency Check...
2021-02-17T14:16:57.0319347Z Not found scanPath: D:\a\1\s\**\*.csproj
2021-02-17T14:16:57.0321424Z ##[error]Not found scanPath: D:\a\1\s\**\*.csproj
2021-02-17T14:16:57.0324662Z ##[error]Unhandled error condition detected.
2021-02-17T14:16:57.0327187Z Ending Dependency Check...
2021-02-17T14:16:57.0390476Z ##[section]Finishing: Dependency Check - DEV

The value D:\a\1\s\**\*.csproj is an invalid folder path.

I see, this validation might be too strict then. The --scan parameter for dependency check allows ANT style paths such as this to recursively find all files with a given extension all directories.

The exemple in the official documentation is a file path. I will fix.

Just re-read the docs. Both the --scan and --exclude support ANT style paths, so this would affect both of those fields.

@alaincroisetiere  Thanks for the bump. Did the new build successfully run for you on your side? I'll test it again shortly.

Yes, build successfully run on my side but i didn't test the dataMirror parameter. Can you test it or provide an URL to me ?

Just ran a couple builds through on my side. One with the data mirror and one without, and confirmed the data mirror is loading the cached data files into the data directory. These were both on a Windows agent. I'm going to test a Linux agent quick and make sure that works OK.
Fields that I have not tested yet, which are mainly for on-premise installations:

Local Install Path
Custom Installer Repository URL

I tested Local Install Path with an agent on my pc: succeed.

Also tested Custom Installer Repository URL with the GitHub release URL: succeed.

@alaincroisetiere Sorry for the delay. Testing on linux agents now, and will release in v6.0 if all goes well! :)

Merged. Tested the v6.* extension on windows and linux, worked great. Awesome work on this @alaincroisetiere


Can I use this from Objective-C?

Im just finding new ImagePicker controller and I found yours is very cool one. Is there any advice to use it from obj-c ?
Thx a lot.
// sry for my bad Eng and Knowledge.

Of couse~! I will update README :)

Thx a lot man. Anyway from many Custom ImagePicker Controller I found a problem that I can only see 49 photos from "All Photos". Do I miss something?

That seemed strange. I've never encountered that...

Look this: https://github.com/zhangao0086/DKImagePickerController#how-to-use-in-objective-c

Thank you very much.

when i press the select button after selecting images , the method
-(void)imagePickerController:(DKImagePickerController * _Nonnull)imagePickerController didSelectAssets:(NSArray<DKAsset *> * _Nonnull)didSelectAssets is not called.
However the block [pickerController setDidSelectAssets:^(NSArray * __nonnull assets) {
NSLog(@"didSelectAssets");
}]; is called. How to get the images selected using your library in objective c? I want to display them on an imageview.


bug: TypeError in unsupported operand type(s) for ** or pow(): numpy.ndarray and Tensor

https://github.com/sp-nitech/diffsptk/blob/053d8888baea01955116035973d76aaf52d8bcb7/diffsptk/misc/utils.py#L137
https://github.com/sp-nitech/diffsptk/blob/053d8888baea01955116035973d76aaf52d8bcb7/diffsptk/modules/freqt.py#L106
When mode="auto", get_alpha returned a numpy.ndarray, which does not support operations with Tensor. In contrast, when mode="hts", get_alpha returned a float. Since PyTorch has built-in compatibility for float types, this will not raise an error.
This PR ensures that "auto" mode consistently return a float, which avoids type errors.

Thanks for the modification!


[ML][Pipelines] [DO NOT MERGE] test: add test for multi-return

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Grammar fixes, and capitalisation fixes

Just a few small fixes

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@PRMerger13: #sign-off

@salmanmkc : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Publishing is suspended until Monday, November 4. Merging will resume 7 AM Pacific Monday.
This PR is ready for merge.

Great, thanks @ktoliver and @ecfan for all the help! Apologies for so many corrections being needed


Wrong information in peer information window

There is IP in SSH info in the peer information window.
It is not an IP itself. It should be fixed to proper message.
Hostname maybe

This how it looks like (My Peers)

Not relevant anymore


Fix issue #4136: 'Restructuring documentation'

[x] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality that this introduces
Restructure documentation to be in an easier-to-follow format

Give a summary of what the PR does, explaining any non-trivial design decisions
Mostly self-explanatory, see #4136 for details

Link of any specific issues this addresses
Fixes #4136

Couple of notes:

Usage Methods shouldn't be that high because most people will not be using CLI, Headless or Github Actions yet.
LLM Configuration doesn't really fall under Advanced configuration. The LLM Configuration is for the most basic configuration of OpenHands with the most common providers. This should really still be right under Getting Started as that is essential for getting OpenHands setup. The Custom Sandbox makes sense as an Advanced Configuration

Rest looks good.

PS: I just realized that I should open this issue about creating a separate page for "GUI" on top of the "Usage Methods" page, which is something I'm thinking of doing after this initial restructure: https://github.com/All-Hands-AI/OpenHands/issues/4142

Hey @mamoodi , thanks!

I'd like to hear other's (particularly @rbren's) take on this as well, but I think we'd kinda like to meet developers where they're at, give them different options, and see which one people like the most. That was the thinking about this.
For basic LLM configuration, this is already included in "Getting Started". I feel that we're trying to move away from getting people to think about lots of different LMs, because only Claude works well at the moment (and maybe gpt-4o).

What do you think?

So is our stance that OpenHands only works with Claude Sonnet 3.5 at the moment?

So is our stance that OpenHands only works with Claude Sonnet 3.5 at the moment?

This is oversimplifying a bit (and we're working on publishing some numbers on which LMs work and which don't), but until we're able to make concrete recommendations about this I think that defaulting to only recommending Claude Sonnet 3.5 seems like a good idea.

So is our stance that OpenHands only works with Claude Sonnet 3.5 at the moment?

This is oversimplifying a bit (and we're working on publishing some numbers on which LMs work and which don't), but until we're able to make concrete recommendations about this I think that defaulting to only recommending Claude Sonnet 3.5 seems like a good idea.

Alright. Cool. Just want to know what to tell people.

I agree w/ Graham--while most people are using the UI, we'd like to start highlighting other ways of interacting with OH. So I think it's worth moving those up.
Re: LLMs, I do think we need better "LLM setup" docs for a few major providers (at least Anthropic and OpenAI--i.e. where do you get a key, etc). Maybe we can expand the getting started docs with specific OpenAI and Anthropic links, and then move the other LLMs into "Advanced"?

Opened an issue for the second one: https://github.com/All-Hands-AI/OpenHands/issues/4146


Can not snipe on certain positions

first of all thank you very much for this amazing program
I can not snipe players at certain position, anyone else has this problem?

@Brambert1233 , Can you give some example positions?

Closing it , please reopen if issue persist


Spacing lost between words when using \styleName

Hello,
Related to the closed issue #50
but with the v0.1.2
Example: Hello {\rHighlight}world{\rDefault} !
Displayed Helloworld!
With others ASS renderers is ok.
Hopefully it will be fixed, as this package is the one with the best rendering. 🙏

If that can help. When I change via the devTools data-wrap-style="2" to 0 or 3 the spaces are good.

Fixed in 0.1.3

Great news, thanks!
I have tested and it's ok.


Create optional byte string wrapper for map keys

Description (motivation)
This allows the parsing of CBOR data containing maps with byte string
keys, which is commonly seen in Cardano blockchain data. The default
behavior remains the legacy behavior (throw an error) for backward
compatibility.
This is related to #312
PR Was Proposed and Welcomed in Currently Open Issue
This PR was proposed and welcomed by maintainer(s) in issue #
Closes #
NOTE: I'm leaving this section blank, since this fix was not necessarily "welcomed" by the maintainer :)
Checklist (for code PR only, ignore for docs PR)

[x] Include unit tests that cover the new code
[x] Pass all unit tests
[x] Pass all 18 ci linters (golint, gosec, staticcheck, etc.)
[x] Sign each commit with your real name and email.
Last line of each commit message should be in this format:
Signed-off-by: Firstname Lastname<EMAIL_ADDRESS>[x] Certify the Developer's Certificate of Origin 1.1
(see next section).

Certify the Developer's Certificate of Origin 1.1

[x] By marking this item as completed, I certify
the Developer Certificate of Origin 1.1.

Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
660 York Street, Suite 102,
San Francisco, CA 94110 USA

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@fxamacker Any chance you could review this PR/allow tests to run?  This functionality would be really useful.

@jwitko My current schedule is swamped, so I allowed the tests to run without review.

@jwitko My current schedule is swamped, so I allowed the tests to run without review.

Thank you!

Until this is reviewed, tested for performance regressions, and fuzzed, maybe we can create a branch in this repo that includes this PR until we can merge into main branch.
@fxamacker and others, what do you think of this idea?

Until this is reviewed, tested for performance regressions, and fuzzed, maybe we can create a branch in this repo that includes this PR until we can merge into main branch.
@fxamacker and others, what do you think of this idea?

For what purpose? This PR's branch can already be checked out using the pull/319/head ref (assuming you've explicitly fetched it)

https://github.com/fxamacker/cbor/issues/320 will handle this use case, just like that "encoding/json" doing.

#320 will handle this use case, just like that "encoding/json" doing.

I'm not entirely sure that this is the same case, since you're converting the byte array map key to a string. The original CBOR contains a map with bytestring/array keys (which is different than text strings), and I see nothing in #342 that will change the way this is handled.

Since it doesn't seem like this change is wanted and there are now conflicts, I'm just going to close this PR.
I implemented a wrapper object that can use the ByteString type from this PR when parsing arbitrary CBOR in order to solve my problem. This can be seen in https://github.com/cloudstruct/go-cardano-ledger/pull/16

@agaffney I'm sorry about the delayed response.  It has been a hectic year and I have some time off at end of year planned to try to take care of pending PRs and other delayed tasks.

@fxamacker Is there still a chance of getting something like this merged? It would be a lot more convenient if I didn't have to use a wrapper object, since it adds a layer of indirection when accessing the parsed value, as well as it not supporting deserializing into an arbitrary struct nested under the top-layer value using the wrapper.

@agaffney I'll have a better idea after I take a closer look at this PR and https://github.com/cloudstruct/go-cardano-ledger/pull/16 during end-of-year holidays.
My job consumed most of my weekends this year, so I'll be catching up on delayed tasks like this during the holidays.

Hey @agaffney I finished reviewing this PR and have a local copy with changes that I've tested and am fuzzing.
p.s.  my father was hospitalized and is going into surgery this week so there may be delays in communications.

Take care of your personal stuff and don't feel compelled to look at this. I'm not blocked on this, but I'll gladly remove my workaround in favor of this (or similar) upstream support if/when it gets merged.

@agaffney Thanks.  It looks like it would be easier for me to open a new PR, mark you as co-author, and merge that one after fuzzing it.
Would that be OK with you if I open a new PR and add you as its co-author?

Yes, that's fine with me.

Closing this because PR #376 includes this PR and some changes, with permission from @agaffney.