[SPARK-9481]Add logLikelihood to LocalLDAModel

@jkbradley Exposes bound (variational log likelihood bound) through public API as logLikelihood. Also adds unit tests, some DRYing of LDASuite, and includes unit tests mentioned in #7760

Merged build triggered.

Merged build started.

Test build #39078 has started for   PR 7801 at commit f0996d8.

Test build #39078 has finished for   PR 7801 at commit f0996d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

This looks good, but it made me realize that gensim defines log perplexity as the negation of the log perplexity in the Online LDA paper's Eq 15.  I prefer the Online LDA paper way, since low perplexity sounds like a good thing to me (whereas with gensim, high perplexity is better).  I checked Stanford NLP, and they also say lower perplexity is better.  Can you please modify the logPerplexity code to negate the returned value?

Merged build triggered.

Merged build started.

Test build #39238 has started for   PR 7801 at commit 6d1b2c9.

Test build #39238 has finished for   PR 7801 at commit 6d1b2c9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

LGTM, thanks!  Merging with master


[INFRA] Introduce metaschema

This PR contains:

metaschema.json: A json schema document that specifies the entire schema language
validate_schema.py: code for loading the schema, dereferencing it, and validating against the metaschema
very minor adjustment to the actual schema, changing "deprecated" to "DEPRECATED", which appears to be the more dominant.
tests ensuring that this correctly identifies erroneous additions to the schema

When I started this at the meeting in Seattle I did not expect the metaschema to be >700 lines, but there it is. I used definitions a bit to make it more concise. It's possible this could benefit from more definitions.

depends on #1790

Made a few small changes that were easier to do than suggest. Please have a look and let me know if you disagree.

@bendichter Please add yourself to https://github.com/bids-standard/bids-specification/wiki/Recent-Contributors if you aren't already on the contributors list.


Determining allowedOrigins programatorically

I'm building an app where multiple parties have api access.
Each party has it's own domains where it can make requests from.
What is the best way of passing the allowed domains of the logged in party to allowedOrigins?
I could swap out the  CorsService registered by the service provider with another instance where the domains of a party are passed via the options of the constructor. But this seems a bit nasty.
Is there a better way?

I think we could perhaps  allow a callback to determine if the origin matches?
$corsService->setOriginCallback(function($request) use ($myApiService) { ... } );

I'm open for improvements :)

setOriginCallback could be useful yes. 👍
Like mentioned on Twitter I'm currently creating my own lightweight cors solution.


[Bug] Image map esc key

When the ESC key is pressed, the selected object is not canceled.

#93 same issue


[es] transaction responses point to glossary.html#response which not exists

Here, under Certificate Authorities, transaction responses point to glossary.html#response which not exists. I checked and response never existed since release-1.1 (the older at Read the Docs). At the other hand, transaction responses started with release-1.3 (it doesn't exists at 1.2).
I am starting with Hyperledger Fabric (and Hyperledger in general), so I am not sure where to point.
Regards,
Rodrigo

Hi @claudiocebpaz. FYI, it is not a particular problem of the es version, is also a problem in the original en version. Please, someone with experience let me know how to fix it (where to point). Regards.

hello @rodrigomelo9 ..shoot me a message. I think i can help out


feat(config): launch app manually

