Why I cannot see "Users and Groups" in the app I owned?

Hi,
I am attempting to restrict some of the users in my organization to access my app on Azure AD, with the following guide:
https://docs.microsoft.com/en-us/azure/active-directory/develop/howto-restrict-your-app-to-a-set-of-users
However I cannot view the "Users and Groups" options under "Manage" as shown below. Could you please help?
https://htkk-my.sharepoint.com/:i:/g/personal/michaelwingyin_cheng_healthcare-tech_co_jp/EVk1gU1H--1LrxZ9ljXSRyQB5LqM6HAb-EUyLqQj24Hqxg?e=V7dtri
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: be62a693-4dfd-ad56-cce2-b4dbc794f34b
Version Independent ID: bd8bc1b2-925f-a0c3-4286-37eb2f7189f5
Content: Restrict Azure AD app to a set of users - Microsoft identity platform
Content Source: articles/active-directory/develop/howto-restrict-your-app-to-a-set-of-users.md
Service: active-directory
Sub-service: develop
GitHub Login: @kalyankrishna1
Microsoft Alias: kkrishna

@wy10cheng Thank you for your feedback . We will investigate and update the thread.

@shashishailaj kindly please advise if i did wrongly?

@wy10cheng , Thanks for reaching out.
You would find  "Users and Groups" options under "Manage" from Enterprise Application section as shown below screenshot.
If you sign in to the Azure portal as a Global Administrator, then Azure Active Directory > Enterprise Applications (Select the application you want to assign a user) > All applications > Users and groups > Add user/group
If you sign in to the Azure portal as an Owner of application, then Azure Active Directory > App registrations > select the Managed application in local directory link in the Essentials section (On the application's Overview page) > Users and groups > Add user/group .
Hope this helps. Feel free to reach out to me, if you have any additional queries. Thanks.


Lack of explanation

I think this project needs a little explanation on how to use the program....
How do i input my Shifts and employees?

Execute the main.py script with the data file specified as a command line parameter.
python main.py data.json
You can find an example of the data format in https://github.com/Markcanfly/shifts/blob/c1f92c3374639e68b16c37b09438d038338c1f6c/data.py#L92
Currently this is a specific solution to certain problem, and right now I unfortunately have no time to make it generally useful. Thanks for the reminder though! I have added the MIT license, and will archive the repository.


[5.4] Add columnize() method to Collection

We have this setup as a Collection macro in the majority of our projects so thought I'd see if it would be useful to add to the core.
This differs from the chunk() method given that you have to pass the number of items you want in each "column", whereas, more often than not we don't care about the number of items in each column. We want to specify the number of columns, and the items are split equally over those columns.
Say you have 18 items and you want to output 4 columns.

$collection = collect([1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18]);

dd($collection->columnize(4)->toArray());

/*
array:4 [
  0 => array:5 [
    0 => 1
    1 => 2
    2 => 3
    3 => 4
    4 => 5
  ]
  1 => array:5 [
    5 => 6
    6 => 7
    7 => 8
    8 => 9
    9 => 10
  ]
  2 => array:4 [
    10 => 11
    11 => 12
    12 => 13
    13 => 14
  ]
  3 => array:4 [
    14 => 15
    15 => 16
    16 => 17
    17 => 18
  ]
]
*/

How is this different than the existing split method?
$groups = $collection->split(4);

Ah, cool. We are still at 5.2 which doesn't have split()

@JosephSilber ...in fact, split() is quite different from columnize().
split() puts the remainder of the items in the last column whereas columnize() spreads the remainder across all the columns which keeps the columns as even as possible.
Compare these two example
split()
dump(collect([1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13])->split(4)->toArray());
/*
Output:
array:4 [▼
  0 => array:4 [▶]
  1 => array:4 [▶]
  2 => array:4 [▶]
  3 => array:1 [▶]
]
*/

columnize()
dump(collect([1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13])->columnize(4)->toArray());
/*
Output:
array:4 [▼
  0 => array:4 [▶]
  1 => array:3 [▶]
  2 => array:3 [▶]
  3 => array:3 [▶]
]
*/

I don't think I will add this one in core. Prefer to keep as a macro for now.


Catch all runtime exceptions

All RuntimeExceptions should be caught and logged, so we can always tell the user if something went wrong.
Also, link to the logs has to be added in the index start comment, so the user can follow them (right now the logs are only shown in case of error)

done


Geolocation.clearWatch() throws an error

We have Opbeat set up on our project, which for now is not perfect for JavaScript, but still gives us some information about errors.
The new errors we were alarmed were caused by Geolocation.clearWatch() from f.Map.include.stopLocate method.
We have not much information tracked down, but the user agent was:
Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.13 Safari/537.36
The TypeError value was:
Object [object Object] has no method 'clearWatch'
Do you have any idea why this suppose to happen? Is there any solution to this?
FYI, v0.7.3 is used.

Are you able to test with latest master?

Not really. I can't put beta release on the live server, where it occurred, also, I cannot reproduce the error myself.
This came from some user, and Opbeat logged the thrown exception.
I thought maybe you might know, if there's problem with clearWatch in some cases. From the code I see, then it checks if navigator.geolocation exists, and the calls the method. But in this case error indicated, that there was no method clearWatch.

The user agent looks like it might be the new Microsoft Edge browser which tries to mimic itself as all other browsers. :) No idea why there's no clearWatch method, but I'm sure this is not a Leaflet issue, but a browser one and not much can be done on our side. I'd suggest doing a try/catch block to catch errors like that.

What about checking if the method exist before executing it? If, as in this in example, there might be cases where the method does not exist.

@mfc-julius just added a check for that. Thanks!

FYI, this is happening with latest Chrome on Windows 10. Again, not sure why.


[Controller Interruption] No action log from controller.interruption with valid SQS message from eventbridge

Description
Observed Behavior:
I’m checking the controller.interruption handling sqs queue when receiving queue information.

Test case 1

With malform information: “abcd” content in SQS message queue. The controller.interruption log show handling:
2023-10-09T03:23:22.419Z	ERROR	controller.interruption	parsing message, parsing sqs message, unmarshalling the message as Metadata, invalid character 'a' looking for beginning of value	{"commit": "34d50bf-dirty", "queue": "karpenter-venture-staging-queue-interruption"}

Test case 2

Checked the queue karpenter-venture-staging-queue-interruption with so many messages received from eventbridge (event: spot interruption warning, instance state change notification). I checked the log for controller.interruption action, but I don't see any debug/info log related to the messages received or interruption handling.
Expected Behavior:
The controller.interruption should have some info or debug log related to SQS interruption queue handling for the correct SQS queue message
Reproduction Steps (Please include YAML):

Send a valid message from eventbridge to SQS queue for SQS Spot Interruption Event

{
  "version": "0",
  "id": "1e5527d7-bb36-4607-3370-4164db56a40e",
  "detail-type": "EC2 Spot Instance Interruption Warning",
  "source": "aws.ec2",
  "account": "my-account",
  "time": "2023-10-09T03:20:00Z",
  "region": "ap-southeast-2",
  "resources": ["arn:aws:ec2:ap-southeast-2a:instance/i-0be439f993306axyz"],
  "detail": {
    "instance-id": " i-0be439f993306axyz",
    "instance-action": "terminate"
  }
}

Expect received any log for interruption SQS message handling but nothing happen

Versions:

Image Version: public.ecr.aws/karpenter/controller:v0.29.2
Kubernetes Version (kubectl version): 1.26

@huyngo81 Can you recheck in your karpenter-global-settings and make sure you have set the aws.interruptionQueueName value?

hi @ivan-amity I configured with correct queue and the controller.interruption is also show log when having invalid SQS message. Please see the global-configmap as below:
k get configmap karpenter-global-settings -n karpenter -o yaml | k neat apiVersion: v1 data:   aws.clusterEndpoint: ""   aws.clusterName: sc-cluster-venture-staging   aws.defaultInstanceProfile: karpenter-node-role-venture-staging   aws.enableENILimitedPodDensity: "true"   aws.enablePodENI: "false"   aws.interruptionQueueName: karpenter-venture-staging-queue-interruption   aws.isolatedVPC: "false"   aws.vmMemoryOverheadPercent: "0.075"   batchIdleDuration: 1s   batchMaxDuration: 10s   featureGates.driftEnabled: "false" kind: ConfigMap metadata:   labels:     app.kubernetes.io/instance: karpenter     app.kubernetes.io/managed-by: Helm     app.kubernetes.io/name: karpenter     app.kubernetes.io/version: 0.29.2     helm.sh/chart: karpenter-v0.29.2   name: karpenter-global-settings   namespace: karpenter
I see this log for the negative test case with SQS message content: "abcd" in SQS Queue:
2023-10-09T03:23:22.419Z ERROR controller.interruption parsing message, parsing sqs message, unmarshalling the message as Metadata, invalid character 'a' looking for beginning of value {"commit": "34d50bf-dirty", "queue": "karpenter-venture-staging-queue-interruption"}

have some info or debug log related to SQS interruption queue handling

We're a little weary of reacting to every message that enters the SQS queue because of the amount of log spam that that could generate. Currently, we only generate metrics when we handle messages as well as DEBUG logging message if the message that we received is actionable.
Are you wanting us to be more verbose when we receive messages from the queue?

Yes @jonathan-innis , just want make sure the controller.interrup had proper action to replace the spot worker-nodes immediate when they received the event Spot Interruption Warning. I'm seeing many messages (Instante-state-change-notification and Spot-Instance-Interrup-Warning received in SQS queue which was sent from EventBrige rules/target but no logs from controller.interruption. is that message correct? Why was it ignored without logs?
2023-10-09T03:23:22.419Z INFO  controller.interruption parsing message "... Spot Interrup Warning instance: i-xyznanxacda: action terminated" valid
2023-10-09T03:23:22.419Z INFO controller.interruption inititalizing the replacing instance: i-xyznanxacda)
2023-10-09T03:23:22.419Z INFO  controller.interruption trigger controller.provisioning initialize new workers ...

Or, If the message in SQS queue is not correct
2023-10-09T03:23:22.419Z INFO controller.interruption : ignored message "abcd xyz" in queue due to invalid timestamp

