Using camera to scan QR cause the wallet to crash (MacOS)

In testing if Sparrow can scan UPPERCASE QR codes (https://github.com/btcpayserver/btcpayserver/issues/2110) and trying to use the camera to do so makes the app crash.
Process:               Sparrow [12880]
Path:                  /Applications/Sparrow.app/Contents/MacOS/Sparrow
Identifier:            Sparrow
Version:               0.9.3 (100)
Code Type:             X86-64 (Native)
Parent Process:        ??? [1]
Responsible:           Sparrow [12880]
User ID:               501
Date/Time:             2020-12-12 06:51:53.379 -0500
OS Version:            Mac OS X 10.14.6 (18G103)
Model: MacBookPro14,1, BootROM <IP_ADDRESS>.0, 2 processors, Intel Core i5, 2.3 GHz, 8 GB, SMC 2.43f6
Full crash report here: https://pastebin.com/XZhFaLaL

I'm able to successfully scan all the QR codes on that Github issue. Unfortunately, this means the crash is due to the native library that Sparrow uses for the camera (and not the QR code specifically). Thanks for the detailed info - I've heard some reports of crashes but haven't had this kind of info yet.
Does a crash occur every time the camera is opened, or only occasionally? If it's occasional, is there a pattern of usage that seems to trigger it (for example, opening and closing the camera quickly in succession)?

It happens every-time. The pop-up widow opens, with a white screen, holds for maybe 1 second, and then the whole application crashes.

Noting the thread dump that caused the crash here for reference:
Thread 35 Crashed:: Java: atomic-processor-1
0   libsystem_kernel.dylib        	0x00007fff752862c6 __pthread_kill + 10
1   libsystem_pthread.dylib       	0x00007fff75341bf1 pthread_kill + 284
2   libsystem_c.dylib             	0x00007fff751f06a6 abort + 127
3   libjvm.dylib                  	0x00000001010d3a64 os::abort(bool, void*, void const*) + 22
4   libjvm.dylib                  	0x00000001012839c0 VMError::report_and_die(int, char const*, char const*, __va_list_tag*, Thread*, unsigned char*, void*, void*, char const*, int, unsigned long) + 2836
5   libjvm.dylib                  	0x0000000101282e86 VMError::report_and_die(Thread*, unsigned int, unsigned char*, void*, void*, char const*, ...) + 152
6   libjvm.dylib                  	0x0000000101283a0d VMError::report_and_die(Thread*, unsigned int, unsigned char*, void*, void*) + 33
7   libjvm.dylib                  	0x00000001010d737a JVM_handle_bsd_signal + 804
8   libjvm.dylib                  	0x00000001010d4f4c signalHandler(int, __siginfo*, void*) + 45
9   libsystem_platform.dylib      	0x00007fff75336b5d _sigtramp + 29
10  libjvm.dylib                  	0x00000001011ee0eb SymbolTable::do_lookup(char const*, int, unsigned long) + 83
11  libjvm.dylib                  	0x000000010124d5cd Unsafe_CopyMemory0(JNIEnv_*, _jobject*, _jobject*, long, _jobject*, long, long) + 157
12  ???                           	0x00000001124bed94 0 +<PHONE_NUMBER>
13  ???                           	0x000000010b0dca8c 0 +<PHONE_NUMBER>
14  ???                           	0x000000010a9a7d22 0 +<PHONE_NUMBER>
15  ???                           	0x000000010a9a7ee7 0 +<PHONE_NUMBER>
16  ???                           	0x000000010a9a7f2c 0 +<PHONE_NUMBER>
17  ???                           	0x000000010a9a7ee7 0 +<PHONE_NUMBER>
18  ???                           	0x000000010a9a7f2c 0 +<PHONE_NUMBER>
19  ???                           	0x000000010a99e949 0 +<PHONE_NUMBER>
20  libjvm.dylib                  	0x0000000100e45f02 JavaCalls::call_helper(JavaValue*, methodHandle const&, JavaCallArguments*, Thread*) + 598
21  libjvm.dylib                  	0x0000000100e452ec JavaCalls::call_virtual(JavaValue*, Klass*, Symbol*, Symbol*, JavaCallArguments*, Thread*) + 254
22  libjvm.dylib                  	0x0000000100e453a8 JavaCalls::call_virtual(JavaValue*, Handle, Klass*, Symbol*, Symbol*, Thread*) + 98
23  libjvm.dylib                  	0x0000000100ec3090 thread_entry(JavaThread*, Thread*) + 120
24  libjvm.dylib                  	0x000000010122d590 JavaThread::thread_main_inner() + 134
25  libjvm.dylib                  	0x000000010122d3ce JavaThread::run() + 280
26  libjvm.dylib                  	0x000000010122b199 Thread::call_run() + 113
27  libjvm.dylib                  	0x00000001010d332e thread_native_entry(Thread*) + 297
28  libsystem_pthread.dylib       	0x00007fff7533f2eb _pthread_body + 126
29  libsystem_pthread.dylib       	0x00007fff75342249 _pthread_start + 66
30  libsystem_pthread.dylib       	0x00007fff7533e40d thread_start + 13

@pedromvpg
Could I ask you to run a test? I've created a small program to isolate the webcam capture and print out debug logs here: https://github.com/craigraw/webcam-tester

@pedromvpg
Could I ask you to run a test? I've created a small program to isolate the webcam capture and print out debug logs here: https://github.com/craigraw/webcam-tester


Use BehaviorRelay instead of Variable

Sadly I had to add a new dependency to RxCocoa just to use BehaviorRelay.

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff          @@
##           master   #64   +/-   ##
====================================
  Coverage      96%   96%
====================================
  Files           7     7
  Lines         175   175
====================================
  Hits          168   168
  Misses          7     7

Impacted Files
Coverage Δ

Sources/ReactorKit/Stub.swift
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I don't want to include RxCocoa dependency to ReactorKit. I'm going to find a better way to deal with it.

Thank you both for your work. Do you have any ideas on how to fix this warning without RxCocoa?

Resolved by #82


Fixed latest version number in Cosmos DB emulator release notes

Fixed latest version number in release notes to 2.14.12 instead of 2.14.2

@sajeetharan
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

It looks like this change was already published by another contributor.

#please-close


Add changes to support oauth2 security via params.yaml

Purpose
Fixes https://github.com/wso2/product-apim-tooling/issues/786
Backporting https://github.com/wso2/docs-apim/pull/4842

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Naduni Pamudika seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Triangular patches when in smoother mode

Hello,
I managed to make my program create MFM mesh, and also create a Const3D solution file, and I am getting the results below:

when I press the f key o get the smoother on I get the following results:

Some triangular parts are missing. Does this mean that my mesh is generated wrong? Maybe there is a problem with ordering or reverseness of the boundary?
Any ideas/recomendations/help greatly appreciated.
Best Regards,
C.

Hi @altineller :). I'm still learning GLVis myself so I can't say for sure, but your second image has patterns that look very similar to incorrect cell/face ordering (as a quick sanity check you could permutate your indices and see if those artifacts still appear). Please post your mesh/sol file and I can take a look. If you can generate a smaller example illustrating your problem that would be helpful.

Hello @justinlaughlin,
Yes, the ordering of certain boundaries were in wrong order. We have fixed it.
Thank you.


[BEAM-1555] Disable newly added tests, that is failing is post-commits.

R: @pabloem

Coverage decreased (-0.003%) to 69.309% when pulling 32cd4836d2441c10fa71df4344c6b546c2c0ca3a on aaltay:post into 437ba250525f5b611a3e3c31bee782aea24a16ce on apache:master.

R: @chamikaramj

LGTM

Thank you.


D3log UUID

a bit of a hodgepodge -
stop encoding network addresses directly in the location and provide a relation to map opaque locations to network endpoints
continue to add comments and assuage clippy
since send is no longer exposed as async, make it a proper trait so we can genericize the plumbing a little better

since we're are going to high school next week, we can put aside this 6th grade homework


Fix accidental typo in test name

This was my fault when I moved some tests around in #6440

Motivation

:shame:
Test plan

See included automated tests.

Current dependencies on/for this PR:

master

PR #6510   👈

PR #6422

This comment was auto-generated by Graphite.


Optional and optional named query parameters are not handled properly

Description
This is arguably a feature request, however it can be considered a bug as well as the optional and optional named parameters are accepted.
To Reproduce
Implement something like:
import 'package:dio/dio.dart';
import 'package:flutter/cupertino.dart';
import 'package:retrofit/http.dart';

part 'my_client.g.dart';

@RestApi()
abstract class MyClient {

  factory MyClient(Dio dio) = _MyClient;

