[MEDIUM] Company Card expense shows as Cash in NewDot report.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.41-12
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail: n/a
Email or phone of affected tester (no customers): Customer
Logs: https://stackoverflow.com/c/expensify/questions/4856 n/a
Expensify/Expensify Issue URL: n/a
Issue reported by: @kadiealexander / @SofiedeVreese
Slack conversation: https://expensify.slack.com/archives/C05DWUDHVK7/p1707958713660189
Action Performed:
Use a company card/Expensify card to make an expense on a Free Plan workspace
Expected Result:
The NewDot report for that expense should show that the expense is a company card expense.
Actual Result:
The expense shows as a Cash expense in the NewDot report.
Workaround:
None at this stage.
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub

Just asking some questions in the Slack thread about where this needs to go.

Tagging @kevinksullivan @grgia based on the Slack thread :)

@lschurr do you have the transaction ID handy?

User email<EMAIL_ADDRESS>Transaction ID:<PHONE_NUMBER>078725321

Just assigning to you for now @grgia - feel free to do something different with it :)

I'm going OOO till the end of the week, going to unassign incase someone else wants to take this over while I'm out. Will reassign when Im back if not!

If someone takes this over, it's most likely an issue with the reimbursable flag not being sent in OnyxUpdate for some reason

Are you able to grab this back this week @grgia?

Calling this out in the weekly update

Looks like we're waiting for proposals or an internal volunteer to grab this one. Is that right @greg-schroeder @grgia?

It should have been fixed here. Can we retest?

That being said, my fix was deployed on v1.4.36-5 and this issue was reported on v1.4.41-12, so maybe there's something else going on here.

Looks like they've moved from a free workspace to a paid collect plan now and everything's on OldDot, so I don't think we can test this one anymore.

Hm, ok. Should we close this one @luacmartins @grgia?

Yea, I think we can close this one and reopen if it happens again.


Add OrgaSegment dataset

Let's revisit this for the next release cycle. Also pining @sushmita-nair.


custom date consideration

import dateparser
print(dateparser.parse("1 week"))
print(dateparser.parse("1 month"))
print(dateparser.parse("6 months"))
print(dateparser.parse("1 year"))

The above code lines return the below output:
2022-01-27 11:30:50.229535
2022-01-03 11:30:50.231393
2021-08-03 11:30:50.232393
2021-02-03 11:30:50.232938

Essentially they consider the current timestamp and return prev one week or prev one moth or prev 6 months or prev one year  date time. What if I want date parser to consider a custom timestamp and based on that return the prev one week or prev one moth or prev 6 months or prev one year  date time?? For Example: For the same above code, I want the below output because I want date parser to consider the start date as 2021-06-13 11:30:50.229535
2022-06-06 11:30:50.229535
2022-05-14 11:30:50.231393
2021-01-13 11:30:50.232393
2020-06-13 11:30:50.232938

how to go about configuring this start date?

https://stackoverflow.com/questions/70975117/dateparser-custom-current-date-time-consideration-config-update/70977189#70977189


Enable the preview

Organization wide share settings seems to be broken in some places, specifically Additional Settings, enable the preview and set sharing per-site to bypass. I wasn't even able to find an acceptable user or group using the lookup.

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

ID: 6ed46514-8036-a543-d690-73d48acef690
Version Independent ID: 8542c3ce-3012-3d23-c3e4-4331dac033d2
Content: Turn external sharing on or off for SharePoint Online
Content Source: SharePoint/SharePointOnline/turn-external-sharing-on-or-off.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@nicksequeira Thank you for this feedback
I will get this information over to the SharePoint team for investigation.
The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

hello, @nicksequeira I've made a couple clarifications and they'll be live as soon as repo owner will merge it, so it should be more accurate from a content perspective. as for your mentioned issues, it looks more like a product issue and not as a content one.

Right on, glad to see y'all working on it. I'll check it out in a few weeks
thanks
On Thu, Feb 21, 2019, 08:33 Dmitri Plotnikov<EMAIL_ADDRESS>wrote:

hello, @nicksequeira https://github.com/nicksequeira I've made a couple
clarifications and they'll be live as soon as repo owner will merge it, so
it should be more accurate from a content perspective. as for your
mentioned issues, it looks more like a product issue and not as a content
one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/337#issuecomment-466000815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALJF0wCJTLDqCcilnYbnupwez-TZytnqks5vPqAkgaJpZM4XzqiF
.


validation of forms inside ng-repeat

http://plnkr.co/edit/WSjjdpVC9ck3NUw7iuKy?p=preview
I have run into angular scope issues while trying to apply validation on forms inside ng-repeat.
Since i am creating the forms as well as the fields dynamically i am assigning the form name dynamically. Now i know that ng-repeat creates an isolated scope and i feel that, that is the reason why on calling validation on the form it is giving me error that the form object is not valid since the form exits inside the isolated scope of ng-repeat and not on the controller scope.
My problem is how do i share the scope in this situation. I have bumped into a couple of solutions but nothing seems to work. Was hoping maybe I could get some help here..

This is a though one but so far, I manage to find the validationSummary inside the child scope. I get the angular element with the form name given in your for loop and then get the scope() from that object, then I can finally pass that to the checkFormValidity() without it complaining.. like so:
$scope.validate=function(){
      for(var key in $scope.items){
        var formName=$scope.items[key].formName;
        var form1 = angular.element(document.querySelector('[name="'+formName+'"]'));
        var childScope = form1.scope();
        console.debug(childScope[formName]);
        if(new validationService().checkFormValidity(childScope[formName]))
          alert("form "+formName+" is valid");
        else
          alert("form "+formName+" is invalid");
      }
  };

but the problem is that you need the Debug mode enabled, which is a mode you can disable for performance wise in Angular1.3+... See it here: Disabling Debug Data
So I manage to find the $validationSummary inside that child scope, but I did not go further to try to display the errors or anything, you could try and see what happen... Let me know

I added your code and also made the validation rule to be applied dynamically http://plnkr.co/edit/WSjjdpVC9ck3NUw7iuKy?p=preview and have some observations
-after checking for form validity through validate() the last field with required rule remains in error state even after adding text to it
-if you remove the required rule from the last field in form2 the form's validity through validate() comes as valid even though the first required field is not filled
i feel the validation fails for the last required field consistently because of which the form's validation is not calculated correclty...

Well I'm sorry, it's hard for to help you debug that one, I rarely have so much dynamic forms. If you find a problem with regular code then I can help you for sure but something that is dynamic is harder to debug. I would suggest you to try using the non-minified code if you want to try debugging it.

I'm working on something with dynamic forms and need to apply validation on those forms too. But, using your framework, validation keeps failing for the last required field and other than that validationservice's checkformvalidity() doesn't show error messages on fields. I can try to debug as you suggested using the non-minified code but as you have developed it ,you have a better understanding of the code rather than me.
Do you have any guess as to where the framework would be failing when validating dynamic forms and fields?
Meanwhile I'll also keep trying to find out and will keep you posted..

I made an update to my code last night which seems to have broke even your last code. What I can tell you is that the checkFormValidity() works in combo with the $validationSummary and the summary is empty then your form validity becomes valid, that is how I coded it. Also the summary is saved in 2 locations, the parent $scope and the $scope.form1 (if found) and in your particular case it doesn't seem to exist in any of them which is why it's failing. So the best would be to try debugging why or where does the summary save the failures and in my code that portion happens in the addToValidationSummary(). Not sure when I'll have time to debug it, but if you have time yourself that would help.

Alright, I made a copy of your plunker to my local code (it will be under the folder "more-examples") and debug everything, I found something that now seem to work but I need to do more much more tests just to make sure that I don't break anything else. It's getting late, so I'll go to bed, I will probably get something working tomorrow after all my tests are done.
In the mean time, you should try to find another alternative to this piece of code:
var form1 = angular.element(document.querySelector('[name="'+formName+'"]'));
var childScope = form1.scope();

Angular does not recommend finding the scope that way, as it is written in the documentation:
Methods

scope() - retrieves the scope of the current element or its parent. Requires Debug Data to be enabled.

I got the code wokring http://plnkr.co/edit/6HwgcYZpy4fgG0gQeQWQ?p=preview
when i added another form field outside ng-repeat i found that the validation summary showed this

then i realized that interpolation is failing for the dynamic fields inside ng-repeat and since your validation framework depends on the element name to validate it, it was failing
Adding this directive as suggested in http://stackoverflow.com/questions/21455695/angularjs-dynamic-form-field-validation/21457121#21457121
app.directive("dynamicName",function($compile){
return {
restrict:"A",
link:function(scope,element,attrs){
element.attr('name', scope.$eval(attrs.dynamicName));
element.removeAttr("dynamic-name");
$compile(element)(scope);
}
};
});
added the name to the field and then compiled it during linking. With this the validation is able to find the element using name and validates it :)

@ghiscoding, validationCommon is unable to handle dymanic field-names (which are supported in Angular 1.3). E.g. trying to use <input name="{{xyz}}" ... /> will result in the field being registered as {{xyz}} instead of the result of the interpolation.
From a quick look, you seem to be using elm.attr('name') at a point in time where it has not yet been updated with the result of the interpolation (i.e. it is still {{xyz}}). You could/should use attrs.name instead which has the correct value and should be more efficient.
--
As a sidenote, Angular is able to handle fully dynamic names (e.g. a fields name changing at any time and as many times as we wish). It might be of little value and not worth the trouble trying to support this, but it would be very useful to at least support interpollated names (which are often used for ngRepeated fields).

Good, I will consider your approach... I was actually changing my code to use the $interpolate because of another Dynamic Form issue https://github.com/ghiscoding/angular-validation/issues/26 that is open. I have some code change that seems to be working for dynamic names and was going to test everything tonight to make sure nothing is broken. Using your approach though might not always work with my Service implementation, I'll have to look at that.
Thanks for the info, you should expect a code commit later tonight. :)

I push the code (v1.3.20) wish should fix your Dynamic Forms with ng-Repeat. I also re-created your plunker in the folder more-examples with the name of dynamicFormIndex.html, which is what I used for troubleshooting your problem.
That should fix it. Enjoy it :)