hi all, I can test the interruption by using AWS FIS. The log as below:
2023-10-10T05:24:20.220Z	DEBUG	controller.interruption	removing offering from offerings	{"commit": "34d50bf-dirty", "queue": "karpenter-venture-preprod-queue-interruption", "messageKind": "SpotInterruptionKind", "machine": "karpenter-spot-worker-preprod-66tr5", "action": "CordonAndDrain", "node": "ip-172-27-13-123.ap-southeast-2.compute.internal", "reason": "SpotInterruptionKind", "instance-type": "t3a.small", "zone": "ap-southeast-2a", "capacity-type": "spot", "ttl": "3m0s"}

2023-10-10T05:24:20.236Z	INFO	controller.interruption	initiating delete for machine from interruption message	{"commit": "34d50bf-dirty", "queue": "karpenter-venture-preprod-queue-interruption", "messageKind": "SpotInterruptionKind", "machine": "karpenter-spot-worker-preprod-66tr5", "action": "CordonAndDrain", "node": "ip-172-27-13-123.ap-southeast-2.compute.internal"}

Close the issue as no bug found.


[HELP NEEDED] Migrate dive to use newer gocui code base

dive current still uses the 0.x branch of gocui which is quite old and also the reason why the following is required:
// related to https://github.com/awesome-gocui/gocui/blob/master/migrate-to-v1.md
replace github.com/awesome-gocui/gocui => github.com/awesome-gocui/gocui v0.6.0

I do not have cycles to work on this at this time but would welcome anyone submitting a pull request.

Happy to take a look. Love the project and would like to help keep it going.

@wagoodman has fixed this upstream. No longer needed.


[DDW-595] Improve input popover handling

This PR improves the configurability of the Input and FormField components to support:
New Features

Showing form field error on hover
Showing form field error on focus
Styling the form field errors differently via injected css variables

which is required by https://github.com/input-output-hk/daedalus/pull/2506

BREAKING CHANGE :boom:
❗next release must increase major version

FormField now shows errors on hover and focus by default which can be configured via two new props:

  isShowingErrorOnFocus: boolean,
  isShowingErrorOnHover: boolean,

❗This means that we need to check all form elements (input, select, autocomplete, etc.)
Chores

Added Github workflow for automated testing 🎉

@aleksandardjordjeviciohk this PR is now released as temporary release under 0.9.7-next.1 - you need to update your PR to include all the latest changes + this PR

This PR waits for https://github.com/input-output-hk/daedalus/pull/2506 to be reviewed and merged.


Ensure sys.executable set correctly on MacOS systems

@Forsworns oh this is an interesting one. Based on the output that you provided I tracked the offending lines of code to packaging.types: https://github.com/pypa/packaging/blob/3030822b0b06a43fbbb2710da7b0846d1bebd2ba/src/packaging/tags.py#L399-L410
The code calls subprocess.run where the target executable is pulled from sys.executable. sys.executable is set correctly in my Linux test environment:
>>> import sys
>>> sys.executable
'/home/spring/dragon/.env/bin/python3`

Can you run the code listed above from your Ghidrathon interpreter? I'm guessing sys.executable is set to the Java executable used to run Ghidra in your Mac environment...
Originally posted by @mike-hunhoff in https://github.com/mandiant/Ghidrathon/issues/59#issuecomment-1660841345

It appears the Jep developers know of this issue: https://github.com/ninia/jep/blob/056ce9907f5ecbf2364df1ec55755404b2e8a947/commands/test.py#L73-L76 which they avoid by setting the environment variable PYTHONEXECUTABLE. Ghidrathon probably needs to set this as well...or manually configure sys.executable during the init phase if running in a Mac environment. Let's open a new issue to track this work
Originally posted by @mike-hunhoff in https://github.com/mandiant/Ghidrathon/issues/59#issuecomment-1660865331

Yes, it is set to the Java executable

I think an optional configurable item in GhidrathonConfig.xml will be helpful. I manually revise it to my python executable and now it works well :cat:

I'll try to resolve the issue later this evening (my time zone), but  at the moment I must depart for work :(

This also appears to be an issue on Windows systems.

This should be addressed in https://github.com/mandiant/Ghidrathon/commit/32f3969e09a5d29197cabbc28d93dc9c572d7e45 - please reopen if you continue to have issues after this is released.


[Run] COMException (0xD0000701)

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
Unknown, sorry. Returned from Sleep, and found the 'PowerToys Run ran into an issue' screen on my desktop, requesting that I post this bug
✔ Expected Behavior
No error? 😄
❌ Actual Behavior
Dialog box 'Something went wrong' on my desktop after a resume from sleep
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 05/01/2024 23:16:39
Exception:
System.Runtime.InteropServices.COMException (0xD0000701): 0xD0000701
   at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
   at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
   at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
   at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

Other Software
No response

xref #30507

Also received this today. I walked past my workstation (that is configured to never sleep), but I happened to notice the screen flashed to black and turned on again. I have the same message that @TinyBigJacko ran into.
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22631.0
IntPtr Length: 8
x64: True
Date: 4/24/2024 5:51:26 PM
Exception:
System.Runtime.InteropServices.COMException (0xD0000701): 0xD0000701
   at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
   at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
   at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
   at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)


Update user guide

Added section for undo and redo feature.
Updated section for filter feature.
Yet to add more terms to glossary.

Codecov Report

Merging #139 (28436b7) into master (1e1c2dc) will increase coverage by 0.26%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #139      +/-   ##
============================================
+ Coverage     69.92%   70.19%   +0.26%
- Complexity      544      547       +3
============================================
  Files            84       84
  Lines          1852     1852
  Branches        199      199
============================================
+ Hits           1295     1300       +5
+ Misses          503      500       -3
+ Partials         54       52       -2

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <0.00%> (+17.24%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[CARBONDATA-4081] In Clean files operation, only consider ".segment" files in the segments folder for cleaning stale segments

Why is this PR needed?
While getting stale segment, we do a list files and take all the files, if there is any folder/file other than .segment file, it will lead to further issues while copying data to the trash folder.
What changes were proposed in this PR?
Added a filter, to only consider the files ending with ".segment"
Does this PR introduce any user interface change?

No

Is any new testcase added?

No

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5137/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3375/

can you add a test case for fault testing?

@QiangCai , i have changed current test cases itself. Please check

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5166/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3404/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5168/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3406/

@vikramahuja1001 : when the issue is found, what other file was present in segment metadata directory ? how it is impacting ?

@ajantha-bhat , so in the case of partition table, we have a tmp directory in the segment folder. In this case, while we check for stale segments, we are blindly reading all the files in the segment folder and collecting the name of it and then based on that we read the segment files, since it is not a file, thus it cannot be read and would give an exception. In this fix, we just need to filter if the file ends with ".segment"

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5175/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3413/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5182/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3420/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3434/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5194/

LGTM


Add XUnit Infrastructure

Xunit Infrastructure and Skeleton
First Steps done!

2 test assemblies created for

2 different implementation assemblies on

2 different layers (Controller and Core)

BaGet.Core.Tests
First sample that allows testing Services
BaGet.Tests
First sample that allows testing asp.net Controllers using a TestServer from Microsoft.AspNetCore.TestHost
Microsoft.Extensions.Logging redirected to Xunit.ITestoutputHelper
Notes

Naming guidelines like suggested in #7

Questions
(to discuss - probably on another place)
Test Infrastructure Code
now we have code like TestServerBuilder or Xunitlogger.
if this code base is growing we should create a separate assembly that contains all that code an can be references by multiple test-assemblies
Test Data
we need some testdata - specially for unit tests.
if the test data can't be generated via code then a file based solution (resource files compiled into a assembly) are the most common/easiest solution
Resource Assemblies can be bigger => needs more time during compiling.
We should create a separate assembly for that. The content normally is not changed so incremental build (msbuild) can do his work.

Okay, finally reviewed it. Looks great! I left some minor nitpicks, I'll go ahead and merge it once it's fixed. Thank you! 😄


Enhancement: Ability to toggle on/off the charting clap

What is your suggestion, and why should it be implemented?
The clap is a little echoy making high nps sections difficult to check while charting. It also means theres no way to listen to the music on its own while charting if there are notes on the section. Its also a little buggy when it comes to pressing pause/play.
Being able to toggle it on/off would be a great QoL improvement.

are you referring to the note hit sounds ? because if you are , that is a toggleable in the Data Section. if not , then this is confusing of what this enhancement is or im just dumb.

 Lol

yeah this exists already


fixed dependencies

I couldn't make this build locally - I've changed the OpenCV version (explicitly finds opencv 2 - we had v3 installed as well, so the build wouldn't work).
Also, the correct ros dependency is called libopencv-dev (not opencv2).
Finally, I've changed the looping over libraries to just using "OpenCV_LIBRARIES". This contains all the dependencies and seems to work fine... I hope I've not missed something crucial ;)

I couldn't make this build locally - I've changed the OpenCV version (explicitly finds opencv 2 - we had v3 installed as well, so the build wouldn't work).
Also, the correct ros dependency is called  libopencv-dev (not opencv2).
Finally, I've changed the looping over libraries to just using "OpenCV_LIBRARIES". This contains all the dependencies and seems to work fine... I hope I've not missed something crucial ;)

Everything seems to be working fine. CMakeList was done under ROS hydro. I just checked on Jade and it does compile fine. Thanks!


pip install mediapipe error

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
None
OS Platform and Distribution
Windows
Mobile device if the issue happens on mobile device
No response
Browser and version if the issue happens on browser
No response
Programming Language and version
python
MediaPipe version
latest
Bazel version
No response
Solution
Cannot Find one so ask here
Android Studio, NDK, SDK versions (if issue is related to building in Android environment)
No response
Xcode & Tulsi version (if issue is related to building for iOS)
No response
Describe the actual behavior
cannot use pip install for mediapipe with error

ERROR: Could not find a version that satisfies the requirement mediapipe (from versions: none)
ERROR: No matching distribution found for mediapipe

Describe the expected behaviour
should be able to install mediapipe

Standalone code/steps you may have used to try to get what you need
I tried to install different version but does not work

Other info / Complete Logs
No response

Hello @tomyanth Please provide the details on

Windows version
Python version

Windows 10
Python 3.9.7

@tomyanth,
Thank you for bringing out to our notice. As workaround, Please upgrade to Python version to 3.10.7 and let us know if still an issue to build MediaPipe. However, I tried to build using the above mentioned Python version, Pip 22.2.2 with latest version 0.10.0 successfully as screenshot attached below.

This also happen for

windows 11
python 3.11.3

ok. I will try and return to you as soon as possible. Thanks

@tomyanth,
Could you please update the status of the above. Thanks!!

It seems to work now even I did not update anything. Thanks.

Sorry for the late reply because I need to restore the computer environment for the test

@tomyanth,
Thanks for the confirmation, Can we mark the issue has been resolved and close it as this is no longer issue?

Of course, will let you know if that happens again

Hello @dkackman,
To keep context of the easier to filter out the issue with title. We suggest you to raise a fresh issue with problem facing in the Windows 11. As, This issue about the Windows 10. Thank you!!

@dkackman,
Thanks for the confirmation, Closing this issue as resolved.


Baseline codeowners and maintainers

Description
Baseline codeowners and maintainers
Issues Resolved
#481
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #547 (be362a9) into main (d66bd60) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #547   +/-   ##
=======================================
  Coverage   52.82%   52.82%
=======================================
  Files         212      212
  Lines        5594     5594
  Branches      782      782
=======================================
  Hits         2955     2955
  Misses       2637     2637
  Partials        2        2

Synced and verified access. I removed @praveensameneni.


How to add a java system argument when starting the actor

Hi
We need to add system arguments when starting the actor to avoid an error when accessing a database.
Do we have a way to do it?
Example:
-Doracle.jdbc.timezoneAsRegion=false
Thanks

Hi,
I solve this by adding the property via de JDBC string connection based on this page Setting JDBC Connection Properties
My string connection was like this jdbc:oracle:thin:@XX.XX.XX.XX:YYYY/MDWAPIQA?oracle.jdbc.timezoneAsRegion=false
Since my database was oracle I use this way, it will depends on the database driver specification


create_openai_tools_agent (RunnableSequence) doesn't return input_keys, breaks AgentExecutor.input_keys

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain.agents import AgentExecutor, create_openai_tools_agent

from langchain_core.prompts import (
    ChatPromptTemplate,
    HumanMessagePromptTemplate,
    MessagesPlaceholder,
    PromptTemplate,
    SystemMessagePromptTemplate,
)

prompt = ChatPromptTemplate.from_messages(
    [
        _system_message,
        MessagesPlaceholder(variable_name=memory_key, optional=True),
        HumanMessagePromptTemplate.from_template("{input}"),
        MessagesPlaceholder(variable_name="agent_scratchpad"),
    ]
)

agent = create_openai_tools_agent(llm, [], prompt)

agent_executor = AgentExecutor(
    agent=agent,
    tools=[],  # type: ignore
    memory=memory,
    verbose=True,
    return_intermediate_steps=True,
    handle_parsing_errors=True,
)

print(agent_executor.input_keys) # Return empty list

Error Message and Stack Trace (if applicable)
No response
Description
Since create_openai_tools_agent returns a RunnableSequence and not a BaseSingleActionAgent, the property input_keys of the AgentExecutor doesn't work for this agent anymore
@property
def input_keys(self) -> List[str]:
    """Return the input keys.

    :meta private:
    """
    return self.agent.input_keys

System Info
System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.4.0: Wed Feb 21 21:44:54 PST 2024; root:xnu-10063.101.15~2/RELEASE_ARM64_T6030
Python Version:  3.9.18 (main, Sep 11 2023, 08:25:10)
[Clang 14.0.6 ]

Package Information

langchain_core: 0.1.27
langchain: 0.1.9
langchain_community: 0.0.24
langsmith: 0.1.10
langchain_experimental: 0.0.52
langchain_google_genai: 0.0.6
langchain_openai: 0.0.6

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langgraph
langserve

Yeah the input_keys variables work with arbitrary runnables. What are you trying to use the property for?

@hinthornw , the input_keys are used in the LangFlow application to dynamically set the input fields required to test out the agent.


Control lights via Home Assistant

Night time lights need to be able to be controlled via Home Assistant API.
Feature editor be exportable to use in other future applications

Due to architecture change, will not be doing this task.


restructure file tree

In this PR:

restructure the file tree according to the layout in #9, it leaves the file content of all files unchanged (we'll work on that later)
rename *.cc files to *.cpp
rename *.h files to *.hpp
added 4 usage directories. These are examples of how users may include the library:

by including the library as a plain copy (plain-copy/)
by includign the library as a copy made with git submodules (git-submodule/)
by locally installing the library to their system (locally-installed/)
by using cmake to collect a copy of the dependency from the internets (cmake-pull/)

added cudawrappers config cmake file, to help users use this library in their cmake files without hassle

PS we can still split off the usage examples subtree later if we feel that's more clear for consumers.
Refs:

#9

How to check:

look at directory structure online or in locally checked out copy of the repo

Thanks for reviewing!


Use native promises

Refactored the code to use native promises and remove a few dependencies.

Thank you. I will have a look on this and solve the conflict once I'm back home.

Works now.
Previously it just never initializes bptf-listings.

I think returning the promise from the start function should fix the error handling issue. Just added a commit which does that.

nah.

I think should have catch here.
https://github.com/TF2Autobot/tf2autobot/blob/aaf13c5500531c65444945a3353e524c7f260ccd/src/app.ts#L144-L152
.catch(err => throw err));

