init metaContract

meta contract. allows for versioning of TokenMint contracts. any contract can read the metaContract to find the latest version of token, crowdfund, or registry,

Can we add a README or come up with a naming system for the contracts in the directory? It's hard to keep track of which ones are live/deployed and which ones are WIP.


Add index responsiveness.

@rhytah @JamesMudidi @RachealN @nearjay06 @bashman1
@mderek27 @sanya-kenneth @chrismatgit @KabohaJeanMark
@mwinel

Good work but centre the nav-bar and we all need to change from  tags to  tags

haha Dude what do the last four words mean. haha

Virus-free. www.avast.com

On 11/28/18, JamesMudidi<EMAIL_ADDRESS>wrote:

Good work but centre the nav-bar and we all need to change from
 tags to  tags
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/neelxie/ireport/pull/4#issuecomment-442316336

I meant from div tags


Where to find ArucoUnity/Calibrations/USB Camera - 2017-10-27_19-20-12.xml

In calibration scene, unity project cannot find the xml file to open camera. And it is not optional. I checked all the commits in 2017-10-27 and failed to find the xml file. Can you commit it again or give an example? Thanks

Here is the detail about the error:

FileNotFoundException: Could not find file "D:\Projects\aruco-unity\src\aruco_unity_package\Assets\ArucoUnity\Calibrations\USB Camera - 2017-10-27_19-20-12.xml".
ArucoUnity.Cameras.ArucoCameraWebcam.Configure () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCameraWebcam.cs:147)
ArucoUnity.Cameras.ArucoCamera.Start () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCamera.cs:203)

Although you mentioned the file is optional and will be filled automatically, the project does need that file to config the camera.

Hi,
I didn't commit this calibration file, it is particular to the webcam I use to work on this project. Each camera needs to be calibrated before used for marker tracking. I can't provide a calibration file for you camera. But I forgot to remove the file reference in the Calibration and Tracking scenes.
So, you first need to calibrate your camera with the Calibration scene: it will generate a calibration file (by default on the ArucoUnity/Calibrations/ folder if ArucoCameraWebcam.CameraParameterFilePath property is empty).
Then use this file in the Tracking scene in the ArucoCameraWebcam.CameraParameterFilePath property.
I've updated the README.md file to make the marker creation, camera calibration and marker tracking easier to understand.

Thanks

Thanks for the reply. Actually I already have some configuration file generated by opencv. However, they are all in opencv-matrix format rather than normal xml. I don't know how to adapt them. That is why I want an example so that I can manually type my camera parameter to use your tracking scene.
My question is, what do you use for calibration, normally I used chessboard. But I guess you are using some marker to do the calibration here? Correct me if I am wrong.
(1) Then I tried your creation scene.
when I checked GridBoard_Dict4x4_50_X_7_Y_5_MarkerSize_160 and ChArUcoBoard_Dict4x4_50_X_7_Y_5_SquareSize_200_MarkerSize_120  for creation scene. There will be errors too. I strongly suggest you try that again.
(2) And then I use your calibration scene and leave the parameterFilePath empty. Turns out I still met errors like:
NullReferenceException: Object reference not set to an instance of an object
ArucoUnity.Plugin.Aruco.RefineDetectedMarkers (ArucoUnity.Plugin.Mat image, ArucoUnity.Plugin.Board board, ArucoUnity.Plugin.VectorVectorPoint2f detectedCorners, ArucoUnity.Plugin.VectorInt detectedIds, ArucoUnity.Plugin.VectorVectorPoint2f rejectedCorners, ArucoUnity.Plugin.Mat cameraMatrix, ArucoUnity.Plugin.Mat distCoeffs, Single minRepDistance, Single errorCorrectionRate, Boolean checkAllOrders, ArucoUnity.Plugin.VectorInt recoveredIdxs, ArucoUnity.Plugin.DetectorParameters parameters) (at Assets/ArucoUnity/Scripts/Plugin/Aruco/ArucoUnity.cs:436)
ArucoUnity.Plugin.Aruco.RefineDetectedMarkers (ArucoUnity.Plugin.Mat image, ArucoUnity.Plugin.Board board, ArucoUnity.Plugin.VectorVectorPoint2f detectedCorners, ArucoUnity.Plugin.VectorInt detectedIds, ArucoUnity.Plugin.VectorVectorPoint2f rejectedCorners, ArucoUnity.Plugin.Mat cameraMatrix, ArucoUnity.Plugin.Mat distCoeffs, Single minRepDistance, Single errorCorrectionRate, Boolean checkAllOrders, ArucoUnity.Plugin.VectorInt recoveredIdxs) (at Assets/ArucoUnity/Scripts/Plugin/Aruco/ArucoUnity.cs:446)
ArucoUnity.Plugin.Aruco.RefineDetectedMarkers (ArucoUnity.Plugin.Mat image, ArucoUnity.Plugin.Board board, ArucoUnity.Plugin.VectorVectorPoint2f detectedCorners, ArucoUnity.Plugin.VectorInt detectedIds, ArucoUnity.Plugin.VectorVectorPoint2f rejectedCorners, ArucoUnity.Plugin.Mat cameraMatrix, ArucoUnity.Plugin.Mat distCoeffs, Single minRepDistance, Single errorCorrectionRate, Boolean checkAllOrders) (at Assets/ArucoUnity/Scripts/Plugin/Aruco/ArucoUnity.cs:455)
ArucoUnity.Plugin.Aruco.RefineDetectedMarkers (ArucoUnity.Plugin.Mat image, ArucoUnity.Plugin.Board board, ArucoUnity.Plugin.VectorVectorPoint2f detectedCorners, ArucoUnity.Plugin.VectorInt detectedIds, ArucoUnity.Plugin.VectorVectorPoint2f rejectedCorners, ArucoUnity.Plugin.Mat cameraMatrix) (at Assets/ArucoUnity/Scripts/Plugin/Aruco/ArucoUnity.cs:463)
ArucoUnity.Plugin.Aruco.RefineDetectedMarkers (ArucoUnity.Plugin.Mat image, ArucoUnity.Plugin.Board board, ArucoUnity.Plugin.VectorVectorPoint2f detectedCorners, ArucoUnity.Plugin.VectorInt detectedIds, ArucoUnity.Plugin.VectorVectorPoint2f rejectedCorners) (at Assets/ArucoUnity/Scripts/Plugin/Aruco/ArucoUnity.cs:470)
ArucoUnity.Controllers.ArucoCalibrator.Detect () (at Assets/ArucoUnity/Scripts/Controllers/ArucoCalibrator.cs:246)
ArucoUnity.Controllers.ArucoCalibrator.ArucoCamera_ImagesUpdated () (at Assets/ArucoUnity/Scripts/Controllers/ArucoCalibrator.cs:191)
ArucoUnity.Cameras.ArucoCamera.OnImagesUpdated () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCamera.cs:409)
ArucoUnity.Cameras.ArucoCameraWebcam.UpdateCameraImages () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCameraWebcam.cs:188)
ArucoUnity.Cameras.ArucoCamera.Update () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCamera.cs:215)
ArucoUnity.Cameras.ArucoCameraWebcam.Update () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCameraWebcam.cs:114)

Hope you can help me. Thanks.

Sorry for the late reply, I'm currently working on a v2 of the project. It will be released this week.
Since the API are going to change, you could retry with this new version. I've created better prefabs and made the scenes simpler to use.
The xml configure file are going to change too. I'll ship an example on the release.
What are the errors for (1) ?
Concerning (2) that should not be normal, could you retry when the
Hope it will fix the issue

Hello,
I have the same problem.
You say the program should generate the xml file, but if I don't give it one, errors  occus.
What should I put in the "Camera Parameters File Path" field if I don't have yet one?
Thank you

@NormandErwan I think the confusion comes from a circular requirement. The docs indicate that you need to use the calibration sceneto generate the calibration file, however the calibration scene requires a calibration file.
When I set the camera calibration file path to an empty string I get this error when I hit play
I'm using release 1.2, which I just noticed looks pretty different from master.
NullReferenceException: Object reference not set to an instance of an object
ArucoUnity.Cameras.ArucoCameraWebcam.ConfigureCameraAndBackground () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCameraWebcam.cs:194)
ArucoUnity.Cameras.ArucoCameraWebcam.Update () (at Assets/ArucoUnity/Scripts/Cameras/ArucoCameraWebcam.cs:96)

Hi!
I have same problem. When I start I not have calibration xml, and if I clear "CalibrationFilename" field in inspector then i get NullReferenceException.

Same issue. Please reply and fix it.

I got it to work by cloning the master (he fixed the problem there). However you need to fix compile errors by copying plugins/x64 folder and renamed ArucoUnity dll to ArucoUnityPlugin and in Plugin/Cv/Size.cs rename au_cv_Size_new1 to au_cv_Size_new.

Hi everyone,
Very sorry for the late reply.
Before doing any tracking, you need to calibrate your camera. It's required by the aruco module of OpenCV. This process measures some parameters of the camera (essentially the focal length, the optical center point and the image distortions). It allows aruco to determines precisely the position and rotation of the markers. You can read the doc for more details: https://normanderwan.github.io/ArucoUnity/manual/build-from-sources.html.
However, I think it could be possible to add a dummy calibration just to test the tracking but with less accurate detected positions...
You all right, it's better to use the master branch actually. You can get the updated ArucoUnityPlugin here. I moved the plugin to a dedicated repository and added continuous integration build. So the plugin in the release is up to date!
@FreakTheMighty the calibration file is optional during the calibration process. The field is visible because OpenCV allows to refine a calibration. I'll update the documentation to make this point clearer.  Thank you for your comment!
I close the issue. I think the documentation is explicit. But feel free to open any issue if some points on the docs can be improved! Finally, I need to release the v2 as soon as possible...

I've added a pre-release version of the package, with up to date plugin: https://github.com/NormandErwan/ArucoUnity/releases/tag/v2.0-alpha


Add libcupti support

From the official TensorFlow Docker image:
# For CUDA profiling, TensorFlow requires CUPTI.
ENV LD_LIBRARY_PATH /usr/local/cuda/extras/CUPTI/lib64:/usr/local/cuda/lib64:$LD_LIBRARY_PATH

Without it, I can't run any examples on TensorFlow.

Actually, I can. The training works. But the error has to be fixed.


The process to create lighting from reference image

Hi, thanks for the great work. I would like to ask you a few questions about the lighting transfer.

How can I transfer the light from the source to the reference image?
Do I need other methods to perform the lighting direction estimation? I tried the EstimationLightDir.m and it outputs 8 envir_map.csv files, I don't know the meaning of those 8 files and the next step. Could you show and explain to me the step to doing this?

Thanks

