Fixing README, eip on outside interface, existing vpc route table control

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
I'm raising this issue to discuss about the issues below. Some are feature requests and some are defects. I'm also mentioning the probable solution for each of these points in the relevant section. In parallel to this discussion, I've already forked this repo to fix the below points based on the solution expected to the best of my knowledge and ability
Defects

Typo in README files
Few existing inputs and outputs were not documented
When creating FTD and FMC with existing subnets, new route tables getting created and existing subnet associations are getting removed from main routing table
FTD user data is not populated with FMC mgmt IP thus 'show managers' on FTD doesn't show the FMC as manager
usage of deprecated argument 'vpc' for resource 'aws_eip'

Feature suggestions

adding an option to associate an eip with outside interface. This is so that internal hosts can reach internet through outside interface

New or Affected Resource(s):
Affected

data "template_file" "ftd_startup_file"
resource "aws_route_table" "ftd_outside_route"
resource "aws_route_table" "ftd_inside_route"
resource "aws_route_table" "ftd_diag_route"
resource "aws_route_table_association" "inside_association"
resource "aws_route_table_association" "diag_association"
resource "aws_eip" "ftd_mgmt_eip"
resource "aws_eip" "fmcmgmt_eip"

New

resource "aws_eip" "ftd_outside_eip"
resource "aws_eip_association" "ftd_outside_ip_assocation"

Describe the solution expected:
The typos in Readme files should be fixed. Also, the missed and new objects needs to be added in the Readme files.
For the points,

When creating FTD and FMC with existing subnets, new route tables getting created and existing subnet associations are getting removed from main routing table
Solution expected: When FMC and FTD are deployed using existing VPC, subnets and igw, the associations of the subnets should not change from the main routing table and also the any form of changes to those subnets would also affect other instances using the same subnets. One way of controlling this is behaviours is the usage of existing variable rta

FTD user data is not populated with FMC mgmt IP thus 'show managers' on FTD doesn't show the FMC as manager
Solution expected: For FTD to populate the correct FMC IP, FMC needs to be first deployed.

usage of deprecated argument 'vpc' for resource 'aws_eip'
Solution expected: The argument 'domain' should be used.

For the below feature suggestion

adding an option to associate an eip with outside interface. This is so that internal hosts can reach internet through outside interface
Solution expected: There should be variable introduced to control whether to associate any eip to the outside interface.

References

#0000

Fix for all the above mention defects and enhancements can be found at https://github.com/CiscoDevNet/terraform-aws-secure-firewall/pull/3

@Maintainers, can someone please comment on this issue about your thoughts


Help chat

Pull Request Description
Closes #6610
Important Notes

Should be tested alongside https://github.com/enso-org/enso-bot/pull/1.
Unread messages notification has not been implemented

Screencasts
screen-recorder-mon-jul-03-2023-20-29-41.webm
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

server implementation is linked in the original post. can be easily moved into this repo, but thinking about it it might make deploy on commit a bit harder unless the app hosting platform we use supports sparse checkouts... or actually we could use CI to build to a separate repo. thoughts?
design not identical to figma - whoops, forgot to bring this up. I don't think i have a link to the figma, and the screenshot seems like it's scaled, likely because of retina displays, so its a bit difficult for me to get the right spacings (etc)
the discord messages land in a test server, yes - see the linked PR about the server impl for details
deduplication: currently the duplicated code is across two different repos, and that's the part making it hard to deduplicate. if you have any ideas that'd be great
repeated code: fair point, originally i refactored it to this to make it consistent with the backend, but yeah the constant can be copied to the backend instead. it won't actually be used in the backend but itll be consistent, and there will be no repetition

400 line long function: I'll see whst i can do

@somebody1234 oh, sorry. Keeping it in a separate repo makes sense. I wanted to tell "this organization", not necessarily this repo, sorry for confusion. I totally missed that link!
Link to Figma: https://www.figma.com/file/YdrxRRNoZWqePEJQye80YG/ide2-v49.8?type=design&node-id=0%3A1&mode=design&t=1qh6VVj0wKvvfddF-1
Regarding duplicated code across repos – hmmmm. What about making that repo an NPM library? Just an idea, probably not best, I'm asking about your opinion here to chat about possibilities.
Regarding long functions - I'd be thankful if you could try splitting them down, it would make the code way more maintainable long term :)

re: using a npm library - it would probably work, but upon some investigation it seems like it's possible to use git dependencies, which is probably more convenient, since it wouldn't require publishing new tags for local development
also, it's not like it should be publicly visible on npm - not that it can't be, but i do think what determines whether something belongs on npm should be an intentional decision, rather than "we have to put it on npm for things to work" - when there are other options, at least

Oh, I did not know that nom supports git dependencies. This is way better solution, let's use it! :)

@somebody1234

The arrow in threads list is not animated. Should be toggled (down for rolled up; up for expanded)
The emoji menu should be visible only on hover.
Emoji ... at the end with full emoji menu? This one please estimate if its doable at all and how hard would it be.
Creating new thread should append +x to the default name New chat thread 1. Right now renaming threads doesn't work on mac and I can follow only latest thread. Previously created are gone.
4.a Weirdly threads appeared after restarting enso but creating new ones do not appended them to thread list
No highlighting when moving mouse over the thread list. The current opened should be highlighted but also the one I have my pointer over
Thread renaming. When you hover your mouse over the thread name the mouse pointer should be changed to text ⵊ and after click name should be changed to input widget with save on enter
Also on thread list we should have 1 additional icon on the left (if its doable):

trash for delete

With expanded thread list clicking outside it does not roll it up
padding left / right should be the same as top and down for "New question?" and "Reply"

Bug with opened workspace:

