Add tooltips to explain the UI

@dmtrKovalenko , @oliviertassinari :

So far, the motivation for not including these tooltips was to reduce the bundle size of the date picker, not requiring to import the Tooltip component. However, as we are now moving from Popover to Popper, the bundle size consideration has decreased. I wonder if we can get away with the usage of the title attribute.

@oliviertassinari :
My suggestions are:

Have a flag where you decide whether or not to have this option enabled, default enabled
Use a component wrapper, so the developer could inject his own component tooltip, and use a default one if the user has not injected any or specify that it is necessary to inject one and use the one of material ui as an example so it does not overload the module material- ui-pickers.

@oliviertassinari : Another thing I don't know if you noticed, for example take the first image.
If you see the first calendar is made up of 30 days, then there after the 30th day, there are the following days which are part of the month after, but are displayed, this option exists.
Same for the days before the 1st of this month, many calendars have this option, I don't know if I explained myself well.

@Angelk90 Regarding the preview and next month, please open a new issue. I think that it's a legit request, but it's off-topic.

@oliviertassinari : Could you open it?

I think that we should go with the simplest option: the title attribute. You will find the same one in:

https://jqueryui.com/datepicker/ for the arrow keys.
Material-UI Autocomplete and TablePagination for the actions.
http://react-day-picker.js.org/ for the shortened days.

The diff would look something like this for the arrows:
diff --git a/lib/src/_shared/ArrowSwitcher.tsx b/lib/src/_shared/ArrowSwitcher.tsx
index 3ea35d48..a229b303 100644
--- a/lib/src/_shared/ArrowSwitcher.tsx
+++ b/lib/src/_shared/ArrowSwitcher.tsx
@@ -80,6 +80,7 @@ const PureArrowSwitcher: React.FC<ArrowSwitcherProps> = ({
         data-mui-test="previous-arrow-button"
         size="small"
         aria-label={leftArrowButtonText}
+        title={leftArrowButtonText}
         {...leftArrowButtonProps}
         disabled={isLeftDisabled}
         onClick={onLeftClick}
@@ -101,6 +102,7 @@ const PureArrowSwitcher: React.FC<ArrowSwitcherProps> = ({
         data-mui-test="next-arrow-button"
         size="small"
         aria-label={rightArrowButtonText}
+        title={rightArrowButtonText}
         {...rightArrowButtonProps}
         disabled={isRightDisabled}
         onClick={onRightClick}
diff --git a/lib/src/views/Calendar/CalendarHeader.tsx b/lib/src/views/Calendar/CalendarHeader.tsx
index ab9c1302..50501c04 100644
--- a/lib/src/views/Calendar/CalendarHeader.tsx
+++ b/lib/src/views/Calendar/CalendarHeader.tsx
@@ -96,8 +96,8 @@ export const CalendarHeader: React.SFC<CalendarHeaderProps> = ({
   rightArrowButtonProps,
   leftArrowIcon,
   rightArrowIcon,
-  leftArrowButtonText = 'previous month',
-  rightArrowButtonText = 'next month',
+  leftArrowButtonText = 'Previous month',
+  rightArrowButtonText = 'Next month',
   getViewSwitchingButtonText = getSwitchingViewAriaText,
 }) => {
   const utils = useUtils();

Regarding the parts of the UI we might want to cover, I can think of:

arrow previous month
arrow next month
arrow open year display
days abbreviation

@Angelk90 Do you want to work on a pull request that goes in this direction? In any case, I don't think that it's more of an attention to the details.

Later on (in 1 year+?), I think that we could accept a TooltipComponent prop at the root, something we would forward down with the context, and have:
const TooltipComponent = React.forwarfRef(function TooltipComponent(props, ref) {
  return React.cloneElement(props.children, props));
}));

as a default value, a value we could switch with TooltipComponent={MuiTooltip};

@oliviertassinari :
 const TooltipComponent = React.forwardRef((props, ref) =>  React.cloneElement(props.children, props));

I believe we have two sides for the issue:

adding the missing title
allowing to customize the title rendering with the Tooltip component.

We now have https://github.com/mui-org/material-ui/issues/20991 to collect feedback and traction on 2. I think that we can only focus on the first part of the problem for closing this issue.


Subset command error

Subset Command:
$ ./allsorts subset --text "abcdefg" SourceSerif4Variable-Roman.otf d1.otf
Output:
Number of glyphs in new font: 8
Error: subset: parse error: an expected data value was missing

SourceSerif4Variable-Roman.zip

Allsorts is failing to parse the font because it's missing a glyf table. The name of the font suggests it's a variable font, which isn't supported yet: https://github.com/yeslogic/allsorts/issues/51

Thank you for your reply, is there any plan to implement this?

We would like to implement support for variable fonts but it's a big job and not scheduled yet.


Add internal MarkdownView component

Related Apollo 810
In preparation for the coming TextFormElement as apart of the FeatureFormView component.

I'm looking into a build compatibility issue with swift-markdown, ArcGIS, and Previews in Xcode. In the meantime you'll likely need to write a small view for testing this.

By also adding swift-markdown as a package dependency to the Test Runner project and as a library to the UITests target, the tests will run again.
While not optimal, and the versions will need to be kept in sync, this is the only fix I've found so far.


Failing OCR for good quality ID card

We are trying to extract the text from the ID cards but the tesseract is not giving satisfactory results even on Good quality images
Environment

Tesseract Version: tesseract 4.00.00alpha
Platform:  Windows 10 64-bit

Current Behavior:
we are not bale to extract correct text from this image

whereas this image is working fine

Expected Behavior:
good result on this image

In my case, we had to use ocrb fonts in order to parse European id cards. So, maybe you can try that or use tesseract based on font type present in your id card.

@MagicAP-QA  Hi, have you resolved this issue, I am also working on OCR extraction for id cards.


updated wordcounter (1.3.5)

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


[WIP - do not merge] Game and menu styling

I added some styling and put in a fix so that the canvas can be anywhere on the page and the click events will still render correctly.
However, I ran into some bugs so I commented out some code because I can't figure it out.

I made a div with the id number-of-lives but when I try to call document.getElementById('number-of-lives') it returns null and won't get me get or set the text. You can see what I am trying to do on lines 56 - 58, and 62.
I want to eventually implement a "start level" or "call wave" button of some kind, so I figure we can wrap line 75 in an if statement that waits for that button to be clicked. However, before that, we would still want to render the board and any towers, and allow the user to build out the towers. I tried to do even just part of this (statically render the board) on line 68, but it renders nothing if I do that.

@stevekinney hoping you can help with some of the issues I outlined above. I can't figure out what I'm doing wrong.

Also adding @rrgayhart

This is what I get when I try to call document.getElementById('number-of-lives').innerHTML();
Uncaught TypeError: document.getElementById(...).innerHTML is not a function

Also if I run jQuery in the console:
$('#number-of-lives').text()
""

@ToniRib - I think you might be getting a false-positive on the assumption that you can't grab number-of-lines.
So if you stick a debugger in that function - you can actually grab document.getElementById('number-of-lives') (but only after the document loads).
If you try to just do it in the chrome console, outside of the debugger statement, you don't find anything. The reason is that Webpack is displaying your page in an iFrame.. which is annoying.
I do still see the error you're talking about - I'll dig into that a little bit more!

innerHTML without the () - so innerHTML = 8

@rrgayhart is a genius, so I fixed the issue with updating the number of lives.

This branch is now ready to merge in. Both of the errors above were fixed.

Looks great, like a real game now!


Escape the Pattern special character before passing to string.gsub()

This will fix #17. It's not pretty, but Lua doesn't seem to have a way to disable patterns in string.gsub(). The string.find() function can be forced to use plain strings with no patterns, but the only way I found to do substitution involved escaping the input.
Since the input is in a *.sil document as one big long string then split by character, there is no way to enter the escape there. SILE itself gets the % character just fine, but since it's sending everything a character at a time to gsub() any escaping is lost. This injects the necessary escape at the last possible second.

Thanks for that. I've just verified it works with our toolchain, @jvgaultney should I go ahead and merge?


use light/dark images for logo so it doesnt look bad in dark mode

What Changed
Really pedantic change but the logo looked bad in dark mode
Why
Todo:

[ ] Add tests
[ ] Add docs
[ ] Add release notes

Didn't seem to work


Timer embedded in Multi Instance subprocess never finishes execution

Describe the bug

The above diagram creates multiple instance subprocesses with an embedded timer running at PT3S. The instances appear to run indefinitely and the entire process never finishes.
The Multi Instance is running parallel.
Expected behavior
Instantiating three subprocesses should result in the following output:
Starting [{drinkType=string, cupSize=string, orderPrice=0}, {drinkType=string, cupSize=string, orderPrice=1}, {drinkType=string, cupSize=string, orderPrice=2}]
Processing {drinkType=string, cupSize=string, orderPrice=0}
Processing {drinkType=string, cupSize=string, orderPrice=1}
Processing {drinkType=string, cupSize=string, orderPrice=2}
2022-03-02 13:59:59,455 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-19-thread-1) Job TIMER:0:596cb1e1-ee08-477c-bb7e-288c8ad0ada9 started
2022-03-02 13:59:59,458 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-19-thread-2) Job TIMER:0:dfaa7725-3cdd-46bf-9ab6-8ab56c3ace87 started
2022-03-02 13:59:59,459 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-19-thread-3) Job TIMER:0:c35a212d-3146-48d6-9537-812ca1b39a67 started
End
End
End
Complete