Hi there, thanks for your interest in our work!
Our method can estimate the lighting from one image (reference image) and transfer it to another (source image). First, feed the reference image to estimate its lighting. In this line "predicted_ratio = self.sess.run([self.predicted_ratio], feed_dict=ffeed_dict)", you can add self.predicted_source_lighting to the sess.run call and save out the estimated SH lighting to a text file. You would feed the reference image to the model in this case and some 9D SH vector as the target light (doesn't really matter in this first call since you're just saving out the lighting of the reference image). Then, run our model again feeding the source image and the saved estimated lighting as the target light. Let me know if any of this is unclear.

Thank you for your response. I got an idea of this. thank you very much.
So what's the use of EstimationLightDir.m and envir_map.csv?

Oh that code is how to convert a lighting direction into SH and vice versa.


format for weekday labels on month view

Added an attribute to the main directive: calendar-day-label-format, which allows the user to specify 'dd', 'ddd', or any other moment format. In the helper I switched from Angular's date $filter to moment's format() because it offers way more flexibility: http://momentjs.com/docs/#/displaying/format/
The rest of the calls to getWeekDayNames already pass false, which will use the default 'dddd' format (same as Angular's 'EEEE').
For example, I want to display 'S', 'M', 'T', etc. for my weekdays on top of the month view calendar. I can set calendar-day-label-format="dd", then globally override moment's weekdaysMin to just the first letter:
moment.locale('en', {
    weekdaysMin : ["S", "M", "T", "W", "T", "F", "S"]
});

Hi, thanks for the pull request.
I think the default behaviour should be to fallback to using angular's date filter if no value is set, this would preserve backwards compatibility and avoid users having to load moment's locale files as well as angular's.
For the sake of completeness I would suggest adding it to the week view and add a similar option for the year view as well.
I'll happily make these changes myself on the weekend or feel free to send an updated PR :)

With this implementation, don't you only need to load moment's locale files if you're providing another translation on top of 'en'? Anyway, the reason I choose to use moment().format() instead of angular's date filter is that allows me to use custom day names, plus all of moment's day formatting options. With Angular, you only have the options of EEE or EEEE --- which you had already accounted for with the short = true/false param.
Thanks for the feedback. I figured I might have to maintain a fork in order to make the tweaks I need for a specific implementation, but I thought I'd make them generic enough for you to use, if you so choose.

https://github.com/mattlewis92/angular-bootstrap-calendar/issues/43 was asking for something similar. So I've made all date formats configurable in a provider. You can use it like this:
calendarConfigProvider.configureDateFormats({
  weekDay: 'dd'
});

This should satisfy your use case :)

@mattlewis92 Does this still work? I couldn't seem to use that code or the issue you referenced to achieve the single character weekday abbreviation.


feature:  下架 gsecmdline 插件 (closed #1604)

Codecov Report

Merging #1605 (7102278) into v2.3.2-dev (a64bb17) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@              Coverage Diff               @@
##           v2.3.2-dev    #1605      +/-   ##
==============================================
- Coverage       73.42%   73.40%   -0.03%
==============================================
  Files             389      389
  Lines           26244    26244
==============================================
- Hits            19270    19264       -6
- Misses           6974     6980       +6

Impacted Files
Coverage Δ

apps/node_man/apps.py
76.59% <ø> (ø)

apps/node_man/constants.py
96.17% <100.00%> (ø)

... and 4 files with indirect coverage changes


Alternative initialization methods for CLI tools

Rationale
Unlike WSGI applications, CLI tools usually offer no mechanism to be consistently monkeypatched, thus hampering any call to sentrysdk.init(). If the tool configuration was a python file or the shell script was fully wrapped in an importable main function, one could still hack a way to initialize sentry. But this is not always a possibility.
Still, tools can usually be told to use a particular file configuration for the standard logging system.
Proposal
I'd love that sentry provided a mechanism to initialize itself on this kind of tooling, specially for scripts that are meant to be run as daemons in some frameworks.

This as a follow-up for a naive implementation that was proposed as part of the Tryton Server Framework Sentry integration https://github.com/getsentry/sentry-python/pull/548
The implementation can be reduced to
class SentryInitHandler(logging.NullHandler):
    def __init__(self, dsn):
        super(SentryInitHandler, self).__init__()
        sentry_sdk.init(dsn)

There is no need to inherit sentry_sdk.integrations.logging.EventHandler because sentry_sdk.init will already install the default sentry_sdk.integrations.logging.LoggingIntegration that will already patch any logger.error call. Then, to use it, this handler must be configured once in an arbitrary logger
[loggers]
keys = root
[handlers]
keys = console, sentry
[formatters] keys =
[logger_root]
level = INFO
handlers = console, sentry
[handler_console]
class = StreamHandler
args = (sys.stdout,)
[handler_sentry]
class=sentry_sdk.integrations.trytond.TrytondSentryHandler
args =<EMAIL_ADDRESS>
As can be seen in the former discussion and https://github.com/getsentry/sentry-python/pull/548/commits/2bdfe53fc77ada9189b1ebd36d1b18882e783882 ,
@untitaker was legitimately concerned about this implementation being tailored for a simplistic usage of the SentrySDK.
Accepting this implementation as-is would be controversial since it would be arbitrarily unusable for advanced sentry configurations, so this issue is opened in the hope that this idea can be improved and merged into this repo.

Alternatively, this could be addressed with a sentry-sdk CLI that wrapped other CLI tooling, like
sentry-init --dsn=https://example.com --whatnot -- my-tool

but I believe this path is much harder. For executable python modules (__main__.py) this could be addressed safely, but again this is not always a possibility.

There is no need to inherit sentry_sdk.integrations.logging.EventHandler because sentry_sdk.init will already install the default sentry_sdk.integrations.logging.LoggingIntegration that will already patch any logger.error call.

I believe the integration should generally function regardless of whether other integrations are enabled. There's no real harm in reporting errors twice, the core SDK will deduplicate them based on object ID if they are reported consecutively from within the same thread.

I believe the integration should generally function regardless of whether other integrations are enabled.

I have updated the link to our former discussion, github was cutting it to a silent commit. The point is: this is NOT an integration. Maybe the problem is depicting this handler in the integrations submodule.
The purpose of this handler is not capturing logging events. The purpose is to abuse the logging system to inject a sentry_sdk.init call in CLI tools that refuse to be monkeypatched!
It should be orthogonal to the existing sentry_sdk.integrations.logging usage, whether it is the default logging integration or a carefully crafted EventHandler & BreadcrumbHandler tree.
For example: if someone needs this initialization method but also wants not to capture any log events into sentry, this is only possible if this handler inherits logging.NullHandler and not sentry's EventHandler! Then, if someone did want to capture logging events... they could simply rely on the default LoggingIntegration or work out their Breadcrumb/EventHandler config.

I think the real challenge is how to unleash the full sentry configurability with this kind of approach. One solution could be to simply forward any arbitrary args and kwargs from the logging configuration into the sentry_sdk.init call like
class SentryInitHandler(logging.NullHandler):
    def __init__(self, *args, **kwargs):
        super(SentryInitHandler, self).__init__()
        sentry_sdk.init(*args, **kwargs)

and then use it with
[handler_sentry]
class=sentry_sdk.?????.SentryInitHandler
args =<EMAIL_ADDRESS>, )
kwargs = {
 'max_breadcrumbs': 50,
 'debug': False,
 'default_integrations': True,
 'with_locals': False,
 'whatnot': None}

which would already leverage all "static" parametrization of the sdk. Yet, options requiring references to functions or classes like integrations and hooks would still be limited, since the eval context of the logging fileConfig is also limited (thankfully).
Would this be an acceptable trade-off? Otherwise this could be addressed if sentry_sdk.init accepted full import paths for functions and classes for those options? But this would better be another discussion.

Hello @n1ngu
Thanks for the proposal and sorry for the late reply.
This sounds interesting and I think it could be useful for a lot of python scripts out there.
I have not yet had time to really wrap my head around using the logging infrastructure to sneak in a sdk init. From the gut feeling it is a little bit hackish, but maybe its a brilliant idea and just need some quiet time to think it through.
Anyways. Could you create sort of an very minimal proof of concept project that we could use for further discussions? This would be great!

Hi @antonpirker
See the example proof of concept in this minirepo https://github.com/n1ngu/sentry-init-handler
Indeed, hackish is the word. It is working and is stable in my setup, but shipping it within sentry_sdk... I think a big challenge would be documenting this so that no one confuses it with the other useful logging handlers that sentry_sdk already provides, and also so that target users that could benefit from it would easily find the documentation while keeping away all the other use cases that should NOT use such technique.

Thanks for the minirepo @n1ngu! Amazing!
Basically this would be something like a CLI integration. (no code changes needed)
I have now put this on the internal backlog at low priority. (So please be patient, we have always a lot on our plates)


Feature proposal: Add DelegateAdaptor as a Property

Feature proposal: Add DelegateAdaptor as a Property
With this proposal, developers will have the option to add a DelegateAdaptor during the request specification, allowing them to receive exclusive notifications from URLSession.
Although RequestDL may not cover all possible scenarios, this functionality will give developers more control over the progress of the request, which is just as important as the other resources being developed.
Proposed solution
DataTask {
    // Properties specifications
    DelegateAdaptor(someOtherDelegate)
}

Technical Details
Currently, the way the delegate is used may not be straightforward to directly connect with the DelegateAdaptor. Therefore, it is necessary to review this proposal to identify the best way to implement it and make it available to developers.

We inform you that the proposed feature has been closed due to the launch of the new development course for version 2.0.0. We thank all developers who have submitted their suggestions and ideas to improve our platform Topic. We will continue to work to provide the best possible learning experience.


DAP is deprecated. Update for GDAP and using MS Graph PowerShell module

Instructions for pre-consenting has changed. DAP is deprecated.
Also Azure AD PowerShell is deprecated.
This article needs to be re-written using supported methods

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

ID: a8f7169c-463e-61c1-dfe1-38a43f4d696b
Version Independent ID: a27fdad3-5138-6cd5-cb17-30be944dfb25
Content: Call Microsoft Graph from a Cloud Solution Provider application - Microsoft Graph
Content Source: concepts/auth-cloudsolutionprovider.md
Product: partner-customer-administration
Technology: microsoft-graph
GitHub Login: @koravvams
Microsoft Alias: MSGraphDocsVteam

@omondiatieno - I believe this issue is also related to the Azure AD PS references work. Can you add the details to the related work item?
@koravvams, please note.


Green Drakes lose ability to target/attack ground units after Elemental Invasion occurs

Likely to be the upgrade "emerald flames" rather than fireland invasion.


Update Caprine to 0.5.1

Thank you for the contribution. It was merged directly as 98c55becc8f4bf2099d2ca02e90810b8dbc69184 to keep commit history cleaner. Your contribution is still credited to you.

:+1:


Add event for creating scheduled updates

I'm talking about World#scheduleUpdate.
This event will allow plugins to change delay of liquid/fire spreading, blocks falling, buttons, pressure plates, redstone torches, tripwires, rail detectors. Or even stop them from ticking at all (for something like one-time use buttons or static liquids)