I also have double cursor pointers when working as above (can't show on screenshot)
There is no help chat on workspace view (needed to go back to dashboard and leave opened to have it on project view). I guess this one is not doable without either having it in rust or having header in react right?

done locally
not sure how this should work
sounds reasonable, naively it should just be a scrolling div with all the emoji - so like the current reaction bar, just with more emojis. one problem though, would be whether it's needed - i'd imagine some (or many) emojis should be omitted, either because they aren't professional (e.g. the vomiting emoji), or because they are rarely useful (e.g. the regional indicator emojis (🇦-🇿))
will do
done locally
done locally
tbd
done locally
done locally (visually). they actually were the same size - it's because of the rounded corners that it looks like they're different sizes
done locally (haven't tested yet)
will investigate
yeah. i think the simplest way is to do things the same way as the "back to ide" event - have rust implement the ui to open it, and then use events to send information back and forth

still having issues that threads are not appending to the list but only current is visible. now even reopening doesn't help
https://github.com/enso-org/enso/assets/2855109/0dc53011-32de-40e4-9f44-8255c6d0feaf

will do tomorrow
on it. seems like it's harder than i expected... i'm thinking of using http + polling, because i'd imagine most users would rarely (if ever) use the help chat

i just realized, i don't think this works with multiple users in an organization. i'll have to take a look at that sometime

i just realized, i don't think this works with multiple users in an organization. i'll have to take a look at that sometime

this one is not obvious. lets fix conflicts and open for review and merge (deploy) this implementation then we see


Issue with base href and nv-point-path

I'm trying to get the linePlusBarChart working, but I'm running into some issues with the nv-point-path layer. It ends up being all black and shattered (like a broken phone screen). If I delete that layer I can see the lines and bars, but mouse over on the lines doesn't work.
I bet this is caused by base href tag in my index.html screwing up clip-path="url(#nv-clip-20801-0)", because it's looking for that definition in base instead of this file.
I've encountered this before with some d3 stuff in angular2 and fixed it by using absolute urls. I can't do that here, so I'm looking for a cleaner solution. Any ideas on how I can deal with this?

I think this is the solution: http://stackoverflow.com/questions/19742805/angular-and-svg-filters

First screen shot is the graph. Second is after I delete nv-point-path. I'm using the example data and code.

Are you going to take a look at this or am I on my own? I can try to fix it and do a pull request, but this will probably have a lot of side effects.

@drnknmstrr, did you include css in you project? Please, create a plunker demo with your issue, based on existing demo.

Doh. That was totally it. I didn't realize there was .css in there. Thank you.

Nm, I see a few things that still aren't quite right. I will try to reproduce them in plunker.

@krispo check out https://plnkr.co/edit/9kkjqJ2NlsbVSLXCn1Nl?p=preview
When I include the base tag the bars do not highlight on mouseover. Not as bad as it was before, but I'd like to get that working.


fix XLOG_MULTIXACT_ZERO_MEM_PAGE wal parsing

closes #453

Strange error in test_pageserver_restart.
In pageserver logs:
Aug 31 09:10:21.514 ERRO get_page_at_lsn: no base img found for 1663/13974/2662.0 at blk 0 at LSN 0/2763E78, location: pageserver::page_service, pageserver/src/page_service.rs:313

In pg logs:
2021-08-31 09:10:21.510 GMT [2033] LOG:  [ZENITH_SMGR] libpqpagestore: connected to 'host=localhost port=64000 password='
2021-08-31 09:10:21.514 GMT [2033] FATAL:  could not read block 0 in rel 1663/13974/2662.0 from page server at lsn 0/02763E78

Error happened only in debug, release test succeeded
@hlinnaka is it a known bug or something new?

Strange error in test_pageserver_restart.
In pageserver logs:
Aug 31 09:10:21.514 ERRO get_page_at_lsn: no base img found for 1663/13974/2662.0 at blk 0 at LSN 0/2763E78, location: pageserver::page_service, pageserver/src/page_service.rs:313

In pg logs:
2021-08-31 09:10:21.510 GMT [2033] LOG:  [ZENITH_SMGR] libpqpagestore: connected to 'host=localhost port=64000 password='
2021-08-31 09:10:21.514 GMT [2033] FATAL:  could not read block 0 in rel 1663/13974/2662.0 from page server at lsn 0/02763E78

Error happened only in debug, release test succeeded
@hlinnaka is it a known bug or something new?

@patins noted that last night, too. It's happened a few times in the CI on 'main'. He suspected commit 7c7e89e2ea6e2f75704b7c222704f76f115d8176 as the culprit, because it didn't happen before that, although it's hard to tell because it doesn't happen very often.
So yeah, not this PRs fault. But if you have any suggestions on what might be causing it or how to trace it down, I'm all ears

Created #500 to track this bug, currently I have no idea on why this can happen, probably enabling trace logs and trying to get a repro is the best we can do now


Add support for ReactivePulsarListener annotation

Wow! This looks very thorough. It is a large PR, though. I glanced through the code, looks good, will do a detailed review later. I see the places where you try to combine with the imperative model. I will let @onobc take a look as well. Thanks for this great work.


Date Input has incorrect sizing in Safari

In Safari 14.1 (on macOS Big Sur) the following will render 'incorrectly' on Safari - the date input element is over-sized.
The issue isn't related to having another column or another field - but having a regular text input helps to highlight the sizing bug.
FF seems to render it correctly.

    <input type="text" class="form-control" placeholder="First name" aria-label="First name">

    <input type="date" class="form-control"">

Made a reduced test case on CodePen so I can confirm with BrowserStack:

I'm unable to find out what's happening here, for now.

I did some digging before filing the issue, it's quite bizarre (but CSS isn't really my regular gig). The shadow DOM for browser-defined parts of the element don't show as having the extra space (although they do have white-space: pre rules) - there's just this mysterious extra spacing below it.
Screenshots from Safari inspector showing the details for the input and the root div inside the shadow DOM. Let me know if there's any more info/tests/debugging I can help with from a 'real' safari.
The input itself:

The shadow dom root div:

Updated demo on codepen (previous one has a broken CSS link)
https://codepen.io/stephenreay/pen/ExbWoRz

Doing some digging, the trigger for this is the display: block rule on the input element.
I've filed a webkit bug about this: https://bugs.webkit.org/show_bug.cgi?id=236352

This one is super rough because the display: block makes it worse, but it's not the only issue at play. Plus, the styles are inconsistent between Chrome and Safari, so it's unclear how we could make this 100% consistent between browsers. We even had issues in the past when using a fixed height.

This can be fixed from the webkit pseudo-element ::-webkit-datetime-edit
input::-webkit-datetime-edit {
  display: block;
  padding: 0;
}

This can be fixed from the webkit pseudo-element ::-webkit-datetime-edit
input::-webkit-datetime-edit {
  display: block;
  padding: 0;
}

Now, this worked like magic! Thank you so much, Safari was really ruining my day.


Hugo Addobbati

Hey @hugoaddobbati thnx for contributing , Contribute in this year's Hacktoberfest in October 2020 and follow me on Github and Star this repo for updates and notifications about the project.


Segmentation fault: 11th nathantreid:css...

Hey,
when I add the following to my package.json, my Meteor app exists with the error above:
"cssModules": {
"extensions": ["scss"]
 }

App crashes:
Segmentation fault: 11th nathantreid:css...

Was a configuration error


Update to reduce Spamming in Hacktoberfest2020

Update by Digital Ocean
We will honor all valid pull requests prior to this change, and as of October 3, 2020 at 12:00:00 UTC – and October 3 in all time zones – pull requests will only count toward earning a T-shirt or planting a tree if they are submitted in a repository classified with the ‘hacktoberfest’ topic. The pull requests will also need to be merged, approved by a maintainer, or labeled as ‘hacktoberfest-accepted’ in order to qualify. The deadline for completions, merging, labeling, and approving is November 1.
To put it in terms that folks will more easily understand:
PRs count if:
Submitted in a repo with the hacktoberfest topic AND
during the month of October AND (
The PR is merged OR
The PR is labelled as hacktoberfest-accepted by a maintainer OR
The PR has been approved
)

Add hacktoberfest as a topic due to new update 🎇in policy


[11.0][PORT] base import map port

Port base_import_map to v11. Functionally tested.
P.S: Thanks guys for a great module!

@Railia check functional part please


Add assimp

Fix https://github.com/conda-forge/idyntree-feedstock/issues/14 .
Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


feat(topsql): detail

[x] Top SQL Detail
[ ] Fix fluent-ui DetailList Selection bug

Should merge https://github.com/pingcap/tidb-dashboard/pull/1092 first.
only one plan:

multi plans:

I have a question: What's the relationship with this PR and https://github.com/pingcap/tidb-dashboard/pull/1092? Is https://github.com/pingcap/tidb-dashboard/pull/1092 implementing something not complete?

I think you can refer to other pages like StatementDetail, ProfilingDetail to see how these Titles and Tables are presented. They have all been encapsulated into reusable components.

Cause

#1092 are ready to review, but it's better to review #1091 first.

I think you can refer to other pages like StatementDetail, ProfilingDetail to see how these standard Titles and Tables (e.g. Tables with extended margin) are presented. They have all been encapsulated into reusable components.

Oh, right. I missed that.

#1092 are ready to review, but it's better to review #1091 first.

Do we need to review #1092 first in order to review this PR? Is #1092 providing a partial page implementation?

#1092 are ready to review, but it's better to review #1091 first.

Do we need to review #1092 first in order to review this PR? Is #1092 providing a partial page implementation? If the implementation o #1092 is actually incomplete then maybe you need to fine some other way to split PRs. Split PRs doesn't mean to bring something broken into the master branch. Instead, you need to divide your works logically, and always ensure that the master branch is working at any time, by not breaking anything exist, and not introducing any new draft. For example, proposing a new component is complete, but proposing a new component + a page that is actually not useable, it is not a good PR splitting. You will also find it easier when PRs are divided logically. In the case of the example above, you can continuously improve the component, making it more and more perfect in one PR. When this is finished, the improvements can be introduced naturally into the PR that based on it.

Yes, we can first review #1091 for the ngm client. Then #1092 for top sql chart & table, and this PR is the detail part.
Each PR can run independently base on last PR and won't break the master.

/cc @baurine @breeswish
PTAL, thanks!

Top SQL switch option.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@84fb1f0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1099   +/-   ##
=========================================
  Coverage          ?   24.00%
=========================================
  Files             ?       86
  Lines             ?     9064
  Branches          ?        0
=========================================
  Hits              ?     2176
  Misses            ?     6749
  Partials          ?      139

Flag
Coverage Δ

be_unit_test
24.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

e2e is failed. Please check whether log search is working according to the failed test case, or if there are layout changes, please update the e2e test case.


Migrating from v1 to v2

CDN URL CHANGED
<script src="https://cdn.jsdelivr.net/npm/litepicker/dist/litepicker.js"></script>

No CSS version:
<script src="https://cdn.jsdelivr.net/npm/litepicker/dist/nocss/litepicker.js"></script>

Be careful, these links will return the latest version.
To avoid breaking changes, you need to install a specific version.
See jsdelivr docs.

In v2, these the following options methods and all events has been removed:

Options bookedDays , bookedDaysFormat, bookedDaysInclusivity, disallowBookedDaysInRange - Since it duplicates lockDays* options.
Option anyBookedDaysAsCheckout - Since v2.0.0 added option lockDaysFilter, we can use like this:

lockDaysFilter: (date1, date2, pickedDates) => {
   if (pickedDates.length < 1) {
     // this example will disable weekends
     // define your own condition for indicate locked days
     const d = date1.getDay();
     return [6, 0].includes(d);
   }

   return false;
},

Option disableWeekends - Since v2.0.0 added option lockDaysFilter, we can use like this:

lockDaysFilter: (day) => {
   const d = day.getDay();

   return [6, 0].includes(d);
},

Option hotelMode - The option hotelMode is misleading because there many cases for each hotel. The replacement is use option lockDaysFilter as a function and control the behavior you want.
Option moveByOneMonth - Since v2.0.0 added new option switchingMonths. New option allow number, so you can set 1 or 2 or whatever you want.
Options resetBtnCallback, useResetBtn - Since v2.0.0 added new option resetButton.

...
resetButton: true,
...
// OR
...
// Do not need to call clearSelection inside this function.
// function should return HTML element
resetButton: () => {
   let btn = document.createElement('button');
   btn.innerText = 'Clear';
   btn.addEventListener('click', (evt) => {
     evt.preventDefault();

     // some custom action
   });

   return btn;
},
...

All events (onShow, onHide, onSelect, onError, onRender, onRenderDay, onChangeMonth, onChangeYear, onDayHover, onShowTooltip) - Replacement is event emitter (see below).
Method setBookedDays(array) - Since v2.0.0 there is no option bookedDays.

These options has been updated:

Option format - Since v2.0.0 option format support external library for parse/output.
Example with luxon:

...
format: {
  // parse function should return Date object
  // date - Date object or string (perhaps there will be more types, need to check)
  parse(date) {
    if (date instanceof Date) {
      return date;
    }

    if (typeof date === 'number') {
      return new Date(date);
    }

    if (typeof date === 'string') {
      return luxon.DateTime.fromFormat(date, 'yyyy LLL dd').toJSDate();
    }

    return new Date();
  },

  // date - Date object
  // output function should return string
  output(date) {
    return luxon.DateTime.fromJSDate(date)
                        .toFormat('yyyy LLL dd');
  }
}
...

These options, events and methods has been added:

Option resetButton - Replacement for resetBtnCallback and useResetBtn.
Option switchingMonths - Replacement for moveByOneMonth.
Option lockDaysFilter - Mark locked days with custom function.

lockDays: (date1, date2, pickedDates) => {
  // define your condition
  //
  // date1 - start date or day of render ([DateTime](https://github.com/wakirin/Litepicker/blob/master/src/datetime.ts))
  // date2 - end date (DateTime)
  // pickedDates - number of selected days (Number)
  //
  // this function calling on render and after apply
  // thus, you need to check pickedDates.length, if it is:
  // 0 - no dates selected
  // 1 - selected start date
  // 2 - selected start and end dates

  // function should return Boolean, when true - day locked
}

Option tooltipNumber - Allow to correct visible tooltip number.

...
// days - number of selected days
// function should return Number
//
// shows one day less
tooltipNumber: (totalDays) => {
   return totalDays - 1;
}
...

Method DateTime - return DateTime object.

const picker = new Litepicker({ ... });

picker.DateTime(); // return DateTime object
// or
picker.DateTime(someDateObject); // also return DateTime object based on provided Date object

Replacement for events.
Since v2 all events work with EventEmitter.
Example initialize:

...
setup: (picker) => {
   picker.on('show', () => {
     // some action after show
   });

   picker.on('render', (ui) => {
     // some action after render
     // ui is root element of picker
   });
},
...

Event list:

.on('init', () => { ... }); - Event is called after initialization picker.
.on('before:render', (ui) => { ... }); - ui is root element of picker. Event is called before render function.
.on('render', (ui) => { ... }); - ui is root element of picker. Event is called after render function.
.on('render:month', (month, date) => { ... }); - month is element of rendered month. Event is called after renderMonth function.
.on('render:day', (day, date) => { ... }); - day is element of rendered day. Event is called after renderDay function.
.on('render:footer', (footer) => { ... }); - footer is element of rendered footer. Event is called after renderFooter function.
.on('change:month', (date, calendarIdx) => { ... }); - date is DateTime object of month, calendarIdx is index of calendar when splitView is enabled. Event is called after change month.
.on('change:year', (date, calendarIdx) => { ... }); - date is DateTime object of year, calendarIdx is index of calendar when splitView is enabled. Event is called after change year.
.on('error:range', () => { ... }); - Event is called on wrong selection range (when disallowLockDaysInRange is enabled).
.on('preselect', (date1, date2) => { ... }); - date1, date2 is DateTime object. Event is called on select days (before submit selection).
.on('selected', (date1, date2) => { ... }); - date1, date2 is DateTime object. Event is called when selection is submitted.
.on('button:cancel', () => { ... }); - Event is called after click on button cancel (footer).
.on('button:apply', (date1, date2) => { ... }); - date1, date2 is DateTime object. Event is called after click on button apply (footer).
.on('tooltip', (tooltip, day) => { ... }); - tooltip is element of tooltip, day is element of hovered day. Event is called after showTooltip function.
.on('before:show', (el) => { ... }); - el is triggered element. Event is called before show.
.on('show', (el) => { ... }); - el is triggered element. Event is called after show.
.on('destroy', () => { ... }); - Event is called after destroy.

Added plugins.
See examples here: https://github.com/wakirin/Litepicker/tree/master/src/plugins
Documentation v2

Is there a way to add custom classes to the elements in the new version? Would be useful with TailwindCSS

@arpowers you can use these events:
render
render:day
render:month
new Litepicker({
  element: document.getElementById('datepicker'),
  setup: (picker) => {
    picker.on('render:day', (day) => {
      // add custom class to day element
      day.classList.add('some-custom-day-class');
    });

    picker.on('render:month', (month) => {
      // add custom class to button previous
      month.querySelector('.button-previous-month').classList.add('some-custom-btn-class');
    });
  }
})

https://jsfiddle.net/bju4hwzt/

@wakirin thanks! i'll give it a shot and let you know how it goes.

I am using this datepicker with Laravel and Livewire and I need to preset date value (on edit), but init event is not working.
Here is the code (ignore the Laravel and Livewire stuff in code since they don't affect this issue):
<input type="text" autocomplete="off" {{ $attributes }} />

let currentDate = new Date();

new Litepicker({
    element: document.getElementById('datepicker'),
    setup: picker => {
        picker.on('init', () => {
            // This is not working
            console.log('init');
        });

        picker.on('selected', date => {
            // This is working and it sets date upon selected
            console.log('selected', date);
            @this.set("{{ $attributes->wire('model')->value }}", date.format('YYYY-MM-DD'))
        });
    }
});

When I am editing the form input, the value from object (wire:model) is visible in datepicker, but when I open and hide datepicker (without selecting any date) it "resets" to empty because date is not actually selected.
I need a way to preselect date if value exists, but init event is not working.
Version is 2.0.11


Add schemas for YAUAA enrichment

Needed for snowplow/snowplow#4009

also you'll need to generate the DDL and jsonpaths file with igluctl, c.f. https://github.com/snowplow/iglu-central/pull/767/files for example

By DDL you mean the SQL file?

superseded by #942


Support Xamarin

This is some kind of proof of concept, which shows that it's possible to debug Xamarin application using VS Code. I'd like you to review the idea and probably help me with advice.
Now it supports only Xamarin.Android, but Xamarin.IOS is similar.
To start debugging build and deploy Xamarin.Android application via command line or ide, then define "packageName": "com.yourapp.name" in launch configuration, then run debugger. Debugger expects that you have adb in path. I tested debugger only on mac os.
Xamarin uses some "special" way to connect sdb, you can find more details here https://github.com/xamarin/xamarin-macios/blob/xamarin-mac-<IP_ADDRESS>/runtime/monotouch-debug.m#L1270 android has similar code, so I just send few packages before starting connection.
I'm going to continue implementing Xamarin.Anroid debug, then I will implement Xamarin.IOS debug.
Please, share you thoughts about this feature with me.

Added example

I have made this work in the past by setting properties by means of ADB, and using the attach support already available.

@borgdylan , you're right, it is how debugger works.
But it's not enough for comfortable work: debugger functionality should include:

Install application on device: for first release we can work only with one device, but in future it should be some kind of drop down where we can select device. I believe installing should be done by MsBuild tasks because build system know where to find compilation artifacts, and Xamarin already have nice infrastructure for installing. Now I use Install target from Xamarin.Android, but unfortunately I hasn't found something similar for Xamarin.IOS. Any way it's possible to install via libmobiledevice.
Launch application. It's not a problem for Android, hasn't investigated for IOS yet.
Connect output from device.
Connect debugger.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: VysotskiVadim sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Demo with current functionality: https://youtu.be/RnWTy1D031k

Hi, loving this project. This could very well enable Xamarin App development in Linux systems. Anyone tried this on Ubuntu or any other Linux distro yet?

@SimantoR, yes check vscode-xamarin-debug, it's tested on Ubuntu.


terminal_print8 and terminal_measure8 removed from BearLibTerminal

isn't linkable with new  BearLibTerminal
https://bitbucket.org/cfyzium/bearlibterminal/commits/31dabc93b5b4e3d28f79a7847e4f1bc45e7af952

Thanks for the report, I'll push a fix tomorrow.

Fixed in 0cc4f55


Window.Current.CoreWindow.GetAsyncKeyState(VirtualKey.Control) - sometimes incorrect while handling mouse-wheel

Describe the bug
Window.Current.CoreWindow.GetAsyncKeyState(VirtualKey.Control) doesn't always give accurate results when the CTRL hotkey is pressed.
I've encountered this while processing mouse-wheel, but I'm almost certain it happens on other occasions as well.
Steps to reproduce the bug
ctrl.PointerWheelChanged += ctrl_PointerWheelChanged;
private void ctrl_PointerWheelChanged(object sender, PointerRoutedEventArgs e)
{
	var ctrl_key_pressed = Window.Current.CoreWindow.GetAsyncKeyState(VirtualKey.Control) == CoreVirtualKeyStates.Down;
	Console.WriteLine("ctrl-pressed=" + ctrl_key_presed);
}

Run this code.
Press CTRL, and keep it pressed
Keep using the scroll wheel, so you constantly receive events
Every one in 15-20 events will show ctrl-pressed=false

Windows 10 version
Saw the problem?

May 2020 Update (19041)
Yes

November 2019 Update (18363)
Yes

Hi @jtorjo
Can you also create an issue in our feedback hub and share the link please? Be sure to use the ‘reproduce my problem’ option to collect data about what you’re seeing.
Thanks for filing this issue!


Blazor Server side, child component not always updates view

I notices that a child component that used async entityframework not always got updated. So I tried to simulate the situation without Entity framework.
I created a child component with this code:
@if(list == null)
{

        Loading....

}
else
{
    foreach(var i in list)
    {

            @i

    }
}

@functions
{
    protected List<string> list = null;

    protected override async Task OnInitAsync()
    {
        await Task.Run(async () =>
        {
            await Task.Delay(3000);

            list = new List<string>();

            list.Add("Some item 1");
            list.Add("Some item 2");
            list.Add("Some item 3");
            list.Add("Some item 4");
        });

        this.StateHasChanged();
    }
}

In de main page: index.cshtml insert A few child components like:
<Child></Child>
<Child></Child>
<Child></Child>
<Child></Child>

Open debugging with f5.
When the page has been loaded in chrome, reload with f5.
After a few time, a few components won't show but keep saying: loading....

Maybe related to #1223

@Knudel, In asynchronous methods, you must call StateHasChanged()

@cores-system I agree, but this doesn't help. My original code was a connection to a aql server with .net Entitycore. There I did call Statehaschanged().  I updated my code with StateHasChanges but it doesn't change.
Thank you for your help..

@Andrzej-W I don't think calling StateHasChanged at the end of OnInitAsync should be necessary. I believe Blazor captures the returned task and calls StateHasChanged for you aftre the task completes.
@SteveSandersonMS I was able to reproduce this issue:

I had same issue with different variations several times (it is same in SBB and CSB).
Last one was that when I select item there was not updating UI (selected item should change background color and in code notify parent about selection change for future actions like add, delete, update and etc).
I have child component and in parent's BuildRenderTree method I am dynamically rendering them, when I add new child I generate new unique ID.
On click event I call statehaschanged, in result BuildRenderTree is invoked again and is filling ID property with new unique values, when blazor calculates diffs it sees changed IDs and thinks that this changes should be applied to UI, removes all old html and adds new for new items.
Here main problem is that, blazor when calculates diffs sometimes thinks that there is no need to update html but actually it is.
This dirty way worked for me and I was able to force blazor to re-render UI.
I know it is not best solution because I need extra code and my app is doing extra job but for now I will use this before blazor renderer will be more accurate.
If it is possible I think there should be some method like StateHasChanged with will force blazor to re-render entire html on component level and don't calculate only diffs, for example UpdateEntireUI.
@SteveSandersonMS

Thanks for reporting this. It's fixed in #1672.


Bg translation

Hello @domnikl
Kindly review the PR
Cheers

Hello @domnikl
Did you have a chance to check it out? Is there anything wrong?
Regards


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/32017

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/61206

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/62659

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/71659

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83239

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83306

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83645

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83905

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/94608

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/95030

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/96550

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98483

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100893

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/101747

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/109152

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/115065


Fix: can't resume a running fiber

The changes in #14098 require that the fiber be associated to a thread once, instead of doing it on every swapcontext, yet enqueueing the fiber didn't do it while associating the fiber to a thread, which led to further enqueues to associate the fiber to any other thread, in which case a thread B was trying to resume a fiber that was still running on thread A (it didn't swap context, yet).
closes #14124

Note: this is the theory. I didn't try it (yet), though I didn't get any bug on the fix/mt-issues branch which was behaving as this patch does.

This has already been merged (I had issues with my linux linking an invalid executable when building the std specs only), but: I can reproduce the crash 100% on master and can't reproduce it anymore with this patch.
CI may be less affected because there are less vCPU available than on our local laptops, so less parallelism to trigger the bug.

Yeah, I had verified this patch locally as well. The error reproduced 0% while on master it was 100%.


Command key combinations don't send key up events to framework.

Currently, the macOS shell seems to receive the key-up event for command sequences like "⌘-C" without producing a key up event in the framework for them, which is problematic on the framework side when someone wants to re-map it to something else, and it causes the framework to think that the keys never went up, so they stick until someone presses the key again (e.g. if I press "⌘-C", the framework thinks that the "C" key is pressed until it gets pressed again).

cc @franciscojma86

You'll likely need to apply #44130 before you can see what I mean (until it lands).

This might be worth surfacing again. I'm continuing to run this code with no issues.
https://github.com/flutter/flutter/issues/38076#issuecomment-522784379

Thanks @shyndman, we'll definitely look at that method and evaluate it.

Taking a look at this, it seems to be an OS limitation. It seems to only happen with command. While @shyndman suggestion does work, I worry that it might have an unexpected impact given that we would be overriding NSApplication. Another downside I see of that approach, is that it would have to be overriden from the runner app.
@gspencergoog maybe this is something we can workaround on the framework side, to "free" the pressed key when command is released? FWIW if you release cmd before the key, keyUp is called

My suggestion wouldn't actually work. Will keep looking into how we can get the raw event.

I have something to consider.
Both Windows and macOS shortcuts work by evaluating a single keyboard key + a set of zero or more modifiers. The advantage of this approach is that a single keydown event describes whether or not to trigger the shortcut's action — no need to store comprehensive keyboard state.
That said, I appreciate that in the current approach you can ask whether a key is down at any point. Useful. You can do this easily in Windows, too, but not in macOS without using lower-level APIs.

@shyndman the behavior you suggest was implemented recently (using the modifies within the NSEvent of the pressed key) so that's indeed a good observation 😄. However, that doesn't solve the problem at hand since we don't even get the NSEvent from the key being released. I just posted a PR for review, using an approach similar to the one you suggested, but using a listener. Thanks for your suggestion!

🍾


New WPF App(.NET Core) project missing references

I apologize if this is covered somewhere else, but I haven't found any references to it.
The only references that I come across state that I need to "Allow NuGet to download missing packages" and "Automatically check for missing packages during build in Visual Studio".  Both options are/were checked, but every time I start a brand new WPF App(.NET Core) project I get this message:
"Some assembly references are missing.  Building to restore the NuGet cache might resolve this issue".
It's a fairly new install of VS2019 ver 16.4.2 and I've only used it for some Forms tutorials.  That part of VS2019 works fine for me.
I'm new to VS2019 so I may be missing something simple?
Thank you!
Windows10.
ps: I'm not sure how to "build to restore NuGet"...
Edit: in checking the VS installer, it installed with issues: VisualStudio.LiveShare did not install because... it already exists... Impacted workloads: .NET desktop development.
I'm attempting a 'repair'.

@Strydir - this repo if for nuget docs and filing issues with the NuGet documentation - https://docs.microsoft.com/en-us/nuget/ and not meant for product related bugs or questions.
Since your issue seems to be specific to WPF in Visual Studio, I would recommend looking at Developer Community for similar issues and the resolution provided there first - https://developercommunity.visualstudio.com/. If you don't find any similar issues, please file one on Developer Community.

My apologies!


Automatically fetch blockchain modules docs

Cosmos SDK module docs specs should be fetched and added to the content of the docs website as a top-level section.
Blocked on zeta-node repo not being open source.

https://github.com/zeta-chain/docs/pull/54


ABCurve/ABLine EastShiftComponent & NorthShiftComponent

These properties are optional doubles, but their definition is unclear.    What is the UOM?   How does this differ from the GuidanceShift concept applied to the instance of a Logged Data?

Discussion in the 3 August 2022 meeting confirmed these properties are indeed redundant.


use cli xml instead of xml tags

the xml tags had issues on large configs.
When we pushed large configs (80k) the asr9001 router just applied a part of the config (not all of it). The cisco support directed us to this page http://www.cisco.com/c/en/us/td/docs/ios_xr_sw/iosxr_r4-0/xml/programming/guide/xl40api/xl40cli.html
So we tried to use the  syntax to apply our config and it worked.

Coverage remained the same at 87.676% when pulling 8ffbdd40cfcdc72050b9643813d875ddc8eed015 on DENICeG:master into a82de0c7b90160324de07a5811dd21ce4f2902e4 on fooelisa:master.

Hi @thorko - for some reason I missed your PR, apologies for the late reply! Let me look into it and analyze the impact. Do you use this code in production since then? What's the impact, are there any negative downsides of using this method?
Thanks for your work!

Hi micreaulinic,
yes we use this in production. We got this hint from a cisco technician and I can't see any downside of using this method. It could be a little bit slower, but we haven't measured it.
When we used the XML tag  the router commited only part of the config (around 13K). So we had to use the other approach, when our configuration became bigger than 13K.
Cheers
thorko

Thanks @thorko

I will include this in the next release, 0.50.


Revert "Git - Improvements to opening git repositories in parent folders (#171617)"

This reverts commit fe423bbdba5ed6b6d84c37e2f6ab6f65a2ff503d.

It currently breaks the smoke tests. We'll re-revert it again once we fix those.


:bug fix click outside bug

This fixes the issue (https://github.com/harshzalavadiya/react-multi-select-component/issues/118) where clicking outside does not work if user clicks on Select All. It did not work as expected because the <label /> component was not focused.
Fix is straightforward. Just need to set focus on index 1 instead of 0.

@maxnathaniel amazing, thank you for fixing this issue


output corrected

@aftab10662,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@SyntaxC4 Please review these proposed changes. Thanks.

@AftabAnsari10662, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

@SyntaxC4 Please review these proposed changes. Thanks.

This is actually supposed to say Hello World!, we've fixed the repository in which it was changed to Hello Azure!

@SyntaxC4 Hi Cory. Please add an appropriate comment so that automation can take appropriate action. Thanks.

Hi @SyntaxC4
Please assist with resolving the merge conflict. Guidance for resolving merge conflicts.
Please also add an appropriate comment so that automation can take appropriate action.  Thanks

#please-close This has been changed in the sample repo.


Transaction with "used" nonce is not marked as canceled

Looking at Safe https://gnosis-safe.io/safes/0x71B94163b085cc2ea7432D8cC96A94989Da4FD6e/transactions we can see that a transaction with nonce 19 was already executed (id 34), but there is still another tx with nonce 19 that is shown as "awaiting confirmations" (id 29 - should be canceled). Worst case this could block users from performing a transaction, since it will never be possible to execute this transaction.

TODO: figure out how to reproduce

@rmeissner Thanks for flagging this. If you make a test safe for teams with 3 users and try to send ERC20 tokens to >4 people while queueing multiple at once (sending another while one is still pending), you're likely to be able to get it into this state.
If you're going to test it on a testnet, I would try it on one with long block times (not Kovan)

should be fixed in https://github.com/gnosis/safe-react/pull/710

Here's the updated link using the new routing scheme https://gnosis-safe.io/app/#/safes/0x71B94163b085cc2ea7432D8cC96A94989Da4FD6e


angular-cli does build project when using Reflect.getMetadata

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
Mac Osx
Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.10
node: 6.0.0
os: darwin x64
Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.

I created simple decorator in my component class
export function Test() {
    return function (target: Object, key: string) {
        var t = Reflect.getMetadata("design:type", target, key);
        console.log(`${key} type: ${t.name}`);

    }
}

When I try to compile this using angular cli
ng build --dev
I get following error :
 Property 'getMetadata' does not exist on type 'typeof Reflect'.

When I try to compile the TS file just using tsc, it work fine.
am I missing something in terms of configuring angular-cli for use of Reflect API?

Same problem here...

To get around this I had to declare in my TS file a Reflect. then compilation passed ok. I jsut put this into my common file:
declare abstract class Reflect {
public static getMetadata(metadataKey:any, target:Object, targetKey:string | symbol):any;
public static getOwnMetadata(metadataKey: any, target: Object, targetKey: string | symbol): any;
public static getOwnMetadata(metadataKey: any, target: Object): any;
public static defineMetadata(metadataKey: any, metadataValue: any, target: Object, targetKey?: string | symbol): void;
}

Is this declaration in the same file as your component?
How about runtime behavior?

I have it now temporary inside environemt.ts. Regarding the runtime behavior all this declaration are properly hooked with correct implementation as you bring in reflection-meta

Very nice, I'm gonna try later and report my own experience on it.
I'm coming from Java/JSF technology and my goal now is to understand how Angular2/TS works and how can I implement some nice things I found in JSF such as composite, reusable components and so forth.

This is what I'm trying:
<EMAIL_ADDRESS>

Tried here... Added code snippet in environment.ts.
angular-cli is able to compile but:

WebStorm shows a compilation error (is there any change do I need to do?)
Decorator's filename is app.custom.decorator.ts but it is not being copied to dist/

Any thoughts?

Did my test and realized that properties is not being merged. =( It seems Reflect is not bound to declaration during runtime.
Any ideas?

I think this is fixed in the webpack branch, since we changed the build process. I believe the problem before was that reflect was being loaded via a shim in index.html, and you were not importing it.

So what is the "proper" way of using Reflect.metadata in a cli project?
Import reflect-metadata when angular is using core-js or adding to typings.d.ts ect

I was helped by the installation package for babel: @babel/preset-typescript.


Make canvas responsive

the canvas has a fixed width and is not responsive
file to edit:https://github.com/Davidthecode/Stroke/blob/main/src/App.tsx

Hey, I would like to work on this issue. Could you please consider assigning it to me.

Sure @deeksharungta


Add GDCR18 in Vechta

Add GDCR18 in Vechta/Germany to the list of events.

I've moved the JSON file to the right directory and it shows up on the preview page.

Welcome @StefanMacke , have a good one!


DB Notifications dont delete on user delete

Laravel Version: 5.3.21
PHP Version: Any
Database Driver & Version: Any

Description:
Not sure if this is a bug, or something you'd expect users to implement themselves, but on delete of a notifiable user, the notifications don't delete. I appreciate this can't be a simple cascade delete as there are different notifiable types.
Steps To Reproduce:
Create user migration,
Make 'user' notifiable,
Send DB notification to user
Delete user
DB notification remains

For any users wanting to resolve this quickly BTW, just place this in your EventServiceProvider.php boot() method:
        User::deleting(function ($user) {
            return $user->notifications()->delete();
        });

yes this is something you need to handle on your own :)

Fair enough, maybe worth sticking the above example in the notifications comments? As it is something I only thought about nearing entering production :)

Not sure if that belongs to the docs honstly :)


Update appcenter-cli to 3.0.2

Task name: AppCenterTestV1
Description: Updated appcenter-cli version 3.0.0 -> 3.0.2
Documentation changes required: No
Added unit tests: (Y/N) No
Attached related issue: (Y/N) US 107782: [Component Governance] AppCenterTest@1 task - CVE-2024-47554 in commons-io:commons-io 2.11.0. Severity: High (azure.com)
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected: example pipeline with appcenter-cli 3.0.2

It is a copy of forked repository PR in https://github.com/microsoft/azure-pipelines-tasks/pull/20678

Outdated due to appcenter-cli 3.0.3 update https://github.com/microsoft/azure-pipelines-tasks/pull/20684


'Have a project and experience better IntelliSense and code navigation' needs polish

Related to #4007

VSCode Version:
OS Version: Linux

Steps to Reproduce:

Open any .js workspace, get a notification
'Have a project and experience better IntelliSense and code navigation' does not sound specific enough imho. I would mention JavaScript, and we never surface 'InteliSense' in UI, so I would go for 'suggestions'

So something like:
'Configure a JavaScript project and experience better suggestions and code navigation'

"Create a jsconfig.json to enable richer IntelliSense and code navigation across the entire workspace."


The extension errors every time a checkpoint or VAE is loaded (SD.Next)

Hi!
I noticed, that for some time, with recent update of the extension, there's an error every time I load a new checkpoint or VAE (the loading of those at startup is fine):
23:03:17-689824 ERROR    executing callback: X:\AI\automatic\extensions\a1111-sd-webui-tagcomplete\scripts\tag_autocomplete_helper.py
                         model_loaded_callback: TypeError
╭─────────────────────────────────────────────────── Traceback (most recent call last) ────────────────────────────────────────────────────╮
│ X:\AI\automatic\modules\script_callbacks.py:151 in model_loaded_callback                                                                 │
│                                                                                │
│   150 │   │   │   t0 = time.time()                                                                                │
│ ❱ 151 │   │   │   c.callback(sd_model)                                                                                │
│   152 │   │   │   timer(t0, c.script, 'model_loaded')                                                                                │
│                                                                                │
│ X:\AI\automatic\extensions\a1111-sd-webui-tagcomplete\scripts\tag_autocomplete_helper.py:184 in get_embeddings                           │
│                                                                                │
│   183 │   │   elif (emb_b_shape == V2_SHAPE):                                                                                │
│ ❱ 184 │   │   │   emb_v2 = [(Path(v.filename), k, "v2") for (k,v) in emb_type_b.items()]                                                 │
│   185                                                                                │
│                                                                                │
│ X:\AI\automatic\extensions\a1111-sd-webui-tagcomplete\scripts\tag_autocomplete_helper.py:184 in <listcomp>                               │
│                                                                                │
│   183 │   │   elif (emb_b_shape == V2_SHAPE):                                                                                │
│ ❱ 184 │   │   │   emb_v2 = [(Path(v.filename), k, "v2") for (k,v) in emb_type_b.items()]                                                 │
│   185                                                                                │
│                                                                                │
│ C:\Users\user\AppData\Local\Programs\Python\Python310\lib\pathlib.py:960 in __new__                                                       │
│                                                                                │
│    959 │   │   │   cls = WindowsPath if os.name == 'nt' else PosixPath                                                                   │
│ ❱  960 │   │   self = cls._from_parts(args)                                                                                │
│    961 │   │   if not self._flavour.is_supported:                                                                                │
│                                                                                │
│ C:\Users\user\AppData\Local\Programs\Python\Python310\lib\pathlib.py:594 in _from_parts                                                   │
│                                                                                │
│    593 │   │   self = object.__new__(cls)                                                                                │
│ ❱  594 │   │   drv, root, parts = self._parse_args(args)                                                                                │
│    595 │   │   self._drv = drv                                                                                │
│                                                                                │
│ C:\Users\user\AppData\Local\Programs\Python\Python310\lib\pathlib.py:578 in _parse_args                                                   │
│                                                                                │
│    577 │   │   │   else:                                                                                │
│ ❱  578 │   │   │   │   a = os.fspath(a)                                                                                │
│    579 │   │   │   │   if isinstance(a, str):                                                                                │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
TypeError: expected str, bytes or os.PathLike object, not NoneType
23:03:18-042325 INFO     Model loaded in 39.20s { load=18.35s create=0.60s apply=0.68s vae=0.53s move=1.32s embeddings=17.29s
                         callbacks=0.38s }
23:03:18-635823 INFO     Model load finished: {'ram': {'used': 11.72, 'total': 63.68}, 'gpu': {'used': 3.92, 'total': 24.0}, 'retries': 0,
                         'oom': 0} cached=0
Loading weights: X:\AI\automatic\models\Stable-diffusion\boymix13.safetensors ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 7.6/7.6 GB 0:00:00
23:04:30-433824 INFO     Torch override VAE dtype: no-half set
23:04:30-435324 INFO     Setting Torch parameters: device=cuda dtype=torch.float16 vae=torch.float32 unet=torch.float16
                         context=inference_mode fp16=True bf16=False
23:04:31-021323 INFO     LDM: LatentDiffusion: Running in eps-prediction mode
23:04:31-023324 INFO     LDM: DiffusionWrapper has 859.52 M params.
Loading weights: models\VAE\difconsistencyRAWVAE_v1LOW.safetensors ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 334.6/334.6 MB 0:00:00
23:04:32-943824 INFO     Cross-attention: optimization=Scaled-Dot-Product options=[]
23:04:50-309825 ERROR    executing callback: X:\AI\automatic\extensions\a1111-sd-webui-tagcomplete\scripts\tag_autocomplete_helper.py
                         model_loaded_callback: TypeError
╭─────────────────────────────────────────────────── Traceback (most recent call last) ────────────────────────────────────────────────────╮
│ X:\AI\automatic\modules\script_callbacks.py:151 in model_loaded_callback                                                                 │
│                                                                                │
│   150 │   │   │   t0 = time.time()                                                                                │
│ ❱ 151 │   │   │   c.callback(sd_model)                                                                                │
│   152 │   │   │   timer(t0, c.script, 'model_loaded')                                                                                │
│                                                                                │
│ X:\AI\automatic\extensions\a1111-sd-webui-tagcomplete\scripts\tag_autocomplete_helper.py:184 in get_embeddings                           │
│                                                                                │
│   183 │   │   elif (emb_b_shape == V2_SHAPE):                                                                                │
│ ❱ 184 │   │   │   emb_v2 = [(Path(v.filename), k, "v2") for (k,v) in emb_type_b.items()]                                                 │
│   185                                                                                │
│                                                                                │
│ X:\AI\automatic\extensions\a1111-sd-webui-tagcomplete\scripts\tag_autocomplete_helper.py:184 in <listcomp>                               │
│                                                                                │
│   183 │   │   elif (emb_b_shape == V2_SHAPE):                                                                                │
│ ❱ 184 │   │   │   emb_v2 = [(Path(v.filename), k, "v2") for (k,v) in emb_type_b.items()]                                                 │
│   185                                                                                │
│                                                                                │
│ C:\Users\user\AppData\Local\Programs\Python\Python310\lib\pathlib.py:960 in __new__                                                       │
│                                                                                │
│    959 │   │   │   cls = WindowsPath if os.name == 'nt' else PosixPath                                                                   │
│ ❱  960 │   │   self = cls._from_parts(args)                                                                                │
│    961 │   │   if not self._flavour.is_supported:                                                                                │
│                                                                                │
│ C:\Users\user\AppData\Local\Programs\Python\Python310\lib\pathlib.py:594 in _from_parts                                                   │
│                                                                                │
│    593 │   │   self = object.__new__(cls)                                                                                │
│ ❱  594 │   │   drv, root, parts = self._parse_args(args)                                                                                │
│    595 │   │   self._drv = drv                                                                                │
│                                                                                │
│ C:\Users\user\AppData\Local\Programs\Python\Python310\lib\pathlib.py:578 in _parse_args                                                   │
│                                                                                │
│    577 │   │   │   else:                                                                                │
│ ❱  578 │   │   │   │   a = os.fspath(a)                                                                                │
│    579 │   │   │   │   if isinstance(a, str):                                                                                │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
TypeError: expected str, bytes or os.PathLike object, not NoneType

~There's also a bug with embeddings not being listed anymore, despite the option being set at default for them. The embeddings I use are in the symlinked folder.~
It happens on SD.Next UI with 89ba8e3c commit.

After being stuck for a bit, I think that the issue is SD.Next not supporting SDXL embeddings properly, possibly as a side effect of SDXL only working with diffusers there. Do you maybe have any lying around?
From what I've seen in the SD.Next code, they get detected as diffusers concepts, but since the data format is wrong they get skipped.
The issue here is that while in auto1111 they are just ignored but still registered properly, in SD.Next the function exits prematurely, before other info about the skipped embedding like filepaths was added. This is a bug / side effect in SD.Next.
When switching models, tag autocomplete looks up the embeddings to determine their type (sdv1/sdv2, with sdxl being detected as the latter for now). The error then happens when trying to look up these skipped sdxl embeddings, since they have been registered as skipped but are missing filepath and other info, which a1111 on the other hand keeps intact.
I have added a safety check for missing paths on my side, so it should be fixed now. It will also hide those embeddings from autocompletion. In SD.Next it shouldn't be too much of an issue since they seem unsupported anyway.
The root cause of the behavioral difference / bug in SD.Next is still there, but I imagine it will be fixed when they add sdxl support to the original backend instead of just diffusers.

Thank you for quick fix!
The only XL embeddings I have are those 'negatives':

Could they be the cause?

I think these are the same I noticed the problem with, they're from CounterfeitXL, right?
I found an issue on the SD.Next repo that had the same problem, so I mentioned it there. As of now, vlad has already fixed it in the dev branch in response, so it should soon be properly resolved.
It seems specifically related to loading these sdxl embeddings on the original backend (where SD.Next doesn't support sdxl models yet) instead of the diffusers one, which caused the original bug/oversight.
Just out of interest since I don't have one downloaded right now:
In case you have a diffusers XL model you use with SD.Next, do these embeddings get correctly loaded there?
Would be interesting to know if they are incompatible with diffusers or were just not properly skipped on the original backend.

I noticed that in Vlad's dev branch as well, can't wait to get original backend implemented for SDXL. 🙂
For the diffusers backed+SDXL, I have trouble running it at all (and since it's still fresh, I'm patiently waiting for fully fledged SDXL support), so I didn't test it much, yet, and I'm using last 'main' branch of the SD.Next, before Vlad started implementing big changes into 'dev' branch and stated, that it's best to go with fresh git pull because of these changes. 🙂


Names are not prefixed

Please answer the following questions.
Which version of matterbridge are you using?
Running version 0.15.0 822605c
Please describe the expected behavior.
Names to be prefixed
Please add your configuration file
(be sure to exclude or anonymize private data (tokens/passwords))
[mattermost]
[mattermost.work]
useAPI=true
#do not prefix it wit http:// or https://
Server="chat.org"
Team="bot"
Login="dbot"
Password="password"
PrefixMessagesWithNick=true

It's all in the documentation https://github.com/42wim/matterbridge/blob/master/matterbridge.toml.sample
Seems like I'll have to update the sample in the README you probably used though
#RemoteNickFormat defines how remote users appear on this bridge
#The string "{NICK}" (case sensitive) will be replaced by the actual nick / username.
#The string "{BRIDGE}" (case sensitive) will be replaced by the sending bridge
#The string "{PROTOCOL}" (case sensitive) will be replaced by the protocol used by the bridge
#OPTIONAL (default empty)
RemoteNickFormat="[{PROTOCOL}] <{NICK}> "


Add support for device "Switch 1~6 gang"

I'm very appreciate to your program.
Can you add my devices?
Here's my 3gang switch information. (Works well. Test completed for 1~6 gang)
Here's my db : https://pastebin.com/0V8m4fAQ
Here's my log : https://pastebin.com/c2aDs72c
Here's my debug: https://pastebin.com/0yMz71zh
Here's my custom devices.js : https://pastebin.com/4awi82YT
Here's my custom homeassistant.js : https://pastebin.com/NsnRhfGS
And I added postfixes.
/lib/util/utils.js   find  "bottom_" and add 'center_left',  'center_right'
Here's device url
https://www.hej.life/shop/?idx=17
https://cdn.imweb.me/upload/S201901095c3581af7317d/7e2b3c21ea3d8.png
Model name
1gang : GLSK3ZB-1711
2gang : GLSK3ZB-1712
3gang : GLSK3ZB-1713
4gang : GLSK6ZB-1714
5gang : GLSK6ZB-1715
6gang : GLSK6ZB-1716
Thank you.

Please make a pull request

@Koenkk  Oh. I didn't know about that. I'll do that. Sorry. Thank you.


Notes: Text not translated

I have translated everything in Transifex, but the text "opened" and "closed" in the history for Notes are not translated.

good catch @magol - I'll add translation strings for these.


Support PowerShell Core 6.1

Summary of new feature
Research and document what will have to be adjusted/updated in order for us to support PowerShell Core. It is clear that not all commands are going to work because the whole SMO library has not been converted to .NET Core.
Proposed technical details (if applicable)

Look at what will have to happen for the import process
Organization on the two versions of SMO we will have to maintain.
Adjustment to the dbatools.dll library
Adjustments to run Pester test under PS Core (Pester already supports Core as of January 2018).

Creating this to start the discussion being that PS Core 6.1 was released.
// @potatoqualitee @FriedrichWeinmann @niphlod

Thanks, Shawn. @FriedrichWeinmann - would you have any time to look into this? We've engaged the SQL Team about adding more core, but need a proof of concept to show what we're still missing and ask for prioritization.
We can handle the changes to the module psm1, but need help with the DLL.

For Phillip:
0.) I am not competent with C# and barely know what i'm talking about here. Our C# guy, Fred, is unavailable for a while but Microsoft recently made movement with SMO SQL and I'd like to take advantage. We use their SMOs and our dbatools.dll library. Our minimum requirement for regular PS is PS v3 and I'd like to support regular PS and then PS Core.
1.) I merged @IISResetMe 's changes into the netcore branch. His changes remove the only Core incompatible thing from our toolset. We were not using it yet.
2.) We're basically going for what the SqlServer module supports and we have permission to use what we need from them. Their project is on the Gallery
3.) Our dll autocompiles on dbatools load when a version change is noticed. Not sure if this will impact you. Version changes are made at bin\library.ps`` and \bin\projects\dbatools\dbatools\Properties\AssemblyInfo.cs`
SqlServer's DLLs look like this

4.) Thank you 🙇  ! I'm super excited about the potential. I just need the dbatools.core.dll and I can work out the rest with SMO Core and our Psm1 loading everything.

okay, i got everything going except for xplat support. dlls and core detection have been added in 0.9.522
i will add the xplat support in the files (resolve-path galore), but still need help with the DLL itself, which loads in Windows but not os x. i tried with latest version of pwsh and all

Does it load in core on Windows for you?

Yep! with no issues. see twitter 💃

@wsmelton Even @SQLDBAWithABeard got it running smoothly in 6.1.1 😄

https://twitter.com/sqldbawithbeard/status/1065955800823332864

Yup, ran a whole bundle of commands without issue

got this working xplat, not sure how, it was the same dll, but once i get the loader done, we're set! thanks everyone for your help!

💔 xplat appears to work but the load time is so bad that i won't announce. logs suggest library is only partially loaded. but our data types work.

k giving up and letting the pros handle this. i tried with
dotnet build "dbatools.csproj" --configuration Release --framework netcoreapp2.1
but no cigar.

dll works well, Fred pointed out i forgot to add -Path. so now we return to just the runspace timeout and potentially related library issues as seen in Get-DbatoolsLog
CallStack    : {}
ComputerName : forte
File         : /usr/local/share/powershell/Modules/dbatools/0.9.527/allcommands.ps1
FunctionName : Set-DbatoolsConfig
Level        : Warning
Line         : 86182
Message      : Invalid validation name: bool.  Supported validations:
ModuleName   : dbatools
Runspace     : 8bec604b-60c2-4a9f-b7fd-de9158583f10
Tags         : {}
TargetObject : cache.enable.credentialfailover
Timestamp    : 11/24/18 1:01:14 AM
Type         : Debug, Warning
Username     : forte\loulou

CallStack    : {}
ComputerName : forte
File         : /usr/local/share/powershell/Modules/dbatools/0.9.527/allcommands.ps1
FunctionName : Set-DbatoolsConfig
Level        : Warning
Line         : 86182
Message      : Invalid validation name: string.       Supported validations:
ModuleName   : dbatools
Runspace     : 8bec604b-60c2-4a9f-b7fd-de9158583f10
Tags         : {}
TargetObject : type.disable.cimdcom
Timestamp    : 11/24/18 1:01:15 AM
Type         : Debug, Warning
Username     : forte\loulou

CallStack    : {}
ComputerName : forte
File         : /usr/local/share/powershell/Modules/dbatools/0.9.527/allcommands.ps1
FunctionName : Set-DbatoolsConfig
Level        : Warning
Line         : 86182
Message      : Invalid validation name: string. Supported validations:
ModuleName   : dbatools
Runspace     : d27e18f8-eee3-40a7-8522-4922891ea306
Tags         : {}
TargetObject : date
Timestamp    : 11/24/18 1:01:15 AM
Type         : Debug, Warning
Username     : forte\loulou

CallStack    : {}
ComputerName : forte
File         : /usr/local/share/powershell/Modules/dbatools/0.9.527/allcommands.ps1
FunctionName : Set-DbatoolsConfig
Level        : Warning
Line         : 86182
Message      : Invalid validation name: bool.   Supported validations:
ModuleName   : dbatools
Runspace     : d27e18f8-eee3-40a7-8522-4922891ea306
Tags         : {}
TargetObject : datetime
Timestamp    : 11/24/18 1:01:15 AM
Type         : Debug, Warning
Username     : forte\loulou

CallStack    : {}
ComputerName : forte
File         : /usr/local/share/powershell/Modules/dbatools/0.9.527/allcommands.ps1
FunctionName : Set-DbatoolsConfig
Level        : Warning
Line         : 86182
Message      : Invalid validation name: sizestyle. Supported validations:
ModuleName   : dbatools
Runspace     : 231a63d4-3a68-4c19-b407-d7b6d9e8a8ff
Tags         : {}
TargetObject : size.style
Timestamp    : 11/24/18 1:01:15 AM
Type         : Debug, Warning
Username     : forte\loulou
```