The plunker http://plnkr.co/edit/WSjjdpVC9ck3NUw7iuKy?p=preview is not working for some time. Its referring to your git repo for validation js. Previously you had mentioned that you had committed the required changes for this issue but it seems its still not fixed.
With an old version (version 1.3.15) of your validation code this works but fails for your latest one 1.3.20.

I copied the code from your plunker to recreate a sample in the more-examples folder, your original plunker has couple of issues here and there, so instead of trying to debug that, download my whole project and in the more-examples folder run the dynamicFormIndex.html as I mentioned you earlier.  That one work, so please use that.

oh and by the way, your Plunker does not work because you are missing the $translate configuration. I got it working but I will not update it as I want you to use the code in the more-examples folder, it's easier to work with.

I added a new functionality see the Wiki - Isolated Scope and this also simplifies the original code I gave you, no more hacking to find the scope and all that weird way of coding. You can now call directly the checkFormValidity(), you only need to explicitly define the property isolatedScope inside the global options, so your code is now much simpler:
  // redefine which scope to use inside the Angular-Validation
  $scope.$validationOptions = { isolatedScope: $scope };

  $scope.validate=function() {
    for(var key in $scope.items) {
      var formName=$scope.items[key].formName;

      if(new validationService().checkFormValidity($scope[formName])) {
        $scope[formName].isValid = true;
      }
      else {
        $scope[formName].isValid = false;
      }
    }
  };

and the bonus is that you can now also disable the debug mode... $compileProvider.debugInfoEnabled(false); and it will still work and that wasn't the case before.

@ghiscoding  great work :+1:


Website scripts do not load when loading a deep page

problem is they are relative so it fails

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


drag/drop skills to HotkeyBar is broken/bugged

It's not totally broken (I managed to get one skill to end up on the skillbar and usable).
Normally the skill icon just stays there (still attached to UISkillTree) and not being "received" by the Skillbar.
Dragging/dropping items to the skillbar works fine (with some exceptions like the transformation gem).

Huh, nice find. I don't think I've tried to drag/drop skills from the tree in years.
Makes me wonder how long that would've gone unnoticed otherwise...
Not only are they not being attached, but they're not being reset on release either (which I believe should be a call to IconRestore()). Very interesting.

CUISkillTreeDlg isn't handling it like other things.
Ordinarily we go:
		case UIMSG_ICON_UP:
			// ¾ÆÀÌÄÜ ¸Å´ÏÀú À©µµ¿ìµéÀ» µ¹¾Æ ´Ù´Ï¸é¼ °Ë»ç..
			if ( !CGameProcedure::s_pUIMgr->BroadcastIconDropMsg(CN3UIWndBase::m_sSelectedIconInfo.pItemSelect) )
				// ¾ÆÀÌÄÜ À§Ä¡ ¿ø·¡´ë·Î..
				IconRestore();
			break;