We already have this, https://jd.spongepowered.org/4.1.0/org/spongepowered/api/event/block/TickBlockEvent.html

Ticks arn't necessarily scheduled block updates. I believe Meegooo was asked to create this issue by someone.

And I was told on IRC that TickBlockEvent fires for random ticks (growth and stuff), not scheduled ones.

There used to be a sub event for scheduled, it appears to be gone

Isn't this just ScheduledBlockUpdate?
https://github.com/SpongePowered/SpongeAPI/blob/44b5f5ecc6bf3751ef81b1ff6321fbe3f08099fc/src/main/java/org/spongepowered/api/world/extent/Extent.java#L333-L395

@simon816 This is for an event to listen, rather than create - from what I can tell

You can't really use those methods the way event could be used.
I mean, in case of stopping water from spreading you could use something like this.
Listen for ChangeBlockEvent.Place for water and after 1 tick change scheduled updates at that  location (and around it)
But that is quite hacky and might not work for scheduled updates with 1 tick delay.  (scheduled updates are created after ChangeBlockEvent is fired, right?)
PS. I just realized. ScheduledBlockUpdate doesn't have information about future block inside it. So you can't really tell what will happen after that update occurs. Looks like it just ticks whatever is located at Location. World#scheduleUpdate, from what I can tell, is used to place/change blocks


[Held requests] Clean up the hold/unhold logic

Coming from here, we're seeing some inconsistencies in the intended design for hold/unhold logic. For example, a member on a workspace can unhold their expense even if the approver put it on hold.
@robertjchen @BartoszGrajdek let's go through this and make sure we clean up any inconsistencies. Any questions or further clarifiers necessary, let's align in the Slack thread.
Hold
Expenses on expenseReports

Members can hold their own expenses.
Approvers can hold expenses submitted to them.
Admins can hold any expense on a workspace they’re an admin of.

Expenses on iouReports

Members can hold each others expenses.

Unhold
Expenses on expenseReports

Members can unhold their own expenses, but only if they themselves put the expense on hold.
Approvers can unhold expenses submitted to them.
Admins can unhold any held expense on a workspace they’re an admin of.

Expenses on iouReports

Members can only unhold expenses they put on hold themselves.

Thanks for going through this and laying out all the cases! 🙇

Anytime!

Hi! I'm currently quite busy with maintainment of live markdown lib, but @cdOut from our SWM team has the capacity to take this over. He already worked with hold requests feature, so he knows everything necessary to work on this. 🙏🏻

Hi, I'm Tymoteusz Kałuzieński from Software Mansion, the expert agency, and I'm going to be working on this issue.

Great!

Started on the backend, draft PR linked!
Having reviewed the existing logic, it looks like the only key thing we would have to change, at least for the backend, is just the unhold case (for both iouReports and expenseReports): Members can only unhold expenses they put on hold themselves.. All other cases appear to be working as intended.
Should have the backend PR wrapped up with automated tests tomorrow.
@cdOut Let me know if you have any questions on the frontend! In the Hold reportAction, we store the accountID of the person who performed the initial hold action, so you should be able to check that and display the Unhold button if the viewer is authorized to perform the action 👍

Backend PR now under review 👍

@cdOut Just to clarify that the frontend changes can be done independently without having to wait for the backend, let me know if you have any questions on the flows! 🙇

I'll start looking into this one today, I'll let you know if I have any questions.

Assigning @parasharrajat for the review. @cdOut let us know when the PR is up. Thanks!

I'll have a draft PR ready by the EOD.

It is in draft, will move through to review by tomorrow.

Thanks @cdOut ! Let me know if you need anything else, the backend fixes were deployed 👍

Latest update here: https://github.com/Expensify/App/pull/45151#issuecomment-2255272532

Finishing touches on the PR, looks on track to be merged later today.

Under review/testing, looks like a few final bugs to hammer out, but we're close

PR was updated and under review! 🏃

Closing this out as this was internal, please re-open if we missed anything.

Payment summary:

Contributor+: @parasharrajat $250

Please request on NewDot!

Thanks for that, Dylan! With the payment summary done, we can close it now. Thanks!

$250 approved for @parasharrajat


The "Citation" resource have no search parameters in FHIR R5

Describe the bug
When getting Citation resource from the deployed or local running R5 fhir server, the common search parameters like "_lastUpdated" is not supported.
FHIR Version?
R5
Data provider?
CosmosDB
To Reproduce
Steps to reproduce the behavior:

Deploy or locally run a R5 fhir server.
Send the http request like
https://localhost:44348/Citation?_lastUpdated=ge1970-01-01&_sort=_lastUpdated

Expected behavior
The Citation resources.
Actual behavior
The fhir server response indicates that "_lastUpdated"  is not supported, but this parameters is common basic search parameters according to fhir definition.
"issue": [
	{
		"severity": "warning",
		"code": "not-supported",
		"diagnostics": "The search parameter '_lastUpdated' is not supported for resource type 'Citation'."
	},
	{
		"severity": "warning",
		"code": "not-supported",
		"diagnostics": "Sort parameter value '_lastUpdated' is invalid, must be a subset of valid search parameters for resource type 'Citation'."
	}
]

Other
From the codes, seems VersionSpecificModelInfoProvider deliberately removed Citation from supported resources in line 54 when building search parameters definition for resources.
public IReadOnlyCollection<string> GetResourceTypeNames()
{
	List<string> supportedResources = ModelInfo.SupportedResources;

	if (Version == FhirSpecification.R5)
	{
		supportedResources = supportedResources.Where(x => x != "CanonicalResource" && x != "MetadataResource" && x != "Citation").ToList();
	}

	return supportedResources;
}

AB#94600

Agree, this is probably a bug and this should be included


Syntax error in 2.7.0

Summary
I just updated the docker image tiredofit/db-backup from 2.4.0 to 2.7.0. Unfortunately this one doesn't work anymore. There seems to be a syntax error in the run script.
Steps to reproduce
Just execute a previously working config. For me this happens for a mariadb and postgres configuration.
What is the expected correct behavior?
The script should work.
Relevant logs and/or screenshots
./run: line 38: syntax error near unexpected token `)'
./run: line 38: `    "mysql" | "MYSQL" | "mariadb" | "MARIADB")'

Environment

Image version / tag:  tiredofit/db-backup:2.7.0
Host OS: Synology

Possible fixes
In commit https://github.com/tiredofit/docker-db-backup/commit/eb0ee6166200f6b2d6802341fe890cf97e16f30e there have been two ;; removed from the run script. Possibly this causes the syntax error. The previous version 2.6.1 works fine.

Confirmed, the removal of the ;; breaks the case statement, inserting ;; below line 37 of 10-db-backup/run file allows both manual and cron job backups again.

Sorry about this - that was sloppy on my end. Builds should be working now (at least with :latest). I'm away for a few days and will return with a tagged release then.

@tiredofit Could you please make a release? The image on the Docker Hub still fails.


Coupling issue

I've tried coupling my Binary Access Link Layer Service while injecting DICSS directly into the head, but it always prematurely exits. Any known issues about this?

I think this is a known issue. DICSS and BALLS tend to have conflicts when injected together; this is because BALLS is much more sensitive and lower-level than DICSS, and they tend to get in each other's way in terms of registering touch events. I think there is a workaround if you alternate between injecting DICSS and BALLS into the head.

Thanks. In the end I just injected the DICSS into the body and the BALLS in to the head (it was a stretch, but it worked). Had to rewire my BALLS though to try and stop all the child processes being spawned.


Recyclerview scroll with Flexbox layout manager

Hi
Does recyclerview scrolling stops working with FlexBox layout manager?
I have also wrapped recyclerview inside scroll view with android:fillViewport="true" option also but recycler view scrolling is not working.

It should work. Can you link your project or code?

// XML FILE
this is my recyclerview
<android.support.v7.widget.RecyclerView
android:id="@+id/recycler_view"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:scrollbars="vertical"/>
// Recycler view with scroll view that i have tried-

<android.support.v7.widget.RecyclerView
    android:id="@+id/recycler_view"
    android:layout_width="match_parent"
    android:layout_height="match_parent"/>

</ScrollView>

// JAVA FILE
FlexboxLayoutManager layoutManager = new FlexboxLayoutManager();
layoutManager.setFlexWrap(FlexWrap.WRAP);
layoutManager.setFlexDirection(FlexDirection.ROW);
layoutManager.setAlignItems(AlignItems.STRETCH);
recyclerView.setLayoutManager(layoutManager);

Hi
Its been solved...it was some other issue.
Thank you.


Added support for clang with GNU stdlib (relevant on Ubuntu 18.04)

This PR adds support for using clang with GNU stdlib on older versions of stdlib where its required to pass stdc++fs to the linker.
NOTE: In the cmake check for clang_with_gnu_stdlib, there is a generator expression checking $<NOT:$<BOOL:${WIN32}>>. This is not exactly what I would like to have there. Better would be a check for Linux, but I did not find a nice way to put that into a generator expression. Maybe UNIX would do, but the fix is also not required on Apple.
Closes #152

I'll look at this over the weekend and try to do something about the CI failures too

I updated the main branch to fix the CI issues, can you merge and run again?

Rebased on latest main and pushed again, lets see what the CI says...

Thanks!


DaisySP build fails

Hi there. I ran into the follow error trying to build on Windows:
arm-none-eabi-g++ -c -mcpu=cortex-m7 -mthumb -mfpu=fpv5-d16 -mfloat-abi=hard -DUSE_HAL_DRIVER -DSTM32H750xx -DUSE_HAL_DRIVER -DSTM32H750xx -I.  -O3 -Wall -fdata-sections -ffunction-sections -MMD -MP -MF"build/drip.d" -MT"build/drip.d" -fno-exceptions -finline-functions -static -Wa,-a,-ad,-alms=build/drip.lst modules/drip.cpp -o build/drip.o
modules/drip.cpp: In member function 'int daisysp::Drip::my_random(int)':
modules/drip.cpp:20:18: error: 'rand' was not declared in this scope
     return (rand() % (max + 1));
                  ^
modules/drip.cpp: In member function 'float daisysp::Drip::noise_tick()':
modules/drip.cpp:26:24: error: 'rand' was not declared in this scope
     temp = 1.0f * rand() -<PHONE_NUMBER>.5f;
                        ^
Makefile:177: recipe for target 'build/drip.o' failed
make: *** [build/drip.o] Error 1

After a bit of googling, I found that adding #include <cstdlib> to the drip.h file fixed the problem.


Upgrade KafkaCat image used in KafkaSource e2e tests

The 1.6.0 version includes built-in retrying.
Fixes #789

/lgtm

/cherry-pick release-0.23

/cherry-pick release-0.24


[Task]: Add ability to set custom width

This is an integral part of IDE layout grid component, it was completed in the parent task.