It is indeed the library, which is an older version of PSFramework before it went xplat. Fred will work on it when possible as It'll require a rework of Set-DbatoolsConfig and the library.

Nice, let me know if there is anything I can help with.


Impl PartialOrd and Ord for Number

It seems possible to implement both of these for Number.

Integer to float comparisons are hard. In particular, (i as f64) < f is not always equal to i < (f as u64). Any ideas for how to implement this?

If #244 were done then perhaps this could be implemented more easily.

I guess only PartialOrd would work, then:
given Numbers a and b:
	if a and b are both i64
		compare as i64
	if a and b are both u64
		compare as u64
	if a is f64 and b is f64
		compare as f64

	if a is u64 and b is i64
		if b < 0
			(a>b)
		else
			compare a vs (b as u64)
	if a is i64 and b is u64
		invert compare b, a

	if a is f64 and b is u64
		if b < max-int-representable-by-f64 (2^53?)
			compare a, (b as f64)
		else if b < floor(a)
			(b<a)
		else if b > ceil(a)
			(a>b)
		else
			undecidable
	if a is f64 and b is i64
		if b < 0
			invert compare -a, -b
		else
			compare a, (b as u64)

... assume the rest are defined the same way

Are there any flaws with this?

As discussed in the PR: I am concerned that this would limit our ability to implement arbitrary precision number parsing in the future. In particular, #252 would have wanted to replace Number with struct Number { n: String } where PartialOrd becomes way more difficult to define.
Let's keep this issue open and follow up after arbitrary-precision numbers have been settled.