Actual behavior
Running the process would appear to never complete:
Starting [{drinkType=string, cupSize=string, orderPrice=0}, {drinkType=string, cupSize=string, orderPrice=1}, {drinkType=string, cupSize=string, orderPrice=2}]
Processing {drinkType=string, cupSize=string, orderPrice=0}
Processing {drinkType=string, cupSize=string, orderPrice=1}
Processing {drinkType=string, cupSize=string, orderPrice=2}
2022-03-02 13:59:59,455 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-19-thread-1) Job TIMER:0:596cb1e1-ee08-477c-bb7e-288c8ad0ada9 started
2022-03-02 13:59:59,458 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-19-thread-2) Job TIMER:0:dfaa7725-3cdd-46bf-9ab6-8ab56c3ace87 started
2022-03-02 13:59:59,459 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-19-thread-3) Job TIMER:0:c35a212d-3146-48d6-9537-812ca1b39a67 started
End
End
End

The Complete task is never called.
Removing the timer or setting the instance to Sequential or Async (which behaves exactly like Sequential) produces the appropriate result:
Starting [{drinkType=string, cupSize=string, orderPrice=0}, {drinkType=string, cupSize=string, orderPrice=1}, {drinkType=string, cupSize=string, orderPrice=2}]
Processing {drinkType=string, cupSize=string, orderPrice=0}
2022-03-02 14:11:10,853 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-46-thread-1) Job TIMER:0:1722d704-8a5f-4e09-b673-dc3d387ab565 started
End
Processing {drinkType=string, cupSize=string, orderPrice=2}
2022-03-02 14:11:13,863 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-46-thread-2) Job TIMER:0:ed3d978f-d4d7-40d9-b658-27ebe835b352 started
End
Processing {drinkType=string, cupSize=string, orderPrice=1}
2022-03-02 14:11:16,870 INFO  [org.kie.ser.job.imp.InMemoryJobService] (pool-46-thread-1) Job TIMER:0:41933937-3385-4972-ace0-1d37d4cec1a5 started
End
Complete

How to Reproduce?
line-test-kogito.zip
Using the multiprocess.bpmn2
Output of uname -a or ver
No response
Output of java -version
jdk-11
GraalVM version (if different from Java)
No response
Kogito version or git rev (or at least Quarkus version if you are using Kogito via Quarkus platform BOM)
1.17.0.Final
Build tool (ie. output of mvnw --version or gradlew --version)
No response
Additional information
No response

Tracking the issue on https://issues.redhat.com/browse/KOGITO-684


Error running docker-compose up: trust-notebooks.sh fails to execute

Not sure what the issue really is, I'm new to docker. But I got this error in my console after letting docker-compose up do its thing:
 => ERROR [mortar_jupyter-notebook 11/11] RUN bash trust-notebooks.sh                                                                                1.6s
------
 > [mortar_jupyter-notebook 11/11] RUN bash trust-notebooks.sh:
#46 1.489 trust-notebooks.sh: line 2: $'\r': command not found
#46 1.490 trust-notebooks.sh: line 3: syntax error near unexpected token `$'do\r''
'46 1.490 trust-notebooks.sh: line 3: `for nb in `ls *.ipynb` ; do
------
failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/bash -o pipefail -c bash trust-notebooks.sh]: exit code: 2

I think it's related to #3 , so how do I go about editing this particular file? I can't find it in the git repo.

Here's how I solved this particular issue on Windows 10 with VSCode:

Open up docker/jupyter-notebook/trust-notebooks.sh
Follow this guide to change line endings from CRLF to LF.    (If this does not show up, right click on the Status Bar and make sure "Editor End of Line" is checked.)
Run docker-compose up again.


Custom (and callback) rules not working with custom rule messages

Laravel Version: 9.45.1
PHP Version: 8.1.0

Description:
I am using requests for data validation. When using custom rules or callback rules along with custom messages and that rule fails, response is always full custom message array for that attribute, instead of fail message for that custom rule.
Steps To Reproduce:
Controller method:
public function doSomething(MyFormRequest $request)
{
    $data = $request->validated();
    // do something with $data
}

Request:
class MyFormRequest extends FormRequest
{
    public function rules()
    {
        return [
            'date' => ['required', 'date_format:Y-m-d', new MyCustomDateRule],
        ];
    }

    public function messages()
    {
        return [
            'date' => [
                'required' => 'This is custom message for required rule.',
                'date_format' => 'This is custom message for date_format rule.',
                'foo' => 'bar',
            ],
        ];
    }
}

MyCustomDateRule:
class MyCustomDateRule implements Rule
{
    public function __construct()
    {
        //
    }

    public function passes($attribute, $value)
    {
        return false;
    }

    public function message()
    {
        return 'My custom rule error message.';
    }
}

When sending request to this controller method and all rules pass except my custom rule, I get this response:
{"message":"This is custom message for required rule. (and 2 more errors)","errors":{"required":["This is custom message for required rule."],"date_format":["This is custom message for date_format rule."],"foo":["bar"]}}
When sending same request, but without messages method (or just without date messages) in my request, I get actual error from my custom rule:
{"message":"My custom rule error message.","errors":{"date":["My custom rule error message."]}}

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

I described how you can simulate it and see it is bug...

You're using the incorrect syntax for your custom messages:
Change this:
public function messages()
{
    return [
        'date' => [
            'required' => 'This is custom message for required rule.',
            'date_format' => 'This is custom message for date_format rule.',
            'foo' => 'bar',
        ],
    ];
}

To this:
public function messages()
{
    return [
        'date.required' => 'This is custom message for required rule.',
        'date.date_format' => 'This is custom message for date_format rule.',
        'date.foo' => 'bar',
    ];
}

More info here: https://laravel.com/docs/9.x/validation#customizing-the-error-messages

You're using the incorrect syntax for your custom messages:
Change this:
public function messages()
{
    return [
        'date' => [
            'required' => 'This is custom message for required rule.',
            'date_format' => 'This is custom message for date_format rule.',
            'foo' => 'bar',
        ],
    ];
}

To this:
public function messages()
{
    return [
        'date.required' => 'This is custom message for required rule.',
        'date.date_format' => 'This is custom message for date_format rule.',
        'date.foo' => 'bar',
    ];
}

More info here: https://laravel.com/docs/9.x/validation#customizing-the-error-messages

I think both syntaxes are correct, but I will test with this and report here.

You're right, it's actually a valid syntax but it tries to search for these options in order:
["{$attribute}.{$lowerRule}", $lowerRule, $attribute]

Since it cannot find a combination of the attribute and the rule, or the rule itself it takes the value of the attribute, which is an array in your case. This causes all your messages to be overridden.
Source: https://github.com/laravel/framework/blob/9.x/src/Illuminate/Validation/Concerns/FormatsMessages.php#L98-L136


Key frames are swapped

The key frames described in the examples and diagrams are swapped - 0.0 is noon and 0.5 is midnight. The example provided actually demonstrates this correctly.
This diagram causes confusion in particular:
https://learn.microsoft.com/en-us/minecraft/creator/documents/media/deferredlighting/key_frame_times_of_day.png

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

ID: 7d4722db-3980-4b38-a102-53f10132095e
Version Independent ID: 7d4722db-3980-4b38-a102-53f10132095e
Content: Getting Started with Deferred Lighting
Content Source: creator/Documents/DeferredLighting.md
Product: gaming
GitHub Login: @iconicNurdle
Microsoft Alias: mikeam

Thanks for the catch! Corrections are on the way.


use copy-on-write list in InMemoryAppender

Fixes https://github.com/apache/incubator-druid/issues/8759
@leventov

thank you, @leventov
comments addressed

I am learning from better developers and was thinking about contract of logger and how would it could be solved should it be inmemorylogger be used somewhere else some day.
Isn't copy on write an overhead and a bit teaching inappropriate use of copy on write?
Isn't logEvents.toArray() atomic  appropriate?
return Collections.unmodifiableList(Arrays.asList((LogEvent[]) logEvents.toArray()));
Code readability is lower and would require a comment maybe.

@LoriLori logEvents.toArray() doesn't solve the race condition - it may still race with an addition to logEvents and throw some exception, unless you make both append() and getLogEvents() synchronized.
Since this class resides in tests, not production, the performance doesn't matter and we shouldn't waste time comparing synchronized and CopyOnWriteArrayList. While CopyOnWriteArrayList solution is a bit simpler.

@zhenxiao please merge the master branch into your PR branch to make the CI pass.

master merged into this PR. failure seems not related

@zhenxiao don't rebase your PRs. Your should never do this in Druid. Once you open a PR, you only merge other branches into your PR branch, never rebase it.

oh, sure. seems tests passed


Async requests do not return a response map