The process is still not terminated/crashed when there's an error on start.

Can you place a log statement before line 734 in src/classes/Bot.ts to see if the checkIfStopping function is being run after each promise in the chain?

Sure thing. Let me try.
Also, if you want to use my extra account reserved for testing, just let me know.

The old one should be here that will reject the start
https://github.com/TF2Autobot/tf2autobot/blob/f8f7c9a49bd92faad0cd6da9b5b4b1c5c20d3283/src/classes/Bot.ts#L1028-L1031
I am also not sure if using let promise = Promise.resolve(); is the right way to do or not.

Maybe bring back
https://github.com/TF2Autobot/tf2autobot/blob/f8f7c9a49bd92faad0cd6da9b5b4b1c5c20d3283/src/classes/Bot.ts#L735
https://github.com/TF2Autobot/tf2autobot/blob/f8f7c9a49bd92faad0cd6da9b5b4b1c5c20d3283/src/classes/BotManager.ts#L58
and reject on error, so that it will catch the error in app.js

I am unable to reproduce these issues on my machine at the current commit. I tried with node v16.15.1 and node v18.4.0.

If you're unable to reproduce the error, that means Steam is back alive.
Try adding a line that will throw an error anywhere in the promiseChains, like the one I previously did.

I can't reproduce even when manually introducing an error - the code correctly shuts down upon erroring on my machine. See the attached mov file.
https://user-images.githubusercontent.com/3758364/174504862-dd2f43b0-a871-494b-af27-66d90a6f89db.mov

hhmmmm weird.

Merging this into development.
Thank you very much!


pdns: 4.8.1 -> 4.8.2

Changelog: https://doc.powerdns.com/authoritative/changelog/4.8.html#change-4.8.2
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 253871 run on x86_64-linux 1

1 package built:

pdns


Issue when looping activated

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
The problem that I faced with is when you try to go back having looping property activated. The problem is that the slide goes to the last one, which is ok, but the dots shown the second index as you can see in the next video.
https://user-images.githubusercontent.com/101422684/207106565-560af164-0812-4f73-9c03-7ef76ca45803.mov
I've "fixed" this problem with the following code, but doesn't fix it at all because the slide animation move forward instead of backward. Maybe you can improve this fix and fix it at all to make this library more robust.
Here is the diff that solved my problem:
diff --git a/node_modules/vue-ssr-carousel/index.js b/node_modules/vue-ssr-carousel/index.js
index ae566a2..9e5de4f 100644
--- a/node_modules/vue-ssr-carousel/index.js
+++ b/node_modules/vue-ssr-carousel/index.js
@@ -1747,6 +1747,9 @@ Code related to dealing with advancing between pages
     },
     // Go to a specific index
     goto: function (index) {
+      if (index < 0) {
+        index = this.pages - Math.abs(index % this.pages);
+      }
       this.index = this.applyIndexBoundaries(index);
       return this.tweenToIndex(this.index);
     },

This issue body was partially generated by patch-package.

awesome job here, we have the same issue and is very confusing for the users. Thanks!
LGTM!!

I have the same problem 🤔


Training commands on CIFAR100

What are the specific training parameters for the training commands on CIFAR100？

Did you save this  issue?  I want to train the model on custom dataset.


Make ReflectionUsingPrepare multiply controlled

Currently it only accepts a single control_val (and this is a hard assumption as it inherits from SpecializedSingleQubitControlledGate). Sometimes we would like more than one control. Should we enable this?

xref #1008

this should be resolved now as #1456 reduces multiple controls to a single control


symfony-cli: 5.5.5 -> 5.5.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/symfony-cli/symfony-cli/releases.
meta.description for symfony-cli is: Symfony CLI
meta.homepage for symfony-cli is: https://github.com/symfony-cli/symfony-cli
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.5.6 with grep in /nix/store/18f8s8mg63ix2czjr1d5k89d51n322hy-symfony-cli-5.5.6
found 5.5.6 in filename of file in /nix/store/18f8s8mg63ix2czjr1d5k89d51n322hy-symfony-cli-5.5.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
symfony-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/18f8s8mg63ix2czjr1d5k89d51n322hy-symfony-cli-5.5.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A symfony-cli https://github.com/r-ryantm/nixpkgs/archive/c2fd4556d6b1ff70413f66b71d9e3c765d72c856.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/18f8s8mg63ix2czjr1d5k89d51n322hy-symfony-cli-5.5.6
ls -la /nix/store/18f8s8mg63ix2czjr1d5k89d51n322hy-symfony-cli-5.5.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

symfony-cli

Maintainer pings
cc @drupol for testing.

1 package built:
symfony-cli
error: build log of '/nix/store/v6wi11vp9kdf03m2h3agspfhcmb4mckl-symfony-cli-5.5.6.drv!*' is not available
error: build log of '/nix/store/18f8s8mg63ix2czjr1d5k89d51n322hy-symfony-cli-5.5.6' is not available
Result of nixpkgs-review pr 232956 run on x86_64-linux 1

1 package built:

symfony-cli

symfony command runs...


German Postal Bank (Deutsche Postbank)

I'd like for them to have that kinda service.

@arisboch feel free to create a pull request for this site.

How can I do that?
On 05/12/2015 02:55 PM, mxxcon wrote:

@arisboch https://github.com/arisboch feel free to create a pull
request for this site.
—
Reply to this email directly or view it on GitHub
https://github.com/jdavis/twofactorauth/issues/1123#issuecomment-101266098.

--
Boris Baran
Skype:psycho_mantis87  skype:psycho_mantis87
Ceterum autem censeo 'Yahoo! neo' esse delendam.