  @GET("content/api")
  Future<Result> apiCall(@Header(@Query("apikey") String apiKey,
      @Query("scope") String scope,
      @Query("type") String type);

  @GET("content/api")
  Future<Result> namedExample(@Query("apikey") String apiKey,
      @Query("scope") String scope,
      @Query("type") String type, {
        @required @Query("from") int from
      });

  @GET("content/api")
  Future<Result> optionalNamedExample(@Query("apikey") String apiKey,
      @Query("scope") String scope,
      @Query("type") String type, {
        @Query("from") int from
      });

  @GET("content/api")
  Future<Result> optionalExample(@Query("apikey") String apiKey,
      @Query("scope") String scope,
      @Query("type") String type, [
        @Query("from") int from
      ]);
}

Expected behavior
In the case of namedExample, I would expect a null assertion to occur for the from parameter as of the existence of the @required annotation, however this is not the case.
In the case of optionalNamedExample, I would expect no null assertion for the from parameter as @required is not present. This is correct, however I would expect it to only be added to the queryParameters if from is not null (I think Square's Retrofit handles this). This could be done simply with Dart 2.3's control flow collections by doing:
final queryParameters = <String, dynamic>{
      'apikey': apiKey,
      'scope': scope,
      'type': type,
      if(from != null) 'from': from
    };

In the case on optionalExample, I would expect the same sort of behaviour as above.

p.s. great job with the library!

@required is deprecated, Work is in progress for the new required syntax see: https://github.com/dart-lang/language/issues/15.
Temporarily I can enable the @required But It will go away in the future.

is this realized? optional query parameter now adds even it is null


Update SiteStructure.ps1

This patch treats the Web nodes as an array and Count will be available in all circumstances.

Thanks for your fix.


How can i access to csrf_token()

When I do csrf_token() in the tenant, it returns empty.

Can you give more detail? I can't help you with such a vague question.
Also, I'm trying to keep this issue tracker organized. There's quite a few open issues already. Support questions are meant to be asked on our Discord community — this is not a documentation issue. So please ask there, thanks!


passivity causality check

Passivity and causality check added using genequiv run.

Codecov Report
Attention: Patch coverage is 69.38776% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 76.61%. Comparing base (7fe1146) to head (7f67a33).
Report is 54 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4319      +/-   ##
==========================================
- Coverage   82.10%   76.61%   -5.49%
==========================================
  Files         183      107      -76
  Lines       64464    52161   -12303
==========================================
- Hits        52927    39964   -12963
- Misses      11537    12197     +660

There are also Codacy issues


Python3

Updated To Python3!

+1 for 3

It seems this project doesn't get maintained anymore. Does anyone know a fork that gets maintained?

Hi All:  Yes, I'll be migrating all of my Github projects to Python3 over the summer of 2020. Thanks for your patience.
StevenACi: Greatly appreciate the pull request.
Background: Cancer recovery + work + PhD turned out to be a bad mix. Do not recommend. emoji smirk  (All's good now, be safe out there.)

Np. All I did was make a little parcer to go through lines and change every
print statement/ multiline statement and I think I was done. Unfortunately
I didn't save it though.
Good luck updating stuff, I'm sure it's appreciated.
On Wed, May 27, 2020, 11:11 AM TryCatchHCF<EMAIL_ADDRESS>wrote:

Hi All: Yes, I'll be migrating all of my Github projects to Python3 over
the summer of 2020. Thanks for your patience.
StevenACi: Greatly appreciate the pull request.
Background: Cancer recovery + work + PhD turned out to be a bad mix. Do
not recommend. emoji smirk (All's good now, be safe out there.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TryCatchHCF/DumpsterFire/pull/3#issuecomment-634728448,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUA3E7JIYFPZAQH2SKXJ23RTUURTANCNFSM4IE6RH5Q
.

Also, glad the recovery went well. All the best.
On Wed, May 27, 2020, 12:18 PM Steve Carino<EMAIL_ADDRESS>wrote:

Np. All I did was make a little parcer to go through lines and change
every print statement/ multiline statement and I think I was done.
Unfortunately I didn't save it though.
Good luck updating stuff, I'm sure it's appreciated.
On Wed, May 27, 2020, 11:11 AM TryCatchHCF<EMAIL_ADDRESS>wrote:

Hi All: Yes, I'll be migrating all of my Github projects to Python3 over
the summer of 2020. Thanks for your patience.
StevenACi: Greatly appreciate the pull request.
Background: Cancer recovery + work + PhD turned out to be a bad mix. Do
not recommend. emoji smirk (All's good now, be safe out there.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TryCatchHCF/DumpsterFire/pull/3#issuecomment-634728448,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUA3E7JIYFPZAQH2SKXJ23RTUURTANCNFSM4IE6RH5Q
.


How can i use custom dataset to fine-tuning caption task?

Dear author，
It is an awesome job！
I want to use a custom data set for caption Task. I want to know how to generate BPE related  files and 【coco-valid-words.p】 files using custom data sets. I have generated TSV files for the training.

@zml110120 Are you using English data? If so, you just need to use the default BPE file we provided.
coco-valid-words.p contains two items:

ref_len: the number of images in the validation set;
document_frequency: the document frequency of each word.

You can refer to utils/cider/pyciderevalcap/ciderD/ciderD_scorer.py to construct this file

That's right. I'm using data in English. Now I only have a Json file in COCO format. I don't have any other files, like 【Pickle】. How do I generate 【coco-valid-words.p】?

@logicwong That's right. I'm using data in English. Now I only have a Json file in COCO format. I don't have any other files, like 【Pickle】. How do I generate 【coco-valid-words.p】?

@zml110120 coco-valid-words.p is used to compute CIDEr score on the validation set during validation. If you don't need this feature, you can remove --eval-cider, --eval-cider-cached-tokens, and --eval-args.


Improve new issue template selection - reorder and handle "this is crash" check better

Acknowledgement

[X] I acknowledge that issues using this template may be closed without further explanation at the maintainer's discretion.

Comment
I propose that templates would be reordered so that "Types not correct in/with callback" would be first one. It may first feel unnecessary to have it as first but I think it would get people to use it more when it's applicable. -> Meaning those could be handled quickly differently
About "Bug report", I think it would be good to separate compiler / language server crash to different template than the other problems. Or alternatively, improve "This is crash" text to make it clear that what it means. Now so many type error issues are flagged as "this is crash" because people do not understand what it means.  :)

I'm pretty happy with the templates as-is

Types not correct in/with callback
get people to use it more when it's applicable

The whole point of that template is that it's never applicable - it only exists to tell you that the behavior it describes (which people tend to assume is a bug) is by design/design limitation and you shouldn't post the issue at all because it will just be duped to #9998.


TP2000-1029 - feat: Measure list view requires optimising

TP2000-1029 - feat: Measure list view requires optimising

Why

Loading measures list views takes a long time to load. On a typical developement development workstation (normally having more computing resource available than the TAP production environment), the response time is in excess of 60 seconds.
The response time should be reduced to an acceptable amount of time
What

A variety of approaches have been taken, primarily using select related rather than fetching all from certain models, also, setting defaults for things like start dates, prefixes and units to prevent data being fetched when it does not exist.
This has reduced large queries from 40s to 20s.

It may be useful to coordinate with @LaurenQurashi as she's doing work with the measures list view, including optimisations.

Codecov Report
Patch coverage: 85.00% and project coverage change: +0.01% :tada:

Comparison is base (7024848) 92.65% compared to head (a6efc6c) 92.66%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1031      +/-   ##
==========================================
+ Coverage   92.65%   92.66%   +0.01%
==========================================
  Files         451      451
  Lines       32717    32740      +23
  Branches     2499     2497       -2
==========================================
+ Hits        30314    30339      +25
+ Misses       1918     1916       -2
  Partials      485      485

Files Changed
Coverage Δ

measures/filters.py
79.62% <75.00%> (+0.92%)
:arrow_up:

measures/querysets.py
100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

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


Add append strategy (collision_strategy) for Import API

Our current import API works in the assumption that the resources are a whole, so if the import detects the resource has changed, it downloads it and fully replaces the current dataset with the new one.
There are use cases where the resource only reflects an incremental dataset, and it's actually desirable to design resources that way in scenarios where we are only appending new records, so we avoid importing the same old records over and over.
This feature request hence is to add a new strategy for the import API that appends the resource to the current synced table.
Afterthought, maybe a primary key should be also added as a parameter to perform an UPSERT operation instead of a blind INSERT. I realize there may be other details to take into account but hopefully, this is not a can of :bug:s
cc. @CartoDB/solutions

cc @jorgesancha @danicarrion @javitonino

VAMOS @jsanz thanks for documenting!
Other thoughts to consider as ideas, but definately not to overcomplicate anything to prevent this feature from happening:

this would be awesome for sync tables, but not only those.. a manual append request to the import API is just as useful. I'm sure that's obvious, but just to make sure
primary key parameter could be simple options e.g.  a single column with a sequence number or timestamp. Or more custom e.g. department, item_id if we wanted to have that flexibility/complexity
could consider a rolling cut-off feature based on a timestamp column, for example make it easy to have "last 14 days" of data synced automatically. This could help prevent tables filling up, or if we prefer them to fill up so they need to buy more storage, then we can omit this feature 😂

cc @jorgesancha @danicarrion @rochoa because I know that there's some work in progress around improving the importer and they might be interested in this feedback.

@pramsey @rafatower because of that same WIP awareness
@stevelewis99 I think you might be able to do your third bullet already through use of triggers or something...on dataset update, delete before time XXXX...maybe...

@inigomedina @alonsogarciapablo this is a feature that could really improve ETL possibilities :)


eg1.bd not found

Hi, nice project. Quick question:
I'm following your README and trying to run the graphical editor. When I navigate to the examples folder and type "bdedit eg1.bd", I get "ValueError: bdfile eg1.py not found".
Obviously the file doesn't exist, but there doesn't seem to be any *.bd files in the repo. Can you provide example .bd files? Why are they not present?
Thanks!

sorry to be slow.
It should have said eg1.bd not found rather than eg1.py.
I've just done a bit of polishing and eg1.bd is there now.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1027 ms

Resolved: Bofrost.at performance has improved in 10cb4c7 after 6 minutes.


feet: Implement endpoint to fetch billing plan details

This pull request introduces a new endpoint GET /api/v1/organizations/{organization_id}/billing-plans/{plan_name} that allows user to retrieve a billing plans associated with a specified organization. The endpoint responds with a JSON object containing the status of the request, a message, and an array of billing plans, each with its ID, name, price, and features.
{
  "status": 200,
  "message": "details fetched successfully",
  "data": {
    "plan": {
      "id": "premium",
      "name": "Premium Plan",
      "price": 20.00,
      "features": ["Feature 1", "Feature 2", "Feature 3"],
      "description": "Detailed description of the Premium Plan."
    }
  }
}

Related Issue (https://github.com/hngprojects/hng_boilerplate_python_fastapi_web/issue/298)
#298
Motivation and Context
This change is required to provide organizations with the ability to view available billing plans. It addresses the need for enhanced visibility of billing options, allowing organizations to make informed decisions about subscription levels and features they may want to use.
How Has This Been Tested?
The new endpoint has been tested using Postman and swagger to ensure that the correct billing plans are fetched based on valid organization IDs. Testing included the following:
Sending requests with valid authorization tokens.
Verifying the response format, status code, and data integrity.
Ensuring that appropriate status codes are returned for unauthorized tokens
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)
Checklist:

My code follows the code style of this project.
My change requires a change to the documentation.
I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I have added tests to cover my changes.
All new and existing tests passed.

Attach POSTMAN image of the test.

fix conflicts

@joboy-dev please check again
It's someone that actually broke my test

@Goketech you can check now


[Bug] Nested comments - undefined index in Tree

CommentManager->organiseComments is inputted with an array of comments, sorted by Ancestors ASC.
Due to this $path->traverse($ancestor) fails, since the tree is built in wrong order.
Proposed solution is to add NULLS FIRST to sorting statement, except, I'm not sure this is really the correct way to fix this.
Also: I am using ORM instead of MongoDB, perhaps this is why you haven't noticed it?

Hello,
Has this problem still unsolved yet?
I've also same problem in ->orderBy('c.ancesstors', 'ASC') in findCommentsByThread method .
Then i changed column to id instead of ancesstors.


Update README.md

document --privileged flag requirement for Elemental builds

@wrkode, can you also remove the package installation note. It is obsolete now.

@wrkode, can you also remove the package installation note. It is obsolete now.

removed and ready for review

I think we can skip this in favor of what's in #192

We shouldn't skip this. these are two independent documents.
The information for the build must result in a successful execution. this change allows that to occur. I'm ok for a generic README, but must be correct.

I looked at this before noticing #192. Feels redundant to have in both places, and this bleeds specific implementation stuff into an otherwise generic README.

the documents are independent.

@wrkode do we want to proceed with this? I see that you've agreed with the suggestions but there's no commit addressing them yet.

@wrkode do we want to proceed with this? I see that you've agreed with the suggestions but there's no commit addressing them yet.

missed the commit. yes LGTM and committing now

@atanasdinov , I've committed the suggest changes


minor update

added showSelectedLabels and showUnselectedLabels,
Hi, you have done great job. And This my contribution ❤

Hi, thanks for the PR! Im going to merge now and update the pub package this evening.


connectionState missing in types

WebRTC types does not have connectionState property defined on RTCPeerConnection object.
Need to find solution to fix it, or w8 for types update.
semantic error TS2339 Property 'connectionState' does not exist on type 'RTCPeerConnection'.

https://microsoft.github.io/PowerBI-JavaScript/interfaces/_node_modules_typedoc_node_modules_typescript_lib_lib_dom_d_.rtcpeerconnection.html#connectionstate


bad irreducible loop nesting

I'm seeing some bad loop display with amg2006 on theta.  This example
was compiled with gnu 7.x, but you'll get similar results with Intel.
In the picture below, there is a tower of 7 loops.  The top 5 are all
located at line 484, and then #6 at line 514 in par_interp.c.  In the
source, 514 is actually a direct child of 484 (actually, 480).
This is a clear sign of a tower of irreducible loops that all have the
same entry block, so they all get the same location.
I've seen this sort of problem, especially with AMG for a long time.
But now I have a plan.  I can collapse nested irreducible loops into
one big loop.
Of course, this is a heuristic and I have to try various codes to see
that it doesn't cause more harm than good.  But in this case, I'm
pretty hopeful.  My experience with parseapi is that when confronted
with an irreducible loop, it often generates a nest of irreducible
loops where a single loop would be more accurate.
Anyway, I consider this to be moderate to severe, and I want to
address this.  (Just like the prescription ads, do you suffer from
moderate to severe irreducible loop nesting?)
But it's not a blocker because master is just as bad.

Fixed in commit 391c71cf7d1.
commit 391c71cf7d1db74bc298c943d6e45ad0dabda930
Author: Mark W. Krentel<EMAIL_ADDRESS>Date:   Wed Feb 6 14:04:40 2019 -0600

    Merge irreducible loops.

    If L1 is an irreducible loop and L2 is an irreducible subloop and L1
    and L2 have the same file and line attribution, then merge L2 into L1.
    It's likely there is really only one loop in the source code.

    This only applies to irreducible loops.  Natural (reducible) loops are
    assumed to be correct.

M       src/lib/banal/Struct-Inline.hpp
M       src/lib/banal/Struct.cpp

The new picture is:

This rule is only a heuristic, not a clear case of correct vs
incorrect.  I've tried a few codes and I don't see any cases where it
makes things any worse.  (The presentation of irreducible loops is
usually pretty bad to begin with.)
But it's always possible that some new case will appear where this
solution is incomplete or misleading.


Discharged editions of committees can no longer be mailed using their committee alias

If you are in the LuCie, you receive mail sent to lucie@ but also to your specific edition (e.g. 2021) through lustrum21@, for example. When the committee is discharged, you should no longer receive the email intended for lucie@, but you should continue to receive email specifically sent to your edition. This does not seem to work at this moment.
The LuCie may be a complicated issue because it is not an annually occuring committee, but an easier example: if I was part of Rially 2019 I should receive rially@ as long as that committee is active. After the committee is discharged (and even when a Rially 2020 is charged), I should still receive rially2019@, for example.
Preferred solution: make<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>whatsoever still forward to the members of that edition's committee. Same for other committees who have multiple editions.

When a committee is 'discharged' (abolished), they are no longer 'working' for IA and their aliases are deactivated, as well as the active members accounts of people in the committee if they were only in that committee. This is how it has always worked.
If a period of time is wanted where the old committee is still reachable via their committee mail, they should not be abolished.

For specific cases where this is necessary, an extra alias can be created. For regular cases, the committee should be abolished once the work of the committee is finished, and thus the alias by default will stop working.


my post api should accept a model which has some readonly fields

my post api should accept a model which has some readonly fields(like id...), but swagger seems not to be work well,  ok, i found can add 'readonly' tag for work properly, but I think this is not a good way, acctually, the readonly should use for gin or gorm
Problem statement
type MyItem struct {
// id
// Read only: true
ID uint64 json:"id" yaml:"id" binding:"-" gorm:"PRIMARY_KEY;AUTO_INCREMENT" readonly:"true"
}
Swagger specification
Steps to reproduce
Environment
swagger version: x.x.x
go version: x.x.x
OS:

The swagger readOnly specification currently doesn't do anything on generated validations. See #936
There is no interaction between the swagger spec and the db-specific tags, unless you force tags manually.


[feaLib] Parser should understand contextual chained SinglePos rules

feaLib rejects the following feature file. However, makeotf v2.0.90 accepts this input, and interprets it as a chain rule for a SinglePos.
feature kern {
    pos [F T] [period]' -25;
} kern;

Althought the OpenType Feature File specification does not seem to give an example for this construct, its semantics is obvious. So feaLib should recognize it and produce the same output as makeotf.

Yes, makeOTF has a number of built-in optimizations that try to simplify certain constructs and express them in a "simpler" lookupType. This is sometimes a bit of a double-edged sword because the makeOTF feature parsing has an implicit "current lookup type" context. So when a feature definition contains rules immediately, without an explicit lookup definition, makeOTF "somehow" determines what the current lookup type should be. If the processor finds a foreign rule within the same flow, it protests.
This is not obvious behavior because these two snippets should probably yield different results:
feature kern {
pos [Y V] A -30;
pos [F T] [period]' -25;
} kern;
feature kern {
pos T [period comma]' A -45;
pos [F T] [period]' -25;
} kern;
Or not?

(Or maybe I'm mistaken and thissimplification does not happen.)

makeOTF has a number of built-in optimizations that try to simplify certain constructs
and express them in a "simpler" lookupType

Do you have examples where this happens?

I’m opening another issue for the parser bug: https://github.com/behdad/fonttools/issues/485, so that we can continue talking about changing lookup types here. Essentially, this would be an optimization. makeotf doesn’t seem to do it, but perhaps it should; eg., in the initial example on this bug, the result would be 12 bytes smaller and (perhaps more important) one less lookup, so the layout engine would have to hop around less between tables. Optimization will be an interesting topic, but it’s independent of fixing the parser. For now, I’d like to make it correct first.

Moved the optimization proposal to https://github.com/behdad/fonttools/issues/487


Screen refresh problem

Regression?
No
System Info and Version
 System/Version info
Hyprland, built from branch  at commit 9e781040d9067c2711ec2e9f5b47b76ef70762b3  (props: bump version to 0.41.1).
Date: Thu Jun 13 09:54:06 2024
Tag: v0.41.1, commits: 4818

flags: (if any)

System Information:
System name: Linux
Node name: void
Release: 6.9.5_1
Version: #1 SMP PREEMPT_DYNAMIC Mon Jun 17 11:50:08 UTC 2024

GPU information:
06:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] [1002:67df] (rev ef) (prog-if 00 [VGA controller])

os-release: NAME="Void"
ID="void"
PRETTY_NAME="Void Linux"
HOME_URL="https://voidlinux.org/"
DOCUMENTATION_URL="https://docs.voidlinux.org/"
LOGO="void-logo"
ANSI_COLOR="0;38;2;71;128;97"

DISTRIB_ID="void"

plugins:

Description
Sometimes when switching desktops or closing a program, part of the screen may stop updating
How to reproduce
This happens rarely so it is quite difficult to understand why it happens
Crash reports, logs, images, videos
I included the logs after this happened, so perhaps there will not be the necessary error there
hyprland.log

By the way, this started happening after I disconnected the third monitor and changed the settings

do I really have to search the issue tracker for yall every time #5386

Sorry I can't find them


Added get_screenshot_as_base64() inbuilt method to replace get_b64_screenshot() function

The get_b64_screenshot.py can be removed but I dont have a mac to test and confirm.


Fix test, bump sovrin-common-dev==0.3.17

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


Background tasks

Make a background task that runs every 1-2 hours, and refreshes the list over news articles, and then notify the user with a notification

https://developer.mozilla.org/en-US/docs/Web/API/Web_Periodic_Background_Synchronization_API


Update travis and makefile

travis: Do not clone
makefile: readd init target and add condition on include harness

/lgtm


Drop the org.apache.tomcat:catalina dependency from any pom.xml files that might be using it

Task Description
We should no longer need this dependency. This task should verify this theory, and, if so, we should remove this from all pom.xml files.
Tasks
The following tasks will need to be carried out:

[ ] Remove this from the strongbox-proto-parent) project.
[ ] Remove this from the strongbox-parent) project.
[ ] Remove this from all pom.xml files in the strongbox project.

Acceptance Test

[ ] Building the code with mvn clean install -Dintegration.tests still works.
[ ] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Help

Our chat
Points of contact:

@carlspring
@fuss86
@sbespalov
@steve-todorov

I'd like to try this.

Go for it!

I'd like to take up this issue.

@praXin ,
Somebody already expressed an interest in this task, but -- sure -- go ahead, if you think you could do it quickly! :)