Description:
This feature is aimed at improving usability when you need to debug native Detox errors on iOS and Android respectively.
{
  "ios.manual": {
    "type": "ios.simulator",
    "binaryPath": "... path to the app ...",
    "device": "... your device config ...",
    "artifacts": false,
    "behavior": {
      "launchApp": "manual"
    },
    "session": {
      "autoStart": true,
      "server": "ws://localhost:8099",
      "sessionId": "someTestSessionId"
    }
  },
  "android.manual": {
    "type": "android.emulator",
    "binaryPath": "... path to the app ...",
    "device": "... your device config ...",
    "behavior": {
      "launchApp": "manual"
    },
    "session": {
      "autoStart": true,
      "server": "ws://localhost:8099",
      "sessionId": "someTestSessionId"
    }
}

So, in this example, when you run detox test -c ios.manual or detox test -c android.manual on a test you want to debug, at some point of time when you get to device.launchApp() call, instead of launching the app, Detox will pause its execution with a message similar to the ones shown below.
iOS:
detox[16804] INFO:  [SimulatorDriver.js] Waiting for you to manually launch your app in Xcode.
Make sure to pass the launch arguments listed below:
  -detoxServer ws://localhost:8099
  -detoxSessionId com.wix.detox-example

Press any key to continue...

Android:
detox[53038] INFO:  [AndroidDriver.js] Waiting for you to manually launch your app in Android Studio.

Instrumentation class: com.wix.detox.test.test/com.example.DetoxTestAppJUnitRunner
Instrumentation arguments:
------------------------------------
Key            | Value
------------------------------------
detoxServer    | ws://localhost:8099
detoxSessionId | test
------------------------------------

Press any key to continue...

You'll need to run your app with the said arguments from Xcode or Android Studio:
iOS:

Android:

When you run the app, return to your running Detox test, and press any key (of course, except Ctrl+C which just terminates the process). As a result, you are expected to see a confirmation from Detox, e.g.::
Found the app (com.wix.detox-example) with process ID = 16854. Proceeding...

Of course, this does not cancel the necessity of preparing of your Xcode or Android Studio project: e.g. for Xcode, that at least implies unpacking node_modules/detox/Detox-ios-src.tbz and embedding it as a build phase:

Nevertheless, this feature somewhat improves the developer experience compared to ios.none config, especially when you have to orchestrate a sequence of mock servers before you can launch the app since this workflow does not require changes in the source code of your e2e tests.
TODO

[ ] update the docs (config: artifacts, behavior, client/server; troubleshooting guide, debugging natively).
[ ] decide if we want to disable that monitor: https://github.com/wix/Detox/blob/master/detox/android/detox/src/main/java/com/wix/detox/reactnative/ReactNativeLoadingMonitor.kt#L56

Still not clear to me what the difference between ios.none and this is, or why this is needed.

This can work with all artifacts enabled, as usual, for both platforms.
As for the redundant arguments — you are technically right, but you can actually omit them only (!) if they match the default values, which is not always true:
E.g. the configuration below has only behavior: { launchApp": "manual" } override, and that's it:
{
      "ios.manual": {
        "type": "ios.simulator",
        "binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/example.app",
        "behavior": {
          "launchApp": "manual"
        },
        "device": {
          "type": "iPhone 11 Pro"
        }
      }
}

And yet it works the same as the regular ios.simulator version, with artifact plugins enabled as well, which is something you cannot do for ios.none that easily (because it is agnostic to device/simulator distinction).
And look, it also autogenerates URL and sessionId like in default settings:
➜  test git:(feat/launch-app-manually) ✗ npx detox test -c ios.manual e2e/01.sanity.test.js      detox[2448] INFO:  [test.js] configuration="ios.manual" useCustomLogger=true DETOX_START_TIMESTAMP=1608554308067 reportSpecs=true nyc jest --config e2e/config.js --testNamePattern '^((?!:android:).)*$' --maxWorkers 1 e2e/01.sanity.test.js
detox[2450] INFO:  [DetoxServer.js] server listening on localhost:50130...
detox[2450] INFO:  [AppleSimUtils.js] Waiting for you to manually launch your app in Xcode.
Make sure to pass the launch arguments listed below:
  -detoxServer "ws://localhost:50130"
  -detoxSessionId "845f9c9a-9a34-3c90-828a-69bf96baa754"

Press any key to continue...

So this is a case when you actually NEED to copy&paste the said values to your IDE.
Overall, I cannot exclude the possibility of debugging multi-worker issues as well, so I like that this solution can be very close to the real configuration on CI, for example.

Why not fix ios.none to work with artifacts?

Because it is a simulator-agnostic driver, and it will make me think more about how to debug it right in a complex configured suite of tests. Also, it will result in an awkward AndroidNoneDriver, if we decide to override "launchApp" implementation so it waits for you to launch the app, because contrary to IosDriver, the AndroidDriver has a full launchApp implementation over adb.

And I never said I won't fix ios.none artifacts issues you have reported.
I just wanted to group that together with single-timeline rewrite task, not just now.

I didn’t ask just about fixing, but perhaps extend the artifacts to support ios.none fully.

Also, I will be rewriting the Xcode guide soon for XCUITest, and that too will be based on ios.none as I don't want any Detox complexities to mix with XCUITest.


Document ability to playmedia action with offset

I noticed the remote control API was updated and mentions that the play media action now supports the offset argument. Is it possible to get that support added here? I find it would be nice to be able to resume something from where you left off, should the need arise.
I added a line to the setParameters function for it, and it works for me:
if offset is not None: params['offSet'] = offset # offset value in ms

I assume you are talking about the plexapi.client.Client.playMedia() function?  You should be able to do this already by simply adding the additional keyword when calling the function.
client.playMedia(<some_media>, offset=1234)

Cool that does already work.  Thanks for pointing that out.

I'll leave the bug open because this should be documented in the code.

commit 0aac1156a14a7b6a1ece2763dbc0bc5543fd668c
Author: Michael Shepanski<EMAIL_ADDRESS>Date:   Thu Feb 2 01:32:38 2017 -0500
Add offset as optional playMedia() argument; Document it
Clenaup old tests as its the only way to test client actions right now


HPACK circular overflow #12190

@anurse 3.0 candidate
Description
When the client tries to intern more than 127 entries in the dynamic header table it reaches the end of the circular buffer and fails to properly wrap.
Customer Impact
Multiple external reports, depends on client input, hard to diagnose, and with no mitigation but to disable HTTP/2.
https://github.com/aspnet/AspNetCore/issues/12190
https://github.com/aspnet/AspNetCore/issues/12045
Regression? No
Risk
Low. Once we understood the repro the fix was simple and easy to test.
@geoffkizer This bug also exists in corefx:
https://github.com/dotnet/corefx/blob/06d215d9fe6fff08f81add5ed341641f668075c0/src/System.Net.Http/src/System/Net/Http/SocketsHttpHandler/HPack/DynamicTable.cs#L37

Looks good and OK with me for ask-mode. Apply the label when review approvals are received.

Tactics approved


(PDB-3047) Corrective change counts

This PR subdivides successes and noops counts into intentional and corrective parts in event-counts and aggregate-event-counts endpoints for PE. Events are mapped to the corresponding counts based on value of corrective_change flag.
The response formats for FOSS are not changed.

@wkalt @senior Could you please take a look?

Test FAILed

To the questions -- this doesn't require an API version bump because the affected endpoints are marked experimental. The store-corrective-change atom gets flipped when the PE service starts, and is fine to use in tests, but it's probably best to have a test case when it's on and another when it's off.

@puppetlabs-jenkins retest this please

Test PASSed

@wkalt I've changed the PR based on the discussion we had yesterday. The response formats for FOSS are not changed now. Could you take a look?
Tests for @store-corrective-change? = false still need to be added.

Test PASSed

@kbrezina left some super minor stylistic notes. This looks good to me otherwise and behaves as expected. Once we have some tests with the atom set to false I'm good to go.

@wkalt I've fixed the typos. Tests are written with and without support for corrective changes now.
Thanks a lot for your review and valuable feedback. 👍

Test PASSed


Edits for roadmap.md

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

@leemills83 @Rugvip Did I do this wrong...

@AlleyWei all fixed, just needed to run prettier on your changes. @backstage/maintainers over to you 😄


Request: run your own Sigstore documentation

The beginnings of this exist already:

https://docs.sigstore.dev/cosign/custom_components
#19

But I propose the following reorganization:

Run your own Sigstore

Overview
Scaffolding
Managing keys with TUF (this would mostly pull from "Configuring Cosign with Custom Components")

TUF
Revocations (#19)

This should eliminate most of "Configuring Cosign with Custom Components," so we could drop that page and make it an FAQ.

We've started this effort here:
https://docs.google.com/document/d/1fchY8FwEGjrpErjNTMuGtMx2qsGcCKtGx4sNpOhpqbE/edit#


Corporate users cannot access dl.bintray.com

People in #192 and #211 said that they cannot download from:
https://dl.bintray.com/elmlang/elm-platform/

If they do not trust that site, I am not sure what external site they would trust by default. So I do not see a clear path on the Elm side. Instead, it seems to make sense to acquire and vet the binaries (probably building from source so you can look through the source code) and serve it on your internal infrastructure.
If this is not correct, please create a separate issue that explains your situation and why this path is inadequate.

Closing in favor of https://github.com/elm-lang/elm-platform/issues/224 which should address this.


maven-mvnd-0.8.0  Support   idea-2022-1.3 version, methods

this is idea  configuration

then, you should change maven-mvnd  source lib,

for lib Add to
(slf4j-api-1.7.32.jar,  slf4j-api.license, maven-slf4j-provider-3.8.4.jar ) (Available from the maven lib )
In this case, MAVEN in IDEA is using MVND to download dependencies,
If you do not add these three lib(slf4j-api-1.7.32.jar,  slf4j-api.license, maven-slf4j-provider-3.8.4.jar), the IDEA download dependency will report an error,  java.lang.NoClassDefFoundError: org/slf4j/Logger  at org.jetbrains.idea.maven

I hope maven-mvnd can be integrated into IDEA. Just as IDEA naturally supports Maven.
then.  advice
Add three dependencies
maven-slf4j-api-1.7.32. jar, slf4j-api.license, maven-slf4j-provider 3.8.4.jar into maven-mvnd-0.8.0 source code  for lib package To support IDEA 2022.1.3 .
Thank you very much, You can try it
In this way,  maven-mvnd-0.8.0 is used to replace apache-maven-3.8.6 in idea-2022-1.3 work

This method may not use mvnd, but use the mvn that comes with mvnd.

This method may not use mvnd, but use the mvn that comes with mvnd.

The maven layout contained in mvnd is not a stock maven layout, so when you launch maven using the mvn/bin/mvn script, it will look like maven, but will use some components from mvnd.  However, there will be no daemon used, so no caching, classloader reuse, etc...


Implement API for using Astranaut as a library

Currently the functionality of Astranaut may be accessed only from the CLI.
Users have requested the facility to use the project as a Java library.

@rultor release, tag is 0.2

@rultor release, tag is 0.2

@kniazkov OK, I will release it now. Please check the progress here

@rultor release, tag is 0.2

@kniazkov Done! FYI, the full log is here (took me 8min)

Thank you. I am working on API usage guide now

@rultor release, tag is 0.2.1

@rultor release, tag is 0.2.1

@kniazkov OK, I will release it now. Please check the progress here

@rultor release, tag is 0.2.1

@kniazkov Done! FYI, the full log is here (took me 7min)


NIFI-10311: Update Avro From 1.11.0 to 1.11.1

Summary
NiFi-10311 Upgrade Avro 1.11.0 to 1.11.1 to remediate the CVE with a High rating of 7.5
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] NiFi-10311

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 8
[X] JDK 11
[X] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks for the upgrade @UcanInfosec!
CVE-2020-36518 does not apply to Avro, it applies to Jackson 2.13.0, which is already upgraded.
However, upgrading to this version of Avro is still useful.

@exceptionfactory You are correct. But the version 1.11.0 of Avro has an old Jackson-databind, which means that the Avro software should be updated

Thanks for the reply @UcanInfosec. The root Maven configuration overrides transitive versions of Jackson using a Bill-of-Materials dependency, so the current version of NiFi is not impacted, but aligning the version of Avro is helpful.

@exceptionfactory didnt realize this. Thanks for the heads up. Will make note for later.


How can i define a Angular 8 Custom Formio Nested Component

Hello everyone,
how can i create a custom nested component (layout) with  angular elements?
Is it like creating a normal custom component? I couldn't find any documentation about it.
Thank you.

I know that i need to extend NestedComponent. But how can i do that. I couldn't even get the checkmatrix example to work.
I tried importing the CheckMatrix.js file into my app.module.ts file and it can't be compiled.
i tred to register the CheckMatrix component and it won't work.
What am i doing wrong here?
Could you give me an example on how to create and register a  custom nested component with angular 8?


Path edit broken for muscles with wrapping

Steps to reproduce
Reported by @tkuchida in beta testing
Expected result
Actual result
(include screen grabs where appropriate)
Environment and GUI version
4.4 Beta

Possibly related: User's Forum Topic 14880

Description from testing doc. to verify fixed in artifact builds
Muscle GeometryPath editor: "java.lang.IndexOutOfBoundsException: Index: 4, Size: 4". To reproduce: (1) Open arm26 model. (2) In Navigator, select Muscles > TRIlong. (3) Click "…" beside "GeometryPath" in Properties window. (4) Click Add > before 1. (5) On row 1, change X = -0.055 to, e.g., X = 0.1 and press . Expected: muscle point updates. Actual: red circle appears in bottom-right corner of GUI.

Fixed in artifact of this PR. Thanks for testing and reporting @tkuchida
@carmichaelong if you can doublecheck the fix in the artifacts I'd appreciate it, then we can merge and backport to 4.4 branch before posting.

Potentially related issue posted to the Forum (topic 15006).

@aymanhab @tkuchida I tried the Forum issue posted most recently in this thread. I was able to replicate in the 4.4 GUI, and also saw that this issue occurred with a muscle with wrapping objects, but not with a muscle that had no wrapping objects associated.


Use gif for preview

Use gif instead of mov files so the preview appears on the README.

Thank you!


Feat/df-45-create-incomes-and-expenses

Keypoints:

Created "Create Expense" and  "Create Income" mutations
Wrote Tests for them both

Related Issue(s)
closes DF-45
Checklist:
-[x] New/Modified Tests are Passing
-[x] I have commented my code, particularly in hard-to-understand areas.
-[x] I have made corresponding changes to the documentation.
-[x] My changes generate no new warnings or errors.
Additional Notes:
[Please include specific lines/files affected if needed]
[Any special testing/config considerations]

Ready for merge now!


Clean-up of ArtifactType

Changes Unknown when pulling 152ccd963707c0be018aa7bee274c7bc5573ae3f on fix/artifact-type-tech-debt into ** on master**.


Unhelpful error message if trying to create an account with special chars in namee

Steps to repro:
try to create an account like special.chars.
Expected result:
UI that says that special characters are not allowed.
What actually happens:
UI incorrectly says that username is already taken.

@janedegtiareva @kcole16 the idea was not to allow to enter some special chars (if you chose one that is not allowed it will not appear in input). I can see from a repo that we changed text information but didn't change allowed chars in regex (@ and . was allowed before).
@kcole16 want me to change it?

@marcinbodnar Yes, please update the regex. My fault for not changing it when I updated the text.


Datalifecycle

Type of PR

Documentation changes
Code changes

Purpose

Added storage data life cycle sample to the single tech samples collection.

Does this introduce a breaking change? If yes, details on what can break
Author pre-publish checklist

[ ] Added test to prove my fix is effective or new feature works
[ ] No PII in logs
[ ] Made corresponding changes to the documentation

Validation steps

...

Issues Closed or Referenced

Closes #issue_number

References #issue_number

@devlace
could you please review this PR.

@promisinganuj
Could you please review this PR.

``

sample-output.txt
PFA the sample output of the script from local environment.


Specification of START-* events and DONE event

Specify START-* events that clearly state when a new phase is started. START-* events are only dispatched by the WorkflowEngine.
Specify generic DONE event that clearly state when a phase is done (completed, finished). DONE events are sent by external keptn services.

Documentation first

[ ] Update of keptn/specification to specify the new event types (according to current status)
[ ] Update of Postman collection to include the new event types
[ ] Design doc including sequence diagram

The specification needs to be done in the keptn/spec repository.

This issue is not relevant anymore. I closed it.


fix #1099 stackoverflow in example caused by random number generator

like the title says fixes the stackoverflow  (issue #1099 )

Wonderful, thanks for the fix!


drivers: gpio: Combined drive strength flags

Add per-pin drive strength configuration flags for hardware that
does not support per-output-state drive strength configuration.
Fixes #30329
Signed-off-by: Erik Larson<EMAIL_ADDRESS>

@mnkp could you please comment on this proposal?

@mnkp could you please comment on this proposal?


liquidsdr.org down

Just in case you didn't know, the site's been down for a few days now

It's back :+1:

Sorry for that. There was an issue with my Linode and I had to restart the server. I really should sign up for pingdom so it doesn't go down for so long. Thanks!


Peruvian Instant Darkness Powder doesn't target source

Describe the bug
peruvianpowder spell will not target the caster despite target_source: true
To Reproduce
Steps to reproduce the behavior:

Do /mgive peruvianpowder
Left-click while holding the wand
Observe how AOE doesn't target the caster if thrown at feet

Expected behavior
The AOE attached to the CustomProjectile should apply the PotionEffect to all targets within the radius, including the caster.
Screenshots
None.
Additional context
Tried shifting around targeting modes to see how I could make it work to no avail.

My theory is that it's a bug with PotionEffectActions which I reported but I'll keep this open until Nathan gets a chance to reply on that thread.

@NathanWolf Still not sure why something like this won't work. Why can't an AOE inside a projectile apply a potion effect to its caster with target_source on? Is there something tied to the projectile? I've fiddled with targeting mode to no avail.

I think it needs target_self: true (target_source vs target_caster vs target_self are all very confusing, sorry!)


CB-8395 marked unsupported tests pending on wp8

This fixes https://issues.apache.org/jira/browse/CB-8395

LGTM :+1:


No module named 'ortools.constraint_solver'

What version of OR-Tools and what language are you using?
Version: 9.6.2534
Language: Python
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
constraint solver
What operating system (Linux, Windows, ...) and version?
Windows 11 Home 22H2
What did you do?
Steps to reproduce the behavior:
1.) I performed validation of my installation from the command prompt:
D:\> python --version
Python 3.8.3
D:\> python -c "import platform; print(platform.architecture()[0])"
64bit
D:\> python -m pip --version
pip 21.2.4 from C:\Python38\lib\site-packages\pip (python 3.8)
D:\>python -c "from ortools.constraint_solver import routing_enums_pb2"

D:\>python -c "from ortools.constraint_solver import pywrapcp"

D:>python tsp.py
Traceback (most recent call last):
  File "d:\tsp.py", line 1, in <module>
    from ortools.constraint_solver import routing_enums_pb2
  File "d:\tsp.py", line 1, in <module>
    from ortools.constraint_solver import routing_enums_pb2
ModuleNotFoundError: No module named 'ortools.constraint_solver'; 'ortools' is not a package

tsp.py below:
from ortools.constraint_solver import routing_enums_pb2
from ortools.constraint_solver import pywrapcp

def create_data_model():
    """Stores the data for the problem."""
    data = {}
    # Locations in block units
    locations = \
            [(4, 4),  # depot
             (2, 0), (8, 0),  # locations to visit
             (0, 1), (1, 1),
             (5, 2), (7, 2),
             (3, 3), (6, 3),
             (5, 5), (8, 5),
             (1, 6), (2, 6),
             (3, 7), (6, 7),
             (0, 8), (7, 8),]
    # Convert locations in meters using a city block dimension of 114m x 80m.
    data['locations'] = [(l[0] * 114, l[1] * 80) for l in locations]
    data['num_vehicles'] = 1
    data['depot'] = 0
    return data

def create_distance_callback(data, manager):
    """Creates callback to return distance between points."""
    distances_ = {}
    index_manager_ = manager
    # precompute distance between location to have distance callback in O(1)
    for from_counter, from_node in enumerate(data['locations']):
        distances_[from_counter] = {}
        for to_counter, to_node in enumerate(data['locations']):
            if from_counter == to_counter:
                distances_[from_counter][to_counter] = 0
            else:
                distances_[from_counter][to_counter] = (
                    abs(from_node[0] - to_node[0]) +
                    abs(from_node[1] - to_node[1]))

    def distance_callback(from_index, to_index):
        """Returns the manhattan distance between the two nodes."""
        # Convert from routing variable Index to distance matrix NodeIndex.
        from_node = index_manager_.IndexToNode(from_index)
        to_node = index_manager_.IndexToNode(to_index)
        return distances_[from_node][to_node]

    return distance_callback

def print_solution(manager, routing, assignment):
    """Prints assignment on console."""
    print('Objective: {}'.format(assignment.ObjectiveValue()))
    index = routing.Start(0)
    plan_output = 'Route for vehicle 0:\n'
    route_distance = 0
    while not routing.IsEnd(index):
        plan_output += ' {} ->'.format(manager.IndexToNode(index))
        previous_index = index
        index = assignment.Value(routing.NextVar(index))
        route_distance += routing.GetArcCostForVehicle(previous_index, index, 0)
    plan_output += ' {}\n'.format(manager.IndexToNode(index))
    plan_output += 'Distance of the route: {}m\n'.format(route_distance)
    print(plan_output)

def main():
    """Entry point of the program."""
    # Instantiate the data problem.
    data = create_data_model()

    # Create the routing index manager.
    manager = pywrapcp.RoutingIndexManager(len(data['locations']),
                                           data['num_vehicles'], data['depot'])

    # Create Routing Model.
    routing = pywrapcp.RoutingModel(manager)

    # Create and register a transit callback.
    distance_callback = create_distance_callback(data, manager)
    transit_callback_index = routing.RegisterTransitCallback(distance_callback)

    # Define cost of each arc.
    routing.SetArcCostEvaluatorOfAllVehicles(transit_callback_index)

    # Setting first solution heuristic.
    search_parameters = pywrapcp.DefaultRoutingSearchParameters()
    search_parameters.first_solution_strategy = (
        routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)

    # Solve the problem.
    assignment = routing.SolveWithParameters(search_parameters)

    # Print solution on console.
    if assignment:
        print_solution(manager, routing, assignment)

main()

What did you expect to see
Evaluation of the TSP example.
What did you see instead?
Error relating to missing package.
I also have, installed, VC++ 2015-2022 Redistributable 64 bit version 14.36.32532

can you try a python -m pip show ortools to see if it is installed ?

can you try a python -m pip show ortools to see if it is installed ?

D:>pip show ortools
Name: ortools
Version: 9.6.2534
Summary: Google OR-Tools python libraries and modules
Home-page: https://developers.google.com/optimization/
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: c:\users\xxxxxxx\appdata\roaming\python\python38\site-packages
Requires: numpy, scipy, protobuf, absl-py
Required-by:

Can you try running python with -v ?
https://stackoverflow.com/questions/7332299/trace-python-imports

Can you try running python with -v ?
https://stackoverflow.com/questions/7332299/trace-python-imports

It seems to have started working after a reinstall and restart.


Added CSCI 39540; Flutter Dev Syllabus

Created a new folder, added a PNG image of the syllabus, and revised the missing syllabi list.
Ready to merge!

Taken care of in https://github.com/Hunter-Open-Source-Club/syllabi/commit/550e8842d27745dcc4f53e78ffd10c0d30ac43c3


[Feature Request] Set date of all files to the same as video file

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running yt-dlp version 2021.08.02
[x] I've searched the bugtracker for similar feature requests including closed ones

When I open my download folder in a file manager and sort by "Date Modified", the videos and their additional files (subtitles, thumbnail, .description, .info.json, ...) are not grouped together. This could be achieved by setting the date of not only the video file but also all additional files.
This is not the same as #601.

Bump.


Inherited options behavior clarification

@bilal-fazlani  I want to make sure I understand this feature correctly.  Currently, when an option is inherited, it looks like values can be provided in the parent command and the target command.  The option is shown in the help for both commands.  If the option is provided for both commands, the parser fails while parsing the second instance.  This isn't standard behavior in any tool I've used and I don't think the behavior is obvious to a user.  What was the intent of this feature?  Would it make sense to only show inherited options in the help for the ~parent~ child command and also to only accept them in the ~parent~ child command?

@bilal-fazlani I corrected my last question above.  This is the last thing I want to clean up before the release because I think the user experience is not clear.  It's not obvious when options for a command can be used for subcommands and it's also not obvious when options can be used in subcommands.  We can solve the latter by only displaying inherited options in executable subcommands.  The former is only a problem when a default method is used to make parent commands executable.  In those cases, we could display the options separately.  What do you think?

Yes, The inherited feature has been very ambiguous since the beginning. I just ported it as is from Microsoft library.
I agree with your suggestion.

Ok, we're on the same page then.  I'm thinking it would be more clear to obsolete the term Inherited and replace it with UseOnlyInExecutableSubcommands.  Option.Parent will still be the defining command, not the executable command, and the option will still be in the Command.Options lists for defining command and the executable subcommands. Does that seem more clear to you?


Enhance Tag Focus Logic & noborder Patch

This commit includes the following:
1- tag: Focus tag after moving client
2- tagmon: Focus monitor after moving client
3- The "noborder" patch in a superior implementation compared to those in suckless patches, as it is less prone to causing bugs. with this implementation, I have yet to encounter a single bug.

Create a static variable in config.def.h that allows to enable/disable noborder, and add an and in the if checks with this variable.


Ember splat attributes support

The rewriter will put the class attribute on the component invocation side, if the component is a css-blocks component, we should merge the invocations that have block styles with that component's analysis so conflict detection and declaration optimization work correctly.

Note: that block styles cannot straddle the splat. they must all be on one side or the other. The rewriter should ensure the rewritten class attribute is on the same side as those.

Also: Block styles after the splat seem to be ignored.


DPL-913 Submit banked PBMC tubes for cDNA prep

User story
As an SSR, I would like to be able to submit a collection of tubes containing banked PBMCs for cDNA prep, as part of the "scRNA Core" pipeline.
Who are the primary contacts for this story
Katy, Lesley, Abby
Who is the nominated tester for UAT
Lesley, Abby, SSR?, someone from BioResource?
Acceptance criteria
To be considered successful the solution must allow:

[ ] I can submit "LRC Bank Seq" and "LRC Bank Spare" tubes for cDNA prep (new pipeline)
[ ] They can have a mixture of Studies and Projects (cost codes).
[ ] Update documentation, to say relevant submission template.

Outstanding questions for stakeholders

How will the communication happen here? e.g. When they determine there are enough banked cells in storage to go down the pipeline, BioResource team create an RT ticket for the SSR, listing out the tubes?
Who knows / how do they know which cost code (Sequencescape "Project") is relevant for each tube? (Study should be known as it was specified on the manifest, when the samples were first uploaded prior to cell extraction).
OK if this is a normal "manual" submission, managed by SSRs?
Should the submission be just for the thawing step (before the 2nd entry point to the pipeline), or for the whole of the cDNA preparation step? Needs stakeholder and dev agreement. Considerations: what chunks billing will happen in; whether pool plates from SeqOps-banked cells will ever be combined with pool plates from faculty.
Should it have a "library type"? Assume no, as we are not creating "libraries". Library type will come in in the next submission - when enters the existing 5' Chromium pipeline.
How should we refer to the "middle" section of the pipeline - is "cDNA prep" OK? (as opposed to the first section, "cell extraction", and the final section, "library prep" or currently "Bespoke Chromium 5 prime")

Outstanding questions for devs

Should the submission be just for the thawing step (before the 2nd entry point to the pipeline), or for the whole of the cDNA preparation step? Needs stakeholder and dev agreement. Considerations: what chunks billing will happen in; whether pool plates from SeqOps-banked cells will ever be combined with pool plates from faculty.

How will the communication happen here? e.g. When they determine there are enough banked cells in storage to go down the pipeline, BioResource team create an RT ticket for the SSR, listing out the tubes? To be determined for the SSRs and BioResource

Who knows / how do they know which cost code (Sequencescape "Project") is relevant for each tube? (Study should be known as it was specified on the manifest, when the samples were first uploaded prior to cell extraction). To be determined for the SSRs and BioResource

OK if this is a normal "manual" submission, managed by SSRs? To be determined for the SSRs and BioResource

Should the submission be just for the thawing step (before the 2nd entry point to the pipeline), or for the whole of the cDNA preparation step? Needs stakeholder and dev agreement. Considerations: what chunks billing will happen in; whether pool plates from SeqOps-banked cells will ever be combined with pool plates from faculty. If there is just one submission and the cell count is poor post-thawing and as a consequence, it doesn't move forward for cDNA prep will the customer still be billed for cDNA prep (assuming contingency also fails)? Also, how would this look if we had to use contingency samples? Can we discuss combining with SeqOps and faculty plates when we meet on Monday it will be useful to have the process map alongside and there are multiple considerations. The risk of cross-study swaps is high.

Should it have a "library type"? Assume no, as we are not creating "libraries". Library type will come in in the next submission - when enters the existing 5' Chromium pipeline. There are alternative protocols for cDNA prep which will need to be defined here. e.g. 5' or 3' for example

How should we refer to the "middle" section of the pipeline - is "cDNA prep" OK? (as opposed to the first section, "cell extraction", and the final section, "library prep" or currently "Bespoke Chromium 5 prime") 'cDNA prep' is fine

More thoughts on q4 above:

As I understand it, whether the submission covers both thawing and cDNA prep, or whether there is a submission for each of those steps, shouldn't impact the billing report. The report is based on what gets recorded as sequenced (iseq_flowcell table in the MLWH) - so if wells are failed, they shouldn't be recorded as sequenced, so won't be in the billing report.
Whether we are going to represent a pool as a compound sample (DPL-916) may impact both the billing report, and this decision about number of submissions.

Notes from meeting yesterday:
Library types:

Library type - agreed makes sense to create some scRNA-specific versions of the 10X library types, so that the scRNA data can be distinguished from the existing 10X data. Can use the same library type for the cDNA prep stage as for the library prep stage. Discussed adding 'HT' to the library type name, and that 'HT' versions are already being created for the existing pipeline in this story: https://github.com/sanger/sequencescape/issues/3883.
Need to clarify exactly what library types we'll need. Suggest to start off with we could just have "scRNA Chromium single cell 5 prime HT" or similar. (above linked story includes 'v2' in the name - ask why)

Submissions / selecting samples to go onto the 10X chip together:

Also see related comment here - https://github.com/sanger/limber/issues/1388#issuecomment-1764736960
Still lots of uncertainty around this.
BioResource team are managing the banked PBMC tubes. Once there are enough tubes available, a group of 100-160 banked PBMC tubes, destined for the same 10X chip, will be taken out of the freezer together.
Faculty will be sending plates of ready-pooled PBMCs to SeqOps, ready to go on the 10X chip.
It's possible faculty and SeqOps pool plates could be combined onto one chip - could lower costs, but it complicates things (from LIMS point of view, and also from lab point of view, managing tags).
If the above happens, how are the faculty plates coming in coordinated with the tubes being taken out of the freezer?
Discussed possibility of enforcing separate processing of samples from different entry points, at least for MVP.
Discussed whether this submission could specify which samples would end up being pooled together for sequencing. Only if a) they consider the donor id, to prevent 2 samples from the same donor being pooled together, and b) if they are not combined with plates coming in at either the 2nd or 3rd entry points.
Another point at which sequencing pools could be specified is the submission for library prep (aligning with 3rd entry point).
Trade off between more admin and more flexibility. Could be max number of submission points:

Fresh blood for cell extraction (automated submission)
Frozen PBMCs for thawing and pooling (manual submission)
Pool plates for cDNA prep (manual submission)
cDNA for library prep (manual submission)
Tubes for sequencing (manual submission)

In meeting 25/10/23 with R&D we discussed library types. Decision confirmed by Lesley today:

These library types are the best placed to align with the 007C project. As discussed previously, in the early days of the project there will be cross-over with what were bespoke activities and 007C until the point at which they are aligned. However, in the interim we can filter by date and/or study.
Chromium single cell 5 prime HT v2 will the library type we immediately require for 007C.

Chromium single cell 3 prime HT v3
Chromium single cell 5 prime HT v2
Chromium single cell BCR HT
Chromium single cell TCR HT

Added Integration Suite DPL-913 MR on GitLab .

Updated the documentation with the Submitting banked PBMC tubes for Donor Pooling and cDNA Prep section.

Gist: Running integration suite locally for the 'scRNA Core entry point 1 - blood' scenario

Gist: Database Inspection for Donor Pooling and cDNA Prep submission

Deployment note: To enable WIP when we deploy Sequencescape to UAT, we need the following executed on the host:
WIP=011_scrna_core_cdna_prep_submission_templates,014_limber_scrna_core_cdna_prep_request_types,011_scrna_core_cdna_prep_plate_purposes,008_scrna_core_cdna_prep_tube_purposes,011_scrna_core_catalogue bundle exec rake record_loader:all


