ModuleNotFoundError: No module named 'gevent.__hub_local'

Hi! when building a distributable binary, and try to execute it  the following error appear:
Traceback (most recent call last):
  File "hello.py", line 1, in <module>
  File "/home/pc-ia/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module
    exec(bytecode, module.__dict__)
  File "eel/__init__.py", line 2, in <module>
  File "/home/pc-ia/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module
    exec(bytecode, module.__dict__)
  File "gevent/__init__.py", line 87, in <module>
  File "/home/pc-ia/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module
    exec(bytecode, module.__dict__)
  File "gevent/_hub_local.py", line 101, in <module>
  File "gevent/_util.py", line 105, in import_c_accel
  File "importlib/__init__.py", line 126, in import_module
ModuleNotFoundError: No module named 'gevent.__hub_local'

Why did you close the issue? I'm having the same problem and don't know why. Did you figure this out?

@sedir  I do not know why it appeared ... but I fixed it by installing these packages with this version:
gevent==1.2.2
Eel==0.9.7
gevent-websocket==0.10.1
greenlet==0.4.13
PyInstaller==3.3.1

@Marcelpv96 thanks, I'll add these as requirements

@Marcelpv96

@sedir I do not know why it appeared ... but I fixed it by installing these packages with this version:
gevent==1.2.2
Eel==0.9.7
gevent-websocket==0.10.1
greenlet==0.4.13
PyInstaller==3.3.1

Great , it saved my day thanks
I would like to know how this works

linux have similar fails, without way to solve it
os kubuntu 18.04
PyInstaller: 3.3.1
Python: 3.6.7
when i run pyinstaller with last bottle/bottle-websocket/gevent etc versions i have this fail of executable
Traceback (most recent call last):   File "d.py", line 3, in <module>   File "/home/user/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module     exec(bytecode, module.__dict__)   File "eel/__init__.py", line 2, in <module>   File "/home/user/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module     exec(bytecode, module.__dict__)   File "gevent/__init__.py", line 87, in <module>   File "/home/user/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module     exec(bytecode, module.__dict__)   File "gevent/_hub_local.py", line 101, in <module>   File "gevent/_util.py", line 105, in import_c_accel   File "importlib/__init__.py", line 126, in import_module ModuleNotFoundError: No module named 'gevent.__hub_local' [20371] Failed to execute script d
When i rollback to from fresh libs to gevent 1.2.2 (pip3 uninstall gevent -> pip3 install gevent=1.2.2) and rerun same i have this fail with executable
Traceback (most recent call last):   File "d.py", line 3, in <module>   File "/home/user/.local/lib/python3.6/site-packages/PyInstaller/loader/pyimod03_importers.py", line 631, in exec_module     exec(bytecode, module.__dict__)   File "eel/__init__.py", line 8, in <module>   File "bottle.py", line 1797, in load_module ModuleNotFoundError: No module named 'bottle_websocket' [20916] Failed to execute script d

@sedir I do not know why it appeared ... but I fixed it by installing these packages with this version:
gevent==1.2.2
Eel==0.9.7
gevent-websocket==0.10.1
greenlet==0.4.13
PyInstaller==3.3.1

This also worked to me, but to clarify further for anyone else with this problem, it seems like the gevent version is the most important, and having a later version can cause problems. I was able to build but not run an EXE created with gevent==1.3.7 and after downgrading to 1.2.2 (pip install gevent==1.2.2) I was able to successfully build an EXE that ran under Windows.

... issue looks closed and not solved

@3dformortals Exactly what command are you running?

@3dformortals Exactly what command are you running?
0-i am on linux
1- i have worked done (from source) basic eel app helloworld level it just call python3 print() operator from javascript code
https://github.com/3dformortals/data/tree/master/python/DASprogress/eel
2- from d.py level i execute terminal command pyinstaller d.py --onefile , and all done without errors
3- click on programm d without exe, because linux, show me nothing
4- from d.py level terminal execution of ./d show me errors above

If you use python3 -m eel instead of calling pyinstaller directly it will include things like bottle automatically

If you use python3 -m eel instead of calling pyinstaller directly it will include things like bottle automatically. This is mentioned in the readme but not pointed out especially, quite a lot of people miss it

how should looks final command for my d.py file? Something like
python3 -m eel blackmagic d.py --onefile? I need executable output.

You need to either use a callback method, or await keyword to get return value from Python. There are examples in the README

You need to either use a callback method, or await keyword to get return value from Python. There are examples in the README

yep, thanx, you're right.
var x = await pythonsidefunction()() or
let x = await pythonsidefunction()() (let in case when i need error raising vs undefined in var case) work as expected.
Is it possible that in the future pyinstaller d.py --onefile will work same as python3 -m eel d.py web --onefile, or i miss something etc?

The reason for using the -m eel syntax is that it automatically includes some necessary modules that pyinstaller leaves out.
If you look at the code you can see the actual pyinstaller command that it calls


File link is lost when importing .bib item with citation keys

Overall, I want to preserve both the citation keys and the file links (to my large Dropbox folder) when importing .bib to Zotero. I am migrating to Zotero.
The .bib item has both the citation key field (already used in my Latex files) and the file field (linked to a PDF file on disk) populated already. Now, when I enable better bibtex when importing, the file is not linked as an attachment. When I disable the extension, the file will show up in Zotero as an attachment.
Report ID: D1197042708
Full Bib(La)TeX item you are trying to import:
It looks like this and I am importing with the file selected as "Link to files in original location":
@article{my_previously_assigned_citation_key,
abstract = {abstract_here},
author = {author_here},
file = {:C$\backslash$:/Users/user_name/Dropbox/Literature/previously_stored_file_name.pdf:pdf},
title = {{title_here}},
}
Expected behavior:
Citation key assigned and the file linked to the PDF location also shows up as a link to the attachment.
Actual behavior:
Citation key assigned; no file attachment or link visible.
When the better-bibtex extension is disabled when importing, the PDF file shows up, but then the citation key is lost.

I can't read reports like D1197042708, those only go to the Zotero developers. But if I'd have to guess, this is Mendeley-generated bibtex; the Mendeley devs apparently couldn't be bothered to read the bibtex manual. I'll spare you the explanation, but you can either remove file from verbatimFields (but do change that back when done) or rename file to files in the bib file before import.

@retorquere You are absolutely right! The bib file was generated by Mendeley. Once I renamed file to files in the bib file, everything went through just fine! Thanks a lot!

Just as an aside, in case you wanted to know why this happened -- if not, feel entirely free to skip the rest, most people don't need to know this about bibtex. But the Mendeley devs absolutely should have. Not the first grave error I've found in Mendeley-generated bibtex, and likely not the last.
BibTeX has, roughly speaking, two "modes" in which the input is parsed; verbatim mode, and "normal". In verbatim mode, LaTeX commands are not treated as instructions but as bare text -- in things like URLs (or, in this case, file paths), you don't really need anything but bare text, so disabling command handling means you don't have to escape backslashes and underscores and so on.
And here's the thing -- file fields are verbatim according to the bibtex/biblatex manual, so
file = {:C$\backslash$:/Users/user_name/Dropbox/Literature/previously_stored_file_name.pdf:pdf},

means literally a file under the non-existent C$\backslash$: folder (although I wonder why Mendeley would commit the further screw-up of having C$\backslash$: instead of C:$\backslash$). What it should have been is just
file = {:C:\Users\user_name\Dropbox\Literature\previously_stored_file_name.pdf:pdf},

or
file = {:C:/Users/user_name/Dropbox/Literature/previously_stored_file_name.pdf:pdf},

if you would have changed that by hand it also would have worked.
Changing the hidden setting makes BBT not consider file fields as verbatim, so the Mendeley-bibtex would become
file = C\:/Users/user_name/Dropbox/Literature/previously_stored_file_name.pdf

after BBT has parsed it, and that I can import; alternately, what you did, is that BBT supports the unsupported files field, with the semantics "treat this as if you saw file =, but don't parse in verbatim mode.

Thanks a lot for the explanation! I would never have thought the issue is with the gravely-erroneously placed \backslash in the verbatim file field. I hope this will be useful for others migrating from Mendeley.


Feature ColorPicker should allow the hide some parts

Describe the changes you have made to improve this project
Added more properties to ColorPicker

AreRgbChannelsVisible
AreHsvChannelsVisible
IsAlphaChannelVisible
IsColorNameVisible
IsEyeDropperVisible

Additionally the ColorHelper is not static anymore. That way the user may roll his own ColorHelper and override ColorFromString and GetColorName.
Additional content
@seba30 please give me a 👍 or 👎 if this PR suits your needs
@punker76  I'll also update the docs if this PR is merged.
Closed Issues
Closes #4020
Closes #4023

@timunie your changes have been merged, thanks for your contribution 👍


** WIP ** Work in Progress Branch for RHEL6 RPMs

This pull request is for reference to developers.  It is not intended to be merged.
This pull request collects commits from various developers, so that each individual developer doesn't have to collect the various commits.  New development can branch off of this branch, and it will be periodically merged in, so that other developers can benefit, even though the code is not, yet, ready for review and testing.

I don't think there's a way for an RPM to 'use the "names" for these users and groups'.  The files and directories in an RPM are stored as a GZip'ped CPIO archive.  CPIO doesn't store user and group names for files, only their IDs, as I think most archives do.

Work-in-progress no longer needed, since the branches that provide basic functionality have been merged in the upstream feature branch.


Edit phrase parts without refreshing / starting over

Needs UX design. Allow drag and drop, or only (x) to delete?

It gives me great pleasure to resolve this just over two years after opening it. :)
The app no longer supports composing phrases by selecting phrase part types a-la-carte, so editing phrases is no longer possible.


Implement WAYF tenant picker on sign up page

Implement WAYF tenant picker on sign up page

Assigning to @mrvisser for fix

Assigned to @nicolaasmatthijs for review

Depends on: https://github.com/oaeproject/Hilary/pull/1246

Finished review. Back to @mrvisser for follow-up

Followed up and rebased w/ email domain restriction.

The auto-suggest functionality has been updated now as well. This is ready for more review from @nicolaasmatthijs

Looking really good functionally. Finished another round of review. Back to @mrvisser for follow-up

Followed up, and refined the keyboard interactions a bit. Back to @nicolaasmatthijs

