stability and maintenance

Description of changes

Upgrade node from v12 to v16
Whenever a image processing attempt fails, retry at least 2 more times before giving up. This is to avoid losing notifications due to occasional blips during the Ai identification process.
Added readme documentation to describe how to manually publish your own version of this project to docker hub.

Checklist
If your change touches anything under src or the README.md file
these items must be done:

[x] User-facing change description added to unreleased section of CHANGELOG.md

closing.... I should have read the contribution readme, to create pull requests using feature branch

@hiro5id Thanks for the contribution. When you open your next PR please open three issues for the three things you are addressing and do one fix in each PR (three total). Thanks!

I'm not sure how to do that easily @neilenns .  I am making a lot of improvements but they depend on each other, and I don't want to have to wait for each PR to be accepted before I can build on top of it to submit the next PR.  I am trying to fix some deficiencies for myself, and I am more than happy to contribute my changes back.  But I don't know how to split them up into separate PR's without taking weeks of back and forth.
Perhaps I can consolidate each logical change into separate commits, but all in one PR ?  I don't know i'll have a look what I can do.

@hiro5id You have separate feature branches for each improvement you want to make, and have to wait for the PRs to be accepted if there are dependencies. This PR didn't have any, it was three independent things that could be opened and merged concurrently.
Also there have to be issues logged for the changes first, so I know what is coming and we can discuss the approach (if appropriate) to resolving them.


Create test framework to test creation of an Event list to be displayed

This means:

Implement a mock for a Calendar provider. This special Calendar provider will supply events to our tests thus allowing us to test different cases, related to calendar events and Settings of our widget.
The Calendar provider should implement minimal part of this API:
http://developer.android.com/guide/topics/providers/calendar-provider.html
From a calendar-widget point of view, this is something, which can return cursor as a response to a request like this:

context.getContentResolver().query(uri, getProjection(),
                    selection, null, EVENT_SORT_ORDER);