Issue with k8s.io/docs/tutorials/kubernetes-basics/deploy-intro/

This is a...

[ ] Feature Request
[X] Bug Report

Problem:
The link _https://github.com/kubernetes/website/tree/master/docs/tutorials/kubernetes-basics_ to github repository is break and get 404 error
Proposed Solution:
Update the link
Page to Update:
https://kubernetes.io/...

/triage duplicate
See #8547

Closing this as it's a duplicate.
/close


fix: invoke spawn sync with shell true arg

child process was failing to find tsc-files
https://github.com/nodejs/node/issues/8077

Addresses https://github.com/Spidy88/lint-staged-tsc/issues/1


feat: add autofix for alphabetize rule and change sort order to lexicographic

related https://github.com/dotansimha/graphql-eslint/discussions/935

@B2o5T rebase and merge?


outdated tutorial

OpenSMOG/Tutorials/SBM_CA/input/SBM_CA.ipynb
The tutorial requires loading of gro, top and xml at separate lines.  But, there is a single function for this now.  we should use the compact version, to help establish good habits.

The tutorial is now updated


Fix unit test

Fix npm test command. Use && between two commands.
Fix a failed test in eventwrapper

Codecov Report

Merging #411 (26294c6) into stage (b9fb5c9) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            stage     #411      +/-   ##
==========================================
+ Coverage   93.50%   93.63%   +0.13%
==========================================
  Files          26       26
  Lines        2987     2987