Finished final round of review. Back to @mrvisser for remaining comments

Wait a minute ... You followed up during review :o Merged :)


Hostname filtering

See: https://github.com/aptible/api.aptible.com/pull/572
See: https://github.com/aptible/sweetness/pull/947
cc @fancyremarker

TODO:

[ ] Serve a descriptive error page when using the wrong domain

Serve a descriptive error page when using the wrong domain

Should we dynamically write out a few error pages for common error cases (currently just bad upstream / unacceptable Host header), using the existing error page as a template, but with placeholder text for the error title and message?

I think so. Did you have other errors in mind, or just those two?

Those are the only two.

This was pulled in as part of #96.


[BUG]: CSI-PowerStore Node Prefix is ignored

Bug Description
During the installation of the CSI PowerStore with the CSM Operator, I set the variable:
common:
  envs:
    - name: X_CSI_POWERSTORE_NODE_NAME_PREFIX
      value: ocp-dk01

If existing node had a different prefix, the new node in Kubernetes will be incorrectly updated as below. Also, the new prefix is ignored.
spec:
  drivers:
  - name: csi-powerstore.dellemc.com
    nodeID: csi-node-5c3d7fd0eb004604a6036df6eb4853ed-w.x.y.z-w.x.y.z

This prevents to mount volumes with the error :

AttachVolume.Attach failed for volume "ocp-3f9e6a6c00" : rpc error: code = NotFound desc = host with k8s node ID 'csi-node-9a7687da85fb4186b48230878bbfc939-w.x.y.z-w.x.y.z' not found

Logs
AttachVolume.Attach failed for volume "ocp-3f9e6a6c00" : rpc error: code = NotFound desc = host with k8s node ID 'csi-node-9a7687da85fb4186b48230878bbfc939-w.x.y.z-w.x.y.z' not found
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
Install the driver with default prefix.
Update the driver with a different prefix.
Re-install the driver with the updated prefix.
Expected Behavior
The host entry should be modified with the correct name having the updated prefix. Also, the node id in kubernetes cluster should match the host name.
CSM Driver(s)
CSI PowerStore
Installation Type
CSM Operator
Container Storage Modules Enabled
No response
Container Orchestrator
OCP
Operating System
CoreOS

link: 27810


Reset image cache

Good Morning
I am trying to post multiple QR codes as single embedded messages within discords "image cached" period. meaning even though the QR code has changed, discord still grabs the previous QR code that was generated. Anyone have an idea how I can reset this before it sends via MessageManager?

Could you post an example of the input data for you case?

What I do is ping my devices then if its online manipulate the data to create a QR code and add it to the embedded message.
Even though I am deleting the data out of the local directory and creating a new qrcode.png file, discord continues to grab the cached original image, as seen below.

I did some research and found that the only way around would be to make the qrcode generate with a unique name every time by adding a timestamp or even a random number to it. Such as qrcode1.png , qrcode2.png
This works but brings up the issue to storage. Images are small but ideally I wouldn't like to keep an eye out on storage usage if you know what I mean.
I will close this ticket as technically its fixed but if anyone has anything to add please do.

you could make a flow that deletes files older than x minutes/hour or period of time.


Removed duplicated 'the'. Fixed jq link.

Can one of the admins verify this patch?

[test]

merged: e3800d33179dc05b1d32b130ffc06df80e8b3731


Added saves_student_data field for external activities.

@scytacki I've added a migration for adding a saves_student_data column to the external_activities table, and added the field to the external activity authoring form.
Is that all I need to do before I start working on the related notification for teachers when they assign an activity?

This should also change code that sends the description of the material down to the browser, so client side code can render the UI for it.


Data visualizations

Description
@daveragabriel mentioned that it would be nice to have some data visualizations as well. I think we can do this either (a) after all the requirements are finished, or (b) some time before then. @rohanherur If you get part way through the requirements and feel like you want to work on this, I'd say go ahead.
@daveragabriel If you have anything specific, feel free to mention it here. Otherwise, we can probably come up with some stuff that you'd like.
Stuff to visualize

Counts of roles: bar chart
Counts of keywords: bar chart

@DaveraGabriel might be able to help w/ the visualization

@DaveraGabriel Are these done or could still use some tweaks?


How to automatically detect Models using Sequelize

Hey! I'm using swagger-autogen for months and it's been running flawlessly.
I am in the process of refactoring some routes' documentation and I noticed that I can use the $ref attribute under definitions to reuse the same schema across files. However, I'm using Sequelize as an ORM and I cannot see how its models can be detected automatically using swagger-autogen.
How can I achieve that?