Once the base testing framework functionality works, we will add actual test cases one by one and fix a code. We will need to refactor and rethink something...
2. Replace actual Calendars with the mock we will use MockContentResolver ( see http://developer.android.com/tools/testing/testing_android.html#MockObjectClasses )
3. It is not clear yet, how we will collect test data for our tests. As practice shows, currently it is very difficult to reproduce a User's problem, see e.g. #188
So maybe we will need to create special reporting feature for users, which will collect data, required to reproduce a problem.

Eventually we will need to add a test new calendar and events to it.
On adding  (creating ) new calendar:
?
On adding events:
http://stackoverflow.com/questions/3721963/how-to-add-calendar-events-in-android

First test case already created and passed on Emulator.

"Share events for debugging" link added to the Feedback section of Settings.
Clicking on this item creates text data (in JSON format) of events, shown on the Calendar-widget, with application settings, which may be shared with developers (e.g. sent by email) in order to communicate to them User's wishes and problems.
This JSON-formatted data may be re-played at developer's machine to reproduce and fix bugs. Moreover, the data may be used to create automated tests, showings these bugs (and absence of these bugs after fixes have been implemented).
Two such recorded files are already included in a test suite, see https://github.com/plusonelabs/calendar-widget/tree/master/app/calendar-widget/src/androidTest/res/raw
At the moment replaying of "multi_day.json" file by the Test suite helps to see and to fix bugs #188 and #194

Using automated test cases created, all known bugs are fixed. We need confirmation from users to close corresponding issues.

Tip for future improvements:
In order to replay recorded results using original Time Zone, we will need this advices: http://stackoverflow.com/questions/9397715/defaulting-date-time-zone-to-utc-for-jodatimes-datetime

The "Time Zone Lock" feature tested to work as expected internally: it is now a part of a test suite, used when we need to replay a list of events in a specific time zone.

Done years ago...


Retrieve the next earnings date for a ticker but did receive a float not a date

My idea was to retrieve the next earnings date for a ticker so I checked your code and I think I found the corresponding column to use.
So I tried the customer screener with this query:
Python code used:
tmp = (Query()
.select('name', 'earnings_release_date','Recent Earnings Date','earnings_release_next_date')
.get_scanner_data())
df = tmp[1]
df[df.name == 'PFE']['earnings_release_next_date']
but getting floats instead of dates:
  ticker name  earnings_release_date  earnings_release_date earnings_release_next_date

48  NYSE:PFE  PFE           1.706615e+09 1.706615e+09                1.714478e+09
Can you please help/fix?

Sure, so that number is a Unix/Epoch time. You can convert it to a datetime using the datetime.datetime.utcfromtimestamp().
_, df = Query().select('price', 'earnings_release_date').get_scanner_data()
df['earnings_release_date'] = df['earnings_release_date'].map(dt.datetime.utcfromtimestamp, na_action='ignore')
>>> df

          ticker        close earnings_release_date
0    NASDAQ:NVDA     672.8286   2023-11-21 21:21:00
1    NASDAQ:TSLA     183.1301   2024-01-24 21:19:00
2       AMEX:SPY     492.2399                   NaT
3     NASDAQ:QQQ     425.5300                   NaT
4    NASDAQ:SMCI     648.0000   2024-01-29 21:06:00
5     NASDAQ:AMD     166.6300   2024-01-30 21:15:00
6    NASDAQ:META     457.3000   2024-02-01 21:05:00
7      NYSE:PLTR      21.5800   2024-02-05 21:05:00
8     NYSE:BRK.A<PHONE_NUMBER>   2023-11-04 21:11:00
9    NASDAQ:AAPL     188.5027   2024-02-01 21:30:00
...

Your query uses a wrong field 'price', should be 'close'. But thanks for the clarification.
I know observed that not always data gets returned.
Today for eaxmple for CRUS
df[df.name == 'CRUS'][['name', 'close', 'earnings_release_date', 'earnings_release_next_date']]
Empty DataFrame
Columns: [name, close, earnings_release_date, earnings_release_next_date]
Index: []
but on TradingView Earnings View/Screener page they are listed for today after Market

Good catch. Glad you got it working now.
As for the ticket not appearing in the result, please open a separate issue for that.


[Server] Implement Gradle subprojects and scaffolding for the Python server.

Motivation and Context
See https://github.com/awslabs/smithy-rs/issues/1367.
Description
Implement Gradle subprojects and scaffolding for the Python server.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

There is a failing CI action, but it is expected as I am creating a bunch of new projects and the diff tool will need them on main before being able to generate the diff.


Merge revision into develop

No conflicts.
New develop files updated in: d1ad868ae460fe6f7e8d77ada66c187b2c28a28d

test me

Build Passed :ok_hand:
You can safely merge 'merge-revision-into-develop' into 'develop'.

:+1:


MenuItem returns null stage on ClickListener

Try adding a MenuItem with a ClickListener that prints its Stage. null will be returned each time. It would seem sensible to return Stage from the menus that the item is currently in (recursively, returning the first found stage instance) if the it is not set internally. After all, the item itself is added to the stage - but simply not shown just yet. (But, in this case, the item is shown, as you can click it - and it still seems to have no stage.)
See https://github.com/czyzby/gdx-lml/issues/13.

This SSCCE prints clicked, stage null: false. Perhaps I don't understand something but it seems to work correctly, when the listener is called menu is still added to stage so it isn't null.

gdx-lml-vis also uses addItem and addMenu methods to create menus, but the MenuItem reports a null stage. I added a workaround on my part, not sure what to suggest to reproduce this issue other than trying to use gdx-lml-vis-tests project.
Add this method in MainView:
    @LmlAction("print")
    void print(final MenuItem item) {
        System.out.println(item.getStage());
    }

Copy and parse this template, try clicking on the item:
<menuBar>
  <menu text="menu">
    <menuItem>submenu
      <subMenu>
        <menuItem onClick="print">item
        </menuItem>
      </subMenu>
    </menuItem>
  </menu>
</menuBar>

As I've said, gdx-lml-vis seems to be using the standard API, so either there is some weird bug in LML or you haven't considered a corner case. Maybe try adding root to Stage after menuBar is already in it?

After moving that line it still works.
I added System.out.println(item.getStage()); in render. Looks like as long MenuItem is visible in stage, item.getStage() is not null. Right after clicking it, menu gets auto removed and it prints null again which is expected. I don't think I missed something, after all stage (un)setting is managed by libgdx.
I haven't tried reproducing with gdx-lml-vis-tests yet.

I don't think I missed something, after all stage (un)setting is managed by libgdx.

That's what I thought, so I was pretty much surprised to see this kind of bug. Not sure what's causing this, as LML seems to properly use menus API and the items are properly rendered on the screen. See https://github.com/czyzby/gdx-lml/issues/13#issuecomment-209687239

For some reason lml method is called after item has it's stage set to null. In my sscce it is called before that. Initially I thought this was because of using submenu but that's not it. I don't know, I can't reproduce it with just VisUI.

I'm just guessing here, but you could try the following order before we give up on this:

Add menu to table.
Add item to menu.
Add table to stage.

Nope, I tried both and still nothing. Replacing ChangeListener with ClickListener has the same result. Not sure what's causing this. The workaround on my end should be enough for gdx-lml-vis users, and I don't think it affects the rest.
By the way, when using ChangeListener in lml, the correct stage seems to be reported. This is some weird issue with listeners order (I think).
Adding some note about preferring ChangeListeners over ClickListeners in MenuItem docs should be enough. I think this issue can be closed.

ChangeListener should always be preferred for every subclass of Button otherwise you don't support setDisabled. ClickListener doesn't care about it.
This can be related to listener order, lml listener is called right after item.setStage(null) was called by libgdx.
I'll update docs.

ChangeListener should always be preferred for every subclass of Button otherwise you don't support setDisabled.

I know, but you got to admit that ClickListener seems more fitting (judging by its name) for all clickable widgets. I'm pretty sure this was the one that I usually used when I was just starting with LibGDX. In this case, we're got more to worry about that just the disabled status, so I think the javadoc note was justified.

Unfortunately it's more fitting. I've seen issues people have then they used ClickListener and were reporting than they still get events for disabled buttons.


version 0.0.10

Make packages work the same as the web client
Test Plan

[] Ensure works locally (download and install into Alfred /worksflows folder)
here is mine for Alfred 2 /Users/YOUR NAME/Library/Application Support/Alfred 2/Alfred.alfredpreferences/workflows
[] Ensure packages work (no 404s that the web makes successfully (srclib-go is broken on web for some things) )
[] Ensure functions did not break with change

Add a comment explaining that Repo is the correct value for Repo, and that we override if we actually see it is something else.
Then LGTM, merge this puppy!


compose typeahead: Don't autocomplete for suffixes followed by space.

Fixes #8279
Ready for review!

As shown in demo, it only prevents for @, : and # typeahead(if they followed by space) not for "```" and "~~~". We can prevent for both, if we want.

@timabbott Thanks for reviewing! Updated.
Autocomplete only matches query with space(:word text) if words before space(word) match with the item's words.

@shubhamdhama would you be up for reviewing this?

@YJDave the tests look great.  It seems like there's some common logic here that could be usefully factored out into a function that you call 4 times, namely this block:
+    if (query.indexOf("_") > 0) {
+        var queries = query.split("_");
+        var i;
+        for (i = 0; i < queries.length - 1; i += 1) {
+            if (emoji.emoji_name.toLowerCase().split("_")[i] !== queries[i]) {
+                return false;
+            }
+        }
+        query = queries[i];
+    }
+    return (emoji.emoji_name.toLowerCase().indexOf(query) !== -1);

(with the  emoji.emoji_name.toLowerCase().indexOf(query) part being an argument)

@shubhamdhama Thanks for reviewing!  Updated with common function, and add one comment. Let me know your suggestions!

I can still see this problem(I'm not sure it's an issue or not )

@shubhamdhama no, that's not an issue, that's how our solution should work. If the query before space match perfectly with the any value, then and only then we should autocomplete. Therefore pinging @rishig to make sure.

Another refactoring note here could be placing the loop for j outside and placing the loop for i inside(It makes less repetition of statement match_for_all_attrs = false; and corresponding if statement.

I am not sure we can do that, cause this how loop works:

It checks if query has " " in it(i.e.@iago), if not then directly find index of query in source attributes using indexOf function.
If query does have " " (i.e. :king of ham) then we will split keyword with " " so, it will be king, of and ham. Now, we will check if king and of exactly matches(in order) with any source's attributes(full_name/email/description, ..) if they do, then pass. If they don't then return false.
And then again find index of last word ham in source's attributes last word (i.e.hamlet), and return that.
So we can't interchange for loop. We can't loop over attributes, we have to run for loop on queries and check if it matches with at least one attribute's value in source.

Ohh, great :smile: if it's not an issue then this LGTM.

@shubhamdhama Thanks for reviewing :)
@timabbott Waiting for your final review.

Thanks for all your work on this!  The tests are great; I added a couple more asserts for some corner cases I was interested in, but overall this is gerat.  While reading this, I noticed the actual code for the core function was really hard to read, so I rewrote it; the resulting algorithm is much easier to understand on reading.  See 5edbcb87fd40ef87fc4f72bf96a2c5af4b64f0ba for the rewrite.
Merged, thanks @YJDave and @shubhamdhama!  It's definitely worth taking the time to take a function that works and make it nice and readable, especially with one that has a role as important as this one.


Hard refresh needed when HMR expected

This is very simple react component run from 'pages/index.js' (sorry, but I don't know why the code is not formatted properly in this example).
If I change input value to, say, 'click Me' then the page is HMR'ed and changes are immediately reflected in the browser. However, if I change console.log in the handler() to, say, 'some other message' then changes are not reflected in the browser and I have to do a hard refresh to pick up the changes. I would think that this change should be picked up but I could be wrong and it's normal.
export default class Test extends React.Component {
    handler = () => { console.log('some message')}
    render () {
      return ( <input type="button" value="click" onClick={this.handler} />)
   }
}

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

If I change console.log('message') to console.log('some message') and save the file and click the button then I would expect this changes to be picked up by HMR and see 'some message' in the browser's console output

Current Behavior

I don't see these changes and I have to do a hard refresh in order to see them in the browser.

Steps to Reproduce (for bugs)

Just run the sample code provided and change variables as described.

Context

It's not affecting me in any meaningful way. I can do a hard refresh if that's by design.

Your Environment

Next.js version: 3.0.1-beta.8
Environment name and version (e.g. Chrome 39, Node.js 5.4): Node 8.x, NPM 5
Operating System (Linux, maxOS, Windows): OSX Sierra

Use a normal class method like this for the handler
import React from 'react'

export default class Test extends React.Component {
    handler ()  {
      console.log('some message 44')
    }

    render () {
      return ( <input type="button" value="click" onClick={() => this.handler()} />)
   }
}

react-hot-loader doesn't work well with function defined as class variables.


Use (test)pypi in Trusted Publishing placeholder for GitHub Environments

GitHub Environments is a confusingly explained feature within GitHub that represents deployment targets. When projects get uploaded to PyPI — that a deployment target; same for TestPyPI. They don't represent processes but server-like entities. So using release is conceptually incorrect and gives people the wrong idea of what it is.
This is actually connected to Deployments API (and corresponding events) on the GitHub platform. The name Environments is just a misleading interface to describe Deployments that appears in some parts of the ecosystem, like GitHub Actions CI/CD.
In other places, it's called deployments and there's even a tab in repositories using it: https://github.com/cherrypy/cheroot/deployments/pypi. Each deployment can be linked to the corresponding released project version URL.
This patch attempts to align the practices with those used in the PyPUG guide and GitHub docs/starters: https://packaging.python.org/guides/publishing-package-distribution-releases-using-github-actions-ci-cd-workflows/ / https://github.com/actions/starter-workflows/pull/2345.
Additionally, this patch deletes the i18n marker, as I believe it shouldn't have been made translatable in the first place.

@woodruffw we talked about this in the past, I finally got a minute to try to update it. Are there any other places?

@woodruffw we talked about this in the past, I finally got a minute to try to update it. Are there any other places? Also, feel free to take over if something needs updating. I made changes from browser, so there's no translation template file updates.

Thanks for doing this! I think those are the main places.
If you'd like, you can give me the commit bit on your fork and I'll run make translations. Otherwise, a PyPI admin will need to do it (or recreate a new PR).

Oh, I thought ✅ Allow edits and access to secrets by maintainers took care of that..

@woodruffw invited!

Oh, I thought ✅ Allow edits and access to secrets by maintainers took care of that..

Only for real maintainers, I am merely a lowly triager 😉

Not sure if the same makes sense for GitLab. I haven't used it in a while so I didn't attempt updating. Somebody who knows the GL concept mapping should evaluate that, I think.


Add to Liquibase Package Manager

I'd really like to do lpm add liquibase-data -g.

Done!

https://github.com/liquibase/liquibase-package-manager/blob/372324ea50d918e71e816f7a30ec6544b8855772/internal/app/packages.json#L15


Needs GHC >= 7.8 due to GHC.Conc.threadWaitReadSTM

I've revised existing versions (http://hackage.haskell.org/package/socket/revisions/) so there is no need for a new release.
Before:

After: http://matrix.hackage.haskell.org/package/socket

Thanks a lot :-)


Remove Ingestion 2

This changeset removes various files and settings for our discontinued Ingestion 2 system (Krikri and Heidrun).
A user should be able to run any of the playbooks on a legacy inventory (e.g. development) without anything changing.
This will require a squash before it's merged -- just let me know and I'll do it!

Seriously, with 107 files ...
I will squash this.

Squashed.


Phrases unsupported in relevant text search within posts

Phrases like "black hole" are currently unsupported in searches for relevant text within a post - which is a shame as some can be really good choices to include. Currently, only single words can be searched for.
Could be fixed at the same time as #10.

Unlikely to be worked on as the Astronomy feed has been doing really well lately without any phrases or keywords (it just uses the emoji and hashtags), so this isn't necessary.


refactor: rename send to transfer.

Description
Rename send to transfer.

@yenkhoon You forgot to rename  NewSendTx to NewTransferTx

Okie i will change it

@yenkhoon Thanks


Custom folder icons

Closes #1015

@gave92 Is necessary to add this file: Files.Launcher/Assets/FilesOpenDialog/CustomOpenDialog64.dll ?
It seems to me that it's from the Open dialog when Files is default file explorer, is necessary to be added for some reason? (I don't mean to be added on this PR, but added to the repository)

No it can be added to .gitignore

The default dll is working well, but if you manually select another dll, even if it's the shell32.dll, it will throw an exception:

@BanCrash it should not throw an exception any longer, but I was not able to reproduce the .dll failing to load the icons, is it still the case?

@BanCrash it should not throw an exception any longer, but I was not able to reproduce the .dll failing to load the icons, is it still the case?

Nope, I've tried yesterday with a custom dll downloaded from github and the windows standard dll and they worked as expected.

@duke7553 @gave92 thank you for your work on this.


Add Upstream-Status in patches and remove unnecessary patches.

Add Upstream-Status in patches and remove unnecessary patches.
linux : remove disable-explicit-reloc-for-gcc12.patch for gcc13 has been supported.
diffutils: can build whithout diffutils-loongarch.patch now.
other: add Upstream-status  in patches.

@loongarch64/dev-team


refactor: split issue state types and document pullrequest state type

Split Issue Types to match the schema documented at https://docs.github.com/en/rest/issues/issues?apiVersion=2022-11-28#get-an-issue
Add comment to PullRequestStateType as merged and draft are not officially part of https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#get-a-pull-request

merge conflicts now resolved

These small changes are incredibly easy to review, thank you!


Better error response from Flutter testing/driver APIs

Currently when the error occurs when talking to the Flutter testing/driver APIs, the API returns a text containing error message and traces. It would be brittle if Espresso/EarlGrey relies on passing the error text to interpret the error.
Two examples:

When no matching UI element is found:
JSON-RPC Request sent to uri ws://<IP_ADDRESS>:45257/Q-Bxibr0quE=/ws:

{
  "id": "message-2",
  "method": "ext.flutter.driver",
  "params": {
    "isolateId": "isolates/root",
    "text": "Increment2",
    "finderType": "ByTooltipMessage",
    "command": "tap",
    "timeout": 10000
  },
  "jsonrpc": "2.0"
}