Missing characters for WiFi passphrase

Hello, understand there is a limit of memory on the badges. I am unable to connect the badge to my home WiFi because the passphrase uses characters not found on the badge. Could create another SSID for them, not sure I want to do that just for the badges.
Could the firmware be upgraded to include all of the special characters found on a keyboard?

Is there a specific character you're looking for?  @seswho I'll check this in tonight and push a new version such that if you connect to WIFI and then go to settings->update you'll get a new keyboard with the following characters added:
.,/-_
If I missed on: simply add it to the K1 string in the vkeyboard.h file.

Hi,
Sorry, just saw your question.
Thanks for adding the characters listed in your question.
I must be strange or something. My passphrase contains .[(}~+ in it.
--
Steve Steiner
steveATseswhoDOTcom
“Oh, so they have internet on computers now!”
Homer Simpson
------- Original Message -------
On Thursday, October 12th, 2023 at 7:40 PM, CmdC0de @.***> wrote:

Is there a specific character you're looking for? @.***(https://github.com/seswho) I'll check this in tonight and push a new version such that if you connect to WIFI and then go to settings->update you'll get a new keyboard with the following characters added:
.,/-_
If I missed on: simply add it to the K1 string in the vkeyboard.h file.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Ah I’ll need to add those as I don’t think I added  []I’ll do that tonight and push a new release Sent and "spell corrected" by my iPhoneOn Oct 13, 2023, at 11:01, Steven Steiner @.***> wrote:
Hi,
Sorry, just saw your question.
Thanks for adding the characters listed in your question.
I must be strange or something. My passphrase contains .[(}~+ in it.
--
Steve Steiner
steveATseswhoDOTcom
“Oh, so they have internet on computers now!”
Homer Simpson
------- Original Message -------
On Thursday, October 12th, 2023 at 7:40 PM, CmdC0de @.***> wrote:

Is there a specific character you're looking for? @.***(https://github.com/seswho) I'll check this in tonight and push a new version such that if you connect to WIFI and then go to settings->update you'll get a new keyboard with the following characters added:
.,/-_
If I missed on: simply add it to the K1 string in the vkeyboard.h file.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

@seswho symbols added and posted to patch server.


Pythonic refactor

Description
Using responses library now
GitHub Issues

#34

Coverage increased (+12.8%) to 85.586% when pulling bf151b5ac828d609261a3768a0758d2f19b9093f on mcrowson:pythonic-refactor into 3c04ac983a2127d9521c0f0755434db74f1ca18b on refindlyllc:development.


Consider separating tasks into individual crates under a single workspace

(See cargo workspaces).
I propose that each task be split into a separate crate under the same workspace. I can think of a number of benefits:

While most crates would be implemented in a single src/main.rs file, solutions that can be used as libraries could be split into both main.rs and lib.rs. This is particularly important for tasks that require other tasks to be implemented, such as Compare sorting algorithms' performance.
Dependencies can be localized to individual tasks. Expensive-to-compile dependencies like image would only be compiled if you were working directly on a task that requires them. Tasks with no external dependencies can be built without requiring them at all.
cargo test, build, etc. would work on a single crate at a time, no need to specify --bin or --lib arguments. Compile times would greatly benefit, and even if you are working on a task that depends on other tasks, you would only have to recompile your own task.
A single Cargo.lock file would be shared across all crates, meaning that synchronizing dependencies wouldn't be much harder than it currently is.

The current implementation of workspaces doesn't currently have a way to build and test all crates at once, though this is an obvious extension that is mentioned in the RFC. Until that implementation lands, it would be trivial to provide a script to perform that action.
Either way, I think it would beneficial to experiment with this organization of our repository once the implementation lands in cargo. Thoughts?

Sounds pretty interesting! I think it is a good idea.

Some additional thoughts:
I think we could somewhat address #504 by grouping multiple solutions under a single crate. We would be able to maintain a clean structure by having a single folder for each task, but having multiple binaries within each folder. Unfortunately, this doesn't really address the problem of having one solution be the "definitive solution" -- it seems like you'd still have src/main.rs and bin/alternative.rs -- not to mention if there were alternate solutions for a library, you'd have to create a separate module, most likely.
Also, by using the forthcoming metadata keys in Cargo.toml, we could include a package.metadata.task-url (or some other name) key that contains the task URL. We could store the URL in a single place without having to duplicate the URL between Cargo.toml and the task implementation.

I also like this idea!

Just started working on this. The implementation so far:
All solution crates are grouped under a top-level folder called tasks. Within this folder, tasks are still grouped by namespace. For example, all sorting algorithm tasks are located under /tasks/sorting-algorithms/. I've also grouped alternative solutions to the same task under the same crate, just a different binary name.
The root-level crate now serves as a meta-crate for rust-rosetta. This means that the main library contains a number of utilities that can be utilized by various crates (test macros, coverage tools, etc.). coverage is now a binary at this top level. I imagine we could add to the list of binaries: I'll be adding a command to test/build all binaries pending the implementation of cargo test --all (In the future, I'd like to introduce a binary to generate wiki-ready markup from the name of a crate).
All solutions still must be explicitly listed in workspace.members. So, the root-level Cargo.toml still serves as a list of all tasks currently implemented in the repository.
If this organization makes sense, I'll continue implementing it this way. I'll be moving one crate per commit, but I can squash if things become unwieldy.
cc @Hoverbear

@euclio I'd like to take a look at this, is it up on a fork somewhere?

@Hoverbear I've moved a few crates at euclio/rust-rosetta, but I'm currently blocked on a bug in cargo.

Doh! So https://github.com/euclio/rust-rosetta/commit/aefee25f381ff618442a1bf1b5b48177d96b57f7 would be a typical commit for this?

Yeah, that's what's they'll look like :)

I like it. Is there any way I can offer assistance?

Sure, I think that since this is going to be a massive change it makes sense to do it in batches. I guess just pick a few crates and migrate them over once I we merge this first batch.
I'll wait to open a PR for this batch until the pending PRs are merged.

Okay! I'll work on closing them up.

Oh, they are! :)