One way to generate the Models objects automatically is to use a script.
I found out that sequelize-swagger does that and I just had to download and insert it into my project (because it's not available on npm). With the use of describeModel, it was possible to generate the definitions of each model from the database connection file and export it:
// database/index.js
const connection = new Sequelize(dbConfig);

const modelsObject = Object.fromEntries(
  Object.entries(connection.models).map(([key, value]) => [
    key,
    describeModel(value),
  ])
);

module.exports = {
  connection,
  modelsObject,
};

Then I used it in the doc configuration:
// swagger.js
const { modelsObject } = require('../database/index');

const doc = {
   //...
  definitions: modelsObject,
};

Hi @juanbomfim22. Thank you for this information.


Refactor ActivityIndicators #no-public-changes

Summary of Pull Request

Condensed ActivityIndicatorTriangleSkewSpin and ActivityIndicatorSquareSpin
into a new class called ActivityIndicatorSkewSpin

I removed ActivityIndicatorBallZigZagDeflect

Reasoning

ActivityIndicatorTriangleSkewSpin and ActivityIndicatorSquareSpin

These two classes shared the same animation and only differed in their shape.
I provided an initializer that takes the shape as an argument.

ActivityIndicatorBallZigZag and ActivityIndicatorBallZigZagDeflect

These two classes were also nearly identical.
The only difference is ActivityIndicatorBallZigZagDeflect auto-reverses
I provided an initializer that takes a flag deflects as an argument
Extra
I haven't added a CHANGELOG entry yet. This is just record keeping.

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          2 Warnings

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

:warning:
Consider adding / updating the demo ap.

  Generated by :no_entry_sign: Danger

@SD10 Can you just update the CHANGELOG? Otherwise, i think that ready to go 👍


New API 'UpdateRecommendations' to support non-chronological order of data

Implementation of new API - "updateRecommendations" to support non-chronological order of data sent by ROS and provide valid recommendations
Ensure to update the design doc with the new API changes

implemented and ready to close.


Possibly migrate Related Resources from RTD TOC page to DocFX/OPS

At the bottom of our current RTD TOC page docs.asp.net we have a Related Resources section with links to the following:

.NET Core
Entity Framework Core
WebHooks

I don't think we need to migrate .Net Core or EF Core. I plan on adding  WebHooks to our hub page. See #2055

@danroth27  please review and close


FLINK-36332 Add option to select the Fabric8 httpclient implemention

What is the purpose of the change
Enable the building of the operator using alternative Fabric8 http clients.
Brief change log
Enable the building of the operator using alternative Fabric8 http clients.
Verifying this change
This change is already covered by existing tests, such as (please describe tests).
The build should still work even when run with -Dfabric8.httpclinent.impl=vertx or jdk or jetty for that matter 😁
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes.
The public API, i.e., is any changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Do you have any recommendation how we should test this and make sure it works? Maybe we can add a case to the e2e-s somehow?

Eventually we should move to Vertx as default, which is the preferred one in fabric8.

Do you have any recommendation how we should test this and make sure it works? Maybe we can add a case to the e2e-s somehow?

Hmm thats a good question. It looks fairly easy to add it to maven build matrix (though at what cost?) and thus test on every build. That is however of debatable value as there is no change to the projects code and thus all it would really be doing is testing a third party dependency and is that very interesting at unit/integration test level?
A more interesting angle as suggested is to include it in the e2e suite. Which again has a massive implication for the spread of tests but is probably more interesting as it validates it works against various different minikube setups.
I've added both options in different commits so we can easily drop them if they aren't considered helpful.

I think the e2e workflow is now correctly setup (and has passed on my fork

I should note I'm not convinced the workflow is doing what its expected to do I've documented my findings as FLINK-36392 which I'm happy to work on next.
I've also noticed a bunch of warnings about deprecated node version from the actions so have opened FLINK-36393 which I'm currently testing a PR for on my fork.

I think the e2e workflow is now correctly setup (and has passed on my fork

Nope its not. The includes with the http-client seem to be overriding tests and not adding an extra run as intended...
I'm tempted to suggest splitting out test_application_operations.sh as a separate "smoke test" workflow that could run with each http-client with a fixed combination of the other matrix args.
My other thought is to have a workflow per Java version as that controls the supported flink versions as well and possibly the test suite as well as some of the tests have version exclusions.
A workflow per Java version feels easier for people to reason about too, but I don't have a lot of project context or history to work from so could be missing something...

@SamBarker I think it makes sense to split the test workflows. It has grown too much and it's getting a bit annoying as you can see :)
Please feel free to propose any reasonable split, I think your understanding is probably as good as anyone's at this point.

Please feel free to propose any reasonable split, I think your understanding is probably as good as anyone's at this point.

In the scope of this PR i've just pulled out a smoke test job which validates the HTTP clients can do the basics. I don't see that as being a great split for future testing and or other combinations however I'd like to look at a wider restructure in a separate PR.
I'd also like to call out https://github.com/apache/flink-kubernetes-operator/pull/881/commits/bb6c5772f3d0c8e4a279a0ee6cf1d04872937303 my primary motivation for adding it was to have a clearer picture of what was actually being built (in my fork) but I think its reasonable to merge. As I don't see any point in running the full job matrix if the basics aren't working (and thus saving actions minutes/ other resources).  The same argument could be extended to making the smoke test dependent on the test_ci job as well, but I haven't done that here as it might slow down feedback from the edge to edge tests for some un-related or flanky unit test failures.


Remove version: RoyQu.RedPanda-C++ version 3.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


[FEA] HostColumnarToGPU with ArrowColumnVector copy add support for Decimal or nested types

Is your feature request related to a problem? Please describe.
for https://github.com/NVIDIA/spark-rapids/issues/1072  we put in changes to optimize the copy of Arrow data read by HostColumarToGpu for primitive types and strings. We did not add support for Decimal or nested types.
Decimal actually isn't supported in the cudf from_arrow type code, so we would need that first. The other nested types for struct and list are supported so we just need the jni side.

looks like https://github.com/rapidsai/cudf/issues/7116 is filed to add decimal/fixed point to cudf for from_arrow and to_arrow


[5.x] Confusing Question while import process

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem Description
Very confusing Question while import process

Enter the relative path to your uncompiled main file

What? I have started the import like this:

my@ubuntu:~/workspace/my-app$ electron-forge import .

Relative path to main file? What is that? It is on the same level like package.js
Proposed Solution
Chage Question text, so it is more clear what you want
Alternatives Considered

Additional Information

We're no longer fixing any non-critical bug in Electron Forge v5. Please use version 6 instead.

Thank you. Just one question. https://www.npmjs.com/package/electron-forge shows Version 5.2.4. How to get Version 6?
Regards

See https://www.electronforge.io/


Scenario Runner : While using WaypointFollower vehicle is comming back to the original location

Hi,
I am trying to create a new scenario, kindly refer the attached file :
lane_cut_in.txt
config_file :
<?xml version="1.0"?>
<scenarios>
    <scenario name="LaneCutIn_1" type="LaneCutIn" town="Town04">
        <ego_vehicle x="248.547" y="-392.23" z="1.2" yaw="180" model="vehicle.nissan.patrol" />
    </scenario>
</scenarios>

Code Snippet :
#!/usr/bin/env python

"""
Lane Cut In:

Another vehicle overtakes the ego vehicle and cuts-in in front of it.

"""

import random

import py_trees

import carla

from srunner.scenariomanager.atomic_scenario_behavior import *
from srunner.scenariomanager.custom_atomic_scenario_behavior import AtTriggerDistanceToVehicle, LaneChange
from srunner.scenariomanager.atomic_scenario_criteria import *
from srunner.scenariomanager.timer import TimeOut
from srunner.scenarios.basic_scenario import *
from srunner.tools.scenario_helper import *

from agents.navigation.local_planner import RoadOption

LANE_CUT_IN = [
    "LaneCutIn"
]

class LaneCutIn(BasicScenario):

    """
    This class holds everything required for a "Lane Cut In" scenario
    """
    category = "LaneCutIn"

    timeout = 1000            # Timeout of scenario in seconds

    def __init__(self, world, ego_vehicle, config, randomize=False, debug_mode=False, criteria_enable=True,
                 timeout=1000):
        """
        Setup all relevant parameters and create scenario

        If randomize is True, the scenario parameters are randomized
        """

        self._map = CarlaDataProvider.get_map()
        self._ev_tv1_distance = 1000
        self._first_vehicle_location = 25
        self._ego_vehicle_speed = 15
        self._first_vehicle_speed = 25
        self._reference_waypoint = self._map.get_waypoint(config.trigger_point.location)
        self._other_actor_max_brake = 1.0
        self._other_actor_stop_in_front_intersection = 20
        self._other_actor_transform = None
        # Timeout of scenario in seconds
        self.timeout = timeout

        super(LaneCutIn, self).__init__("LaneCutIn",
                                                   ego_vehicle,
                                                   config,
                                                   world,
                                                   debug_mode,
                                                   criteria_enable=criteria_enable)

        if randomize:
            self._ego_other_distance_start = random.randint(4, 8)

            # Example code how to randomize start location
            # distance = random.randint(20, 80)
            # new_location, _ = get_location_in_distance(self.ego_vehicle, distance)
            # waypoint = CarlaDataProvider.get_map().get_waypoint(new_location)
            # waypoint.transform.location.z += 39
            # self.other_actors[0].set_transform(waypoint.transform)

    def _initialize_actors(self, config):
        """
        Custom initialization
        """

        _ego_vehicle_current_waypoint = self._map.get_waypoint(self._reference_waypoint.transform.location)

        # calculate EV's new waypoints.
        #_ego_vehicle_new_waypoint, _ego_vehicle_dist = get_waypoint_in_distance(_ego_vehicle_current_waypoint, self._ev_tv1_distance)

        # calculate TV1's new waypoints
        _first_vehicle_waypoint = _ego_vehicle_current_waypoint.get_left_lane()

        # set new position to EV
        #self.ego_vehicle.set_transform(_ego_vehicle_new_waypoint.transform)

        first_vehicle_transform = carla.Transform(_first_vehicle_waypoint.transform.location, _first_vehicle_waypoint.transform.rotation)

        self._other_actor_transform = first_vehicle_transform

        first_vehicle = CarlaActorPool.request_new_actor('vehicle.nissan.patrol', first_vehicle_transform)

        self.other_actors.append(first_vehicle)

    def _calculate_waypoint(self, source, destination):
        """
        Return list of way points between source and destination.
		Start point and end point of lane change are considered.
		Slope of the line forming from those points is considered.
		waypoints are calculated on that line in this function using slope formula(slope = (Y2 - Y1)/ (X2 - X1))
        """
        _slope = (float(destination.transform.location.y) - float(source.transform.location.y)) / (float(destination.transform.location.x) - float(source.transform.location.x))
        _route = list()
        _resolution = (float(destination.transform.location.y) - float(source.transform.location.y))  / 50
        y = float(source.transform.location.y)
        _copyt_waypoint_cls = destination
        try:
            for _cnt in range(50):
                y = float(y + _resolution)
                x =  ((float(y) - float(source.transform.location.y)) / _slope ) + float(source.transform.location.x)
                _copyt_waypoint_cls.transform.location.x = x
                _copyt_waypoint_cls.transform.location.y = y
                _copyt_waypoint_cls.transform.location.z = 0
                _copyt_waypoint_cls.transform.rotation.yaw = 0
                _copyt_waypoint_cls.transform.rotation.pitch = 0
                _copyt_waypoint_cls.transform.rotation.roll = 0
                _route.append((_copyt_waypoint_cls, RoadOption.VOID))
        except Exception as e:
            print("Error while _calculate_waypoint() : ",e)

        return _route

    def _get_destination_waypoint(self, _vehicle_current_waypoint, lane_side=None, distance=40):
        """
        """
        _vehicle_new_waypoint, _vehicle_dist = get_waypoint_in_distance(_vehicle_current_waypoint, distance)

        if lane_side == 'right':
            _destiantion_waypoint = _vehicle_new_waypoint.get_right_lane()
        elif lane_side == 'left':
            _destiantion_waypoint = _vehicle_new_waypoint.get_left_lane()
        else:
            print("----------------- ERROR while generating waypoint for destination vehicle")

        return _destiantion_waypoint

    def _create_behavior(self):
        """
		The ego vehicle will work to restore a safe following distance. A combination of speed reduction and anticipating future motion of the cut-in vehicle
        should lead to the safe following distance being restored within 5 seconds of the initial maneuver of the cut-in vehicle.
        """
        print("other actor previous :", self.other_actors[0].get_location())
        # STAGE 1
        stage_1 = py_trees.composites.Parallel("stage_1", policy=py_trees.common.ParallelPolicy.SUCCESS_ON_ONE)
        stage_1.add_child(WaypointFollower(self.ego_vehicle, self._ego_vehicle_speed))
        stage_1.add_child(DriveDistance(self.ego_vehicle, 15))

        # STAGE 2
        stage_2 = py_trees.composites.Parallel("stage_2", policy=py_trees.common.ParallelPolicy.SUCCESS_ON_ONE)
        stage_2.add_child(WaypointFollower(self.ego_vehicle, self._ego_vehicle_speed))
        stage_2.add_child(WaypointFollower(self.other_actors[0], self._first_vehicle_speed))
        # TODO: Replace DriveDistance with AtTriggerDistance.
        #stage_2.add_child(AtTriggerDistanceToVehicle(self.other_actors[0], 5, self.ego_vehicle, 5, 20))
        stage_2.add_child(DriveDistance(self.other_actors[0], 100))

        # get vehicle current waypoints.
        _vehicle_current_waypoint = CarlaDataProvider.get_map().get_waypoint(self.other_actors[0].get_location())

        # get destination waypoints.
        _destiantion_waypoint = self._get_destination_waypoint(_vehicle_current_waypoint, lane_side='left')

        # calculate waypoint from source to destination
        _route = self._calculate_waypoint(_vehicle_current_waypoint, _destiantion_waypoint)

        # STAGE 3
        stage_3 = py_trees.composites.Parallel("stage_3", policy=py_trees.common.ParallelPolicy.SUCCESS_ON_ONE)
        stage_3.add_child(WaypointFollower(self.other_actors[0], self._first_vehicle_speed, plan=_route))
        stage_3.add_child(WaypointFollower(self.ego_vehicle, self._ego_vehicle_speed))
        stage_3.add_child(DriveDistance(self.ego_vehicle, 1000))

        # Build behavior tree
        sequence = py_trees.composites.Sequence("Sequence Behavior")
        sequence.add_child(stage_1)
        sequence.add_child(stage_2)
        sequence.add_child(stage_3)

        return sequence

    def _create_test_criteria(self):
        """
        A list of all test criteria will be created that is later used
        in parallel behavior tree.
        """
        criteria = []

        collision_criterion = CollisionTest(self.ego_vehicle)

        criteria.append(collision_criterion)

        return criteria

    def __del__(self):
        """
        Remove all actors upon deletion
        """
        self.remove_all_actors()

stage_1:
Ego vehicle is driving in it's speed using WaypointFollower till 15 m (DriveDistance as end condition)
stage_2:

Traffic vehicle and ego vehicle both are driving in their respective speed using WaypointFollower
end condition = Traffic vehicle drives 150m

stage_3:

Lane change of Traffic Vehicle :
i. Ego vehicle will drive in it's speed using Waypoint Follower
ii. Traffic vehicle will follow the generated route (route is generated using : _calculate_waypoint()) using WaypointFollower
iii. end condition = Ego Vehicle drives 1000m

I have added this stages in a sequence tree and tried to run it.
My Observations :

Till stage_2 it works well.
When stage_3 start running, ego vehicle takes U-Turn and goes back to it's previous location and then again take U-Turn and then drive straight
We tried to give the generated route to traffic vehicle and ego vehicle in the stage_1, then it is working.

Kindly help me to know why this is happening.

Is this related to #302?

@fabianoboril, yes, I will close it. Thanks for you help


add test for #48

Thanks! Looks like Travis CI is noting an unused variable and some stylistic issues.

@graingert do you still plan to work on this?

@graingert do you still plan to work on this?

@fsouza I'm happy for someone else to take this over!


LDAP authentication fails but can ldap user search

Laravel Version: 5.4.36
Adldap2-Laravel Version: 3.0
PHP Version: 5.6.21

Description:
Hi,
Hope anybody can help me on this since I'm stuck at if for days.
I already set up adldap on my laravel project by following on this link: https://github.com/Adldap2/Adldap2-Laravel but I keep on encountering an error that my credentials did not match on my record.   Just to test my connection I tried searching a user using their username and it returned the user so it means my connection settings is correct.
dd(Adldap::search()->where('samaccountname', '=', request('username'))->get());
Steps To Reproduce:

Hi @kevin042, I need more details to help troubleshoot this.
It sounds like a configuration issue, can you post your configuration with sensitive details omitted?

Hello Sir Steve,
Good day,
Thank you for the quick response, please see below my configurations.
File: adldap.php
 [
        'default' => [
            'auto_connect' => true,
            'connection' => Adldap\Connections\Ldap::class,
            'schema' => Adldap\Schemas\ActiveDirectory::class,
            'connection_settings' => [
                'account_prefix' => env('ADLDAP_ACCOUNT_PREFIX', ''),
                'account_suffix' => env('ADLDAP_ACCOUNT_SUFFIX', ''),
                'domain_controllers' => explode(' ', env('ADLDAP_CONTROLLERS', 'XXX.XXX.XX.XX')), //IP Address
                'port' => env('ADLDAP_PORT', 389),
                'timeout' => env('ADLDAP_TIMEOUT', 5),
                'base_dn' => env('ADLDAP_BASEDN', 'OU=XXXXX,DC=XXXXX,DC=XXXXX'),
                'admin_account_suffix' => env('ADLDAP_ADMIN_ACCOUNT_SUFFIX', ''),
                'admin_username' => env('ADLDAP_ADMIN_USERNAME', 'CN=XXXXXXXXX.,OU=XXXXX,OU=XXXXX,OU=XXXXXXX,DC=XXXXXXX,DC=XXXXX'),
                'admin_password' => env('ADLDAP_ADMIN_PASSWORD', 'XXXXXXXX'),
                'follow_referrals' => false,
                'use_ssl' => false,
                'use_tls' => false,
            ],
        ],
    ],
];
[adldap.txt](https://github.com/Adldap2/Adldap2-Laravel/files/1389505/adldap.txt)

File: adldap_auth.php
[adldap_auth.txt](https://github.com/Adldap2/Adldap2-Laravel/files/1389517/adldap_auth.txt)

 env('ADLDAP_CONNECTION', 'default'),
    'provider' => Adldap\Laravel\Auth\DatabaseUserProvider::class,
    'resolver' => Adldap\Laravel\Auth\Resolver::class,
    'importer' => Adldap\Laravel\Auth\Importer::class,
    'rules' => [
        Adldap\Laravel\Validation\Rules\DenyTrashed::class,
    ],
    'scopes' => [
        Adldap\Laravel\Scopes\UpnScope::class,
    ],
    'usernames' => [
        'ldap' => env('ADLDAP_USER_ATTRIBUTE', 'samaccountname'), // Was userprincipalname
        'eloquent' => 'username',
    ],
    'login_fallback' => env('ADLDAP_LOGIN_FALLBACK', false),
    'password_sync' => env('ADLDAP_PASSWORD_SYNC', true),
    'windows_auth_attribute' => ['samaccountname' => 'AUTH_USER'],
    'sync_attributes' => [
        'username' => 'samaccountname',
        'name' => 'cn',
    ],

];

File: app.php (Added to file)
Adldap\Laravel\AdldapServiceProvider::class,
Adldap\Laravel\AdldapAuthServiceProvider::class,

'Adldap' => Adldap\Laravel\Facades\Adldap::class,

File: auth.php (Updated to file)
'providers' => [
        'users' => [
            'driver' => 'adldap', //Was eloquent
            'model' => App\User::class,
        ],
    ],

File: LoginController (in auth directory)

 public function username()
    {
        return config('adldap_auth.usernames.eloquent');
    }

    protected function validateLogin(Request $request)
    {
        $this->validate($request, [
            $this->username() => 'required|string',
            'password' => 'required|string',
        ]);
    }

    protected function attemptLogin(Request $request)
    {

        $credentials = $request->only($this->username(), 'password');
        $username = $credentials[$this->username()];
        $password = $credentials['password'];

        if (Adldap::search()->where('samaccountname', '=', $username)->get()) {
            // the user exists in the LDAP server, with the provided password
            $user = \App\User::where($this->username(), $username)->first();
            if ($user) {
                $this->guard()->login($user, true);

                return redirect('/');
            }else {
                return back()->withErrors([
                    'message' => 'Username and password did not match.'
                ]);
            }
        }
    }

File: .env
ADLDAP_CONNECTION=default
ADLDAP_CONTROLLERS=XXX.XXX.XX.XX // IP Address of AD
ADLDAP_BASEDN=OU=XXXXX,DC=XXXXXX,DC=XXX
ADLDAP_USER_ATTRIBUTE=samaccountname
ADLDAP_USER_FORMAT='CN=XXXXX.,OU=XXXX,OU=XXXX,OU=XXXXXXX,DC=XXXXXX,DC=XXXX'

Those are all the configurations I made on the system.

Hello Sir @stevebauman ,
Good day,
Do you have an update on where I possibly gone wrong on the configuration?

Found the error on the configuration.

Hi @kevin042, sorry for the long response. Thanks for troubleshooting your issue on your own and resolving it :)

Hi @jLopez23, can you post your code here, I'll take a look at it.

@kevin042, my problem I post it in the following link: https://github.com/Adldap2/Adldap2-Laravel/issues/580

@jLopez23 ok will take a look at it.

@kevin042 thanks, I appreciate your help too much.

Hi @jLopez23 are you using Windows AD? if yes try to empty your ADLDAP_ACCOUNT_PREFIX and for the  ADLDAP_ACCOUNT_SUFFIX try using something like<EMAIL_ADDRESS>

@kevin042, I already tried but it does not work, it's just working for me by concatenating the ADLDAP_ADMIN_USERNAME with ADLDAP_ACCOUNT_SUFFIX in my .env file in the following way: ADLDAP_ADMIN_USERNAME =<EMAIL_ADDRESS>but as I mentioned before I do not know if this is a good solution

@jLopez23 if it works on that method then it's fine.  Just don't use the administrator account and make sure to secure your .env file

@kevin042 thanks, some advice on how to protect my .env file


Wtfos live audio

Hi :)
Hoping someone can help.
I have install the live audio onto my goggles and the original air unit but I'm not getting any audio back through the headphones.
I have done a playback from the sd card that was in the air unit and its recorded the sound but I can barely hear it?
Thanks

Hey,
After I installed the"djifpv_live_audio_V2" by Funneld I had to turn the volume up. Also make sure to use headphones or earbuds with TRRS connector (like with mic).
His page explains;
Audio volume can be set:
1)by playing back a video that doesn't contain any audio and set the volume from there.
2)with CLI command "setprop persist.dji.volume.headphone 6" with persistent values from 1-6.
3)with CLI command "tinymix 2 48 48" Values up to 63.
Hope this helps