Hi,
when doing an async request like:
(let [response (defer (http/get "http://example.com" {:async? true} (fn [_] _) (fn [_] _))])

response is org.apache.http.HttpResponse and not a Ring response map.
In our use case we're using the code above in our tests.
Can the return value of an async request also be wrapped in such a map?

Unfortunately there isn't a way to do out of the box because of the async nature. It would break the functional abstraction of a response/response map. The map would not have headers or body available.
However, you can achieve a similiar functionality in tests by putting the async response inside a  promise and then blocking in your tests until the result comes back with deref.
In terms of code, it'd be something like this:
(let [promise-response (promise)]
  (http/get "..." {:async? true} (fn [response] (deliver promise-response response))]
  (deref promise-response 1000)  ; wait up to 1000ms for the response
  )

Hi Raymond, thanks for responding!
The code I've shown is not the real code. In our case the async request is done inside a function which we call in our test where we can't inject a custom callback.
However, our function returns whatever http/get returns which happens to be a Future<HttpResponse>. So, we can and do defer that but then it's not a Ring response map.
Wouldn't it be possible for http/get to return the kind of promise that you're describing?

That does seem like a reasonable request.
I can think of two ways this could be done:
Option 1: Introduce no new options and replace the current BasicFuture response object.
I'm uneasy about this breaking change since the current returned object is a org.apache.http.message.BasicHttpResponse. This original API was designed to mimic async-ring ring, we should respect this.
Option 2: Introduce a different request option to return the response as a fully defined future.
We could create a new option called :async-future which uses uses the existing async machinery that wraps the existing :async API as a tidy future. This approach avoids breaking the existing API contract, allowing users to upgrade at their leisure.

I am leaning towards Option 2 to avoid breaking changes. Do you have any thoughts on this? cc-ing: @dakrone


Nikom does not support INTL, TOPT and FMTP (Fidonet)

It would be nice if NiKom supported these standards.
Standards here: http://retro.fidoweb.ru/docs/index=ftsc&doc=FTS-4001.001&enc=koi:

FMPT

The FMPT control paragraph shall be used to give information about
the point number of the original sender of a message if that point
number is not 0. If the point number of the original sender of a
message is 0 that message should not contain any FMPT control
paragraph.
The format of a FMPT control paragraph shall be:
"FMPT "
where  is the ASCII representation of the point number
of the sender. The point number shall be an unsigned integer in the
range 1-65535.
E.g. a message from point number 1 of a certain node shall contain
the following FMPT control paragraph
"FMPT 1"
Note that the format of a FMPT control paragraph deviates from the
general format specified in separate FTSC documents in that it does
not contain any colon after the control tag.

TOPT

The TOPT control paragraph shall be used to give information about
the point number of the ultimate addressee of a message if that
point number is not 0. If the point number of the ultimate addressee
of a message is 0 that message should not contain any TOPT control
paragraph.
The format of a TOPT control paragraph shall be:
"TOPT "
where  is the ASCII representation of the point number
of the addressee. The point number shall be an unsigned integer in
the range 1-65535.
E.g. a message to point number 1 of a certain node shall contain the
following TOPT control paragraph
"TOPT 1"
Note that the format of a TOPT control paragraph deviates from the
general format specified in separate FTSC documents in that it does
not contain any colon after the control tag.

INTL

The INTL control paragraph shall be used to give information about
the zone numbers of the original sender and the ultimate addressee
of a message.
The format of an INTL control paragraph shall be:
"INTL "" "
where  shall be the representation of the
address of ultimate destination and  is the
representation of the address of the original sender of the message
in question. These addresses shall be given on the form
:/ where  is the ASCII representation of the
zone number,  is the ASCII representation of the net number and
 is the ASCII representation of the node number. Any point
number information shall be given in FMPT and TOPT control
paragraphs.
E.g. a message from address 1:123/4.5 to 2:345/6.7 shall contain the
following INTL control paragraph
"INTL 2:345/6 1:123/4"
Note that the format of an INTL control paragraph deviates from the
general format specified in separate FTSC documents in that it does
not contain any colon after the control tag.
INTL control paragraphs are also often used even when both the
originating and the destination systems are in the same zone. This
gives both the originating system and the destination system as well
as any intermediate routing systems unambiguous zone information
even in a situation where one system may be active in a number of
different (possibly non-FidoNet) zones.
Although it is known that some programs may route messages
incorrectly if the INTL control paragraph is present in messages
where both the originating and the destination systems are in the
same zone, it is recommended that the INTL control paragraph is
always inserted into netmail messages in packet files.

No issue as it turns out.  Closing.


update and add to/fix license & guidelines info

The license change happened so that obviously needed updating
Slimmed it down in a way, although I also linked some useful docs on the OSMF site and also described the automated edits guidelines, since people could use this library to make automated editing scripts.

@christoph-buente @freundchen Can you give feedback here?

@lennerd Looks good to me

@sozialhelden ping?

Note that "The Travis CI build failed" indicates issues with tests rather than with this PR given that it is not changing code.

yeah this is just readme change with things which obviously need correcting


Getting the environment for Mac

I was wondering whether there is something wrong with the environment zip for Mac. When I try to download  it the link does not work. When I navigate to the website the link points to and download it, I get an app file that my Mac cannot open nor unzip (running Ventura 13.0).

Hi @eleninisioti, have you made sure to follow the exact installation process, also the notes section?:
"If you are using MacOS, you may also need to run this command: chmod -R 777 [applicationName].app in your MacOS terminal to unlock permissions for running the application. Edit: Please also make sure when you extract the folder, you move the files inside the sub-directory to its parent folder."
I've downloaded the macOS version of AAI just now and everything is fine. Could you elaborate more on the issue?
-Ibrahim
Lead Software Engineer

Im going to assume that the issue is now solved. If not, please post an issue again.


Tab bar replace, Icons missing, iOS 10

Found an issue and a workaround.
Issue: After a tab bar replace, you'll have no tab bar icons. This likely has to do with Pixate not having been updated.
Use Case:
Define a tab bar in structure, set up your icons through native css
Run the replace command at runtime to change out your tabs (maybe your user logged in and was expecting new options to show up).
The new tabs will not pick up the icons from the native css (the tabs will have text labels and function normally but will have blank spaces where icons should be).
Workaround:
Note that this doesn't work using svgs, instead use pngs with @2x as part of the filename so that pixate knows to half the size on them (filename@2x.png). In my case, I was using 60x60 images.
Define the icon in the tab objects in both structure and in the replace function as a param (legacy steroids style):
title: "whatever"
location:"whatever"
icon:"icons/filename.png"
The icons should show up normally now... at the cost of dropping your svg icons...
Also note that this doesn't address any of the tab bar colour styling issues introduced with iOS 10.

@Schurke Way to GO! Thank you


Support Multiple Templates per Repository

I would like for copier to support multiple templates in a given repository by allowing the subdirectory of the repository to be specified with the vcs ref, and supporting updates by filtering commit history based on the subdirectory and a user-specified suffix/prefix for tags.
Alternatively, I can continue to use single repo per template approach that makes template management more difficult on the user and developer.

I've seen this workaround, https://copier.readthedocs.io/en/stable/configuring/#subdirectory, but I don't believe that will support the use case of having different questions per template. Unless it's possible to pull in questions from each subdirectory. If so, that would be 💯

nvm, it's in the docs, https://copier.readthedocs.io/en/stable/configuring/#include-other-yaml-files. I think I can work with the combination of those two options.

@BrutalSimplicity, if you figure out a working example, I would be interested in seeing it.

Well, I can open this back up. Do you mean the original support in the ticket, or a setup that the configuration supports?

An example layout, I'm interested doing something similar.

Hmm, ok. I assume you've seen this.

Yes, I played with putting extra copier.yaml files in the _subdirectory, but didn't seem to get anywhere with that.

(I am currently including a handful of files in the top level copier.yml to separate out my questions and each question does a when based on the _subdirectory parameter); but this has the downside of everything winding up in every answers file.

Maybe yaml anchors can help you reducing repetition.


Can't download instagram posts

Ripme version: v1.7.94-19-c52c34ab
Java version: Version 8 Update 281 (build 1.8.0_281-b09)
Operating system: Windows 10 Version 20H2 (OS Build 19042.867)
Exact URL you were trying to rip when the problem occurred:
https://www.instagram.com/yaslenxoxo/

Expected Behavior
Should download photos and videos from instagram page
Actual Behavior
After download starts, it immediately stops and either says "Error: null" or "No images found"

@CynicalPlatapus, would you mind copying the error out ot the log.txt in the download directory? especially if it is "Error: null".

@soloturn for some reason it suddenly started working, i have absolutely no idea why
it's been doing this occasionally lately, like it'll refuse to download from a twitter until i close and reopen it

@soloturn for some reason it just started working again, but after 93 downloads it stopped and says "no images found"
it's been a little buggy lately, like it'll refuse to download twitter posts unless i close and re-open it

@soloturn not seeing a log txt file anywhere

let me close it until we have something more concrete.


Move field&index data retrieval responsibilities from TableListProvider to PanelViewProvider

displayData method functionality should be moved to PanelViewProvider or its child objects

Closing as this change requires more analysis and probably is a part of a bigger refactoring.


Add a way to test templates against a locally built version of the libs

@Garbee @surma PTAL

LGTM

Awesome! Thanks @nicolasgarnier :)


createProgram: provide file content with file objects independent of fs access - I. e. for Meteor Build System

To enable meteor developers the use of TypeScript in meteor packages and application code, we are currently working on a compiler plugin that needs to integrate into the meteor specific build system.
_note: the 9th most stared Github repo, before ruby/rails, is the meteor js framework
The problem is that the meteor build system instruments a caching and bundling system that obfuscates access to the file system, which seems to challenge the fs and typescript api coupling.
I. e. during meteor build an [1]array of input file objects that each incl. filename, -path, -hash and -content (data) properties is passed to the compile plugin (here typescript) and expected to be returned as [2]transpiled content via a method handler. Without touching the file-system.
Analog to the "A minimal compiler" API example, it seems to be easy to cover [2] by providing a custom compileHandler: ts.WriteFileCallback into program.emit(undefined, compileHandler) to pass the transpiled content back upstream.
export function compile(fileNames: string[], options: ts.CompilerOptions): void {
    var program = ts.createProgram(fileNames, options);
    var emitResult = program.emit();
//...

But can [1] lead to a full Program instance without providing fs-access just with "virtual" file paths and file contents?

It would be convenient to have ts.createProgram accept file objects that encapsulate file properties such as content etc. to make the compiler file-system agnostic.

Any given advice to prevent redundant disk access while maintaining source maps, incremental builds and diagnostics inside this build chain?

createProgram accepts optional compilerHost argument. This host is responsible for all IO related operations and by default is is file system based. You can always provide alternative implementation that uses some in-memory cache to provide answers.

Simply replacing the host when instantiating a service should usually suffice, but failing that, assigning ts.sys a host-specific implementation of a System should cover everything.

I would consider replacement of sys to be more like a hack then a recommended solution. The fact the default version of compilerHost uses sys is an implementation detail and I would not recommend to rely on it

Thanks @vladima! compilerHost looks like what we need, though not sure how to instrument it yet. Are there by now better API documentations except for the directives?

If all you need is transpilation without diagnostics on inter-file dependencies, check out the transpile functionality.
If you need more (you mentioned incremental builds and diagnostics) then I think you should use the language service instead. I used the language service to build something like what you're talking about for webpack, and while it has a lot of webpack-specific stuff it may help guide you. In particular take a look at the getScriptSnapshot implementation which answers your initial question about loading from memory.

Exactly, a simple transpile would defeat the purpose of compile assurance :)
I originally hoped to get around the language service implementation (after all, its not an IDE) but the getScriptSnapshot looks like a plan in getting around a parallel build implementation.
Since the meteor build system plays the role of the file watcher, starting and killing the process at will, I'd need to check how to translate that into some cache layer.