[WM-2259] Use query params to navigate in Workflows landing page

Jira Ticket: https://broadworkbench.atlassian.net/browse/WM-2259

Summary of changes:

What

Workflows app landing page navigates between subheaders with a query parameter
Submission and run details breadcrumbs now point to submission history subtab

Why

Users should be able to bookmark specific subheaders, e.g. submission history
Users should be able to navigate directly to subheaders from other parts of the UI

Testing strategy

Unit tests
Interacted with UI

https://github.com/DataBiosphere/terra-ui/assets/67511512/30eb4c3e-ad18-44e3-be5d-7fe74705db75

I'm still not 100% sure why this is a query param rather than an extension of the nav-path like we do everywhere else

Responded to this message in Slack first - for anyone curious in the future:
The internal workings (useQueryParameter) comes from within Terra-UI in addition to other examples of similar patterns - for example, the billing projects page. Though we didn’t end up doing all of the possible abstraction, the other primary reasoning was avoiding repetition (no need to duplicate sidebar component/page structure, and we could pull out some of the network calls to avoid repeating them in between tab switches).


[NavigationExperimental] Best way to dynamically change content of overlay from scene?

I want to control / update content of overlay from the scene component.
For example, I want to dynamically change nav bar title based on user interactions inside a scene without doing any navigation.
Afaik, the following is a simplified version of how scene and overlay are rendered inside a AnimatedView or CardStack.
<AnimatedView>
  {scene}
  {overlay}
</AnimatedView>

Correct me if I am wrong, but currently, it seems the only way to trigger rerendering of overlay from the scene is to update navigationState.
That means we need to use a reducer to create a new navigationState, which will cause rerendering of scene as well as overlay.
It sounds like an overkill and too heavy work for such a trivial task.
Another way I can think of is to expose a new API via context that can override overlayProps.
Here's a simplified code snippet for the concept:
class ContainerWithContext extends Component {
  getChildContext() {
    return {
      setOverlayProps: (overlayProps) => this.setState({overlayProps})
    };
  }
  renderOverlay() {
    return this.props.renderOverlay({
      ...this.props,
      ...this.state.overlayProps,
    });
  }
  render() {
    return (
      <View>
        {this.renderScene()}
        {this.renderOverlay()}
      </View>
    );
  }
}

const Overlay = ({title}) => <View><Text>{title}</Text></View>;

class Scene extends Component {
  static contextTypes = { setOverlayProps: PropTypes.func }

  updateNavTitle(title) {
    this.context.setOverlayProps(title);
  }
}

I can help with PR if you think this is a good idea.
What are your thoughts on this?

I would not use context, as it would be an overkill. There seems to be multiple ways to approach this. But I'm not convinced this belongs within NavigationExperimental. The low level nature of NavigationExperimental enables greater abstractions to be built up in higher layers.
Also, NavigationAnimatedView is deprecated. As for NavigationTransitioner, renderScene and renderOverlay are also deprecated. NavigationTransitioner now delegates the rendering of all its children to a user provided render function. This addresses the issue discussed here. A nice side effect of this is that it allows full control over both the scene and overlay components.
 Let me know if this doesn't make sense and I'll put together an example. Oh and then there is Redux 😆
🍺

I've used Redux before, but haven't needed it yet since I started using Relay, but I think it's the time to use it together with Redux.

or you can also use an event emitter like the way exnavigation does:
https://github.com/exponentjs/ex-navigation/blob/6271d5cb1cdfa0336cdf57edff44d0a3ea3f7550/src/ExNavigationRouter.js#L57-L59

I will close this as recommended solutions make sense.

How can I make a reference to the header from my navRoot?
I have this now.
    <NavigationCardStack           onNavigateBack={this._handleBackAction.bind(this)}           navigationState={this.props.navigation}           onNavigate={this._handleNavigate.bind(this)}           renderScene={this._renderScene}           renderOverlay={this._renderHeader} />
`  _renderHeader(props){
const { route } = props.scene
return (
  <Header
        route={route}

        _goBack={this._handleBackAction.bind(this)}
        _handleDrawerAction={this._handleDrawerAction.bind(this)}
        ref={(header) => { return this.header = header  }}>>
  </Header>

)
}`
When I try to do a function on the header from my navroot component
header.consoleLog()
I get an error in my application. this.headerconsoleLog is not a function.
Header component code snippets
`class Header extends Component {
constructor(props){
super(props)
this.consoleLog = this.consoleLog.bind(this)

}
consoleLog(){
console.log('Change header via other component');
}`

@jmurzy What can I do to reference my header in e.g. my scenes?

@jmurzy - I am still battling on how to change the overlay from a scene. What I would like to do is change the buttons and title from some of my view components created in the renderScene function.
Have you built the sample you mentioned above so we see how this is supposed to work?


Fix and enhance subquota in the dashboard

This PR:

Adds the subquota as a filter to the dashboard (sidebar clickable filter)
Translates subquota column in the dashboard list view
Add some db index to speed up this filter

@vilapedro as requested for your article, the link for taxis and stuff, once this is merged and deployed, will be http://jarbas.serenatadeamor.org/dashboard/core/reimbursement/?subquota_id=122

Changes Unknown when pulling<PHONE_NUMBER>be8c8e89513a8559293499e6af3c19 on cuducos-subquotas-list-filter into ** on master**.


Sass/Scss nested rules not sorted by leftovers selector

Hello,
The problem
I found no way to configure position of nested rules when working with scss syntax (not tested on less/sass). I thought that nested rules would be catched by the leftovers selector ("...") but that's not the case : every nested rules always gets at the end
current behavior
with this config :
"sort-order": [
	[ "color", "font-weight" ],
	[ "..." ],
	[ "$include" ]
]

and this main.scss
.container {
    & > p {
        font-weight: bold;
    }
    @include mediaDestkop {
        color: blue;
    }
    color: red;
}

We get this result (note that & > p is at the end)  :
.container {
    color: red;
    @include mediaDestkop {
        color: blue;
    }
    & > p {
        font-weight: bold;
    }
}

Expected :
I would expect that nested rules get catched by the leftovers selector and sorted accordingly :
.container {
    color: red;
    & > p {
        font-weight: bold;
    }
    @include mediaDestkop {
        color: blue;
    }
}

Alternative
Another option could be to add a new identifier to support nested rules in the same way $include, $extend, etc. does. A deleted user mentioned in https://github.com/csscomb/csscomb.js/issues/462#issuecomment-246597260 to add a new "$selector" keyword (not sure the term is ideal, but the idea is interesting).
Help ?
I would be glad to help but after looking at the code at https://github.com/csscomb/csscomb.js/blob/dev/src/options/sort-order.js I could not find where nested declarations are treated.
@tonyganch if you could point me to a direction, I could try to do a PR. :)
NB: I created this issue event if there is a discussion that started in #462  comments, since the original issue isn't exactly the same.

@jdalton I saw you were the last one to udpate the sort-order.js file, maybe you know were to look at ? II would be glad to help and do a PR, but I couldn't find where to start. Thanks

Hi @tfritsch-km!
After forking the repo, cloning the repo, and npm i'ing you can explore the package.json for the various scripts to run (e.g. npm run build and npm test). Start with modifying sort-order option. Possibly the _sortLeftovers method. Then adding a test to test/options/sort-order/process/test.js. The files used in the tests come from the fixture folders of css, less, sass, and scss.

Hey @jdalton,
Thanks a lot for the quick answer and detailed procedure ! 🙂
What I couldn't find when reading the code and comments is where nested rules are treated (it seems not to be in _sortLeftovers unfortunately), I'll try to find out as soon as I get time to work on it, but maybe you have any idea where it can be ?

It may be that only specific nodes are sorted, so the lack of a specific check for nested rules means they are simply ignored and not sorted or left to the end.