Hey, After I installed the"djifpv_live_audio_V2" by Funneld I had to turn the volume up. Also make sure to use headphones or earbuds with TRRS connector (like with mic). His page explains; Audio volume can be set: 1)by playing back a video that doesn't contain any audio and set the volume from there. 2)with CLI command "setprop persist.dji.volume.headphone 6" with persistent values from 1-6. 3)with CLI command "tinymix 2 48 48" Values up to 63.
Hope this helps

Thank you!
I have been using a headset with microphone tried 2x sets and neither worked.
I'll give playing a video back with no audio to adjust volume and try that.
Thank you.

Hey, After I installed the"djifpv_live_audio_V2" by Funneld I had to turn the volume up. Also make sure to use headphones or earbuds with TRRS connector (like with mic). His page explains; Audio volume can be set: 1)by playing back a video that doesn't contain any audio and set the volume from there. 2)with CLI command "setprop persist.dji.volume.headphone 6" with persistent values from 1-6. 3)with CLI command "tinymix 2 48 48" Values up to 63.
Hope this helps

Thank you! I have been using a headset with microphone tried 2x sets and neither worked. I'll give playing a video back with no audio to adjust volume and try that. Thank you.

did you get it to work ??


Remove duplicated 'target_length' attribute

This PR removes the duplicated attribute 'target_length', which existed to form and force diagrams and needed to be copied all the time from one to the other.
Now this attribute is applied only to the form diagram. The value of this attribute is then easily retrieved with the helper dual_edge_targetlength().
As a future update this attribute can be renamed to target_force if the reviewer prefers.
Note, this PR depends on the approved but not yet merged PR #127

This takes the target force using the self.dual.edge_attribute, but currently does not work with the proxy. See issue #131


[Bug Report]: Azure Virtual Network Manager [Min] test failing on Management Group ID token

Describe the bug
Our min test is failing on AVNM when it is testing the management group scope deployment
module testDeployment '../../deploy.bicep' = {
  scope: resourceGroup
  name: '${uniqueString(deployment().name)}-test-${serviceShort}'
  params: {
    enableDefaultTelemetry: enableDefaultTelemetry
    name: '<<namePrefix>>${serviceShort}001'
    networkManagerScopeAccesses: [
      'Connectivity'
    ]
    networkManagerScopes: {
      managementGroups: [
        '/providers/Microsoft.Management/managementGroups/<<managementGroupId>>'
      ]
    }
  }
}

However the resolved token in the template deployment is correct to the ID stored in GitHub