I know that I know nothing.
Only a Sith deals in absolutes.
The cake is a lie.
Cast in the name of God, Ye (not) guilty.
All your base are belong to us. You are on the way to destruction. You have no chance to survive make your time.
Dropbox fanfic copy folder:https://www.dropbox.com/sh/zp4t4e1i0cabar6/mXs8gCsERj

@arisboch look at the wiki

@arisboch Would you like to submit a pull request for this site? If not, you can provide the necessary details about it (see CONTRIBUTING.md) and we will add it to our list of sites to be added. Please note that this site is maintained by volunteers and it may not be handled immediately.


Fix docs for release

Thanks for merge 'dev'!
I fix some typos!

My pleasure. And thank you for the PR :)


Large Amount of data

1.How much of rows and columns supported in serenity grid view system if supported any  perform issues??
2.above 10 lacs of rows and more than 100's of columns is supported or not??

Hi,
rows should be virtually unlimited (limited only by the underlying SQL Server System). Performance will always be very good as Long as you Limit the rows per page in the grid to - let's say 100.
About columns amount I don't know but simply test it and then post here your eperience.
As even on the biggest Screens with a 4K Resolution you cannot Display too many colums at the same time more than 100 columns seems a bit much for me - but of Course I don't know your Project...
With Kind regards,
John


FossGraph. Make changes to the database

Closes https://github.com/saveourtool/save-cloud/pull/2075


Fix avatar resizing for guests browsing the forums.

Add .avatarresize style bit for guests browsing the forums.

I'm a bit surprised of this (unless it's a bug in 1.1 that I can't test right now), on elkarte.net it seems to work fine and the relevant code is added to the template both for logged in members and guests (i.e. browsing the forum as a guest).
Can you give me some instructions on how to reproduce the problem?

It needs for avatar size limits to be enforced only as CSS resizing, not re-encoding or outright disallowing.

So, I have the forum (1.1) set to "let CSS resize them (avatar)":

and I'm browsing as a guest:

but here I see the "resize" code block for the resize (see the bluish area of the Chrome inspect on the right).

It must only affect 1.0.6, then. Close and drop.

It also depends on user avatars on the server being noticeably larger than 65x65, the default size.

Let's keep it open to be tracked as a possible bug, it may be in a certain combination it may fail.

This was a very valid report, sorry @kode54 for dismissing it too quickly.
I pushed here another fix:
https://github.com/elkarte/Elkarte/pull/2458/commits/2210a08ecd96325710895f499e2ade30f178bb1e
I moved entirely the original code up in the chain to loadTheme where it better belongs.


A Speaker Detail Page

スピーカー詳細のガワ作成
セッション一覧のスピーカー情報表示欄をタップ可能に
つなぎ込みはセッション一覧画面の内容がjSONデータになってから

Pull Request Test Coverage Report for Build 11

0 of 49 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/page/time_table_page.dart
0
17
0.0%

lib/page/speaker_detail.dart
0
32
0.0%

Totals

Change from base Build 9:
0.0%

Covered Lines:
0

Relevant Lines:
260

💛  - Coveralls


ACM-7063 Overview page should detect if Observability is installed

Related issue: https://issues.redhat.com/browse/ACM-7063
Overview page should detect if Observability is installed and either hide Alert & operator metrics if disabled show if enabled.

/test unit-tests-sonarcloud

@jlpadilla FYI if you would like to review - Changes to detect if Observability is installed, and if enabled, show the Alerts & Operator metrics.

/lgtm

/retest-required


Controls in a SfTextLayout do not take the Background from the parent

After updating the syncfusion nugets, this stopped working. Now it is not possible to set the Background in a control nested in a SfTextInputLayout.

Not a bug, the underlying control need to overwrite the  VisualStateList or use a Transparent background.


selhost/jakttest: Fix jakttest on MacOS

This fix the jakttest for Mac users

Instead of the commit title simply saying "Fix x on y" -- describe the change made in the title. The rationale can go in the message :^)

jakttest: Fix struct stat member declaration renaming on MacOS
Although it is supposed to follow POSIX.1-2008, struct stat in MacOS headers doesn't have a consistent naming for struct stat timing members (st_atime, st_mtime,...); leaving with us with no choice but to patch our own headers to implement a fix that should've been there 16 years ago.

Thanks @cybergsus
I updated the title and description with your suggestion to make it more understandable

My bad. I amended the commit to reflect the same message.
Thank you again


Removed Codex

Codex API doesn't work anymore

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[ ] Any category I am creating has the minimum requirement of 3 items
[ ] All changes have been squashed into a single commit

@touero When will the other maintainers have a look at this pr ?


Restore "kontena login" and "logout" commands

From community Slack:

This PR restores the good old "kontena login", but with a twist:
kontena login master is exactly the same as kontena master login
kontena login cloud is exactly the same as kontena cloud login
And therefore:
kontena login gives you the subcommand list:
Usage:
    kontena login [OPTIONS] SUBCOMMAND [ARG] ...

Parameters:
    SUBCOMMAND                    subcommand
    [ARG] ...                     subcommand arguments

Subcommands:
    master                        Login to a Kontena Master
    cloud                         Login to a Kontena Cloud account

Same goes with kontena logout.

Should kontena logout behave the same, then?
$ kontena logout
 [error] Command removed. Use kontena master logout to log out of the Kontena Master
$ kontena logout -h
Usage:
    kontena logout [OPTIONS]

  Command removed, use 'kontena master logout' to log out of the Kontena Master
  or 'kontena cloud logout' to log out of the Kontena Cloud

Options:
    -h, --help                    print help
    -D, --debug                   Enable debug (default: $DEBUG)

It does now.

LGTM in terms of implementation, but not sure about the design.
Having both kontena cloud login and kontena login cloud seems more confusing than just killing the deprecated kontena login command entirely.
Alternatively, a single kontena login http://.../ command that does both a cloud login + master login...?
Same for kontena logout - hvae it logout of both master and cloud?

Same for kontena logout - have it logout of both master and cloud?

Are people always wanting to logout from both master and cloud? I think there are cases where one wants to logout only from master but not from cloud.

Are people always wanting to logout from both master and cloud? I think there are cases where one wants to logout only from master but not from cloud.

Sure, that's what the current kontena master logout does.

Sure, that's what the current kontena master logout does.

Yes, but the point was that if kontena logout does both of those tasks, is it more or less confusing for users? :)
TBH I don't know which one is better way, they both have pros and cons. ¯_(ツ)_/¯

@jakolehm WDYT, is the proposed changes to have kontena login master == kontena master login and kontena login cloud == kontena cloud login ok, or source of confusion?

Pehaps the main command help should just have some kind of "getting started" banner.

?

IMO banner approach is (also) good, it pretty much achieves the desired results without adding duplication in commands.

Having both kontena master logout and kontena logout master seems more confusing than just killing the deprecated kontena logout command entirely.

I agree, it's confusing.

So having multiple commands do the same things is now deemed as confusing, for a good reason.
Maybe the best alternative to "solve" this issue is the banner approach.

Replaced with #2385


Always force download

Hello. I have discovered this issue now too. In my applicaton the collection is used as storage, so user can upload any file really. Is there way to always force a download, ? something similar to this:
"So if I would click the link foo.bar/cdn/storage/badFild.html it would redirect you to foo.bar/cdn/storage/badFild.html?download=true, right?
If that always prevents an execution it should be sufficient."
This doesn't happen actually, do i need to force it trough router?
Originally posted by @gjdanskij in https://github.com/VeliovGroup/Meteor-Files/issues/289#issuecomment-557804348
I tried to force with FlowRouter by doing like this:
FlowRouter.route('/cdn/storage/FilesData/:id/original/:fname',{    action: function(params){      FlowRouter.redirect('/cdn/storage/FilesData/' + params.id + '/original/' + params.fname + '?download=true') ;   }
No redirect happens for some reason, can you help how to do this
});

Hello @gjdanskij ,
I'd recommend to set Content-Disposition: attachment header using responseHeaders.
Or even easier solution would be to use interceptDownload hook and set http.params.query.download to true, like:
new FilesCollection({
  interceptDownload(http) {
    http.params.query.download = true;
    return false; // <-- returning false is important here
  }
})

I tried your second solution, and what it does it always forces preview not download. So i removed that immediatly, i will try responseHeaders

it always forces preview not download

What exactly do you mean by "preview"?
It suppose to force the very same behavior as you would use ?download=true — the rest is on browser. Different browsers act differently upon Content-Disposition: attachment header presence. The only 100% way to preserve "download" action is to add download="file.name" attribute to anchor download file element.
I'm afraid using responseHeaders you will get the very same results.
Marked as [help wanted], feel free to add link and ask for more help at forums.meteor.com

Closed due to silence at issue owner end.
Feel free to reopen it in case if the issue still persists on your end.
//cc @gjdanskij


Missing parsed information

Parser unfortunately doesn't return full information about parsed code.
Right now I know about missing