JSON-RPC response received:
 {
  "jsonrpc": "2.0",
  "result": {
    "isError": true,
    "response": "Timeout while executing tap: TimeoutException after 0:00:10.000000: Future not completed\n#0      FlutterDriverExtension.call (package:flutter_driver/src/extension/extension.dart:184:31)\n<asynchronous suspension>\n#1      BindingBase.registerServiceExtension.<anonymous closure> (package:flutter/src/foundation/binding.dart:512:32)\n<asynchronous suspension>\n#2      _runExtension (dart:developer-patch/developer.dart:84:23)\n",
    "type": "_extensionType",
    "method": "ext.flutter.driver"
  },
  "id": "message-2"
}

When there are multiple UI element matches:
JSON-RPC Request sent to uri ws://<IP_ADDRESS>:38582/Bc3XMX9_8wk=/ws:

{
  "id": "message-2",
  "method": "ext.flutter.driver",
  "params": {
    "isolateId": "isolates/root",
    "text": "Increment",
    "finderType": "ByTooltipMessage",
    "command": "tap",
    "timeout": 10000
  },
  "jsonrpc": "2.0"
}

JSON-RPC response received:
{
  "jsonrpc": "2.0",
  "result": {
    "isError": true,
    "response": "Uncaught extension error while executing tap: Bad state: Too many elements\n#0      Iterable.single (dart:core/iterable.dart:554:24)\n#1      WidgetController._getElementPoint (package:flutter_test/src/controller.dart:612:47)\n#2      WidgetController.getCenter (package:flutter_test/src/controller.dart:584:12)\n#3      WidgetController.tap (package:flutter_test/src/controller.dart:256:18)\n#4      FlutterDriverExtension._tap (package:flutter_driver/src/extension/extension.dart:324:19)\n<asynchronous suspension>\n#5      FlutterDriverExtension.call (package:flutter_driver/src/extension/extension.dart:181:53)\n<asynchronous suspension>\n#6      BindingBase.registerServiceExtension.<anonymous closure> (package:flutter/src/foundation/binding.dart:512:32)\n<asynchronous suspension>\n#7      _runExtension (dart:developer-patch/developer.dart:84:23)\n",
    "type": "_extensionType",
    "method": "ext.flutter.driver"
  },
  "id": "message-2"
}

Proposal:

Using error codes in the responses to better present the errors;
Possibly clean up the error messages to make it more user-readable.

@goderbauer

Adding error codes sounds good. Unfortunately, most exceptions are currently just caught by a catch-all. It would be very helpful if you can post all exception types you run into on this bug and I can then go through them to give them specific error codes.

A few off the top of my mind:

Timeout;
No matching widget;
Multiple matching;
...

I think we could start with these three.


Cannot Run Tests

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp

kitty ipsum

kitty ipsum text kitty

@nhuthuynh Your code is incomplete. Please visit the Help Room for help with the challenge. Happy Coding!


No arg order

This defers processing of cipher,digest,rand,key-options until after the argument processing is done so that providers
specified on the command line can be loaded first.
I believe that this fixes #14069.
Do we need a CHANGES entry?
Using "stacks of strings" to collect options and then parse them later is gross.  All you need is something argv-like. But that's not needed for 3.0 so it will have to wait.

fixup commit pushed addressing @t8m's feedback.

app_RAND_load() doesnt seem to be added to few of these files (genpkey, rsa & storeutl). Is this deliberate?

app_RAND_load() doesnt seem to be added to few of these files (genpkey, rsa & storeutl). Is this deliberate?

i only changed the files that should already had the -rand/-writerand flags, I didn't add it to others.  So yeah, kinda.

There were some additional commits so @paulidale do you still approve?

Merged to master, thanks for this.


image/color.NYCbCrA.RGBA(): possible loss of information (Go 1.6)

image/color.YCbCr.RGBA() uses a custom code for the conversion YCbCr to RGB, and doesn't use YCbCrToRGB().
The comment in the method says:
	// This code is a copy of the YCbCrToRGB function above, except that it
	// returns values in the range [0, 0xffff] instead of [0, 0xff]. There is a
	// subtle difference between doing this and having YCbCr satisfy the Color
	// interface by first converting to an RGBA. The latter loses some
	// information by going to and from 8 bits per channel.

Go 1.6 adds a new image|color type: NYCbCrA.
But image/color.NYCbCrA.RGBA() calls YCbCrToRGB().
YCbCrToRGB() returns 8 bits per channel, I think there is a loss of information here.

@nigeltao thank you! 😀


HxGrid.Defaults Page Navigation Settings

I use Font Awesome on my websites (leaving out the Bootstrap icons) and therefore need to set custom page icons for every HxGrid I use. I would like to see HxGrid.Defaults contain settings for these (like it does for the sort icons). May I recommend: FirstPageIcon, LastPageIcon, NextPageIcon, PreviousPageIcon?

These has to be GridSettings.PagerSettings as they have to be passed to underlying HxPager.
We need to add customizability to the HxPager it is currently hardcoded:

FirstPageIcon parameter
LastPageIcon parameter
NextPageIcon parameter
PreviousPageIcon parameter
PagerSettings class
HxPager.Settings parameter

add also CssClass
add also NumericButtonsCount (rename original DisplayNumberCount parameter ⚠)

HxPager.Defaults static property

I also marked TotalPages and CurrentPageIndex using [EditorRequired].
I doubt about HxGrid.PagerCssClass, should or should not it be removed? (Currently it is replaced by HxGrid.PagerSettings.CssClass.)

PR #310.

Thank you @blimkemann for your sponsorship!


[Feature Request] Update the usage of MySQL in the doc of Azure Spring Apps

Context
With all the changes we made to support credential-free connections:

https://github.com/Azure/azure-sdk-for-java/issues/29218
https://github.com/Azure/azure-sdk-for-java/issues/29785
https://github.com/Azure/azure-sdk-for-java/issues/28685

We should also update docs of Azure Spring Apps using MySQL to ensure they will use credential-free connections.
Goal
Update this doc https://docs.microsoft.com/en-us/azure/spring-apps/tutorial-managed-identities-mysql to include credential-free connections.
replace this doc with new tutorial (asir)

Replace the doc https://docs.microsoft.com/en-us/azure/spring-apps/tutorial-managed-identities-mysql with https://learn.microsoft.com/en-us/azure/developer/java/spring-framework/deploy-passwordless-spring-database-app?toc=%2Fazure%2Fdeveloper%2Fintro%2Ftoc.json&bc=%2Fazure%2Fdeveloper%2Fintro%2Fbreadcrumb%2Ftoc.json&tabs=mysql

PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/213151

PR has been merged, close it now.


NavigationView Tooltips on touch screens

Is your feature request related to a problem? Please describe.
When using a touch screen as the input method tooltips are not particularly useful. They some times though appear when a user touches one of the navigation elements. The tooltip then stays on the screen even if the user touches something. It is a minor inconvenience really but it does confuse some users.
Describe the solution you'd like
It would be nice if there was a way to disable all tooltips as a property on the NavigationView. I see that in NavigationViewItem there is a property already which enables/disables tooltips so I think part of a solution is already there.
Describe alternatives you've considered
This might be an issue with the way Avalonia treats touch inputs which erroneously triggers a mouse hover and thus triggering the appearance of the tooltip.
Hopefully something can be worked out and thank you for the work you do on this library.

This might be an issue with the way Avalonia treats touch inputs which erroneously triggers a mouse hover and thus triggering the appearance of the tooltip.

This sounds like a bug in the touch-related code in Avalonia. It might be better to create a minimal repro using just Avalonia controls (like a ListBox or Button with a ToolTip - this just keeps the example simple) and open an issue upstream


Highlights template does not accept changes

The Highlights template does not accept my changes. I changed the "set callout =" to be "note" (red), "important" (green). However, the default remains: "danger" (red), and "check" (green). What do I need to do to get my callout types?

Please provide your full template content to let me better understand the problem. Thanks.

I apologize. The changes were made after I relaunched Obsidian and re-synced.

I apologize. After I closed and opened Obsidian, and resynced, all the new callout types I set in the template work as expected. I  do have another issue regarding header colors; I'll open a new issue for it. Again, I apologize for the false alarm.


precalculate gates for AddInteger

When an AddInteger operation is performed, the conditions for whether or not to add an R gate are known once the Integer is known, hence they can be computed before the circuit is run. This leads to 1 gate per qubit, which almost always will speed up performance.

fixed by #81


Apply strabospot2 look and features to desktop interface?

Will the web-based desktop interface be updated to the StraboSpot2 map interface? That would be nice, there are some great SS2 features that would be nice to use on the more powerful desktop environment. Like the tag color for spots and being able to turn off the spot number.

Web interface for StraboSpot2 released in December 2023.


Where are the docs for writing source analyzers outside VS (for cross-platform CLI)?

I'm not a typical Visual Studio User(because I'm not a full-time windows user). I need to write an analyzer. But all the docs explain how to write analyzers with VS. There is no documentation on how to get started building an analyzer on CLI. How to set up the project? It's really frustrating. Most languages' documentation first explains how to do things on CLI without extra tooling. The IDE documentation is totally separated. These things still give the feeling that .Net is not fully cross-platform.
Yes, it's cross-platform, but all the documentation is about VS, which is a windows-only product. So you need to use VS on windows to get the full gist of .NET.
The docs really should be only about the .NET and doing things on every platform. Yes VS is a good product. But there are many people who like to use CLI or other IDEs. I think C# really went in the wrong direction if you compare it to other languages. In most languages, everything is CLI first. But in C# everything is VS first.
 But all documentation regarding Roslyn should be written as they could be followed on any platform.

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

ID: 6e3fc905-7aa1-7f5e-9bfe-fbcef461ad5e
Version Independent ID: a286e20b-1e43-dc57-e0fb-5a9dee7ec3f6
Content: Tutorial: Write your first analyzer and code fix
Content Source: docs/csharp/roslyn-sdk/tutorials/how-to-write-csharp-analyzer-code-fix.md
Product: dotnet-csharp
Technology: csharp-roslyn
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