Create a custom _meilisearch_id field

#612
When indexing an entry to Meilisearch, the id field used to be transformed with a prepend of its collection name.
For example in a restaurant collection:
{
   "id": 1,
   "name": "pizza mania"
}

Would become in Meilisearch:
{
   "id": "restaurant-1",
   "name": "pizza mania"
}

With this PR, instead of overwriting the id field an additional field is added _meilisearch_id.
The entry in Meilisearch is thus transformed like this:
{
   "id": 1,
    "name": "pizza mania",
    "_meilisearch_id": "restaurant-1"
}

bors merge

@bidoubiwa is there a way to make the primary key different from _meilisearch_id?
I believe this forces it to be _meilisearch_id and does not allow any other field to be the primary id. E.g. in the case where the documents are being indexed by slug and not by an incremental id, this totally breaks that implementation.

Sure @bidoubiwa I'll make sure to create a new issue with this.

Could you use transformEntry (see readme for usage) and change the id field of your document with the slug ?

I was using transformEntry to bypass this but it's not the cleanest workaround due to the collectionName hardcoded prefix.
A solution is to use a filter to query the slug in this case instead of getting the pk directly, but should not behave with the same performance. Anyway I'll move this discussion onto a new issue.
Thank you! 🙏


Sometimes clicking on item drops another item instead.