async flag (https://github.com/buehler/node-typescript-parser/issues/66)
isStatic flag (https://github.com/buehler/node-typescript-parser/issues/43)
propertyIsOptional flag (e.g. { myProperty?: string } )
while parsing property of type Object (e.g. { myProperty: { innerPropery: string } }, the type is not fully parsed and returns values like
{\n        email: string\n    }
I can parse finish parsing by recursive call of parser.parseSource(), but it's very inconvenient

Can we get this fixed, please?

flag for optional property is implemented in this pull request I made https://github.com/buehler/node-typescript-parser/pull/80

Things left open:

[ ] isAsync
[ ] isStatic
[x] isOptional

:)
About the inline types... We need to think about a reasonable and good way to parse those things. It shouldn't kill the parser by loading recursive stuff to do.

Another important feature is:

[ ] support for dictionaries

This code
export interface TestInterface {
    [key: string]: number
}

results in this
InterfaceDeclaration {
  name: 'TestInterface',
  isExported: true,
  start: 13,
  end: 97,
  accessors: [],
  properties: [],
  methods: []
}

and

[ ] inheritance

export interface TestInterface extends OtherInterface {
    count: number
}

Yep, inheritance is already present in #88

@ondratra a question:
would (in your opinion) anything that returns a promise be async? actually yes right?
So if the user has public async function this means (automatically) it returns a promise. And if I code public function foo(): Promise<void> I return a promise which is also async and awaitable.

I'm going to close this issue in favor to propper issues for each feature :)


Changing the golden tests to produce a nice diff output.

Sigh, I think it may be time to admit that we need a plutus-util library, it's dumb having this stuff in language-plutus-core. Not for this PR, though.


"Invalid Location" when deleting recently added blocks

Describe the bug
Blocks are not able to be deleted if they have just been added. I get an "Invalid Location" error. However, the problem disappears after refreshing and they blocks can be deleted.

This is closed with https://github.com/nathanhollows/Rapua/commit/8dae6626c9df1f990f4b185cff9bd38881a44b5c


Contact form to collect emails

it should be present on the contact page, at the end of each article, and on the books page.

at the moment the best solution seems to integrate it with Netlify, since it will be the best method to deploy the website


Improve warnOnce warning

Describe the Feature
Would be nice to make the warnOnce util include caller information.
It currently logs "Warning: RNCNetInfo - You are using the deprecated API. It will still work, but you must upgrade to the new API to receive the new features. The old API will be removed in the future".
This log is helpful, but it would be even more helpful if it included information about WHICH deprecated API is being called. This will make it easier for clients on the upgrade path to identify what is doing the wrong thing. This is non-trivial in some cases for clients that use dependencies that wrap react-native-netinfo as a simple code search won't turn up these cases, but some enhanced logging would facilitate scoping of these issues.
Something like
Warning: RNCNetInfo - ${caller} is using the deprecated API. It will still work, but you must upgrade to the new API to receive the new features. The old API will be removed in the future
Possible Implementations
Add a string: caller param to warnOnce that is hard coded everywhere that calls warnOnce()
The warn once logic would have to be modified to only warn once PER caller not globally. So that if warnOnce gets called via 3 different callers and a 4th duplicate caller, it should log 3 times
Related Issues
https://github.com/react-native-community/react-native-netinfo/issues/123

👋 This is a very good idea. Any help to implement this is appreciated.

Closed as we have no removed all deprecated methods in #230.


Unable to execute codedui tests without installing Visual Studio thrtough Azure DevOps

When the "Visual Studio Test Platform Installer" and "VsTest" tasks are added in the release definition, the CodedUI tests are failed with the below errors.
019-03-26T04:42:39.6678793Z ##[warning]DiscoveryMessage : TestSettings will soon be deprecated for automated unit and functional testing scenarios. It is recommended that you use RunSettings. To learn more, see http://aka.ms/runsettings
2019-03-26T04:42:40.3716628Z ##[warning]DiscoveryMessage : No test is available in F:\CodedUIAutomation\CodedUIAutomation\Test.dll. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.
Please help to resolve the issue.

Please provide the full set of logs (the entire zipfile you get when you click download all logs) with system.debug = true.
Also if you are building with a particular major version of VS then you will have to ensure you use the same major version of the testplatform when running CODED UI tests. My suspicions are on this.

@Selvaraj1989 closing due to no response, please re-open if you are still facing issues (with the requested logs) after following the suggestion above.


Single file articat dump --local-path could set files_pattern

When dumping a single file via the articat dump CLI (--local-path=single_file), it would be convenient if files_pattern got set to the single file's name.
This way the single file artifact could be read without having to know or guess the filename.
Does this make sense or could it cause other problems?

Makes sense!


Files open in editor seem to behave undeterminist when renamed/moved

If the file is open in text editor it seems transform changes applied are reverted. Some kind of atom cache or git cache issue is happening - i think. Need to research tree-view move-dialog and rename functions to see what subtleties are missing in our implementation

Attempted a fix for this issue. Modeled after the below code. Since I currently cannot get a list of updated file paths from jscodeshift I am just passing the entire project dir to git to examine. I would like to make this explicit in the future with paths that actually changed.
https://github.com/atom/tree-view/blame/492e344149bd42755d90765607743173bb9b7c0a/lib/move-dialog.coffee#L40

I've now added code that examines all the open text editors and all the base project directories and calls the git path status on each in order for atom to programmatically register the file change coming from the codemods.
In order to simplify and ensuring no future bugs crop up I need a list of the file paths that were affected by the codemods.

Was able to fix this without modifying jscodeshift.
https://github.com/jurassix/atom-refactoring-codemods/blob/master/lib/renamePaths.js#L48
However, I still need reporting from a codemod so we can support an interactive refactor of files.


Document the usage of Events

We recently included #1246 and #1268 event detectors and they were properly validated. However, there isn't any tutorial on how to use them apart from the LithobrakeEvent in the natural and artificial perturbations notebook.
Let us have a notebook dedicated to Event detectors so users can know about this feature and how to use it! 🚀

Pinging here @Yash-10 as this is related with his GSoC'21 contributions.

Thanks for this issue. I would surely create a tutorial notebook for the implemented events during this period.

If you need help mentally separating a tutorial from a how-to, check out https://diataxis.fr/ :)


Intermediate symlink resolving in realpath

Currently seems that uutils doesn't resolve symlinks that is hidden as a path inside another symlink
$ mkdir dir1
$ mkdir dir1/dir2
$ touch dir1/dir2/file1
$ ln -s dir1 link1
$ ln -s link1/dir2 link2
$ realpath link2/file1
/tmp/example/dir1/dir2/file1
$ ~/repositories/uutils/target/debug/realpath link2/file1
/tmp/example/link1/dir2/file1

As well as other utilities that use canonicalize function don't check for existence of each part of the paths inside symlinks.

PR #3703 closes the issue


Add tutorial on Dirichlet Process Mixture Models

See https://forum.pyro.ai/t/variational-inference-for-dirichlet-process-clustering/98 for extended discussion .
It would be nice to make this work with data subsampling.

cc @vincent6606 @fehiepsi

it's easy enough to write a dirichlet-multinomial mixture model (what y'all have been calling dirichlet process, though that terminology is usually reserved for the infinite version), and pretty easy to write the DP in either stick breaking or CRP form. (see, eg, corresponding code in webppl.) however, getting VI to work for mixture models is not entirely straightforward and for infinite models it's still an open problem as far as i know.
i have a few ideas, and i think this is a great thing to work on, but having a tutorial in 0.2.1 might be overly optimistic? i may be misinterpreting the scope of what y'all are proposing though!

Noah, weren't you working with an MIT student on that topic (svi for dpp) ?
Any updates on how this is going?
On Mar 12, 2018 7:45 AM, "ngoodman"<EMAIL_ADDRESS>wrote:

it's easy enough to write a dirichlet-multinomial mixture model (what
y'all have been calling dirichlet process, though that terminology is
usually reserved for the infinite version), and pretty easy to write the DP
in either stick breaking or CRP form. (see, eg, corresponding code in
webppl.) however, getting VI to work for mixture models is not entirely
straightforward and for infinite models it's still an open problem as far
as i know.
i have a few ideas, and i think this is a great thing to work on, but
having a tutorial in 0.2.1 might be overly optimistic? i may be
misinterpreting the scope of what y'all are proposing though!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/uber/pyro/issues/880#issuecomment-372334673, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABVhL1a2fPIRPqHW-8HQb-7kFHcmpZwoks5tdooQgaJpZM4Sl0xZ
.

hi, any plans on  logistic mixture model?

hi, any plans on logistic mixture model?

@zhf459 not from us, but feel free to make one and open a PR!


Using MassTransit.CastleWindsor 5.3.X results in exception on Consume

Is this a bug report?
Yes, using MassTransit.CastleWindsor when updating Masstransit from 5.2.3 to 5.3.0 or any version above, the consumers throws exceptions (diffrent errors depending on Castle Windsor version) and messages are routed to error queues.

Can you also reproduce the problem with the latest version?
Yes
Environment

Operating system: Windows 10
Visual Studio version: VS 2017 Professional Version 15.9.7
Dotnet version: 4.6.1

Steps to Reproduce

Set up a project with Masstransit 5.2.3, MassTransit.CastleWindsor 5.2.3, and Masstransit.AzureServiceBus 5.2.3 (or use the supplied demo)
Configure according to examples and add consumer
Send messages and watch the messages get consumed
Upgrade Masstransit packages to 5.3.0 or above
Watch conumers throw exceptions

Expected Behavior

I expected 5.3.X to work pretty much the same as 5.2.3
Actual Behavior

If Castle.Windsor is at version 4.1.1 as as is the lowest supported version of Masstransit.CastleWindsor 5.3.0 the exception will be  "Newtonsoft.Json.JsonReaderException -
Unexpected character encountered while parsing value: S. Path '', line 1, position 1."
If Castle.Windsor is at version 5.0.0  the exception will be "Castle.MicroKernel.ComponentNotFoundException -
No component for supporting the service MassTransit.Scoping.ScopedConsumeContextProvider was found, "
Reproducible Demo

https://github.com/prog-re/ServiceBusWindsorTest
This project has Masstransit 5.3.0 and does not work. Rolling it back to 5.2.3 makes it work.
Please note, you need to supply your own service bus connection string in App.config

This has nothing to do with Castle, it has to do with the fact that you used 5.2.3 with Azure Service Bus - which was serializing messages incorrectly. That was fixed in 5.3.2, so that they are properly formatted JSON strings.

Sorry, I don't follow: In 5.2.3 messages was incorrectly serialized, but this was fixed in 5.3.2?
But with 5.2.3 the consumers work. With 5.3.X they don't.

I would suggest you empty your queues, upgrade to 5.3.2, and see if you can reproduce the issue.

It's reproducable with the supplied demo using Masstransit 5.3.2, 5.3.1 and 5.3.0

So the JSON error you're seeing is what was fixed in 5.3.2, and that indeed is fixed in 5.3.2.
The other error is because you aren't configuring MassTransit properly for use with Windsor.
http://masstransit-project.com/MassTransit/usage/containers/castlewindsor.html
I changed your installer to use the configuration and call AddBus(kernel => Create...) and the code works as expected.
You can see what is done by using the configuration, as there are many supporting types that most people get wrong when configuring containers:
https://github.com/MassTransit/MassTransit/blob/develop/src/Containers/MassTransit.WindsorIntegration/Registration/WindsorContainerRegistrationConfigurator.cs#L37
Hope this helps.

Yes, that takes care of it. Thank you so much!


encode dictionary type in fst footer

fix #1827

Codecov Report

Merging #1968 (88b8b63) into main (da28046) will increase coverage by 0.00%.
The diff coverage is 94.11%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##             main    #1968     +/-   ##
=========================================
  Coverage   94.50%   94.51%
=========================================
  Files         314      317      +3
  Lines       57417    58445   +1028
=========================================
+ Hits        54262    55238    +976
- Misses       3155     3207     +52

Impacted Files
Coverage Δ