looks like the original issue has been handled. please reopen if this is not the case.

Hi @D1no, I have the same needs as yours.
After reading some docs about the LanguageService, I see that I need to implements methods, like getCurrentDirectory, readDirectory or fileExists, but I already have the file content in memory.
Any ideas ?


Make az_span_from_str accept a const char * instead of char *.

The signature is by design as the span is used for mutation.  Those creating a span on top of a string literal must ensure they never mutate the underlying buffer.


Unable to pick colors in the ColorPicker node.

I can't select any color other than the default color when loading the node. Normally, when I click the mouse cursor on the color, a color palette pops up, and I just need to move the cursor to the image and pick the color. But today, when I click on the color, nothing happens.

I'm having the same problem. Did you solve it? Can someone help me?

The panel does not appear even though my browser settings do not disable pop-ups.

I have the same problem. In an new install, the color pop screen work flawless. After installing custom nodes, it stop working. It's probably an incompatibility with one of them, but wich one...?


Update bower.json

So it does not collide with other minor versions from projects using the component

could you also please release a new version, so we can update our project and not always get a Unable to find a suitable version for iron-icons message on bower install


Support Central Package Management (CPM) for .NET components

The files Packages.props and Directory.Packages.props used in CPM are not yet supported. As this functionality is now stable in .NET 6, it should be supported.
https://learn.microsoft.com/en-us/nuget/consume-packages/Central-Package-Management
AB#2098999

Scanning those files will only tell us the direct dependencies. We cannot resolve transitive dependencies from a Directory.Packages.props file.


Remove unnecessary props from DatasetExplorer SidePanel class

Description
Looks like cohorts, jointDataset and dataset props can be inferred from this.context. Hence, removing these from being passed around a props for SidePanel.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Closing this as the context seems to have an issue when accessed in constructor.


signInUrl/signUpUrl and Internationalized Routing

[x] Review the documentation: https://clerk.com/docs
[x] Search for existing issues: https://github.com/clerkinc/javascript/issues
[x] Go through package changelog files.
[ ] Provide the Frontend API key from your application dashboard.

Package + Version

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[x] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore
[ ] other:

Dependencies + versions
{
  "dependencies": {
    "@clerk/localizations": "^1.10.0",
    "@clerk/nextjs": "^4.14.2",
    "@clerk/themes": "^1.5.1",
    "@clerk/types": "^3.34.0",
    "@emotion/react": "^11.10.6",
    "@prisma/client": "^4.10.1",
    "@radix-ui/colors": "^0.1.8",
    "@radix-ui/react-aspect-ratio": "^1.0.2",
    "@radix-ui/react-avatar": "^1.0.2",
    "@radix-ui/react-dropdown-menu": "^2.0.4",
    "@radix-ui/react-separator": "^1.0.2",
    "@stitches/react": "^1.2.8",
    "@tanstack/react-query": "^4.28.0",
    "@trpc/client": "^10.19.1",
    "@trpc/next": "^10.19.1",
    "@trpc/react-query": "^10.19.1",
    "@trpc/server": "^10.19.1",
    "clsx": "^1.1.1",
    "formik": "^2.2.9",
    "jotai": "^2.0.3",
    "jotai-devtools": "^0.4.0",
    "jotai-form": "^0.1.0",
    "lucide-react": "^0.130.1",
    "next": "^13.2.1",
    "next-themes": "^0.2.1",
    "plaiceholder": "^2.5.0",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "sharp": "^0.32.0",
    "superjson": "^1.7.4",
    "typesafe-i18n": "^5.24.3",
    "zod": "^3.0.0",
    "zod-formik-adapter": "^1.2.0"
  },
  "devDependencies": {
    "@playwright/test": "^1.26.1",
    "@swc-jotai/react-refresh": "^0.0.7",
    "@types/node": "^18.7.20",
    "@types/react": "^18.0.9",
    "@typescript-eslint/eslint-plugin": "^5.47.0",
    "@typescript-eslint/parser": "^5.47.0",
    "eslint": "^8.30.0",
    "eslint-config-next": "^13.2.1",
    "eslint-config-prettier": "^8.5.0",
    "eslint-import-resolver-webpack": "^0.13.2",
    "eslint-plugin-prettier": "^4.2.1",
    "eslint-plugin-react": "^7.31.11",
    "eslint-plugin-react-hooks": "^4.6.0",
    "npm-run-all": "^4.1.5",
    "prettier": "^2.7.1",
    "prisma": "^4.10.1",
    "start-server-and-test": "^1.12.0",
    "tsx": "^3.12.3",
    "typescript": "^4.8.3",
    "vite": "^4.1.2",
    "vitest": "^0.29.8"
  },
}

Description
I am using nextjs with automatic locale detection, and when you have that on it appends your locale to the URL.
For example, it will append:
/no-NO to the URL.
I noticed that when i use the Clerk SignUp component it doesn't handle this locale properly It ends up appending it twice for some reason.
See this:
https://user-images.githubusercontent.com/3880008/230762258-0270735f-38f3-47c0-b75f-15d215782260.mp4
My SignIn component basically looks like this:
      <SignIn path={clerkPath} routing="path" signUpUrl="/sign-up" />

Is there a way for me to handle where it goes to entirely in my app rather than pass it a route? Is there a workaround or something I should be doing?
Thank you,

I'm facing a similar internationalized routing issue on Next JS. When I load the page /sign-in/, everything loads but when I use /en/sign-in/ then it doesn't load.
I have my folders stored as pages/sign-in/[[...index]].tsx.

just wanted to add: we're not launched yet but this is a pretty big problem for us as our app needs to be translated. I would hate to have to write my own UI components just because of this.

How I fixed it for me.
import { useRouter } from 'next/router';
import React from 'react';
import { SignIn } from '@clerk/nextjs';
import { Page } from '~/components/page';

const SignInPage = () => {
	const router = useRouter();

	const once = React.useRef(false);
	React.useEffect(() => {
		if (once.current) return;
		once.current = true;

		if (router.locale === 'de') return;
		router.replace(router.asPath, router.asPath, { locale: 'de' });
	});

	return (
		<Page
			style={{
				justifyContent: 'center',
				alignItems: 'center',
			}}
		>
			<SignIn
				path="/sign-in"
				routing="path"
				signUpUrl="/sign-up"
				redirectUrl="/"
			/>
		</Page>
	);
};

export default SignInPage;

Hello @harrisrobin and @theramjad , we're currently investigating - we'll get back to you as soon as possible.

Hello. We've identified the issue. The fix will be available in the next release. @theramjad, you need to include the router.locale in the paths that are passed to the <SignIn /> component. That should do the trick.
Example:
import { SignIn } from "@clerk/nextjs";
import { useRouter } from "next/router";

const SignInPage = () => {
  const router = useRouter();
  return <SignIn path={`/${router.locale}/sign-in`} routing="path" signUpUrl={`/${router.locale}/sign-up`} />;
};

export default SignInPage;

We've identified the issue - a fix will be deployed with our next release.

Thank you @desiprisg for the trick.
Unfortunately, there is a little bug with the signUpUrl={`/${router.locale}/sign-up`}. When I click on the link, it takes me to /en/sign-up as expected but then I get automatically redirected to /en/en/sign-up after which the page does not appear.
Any idea what is happening? I'm unsure whether this is a Next.js problem or a Clerk problem.
My code is,
import { SignUp } from "@clerk/nextjs";
import { useRouter } from "next/router";
import { Card, Container } from "react-bootstrap";

export default function Page({ allow_sign_up }) {
  const router = useRouter();
  let locale;

  // In path and signUpUrl, you have to use locale as a path parameter
  if (router.locale === process.env.NEXT_PUBLIC_DEFAULT_LOCALE) {
    locale = "";
  } else {
    locale = "/" + router.locale;
  }

  return (
    <div className="align-items-center d-flex justify-content-center min-vh-100">
      <SignUp
        path={`${locale}/sign-up`}
        routing="path"
        signInUrl={`${locale}/sign-in`}
        appearance={{
          layout: {
            socialButtonsPlacement: "bottom",
          },
        }}
      />
    </div>
  );
}

@theramjad This sounds like the original issue of this thread. It should be fixed in the upcoming release. Let us know if you experience any more problems after that!

experiencing a similar issue when I have the signin form in a nested path e.g /auth/signin/[[..index]].tsx and a similar setup for signup.

@theramjad This sounds like the original issue of this thread. It should be fixed in the upcoming release. Let us know if you experience any more problems after that!

Thank you. Is there a timeline for the next release?


GetAppModel and ApplicationFromSummary are using time.Time pointer members

While trying to build a CLI plugin, I ran into an issue with the data being returned from cliConnection in my Run.
If I do a call cliConnection.GetApp("my_app"), the return result always had a nil value on the PackageUpdatedAt member on the GetAppModel struct.
After tracing it down, it appears to be associated with GetApp and ApplicationFromSummary. JSON unmarshalling cannot capture a string time into time pointer.
When we changed the member to be a non-pointer time.Time everything worked for us.

We thought we'd be able to file a pull request, but we are unable to fix the tests to pass correctly.