Is insert with null value generate wrong sql query ?

Bug Report
It looks like a bug
CryptoExchanges(null, null) Should generates this request : INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT, DEFAULT);
But I see it generates: INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT)
With an error : Caused by: org.h2.jdbc.JdbcSQLSyntaxErrorException: Column count does not match; SQL statement:

I join a project with last version of Spring boot, Java, Kotlin and H2 R2dbc
You can find a test inside that generate this error
And an SQL script for init the DB
    @Test
    fun findByExchangeCodeNotNull() {

        // given
        val uni = CryptoExchanges(null, "uni")
        val null1 = CryptoExchanges(null, null)
        cryptoExchangesRepository.saveAll(listOf(uni, null1)).collectList().block()

        // when
        val res = cryptoExchangesRepository.findByExchangeCodeNotNull()

        // then
        res.`as`(StepVerifier::create)
            .expectNextMatches { it.id == 1L && it.exchangeCode == "uni" }
            .expectNextMatches { it.id == 2L && it.exchangeCode == null}
            .verifyComplete()
    }

    CREATE SCHEMA "common";

    CREATE TABLE "common"."CryptoExchanges"
    (
        "Id"           SERIAL NOT NULL,
        "ExchangeCode" character varying(30)
    );

    INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT, 'crypto');

    INSERT INTO "common"."CryptoExchanges" VALUES (DEFAULT, DEFAULT);

demo.zip
Best

This issue tracker hosts bug reports and feature requests for the R2DBC specification. Looking at your code, it seems related to Spring Data.

@mp911de thx I write to spring data now


Push the application to IBM Cloud

As a developer
I need to push our application to IBM Cloud
So that the public can access our latest version of app
Details & Assumptions:

Use ibmcloud cf push to push our application in the VM

Acceptance Criteria:
Given that the team add some features to our application
When I do ic cf push in the VM
Then the Cloud app will be re-deployed and it should have those new features

Added details and AC


rsync error: unexplained error (code 255)

I got this error when deploying.
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: unexplained error (code 255) at /BuildRoot/Library/Caches/com.apple.xbs/Sources/rsync/rsync-47/rsync/io.c(453) [sender=2.6.9]

I can ssh into the server and I only modified the html code.

The core Middleman project does not use rsync so that error must be coming from somewhere else in your system.


can't execute example

Environment
Ubuntu 16.04
go version go1.9 linux/amd64
Steps

go get -u github.com/John-Lin/tinynet
execute tinynet/example/topo-single-8.go

Issue

When I type go run topo-single-8.go, I got following error.
/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
/tmp/go-link-905310536/000003.o: In function `_cgo_f36fc5609ae4_C2func_load_simple_bpf':
/tmp/go-build/github.com/containernetworking/plugins/vendor/github.com/vishvananda/netlink/_obj/cgo-gcc-prolog:32: multiple definition of `_cgo_f36fc5609ae4_C2func_load_simple_bpf'
/tmp/go-link-905310536/000001.o:/tmp/go-build/github.com/vishvananda/netlink/_obj/cgo-gcc-prolog:32: first defined here
/tmp/go-link-905310536/000003.o: In function `_cgo_f36fc5609ae4_Cfunc_load_simple_bpf':
/tmp/go-build/github.com/containernetworking/plugins/vendor/github.com/vishvananda/netlink/_obj/cgo-gcc-prolog:55: multiple definition of `_cgo_f36fc5609ae4_Cfunc_load_simple_bpf'
/tmp/go-link-905310536/000001.o:/tmp/go-build/github.com/vishvananda/netlink/_obj/cgo-gcc-prolog:55: first defined here
collect2: error: ld returned 1 exit status

This issue only happens if I install tinynet via go get.

And how to avoid this error?

I try to download source from git and use govendor sync to install dependency packages and it works now.

@hwchiu thanks. I should add a dependencies installation in README.
@07skat Use govendor and govendor sync to install dependencies.


Uncaught Error: Null object reference in the character editor

Describe your bug here.

Can someone please help me fix this?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I only added xml, png and json files for characters

It's not aa bug, must've been a mistype of the character file path or a corrupted file(most of the time xml)

Oh well okay... I used the FNF xml generator program that everyone suggests in tutorials etc.


Update dependencies, go-gh to v2

Hey!
I have a fairly chunky PR on the way that does this update.. along with a bunch of other cleanup tasks.
I'll add you as a reviewer if you are interested?
Cheers!


Use haptic feedback when favoriting

Overview
Like last year, it might be better to have haptic feedback when you tap the favorites button.

https://github.com/DroidKaigi/conference-app-2023/issues/1056

✋


Document naming convention

What this PR does / why we need it:
Proposal to document naming guidelines for developers extending Arki
Provides:

Consistency
Simplifies "naming things"

Special notes for your reviewer:
If applicable:

[X] this PR contains documentation
[ ] this PR contains unit tests
[ ] added code adheres to standard Rust formatting (cargo fmt)
[ ] code builds properly (cargo build)
[ ] code is free of common mistakes (cargo clippy)
[ ] all Akri tests succeed (cargo test)
[ ] inline documentation builds (cargo doc)
[ ] version has been updated appropriately (./version.sh)

Thanks for the feedback @kate-goldenring !
@bfjelds proposed adding these to ./docs/development.md which I will do ... I was wondered whether others would like to weigh-in on these before they became definitive.
In hindsight, perhaps I should have used the HackMD site?
If I get your thumbs-up to this comment, I'll move these into ./docs/development.md now.
If not, I'll wait for more feedback.

Thanks for the feedback @kate-goldenring !
@bfjelds proposed adding these to ./docs/development.md which I will do ... I was wondered whether others would like to weigh-in on these before they became definitive.
In hindsight, perhaps I should have used the HackMD site?
If I get your thumbs-up to this comment, I'll move these into ./docs/development.md now.
If not, I'll wait for more feedback.