common/src/dictionary_footer.rs
68.42% <ø> (+21.05%)
:arrow_up:

sstable/src/lib.rs
99.15% <93.33%> (-0.26%)
:arrow_down:

sstable/src/dictionary.rs
87.84% <100.00%> (+0.03%)
:arrow_up:

... and 37 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Shouldnʼt there be layer above the dictionary, that stores the type?

If dictionary type is inside the dictionary, then we canʼt use it in the future if we want to support multiple dictionaries types.
Version should be part of the dict itself, so we can change the fst, sstable format separately.


3Dconnexion.3Dconnexion3DxWare10 version <IP_ADDRESS>88

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I accidentally submit this pr, anyway there is a more recent version available. Closing


Can't clean up images from URL previews safely

The uploads are associated with the user doing the preview, and use the file name from the website itself. This can cause the media to be considered "duplicate" based on hash (if the user has uploaded the same image with same file name before) and therefore reuse the same MXC URI.
The recurring tasks to purge old previews only deletes the database records but doesn't issue a purge on the media, presumably because doing so could actually be a problem. This can leave unused media in the database.
However, the media does get considered against the user's quota, which makes this moderately sane. In practice though, the user ID shouldn't be used for the upload and we should guarantee that the MXC URI won't be reused. Fixing this for old previews will be tough, though.
A possible advantage of keeping the media around is URL previews are likely to happen multiple times, even after expiring, and therefore can "save" data by caching the hash. More research would be required to determine if this is actually true, though.
This issue applies both pre and post refactor.

Related: https://github.com/turt2live/matrix-media-repo/issues/354


Microsoft Teams service does not load chats

Describe the bug
When adding a Teams service to Franz, the login to Teams is successful. When navigating to the Chat list and selecting any private chat, chat messages do not show at all.
To Reproduce
Steps to reproduce the behavior:

Add Microsoft Teams service.
Login to Teams
Navigate the Chat list
Select any private chat

Expected behavior
The chat messages exchanged with the contact are displayed.
Screenshots

Desktop (please complete the following information):

OS: Windows Pro 19043.1288
Franz version: 5.7.0

Additional context
Teams service has worked normally until last week. I have 6 different Teams services added (6 different guest accesses to different tenants) - then they stopped working one by one.
I have tried clearing the cache, removing the Teams recipe from %appdata%/franz, uninstalling and re-installing Franz - none of the methods work.
Debug information:
{
"host": {
"platform": "win32",
"release": "10.0.19043",
"screens": [
{
"id":<PHONE_NUMBER>,
"bounds": {
"x": 0,
"y": 0,
"width": 1920,
"height": 1080
},
"workArea": {
"x": 0,
"y": 0,
"width": 1920,
"height": 1050
},
"accelerometerSupport": "unknown",
"monochrome": false,
"colorDepth": 24,
"colorSpace": "{primaries:BT709, transfer:IEC61966_2_1, matrix:RGB, range:FULL}",
"depthPerComponent": 8,
"size": {
"width": 1920,
"height": 1080
},
"displayFrequency": 60,
"workAreaSize": {
"width": 1920,
"height": 1050
},
"scaleFactor": 1,
"rotation": 0,
"internal": false,
"touchSupport": "unknown"
},
{
"id":<PHONE_NUMBER>,
"bounds": {
"x": 1920,
"y": 0,
"width": 1920,
"height": 1080
},
"workArea": {
"x": 1920,
"y": 0,
"width": 1920,
"height": 1050
},
"accelerometerSupport": "unknown",
"monochrome": false,
"colorDepth": 24,
"colorSpace": "{primaries:BT709, transfer:IEC61966_2_1, matrix:RGB, range:FULL}",
"depthPerComponent": 8,
"size": {
"width": 1920,
"height": 1080
},
"displayFrequency": 60,
"workAreaSize": {
"width": 1920,
"height": 1050
},
"scaleFactor": 1,
"rotation": 0,
"internal": false,
"touchSupport": "unknown"
}
]
},
"franz": {
"version": "5.7.0",
"electron": "12.0.6",
"installedRecipes": [
{
"id": "linkedin",
"version": "1.0.0"
},
{
"id": "messenger",
"version": "1.3.0"
},
{
"id": "msteams",
"version": "1.1.0"
},
{
"id": "trello",
"version": "1.0.1"
},
{
"id": "tweetdeck",
"version": "1.0.2"
},
{
"id": "whatsapp",
"version": "1.0.28"
},
{
"id": "yammer",
"version": "1.0.0"
},
{}
],
"devRecipes": [],
"services": [
{
"id": "92423bbd-cf09-459f-9c24-e9ef1dabde1a",
"recipe": "whatsapp"
},
{
"id": "8cf09046-8946-4c89-994b-c1e6dceb57be",
"recipe": "messenger"
},
{
"id": "d72ef5de-7406-419a-ac32-e5fcca54b61c",
"recipe": "linkedin"
},
{
"id": "314704a9-fb4e-4123-a804-9dfd39d91ed0",
"recipe": "tweetdeck"
},
{
"id": "3de6f05c-fb02-4133-8a86-526dffcdb206",
"recipe": "trello"
},
{
"id": "0c050c58-9fbc-454e-bfa7-a8936870cffa",
"recipe": "msteams"
},
{
"id": "7bffcbd9-5d82-4efa-b327-3b503eee52de",
"recipe": "yammer"
}
],
"workspaces": [],
"windowSettings": {
"width": 1316,
"height": 832,
"x": 357,
"y": 161,
"isMaximized": false,
"isFullScreen": false,
"displayBounds": {
"x": 0,
"y": 0,
"width": 1920,
"height": 1080
}
},
"user": "7ebbcf39-4685-48f6-9225-59553d084196"
}
}

Same for me!

Same here on clean install on PC and on existing setup on Mac.

Same here

Same problem.
Version: 5.6.1
I try:

clear cache
recreate service

We need do downgrade?

This workaround solved my problem:
https://github.com/meetfranz/franz/issues/2093#issuecomment-960291597

same issue here
version: 5.7.0

Same issue on a Mac with macOS Monterey


GoAccess 1.3 crashed by Sig 11

pyserver@ubuntu:/var/log/nginx$ tail -f access.log | egrep -v "jsapiver|token|hasnew|images|info|search" | goaccess -

==13234== GoAccess 1.3 crashed by Sig 11
==13234==
==13234== VALUES AT CRASH POINT
==13234==
==13234== Line number: 0
==13234== Offset: 0
==13234== Invalid data: 0
==13234== Piping: 1
==13234== Response size: 0 bytes
==13234==
==13234== STACK TRACE:
==13234==
==13234== 0 goaccess(sigsegv_handler+0x13e) [0x40df9e]
==13234== 1 /lib/x86_64-linux-gnu/libpthread.so.0(+0x11390) [0x7fede48df390]
==13234== 2 goaccess(main+0x129e) [0x40a8de]
==13234== 3 /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0) [0x7fede4524830]
==13234== 4 goaccess() [0x40af21]
==13234==
==13234== Please report it by opening an issue on GitHub:
==13234== https://github.com/allinurl/goaccess/issues

Thanks for reporting this. Would you be able to run this through gdb and/or strace?
$ curl tar.goaccess.io/goaccess-1.3.tar.gz | tar xvz
$ cd goaccess-1.3/
$ ./configure --enable-debug --enable-tcb=btree --enable-geoip=legacy --enable-utf8
$ make
$ gdb --args ./goaccess access.log
(gdb) r <enter>
(gdb) bt full <enter>

Oh, It's so sad that I can't reproduce this bug now ... I run the same command tail -f access.log | egrep -v "jsapiver|token|hasnew|images|info|search" | goaccess -, but this time it works well. Maybe farther observation is required.

Actually I could only encount this bug when I use the grep or egrep command in pipe, if I directly use tail -f access.log | goaccess - or goaccess -f access.log, there is no bug at all.

I tried to build from src today. When I ran command ./configure --enable-debug --enable-tcb=btree --enable-geoip=legacy --enable-utf8, I encounted these two errors:
checking for GeoIP_new in -lGeoIP... no
configure: error:
    *** Missing development files for the GeoIP library

checking for tchdbnew in -ltokyocabinet... no
configure: error: *** Missing development libraries for Tokyo Cabinet Database

I ran these two commands, then everything went well.
$ sudo apt-get install libgeoip-dev
$ sudo apt-get install libtokyocabinet-dev

I updated goaccess to the latest version from apt by following the README.md guide under this repo two days ago.
$ echo "deb https://deb.goaccess.io/ $(lsb_release -cs) main" | sudo tee -a /etc/apt/sources.list.d/goaccess.list
$ wget -O - https://deb.goaccess.io/gnugpg.key | sudo apt-key add -
$ sudo apt-get update
$ sudo apt-get install goaccess

Today I checked the version of this goaccess:
$ which goaccess
/usr/bin/goaccess
$ goaccess --version
GoAccess - 1.3.
For more details visit: http://goaccess.io
Copyright (C) 2009-2016 by Gerardo Orellana

Build configure arguments:
  --enable-utf8
  --enable-geoip=legacy
  --with-openssl

I found out that I don't have libgeoip-dev in the past, but this goaccess built with   --enable-geoip=legacy. I don't know whether this will cause a crash.

I've also ran gdb as mentioned above, but no error occured.


fixed the page load issue in Firefox Desktop and other browsers.

Fixes

Fixes #138  by @TechSpiritSS
Types of changes