@jtarchie @dkoper  We confirmed the bug. The plugin model is being populated by a partially filled app model, hence PackageUpdatedAt is nil.

Hi @jtarchie. Thanks for trying with the PR! We're a bit low-staffed next week with the CF Summit and release preparation, but you can try reaching out to @krishicks to pair with you on your fix if you want it fast-tracked.

This isn't something that @jtarchie and I need to pair on; this is something that @simonleung8 and I should pair on. It'll require a fair amount of changing how that model is populated.

This issue has been fixed and confirmed by @jtarchie. Thanks guys!


InvalidArgumentError: Key: roadgraph_samples/xyz.  Can't parse serialized Example. [Op:ParseExampleV2]

When I run tutorial_motion.ipynb,  the error shows:
Example data downloaded from here
Output exceeds the [size limit](command:workbench.action.openSettings?%5B%22notebook.output.textLineLimit%22%5D). Open the full output data [in a text editor](command:workbench.action.openLargeOutput?2b71df5f-3488-4670-99b7-aa11fb6ea35a)
---------------------------------------------------------------------------
InvalidArgumentError                      Traceback (most recent call last)
Cell In[15], line 3
      1 dataset = tf.data.TFRecordDataset(FILENAME, compression_type='')
      2 data = next(dataset.as_numpy_iterator())
----> 3 parsed = tf.io.parse_single_example(data, features_description)