adding @KathleenDollard
We should discuss the strategy for this.


Fix a bug that causes task status updates not correctly sent when the task is killed.

Testing: Updated the test in 57857ee which previously logs a TASK_LOST but we never checked it.

:+1:


Hide empty test suites when planning or executing

No reason to display the test suites if they are empty.

Relevant https://github.com/mar10/fancytree/wiki/ExtFilter

Better to do this after we have system-wide settings. This way we can allow users to enable/disable this feature.


stop vispy error catching

Description
As had been previously observed in our own event system which is based on the one from vispy, the default behavior of the event system is to ignore errors. We want this behavior turned off so that we know about errors. This PR now does this for the vispy event system on the canvas.  Doing so in #544 exposed some windows failures that might actually be causing real user errors that we were previously missing - see #527 and #428.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

Our windows build now fails, which is great news as this error was happening in the wild - see #527 and we weren't really aware of it. Now we can have some hope that if our windows tests start passing again napari will actually work for our windows users.
Here's the stack trace that we get when our tests fail:
---------------------------- Captured stderr call -----------------------------
WARNING: Error drawing visual <Image at 0x1200da71da0>
Qt exceptions in virtual methods:
________________________________________________________________________________
Traceback (most recent call last):
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\app\backends\_qt.py", line 818, in paintGL
    self._vispy_canvas.events.draw(region=None)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\util\event.py", line 455, in __call__
    self._invoke_callback(cb, event)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\util\event.py", line 475, in _invoke_callback
    self, cb_event=(cb, event))
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\util\event.py", line 471, in _invoke_callback
    cb(event)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\scene\canvas.py", line 217, in on_draw
    self._draw_scene()
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\scene\canvas.py", line 266, in _draw_scene
    self.draw_visual(self.scene)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\scene\canvas.py", line 304, in draw_visual
    node.draw()
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\scene\visuals.py", line 99, in draw
    self._visual_superclass.draw(self)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\visuals\visual.py", line 443, in draw
    self._vshare.index_buffer)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\visuals\shaders\program.py", line 101, in draw
    Program.draw(self, *args, **kwargs)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\gloo\program.py", line 533, in draw
    canvas.context.flush_commands()
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\gloo\context.py", line 175, in flush_commands
    self.shared.parser.parse([('CURRENT', 0, fbo)])
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\gloo\glir.py", line 819, in parse
    self._parse(command)
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\gloo\glir.py", line 743, in _parse
    self._gl_initialize()
  File "c:\users\containeradministrator\anaconda\lib\site-packages\vispy\gloo\glir.py", line 851, in _gl_initialize
    if this_version < '2.1':
  File "c:\users\containeradministrator\anaconda\lib\distutils\version.py", line 52, in __lt__
    c = self._cmp(other)
  File "c:\users\containeradministrator\anaconda\lib\distutils\version.py", line 335, in _cmp
    if self.version == other.version:
AttributeError: 'LooseVersion' object has no attribute 'version'

This trace has been discussed in a number of vispy issues, including https://github.com/vispy/vispy/issues/1503, https://github.com/vispy/vispy/issues/1342, https://github.com/vispy/vispy/issues/1450, and most recently https://github.com/vispy/vispy/issues/1688, but never seems to be have been fully resolved.

Thanks for approving, but I think I'd rather hold off on the merge - it's such a trivial change we can make it at any point, and it will only cause all our windows builds to break and code-cov not to run. I will try and get my hands on a windows machine next week and investigate there.

@sofroniewn just reviewing open PRs and found this... Which I thought was the reason our windows builds were failing! :man_facepalming: Should we just merge, since windows isn't working anyway?

yup, will merge

Hey folks, sorry if I'm a little late to the game here and this is all known... but I was just playing a bit with windows builds both on a local Win10 machine (where the tests in the master branch always pass) and on Cirrus, where they of course fail.  I found that enabling the pytest setting qt_no_exception_capture = 1 allows the builds to pass on Cirrus (example).  I'm not that familiar with this pytest-qt virtual methods issue, and it's still not clear to me whether qt_no_exception_capture is tantamount to just skipping all the important tests anyway (as this PR was trying to avoid in the first place), or whether those exceptions would be unlikely to be napari bugs (making it a potentially useful option).  But since I didn't see that pytest option discussed anywhere in these issues or in the vispy ones, i just wanted to check

I don't really understand what they're saying, nor why disabling this would actually cause the tests to pass rather than the interpreter to crash in a different way. It could be that their exception capture is actually broken, and that's the thing causing our crashes. For me that's the only reasonable explanation, since, according to that documentation, exceptions raised in our own custom methods will crash the interpreter during testing, and that's clearly not what we're observing.
All this to say: I think it's probably ok to disable it! @sofroniewn?


mosparo shows cloudflare ip not the origin ip

When a user submits a form I see the Cloudflare servers ip not the ip of the users pc.

@Ratingthomas Thank you very much for the issue!

@Ratingthomas, I've added a workaround for this issue.
Can you please update to v0.3.8?

After the update, there is a file called .env.local.dist in the root directory of mosparo.
Please copy (or rename) this file to .env.local (with the dot at the beginning).
Edit the file .env.local and uncomment the correct line. For your case (CloudFlare), uncomment line 13 (the line after the text with # If you use CloudFlare...)
Submit a form and check the IP address in mosparo

We will add a better solution with the next major version (v0.4), where this setting (and settings for other providers than CloudFlare) are manageable in the administration interface instead of changing environment files.
I'm looking forward to your feedback.

Thank you very much, now I see the right ip address!


Test migration for Mac OS X ARM 64

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] 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.

@conda-forge-admin, please rerender

Prepare for https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/4931

Invalid configuration `arm64-apple-darwin20.0.0': machine `arm64-apple' not recognized
configure: error: /bin/sh config/config.sub arm64-apple-darwin20.0.0 failed


Changes Travis configuration

So that it's faster, and also it shows the green color. Closes #15

I like the green color.  Sorry I missed this.


feat: discover

Layout only:

Icons are provisory

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4ec80f0) 48.26% compared to head (57e662e) 48.28%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   48.26%   48.28%   +0.01%
==========================================
  Files         142      142
  Lines       17211    17211
==========================================
+ Hits         8307     8310       +3
+ Misses       8904     8901       -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Generated tech debt:
Round shader for textures on TextureRects for Discover mainly and other places
added to: https://github.com/decentraland/godot-explorer/issues/197


tracking issue to continue v2 support for upgraded protos in Q1.

Title: Opening this issue to track  v2 support for protos in Q1.
Description: Opening this issue to track  v2 support for following protos in Q1.
refer #12841  for details.
protos:
"//envoy/config/filter/thrift/router/v2alpha1:pkg",
"//envoy/config/health_checker/redis/v2:pkg",
"//envoy/config/resource_monitor/fixed_heap/v2alpha:pkg",
"//envoy/config/resource_monitor/injected_resource/v2alpha:pkg",
"//envoy/config/retry/omit_canary_hosts/v2:pkg",
"//envoy/config/retry/previous_hosts/v2:pkg",

Describe the issue.

[optional Relevant Links:]

see #12841 for details

@htuch may be this issue can be reopen since it is closed automatically

@htuch may be this issue can be reopen since it is closed automatically


Broken cygwin (32 and 64-bit) builds using test, ::mkstemp, ::select, ::kill undefined with cxxstd=03

I've been adding CI environments to the CMT projects and there's a common problem I have found with cygwin targets on appveyor, they show up like this:
gcc.compile.c++ bin.v2\libs\property_map\test\compose_property_map_test.test\gcc-6.4.0\release\cxxstd-03-iso\target-os-cygwin\compose_property_map_test.o
In file included from ./boost/test/minimal.hpp:54:0,
                 from libs\property_map\test\compose_property_map_test.cpp:11:
./boost/test/impl/debug.ipp: In function 'const char* boost::debug::{anonymous}::prepare_gdb_cmnd_file(const boost::debug::dbg_startup_info&)':
./boost/test/impl/debug.ipp:424:23: error: '::mkstemp' has not been declared
     fd_holder cmd_fd( ::mkstemp( cmd_file_name ) );
                       ^~
In file included from ./boost/test/minimal.hpp:54:0,
                 from libs\property_map\test\compose_property_map_test.cpp:11:
./boost/test/impl/debug.ipp: In function 'void boost::debug::debugger_break()':
./boost/test/impl/debug.ipp:711:5: error: '::kill' has not been declared
     ::kill( ::getpid(), SIGTRAP );
     ^~
./boost/test/impl/debug.ipp: In function 'bool boost::debug::attach_debugger(bool)':
./boost/test/impl/debug.ipp:892:34: error: '::mkstemp' has not been declared
     fd_holder init_done_lock_fd( ::mkstemp( init_done_lock_fn ) );
                                  ^~
./boost/test/impl/debug.ipp:929:9: error: '::select' has not been declared
         ::select( 0, 0, 0, 0, &to );
         ^~

It looks like missing includes, perhaps.
Here's a build job showing it.
https://ci.appveyor.com/project/jeking3/property-map/build/1.0.2-develop/job/9tt6av7lxe14qy8h
Is this a known issue?

This appears to happen only when cxxstd=03.  I am working on a fix.

Thanks for the report. It would be good that you add those issues on Trac - as explained in the contributions document -:  there are some compilation errors already reported by other people. I believe the one of ::mkstemp is already there.
I would be interested in any patch, since I do not have any cygwin knowledge. Thanks

I will work on a patch that fixes this on cygwin.  Too bad we cannot all use github issues. :)

So in general here, I think it would be best to check the value of __POSIX_VISIBLE if the platform is CYGWIN and output a better error message, telling folks they need to define _POSIX_C_SOURCE=200112L in order to use Boost.Test.  If the cygwin headers are included in the compilation unit first then Boost.Test cannot change them.

