Workaround for missing message destination in message-pacts

A suggestion how to use the description field as workaround for the sentTo field. I thought it is similar to the translation from SC contracts to pact where sentTo is used as part of the expectation-description (in MessagePactCreator).

Codecov Report

Merging #842 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #842      +/-   ##
============================================
+ Coverage     49.68%   49.68%   +<.01%
- Complexity     2177     2178       +1
============================================
  Files           268      268
  Lines         10469    10474       +5
  Branches       1690     1691       +1
============================================
+ Hits           5201     5204       +3
- Misses         4441     4442       +1
- Partials        827      828       +1

Impacted Files
Coverage Δ
Complexity Δ

...rifier/spec/pact/MessagingSCContractCreator.groovy
33.33% <100%> (+2.29%)
3 <0> (+1)
: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 2fdb990...9a50ac7. Read the comment docs.

Codecov Report

Merging #842 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #842      +/-   ##
============================================
+ Coverage     49.68%   49.68%   +<.01%
- Complexity     2177     2178       +1
============================================
  Files           268      268
  Lines         10469    10474       +5
  Branches       1690     1691       +1
============================================
+ Hits           5201     5204       +3
- Misses         4441     4442       +1
- Partials        827      828       +1

Impacted Files
Coverage Δ
Complexity Δ

...rifier/spec/pact/MessagingSCContractCreator.groovy
33.33% <100%> (+2.29%)
3 <0> (+1)
: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 2fdb990...9a50ac7. Read the comment docs.


Comment message tweak

Current dependencies on/for this PR:

main

PR #99   👈

This comment was auto-generated by Graphite.


Nothing Related to IOS Development

@Aviral190694 some help here?

@yash0707 @saranshkataria I will surely add some resources here for quick start by end of this week. Thank you for creating the issue @yash0707

Some resources for iOS have been added, this issue can be closed now


Try to fix a test

Looks like files might not be sorted.

This was separately addressed in a now-merged PR from @meffij.


FloatingActionButtonVisibilityValue not change when showBottomSheet Calling

Is there an existing issue for this?

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

Steps to reproduce

use showBottomSheet and DraggableScrollableSheet
set FloatingActionButton in Scaffold

Expected results

When using showBottomSheet and DraggableScrollableSheet, FloatingActionButton Visibility change.

Actual results
Currently FloatingActionButton Visibility not change and BodyScrim not also change.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(useMaterial3: true),
      home: const ExampleScreen(),
    );
  }
}

class ExampleScreen extends StatefulWidget {
  const ExampleScreen({super.key});

  @override
  State<ExampleScreen> createState() => _ExampleScreenState();
}

class _ExampleScreenState extends State<ExampleScreen> {
  final _scaffolKey = GlobalKey<ScaffoldState>();