File [~/miniconda3/envs/mtr/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py:206](https://file+.vscode-resource.vscode-cdn.net/home/zetlin/Code/WaymoDataset/waymo-od/tutorial/~/miniconda3/envs/mtr/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py:206), in add_dispatch_support..wrapper(*args, **kwargs)
    204 """Call target, and fall back on dispatchers if there is a TypeError."""
    205 try:
--> 206   return target(*args, **kwargs)
    207 except (TypeError, ValueError):
    208   # Note: convert_to_eager_tensor currently raises a ValueError, not a
    209   # TypeError, when given unexpected types.  So we need to catch both.
    210   result = dispatch(wrapper, args, kwargs)

File [~/miniconda3/envs/mtr/lib/python3.8/site-packages/tensorflow/python/ops/parsing_ops.py:452](https://file+.vscode-resource.vscode-cdn.net/home/zetlin/Code/WaymoDataset/waymo-od/tutorial/~/miniconda3/envs/mtr/lib/python3.8/site-packages/tensorflow/python/ops/parsing_ops.py:452), in parse_single_example_v2(serialized, features, example_names, name)
    450 serialized = ops.convert_to_tensor(serialized, name="serialized")
    451 serialized = _assert_scalar(serialized, "serialized")
--> 452 return parse_example_v2(serialized, features, example_names, name)

File [~/miniconda3/envs/mtr/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py:206](https://file+.vscode-resource.vscode-cdn.net/home/zetlin/Code/WaymoDataset/waymo-od/tutorial/~/miniconda3/envs/mtr/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py:206), in add_dispatch_support..wrapper(*args, **kwargs)
    204 """Call target, and fall back on dispatchers if there is a TypeError."""
    205 try:
--> 206   return target(*args, **kwargs)
...
     60                                       inputs, attrs, num_outputs)
     61 except core._NotOkStatusException as e:
     62   if name is not None:

InvalidArgumentError: Key: roadgraph_samples[/xyz.](https://file+.vscode-resource.vscode-cdn.net/xyz.)  Can't parse serialized Example. [Op:ParseExampleV2]

The max number of map points in tf_examples is 30k in the new motion dataset, instead of 20k

Thank you for flagging this.  We will update the tutorial.

That's it, thank you very much!

That's it, thank you very much!

Hi. I had a same problem as you stated. Have you solved it now?


fix DeprecationWarning for win ascent & descent fallbacks

Small edit to get rid of DeprecationWarning when openTypeOS2WinAscentFallback and openTypeOS2WinDescentFallback are called.

thanks!


WIP tests: record listing with multiple serializers

Adds tests for record listing with multiple serializers.

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

Coverage decreased (-0.5%) to 93.236% when pulling 47201ed7ff8fbbd63e1834408531d1f0d9179efa on jirikuncar:resumption-tests into a628f491c48b58b7ba4d349d1baf5ee69b365f78 on inveniosoftware:master.

Coverage decreased (-0.4%) to 93.273% when pulling 469a79959fed14f19a6edafd84ad0ca95518a193 on jirikuncar:resumption-tests into a628f491c48b58b7ba4d349d1baf5ee69b365f78 on inveniosoftware:master.


Is the document possible with multi-language support？

There seems to be only English documents

I'd be happy to translate it into Chinese if I could


@fenwang's 7 GB to 5.5 GB (512x512)

https://github.com/fengwang/Stable-Diffusion-NCNN

hi, It looked like there was this memory optimization applicable to 8gb laptops (no swap requirement)
Is this also applicable to android mobile, specifically?
You have mentioned a while ago the f16 512x512 version working on your 12gb ram android.
Also, is the 256x256 model a fixed shape, and is a quantized model?

thanks. my bad, the models were obviously labeled.
I used your diffusers conversion repository successfully, may I ask if the current vae decoder provided via cloud drive is the separate nai vae, or the one built-in (regular SD)?
The int8 and/or uint8 quantization process is easy enough with onnx, but I don't know how to do this with .pt (pnnx)
Your ncnn implementation is currently as fast as the openvino implementation, and supports multiple sizes.
I am interested in quantization because you have an apk release that supports 6gb currently, and 4gb may be supported if the quantized model is used, or maybe some low ram optimization. I'm not sure.

But considering that diffusion is a noisy computational process, I don't think quantifying such data will give a good result.


Project output ordering at the source

Describe the bug
Assume that input source is already ordered by Column a. Also assume that it consists of Columns a,b (with this order in the schema). When I run the query below.
SELECT a FROM annotated_data
  ORDER BY a

It produces following plan
"CsvExec: files={1 group: [[FILE_PATH]]}, has_header=true, limit=None, projection=[a]",

However, If input source schema were consist of Columns b, a (with this order in the schema). The query above produces following plan
"SortExec: expr=[a@0 ASC NULLS LAST]",
"  CsvExec: files={1 group: [[FILE_PATH]]}, has_header=true, limit=None, projection=[a]",

To Reproduce
No response
Expected behavior
I expect for second case to not produce SortExec in its physical plan.
Additional context
I think during output_ordering calculation for sources, we do not consider projection information. Hence output_ordering generated may not always be valid.

I fixed a bug related to the output_ordering and output_partitioning in ProjectionExec.  I'm not sure whether it is related or not, looks like it is related to the projection pushed down to the scan ?

I fixed a bug related to the output_ordering and output_partitioning in ProjectionExec. I'm not sure whether it is related or not, looks like it is related to the projection pushed down to the scan ?

I have verified that after your fix is introduced. This problem still occurs. I traced the bug, it is related to the projection pushed down to the scan. Source doesn't use projection information during calculation of output_ordering. I will probably file a fix for this bug today.


'Back to Top' button is not working properly

Describe the bug
On clicking the Back to Top button the webpage should scroll back to the top but the button is not working
Expected behaviour
On clicking the Back to Top button the webpage should scroll back to the top smoothly
Screenshots
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

hello, @Sanglap-Halder can you assign to me


Automated Package Upgrade

:tada: This PR is included in version 2.2.116 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


traceur-runtime remote

In the index.html there's still one file that is loaded from http and not from local package

just do:
jspm install traceur-runtime

That shouldn't be required if you specify Traceur as the runtime during jspm init
I think jspm changed this recently. Before you had to add a --runtime flag to bundle-sfx. The most recent version should bundle the runtime by default unless you specify --no-runtime.


Question: is there a way to define an exact URL as a starting point for an app?

Given I'd like to load a Trello instance with a specific board, could I achieve it within Ferdi settings?
ATM I did it with a custom recipe, but keeping it up to date is rather awkward.

You could achieve this via a custom userscript - something like:
if (window.location.pathname === "/initial/path") {
    window.location.pathname = "/new/path"
}

For now I changed config.serviceURL in package.json. The first time it redirects to a sign-in page and then always opens the board I need.
Do you mean user.js? As I can see it is the same for all instances of a service. Meantime, I want to have Trello in different workspaces with different starting boards respectively.

Yes, I mean user.js. You can use the config option there to switch to different workspaces on different service instances (see https://github.com/getferdi/ferdi/wiki/Using-user.css-and-user.js#userjs)

Sorry, but I can't find anything on workspaces on the page you linked.
These two options look interesting hasCustomUrl and hasPredefinedUrl though. I guess I'll need to check Franz docs for that.

Oh sorry, I misunderstood you there. You can only detect different instances of Trello using the config.id. In that case, I'd suggest creating multiple Trello instances and putting them on different Workspaces.

hasCustomUrl mostly solves my issue. Now I can define different url for each instance using the same recipe.

Where could I find information on hasPredefinedUrl option? It looks like it is defined in src/models/Recipe.js but never actually used in the app.

No, its never used in the app - it's may have been an old option used in Franz 4 plugins.

ОК, @vantezzen thank you for helping me with this!

I need the exact same thing. Why isn't this a feature for trello boards? Or... better yet. Make every service "pick up" where last left off.

hasCustomUrl in the recipes' package.json will allow the user to do this. I haven't come across examples where trello has this sub-domain-based urls (like slack has the {teamId}.slack.com. IF there are evidence of such a pattern being used, we can gladly turn on this feature in the main repo.


Styling incorrect for object requests

Describe the bug

This is a query parameter. The description is at the bottom after all the properties are defined.
To Reproduce
Parameter defined as
{
    "name": "productId",
    "in": "query",
    "description": "Unique identifier for the product.",
    "schema": {
      "type": "object",
      "properties": {
        "eq": {
          "type": "string",
          "description": "Matches values that are equal to the specified value.",
          "format": "string"
        },
        "ne": {
          "type": "string",
          "description": "Matches values that are not equal to the specified value.",
          "format": "string"
        },
        "in": {
          "type": "string",
          "description": "Matches any of the values specified.",
          "format": "comma-separated-values"
        },
        "nin": {
          "type": "string",
          "description": "Matches none of the values specified.",
          "format": "comma-separated-values"
        }
      }
    }
},

Expected behavior
The parameter description should come first and the properties and their descriptions should come later. We can also add some sense of hierarchy using indentation.
How often does this bug happen?
Every time
System Info
No response
Additional Context
No response

The issue should now be fixed in v0.6.0 that I just published.
Before:

After:


Pick command

WIP

Codecov Report
Base: 34.32% // Head: 33.99% // Decreases project coverage by -0.33% :warning:

Coverage data is based on head (9514555) compared to base (63f1c9e).
Patch coverage: 2.85% of modified lines in pull request are covered.

:exclamation: Current head 9514555 differs from pull request most recent head 110dbe8. Consider uploading reports for the commit 110dbe8 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #84      +/-   ##
============================================
- Coverage     34.32%   33.99%   -0.34%
  Complexity      422      422
============================================
  Files           133      136       +3
  Lines          3263     3298      +35
  Branches        337      341       +4
============================================
+ Hits           1120     1121       +1
- Misses         2036     2070      +34
  Partials        107      107

Impacted Files
Coverage Δ

...in/java/nus/climods/logic/commands/AddCommand.java
64.70% <ø> (ø)

...n/java/nus/climods/logic/commands/PickCommand.java
0.00% <0.00%> (ø)

...n/java/nus/climods/logic/parser/CliModsParser.java
0.00% <0.00%> (ø)

...va/nus/climods/logic/parser/PickCommandParser.java
0.00% <0.00%> (ø)

.../logic/parser/parameters/FacultyCodeParameter.java
100.00% <ø> (ø)

...s/logic/parser/parameters/LessonTypeParameter.java
0.00% <0.00%> (ø)

...s/logic/parser/parameters/ModuleCodeParameter.java
100.00% <ø> (ø)

...main/java/nus/climods/logic/parser/ParserUtil.java
51.61% <20.00%> (-6.08%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Improvements in style

[x] Use a free font
[ ] Use a dark colour scheme for codeblocks
[x] Fix the icon footer style for about.md
[x] Give border radius to the profile image.
[x] Split the about in two paragraphs

This is done in #36


protoize

Protoize old K&R style function declarations and definitions.

Thanks for the PR. Will likely merge on Monday.


Doesn't search webpage sources

I noticed this a short while ago and just checked with the new update. If you look at a page's source in-browser (such as with '<Ctrl+U>'), you cannot search the source with the extension. To confirm this, just open this page's source, and look for your name, not even with metacharacters. You'll be able to find it with '<Ctrl+F>', but not with Regex Search.

I'm not sure what you mean by '', but if you're talking about pages that start with view-source:, then I've looked into this and I believe it isn't possible. The Chrome extension API doesn't allow for it.

Sorry, I was trying to use Markdown on each of those keyboard shortcuts and didn't preview it. Had to rush away. ^^; If it's not possible, then it's not possible, I guess.

Also related: https://github.com/gsingh93/regex-search/issues/7


function ruuter.core/route+req->response overwrites ":param" keyword from ring request map

Hi o/
I was using ruuter with some ring middleware: ring.middleware.params/wrap-params and ring.middleware.keyword-params/wrap-keyword-params from ring/ring-core "1.10.0"
The first one parses any payload in a request body a puts it in the :params and :form-params keys of the request map (it also parses any query params but I was using form params)
The second one turn to keywords the keys of the :params map.
But I was getting an empty params map in my handler.
I noticed that in L108 and L109 of ruuter.core there's:
    (response (->> {:params (path+uri->path-params path uri)}
                   (merge req)))

in the route+req->response function
Since it's using thread last it's overwriting the parsed form params from the middleware
  (merge
   ;; form params from wrap-params ring middleware
   {:params {:content "hello world"}}
   ;; empty query params from ruuter (OK since I don't have any in that route)
   {:params {}})
  ;; => {:params {}}

I don't know if changing it to a thread first macro breaks anything :thinking:
 (merge
   ;; empty query params from ruuter (OK since I don't have any in that route)
   {:params {}}
   ;; form params from wrap-params ring middleware
   {:params {:content "hello world"}})
  ;; => {:params {:content "hello world"}}

Hi!
Thank you for the bug report, I'll give it a look later today unless you want to make a PR. I think you're right in that changing it to be thread-first would solve it, since then the :params from Ruuter would be the thing being overwritten, not the req, allowing Ring middlewares to overwrite this as they wish.

@galuque this issue is now fixed in the newly released 1.3.4 version.

Awesome! that's a really nice solution


Minor tweaks to go CI

Scope

Removing redundant go vet CI step
Making sure go tooling is standardized on 1.17
Moving to pre-built golangci-lint action

Why?

Less variability, faster builds, simpler CI.

Todos
None.

Thanks @vtleonardo!


Split bill - 'No results found' appears

I have read the CLA Document and I hereby sign the CLA
Details

On here
Selected members who is in the search result  isn't considered in the headerMessage function  . so we need to consider it.
Fixed Issues

$ https://github.com/Expensify/App/issues/22959
PROPOSAL: https://github.com/Expensify/App/issues/22959#issuecomment-1645202830
Tests

Click + icon > Split bill > Enter amount > Next.
Type an email address.
Select the member in search result
verify 'No results found' does not appears

[x]  Verify that no errors appear in the JS console

Offline tests

Click + icon > Split bill > Enter amount > Next.
Type an email address.
Select the member in search result
verify 'No results found' does not appears

QA Steps

Click + icon > Split bill > Enter amount > Next.
Type an email address.
Select the member in search result
verify 'No results found' does not appears

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/139998585/baf9c6ca-d9a0-4b56-9307-8e0f71f84ba5

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

I have read the CLA Document and I hereby sign the CLA

I have rearise PR.
@fedirjh  @madmax330  please review it.

The linter is failing

@madmax330  I have pushed again. would you please check again?

@madmax330  pushed. Please check.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/madmax330 in version: 1.3.53-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.53-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[SPARK-14137][SPARK-14150][SQL] Infer IsNotNull constraints from non-nullable attributes

What changes were proposed in this pull request?
This PR adds support for automatically inferring IsNotNull constraints from any non-nullable attributes that are part of an operator's output. This also fixes the issue that causes the optimizer to hit the maximum number of iterations for certain queries in https://github.com/apache/spark/pull/11828.
How was this patch tested?
Unit test in ConstraintPropagationSuite

Test build #54157 has started for PR 11953 at commit b630c17.

Test build #54157 has finished for PR 11953 at commit b630c17.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54157/
Test FAILed.

Merged build finished. Test FAILed.

cc @davies: I verified that this patch fixes union20 in HiveCompatibilitySuite. The issue apparently was that when NullFiltering rule removed the filters for non-nullable attributes, we lost track of their corresponding constraints (and they'd then be pushed back by InferFiltersFromConstraints)

Test build #54159 has started for PR 11953 at commit 6421ba7.

Test build #54159 has finished for PR 11953 at commit 6421ba7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54159/
Test PASSed.

Merged build finished. Test PASSed.

#11828 Has been merged, could you enable union20 here ?

thanks, done

Test build #54191 has started for PR 11953 at commit 2f43d41.

LGTM

Test build #54191 has finished for PR 11953 at commit 2f43d41.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54191/
Test PASSed.

Merged build finished. Test PASSed.

Merging this into master


Editor GPU: Opening a folded section double renders the line

Something odd with this one, I got in a state where the DOM renderer rendered it twice, or once with the GPU line. So maybe the problem is somewhere in ViewLine but caused by the canRender condition?


[WIP] Adding cleanup task

Optional task useful for removing users in Intercom who don't exist
in the local database. This can be a destructive task, so it's not bundled
into the IntercomBulkLoadTask for that reason.

A simple solution could be to add sleep(10); inside the BulkUpload Loop - it seems to be the only way to get around the rate-limiting.
At the moment, the Bulk docs are missing, and I was unable to get the rate-limiting headers as described to be able to have a more accurate sleep, so we are best off just throwing in a sleep.

Here you go: https://github.com/silverstripe/silverstripe-intercom/pull/10


New version: OBSProject.OBSStudio.Pre-release version 28.1.0-beta1

Result: Installation Successful

Manifest
Add/Remove Programs

Name
OBS Studio Beta
OBS Studio

Version
28.1.0-beta1
28.1.0

Publisher
OBS Project
OBS Project

ProductCode

OBS Studio

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 3190
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Color R^ error

When I try to run the scripts they all generate the following error.
E:\My_GitHub_Repo\Git\My_Scripts\LTE_Simulation>python smallNetworkSINR.py
C:\Users\Yoga 12\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyltes\printer.py:102: UserWarning: color is redundantly defined by the 'color' keyword argument and the fmt string "r^" (-> color='r'). The keyword argument will take precedence.
ax.plot(bs_x_locations, bs_y_locations, 'r^', color="black", markersize=10)
C:\Users\Yoga 12\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyltes\printer.py:110: UserWarning: color is redundantly defined by the 'color' keyword argument and the fmt string "b*" (-> color='b'). The keyword argument will take precedence.
ax.plot(ue_x_locations, ue_y_locations, 'b*', color="blue", markersize=10)

Hi!
It's just a warning, I just checked it and everything works fine.
To be honest, I don't know how to correct arguments in these functions, it's related to the newer version of matplotlib.


Link doesn't work

see Packages, runbooks, and the AXUpdateInstaller in depth at the end of this topic.

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

ID: 4008b2e3-4d1a-e146-8bdb-f39280124e9f
Version Independent ID: 22f8390d-bb4d-1755-515c-7e7ebf1ddaae
Content: Apply updates to cloud environments - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/deployment/apply-deployable-package-system.md
Service: dynamics-ax-platform
GitHub Login: @manalidongre
Microsoft Alias: manado

Thanks for letting us know!  I've fixed the link.  The updated article will be published this week.


Select: Truncates longer Options

I 100% admit I may just be stupid, but is there a way to have the dropdown on select as wide to fit the longest option name so it doesn't get truncated? (:

At this time select has not attribute for automatic  set width

Ahh OK so at least its not just me being stupid (:   Is it possible to ask for that in future?

You can use templates for options and add title attribute to template https://metroui.org.ua/select.html#_select_template

Ahh OK so at least its not just me being stupid (: Is it possible to ask for that in future?

I'll think about whether it can be done without much effort.

(: no rush, templates will work for now
On Tue, 21 May 2019 15:36 Sergey Pimenov<EMAIL_ADDRESS>wrote:

Ahh OK so at least its not just me being stupid (: Is it possible to ask
for that in future?
I'll think about whether it can be done without much effort.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/olton/Metro-UI-CSS/issues/1369?email_source=notifications&email_token=AHP6BN2DTJPVXKHQY47QKVDPWP3GXA5CNFSM4HOK7G3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV35WCA#issuecomment-494394120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHP6BN4XUF6DWL6GIS6IIIDPWP3GXANCNFSM4HOK7G3A
.


Plugin tool search path on Windows does not take into account drive letter

Is it reproducible with SwiftPM command-line tools: swift build, swift test, swift package etc?

[X] Confirmed reproduction steps with SwiftPM CLI. The description text must include reproduction steps with either of command-line SwiftPM commands, swift build, swift test, swift package etc.

Description
Minimal plugin example:
import PackagePlugin

@main
struct IssueDemoPlugin: BuildToolPlugin {
  func createBuildCommands(context: PluginContext, target: Target) async throws -> [Command] {
    print(try context.tool(named: "clang").path)
    return []
  }
}

The user folder on my computer is installed on the D drive. When printing the contents of context, I see this part:
toolSearchDirectories: [/Users/Admin/AppData/Local/Programs/Swift/Toolchains/6.0.2+Asserts/usr/bin], toolSearchDirectoryURLs: [file:///Users/Admin/AppData/Local/Programs/Swift/Toolchains/6.0.2+Asserts/usr/bin]

Needless to say, the file D:\Users\Admin\AppData\Local\Programs\Swift\Toolchains\6.0.2+Asserts\usr\bin\clang.exe exists, and yet I get this error:
error: Plugin does not have access to a tool named ‘clang’

Expected behavior
Should print the path to clang.exe that comes with the toolchain
Actual behavior
error: Plugin does not have access to a tool named ‘clang’

Steps to reproduce

mkdir PluginToolMVP
cd PluginToolMVP
swift package init
Edit Package.swift

// swift-tools-version: 6.0
// The swift-tools-version declares the minimum version of Swift required to build this package.

import PackageDescription

let package = Package(
	name: "PluginToolMVP",
	products: [
		// Products define the executables and libraries a package produces, making them visible to other packages.
		.library(
			name: "PluginToolMVP",
			type: .dynamic,
			targets: ["PluginToolMVP"]),
		.plugin(
			name: "IssueDemoPlugin",
			targets: ["IssueDemoPlugin"]),
	],
	targets: [
		// Targets are the basic building blocks of a package, defining a module or a test suite.
		// Targets can depend on other targets in this package and products from dependencies.
		.target(
			name: "PluginToolMVP",
			plugins: ["IssueDemoPlugin"]),
		.testTarget(
			name: "PluginToolMVPTests",
			dependencies: ["PluginToolMVP"]),
		.plugin(
			name: "IssueDemoPlugin",
			capability: .buildTool()),
	]
)

Create Plugins/IssueDemoPlugin/plugin.swift:

import PackagePlugin

@main
struct IssueDemoPlugin: BuildToolPlugin {
  func createBuildCommands(context: PluginContext, target: Target) async throws -> [Command] {
    print(try context.tool(named: "clang").path)
    return []
  }
}

swift build

Swift Package Manager version/commit hash
6.0.2
Swift & OS version (output of swift --version ; uname -a)
>swift --version
Swift version 6.0.2 (swift-6.0.2-RELEASE)
Target: x86_64-unknown-windows-msvc

OS Name: Microsoft Windows 10 Pro
Version: 10.0.19045 Build 19045

P.S. Not directly related to the issue, but the URLs in target.sourceFiles seem to lack a drive letter as well. This can be an issue when passing them to tools.


PowerShell Core: Unable to Prompt For SecureString Over PSSession

Steps to reproduce
On a Windows Client running PowerShell Core v6.1.0-preview.3, create a PSSession (via SSH) to a CentOS7 host running v6.1.0-preview.2 and do the following
PS C:\Users\zeroadmin> $CentOS7PSSession = New-PSSession -HostName <IP_ADDRESS> -KeyFilePath "$HOME\.ssh\vagrant_unsecure_key" -UserName vagrant
PS C:\Users\zeroadmin> Enter-PSSession -Session $CentOS7PSSession
[<IP_ADDRESS>]: PS /home/vagrant> $test = Read-Host "Pwd"
Pwd: PlainTextPwd
[<IP_ADDRESS>]: PS /home/vagrant> $test2 = Read-Host "Pwd" -AsSecureString
WARNING: A script or application on the remote computer <IP_ADDRESS> is sending a prompt request. When you are prompted, enter sensitive information, such as credentials or passwords, only if you trust the remote computer and the application or script that is requesting the data.
Pwd: ***********

# Hangs indefinitely here until ctrl+c

There is an error processing data from the background process. Error reported: Object reference not set to an instance of an object..
PS C:\Users\zeroadmin> Enter-PSSession -Session $CentOS7PSSession
Enter-PSSession : Cannot enter session Runspace5 because it is not in the disconnected state or is not available for connection. Retrieve the remote session using Get-PSSession -ComputerName <IP_ADDRESS> -InstanceId d8e069fb-f075-432c-a394-51e50a9f86b9.
At line:1 char:1
+ Enter-PSSession -Session $CentOS7PSSession
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : InvalidArgument: (:) [Enter-PSSession], ArgumentException
+ FullyQualifiedErrorId : PushedRunspaceMustBeOpen,Microsoft.PowerShell.Commands.EnterPSSessionCommand

PS C:\Users\zeroadmin>

Expected behavior
Successfully create the securestring in a remote session
Actual behavior
Hangs indefinitely and breaks the PSSession.
Environment data
Windows Client Info:
PS C:\Users\zeroadmin> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.1.0-preview.3
PSEdition                      Core
GitCommitId                    v6.1.0-preview.3
OS                             Microsoft Windows 10.0.16299
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

PS C:\Users\zeroadmin> Get-Command ssh

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Application     ssh.exe                                            <IP_ADDRESS>    C:\Program Files\OpenSSH-Win64\ssh.exe

CentOS7 Remote Host Info:
[<IP_ADDRESS>]: PS /home/vagrant> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.1.0-preview.2
PSEdition                      Core
GitCommitId                    v6.1.0-preview.2
OS                             Linux 3.10.0-862.6.3.el7.x86_64 #1 SMP Tue Jun 26 16:32:21 UTC 2018
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

[<IP_ADDRESS>]: PS /home/vagrant> sudo pwsh -Command 'Get-Content /etc/ssh/sshd_config'
#       $OpenBSD: sshd_config,v 1.100 2016/08/15 12:32:04 naddy Exp $

# This is the sshd server system-wide configuration file.  See
# sshd_config(5) for more information.

# This sshd was compiled with PATH=/usr/local/bin:/usr/bin

# The strategy used for options in the default sshd_config shipped with
# OpenSSH is to specify options with their default value where
# possible, but leave them commented.  Uncommented options override the
# default value.

# If you want to change the port on a SELinux system, you have to tell
# SELinux about this change.
# semanage port -a -t ssh_port_t -p tcp #PORTNUMBER
#
#Port 22
#AddressFamily any
#ListenAddress <IP_ADDRESS>
#ListenAddress ::

HostKey /etc/ssh/ssh_host_rsa_key
#HostKey /etc/ssh/ssh_host_dsa_key
HostKey /etc/ssh/ssh_host_ecdsa_key
HostKey /etc/ssh/ssh_host_ed25519_key

# Ciphers and keying
#RekeyLimit default none

# Logging
#SyslogFacility AUTH
SyslogFacility AUTHPRIV
#LogLevel INFO

# Authentication:

#LoginGraceTime 2m
#PermitRootLogin yes
#StrictModes yes
#MaxAuthTries 6
#MaxSessions 10

#PubkeyAuthentication yes

# The default is to check both .ssh/authorized_keys and .ssh/authorized_keys2
# but this is overridden so installations will only check .ssh/authorized_keys
AuthorizedKeysFile      .ssh/authorized_keys

#AuthorizedPrincipalsFile none

#AuthorizedKeysCommand none
#AuthorizedKeysCommandUser nobody

# For this to work you will also need host keys in /etc/ssh/ssh_known_hosts
#HostbasedAuthentication no
# Change to yes if you don't trust ~/.ssh/known_hosts for
# HostbasedAuthentication
#IgnoreUserKnownHosts no
# Don't read the user's ~/.rhosts and ~/.shosts files
#IgnoreRhosts yes

# To disable tunneled clear text passwords, change to no here!
#PasswordAuthentication yes
#PermitEmptyPasswords no
PasswordAuthentication no

# Change to no to disable s/key passwords
#ChallengeResponseAuthentication yes
ChallengeResponseAuthentication no

# Kerberos options
#KerberosAuthentication no
#KerberosOrLocalPasswd yes
#KerberosTicketCleanup yes
#KerberosGetAFSToken no
#KerberosUseKuserok yes

# GSSAPI options
GSSAPIAuthentication yes
GSSAPICleanupCredentials no
#GSSAPIStrictAcceptorCheck yes
#GSSAPIKeyExchange no
#GSSAPIEnablek5users no

# Set this to 'yes' to enable PAM authentication, account processing,
# and session processing. If this is enabled, PAM authentication will
# be allowed through the ChallengeResponseAuthentication and
# PasswordAuthentication.  Depending on your PAM configuration,
# PAM authentication via ChallengeResponseAuthentication may bypass
# the setting of "PermitRootLogin without-password".
# If you just want the PAM account and session checks to run without
# PAM authentication, then enable this but set PasswordAuthentication
# and ChallengeResponseAuthentication to 'no'.
# WARNING: 'UsePAM no' is not supported in Red Hat Enterprise Linux and may cause several
# problems.
UsePAM yes

#AllowAgentForwarding yes
#AllowTcpForwarding yes
#GatewayPorts no
X11Forwarding yes
#X11DisplayOffset 10
#X11UseLocalhost yes
#PermitTTY yes
#PrintMotd yes
#PrintLastLog yes
#TCPKeepAlive yes
#UseLogin no
#UsePrivilegeSeparation sandbox
#PermitUserEnvironment no
#Compression delayed
#ClientAliveInterval 0
#ClientAliveCountMax 3
#ShowPatchLevel no
#UseDNS yes
UseDNS no
#PidFile /var/run/sshd.pid
#MaxStartups 10:30:100
#PermitTunnel no
#ChrootDirectory none
#VersionAddendum none

# no default banner path
#Banner none

# Accept locale-related environment variables
AcceptEnv LANG LC_CTYPE LC_NUMERIC LC_TIME LC_COLLATE LC_MONETARY LC_MESSAGES
AcceptEnv LC_PAPER LC_NAME LC_ADDRESS LC_TELEPHONE LC_MEASUREMENT
AcceptEnv LC_IDENTIFICATION LC_ALL LANGUAGE
AcceptEnv XMODIFIERS

# override default of no subsystems
Subsystem       sftp    /usr/libexec/openssh/sftp-server
Subsystem powershell /bin/pwsh -sshs -NoLogo -NoProfile

# Example of overriding settings on a per-user basis
#Match User anoncvs
#       X11Forwarding no
#       AllowTcpForwarding no
#       PermitTTY no
#       ForceCommand cvs server
[<IP_ADDRESS>]: PS /home/vagrant>

Hi @pldmgg,
I notice your sshd_config has the Port 22 line still disable. You need to make sure both Windows and Linux has that port available for openssh to work.
Also, make sure you don't include the path in CentOS as it will find it automatically:
Subsystem	powershell pwsh -sshs -NoLogo -NoProfile

:)

@pldmgg, Apologies!!  I see now.
So you're getting connected but creating the secure string hung the session. I'm trying it now with Preview.3.

Yeap! I can replicate with simply doing:
Enter-PSSession -HostName mercury -UserName maxt
maxt@mercury's password:
[mercury]: PS /home/maxt>
[mercury]: PS /home/maxt> $test2 = Read-Host "Pwd" -AsSecureString
WARNING: A script or application on the remote computer MERCURY is sending a prompt request. When you are prompted, enter sensitive information, such as credentials or passwords, only if you trust the remote computer and the application or script that is requesting the data.
Pwd: *****

Hung Here

There's the possibility that's happening in all distro of Linux (not only CentOS).

Yeap! Linux CentOS to Ubuntu the same thing.

@MaximoTrinidad Thanks for taking the time to reproduce. I have a feeling there might be something about this somewhere in https://github.com/PowerShell/Win32-OpenSSH/issues ... I'll try and take a look tomorrow

On the contrary! Thanks @pldmgg for catching it now.  For sure, someone will take a look at this next week.
:)

Dup of #1654.  Closing...


Update workflow template documention

Add documentation for new postproc and module section in the workflow template documentation.

Updated parameter declaration section.
Still needed are:

[ ] Result schema and post-processing workflows
[ ] Output file formats and widgets


FMUK66 UDP telemetry stream terminating after a few seconds

Describe the bug
MavLink Telemetry stream via UDP over T1 ethernet port on FMUK66 is terminating with current PX4 master after a few seconds. This was not the case with https://github.com/PX4/Firmware/pull/15154
To Reproduce
Steps to reproduce the behavior:

adjust /etc/extras.txt on SDcard of FMUK66 with "mavlink start -x -u 14550 -r 200000 -t <IP_ADDRESS> -m onboard" where -t flag defines IP address of computer running QGC.
connect ethernet cable via T1 media converter to T1 ethernet port on FMUK66.
Set own computer IP to <IP_ADDRESS>
Open QGC on computer
connect to FMUK66 via USB and set MAV_BROADCAST parameter to 2 - Only multicast
disconnect USB and power FMU from battery
reboot FMUK66
telnet into FMU (FMUK66 has <IP_ADDRESS> as default IP) to verify connection
FMU sends Mavlink data to QGC via UDP over T1 ethernet port
after a few seconds mavlink stream terminates
revert PX4 FW on FMUK66 to commit df73662
reboot FMUK66
FMU sends Mavlink data to QGC via UDP over T1 ethernet port without interruption

Expected behavior
Expectation is that UDP data stream via T1 ethernet port runs stable with current master as it was implemented with commit df73662.
Log Files and Screenshots
na
Add screenshots to help explain your problem.
Drone (please complete the following information):
NXP test drone
FMUK66 with PX4 master
NavQ companion computer
Additional context
My test setup

QGC is on linux machine in local network as <IP_ADDRESS>.
Same happens when QGC is connected directly via T1 media converter to FMU. somwhere between commit df73662 and now an change has been applied to let the UDP stream terminate after a while.

@davids5 is this something you can look at please?

hmm.. RTFM.. looks like MAV_BROADCAST set to 1 solves the problem.

Ok, the problem is solved for me if i set the MAV_BROADCAST parameter to 2 - Only multicast

Ok, the problem is solved for me if i set the MAV_BROADCAST parameter to 2 - Only multicast

Ok, the problem is solved for me if i set the MAV_BROADCAST parameter to 2 - Only multicast

@davids5 - Hi David, now it's an issue again. May you please look at that. Something has happened between between commit df73662 and now.

@dk7xe - I can get to it the latter half of the week

@dk7xe See https://github.com/PX4/NuttX/pull/116 once that merges I will do a Firmware update.

Hi @davids5  PX4/NuttX#116 has been merged already. Have u done the FW update?

@dk7xe yes
You can see what is on master of Firmware by following https://github.com/PX4/Firmware/tree/master/platforms/nuttx/NuttX
to nuttx, then list commits
https://github.com/PX4/NuttX/commits/ec20f2e6c5cc35b2b9bbe942dea55eabb81297b6

Thanx David. Meanwhile successfully tested.


Integration Test - Release - 2022/01/14 03:17:55

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign NonAllowedUser

#assign PRMergerTest1

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign NonAllowedUser

#reassign PRMergerTest2

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#unassign PRMergerTest1

#unassign NonAllowedUser


Demon Sect False(?) Cloudflare Error

Source information
Demon Sect 1.4.44
Source language
Brazilian Portuguese
Steps to reproduce

Open the extension
Pass cloudflare by opening it on webview
Try reloading the extension

Expected behavior
it should pass cloudflare as shown in the webview and the fetch the manga from the extension
Actual behavior
it keeps giving cloudflare error even though everything is alright on the webview
Mihon/Tachiyomi version
0.16.5
Android version
Android 13
Other details
tried cleaning cookies, changing UA and even reinstalling the extension, DoH is already enabled
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Aqui está funcionando

?blurb cf

try navigting to the popular/latest url in webview and passing cloudflare there, like https://dsectcomics.org/manga/?m_orderby=views

tried that and worked one time, now it is the same as before, it passes on webview but on the app it keeps giving cloudflare error, maybe it's a problem with only me? idk, kinda strange since even after changing the UA the same thing happens


absl/debugging: use <execinfo.h> only when available

Instead of relying on GLIBC or other unreliable detection
mechanism, simply detect if <execinfo.h> is available before using the
stacktrace_generic-inl.inc implementation.
Signed-off-by: Thomas Petazzoni<EMAIL_ADDRESS>

@googlebot I signed it!

The proposed change only works with CMake. We need something that works with Bazel as well. While we understand that the only reliable thing to do is check for the presence of the header, there is a reason we currently do it the way we do.
What platform are you trying to fix with this change?

The uClibc-ng C library defines GLIBC, but its execinfo.h support is optional, so some uClibc-ng toolchains might have execinfo.h, some might not. The musl C library does not implement execinfo.h, and does not define GLIBC. So this is handled today on ARM by the check on GLIBC in your code, but it is not handled for any other CPU architecture.
Checking the execinfo.h header is really the only reliable thing to do here. I am not familiar with Bazel, but checking whether a system header is available or not is a pretty standard functionality for build systems, so certainly there should be a way to do that as well ?

Bazel has different design goals from many traditional build systems, so it doesn't actually have functionality for checking for the existence of a header.
You might be able to do something with __has_include to check directly during compilation.

Isn't has_include quite new (C++17 maybe ?) and therefore only available in very recent compilers?
Also, wouldn't it be possible for the Bazel build system to always define HAVE_EXECINFO_H, which would pretty much preserve the same behavior as today ?

I see support for __has_include going back all the way to gcc 4.9.2 (https://godbolt.org/z/GWdM9E), which is good enough for me. We should prefer mechanisms that are build-system neutral given that Bazel is very important to us, and maintaining two build systems is a burden.

I'm closing this issue as I'm working on a different change to address this issue.