By adding a comment to the code that fails to compile, it makes it a lot easier for someone to understand what happened and how to fix it:
$ ../../b2 -q cxxstd=03
Performing configuration checks
...found 4452 targets...
...updating 458 targets...
gcc.compile.c++ ../../bin.v2/libs/test/build/gcc-6.4.0/debug/cxxstd-03-iso/link-static/debug.o
In file included from ../../libs/test/src/debug.cpp:16:0:
../../boost/test/impl/debug.ipp: In function ‘const char* boost::debug::{anonymous}::prepare_gdb_cmnd_file(const boost::debug::dbg_startup_info&)’:
../../boost/test/impl/debug.ipp:424:23: error: ‘::mkstemp’ has not been declared
     fd_holder cmd_fd( ::mkstemp( cmd_file_name ) ); // if mkstemp is not defined on Cygwin, define _POSIX_C_SOURCE appropriately
                       ^~
In file included from ../../libs/test/src/debug.cpp:16:0:
../../boost/test/impl/debug.ipp: In function ‘void boost::debug::debugger_break()’:
../../boost/test/impl/debug.ipp:711:5: error: ‘::kill’ has not been declared
     ::kill( ::getpid(), SIGTRAP );
     ^~
../../boost/test/impl/debug.ipp: In function ‘bool boost::debug::attach_debugger(bool)’:
../../boost/test/impl/debug.ipp:892:34: error: ‘::mkstemp’ has not been declared
     fd_holder init_done_lock_fd( ::mkstemp( init_done_lock_fn ) );
                                  ^~
../../boost/test/impl/debug.ipp:929:9: error: ‘::select’ has not been declared
         ::select( 0, 0, 0, 0, &to );
         ^~

Normally I would submit a PR, but folks should be able to figure this one out on their own.  A note in the documentation about cygwin seems appropriate however.  In any case I'm going to close this.  In each of the affected repositories I will set this macro as needed.


Refs #24121 -- Added repr() to SQLCompiler

Hello,
I'm not sure where I should write tests,
I have searched around if they are tests against   SQLCompiler but I couldn't find them.
If someone could check if this one is the most suitable place for the tests.
thanks

@saeedblanchette Thanks for this patch, however SQLCompiler.__repr__() is already handled in #14111. I updated the ticket description.


Maps not loading online

My map is working on localhost but not online. I have removed keys restrictions but still doesn't load the map online with no errors in the console.

@MachariaK
This should have something to do with your network.
Can you show me the network tab of Chrome Devtools?
Is your app accessible somewhere? so that I have a look at it myself?

Hello. I have sent the link and login credentials via Twitter. I couldn't find a way to send the details privately on here. Thank you in advance.

@MachariaK
Looking at your Dashboard, maps are properly loaded in your Dashboard.
You forgot to pass the style attribute to the GMapMap component

<GMapMap
      :center="center"
      :zoom="7"
      map-type-id="terrain"
      style="height: 100%; width: 100%;"
  > </GMapMap>

Or alternatively you should define .vue-map styles in CSS.

I had passed the style attribute and I don't know why it was not working. As you have advised, I have added the .vue-map style and the map can now show both online and on localhost. Thanks.


Add dark mode functionality

Closes #1

Wow, thank you! Very impressive how quickly you made this PR. Just a couple things if you wouldn't mind tweaking:

Buttons look a little bit weird when you hover over them in dark mode.

It's hard to read the users' names in the dark background. You could change the L part of the HSL to be higher in dark mode. The purple "Read the code" button is also hard to see.

White inputs and selection looks like a harsh color compared to the surrounding dark background.

Very minor, but could you capitalize the "Dark mode" heading so that it's consistent with the rest of them?

Thanks for the review, I'll take care of it when I wake up tomorrow.

@ekzhang I've made the requested changes, let me know if there's anything else :smiley:


XSS possible in data-container property of tooltip

XSS possible in tooltip data-container attribute
data-container="<img src=1 onerror=alert(123) />"
Bootstrap 4.1.1 in combination with JQuery 3.3.1

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and OS details.

https://jsbin.com/bimipayoda/edit?html,output
Browser: Chrome Version 66.0.3359.181 64x
OS: Windows 10 64x


[Bug and typo fix]

Job d2l-zh/PR-1077/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-1077/

Job d2l-zh/PR-1077/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-1077/


Support dynamically-sized lists in markup

For example a list of partners that need to be added
support a new format parameter that specifies the number of elements in the list e.g [[List of Partners: array[5] ]]
maybe use text on left side of colon as key and reuse key like other keys

could also use nested double brackets for the input 2d array so that the input is pushed back to the user

So what you're proposing is a way to "force" a number of elements in a list is that correct?
Because you have already the type Collection that let's you have a dynamic number of elements.
https://docs.openlaw.io/markup-language/#collection


Add contentypes to Tag

Fixes: #872

Add content_types to TagFilterSet, TagForm and TagTable
Create PrimaryModelRelatedContentType as an helper class to get ContentType models that are subclasses of PrimaryModel

With this as a draft, it's not clear to me how complete this is currently considered to be - I presume you're yet to implement the corresponding model logic to only allow assigning a given tag to a given object if the content_type is a permitted one for that tag?

Acceptance Criteria

[ ]  Similar to the status pattern already implemented, a tag constrained to a particular content type cannot be applied to other content types
[ ] Similar to the status pattern, a tag could be constrained to multiple content types
[ ] For migration, tags would by default be available to all content types
[ ] Enforcement should be done both in the UI & API
[ ] Should be able to filter tags by content types


refactor: simplify state pruning

Like #2152, this PR has been checked out of #2130.
This PR breaks a pruning down into a function with helpers, instead of a single large function.

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

@mzabaluev fyi, this has been updated to fix the missing batching.  Thanks!

this now needs rework, and I'm hesitant to do the merge because of the risk of missing something.


Validation for Confirm Button

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I am frustrated that I cannot customize the dialogue box in Buttons to validate something before confirming
Describe the solution you'd like
A clear and concise description of what you want to happen.
A callback to be called before action is issued
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Tried to use modal but to no avail
Additional context
Add any other context or screenshots about the feature request here.

I believe that for the implementation of this, the recently released Laravel package by precognition https://laravel.com/docs/10.x/precognition is likely suitable. At the moment, I don't plan to work on implementing it into the package myself, but I would be happy to accept a pull request (PR).


[ISSUE #6752] Prepare to release RocketMQ 5.1.1

Which Issue(s) This PR Fixes

Fixes #6572
Brief Description
Prepare to release RocketMQ 5.1.1
How Did You Test This Change?

Codecov Report

Merging #6753 (478b46a) into develop (9d411cf) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #6753      +/-   ##
=============================================
- Coverage      42.85%   42.85%   -0.01%
+ Complexity      9002     8999       -3
=============================================
  Files           1113     1113
  Lines          78615    78615
  Branches       10229    10229
=============================================
- Hits           33692    33688       -4
- Misses         40701    40704       +3
- Partials        4222     4223       +1

Impacted Files
Coverage Δ

...ain/java/org/apache/rocketmq/common/MQVersion.java
99.83% <100.00%> (ø)

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


Formatting for Lesson 5 is different than other lessons

Formatting for Lesson 5 is different
on the web page:
http://learn.openwaterfoundation.org/cdss-learn-git/05-lesson-new-repo/lesson-new-repo/
it appears that the formatting is lost. The lesson has content, but looks different than the other lessons.
This may be purely aesthetic, or something could be amiss?

I'm not seeing an issue.  Maybe it was due to me updating the files?  Please advise if this is still an issue or provide more specific feedback.

It is also now working for me. Going to assume something didn't get fully updated, but now it is running okay!


Some Kubeflow cluster base files missing

For example:
https://github.com/kubeflow/gcp-blueprints/blob/dbde75f8c627a3558d3b21216c5012e54a60635e/kubeflow/common/istio/kustomization.yaml#L4
When running make hydrate, it fails because it can't find the base files.


Children of instantiated scenes will appear with the same name and type of the root parent when hovered

Godot version:
baa3933
Issue description:

I note that but i think that bug is not with children, i think that simply the label is duplicated , it appears two times ever, no mathers circustances, if the node have the new placeholder cross

@Ranoller No it isn't. The number of times that the label appears is the same number of Node2D types in the scene:

When there's no children, only one label appears.

@groud
Test.zip

In fact those should not even be visible. I'll work on that.


Sync: Auth is broken

Today's insiders broke auth on my Windows box.
It tells me I'm not signed in to sync. I go through the whole flow. Once back in Code, I get this:

This was already opened 3 days ago: https://github.com/microsoft/vscode/issues/108362


[BUG] Problem with VMOVDQU8 opcode

In the assembly window, hovering the mouse over the VMOVDQU8 opcode does not give actual documentation on the opcode. Instead, the hover text says:
"Note: VEX.vvvv and EVEX.vvvv are reserved and must be 1111b otherwise instructions will #UD"
When right clicked and the opcode doc is selected, this note is above the actual documentation. I think this note needs to be moved below the doc to show up on hover.

see #1376


Implementing sar binary decoder

Hello
I want to implement a sar binary file decoder which converts sar binary into json format with sadf command. After that, it is converted into csv and passed to the csv parser.
It requires the entire file to be converted into JSON at once during guess. For this, the sample buffer provided during guess is insufficient.(Only 30K)
Any idea?
cpu-load-all.1.cpu.full,network.net-nfs.getatt,cpu-load-all.2.iowait,network.net-sock.tcpsck,power-management.cpu-weighted-frequency.3.number,cpu-load-all.1.cpu ...
0,181,1.98,74,0,0 ...

I noticed that it's wrong to write a decoder plugin in this situation. I'll try to extend LocalFileInputPlugin.
Thanks.

Hello
I think you can also use ParserPlugin.
For example.

embulk-parser-pcapng
embulk-parser-roo-excel

I'll comment later about guess.


PreventDefault for devtool button events

Hi there.
Thanks for your work on this. Super useful.
One issue I'm running into is that, where I'm using the devtool, clicking any button causes the page to reload. Is preventDefault set on the click event for those buttons? If not, could it be?

@dsifford, that happens when you click devtool panel button, or picking observable component in «inspect dependencies» mode?

is totally unbelievable, as those buttons are quite ordinary react components with onClick props https://github.com/mobxjs/mobx-react-devtools/blob/master/src/Panel/PanelButton.jsx#L49
shouldn't happen because capturing is being used https://github.com/mobxjs/mobx-react-devtools/blob/df0cd13ed2c708137d552f7d32d52febec045cb9/src/Controls/GraphControl.jsx#L19 and the handler stops propagation if clicked on observable component https://github.com/mobxjs/mobx-react-devtools/blob/1594b6181dfc6ef244890564547eef88e9a264c0/src/globalStore.js#L98

Could you provide more details on how to reproduce the issue?

@dsifford Hmm, I can imagine wordpress reloads after click on any button, just to make people suffer. Could you try placing <button>test</button> just after  component? Does it also leads to page reload? By the way, is it possible, that your DevTool panel is somehow placed inside <form></form> html element?

If the answer to the last question is yes, this commit should fix this https://github.com/mobxjs/mobx-react-devtools/commit/b673df5b53a225812919d2625c105add6b0b26f4 (mistyped issue number)

Yep, without even looking, that must be the case. The component is rendered within a metabox, which itself is completely wrapped in a form element.
Nice catch! I hadn't even thought of that. Thanks for the help 😄

Let me test part 1 of your question here real quick... Give me just a sec.

@dsifford, no need, it will reload page without type="button" :-)