Thanks for the feedback @kate-goldenring !
@bfjelds proposed adding these to ./docs/development.md which I will do; I wondered whether others would like to weigh-in on these before they became definitive.
In hindsight, perhaps I should have used the HackMD site?
If I get your thumbs-up to this comment, I'll move these into ./docs/development.md now.
If not, I'll wait for more feedback.

I think it is going to take a bit of time to get in the flow of using HackMD, especially when this flow was fairly smooth!

Thanks for the feedback @kate-goldenring !
@bfjelds proposed adding these to ./docs/development.md which I will do; I wondered whether others would like to weigh-in on these before they became definitive.
In hindsight, perhaps I should have used the HackMD site?
If I get your thumbs-up to this comment, I'll move these into ./docs/development.md now.
If not, I'll wait for more feedback.

I think it is going to take a bit of time to get in the flow of using HackMD, especially when this flow was fairly smooth!


Include a profile that will connect to Visual Studio Codespaces

Description of the new feature/enhancement
When using VSCode with Visual Studio Codespaces , the integrated terminal is "connected" to the codespace and all commands execute in the Codespace environment.
I would like a profile in Terminal that connects to a Codespace. I would expect it to function similar to the Azure Cloud Shell profile.

This isn't a terrible idea - and it's a great extension idea.

If you are trying to find this, though there is no direct way of doing this currently.
ssh can be used to do the same with Github Codespaces as per

https://github.community/t/will-it-be-possible-to-create-an-ssh-tunnel-to-a-codespaces-instance/130823/8
https://docs.microsoft.com/en-us/windows/terminal/tutorials/ssh

Was there any update on this now that codespaces is starting to approach GA?


Switch to java config

Also bumping spring-security-oauth to version 2.0.7.RELEASE
Closes #20

This is more a draft. There are still todos:

I'm not sure right now how to handle the new introduced OAuth2Request
The access configuration has changed and the AccessDecisionVoters can't handle expressions like isAuthenticated(). That's why the resource-server isn't deployable at the moment org.springframework.beans.BeanInstantiationException: Failed to instantiate [javax.servlet.Filter]: Factory method 'springSecurityFilterChain' threw exception; nested exception is java.lang.IllegalArgumentException: Unsupported configuration attributes: [authenticated]

Here is a good example of what can be used to configure the access to the endpoints. I guess we need to rethink the SCOPE_DYNAMIC.

I think we need our own security expressions. Something like:
hasScope('ADMIN') or hasScope('ME') and isOwnerOfResource()

hasScope already exists and custom expressions can be added like this: http://stackoverflow.com/questions/21467823/how-to-parse-spring-security-expressions-programmatically-e-g-in-some-controll/21602246#21602246 (thanks to @dacrome for finding this)

Seems that all issues have been solved. Thanks to @dacrome for the tremendous efforts made to realize this and to @sputnik27 for the implementation of the custom SecurityExpressionHandler!


New version: CodecGuide.K-LiteCodecPack.Basic version 17.7.0

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

@microsoft-github-policy-service agree


required features of dependencies in examples

some_feature of a is not enabled in examples/example.rs file.
[[example]]
name = "example"
required-features = ["a/some_feature"]

Minimal, Reproducible Example
extension version: 0.2.432 stable
server version: 1d53075

I discussed this with @Maan2003 on zulip: https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0/topic/example.20required.20feature/near/221343767
While this is expected, as required features are not automatically enabled by cargo, it is confusing. We could either skip reporting errors when the required features of a target are not enabled, or maybe automatically enable them when rust-analyzer.cargo.allFeatures is set.

I discussed this with @Maan2003 on zulip: https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0/topic/example.20required.20feature/near/221343767
While this is expected, as required features are not automatically enabled by cargo, it is confusing. We could either skip reporting errors when the required features of a target are not enabled, or maybe automatically enable them when rust-analyzer.cargo.allFeatures is set.

Same thing applies to benchmarks as well.

By the way, why don't we have an option like enableRequiredFeatures and have it set by default for targets with required-features? This doesn't sound like a very clean solution, but at least it would resolve the problem.

Yes, this is a little strange… rust_analyzer complains about a field not existing (because of a feature-flag) and ignores required-features in my Cargo.toml… and then complains about inactive code in my lib files because of the same feature.


Fix shell detection for Powershell core

Description
This PR fixes shell detection for newer versions of powershell. The executable for Powershell Core is pwsh.exe as opposed to powershell.exe for previous versions.
Related issue(s)
#4789

@guillaumerose Can you please merge this?

Sorry I can't merge anymore. Perhaps @jeanlaurent or @gtardif can help.
If you want a fix for minikube, I think you should open the PR here: https://github.com/machine-drivers/machine

(to clarify, I don't know what minikube is and I'm only interested in docker-machine :)

If you want a fix for minikube, I think you should open the PR here: https://github.com/machine-drivers/machine

The machine-drivers is only for external drivers and for fixes not (yet) accepted into machine upstream.
 drivers/hyperv/hyperv.go      | 71 ++++++++++++++++++++++++++++++++++++++++++++---------------------------
 drivers/hyperv/powershell.go  |  4 ++--
 libmachine/mcnutils/b2d.go    |  7 ++++++-
 libmachine/provision/utils.go |  2 +-
 libmachine/ssh/keys.go        | 14 ++++++++++++++
 5 files changed, 67 insertions(+), 31 deletions(-)