To reproduce
Run the AVNM module pipeline
Code snippet
VERBOSE:
  VERBOSE: 20:19:11 - Template is valid.
  VERBOSE: 20:19:15 - Create template deployment 'n-nm-min-t1-20230110T2001518007Z'
  VERBOSE: 20:19:15 - Checking deployment status in 5 seconds
  VERBOSE: 20:19:22 - Resource Microsoft.Resources/deployments 'hegpjweihceac-test-nnmmin' provisioning status is running
  VERBOSE: 20:19:22 - Resource Microsoft.Resources/deployments 'pid-47ed15a6-730a-4827-bcb4-0fd963ffbd82-4qlgz35f3lzk2' provisioning status is running
  VERBOSE: 20:19:22 - Resource Microsoft.Resources/resourceGroups 'ms.network.networkmanagers-nnmmin-rg' provisioning status is succeeded
  VERBOSE: 20:19:22 - Checking deployment status in 12 seconds
  VERBOSE: 20:19:37 - Checking deployment status in 14 seconds
  VERBOSE: Resource deployment Failed.. (1/3) Retrying in 5 Seconds..

  VERBOSE: 20:19:52 - The deployment 'n-nm-min-t1-20230110T2001518007Z' failed with error(s). Showing 2 out of 2 error(s).
  Status Message: Resouce [/providers/Microsoft.Management/managementGroups/***] is not within tenant [***] (Code:BadRequest)

  Status Message: At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details. (Code: DeploymentFailed)
   - {
    "error": {
      "message": "Resouce [/providers/Microsoft.Management/managementGroups/***] is not within tenant [***]",
      "code": "BadRequest"
    }
  } (Code:BadRequest)

  CorrelationId: 9a8ff8c9-3d2f-4865-82fb-16f029ce4896

Relevant log output
No response

cc @AlexanderSehr

Potential bug in AVNM. As it appears that if the Management Group ID is a GUID, AVNM is unable to interpret it and throws an error regarding the management group not being in the same tenent


Fix installation problems for openapi_go_client and other links

These links are currently installed into $(GOPATH):
ln -s $(GOPATH)/bin/openapi_go_generator $(GOPATH)/bin/openapi_go_client
But GOPATH can be a ":"-separated list of paths, and that breaks this process.
Instead modify the installation to link in the same directory that contains openapi_go_generator, which can be obtained as follows:
GENERATOR = `which openapi_go_generator`
GOBIN = `basepath $GENERATOR`
ln -s $(GOBIN)/openapi_go_generator $(GOBIN)/openapi_go_client

The "which" command works by iterating through $PATH, not $GOPATH...I'm not sure of the context for this problem, but maybe you need your own command to use $GOPATH?  And on my linux host, I see no "basepath" command -- I think you want "dirname" instead...so in this vein:
GENERATOR=PATH=$GOPATH:$PATH which openapi_go_generator
GOBIN=dirname $GENERATOR
ln -s $GOBIN/openapi_go_generator $GOBIN/openapi_go_client
or...
(cd $GOBIN && ln -s open_go_generator openapi_go_client)
You don't really need the full path in the symlink...it will be interpreted relative to the containing directory.

openapi_go_generator is no longer included with this project


Produce slash instead of space as the first array's element in configurable router links

Will be delivered in one PR together with GH-2293


AIXpb: version02: cleanup stage 02

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Change Summary

main.yml: rewrite paging space task so no reboot is needed, localize variables; only one bootjdk role called
roles/bootjdk/tasks/main.yml: rewrite task to use include_tasks: and move/rewrite tasks into includeable tasks; localize variables
roles/aixfs/tasks/main.yml: remove /usr from the list and include a chfs.yml task for growth as needed by bootjdk

Generally good, although I'd prefer if we left the list of versions of the JDK to install in the top level of the playbook to avoid diverging from how the other ones do it, to ease maintenance.

My idea - moving forward - made applicable for all playbooks - is to have the jdk versions to be installed assigned to a variable, and processed as a list. I'll even take that task upon myself for both all platforms. Will just need some assistance in learning how to test - adequately - on non-AIX platforms :)


Fix lime.ndll build on MinGW

Fixes some issues to allow compiling lime.ndll with MinGW.
I'm not too sure what the best way to deal with the System.cpp situation would be.

I separated out some of the commits into #1661, because with those commits it's already possible to cross compile projects with mingw with a precompiled lime.ndll.

I'm interested in testing this build! Could you offer any links to the new minGW version of lime.ndll? If not, I can look into building from source, but it might be a while.

I'm interested in testing this build! Could you offer any links to the new precompiled minGW version of lime.ndll?

Hi sorry, I haven't uploaded the precompiled lime.ndll anywhere. Building from source isn't too complicated as long as you have mingw set up, when I have a look at this again I can try to provide some proper instructions for how to do this. I should have provided instructions originally in the PR, sorry. Once I have a look again I could probably upload a copy of the ndll here as well.
Just a heads up, I managed to get things compiling but the build wasn't really working. When I tried to run a project with the mingw ndll there was no window showing up unfortunately. This will require more investigating to figure out what exactly is wrong. In the meantime I have just been building my actual project code with mingw (with #1661 and hxcpp patches https://github.com/HaxeFoundation/hxcpp/pull/1044 https://github.com/HaxeFoundation/hxcpp/pull/1045 https://github.com/HaxeFoundation/hxcpp/pull/1046) but using the regular Windows lime.ndlls instead of compiling my own mingw copy.

I'm interested in testing this build! Could you offer any links to the new precompiled minGW version of lime.ndll?

Hi sorry, I haven't uploaded the precompiled lime.ndll anywhere. Building from source isn't too complicated as long as you have mingw set up, when I have a look at this again I can try to provide some proper instructions for how to do this. I should have provided instructions originally in the PR, sorry. Once I have a look again I could probably upload a copy of the ndll here as well.
Just a heads up, I managed to get things compiling but the build wasn't really working. When I tried to run a project with the mingw ndll there was no window showing up unfortunately. This will require more investigating to figure out what exactly is wrong. In the meantime I have just been building my actual project code with mingw (with #1661 and hxcpp patches HaxeFoundation/hxcpp#1044 HaxeFoundation/hxcpp#1045 HaxeFoundation/hxcpp#1046) but using the regular Windows lime.ndlls instead of compiling my own mingw copy.

I'll give that a try! Thanks again for all of the work you've put into this build.

When trying compiling my app, a separate library reports this error:
Error: /Users/E/Documents/Haxe/hxdiscord_rpc/git/project/discord-rpc/src/connection_win.cpp:8:10: fatal error: 'windows.h' file not found
Do you know what I may have to do in order to get the windows.h header file to be included?

I'm not too sure what the best way to deal with the System.cpp situation would be.

I figured out that this issue can be worked around by avoiding _bstr_t(char*) calls and replacing them all with  _bstr_t(wchar*) calls, which seems more consistent anyway. I've tested it with the native windows ndll and that still works too.
With current hxcpp, when cross compiling from linux it is necessary to rebuild with lime rebuild windows -Dmingw to make sure the correct flags are used for the build, see: https://github.com/HaxeFoundation/hxcpp/pull/1168.
This compiles properly now, and also runs successfully from my testing!


Empty output.png cannot be parsed

I created empty grid:
>Pixie.exe generate -w 10 -h 20 -n Hex
I used this config file:
{
  "BitsPerPixel": 2,
  "SymbolWidth": 12,
  "SymbolHeight": 18,
  "DelimeterWidth": 1,
  "DelimeterHeight": 1,
  "DelimeterColor": "#FF0000",
  "BackgroundColor": "#808080",
  "ColorMapping": {
    "#FFFFFF": 0,
    "#000000": 1,
    "#C8C8C8": 2,
    "#808080": 3
  }
}

However, when I try to parse it, I get this error:
Critical error!
Problem detected while processing pixel at 17,6
Can't find corresponding bits to pixel color #FF0000. Check your config

same as #11 insufficient help text
image with row  and column numbers shoul be parsed with-hoption


remove warning about patch not found for external-const-factory

To repro:
./tools/build.py -m release patched_dart2js_sdk

Will produce these warnings:
warning: patch not found for bool.fromEnvironment: external const factory bool.fromEnvironment(String name, {bool defaultValue : false});
warning: patch not found for int.fromEnvironment: external const factory int.fromEnvironment(String name, {int defaultValue});
warning: patch not found for String.fromEnvironment: external const factory String.fromEnvironment(String name, {String defaultValue});

We shouldn't complain about these though.
In the past we had a body for these methods, but the FE would complain const factory constructors were not allowed to have a body either. We removed the bodies in dd70d8b655.

Turns out this was an issue in patched_sdk, fix is out for review.

fixed in https://github.com/dart-lang/sdk/commit/62d1d28428ccdcdf76cd65d347f7f8d3489a92f8


Socket connection for autodriving

In the VisionExport.cs, there is socket connection to some server here and apparently being used to instruct this mod here
but it is nowhere mentioned in the documentation, neither I found source code for this.
Do you plan to opensource it?
Did anyone manage to get this working with their own implementation?

we ended up using autostarting and automatic reloading instead of the socket connection.

Oh, I see. Well, I created socket client with webserver so it can be started by clicking buttons. Hope someone will find it helpful.
https://github.com/racinmat/GTAVisionExport-Server


subsys/net/ip/net_mgmt: Use mutex for net_mgmt_lock

Conceptually the net_mgmt_lock should be a mutex instead of a semaphore. It is easier to identify the owner of a mutex and debug when deadlock happens, so convert it.

There should be no subsys:  prefix in the commit title, looks good otherwise.

Thanks for the quick review, fixed.

Thanks for the quick review, fixed.

You need to fix the individual commit and then force push, editing the PR title does nothing to the commit.

My point was that the commit title should not contain subsys keyword at all (which is banned by the compliance, not sure why did it pass in current form), i. e. just use net :ip: net_mgmt:  or even simply net: mgmt:  for the prefix.

My point was that the commit title should not contain subsys keyword at all (which is banned by the compliance, not sure why did it pass in current form), i. e. just use net :ip: net_mgmt:  or even simply net: mgmt:  for the prefix.

fixed

Thanks for the quick review, fixed.

You need to fix the individual commit and then force push, editing the PR title does nothing to the commit.

Yup, sure, I edited both.


Request: Ability to distribute a JSON file along with an MSU pack to shuffle alts

Use case: I'm developing an MSU pack that has tons of great tracks that unfortunately won't fit into a 61-track pack. But what would be nice is if we can create a bunch of alt tracks that MSU Shuffler could swap in on a per-run basis, but only for tracks within one pack, and us pack makers just simply include a json file for the shuffler to utilize more tracks for a pack than previously before.
This request also calls for shuffling beyond the scope of simply looping and non-looping tracks, but the dungeon tracks amongst the dungeon tracks, boss amongst boss, etc.
It would also be nice if there could be a way to keep certain dungeon tracks tied to their respective boss tracks as well, or at least just have the option to define tracks that must remain together.

@miketrethewey I thought you'd find this interesting as well

@miketrethewey I thought you'd find this interesting as well

miketrethewey/ALttPMSUShuffler:expand/resources/snes/zelda3/manifests/packs/codemann8.json


I am start translating to - installation.md

Estimate finish time is 3 days.

https://laravelmyanmar.com/docs/5.7/installation


community: langkit dependency

Issue: the langkit package is not presented in the pyproject.toml but it is a requirement for the WhyLabsCallbackHandler
Change: added langkit

Hey team - going to revert this because it doesn't seem necessary for testing. We should only be adding optional + extended_testing dependencies for deps that have extended tests.
otherwise it just increases probability of dependency conflicts in the community lockfile.


RyuJIT is the x86 JIT in .NET Core 2.0 – Replacing JIT32

RyuJIT is the x86 JIT in .NET Core 2.0 – Replacing JIT32
Summary
The RyuJIT JIT compiler will be used for all platform combinations for .NET Core 2.0 and beyond. JIT32 will be removed from the product. For .NET Core 1.x, RyuJIT is used for x64 and JIT32 is used for x86 chip support.
We intend for RyuJIT to be a 100% functionally compatible replacement for JIT32. RyuJIT has gone through extensive stress testing. This brings the .NET Core 32-bit product behavior much closer to the 64-bit product behavior.
We will publish more information on this change later. For now, this is just the announcement.
Note: Windows is the only supported 32-bit platform for .NET Core 1.x and 2.x.
Details:

Make RyuJIT/x86 the default x86 JIT (dotnet/coreclr #8260)

Impact
Open Source - RyuJIT is open source. JIT32 is not open source.
Single JIT Codebase - There is now a single codebase for the CoreCLR JIT. This helps everyone who is working on platform expansion and who is investing in JIT improvements. An example is Samsung, who is investing in Linux x86 with RyuJIT x86.
Performance - RyuJIT generally takes longer to compile than JIT32, but its optimizer can generate faster code. The performance difference may not be significant, depending on the application. In initial testing with ASP.NET, we found that the difference didn't matter. Please help us make RyuJIT faster! We will continue investing in RyuJIT performance (which often helps all architectures). If you notice performance regressions, we’re very interested to hear your results. Please contact us.
Features! - RyuJIT has more features than JIT32. The feature gap with JIT32 will grow over time. The following features are now available to x86 workloads (due to this change):

SIMD
Fast Span<T> support
Inlining improvements
Devirtualization
Significantly better floating point performance
Better performance with value types (structs)

Possible compatibility concerns:

floating-point precision
undefined behaviors (programs that happened to work that had bugs in them) especially around interop, inlining differences (GetCallerAssembly intrinsic).

Please test! If an x86 program was working in 1.x, and failing in 2.x, please report it on GitHub or at<EMAIL_ADDRESS>Please see some of our past blog posts on RyuJIT to learn more about it.
Rationale
RyuJIT is the current and future platform of JIT compilation for .NET. We will use it for all platform combinations going forward. We're  more efficient as a developer community if we have a single codebase to learn, work on and improve.

Reopening according to process.


Update Readme.cd

Hey , actually i go through your sites ..I feel to put my experience on it , so when i read the above stuffs ..i feel to edit some lines in a better way. Hope you like it

Hello @ayushi-blip, Thanks for your contribution, but adding more links is more relevant than just editing those texts.
Please consider looking for good interesting links instead :)


Syntax highlighting causes panic

Attempting to apply highlighting to certain code causes a panic that prevents the site from building.
For example, writing
```js
function addTwo(number) {
return number * 2;
}
```
results in the following error:
$ gutenberg serve
Building site...
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Error(-114, target of repeat operator is invalid)', libcore/result.rs:945:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

(That wasn't the actual code I wanted to highlight; it's a simplified example that I found after playing around.  In that simple example, the return keyword seems to trigger the bug.  However, the full text I wanted to highlight (shown below) still exhibited the bug even after removing all uses of return.)
Using backticks without syntax highlighting didn't trigger the bug.
Here's the full text I was attempting to highlight:
```js
(window.NREUM||(NREUM={})).loader_config={
  xpid:"UQ8EUVRACQAFVVdbAQk="
};
window.NREUM||(NREUM={})
,__nr_require=function(t,n,e){
  function r(e){if(!n[e]){var o=n[e]={exports:
    {}};t[e][0].call(o.exports,function(n){
      var o=t[e][1][n];return r(o||n)},o,o.exports)
    }return n[e].exports}if("function"==typeof __nr_require)
    return __nr_require;for(var o=0;o<e.length;o++)r(e[o]);
    return r}
\```

Which version are you using? This kind of error means that syntect doesn't understand some part of the sublime syntax files, which happens sometimes when the syntaxes get updated and needs to be fixed upstream.

I'm using 0.3.4, which I cloned from GitHub and built from source earlier today, so it should be up-to-date.

You were absolutely correct; the issue was upstream and had already been resolved.  I fixed it by downloading the AUR package instead of building from source.  That's what I get for building from source when I apparently don't (yet!) understand the Rust package system well enough to upgrade all the dependencies.  Sorry for taking up your time with this.


Leading 0x in private key results in different addresses.

Issue Type

[X] Bug (https://github.com/ethjs/ethjs-sha3/blob/master/.github/CONTRIBUTING.md#bug-reports)

Description
The following program spits out two different addresses:
import { privateToAccount } from 'ethjs-account';
console.log(JSON.stringify(privateToAccount('ab18faade35ee0c65210782aff7ed995c4cbd351e204e9fe5e4a9caa5fa0dc2b').address));
console.log(JSON.stringify(privateToAccount('0xab18faade35ee0c65210782aff7ed995c4cbd351e204e9fe5e4a9caa5fa0dc2b').address));

Output:
"0xc2885646840673a2feD7080C29E36555A9F0ACD9"
"0xbC02A3FF56CD1D98197a742A626C4A339F002B8C"

Versions

Node/NPM: 8.2.1

Wow, I'll look into this right away. It should always strip hex prefixes.
Sent from my iPhone

On Aug 14, 2017, at 2:09 PM, Micah Zoltu<EMAIL_ADDRESS>wrote:
Issue Type
Bug (https://github.com/ethjs/ethjs-sha3/blob/master/.github/CONTRIBUTING.md#bug-reports)
Description
The following program spits out two different addresses:
import { privateToAccount } from 'ethjs-account';
console.log(JSON.stringify(privateToAccount('ab18faade35ee0c65210782aff7ed995c4cbd351e204e9fe5e4a9caa5fa0dc2b').address));
console.log(JSON.stringify(privateToAccount('0xab18faade35ee0c65210782aff7ed995c4cbd351e204e9fe5e4a9caa5fa0dc2b').address));
Output:
"0xc2885646840673a2feD7080C29E36555A9F0ACD9"
"0xbC02A3FF56CD1D98197a742A626C4A339F002B8C"
Versions
Node/NPM: 8.2.1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I glanced at the code and it definitely looks like it is intending to strip the 0x so the above two should yield the same result.  I was surprised by the behavior as well.   Thank you so much for finding this. Looks like I had input validation, but it wasn't functioning properly.
Fixed in commit:
https://github.com/ethjs/ethjs-account/commit/e64a68ae6782e4235a38a00d0842e8f2f08fb3f0
Please upgrade to version<EMAIL_ADDRESS>


XDG specification to scripts dir in Linux

Three years ago I asked if there was a way to change the scripts directory of Ghidra(#1347 ). Since then I've been messing with trying to make the scripts directory better conform with XDGBaseDirectorySpecification, so there it is.

Thank you, but we are not interested in a Linux-only change at this time.  There is some discussion in #908 regarding XDG.  It's on our radar.


Ensure Kubernetes installation docs are prominent in the reference

I think this is much better with the latest doc updates.


Add task_name to built-in variables

With this feature, users can use a task name in workflow like this:
+task1:
    td>: queries/${task_name.replace(/\+/g, '_')}.sql    # >>> "_myworkflow_task1.sql"
    insert_into: hourly_summary

@frsyuki Can you take look at this pull request when you get a chance?

Very nice :+1:
I wondered whether the task name should include the root task name, or not.
In +built_in_variables+get_variables+task_name, +built_in_variables is not precisely task name because it's workflow name. However, internally, workflow name is used as the "root" task name. So it's a sort of task name.
I honestly don't have strong opinion on it. How do you think?
Code itself looks very good.

Yeah. It's probably better to remove a workflow name from it since users don't know a workflow name is internally included in a task name. I'll modify the pull request later.

Separated the workflow name from a task name like this.
+parent:
  +task0:
      td>: queries/${workflow_name}/${task_name.join('__')}.sql     # >>> "queries/mydig/parent__task0.sql"
      insert_into: hourly_summary

@frsyuki Could you look over the updated one?

@komamitsu yes, LGTM 👍

Oh Mitsu, I recalled one problem. Task name delimiter is not always +. Dynamically generated tasks use ^ as a delimiter (e.g. ^sub, ^failure-alert). So, removing + actually loses some information...

How about just using the first idea? It seems most straightforward way.
Calling task_name.replace(/\W/g, '_')} or tsak_name.split(/\W/).join('__') still works.

Ah. Good catch!

How about just using the first idea? It seems most straightforward way.

👍


Add Step 1 of Installation

Added Installation Steps.md file with Step 1 of Installation

@yamihe19 Looks good, and I have made some minor edits on wordings in the latest commit (055589b). If you have nothing to add then please merge it into main. Thanks!


Mark FileManager.createFile as @discardableResult

On iOS / macOS this API comes from Objective-C which is implicitly
@discardableResult, this means it doesn't warn on those platforms, but
previously did on Linux

@swift-ci test

I'm really not into adding this piecemeal. It's fine, but we should do it to all the equivalent API, or none.


Domain User Info

Following APIs should return UserDetailWithDomainRole  instead:

Get domain user:
https://github.com/joint-online-judge/horse/blob/1ba920196656002bdd11317a6b8f86b42bc24879/joj/horse/apis/domains.py#L221
List domain user:
https://github.com/joint-online-judge/horse/blob/1ba920196656002bdd11317a6b8f86b42bc24879/joj/horse/apis/domains.py#L185

The permission for these is Permission.DomainGeneral.view, which is a little too wild. Does it mean everyone in the domain can see the real name of other users?


Create issues to get an initial POC out

It would be great to get an initial POC out as part of 0.3.
What issues need to be resolved in order to get to an initial POC?
Admittedly this doesn't leave much time since we are trying to cut 0.3 this week although we could always cherry pick it into minor releases.
Lets create specific issues for all the work that needs to be addressed.

/assign @wbuchwalter

@jlewi There is no blocking issue for the initial POC, it should be working already.
The only blocker I have currently is to migrate the repo itself, changing licence requires a bit of legal overhead on my side.
But if you are fine with working on wbuchwalter/fairing until this get resolved there is no real blocker.

Are you talking about a python library or a jupyter integration? This would be relevant in the case of the former: https://github.com/cwbeitel/tk/blob/master/docs/demo.ipynb


Add new Performance/UriDefaultParser cop

This cop is inspired by https://github.com/bbatsov/rubocop/pull/4694#discussion_r136707061.
Feature
This cop identifies places where URI::Parser.new can be replaced by URI::DEFAULT_PARSER.
% cat /tmp/test.rb
# frozen_string_literal: true

require 'uri'

URI::Parser.new.make_regexp

% bundle exec rubocop /tmp/test.rb
Inspecting 1 file
C

Offenses:

/tmp/test.rb:5:1: C: Use URI::DEFAULT_PARSER instead of URI::Parser.new.
URI::Parser.new.make_regexp
^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected

Target Problem
The following is a benchmark script.
require 'benchmark/ips'
require 'uri'

Benchmark.ips do |x|
  x.report('URI::Parser.new')     { URI::Parser.new }
  x.report('URI::DEFAULT_PARSER') { URI::DEFAULT_PARSER }
  x.compare!
end

It is the result of running the above benchmark script.
% ruby bench.rb
Warming up --------------------------------------
     URI::Parser.new    70.000  i/100ms
 URI::DEFAULT_PARSER   315.505k i/100ms
Calculating -------------------------------------
     URI::Parser.new    714.537  (± 4.2%) i/s -      3.570k in   5.005495s
 URI::DEFAULT_PARSER     10.950M (± 6.2%) i/s -     54.582M in   5.007247s

Comparison:
 URI::DEFAULT_PARSER: 10949722.8 i/s
     URI::Parser.new:      714.5 i/s - 15324.22x  slower

It is a constant as follows and it is fast because it does not instantiate every time.
https://github.com/ruby/ruby/blob/v2_4_1/lib/uri/common.rb#L22

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

👍

Thank you!


makeInstDirectory does not support vector masks or any other special fields

I have tried to make changes for https://github.com/sparcians/mavis/issues/15 Pls suggest changes

@kathlenemagnus, @bdutro am I missing something?

Ok I ll check this out and get back to you.
On Tue, 20 Aug 2024 at 7:17 PM, Knute @.***> wrote:

@kathlenemagnus https://github.com/kathlenemagnus, @bdutro
https://github.com/bdutro am I missing something?
—
Reply to this email directly, view it on GitHub
https://github.com/sparcians/mavis/pull/19#issuecomment-2298907855, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A5WMIMSVSXJPFX7WBUWEBS3ZSNCIVAVCNFSM6AAAAABMZNHXE6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJYHEYDOOBVGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

How about adding a new constructor type for the ExtractorDirectInfo class that takes in a map of special fields and their values? See ExtractorDirectInfo.h#L242C1-L276C7
Actually now that I look at this more closely, I do see a constructor that we could use:
ExtractorDirectInfo(const InstructionUniqueID uid, const RegListType &sources, const RegListType &dests,
                        const ValueListType& specials) :
        ExtractorDirectBase(uid), sources_(sources), dests_(dests), specials_(specials)
    {}

@Shubhf can you add a test for a vector add with a mask vm to see if this constructor will work?

yeah thats a good idea .. i ll try to implement it
On Wed, Aug 21, 2024 at 11:52 PM Kathlene Magnus @.***>
wrote:

How about adding a new constructor type for the ExtractorDirectInfo class
that takes in a map of special fields and their values? See
ExtractorDirectInfo.h#L242C1-L276C7
https://github.com/sparcians/mavis/blob/fe4a557091d5322ded4e090f10fca1e9e0b1704a/mavis/ExtractorDirectInfo.h#L242C1-L276C7
Actually now that I look at this more closely, I do see a constructor that
we could use:
ExtractorDirectInfo(const InstructionUniqueID uid, const RegListType &sources, const RegListType &dests,
const ValueListType& specials) :
ExtractorDirectBase(uid), sources_(sources), dests_(dests), specials_(specials)
{}
@Shubhf https://github.com/Shubhf can you add a test for a vector add
with a mask vm to see if this constructor will work?
—
Reply to this email directly, view it on GitHub
https://github.com/sparcians/mavis/pull/19#issuecomment-2302699125, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A5WMIMXF2BJDYCNTWVLE55TZSTLGDAVCNFSM6AAAAABMZNHXE6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBSGY4TSMJSGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@kathlenemagnus As you suggested to add a test vector with a mask "vm" to test to verify "ExtractorDirectInfo" , which will initialise "ExtractDirectInfo" with special fields including vm and validate that the mask is correctly set.
void test_vector_add_with_mask_vm() {     ExtractorDirectInfo ex_info(123, {1, 2}, {3}, {{mavis::ExtractorIF::SpecialField::VM, 0}});     auto specials = ex_info.getSpecialFields();     assert(specials.at(mavis::ExtractorIF::SpecialField::VM) == 0); }
I have thought about this function suggest necessary changes


Phase5: Update the AccuracyScorer.score signature

Update all the AccuracyScorer signature to take modelcontext and sourcescontext.
Addresses #732
Signed-off-by: Sudhanshu<EMAIL_ADDRESS>

Good stuff! Just wanted to make a note here that we need to remember to

[ ] Create a new AccuracyScorer based on the old NER accuracy() method

Correct me if I'm wrong because I didn't check around to see if you'd already done that, or if for some reason that doesn't need to be done. I just read the commits individually when reviewing this just now I didn't look at the codebase as a whole.

@pdxjohnny , yes for the ner model we need to do #866 , In the ner accuracy the spacy.scorer was used.

Very nice! I think we have some issues with the tests in ., it would be good to investigate those before doing the rebase, perhaps in an other PR.
$ python -m unittest discover -v tests

======================================================================
ERROR: test_docstring (test_docstrings.high_level_accuracy)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/dffml/dffml/tests/test_docstrings.py", line 209, in testcase
    run_doctest(obj, state)
  File "/home/runner/work/dffml/dffml/tests/test_docstrings.py", line 178, in run_doctest
    raise Exception(output.getvalue())
Exception: Trying:
    import asyncio
Expecting nothing
ok
Trying:
    from dffml import *
Expecting nothing
ok
Trying:
    model = SLRModel(
        features=Features(
            Feature("Years", int, 1),
        ),
        predict=Feature("Salary", int, 1),
        directory="tempdir",
    )
Expecting nothing
ok
Trying:
    async def main():
        print(
            "Accuracy:",
            await accuracy(
                model,
                {"Years": 4, "Salary": 50},
                {"Years": 5, "Salary": 60},
            ),
        )
Expecting nothing
ok
Trying:
    asyncio.run(main())
Expecting:
    Accuracy: 1.0
**********************************************************************
File "/home/runner/work/dffml/dffml/dffml/high_level.py", line 436, in accuracy
Failed example:
    asyncio.run(main())
Exception raised:
    Traceback (most recent call last):
      File "/usr/share/miniconda/lib/python3.8/doctest.py", line 1336, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest accuracy[4]>", line 1, in <module>
        asyncio.run(main())
      File "/usr/share/miniconda/lib/python3.8/asyncio/runners.py", line 43, in run
        return loop.run_until_complete(main)
      File "/usr/share/miniconda/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
        return future.result()
      File "<doctest accuracy[3]>", line 4, in main
        await accuracy(
      File "/home/runner/work/dffml/dffml/dffml/high_level.py", line 440, in accuracy
        async with sources as sources, model as model, accuracy_scorer as accuracy_scorer:
    AttributeError: __aexit__

======================================================================
ERROR: test_docstring (test_docstrings.noasync_accuracy)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/dffml/dffml/tests/test_docstrings.py", line 209, in testcase
    run_doctest(obj, state)
  File "/home/runner/work/dffml/dffml/tests/test_docstrings.py", line 178, in run_doctest
    raise Exception(output.getvalue())
Exception: Trying:
    from dffml import *
Expecting nothing
ok
Trying:
    from dffml.noasync import *
Expecting nothing
ok
Trying:
    model = SLRModel(
        features=Features(
            Feature("Years", int, 1),
        ),
        predict=Feature("Salary", int, 1),
        directory="tempdir",
    )
Expecting nothing
ok
Trying:
    def main():
        print(
            "Accuracy:",
            accuracy(
                model,
                {"Years": 4, "Salary": 50},
                {"Years": 5, "Salary": 60},
            ),
        )
Expecting nothing
ok
Trying:
    main()
Expecting:
    Accuracy: 1.0
**********************************************************************
File "/home/runner/work/dffml/dffml/dffml/noasync.py", line 79, in accuracy
Failed example:
    main()
Exception raised:
    Traceback (most recent call last):
      File "/usr/share/miniconda/lib/python3.8/doctest.py", line 1336, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest accuracy[4]>", line 1, in <module>
        main()
      File "<doctest accuracy[3]>", line 4, in main
        accuracy(
      File "/home/runner/work/dffml/dffml/dffml/noasync.py", line 23, in accuracy
        return asyncio.run(high_level.accuracy(*args, **kwargs))
      File "/usr/share/miniconda/lib/python3.8/asyncio/runners.py", line 43, in run
        return loop.run_until_complete(main)
      File "/usr/share/miniconda/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
        return future.result()
      File "/home/runner/work/dffml/dffml/dffml/high_level.py", line 440, in accuracy
        async with sources as sources, model as model, accuracy_scorer as accuracy_scorer:
    AttributeError: __aexit__

We are seeing an issue with transformers, not sure if it's a versioning issue or not, re-running the CI in master to double check.


Remove duplicate code to import MSS

Detailed Description

The code to import the MSS module is duplicated. You first try to import OS specific code, and if no OS was good, you still do from mss import mss.
But the current code is a duplicate from the mss.mss() factory, which already handle OS specific imports for you :)
The current code will simply double the checks and the result would be the same in any cases (when the OS is not handled).
The patch would be quite simple:
-			import platform
-			if platform.system() == 'Linux':
-				from mss.linux import MSS as mss
-			elif platform.system() == 'Windows':
-				from mss.windows import MSS as mss
-			elif platform.system() == 'Darwin':
-				from mss.darwin import MSS as mss
-			else:
-				from mss import mss
+			from mss import mss

Do you are interested in a PR?
BTW I am the MSS author, glad to see the module in use there :+1:
Context

Your Environment

VidGear version: 0.1.5
Branch: all
Python version: all
pip version: all
Operating System and version: all

Any Other Important Information

Hey @BoboTiG, Thanks for the great mss library and point this fault out. I thought it will be safe to implement my own wrapper around your system specific imports, while going through the mss docs. I should have checked the source-code too. Thanks again, I'll merge this update as soon as I can.

@BoboTiG Aren't these lines in mss docs redundant?

As mss automatically detects the right import according to your system OS?

You are right. I thought it was a good thing to let users choose a specific import. I will remove that part from the documentation ;)

Successfully resolved in https://github.com/abhiTronix/vidgear/commit/bad5d0c07d486b2a0a7b6801d21b9361b3189f63


Automate - Enhanced messaging for Cloud Orchestration provisioning.

Modified update_serviceprovision_status in
Cloud/Orchestration/Provisioning/StateMachines/Provision/ Methods class.
This generates an enhanced status message for the UI.
Previous Enhanced Messaging PR(s): 9333 12270

@tinaafitz
Please Review

@billfitzgerald0120 Looks good.
@mkanoor Please review.

@billfitzgerald0120
Can we add specs for these

@billfitzgerald0120 Some methods for orchestration provisioning explicitly set request.user_message. They can be removed because of this enhancement.

@tinaafitz
Please Review

@billfitzgerald0120 @tinaafitz Why can't we make update_serviceprovision_status itself a separate class and method under System/Provision/ or somewhere, so that all the three uses point to the same rel?

@bzwei @tinaafitz
I can't combine them now because they are different currently and I still need to add notifications to all three.

@mkanoor
I added spec tests for all three versions as requested.
Please Review

bump @mkanoor

See PR: https://github.com/ManageIQ/manageiq-content/pull/9


dfx start fails if dfx.json is malformed

When starting dfx with a bad dfx.json, the command will fail, although I don't understand why dfx is even reading the dfx.json:
$ cat dfx.json
{ "canisters": "yes" }
$ dfx start
Error: Failed to read config from current working directory.
Caused by: Failed to read config from current working directory.
  Failed to read config from directory /Users/nicolas/internet-identity/demos/using-dev-build.
    Failed to load config from /Users/nicolas/internet-identity/demos/using-dev-build/dfx.json.
      invalid type: string "yes", expected a map at line 1 column 20

I can definitely work around this, although I still find the behavior surprising. No error is thrown if there is no dfx.json.

dfx.json can enable replica features, such as canister_http or the BTC adapter. If the file is malformed, dfx start may run with the wrong configuration.

Does that mean that I should have exactly one running replica per dfx.json? how should I deal with multiple canister projects?

@ericswanson-dfinity is working on the system-wide dfx feature, which has progressed quite far already. Before this is ready, it's simply painful. You probably should have one main place where you start the replica from, and go from there. Some people had some success with symlinking different dfx.jsons together.


[ACM-13136] Updated IBIO component to GA status

Description
In MCE 2.7, IBIO will GA this release; therefore, we are updating the IBIO component to reflect the new status.
Related Issue
https://issues.redhat.com/browse/ACM-13136
Changes Made
Updated MCE to use IBIO GA version and remove preview version.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/test all

/test all

/test all

/lgtm


Don't wrap console methods for metro logging in Chrome debugger

Summary:
This diff fixes an issue reported in https://github.com/facebook/react-native/issues/26788 where logs in the Chrome console were showing a different location than previous versions.
In the change here, we stop wrapping the console functions to attach the metro websocket in any environment that isn't a native app environment. We do this by checking global.nativeLoggingHook which is bound only by native apps and not environments like the Chrome DevTools.
Differential Revision: D17951707

Closes https://github.com/facebook/react-native/issues/26788


Can not be visualized all accounts

I have three accounts in my Chrome Badget Wallet, but when I import it to the app only watch  the first account, the other are missing.

Thanks for flagging this issue.
This is a feature we are planning to add to the mobile app very soon.  Until then, only the first account is accessible in the mobile app.


fbdev: fixup BLANK macro usage on FreeBSD

The FreeBSD macro's name contains an underscore.
This fix was suggested [0] by @thelunatic.
[0] https://github.com/lvgl/lv_drivers/pull/142#issuecomment-896125290

Looks good. Thanks.

Thanks!


Credentials missing when using SnowflakeTargetConfigs in CLIProfile

Expectation / Proposal
I define four block entities (Snowflake credentials, Snowflake connector, Snowflake target config and dbt CLI profile) and link them together. In this case credentials can be changed via block configuration.
I expect that the CLI profile holds the credential/connection information as well but it didn't. This is maybe related to the TargetConfigs which can't be linked to another block entity.
Traceback / Example
from prefect_snowflake.credentials import SnowflakeCredentials
from prefect_snowflake.database import SnowflakeConnector

from prefect_dbt.cli.credentials import DbtCliProfile
from prefect_dbt.cli.configs import SnowflakeTargetConfigs

if __name__ == "__main__":
    credentials = SnowflakeCredentials(
        user="user",
        password="password",
        account="account.region.aws",
        role="role",
    )
    credentials.save("sf-cred", overwrite=True)

    connector = SnowflakeConnector(
        schema="public",
        database="database",
        warehouse="warehouse",
        credentials=SnowflakeCredentials.load("sf-cred"),
    )
    connector.save("sf-conn", overwrite=True)

    target_configs = SnowflakeTargetConfigs(
        connector=SnowflakeConnector.load("sf-conn"), schema="TEST"
    )
    target_configs.save("dbt-targetconf", overwrite=True)

    dbt_cli_profile = DbtCliProfile(
        name="jaffle_shop",
        target="dev",
        target_configs=SnowflakeTargetConfigs.load("dbt-targetconf"),
    )
    dbt_cli_profile.save("example", overwrite=True)

    dbt_cli_profile = DbtCliProfile.load("example").get_profile()

    print(dbt_cli_profile)

Output:
{'config': {}, 'jaffle_shop': {'target': 'dev', 'outputs': {'dev': {'type': 'snowflake', 'schema': 'TEST', 'threads': 4}}}}

[ ] I would like to help contribute a pull request to resolve this!

Thanks for the details! I believe this has been fixed in https://github.com/PrefectHQ/prefect-dbt/pull/115.

So we need to wait for a new release then.

Hi @ahuang11 ,
I've tested the behavior with the latest main branch and this doesn't work for me when we specifiy a schema in the SnowflakeConnector (which is required) and in the Snowflake Target Config (which is optional).
This ends up in a ValueError: The keyword, schema, has already been provided in TargetConfigs; remove duplicated keywords to continue error.
As a workaround we could only specify it in Snowflake Connector but if we use the same Connector for different dbt Configs (or may for other things) it would be a better solution to overwrite the schema from Snowflake Connector with the schema from the target config.

Thank you for reporting this. I'll try to get this fixed soon.


bug:

Neovim version (nvim -v)
v0.10.0
Operating system
MacOS
Terminal emulator / GUI
Wezterm
Describe the bug
Checkboxes renders are "eating" up the first two characters of text.

Expected behavior
Checkboxes should show the full text.
Healthcheck output (:checkhealth render-markdown)
==============================================================================
render-markdown: require("render-markdown.health").check()
markdown.nvim [neovim version] ~

OK Version >= 0.10

markdown.nvim [configuration] ~

OK valid

markdown.nvim [nvim-treesitter] ~

OK installed
OK markdown: parser installed
OK markdown_inline: parser installed
WARNING latex: parser not installed

ADVICE:

Disable LaTeX support to avoid this warning by setting { latex = { enabled = false } }

OK highlights enabled

markdown.nvim [executables] ~

WARNING latex2text: not installed

ADVICE:

Disable LaTeX support to avoid this warning by setting { latex = { enabled = false } }

Additional information
No response

Looks like this is an issue only in combination with obsidian.nvim. Will close.

I can confirm this is an issue happening when used with obsidian.nvim. Anyone know of a workaround?
Currently I use { ui = { enable = false } } on obsidian.nvim's options, which fixes this, but wikilinks are no longer displayed properly.


Globally install Python libraries in GitHub Actions

Apparently GitHub doesn't already have ~/.local/bin set up properly, so we need to do this manually ¯\_(ツ)_/¯
This warning is produced with the current master (as of #40335), and calling black silently fails.
WARNING: The scripts black and blackd are installed in '/home/runner/.local/bin' which is not on PATH.
Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.

Thanks!


CRDs: use apiextensions.k8s.io/v1

Signed-off-by: Zbynek Roubalik<EMAIL_ADDRESS>
Moving the KEDA CRDs to v1, that means supporting only Kubernetes >= 1.16
Fixes https://github.com/kedacore/keda/issues/701

PR for Documentation on the minimal k8s version:
https://github.com/kedacore/keda-docs/pull/272

Can we update the Helm chart as well?

@tomkerkhove yeah, I'll send a PR for charts once/if this is merged.


Save workspace by name

Please ignore that first test entry (was testing if my fresh devbox was working and forgot to remove it).

@McNull  thank you for the PR!
I'll try to take a look at weekend

@McNull meanwhile, can you please check at linter complaints (see failed job)?
To run ESLint locally, run yarn lint inside web directory.

@McNull can I continue to work on your PR to polish it?

@McNull can I continue to work on your PR to polish it?

Sure, np. The code is all yours. I probably don't have the time till end of this week.