@andykog Ok cool, thanks for the quick fix!

@andykog this fix is not released yet if I am correctly? feel free to do so :)

Published as v4.2.5


Clear warning from alerts when retry of PubMed, OMIM, etc results in successful response

We show a warning glyph next to a gene that failed to on an NCBI request (e.g. get PubMed entries, get NCBI gene, etc). When the user clicks on the gene a second time, the NCBI request will likely succeed. In this case, the 'warning' alert should be removed from the alert panel and the warning glyph should no longer show next to the gene button in the genes side panel.

To test, you can force an error with PubMed by clicking very quickly through the genes during or after 'analyze all'. The NCBI request will fail because we have exceeded the number of requests allowed in an interval of time (a minute?).
So you should see some warnings next to the genes in the side panel:

You can click on the warning glyph to see the warning in the alerts right side panel.


Convert test_layout module to py.test style.

Also move TEMPLATE_DIRS and TEMPLATE_LOADERS settings to test_settings module.

There was a bug:
self.assertTrue(html.count('1000'), 2)

And proper assert should be assertEqual.
In this branch this bug is fixed with conditional string check.


Windows build

@saeedvaziry
First, thank you for making this free and open-source alternative to tinkerwell, it's great!
Can we please have Windows binaries? It's electronjs so I think it won't be much work 😅
Thanks!

@aneesdev Even electronjs apps need to be tested and built properly so they can be ran without any problems on the end-user machines.
But Windows and Linux builds will come early 2025 🚀

The Windows build is blocked because phpactor doesn't support Windows at the moment
Need to find a workaround

One way is to use WSL, not sure every Windows developer have WSL ready on their machines


Create it-IT.json

https://jira.shopgate.guru/browse/CCP-802

Pull Request Test Coverage Report for Build 1773

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.226%

Totals

Change from base Build 1769:
0.0%

Covered Lines:
1513

Relevant Lines:
3446

💛  - Coveralls


[REVIEW] prakhartiwari0

GitHub Profile Link
https://github.com/prakhartiwari0

cool profile , loved the anime  refs , but do some changes , that's all , 😄

Less Gifs  more Pngs ( mostly becoz speeds )
too long 😢
tell a bit more bout yourself


[FSDE-10976] Fixed Data Table activePage issue

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-251/1/. :x:

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-251/2/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-251/3/. :white_check_mark:

You can check this PRs instance in https://02a28dae44eaf16f8b245ba3dd284161ae9ea5a3-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-251-ux-chi.kubeodc-test.corp.intranet/ (internal)


chore: Readme rewrite

General feedback from Brad:

I'd suggest we avoid direct comparisons to other tools, like the grid or  some of the content in the what sets it apart section. I'd probably send this a bit more toward a framing like "Goose is not looking to do code line completions or pair programming" or similar, and then say what it is trying to do. And leave the vs copilot, vs aider, vs cursor implied? And we can put lines about why we think that is a valuable approach/principle compared to other approaches
Maybe the examples can grow into a "what can you do with goose" section? where we have prompts like you have here, and then maybe talk through why we prompt it like that (effectively tips again) and how we have seen it solve that thing
a. we can probably manufacture more open examples btw, these are real which is great but some are block specific

Will reopen a new version of this. We plan to discuss how we compare capabilities to other related tools, and how we can best keep goose open for collaboration with them.


modify serac_add_tests to support pch

see: https://github.com/LLNL/serac/issues/567
This PR adds a new optional argument, PRECOMPILED_HEADER to the cmake macro serac_add_tests, that lets the user specify a header file to be precompiled and applied to each of the sourcefiles passed to serac_add_tests. This feature is opt-in:
code that does not pass this optional argument to serac_add_tests behaves as it did before (where each translation unit is compiled "from scratch").
Using functional_boundary_test.cpp as an arbitrary example, compilation times decreased significantly (6.0 seconds -> 3.0 seconds) by precompiling pch.hpp.
Some thoughts:

Each call to serac_add_tests that specifies PRECOMPILED_HEADER will compile that header file once. Since serac makes many calls to serac_add_tests, some headers may be precompiled multiple times. This is still an improvement to compile times, as those headers are processed once per call to serac_add_tests, as opposed to once per translation unit.
As a result of the point above, Serac would compile even faster if we made few calls to serac_add_tests with many sourcefiles as opposed to many calls to serac_add_tests with a couple of sourcefiles. Is it worth restructuring our tests to do this or not?
Right now, the specified header to be precompiled can be different for each call to serac_add_tests. Is this helpful, or should we just have a single precompiled header for all of serac, and make it globally applied to each translation unit?

Codecov Report

Merging #572 (465bda1) into develop (1bea77b) will decrease coverage by 0.79%.
The diff coverage is 86.73%.

@@             Coverage Diff             @@
##           develop     #572      +/-   ##
===========================================
- Coverage    95.61%   94.82%   -0.80%
===========================================
  Files          109      112       +3
  Lines         6552     6534      -18
===========================================
- Hits          6265     6196      -69
- Misses         287      338      +51

Impacted Files
Coverage Δ

src/serac/infrastructure/accelerator.hpp
100.00% <ø> (ø)

src/serac/infrastructure/initialize.cpp
78.57% <ø> (-0.74%)
:arrow_down:

src/serac/physics/solid.hpp
100.00% <ø> (ø)

.../serac/physics/utilities/finite_element_vector.hpp
100.00% <ø> (ø)

.../serac/physics/utilities/functional/quadrature.hpp
100.00% <ø> (ø)

...unctional/tests/functional_material_state_test.cpp
64.28% <0.00%> (-35.72%)
:arrow_down:

src/serac/physics/utilities/functional/tuple.hpp
100.00% <ø> (ø)

...physics/utilities/functional/boundary_integral.hpp
48.75% <25.00%> (-19.00%)
:arrow_down:

...ties/functional/tests/functional_boundary_test.cpp
87.38% <33.33%> (-12.62%)
:arrow_down:

src/drivers/serac.cpp
93.25% <66.66%> (-1.87%)
:arrow_down:

... and 26 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 f47bbcf...465bda1. Read the comment docs.


Suggestion for better decoupling/separation of concerns

Thank you for writing this very helpful HATEOAS library. I have a suggestion that I think might improve it. Please let me know if you think it's a good idea.
Currently whatever library contains the data transfer objects (DTOs) that are used to pass representations of domain objects between the server and the client must have a dependency on RiskFirst.Hateoas. Furthermore, every DTO class must inherit from RiskFirst.Hateoas.Models.LinkContainer, and every collection of DTOs must be an ItemsLinkContainer.
It seems to me that this is not ideal. We might (at least hypthetically) want to reuse those same DTOs in another part of the application that deals clients that aren't RESTful or aren't even using HTTP...perhaps a Console client or WinForms or WPF or something else entriely.
Because DTOs are part of the core, maybe they shouldn't be tightly coupled to a library that specializes in HTTP communication. I was wondering if LinkContainer could become LinkContainer<T> where T is your DTO class, thus keeping DTO separate from the HATEOAS object. Something similar could be done with the collection class ItemsLinkContainer<T>.
What do you think... Is it possible? Is it a good idea? Could it be done without negatively affecting the output when the LinkContainer<T> is serialized to JSON?

Hi,
The coupling-to-http problem is why this library was split into 2 packages
RiskFirst.Hateoas.Models only depends on newtonsoft.json which is usually ok for a DTO project.
RiskFirst.Hateoas is what you would use for the REST api end, which has all the dependencieson http/web stuff.
The dependency is actually only on an interface, so theres nothing stpping you implementing the functionality differently. If you wanted to write your own eg LinkContainer<T> it should be esy enough to do, and you can control extactly how you want it serialized to JSON.
I'm not sure we would add this to the library itself, as (I think) we've provided everything you would need to implement this for your own project - but by all means if you think there is something that would benefit the library itself feel free to submit it as a pull request.
Thanks


gototop button feature added

I have added the button. You can check it out.

https://github.com/Jagroop2001/AcmeRockets/issues/37

@debuuuu007 , you can also contribute to this project :
https://github.com/Jagroop2001/validify-string
If you want to

Yess i would love to contribute, lets talk about this in the project page itself.


Allow extensions to participate in tasks using commands

Currently the task framework can only trigger external commands (programs or shell scripts). Since Ctrl+Shift+B is taken by the task framework we should allow extension to participate in 'build' and 'test' (and any other composite task) by executing a command in the extension itself.

Duplicate of #11396?

Correct :-)


Custom themes

There are 2 parts to this:

a refactor of the internal css variables allowing for custom theme support (https://github.com/joeattardi/emoji-button/commit/2834f30289db904f4ddceedb6219cae732e2e14e)
the custom theme support itself, which is a thin wrapper that adds classes to the proper elements

One thing of note is that base themes are applied to the wrapper, while custom themes are applied to the picker.  This allows custom themes to extend base themes, while still overriding variables since they have the same specificity.
The one thing I didn't add to the documentation is a description of the available CSS variables for theming, but I think the names are self explanatory, and I provided an example which overrides every variable, which serves as a kind of reference.
Of course, let me know if you think anything should be added or changed.  Thanks!

Haven't forgotten about this, sorry, will try to take a look soon!

Currently I'm rethinking how theming will work for the upcoming v5 release.


JavaScript minification errors when trying to use JsMinify

Looks like "not all code paths return a value"
/* Minification failed. Returning unminified contents.
(400,6304): run-time error JS1004: Expected ';'
(400,6304-6305): run-time error JS1195: Expected expression: :
(400,6346-6347): run-time error JS1002: Syntax error: }
(400,6362-6363): run-time error JS1002: Syntax error: }
(400,6363-6364): run-time error JS1195: Expected expression: )
(400,6364-6365): run-time error JS1002: Syntax error: }
(400,6365-6366): run-time error JS1195: Expected expression: )
(400,6366-6367): run-time error JS1002: Syntax error: }
(400,6367-6368): run-time error JS1195: Expected expression: )
*/

From experience never compress a file already compressed it always generates errors.
Try using the uncompressed file in your project.


Remove plugin from README.md

This is not necessary and produces an error: ESLint couldn't find the config "react-refresh" to extend from. Please check that the name of the config is correct.

Are you using the new configuration format?
The doc clearly shows that adding the plugin to the plugin list is expected: https://eslint.org/docs/latest/use/configure/plugins#use-a-plugin
The error suggest that there is either an issue with the resolution or the installation of the package

I'm not sure what the "new configuration format" is and I'm not an expert at eslint so totally feel free to close! Just wanted you to know that following the Readme produced an error for me and offer to fix up the docs.

Are you sure the rules still works when removing the plugin entry? Are ou able to get a warning when a file export both a basic variable and a react component?


Update TOC.md

fixed advanced HR

Docs Build status updates of commit e7a30f9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 05a8c9d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/financials/MulticurrencyManagement.md
:warning:Warning
View
Details

dynamics-gp/payroll/AdvancedHumanResource.md
:warning:Warning
View
Details

dynamics-gp/financials/MulticurrencyManagement.md

Line 215: [Warning] Invalid file link:(~/financials/media/97e37b9faa5f7623033ae2a1b014d7ea.jpg).
Line 833: [Warning] Invalid file link:(~/financials/media/0f0af18c6d0fadcdc0d9ed0c11ea54c2.jpg).
Line 1340: [Warning] Invalid file link:(~/financials/media/image8.jpg).

dynamics-gp/payroll/AdvancedHumanResource.md

Line 515: [Warning] Invalid file link:(~/payroll/media/9fc14a3a23676cb02b8177f72b72a1d1.jpg).
Line 704: [Warning] Invalid file link:(~/payroll/media/aef7d41c75511a0b9c39b7e944629bb0.jpg).
Line 736: [Warning] Invalid file link:(~/payroll/media/4341819673bb483f27befe0e7bf70ece.jpg).
Line 772: [Warning] Invalid file link:(~/payroll/media/b1ed05943d5e7de57ef712998bb236ca.jpg).
Line 799: [Warning] Invalid file link:(~/payroll/media/faf062154592f53ea4f1a15d66bca5d4.jpg).
Line 853: [Warning] Invalid file link:(~/payroll/media/da6d7897048bebdeb57e7d4db16c7719.jpg).
Line 894: [Warning] Invalid file link:(~/payroll/media/1cd28ed5b5629e37581c7374f930f486.jpg).
Line 953: [Warning] Invalid file link:(~/payroll/media/442e889d717a4be412c2448f61e209cb.jpg).
Line 997: [Warning] Invalid file link:(~/payroll/media/31780b0c27992aefbc525bd2adbf7a77.jpg).
Line 1045: [Warning] Invalid file link:(~/payroll/media/aaab29d5481f1423098576df5af5f4ed.jpg).
Line 1132: [Warning] Invalid file link:(~/payroll/media/e200fd48fb3a0f3622532c5d5f5c7b1c.jpg).
Line 1370: [Warning] Invalid file link:(~/payroll/media/1ade2b756a776e7e0a648332366d701c.jpg).
Line 1439: [Warning] Invalid file link:(~/payroll/media/98a5dbc9b7beb772987bd6f60ceafdb3.jpg).
Line 1488: [Warning] Invalid file link:(~/payroll/media/994e2eac7970533aaad91c7453d9f2ba.jpg).
Line 1523: [Warning] Invalid file link:(~/payroll/media/390bb07315226e87967e6126fbeb485a.jpg).
Line 1638: [Warning] Invalid file link:(~/payroll/media/01f4a8079047950b6cc79763cae9b080.jpg).
Line 1794: [Warning] Invalid file link:(~/payroll/media/232fbd0fb458f93e27b3d063fe7c83c6.jpg).
Line 1837: [Warning] Invalid file link:(~/payroll/media/69da87abc6a0daeda941826219b9d7e4.jpg).
Line 1839: [Warning] Invalid file link:(~/payroll/media/84f06a0657467fa18d5077df794f1753.jpg).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Certain form categories overflow on mobile.

Overview
When navigating to the profile-page while viewing the page on mobile, certain div elements representing form categories overflow the page.
Reproduction steps

Ensure the screen size of the device being used is smaller than 695 px wide.
Navigate to the profile page.
Observe the Education and Work Experience categories overflowing the page.

Fixes
This happens due to date picker containers not being responsive and having a rigid size. They should instead use Flexbox and be collapsed on smaller screens.
Documentation
Screenshots

Still having this issue.


I made TextMate2 Bundle for ruff

Hi, if you are a dinosaur like me :) you should probably like this;
https://github.com/vigo/textmate2-ruff-linter
I'm not sure how many people are still using TextMate2 but I do :) (macOS only text editor)

Thanks so much for this! I can add a note in the docs.


Extract SaslClient/SaslServer logic from handlers as abstract classes

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/4417/
Test FAILed.

Jenkins, test this please

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/4420/
Test FAILed.

Jenkins, test this please

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/4423/
Test PASSed.

@bf8086 PTAL. Thanks!

@bf8086 PTAL.

@gpang PTAL. Thanks!

alluxio-bot, merge this please


Bugfix for async requests made through plugins failing making calls through agent.

This auth parameter being passed to the method results in a duplicate keyword argument since auth doesn't exist on the parameters.
This is an issue fix for https://github.com/langchain-ai/langchain/issues/7953
It was introduced as part of this change.
https://github.com/langchain-ai/langchain/commit/663b0933e488383e6a9bc2a04b4b1cf866a8ea94
@agola11  @baskaryan

@hwchase17 @baskaryan @agola11 can you please take a look

@baskaryan Please, review it


[column_with_row] BE crash when running pk table with storage_type = column_with_row cases

Steps to reproduce the behavior (Required)
The permanent case has not been found yet.
be.out:
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1703854915" if you are using GNU date ***
PC: @          0x2bc5e0c starrocks::BinaryColumnBase<>::append()
*** SIGSEGV (@0x4) received by PID 2706 (TID 0x7f17925e3700) from PID 4; stack trace: ***
    @          0x6627242 google::(anonymous namespace)::FailureSignalHandler()
    @     0x7f18462e0630 (unknown)
    @          0x2bc5e0c starrocks::BinaryColumnBase<>::append()
    @          0x59fb609 starrocks::ShortCircuitHybridScanNode::_process_key_chunk()
    @          0x59fcce1 starrocks::ShortCircuitHybridScanNode::get_next()
    @          0x59f8980 starrocks::ShortCircuitExecutor::execute()
    @          0x57a709d starrocks::PInternalServiceImplBase<>::_exec_short_circuit()
    @          0x57b2e15 starrocks::PInternalServiceImplBase<>::exec_short_circuit()
    @          0x684e04d brpc::policy::ProcessRpcRequest()
    @          0x67f5807 brpc::ProcessInputMessage()
    @          0x67f66db brpc::InputMessenger::OnNewMessages()
    @          0x67c388e brpc::Socket::ProcessEvent()
    @          0x679373f bthread::TaskGroup::task_runner()
    @          0x6788f21 bthread_make_fcontext

Expected behavior (Required)
Real behavior (Required)
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

branch-3.2-53f8343

Ref : https://github.com/StarRocks/starrocks/pull/38343


Openstack driver: Unclear instructions on overrides/configuration handling

Issue Type

Bug report