It's not a place to do development, that is supposed to happen on https://github.com/docker/machine
But since this repository is not maintained anymore, we had to add some patches needed for hyperv...
The next step is to make a more proper fork, like crc has done. After that it will no longer be compatible.
@NotWearingPants : docker-machine is in "maintenance mode" and is not accepting feature patches :-(
I think support for newer Windows and newer Docker qualifies as feature, so it is basically dead/deprecated.

Boot2Docker was only really still maintained for Windows 7 and Windows 10 Home users. The former should be off Windows 7 now that it's EOL, and the latter are now supported by Docker Desktop thanks to WSL2 (as noted in the changes in this PR). Docker Toolbox, which is the main way Windows 7 and Windows 10 Home users would've been consuming this hasn't seen a release since 19.03.1 (Jul 31, 2019). Docker Machine is in a similar boat with the last release being Sep 2, 2019.


[palo-alto-network] Refactor script

Make the script more readable, mostly by:

using the Product class,
removing the use of functions when unnecessary,
a little bit of renaming and documentation.

No version update as expected, merging.


Add Punctuated::drain(range)

related to #1314
Add a drain(range) method to Punctuated that is similar to Vec::drain(range).

Could you share the use case you have for this function?

I don't have a specific use case in mind. I added it to more convenient than calling remove (https://github.com/dtolnay/syn/pull/1395) multiple times to remove a range.


feat(mat-list-option): 2-way value data binding.

Feature Description
The mat-list-option currently has an @Input selected: boolean which is bound to the checkbox' selected status.
I would like that (de)selecting can also change the boolean in my typescript like follows:
<mat-selection-list>
    <mat-list-option [(selected)]="isChecked">Check me!</mat-list-option>
</mat-selection-list>

Is my option checked? {{ isChecked }}!

Use Case
This would allow me to bind the selected status to a boolean of the model I have bound to my component, instead of accessing the selection list. Because I have references to the models, from my main component.

I really appreciate this, you guys are amazing! ❤


bump version to 0.28

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@alixdamman our release script let you make a PR on a un-rebased master? doh!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Routing Callback

In order to make the documentation menu animate correctly, I think I need to know when a new route is being navigated to. Is there a way to trigger an event in this case that can be responded to by the other nav items?
Here's what happens:

when a section is expanded, the height of the child section is calculated so that it can be animated to (CSS won't animate to 'auto' height).
when the animation is finished, I will set it back to 'auto' in order to accommodate changes inside it which change its height.
when another menu item is navigated to, that may cause the current section to collapse, which means the height needs to be set back to a number, instead of auto but I don't know how to hook up an event in that scenario.

ref #160


Add support for binding volumes to all containers

Add an agent-level argument for binding a volume into all containers. This
is handy if you have a directory on all agents that you want accessible in
every container.
Our use case at Spotify is to mount the host's /etc/ssl/certs into all
containers.

:+1:


Support stylish-haskell

fixes https://github.com/Glavin001/atom-beautify/issues/628

This is great!
Could you also add a simple test example file and run the command Window: Run package spec. See step 4 from How to add a Beautifier for a Language: https://github.com/Glavin001/atom-beautify/blob/master/docs/add-languages-and-beautifiers.md#how-to-add-a-beautifier-for-a-language
You may have to edit .travis.yml and .appyveyor.yml so the tests pass when run on CI.

@Glavin001 I've added a test case. However I think that the only real way to install stylish-haskell is via Haskell source, so it might be more trouble than it's worth to get this working with CI :/

I've added a test case.

Thanks!

However I think that the only real way to install stylish-haskell is via Haskell source, so it might be more trouble than it's worth to get this working with CI :/

Bummer. If you can get it working on Travis CI that would be great -- I find Windows with AppVeyor to be too difficult to get working sometimes anyway, so Travis CI (Linux) is my Go-To CI for verifying Pull Requests.
If you are unable to get Travis CI working, then change the original/test.hs to original/_test.hs which will disable that test (tests with _ prefix are ignored).
After that, I can merge this as is.
Thank you for contributing!

I got travis to pass! Installing the hs dependencies adds to the travis build time but that's probably OK?

Absolutely OK. This is great! Thank you for contributing.

Published to v0.28.16

:beers:


Gala Cat Sìth 180% may be Skill Damage buffs (Not confirm)

Gala Cat Sìth 180% may be Skill Damage buffs. (Not confirm)
If 180% is buffs, Candy Couriers 40% (Skill Damage abilities) * Gala Cat Sìth 180% Skill Damage(Not 180%+40%), THX

gala cait sith is already implemented as skill damage buff.


Merge sort.java

Merge sort Algo in java

Pls open an issue first. I will assign it to you and then contribute. Also please try to do a contribution like this as a microbyte.  Also give PR description like this. Please follow a formal workflow for correct recognition.

ok thanks

I see there are no changes that have happened since then. if you have any queries or problems pls share. We can sort it out.
Cheers

I have opened a new pull request.


Add SCM ip restrictions programmatically

Is there a way to automate the "Box Check" the article mentions via powershell, az cli or ARM templates?
This part only mentions the application of rules for the main site.

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

ID: d90bbf62-44f9-30a6-d6ea-bbf776f6efc5
Version Independent ID: b3961eac-6c70-b345-f48c-e4c0ec67059c
Content: Restrict access - Azure App Service
Content Source: articles/app-service/app-service-ip-restrictions.md
Service: app-service-web
GitHub Login: @ccompy
Microsoft Alias: ccompy

@AugerC Thanks for the comment! We will check on this request and get back to you soon.

There is a setting in the config section under your app where all of the rules go.  Also there is a config item.  When set to true, your scm site uses the rules set for your app.
"scmIpSecurityRestrictionsUseMain": true
We do not have CLI/PowerShell for all of this yet.

@AugerC, Just checking in to see if you have had a chance to see the previous response. Kindly let us know if you need further assistance on this issue or is good to be closed.

Oh, i see. It's ok then, thanks a lot.
To future people, what @ccompy mentions is correct, located here:
https://docs.microsoft.com/en-us/azure/templates/microsoft.web/2018-02-01/sites
An example:
    "resources": [
        {
            "apiVersion": "2018-02-01",
            "name": "[concat(parameters('appName'), '/', parameters('slotName'))]",
            "type": "Microsoft.Web/Sites/slots",
            "location": "[resourceGroup().location]",
            "tags": {
            },
            "properties": {
                "siteConfig": {
                    "ipSecurityRestrictions":  "[variables('firewallRules')]",
                    "scmIpSecurityRestrictionsUseMain": true
                }
            }
        }
    ]


Broken link on README

The translate-html library is built on top of Argos Translate.

The broken link is "The argos-translate-files library". Sorry, I copied the wrong link.


js-mgmt-pr is failing with type errors

js-mgmt-pr is failing since June 24th with the following error -
[03:20:15] Requiring external module ts-node/register
.scripts/readme.ts(106,12): error TS2532: Object is possibly 'undefined'.
.scripts/readme.ts(106,12): error TS7053: Element implicitly has an 'any' type because expression of type '"typescript"' can't be used to index type 'string | object'.
  Property 'typescript' does not exist on type 'string | object'.

This is an example- https://dev.azure.com/azure-sdk/public/_build/results?buildId=437873&view=logs&j=ca395085-040a-526b-2ce8-bdc85f692774&t=2997d1dd-cd01-525b-c0a8-6ec662309496

@KarishmaGhiya Is this still applicable?

@ramya-rao-a I cannot see this error in latest pipeline, and I think we can close it. thanks


Add Gradle Scan

Ask the team what version should we use first:

Free but we don't own the the reports uploaded
Use the Internal one but we have many issues with it and found a way for it to communicate with Github (currently it use a VPN)

For now, there is an attempt to get local Gradle profiles in #95.


[BUG] Can't exclude some of test code when build sdist

setuptools version
setuptools==67.2.0
Python version
3.10.6
OS
Debian GNU/Linux 11 (bullseye)
Additional environment information
Package         Version
--------------- -------
build           0.10.0
packaging       23.0
pip             23.0
pyproject_hooks 1.0.0
setuptools      67.2.0
tomli           2.0.1
wheel           0.38.4

Description
When build sdist, some of test codes are included.

When built in 2022-09-29, test codes are not included.

However, I couldn't reproduce past behavior by rollbacking package versions.
I don't change any settings like pyproject.toml since last time I built.
Even I change pyproject.toml, I couldn't exclude test package.

For example, remove [tool.setuptools.packages.find] block

Wheel is no problem, test code were successfully excluded.
I couldn't identify what package and code cause this behavior change.
python setup.py sdist doesn't includes test code.
It seems undecided whether tests should be included or not:

Should sdists include docs and tests? - Packaging - Discussions on Python.org

There are no mention of specifications about whether tests should be included or not in official docs:

The Internal Structure of Python Eggs - setuptools 67.2.0.post20230207 documentation

Relates?: #3243

Expected behavior
Exclude tests directory from sdist. (original behavior)
How to Reproduce
1.
Clone this project:
yukihiko-shinoda/pyvelocity: Automates task to check configurations about Python project to follow best practices to increase development velocity.
2.
Install dependencies:
pip install setuptools==67.2.0 build==0.10.0 pyproject_hooks==1.0.0 tomli=2.0.1

3.
Build:
python -m build

4.
Check sdist:
tar tf dist/*.tar.gz

Output

Click me
pyvelocity-0.1.2/
pyvelocity-0.1.2/LICENSE
pyvelocity-0.1.2/PKG-INFO
pyvelocity-0.1.2/README.md
pyvelocity-0.1.2/pyproject.toml
pyvelocity-0.1.2/pyvelocity/
pyvelocity-0.1.2/pyvelocity/__init__.py
pyvelocity-0.1.2/pyvelocity/checks/
pyvelocity-0.1.2/pyvelocity/checks/__init__.py
pyvelocity-0.1.2/pyvelocity/checks/aggregation.py
pyvelocity-0.1.2/pyvelocity/checks/line_length.py
pyvelocity-0.1.2/pyvelocity/checks/using_py_project_toml.py
pyvelocity-0.1.2/pyvelocity/cli.py
pyvelocity-0.1.2/pyvelocity/configurations/
pyvelocity-0.1.2/pyvelocity/configurations/__init__.py
pyvelocity-0.1.2/pyvelocity/configurations/aggregation.py
pyvelocity-0.1.2/pyvelocity/configurations/files/
pyvelocity-0.1.2/pyvelocity/configurations/files/__init__.py
pyvelocity-0.1.2/pyvelocity/configurations/files/aggregation.py
pyvelocity-0.1.2/pyvelocity/configurations/files/py_project_toml.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/__init__.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/black.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/docformatter.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/factory.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/flake8.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/isort.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/pylint/
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/pylint/__init__.py
pyvelocity-0.1.2/pyvelocity/configurations/files/sections/pyvelocity.py
pyvelocity-0.1.2/pyvelocity/configurations/files/setup_cfg.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/
pyvelocity-0.1.2/pyvelocity/configurations/tools/__init__.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/black.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/docformatter.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/flake8.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/isort.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/pylint.py
pyvelocity-0.1.2/pyvelocity/configurations/tools/pyvelocity.py
pyvelocity-0.1.2/pyvelocity/py.typed
pyvelocity-0.1.2/pyvelocity/pyvelocity.py
pyvelocity-0.1.2/pyvelocity.egg-info/
pyvelocity-0.1.2/pyvelocity.egg-info/PKG-INFO
pyvelocity-0.1.2/pyvelocity.egg-info/SOURCES.txt
pyvelocity-0.1.2/pyvelocity.egg-info/dependency_links.txt
pyvelocity-0.1.2/pyvelocity.egg-info/entry_points.txt
pyvelocity-0.1.2/pyvelocity.egg-info/not-zip-safe
pyvelocity-0.1.2/pyvelocity.egg-info/requires.txt
pyvelocity-0.1.2/pyvelocity.egg-info/top_level.txt
pyvelocity-0.1.2/setup.cfg
pyvelocity-0.1.2/setup.py
pyvelocity-0.1.2/tests/
pyvelocity-0.1.2/tests/test_cli.py

This seams to depends exported SOURCES.txt:

Click me
LICENSE
README.md
pyproject.toml
setup.cfg
setup.py
pyvelocity/__init__.py
pyvelocity/cli.py
pyvelocity/py.typed
pyvelocity/pyvelocity.py
pyvelocity.egg-info/PKG-INFO
pyvelocity.egg-info/SOURCES.txt
pyvelocity.egg-info/dependency_links.txt
pyvelocity.egg-info/entry_points.txt
pyvelocity.egg-info/not-zip-safe
pyvelocity.egg-info/requires.txt
pyvelocity.egg-info/top_level.txt
pyvelocity/checks/__init__.py
pyvelocity/checks/aggregation.py
pyvelocity/checks/line_length.py
pyvelocity/checks/using_py_project_toml.py
pyvelocity/configurations/__init__.py
pyvelocity/configurations/aggregation.py
pyvelocity/configurations/files/__init__.py
pyvelocity/configurations/files/aggregation.py
pyvelocity/configurations/files/py_project_toml.py
pyvelocity/configurations/files/setup_cfg.py
pyvelocity/configurations/files/sections/__init__.py
pyvelocity/configurations/files/sections/black.py
pyvelocity/configurations/files/sections/docformatter.py
pyvelocity/configurations/files/sections/factory.py
pyvelocity/configurations/files/sections/flake8.py
pyvelocity/configurations/files/sections/isort.py
pyvelocity/configurations/files/sections/pyvelocity.py
pyvelocity/configurations/files/sections/pylint/__init__.py
pyvelocity/configurations/tools/__init__.py
pyvelocity/configurations/tools/black.py
pyvelocity/configurations/tools/docformatter.py
pyvelocity/configurations/tools/flake8.py
pyvelocity/configurations/tools/isort.py
pyvelocity/configurations/tools/pylint.py
pyvelocity/configurations/tools/pyvelocity.py
tests/test_cli.py

However, the cache of SOURCES.txt when built in 2022-09-29 is:

Click me
LICENSE
README.md
pyproject.toml
setup.cfg
setup.py
pyvelocity/__init__.py
pyvelocity/cli.py
pyvelocity/py.typed
pyvelocity/pyvelocity.py
pyvelocity.egg-info/PKG-INFO
pyvelocity.egg-info/SOURCES.txt
pyvelocity.egg-info/dependency_links.txt
pyvelocity.egg-info/entry_points.txt
pyvelocity.egg-info/not-zip-safe
pyvelocity.egg-info/requires.txt
pyvelocity.egg-info/top_level.txt
pyvelocity/checks/__init__.py
pyvelocity/checks/aggregation.py
pyvelocity/checks/line_length.py
pyvelocity/checks/using_py_project_toml.py
pyvelocity/configurations/__init__.py
pyvelocity/configurations/aggregation.py
pyvelocity/configurations/files/__init__.py
pyvelocity/configurations/files/aggregation.py
pyvelocity/configurations/files/py_project_toml.py
pyvelocity/configurations/files/setup_cfg.py
pyvelocity/configurations/files/sections/__init__.py
pyvelocity/configurations/files/sections/black.py
pyvelocity/configurations/files/sections/docformatter.py
pyvelocity/configurations/files/sections/factory.py
pyvelocity/configurations/files/sections/flake8.py
pyvelocity/configurations/files/sections/isort.py
pyvelocity/configurations/files/sections/pyvelocity.py
pyvelocity/configurations/files/sections/pylint/__init__.py
pyvelocity/configurations/tools/__init__.py
pyvelocity/configurations/tools/black.py
pyvelocity/configurations/tools/docformatter.py
pyvelocity/configurations/tools/flake8.py
pyvelocity/configurations/tools/isort.py
pyvelocity/configurations/tools/pylint.py
pyvelocity/configurations/tools/pyvelocity.py

This is not range of setuptools but distutils.
see: https://github.com/pypa/distutils/issues/187
The current specification seams to be that:

[tool.setuptools.packages.find] block in pyproject.toml effects only for bdist
The sdist includes some of test files however it's not completed


CentOS 8 package

Are you interested in packaging for CentOS 8? I'd happily share the spec file I used, but it's not ready for prime time yet.

Yes please share it with us in a Pull Request

I will close this issue. Many thanks for the contribution!


api:routes not work when I use Illuminate\Foundation\Http\FormRequest

api:routes command will cause a Fatal error:Call to a member function make() on a non-object,
this is the code:
app/Http/Requests/Request.php
namespace App\Http\Requests;
use Illuminate\Foundation\Http\FormRequest;
abstract class Request extends FormRequest{
}

app/Http/Requests/TestRequest.php
<?php
namespace App\Http\Requests;
use App\Http\Requests\Request;
class TestRequest extends Request
{
    //Determine if the user is authorized to make this request.
    protected $rules = [
        'GET'   =>  [],
        'DELETE'=>  [],
        'POST'  =>  [
            'title'     =>  ['required', 'unique:articles', 'between:1,63'],
            'content'   =>  ['required', 'max:65533'],
        ],
        'PUT'   =>  [
            'title'     =>  ['between:1,63'],
            'content'   =>  ['max:65533'],
        ],
        'PATCH' =>  [
            'title'     =>  ['between:1,63'],
            'content'   =>  ['max:65533'],
        ],
    ];

    public function authorize()
    {
        return true;
    }

    //Get the validation rules that apply to the request.
    public function rules()
    {
        return $this->rules[$this->getMethod()];
    }
}

and I imported TestRequest in TestController
<?php
namespace App\Http\Controllers;
use App\Http\Requests\TestRequest;
use App\Http\Controllers\Controller;
class TestController extends Controller
{
    protected $request;
    public function __construct(TestRequest $request) {
        $this->request = $request;
    }

    public function index()
    {
       echo 'everything is fine';
    }

the problem is : run php artisan api:routes in terminal.you will see this error:

How to solve this problem?

that is not dingo problem,is the laravel problem


Pycytominer v1.2.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Opening after a premature close on #9 .

After setting Python constraints to address the original errors I'm finding there's an additional error from pip check:  backports.tempfile 1.0 is not supported on this platform. backports.tempfile is a dependency of Pycytominer via cytominer-database. I believe the error may be stemming from the build platform (OS and architecture), Python version (3.10.15), and possibly other constraints. The last successful build used Python 3.10.13 (see here).

Just remove it as a dependecy from https://github.com/conda-forge/cytominer_database-feedstock/blob/main/recipe/meta.yaml
Python 3.4 is way out of date, but even that should support TemporaryDirectory natively.
https://pypi.org/project/backports.tempfile/

There are other hacks you can do, but generally they aren't worth it for Python 3.3. compatibility.
This seemed easy: https://github.com/conda-forge/dataclasses-feedstock/pull/3
Then the churn: https://github.com/conda-forge/dataclasses-feedstock/pull/9
continues: https://github.com/conda-forge/dataclasses-feedstock/pull/14

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Thanks @gwaybio for the review! Also, thank you @hmaarrfk for the continued guidance throughout this PR and the other related ones along the journey, we really appreciate it! Merging this in now.


support multiple git-sync repos (for dags & plugins)

Currently we only support a single git repo with dags.gitSync, we should consider extending this to allow for multiple dags git repos, if this can be done with minimal impact to the values structure.

I thought about this a bit and would propose something along the lines of:
dags.gitSync.repolist:
- repo: ""
  repoSubPath: ""
  branch: ""
  revision: ""
  depth: 1
  ...

Currently I am considering adding default options like secretKey, branch etc. so that you don't have to define them in each repo entry but would still be able to overwrite it foreach repo.
Sadly the git-sync sidecar container is specifically designed to clone only one repo. It was kinda discussed in here: https://github.com/kubernetes/git-sync/issues/261
We could always just range over the repolist and add an additional sidecar foreach repo. Altough this would probably lead to a lot of sidecars in bigger environments.
Would you rather have multiple sidecars or add a custom git-sync mechanism?

I've been thinking about an alternative, which is some kind of python library which would be used as the only main.py file in your gitSync repo (which itself is configured by some kind of config.yaml in the gitSync repo).
The purpose of this library would be to target specific tags/revisions of other repos, to allow a more git-ops approach to airflow in general, specifically enabling one git-repo per dag. (While letting you pin version tags from these repos from the config.yaml, so you can change-manage your dags)
The library would handle being called at each AIRFLOW__SCHEDULER__DAG_DIR_LIST_INTERVAL, and sync any updates from the remote repos if their targeted tags/branches have new/changed commits.
NOTE: it might not even need to be a library, and might be fine as a basic script

So you would basically still have only one gitsync repo defined in k8s but the given repo would add the possibility of a config.yaml or wathever else.
This config.yaml then includes other repos with their tag/rev.
I like the idea of managing this outside of the helm chart that I don't have to update my k8s ressources for a new dag / project repo. So hard agree on that one.
I think then it would be needed to use something like the old setup with a basic alpine linux and script mounted by cm. I kind of dislike the idea of a main.py within my gitsync repo since it introduces additional complexity in handling this mechanism. Adding just the possibility of a config.yaml with a given list of repos should do the work. (Altough this means that the script that handles the config.yaml will be more complex)
Obviously it should still be possible to just use the git-sync repo as an "all-in-one" dag bucket.
The config.yaml would then be picked up by a basic script and include the "sub-repos" within airflow_home/dags/.
Giving the user the ability to define a main.py sounds more like a 3rd option of this mechanism. In that way a "power-user" can do whatever he pleases but less experienced users can default to the "all-in-one" repo or to a config.yaml with a given structure.

Just curious, if you were to go with @yehlo's original repoList proposal with a range of sidecars over repos in the list, how would you configure airflow to read the dags from each repoSubPath? DagBags?

Bumping for the bot, I will work on this after the next minor release (which should add PGBouncer).

For those watching, it is already possible to use multiple git repos. Git has a feature called .gitmodules which allows you to "recursively" reference git repositories from a single repo.
There is an issue to document this here: https://github.com/airflow-helm/charts/issues/434

Given this, I am not sure that we should support multiple git repos directly in the chart, as the .gitmodules approach is better for a few reasons:

It allows you to define (in git) which versions of other repos are synced, meaning you don't have to update your helm values to change dag versions.
It reduces the overhead by not requiring a sidecar container per repo (per deployment)

Reopening

Just curious, if you were to go with @yehlo's original repoList proposal with a range of sidecars over repos in the list, how would you configure airflow to read the dags from each repoSubPath? DagBags?

@gillbuchanan The trick for getting multiple git-repos to work, is to use symbolic links that are auto-created as the main airflow container starts.

There would be an emptyDir volume for each git-repo:

populated by its own init-container
kept updated by its own sidecar container

These emptyDir volumes would be mounted at /mnt/git-sync/my-first-repo/, /mnt/git-sync/my-second-repo/

NOTE: the kubernetes/git-sync container creates its own symbolic links, so our symbolic links need to point to the repo symbolic link created under /mnt/git-sync/my-first-repo

As the main airflow container starts, it would create symbolic links under /opt/airflow/dags that point to /mnt/git-sync/my-first-repo/repo and /mnt/git-sync/my-second-repo/repo.

Would it make sense to just git submodules?
https://git-scm.com/book/en/v2/Git-Tools-Submodules
this way there is still one git repository that contains the DAGs and additional code from other repos can be fetched by setting GIT_SYNC_SUBMODULES to "recursive" (this is the default value anyway).
This would be a solution with minimal effort and probably according to what the developers of git-sync had in mind.

@juliandm previously, I had the idea to use submodules and talked about it in https://github.com/airflow-helm/charts/issues/166#issuecomment-929671884, but I think it has some serious limitations for this use-case. Namely that submodules are not well understood by most users, and things like authentication will become problems.

Using the method described in https://github.com/airflow-helm/charts/issues/166#issuecomment-993123687, I think the best approach is to allow a list of repos to be used in dags.gitSync, with the two remaining issues being:

Not requiring a git-sync sidecar and git-clone init-container for each repo:

Otherwise, the number of containers will exponentially increase with the number of repos.
Instead, we should make a single container that is responsible to sync all the repos.
Hopefully, someone has made something like the kubernetes/git-sync container which works for multiple repos

How to arrange the dags.gitSync values:

We can probably create a new section called dags.gitSyncMultiand use mappings for things like SSL configs.

@thesuperzapper Is there any progress on this? Could we have a workaround to pass Env var for the git branch so we can have separate repos per deployment?

@thesuperzapper Is there any progress on this? Could we have a workaround to pass Env var for the git branch so we can have separate repos per deployment?

@hkvia what do you mean "separate repos per deployment", all Airflow workers need to have the same dags?

@thesuperzapper ignore that last bit. Can we use an Env var for the branch name?

Just wanted to note that git-sync v3.4.0+ supports having GIT_SYNC_DEST not under GIT_SYNC_ROOT.
This makes the solution slightly cleaner than what was proposed in https://github.com/airflow-helm/charts/issues/166#issuecomment-993123687, as we now don't have to modify the startup script of the container to create the symbolic links (as git-sync will create them for us if we specify GIT_SYNC_DEST = /opt/airflow/dags/REPO_1/).

@thesuperzapper have you accomplished what you have been describing in this thread? I am trying to make this work :

initContainers for different repos are ok, created with extra emptyDirs
sideContainers also work and keep the repos in sync
I have to manually edit the statefulSet of the worker, though, to create the symbolic links.

For this 3rd asterisk, is there something I am missing? Can it be done in a more streamlined way? The only thing I have though of so far is to support in the helm chart editing the command of the airflow-worker container - would you be open to that?

any update?

I was able to solve this by creating a git repository(let's say A) which has git submodules(B,C,D), and setup a trigger in git-submodules' CI/CD (on new commit) to trigger the job in git repository A. The CI/CD job in git repo A would be something like below :
  git submodule sync --recursive
  git submodule update --recursive --remote
  git add .
  CHANGES=$(git status --porcelain | wc -l)
    if [[ "${CHANGES}" -gt 0 ]]; then
      git config user.email<EMAIL_ADDRESS>      git config user.name "test"
      git commit -m "Submodule Sync"
      git push origin HEAD:main
    else
      echo "Nothing to commit"
      fi

And git-sync is syncing to git repo A.

bitnami's airflow helm chart can support this feature

@snowsky can you share the place where this is supported? probably we can create PR against official airflow helm chart

https://github.com/bitnami/charts/blob/main/bitnami/airflow/values.yaml#L956
Bitnami Chart can support multiple repos ^^^ :)

any update of this feature?

Any updates?

Guys one workarounds?

Still waiting...


⚠ Bofrost.lu has degraded performance

In 94204cd, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1815 ms

Resolved: Bofrost.lu performance has improved in 4416ffc after 13 minutes.


[YUNIKORN-422] Automated install of OpenShift 4.5 with Yunikorn scheduler for BigData and ML on your laptop, virtual machine or baremetal server using CodeReady Containers

Here are my steps which uses Marc's steps. After you've set up a local CRC cluster you can use these steps to test any patches in YuniKorn locally.

Uploaded a new patch with the changes. Please review this.


chore(deps): upgrade to GraphQL v15.0.0

Upgrades the GraphQL dependencies to v15, includes the alpha for codemirror-graphql.

looking good! can we replace this constant with the getIntrospectionQuery() result? it's a simple change that reduces code and makes us more graphql 15 ready

re-recreating here #1199


Added my okalu.txt file

Hello

@okalu Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Please don't add pull requests tat don't add anything to the root repository. Please remove this and work on your own branch.


Pack command produces not correct lib folder name on Mac OSX

I work on mac in Parallels VM. Packed in this environment nupkg, has invalid lib folder structure. Looks like bug in file path encoding.

Seems copy of #963


Install libsndfile 1.2.2 from source to support 0.12 OPUS format

Install libsndfile 1.2.2 from source in Docker image.
Fix #2584.

Thanks for the feedback: honestly, I took this issue without having a clear idea what to do... :sweat_smile:

@severo, while trying to implement a test, I discovered that I am not able to reproduce the issue with libsndfile-1.0.31 (our minimum required version).
I have checked our CI, and our tests also use libsndfile-1.0.31. See: https://github.com/huggingface/dataset-viewer/actions/runs/9079619818/job/24949210823

Setting up libsndfile1:amd64 (1.0.31-2ubuntu0.1) ...

Do you know why our prod has libsndfile-1.2.0?

Maybe I could try to pin the version of libsndfile1 in all the Dockerfiles, e.g. https://github.com/huggingface/dataset-viewer/blob/af80a2486ffed3090ec08bd4f27f938848bec942/services/worker/Dockerfile#L19
What do you think?

Do you mean that your local system package is libsndfile-1.0.31, not libsndfile-1.2.0?

Yes, I think we first should confirm which sets of system + pip packages work wit opus, and which ones don't. Then ensure we use this set in our CI and prod (+ local setup for development)

Yes, of course, @severo
What I wanted to pint out is that currently I are not be able to test in our CI what will happen in our prod, because the libsndfile1 versions are misaligned:

CI uses 1.0.31
prod uses 1.2.0

We should first align both. Do you agree?

Ah OK. It's because the tests in the CI don't use the docker file. The system (ubuntu, apt) is set in the GitHub action. Yes, we should align, you're right!

Superseded by:

#2811


KAFKA-7650: make auto.create.topics.enable dynamically configurable

KAFKA-7650: make auto.create.topics.enable dynamically configurable
tested auto.create.topics.enable can be dynamically configured after change.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

About code changes, Are we stopping the client in other scenario like this,
when auto.create.topics.enable=false at broker side especially in multi-tenant cluster, but client make the configuration to true and start creating as many topics as they want. That should not happen.

About code changes, Are we stopping the client in other scenario like this,
when auto.create.topics.enable=false at broker side especially in multi-tenant cluster, but client make the configuration to true and start creating as many topics as they want. That should not happen.

If either client or broker don't allow auto topic creation,  the auto topic creation will be disabled.
Setting this at broker side, it is sufficient to prevent auto topic creation.   This configuration is only for auto topic creation, it doesn't affect the topic creation via create topic API.

@mrsrinivas I don't know if it requires a KIP, since it is a very small change.  It only makes existing configuration dynamic configurable.


⚠ Bofrost.at has degraded performance

In 9a16578, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1134 ms

Resolved: Bofrost.at performance has improved in 56558a4 after 20 minutes.


Add validation to search field

Hi @surajondev
Have made the changes to display validation toaster message when invalid input is entered.
Can you please review the PR
Issue Link: https://github.com/surajondev/get-weather/issues/26#issue-1400817421
Screencast for changes https://www.loom.com/share/3a88d563b858414cb421dd94226f3153

Thanks for the PR. The website looks more complete with the toastify.
I will be glad if you can leave a star on the repository.

sure


AnnouncementCollatorFactory should use native fetch

The AnnouncementCollatorFactory was moved to use the FetchApi Backstage provides, but this is a front-end concept, and should not be used here.
It's causing a headache for end users and should be switched to using native fetch, or node-fetch as last resort.
Discussed in https://github.com/procore-oss/backstage-plugin-announcements/discussions/61

Originally posted by gaelgoth October 23, 2023
Hello,
We recently used your (archived) plugin fork. Since I would like to migrate to the procore-oss organization version, you have introduced the usage of the Backstage API, as referenced in this commit. However, I have not been able to pass fetchApi in my search.ts. Do you have an example of how to do it properly? I would be happy to contribute and help by updating the documentation 😄
...
import { AnnouncementCollatorFactory } from '@procore-oss/backstage-plugin-announcements-backend';
import { PgSearchEngine } from '@backstage/plugin-search-backend-module-pg';
import { Router } from 'express';

export default async function createPlugin(env: PluginEnvironment): Promise<Router> {
  const searchEngine = await PgSearchEngine.fromConfig(env.config, { database: env.database });
  const indexBuilder = new IndexBuilder({
    logger: env.logger,
    searchEngine,
  });

...

  // Announcements indexing
  indexBuilder.addCollator({
    schedule: schedule,
    factory: AnnouncementCollatorFactory.fromConfig({
      logger: env.logger,
      discoveryApi: env.discovery,
      fetchApi: env.fetchApi, // FIXME: Correctly pass fetchApi
    }),
  });

  const { scheduler } = await indexBuilder.build();
  scheduler.start();

  useHotCleanup(module, () => scheduler.stop());

  return await createRouter({
    engine: indexBuilder.getSearchEngine(),
    types: indexBuilder.getDocumentTypes(),
    permissions: env.permissions,
    config: env.config,
    logger: env.logger,
  });
}

```</div>

Resolved in https://github.com/procore-oss/backstage-plugin-announcements/pull/70


Fix typo that leads to compilation error

Thanks! It appears on my machine SDL2 has an enum called SDL_KeyCode which is how it managed to compile and slip through the cracks but that wasn't my intention. Good catch!

Thanks! It appears on my machine SDL2 has an enum called SDL_KeyCode which is how it managed to compile and slip through the cracks but that wasn't my intention. Good catch!


Installation broken

Component
Bifrost, Heimdall (CLI)
Have you ensured that you are up to date?

[X] Bifrost
[X] Heimdall

What version of Heimdall are you on?
7.3
Operating System
Linux
Describe the bug
i ran
bifrost -B

with heimdall 7.3 installed, but when i ran heimdall in my command line i got:
$dev5# heimdall
heimdall: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by heimdall)
heimdall: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by heimdall)
heimdall: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by heimdall)

/dev5# ldd --version
ldd (Debian GLIBC 2.31-13+deb11u6) 2.31
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.

Hey! Thank you for raising this! It looks like there isn't a precompiled binary for your platform. For now, you should run bifrost instead to compile the binary on your platform.
I'll look into adding support for your plat today or tomorrow!

resolving next

does installing the requested  GLIBC versions help?

closing as stale/can't reproduce


Typo: codecept.js ➡ json

The configuration is in a json file not in a js file.

Indeed. Thank you


pass equal_labels configuration to worker

Signed-off-by: Robert Cerven<EMAIL_ADDRESS>

Needs corresponding changes to test_render_orchestrate_build in test_build_request.py.


up-to-date indicator: don't use filtered entries

Don't use the filtered entries to get account activity indicators.

Works, thanks!


North setting does not work (syntax ?)

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue:
all
Last working release (if known):
none
Browser and Operating System:
Firefox/Chrome/Edge Windows

Description of problem:
The "North" parameters does not seem to adjust anything. What is the correct syntax for this parameter ?
even if I enter the default value of :   {x: 0, z: 1}  it does not work (shading is gone then).
With empty parameter value, shading is there, but with wrong North position.
I want to change this but I don't get any working North parameter values working.
Give some working examples please.
Further the editor is very slow and buggy.
"Sky" & "Shadow" & "show_axes" parameters are set to "yes"

Javascript errors shown in the web inspector (if applicable):

Additional information:

Hello @8OND007, try to use the code editor and enter the north this way. I do not know why but the object format is not working correctly:
north:
  x: 0
  z: 1

Related to the editor have you seen the performances going down with the last version (v1.3.11) ?
You have to wait for the editor to load the card in the beginning, this is why it is taking a little more time, but during the editing the performances are quite acceptable (in my config)....

Thanks for feedback @8OND007, the secret is a good GPU (this is fundamental for 3d rendering). Unfortunately this is against the HA principles. All HA software (add-ins, custom integraions and cards) should be working on any HW. I was thinking to have the 3d render built as a separate UI communicating to HA via API; I do not want to be reponsible for performance issue of the HA plarform.
I need to find a way to edit the card with or without preview in order to make the editing flawless

@8OND007 can you please close this issue if you consider that it is fixed. I have updated the readme


sample_lanelet_viz error

Hi,
I can't get sample_lanelet_viz.launch to load due to parsing errors of the provided example map. The exact error message is:

Error during map loading with lanelet2_interface_ros: Errors ocurred while parsing Lanelet Map: - Error parsing primitive 45034: Failed to generate boundary (self-intersecting?) - Error parsing primitive 45034: Areas must have exactly one outer ring! - Error parsing primitive 45042: Failed to generate boundary (self-intersecting?) - Error parsing primitive 45042: Areas must have exactly one outer ring! [...]

Is the provided map outdated ?

Hi,
sounds like the xml parser doesn't understand the map. Do you have , as decimal separator? It should be ..
You can check your settings with
$ locale

it should be
LC_NUMERIC="en_US.UTF-8"

Please let me know whether changing your locale settings fixes the problem.

Sorry for the late reply,
changing my locale settings fixed the problem. I can now see 3 intersections.
Thanks a lot.