In multiplayer mode(haven't tested in singleplayer), on clicking an item sometimes results in dropping another item or even an item I currently don't have. I hope this problem can be fixed so the feature is much more usable. Thank you!

Which version are you using? Also if you could provide steps to reproduce, I can further debug this.

I'm using v1.2.1-alpha. I tested it in singleplayer mode today and it happened again. I had 12 different types of items and when I press tab and click the item I want to drop in the middle section item list (not the one on top of screen), it drops other items. I captured screen when it happened, let me show you.
https://drive.google.com/file/d/1ZueDdGJj2vxpYPkQ2BnYwP3W98HgrMoP/view?usp=sharing

Based on the code I'm seeing the game reuses item icons. Simply keeping track of the ItemIndex doesn't guarantee ItemIcons have the same ItemIndex as the DropItemController attached to it.


The code doesn't run on real hardware raspi 2 B

I downloaded and built the "master" branch code, or part 06 branch code.
It runs smoothly on qemu-system-arm simulator,
but when I renamed the kernel.img to kernel7.img and copied to the raspbian SD card of the real raspi 2 hardware, it doesn't run at all, just blank black screen (I disabled the raspi colour splash test). I also tried to change the gpu.c BLACK colour to BLUE, it worked in simulator, but doesn't work on real hardware.
To double check, when I switch back the raspbian kernel7.img, the raspberry pi runs without issue. The hardware is raspberry pi 2 B model.

Just an update: after many trial-and-error, it starts to work on real pi 2 (v1.1) now.  Few fix

when setting integer & fractional part of uart baud rate, the parameters based on 3MHz UART_CLOCK doesn't work, need to re-calculate the integer and frac part parameters based on 42MHz uart clock. it helps debugging tremendously.

The return address from mail box fbinfo.buf doesn't work (exceeded 1G address limit), it needs to be add by 0x40000000 (or do the bit wise AND 0x3FFFFFF) to translate to physical address.

color depth 24 doesn't work properly. Need to set to 16 (need to modify pixel accordingly). No matter how I change the frame buffer parameters, the return buffer size is always x96000, which is corresponding to 640x480x2 bytes, but the fbinfo.pitch seems changing with different parameters. The code returned from mail box message shows success. I wonder if it is caused by raspi config.txt file.

After these changes, it can print "hello world!" correctly on the screen. However, there are still other issue: ATAG returns 0 mem_size. I temporarily fix it by hard code the mem-size 128MB when debugging gpu code, need to go back to investigate frame buffer and ATAG issue.
By the way, I downloaded most recent raspbian and installed on pi 2 to create boot SD card and then replace the kernel7.img with this img. I don't know if it is because the new raspbian that caused the color depth and ATAG issue.

update: now I can set color depth to 24 or 32, also different screen size. I fixed the issue by merging the frame buffer initialization mail box message and the request frame buffer mail box message into one send_messages() call in the framebuffer_init(). For some reason, if send_messages() is being called twice or more, it results in strange behaviour, I suspect the heap  kmalloc(bufsize) or kfree(msg) still have bugs.
Nevertheless, now I can successfully set and request the frame buffer on real hardware.
Here is the code
tags[0].proptag = FB_SET_PHYSICAL_DIMENSIONS;
tags[0].value_buffer.fb_screen_size.width = 800;
tags[0].value_buffer.fb_screen_size.height = 600;
tags[1].proptag = FB_SET_VIRTUAL_DIMENSIONS;
tags[1].value_buffer.fb_screen_size.width = 800;
tags[1].value_buffer.fb_screen_size.height = 600;
tags[2].proptag = FB_SET_BITS_PER_PIXEL;
tags[2].value_buffer.fb_bits_per_pixel = COLORDEPTH;
tags[3].proptag = FB_ALLOCATE_BUFFER;
tags[3].value_buffer.fb_screen_size.width = 0;
tags[3].value_buffer.fb_screen_size.height = 0;
tags[3].value_buffer.fb_allocate_align = 16;
tags[4].proptag = NULL_TAG;

if (send_messages(tags) != 0) {
    return -1;
}

// copy buffer info
fbinfo.width = tags[0].value_buffer.fb_screen_size.width;
fbinfo.height = tags[0].value_buffer.fb_screen_size.height;
fbinfo.chars_width = fbinfo.width / CHAR_WIDTH;
fbinfo.chars_height = fbinfo.height / CHAR_HEIGHT;
fbinfo.chars_x = 0;
fbinfo.chars_y = 0;
fbinfo.pitch = fbinfo.width*BYTES_PER_PIXEL;

fbinfo.buf = tags[3].value_buffer.fb_allocate_res.fb_addr;
fbinfo.buf += 0x40000000;

fbinfo.buf_size = tags[3].value_buffer.fb_allocate_res.fb_size;

After some help from Raspberry pi forum, I fixed the ATAG issue. Because I use new version Raspbian to create boot SD card, the new boot loader doesn't create ATAG by default. After I moved all device tree files into a different folder, boot loader creates the ATAGS, then everything runs correctly on PI2 now.
Here is the short recap of the fix:

UART fix in uart.c
mmio_write(UART0_IBRD, 26);  // New raspi uart clock is 48M Hz (not 3M Hz)
mmio_write(UART0_FBRD, 3);
Fix the kfree()
....
// try to coalesce segments to the right
while(seg->next != NULL && !seg->next->is_allocated) {
seg->segment_size += seg->next->segment_size;
if(seg->next->next != NULL) seg->next->next->prev = seg;
seg->next = seg->next->next;
}
...
Merge the frame buffer initialization and buffer request into one message call.
Add the bfinfo.buff address by 0x40000000, the codes are in previous post.
For new raspbian, delete or move all the device tree related files:0 /boot/*dtb etc.


Minimize and simplify the breadcrumbs

On the GO project, the current design of breadcrumbs is too heavy, particularly on mobile devices, where they also break the design.

We could consider a less prominent approach, that would still serve the objective of telling users where they are, but won’t take too much space or confuses them with the main navigation.

Besides that, on some cases showing the full title of the page on the breadcrumbs does not work. Short or alternative titles should be allowed. For example, on newgo.wfp.org this path
home> About wfp >Operations Services Department > Emergency Preparedness and Support Response Division > Emergency Preparedness and Response Branch > Operational Readiness Management Unit

could be improved by using the acronyms instead
home> About wfp > OS > OSE> OSEP> Operational Readiness Management Unit

After the initial proposal from @danielapizzurro and a few minor improvements, we've agreed on the following two variants, which should replace the existing breadcrumbs.
Standard

Integrated

Specification

Text size: 0.875rem (14px)
Colours:

Arrow: $grey;
Anchor: $blue-dark (Integrated: $blue-light);
Label: $text-color (integrated: $white);

Home icon size: 16 x 16 (can stay exactly the same);

Below our implementation of the simplified breadcrumb for the GO project:
.wfp-breadcrumbs-light {
  @extend %seg-control;
  height: auto;
  margin: .25em 0;
  text-align: left;

  .breadcrumbs--item {
    @extend %seg-control--item;
    display: inline-block;
    border: 0;
    margin: 0;
    &:first-child {
      border-left: 0;
    }
  }

  .breadcrumbs--link {
    @extend %seg-control--link;
    padding: 0 0.4em 0 0;
    font-weight: normal;
    color: darken($primary, 12%);

    &:after {
      content: '\203A';
      color: $grey;
      font-size: 1.125em;
      margin-left: .5em;
    }

    &.home {
      [class*="icon-"] {
        margin: 0;
      }
      svg {
        path {
          fill: $blue-dark;
        }
      }
      &:hover {
        background-color: inherit;
        color: $primary;
      }
      &:after {
        margin-left: .25em;
      }
    }

    &:hover {
      background-color: inherit;
      color: $primary;
    }
  }

  .breadcrumbs--last {
    @extend %seg-control--link;
    display: inline-block;
    padding: 0 0.4em 0 0;
    font-weight: normal;
    color: $grey-dark;
    cursor: default;

    &:hover {
      background-color: inherit;
      color: inherit;
    }
  }
}

This is to be paired with the following Drupal template, so it will need some adaptation when porting it in the upstream project:
{#
/**
 * @file
 * Theme override for a breadcrumb trail.
 *
 * Available variables:
 * - breadcrumb: Breadcrumb trail items.
 */
#}
{% if breadcrumb %}

{{ 'Breadcrumb'|t }}

      {% for item in breadcrumb %}
        {% if loop.index == 1 %}

            <svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="16" height="16" viewBox="0 -5 24 24">
              <path d="M10 20v-6h4v6h5v-8h3l-10-9-10 9h3v8z"></path>
            </svg>

        {% elseif item.url %}

            {{ item.text }}
        {% else %}

{{ item.text }}
        {% endif %}

      {% endfor %}

{% endif %}


Mock Bank Transfers do not work despite whitelisting server IP Address

Hi Flutterwave,
I have been trying to mock a bank transfer to a Nigerian Bank Account, but I keep getting the error, "This transaction cannot be competed, please contact the administrator". Please also try to make your forum question and answers forum open to the public. There seems to be a famine of information on how to resolve Flutterwave integration problems and your forum which could do a world of difference seem not to be working at all. I registered with two different email addresses but none got any confirmation email.
Please consider making Youtube videos just like Stripe on integration of Flutterwave.  I will be looking forward to your response.

Hello @jzagalo,
Bank transfers(payouts) are not supported in this library. Kindly contact our support team for help with your integration.
Thank you for your feedback.


Add retcode error checking, solc_binary argument

Mythril compile_solidity:

doesn't use its own solc_binary argument
doesn't offer NIX signal tracking if solc has internal fatal issues,
such as SIGSEGV

This commit adds a more verbose message for a fatal error in solc and uses the solc_binary argument. It may be worthwhile to set the default argument for solc_binary in the function declaration.
Note: My system has a strange issue in which my solc will segfault when run only from the Mythril subprocess. Running the same code as below in the Python repl. It wasn't clear what the problem Mythril was running into was, as it just displayed a blank string on the segfault. This helps others debug this same issue, and fixes a bug in which solc_binary was not passed in.
Tested on Python 3.6.3, Arch Linux x64 4.14.5-1.

Thanks @10a7

Included in release 0.8.25.


TIFF/EXIF tags: Fix potential 32bit unsigned integer overflow

...for rational type in metadata.
I ran into an issue with for example using 0.000493 secs for exposure time. This change fixes this.

Thanks! Let me give some time to review the PR

This is the code-snippet I used to investigate this issue: godbolt
Line #10 lets you toggle between my fix and the original code
It may be of help


[Enhancement] Improve broker load error message

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

This PR is to refactor broker load error message.
When broker load and error message type is ETL_QUALITY_UNSATISFIED, there is a extra field TrackingSQL to see the detailed error message, but many users don't know this. So I add a message "You can find detailed error message from running TrackingSQL" at the error message ErrorMsg field.

And in detailed error message, I rephrase it a little bit.

// This one means it failes to fill a csv field.
Error: The field (name = pair, pos = 2) is out of range. Type: ARRAY<VARCHAR(65533)>, Value: [ "1", "2" ]. Row: 6817;a;"[ ""1"", ""2"" ]"

// This one means value count does not match column count.
Error: Value count does not match column count: expected = 3, actual = 5. Column delimiter: 59, Row delimiter: 10. Row: 6418;a\b;"[ ""a\\b"", ""c\""d"" ]"
6419;"c""d";

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:disappointed: fail : 14 / 38 (36.84%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/csv_scanner.cpp
14
38
36.84%
[28, 42, 43, 56, 58, 330, 331, 333, 337, 374, 423, 425, 426, 429, 430, 431, 436, 437, 440, 456, 457, 458, 461, 462]

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)


Sws extension take Fx big

Hi,
I found an issue with the copy pasting of take fx with SWS extension.
Scenario : I have an "X" media item where I have a 6 in 6 out plugin as take fx. I have a "y" media item with no plugins. Now I want to copy the plugins from the "X" item to "Y" clip item.
For this I use these shortcuts.
SWS/S&M: Copy FX Chain (Depending on Focus)
and then
SWS/S&M: Paste FX Chain (Depending on Focus)
When I do this the "Y" media item only has 2in6 out in the plugin pin configuration. Then I have to manually open the plugin and then modify the plugin I/O config to 6 in 6 out.
But if I just cmd drag and drop the take fx from "X" item to "Y" item then I dont have this problem. This is why I am sure that this is a SWS extension bug.
Can you pls fix this ?
svijayrathinam is online now  Report Post