[x] Bug/Hot fix (non-breaking change which fixes an bug)
[ ] New feature (non-breaking change which adds a functionality/feature)
[x] Refactoring Old Code ( non-breaking change replacing already existing code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

This issue was caused because of the use of latest ES6 feature assert, which is not supported by all the browsers at the moment especially non-chromium based browsers.
I replaced that implementation with the old widely accepted import statement, by making minimum changes required.
It is advised that futures coders don't implement the beta features in the code.
Preview Link

https://anuragshukla.xyz/Terminal-Portfolio/

Checklist:

[x] I have read the CONTRIBUTING guidelines in repository.
[x] My pull request has a descriptive title and is attached to an existing issue.
[x] I haven't repeated the code and it follows established code style of repository.
[x] My branch is up-to-date with upstream branch.

Great 🚀


SW-3932 Update editable map instructions

Based on user feedback, update the instructions for the planting site map editor
to tell people to zoom in first, and move the instructions above the map.


Disable codecov temporarily until it works again

See link for more details regarding Codecov error:
nus-cs2103-AY2021S1/forum#374

Bug appears to have gone, PR is closed.


[QUESTION] Deployment with Traefik

I managed to configure Traefik with Docker but how can I make my FastAPI application point to it? I already can access Traefik and Consul UI, but I'm unable to access my application on port 8000 with SSL.

You should terminate SSL at Traefik and then traefik will forward the request to your fastapi container as http.  You can add something like the following (see the labels section):
networkstore:
  container_name: pynetworkstore
  image: biodati/pynetworkstore:localdev
  build:
    context: ./pynetworkstore
    dockerfile: ./docker/Dockerfile
  environment:
    - LOGGING=INFO
  depends_on:
    - arangodb
    - elasticsearch
  command: ["/start-reload.sh"]  # Used for development only
  volumes:
    - ./pynetworkstore/app:/app
  networks:
    biodati:
  labels:
    - traefik.enable=true
    - traefik.backend=networkstore
    - traefik.frontend.rule=Host:networkstore.${BS_HOST_NAME:?err};
    - traefik.port=80

The traefik labels tell traefik to route traffic for networkstore.example.test:80 to the networkstore container.

These types of questions are a better fit for the Gitter FastAPI channel (https://gitter.im/tiangolo/fastapi) or the Traefik Slack channel.

Thanks @wshayes for the help!
@marcosmmb you can check the project generators, they have everything already prepared to be connected to Traefik: https://fastapi.tiangolo.com/project-generation/

@tiangolo I found a more interesting solution for me. I'm using Caprover, which works as a self hosted Heroku, so I don't need to worry about all that stuff. Since Caprover works with docker images, it is pretty easy to launch a FastAPI container with it. If you like, I could PR a tutorial for that.

Great @marcosmmb ! Thanks for reporting back about your solution. I think that would make an interesting blog post :wink:
As I see you already solved your problem, I'm going to close this issue now. But feel free to add more comments or issues.


Compare constraints

I have the following code:
sectionView?.snp.updateConstraints { make -> Void in
    make.width.equalToSuperview()
    make.height.equalTo(SECTION.kHeaderHeight)

    if sectionHeader.scrollPositionOffset != sectionHeader.headerPositionOffset {
        make.top.equalTo(safeAreaInsetsTop + sectionHeader.scrollPositionOffset)
    } else {
        make.top.equalTo(safeAreaInsetsTop + sectionHeader.headerPositionOffset)
    }
}

which gets repeatedly called in updateViewConstraints().
To optimize it, I'd like to compare if the width and height constraints are already correctly set so that I don't have to call make on them again, and simply update the top constraint alone.

@sedwo if you need high performance code I recommend storing a reference to a constraint the first time you created it:
self.topConstraint = make.top.equalTo(...)
and then in your updateViewConstraints code:
self.topConstraint.constant = 5


lib: provide an "extended lib" to our modules

lib/extend-lib.nix returns a nixpkg's lib, extended with our own helpers available as lib.nixvim.
The extended lib also merges our maintainer list into lib.maintainers.
This is exposed as helpers.extendedLib, but when evaluating our modules it should be assigned to specialArgs.lib.
Outside of our modules you must still access our helpers via config.lib.nixvim or config.lib.nixvim.extendedLib.
Within helpers' sub-sections, lib is the extended lib.
I added a test, but currently it is only run with a standalone build. For completeness it'd be nice to also run the test with out other module wrappers, but I'm not sure the best way to go about this.
This PR makes no restructuring changes to the lib's layout, as discussed in the doc. #1962 is a first step towards doing that, but also makes no breaking changes.

Currently this is just an alternative way to access helpers, however I think new code should prefer using lib.nixvim.
Should this PR update documentation and CONTRIBUTING or should we do that after some incubation time?
If we ever get around to migrating all internal usage of helpers to lib.nixvim then perhaps we can add a deprecation warning to helpers. I don't think this should be a priority though.

I think we should indeed change the mentions from helpers to lib, as it is more inline with what the other projects are doing, like HM.
Refactoring the entire codebase to get rid of helpers should not be too hard, we already did it once (twice maybe?) to refactor how they were imported

Changes look good to me, and I am fine with the change.
Thanks !

@mergifyio queue

Thank you both for your reviews!

I think we should indeed change the mentions from helpers to lib, as it is more inline with what the other projects are doing, like HM.

Agreed! As well as updating the docs, there's probably other minor polishing to be done in follow up PRs. E.g. we may not need to explicitly set specialArgs.lib if we use extendedLib.evalModules.

Refactoring the entire codebase to get rid of helpers should not be too hard, we already did it once (twice maybe?) to refactor how they were imported

If we can do it without much effort that's great! I just didn't want migration to be a blocker for the initial refactoring.


fix line 182 indentation

@ramimsft
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

Thanks for your contribution!
#sign-off


InitialPlaybackTime and EndPlaybackTime does not work.

InitialPlaybackTime and EndPlaybackTime does not work.
The duration bar show the right play time from my setting.
But the player does not start and end from my setting.
Any suggestions?

Please see my answer in issue #155.


Do not check whether tx is coinstake in PoW networks

This bug prevents accepting btc testnet block 1439021, thereby freezing IBD at block 1439020. The tx 55 is the culprit.

@yegorpetrov So this is an issue when downloading BTC test net?

@yegorpetrov So this is an issue when downloading BTC test net?

Yep, it's btc testnet. I didn't really try any mainnets, and the only other network I did try was litecoin testnet, which synced successfully prior to this fix (I haven't published LTC support code yet). And so with this fix, the btc testnet resumed syncing.
May be the definition of the IsCoinStake property is also a problem, but seeing how the transaction class does not have consensus context there, I don't see any better way to fix this.

I can see why this occurs, although I think it may be better to separate the PoS-specific logic out of the rule into the PoS version instead of using IsCoinStake in a PoW context

This is the condition for a POS trx
        public bool IsCoinStake
        {
            get
            {
                return this.Inputs.Any()
                    && !this.Inputs.First().PrevOut.IsNull
                    && this.Outputs.Count() >= 2
                    && this.Outputs.First().IsEmpty;
            }
        }

And for ouputs
        public bool IsEmpty => (this.Value == 0 && this.ScriptPubKey.Length == 0);

I always thought that an output with empty script and not value is not allowed on BTC

Confirmed for smart contracts too. Will leave it to you to merge @dangershony.  Thanks @yegorpetrov

I have synced testnet on both Stratis and Bitcoin this is good to go, however we need to merge conflicts.
@yegorpetrov if you manage to merge conflicts today please do otherwise I will to it later and push.

I have synced testnet on both Stratis and Bitcoin this is good to go, however we need to merge conflicts.
@yegorpetrov if you manage to merge conflicts today please do otherwise I will to it later and push.

@dangershony I've resolved the conflict.


Newly Created Repo Throws Not Found Error When Trying To Download Image

Expected behavior
When clicking the "Create" button under one of my newly created repos, Docker should download the image.
Actual behavior
I get an error message
(HTTP code 404) unexpected - {"message":"manifest for canis123/keychainatopservice:latest not found"}
Information about the Issue
Running Windows 10 Pro
Client:
Version:      17.09.0-ce
API version:  1.32
Go version:   go1.8.3
Git commit:   afdb6d4
Built:        Tue Sep 26 22:40:09 2017
OS/Arch:      windows/amd64
Server:
Version:      17.09.0-ce
API version:  1.32 (minimum version 1.12)
Go version:   go1.8.3
Git commit:   afdb6d4
Built:        Tue Sep 26 22:45:38 2017
OS/Arch:      linux/amd64
Experimental: false
Steps to reproduce the behavior

Create new private Repo.
Open kitematic.
Click the "+ New" button.
Click the "My Repos" button.
Click the "Create" button on my private repo.

Without having access to the image, it's quite difficult to figure out what actually happened.
Can you see the private repo when you go to:
https://hub.docker.com/r/canis123/keychainatopservice/
What tags are available under:
https://hub.docker.com/r/canis123/keychainatopservice/tags/

@FrenchBen , I can see the private repo. There are no tags available under:
https://hub.docker.com/r/canis123/keychainatopservice/tags/

@FrenchBen , were you able to reproduce my error by following the steps I posted?

You need to have the latest tag available, in order for Kitematic to pull the latest.

@FrenchBen , how do you add the "latest" tag in Kitematic?

@nathvi you push an image to the repo with a tag of latest


[WIP] Wos resilience

:+1:
is 2018-01-01+2021-12-31 the WOS syntax for a range? I'm assuming the + is due to the - in the dates, just looks odd.

@aaron-collier Yep:

This element specifies a range of publication dates. If publishedTimeSpan is used, the loadTimeSpan parameter must be omitted. If publishedTimeSpan and loadTimeSpan are both omitted, then the maximum time span will be inferred from the editions data. Beginning and end dates should be specified in the yyyy-mm-dd format separated by '+’, ex: 1993-01-01+2009-12-31.

From https://api.clarivate.com/swagger-ui/?url=https%3A%2F%2Fdeveloper.clarivate.com%2Fapis%2Fwos%2Fswagger%3FforUser%3D6465b3d808e4d901b25323794b69dc0540441355


Configure sample gemini tests

Installation:
npm install
npm install -g gemini
npm install -g selenium-standalone
selenium-standalone install

Start selenium-standalone start in separate terminal tab
Run gemini update to update test suite and generate base images
Run gemini test to perform testing against base images
See gemini-reports/index.html for detailed report

We have selenium-standalone installed in the project with locked browser driver versions.
yarn selenium essentially does selenium-standalone install && selenium-standalone start 😉

No need to keep it. Just keeping the branch


Make the coded variable file external

Would speed up development somewhat, .gitignore isn't behaving for me...

Fixed by saving the template at homeserver_variables.py.sample, simple!
https://github.com/PC-Admin/matrix-moderation-tool/commit/401206c8c5b8e8b651d1117d7dc9577f32e915e2


Improve collection of user data

Collect all appropriate information from users, including first, last name, title, etc.
May need to update enhancement request form accordingly.

Current fields:

ID
name
role
username
password

Will remove password in #17.

Need to modify create_update_user() in db.py to use first and last name instead of single name field, and deprecate name field.

create_update_user() is updated for user creation. Name field is deprecated. Issue #23 covers other changes related to name field deprecation.
Email field is added to registration form.


dependsOnOptions compatibility with nova-flexible-content

Hello @Tarpsvo
I try multiselect with nova-flexible-content, just nothing happend when use dependsOnOptions, something i can look at? thanks for all!
Maybe it has something to do with the field index?

Hi! I added compatibility for dependsOn and Flexible in version 1.9.6. Check it out! Good luck!


Notify users of comments on sessions they have commented on

Notify not only the authors of the session of a comment in a session but also the users that have commented on that session as well.
This is probably something we want to enable by default but allow to remove. Something like 'watch'/'unwatch' for github.

@raphaelmolesim: Are you waiting for something to pull this in? Should this issue be closed? Or do we want the full 'watch' implementation like for github?

I'm closing this one since we have an OK implementation for now. A better solution might be #230 whenever it gets picked up.


fix issue #997 by forcing and

#997 Fix

@kYann I'm gonna get this right now, but add a regression test. Next time though, can you add a test demonstrating the problem and locking down the problem?

Got it in with a rebase.


To Build a MTR Mod Wiki and download station in China Mianland

Hello Jonathan,I am a Minecraft server owner in China Mainland.I want to build a MTR Mod Wiki and download station in China Mainland,because use offical link to download MTR Mod is slow in China Mainland.And many people don't know how to use the mod.So I want to build a MTR Mod Wiki and download station in China Mainland.
Hope you agree.
Best regards.

你其实可以用中文的（
而且不一定需要用 Wiki 这种动态站点。听我的，用 VuePress + GitHub Pages（

我已经收到你的邮件，会及时查看 --可爱的铁仔

There are already many Chinese sites for this mod, please see if those work for you.


Can't uninstall global npm package

I'm trying to uninstall a package with npm uninstall -g <package> but it's not removing it from .nvm/versions/node/v6.0.0/bin/<package>

What's nvm debug and npm root -g say? When you installed that package, did you use sudo? (you never want to use sudo with nvm)

$ nvm debug
nvm --version: v0.31.0
$SHELL: /bin/bash
$HOME: /Users/cameronroe
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
nvm current: v6.0.0
which node: $NVM_DIR/versions/node/v6.0.0/bin/node
which iojs:
which npm: $NVM_DIR/versions/node/v6.0.0/bin/npm
npm config get prefix: $NVM_DIR/versions/node/v6.0.0
npm root -g: $NVM_DIR/versions/node/v6.0.0/lib/node_modules

$ npm root -g
/Users/cameronroe/.nvm/versions/node/v6.0.0/lib/node_modules

Try chown -R "$(whoami)": "$(npm root -g)" (you may need sudo for it) and then npm uninstall -g <packagename> again?

I think it removed it, but the shell didn't update after uninstall.

Now I get this on login: -bash: /usr/local/Cellar/nvm/0.31.6/nvm.sh: No such file or directory

Ah, "Cellar" - did you previously install nvm with homebrew? If so, you may want to remove all of the nvm-related lines from your profile files, and then rerun the install script in the readme.

Yeah, I did though.

hmm - that line must exist somewhere, otherwise it wouldn't try to source that file :-/

Okay, so I had some strange configuration because now that I've removed the brew version, I still have a .nvm and it's giving me this when I try to install.
=> Downloading nvm from git to '/Users/cameronjroe/.nvm'
=> fatal: destination path '/Users/cameronjroe/.nvm' already exists and is not an empty directory.
Failed to clone nvm repo. Please report this!

Ah, that's fixed in 68bf93514bdf5d2852ce74819350bed99eb472a6 but not yet released in master. Try curl -o- https://raw.githubusercontent.com/creationix/nvm/68bf93514bdf5d2852ce74819350bed99eb472a6/install.sh | bash to work around that for now?

I did that and I get this:
$ nvm
-bash: nvm: command not found

After restarting your terminal?
Can you verify the nvm-related lines in one of your profile files, and in which one?

I have a .bash_profile with:
# nvm/node
export NVM_DIR=~/.nvm
export NODE_ENV='development'

.bashrc with:
export NVM_DIR="/Users/cameronjroe/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm

$ which nvm gives nothing

ok so, in bash_profile - remove that NVM_DIR line entirely, the one in .bashrc is correct. which nvm isn't supposed to give anything, because nvm is a shell function and not a binary. type nvm should, however.

still nothing

opened a new shell too

I also keep seeing this in new shell windows on login -bash: /usr/local/Cellar/nvm/0.31.6/nvm.sh: No such file or directory

Can you gist the entire contents of .bash_profile and .bashrc? Feel free to asterisk out any sensitive parts.

Also, make sure to run brew uninstall nvm

I did find this source $(brew --prefix nvm)/nvm.sh

Ah, please remove that line :-)

Ah, okay turned out my .bashrc wasn't sourced in my .bash_profile

that'd do it :-) you definitely want to add that also.

Would this be messing it up in .bashrc -> [ -n "$PS1" ] && source ~/.bash_profile

yes, generally you want bash_profile to source bashrc, not the other way around.

Awesome, got it. Thanks!

np, let me know if it's still not all working!

Not sure whats wrong with my npm but I can not use any packages i install using npm install -g <package>. Can someone please help
root@pace:~# nvm debug
nvm --version: v0.33.8
$SHELL: /bin/bash
$HOME: /root
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 4.3.48(1)-release (x86_64-pc-linux-gnu)'
uname -a: 'Linux 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux'
OS version: Ubuntu 16.04.3 LTS
curl: /usr/bin/curl, curl 7.47.0 (x86_64-pc-linux-gnu) libcurl/7.47.0 GnuTLS/3.4.10 zlib/1.2.8 libidn/1.32 librtmp/2.3
wget: /usr/bin/wget, GNU Wget 1.17.1 built on linux-gnu.
git: /usr/bin/git, git version 2.7.4
grep: /bin/grep (grep --color=auto), grep (GNU grep) 2.25
sed: -e expression #1, char 9: Unmatched ) or \)
awk: , GNU Awk 4.1.3, API: 1.1 (GNU MPFR 3.1.4, GNU MP 6.1.0)
sed: /bin/sed, sed (GNU sed) 4.2.2
cut: /usr/bin/cut, cut (GNU coreutils) 8.25
basename: /usr/bin/basename, basename (GNU coreutils) 8.25
sed: -e expression #1, char 9: Unmatched ) or \)
rm: , rm (GNU coreutils) 8.25
sed: -e expression #1, char 9: Unmatched ) or \)
mkdir: , mkdir (GNU coreutils) 8.25
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.7.0-git
nvm current: v8.9.4
which node: $NVM_DIR/versions/node/v8.9.4/bin/node
which iojs:
which npm: $NVM_DIR/versions/node/v8.9.4/bin/npm
npm config get prefix: $NVM_DIR/versions/node/v8.9.4
npm root -g: $NVM_DIR/versions/node/v8.9.4/lib/node_modules

You seem to have a node_modules folder in your home dir; I’d delete that. Either way tho, please file a new issue (and fill out the issue template) instead of commenting on this old, closed one, and I’ll be happy to take a look.

https://github.com/creationix/nvm/issues/1945


Broken link to the documentation in yt/cpp/mapreduce/interface/operation.h

This line https://github.com/ytsaurus/ytsaurus/blob/215800c48f5eab62e6ecac62fa59781ecb809790/yt/cpp/mapreduce/interface/operation.h#L1205 contains a link to the https://ytsaurus.tech/docs/en/user-guide/storage/static_schema.html#schema_inference but if I try to open it I get an error 404.

Thank you for the report. I have fixed it!


[on hold] Release lite models

This PR is preparing the release of our lite models. It introduces a few concepts that should make it easier for the user to switch between different models when running a demo. Demos that can be run by multiple models now includesa section like this at the top of the script:
SUPPORTED_MODEL_CONFIGURATIONS = [
    ModelConfig('StridedInflatedMobileNetV2', 'pro', ['met_converter']),
    ModelConfig('StridedInflatedEfficientNet', 'pro', ['met_converter']),
    ModelConfig('StridedInflatedMobileNetV2', 'lite', ['met_converter']),
    ModelConfig('StridedInflatedEfficientNet', 'lite', ['met_converter']),
]

indicating which models are supported by this script. It is possible to select which model to use from the command line (see --model_name and --model_version arguments). When these arguments are not used (default behavior), it will loop over SUPPORTED_MODEL_CONFIGURATIONS, from top to bottom, until it can find a model configuration for which the user has all necessary files. With the example above, this means that the script will try to load "pro" weights first and will only fallback to "lite" weights if "pro" weights were not found.
Please, also check out models.yaml. This is my suggestion to map classifier weights to their corresponding backbone weights. This is what we will have to edit when we add support to another demo.
I've left a bunch of TODOs here and there. We should also edit the README I guess.

Will fix conflicts tomorrow

Will fix conflicts tomorrow

@corneliusboehm Thanks for the review! I've addressed the issues that were easy to fix.
Moving get_relevant_weights  into ModelConfig would probably make sense and I think there are a few things that could be simplified with regards to how we handle loading weights on Travis (not sure we need this many running_on_travis conditions)...
I haven't made up my mind yet about your load_model suggestion. One disadvantage I see is that it would hide high-level information about the architecture (e.g. "the model for this example is simply a backbone network + a fully connected layer with X classes") and, as a user, I kind of liked that this was exposed in the example scripts. Not sure... Maybe we can discuss it with a larger round of people and see what they prefer?


Fix widget scrolling with magicgui>=0.5

magicgui 0.5 broke the scrollbars in cellfinder-napari. This was kind of unsurprising though, because the previous solution in cellfinder-napari was very hacky, and involved using private API in magicgui (always a bad idea...). This PR fixes that.
The fix is simple, and robust against future changes in magicgui - take the QtWidget magicgui provides, wrap it in a ScrollArea and return the ScrollArea.
Please could the reviewer manually check the scrollbar works with this PR and magicgui==0.5.1?

😱 it seems like this breaks layer selection, so moving to draft

Codecov Report

Merging #127 (c7d1cc5) into main (2a44950) will decrease coverage by 0.31%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
- Coverage   81.57%   81.25%   -0.32%
==========================================
  Files          15       15
  Lines         662      667       +5
==========================================
+ Hits          540      542       +2
- Misses        122      125       +3

Impacted Files
Coverage Δ

cellfinder_napari/detect/detect.py
48.33% <0.00%> (-3.40%)
:arrow_down:

cellfinder_napari/detect/thread_worker.py
96.15% <0.00%> (+0.50%)
:arrow_up:

Continue to review full report at Codecov.

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

I've opened an issue upstream for this: https://github.com/napari/napari/issues/4749