  void _showHomeDraggableScrollableSheet() {
    _scaffolKey.currentState!.showBottomSheet(
      clipBehavior: Clip.antiAlias,
      (context) {
        return DraggableScrollableSheet(
          snap: true,
          expand: false,
          minChildSize: 0.0,
          maxChildSize: 0.9,
          initialChildSize: 0.9,
          snapSizes: const [0.0, 0.9],
          builder: (context, scrollController) {
            return Scaffold(
              body: CustomScrollView(
                controller: scrollController,
              ),
            );
          },
        );
      },
    );
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      key: _scaffolKey,
      appBar: AppBar(title: const Text("Example")),
      floatingActionButton: FloatingActionButton(
        onPressed: _showHomeDraggableScrollableSheet,
        child: const Text("Open"),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
octor summary (to see all details, run
flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.2, on
    macOS 13.5.1 22G90 darwin-x64,
    locale fr-CI)
[✓] Android toolchain - develop for
    Android devices (Android SDK
    version 33.0.2)
[✓] Xcode - develop for iOS and macOS
    (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.81.1)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

Hi @allou-deyforyou, the code sample that you have provided is not complete. Please provide a complete minimal reproducible code sample along with detailed steps to reproduce so that we can investigate this issue. Please also provide any screenshots/recordings that capture the issue.
Thank youj

Hi @danagbemava-nc, I just modified the code. You can try.
Thank you

Hi, @allou-deyforyou, your description of the issue is quite vague. Please provide a detailed description of the issue. Screenshots/recordings will help in this case.
If you could also share anything that supports your expected case, that would be helpful.
Thank you


Remove code.interact or replace with raise Exception

Program running in infinite loop which can handle normal exception will be disrupted by the code.interact and stop running. Removing code.interact completely or add new parameter to suppress this and raise exception instead.

Added environment variable INTERACT_AFTER_ERROR to suppress code.interact and raise error as normal with fa22f3e48490f3c09e136d2fe8bf9864da68d12c


Tool:AndroidOS/ZkarletFlash trigger from system informer

Brief description of your issue
Getting warning from microsoft defender about "Tool:AndroidOS/ZkarletFlash"

What is this?  Is this a false positive?  I've had system informer for months now.  And why is it triggering on the zip file itself and not the extracted executables?  From my knowledge this is a completely legit copy I've downloaded from the official systeminformer site.
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
Windows 10 home

I want to add that I ran the .zip file on virus total and i got 13 warnings.  Is this version of the file just a bit full of false positives?  What's going on here?

I want to add that I ran the .zip file on virus total and i got 13 warnings. Is this version of the file just a bit full of false positives? What's going on here?

@ak33tah
The latest version is 3.1.24305 but your screenshot shows you're uploading 3.0.7680 from 2022? Those versions would pre-date signing of the nightly branch and +2500 other changes.

Do you know what this is? Is this a false positive? And why is this triggering on the zip file itself and not the extracted executables lol?

Microsoft releasing Android signatures that flag Windows software as malicious but only zip files not the content of the zip like the executables or binaries makes no sense.
You can submit the binary to Microsoft and ask them: https://www.microsoft.com/en-us/wdsi/filesubmission
Re-open this thread if they don't resolve the issue 👍

I want to add that I ran the .zip file on virus total and i got 13 warnings. Is this version of the file just a bit full of false positives? What's going on here?

@ak33tah
The latest version is 3.1.24305 but your screenshot shows you're uploading 3.0.7680 from 2022? Those versions would pre-date signing of the nightly branch and +2500 other changes.

Hmmm idk if we're looking at the same thing but according to the source forge page v3.0.7660 is released June 2024, not 2022.

Do you know what this is? Is this a false positive? And why is this triggering on the zip file itself and not the extracted executables lol?

Microsoft releasing Android signatures that flag Windows software as malicious but only zip files not the content of the zip like the executables or binaries makes no sense.
You can submit the binary to Microsoft and ask them: https://www.microsoft.com/en-us/wdsi/filesubmission
Re-open this thread if they don't resolve the issue 👍

My only theory is that the contents inside v3.0.7660.zip file is triggering something from defender whereas the already extracted executables probably underwent changes from auto updates that no longer trigger defender.   I mean it's still weird bc that zip file never triggered anything from Microsoft defender until now.  I just want to check if its my specific pc issue or a microsoft defender as a whole issue.  When you run v v3.0.7660 do you get a "Tool:AndroidOS/ZkarletFlash trigger" as well?

Reopen

Do you know what this is? Is this a false positive? And why is this triggering on the zip file itself and not the extracted executables lol?

Microsoft releasing Android signatures that flag Windows software as malicious but only zip files not the content of the zip like the executables or binaries makes no sense.
You can submit the binary to Microsoft and ask them: https://www.microsoft.com/en-us/wdsi/filesubmission
Re-open this thread if they don't resolve the issue 👍

@dmex My only theory is that the contents inside v3.0.7660.zip file is triggering something from defender whereas the already extracted executables probably underwent changes from auto updates that no longer trigger defender. I mean it's still weird bc that zip file never triggered anything from Microsoft defender until now. I just want to check if its my specific pc issue or a microsoft defender as a whole issue. When you run v3.0.7660.zip on defender do you get a "Tool:AndroidOS/ZkarletFlash trigger" as well?

I mean it's still weird bc that zip file never triggered anything from Microsoft defender until now. I just want to check if its my specific pc issue or a microsoft defender as a whole issue.

@ak33tah Security products update their definitions/signatures all the time. Looks like a clear false positive to me. System Information has nothing to do with "AndroidOS" or "ZkarletFlash". Report it to Microsoft or ignore it. @dmex already provided you instructions.

Microsoft says there are no detections.

My only theory is that the contents inside v3.0.7660.zip file is triggering something from defender

We haven't changed anything related to the setup since last year and our zip support hasn't changed in almost 10 years.... Google search shows that signature targeting something different every month so it's something that Microsoft does to cause the issue:
https://www.google.com/search?q=Tool:AndroidOS/ZkarletFlash


Inconsistent page names

Current Behavior
Longer page names follow inconsistent naming conventions:

no separation: https://portal.dev.demo.catena-x.net/companyrolesonboardingserviceprovider
kebap-case: https://portal.dev.demo.catena-x.net/usecase-participation
snake_case: https://portal.dev.demo.catena-x.net/appreleaseprocess_form
camelCase: https://portal.dev.demo.catena-x.net/companyCertificate

Expected Behavior
Page names should follow a consistent naming convention
Steps To Reproduce
Open the definitions of the page names:
https://github.com/eclipse-tractusx/portal-frontend/blob/main/src/types/Constants.ts#L25-L107
Check the name values - also some constant keys are inconsinstent like: APPRELEASEPROCESS and APP_RELEASE_PROCESS_FORM

@manojava-gk @oyo this issue is in status in review, why are no pull requests linked?

@manojava-gk @oyo this issue is in status in review, why are no pull requests linked?

Its already merged

following link is not adjusted:
https://portal.int.catena-x.net/semantichub

Hi @evegufy, are you already considering changes in helm values for PR eclipse-tractusx/portal-frontend#895 ?
Most camel case changes do not make a difference but something like consent_osp -> consentOsp is breaking the OSP path.


Ship backup versions of templates with vsix (version 2)

These backup templates will be used if the user has no internet or the func portal api changes.
This PR also has two main differences than my original PR from last week:

Much more testing - all function unit tests are run against the production portal, staging portal, and backup files
Telemetry for each case (getting templates from portal, from cache, and from backup files). NOTE: I added a custom 'isActivationEvent' property since these events always run on extension activation and we don't necessarily want to track that for usage

The backup templates should rarely be used and we should keep them up-to-date (the unit tests help ensure this). I don't want users to even know they're using backup templates since there should be little difference.
I filed a separate issue to fix the unit tests on windows: https://github.com/Microsoft/vscode-azurefunctions/issues/239


renew credentials too often

Hi, I found that the server/writer/reader are all renew credentials very often. And the kerbs ticket cache is refresh very frequently.
Is that normal?

docker version : latest
hdfs cluster :  exist hdfs cluster

Hi @wordhardqi , could you help to answer this question?

@dmthuc  i am not familiar with this part. I spot one potential issue. Whenever we created a new hdfs filesystem, we will run kerberosInit and whenever we created a new readBufferFromByteHdfs, we will create a new filesystem. We might call the kerberosInit too often. As for why the renew request failed, @xingcici could u check the KDC log?

@dmthuc i am not familiar with this part. I spot one potential issue. Whenever we created a new hdfs filesystem, we will run kerberosInit and whenever we created a new readBufferFromByteHdfs, we will create a new filesystem. We might call the kerberosInit too often. As for why the renew request failed, @xingcici could u check the KDC log?

yeah, this is part gc job. It try to remove outdated file. As I can see here is that is is failed to remove certain file

We have found the issue with the file not found

@dmthuc i am not familiar with this part. I spot one potential issue. Whenever we created a new hdfs filesystem, we will run kerberosInit and whenever we created a new readBufferFromByteHdfs, we will create a new filesystem. We might call the kerberosInit too often. As for why the renew request failed, @xingcici could u check the KDC log?

Hi, I do not find the KDC log on byconity-server（Or can you tell me in which dir.）。And the KDC sever log shows like below.

@dmthuc i am not familiar with this part. I spot one potential issue. Whenever we created a new hdfs filesystem, we will run kerberosInit and whenever we created a new readBufferFromByteHdfs, we will create a new filesystem. We might call the kerberosInit too often. As for why the renew request failed, @xingcici could u check the KDC log?

Hi @wordhardqi , I have found the bug with part gc that that cause failed to delete file. But about the issue run kerberosInit whenever we create a new readBufferFromByteHdfs, is this normal?

what is the gc bug? If kerberos authentication is enabled, I think it is expected with kerberosInit when reading.

@dmthuc i am not familiar with this part. I spot one potential issue. Whenever we created a new hdfs filesystem, we will run kerberosInit and whenever we created a new readBufferFromByteHdfs, we will create a new filesystem. We might call the kerberosInit too often. As for why the renew request failed, @xingcici could u check the KDC log?

Hi @wordhardqi , I have found the bug with part gc that that cause failed to delete file. But about the issue run kerberosInit whenever we create a new readBufferFromByteHdfs, is this normal?

what is the gc bug? If kerberos authentication is enabled, I think it is expected with kerberosInit when reading.

The issue is that the GC try to remove the data part that already have its data file in HDFS removed. Yeah, it is expected with kerberos init when reading but i think we don't need to call it everytime. caichen is handling the issue with kerberos


Inode type adapts to the situation

The goal of this PR is to reflect in the types the inode structure in order to remove the assert false statements, to save memory and to make the code more explicit on edge cases.
I did not re-implement the runtime types in Val_impl. I felt it was very redundant with the ones in Bin.t.

I have finished implementing that PR so i'm opening it for review

I have finished implementing that PR so i'm opening it for review

I'm changing it back to Draft as it has to be rebased on #1265 and that a bug was just found that might change the design

I'm changing it back to Draft as it has to be rebased on #1265 and that a bug was just found that might change the design

#1265 is merged
#1266 will be fixed by #1283
I'm happy with the implementation

So I'm opening it back review again!

I've pushed a batch of diffs.
@icristescu you mentioned on slack that shallow may be less confusing than broken. I think broken is better because it reveals that the situation with that inode is not ideal.
In order to simplify the code, I've matched the names of the pointer types with their tag counterpart. Let me know if you disagree on that change.
type _ layout =
-  | Total : direct_ptr layout
+  | Total : total_ptr layout
-  | Partial : (hash -> lazy_ptr t option) -> lazy_ptr layout
+  | Partial : (hash -> partial_ptr t option) -> partial_ptr layout
-  | Truncated : flaky_ptr layout
+  | Truncated : truncated_ptr layout

Let's merge this one today!

Ready for merge, thanks for the reviews.
@CraigFe Merge it depending on whether you want it in the release or not.

Let's get it out there :-)


Test fixes related to changes in Jest's Fake Timer behavior

Summary
Fixed tests related to changing Jest's fake timer behavior.
And some of the test code in LogBoxData-test.js itself was incorrect so I am fixing it.
LogBoxData-test.js fails when I try to build a new local development environment and fork the React Native source to run a JavaScript test.
The error message is:
runAllImmediates is not available when using modern timers
After checking, runAllImmediates could not be used with the following modification.
https://github.com/facebook/jest/commit/71631f6bf9ccf8937f02a5ecfb64b62712b86c19
Changelog
[Internal] [Fixed] - Test fixes related to changes in Jest's Fake Timer behavior
Test Plan
Ran JavaScript Tests locally.

Thanks @makitake2013!

Thanks @rickhanlonii !
This was my first pull request. I learned a lot and it was fun, so I would like to try again!

Please do!


Blazor common sample enhancements

Over the last few preview releases, a number of great examples have been added to the text of the docs but not the common sample app.

Provide a list of suggestions for sample enhancements to management.
Enhance the sample based on feedback.
Work this issue after Review Blazor Sample code for API review changes (aspnet/AspNetCore.Docs #13787) because I think it's best to make sure that everything is technically correct across the text and code examples before working new code into the sample app.
Move in-text code examples that require highlighting into snippets and apply highlighting.

Test ping @Rick-Anderson


App rail nav item visited color issue in dark mode.

In dark mode, the FeatherAppRailNavItem applied visited color makes menu unreadable.
Seems to be dependent upon the ordering of the style loads.

Fix released in v0.12.20


[Enhancement] New codebase changes and the new direction of the tabs of the United Sets

I am looking into the codebase upgrade in United Sets. Note that I don't have any dates (as I have a lot of IRL stuff to be done)
Introduction: Debate about United Sets and Taskbar
We do recognize of the idea that some people have commented that we don't need tabs for the apps because we have "Windows Taskbar." And while it may be unexpected for one of the devs to speak out, I do agree with this argument. I sometimes feel like Alt+Tab is already getting the job done.
Therefore, I would like to actually give the purpose to United Sets. I would like to give a clear reason to use United Sets other than just an app that gives tabs that could be replicated with an already existing taskbar.
Here are the steps that have already been taken in order to give a purpose to United Sets

Splittable Tabs. This is the first step that I took to give something that was harder to replicate with taskbar. although right now support for Alt+Tab and Window group is a thing in Windows 11, but we still don't have a single icon on a taskbar that represent that group, so I would say that this is the first step towards making United Sets have a purpose
Cropping Window. This is also another feature that was added to the preview version of Untied Sets. By having the cropping window feature, user can choose to crop out the top part of the window if they wish. For example, one can hide the File Explorer's tabs or browser tabs with this feature.

Here are the plans that I have in the future:

Tabs and the Splittable, Recursive Tab System (more below)
Tab Templates (Open multiple apps in one click... and group them, split them like you were using them before saving them as the template)

Window Hoster
The current state of United Sets has made it hard to make additional changes due to the assumption that the window is linked to a single UIElement, and all are linked to a single United Sets window. This makes it hard to make changes to the core system, leads to several bugs and issues that are hard to track down, and leads to a very complex UI code.
I will make a new WindowHoster library which will overcome the weakness of the WinUI3HwndHostPlus library. This library will minimize the use of polling and relying on Windows events. It will also decouple the UI logic from the window-controlling logic, making it possible to have a window with no UI attached to it. This would be helpful in the development of setting tabs aside as well, as there were some bug reports of that system currently.
This would have a neglectable change to the end user perspective. Maybe some performance improvements but I won't make guarantees. This change is designed to help in the future development flow.
Tabs and the Splittable, Recursive Tabs System
The current tab system has the downside of being complex. The new window hoster would help reduce the complexity of the system, but wouldn't make enough impact.
Therefore, I am looking to upgrade to the new tab system that is currently being developed internally. This would provide smooth dragging tabs and a smoother multi-window experience for United Sets. This system will replace the current splittable tabs system to be more integrated. It's hard to explain with words, so this is the video prototype. Note that this system is still not finished and still needs polishing.
https://github.com/Windows-Apps-Hub/UnitedSets/assets/41405393/4af9555e-c441-4e4b-ab6e-fe93ee6d508e
The downside is that initially multi-process United Sets may not be supported (multi-window will be supported instead). However, in the long run, it might be possible to add this back).

At least for me UnitedSets has the ability to create app clusters.   I would describe app clusters as microsoft desktops except clearly in window form, so you don't need to put all other windows aside to have a group of windows.   App clusters allows me to use the same app for unrelated things and I don't have to guess (or remember if its app instance 1 or 2 in the taskbar) goes with what, as its already in a set with the other apps its related to.
App clusters also allows me to easily access all the related apps with a singular shortcut key and in a specific ui space.   I can code something in VS and instantly switch to the terminal that goes with what im working on.

It's great that the development is continuing. Thank you! 👏🏼


why causal  model flops divide 2 ？

f = 4 * batch * seqlen**2 * nheads * headdim // (2 if causal else 1)

As mentioned in the technical report, it's because with causal mask we only need to compute approximately half the entries of the attention matrix.

ok


Run tests on macOS

Things added/changed:

Run tests on macOS.

LGTM. Thanks for your contribution.


Adonis will not load in my place

Adonis will not load in the place even after I tried putting a new one and relaunching the game multiple times. No errors.

Are you positive the place is saving? Sometimes Roblox won't actually update the place. So far that's been the issue with most reports like this. If you're able to confirm the place is actually updated, and Adonis is in either Workspace or ServerScriptService when you save it, try using a test server & client in studio to see if it runs. Note: Use Test > Start Server (with 1 player) and NOT Play/Solo mode.

@Sceleratis This is an issue for a lot of games on the ROBLOX platform. Either if it's the old version or the moved development version. It takes about 10 minutes for it to load and kick only a few people with "ACLI Error". This has been rare and never happened for a while.

Right, but there's also the case where it's not loading at all. I'm trying to determine if this specific situation is an actual bug or if the script just isn't running in the first place.

I've only had issues with Adonis failing to load properly when the server was overwhelmed by scripts that are maliciously designed to put a burden on server performance. Scripts like the fire spread, RotateP spread, or scripts that are just poorly scripted with unnecessary loops can cause Adonis to load late, some clients ending up with a kick because Adonis wasn't able to communicate with the client due to flooding of Remote Events.
Be sure you throttle the use of Remote Events as one script can pull the server to an halt.


⚠ Bofrost.fr has degraded performance

In 8cdcab2, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1501 ms

Resolved: Bofrost.fr performance has improved in bdf2bf0 after 6 minutes.


Update prevent-changes-to-security-settings-with-tamper-protection.md

Added licenses which includes Intune

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7181

@JohanFreelancer9 Please copy review

@denisebmsft Please merge this PR. Thanks


Add worker master periodical connections timeout

In a cluster with multiple masters using embedded journal, when network partitioned the original leading master, leadership changes to a new leading master. The workers hang in GRPC connections with the previous leading master and don't register with the new leading master.
This PR adds the timeout to periodical connections (operations) between workers and leading masters to prevent workers from hanging forever.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11320/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11321/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11327/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11334/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11336/
Test PASSed.

alluxio-bot, merge this please.


Add Dart commands

References - https://dart.dev/tools/dart-tool

Hi, I'm Thareendra, and I'm interested in working on this issue. Can I be assigned to this issue?


Fix/share replay ref count sync

Description:
Bug fix: shareReplay with refCount sync sources shouldn't restart
Related issue (if exists):
It addresses #5548

For the moment, we've just reverted the change. There is a little more to this, as the reason subscription was assigned undefined was to 'null out' the reference to the source subscriber (the subscription), but the problem the reverted PR caused exposed a separate issue that probably needs to be looked at: sources that complete synchronously won't end up with an undefined subscription and that wasn't the intention. In short, this reference-releasing behaviour needs more thought, so we've reverted it for now.


Replace use of Spegel fork with upstream

Proposed Changes

This replaces the k3s fork of Spegel with the upstream version. This change can be done as the changes done in the fork have been merged upstream.
Types of Changes

No changes introduced, just a replacement of dependency.
Verification

Rely on existing testing of Spegel integration in k3s.
Testing

Linked Issues

User-Facing Change

Further Comments

An easy way to check the change is to look at the diff between the k3s fork and upstream. The only changes are related to changes that have been done upstream.
https://github.com/XenitAB/spegel/compare/main..k3s-io:spegel:k3s-main

LGTM, but it looks like there's a conflict. If you're up for a rebase, we're planning to go into code freeze for next week's releases on Monday.

@brandond missed the code freeze but i rebased now, and can rebase again later when it is possible to merge. Do you know when that will be possible?

Code Freeze usually lasts about 2 weeks, so likely towards the end of February.

@phillebaba FYI I might be looking into adding support for mirroring images from all registries - see https://github.com/k3s-io/k3s/discussions/9061#discussioncomment-8494097


Book Agent Setup

Add classes to store information from Google Books API that will be used by the Books Agent to output audio and text to users
Create Books.java (Books agent)

Set up Books agent to retrieve and store parameters necessary to retrieve information from Books API

Create BookQuery.java class

BookQuery objects contain information about query requests retrieved from user input and parameters from Dialogflow
Information will be used to make appropriate request to Google Books API to retrieve requested information

Create Book.java class

Book objects contain the information that will be used to create display for book requests

Create BooksUtils.java class

Fetches information from Google Books API based on BookQuery objects (containing info about query requests)
Returns an ArrayList of Book objects that can be used by agent to build responses and displays

will be fixed in follow up


[PRP-347] Participant Documentation

Ticket
https://wicmtdp.atlassian.net/browse/PRP-347
Changes

What was added, updated, or removed in this PR.

Documentation pages in /docs/participant
New documentation pages rendered in storybook
Breadcrumbed form page flow in Storybook

This is looking great! I've gone through and made a number of edits and suggestions. In addition, could you add documentation for these items:
* Document how local agencies are created in the database and how to replace those with the correct ones for "your WIC state agency"

* Document how to insert pages or remove pages from the routing structure

* How to change the session timeout

Added new readme file docs/participant/application.md to talk about routing, database, internationalization, and session


get_user call hangs when accounts.db already exists

Hello, I would like to thank for this wonderful library,
while testing it today, I've came to this case: when running code in the
first run get_user_by_id call works correctly and return user information, and as I see accounts.db
sqlite db is created.
However, when I try to run the code 2nd time I see no login is being made, so I assume account session from
accounts.db is being reused, however call to  get_user_by_id is never returned.
When enabled DEBUG mode, I see this looping:
No accounts available for queue 'UserByRestId' (sleeping for 5 sec)

@0xGhostCasper
Hi. Most likely the cause is the same as in https://github.com/vladkens/twscrape/issues/7. I added a way to see accounts status through the CLI.
Don't forget to update:
pip install twscrape --upgrade

Old issue, closing it.


UI Adjustments for MoonID

Artistic —> Artisticity

Score of Balance -> Treasure Score
Item -> Type
Balance: Wallet Balance
Points Calculation Instructions -> Score Calculation Instruction

Score of Transaction: Vitality Score
Item -> Type
Number of Transactions(DEX) -> Number of Transactions (DEX)  [Space before parentheses]
Number of Transactions(except DEX) -> Number of Transactions (non-DEX)

Rule -> Value
Score Items -> Type
Balance -> Wallet Balance
Metal -> Badges for a Metal status

update the description of items

fixed


affected/package: 1.18 Generics when using interfaces as constraints

Go version:
go version go1.18beta1 windows/amd64

Does this issue reproduce with the latest release?
No if the latest release is 1.17.
What operating system and processor architecture are you using (go env)?
Windows 11 - AMD x64
What did you do?
When creating interfaces like the following:
type RepositoryEncodable interface {
	ToRepository() RepositorySnapshot
	FromRepository(input RepositorySnapshot) error
}

And implemented in any struct like this:
func (r Race) ToRepository() repository.RepositorySnapshot {
	// Some code
}

// Here I need a pointer to race because I work with struct's properties
func (r *Race) FromRepository(input repository.RepositorySnapshot) error {
	// Some code
	return nil
}

If I try to use generic constraints in a method, for example:
func ReadToNStructFrom[S RepositoryEncodable](s RepositorySnapshot, fieldName string, dest *Nullable[S]) error {}

When I try to call the method using Race as S type param, it reports the following compile time error:
"data/models/ordering".Race does not implement repository.RepositoryEncodable: wrong method signature
	got  func (*"data/models/ordering".Race).FromRepository(input repository.RepositorySnapshot) error
	want func (repository.RepositoryEncodable).FromRepository(input repository.RepositorySnapshot) error

But if I remove the *Race pointer from this method:
func (r Race) FromRepository(input repository.RepositorySnapshot) error {
	// Some code
	return nil
}

No compile time error is shown.
What did you expect to see?
I expect to see no compile time error.
What did you see instead?
A compile time error shown above.

Hi, would it be possible for you to provide the full definition of the types involved, perhaps in a playground example?(play.golang.org -> choose 'Go dev branch' in the dropdown next to Run)
CC @griesemer

@tomasweigenast the FromRepository method has a pointer receiver, which means it is only available to on values with pointer type. I believe this error message has been improved since the beta1: https://go.dev/play/p/_ZPAC8Pk_xx?v=gotip
Could you please try your code with the Go 1.18 release candidate?

Even if the error message is not great, at this point this will have to wait for 1.19 assuming everything else (but the error message) is behaving as expected.
In any case, if you can provide a minimal, self-contained example that is always best, as it will make it much easier for us to actually take action. Thanks.

@griesemer I think this has already been addressed since beta1. Marking this as WaitingForInfo. Barring something I'm missing, I think we can close this.

I agree.  I don't see anything to do here on tip.  Please comment if you disagree.

Even if the error message is not great, at this point this will have to wait for 1.19 assuming everything else (but the error message) is behaving as expected.
In any case, if you can provide a minimal, self-contained example (ideally single file, or at least playground link) hat is always best, as it will make it much easier for us to actually take action. Thanks.

Can you provide a link to see what is coming for Go 1.19?

@griesemer I think this has already been addressed since beta1. Marking this as WaitingForInfo. Barring something I'm missing, I think we can close this.
There is nothing can I do to make this work?

Can you provide a link to see what is coming for Go 1.19?

Things may change, but currently there are no features currently planned for 1.19.  Work will be focused on fixing bugs, improving error messages, and speeding up compiling and execution time.

There is nothing can I do to make this work?

People have tried to explain above that the problem is that a pointer method is only available on the pointer type.  There is nothing you can do to change that.  I don't know what the larger problem is.  You may have to specify type arguments explicitly, using the pointer type.  Or you may have to pass a pointer.  Or it may help to take a look at https://go.googlesource.com/proposal/+/refs/heads/master/design/43651-type-parameters.md#pointer-method-example.  I don't know.  Sorry.

What I did is to change method's signature trying to avoid using generics. Thank to everyone.


Add NF_Track/TakeFX_Get/SetOffline API functions

Closing this, because I made some additions to this branch so the branch name doesn't reflect the content properly anymore. And apparently it's not possible to rename a branch under PR.


State does not update properly?

Renamed my TV from Samsung QN95B to just Samsung and since then I have been having problems with state not working as it should. Removed the TV from Home Assistant and added it once again. Still the same issue. Tried switching from WiFi to Ethernet and still having the same problem.
I don’t know why this started happening, and how to solve it?
It was working good before!

Same here, cannot update the tv on status since last 4 days. I have tried to change port settings but no luck. Seems there is a problem with the power on status.

Same here... the power state hasn't been updating anymore. I have a Samsung S95C, and until now it worked without any problems. However, after updating Home Assistant, the media player entity shows as turned off all the time. The only way to bring it back on is to restart Home Assistant.

NOTE:  This proposes using an alternative integration.  I realize this does not solve the problem with THIS integration, but it MAY work with this integration if you execute similar steps.  I haven't tried (yet).  I just wanted something that works for now and am posting this because someone else might as well.
Device: Samsung QN85B
I started having this problem after updating my Home Assistant and Home Assistant OS versions recently ( ~ last week or so ).  My status stopped updating when the tv was turned on/off.  Ultimately I ended up switching back to the Samsung integration, which had its own set of annoyances but is working ( for now, at least ).
After switching to the Samsung integration, I kept having an issue where notifications kept popping up for permission requests when (presumably) the state was being polled.  This kept happening even after my HA instance was added as an allowed device and notifications were turned off.  Here are the steps as I took them if anyone is interested.  I just did this today so there could be unnecessary steps included.  ( I didn't have issues beyond the tv connection, but you may want to back up your HA configuration before doing this just to be safe )

Uninstalled the Samsung Tv Smart integration
On my Samsung tv, navigated to Settings -> connection -> External Device manager -> Device Connect Manager -> Device List
Deleted the Home Assistant device entry
Restarted Home Assistant
On the Samsung Tv, went back to the device connect manager and set "Access Notification" to "First Time Only"
Installed the Samsung Smart Tv Integration ( be sure to keep the entity name(s) the same so you don't have any issues with any existing functionality related to your tv )
Multiple notifications requesting permission(s) popped up on the tv, which I allowed
Went back to the Device List and verified that Home Assistant showed up as an allowed device
In the Device Connect Manager, I set the "Access Notification" to "Off"
Restarted Home Assistant
Was prompted for permission one more time ( not sure why )
Turned off the tv and verified that the on/off state updated in HA
Turned on the Tv and verified that the on/off state updated in HA without being prompted for permission

I understand that this is just an alternative and not a solution or even a workaround using this integration, but if you just want something that works for now, I suppose it's worth a shot. 🤷🏿♂ Good luck!  👍🏿
I'll report back if I manage to figure out a way to get it working using the Samsung Tv Smart integration

In the last few days I've been seeing different behaviour for my Samsung Frame.  Previously the media_player would have a status of 'on' if art mode was on. Now the media_player is off but there's a new art_mode_status which toggles on/off.  This is actually a good thing because it makes it easier to distinguish between art mode on vs TV on.
However, I've found that after the TV goes to sleep, next time art mode comes on the new art_mode_status stays off. It only works again after I've fully turned on the TV.
Not sure if this is the same issue you're reporting, and whether it is new intended behaviour or a change on the Samsung side?

I have the same problem, after turning off the TV when I turn it on again it does not update the status. I have to restart the integration

Just as an FYI, reloading the configuration does seem to update the status in the integration. Until this is fixed I've added the homeassistant.reload_config_entry Service at the beginning of all my automations I have using this integration telling it to reload the integration before running the automation to avoid any issues.

Just as an FYI, reloading the configuration does seem to update the status in the integration. Until this is fixed I've added the homeassistant.reload_config_entry Service at the beginning of all my automations I have using this integration telling it to reload the integration before running the automation to avoid any issues.

Great idea, I've used it before for different purposes, but I forgot about it. Thanks for the reminder!

I am facing exactly the same issue since the last update.

Same here. Multiple requests from HA every few seconds.

Is there any solution to this?

I’m having an issue with adjusting the volume which I think is related, I can turn the volume up or down a few ticks and then it stops responding and I have to mute and unmute for volume state to update again.

I had the same issue. But I installed Samsung smartthings integration. This gave me a reliable on/off switch for my Frame TV in HA. Then I made an automation that reload the custom TV integration every time smarthings tell HA that the TV turns on. Has worked very well so far.

Just for the record, I have 6 Samsung Tvs running through a matrix at my house and the only affected tv has been the QN85B.  The others are CU7000s, MU6300s, and NU6900s and they haven't shown any issues.  So it seems that either some models are unaffected or that this issue is triggered somehow.

Just for the record, I have 6 Samsung Tvs running through a matrix at my house and the only affected tv has been the QN85B. The others are CU7000s, MU6300s, and NU6900s and they haven't shown any issues. So it seems that either some models are unaffected or that this issue is triggered somehow.

I only have one, a QN95B, which is affected!

Same problem here.
Only work when restart integration

Same here with a Frame 2022. I’ve stopped using the integration because of that :( can’t find a solution except reloading the integration permanently…

I have the same issue: Samsung The Frame 32LSO3B (2023)

Same problem with my 2020 Frame TV

Same problem with my Frame TV.  If I send a media_player.turn_on service to the tv, it will turn it on, but the status of the tv remains off.  However, once the tv is on, if I send a "SamsungTV Smart: Set Art Mode" service, the entity remains off, but the entity attribute changes to "art_mode_status: on".  If I then send another "media_player.turn_on" service when art mode is "on", then the samsung tv entity WILL update and show a status of "on".  I.e. I am able to get the entity to update the power status, but it requires that the art mode is "on" first.  Can others with Frame TVs confirm that this method works for them?
@ollo69 does this give any insight into what might be happening?

Or are there any logs I can provide or look into for clues?

So, looks like I'm not alone on this.
My work around is to create a script to reload the integration.
If you manually reload the integration, it will show the correct entity state after that.
So, here the script or you can just add it as an action in your automation:
Replace the entity device name to your device - on mine I named it smarthingstv )
service: homeassistant.reload_config_entry
metadata: {}
data: {}
target:
  entity_id: media_player.smarthingstv

Thank you @mytasker !    This workaround is working very well for me.

I'm also seeing this issue on my S95BA. As others have noted, reloading the config correctly detects the state. Still, I would like it to update automatically because I have an energy consumption integration connected to the device's state.

Also have the mediaplayer for this integration incorrectly reporting as off when the TV is on for my S95B. Got rid of the problem by uninstalling and reinstalling all integrations related to Samsung TVs, and doing the trick with my "Allow Access" notifications mentioned above. However, when I use the Ambient mode, it breaks again. It also breaks my app buttons, which will turn the TV off if I press them. Strangely, if I press the app buttons again, they do not turn the TV on. I noticed one instance where ambient mode does not break the power status of the tv reporting correctly... If I initiate ambient mode using the app button in HA I created when the TV is already off, and then switch to another app, the powered on/off status still reports correctly. Naturally, I have get power status from SmartThings enabled.

Not stale

I seem to also have this problem with my Samsung 43 Serif. But I found a decent workaround and that is to set the TCP port in the advanced setting to 0 so it uses ICMP for power status instead.

I seem to also have this problem with my Samsung 43 Serif. But I found a decent workaround and that is to set the TCP port in the advanced setting to 0 so it uses ICMP for power status instead.

mmm but.... is 0 by default.... no?

I seem to also have this problem with my Samsung 43 Serif. But I found a decent workaround and that is to set the TCP port in the advanced setting to 0 so it uses ICMP for power status instead.

mmm but.... is 0 by default.... no?

I have the smartthing credentials so my tcp port was like 9xxx but i removed my last comment as it only worked decently for 5 min and now I'm back to square one again.

Based on what you mentioned there @aheagel
I ran the command below to check which network ports my TV is reporting in use.
nmap -sT <IP_ADDRESS>
Then, double confirmed the port using a second application (WireShark) and I could see my TV's IP address (above) constantly talking via the port:
8001 open "vcom-tunnel"
So, for everyone still having the issue, please try the port above.
If it does not work, try to run a scan against your TV IP address.
If you use the Nmap application (found at https://nmap.org) - you will be looking for a port with the description  "vcom-tunnel".
Then, go to the integration Samsungtv_Smart > click on Configure > check the box to "Show options menu" > click on Submit > click on Advanced Options > then change the TCP port to what you found - in my case 8001
That should do it.
If that is too much, just use the work around above.
@skyw33
@olythoreau, this may help you as well.
Cheers,
PS:
If port 8001 fixes everyone's issue. Then, we may be lucky to get it implemented in the next update.
Otherwise, if everyone port is different, then it would be hard, and best is to just follow the documentation that mention about it:
https://github.com/ollo69/ha-samsungtv-smart (search for "power status")

I'm also been experiencing these issues.  I just checked and I also have vcom-tunnel available at port 8001.  I updated the configuration as recommended and everything is working fine so far (still works through multiple ON/OFF cycles).   Will report back as I have a chance to use it more.  My problem with losing state has definitely been intermittent; hard to pin down even.
One thing I noticed though too while testing just now was that when reloading the configuration it seemed to revert the port (or at least it no longer showed up in the menu for me).  Not sure if this messing up anyone's testing but I thought I'd mention it just in case.

For me it's also port 8001. However, when using this port, when the TV is turned off, after a couple of minutes the power state changes to "on" and then a few seconds later to "off" again - this then happens in 5 minutes intervals which is very inconvenient if you want to use the integration for automations. Anyone else facing the same issue?

For me it's also port 8001. However, when using this port, when the TV is turned off, after a couple of minutes the power state changes to "on" and then a few seconds later to "off" again - this then happens in 5 minute-intervals which is very inconvenient if you want to use the integration for automations. Anyone else facing the same issue?

Yes this happens to me too.

Same here. I confirm that this is workaround gives you the on/off state more reliably, but every few minutes the tv shows on then off again, when it's in fact off m

This integration has been very flaky from the start.... I gave up on using it for the majority of my actions and use SmartThings' API directly: https://developer.smartthings.com/docs/devices/capabilities/capabilities-reference/
I installed a smart outlet on my TV and if the power is over 50W, then I know the TV is on and then I tell my automation to query the Smarthings API every 3 seconds. You call the API to check the TV status, or turn the device on/off....
It's so far been 100% reliable.
Example to turn on your TV:
POST: https://api.smartthings.com/v1/devices/{device_id}/commands
body = {
"commands": [
{
"component": "main",
"capability": "switch",
"command": "on"
}
]
}

I may have the wrong advice, but for those who have a problem, try this PR:
https://github.com/ollo69/ha-samsungtv-smart/pull/300
I have a Samsung The Frame 2022 connected via wifi + smartthings token. everything works perfectly

This integration has been very flaky from the start.... I gave up on using it for the majority of my actions and use SmartThings' API directly: https://developer.smartthings.com/docs/devices/capabilities/capabilities-reference/
I installed a smart outlet on my TV and if the power is over 50W, then I know the TV is on and then I tell my automation to query the Smarthings API every 3 seconds. You call the API to check the TV status, or turn the device on/off....
It's so far been 100% reliable.
Example to turn on your TV: POST: https://api.smartthings.com/v1/devices/{device_id}/commands
body = { "commands": [ { "component": "main", "capability": "switch", "command": "on" } ] }

Another zero-cost solution is to install the original Samsung integration for HA:
https://www.home-assistant.io/integrations/samsungtv
Them use power status from that integration to reload the "SmarthingsTV integration" as it offers more control
The automation below takes around 2 seconds to reload the integration.
So, it is a good/reliable option;
It only runs once as the TV is detected as ON;
and all your automations related to "SmarthingTV" may NOT require any change or,
probably a 5 seconds delay before running if the TV is OFF and you are trying to turn it ON
alias: Reload smarthingstv integration
description: ""
trigger:
  - platform: state
    entity_id:
      - media_player.livingroomtv
    from: "off"
    to: "on"
condition: []
action:
  - metadata: {}
    data: {}
    target:
      entity_id: media_player.smarthingstv
    action: homeassistant.reload_config_entry
mode: single

I may have the wrong advice, but for those who have a problem, try this PR: #300
I have a Samsung The Frame 2022 connected via wifi + smartthings token. everything works perfectly

I just copied the file, and I'll give it test as well @kamashkin

Yes, I've also manually copied the file from #300. I haven't had it running for too long yet but I think at least the power state isn't endlessly toggling in HA now.

I also tried this PR for some time. Now, the behavior basically turned around: When the TV is turned off, the power state does not change randomly any longer, i.e. it remains off. However, when the TV is turned on, the state changes frequently...

YES! The https://github.com/ollo69/ha-samsungtv-smart/pull/300 is the fix for this one.
Working well for the past 2 weeks.

YES! The https://github.com/ollo69/ha-samsungtv-smart/pull/300 is the fix for this one.
So far no more issues and it well again.

+1 on #300 from me. I tried @KimuraFTW's steps initially which didn't work, but manually copying over the file from this PR and then repeating those steps has resolved the issue.

I have experienced the same issues, I even tried to get a template binary sensor based on the smartthings api and use this as "power on"-state. But unfortunantely this didn't do the trick (guess related to https://github.com/ollo69/ha-samsungtv-smart/issues/293).
My sensor itself is working correctly:

But it seems https://github.com/ollo69/ha-samsungtv-smart/pull/300 does the trick also for me!

This is still happening. Any chance PR #300 can get merged anytime soon?

PR https://github.com/ollo69/ha-samsungtv-smart/pull/300 fixed the updating issue for me, as well!
I manually copied the updated version of media_player.py from PR https://github.com/ollo69/ha-samsungtv-smart/pull/300  over custom_components/samsungtv_smart/media_player.py and restarted the integration - and the state is now refreshing almost immediately upon change!  Thank you @kamashkin !
+1 to merge this change : )
As a side note, I am not able to consistently nudge the TV into art mode with this integration, even though I have the Power button switch to art mode  attribute set to true.
I also tried creating a stand-alone button card, to invoke the service in this integration, but that didn't work:
show_name: true
show_icon: true
type: button
tap_action:
  action: perform-action
  perform_action: samsungtv_smart.set_art_mode
  target: {}
entity: media_player.ha_samsungtv_smart_living_room_frame_tv
name: Art Mode On

What DID work was using the frame_art integration. So, for those looking for a HA based solution for when a CEC signal (from an Apple TV for example) powers off a 2024 Frame TV (instead of putting it into ART mode like the 2022 version I have does) - check out this "Proof of Concept" automation I put together.
Note: I am new to Home Assistant, so please take my suggestion with a "grain of salt" - and also feel free to suggest other ways around this or improvements to my automation.
Also, I have previously been able to create the same automation directly in "Smart Things" routines, but I wanted to see if could get it to work in Home Assistant, and then add some more conditions and triggers and such...


[1.4.5] Right click functions for off hand items blocked.

Just Dire Things version
1.4.5
Minecraft Version
1.21.1
(Neo)Forge Version
21.1.71
Modpack & Version
No response
Do you have optifine or Rubidium installed?
I have other rendering based mods installed
Describe the issue
Right-clicking with JDT tools in main hand does not right-click use item in off hand.
Steps to reproduce

Install or update JDT to version 1.4.5 in your modpack.
Attempt to right-click use/place item from offhand while JDT tool is in main-hand. (Found with Blazegold tier in survival world. Tested with all 4 tiers in test world.)

Expected behaviour
Items to be used normally from off hand.
Screenshots
No response
Log files
No response
Additional information
This issue is new as of version 1.4.5.

Thanks for the report ;). I changed some mechanics to this in 1.4.5 and must have added a bug.
You mean like placing a torch from your off hand right ?

Yes, tested with both placing a torch and casting spells from Ars Book in the offhand.


chore: update vitest to v2

Description

Updates vitest to V2 in root package.json
Adds vitest as dev dependency in core

Type of change

[X] Chore (non-breaking change that addresses non-functional tasks, maintenance, or code quality improvements)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[X] Code in this PR is covered by automated tests (Unit tests, E2E tests)
[X] My changes generate no new warnings
[X] I have reviewed my own code
[X] I have filled out all required sections

@tomiir should we add "@vitest/coverage-v8": "2.0.5" to wallet package?


top level /inscription_query route

Closes https://github.com/ordinals/ord/issues/2807

@raphjaph I didn't realize this issue https://github.com/ordinals/ord/issues/3099 when I made this PR. Do we want to do more than just the inscription on the home route?
It would be pretty easy if we redirected like /search is doing. Or do we want to keep people on the route for issue 3099? Like if I go to /:tx I wouldn't get rerouted to /tx/:tx?

Going to close this for now, but we should revisit the generalized search idea


By default split?

I've been thinking that we should by default split, and behave similarly to the builtin help command. Except if you use a bang, it opens in the current window just like it does now.
@lervag what do you think? (sorry I have no one else to discuss this with)

Emulating :help behavior sounds like a good thing to do. :+1:

Actually, you should emulate :Man which opens the result in a new tab in NeoVim.

@sunaku i find that super annoying. Thats part of the reason I created this plugin. I much prefer the way :help works to :Man.

Ah, thanks for explaining. :sweat_smile:

I agree, it should split similar to :help by default. I would say bang should not open in current window, it should instead open in current Neoman window if any exists, else open in a split window.
You could add an option that specifies how it splits. This would allow @sunaku and others to instead open in new tab.

I'll add that all of that with mappings for splitting. Should I rename the command Neoman to simply Nman?

Both names work fine for me (if I end up using this plugin, I will use mappings). Neoman sounds better, whereas Nman is shorter.

Alright for now I just renamed it to Nman and added a notice on the readme.

Sounds good to me.

What do you guys think of the new commands now?

I think it works very nicely, thanks.

Great.


Use a public java_compile_toolchain attr instead of _java_toolchain

Description

Allow per-target selection of Java compilation toolchain.

Note: This change doesn't add similar functionality to thrift_library.
Motivation

Helps with JDK migrations by allowing targets to incrementally update.
Also allows targets to opt into newer Java syntax.
Co-authored-by: Shane Delmore<EMAIL_ADDRESS>

Thanks, @wiwa!


TextColumn.contains()

Hey.
I think its super basic use case when someone wants to get results from DB by string query.
Why there isn't somethin similar to RealmQuery.contains() in moor?

You can use the like and regexp methods, does that work for you?
Maybe we can have contains('foo') as alias for like('%foo%').

I've added support for contains which will be available in the next moor version.

Its great that we have like method, but having contains method would have some nice advantages:

Its intuitive for people that are not as much familiar with raw SQL queries.
No regex required.
You could make method that would take optional caseSensitive parameter.

I've added support for contains which will be available in the next moor version.

You could make method that would take optional caseSensitive parameter

I agree that a caseSensitive parameter would be useful, but I couldn't find a good way to implement that in sqlite. It looks like LIKE is always case-insensitive for the English alphabet, even when I add COLLATE BINARY.

I agree that a caseSensitive parameter would be useful, but I couldn't find a good way to implement that in sqlite. It looks like LIKE is always case-insensitive for the English alphabet, even when I add COLLATE BINARY.

You need PRAGMA case_sensitive_like = true; and then for case-insensitive you would probably need to do something like LOWER(field) LIKE LOWER(:value). Not the best solution though

Using case_sensitive_like should work, but it would be breaking if moor started changing that pragma suddenly. That change would also be hard to detect for users, so it's not really an option.
I've implemented this using custom sql functions in moor_ffi. The next moor version will have the containsCase extension method which takes a caseSensitive parameter that defaults to false. You need to import package:moor/extensions/moor_ffi.dart to use it, since it's a moor_ffi-only feature.
This function is implemented in Dart and uses String.contains or String.toLowerCase().contains(other.toLowerCase()). That doesn't fix all internationalization issues, but it's more consistent than using LIKE.


Create patch/content/secondary branch (based on feature/body) branch

Create patch/content/secondary branch (based on feature/body) branch

To solve this question, you need to enter the command:  git checkout -b patch/content/secondary

Thanks for the provided solution!


[Docs] Example for auto_insert_metric_name

Example for auto_insert_metric_name:

Add an example in case you have slash in metric name

What does this PR do?

Fixes #4012
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@carmocca Why it take long time to review?


create-tamagui with App Router throws error NextRouter was not mounted

I started this issue in the tamagui repo, but decided to open it here after finding that removing the calls to Solito solved the problem.

Changes to the original next-expo-solito-monorepo:

Run cd packages/app && yarn add solito@canary to install Solito v4
Run cd packages/app && yarn add next@latest && yarn add -D eslint-config-next@latest to update next.js
Delete folder apps/next/pages
In apps/next/next.config.js set experimental.appDir: true (optional: This option is no longer needed as of Next.js 13.4. The App Router is now stable)
Run cd packages/ui && yarn add @tamagui/config@latest to fix Cannot find module '@tamagui/config' or its corresponding type declarations. in apps/next/app/TamaguiProvider.tsx
In apps/next/app/TamaguiProvider.tsx change Tamagui.getCSS to Tamagui.getNewCSS
In packages/app/features/user/detail-screen.tsx change import { createParam } from 'solito' to import { useParams } from 'solito/navigation'

Throws error when cd apps/next && yarn build:
- info Creating an optimized production build
- info Compiled successfully
- info Skipping validation of types
- info Linting
- info Collecting page data
[    ] - info Generating static pages (0/3)Error: NextRouter was not mounted. https://nextjs.org/docs/messages/next-router-not-mounted
    at useRouter (/project/sandbox/apps/next/.next/server/chunks/638.js:26250:15)
    at useRouter (/project/sandbox/apps/next/.next/server/chunks/638.js:47723:44)
    at useLink (/project/sandbox/apps/next/.next/server/chunks/638.js:47795:20)
    at HomeScreen (/project/sandbox/apps/next/.next/server/app/page.js:450:57)
    at Je (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:121:272)
    at Z (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:128:91)
    at Je (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:122:9)
    at Je (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:127:11)
    at Z (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:128:91)
    at Ke (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:131:155)

Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error
Error: NextRouter was not mounted. https://nextjs.org/docs/messages/next-router-not-mounted
    at useRouter (/project/sandbox/apps/next/.next/server/chunks/638.js:26250:15)
    at useRouter (/project/sandbox/apps/next/.next/server/chunks/638.js:47723:44)
    at useLink (/project/sandbox/apps/next/.next/server/chunks/638.js:47795:20)
    at HomeScreen (/project/sandbox/apps/next/.next/server/app/page.js:450:57)
    at Je (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:121:272)
    at Z (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:128:91)
    at Je (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:122:9)
    at Je (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:127:11)
    at Z (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:128:91)
    at Ke (/project/sandbox/node_modules/next/dist/compiled/react-dom/cjs/react-dom-server.edge.production.min.js:131:155)
- info Generating static pages (3/3)

> Export encountered errors on following paths:
        /page: /

Reproduction
https://codesandbox.io/p/sandbox/create-tamagui-appdir-build-error-65jk6z
System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
    Memory: 12.88 GB / 23.93 GB
  Binaries:
    Node: 18.16.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 9.8.0 - C:\Program Files\nodejs\npm.CMD
    Watchman: 20210110.135312.0 - C:\ProgramData\chocolatey\bin\watchman.EXE
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (114.0.1823.67)
    Internet Explorer: 11.0.19041.1566

Thanks for the good reproduction. I wonder if some Next.js-related code is being imported by your app directly at the wrong time. Checking now.

So, the issue is, as far as the repro goes, it's using the tamagui starter rather than the solito one...so it has a lot of tamagui-related code that may be causing this, rather than solito's. Can you clarify which line getting removed fixes it? is it the homepage that's broken because of a solito import? Thanks!

Ahhh, I see the issue! You imported import { useLink } from 'solito/link'. However, as the docs state, to use this with app router, you need to import { useLink } from 'solito/navigation'. That should fix it!

Thanks for the reply @nandorojo
I had try this but I get Module '"solito/navigation"' has no exported member 'useLink'.

please upgrade


Add PullsURL to PushEventRepository

https://developer.github.com/v3/activity/events/types/#pushevent defines the statuses_url sent with each PushEvent webhook.

Also, please perform the go generate ./... step found in CONTRIBUTING.md to fix the Travis CI build failure.  Thanks.

I can rebase if you need :)


Unable to blacklist host "localhost"

In the following test, I am trying to blacklist "localhost". When I create client with hostname "localhost" and initialize a WriteBatcher, the following is the forest config

17:22:03.124 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - (withForestConfig) Using [localhost] hosts with forests for "WriteHostBatcher"

After I blacklist "localhost", the forest config becomes

17:22:03.171 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - (withForestConfig) Using [hp1040-3505.marklogic.com] hosts with forests for "WriteHostBatcher"
which is the alternate hostname

If I try to blacklist "hp1040-3505.marklogic.com, it has no effect and the WriteBatcher continues to run with [localhost] .

	@Test
	public void WBBlackList() throws Exception{
		DatabaseClient db = DatabaseClientFactory.newClient("localhost", 8000, "admin", "admin", Authentication.DIGEST);
		DataMovementManager dmManager = db.newDataMovementManager();

		final String query1 = "fn:count(fn:doc())";

		FilteredForestConfiguration forestConfig = null;
		DocumentMetadataHandle meta6 = new DocumentMetadataHandle().withCollections("NoHost").withQuality(0);

		Assert.assertTrue(dbClient.newServerEval().xquery(query1).eval().next().getNumber().intValue() == 0);

		WriteBatcher ihb2 =  dmManager.newWriteBatcher();
		forestConfig  = new FilteredForestConfiguration(dmManager.readForestConfig()).withBlackList("localhost");

		ihb2.withBatchSize(50).withForestConfig(forestConfig);

		ihb2.onBatchSuccess(
		        batch -> {
		        	}
		        )
		        .onBatchFailure(
		          (batch, throwable) -> {
		        	  throwable.printStackTrace();

		          }
		);
		for (int j =0 ;j < 1000; j++){
			String uri ="/local/string-"+ j;
			ihb2.addAs(uri, meta6 , jsonNode);
		}

		ihb2.flushAndWait();
		Assert.assertTrue(dbClient.newServerEval().xquery(query1).eval().next().getNumber().intValue() == 1000);

	}

Logs:
17:22:02.202 [main] DEBUG c.m.client.impl.JerseyServices - Connecting to localhost at 8000 as admin
17:22:02.609 [main] DEBUG c.m.client.impl.JerseyServices - Getting internal/forestinfo as application/json
17:22:02.827 [main] DEBUG c.m.client.impl.JerseyServices - Connecting to localhost at 8000 as admin
17:22:03.015 [main] DEBUG c.m.client.impl.JerseyServices - Posting eval
17:22:03.093 [main] DEBUG c.m.client.impl.JerseyServices - Getting internal/forestinfo as application/json
17:22:03.124 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - (withForestConfig) Using [localhost] hosts with forests for "WriteHostBatcher"
17:22:03.139 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - Adding DatabaseClient on port 8000 for host "localhost" to the rotation
17:22:03.139 [main] DEBUG c.m.client.impl.JerseyServices - Getting internal/forestinfo as application/json
17:22:03.171 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - (withForestConfig) Using [hp1040-3505.marklogic.com] hosts with forests for "WriteHostBatcher"
17:22:03.171 [main] DEBUG c.m.client.impl.JerseyServices - Connecting to hp1040-3505.marklogic.com at 8000 as admin
17:22:03.249 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - Adding DatabaseClient on port 8000 for host "hp1040-3505.marklogic.com" to the rotation
17:22:03.264 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - threadCount=1
17:22:03.264 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - batchSize=50
17:22:03.312 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:03.312 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:03.312 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:03.312 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.319 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.319 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.335 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.335 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.507 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.507 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.616 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.616 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.726 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.726 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.772 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.772 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.838 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.838 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.901 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.901 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:05.947 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:05.947 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.025 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.025 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.072 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.072 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.135 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.135 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.181 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.181 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.244 [main] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.244 [main] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.275 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.275 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.335 [main] INFO  c.m.c.d.impl.WriteBatcherImpl - flushing 0 queued docs
17:22:06.388 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.388 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.467 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.467 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.545 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Posting documents
17:22:06.545 [pool-1-thread-1] DEBUG c.m.client.impl.JerseyServices - Sending multipart for /v1/documents
17:22:06.608 [main] DEBUG c.m.client.impl.JerseyServices - Posting eval
S is true
WriteHostBatcher-1

When you black-list a host, it's going to try to use any valid host for the available forests.  That includes the host, alternate host, or replica host.
Could you try black-listing both localhost, and hp1040-3505.marklogic.com?

I am able to blacklist after including both the hostname and alternate hostname in the blacklist


Merge branch 'release/0.92.0'

Coverage remained the same at 80.089% when pulling e85d63cad3b7e3605c777a84d21555082720b136 on master into 60083b05d0215a17b2e93974be41a1cbf6cef56b on develop.

Coverage remained the same at 81.116% when pulling f2399aee0813a7ec9ef62e79e59c9f0247a78710 on master into 9d00dba74ee23d7d0a628ccb8faf70484f8e6f10 on develop.


[issue:411] When displaying the update candidates, display the name of the channel the update was found in

Fixes: https://github.com/wildfly-extras/prospero/issues/411

Do not review, I am looking at some Tests failures.

Hi @spyrkob ,
I've updated the PR, please review it.
Thanks.

Hi @spyrkob ,
I've updated the PR, please review it.
Thanks.


Game Crashing?

As of September 26, the hack is crashing CSGO, probably the offsets changed due to an update

#583 try this, it should work.

*too lazy to log the crash
(me too)

#583 try this, it should work.

I'm really confused on what to do, I'll change the offsets in the source code then?

#583 try this, it should work.

I'm really confused on what to do, I'll change the offsets in the source code then?

Change the offsets, compile, inject, play, win. (not so sure about the last one)

constexpr Vector getEyePosition() noexcept
{
Vector vec{ };
callVirtualMethod<void, Vector&>(this, 283, vec);
return vec;
}
constexpr bool isWeapon() noexcept
{
return callVirtualMethod(this, 165);
}
constexpr bool isAlive() noexcept
{
return callVirtualMethod(this, 155) && health() > 0;
}
constexpr bool isPlayer() noexcept
{
    return callVirtualMethod<bool>(this, 157);
}

constexpr Entity* getActiveWeapon() noexcept
{
    return callVirtualMethod<Entity*>(this, 267);
}

constexpr int getWeaponSubType() noexcept
{
    return callVirtualMethod<int>(this, 279);
}

constexpr WeaponData* getWeaponData() noexcept
{
    return callVirtualMethod<WeaponData*>(this, 456);
}

constexpr float getInaccuracy() noexcept
{
    return callVirtualMethod<float>(this, 478);
}
constexpr Entity* getObserverTarget() noexcept
{
    return callVirtualMethod<Entity*>(this, 293);
}


config file bug?

Hi! when I run the code with script , I find the following  key is not shown in default.py, thus caused failure.
https://github.com/ChCh1999/RTPB/blob/4f72279d4b63f8818999678507e4f0bafc6cb1b4/configs/e2e_relation_X_101_32_8_FPN_1x_trans_predcls.yaml#L91

It's a bug and we will fix it. You can just delete the following code block.
https://github.com/ChCh1999/RTPB/blob/4f72279d4b63f8818999678507e4f0bafc6cb1b4/configs/e2e_relation_X_101_32_8_FPN_1x_trans_predcls.yaml#L91-L102

Thanks!


Fix build issues for MP starter

Fixing issues to execute build tests

@breakponchito there are many failures

@breakponchito there are many failures

yeah checking what to do for the following error:

in all cases is the same issue

@breakponchito there are many failures

@Emily-Jiang  I already upgrade the version for the upload-artifact to use v4, please run again the validations

@Emily-Jiang Could you help me to run again the tests? it seems  a network problem, on my local I can't reproduce same error

@Emily-Jiang please run again the validation. I'm testing to increment connect timeout and read timeout to minutes

We discussed at today's call and agreed to merge this PR for now.


istio-proxy can't connect to upstream/pilot on AKS

Bug report
General Information

Cilium version (run cilium version):
Client: 1.9.1 975b66772 2020-12-04T18:16:09+01:00 go version go1.15.5 linux/amd64
Daemon: 1.9.1 975b66772 2020-12-04T18:16:09+01:00 go version go1.15.5 linux/amd64
Kernel version (run uname -a):
Linux aks-vmss01-42431932-vmss000000 5.4.0-1035-azure #36~18.04.1-Ubuntu SMP Wed Dec 16 23:49:28 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Orchestration system version in use (e.g. kubectl version, Mesos, ...):
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.2", GitCommit:"faecb196815e248d3ecfb03c680a4507229c2a56", GitTreeState:"clean", BuildDate:"2021-01-14T05:14:17Z", GoVersion:"go1.15.6", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.6", GitCommit:"fbf646b339dc52336b55d8ec85c181981b86331a", GitTreeState:"clean", BuildDate:"2020-12-18T16:06:08Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Link to relevant artifacts (policies, deployments scripts, ...):
https://docs.cilium.io/en/stable/gettingstarted/k8s-install-aks/
https://docs.cilium.io/en/stable/gettingstarted/k8s-install-azure/
Generate and upload a system zip:
cilium-sysdump-20210120-112307.zip

How to reproduce the issue

Install AKS Cluster
Install Cilium into AKS Cluster using either of manuals https://docs.cilium.io/en/stable/gettingstarted/k8s-install-aks/ or https://docs.cilium.io/en/stable/gettingstarted/k8s-install-azure/. The very Helm command I've used last time is:

helm install cilium cilium --repo https://helm.cilium.io/ --version 1.9.1   --namespace cilium   --set azure.enabled=true   --set azure.resourceGroup=$AZURE_NODE_RESOURCE_GROUP   --set azure.subscriptionID=$AZURE_SUBSCRIPTION_ID   --set azure.tenantID=$AZURE_TENANT_ID   --set azure.clientID=$AZURE_CLIENT_ID   --set azure.clientSecret=$AZURE_CLIENT_SECRET   --set tunnel=disabled   --set ipam.mode=azure   --set masquerade=false   --set nodeinit.enabled=true   --set nodeinit.restartPods=true

Install Cilium's modified version of Istio using manual: https://github.com/cilium/cilium/blob/master/Documentation/gettingstarted/istio.rst

More Details
On freshly created Azure AKS cluster v1.19.6 (but it doesn’t really meter, I believe, did the same on v1.18.10), after installing Cilium using either ordinary CNI chaining method https://docs.cilium.io/en/stable/gettingstarted/k8s-install-aks/ or Azure IPAM https://docs.cilium.io/en/stable/gettingstarted/k8s-install-azure/ Istio components becomes really flaky communicating to each other. Actually all the component suffering from DNS errors like curl: (6) Could not resolve host: git.io or ;; connection timed out; no servers could be reached, but I'm interested in Istio most.
I’ve used latest two Cilium’s Istio releases v1.6.14 and v1.7.6 the picture is the same.
The main problem that istio-proxy couldn’t reliably connect to istiod (pilot).
Log is full of errors like:
2021-01-19T17:13:55.405297Z	warning	envoy config	StreamSecrets gRPC config stream closed: 14, connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:15012: i/o timeout"
2021-01-19T17:16:18.727112Z	warning	envoy config	StreamSecrets gRPC config stream closed: 14, connection error: desc = "transport: Error while dialing dial tcp: lookup istiod.istio-system.svc on <IP_ADDRESS>:53: no such host"

And so on...
Cordoning/rebooting nodes and trying to reschedule those pods on different nodes sometimes help sometimes not.
Just to have source of truth created another cluster without Cilium and installed not modified version of Istio v1.7.6 there. Everything is working fine there as was expected.
So here the question, is there some tested configuration for AKS and Cilium’s version of Istio (I need L7 policies in the end) which I can use? Or maybe someone can advice me what should I check?

/cc @aditighag @jrajahalme who probably know best about Istio proxy in Cilium.

/cc @aditighag @jrajahalme who probably know best about Istio proxy in Cilium.

I bet it has something to do with Azure's CNI, but I don't know how to troubleshoot it properly. We have pretty old AKS cluster (v1.17.11) where Cilium v1.9.1 and Cilium's Istio v1.5.9 work just fine.

I bet it has something to do with Azure's CNI, but I don't know how to troubleshoot it properly. We have pretty old AKS cluster (v1.17.11) where Cilium v1.9.1 and Cilium's Istio v1.5.9 work just fine.

We recently fixed Istio redirection in Cilium. Here is the PR - https://github.com/cilium/cilium/pull/14271
Could you try with Cilium v1.9.2 and Istio v1.6.15?

We recently fixed Istio redirection in Cilium. Here is the PR - https://github.com/cilium/cilium/pull/14271
Could you try with Cilium v1.9.2 and Istio v1.6.15?

Looks like unrelated or partially related to Cilium. Sometimes it isn't working even without Cilium at all. Sometimes it's working fine on one Cilium version, but failing on another.
Can't confirm, or reliably reproduce, hence closing.


[Mobile] The mobile version does not display a button (frame) around the text when deleting

The mobile version does not display a button (frame) around the text when deleting, I don't know, maybe this is the author's idea, and it has always been so, but before I did not pay attention to it.

This is intentional as of the latest release to reduce accidental input

This is intentional as of the latest release to reduce accidental input

Well, it looks a bit strange, well, or just unusual, but for that matter, I suggest you transfer this style to the desktop version, I think it would be appropriate, so that both versions correspond to the general style.

The desktop screenshot you shared is not the latest version. You can see more about the discussion on this topic in the Obsidian Discord

The desktop screenshot you shared is not the latest version. You can see more about the discussion on this topic in the Obsidian Discord

Well..., the latest version of the program is 0.13.19 (excluding insider build), the version of the theme is 4.4.8, doesn't matter, the main thing is that you are aware 😁


Can' find browser.waitUntil() function

I've started to using WebdriverIO v3 for testing my application, and it's working awesome when I'm running tests with wdio command. But when I'm running the same tests with grunt-webdriver plugin,  I am getting a TypeError: undefined is not a function error, at the line of code that calls browser.waitUntil() function.

@valekask grunt-webdriver still runs on v2.4.5. I will need to update that project to work with the wdio test runner. I would recommend to use something that can run the wdio command from your gruntfile.


chore: update README to link to official client

The official client package is now github.com/InfluxCommunity/influxdb3-go, direct users there.

we should probably archive the repo using GitHub's official functionality

we should probably archive the repo using GitHub's official functionality

Yes that's the plan, just wanted to get the redirect in first.


PermissionError in phold plot

phold version: 0.1.4
Python version: 3.10.14
Operating System: Windows (WSL2, Ubuntu)

Description
Trying to run phold plot and I'm getting this error:
2024-04-12 16:30:33.499 | INFO     | phold.utils.validation:instantiate_dirs:70 - Checking the output directory /home/ctbus/Penn/sunbeam/projects/virus/sunbeam_output/virus/phold/LONG_plot
2024-04-12 16:30:33.499 | INFO     | phold.utils.validation:instantiate_dirs:73 - Removing /home/ctbus/Penn/sunbeam/projects/virus/sunbeam_output/virus/phold/LONG_plot because --force was specified
Traceback (most recent call last):
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/bin/phold", line 10, in <module>
    sys.exit(main())
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/phold/__init__.py", line 1355, in main
    main_cli()
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/phold/__init__.py", line 1236, in plot
    instantiate_dirs(output, force)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/site-packages/phold/utils/validation.py", line 74, in instantiate_dirs
    shutil.rmtree(output_dir)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/shutil.py", line 725, in rmtree
    _rmtree_safe_fd(fd, path, onerror)
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/shutil.py", line 681, in _rmtree_safe_fd
    onerror(os.unlink, fullname, sys.exc_info())
  File "/home/ctbus/Penn/sunbeam/.snakemake/160f8892b5f849b64b315e3ee4674746_/lib/python3.10/shutil.py", line 679, in _rmtree_safe_fd
    os.unlink(entry.name, dir_fd=topfd)
PermissionError: [Errno 13] Permission denied: 'phold_plot_1712951468.829077.log'

I'm running this locally on a computer where I'm the only user. Permissions for the output of phold compare:
drwxr-xr-x 4 ctbus ctbus 4096 Apr 12 15:47 ./
drwxr-xr-x 8 ctbus ctbus 4096 Apr 12 16:29 ../
drwxr-xr-x 2 ctbus ctbus 4096 Apr 12 15:47 logs/
-rw-r--r-- 1 ctbus ctbus 6324 Apr 12 15:47 phold.gbk
-rw-r--r-- 1 ctbus ctbus  624 Apr 12 15:47 phold_3di.fasta
-rw-r--r-- 1 ctbus ctbus  624 Apr 12 15:47 phold_aa.fasta
-rw-r--r-- 1 ctbus ctbus 1945 Apr 12 15:47 phold_all_cds_functions.tsv
-rw-r--r-- 1 ctbus ctbus 9244 Apr 12 15:47 phold_compare_1712951223.3917286.log
-rw-r--r-- 1 ctbus ctbus 1596 Apr 12 15:47 phold_per_cds_predictions.tsv
drwxr-xr-x 2 ctbus ctbus 4096 Apr 12 15:47 sub_db_tophits/

What I Did
Ran install, predict, compare with minimal options, just specifying inputs, outputs, db path, and --force. Plot command:
phold plot -i {input.gbk} -o $(dirname {output.png}) --force

Getting this using a conda install and a docker install. I suspect it's just a matter of earlier steps writing with overly restrictive permissions for what plot is trying to do. Let me know if there's any more information/help I can give.
Thanks,
Charlie

Hi @Ulthran ,
Thanks for raising this. I am not sure myself what could be the issue - I don't think it has anything to do with the output for phold compare.
It looks like it can't force remove the output directory  shutil.rmtree(output_dir) but weirdly throws an error to the log file... I am not really sure - maybe try without --force?
George

Thanks @gbouras13, without --force it's complaining about the directory existing:
2024-04-16 09:33:38.406 | INFO     | phold.utils.validation:instantiate_dirs:70 - Checking the output directory /home/ctbus/Penn/sunbeam/projects/virus/sunbeam_output/virus/phold/LONG_plot2
2024-04-16 09:33:38.406 | ERROR    | phold.utils.validation:instantiate_dirs:81 - Output directory already exists and force was not specified. Please specify -f or --force to overwrite the output directory

Unfortunately, that's because of snakemake making the output directory before running the command (which I don't think is behavior I can change). So I need to run with --force in this context.

@Ulthran I've run into that with snakemake before too :(
I'll implement phold into a snakemake pipeline when I get some time and see if I get the same error as well - otherwise, not quite sure how to go about fixing this!
George

Ok thanks George, this is at the end of our pipeline so it's not hugely important that it be automated. If you do come up with a fix though we'll be happy to implement it!

Hi @Ulthran ,
We have implemented phold in a snakemake pipeline (https://github.com/linsalrob/sphae) and don't have such an error - maybe have a look at the rules in sphae?
George

Thanks @gbouras13 , it's only happening for phold plot though. Do you get the same issue if you add a rule for that to sphae?


Generic handlers & DE OMAS performance improvements

Description
This PR covers the following changes:

in DE OMAS the anchor is set on the SchemaType/SchemaAttribute at creation
in the uncheckedLinkElementToElement method the addLatestChanges classification is no longer added to the anchor

Related Issue(s)

Testing

Release Notes & Documentation

Additional notes

@mandy-chessell can you please take a look at the change in generic handler class and apporve, we would like to proceed with this PR. Thanks.

@mandy-chessell Can you please take a look at this change?


do not vendor parmed

Checklist

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

Will close #21

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin, please rerender

PPC64LE blocked by https://github.com/conda-forge/parmed-feedstock/pull/10

That PR was merged, but Travis failed one upload for PPC. Retrying: https://travis-ci.com/github/conda-forge/parmed-feedstock/jobs/480397033

I think the parmed replacement worked.

@jchodera New parmed-less AmberTools should be available ~1h after this is auto-merged.


meteorite example: add dependency workaround

The meteorite example is currently not working.
It gives the following error:
cannot import name 'soft_unicode' from 'markupsafe'

This seems to be due to markupsafe removing soft_unicode:
https://github.com/pallets/markupsafe/issues/304
We can work around this by pinning to an earlier version of markupsafe.
This gives the following warning:
pandas-profiling 3.2.0 requires markupsafe~=2.1.1, but you have markupsafe 2.0.1 which is incompatible.
but the example seems to work ok.

Hi @stuart-mclaren, thanks for flagging this. The issue has been resolved here and will be included in the next release.


Autofund logging

Codecov Report

Merging #343 (c8c1592) into main (3b8c3c3) will decrease coverage by 0.1%.
The diff coverage is 0.0%.

@@           Coverage Diff           @@
##            main    #343     +/-   ##
=======================================
- Coverage   14.2%   14.0%   -0.1%
=======================================
  Files         27      27
  Lines       7183    7257     +74
=======================================
  Hits        1019    1019
- Misses      6164    6238     +74

Impacted Files
Coverage Δ

src/config.rs
0.0% <0.0%> (ø)

src/farcasterd/opts.rs
0.0% <ø> (ø)

src/farcasterd/runtime.rs
0.0% <0.0%> (ø)

src/swapd/runtime.rs
0.0% <0.0%> (ø)

src/walletd/runtime.rs
0.0% <0.0%> (ø)

Continue to review full report at Codecov.

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

tACK


Port connection string changes to Python

Secondary endpoints may now be specified in connection strings, provided the corresponding primary endpoint is specified. See the connection string documentation for details.

@kfarmer-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


I1258 b update release notes

The is a second attempt to push changes to 7160007.html - update release notes.

Let's push this to github docs for easier review

It looks like our known issues have gone stale. It may be time to update those.

The description for resolved issue #171 is confusing. I think it would be best to change it to the issue title (Add advanced parameters to geserver installer).

Nitpick: on resolved issue #126 we can delete the phrase "No current workaround." since the workaround is listed right after that sentence.

The workaround for known issue #343 is missing a letter: Void opening files with .kip extension. --> Avoid opening files with .kip extension.

Resolved issue #279 is out of order and should be moved up so that the resolved issues are in alphabetical order.

I think issue #171 merits inclusion as a new feature (i.e., it should be moved to the top rather than the resolved issues section).

After looking through the notes here's what seems to be missing:
New features
Issue #1020
Issue #1192
Issue #1275
Resolved issues
Issue #1104
Issue #1169
Issue #1201
Issue #1276

@tst-lsavoie  what items are out of order?  I can't find any...  Are you looking at https://tst-eclamar.github.io/earthenterprise/geedocs/5.3.0/answer/7160007.html or an older webpage?

Hm, I must have been looking at the wrong one. Let me take another look.

Here's two issues that I think could be added to the known issues, together with a rough cut at a workaround:
Issue #1257: Ensure you use a valid date when creating resources from the command line
Issue #1202: Add the resource from the command line