Yeah, that's what I thought too, but I was wondering where this detection is done.
Anyway I think I've found something that works ! I'll send a PR in a couple of days if it really works (it's late here and I'm not sure i'm not dreaming 😄 )

Awesome! Thank you!

So I've got tested it a little bit more and :
good news : it works ! Nested rules are sorted alongside leftovers by default and sorted by a new "$ruleset" selector if provided in "sort-order" config
bad news : it messes up about 7 or 8 tests, because since nested rules are considered as part of the "leftovers" group, csscomb adds a new line before them
ex.
with config
{
    "sort-order": ["top", "color"]
}

the sass file :
div
    color: tomato
    a
      color: nani
      top: 0

becomes :
div
    color: tomato

    a
      top: 0
      color: nani

(note the new line before the "a" ruleset)
I think I could test if "$ruleset" exists in config, and if not, assign the ruleset nodes to the last group of config that is not leftovers. I'm not sure this approach wouldn't lead to another bunch of test failures...

Awesome! Please open a PR and we can work through the issues together 🤝


x/mobile: gobind type-checking fails with v2+ modules

What version of Go are you using (go version)?

go version go1.12.7 darwin/amd64

Does this issue reproduce with the latest release?
Yes, it also happens on 1.12.6
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN="/Users/Tristian/go/bin"
GOCACHE="/Users/Tristian/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/Tristian/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/Cellar/go/1.12.7/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.12.7/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/go-build089542378=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
The setup consists of 2 packages a gomobile-v2target (mobile) which imports a package that is
v2 suffixed, and a gomobile-v2mod (v2mod) package that has a sub package that is imported in the root package and that is declared as v2.
The project that produces the framework can be located here:

https://github.com/triztian/gomobile-v2target

The project that is v2 and is imported by the previous one can be found here:

https://github.com/triztian/gomobile-v2mod

Both projects exist under the $GOPATH and I've verified that they build successfully without using gomobile and with GO111MODULE=off:
Framework producer Build Output
 λ gomobile-v2target ~> go build -a -v ./...
errors
math/bits
runtime/internal/sys
runtime/internal/atomic
internal/cpu
unicode/utf8
runtime/internal/math
internal/race
sync/atomic
unicode
internal/bytealg
math
internal/testlog
github.com/triztian/gomobile-v2mod/subpkg
github.com/triztian/gomobile-v2mod
runtime
strconv
sync
io
reflect
syscall
internal/syscall/unix
time
internal/poll
os
sort
internal/fmtsort
fmt
github.com/triztian/gomobile-v2target

v2 module Build Output
 λ gomobile-v2mod ~> go build -v -a ./...
github.com/triztian/gomobile-v2mod/subpkg
github.com/triztian/gomobile-v2mod

When trying to generate the framework with the following command:
$ gomobile bind -x -a -v -target ios .

We get the following output:
GOMOBILE=/Users/Tristian/go/pkg/gomobile
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-104674381
GOOS=darwin CGO_ENABLED=1 $GOPATH/bin/gobind -lang=go,objc -outdir=$WORK -tags=ios github.com/triztian/gomobile-v2target
type-checking package "github.com/triztian/gomobile-v2target" failed (/Users/Tristian/go/src/github.com/triztian/gomobile-v2target/export.go:6:8: could not import github.com/triztian/gomobile-v2mod (type-checking package "github.com/triztian/gomobile-v2mod" failed (/Users/Tristian/go/src/github.com/triztian/gomobile-v2mod/math.go:3:8: could not import github.com/triztian/gomobile-v2mod/v2/subpkg (cannot find package "github.com/triztian/gomobile-v2mod/v2/subpkg" in any of:
	/usr/local/Cellar/go/1.12.7/libexec/src/github.com/triztian/gomobile-v2mod/v2/subpkg (from $GOROOT)
	/Users/Tristian/go/src/github.com/triztian/gomobile-v2mod/v2/subpkg (from $GOPATH)))))

rm -r -f "$WORK"
gomobile: /Users/Tristian/go/bin/gobind -lang=go,objc -outdir=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-104674381 -tags=ios github.com/triztian/gomobile-v2target failed: exit status 1

if it helps, this is the structure of my GOPATH:
λ triztian ~> pwd && tree gomobile-v2*
/Users/Tristian/go/src/github.com/triztian
gomobile-v2mod
├── LICENSE
├── README.md
├── go.mod
├── math.go
└── subpkg
    └── sub.go
gomobile-v2target
├── README.md
├── export.go
└── go.mod

1 directory, 8 files

What did you expect to see?
A Mobile.framework bundle.
What did you see instead?
The following error message:
λ gomobile-v2target ~> gomobile bind -x -a -v -target ios .
GOMOBILE=/Users/Tristian/go/pkg/gomobile
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-431765436
GOOS=darwin CGO_ENABLED=1 $GOPATH/bin/gobind -lang=go,objc -outdir=$WORK -tags=ios github.com/triztian/gomobile-v2target
type-checking package "github.com/triztian/gomobile-v2target" failed (/Users/Tristian/go/src/github.com/triztian/gomobile-v2target/export.go:6:8: could not import github.com/triztian/gomobile-v2mod (type-checking package "github.com/triztian/gomobile-v2mod" failed (/Users/Tristian/go/src/github.com/triztian/gomobile-v2mod/math.go:3:8: could not import github.com/triztian/gomobile-v2mod/v2/subpkg (cannot find package "github.com/triztian/gomobile-v2mod/v2/subpkg" in any of:
	/usr/local/Cellar/go/1.12.7/libexec/src/github.com/triztian/gomobile-v2mod/v2/subpkg (from $GOROOT)
	/Users/Tristian/go/src/github.com/triztian/gomobile-v2mod/v2/subpkg (from $GOPATH)))))

rm -r -f "$WORK"

gomobile does not work in module mode. See #27234

@AlexRouSg Thank you for the reply yes and thank you for the link!,  I was aware of this limitation, however this issue is slightly different, in the issue that you mention the build is done with GO111MODULE=on (and seemingly outside a GOPATH) as it can be seen from the command output:
gomobile: go build -buildmode=c-shared -o=/var/folders/nt/0hzfppd92jvbk99fhvnhy8ph0000gn/T/gomobile-work-327189385/android/src/main/jniLibs/armeabi-v7a/libgojni.so gobind failed: exit status 1
go: finding github.com/dustin/go-humanize v0.0.0-20180713052910-9f541cc9db5d
go: finding golang.org/x/mobile v0.0.0-20180808221059-bceb7ef27cc6
can't load package: package gobind: unknown import path "gobind": cannot find module providing package gobind

(notice the go: finding ... statements, those indicate that the module system is being used)
In this issue the the build is done inside a GOPATH and with GO111MODULE=off, I was under the impression that the rules defined here would take effect:

"Additional Details – "Minimal Module Compatibility": https://github.com/golang/go/wiki/Modules#how-are-v2-modules-treated-in-a-build-if-modules-support-is-not-enabled-how-does-minimal-module-compatibility-work-in-197-1103-and-111

The type-checking package "github.com/triztian/gomobile-v2target" ... error message is generated from srcimporter.go however the minimal compatibility notes mentions:

The entirety of the logic is – when operating in GOPATH mode, an unresolvable import statement containing a /vN will be tried again after removing the /vN if the import statement is inside code that has opted in to modules (that is, import statements in .go files within a tree with a valid go.mod file).

And the type-check error points towards the module not being tried again (for whatever reason), in this situation the imported package gomobile-v2mod has opted into modules and is inside the GOPATH so I'd expect for the retry to happen but removing the /v2 suffix.
After all of that I'm not denying that the root issue is the one you linked, but it could be that it's a different one.

The issue is github.com/triztian/gomobile-v2target imports github.com/triztian/gomobile-v2mod, but github.com/triztian/gomobile-v2mod then imports github.com/triztian/gomobile-v2mod/v2/subpkg
In gopath mode this will not work since there is no such package.

cc @eliasnaur @hyangah for comments about retrying without /v2

I have a bit more info, after playing a bit with the source, I narrowed the error to this line:

https://github.com/golang/mobile/blob/master/cmd/gobind/main.go#L145

Basically a syntax-check is performed before any build is attempted, the syntax check happens without the minimal version compat that the modloads and several other modyyy packages provide when the go build command is ran on source files.
I think it might be possible to provide an Importer (interface) that is able to handle the module that opted in packages through an in-memory rewrite of the source for the package that has the exports
for the framework:
Example Excerpt: https://golang.org/pkg/go/types/#example_Info
fset := token.NewFileSet()
f, err := parser.ParseFile(fset, "fib.go", input, 0)
if err != nil {
	log.Fatal(err)
}

// Type-check the package.
// We create an empty map for each kind of input
// we're interested in, and Check populates them.
info := types.Info{
	Types: make(map[ast.Expr]types.TypeAndValue),
	Defs:  make(map[*ast.Ident]types.Object),
	Uses:  make(map[*ast.Ident]types.Object),
}

var conf types.Config
pkg, err := conf.Check("fib", fset, []*ast.File{f}, &info)
if err != nil {
	log.Fatal(err)
}

I've provided a working patch here, well at least for the 2 sample projects:

https://github.com/golang/mobile/compare/master...triztian:issue/33272

Basically it provides a custom importer that performs an in-memory rewrite of import paths of packages that have major versions in their paths i.e: "example.org/lib/v6" into "example.org/lib"
This does not add support for modules but rather it allows you to import packages that have opted into go modules and that have a vN path component without having to edit their sources.
To test the patch, just clone the branch from here https://github.com/triztian/mobile/tree/issue/33272 and reinstall the gobind command:
go install ./cmd/gobind

You may also need to re-init gomobile but I'm not entirely sure if thats necessary:
gomobile init

Setup
Clone the sample projects into your gopath (they've been updated):

https://github.com/triztian/gomobile-v2target
https://github.com/triztian/gomobile-v2mod

Then try and build the gomobile-v2target package:
The relevant proofs are that v2target imports v2mod and it has a v2 part in it's import:
// github.com/triztian/gomobile-v2target/export.go:8
v2mod "github.com/triztian/gomobile-v2mod/v2"

gomobile-v2target also imports one of it's subpackage with a v2 path component:
// github.com/triztian/gomobile-v2mod/math.go:4
import "github.com/triztian/gomobile-v2mod/v2/subpkg"

To build gomobile-v2target execute the following:
make clean run

You will need a swift compiler installed:
λ gomobile-v2target ~> swift --version
Apple Swift version 5.0.1 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin18.6.0

Successful Output:
gomobile bind -target ios -work -a -v -tags "" .
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-132492778/go-build699643396
runtime/cgo
golang.org/x/mobile/internal/mobileinit
gobind
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-132492778/go-build206905632
runtime/cgo
golang.org/x/mobile/internal/mobileinit
gobind
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-132492778/go-build468413038
runtime/cgo
golang.org/x/mobile/internal/mobileinit
gobind
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-132492778/go-build230851473
runtime/cgo
golang.org/x/mobile/internal/mobileinit
gobind
write Mobile.framework/Versions/A/Headers/Mobile.objc.h
write Mobile.framework/Versions/A/Headers/Universe.objc.h
write Mobile.framework/Versions/A/Headers/ref.h
write Mobile.framework/Versions/A/Headers/Mobile.h
write Mobile.framework/Resources/Info.plist
write Mobile.framework/Versions/A/Modules/module.modulemap
WORK=/var/folders/7s/cnfg6ksj44gb3br_lc4v3gn40000gp/T/gomobile-work-132492778
swiftc -F. -framework Mobile -framework Foundation ./main.swift
-------------
./main
Testing
Result 21 + 21 =  42
Sum call: 18

There is a minimal swift file to test the framework and functionality:
import Mobile
import Foundation

print("Testing")
MobileMain()
print("Sum call: \(MobileSum(10, 8))")

I've not tested how it behaves when a vendor directory exists in the gomobile-v2target, happy to receive feedback and reports of issues, the test packages are quite simple so testing with more real-world complex packages would be beneficial.

We are discussing to support Go module at https://github.com/golang/go/issues/27234

Is this still an issue?

Hi @hajimehoshi I can try and repro with the latest releases, I'll give it a go most likely this weekend. Dealing with iOS 14. Thank you for checking in!.

Assume this is not an issue anymore because there are modules that use gomobile but also in v2 mode, eg. fyne-io/fyne. Close as outdated.


Source Code

Describe your problem here.
Source Code compilation. Everyone says it's easy, but I've found all of the tutorials are outdated and I have no idea how.
Version is 0.5.2
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

?, wdym with that, there is an explanation about installing the source in README.md

?, wdym with that, there is an explanation about installing the source in README.md

And then you select the folder of the code, select the open with git bash, and then write if you are in windows write this: lime test windows, if you are in Mac type: lime test mac, and if you are with linux write: lime test linux


[AutoRelease] t2-nginx-2022-09-20-16466(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3202
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=4948

issue link:https://github.com/Azure/sdk-release-request/issues/3202


Add export for Types

Hi, thanks for the PR. I've exported the types under paseto-ts/lib/types along with everything else in the lib directory in 1.4.4.


Make tum live pwa compliant

Make TUM Live PWA compliant:

Add WebApp-Manifest
Add ServiceWorker (w. Network-First-Cache)

https://user-images.githubusercontent.com/17506411/167293714-0981fa36-943f-484f-8c28-7ee84378d5b1.mov

Let's merge this and see how devices behave :D Looks great :)


How do I fix this?

Tried 3 | Line : 378 | Message: no such element: Unable to locate element: {"method":"css selector","selector":"button.ytp-large-play-button.ytp-button"}
(Session info: headless chrome=90.0.4430.85)
and
Tried 15 | Line : 347 | Message: unknown error: net::ERR_TUNNEL_CONNECTION_FAILED
(Session info: headless chrome=90.0.4430.85)

what proxy you're using? you are not using the latest script or release.
Well, the first problem may occur if the script fails to find your video from the search list or can't find the play button.
The second problem is related to your proxy connection. Check you configured proxy correctly or for free proxies that are dead proxy.
you don't need to worry about that as long as your script finds videos and plays them.

It finds everything perfectly, proxies are also suitable, and the script version is the latest


Android "elevation" style property doesn't work on map markers

The React Native Android "elevation" style property (to raise a view and cast a native-looking shadow) does nothing for map markers -- is this because of the way markers are rendered as bitmaps on Android? Any way to work around it?

+1 on this.

+1 on this too. It would be nice to implement this functionality, and it doesn't require a massive amount of changes (I think).

+1 on this

+1

@christopherdro please reopen this issue because the issue is not resolved on the latest stable version (currently 0.23.0). Opening an new issue would only make this problem less visible for people searching for it on Google, etc.

BTW: is it right next thing
Do Markers behave like they have overflow = hidden?
as to add a shadow in iOS I need to add additional View to wrap View with shadow

This is really needed things and its been 3 years this issue is in open state.
Is there any workaround?

+1

+1

looks like markers callouts have overflow hidden?

Please remove overflow hidden from markers.

+1

You can use png with transparency as marker icon

You can use png with transparency as marker icon
PNG is not a solution. This leads to performance problems.

+1

did people forget it?

+1

+1

+1, open issue again please

Please open the issue again, as marker shadows are a nice feature to have.

I will lock this issue because there seem to be a big misunderstanding here as to what this library can and can't do.
The android google maps library doesn't use the native view system so introducing things that apply to the normal views like shadow is not possible out of the box. a feature request about this in google maps android sdk makes more sense than demanding it from this library.


Fix bug where old objects could get stuck and never updated.

Also modernize typescript version and some simplifications.

@wally-the-cartographer merge

Merge requested by authorized user SirVer. Merge queue now has a length of 1.


Use the indent as column_number

This way the error message would show up where the text starts.

I started also a conversation if Lint.jl could provide the error's columns https://github.com/tonyhffong/Lint.jl/issues/177, after all it could be more interesting to implement in Julia.

Thanks for reporting these issues and helping to make Lint better. This is, as far as I can tell, a very big undertaking. It will need Lint to be able to understand source, not just ASTs. It's a desirable feature but there is some infrastructure that needs to be done first.
We first need a concrete syntax tree representation of syntax that does not throw away semantically meaningless but syntactically meaningful things like whitespace.

Using the whole line width (80) works pretty ok. This is easy to add later, if Lint.jl or some other implementation will provide the column information.


Data migration different class name

This fixes an issue when generating both schema and data migrations at the same time, where you don't use the default up and down class methods in the migration.
For example:
You run
rails g data_migration create_headaches
Which generates two files:
db/migrate/20150924152519_create_headaches.rb
db/data/20150924152519_create_headaches.rb

Where the contents of both look like this
class CreateHeadaches < ActiveRecord::Migration
  def self.up
  end

  def self.down
  end
end

The problem occurs when you modify the schema migration to use the change method instead of up and down. Then what happens is, the data migration ends up re-opening the CreateHeadaches class and adding up and down methods to it.  Then you've got a migration class with up, down and change methods and things end up getting run twice.
The solution is to not use the same class for both schema and data migrations. Instead, we modify data_migrate's generator to append Data to the data migration's class name.

Are there any tests?

@jcmuller no tests in this repo unfortunately.

If there are no objections, I'm going to merge this so I can get the greenhouse Gemfile changes reviewed/merged.

raise Objection if false


Feel free to ignore it.

can you add some description as for what benefits does it bring ?

can you add some description as for what benefits does it bring ? maybe like a release file ?

Basically we strayed far from the original fork, so this updates our version with all of their changes.
It includes adherence to all new REST API changes since 2020, as well as a few specific features we wanted.
The specific feature we wanted is the CachingHttpClient which allows us to cache responses and attempt to save on rate limit.

Did you try to use a built version of this with LIM/GH service?

Did you try to use a built version of this with LIM/GH service?

Yes, I have tested this with my local lim environment.
I haven't tried testing the github service.


Error with running make bootstrap: google-chrome-unstable : Depends: libgbm1 (>= 17.1.0~rc2) but 13.0.6-1+b2 is to be installed

runinng
make bootstrap  or sudo make bootstrap
...
Hit:1 http://security.debian.org/debian-security stretch/updates InRelease
Get:2 http://dl.google.com/linux/chrome/deb stable InRelease [1811 B]
Get:3 http://dl.google.com/linux/chrome/deb stable/main amd64 Packages [1112 B]
Ign:4 http://deb.debian.org/debian stretch InRelease
Hit:5 http://deb.debian.org/debian stretch-updates InRelease
Hit:6 http://deb.debian.org/debian stretch Release
Fetched 2923 B in 0s (3261 B/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
google-chrome-unstable : Depends: libgbm1 (>= 17.1.0~rc2) but 13.0.6-1+b2 is to be installed
E: Unable to correct problems, you have held broken packages.

I changed in the Dockerfile
FROM node:1**4**-slim as core
instead of:
FROM node:10-slim as core
and now it works!!!


getPeerCertificate() is an empty object when using dev server

I've got an Express server that's set up to use PKI authentication for the users (corp app where all users have certs in their browser). For my create-react-app setup, I've updated package.json to proxy requests to the Express server, but it looks like the dev server doesn't pass along the user's cert. When my authentication code runs on the Express server, and the app is accessed through the webpack dev server, an empty object is returned: (though it works fine when accessing the Express server API directly through the browser)
var peerCertificate = req.connection.getPeerCertificate()

I don't see a way to to configure things so that this information gets passed along too. Am I missing something or doing something incorrectly? Or is this something that could be added in?

Could you find out for me if this also happens when using webpack-dev-server directly? I'm the maintainer of webpack-dev-server and could fix this if it is an issue in our package.

I created a light weight app using just webpack-dev-server and got the same issue. I can't cut and paste, so I just re-typed it below. The app it's trying to proxy to is running on port 8080 (on the same box as this app) and is running using expressjs.
var webpack = require('webpack');
var WebpackDevServer = require('webpack-dev-server');
var fs = require('fs');
var splitca = require('split-ca');

var compiler = webpack({
    context: __dirname,
    entry: './entry',
    output: {
        path: __dirname + '/dist',
        filename: 'bundle.js'
    }
});

var devServer = new WebpackDevServer(compiler, {
    proxy: {
        '*': {
            target: 'https:// ... :8080',
            secure: false
        }
    },
    https: {
        key: fs.readFileSync('file_name_here'),
        cert: fs.readFileSync('file_name_here'),
        ca: splitca('file_name_here')
    }
});

devServer.listen(3000);

The expressjs app gets the request, but peerCertificate object is empty.

Okay, AFAIK that's not really a bug (at least not in WDS). webpack-dev-server only passes through the certificates and uses the standard https.createServer from Node.js.

Would it be possible to additionally make the "headers" option in the WebDevServer configuration a function that took in the request object and returned an object for the headers? If so, I could set a header with the needed information from the user's cert and that could get passed along to the expressjs server.
The expressjs server reading a header for the user cert information wouldn't be secure for production, but it should be fine for development, and this technique seems to be popular for getting around this kind of situation [1, 2, 3].
[1] http://www.zeitoun.net/articles/client-certificate-x509-authentication-behind-reverse-proxy/start
[2] http://serverfault.com/questions/622855/nginx-proxy-to-back-end-with-ssl-client-certificate-authentication
[3] http://webcache.googleusercontent.com/search?q=cache:IaZk_nEeCyIJ:www.gossamer-threads.com/lists/apache/users/350827+&cd=7&hl=en&ct=clnk&gl=us

Sure, I'll accept a PR for that ;).

I’m going to close since it doesn’t appear like there’s anything actionable here for this repo.
Let me know if I’m wrong!


Update to latest versions of dependencies

We were using some quite old versions of the lambda_http,
lambda_runtime, etc. crates.

Closing in favor of #160


Added TabBarIOS and TabBarItemIOS

Added missing components for my own use and i thought they may come in handy for somebody else.

Really cool! We just need to fix the .reis and we are good to got :)

I've added types, one thing that i found out is that it might be a good idea to extract imageSource from the ImageRe module to have other modules use it like this one, let me know your opinion about it. Thanks!

This looks good now :)
I think it would make sense to extract imageSource but we should do this separate since this occurs on other places as well. I will open a separate issue to discuss the best approach.


cmd/encrypt: enhance success message

the message to provide the password on the command line is a
bit misleading, as it is also possible to provide the password
via stdin when invoking 'start'
providing it via stdin has the advantage not to be visible later
in the shell's history
Signed-off-by: Maximilian Meister<EMAIL_ADDRESS>

Coverage remained the same at 26.114% when pulling 2937f063b53c3e747f4cf1ba46d025c22e8c7248 on MaximilianMeister:encrypt-message into 1e7e9f39931b1f9e11c092bd6e06861d8bc6371d on OpenBazaar:master.


Fix markdown

Fixes #49860

@Potapy4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified editorial correction. Thank you!


Add routing constraints to restrict icon formats

Previously the icon redirect routes would accept any format and be recognised. Since this path is passed directly into asset_path to find the destination digest path for the redirect it seems advisable to only allow the expected types.
The paths could've been set to format: false but this leaves them being recognised as the default type (text/html) which could have unintended side effects.

I'm not normally looking at GOV.UK repos on a Saturday but I was looking at what you're doing for /favicon.ico as I can't get CloudFront to compress an image/x-icon correctly. Whilst looking at the implementation I noticed an issue with the defined routes and thought best to contribute back any changes I'd be making.
I think the Fastly config prevents the additional file extension but if you request /favicon.ico.txt locally you end up with a Sprockets::Rails::Helper::AssetNotFound exception. I'm pretty sure it's not exploitable but I thought it best that a user controlled path doesn't get sent into that code since we've had path traversal exploits in Sprockets before: https://blog.heroku.com/rails-asset-pipeline-vulnerability - though that one needed assets.compile = true in production.


[Bug] [txn] can not do txn-produce after several pulsar-perf op and cluster restart

Search before asking

[X] I searched in the issues and found nothing similar.

Version

os: Linux  4.19.160-0419160-generic x86_64
pulsar-server: 2.9.3
pulsar-client: 2.9.2
3 nodes cluster
use pulsar-perf to simulate produce/consume, add config "-txn -nmt 50" to enable txn-produce or txn-consume
server conf: transactionCoordinatorEnabled=true, acknowledgmentAtBatchIndexLevelEnabled=false, E-Qw-Qa is 3-3-2

Minimal reproduce step

use pulsar-perf to do several perf-test, both txn and non-txn produce/consume work well
during the test, cluster have several time restart broker and bookie
txn-produce failed, but non-txn-produce succeed

What did you expect to see?
both txn-produce and non-txn-produce succeed
What did you see instead?
Broker log do not have any error log, and as the following log shown, it look like the operation timeout is over 30s and client close the producer. But although I add operation-timeout client-config in pulsar-perf and change it to a large value 8min, the problem also occur.
And if I remove "-txn -nmt 50" in pulsar-perf, change it to non-txn produce, it can succeed. The problem seems refer to the txn module.
Besides, although I restart broker and bookie again, txn-produce still fail. I don't know how to solve it without cleanup the whole cluster.
client error log:
15:58:33.534 [pulsar-client-io-3-1] ERROR org.apache.pulsar.client.impl.ProducerImpl - [persistent://test/test/test_txn_perf_tmp-partition-2] [null] Failed to create producer: request timeout {'durationMs': '30000'}
15:58:33.534 [pulsar-client-io-1-1] ERROR org.apache.pulsar.client.impl.ProducerImpl - [persistent://test/test/test_txn_perf_tmp-partition-2] [null] Failed to create producer: request timeout {'durationMs': '30000'}
15:58:33.534 [pulsar-client-io-2-1] ERROR org.apache.pulsar.client.impl.PartitionedProducerImpl - [persistent://test/test/test_txn_perf_tmp] Could not create partitioned producer.
15:58:33.534 [pulsar-client-io-3-1] ERROR org.apache.pulsar.client.impl.PartitionedProducerImpl - [persistent://test/test/test_txn_perf_tmp] Could not create partitioned producer.
15:58:33.534 [pulsar-client-io-1-1] ERROR org.apache.pulsar.client.impl.PartitionedProducerImpl - [persistent://test/test/test_txn_perf_tmp] Could not create partitioned producer.
15:58:33.538 [pulsar-perf-producer-exec-1-3] ERROR perf.PerformanceProducer - Got error
java.util.concurrent.ExecutionException: org.apache.pulsar.client.api.PulsarClientException$TimeoutException: request timeout {'durationMs': '30000'}
        at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357) ~[?:1.8.0_121]
        at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1895) ~[?:1.8.0_121]
        at perf.PerformanceProducer.runProducer(PerformanceProducer.java:540)

server log:
17:15:32.715 [pulsar-io-27-15] INFO  org.apache.pulsar.broker.service.ServerCnx - New connection from
17:15:32.715 [pulsar-io-27-16] INFO  org.apache.pulsar.broker.service.ServerCnx - New connection from
17:15:32.715 [pulsar-io-27-17] INFO  org.apache.pulsar.broker.service.ServerCnx - New connection from
17:15:32.721 [pulsar-io-27-16] INFO  org.apache.pulsar.broker.service.ServerCnx -[persistent://test/test/test_txn_perf_tmp-partition-0] Creating producer. producerId=0
17:15:32.721 [pulsar-io-27-15] INFO  org.apache.pulsar.broker.service.ServerCnx - [persistent://test/test/test_txn_perf_tmp-partition-0] Creating producer. producerId=0
17:15:32.721 [pulsar-io-27-17] INFO  org.apache.pulsar.broker.service.ServerCnx - [persistent://test/test/test_txn_perf_tmp-partition-0] Creating producer. producerId=0
17:15:32.722 [ForkJoinPool.commonPool-worker-43] INFO  org.apache.pulsar.broker.service.ServerCnx -  persistent://test/test/test_txn_perf_tmp-partition-0 configured with schema false
17:15:32.722 [ForkJoinPool.commonPool-worker-50] INFO  org.apache.pulsar.broker.service.ServerCnx -  persistent://test/test/test_txn_perf_tmp-partition-0 configured with schema false
17:15:32.722 [ForkJoinPool.commonPool-worker-50] INFO  org.apache.pulsar.broker.service.ServerCnx -  persistent://test/test/test_txn_perf_tmp-partition-0 configured with schema false
17:16:32.716 [pulsar-io-27-16] INFO  org.apache.pulsar.broker.service.ServerCnx - Closed producer before its creation was completed. producerId=0
17:16:32.726 [pulsar-io-27-17] INFO  org.apache.pulsar.broker.service.ServerCnx - Closed producer before its creation was completed. producerId=1
17:16:32.726 [pulsar-io-27-15] INFO  org.apache.pulsar.broker.service.ServerCnx - Closed producer before its creation was completed. producerId=2
17:16:32.746 [pulsar-io-27-15] INFO  org.apache.pulsar.broker.service.ServerCnx - Closed connection from
17:16:32.746 [pulsar-io-27-16] INFO  org.apache.pulsar.broker.service.ServerCnx - Closed connection from
17:16:32.746 [pulsar-io-27-17] INFO  org.apache.pulsar.broker.service.ServerCnx - Closed connection from

Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@congbobo184 @liangyepianzhou Could you please help check this issue?

I have added many log in the code, and found that it is blocked in topic.checkIfTransactionBufferRecoverCompletely in handleProducer().
Then I found that the object transactionBufferFuture in TopicTransactionBuffer was not completed, because it did not finish TopicTransactionBufferRecover#run().
The problem was blocked in the reader.readNext() method. I found that reader can read the 1,2,3 message successfully, but when reader read the 4th message, it would block and thus over the 30s timeout.

https://github.com/apache/pulsar/pull/17051 This Pr will fix the reader.hasMoreEvents has been blocked problem. please re-build the branch-2.9 or wait for tag 2.9.4

hi @TakaHiR07 , sorry it took a long time to reply. #17051 This Pr will fix the reader.hasMoreEvents has been blocked problem. please re-build the branch-2.9 or wait for tag 2.9.4. If still have the same problem, please ping me.

@congbobo184 I re-build the branch-2.9, restart my cluster. This time I can successfully txn_produce on some topic, but when I try txn_produce on a 10-partition topic, client still has the same timeout exception.
The client log is the same as before. For the server log, this time it seems some partition has finish the transaction recover and can successfully create producer. But some other partition do not finish.
Besides, I observe this log between I restart cluster and create txn_producer, which is not occur before. I don't know whether it is the possible reason.
18:27:19.260 [transaction_coordinator_TransactionCoordinatorID(id=6)thread_factory-70-1] ERROR org.apache.pulsar.broker.TransactionMetadataStoreService - End transaction fail! TxnId : (6,2309004), TxnAction : 1 org.apache.pulsar.transaction.coordinator.exceptions.CoordinatorException$CoordinatorNotFoundException: Transaction coordinator with id 6 not found! 18:27:19.260 [transaction-coordinator-thread-factory-66-1] INFO  org.apache.pulsar.broker.TransactionMetadataStoreService - Added new transaction meta store TransactionCoordinatorID(id=6)

@TakaHiR07 Hi
what version of the client you are using?
18:27:19.260 [transaction_coordinator_TransactionCoordinatorID(id=6)thread_factory-70-1] ERROR org.apache.pulsar.broker.TransactionMetadataStoreService - End transaction fail! TxnId : (6,2309004), TxnAction : 1 org.apache.pulsar.transaction.coordinator.exceptions.CoordinatorException$CoordinatorNotFoundException: Transaction coordinator with id 6 not found! 18:27:19.260 [transaction-coordinator-thread-factory-66-1] INFO org.apache.pulsar.broker.TransactionMetadataStoreService - Added new transaction meta store TransactionCoordinatorID(id=6)

this log is normal and does not affect behavior, we will change it to warn or debug level.
for creating a producer timeout problem, can you conveniently provide the jstack, dump and log file?
https://github.com/apache/pulsar/issues/17040#issuecomment-1214544142 Is the thread still stuck here?

@TakaHiR07 Hi what version of the client you are using?
18:27:19.260 [transaction_coordinator_TransactionCoordinatorID(id=6)thread_factory-70-1] ERROR org.apache.pulsar.broker.TransactionMetadataStoreService - End transaction fail! TxnId : (6,2309004), TxnAction : 1 org.apache.pulsar.transaction.coordinator.exceptions.CoordinatorException$CoordinatorNotFoundException: Transaction coordinator with id 6 not found! 18:27:19.260 [transaction-coordinator-thread-factory-66-1] INFO org.apache.pulsar.broker.TransactionMetadataStoreService - Added new transaction meta store TransactionCoordinatorID(id=6)

this log is normal and does not affect behavior, we will change it to warn or debug level.
for creating a producer timeout problem, can you conveniently provide the jstack, dump and log file? #17040 (comment) Is the thread still stuck here?

Oh, still 2.9.2, I notice the pr-17040 fix on client side, not on broker side. I would try to update the client version and test again. Thanks a lot.


Social engagement and social selling discontinued?

The 16 January 2020 Microsoft Social Engagement was discontinued, and I can't find the Social selling app in AppSource. you should update the documentation with a notification about its status.

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

ID: c8cb33b1-f624-71dd-5cd6-66b2ad9e3562
Version Independent ID: 278c97e4-60af-2a85-0b38-1dabc92de60d
Content: Install the Social Selling Assistant - Power Platform Admin center
Content Source: power-platform/admin/install-social-selling-assistant.md
Service: power-platform
GitHub Login: @m-hartmann
Microsoft Alias: mhart

@cxr-dev
Thanks for the feedback. We've added a link to an FAQ for more information.


Installing - Wrong Version of Node

I have a problem as i install everything for the first time, this is it:
npm ERR! Linux 4.18.0-16-generic
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "run" "build-assets"
npm ERR! node v8.10.0
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build-assets: `webpack -p`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build-assets script 'webpack -p'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the saleor package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     webpack -p
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs saleor
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls saleor
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/sugar/sale/npm-debug.log

this is the log
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node', '/usr/bin/npm', 'run', 'build-assets' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild-assets', 'build-assets', 'postbuild-assets' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prebuild-assets, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/share/npm/bin/node-gyp-bin:/home/sugar/sale/node_modules/.bin:/home/sugar/saleor/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/
usr/games:/usr/local/games:/snap/bin
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/sugar/sale
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'webpack -p' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 2  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec build-assets script
14 verbose stack Error<EMAIL_ADDRESS>build-assets: `webpack -p`
14 verbose stack Exit status 2
14 verbose stack     at EventEmitter.<anonymous> (/usr/share/npm/lib/utils/lifecycle.js:232:16)
14 verbose stack     at emitTwo (events.js:126:13)
14 verbose stack     at EventEmitter.emit (events.js:214:7)
14 verbose stack     at ChildProcess.<anonymous> (/usr/share/npm/lib/utils/spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:126:13)
14 verbose stack     at ChildProcess.emit (events.js:214:7)
14 verbose stack     at maybeClose (internal/child_process.js:925:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd /home/sugar/sale
17 error Linux 4.18.0-16-generic
18 error argv "/usr/bin/node" "/usr/bin/npm" "run" "build-assets"
19 error node v8.10.0
20 error npm  v3.5.2
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>build-assets: `webpack -p`
22 error Exit status 2
23 error Failed at the<EMAIL_ADDRESS>build-assets script 'webpack -p'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the saleor package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     webpack -p
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs saleor
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls saleor
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

The versions
(saleor) sugar@charco:~/sale$ node -v v11.12.0 (saleor) sugar@charco:~/sale$ npm -v 3.5.2

Something is wrong with your nodejs and npm versions.
In the first log it says:
npm ERR! npm  v3.5.2```
Then in the last
```(saleor) sugar@charco:~/sale$ node -v v11.12.0 (saleor) sugar@charco:~/sale$ npm -v 3.5.2```

You should use node v11.x and npm 6.x

Thanks!! that had been very useful! now i'm at the right versions, but i get this in the error logo
0 info it worked if it ends with ok 1 verbose cli [ '/usr/local/bin/node', '/usr/bin/npm', 'run', 'build-assets' ] 2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild-assets', 'build-assets', 'postbuild-assets' ] 5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true 8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/node_modules/npm-lifecy cle/node-gyp-bin:/home/sugar/sale/node_modules/.bin:/home/sugar/saleor/bin:/usr/local/sbin:/usr/local/ bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin 9 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/sugar/sale 10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'webpack -p' ] 11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 2  signal: null 12 info lifecycle<EMAIL_ADDRESS>Failed to exec build-assets script 13 verbose stack Error<EMAIL_ADDRESS>build-assets: webpack -p13 verbose stack Exit status 2 13 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle /index.js:301:16) 13 verbose stack     at EventEmitter.emit (events.js:197:13) 13 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle /lib/spawn.js:55:14) 13 verbose stack     at ChildProcess.emit (events.js:197:13) 13 verbose stack     at maybeClose (internal/child_process.js:988:16) 13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:265:5) 14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /home/sugar/sale 16 verbose Linux 4.18.0-16-generic 17 verbose argv "/usr/local/bin/node" "/usr/bin/npm" "run" "build-assets" 18 verbose node v11.12.0 19 verbose npm  v6.7.0 20 error code ELIFECYCLE 21 error errno 2 22 error<EMAIL_ADDRESS>build-assets:webpack -p22 error Exit status 2 23 error Failed at the<EMAIL_ADDRESS>build-assets script. 23 error This is probably not a problem with npm. There is likely additional logging output above. 24 verbose exit [ 2, true ]


turn off TF warnings

Description

Related Issues

fixes https://github.com/microsoft/recommenders/issues/1256
Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[ ] This PR is being made to staging branch and not to master branch.

Wonder if we want to clean up output of the notebook before checking in?  Asking as a general question.

Wonder if we want to clean up output of the notebook before checking in?  Asking as a general question.

Wonder if we want to clean up output of the notebook before checking in? Asking as a general question.

Not sure if you refer to the warning outputs of Tensorflow, in that case, they should be cleaned with the latest changes. If you refer to completely erase the output of all the notebooks, we had a discussion about that matter some months ago, we decided to leave the outputs because it helped the user understand each notebook and algorithm. We can have the discussion again if you consider it

Wonder if we want to clean up output of the notebook before checking in? Asking as a general question.

Not sure if you refer to the warning outputs of Tensorflow, in that case, they should be cleaned with the latest changes. If you refer to completely erase the output of all the notebooks, we had a discussion about that matter some months ago, we decided to leave the outputs because it helped the user understand each notebook and algorithm. We can have the discussion again if you consider it


UV incorrectly scaled for some but not all parts

Hello! I've encountered a bug of some sort (or, I may just be dumb), wherein a few of the textures for my model's parts are scaled incorrectly??? They are just a single pixel larger than they should be, while the rest of the parts are just fine.
I did notice that it seems to have only affected parts that are paired inside a bone, with one component scaled normally, and another scaled just 1 pixel smaller (or, .1 inflation in blockbench). It seems that perhaps this has gotten the texture editor confused in some way?
Any help or even just clarification would be appreciated.
I am using the v0.6.17_pre1 version of the cpm_plugin for blockbench to do my editing
the bugged texture:

normal behavior in blockbench:

I do appear to have found a way to circumvent it, by applying the UV for each face individually, however it is a tedious stopgap solution that does not address whatever the root issue for this problem might be.

Ok, looks like it may just be a visual glitch. when viewed in-game, the glitch does not persist and texture behave normally


Contract request does not find contract  because of invalid consumer

A contract agreement request is requested from consumer. The provider method ContractUtils.removeContractsWithInvalidConsumer(contracts, issuer) removes all contracts because the issuer does not fit to the contract consumer. The method works correct, but the process seems to be incomplete.
I created provider data like in the documentation of the project. For the contract I used this data:
{
"consumer": "http://testconsumer",
"provider": "http://testprovider",
"title": "test contract 1",
"start": "2021-04-19T10:20:30.937Z",
"end": "2021-10-19T13:20:30.937Z"
}
Then I requested a contract agreement from the consumer. While agreement processing, the provider clears all contracts because
the contract consumer has the value http://testconsumer and the issuerConnector is
https://w3id.org/idsa/autogen/baseConnector/7b934432-a85e-41c5-9f65-669219dde4ea
public static List<io.dataspaceconnector.model.Contract> removeContractsWithInvalidConsumer(
final List<io.dataspaceconnector.model.Contract> contracts,
final URI issuerConnector) {
Utils.requireNonNull(contracts, ErrorMessages.LIST_NULL);
Utils.requireNonNull(issuerConnector, ErrorMessages.URI_NULL);
    return contracts.parallelStream()
            .filter(x -> x.**getConsumer**().equals(issuerConnector) || x.**getConsumer**()
                    .toString()
                    .isBlank())
            .collect(Collectors.toList());
}

Connector Version: 5.1.2
What is going wrong, should I use another value for the consumer into the contract data?  Why is the issuer connector compared with the consumer address? The issuerConnector value seems to be auto generated, is this a correct behaviour?

There are two options when creating a contract offer: Either you don't set a consumer, then any consumer connector can negotiate a contract agreement for a resource. Or you set a consumer, then this id will be compared to the one of the issuer connector, as you already observed. We implemented this as a basis for hiding contract offers for one specific company from other requesting companies. The connector id in the default config.json is auto generated, correct. As described here this should be replaced by the correct connector address. This will be then referenced in the ids message and, on the provider side, compared to the prepared consumer id (or not if this was left empty).

Hi Julia, thank you for your explanation. I now understand the functionality for this part of contract agreement negotiation.


TypedFactory works with multi-Dispose calls and Release after Dispose

Fixes #458

It does not seem that the fail has anything to do with the changes in this PR...


fix: sasjs add cred does not use defaultTarget property fixed

Issue
Closes #763
Intent
When user does not provide target it should use default target
Implementation
Instead of throwing error when targetName contains falsy values just return that value.
Checks

[x] Code is formatted correctly (npm run lint:fix).

[ ] Any new functionality has been unit tested.
[ ] All unit tests are passing (npm test).
MOcked:

[ ] All CI checks are green.
[ ] JSDoc comments have been added or updated.
[ ] Reviewer is assigned.

Tests all passing now


Add Usama's slide deck

Signed-off-by: Thomas Fossati<EMAIL_ADDRESS>

@KeithMoyer @gkostal @shnwc : it seems we've lost track of this.  Can we merge it?  (modulo conflict resolution)


cann't find executable file in $PATH. cannot init workspace

What happened?
generate alicloud-jet-provider using terrajet and followed the guide.
build provider binary and runs with:
./provider --terraform-provider-source=aliyun/alicloud  --terraform-provider-version=1.155.0  --terraform-version=terraform-1.1.5 --debug

How can we reproduce it?
deploy crossplane version 1.6.2 in k8s cluster; run provider binary as metioned above.  apply crds and securitygroups template.

the terraform-1.1.5 binary is in my $PATH， I can execute from a terminal

fixed

@bretagne-peiqi
Hi,
I am having exactly the same error, how did you resolve it please?

check the network. by default, it will download it from terraform.io


Special exclusions for well known versions of well known libraries

Currently refreshVersions proposes old versions of some software products as upgrades when they are actually downgrades.
The classic example is commons-io: latest version is 2.8.0, yet refreshVersions proposes 20030203.000550 as an upgrade.
I understand the logic behind that: who published the library used inconsistent versioning and refreshVersions is actually doing nothing wrong. This however plays badly with further automation, see e.g. https://github.com/Protelis/Protelis/pull/411.
I was wondering if it would make sense to create a list of artifacts/versions not to consider into updates.
I realize this is something hard to maintain, but I expect it not to be particularly big (Apache commons stuff, possibly Scala, not sure anything else).
What do you think?

Hi Dany, it will take more time for us to get there, but we've laid the foundation to allow future metadata comments which will be able to control/filter the versions shown as available by refreshVersions later down the road.
Thanks for letting us know about that unfortunate real world example, it'll give food for thought when we design the details of these features.

Hello Dany, this will be doable when https://github.com/jmfayard/refreshVersions/pull/347 is merged and released

Very nice! Thanks!


Make updating kernel and initramfs more reliable

After extracting the btrfs image, we copy the Linux kernel and initramfs images to the boot partition.
If the user restarts the system in the middle of this copy or the copy fails, this can result in an unbootable system.
A good strategy might be to copy the kernel and initramfs files to a subdirectory named after the version string of the system.
Not confirmed but this issue may result in the following error at boot.

This was fixed in GamerOS 24.


Security backports should be marked as such

Describe the bug
If a security fix gets backported by our new GH action, I'd expect the bot to add the 1. severity: security-label to the backport PR automatically.
Currently this has to be done manually (see for instance https://github.com/NixOS/nixpkgs/pull/127131), but it would IMHO be a nice addition to let the backport-action do this.
Notify maintainers
@domenkozar

Still relevant.

Depends on https://github.com/zeebe-io/backport-action/issues/87

Fixed in https://github.com/zeebe-io/backport-action/pull/303 but I don't want to deal with CLA crap :(

@lheckemann since it's MIT licensed, can't we just use your patch in a fork in the nixos org? %)

This seems to be happening nowadays: https://github.com/NixOS/nixpkgs/pull/300527, https://github.com/NixOS/nixpkgs/pull/300115, https://github.com/NixOS/nixpkgs/pull/299815, https://github.com/NixOS/nixpkgs/pull/299541


[usockets,uwebsockets] Update uwebsockets to 20.71.0, install and test CMake config, Cleanup

Install unofficial CMake config. Resolves https://github.com/microsoft/vcpkg/issues/42239.
uwebsockets: Update.

Move zlib to optional feature.
Wire ssl support.

usockets: Cleanup. Align CMakeLists.txt with upstream Makefile.

Support only the default event loop for the target platform as in
https://github.com/uNetworking/uSockets/blob/8606de6414a102c55bef8e8ef3391932d7e8df6a/src/libusockets.h#L339-L348
i.e. libuv is used on Windows only.
Remove the network feature with its unique winsock2 dependency.
usockets was the last port modeling the winsock2 dependency. After merge of this PR, winsock2 should be delisted. (Another PR.)

Ping @FrankXie05.