==========================================
+ Hits         2793     2797       +4
+ Misses        194      190       -4

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[5.4] Fix make:policy Command

When create the User policy  php artisan make:policy UserPolicy -m User
will duplicate the App\User class imports

This PR has typos.

@taylorotwell , I'm wondering why are you closing PR instantly if issues are found during review process. This completely disables dialog possibility on PR code or even fixing attempt as separate commit on same PR.
Any practical reasons behind this?


FileService object in azure.storage.file has SSL 'certificate verify failed' error

just updated on one setup to the latest azure python sdk, then when trying to create a file share in storage account, it's giving 'certificate verify failed':
azure.common.AzureException: HTTPSConnectionPool(host='smallflowesa.file.core.windows.net', port=443): Max retries exceeded with url: /hett?restype=share (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')],)",),))
on the other setup with older version of python sdk, everything works fine.
my code:
from azure.storage.file import FileService
file_service = FileService(account_name='',account_key='',endpoint_suffix='core.windows.net')
file_service.create_share('***')

Hi @utadamai, thanks for reaching out!
It's most likely not a problem in the SDK itself. Please refer to this relevant post.

Hello @zezha-msft I need your help. I am trying to create an Azure file share but I am getting a HTTPSCONNECTIONPOOL error. What could be the problem? Please help me on urgent basis as I have the certification exam in 3 days. Thank you!

Hi @championbally
Did you you see the relevant post https://stackoverflow.com/questions/27835619/urllib-and-ssl-certificate-verify-failed-error/42334357#42334357 zezha referred to?

Hi @xiafu-msft
Thanks for your reply, I read that post but that post seems  to not answer to my specific problem. I have an HTTPs problemand I use Windows but that post talks about Python certificate for MacOS

Hi @championbally
Did you try some suggested way and see if it can be resolved? eg. https://answers.microsoft.com/en-us/windows/forum/all/ssl-error-preventing-connection-in-windows-10/192436e5-e37b-4b4c-a4e0-c4ec744b0f5c

Hi @xiafu-msft
Thanks for your time. I think the SSL configuration helped me a little bit because before I was getting the HTTPSCONNECTIONPOOL error along with the port 443 error but now I am getting only the HTTPSCONNECTIONPOOL error only. Kindly find attached a screenshot of the error I am getting along with the command I am getting error on.

Also can you explain what this HTTPSCONNECTIONPOOL error is all about? I think if I understand the error itself I can be able to solve the issue. Thank you so much


Complete become a lessor registration

Complete #115

@futurizing I have completed all your requests (Including router resolver). Please review again. Hope we'll not have anything to change more, so I can move on to the next task :D


Http error 400 when I try to invoke  char completion

Are you using the OpenAiService in the service module? It will give more detailed error messages.

Error has been solved, Reason: max tokens too high. OpenAI explained this problem in their documents.


Unify behavior of plastic strain without or with transports

Summary
I consider the fact that PyRolL switches the way the plastic strain is treated based on the fact whether a transport is present or not a severe design flaw. Without transport plastic strain is accumulated with transport present it is reset to zero between stands.
As it seems to be an accepted assumption that for most materials recrystallization occurs between stands this should be the default PyRolL-behavior regardless whether transports are present.
It should be possible for the user to request accumulative plastic strain via a PyRolL-setting.

There is no behavior switched because of present or not present transports. The transport itself resets the strain to 0 by default (assuming total recrystallization within the transport).
We do not think, that it is a design flaw, since without a transport (aka inter-pass range) between there would be immediate input from the previous pass to the next (which is of course physically not possible). So where shall the recrystallization happen?
We designed it to model the inter-pass ranges explicitly as a transport unit, rather than implicitly.
So this behavior will not be changed.

Agreeing with @axtimhaus in that case. Our assumption tends to be wrong when higher rolling speeds are present.


Problems saving

Things happen when i save
When i save my brew the right side of the brew disappears and im not sure how to stop this from happening

Issues like this resulting from invalid code in the editor should be fixed now with v2.10.3.


Unique identifier columns are not allowed to be a primary key in table designer.

Steps to repro:

Open table designer.
Change the column type to unique identifier.
The UI does disables the primary key checkbox.

Column 3: is not allowed to be primary key in table designer.

@alanrenmsft Assigning this to you for Feb release


Duplicate characters in textbox

I have a textbox in a form:
var lname=this.blessed.textbox({ 			parent: this.form, 			top: '40%', 			left: '30%', 			height: '20%', 			width: '40%', 			name: 'name', 			inputOnFocus: true, 			border: 'line', 			style: { 				bg: 'blue', 				focus: { 					bg: 'white', 					fg: 'blue' 				}, 				hover: { 					bg: 'white', 					fg: 'blue' 				} 			} });
After you focus on the textbox the second time using the mouse, characters input into it are duplicated (aaddmmiinn). Is this a bug or did I mess something up? Code is at:
https://github.com/kristerj/dadOS/blob/master/termView.js

I had same problem. Just remove lname.focus() if you have it somewhere. But yes, somehow input can be focused twice, so any input doubles.


Add socat Serial over TCP support

Adds the very small socat package (https://pkgs.alpinelinux.org/package/edge/main/x86/socat) and a script to fork a socat process within the zigbee2mqtt container. The goal is enable zigbee2mqtt to access a serial device over the network via TCP.
This can be very usefull in the scenario where the zigbee stick isn't physically connected to the hass.io machine. Also when you do not have access to the operating system itself (like with the hassos appliances or VMs) it can be very difficult to add a virtual serial device within the zigbee2mqtt docker container.
Socat is a relay tool for bidirectional data transfer between two data channels. By default the socat parameters are configured to listen on port 8485 and redirect it to a virtual device named /dev/ttyZ2M. This device can then be used in zigbee2mqtt for to access the serial device.
Socat is configured via a separate 'socat' section within the options. The section is completely optional to make it backwards compatible with older configurations. The socat functionality is disabled by default on new installations.

Hi @EssentialNRG @danielwelch
I just refreshed my 1.6 edge and the config no-longer worked.
This was because adding socat to config schema is a breaking change.
Its necessary to add (at least)
  "socat": {}

to the addon config to allow the addon config to save.
Unfortunately the hassio error message is truncated so I need to search up recent schema changes in GH to fix it.
Should I open an issue to get the docs for 1.6 to indicate that there is a breaking change?

Hey @james-fry
Thanks for clarifying. Yes, this is a breaking change, and docs will need to be added before the next release. New configs will be ok because of the defaults in the schema, but old configs will need to be updated.
Nothing needs to be changed for 1.6 as the release doesn't include socat support. I'm ok with breaking changes on the edge version (without docs even), as its purpose is testing new features and using the bleeding-edge version of zigbee2mqtt. But we will need docs before the next stable release if it is to include socat support.


Improve Azure Search continuation token

Probably want to add service version in addition to current base64 encode

The relevant code is here:
https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/search/azure-search-documents/azure/search/documents/_index/_paging.py
Currently the request is already base64, the ask is to also include the service version somehow (TBD with team)

See #10571

Fixed in #10571


when drag start to prevent google chrome  back??

Doesn't use issue template. Improper punctuation. No explanation. Locked and close.


Fix sourcemaps

Publish 1.0.0 please. Or at least let's discuss before publishing.
You don't need more tests. Everything is fine now. It's not so heavy plugin to keep it in "unready for use" version.

Practical example: you publish "minor" and my project should automatically load new version on npm i, but it won't. Because 0.* versions are not semver => not safe.

Sure

Thanks


Cannot view source on Coveralls

When you try and view the source of a file on Coveralls you see the following error instead:
The owner of this repo needs to re-authorize with Github; their OAuth credentials are no
longer valid so the file cannot be pulled from the Github API.

https://coveralls.io/builds/2561958/source?filename=request.js

@nylen can you first Revoke the access to Coveralls from here https://github.com/settings/applications and then login again here https://coveralls.io/ (you might as well delete the coveralls cookies from the Resources tab before that). And this might fix the GitHub status API reporting on the PRs.

Actually there might some other issue with coveralls because I'm not seeing the code for my own projects either. Still the above procedure can fix the GitHub's status API at least.

It was working fine for me, but I remoked Coveralls access from GitHub.  I did not re-add it again, @simov I'll let you do that.  One more thing to try later would be to do "delete repo" in Coveralls - I didn't do it because then we would lose all of our build history.

@nylen that's the thing, there is some weird logic in coveralls that you have to be the one initially added the project to be able to delete it, also your login token will be used for calls to the GitHub API, not mine, so without you re-authenticating it won't work.

Hmm.. want me to delete the repo from coveralls and see if that helps?

Deleting anything sounds scary, but if that's an option, after that I can try to add it myself.

Well.. I tried twice, and got this ClouldFlare 504 message both times:

Then, the repo is still up at https://coveralls.io/r/request/request .  Looks like there are some things broken with @coveralls.

@eugirdor currently we have two coverage badges in our readme - codecov and coveralls Currently coveralls file coverage for request is still broken for some reason, although it magically started to work for my personal projects, but you can see the actual file coverage in codecov. Also the GitHub's status API is finally working.

Excellent, thank you.


Added option redundant_servers

This will fix https://github.com/php-cache/issues/issues/69
Ping @panki3a, Can you test this patch?
cache_adapter:
  providers:
    memcached:
      factory: 'cache.factory.memcached'
      options:
        persistent_id: 'story_pool'
        host: 'foo'
        port: 123
        redundant_servers:
          - host: 'bar'
            post: 123
          - host: 'baz'
            post: 123

Current coverage is 21.90% (diff: 0.00%)

Merging #48 into master will decrease coverage by 0.86%

@@             master        #48   diff @@
==========================================
  Files            35         35
  Lines           606        630    +24
  Methods          77         77
  Messages          0          0
  Branches          0          0
==========================================
  Hits            138        138
- Misses          468        492    +24
  Partials          0          0

Powered by Codecov. Last update 775dabc...0362f8b

i tested it, and its functional.

Thank you for trying this out.


⚠ Bofrost.de has degraded performance

In 0ca6f7a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1342 ms

Resolved: Bofrost.de performance has improved in 4e58810 after 6 minutes.


Git Mirroring doesn't respect BUILDKITE_REFSPEC

One of our projects uses a rather tricky Git mirroring approach that allows agents external to our network to gain access to our code via a GitHub.com mirror.
For this to work, we set BUILDKITE_REFSPEC in the environment hook – this ensures that subsequent steps don't try to pull from the "magic" PR branches used by GitHub, and it works quite well. Unfortunately, the Git Mirroring logic breaks this by trying to checkout the PR branch (which doesn't exist on the mirror).
The only workaround we've found is to add the following to our environment hook:
export BUILDKITE_GIT_MIRRORS_SKIP_UPDATE="true"

if [ -d "${GIT_MIRROR_DIR_THAT_WE_MAINTAIN_MANUALLY}" ]; then
  git --git-dir ${GIT_MIRROR_DIR_THAT_WE_MAINTAIN_MANUALLY} fetch origin
fi

It'd be really great if we didn't need to maintain this ourselves, but for that to happen we'd need to ensure that the mirroring system respects the BUILDKITE_REFSPEC setting – IINM, the line linked above should evaluate to false if BUILDKITE_REFSPEC is present – I suspect that would resolve the problem.

Thanks for raising this issue @jkmassel. I agree that BUILDKITE_REFSPEC should probably be overriding the git mirror logic there. We'll look into fixing it.


How to disable change address in Sparrow

I could use use_change: false in Electrum Wallet, But how in Sparrow?

It's not possible. I'm not adding this "feature" either, since it makes transactions less private for both sender and receiver, and Bitcoin overall less private as well.

It's not possible. I'm not adding this "feature" either, since it makes transactions less private for both sender and receiver, and Bitcoin overall less private as well.

However, I’d like to do something similar to an exchange or company that doesn’t prioritize privacy.
Sparrow is a very nice wallet, but it should offer some options for this, just like the coin control feature in the Bitcoin Core GUI or use_change: false Electrum.

However, I’d like to do something similar to an exchange or company that doesn’t prioritize privacy.

New Bitcoin addresses are effectively free. What is the cost of additional privacy?

It’s costly to manage. Why do Binance, Coinbase, or other companies keep using fixed addresses?

Isn't costly?

Not really. I test Sparrow on active wallets with millions of addresses, that have 10s of new transactions in every block. It handles this well. Modern Electrum servers like Fulcrum scale very well, particularly if they are running on decent hardware. You can even configure Sparrow to only watch the last x addresses in the Advanced Settings.
Giving up privacy is a very serious tradeoff, which IMO should only be made if there are no other alternatives. I'm not convinced that is the case.

Not really. I test Sparrow on active wallets with millions of addresses, that have 10s of new transactions in every block.

How do you set the gap limit?


python311Packages.google-ai-generativelanguage: 0.5.1 -> 0.5.2

Changelog: https://github.com/googleapis/google-cloud-python/blob/google-ai-generativelanguage-v0.5.2/packages/google-ai-generativelanguage/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

5 packages built:

home-assistant-component-tests.google_generative_ai_conversation
python311Packages.google-ai-generativelanguage
python311Packages.google-ai-generativelanguage.dist
python311Packages.google-generativeai
python311Packages.google-generativeai.dist


Getting Started with Java SDK content out of date

The content for the instructions and dependencies need to be updated.
Link to page: https://aws-otel.github.io/docs/getting-started/java-sdk

Updated with PR https://github.com/aws-otel/aws-otel.github.io/pull/15


Add partition statistics

Merged build finished. Test PASSed.

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

Right now partitionInfo also contains partitionStatistics, we might want to move it out of partitionInfo in the future so that we can return them separately for more efficient rpcs

Merged build finished. Test PASSed.

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

@gpang Will move statistics information to an object outside of the layout data so we can return them independently to the client as we discussed.

@gpang separate stats from the rest of the partitionInfo. PTAL

Merged build finished. Test PASSed.

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

Merged build finished. Test FAILed.

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

jenkins, test this please

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

Merged build finished. Test PASSed.

@ZacBlanco PTAL, thanks

@ZacBlanco Updated, PTAL

Merged build finished. Test PASSed.

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

alluxio-bot, merge this please