Confirmed.
Reason (@devs):
The Copy action copies the TAKEFX chunk but misses the TAKEFX_NCH line becuase it's outside of the chunk.
(https://i.imgur.com/us4z1Kh.jpg)
I've been looking into it but don't know how to fix currently as it's (again) dealing with the ChunkParserPatcher which I don't fully understand. I'll continue trying but if anyone else wants to tackle this, it's also welcome of course.

Hi,
asking just in case, have you tried pressing Play after you've pasted the FX chain ?
Here at least when I do the above steps it also pastes with 2in/6out as you say, but it updates the In configuartion to 6 automatically when I press Play.
!(https://i.imgur.com/IA20wnd.gif)

Yes.. I have tried that..This doesn't happen to me...Check the below Link
https://drive.google.com/file/d/11pG2HUftlQsHeVnH1T7HdtEzJ9E3Oa2b/view?usp=sharing

Thanks for the screencapture.
Could you please test with the version linked below and let us know if it works for you ?
(I don't have the plugin you're using, so I can't test with it here.)
https://www.dropbox.com/s/r9y1xobok16mw9u/sws_osx.dmg?dl=1
Note:
It's a version only meant for testing, so don't forget to reinstall the current official SWS version afterwards.
Thanks.

Hi Thank you very much for your work. I tried this. But the problem is still there. If you want the plugin you can download the trial here
http://www.iosono-sound.com/vstaax-plug-ins/
Its not just with this plugin but with every plugin.

Ah, I almost missed it in the screencapture, you have a default preset set for the plugin, this means the correct channel assigment is done automatically when inserting the plugin, right ?
Not sure yet, but seems to me what's happening is that the the default preset (and thus the channel assignment) is not propagated when copy / pasting via the SWS action. I'll try to investigate further.
(As always, help from other SWS collaborators appreciated).

@devs
The original issue (as I understand) is that  the TAKEFX_NCH line is not part of the TAKEFX chunk and so doesn't get copied via SWS/S&M: Copy FX Chain (Depending on Focus) resulting in setting the pasted item FX chain back to 2 channels.

My patch seems to fix (also confirmed by OP above) but I'm hesitant submitting as PR because of risk of corrupting projects (when modifying chunk) so I'd appreciate review / help.
Would probably better to fix directly in SNM_FXChainTakePatcher but I don't good enough to do that.

Justin says it's ok..
https://askjf.com/index.php?q=4041s

Hi This take fx channel bug is still not fixed in ver 2.9.8.
Can you pls fix this ?
I reported this long time ago and nofish made a temporary fix. But Now I find new feautures in SWS extension ver 2.9.8 that I want to use. But the old bug still remains..Can you pls fix this ? if not..@nofish could you pls make another temporary fix with the new version ?


Finalize Broker split

Move xplat initialization code into Broker
Remove static code from CommonStartup
Remove IExitService that has been creating duplicated messages in the log
Use IWebHost.Start instead of IWebHost.Run in service
Use IApplicationLifetime cancellation tokens instead of custom ones
Use configuration after it has been read from config file
Use urls startup parameter instead of legacy server.urls
Replace start.as parameter with startup:isService

👍

@karthiknadig , can you also please take a look?

One minor change.
👍


Text explanations

Added a function that converts MACCS descriptor explanations to text

Looks like tests still fail @hgandhi2411


FILEUPLOAD-324 - Use java9 profile on JDK 9 and later

Coverage remained the same at 78.359% when pulling d0e1b070187f2bc4fb91df7e1054090364b0b9f5 on arturobernalg:feature/FILEUPLOAD-324 into 32eade4883d367a2103ea9a0be68f057604adb37 on apache:master.

Could you, please, add a reference for the trouble with Jacoco in Java 13? So, that there is something, that we can track and use as a source for the decision to revert.

Hi @jochenw
since fileupload inherit from commons parent use the version 0.8.6 that support java 13 no need skip jacoco.
Remove the comment
TY

Okay, but another question: As we are inheriting from the parent, wouldn't it be wise to retain the profile id ("java9", rather than "java9+")?

HI @jochenw
Well, maybe you're right. Maybe I haven't thought it through. Rollback madeed
YY

As far as I can tell, this is now reduced to introducing a new profile "java13", that is activated when building on Java 13, but not later.
Okay, but that profile is empty. So, what's the point?

As far as I can tell, this is now reduced to introducing a new profile "java13", that is activated when building on Java 13, but not later.
Okay, but that profile is empty. So, what's the point?
Hi @jochenw . you are right, now seems logical close this PR


autocomplete does not work on the server

I used google maps api for autocomplete of places.
Everythings works in localhost, but when i publish and upload it on server, it doesn't work...Why it happen? the dropdown of places does not show.

I'm finding that the autocomplete isn't working at all for me including localhost.  can you give me the code you're using to get even that far?

use this :

and :
`$('#us1').locationpicker({
  inputBinding: {
    latitudeInput: $('#lat'),
    longitudeInput: $('#lon'),
    radiusInput: $('#cover_area'),
    locationNameInput: $('#uaddress')
  },
    enableAutocomplete: true
});`

I didn't have enableAutocomplete: true which seems to have at least changed the input for city but for some reason it is still not auto suggesting locations as you enter in the details.  Do I need a google api key?
`

					Location:
					<input type="text" id="us2-address" style="width: 200px"/>
					Radius: <input type="text" id="us2-radius"/>

					Lat.: <input type="text" id="us2-lat"/>
					Long.: <input type="text" id="us2-lon"/>

`
	$('#select-location-modal').on('shown.bs.modal', function (e) {
		$('#us2').locationpicker({
			location: {latitude: <?php echo $currentLat; ?>, longitude: <?php echo $currentLng; ?>},
			radius: 300,
			inputBinding: {
		        latitudeInput: $('#us2-lat'),
		        longitudeInput: $('#us2-lon'),
		        radiusInput: $('#us2-radius'),
		        locationNameInput: $('#us2-address')
		    },
				enableAutocomplete: true
		});
	});

You don't need a Google api but you have to attach this :
`

`

I am including it though I'm getting the following errors in the console:

Google Maps API warning: NoApiKeys https://developers.google.com/maps/documentation/javascript/error-messages#no-api-keys

util.js:208 Google Maps API warning: SensorNotRequired https://developers.google.com/maps/documentation/javascript/error-messages#sensor-not-required

js?sensor=false&libraries=places:34 This service requires an API key.  For more information on authentication and Google Maps Javascript API services please see: https://developers.google.com/maps/documentation/javascript/get-api-key

I also have this problem and I don't know where is the problem but is work on the localhost but not on the server

What packages / systems you have running on your local server that isn't on the web server?
Get Outlook for iOS
On Thu, Sep 8, 2016 at 4:02 PM +0100, "Mseif"<EMAIL_ADDRESS>wrote:
I also have this problem and I don't know where is the problem but is work on the localhost but not on the server
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@T3RON and @colerua try to create API key in google developers console.

Create project.
Go to credentials.
Create credentials and select API key. (use it API key)
Open your new credential and set HTTP referrers (web sites), then add link of your site. (it important)
Enable Google Maps JavaScript API in library.

Be sure that you include google maps library to your project

And of course property enableAutocomplete must be enabled
$('#mapId').locationpicker({
   inputBinding: {
      locationNameInput: $('#nameInputId')
   },
   enableAutocomplete: true
});

P.S. It helped me.

@Sumragen Tanks it's work

I must be missing something as this didn't work for me.  Instead the map isn't appearing at all now.

ok, fixed that.  was the http rule but the autocomplete is still not working :(

Here is a best and simple autocomplete using PhpMySql and JQuery Ui plugin Link

 1.  Go to google map credentials.  https://console.developers.google.com/apis/credentials
 2. Create credentials and select API key. (use it API key)
 3. Click on HTTP referrers (web sites).
 4. add you web site link.
 5. Enable Google Maps JavaScript API in library.


Improve compile time

Compilation with some features makes the crate take ages to compile. Need to figure out why, and try to mitigate it.
Experiments need to be done to figure out what is taking all the time, I suspect codegen is a big timewaste.
Related to #203 and #204

These tests should be done on windows and linux

Should try to make a reproducible benchmark to keep track of this, I feel like the compile time has improved with rustc versions

❯ summarize summarize --percent-above 10 twitch_api-0025240.mm_profdata
+------------------------------+-----------+-----------------+--------+------------+---------------------------------+
| Item                         | Self time | % of total time | Time   | Item count | Incremental result hashing time |
+------------------------------+-----------+-----------------+--------+------------+---------------------------------+
| LLVM_module_codegen_emit_obj | 59.75s    | 53.079          | 59.75s | 256        | 0.00ns                          |
+------------------------------+-----------+-----------------+--------+------------+---------------------------------+
| LLVM_passes                  | 15.30s    | 13.594          | 15.30s | 1          | 0.00ns                          |
+------------------------------+-----------+-----------------+--------+------------+---------------------------------+
| codegen_module               | 11.34s    | 10.076          | 14.73s | 256        | 0.00ns                          |
+------------------------------+-----------+-----------------+--------+------------+---------------------------------+
Total cpu time:<PHONE_NUMBER>93s
Filtered results account for 76.749% of total time.
+----------------------------+-----------------+
| Item                       | Artifact Size   |
+----------------------------+-----------------+
| codegen_unit_size_estimate | 3781590 bytes   |
+----------------------------+-----------------+
| crate_metadata             | 53295483 bytes  |
+----------------------------+-----------------+
| dep_graph                  | 158831053 bytes |
+----------------------------+-----------------+
| linked_artifact            | 323684368 bytes |
+----------------------------+-----------------+
| object_file                | 251006424 bytes |
+----------------------------+-----------------+
| query_cache                | 93758624 bytes  |
+----------------------------+-----------------+
| work_product_index         | 14794 bytes     |

cargo llvm-lines
  Lines                  Copies                Function name
  -----                  ------                -------------
  7150872                136439                (TOTAL)
   715551 (10.0%, 10.0%)  17022 (12.5%, 12.5%) core::result::Result<T,E>::map_err
   544038 (7.6%, 17.6%)    2349 (1.7%, 14.2%)  <serde_json::de::SeqAccess<R> as serde::de::SeqAccess>::next_element_seed
   461914 (6.5%, 24.1%)     864 (0.6%, 14.8%)  <&mut serde_json::de::Deserializer<R> as serde::de::Deserializer>::deserialize_struct
   326040 (4.6%, 28.6%)     247 (0.2%, 15.0%)  <twitch_api::eventsub::event::_::<impl serde::de::Deserialize for twitch_api::eventsub::event::EventType>::deserialize::__Visitor as serde::de::Visitor>::visit_enum
   261818 (3.7%, 32.3%)    2405 (1.8%, 16.8%)  <serde_path_to_error::de::MapAccess<X> as serde::de::MapAccess>::next_value_seed
   226709 (3.2%, 35.5%)     316 (0.2%, 17.0%)  <&mut serde_json::de::Deserializer<R> as serde::de::Deserializer>::deserialize_any
   213487 (3.0%, 38.5%)    2349 (1.7%, 18.7%)  <serde_path_to_error::de::SeqAccess<X> as serde::de::SeqAccess>::next_element_seed
   208653 (2.9%, 41.4%)    2496 (1.8%, 20.6%)  <serde_path_to_error::de::TrackedSeed<X> as serde::de::DeserializeSeed>::deserialize
   196638 (2.7%, 44.1%)    2112 (1.5%, 22.1%)  <serde_ignored::MapAccess<X,F> as serde::de::MapAccess>::next_value_seed
   156592 (2.2%, 46.3%)     160 (0.1%, 22.2%)  <twitch_api::eventsub::_::<impl serde::de::Deserialize for twitch_api::eventsub::EventSubscriptionInformation<E>>::deserialize::__Visitor<E> as serde::de::Visitor>::visit_map
   144270 (2.0%, 48.3%)    2403 (1.8%, 24.0%)  <serde_json::de::MapAccess<R> as serde::de::MapAccess>::next_value_seed
   143722 (2.0%, 50.3%)    2053 (1.5%, 25.5%)  <serde_ignored::SeqAccess<X,F> as serde::de::SeqAccess>::next_element_seed
   128832 (1.8%, 52.1%)     244 (0.2%, 25.7%)  <twitch_api::eventsub::_::<impl serde::de::Deserialize for twitch_api::eventsub::Status>::deserialize::__Visitor as serde::de::Visitor>::visit_enum
    87906 (1.2%, 53.4%)     289 (0.2%, 25.9%)  <&mut serde_json::de::Deserializer<R> as serde::de::Deserializer>::deserialize_enum
    76217 (1.1%, 54.4%)     364 (0.3%, 26.1%)  <serde_json::de::MapAccess<R> as serde::de::MapAccess>::next_key_seed
    75925 (1.1%, 55.5%)    1247 (0.9%, 27.1%)  serde_path_to_error::de::<impl serde::de::Visitor for serde_path_to_error::wrap::Wrap<X>>::visit_seq
    73786 (1.0%, 56.5%)    1212 (0.9%, 27.9%)  serde_path_to_error::de::<impl serde::de::Visitor for serde_path_to_error::wrap::Wrap<X>>::visit_map
    70721 (1.0%, 57.5%)     864 (0.6%, 28.6%)  <serde_path_to_error::de::Deserializer<D> as serde::de::Deserializer>::deserialize_struct
    67832 (0.9%, 58.5%)     160 (0.1%, 28.7%)  <twitch_api::eventsub::_::<impl serde::de::Deserialize for twitch_api::eventsub::EventSubscriptionInformation<E>>::deserialize::__Visitor<E> as serde::de::Visitor>::visit_seq
    61713 (0.9%, 59.3%)     332 (0.2%, 28.9%)  <&mut serde_json::de::Deserializer<R> as serde::de::Deserializer>::deserialize_str
    45126 (0.6%, 60.0%)    2232 (1.6%, 30.6%)  serde_path_to_error::wrap::Wrap<X>::new
    41007 (0.6%, 60.5%)    1005 (0.7%, 31.3%)  core::result::Result<T,E>::map

A lot of the time comes from llvm, and most of it seems to be serde, I wonder if there's a neat way to avoid this, perhaps making json the only supported serialization/deserialization unless something else is wanted, thus skipping serde

Maybe miniserde or serde-lite could help?
miniserde doesn't have all the features we're using currently but serde-lite could maybe be interesting, but doesn't either have all the features we use

Removing a single module in helix makes quite a difference, I wonder if there's something bad we're doing that can be fixed by forcing the compiler to not inline

Removing a single module in helix makes quite a difference

Maybe modules could be enabled one-by-one? I think it's rarely the case that you need every helix/eventsub/... module.

Maybe modules could be en-/disabled one-by-one? I think it's rarely the case that you need every helix/eventsub/... module.

That sounds very easy, could possibly have multiple scopes, and with the new feature where cargo tell you that an item is behind a cfg, quite discoverable
- bits
- channels
- charity
- chat
- client
- clips
- eventsub
- games
- goals
- hypetrain
- moderation
- points
- polls
- predictions
- raids
- request
- response
- request
- schedule
- search
- streams
- subscriptions
- tags
- teams
- users
- videos
- whispers

that's a lot of feature flags though :3

the problem is not serde or serde_derive, the problem is the code it generates @simonsan, but we could definitely switch to using serde_derive to make compilation more parallel. Related https://github.com/serde-rs/serde/issues/2584#issuecomment-1685760722

cargo build -F all (1 thread)
cargo build -F all (8 threads)


Added Trubrics Feedback Collector to DemoGPT

Hey @melih-unsal, this PR enables you to gather and analyze feedback for the results and sources of the DemoGPT model by integrating the Trubrics SDK. You are then also able to analyse the feedback received through registration on the Trubrics platform.
Setup
To store feedback in your Trubrics account, follow these steps:

Register for a free account with Trubrics here
Input your Trubrics email & password into st.secrets

Preview
Here is an example of gathering and storing feedback using the faces feature for the generated code from DemoGPT.

We have also integrated feedback into the Streamlit app with the generated model:

In the following image, we can see the feedback saved in our Trubrics account.

Useful links

See here for information on how to integrate Trubrics with Streamlit.
Here you can find descriptions of the various feedback components (thumbs, faces, custom text).

We believe it would be fascinating to investigate the possibility of the feedback collector being automatically incorporated into the code when the model is generated, as opposed to being added externally as we did. Please let me know what you think.
If you have any questions, feel free to contact me.


setCurrentView(): Improve example

CRM 9.0 on prem.
osGrid.getViewSelector().setCurrentView(...) throws error if trying to set within form onLoad, if grid not yet loaded.
It should be noted that grid onLoad event must be waited for:
            let changeDefaultView = () => {
                //Can't change view directly, must wait onload. Otherwise GetViewSelector may not be immediatelly available and throw error: TypeError: Cannot read properties of null (reading 'getViewSelector')
                console.log(`Change view for ${osGrid.getName()}`);
                osGrid.getViewSelector().setCurrentView({
                    entityType: 'savedquery',
                    id: "{A41DE317-017A-EC11-A2EF-00155DE7E766}",
                    name: "Outsourced Services Including Unedited"
                });
                osGrid.removeOnLoad(changeDefaultView);
            };
            osGrid.addOnLoad(changeDefaultView);

Moreover, setCurrentView object entityType documents that it must be a Number. And conveniently, both actually works: string entityType: 'savedquery' or Number entityType: 1039
I'v posted #2990 too - it actually returns object where entityType is string. If I choose to save this variable and afterward set it - it may mislead developer who thinks that getCurrentView returned object cannot be used on setCurrentView. So perhaps would be nice to note that entityType can be either string or Number.

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

ID: 744c1b98-3e68-6091-981f-c146152aaaeb
Version Independent ID: 41ac2c71-1869-54b1-19eb-556bffa9f1ce
Content: getCurrentView (Client API reference) in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/grids/viewselector/getCurrentView.md
Service: powerapps
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

Oh no, Document details are actually as follows. Should I edit original answer or will I cause too much trouble?

ID: c22a3770-5c77-49ff-3384-f2b16d9edd11
Version Independent ID: 2230e0c7-faff-bdc3-73ac-1b6a27d741fe
Content: setCurrentView (Client API reference) in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/grids/viewselector/setCurrentView.md
Service: powerapps
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk


Installation of an Operator fails when BundleDeployment with the same bundle already exists

Steps to reproduce:

Create a catalog on cluster: kubectl apply -f config/samples/catalogd_operatorcatalog.yaml
Create a BundleDeployment:

apiVersion: core.rukpak.io/v1alpha1
kind: BundleDeployment
metadata:
  name: operator-sample-manual
spec:
  provisionerClassName: core-rukpak-io-plain
  template:
    metadata: {}
    spec:
      provisionerClassName: core-rukpak-io-registry
      source:
        image:
          ref: quay.io/operatorhubio/argocd-operator@sha256:1a9b3c8072f2d7f4d6528fa32905634d97b7b4c239ef9887e3fb821ff033fef6
        type: image

Create an Operator which resolves to the same bundle:

apiVersion: operators.operatorframework.io/v1alpha1
kind: Operator
metadata:
  labels:
    app.kubernetes.io/name: operator
    app.kubernetes.io/instance: operator-sample
    app.kubernetes.io/part-of: operator-controller
    app.kubernetes.io/managed-by: kustomize
    app.kubernetes.io/created-by: operator-controller
  name: operator-sample
spec:
  packageName: argocd-operator
  version: 0.6.0

Actual result:
apiVersion: operators.operatorframework.io/v1alpha1
kind: Operator
metadata:
  # ...
  name: operator-sample
spec:
# ...
status:
  conditions:
  - lastTransitionTime: "2023-10-06T12:26:51Z"
    message: resolved to "quay.io/operatorhubio/argocd-operator@sha256:1a9b3c8072f2d7f4d6528fa32905634d97b7b4c239ef9887e3fb821ff033fef6"
    observedGeneration: 1
    reason: Success
    status: "True"
    type: Resolved
  - lastTransitionTime: "2023-10-06T12:26:57Z"
    message: 'bundledeployment not ready: rendered manifests contain a resource that
      already exists. Unable to continue with install: Namespace "argocd-operator-system"
      in namespace "" exists and cannot be imported into the current release: invalid
      ownership metadata; annotation validation error: key "meta.helm.sh/release-name"
      must equal "operator-sample": current value is "operator-sample-manual"'
    observedGeneration: 1
    reason: InstallationFailed
    status: "False"
    type: Installed
  resolvedBundleResource: quay.io/operatorhubio/argocd-operator@sha256:1a9b3c8072f2d7f4d6528fa32905634d97b7b4c239ef9887e3fb821ff033fef6

Expected result:
Not sure what would be the expected behaviour in here. Perhaps a better error message indicating conflict. Or, if we can successfully resolve, updating ownerReferences on already existing BundleDeployment instead of creating a new one?

This probably requires more thought, but my instinct is that we should not automatically adopt existing BundleDeployments.

How is it expected to work with dependency resolution? Would you check for on-cluster availability as part of the resolution and not add the resolved bundle if it has already been installed?
What are the conflicts when multiple operators share the same dependency?

RBAC
upgrades: channel, constraint policy
anything else?

Cross posting @joelanford's comment from this thread. I believe it is related to this ticket.

I don't think we need to worry too much about this [conflicts between unmanaged content and managed via Operator]. An unmanaged BundleDeployment isn't all that different than an unmanaged CRD or unmanaged cluster role or whatever else that ultimately conflicts with something that comes from an Operator.
What I would expect to happen is:

Resolution is not affected by the conflicting BundleDeployment
If the conflicting BundleDeployment has the same name as the one that we want to create/manage, there should be an error along the lines of "couldn't create BD foo, already exists" or "couldn't update BD foo, not owned by Operator"
The the conflict BundleDeployment does not have the same name, operator-controller should create/update it, and then rukpak should report the conflict along the lines of "could not create CustomResourceDefinition foos.acme.org, already exists or not owned by BD", which would then bubble up on the Installed condition back to the Operator.

No BundleDeployment - no problem. Closing this.


Bootstrap accessibility component plugin

Consider adding this?
https://github.com/paypal/bootstrap-accessibility-plugin

Updated via Pull request #269 that integrates new bootstrap version and latest a11y plugin

This was merged some time ago.


fix: Add validation for same secret to vendir

What this PR does / why we need it:
When an AppCR is provided as part of an kapp deploy execution, and the AppCR have in the fetch section multimples entries of git type and those entries are against a private repository and uses the same secretRef, kapp controller is generating the Directory with multiple times the secret causing the error described in the issue, for example
spec:
  serviceAccountName: default
  fetch:
  - git:
     # creates a config map
      url: git@github.com:odinnordico/kapp-controller-private-test.git
      ref: origin/main
      secretRef:
        name: ssh-key-secret
      subPath: cm
  - git:
      # creates a secret
      url: git@github.com:odinnordico/kapp-controller-private-test.git
      ref: origin/main
      secretRef:
        name: ssh-key-secret
      subPath: secret

The ssh-key-secret is twice and the error  Expected to find one secret 'ssh-key-secret', but found multiple is shown.
This PR filters the resourcesYaml to be used in the vendir command by only adding one time each secret.
This can work assuming that the AppCR only supports secrets in the same namespaces as it is, then only one secret with a given name can exist within.
Which issue(s) this PR fixes:

Fixes #1002
Does this PR introduce a user-facing change?

User is now able to provide the same secret on multimple fetch entries under the `secretRef.name` of each

Additional Notes for your reviewer:
Review Checklist:

[x] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[x] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

Done as part of https://github.com/carvel-dev/vendir/pull/239


[Types] useMutation returning union undefined with new types?

Describe the bug
Mutate now potentially returns an undefined type, which is different from the return type before I believe?  Maybe that's the expected behavior but by looking at the docs you generally aren't checking that something is returned.  Shouldn't it just throw an error rather than return undefined if something goes wrong?
So using the new types you now have to check that the data exists before using it.
Before
const [mutate] = useMutation(mutateFunction);

try {
  const data = await mutate();
  // do something
} catch (err) {
// error handling
}

After:
const [mutate] = useMutation(mutateFunction);

try {
  const data = await mutate();
  if (data) {
    // do something
  }
} catch (err) {
// error handling
}

There might be a good reason for this but it does deviate from the docs (https://react-query.tanstack.com/docs/guides/mutations#basic-mutations) and previous types.

Is this still a valid problem now #767 is merged? 🤔

Is this still a valid problem now #767 is merged? 🤔

I just installed it today so I think so?  WIth:
npm i react-query@tsnext

My package.json:
"react-query": "^2.5.7-tsnext.1",

Hi @Buuntu, thanks for giving feedback on the new types! The type has indeed changed because the mutation returns undefined when an error occurs and the throwOnError option was not set to true. The documentation does seem to imply that the mutate function will throw by default because of the try catch in the example. This could either be a bug in the code or in the docs. What should be the default behaviour here @tannerlinsley ?

Mutate does not throw by default unless throwOnError is true. The docs should be updated to reflect this.

I suppose it's not doable to set a conditional type depending on what the config is.

Is this still a valid problem now #767 is merged?

Yes it is, I get the error on 2.5.11 but not on 2.5.7-tsnext.1

Looks like with the docs and the new 2.5.7-tsnext.1 types, we'll be in good shape.


How to disable update message on start of "ionic serve" command.

I have lastest version of ionic. After the new version of Ionic is comming out I got message that ask me to update or not my project to new version. I need to disable this option because I use creating of apps on ionic framework via commands that starts from cron. For this I use PHP server. The problem that I don't know then it message coming out and program is will be stucked because it want for answer always.
How to disable this message?

I believe the following will disable them for your user: ionic --no-interactive

Thanks, I will try!

ionic --no-interactive serve indeed prevents the update requests from appearing!


checkSSLCert: 2.33.0 -> 2.34.0

Description of changes
https://github.com/matteocorti/check_ssl_cert/releases/tag/v2.34.0

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

checkSSLCert


Fix back button.

This should fix https://github.com/colszowka/simplecov/issues/263
It's unclear to me on where and how to add tests for this. Any guidance is appreciated.

What are the chances we both decided to fix this today? Guess I should have checked that bug once I got back to my computer this evening..
https://github.com/colszowka/simplecov-html/pull/36

@justinsteele Sorry about that. :(

Closed by https://github.com/colszowka/simplecov-html/pull/36

@whatasunnyday Thanks for your contribution. I will add both you and @justinsteele to the changelog for the 0.9.2 release of SimpleCov itself.


Updated EDI Model POCO to better match message format and added addit…

Updated EDI Model POCO with more fields to match message. Added additional tests for changes to the POCO.

Thanks @cwhellams .
Now maybe I need to think of a better way to organize test classes.


add source license full text into source license

What is the purpose of the pull request
add source license full text into source license
Brief change log
add source license full text into source license

Codecov Report

Merging #2840 into dev-1.3.0 will increase coverage by 0.28%.
The diff coverage is 36.84%.

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


Builder: Add Button/Icon to Remove all Items

Heya,
i would like to have a button in the builder to remove all (not fixed) assigned items by the builder for each unit.
That way its easier/faster to campare the units.
Atm i do this with JS in the console:
$(".delete").trigger("click");

@reloxx13 PR incoming

Thank you very much :D


Make it clear to the reader that it is Azure PowerShell

Update document to make it clear to the reader that this Quickstart requires Azure PowerShell instead of the regular PowerShell.

@AzureMentor Thanks for your contribution.
@kevinvngo Could  you review the proposed changes? Thanks.

Bug opened for PRMerger issues: https://ceapex.visualstudio.com/SRE/_workitems/edit/112917

@ktoliver Please review this PR and merge.  Thanks!

@ktoliver Please review this PR and merge. Thanks!

hi @AzureMentor - the article author @kevinvngo needs to approve / sign off on the PR changes before we can merge. thanks.

@barbkess, can you review the proposed changes in this PR? Thanks.

Hi @julieMSFT - Could you review the proposed changes or suggest a suitable reviewer? Thanks!

@kevinvngo Hi Kevin, in the change suggested correct? Are we only talking about Azure Power Shell?

#sign-off

@julieMSFT: I'm sorry - only the author of this article, @kevinvngo, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


No network error should be improved

When trying to install an extension without an internet connection I get the following error:

getaddrinfo ENOTFOUND marketplace.visualstudio.com marketplace.visualstudio.com:443

Something like this would be better:

Cannot install extension without internet connection

Closing as a detailed error message is useful for us to know what's happening. Without a logger this can't be solved.


Getting EMFILE errors on macOS with v3.x.x

Describe the bug
When watching recursively a folder containing a lot of folders, we'll reach the system's limits and start getting EMFILE errors on some paths.
This seems to be happening only with a large number of folders, not files.
Also, I don't see the issue with Chokidar v2.1.8.
I've tried increasing the ulimit value of my computer from 256 to 10480 without improvements.
FYI:
$ ulimit -a
core file size          (blocks, -c) 0
data seg size           (kbytes, -d) unlimited
file size               (blocks, -f) unlimited
max locked memory       (kbytes, -l) unlimited
max memory size         (kbytes, -m) unlimited
open files                      (-n) 10480
pipe size            (512 bytes, -p) 1
stack size              (kbytes, -s) 8192
cpu time               (seconds, -t) unlimited
max user processes              (-u) 1418
virtual memory          (kbytes, -v) unlimited

Versions (please complete the following information):

Chokidar version 3.1.0
Node version 8.12.0
OS version: macOS 10.14.6

To Reproduce
I've found that in my case, the smallest number of directories required to see the errors pop out is 4096.
$ mkdir dirs
$ for i in `seq 1 4096`; do mkdir "dirs/dir-$i"; done

const chokidar = require('chokidar')

const watcher = chokidar.watch('./dirs', {
  cwd: '.',
  followSymlinks: false,
  alwaysStat: true,
  usePolling: false,
  useFsEvents: true,
  atomic: true,
  awaitWriteFinish: {
    pollInterval: 200,
    stabilityThreshold: 1000
  },
  interval: 1000,
  binaryInterval: 2000
})

watcher
  .on('all', console.log)
  .on('error', console.error)

Expected behavior
We should see every event and path printed out without any errors.
Additional context
We're using Chokidar as watcher in https://github.com/cozy-labs/cozy-desktop for macOS. We don't have any way to control the ulimit value on our users' computers and don't want to add arbitrary folders limits.

Node 8.12 is not supported by fsevents. Only 8.16 and higher.

I tried to reproduce on Node 12.4.0 and I couldn't see the error. Thank you for the pointer and sorry for the noise.
I don't see any mention of this limitation in chokidar's repo nor in fsevents' repo. Did I miss some wiki page?

Added it to changelog in README, and added it to fsevents - should be easier.


Make Gradle metadata format 1.0

Context
This commit bumps the Gradle metadata format to 1.0.
The specification file of the format has been changed
so that the name of the file indicates which version
it covers.
Starting from this commit, it is illegal to change
the metadata file format without backwards compatibility
in mind.

Also we should add a cross version test to verify that something published by the current Gradle version can be consumed by older versions of Gradle that support the 1.0 format, and vice versa.

Cross version test added for Java libraries. Maybe native folks would like to do something similar, but I don't have enough knowledge to do it. In any case this is going to be more valuable with 5.3.1+ (because of the test matrix).


Manual ticking.

It’s not currently possible because simulation.stop sets the iteration to Infinity.

We probably need simulation.pause and simulation.resume.


dockerTools: Add fakechroot to fakeRootCommands

Motivation for this change
Make installation scripts work unmodified in fakeRootCommands, by making programs behave like the image root is /.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg test docker-tools

cc @vroad as contributor of the fakeroot change.

@roberth It does look like a minimal change to me, but unfortunately I do not know much about chroot or fakeroot, so I think someone else (maybe @vroad) would be a better reviewer.

@roberth @utdemir Unfortunately I don't know much about fakeroot or fakechroot either, I tried fakeroot for the first time in my previous PR just because it was exactly what I have been looking for solving my issue (file owner issue).

Fair enough. I've disabled it by default, so the change is entirely opt-in. I'll merge it so we can gain some experience over time.


Add 'Fit Selected'

GET /v1/apply/fit/{networkId} could use a parameter to allow fitting to selection.

Commands offers this in the meantime: /v1/commands/view/view_fit_selected