Molecule and Ansible details
$ ansible --version && molecule --version
ansible 2.7.6
  config file = None
  configured module search path = [u'/home/ekan/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/ekan/dev/github/maas-venv/local/lib/python2.7/site-packages/ansible
  executable location = /home/ekan/dev/github/maas-venv/bin/ansible
  python version = 2.7.15rc1 (default, Nov 12 2018, 14:31:15) [GCC 7.3.0]
molecule, version 2.19.0

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:

yamllint
testinfra

Desired Behavior
I find it somewhat unclear which ansible/molecule settings that should be overridable from molecule.yml for the Openstack driver.
Example:
platforms:
  - name: maas-single-1604
    image: Ubuntu 16.04 Xenial Xerus
    flavor: 1C-1GB-50GB
    networks: Default network
    ssh_user: ubuntu
    sshuser: ubuntu
   user: ubuntu
    groups:
      - maas

Above configuration snippet properly overrides image and flavor also default set in create.yml. Overriding networks and the three different tests for user does not seem to work.
I would expect common variables such as which network to use in OpenStack project, and which remote user to run tests as, to be overridable. Alternatively, a reference to values that are overridable and which are not.
Changing values for neutron_network_name and ssh_user directly in create.yml resolves the issue, but I'm not sure this is the intended behavior..
create.yml:
---
- name: Create
  hosts: localhost
  connection: local
  gather_facts: false
  no_log: "{{ not (lookup('env', 'MOLECULE_DEBUG') | bool or molecule_yml.provisioner.log|default(false) | bool) }}"
  vars:
    *ssh_user: ubuntu*
    ssh_port: 22
....
    *neutron_network_name: Default network*

Actual Behaviour
See ^

Openstack Driver was moved to https://github.com/pycontribs/molecule-openstack -- please re-add ticket there if still valid.


Trouble getting the server to work

I got this on the command prompt (Windows): sh: ./goMarkableStreamServer.arm: Permission denied

found that i needed to make it executable first with chmod +x goMarkableStreamServer.arm


How to load vue-rellax after page is rendered?

Hi,
I have like 25+ images (from api get method) in a v-for, I want to load rellax after the entire page is loaded.
When i scroll to the bottom, the images are all messed up in position. any way i can make the parallax start only when its visible in the screen after scroll?
Thanks!

Im having this problem also, especially on long-form pages. I am using Vue-rellax globally, with Nuxt js so the answer in this ticket is not ideal for me. https://github.com/croutonn/vue-rellax/issues/4
is there no other way of recalculating on window load globally when I initiate the plugin in Nuxt?
import Vue from 'vue';
import VueRellax from 'vue-rellax'

Vue.use(VueRellax)


Develop a code mod to guide through v2 to v3 upgrade

Summary
Look into whether we can provide a code mod to do simple changes for the upgrade from v2 to v3 for users.
💬 Description
Having a code mod that would automatically update any prop/event names or remove any props that no longer exist would make a users life much easier when upgrading from v2 to v3.
Investigate whether it will be possible/if there's an external package we can use.
💰 Use value
Much quicker migration for users.

Include testing in test app/website, then a new issue to test with other apps


Support mariner OS

For ROCm 6.2 release, as same as #807.

Tests are all passed on local Mariner docker container:
[----------] Global test environment tear-down
[==========] 34754 tests from 13 test suites ran. (1160430 ms total)
[  PASSED  ] 34754 tests.
hipBLASLt version: 800

command line: ./hipblaslt-test


DIP-158: Follow ups

Comment about error codes (just a recommendation)
Change sequences to QR code
Update typos

Opened by error


Fixes exception when retrieving location from profile.

refer #1
since Map.get returns nil if there's no result that'd be the default value from the struct.

Coverage remained the same at 100.0% when pulling 552556c0fc4bd79836e6912c0ae05bb349e8b1d0 on aguxez:location_update into f9167d6cbf88fa88204c3c40c0e74348477dcf18 on shinyscorpion:master.


Addic7ed is not getting the subtitles for some series

Hi,
First, thanks for this beautiful tool.
I am using subliminal lib from last version of sickrage (https://github.com/SickRage/SickRage). I compared addic7ed.py and file from my sickrage is the same that the one on subliminal master repo.
I configured it to use my account on addic7ed site and my profile on this site is configured to display subtitles of all languages.
It works pretty well except for 2 series where it gets nothing for both my languages (FR, EN) while there are files on addic7ed for the languages with same release group:

The 100 - id=268592
Scandal (2012) - id=248841

Log shows it didn't find any file:
2016-04-11 22:23:29 DEBUG    Thread-13 :: 248841: No subtitles downloaded for Scandal (2012) S05E15
2016-04-11 22:23:29 DEBUG    Thread-13 :: No subtitles found for Scandal (2012) S05E15
2016-04-11 22:18:13 DEBUG    Thread-13 :: Checking subtitle candidates for Scandal (2012) S05E15 (Scandal.US.S05E15.SDTV-lol.mp4)

Last update of Sickrage fixed my issue.
I am now very happy :)

Issue is back with last commit "Update subliminal to 14707b6"
https://github.com/SickRage/SickRage/commit/079958e620920214e79fb42944306c64ae83d0ed

Maybe issue is different this time.
For "The.Flash.2014.S02E22.SDTV-lol.mkv", I get following log:
2016-05-25 09:40:36 DEBUG Thread-13 :: [addic7ed] Subtitle score for updated/8/112311/0 is: 100 (min=198)
While subtitle on Addic7ed looks perfect:
2	22	Invincible	French	LOL	Completed		✔	✔	Download

Please report to SickReage or see Contributing rules. I need debug logs.


Style FillForegroundColor not being applied

When creating custom cell styles, the value set with XssfCellStyle.SetFillForegroundColor is not being applied to my workbook however if I user XssfCellStyle.SetFillBackgroundColor I can see the background color has been applied when I double click on a cell to edit it.
var workbook = new XSSFWorkBook();
var style = (XSSFCellStyle)workbook.CreateCellStyle();
var xssfColor = new XSSFColor(Color.Aqua);
style.SetFillForegroundColor(xssfColor);
style.SetFillBackgroundColor(xssfColor);

FillPattern is required for foreground colors but not background


Add ECHO to makefiles for real silence

I know I could not remove the echo silliness, but at least now it's possible to turn it off.

The whole file was rife with spaces ☺  fixed.

...  @richsalz, I commented on one line, not the whole file.  I was not at all complaining about the embedded perl sections.
The only reason to retain tabs in the actual Makefile parts is that make requires action lines to be indented with a tab.  Had it not been for that, I would have removed those tabs a long time ago.

New commits pushed.  Will be squashed.

thanks for the review!


Added weak/assign value support for PFPropertyInfo, and improved @synthesize detection.

We now walk the class heirarchy for the synthesized variable. This may help in nested subclass use cases with a different synthesized name.

This needs documentation, but otherwise LGTM.


fix(types): use teeny-request types where applicable

closing in favor of #264 (whoops, sorry about not coordinating) which we're moving conversation to.


cell config visual update, add docs

@christian-bromann can you do a review and help me QA this. Some new tech here for me so point out anything learning worthy please. I also see some errors in the console log related to the kernel and I don't know what they mean.

Doesn't yet have links to everything, but there are tickets for those docs:

https://github.com/stateful/docs.runme.dev/issues/186
https://github.com/stateful/docs.runme.dev/issues/185

Doesn't yet have links to everything, but there are tickets for those docs:

Working on them now.

Working on them now.

There are PRs for both of them now:

https://github.com/stateful/docs.runme.dev/pull/187
https://github.com/stateful/docs.runme.dev/pull/188

Looks great overall. In the light theme, though, the headlines are difficult to read:

Oh gosh, totally meant to update to the css variables, woopsy, nice catch.

The tab underlines are still a bit light for the light themes, but that will require some more css digging to fix.

@sourishkrout give that a try when you have a min.


Microsoft.AspNetCore.SignalR.Client refactor

Add overload of HubConnection..ctor that doesn't require an IServiceProvider (and update disposal)
Make HubConnection.Closed Func<Exception, Task> and await it when called (but make sure it's not in any locks and that it's on the thread pool)
Rename HubConnection.InvokeAsync, .SendAsync and .StreamAsChannelAsync to .[name]CoreAsync to avoid conflict with extension methods
Add CancellationToken to StopAsync
Remove HubConnectionBuilderExtensions.WithConnectionFactory and .WithHubProtocol
Rename HubConnectionBuilderExtensions.WithLogging to ConfigureLogging
Rename HubConnectionExtensions.InvokeAsync(string, object[], CancellationToken) to InvokeCoreAsync
Rename HubConnectionExtensions.InvokeAsync(string, object[], CancellationToken) to InvokeCoreAsync

You have to merge after me 😄

:(((((((((


[bug] Hosted Gorilla: MULTIPLE FUNCTIONS calls

Describe the bug
I am using gorilla-openfunctions-v1 to allow multiple function calling but it calls just the first one in the list
To Reproduce
Steps to reproduce the behavior:
I tried the following code using LMStudio as inference for Gorilla on port 1234:
import openai
import urllib.parse
import json

# Query Gorilla server
def get_gorilla_response(prompt="", model="gorilla-openfunctions-v1", functions=[]):
  openai.api_key = "EMPTY"
  openai.api_base = "http://localhost:1234/v1"
  try:
    prompt = f'<<question>> {prompt} <<function>> {json.dumps(functions)}'
    completion = openai.ChatCompletion.create(
      model="gorilla-openfunctions-v1",
      temperature=0.0,
      messages=[{"role": "user", "content": prompt}],
      functions=functions,
    )
    return completion.choices[0].message.content
  except Exception as e:
    print(e, model, prompt)

here the query:
#query = "Call me an Uber ride type confort in Rome hotel in 10 minutes"
query= "send a message to Jhon using the email<EMAIL_ADDRESS>and tell him that I need some money to buy a gift"
functions = [
    {
        "name": "Uber Carpool",
        "api_name": "uber.ride",
        "description": "Find suitable ride for customers given the location, type of ride, and the amount of time the customer is willing to wait as parameters",
        "parameters":  [{"name": "loc", "description": "location of the starting place of the uber ride"}, {"name":"type", "enum": ["plus", "comfort", "black"], "description": "types of uber ride user is ordering"}, {"name": "time", "description": "the amount of time in minutes the customer is willing to wait"}]
    },
    {
        "name": "Send Message",
        "api_name": "send.message",
        "description": "Send an email message to the destinatary",
        "parameters":  [{"name": "dest", "description": "email address, example<EMAIL_ADDRESS>{"name":"message", "description": "body of the message, here you have to summarize the content of the message to be sent"}]
    }
]
get_gorilla_response(query, functions=functions)

I was expecting to call the function for sending a message
But i got:
'uber.ride(loc="LOCATION", type="TYPE", time=30)'

how to make multiple function call with "gorilla-openfunctions-v1"?
Let's suppose we have a list function call:

send email
get weather condition for a location

Or simply reply to a question without involving a function call

Hi @HAL9KKK you got the solution for the multiple function?

Hi @CHIRU98 unfortunately not....I do not know why this multiple function feature is not working!
@ShishirPatil can you please explain how to use your code to make 2 function calls and the LLM should understand how to classify the call based on the context?

send email
get weather condition for a location

The same problem, only one function can be called each time.

Hey @HAL9KKK were you able to solve this? I'm having the same problem..


Restrict every outgoing traffic except external services

Dear contributor team,
I would like to trigger a discussion on the following nice feature:

In "block" mode every outgoing traffic should be blocked
Watcher scans the stack for statically defined "external services"
Those External Service IPs are allowed to be reached, nothing else

What do you think about this idea?

Can you create this in the rancher/rancher repo? I will be closing the ability to open issues in this repo.