And then handle behaviour appropriately in ReceiveIconDrop() (which is called by BroadcastIconDropMsg().
However, CUIHotKeyDlg only handles dragging items from the inventory, as you can see here:
	if ( CN3UIWndBase::m_sSelectedIconInfo.UIWndSelect.UIWnd != UIWND_INVENTORY )
		return false;

It would have to handle the instances specifically from the skill tree dialog, because they'll be skill icons not item icons (i.e. they won't have item data attached and the logic won't make sense).
Hm. Interesting.

Just update the function CUI HotKeyDlg :: IsSelectedSkill RealIconArea ()
`	CN3UIArea* pArea;
bool bFound = false;
POINT ptCur = CGameProcedure::s_pLocalInput->MouseGetPos();
for( int i = 0; i < MAX_SKILL_IN_HOTKEY; i++ )
{
	pArea = CN3UIWndBase::GetChildAreaByiOrder(UI_AREA_TYPE_SKILL_HOTKEY, i);
	if (pArea && pArea->IsIn(ptCur.x, ptCur.y))
	{
		SetReceiveSelectedSkill(i);
		bFound = true;
		break;
	}
}

if (!bFound) return false;
if (!CN3UIWndBase::m_sSkillSelectInfo.pSkillDoneInfo) return false;

return bFound;`

Unintentionally triggering behaviour (from the caller's perspective) in a method indicated as a check only is probably not the way to go there.

Unintentionally triggering behaviour (from the caller's perspective) in a method indicated as a check only is probably not the way to go there.

I could not see a properly controlled method across the source code.
To explain;
Detects an icon drifting from "UISkillTreeDlg" here; CUISkillTreeDlg::ReceiveMessage case UIMSG_ICON_UP:
The icon is checked if it is on "UIHotKeyDlg" with "IsSelectedSkillInRealIconArea" method, but it does not fix the icon in any way.
The "CUIHotKeyDlg :: SetReceiveSelectedSkill" line I added above acts as "IconRestore" here and fixes the icon in the area.
There is no problem working properly. Could you please open some more what you mean?
@twostars


Handle spurious failure were rename succeeded but reported it failed (#1588)

Fix #1588 : Refine rename during minor compaction
PR addresses a few issues:

Spurious rename failures where rename failed but actually succeeded
The check for file existence should only be done before attempting rename
Should not delete file if it exists (an admin may want to investigate).

I think #1588 is a more general issue that could still be beneficial and be done at some point in the future.  Could change the commit message for this PR to something like Handle spurious failure were rename succeeded but reported it failed and omit #1588 or just reference #1588 in the commit message without the preceding fix

@tushardhadiwal  thanks for the contribution.  If you would like to be listed as a contributor on the people page, please submit a PR to add yourself.  Let me know if you have any questions about editing the website.


Show result on empty history

Right now, on the first run of the cf Script Filter, the following happens:

Type cf and see results in Alfred.
Type space to start the prompt. Results in Alfred disappear.
Type first letter. Result reappears.

Step two makes is seem something went wrong, but it’s just the script returning zero results because there is no history yet. Maybe it’s worth it to output “No prompt history yet” as a subtitle or something in that case?

Improved in: v1.3.1


Wrong screenshot for "Queries & Connections" option for Refresh All Objects

Under the Refresh All Objects section the screenshot for the second option using the Queries & Connections pane does not show that UI. Instead the screenshot is the same as the one shown for the Refresh All->Refresh option.

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

ID: 7a4819a8-3056-4134-248c-b216300f62dd
Version Independent ID: 7a4819a8-3056-4134-248c-b216300f62dd
Content: Design refreshable reports in Excel with Power BI data - Power BI
Content Source: powerbi-docs/collaborate-share/service-analyze-power-bi-datasets-excel.md
Service: powerbi
Sub-service: pbi-collaborate-share
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @keithrjones. This issue has been assigned to the doc author for review.

Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. Both the product and documentation might have changed since this issue was opened. Therefore, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.


BTC-ETH Converter

Student, group: Божко Александр, 3530901/90201
URL:  https://github.com/BozhkoAV/BTC-ETH-Converter
Git approach: git-flow

`+


Add headers for ruby-2.7.7-p221

Not sure if there's a way, but can/should I target a patch version in 0.10.x instead?


Fail to replicate if a table has more than one primary/unique key

For example, I creates a table like this
create table account(
  id int,
  name char(16) unique,
  balance float,
  primary key(id)
);
insert into account values(12500000, "name0", 514.35);

Then when I try to replica this from MariaDB to MyDuck, an error occurs:
ERROR: TGT: [worker000]: test@account@@0.tsv: MySQL Error 1105 (HY000): Binder Error: Conflict target has to be provided for a D0 UPDATE operation when the table has multiple UNIQUE/PRIMARY KEY constraints: LOAD DATA LOCAL INFILE 'memory/test@account@@0.tsv' REPLACE INTO TABLE `test`. `account` CHARACTER SET 'utf8mb4' FIELDS TERMINATED BY '   ' ESCAPED BY '\\' LINES STARTING BY '' TERMINATED BY '\n' (`id`, `name`, `balance` )

This error disappeared after removing unique.

Does this occur in the invocation of mysqlsh? If so, can we bypass it by using the deferTableIndexes options? https://dev.mysql.com/doc/mysql-shell/8.4/en/mysql-shell-utils-copy.html#:~:text=restrictions and configuration.-,deferTableIndexes,-%3A [ off | fulltext | all

Interesting. Maybe we should investigate why --defer-table-indexes all isn't working. Is there anything noteworthy in the MySQL Shell log?

You can run ./myduckserver --loglevel 6 to observe whether the secondary index creation is actually defered. If not, you can search deferTableIndexes  in MySQL Shell's source code to find why it does not take effect.


Cannot Drag Folders to Bundle - Instant Crash

Describe the bug
Cannot drag folder into a bundle. When I do, Files crashes
To Reproduce
Steps to reproduce the behavior:

In dual-pane view drag a folder (after opening in primary pane) to a bundle on the secondary pane
Files crashes

Expected behavior
The folder appears in the bundle
Screenshots
https://www.dropbox.com/s/t0qbksom7sguwv8/2021-03-12_11-05-09.mp4?dl=0
Desktop (please complete the following information):

OS Version: Windows  Version 10.0.17763 Build 17763

App version: [e.g. v1.0]
<IP_ADDRESS>

Additional context
The problem occurs every time
debug_fulltrust.txt
debug.txt
Log file
attached

@mannycastaneda this issue should be resolved.


Expanding Patch Operations

This PR expands the patch operations to support Set, Add, Remove, and Increment.
@mumby0168 I understand this was assigned to you, I hope I am not stepping on your toes, this is just a draft to make sure I haven't overstepped, I will add tests after I know if this PR is needed.
I also had a question about the _rawPatchOperations is it needed?
Fixes #297

This looks awesome, thank you for this contribution @evancjohnson! As long as @mumby0168 is good with this, let's make it happen. Yes, please add tests. As for the _rawPatchOperations I'm not certain, Billy would know.

@all-contributors please add @evancjohnson for code and tests. Thank you 💜

@mumby0168 @IEvangelist I see where _rawPatchOperations is used, the current implementation of  InMemoryRepository expects property info to be provided in the internal patch operation. Is this approach reasonable given that the caller can also provide an explicit path? Using the direct path would allow users to patch a nested object.
Should the InMemoryRepository use Newtonsoft.Json in some way (I haven't actually looked up if this possible, I believe I have done something like this in the past) to apply the replace operation to the TItem by path removing the need for _rawPatchOperations and instead leaning on the information provided by the PatchOperation

Hi @evancjohnson yes, from my distant memory the _rawPatchOperations allowed the in-memory repo to support this feature, it was a challenge and I suspect keeping the support for all the patch operations maybe be difficult ... if there is a way to apply the modifications to JSON from the in memory repo that would be great! Check out the json path support on the JObject type that might help in this scenario?

Thanks for the tip @mumby0168. I will do some research on a good approach. I've got a quick questions.
How rigid is the InMemoryRepository? Is it expected to perform exactly the same way as CosmosDB?

Hi @evancjohnson - it's not super rigid. It was intended to be a helper so that you could use it for unit testing, to help ensure other functionality that might rely on the underlying data store. Does that make sense? We know that there are functional gaps, and we're ok with that for now.

@IEvangelist I'll have time to work on this during the weekend, sorry for leaving it open for so long.

@IEvangelist I'll have time to work on this during the weekend, sorry for leaving it open for so long.

Will you be able to finish working on this code?

@IEvangelist I'll have time to work on this during the weekend, sorry for leaving it open for so long.

Will you be able to finish working on this code?

Hi @MatKumpf - this was something that @evancjohnson was working on, but hasn't updated. If the op is unable to finish this, I could work on it. Unless it's something that you're interested in?

@IEvangelist I'll have time to work on this during the weekend, sorry for leaving it open for so long.

Will you be able to finish working on this code?

Hi @MatKumpf - this was something that @evancjohnson was working on, but hasn't updated. If the op is unable to finish this, I could work on it. Unless it's something that you're interested in?

I won't conceal, it would be one of the most important functionalities for me 😄 So far I have to in few places create my custom connection with container and use the rest of patch operations on it.

@MatKumpf @IEvangelist Sorry for the delay friends I have just finished a project and am about to graduate school, so I have much more time. I am pretty close to finishing this one up. I do have a question which is more architectural for David and Billy. Should test host support patch operations? I personally do not believe that it should, and those tests should be run against the emulator.
The reason I suggest that due to the fact patch operations have some not at first noticeable quirks. Check the troubleshooting for an example.
What are the thoughts on just supporting testing of patch operations internally and not exposing those in the test harness provided by the library?

@MatKumpf @IEvangelist Sorry for the delay friends I have just finished a project and am about to graduate school, so I have much more time. I am pretty close to finishing this one up. I do have a question which is more architectural for David and Billy. Should test host support patch operations? I personally do not believe that it should, and those tests should be run against the emulator.
The reason I suggest that due to the fact patch operations have some not at first noticeable quirks. Check the troubleshooting for an example.
What are the thoughts on just supporting testing of patch operations internally and not exposing those in the test harness provided by the library?

This is a decision we've made with some of the other features, as in not to support those features in the in memory implementation I think that is also fine in this case due to the complexity, would you agree @IEvangelist ?

Hi again @evancjohnson - any updates on this?

We can always reopen this PR. But it's been nearly a year since it was first introduced. And months since the OP has replied.


Add remap command to sphere simulations

Purpose

Closes #1382.

Adds remap command only to sphere simulations because the current remap pipeline only works for the sphere.
Remapping is not added for jobid sphere_baroclinic_wave_rhoe_equilmoist_conservation_ft64 because it doesn't work for moist simulations with no microphysics schemes (the remap script assumes precipitation_removal exists, we can fix it later, but it is not that important as this job is mostly to check conservation).

To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+


Replace MonotonicTimestampNs() by orbit_base::TimestampNs()

This PR is split in two commits.
The first one defines Orbit's main Linux clock as "kOrbitTimelineClock" and use it
for internal timestamping and for events obtained through the perf_event_open api.
This is done to explicitly link perf_event_open's clock to Orbit's timestamping clock.
The second commit puts the timestamping functions in the orbit_base namespace
and removes the "Monotonic" prefix.
This change stems from a discussion in https://github.com/google/orbit/pull/1772.

I don't know, for me what I said in the other pull request still applies with no change. This function has different semantics on Windows and Linux, as on Windows it's a generic monotonic timestamp, and on Linux it's a timestamp from a very specific clock for a specific reason. Also the name now is extremely generic (I thought we wanted to make the name more specific, not the opposite?) and this reason is lost even more.
So in the end I still feel the same: This should be a Linux-only function as it serves a specific Linux purpose—I really don't get why you want this function to have a Windows implementation. If necessary there could be a more generic timestamp function (or not, just use what you need from std or absl). The very special case of UI introspection (where we want a specific clock on Linux and a generic monotonic clock on Windows) can easily be handled with an #ifdef in the introspection code, not here.
This said, I don't really want to go through another long discussion for this, so if Ronald and Dimitry are fine I guess I'm fine, worst case we change it again.


Could not find maven.modrinth:vanish:1.5.5-1.20.1.

> Could not resolve all files for configuration ':modCompileOnly'.
   > Could not find maven.modrinth:vanish:1.5.5-1.20.1.
     Searched in the following locations:
       - file:/home/killerz/TimeRecorder/.gradle/loom-cache/remapped_mods/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - file:/home/killerz/.gradle/caches/fabric-loom/minecraftMaven/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - file:/home/killerz/TimeRecorder/.gradle/loom-cache/minecraftMaven/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://maven.fabricmc.net/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://libraries.minecraft.net/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://libraries.minecraft.net/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.jar
       - https://repo.maven.apache.org/maven2/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://repo.codemc.io/repository/maven-releases/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://maven.nova-committee.cn/releases/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://raw.github.com/Nova-Committee/maven/release/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://api.modrinth.com/maven/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
       - https://maven.parchmentmc.org/maven/modrinth/vanish/1.5.5-1.20.1/vanish-1.5.5-1.20.1.pom
     Required by:
         project :

What is the context of this? The correct version would be 1.5.5+1.20.1, not 1.5.5-1.20.1.

Just a fork of stopped projectLOL
thank you，I didnt learned java before
---Original---
From: @.>
Date: Fri, Aug 9, 2024 20:45 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [DrexHD/Vanish] Could not findmaven.modrinth:vanish:1.5.5-1.20.1. (Issue #47)
What is the context of this? The correct version would be 1.5.5+1.20.1, not 1.5.5-1.20.1.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Rename keepassx2-http to keepassx-reboot

Motivation for this change
The keepassx2-http fork has been moved to a new organization and
renamed to keepassx-reboot. For more details on the change, see the
discussions in GitHub issues [1][2].
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Included changes:

Rename the keepassx2-http package to keepassx-reboot
Fetch source from correct (moved) GitHub repository
Update the version to the latest release
Change the homepage, as these projects are likely to diverge over
time
Add keepassx2-http to `aliases.nix

[1] https://github.com/keepassx/keepassx/pull/111#issuecomment-250639109
[2] https://github.com/keepassxreboot/keepassx/issues/40

Ping @globin Can you merge this?


az login --identity giving error in system managed identity

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 654, in execute
    raise ex
cli/core/commands/__init__.py, ln 718, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 711, in _run_job
    six.reraise(*sys.exc_info())
lib/python3.6/site-packages/six.py, ln 703, in reraise
    raise value
...
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 600, in set_token
    token_entry = self._vm_msi.get_token(self.resource)
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 649, in get_token
    token_entry = self._retrieve_token_from_imds_with_retry(resource)
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 704, in _retrieve_token_from_imds_with_retry
    raise HTTPError(request=result.request, response=result.raw)
requests.exceptions.HTTPError

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --identity

Expected Behavior
Environment Summary
Linux-5.3.0-1020-azure-x86_64-with-debian-buster-sid
Python 3.6.10
Installer: DEB

azure-cli 2.7.0

Additional Context

add to S172

Could you follow Get a token using CURL to get an access token directly with curl instead of az?
If you are receiving an error using curl:
{"error":"invalid_request","error_description":"Identity not found"}

this simply suggests that Managed Identity is not turned on.
Also see

Configure managed identities for Azure resources on an Azure VM using Azure CLI
Sign in with a managed identity


Trying Caddy with own Boulder

Hi,
i am trying to get automatic certificates from my own boulder instance. Since the boulder HTTPS server is running with an untrusted TLS Certificate singed by a untrusted CA, I added the CA to ubuntu's /etc/ssl/certs and did update-ca-certificates -f to get it merged to /etc/ssl/certs/ca-certificates.crt
./caddy
Activating privacy features...2017/10/11 15:18:55 get directory at 'https://wfe.boulder:4430/directory': failed to get json "https://wfe.boulder:4430/directory": Get https://wfe.boulder:4430/directory: x509: certificate signed by unknown authority

I can see with strace caddy that caddy opens the CA file:
openat(AT_FDCWD, "/etc/ssl/certs/ca-certificates.crt", O_RDONLY|O_CLOEXEC) = 6
With openssl as client it works
openssl s_client -connect wfe.boulder:4430 -CAfile /etc/ssl/certs/ca-certificates.crt
...
Verify return code: 0 (ok)

Adding the CA to chrome's CA Store also shows a green OK.
Anyone have an idea how I can get a verify OK with caddy for the custom CA?
(i am using the latest caddy version)

It finally worked adding the CA manually to /etc/ssl/certs/ca-certificates.crt
update-ca-certificates only updates certs listed in file /etc/ca-certificates.conf

Great! Glad you managed to fix it.
For future reference, it's usually best to ask questions on https://caddy.community/ instead. There's more activity there. This issue board is more for bugs and feature requests.

This is good to know! Thanks for reporting back. I'm sure others will find this solution useful too.
@mschneider82 If you happen to decide to deploy boulder internally inside your company (I'm presuming?), tell Let's Encrypt -- not because you have to, but because I think they'd be interested in knowing how people are using it. :+1:


Created Folder structure for tests for app

Created a folder tests
Segregates all the class tests into their respective files for easier test handling

Hi @rtgdk can you please check this once and let me know if I am doing it correctly, like we need to do something similar for the api folder so, if this one looks good, I will follow the similar structure for it as well

Please provide the screenshot and the command used to run all test cases. Show the output as successfull

Hi @rtgdk I think all the tests are working like I tried this on both the main and Test-Files branch and the tests which are failing are common like they are the firefox ones I think it cannot find the relevant webdriver on my local wsl setup
So once I like do the same for the other modules can you please run my branch locally before merging it

Overall changes LGTM. I hope these is no code leakage in this - very difficult to review for that. I would have loved to have a common setup/teardown for all tests and having all tests run without failure/skipping but I guess these will need some rethinking on how to run test cases run - currently a lot of duplicate code exists .
@BassCoder2808 Can you please list down the test cases not working on your machine?  This will make the dataset for tests affected by selenium/firefox usecases and provide us a start to move these to unit based framework.
There are already some issues opened for fixing these testcases. Let's try to fix some of them.
Holding to merge this diff for sometime - until I run tests on this branch locally.


Incomplete context when validating additionalProperties

When additionalProperties is defined as:
"additionalProperties": {
    "type": "string"
}

and the validation of such property fails, the property name is not included in the error context.
For example I want validation of this document:
{
  "Device": {
    "Color": true
  }
}

To fail with an error message:
Device.Color: Invalid type. Expected: string, given: boolean

but instead the message is:
Device: Invalid type. Expected: string, given: boolean

With multiple keys under Device it is hard to understand what part of the document is incorrect.
The problematic schema:
{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "type": "object",
  "properties": {
    "Device": {
      "type": "object",
      "additionalProperties": {
        "type": "string"
      }
    }
  }
}

When using patternProperties instead of additionalProperties then the error message is correct.

From what I understand the issue is in validation.go:672 there's a call:
validationResult := additionalPropertiesSchema.subValidateWithContext(value[pk], context)

which should instead get as a second parameter subContext := NewJsonContext(pk, context)
Here's a test for this issue:
CLICK ME

func TestAdditionalPropertiesErrorMessage(t *testing.T) {
	schema := `{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "type": "object",
  "properties": {
    "Device": {
      "type": "object",
      "additionalProperties": {
        "type": "string"
      }
    }
  }
}`
	text := `{
		"Device":{
			"Color" : true
		}
	}`
	loader := NewBytesLoader([]byte(schema))
	result, err := Validate(loader, NewBytesLoader([]byte(text)))
	if err != nil {
		t.Fatal(err)
	}

	if len(result.Errors()) != 1 {
		t.Fatal("Expected 1 error but got", len(result.Errors()))
	}

	expected := "Device.Color: Invalid type. Expected: string, given: boolean"
	actual := result.Errors()[0].String()
	if actual != expected {
		t.Fatalf("Expected '%s' but got '%s'", expected, actual)
	}
}

I can create a PR if you prefer that but I don't see a good place where I could add a test for an error message.

I was working on this part of the code doing some refactoring. I'll include a fix for this and merge the fix today or tomorrow.


Cannot create resource in Portal (SignalR service)

Describe the bug
Cannot create a signalR service resource in azure portal
To Reproduce
go to azure portal and try to create a signalr service in any west US region
Exceptions (if any)
{"type":"MsPortalFx.Errors.AjaxError","baseTypes":["MsPortalFx.Errors.AjaxError","MsPortalFx.Errors.Error"],"data":{"uri":"https://management.azure.com/api/invoke","type":"POST","pathAndQuery":"/subscriptions/0bb0d1d5-e384-49ea-99ab-134be74bcfbf/providers/Microsoft.SignalRService/locations/westus2/checkNameAvailability?api-version=2023-08-01-preview","requestId":"8f7f5186-7f84-49fb-8e2c-243e7e22c00a","failureCause":"","sessionId":"f6ed647e814944d2b8d1085c5604d109","commandName":"Microsoft_Azure_SignalR.","status":500,"statusText":"error","duration":985.1999999880791},"extension":"Microsoft_Azure_SignalR","errorLevel":2,"timestamp":214808.60000002384,"name":"AjaxError","innerErrors":[],"textStatus":"error","errorThrown":"","jqXHR":{"readyState":4,"responseText":"{"error":{"code":"InternalServerError","message":"Unhandled Exception in Azure SignalR service. Please try again later."}}","responseJSON":{"error":{"code":"InternalServerError","message":"Unhandled Exception in Azure SignalR service. Please try again later."}},"status":500,"statusText":"error"}}

was able to do it through the powershell command

was able to do it through the powershell command


[Rumble] Throttling

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that a supported site is broken
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Norway
Provide a description that is worded well enough to be understood
For some time now I have had reduced downloading speeds on Rumble. When the download begins the speed is several MB/s, but it very quickly reduces to 300-400 KiB/s.
When similar throttling has happened on Youtube this has been fixed in consequent yt-dlp releases, so I wonder if perhaps this issue on Rumble hasn't been identified.
The session quoted below started out >2.5 MB/s
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU https://rumble.com/v25xfcm-leo-zagami-hollywood-the-military-industrial-complex-the-occult-and-the-gre.html
[debug] Command-line config: ['-vU', 'https://rumble.com/v25xfcm-leo-zagami-hollywood-the-military-industrial-complex-the-occult-and-the-gre.html']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (pip)
[debug] Python 3.10.6 (CPython x86_64 64bit) - Linux-5.15.0-67-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, secretstorage-3.3.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[Rumble] Extracting URL: https://rumble.com/v25xfcm-leo-zagami-hollywood-the-military-industrial-complex-the-occult-and-the-gre.html
[Rumble] v25xfcm-leo-zagami-hollywood-the-military-industrial-complex-the-occult-and-the-gre.html: Downloading webpage
[RumbleEmbed] Extracting URL: https://rumble.com/embed/v23bajo
[RumbleEmbed] v23bajo: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] v23bajo: Downloading 1 format(s): mp4-720p
[debug] Invoking http downloader on "https://sp.rmbl.ws/s8/2/2/J/t/Z/2JtZh.gaa.mp4"
[download] Resuming download at byte 319706681
[download] Destination: Leo Zagami： Hollywood, the Military-Industrial-Complex, the Occult, & The Great Reset [v23bajo].mp4
[download] 100% of  931.19MiB in 00:32:57 at 482.18KiB/s

try with aria2c   yt-dlp  --downloader aria2c "https://rumble.com/v25xfcm-leo-zagami-hollywood-the-military-industrial-complex-the-occult-and-the-gre.html"

test


Always run code quality tasks in buildSrc

Context
Back to https://github.com/gradle/gradle/commit/e215b27afaada8462e4824a6eb9bbe5c9e3a0532,
we disabled buildSrc code quality tasks to speed up the build. Now is 2018, with build cache
this is no longer a problem. Plus, conditionally enabling these tasks causes cache misses,
so we always enable the buildSrc code quality tasks.

Thanks @wolfs !


Problems in walkthrough: Creating an SDK using C++

https://github.com/MicrosoftDocs/visualstudio-docs/blob/master/docs/extensibility/walkthrough-creating-an-sdk-using-cpp.md

The Prerequisites element is missing from the .vsixmanifest
Wrong pri file name. Should be NativeMathWRT.pri instead of SimpleMath.pri
Incorrect path in the props file. Should be DesignTime\Debug\x86 instead of DesignTime\Retail\x86 (Or the example should work with a Release build)
In step 11 the paths are formatted incorrectly without back slashes

InstallationTarget for VS 2017
Properties will not be imported (I only tried in VS2017, maybe it works elsewhere)
The problem is <InstallationTarget Id="Microsoft.ExtensionSDK" TargetPlatformIdentifier="Windows" TargetPlatformVersion="v8.0"... will install the extension at C:\Program Files (x86)\Microsoft SDKs\Windows\v8.0\ExtensionSDKs\ and not C:\Program Files (x86)\Microsoft SDKs\UAP\v<IP_ADDRESS>\ExtensionSDKs\ where a sample UWP app made with VS2017 expects them to be.

Thanks for submitting your feedback and we apologize for the delayed response. Due to current resourcing and substantial extensibility changes we are currently working on that will result in significant doc changes, we are closing this lower priority issue. We will return to this issue on a later date and address it based on it's relevance after those extensive extensibility updates are made. Feel free to comment here as necessary.


date datetime helper

Hello Umer,
thanks for this new helper. It looks good to me.
Please remove the lines that we dont need.
this.$expr = new SQLBuilder.SQLPredefined.StringValueParam(sql);
this.$format = new SQLBuilder.SQLPredefined.StringValueParam(sql);

You only need to to declare them if your syntax has used them like:
this.Types({
  String: { syntax: this.Syntax('DATE(<value-param>, $expr[, $format])') },
});

Yes, I have removed the extra code which is not used in the helper. PR is
updated. Thanks
Regards,
Umer Mehmood
NorthBay
Direct: +92<PHONE_NUMBER>
<EMAIL_ADDRESS><EMAIL_ADDRESS>www.Northbaysolutions.com http://www.northbaysolutions.com/
On Mon, Nov 11, 2019 at 8:23 PM Planetary Development <
<EMAIL_ADDRESS>wrote:

Hello Umer,
thanks for this new helper. It looks good to me.
Please remove the lines that we dont need.
this.$expr = new SQLBuilder.SQLPredefined.StringValueParam(sql);this.$format = new SQLBuilder.SQLPredefined.StringValueParam(sql);
You only need to to declare them if your syntax has used them like:
this.Types({
String: { syntax: this.Syntax('DATE(, $expr[, $format])') },
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/planetarydev/json-sql-builder2/pull/25?email_source=notifications&email_token=AMMDKPBSWQG24B5CNGUKFSLQTF2G7A5CNFSM4JLWWO5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDXE5YA#issuecomment-552488672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMMDKPDT6JMYFFUC5YA4MCLQTF2G7ANCNFSM4JLWWO5A
.

THANKS!

Hello,
I have a use case in which we want to use $count operator on a column value
and compare the result to any numeric value. And i am not able to use
$where helper to accommodate the $count helper.
E.g.
Select * from table1
Where count(column) = 10
On Mon, 11 Nov 2019, 8:35 pm Planetary Development, <
<EMAIL_ADDRESS>wrote:

Merged #25 https://github.com/planetarydev/json-sql-builder2/pull/25
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/planetarydev/json-sql-builder2/pull/25?email_source=notifications&email_token=AMMDKPBPCDOLBPFEV7CFUILQTF3VNA5CNFSM4JLWWO5KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUYZPTPA#event-2788358588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMMDKPG635LBTRA2KBGWJE3QTF3VNANCNFSM4JLWWO5A
.

In addition to this, if i want to cast any column type while using it in
where.
E.g.
Select * from table1
Where cast(date) = '2019-09-01'
Also i have to cater this kind of use case.
On Mon, 11 Nov 2019, 9:16 pm umer mahmood, <
<EMAIL_ADDRESS>wrote:

Hello,
I have a use case in which we want to use $count operator on a column
value and compare the result to any numeric value. And i am not able to use
$where helper to accommodate the $count helper.
E.g.
Select * from table1
Where count(column) = 10
On Mon, 11 Nov 2019, 8:35 pm Planetary Development, <
<EMAIL_ADDRESS>wrote:

Merged #25 https://github.com/planetarydev/json-sql-builder2/pull/25
into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/planetarydev/json-sql-builder2/pull/25?email_source=notifications&email_token=AMMDKPBPCDOLBPFEV7CFUILQTF3VNA5CNFSM4JLWWO5KYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUYZPTPA#event-2788358588,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMMDKPG635LBTRA2KBGWJE3QTF3VNANCNFSM4JLWWO5A
.

One thing ahead - please open an new issue for further questions. So anybody can find if he got the same problem/question.
Using the count-function inside the where clause will be a little bit tricky. You have to use the anyExpr helper and it works like this:
sql.build({
  $select: {
    $from: 'people',
    $where: {
      $and: [
        { $anyExpr: { $count: '*', $eq: 10 } }
      ]
    }
  }
});

https://github.com/planetarydev/json-sql-builder2/tree/master/sql/helpers/misc/anyExpr
Here you will find an example for the having clause.

I have opened an new issue #26
Please have a look.


hiredis-client: refactor compilation flags

Rather than to use an exported symbols list, we compile with -fvisibility=hidden and mark the init function with RUBY_FUNC_EXPORTED.
This was suggested by @nobu.
Ref: https://github.com/redis-rb/redis-client/issues/58#issuecomment-1397802949

Yeah, but CI is broken for some reason, not sure if related or not.

Somehow the hiredis test suite pass on my machine, but on CI it's consistently broken for 3.1 and 3.2.  😫

Is it also broken on master?

No it passes fine: https://github.com/redis-rb/redis-client/actions/runs/3968428783/jobs/6801585027
the pure-ruby part of the test suite also pass fine, only the hiredis (binding) part is failing with a weird EAGAIN issue.
I probably won't have time to debug this much before a week or two though :/

interesting, if I try to repro locally inside the ruby:3.2 docker image, I get a ENOTSOCK error on connect, that's even wierder.
@eregon given that you are on Linux if I understand correctly, I wonder if you get the same issue?

So the issue seem to be that on Linux the visibility=hidden doesn't apply to the symbols coming from the libhiredis.a, I tried a few solution I found on the web for no luck so far.

Well, somehow on 3.2 it is not passed:
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb alloc.c
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb net.c
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb hiredis.c
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb sds.c
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb async.c
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb read.c
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb sockcompat.c
ar rcs libhiredis.a alloc.o net.o hiredis.o sds.o async.o read.o sockcompat.o
cc -std=c99 -pedantic -c -g -fPIC  -DHIREDIS_TEST_SSL -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb ssl.c

But on 3.0 it is:
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb alloc.c
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb net.c
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb hiredis.c
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb sds.c
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb async.c
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb read.c
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb sockcompat.c
ar rcs libhiredis.a alloc.o net.o hiredis.o sds.o async.o read.o sockcompat.o
cc -std=c99 -pedantic -c -g -fPIC  "-I/opt/hostedtoolcache/Ruby/3.0.5/x64/openssl/include" "-fvisibility=hidden" -Wall -W -Wstrict-prototypes -Wwrite-strings -Wno-missing-field-initializers -g -ggdb ssl.c

So that's likely the source of the issue yeah.

Ok, I finally figured it out.
@flavorjones is working on a cleanup of the extconf.rb though, so I'll wait for his PR first.

@flavorjones is working on a cleanup of the extconf.rb though, so I'll wait for his PR first.

Sounds like heaven to me :) (I've come across many extconf.rb, and most could afford a good cleanup, it tends to be the least readable Ruby scripts of all)

Ok, all green. Thank you all! It's much cleaner now.


Add TypeScript support to Svelte

We already have _extensions/sverto/setupTypeScript.js from https://github.com/sveltejs/template, and it looks like that really just adds the dependencies to package.json, so I'm hoping this won't be too hard!


docs(transformer/class-properties): re-order file list in doc comment

Re-order list to reflect order transforms occur.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7912
#7911  👈 (View in Graphite)
#7910
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Trivial change, so merging without review.

Merge activity

Dec 15, 8:01 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


8:39 AM	Plugin error: Plugin 'Poetry' (version '1.1.5-212') is not compatible with the current version of the IDE, because it requires build 212.* or older but the current build is PC-213.4631.9

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

PyCharm 2021.3 was rolled out today in updates doesn’t seem to be any support for poetry at this stage.

https://www.jetbrains.com/pycharm/whatsnew/#poetry-support
I have used this version for poetry.

Does this mean this plugin is being deprecated?

@sseveran
Yes.
PyCharm 2021.3 has the same code as this plugin.
So, This plugin will not be supported on PyCharm 2021.3

We should write the information at the top of the repository.

I just updated README.md


Sometimes app does not display result from Latency Test

To REPRODUCE:
Insert fun plug
Turn up volume to about 80%
Launch Loopback app
Press Calibrate Sound Level button
EXPECT
Just calibrate volume
ACTUAL
The Glitch test icon turns red indicating that it is running.
Sometimes the calibration sows a latency result.
Sometimes the Glitch test stays red and the Latency Test does not show a result.
If you run the latency test multiple times it starts to work.

This is very intermittent and hard to reproduce.
This is sometimes a symptom of call UI methods not on the UI thread.
The setTransportButtonsState() method makes UI calls.
It is called from a Handler in LoopbackActivity.java
The Handler is not on the UI thread:
private Handler mMessageHandler = new Handler()

It may need to be on the UI thread:
private Handler mMessageHandler = new Handler(Looper.getMainLooper())

Info here:
https://developer.android.com/training/multiple-threads/communicate-ui#Handler
Also I think lines 1104 and 1115 should set latencyStart not bufferStart.

I have never seen this particular failure, nor was I able to reproduce it when I tested again today numerous times. Ideally I would prefer to see a fail -> pass transition before accepting a patch. However, in this case I am accepting the pull request without a transition, on the assumption that it will solve the problem based on the thread analysis.


OSSM-1847 ensure ProxyUID and ProxyGID are initialized when using gat…

…eway injection template
Signed-off-by: rcernich<EMAIL_ADDRESS>Please provide a description of this PR:
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

/test integration

/test integration


Add Travis configuration

Registry has already tests.

This looks good to me, @kelunik is it ready to merge?

Yes.


Add /deployment/processes REST call

Add /deployment/processes REST call which is already implemented in DeploymentsResourceImpl.
(docbook is generated from adoc following the guidance in https://issues.jboss.org/browse/JBPM-4568)

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

ok to test

add to whitelist


PSCredential Parameterbinding quirks

Hi,
I'm trying to create a parameter class that can be directly bound to a PSCredential parameter.
A simple example use of the objective:
$cred = [DbaCredential]"foo\bar"
Get-WmiObject -ComputerName "computer1" -Credential $cred -Class "Win32_OperatingSystem"

This parameter class is basically a functionality wrapper around the PSCredential object (So I didn't try to reinvent security. Not good enough for that). When using it like this however I am rewarded by a seriously awesome message in red:

Now, this parameter class is for module-internal use only, so I suppose I could just call it like this:
$cred = [DbaCredential]"foo\bar"
Get-WmiObject -ComputerName "computer1" -Credential $cred.Credential -Class "Win32_OperatingSystem"

Which would work just fine. My queen and project leader however would appreciate to not confuse random contributors (because we already have a lot of fancy features and we don't want to scare them away. They are trying to help after all). Every little thing adds up and so I've spent a few hours trying to make it work.
By now I really want to know what the heck is happening here, out of sheer bloody-mindedness!
What I've tried so far:

Set up implicit conversion
Implemented IConvertible
Created a PSTypeConverter for the conversion and attached it to the input type.

Guess what ... none of it worked.
Giving up on it, I dug down into the exception and PowerShell code (well ... it was more a parallel thing, alternating between the two):

The parameter binding passes it to the CredentialAttribute's Transform method. This in return calls LanguagePrimitives's T FromObjectAs<T> method to have it bake a PSCredential object. In that method it tries to cast to PSCredential which should be working, dangit! , returns null instead and the Transform method throws the exception which is then relayed by the parameter binding.

What is happening here and what did I forget? Any ideas?
The current implementation can be checked out on our dbatools project's DbaCredentialParameter branch
Thanks in advance for any ideas, inspiration or suggestions,
Cheers,
Fred

In case you don't strictly need an actual .NET type, here's a simple all-PowerShell alternative using PowerShell's ETS (extended type system):

Define a function New-DbaCredential that creates [PSCredential] instances that are ad-hoc-decorated with the additional members as needed.

The resulting instances will still behave like [PSCredential] in the context of cmdlet calls, yet you can access the extra members as needed; Get-Member will report the made-up type name DbaCredential.

function New-DbaCredential {

  param([string] $UserName = (whoami), $Foo, $Bar)

  # Construct a [PSCredential] instance...
  # Note: This will prompt for a password.
  $cred = Get-Credential -UserName $UserName -Message 'Enter password:'

  # Now shamelessy lie about the output object's type...
  $cred.pstypenames.Insert(0, 'DbaCredential')

  # ... and decorate it with your custom properties and output the result.
  Add-Member -InputObject $cred  -PassThru -NotePropertyMembers @{ Foo = $Foo; Bar = $Bar }

}

Now you can construct instances of your custom "type":
> $custCred = New-DbaCredential -UserName jdoe -Foo f -Bar b; $custCred

Foo Bar UserName                     Password
--- --- --------                     --------
f   b   jdoe     System.Security.SecureString

> $custCred.pstypenames
DbaCredential
System.Management.Automation.PSCredential
System.Object

> $custCred -is [PSCredential]
True

Heya, thanks for the input.
Indeed, that would do for adding some new members on a limited scale.
Design goal:

Add input validation that gets inherited through
Avoid any changes in handling internally
Eliminate credential prompts on older PowerShell versions, without an eternal chain of if/else
Flexibly convert input of specified types without blanket-accepting everything

So basically, all we want to tell our contributors is Instead of a [PSCredential], make it a [DbaCredential] parameter and the rest will take care of itself.
However - sorry - it really is beside the point for the purpose of this question. Because for me it really is about what is happening here on parameter binding. Maybe it is a bug. Maybe I am missing something. I want to know, in order to further my understanding of how parameter binding works.

Without debugging, I'm guessing the problem is here - LanguagePrimitives.FromObjectAs is essentially the C# as operator, but this method also unwraps a PSObject.
This means your implicit conversion operator will not help, the transformation is converting your value to null.  The fix might be as simple as using LanguagePrimitives.TryConvertTo<PSCredential> instead.

@FriedrichWeinmann: Got it, thanks for the feedback (on my somewhat self-indulgent workaround).
Kudos for a sophisticated approach.
The link to your source doesn't seem to work, but this one does.

Heya @lzybkr ,
thanks for the pointer, though it's more this position that had me confounded. That string wouldn't work was kinda expected, but in the call from line 53 it tries to coerce it into a PSCredential object by casting it - which bloody darned well should be working, as far as I can see.
Just did a short code extraction of it though, in order to test things out:
Add-Type @"
 using System;
 using System.Management.Automation;

 public static class test
 {
     public static PSCredential FromObjectAs(Object castObject)
         {
             PSCredential returnType = default(PSCredential);

             // First, see if we can cast the direct type
             PSObject wrapperObject = castObject as PSObject;
             if (wrapperObject == null)
             {
                 returnType = (PSCredential)castObject;
             }
             // Then, see if it is an PSObject wrapping the object
             else
             {
                 returnType = (PSCredential)wrapperObject.BaseObject;
             }

             return returnType;
         }
 }
 "@
[test]::FromObjectAs($cred)

And it definitely is throwing an invalid cast exception.
I just don't see why ...

@FriedrichWeinmann - in your example, your implicit operator is not called because the static type of the parameter is object. Something like this should work:
   returnType = (PSCredential)(castObject as DbaCredentialParameter)

Of course this can't work in PowerShell because PowerShell won't statically know your type.

argh. Alright and good to know, thanks for the info.
So basically, in order for it to work, I'd have to add that operator to Object itself? Somehow I doubt the system makes that possible ... :(
But thank you very much for explaining that. Doesn't look like there's much I can do then (Still gonna poke around a bit when I've got the time).

Alright, did some poking around:

Cannot extend class to add operators when it's not in my own library
Cannot extend PSCredential, as it is sealed (whyever. I'm sure that made sense to some people, can't for the heck of me see it though)
Cannot undo the credential attribute

Ergo: Mission impossible.
While I'm somewhat disgruntled over the entire parameter binding/type coercion system being ignored for pscredentials, just to force credential prompt (which can be implemented in a different way btw - at least our prompt works just fine as well and we don't mess with parameter attributes), there is little I can do about it. Given that ... well, time to call it a day and spend my time on something with a better effort/success ratio ;)
Cheers,
Fred

Your questioned is answered - but there are changes we can make to the engine to enable your scenario - it won't help with Windows PowerShell, but going forward it might be helpful, right?

Indeed you could and in the long term it would be helpful.
Basically, supporting consistency in the type coercion process would help. It's extremely flexible and powerful, so hardcoding for a specific input type is not helpful as far as I can see it, so yes, I'd appreciate it. I don't really know how much impact 'fixing' this would have compared to the effort to do it - I use parameter classes heavily, but I don't know anybody else who does so - but even then it would make powershell behavior more uniform and consistent.
Btw, I solved the prompt-for-credential issue with a constructor that accepts string. So when you just pass a name, it'll prompt as part of the default coercion process while binding the parameter (and I added a "remember" checkbox, so it'll remember the credential matched to the name for the duration of the process).
The link above is dead, but that's because the branch was merged into development. So if you want to check out our implementation, just download it and run [DbaCredential]"foo" (or build a dummy function to actually have it operate as parameter type).

I believe we could enhance binding to support IConvertible/PSTypeConverter

I believe we could enhance binding to support IConvertible/PSTypeConverter


Bug with jax.disable_jit (or lax.cond, or vmap?)

Attempt at minimal reproducing example:
def fn(t):
  return jax.lax.cond(t > 0, 0, lambda x: 0, 0, lambda x: 1)
fn = jax.vmap(fn)

with jax.disable_jit():
  outputs = fn(jnp.array([1]))

fails with
Encountered value: Traced<ShapedArray(bool[])>with<BatchTrace(level=11/0)>
  with val = DeviceArray([ True], dtype=bool)
       batch_dim = 0

It does not fail without the disable_jit() and it does not fail without the vmap (input a scalar in that case).

Thanks for raising this! This is a known problem, though I forget if there's already an issue for it.
The problem is that:

when in a disable_jit context, the control flow primitives like cond just fall back to a Python implementation motivated by better debugging, yet
this Python implementation isn't vmap-traceable (because in this case vmap abstracts the boolean t > 0 to the Shaped level).

Probably the best option is to run the Python implementation only when bool(pred) can be evaluated at trace time (i.e. when that call doesn't raise an abstract value error), and call into the non-Python version when it fails.


filter out null before replacing module source

Fixes #92.

@mihkeleidast looks like this is blocked from being merged as you need to have signed GitHub commits setup.
Do you wanna want to do that and recreate this PR with a signed commit?

Uhh I have no idea currently how to make that work. Is that the only thing blocking the merge/release of this? Because, by all means, this oneliner fix does not need to be merged by this commit, it can be committed directly if someone has permissions.

TBH am not sure, I stumbled upon this PR from issue #92 you raised as I had the same thing. I tried your fix locally and it worked for me so I was trying to see if this got get added.
@bensampaio would you be able to add this to an upcoming version?

FYI @mihkeleidast you can find out how to set up verified GitHub commits here 👍🏻

Yeah, I know about that guide. I tried it once and failed, probably because I'm coding in WSL2 but using Fork as my Git GUI. Those two use different git executables (I think) and sharing stuff like keys is a hassle.

Closing since it's a dupe of already merged #96.


Fetch bot name automatically

Wobot now fetches the name of the bot automatically (change) and it'd be take advantage of this to make configuration simpler. Unfortunately Hubot core seems very set on knowing the name ahead of time since it builds up the regexes to run on incoming messages (src) at the time scripts are loaded.
Deferring loading of scripts until we've connected would allow this to work but I believe that would require a change in Hubot core.

We could defer loading of scripts until a "connected" event was emitted, or something similar.
/cc @tombell


How to display media type field

I want to display a media type field on a card page, I tried the usual way but the field did not appear.
I can see there is a media id in the db column, but I have no idea how to link it to the actual image.
Please help, thanks in advance!
Book.al
field(6; "Book Cover Picture"; Media)
{
Caption = 'Book Cover Picture';
}
BookCard.al
field("Book Cover Picture"; "Book Cover Picture")
{
ApplicationArea = All;
}

Check standard pages like customer card .You will get an idea.

This repository no longer handles general questions and is only dedicated to bug reports and feature suggestions for AL compiler and developer tools
To get help please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


Create RadiusNeighborsClassifier_MaxAbsScaler_QuantileTransformer.ipynb

Jay Shimpi has named his RadiusNeighborsClassifier with MaxAbsScaler & PowerTransformer file with the same name as this file RadiusNeighborsClassifier_MaxAbsScaler_QuantileTransformer.ipynb and needs to be changed to RadiusNeighborsClassifier_MaxAbsScaler_PowerTransformer

Jay Shimpi has named his RadiusNeighborsClassifier with MaxAbsScaler & PowerTransformer file with the same name as this file RadiusNeighborsClassifier_MaxAbsScaler_QuantileTransformer.ipynb and needs to be changed to RadiusNeighborsClassifier_MaxAbsScaler_PowerTransformer

Thanks for noticing the mistake


CFD-DEM coupling engine

This PR introduces a coupling engine between lethe-dem and the gls_vans solver to create a CFD-DEM solver for multiphase flows. It adds. separate assembler for each particle fluid interactions and introduce an new force defined as the buoyancy.

All comments have been addressed. A particle sedimentation test has been created

@tonielgeitani  are we ready for merge? :)!

Merged! Congrats! Huge chunk of work done in so little time :). Now onwards to validation :)


Update learning/sls-gallery to PSQL 13

Includes some of the commits from #360 to bump the roles to versions supporting PSQL 13+.
Also migrates the postgresql_version under vars as it is transitively used in the om.postgresql_client dependency

On ome-learning, given I had less than 50% space on the /var/lib/pgsql partition, I had to go for a modified version of https://github.com/ome/prod-playbooks/pull/360#issuecomment-1179182594

have a VM snapshot and check the nightly PG dump is presen int
stop the services sudo service omero-web stop && sudo service omero-server stop && sudo service postgresql-10 stop
run the playbook until the completion of the ome.postgresql role to install the new packages/initialize the cluster
as the postgres user, first delete the existing cluster rm -rf /var/lib/pgsql/10 then unzip and restore the dump cd /OMERO/pgbackup && gunzip nightly-omero-Mon.pgdump.gz && psql -f nightly-omero-Mon.pgdump && gzip nightly-omero-Mon.pgdump
run the playbook again until completion to restart all services
cleanup the former packages sudo yum remove postgresql10*

It sounds a little bit like the openstack creation routines which can choose what their source "volume" is. Here it's source "database" which could be local if there's enough space but otherwise is remote and needs to be copied/unzipped into place.

It sounds a little bit like the openstack creation routines which can choose what their source "volume" is. Here it's source "database" which could be local if there's enough space but otherwise is remote and needs to be copied/unzipped into place.

In the back of my mind, I was considering how to perform such an upgrade in the case of IDR where we are at also over 50% usage
[sbesson@prod107-database ~]$ df -h | grep pgsql
/dev/vdb        800G  553G  248G  70% /var/lib/pgsql

A source volume containing a dump of the database might effectively mirror what happens here.


pip-compile doesn't support the new pip resolver

pip-compile doesn't support using the 2020 resolver.
Environment Versions

OS Type macOS
Python version: $ python -V: Python 3.7.8
pip version:  $ pip --version: pip 20.2
pip-tools version: $ pip-compile --version: pip-compile, version 5.3.0

Steps to replicate

Create t.txt:--use-feature=2020-resolver
boto3

Run pip-compile t.txt:

Traceback (most recent call last):
  File ".tox/pip-compile/bin/pip-compile", line 8, in <module>
    sys.exit(cli())
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/scripts/compile.py", line 458, in cli
    results = resolver.resolve(max_rounds=max_rounds)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/resolver.py", line 169, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/resolver.py", line 274, in _resolve_one_round
    their_constraints.extend(self._iter_dependencies(best_match))
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/resolver.py", line 384, in _iter_dependencies
    dependencies = self.repository.get_dependencies(ireq)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/repositories/local.py", line 75, in get_dependencies
    return self.repository.get_dependencies(ireq)
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/repositories/pypi.py", line 232, in get_dependencies
    download_dir, ireq, wheel_cache
  File "/Users/andy/gr/gold/.tox/pip-compile/lib/python3.7/site-packages/piptools/repositories/pypi.py", line 184, in resolve_reqs
    results = resolver._resolve_one(reqset, ireq)
AttributeError: 'Resolver' object has no attribute '_resolve_one'

Expected result
pip-compile completes successfully.
Actual result
pip-compile errors out.

Hello @rouge8,
Thanks for the issue! Yes, currently, pip-tools doesn't support new pip's resolver. I think it makes sense to adopt it after the resolver being stable, so we can use the internal API.

Well pip never considers the internal API stable right? :)
For my purposes I don’t actually need the new resolver logic except to deal with https://github.com/pypa/pip/issues/5780. If you don’t think the new resolver internals are stable enough for pip-tools yet, I’m sure I can come up with some hack to work around this issue. :)

I got a different error (but I suspect related to this issue) and I didn't specify the --use-feature=2020-resolver flag in any file and using pip 20.2.1 :
pip-compile  --output-file=requirements.in ./requirements.in ../dirA/requirements.in ../dirB/requirements.in

Traceback (most recent call last):
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/bin/pip-compile", line 8, in <module>
    sys.exit(cli())
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/scripts/compile.py", line 392, in cli
    results = resolver.resolve(max_rounds=max_rounds)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/resolver.py", line 169, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/resolver.py", line 255, in _resolve_one_round
    constraints = sorted(self.constraints, key=key_from_ireq)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/resolver.py", line 128, in constraints
    self._group_constraints(chain(self.our_constraints, self.their_constraints))
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/resolver.py", line 231, in _group_constraints
    self.repository.get_dependencies(ireq)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/repositories/local.py", line 74, in get_dependencies
    return self.repository.get_dependencies(ireq)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/repositories/pypi.py", line 228, in get_dependencies
    download_dir, ireq, wheel_cache
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/piptools/repositories/pypi.py", line 180, in resolve_reqs
    results = resolver._resolve_one(reqset, ireq)
  File "/home/salotz/tree/lab/devel/wepy/_conda_envs/dev/lib/python3.7/site-packages/pip/_internal/resolution/legacy/resolver.py", line 425, in _resolve_one
    assert req_to_install.user_supplied
AssertionError

@salotz which pip-tools version is used?

apologies, that was 5.1.2, works with 5.3.1 :+1: @atugushev

I'm hitting this error too.
The new resolver is going to be the default from middle or end of October if I understand correctly.
https://github.com/pypa/pip/issues/8937
(This will be very exciting)
It probably makes sense to ensure pip-tools works with both --use-feature=2020-resolver and --use-deprecated-feature=legacy-resolver

pypa/pip#988 (comment):

We are aiming to release pip 20.3 about a week from now, on Wednesday or Thursday, Oct 28 or 29. We are preparing to change the default dependency resolution behavior and make the new resolver the default in pip 20.3.

@atugushev @graingert How much do you already know about what needs to be done here in pip-tools?

This issue is critical as the new resolver is days away from becoming default. If you was one of those that added use-feature = 2020-resolver to your  ~/.config/pip/pip.conf in order to benefit from extra conflict resolution in newer resolver you are unable to use pip-compile at all.

So weirdly the 5.4 release seems to have fixed this for Python 3. But it still breaks for Python 2 (I know, I know). Can someone confirm this behavior, or is it just me?

@hynek pip-tools==5.4.0 adds only pip==20.3 support. I've tried locally py27, but works fine. Could you create an issue with a traceback?

I…uh…played a bit with the settings hence and forth and suddenly it started working and I can't break it anymore. 😳 It was 100% 5.4.0/20.3 before. I guess my computer is just haunted.

Well pip never considers the internal API stable right? :)

Can't we just use resolvelib directly instead of the pip's vendored copy? This way pip-tools wouldn't need to rely on the internal pip's resolver wrapper but use it directly. It's API is quite simple (I'm in the progress of integrating resolvelib into ansible-galaxy CLI, for example) — using it would mean having a stable public API to rely on but the behavior may differ from what a given version of pip does, of course.

This bus is still a big PITA because even latest version will choke if you happen to define:
export PIP_USE_FEATURE=2020-resolver

This used to be a popular method to enable the new resolver before it was released, so we would detect issues at CI level.

I believe you can work around that for now using pip check?

Well,.. I removed that line from that CI pipeline that was running pip-compile, but having to do this on multiple projects is no joy. We should still  have to fix it. Sadly lots of people interested but nobody willing to do it.

Is anyone working on this? Thinking about taking a look but don't want to duplicate work.

This should actually be working now. I think this issue could be close.
Which version of pip are you seeing errors with? I use pip==21.0.1 (new resolver enabled by default) with the latest pip-compile and things work correctly.

okay well I have something that works with pip install but not pip compile. I'll see if I can boil it down to the base case.

I have simplest example:
req.txt:
asyncpg ~= 0.15.0
asyncpgsa >= 0.22.0

pip-compile fails with:
Could not find a version that matches asyncpg>=0.22.0,~=0.15.0 (from -r req.txt (line 1))
Tried: 0.5.0, 0.5.1, 0.5.2, 0.5.3, 0.5.4, 0.6.1, 0.6.3, 0.7.0, 0.8.0, 0.8.1, 0.8.2, 0.8.3, 0.8.4, 0.9.0, 0.10.0, 0.10.1, 0.11.0, 0.12.0, 0.13.0, 0.14.0, 0.15.0, 0.16.0, 0.17.0, 0.18.0, 0.18.1, 0.18.2, 0.18.3, 0.19.0, 0.20.0, 0.20.0, 0.20.1, 0.20.1, 0.21.0, 0.21.0, 0.22.0, 0.22.0
There are incompatible versions in the resolved dependencies:
  asyncpg~=0.15.0 (from -r req.txt (line 1))
  asyncpg>=0.22.0 (from asyncpgsa==0.27.1->-r req.txt (line 2))

but pip install succeed with:
...
Successfully installed asyncpg-0.15.0 asyncpgsa-0.27.0 greenlet-1.0.0 sqlalchemy-1.4.6

Tested on both pypi and github-master versions of pip-tools. Latest pip (since pip-tools required it).
If I change req.txt as:
asyncpg ~= 0.15.0
asyncpgsa >= 0.22.0, < 0.27.1

Then I get pip-compile succeed with:
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile req.txt
#
asyncpg==0.15.0
    # via
    #   -r req.txt
    #   asyncpgsa
asyncpgsa==0.27.0
    # via -r req.txt
greenlet==1.0.0
    # via sqlalchemy
sqlalchemy==1.4.6
    # via asyncpgsa

confirmed pip-compile can't handle,
boto3>=1.15.8
botocore==1.20.20

but valid solution is
boto3==1.17.20
botocore==1.20.20

So sounds like no ones working on this?

Hi, I'm a maintainer at https://github.com/pantsbuild/pants. We're working on a revamp of our 3rd-party support with lockfiles at the crux of the changes. We've been using pip-compile so far and are really happy with it so far, great UX!
This issue is a blocker for us to call the project complete. We're interested in taking on this issue, but are trying to scope it out: how much work would you anticipate this change will take?
Would anyone be available to help, like giving some tips or pair programming?
--
Also, Pants still supports Python 2.7, so we'd ideally backport this feature to pip-tools 5, in addition to pip-tools 6. How feasible do you expect a backport like that would be?

@Eric-Arellano there's a new lock file spec here too https://www.python.org/dev/peps/pep-0665/

Ohh yeah, it only took me to read the header to guess what this pep will propose: more TOML :( ... few pages later, I got my guess confirmed.

#1539 adds support for new resolver and fixes the issue:

Details
❯ pip-compile --resolver backtracking
#
# This file is autogenerated by pip-compile with python 3.8
# To update, run:
#
#    pip-compile --resolver=backtracking
#
boto3==1.21.4
    # via -r requirements.in
botocore==1.24.5
    # via
    #   boto3
    #   s3transfer
jmespath==0.10.0
    # via
    #   boto3
    #   botocore
python-dateutil==2.8.2
    # via botocore
s3transfer==0.5.1
    # via boto3
six==1.16.0
    # via python-dateutil
urllib3==1.26.8
    # via botocore

The resolver is released as part of pip-tools v6.8.0 🎉

@atugushev  I can't say how much this saved the day for me - thank you for getting this in there when you did!


use new cargo [lints] section

this significantly reduces copy-paste and source of errors to have project wide linting rules

done via db1d46b


fixed RangeError: Maximum call stack size exceeded.

Fixed RangeError: Maximum call stack size exceeded when dragging an element over a static element causes.
The point of the solution is that when we drag an element over a static one, a collision occurs and we have to interrupt the operation.
Fixes #941

Thanks for the contribution!


models-only produces broken output

Describe the bug
I only export types because I use nuxt's useFetch() for the client, unfortunately restricting the output with this fork is broken.
--exportCore false is ignored, core/ is always generated and imported
--exportSchemas false is ignored, schemas.ts is always generated and imported
--exportService false is partially respected, but we get a blank services/index.ts imported from index.ts breaking the builds.
Severity
serious, but I can work around it
Additional information
@hey-api/openapi-ts 0.31.1
node v18.19.1
ubuntu 22.04.4 amd64

@jordanshatford i think this is related to https://github.com/hey-api/openapi-ts/issues/155
Separate issue are those partially exported files

@amery i think you'll get a different result if you use the config file, this seems to be related to the CLI options parsing

@amery can you share an example call using useFetch()? Curious how you use models with it

@amery this is fixed in v0.32.0, thanks for catching! 🤝

I think we will want to split the type generator into a separate package at some point btw

@amery this is fixed in v0.32.0, thanks for catching! 🤝

thank you for fixing it so quickly!

I'm still curious how you use these models with useFetch()?

I am fairly new to the typescript and reactive world, but I have a types package that exports everything from models and some extra details, then I have a composable wrapping useFetch() injecting content-type and the authentication token like this:
export function useAPI<T>(url: string, opts: UseFetchOptions<T> = {}) {
  const { public: config } = useRuntimeConfig();
  const Authorization = useAuth().tokenStrategy?.token?.get();

  // headers
  const headers = opts.headers || {};
  if (typeof Authorization === 'string') {
    opts.headers = {
      Authorization,
      ...headers,
    };
  }

  const defaults: typeof opts = {
    baseURL: config.apiPrefix,
    key: url,
    server: false,
    headers: {
      Accept: 'application/json',
    },

    onRequest(ctx) {
      useLoadingIndicator().finish();
      return ctx;
    },

    onResponseError({ response }) {
      useLoadingIndicator().finish();

      if (response.status !== 401) {
        throw createError({
          statusCode: response.status,
          statusMessage: response.statusText,
          stack: response._data?.stack,
        });
      }

      redirectToLogin();
    },
  };

  const params = defu(opts, defaults);
  useLoadingIndicator().start();
  return useFetch(url, params);

then per-service composables using it
export function useFunctionTypes(opts: UseFetchOptions<FunctionTypeInfo> = {}) {
  return useAPI<FunctionTypeInfo>('/config/function/types', opts);
}

and finally on .vue files things like
const { data, error, pending, refresh } = await useFunctionsConfig({
  transform: (data):TransformedFunctionCollection => asTransformedFunctionCollection(data),
});

or the same within a pinia store

Thanks for the additional context @amery. Do you use this wrapper to make POST/PUT requests too?
We'd eventually want to offer this kind of wrappers for clients such as Nuxt too, so this is helpful. I see for example that your URLs aren't type-safe, we could improve it a bit. In the meantime, let me know if there's anything we can do to make your life easier!

Thanks for the additional context @amery. Do you use this wrapper to make POST/PUT requests too?

Yes

We'd eventually want to offer this kind of wrappers for clients such as Nuxt too, so this is helpful. I see for example that your URLs aren't type-safe, we could improve it a bit. In the meantime, let me know if there's anything we can do to make your life easier!

Generating the 3-line wrappers would be enough to save me some pain and reduce simple mistakes.
I'll be happy to test such experiments, but my typescript is still too weak to be able to get involved in the code itself yet.


Bug in Non-Linear SVM Kernel Calculation

I think there's a bug in the calculation of the Gaussian Kernel for the multiclass SVM. Your code is as follows:
# Gaussian (RBF) kernel
gamma = tf.constant(-10.0)
dist = tf.reduce_sum(tf.square(x_data), 1)
dist = tf.reshape(dist, [-1,1])
sq_dists = tf.multiply(2., tf.matmul(x_data, tf.transpose(x_data)))
my_kernel = tf.exp(tf.multiply(gamma, tf.abs(sq_dists)))

Initially it bothered me that the variable dist was calculated but never used. I then tried to figure out what you were doing in this code, and I think it is supposed to be doing something like this (dist is K in the below example)

In which case, the code should read something more like:
# Gaussian (RBF) kernel
gamma = tf.constant(-10.0)
K = tf.reduce_sum(tf.square(x_data), 1)
K_cols = tf.expand_dims(K, 1)
K_rows = tf.expand_dims(K, 0)

D = K_cols + K_rows - 2*tf.matmul(x_data, tf.transpose(x_data))
my_kernel = tf.exp(tf.multiply(gamma, D))

(The sq_dists calculation uses broadcasting so we don't need to create the J matrix as in the math above)

HI @fbcotter ,  Thanks for this!  I'm just getting around to triaging issues in preparation for a v2-update of the code.
I suspect you are correct.  When I get around to chapter 4, I'll investigate and most likely accept your proposal.  Expect more in the next month or so.  Thanks again!

Yes, thanks. I just looked into that and found you are correct.  I've implemented a very similar fix and kept the 'dist' variable terminology in both the *.py and the jupyter notebook.  Thanks again!


Expose gps.sourceGateway, and a method on gps.SourceManager for getting them

I'm undecided on whether this is a good idea, but I'm leaning towards 👍 , so posting it.
The SourceManager currently hides away access to all of the underlying sourceGateway objects to which it delegates responsibility for running requests. We could continue with this model, but it might also be nice to simply make those objects available directly for client interaction, and allow SourceManager to keep a skinnier interface.
I swear I had a concrete use case a second ago, though it's eluding me now. If we can't come up with one, we definitely shouldn't do this, it has complexity costs.
The main challenge this would pose is object lifetime. Once sourceGateway objects have escaped the SourceManager, we have no control over their lifetime. Because the resources they manage are still being orchestrated by the SourceManager, but we can no longer directly couple that to object lifetime, we'd have to introduce and manage a killswitch inside the source objects - allow them to be terminated centrally by the SourceManager, causing all subsequent calls to return errors.
We already do some of this, of course, but distributing that responsibility across multiple objects necessarily makes the task more complex.

goroutine 1 [running]:
github.com/golang/dep/internal/gps.(*metrics).push(0x0, 0x8a0627, 0xb)
        E:/gocode/src/github.com/golang/dep/internal/gps/metrics.go:33 +0x3e
github.com/golang/dep/internal/gps.versionUnifier.intersect(0xa83ec0, 0xc0426be3c0, 0x0, 0xc0423d29a0, 0x1a, 0x0, 0x0, 0xa829a0, 0xc0422baaa0, 0xa82b20, ...)
        E:/gocode/src/github.com/golang/dep/internal/gps/version_unifier.go:119 +0xcf
github.com/golang/dep/internal/gps.(*selection).getConstraint(0xc0424cd200, 0xc0423d29a0, 0x1a, 0x0, 0x0, 0xc0424e8801, 0x49aed9)
        E:/gocode/src/github.com/golang/dep/internal/gps/selection.go:131 +0x1aa
github.com/golang/dep/internal/gps.(*solver).getLockVersionIfValid(0xc042258d80, 0xc0423d29a0, 0x1a, 0x0, 0x0, 0x1, 0x0, 0x0, 0x1)
        E:/gocode/src/github.com/golang/dep/internal/gps/solver.go:925 +0x1b3
github.com/golang/dep/internal/gps.(*solver).createVersionQueue(0xc042258d80, 0xc0423d29a0, 0x1a, 0x0, 0x0, 0xc04247e210, 0x1, 0x1, 0x0, 0x0, ...)
        E:/gocode/src/github.com/golang/dep/internal/gps/solver.go:745 +0xcf7
github.com/golang/dep/internal/gps.(*solver).solve(0xc042258d80, 0x0, 0x0, 0xc0426be330)
        E:/gocode/src/github.com/golang/dep/internal/gps/solver.go:449 +0x478
github.com/golang/dep/internal/gps.(*solver).Solve(0xc042258d80, 0x12, 0xa7ec80, 0xacd188, 0xc04211bb4e)
        E:/gocode/src/github.com/golang/dep/internal/gps/solver.go:399 +0xb2
main.(*initCommand).Run(0xc0421473da, 0xc042127900, 0xc0420583e0, 0x0, 0x0, 0x0, 0x0)
        E:/gocode/src/github.com/golang/dep/cmd/dep/init.go:185 +0xe8b
main.(*Config).Run(0xc042041260, 0xc042041260)
        E:/gocode/src/github.com/golang/dep/cmd/dep/main.go:160 +0x89f
main.main()
        E:/gocode/src/github.com/golang/dep/cmd/dep/main.go:44 +0x25a


Property "..." does not exist on value ot type "....."

I am using this framework and ran into these errors. Not sure what cause:
error TS2094: The property 'registerMediator' does not exist on value of type '() => puremvc.IFacade'
error TS2094: The property 'retrieveProxy' does not exist on value of type '() => puremvc.IFacade'
error TS2094: The property 'registerProxy' does not exist on value of type '() => puremvc.IFacade'
Any idea on how to fix these error? By the way, the d.ts file is kinda out of date too.

Published a brand new update. This should have fixed the issue.