Can't login anymore - Cannot read properties of undefined (reading 'split')

I think there might be login issues with newly created accounts. After a few publishes, the API cannot connect to my HackerNews bot, neither can I. When trying to connect manually (to either Instagram or Threads), I just get generic error messages, and it looks like I can't even reset my password and use the magic login Instagram link.
When trying to connect through the API I get TypeError: Cannot read properties of undefined (reading 'split'). It comes from line 496 of src/threads-api.ts, because I don't get the Bearer token.
I'm not sure we can fix this through the API given that I can't even reset my bot password manually, it most likely comes from automated Instagram anti-hack protection.
So be careful when login and publishing a lot, especially with your own account (although maybe real accounts have higher hidden "limits" than newly created ones like my bot).

Exactly, so I think we need to come up with a way to store the token separately in a local environment once logged in as an additional measure.
This is something we'll be doing an update on soon.

I think there's already everything in the API to let devs store the token how they see fit (through getToken and the constructor's token arg).
Maybe just a warning (in the console) that says that "a new ThreadsAPI instance without a token will try to get one anytime at the first write of the instance, which can lead to disabling the account"

Update: the account works again, I tried various things and not sure what did it:

changed email, then password (I could because I was still connected to the account on my phone)
I followed someone on IG, maybe it unflagged the bot?

But the most likely is that none of the things I did above really had an impact


Wrong y axis label position for line/circle in case of both axis dimension

The best solution would be to put the markers in the middle instead of at the top of the positions where the bars would be

This is not working yet, because orientation change is not recognized as an animation step (jumps)


Add internal evaluation to sir-lancebot

Description
Admins should be able to run evaluations on sir-lancebot at runtime.
Proposed Implementation
The implementation can be similar to the one in the bot, but it's possible the code is outdated or over-complicated, so alternative suggestions are fine.
Would you like to implement this yourself?

[ ] I'd like to implement this feature myself
[x] Anyone can implement this feature

We should be using @SebastiaanZ's implementation on Black Knight for that.


Create separate, not-modular, program to do the work for JBoss EAP

Date: Wednesday, March 22, 2023 at 19:21
Ed Burns wrote:
I can commit to hacking upon the existing code in a fork to create separate, not-modular, programs that do the work for WLS, JBoss EAP, and tWAS.

Internal User Story https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1779935

Hi @edburns, which do you prefer?
a.  So

So component is reusable like server, process...
But we need different discoverApp processor.
For Java EE, we can discovery multiple apps from one single process.


Fix #100: Enable Micrometer Metrics

Fix #100: Enable Micrometer Metrics
@rmanibus i am sure you can come up with something more clever but i did follow the pattern of how to detect Micrometer Metrics is installed and running.

I want to do some more testing on this first too.

OK all tested this is ready for review @rmanibus

Looks good to me !


Centralised logging

Hey folks, thanks for the great work on Metaflow.
This is a question, not an issue. How do you typically do distributed log tracing with Metaflow? We're deploying our Flows as AWS Step Functions, and we're doing a lot of clicking on AWS Console to get to the logs. And it gets especially tedious when we have SFNs that fan out to 50+ Tasks.
We have a centralised logging server/UI (something like Splunk), and we're shipping the logs produced by our Metaflow SFNs there, but without a single correlation id (e.g. Metaflow run-id) formatted in each log lines, it's not possible to do distributed log tracing.
Keen to hear how people typically solve this problem. Cheers

@davidtan-tw - Depending on how you are shipping logs back, you can access the run ids from the environment of the container. All user logs are also stored in S3 by Metaflow. You can access them as Run('myflow/run-id')['step'].task.stdout and Run('myflow/run-id')['step'].task.stderr. You may want to ask the question in chat.metaflow.org where there are quite a few folks who use Splunk with Metaflow.

@davidtan-tw Any follow-ups on this? Please re-open the issue if you would like to chat further.

@davidtan-tw we use structured logging using structlog ( see https://www.structlog.org/en/stable/logging-best-practices.html#canonical-log-lines) to add various ids etc to each logline (but then output only a few log lines)

Thanks @seanv507 . That's what we ended up doing as well. In our log formatter, we added run-id and also task-id (in case we need to drill down to logs for one of the specific parallel steps.


Mute command should be allowed to run without a time given (unlimited)

If someone runs the ;mute command without a length/time given, it should simply add the role and require them to run ;unmute to have the role removed.

Mute command is now redundant due to the built-in timeout feature


Feature: Optional Pillow/PIL

This improves the PNG renderer iff PIL/Pillow is available. Uses PyPNG as fallback

Note: Reduces the PNG rendering significately. It does not depend on PIL/Pillow but uses it iff available. Further optimazions (file size) would be possible if the image mode would be choosen more carefully...

TODO: Add tests with Pillow/PIL available vs. not available (current tests pick PIL/Pillow if available, otherwise PyPNG)

Note that test_png.test_png_as_base64_str fails with PIL/Pillow, all other tests pass.


Need ability to copy / overwrite Azure storage account contents from one container to another

We need the ability copy / overwrite assets from one Azure storage account container to another container of the same Azure storage account.  We need this as a part of a bigger process to restore one environment to another when needing to pull a production --> staging --> testing.  Need to determine which command argument this belongs under.  Not sure if deploy makes sense or if we should expand copy to include cloud assets or maybe it makes sense to add a new argument.  Any and all thoughts are welcome.
CC: @brandongregoryscott @wintondeshong @Stefanie899

Hmm, I'd hesitate to tack it on to copy at the moment because there's likely some authentication/configuration needed that would clutter up that command and in most cases won't be applicable. I agree, though, it doesn't seem like a standard deploy command, either. For now, I would probably name it azure-copy and build out the functionality there first.
Thinking ahead, we may have the need to add other cloud-based utility commands that are necessarily 'deployment' related, in which case it'll probably make sense to make azure a parent command and then register subcommands that contain more specific operations, instead of building out an azure-x command or adding a bunch of flags for each specific operation.
(For example, our github command is probably going to go the same route - it will become a parent command for the purpose of holding subcommands for topics, repos, issues, etc. since it is getting larger and operating w/ several different concerns)
I mentioned it in our Slack conversation, but if the existing and-cli-deploy-azure-storage.ts command can be used in some sequence/combination of flags to fit this purpose, let's try to use that first unless it ends up being overly complex or clunky.

Hmm, I'd hesitate to tack it on to copy at the moment because there's likely some authentication/configuration needed that would clutter up that command and in most cases won't be applicable. I agree, though, it doesn't seem like a standard deploy command, either. For now, I would probably name it azure-copy and build out the functionality there first.
Thinking ahead, we may have the need to add other cloud-based utility commands that are necessarily 'deployment' related, in which case it'll probably make sense to make azure a parent command and then register subcommands that contain more specific operations, instead of building out an azure-x command or adding a bunch of flags for each specific operation.
(For example, our github command is probably going to go the same route - it will become a parent command for the purpose of holding subcommands for topics, repos, issues, etc. since it is getting larger and operating w/ several different concerns)
I mentioned it in our Slack conversation, but if the existing and-cli-deploy-azure-storage.ts command can be used in some sequence/combination of flags to fit this purpose, let's try to use that first unless it ends up being overly complex or clunky.

After talking to @brandongregoryscott , we decided to create a new AzureStorage module and, at a bare minimum, add functionality to it to allow us to use the remove capability (see https://docs.microsoft.com/en-us/cli/azure/storage?view=azure-cli-latest#az_storage_remove).  The current and-cli-deploy-azure-storage command can then be updated to optional (through another optional argument) clear the existing destination's assets recursively.

After talking to @brandongregoryscott , we decided to create a new AzureStorage module and, at a bare minimum, add functionality to it to allow us to use the remove capability (see https://docs.microsoft.com/en-us/cli/azure/storage?view=azure-cli-latest#az_storage_remove).  The current and-cli-deploy-azure-storage command can then be updated to optional (through another optional argument) clear the existing destination's assets recursively.


Migrate to new JSONObject/JSONArray

@labkey-adam, @labkey-jeckels or others: i cant make branches on the LabKey org, which is needed for TeamCity to pick this up. Would someone be able to do this?

@labkey-adam, @labkey-jeckels or others: i cant make branches on the LabKey org, which is needed for TeamCity to pick this up. Would someone be able to do this?

@bbimber yes, we'll take care of this shortly

@bbimber I believe I've created TeamCity-accessible feature branches for all your PRs (platform, onprc, discvr, labdevkit, bimberlabkey). Tests started running (and failed) before all FBs were created. I'll re-queue the failed tests.

@labkey-adam, thanks. for my knowledge, is there a more correct way to kick off multiple coordinated PRs that depend on one another?
If anyone knows gradle, I think this issue might cause another build failure. I tried to migrate LDK/laboratory code from platform, and gradle isnt recognizing the new dependencies: https://www.labkey.org/ONPRC/Support Tickets/issues-details.view?issueId=47287

@labkey-adam, thanks. for my knowledge, is there a more correct way to kick off multiple coordinated PRs that depend on one another?
If anyone knows gradle, I think this issue might cause another build failure. I tried to migrate LDK/laboratory code from platform, and gradle isnt recognizing the new dependencies: https://www.labkey.org/ONPRC/Support Tickets/issues-details.view?issueId=47287

@bbimber I'm not aware of a better workflow. Normally, it would go faster, but I haven't ever created FBs against fork PRs I was slow to get them all created.
I'll look at the build failure, but we might need to bring in @labkey-susanh , who's mostly OOO today.

I'll look at the build failure, but we might need to bring in @labkey-susanh , who's mostly OOO today.

@bbimber see my reply in the issue

@labkey-adam thanks - just a reminder i dont have write permission to actually merge this.

@labkey-adam thanks - just a reminder i dont have write permission to actually merge this.

Yep, I plan to initiate all the merges myself (after one more PR review and I figure out how to get this PR to see the successful premium build)... unless you're dying to do some of the merges.


Go from sleeve to fastdp

I have seen a few nodes that have connections which are in "sleeve" but all others are in "fastdp".  This seems to occur somewhat randomly and the node never goes back to fastdp - is it possible to force a connection back to "fastdp"?

Hi @cwolfinger this is a known issue (#1737) - the only workaround available at the moment is to stop and restart the router I'm afraid.

This is a known issue and further debugging can be found here:
https://github.com/weaveworks/weave/issues/3252


WRQ-10126: Updated placeholder images URLs

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Updated placeholder images URLs
Resolution
Updated placeholder images URLs
Additional Considerations
Links
WRQ-10126
Comments
Enact-DCO-1.0-Signed-off-by: Paul Beldean<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: paul-beldean-lgp:x: dongsuwonYou have signed the CLA already but the status is still pending? Let us recheck it.

LGTM


rebase on main

rebase on main -- former comments/test plan:
Fixes errors where lines like this.foo = bar don't parse.
To test: use
npx tree-sitter test

The this assignment test should pass.
Pad's Comment:

'This is nice, and it might simplify the original grammar of Kotlin, but I'm a bit worried that we deviate now even
more from the original grammar. Fwcd already deviated from the original grammar a little on how he is handling expressions. I'd rather see us go back to the original spec of Kotlin where there is a directly_assignable_expression,
and a postfix_unary_operator, etc.'

Looks good, but we need to make this PR on the original fwcd repo no?

yes definitely 🤦♀


Display the created component and blade file in terminal like Livewire

@stefvanesch  Added in Development branch, see for test;
php artisan make:flex-component Stef/Lars


[OTHER] Show github greetings to new members only

What would you like to share?
@Dun-sin is it possible to show GH greetings to new members only when creating new issues? The messages are too much
Additional information
No response

What would you like to share?
@Dun-sin is it possible to show GH greetings to new members only when creating new issues? The messages are too much
Additional information
No response

That's because you create too many issues🤣🤣🤣
But yeah it is, had that before, people forget about the rules so brought this one back, plus it also changes


Update GaiResolver to use a tokio-threadpool

As part of the update to std::future, futures-cpupool was removed, and the GaiResolver is just blocking directly in the connect future 😢 . This should be fixed!
Things I've been thinking to make it better:

Use a tokio-threadpool, as it has better support for only spawning threads as needed, and sleeping and shutting down threads when not.
In some cases, if being executed on Tokio's multi-threaded runtime, we don't need a separate pool and could just use tokio_threadpool::blocking.
But if not, we'd need to spawn one inside the GaiResolver.

It'd be neat if we could handle all these cases automatically, instead of requiring configuring that could be configured wrongly...

The first part is done: a tokio-threadpool is used internally.


Mark every in progress-policy as broken in 100 days category

Description of your issue, suggested solution and other information
Every 100 days promise that is now on 'in progress' should be marked as broken until it is executed, then it should be compromised.

Make sure you are using the latest version: pull latest commits before reporting any issues.

[x] I've verified and I assure that I'm looking at the latest version of Trump Tracker.

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably the To Do List section.
[x] Searched the bugtracker for similar issues including closed ones

I will try to do this but I'm not sure if I can do this alone.

Can we add a clause to the 100 days promise so that people know and understand that if the president does complete any of the aforementioned promises, we will mark it as compromised?

24a5799ca8b75f1057e8b9cdb1a74375dac5fafd


Add intro to rust md file info section

This PR adds an info note for Rust beginners pointing them to materials on getting started with the language.

The broken link in the CI works fine from my machine


Fix tests

What does this PR do?
Refactors tests
Adds a test for s3 directories
Refactors configuration files
Fixes scrapers archiving
Adds color printing for errors on console

@DavidLemayian i've tested this changes on morph.io and they seem to run fine. Check it out

@Gathondu Thanks for this. I don't think it's necessarily a "fix tests" PR so you might want to make sure that you're working off the right branch and the description matches the work being done. 👌

Noted.


making LexRequirement an actual struct

I have always thought that making `LexRequirement` an actual struct with methods and documentation would be even better

For example,  something like

/// Describes a lexical (multi-column) sort requirement
struct LexRequirement {
  inner: Vec<PhysicalSortRequirement>>
}

impl LexRequirement {
// documented methods here
...
}

Otherwise the rust docs just show as `Vec<PhysicalSortRequirement>` and it is unclear what operations are used

Originally posted by @alamb in https://github.com/apache/datafusion/pull/12130#pullrequestreview-2257616876

FYI @ozankabak  and @mustafasrepo

Hi! Mind if I try taking this one up? My look-over of this is that most of the code can stay the same, we would just need Deref, and a few util functions for accessing the inner Vec on the struct.


build(cli): add scaffolding for pipeline status, incl help menu

This is the scaffolding for pipeline_status.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

manual output of help menu:
$ ecs-preview pipeline status --help
Shows the status of each deployed pipeline's stages.

Usage
  ecs-preview pipeline status [flags]

Flags
  -a, --app string    Name of the application.
  -h, --help          help for status
      --json          Optional. Outputs in JSON format.
  -n, --name string   Name of the pipeline.

Examples
Shows status of the deployed pipeline "pipeline-mycompany-myapp-myrepo".
`$ copilot pipeline status -n pipeline-mycompany-myapp-myrepo`
$


Deleting dataset share request does not delete row in the DB

The row is still left in the dataset_request table and a user cannot resubmit the share request because of that.


Update to 0.0.21 causing compile failure

After updating to get the latest changes I hit the following issue.
Looking it seems to be something to do with the type definition. I tried to add the typings manually but it is still failing. I loked through your repo to see if there is a setting I am missing but cannot see anything and could also not see any path changes you made for import using the scss: prefix in the import statement.
Suggestions?
✘ [ERROR] Cannot find module 'scss:./dialog.scss'
Require stack:
- D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\esbuild-plugin-sass<EMAIL_ADDRESS>[plugin sass]

    node_modules/.pnpm/@ophidian+core@0.0.21_@codemirror+state@6.4.1_@codemirror+view@6.26.3/node_modules/@ophidian/core/src/ui/dialog.ts:2:24:
      2 │ import dialogStyle from "scss:./dialog.scss";
        ╵                         ~~~~~~~~~~~~~~~~~~~~

  This error came from the "onResolve" callback registered here:

    node_modules/.pnpm/monkey-around@2.3.0/node_modules/monkey-around/index.js:24:23:
      24 │         return current.apply(this, args);
         ╵                        ^

    at Object.<anonymous> (file:///D:/repos/gh/obsidian-queryallthethings/node_modules/.pnpm/github.com+sytone+ophidian-lib-build@a8bd3c03867f1f7541a6f94493fbe0436dc0e179/node_modules/@ophidian/build/index.js:198:20)
    at Object.wrapper (D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\monkey-around@2.3.0\node_modules\monkey-around\index.js:24:24)
    at setup (D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\esbuild-plugin-sass@1.0.1_esbuild@0.17.6\node_modules\esbuild-plugin-sass\index.js:100:15)
    at file:///D:/repos/gh/obsidian-queryallthethings/node_modules/.pnpm/github.com+sytone+ophidian-lib-build@a8bd3c03867f1f7541a6f94493fbe0436dc0e179/node_modules/@ophidian/build/index.js:180:28
    at wrapper (D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\monkey-around@2.3.0\node_modules\monkey-around\index.js:24:24)
    at handlePlugins (D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\esbuild@0.17.6\node_modules\esbuild\lib\main.js:1279:21)
    at buildOrContextImpl (D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\esbuild@0.17.6\node_modules\esbuild\lib\main.js:968:5)
    at Object.buildOrContext (D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\esbuild@0.17.6\node_modules\esbuild\lib\main.js:776:5)
    at D:\repos\gh\obsidian-queryallthethings\node_modules\.pnpm\esbuild@0.17.6\node_modules\esbuild\lib\main.js:2172:68

If I manually update the dialog.ts file to use `import "./dialog.scss"; the compile works without error.

Are you using the latest ophidian/build?  It adds support for those import prefixes.
Future versions of ophidian/core should be able to drop this requirement, at least once I figure out how to get pkgroll to do it.  The issue is that newer versions of esbuild have a problem with tree-shaking CSS support, such that it includes all the project CSS even if you never use any.  I switched to a different way of managing Ophidian's CSS that's compatible with newer esbuild, but the cost is that your build process now has to support those file types.
If you can't use or upgrade ophidian/build, here's the code I'm using that should be adaptable to other esbuild-based build processes.

I am now, I had a forked version as there were path issues, I moved the build script to root and the version you have is working fine. The changes I made were for the copy of the files to the dist directory. I updated the paths to be ./dist/... adding in the prefix of ./
It is all working as expected thanks.

Closing up as the issue is now resolved. Thanks!

Closing up as the issue is now resolved. Thanks!

I'm not quite following there -- do you mean that there's a problem in ophidian/build's paths?  And if so, in what part exactly?  I'm looking at your fork but not seeing any commits in it.


How to re-use acquired connection?

Is there a way to reuse the connection after retrieving a record? For example, in a scenario where you retrieve the record, perform some checks, and then make an update.

Not sure what you're trying to do, but once you have a connection from a pool, you can re-use it for multiple queries.
What you can't do is execute a query while you have an open result/row. To do that, you need another query from the pool. As far as I know, this is simply not possible with PostgreSQL, but if you know of a driver that allows it, let me know and I'll try to see what they're doing.

Got it! If I deinit the row or result, I can reuse the connection.


Upgrade lienzo core to 2.0.277 and lienzo tests to 1.0.0-RC3.

Hi @manstis , this is the lienzo upgrade we've commented on today's call.. once get merged I will be able to create the PR for integrating stunner into kie-wb-common. Thanks!

Added lienzo nativetools & commons as well.

We should also create PR(s) for IP BOM...

ok - this was the only question I had - don't have these dependencies to be added to IP_BOM? but @psiroky answered this. I will do the PR for IP-BOM

@mbiarnes @romartin Thanks Michael for looking after ip-bom.

@romartin You'll also need to update https://github.com/uberfire/uberfire/blob/master/pom.xml

@manstis @mbiarnes @psiroky Thanks guys for moving this forward!!

@manstis Thanks for the feedback! Created this for UF -> https://github.com/uberfire/uberfire/pull/523


Ability for the client to receive notifications

I'm working with the Chrome Remote Debugging protocol, which allows for notifications to be sent from the server to the client. How can I go about receiving these event notifications?

I'm sorry, I must have missed this report. GitHub hasn't been emailing me for years. I hope you managed to solve this issue. I'll close because it is very old and you probably managed to resolve it already.


Master and Details Table with service implementation

Hi,
Can i implement a master-details table using a service implementaion , if yes can i get a little hint?

Yes it's possible, the trick would be to add a column for the details like the snippet below from my actual project.
protected function getColumns() : array
    {
        return [
            [
                "className"  => 'details-control text-success',
                "orderable"  => false,
                "searchable" => false,
                "data"       => 'details',
                "title"      => "",
            ],
        .....

On ajax, append the details icon:
public function ajax()
    {
        return $this->datatables
            ->queryBuilder($this->query())
            ->addColumn('details', "")
...

thanks @yajra  i will try it :)

Hi @yajra can you provide a more detailed example, can't get it work as a service..
Tnks

did any one get working example with service implementation ?

@yajra Help please.
Master-Details and Row-Details in service implementation are confusing.
The main question of Row-Detail in service implementation is:
How to pass data from DataTables class into detailed row(Handlebars template) ?

Anyone got a full example please?

@elpoleto Send an E-Mail to me. I think can help you. (h.kamrava1 [AT] gmail.com)

Here's a reasonably clean pattern for anyone still stuck:-
In your Datatable class:-
class OrdersDatatables extends DataTable {

protected function getColumns()
    {
        return  [
            Column::make('details')
                  ->title("")
                  ->className('details-control text-info')
                  ->searchable(false)
                  ->orderable(false),
            Column::make('id')->title('#'),
            Column::make('name')
        ];
....
}

Setup the return values and in the detail column add a data-url property to the icon setting a link to load the relevant details table by passing in the model->id of this row.
public function dataTable($query)
    {
        return datatables()
            ->eloquent($query)
            ->editColumn('details', function ($model) {
                return "id)."'  class='fa fa-chevron-circle-right'>";
            })
            ->editColumn('id', function ($model) {
                return \Html::link(route('order.show', $model->id), "#".$model->id);
            });
//add your other columns
}

In the html builder include a function on the initComplete to be run once when the table has loaded.   This is
the juicy stuff, the initComplete function includes all the DT settings so we can get the tableId to refer to it.
Clicking on the expand icon ajax loads another datatable view.  I used a parameter (loadview) to tell the controller we're requesting the table HTML not the data.
 public function html()
    {
        // I use a common helper to provide a consistent layout and save repeating code
        $builder = Helper::datatablesBuilderDefaults($this->builder(), 'Orders with Notes');

       // Override the defaults and add some custom JS
      // I was today years old when I discovered formatting of JS in a php string with @lang JavaScript - revelation :-)
        $builder->setTableId( "dt_".Str::snake(class_basename($this)))
                ->columns($this->getColumns())->orderBy(1, 'desc')
//Prevent the default responsive show more button from messing us up
                ->responsive(['details' => false])
                ->parameters(array(
                    'initComplete' =>

                    /** @lang JavaScript */ "
                        function(settings, json){
                            jQuery('#'+settings.sTableId+' tbody').on('click', 'tr td.details-control', function () {
                                let el = jQuery(this).find('i');
                                let url = el.data('url');
                                var tr = jQuery(this).closest('tr');
//get the datatable js row object
                                var row = LaravelDataTables[settings.sTableId].row(tr);
                                if (row.child.isShown()) {
                                    row.child.hide();
                                    el.addClass('fa-chevron-circle-right text-info').removeClass('fa-chevron-circle-down text-danger');
                                } else {
                                    el.removeClass('fa-chevron-circle-right text-info').addClass('fa-chevron-circle-down text-danger')
                                    row.child(loadView(url)).show();
                                }
                            });

                            function loadView(url) {
                                var div = jQuery('
').addClass('loading');
                                jQuery.ajax({
                                    url: url,
                                    data: {loadview: true},
                                    success: function (data) {
                                        div.html(data).removeClass('loading');
                                    }
                                });
                                return div;
                            }}"
                ));

        return $builder;
    }

Then the subtable controller action:-
    public function detailsTable(Request $request, Order $order, OrderNotesDetailsDatatables $dataTable)
    {
        //if loadview is set this is the first request to get the table html
        if($request->ajax() && $request->loadview) {
            return $dataTable->with(['order' => $order])->render('reporting.client_status_report_detail');
        }
        //otherwise it's the next call requesting the data
        return $dataTable->with('order', $order->id)->ajax();

    }

Note in this case the $order is already bound via the Route definition:-
Route::get('order/{order}/notes','OrderNoteController@detailsTable')->name('order.notes');

The detail view is simply 2 lines.
{{ $dataTable->table(['class'=>'table table-bordered table-vcenter table-sm'],false) }}

{{$dataTable->scripts()}}

Just for completeness the primary table view is:-
@extends('layouts.app')

@section('metaTitle', 'Client Orders')

@section('content')

        {{ $dataTable->table(['class'=>'table table-bordered table-striped table-vcenter'],false) }}

@endsection

@push('scripts')
    {{$dataTable->scripts()}}
@endpush

And controller action:-
    public function index(OrdersDatatables $dataTable)
    {
        return $dataTable->render('reporting.orders');
    }

It's a few things to hold in your head at once but Yajra's module makes it pretty easy. Still so happy to have found this.  My current project has 47 datatables which were originally created by hand with separate JS for each table  URGH:-( Now it's easy.  Thanks @yajra you've saved  me hours.


WiP Compare failing test with CI

intends to fix #357

Hi Simon, this PR is really old, should we finish or close it?

This need some JTravis work to be finish first: https://github.com/Spirals-Team/jtravis/issues/6
close it for now.


/uploaded/preview media broken

Non-lecture recording material on mediastore breaks Canvas Video Enhancer.
Example: https://mediastore.auckland.ac.nz/uploaded/project/CMS_ARTS/07-2017/BA052DE88CDE42B3BE7A8AF3EC495079.preview

Seems to be due to some expected formats (.m4v, -slides.m4v, .mp3) not being present. Should probably change it to not skip the first 42 seconds though since there's no copyright notice in these videos.


Clean up of #12

Cleaned up some of the comments from @bcherny on @benjie's #12

[x] There's some issues still with the types that I'm hoping to solve by adding the @types/babel__traverse package.
[ ] There's also one test error: TypeError: Property body of DeclareClass expected node to be of a type ["ObjectTypeAnnotation"] but instead got "TSTypeLiteral"

Any ideas?

My guess is DeclareClass is a Flow type (which expects only Flow types as children), while TSTypeLiteral is a TS type. This might be a limitation of the walker approach in this PR, as opposed to a recursive descent approach to transforming the tree from Flow -> TS.

I had to work around the types quite a lot to make it work correctly; hence the ts-ignores/etc everywhere. I think some of it was down to mismatched versions, was planning to upgrade all the dependencies and try again but still have yet to find time.


Don't use brackets in a return statement?

Does this mean "Don't wrap return statements in braces"?
Would this be acceptable?
   private static int calculate(int x, int y, int z) {
        return (x + y) * z;
    }

Ignore the fact that x, y and z are not acceptable variable names ;)

Not sure if "brackets" is the correct terminology either? https://www.cis.upenn.edu/~matuszek/General/JavaSyntax/parentheses.html

return (x + y) * z; is technically fine.
Though since we have a preference for named arguments this would possibly be better.
private static int calculate(int x, int y, int z) {
    final int w = (x + y) * z;
    return w;
}

Back to the original point though. "Don't wrap statements in braces" - don't do this:
return (x + y);
I'll give an example and change "braces" to "parentheses"


Refactor delete handling to remove the workspace

Signed-off-by: Bob Haddleton<EMAIL_ADDRESS>Description of your changes
Refactor the delete handling code to use "terraform workspace delete" to remove the workspace cleanly.
Fixes #55
I have:

[X] Read and followed Crossplane's contribution process.
[X] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
The code ran in my local environment for a week with no issues.

@ytsarev - can you please take a look when you get a chance?  Thanks!


Allow regexp-based routing and param capture

A handy feature of Martini is the ability to use regexps and named capture groups to route requests (https://github.com/go-martini/martini#routing).
This can be helpful for constraining the format of parameters, or for parsing more complex routes than simple globs allow.
Looks like you're using route-recognizer for globbing, so I guess this would require re-writing the path matcher.

The current approach is much faster and has a simpler implementation (from Iron's perspective), so I doubt this change will be made. If you have any killer reasons or examples of why regex based routing is needed over our current glob and param matching, I might be convinced, but otherwise closing as wontfix.


Backend testing

Describe your changes
Creates tests for following backend endpoints

HobbyType
Hobby
Onboarding
Availability
ZipCode
SuggestionResults
SubmitOnboarding

Checklist before merging

[X] My code produces no errors when run
[X] My code is linted and properly formatted

Something is weird here... why are there 209 files changed 😆 . It should only just be the tests you made.

I ran the precommit and it ended up changing literally everything

Oh makes sense. Well all tests pass so I'll approve, hopefully merging the precommit changes wont do too much damage


Warning & Separation Circles misleading

The white warning circles and red separation circles are misleading... They have a radius of 2.5nm, and the separation required is 3.0nm, resulting in a/c generating sep losses when they are still outside of the circle.
I recommend that we:

Change the radius from 2.5nm to 3nm
dim the "warning" from the bright white to barely visible white

Example:

@erikquinn
I've noticed this too, I just assumed it was a very string warning ;)
On a similar topic, would it be possible to have a faint circle surround any craft whenever it is selected? This would be a big help when I merge two lines of multiple craft for approach by providing further information to set their heading and speed to space them correctly.

@sixeyeco that's possible, sure, I personally don't think I would like that though. I find that the 5nm range rings provide more than enough information to get a sense of where you're at separation-wise.

Yes, but they only appear when vertical sep is fewer than 1k ft. I like to dovetail a flight at 3k in between two others that are at 2k. If the ring appeared when I select the 3k flight I could easily tell if it's coming in correctly. Just my opinion.. AISB, I'm neither a controller nor a pilot.


Unable or lack of feedback on the Sign-In link for "Use Update Management v2 that does not require agent"

[Enter feedback here]
Customer want to join the private preview, - but link does not provide access, - or maybe the engine behind the does not trigger?
Kr, Torben
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bd3dca2f-629d-36e9-f0f0-8c7e64cc04ab
Version Independent ID: f40ee991-6cb8-1eee-9a35-ebc5833d7c7f
Content: Azure Monitor agent overview - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-overview.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@sorensent Thanks for your feedback! We will investigate and update as appropriate.

@sorensent The sign-up link redirects to Yammer Group. Since this is in currently in Private Preview, Yammer group will only be accessible to MVPs who are under NDA.

@sorensent Did you get chance to look into my previous comment ? Let me know if you have any questions.

@sorensent  As we haven't heard from you, we will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


add redhat8, centos8  and debian11 support

Strange. No tests on centos 8 ..

Ah, I've see the discussions.
This is a Ubuntu host / el8 container issue then?
Because it runs fine on a rockylinux8 host with docker.


[Port Manager] Use IP/Mac replacement APIs for port update

Request
After IP Manager (Issue https://github.com/futurewei-cloud/alcor/issues/218) and Mac Manager (Issue https://github.com/futurewei-cloud/alcor/issues/217) provide replacement APIs, we would like to use the new APIs for port update to allow "atomic" operation.

@chenpiaoping For you tracking purpose.

Finished

Finished


RealmSwift compiled with older Swift language (2.0) than previous files (3.0)

When creating a new Xcode project and setting up Realm, I built the default project to make sure the settings were correct, and I get the Mach-O Linker Error "Linker command failed with exit code 1 (use -v to see invocation)"
Here is the raw output from the whole error:

Ld /Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Products/Debug-iphonesimulator/Record\ Store.app/Record\ Store normal x86_64
cd "/Users/jonathanstowe/Xcode Projects/Record Store"
export IPHONEOS_DEPLOYMENT_TARGET=9.3
export PATH="/Applications/Xcode-beta.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode-beta.app/Contents/Developer/usr/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -arch x86_64 -isysroot /Applications/Xcode-beta.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator10.0.sdk -L/Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Products/Debug-iphonesimulator -F/Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Products/Debug-iphonesimulator -F/Users/jonathanstowe/Xcode\ Projects/Record\ Store -filelist /Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Intermediates/Record\ Store.build/Debug-iphonesimulator/Record\ Store.build/Objects-normal/x86_64/Record\ Store.LinkFileList -Xlinker -rpath -Xlinker @executable_path/Frameworks -mios-simulator-version-min=9.3 -Xlinker -object_path_lto -Xlinker /Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Intermediates/Record\ Store.build/Debug-iphonesimulator/Record\ Store.build/Objects-normal/x86_64/Record\ Store_lto.o -Xlinker -export_dynamic -Xlinker -no_deduplicate -Xlinker -objc_abi_version -Xlinker 2 -fobjc-link-runtime -L/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphonesimulator -Xlinker -add_ast_path -Xlinker /Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Intermediates/Record\ Store.build/Debug-iphonesimulator/Record\ Store.build/Objects-normal/x86_64/Record_Store.swiftmodule -framework RealmSwift -framework Realm -Xlinker -dependency_info -Xlinker /Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Intermediates/Record\ Store.build/Debug-iphonesimulator/Record\ Store.build/Objects-normal/x86_64/Record\ Store_dependency_info.dat -o /Users/jonathanstowe/Library/Developer/Xcode/DerivedData/Record_Store-dxhwlltjuofyzfccpubkdqlpgngi/Build/Products/Debug-iphonesimulator/Record\ Store.app/Record\ Store

ld: /Users/jonathanstowe/Xcode Projects/Record Store/RealmSwift.framework/RealmSwift compiled with older version of Swift language (2.0) than previous files (3.0) for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

This is just the error I'm stuck at, and saw that there wasn't an issue written up about it yet. It seems to only stick with the error because of the Swift 3.0 compiler.
Realm version: 1.0.2
Xcode version: 8.0 beta 6 (8S201h)
iOS/OSX version: iOS 10

Prebuilt binaries of Swift frameworks can only be consumed by the same version of Xcode that compiled them. Currently our releases are built with Xcode 7.3. Since you're using a prerelease version of Xcode, you'll need to build Realm from source from our master branch. Once Xcode 8 officials ships we'll release a version of Realm built with it.

Ah perfect, the binary is building now!


Minified v3.1.1 missing code

The beginning part of the code in the minified version of 3.1.1 is missing (from CDN and dist in download).

Hey, sorry about that.
The correct version has been put in the 3.1.2 release

Still looks like the same issue is there...


Decahedron Calculator

Add  Decahedron Calculator (3d shape with 10 faces)

i would like to work on this issue


TFLite?

Sorry if this question is pretty noob, but I was wondering if the facial deep fake with a driving video and picture can be compressed into a tflite file or not?

This is repository is in pytorch. Conversion to tensorflow will be problematic.


Feature : Login/Signup by Github Authentication.

Is your feature request related to a problem? Please describe
Currently, users may find it inconvenient to create a new account or remember their passwords for different platforms. Offering GitHub authentication will allow users to quickly sign in or sign up with their GitHub account, simplifying the login process and enhancing user experience. This feature will also reduce friction for developers who frequently use GitHub, encouraging them to engage with Circuitverse more easily.
Describe the solution you'd like
CircuitVerse should integrate GitHub OAuth with Vue.js to allow users to log in or sign up using their GitHub credentials. Here’s how it would work:
Users click a "Login with GitHub" or "Sign up with GitHub" button.
They are redirected to GitHub’s authorization page, where they grant permission for the app to access their profile.
After successful authentication, the app retrieves basic user information (e.g., email, GitHub ID) and either logs the user in or creates a new account if they are signing up for the first time.
The user is then redirected back to the application and logged in automatically.
This feature will be implemented using the OAuth flow in Vue.js. The process should be secure, responsive, and seamless, with proper error handling in place
Describe alternatives you've considered
Manual Signup/Login with Email and Password:
While this is a traditional option, it introduces friction and isn't as convenient as social login, particularly for the developer community.
Other Social Logins (Google, Facebook):
Although Google authentication is another viable option, GitHub is more appropriate for CircuitVerse’s target audience, which includes developers and engineers.
API Key Authentication:
This could be an alternative for some user types, but it doesn't address the ease-of-use and quick access GitHub OAuth provides.
Additional context
Technology Stack: Since CircuitVerse uses Vue.js, this feature will involve integrating Vue with GitHub’s OAuth API, managing the authentication process using Vue components and backend support to securely handle access tokens.
Are you working on this?
Yes

@subhamkumarr @tanmoysrt
Please review this. I encounter redirection to signup page while login/signing-up via github.

Have you tried using Circuitverse? We have big login with GitHub button in out sign in page !

https://github.com/user-attachments/assets/de33dafe-cc88-4ebc-a959-33123a331cba
so is that a bug?


Maunope/network added firewall metrics

I think the PR is good now, thanks a lot for your contribution Maurizio!
LGTM


Test cases for serialization

This pull request derives from #371. So needs to be handled after that.

The repository has virtually no test for serialization performed by mode="test:serialize" in src/reporter/format-utils.xsl.
This pull request expands test/end-to-end/cases/xspec-356.xspec and adds some serialization tests.

@cirulls
Merged this set of tests.
Serialization (i.e. how results are rendered in HTML report) is now tested.


Not all Cargo airport stand is used

When using vanilla , say if i have 3 Cargo airport stand, all placed as row  then cargo planes seems to coming to all 3. but with our Mod, it seems always goes to first Cargo stand. I am not sure because of this path finding. Is it possible to switch off using path finding for airport ?

The thing is the aircraft must want to go to different stand. That decision is done outside of TMPE. I'll take a look later.

Again last night , i have ran with vanilla. It goes to all airport stand and noticed, some airplanes are circling around if no taxi way.
Morning i have switched on, cleared traffic. all goes to 1 stand + noticed, airplanes are not circling instead it just lands and vehicles overlapes to each other, means 2 or 3 planes in 1 location standing,
I badly need this mod for main junction , lane connector, vehicle trestriction , dynamic lane , speed limit and timed traffic light. Please help to fix for air craft or ignore it to vanilla for now is highly appreciated.
Thanks a lot brother.

TM:PE is only searching for path for the aircrafts for now, because TM:PE is replacing whole pathfinding. Pathfinding for aircrafts is 1:1 as vanilla, no tweaks or any changes. Other behaviour like stands assignment, taxiing, taking off, landings are fully handled by vanilla code. Because of that none of TM:PE tools work for taxiways - we don't support such vehicles at all and I don't know yet why they do what they do with TMPE enabled 🤷♂
Adding support will be quite hard as all code is designed for road/rail vehicles only and Aircrafts when moving can do both (move on the ground and also fly, obviously and transition between states (take off/landing)) so a lot of additional code for state detection (on ground/flying) will be required.
It will be huge task.

Thanks bro. Is there a way to enable full log?

Looks like problem with CitiesHarmony used. I just replaced CitiesHarmony new version ( yeah may be it is under beta not sure) and recompiled. Now Cargo stand is going like expected + all my garbage collector is collecting too. I dont what is the purpose of CitiresHarmony dll. Anyway let me try to check.

Which version of harmony were you using previously?

Currently used in nuget and this mod uses below

I have used the pre-release vesion as below

I'm assuming you're on Linux?

Windows 10 brother

You can use stable version of Harmony, I also tested experimental on 2 days ago, both work without any issues. TMPE is shipped with CitiesHarmony,Api.dll which in game communicates and requests the main Cities.Harmony.dll, either this (stable) or that (experimental) depending on which is installed.
Experimental is currently at final phase so will be released as Stable soon :tm: AFAIK
Without Cities Harmony mod TM:PE won't work at all and it's required since 11.6.x+. Up to v11.5.2 previous version was included(embedded) but now migrated to external version to be easier to keep updated. Also you can't have multiple versions of Harmony installed, just a side note.

Agreed bro,let me wait for stable release then and check it again. Currently TM PE mod screws up cargo whereas vanilla pathfinding works fine.

I can't reproduce that problem... What I'm doing wrong? (city is super tiny, <1k)🤔


Added Swedish translation

Hi @YouKnowBlom, with the strings nearly finalized, would you mind updating the Swedish translation to match the English?

Can do. I have very varying amounts of time as I'm closing in on a move but I'll get around to it. If not, please remind me in ED :p

@YouKnowBlom Would you mind updating the translations through POEditor by joining here https://poeditor.com/join/project?hash=nRljcnV0ET this is the recommended way now. You will also be able to check periodically in case there are new strings added as BD adds new features.


OSL shaders are not supported by interactive rendering mode.

Currently I have tried a number of different syntax options to point PRMan to the shaders. So far the only one that works with the engine is ./shaders:@ any other including the full path will give a shader not found error. Interactive rendering will not accept any version of this. Let me know if you need any more information.

Does my latest changes for this work for you?  You were adding spaces to the search path which is a no no.

Thanks for the information about the search path. So far, your and my attempts at getting interactive rendering to accept osl has resulted in the same error message. (Plugin Error: Could not open shader "shader name" [File "file name" could not be found.])

Can you paste the search path in your scene.rib file?
On Friday, July 31, 2015, MWY3510<EMAIL_ADDRESS>wrote:

Thanks for the information about the search path. So far, your and my
attempts at getting interactive rendering to accept osl has resulted in the
same error message. (Plugin Error: Could not open shader "shader name"
[File "file name" could not be found.])
—
Reply to this email directly or view it on GitHub
https://github.com/bsavery/PRMan-for-Blender/issues/76#issuecomment-126854788
.

--
<EMAIL_ADDRESS>508-274-8700

Sure,
This works ["./shaders:@://C/Program Files/Pixar/RenderManProServer-20.0/lib/RIS/pattern://C/Program Files/Pixar/RenderManProServer-20.0/lib/RIS/bxdf://C/Program Files/Pixar/RenderManProServer-20.0/lib/rsl/shaders://C/Program Files/Blender Foundation/Blender-2.75a/2.75/scripts/addons/PRMan-for-Blender/shaders"]
This does not work ["//C/Users/____/My Documents/prman_for_blender/Advanced3//shaders:@://C/Program Files/Pixar/RenderManProServer-20.0/lib/RIS/pattern://C/Program Files/Pixar/RenderManProServer-20.0/lib/RIS/bxdf://C/Program Files/Pixar/RenderManProServer-20.0/lib/rsl/shaders://C/Program Files/Blender Foundation/Blender-2.75a/2.75/scripts/addons/PRMan-for-Blender/shaders"]
Underscore added in place of my username.
Removing or moving the colons or @ 's don't change anything.
Now that I think about it do you think the colon after the drive letter might make the difference?
Anyway I'm going to bed so have a great evening.


[k8s] Ingress and deployment for Daphne

[k8s] Adding daphne properties
[k8s] Add config hash to deployments
[k8s] Ingress and deployment for Daphne

@gmsantos FYI


Storing Address programmatically

HI, I am loading address into db programmatically as shown below but the map doesnt render when i try to access the address using the template tags(the computed address and lat long fields are loaded by itself). A strange thing i noticed is if i open the instance from django admin and save it again then i can view the map! Not sure if am doing something wrong on my end.
File   :
Airoli,Mumbai,India
Airoli East,Mumbai,India
Airoli West,Mumbai,India
Ghansoli,Mumbai,India
Rabale,Mumbai,India
for line in file.readlines():
address = Address(address=line)
address.save()

Bitbucket: https://bitbucket.org/kmike/django-easy-maps/issue/24
Originally reported by: Siddharth Shah
Originally created at: 2014-08-14T20:23:12.246

Strange, but works for me. Please try latest version


[Feature] Add total relic count / filtered relic count to the relics grid

Motivation
Currently its not possible to see how many relics are in my inventory, and how many have passed the filters
Goal
Use the aggrid status bar or some other component https://www.ag-grid.com/javascript-data-grid/status-bar/ to display it

Released in https://github.com/fribbels/hsr-optimizer/pull/442


Migrate into Postgres Database Schema other than 'public'

Environment
Knex version: 0.15.2
Database + version: postgres 9.5.3
OS: mac
Questions about how to use knex
Is there a way to run a migration and have it execute inside a postgres SCHEMA other than 'public'? Note I am running the migration programatically, not through the CLI.
I've tried this (https://github.com/tgriesser/knex/issues/1138) by setting tableName config option to be '.db_migrations' (substituting the schema name in). Did not work.
I've searched the web, stack overflow, and issues. Can't seem to find an answer. Please advise.

@kibertoad, absolutely.
Ultimately we have a multi-tenant system. Each tenant lives in a postgres 'schema' other than the default 'public' schema. So if I have a database with tenant1 and tenant2 in it, then that database would contain 3 postgres schemas:
'public' - nothing stored here
'tenant1' - tenant 1 has tables
'tenant2' - tenant 2 has tables
I want to define a general migration for any tenant.
Then I want to be able to run the migration on each of the tenant's schemas, which would update the tables (the same tables in each schema). Again, I am not using the CLI, but doing this programatically by instantiating a Migrator.
So to be clear re your suggestions:

to use withSchema, I need to know the schema name ahead of time. Is there a way to pass variables in dynamically to the migration scripts to be consumed when the script is run?
prefix table names with schema, would require me to know the schema name ahead of time, which would not be possible, again, unless they are passed into the migration as a variable.

The approach I found working is:

steps:

set searchPath in KnexConfig on database connection.
then when running migration, migration is done to schema specified in the search path.

I don't love this approach because I need to create a connection per migration, but it is workable if no better solution exists

Is there a way to pass variables in dynamically to the migration scripts to be consumed when the script is run?

Elegant solution would be https://github.com/tgriesser/knex/pull/2778. Since it is not currently merged and it's unclear at this point whether it would ever be, you could take advantage of the fact that migrations are executed sequentially and set schema as a "global" variable somewhere that migration script would read from.
Alternatively, since you have access to knex instance in your migrations, probably you could set some variables on it directly and then access them from migration script.

#2778 is definitely elegant. When would a decision be made one way or the other? Why would it not be merged? I think it adds even more power to an already fantastic tool!
I don't want to hack the knex object. Its always best to have a supported methodology that'll be maintained and will work in future releases. So I'd lean away from that and use #2778 or the solution I discussed above.

I pinged one of the maintainers on this topic already, we'll see. I agree that #2778 makes sense.

@mashaalmemon Solution coming in a "@next" release soon, would be great if you could test it on real-world scenarios!

@mashaalmemon next with this feature is out, please give it a try and let me know how it works for you!
Here's the documentation - https://github.com/knex/documentation/pull/135


Different tokenisation results between oniguruma and RE2

Right now envy uses regex-based tokeniser (until #218 at least).
We have 2 drop-in replacement regex engines: default from Go RE2 and default from Ruby oniguruma.
Recent improvements done with Go module migration #219 surfaced a new issue: it seems that tokeniser produces a bit different results, depending on which regex engine is used :/
More specifically, the token frequencies built from linguist samples are different and high-level code-generator test catch by comparing with a fixture (pre-generated with RE2) and fail on oniguruma profiles like this https://github.com/src-d/enry/pull/219#issuecomment-482525632
We need to find the exact reason and depending on it decide, if we want to support 2 versions of fixtures or change something so there is no difference in output.
This also potentially affects #194

Often I have found the differences between engines on the same regexp boil down to small variations in the handling of greed (in particular: whether operators are greedy), zero-width assertions, anchoring (e.g., whether it is on by default), case-sensitivity, and other meta-properties of the execution model that aren't well-specified. Obviously I don't know what the causes are in this cases yet, but that's where I would start looking for cases.

Steps to reproduce
$ go test ./internal/code-generator/... -run Test_GeneratorTestSuite -testify.m TestGenerationFiles
ok  	gopkg.in/src-d/enry.v1/internal/code-generator/generator	35.537s

$ go test -tags oniguruma  ./internal/code-generator/... -run Test_GeneratorTestSuite -testify.m TestGenerationFiles
FAIL	gopkg.in/src-d/enry.v1/internal/code-generator/generator	34.583s

Both should produce the same results if results of tokenisation of all linguist samples are the same.;
Going to add extra tests to verify just that.

I localised the problem down to the difference in handling what seems to be latin1 encoded file.
While tokenising thå filling encoded in latin1 and read as UTF-8
RE2 gets
th
filling
�

while Oniguruma gets
th
illing
�
f

Of the three, only Flex really seems to be doing anything reasonable here. RE2 is close—but the order of the tokens is weird. I have no idea what Oniguruma is doing there, but it seems obviously broken in at least two ways.

Yup. And if the content is decoded from latin1 and encoded to utf8 with charmap.ISO8859_1.NewDecoder().Bytes(content) the result is an expected one:
RE2
th
filling
å

oniguruma
th
filling
å

I know that Linguist does use ICU-based https://github.com/brianmario/charlock_holmes but am not sure yet if it's part of the tokenisation.

Yeah, I think either we should normalize the encoding or find a way to treat the Unicode replacement character as part of the token, e.g., xxx�yyy ⇒ xxx�yyy, or use it as a separator and discard it, e.g. ⇒ xxx � yyy ⇒ xxx yyy. It seems like Linguist does the latter maybe.

True. And linguist with flex-based tokeniser does not have this issue, so no need to encoding detection there. Thank you for suggestions, let me think a bit more about that..

After digging deeper - ot seems that the offending tokenization rule is extractAndReplaceRegular(content).
The extractRemainders is then called on it's output and does bytes.Fields(content) on  " \xe5 "which results in extra�` in RE2 case.

Doing equivalent operation in Ruby results in ArgumentError: invalid byte sequence in UTF-8
$irb

"th\xdd filling".scan(/[0-9A-Za-z_\.@#\/\*]+/)

Digging a little bit deeper with oniguruma's C API using awesome examples/simple.c, it starts to look like this may rather be a bug in go-oniguruma Regexp.findAll() implementation.

Even in C, oniguruma does consistently produce strange result in UTF8 mode for the non-valid input bytes in UTF8, like above. Seems like a bug upstream.
As all regex for tokenization are not using any Unicode character classes, all RE2-based matches are conducted in ASCII-only mode, while go-oniguruma has hardcoded UTF8.
For our use-case the fix would be to force Oniguruma also use ASCII mode and that indeed produces identical results even for non-valid bytes in UTF8.
I will submit a patch to cgo part of https://github.com/src-d/go-oniguruma to have an option to override hardcoded UTF8,  expose it in Go as MustCompileWithEncoding (similar to MustCompileWithOption)  and as soon as it's merged, move entry regex/oniguruma.go to use it.

Fixed by #227


V1.0.0 release notes

1.0.0 November 14th 2018
HOCON 1.0.0 is a complete and total rewrite of the HOCON parsing engine, designed to work with both the .NET Framework (4.5 and later) and .NET Standard 1.3.
The performance has significantly improved, as have the range of features supported by this library, including:

Built-in enviroment variable substitution;
Support for better byte and time formats;
Improved object substitution and merging;
and many more features.

This library will be developed and released independently from Akka.NET itself, but beginning with Akka.NET v1.4 it will be taken as a dependency by the core Akka libraries.
HOCON ships as two NuGet packages:

HOCON - includes all core parser, substitution, tokenization, and other functionality;
HOCON.Configuration - adds App.config and System.Configuration integration to HOCON.

Need to update the release notes to mention the changes we just merged in https://github.com/akkadotnet/HOCON/pull/62


Hiscores level 1 combat

When a player is unranked in combat skills, have a minmum combat level of 3 to prevent it from displaying 1.
@devLotto

This bug seems to be back?


fix: repair bug_report.md to one standard

Describe the PR
bug_report.md file was fixed to one standard
Relation issue
None
Additional context
None

Codecov Report

Merging #366 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #366   +/-   ##
=======================================
  Coverage   85.17%   85.17%
=======================================
  Files           7        7
  Lines        1322     1322
=======================================
  Hits         1126     1126
  Misses        113      113
  Partials       83       83

Continue to review full report at Codecov.

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


getting 'mime must be a string' error when loading image

Hi,
I've encountered issue when loading and image. Error says 'mime must be a string'.
It's interesting because for 2 different images the same code works differently (works fine with one image and crashes when using the other. Both images are .png).
I'm using jimp for browsers.
Thanks.

Figured out. My bad. Sorry:)

can you tell me why?possibly i encountered the same issue....

i'm having similar issues as well

@Crise420WMT @andkhong that error comes from getMIMEFromBuffer function. Make sure you are passing correct image data to jimp. You may be passing an unsupported image type. I had the same issue when I was accidentally passing in an SVG file instead of PNG.


Suggestion

Describe the bug
In your repository when clicking on your coders-evoke site it is not redirecting to your site please solve it :))
To Reproduce
Steps to reproduce the behavior:
Edit the repo and change that link with that link that is working

Describe the bug
In your repository when clicking on your coders-evoke site it is not redirecting to your site please solve it :))
To Reproduce
Steps to reproduce the behavior:
Edit the repo and change that link with that link that is working

thnx @anubhav201241  for suggestion
:)

Welcome :)


⚠ Samsung has degraded performance

In cd8d71d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2349 ms

Resolved: Samsung performance has improved in 0675420.


Fix issue #64: Add request headers

Proposal to add request headers.

@aweisser This looks like it satisfies the bulk of the first instance we were discussing. What do you think?

@tofcoder Thanks for taking the time to work on this PR :)

Everything looks fine. Nice and clean solution.
Thanks for your contribution @tofcoder.


FitBounds changes y and height when called on osd with display:none parent

If am calling viewport.fitBounds(rect) when the canvas is in display:none parent there is mismatch of the rect I pass and I get after I set the canvas'  to display:block. Setting visibility:hidden does the math okey so it is display:none issue. I think it is problem with the aspectRatio and will investigate it further. This happens under all browsers. Any ideas how to fix it?

I think it is problem with the aspectRatio and will investigate it further

You are correct. With display: none the  width and height of the viewport are 0.
Your best bet is probably to call viewport.fitBounds(rect) immediately after removing display: none.

Yea I was thinking about it, however this is unacceptable solution for now. This problem will stay because of how browsers threat display:none , won't it?...

Probably yes, except if someone has a good idea to workaround that...

I think if you put autoResize: false in your options you can avoid this (assuming your viewer is visible to begin with). At least that might be an avenue worth investigating.


Update ConfigMap doc to explain TTL-based cache updates

Based on recent change: https://github.com/kubernetes/kubernetes/pull/46470
cc @wojtek-t

This change is

@kubernetes/sig-docs-maintainers Seems like I don't have the permission. Could someone assign v1.7 milestone and add a reviewer to this PR? Thanks.

/lgtm

@chenopis @wojtek-t Thanks for reviewing.


Issue with trading updates

I discovered the Topic that the client did not delivery any trade updates that are happening on the account.
For me the issue is that some lines of tradeData at null.
With the change below i could fix the issue locally.
void SubscriptionListener.onItemUpdate(ItemUpdate itemUpdate)
{
var json = JsonConvert.SerializeObject(itemUpdate.Fields, Formatting.Indented);
var tradeData = JsonConvert.DeserializeObject(json);
if (tradeData.Confirms == null)
{
tradeData.Confirms = String.Empty;
}
if (tradeData.Opu == null)
{
tradeData.Opu = String.Empty;
}
if (tradeData.Wou == null)
{
tradeData.Wou = String.Empty;
}
OnItemUpdateHandler(new TradeListenerEventArgs(tradeData));
}

the change was done in tradelistener.cs

Hi, it is not a good idea to hard-code strings, instead of that check if  they are null before processing them at the end. Do not hard-code values.
This is not an issue.


Bump ms 0.7.2

Yui compressor users will be able to minifier debug.

👍

👍

@thebigredgeek Can you release this? Thanks a lot


Revamp The Current Search Bar

Description
This will revamp the current Searching experience with interactive, responsive and user-friendly Search bar.
Hence, user experience will be improved.
Fixes #144
Type of change
Please check options that are relevant to your PR.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

Checklist:

[x] I have squashed my commits
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

@leoshrey Looks perfect. Please pull the latest changes and then I'll merge your PR.


Hexagons Resolution

Hello!
I would like to know if there is a way to use a resolution between 3 and 4 for the hexagons, because in resolution 3 they are very large, and in resolution 4 the hexagons are very small, I would like the hexagons to be larger than resolution 4 but smaller than resolution 3.
resolution 3:

resolution 4:

thanks!

Unfortunately no - H3 has 16 resolutions, and they are all fixed in size, with each subsequent resolution using cells that are approximately 1/7th the size of the parent resolution. Because of the fixed relationship between parent cells and child cells, there's no way to support custom or partial resolutions.


Json schema nested array properties incorectly mapped.

Description
Json schema nested array properties incorectly mapped in array on second level (don't know what happens deeper).
At first level in root array we can see that hidden property is correctly mapped and age is not displayed for two of three elements but on the second array with hidden property specified exacly the same like in root array age is displayed.
You can observe that mapFn param mapSource passed to map for every field is passed (as it seams) as first element from second level array properties.
Minimal Reproduction
https://stackblitz.com/edit/angular-ngx-formly-array-type-with-different-item-properties

This issue has been fixed and released as part of v5.5.5.
Please let us know, in case you are still encountering a similar issue/problem.
Thank you!


Customize Template is empty

Hello,
I just update our Oparin instance to Petrosian and when i want to deploy a new vm using template i cant choose any scripts in "Customize template" section. It's only displayed " <None>". Yet the scritps were there before upgrading.
Before with Nadjorf/Oparin :

After with Petrosian :

Any information to help me?
Regards,

Wow that's a coincidence, I was looking into something different and I just saw this exact issue today.  Thank you for the bug report I'll dig into this and hope to post back with what the issue is shortly.

Have a fix here: https://github.com/ManageIQ/manageiq-ui-classic/compare/master...GilbertCherrie:fix_provision_table?expand=1

Hello guys,
Thanks for the quick responses and fix. I'm not sure what i'm supposed to do. I have edited the only "request_info_helper.rb" file i have found here :
/opt/manageiq/manageiq-gemset/bundler/gems/manageiq-ui-classic-5e063889dad7/app/helpers/request_info_helper.rb
I put the fix manually and restart evmserverd service but the issue still appear im  my UI.
I missed something?
Regards,

@Jonathan-Caruana no you aren't missing anything, turns out the was a similar bug with RHV/Ovirt Customization Templates.  We were looking at the vSphere provision page but I see an issue with customization_templates as well.

I can look into this

https://github.com/ManageIQ/manageiq-ui-classic/pull/8939

Hi @agrare
Everything works perfectly now thank you.
I also closed this issue : https://github.com/ManageIQ/manageiq-providers-ovirt/issues/651
Petrosian upgrade fixed this one too.
Have a good day.

Thanks for testing @Jonathan-Caruana !


policy issue

Hello,
I have an issue with policies when triggered from any of my modules. I believe all is declared properly, however access is never granted.
Looking into it a bit more to see where it fails, I noticed the Raw function from  vendor\laravel\framework\src\Illuminate\Auth\Access\Gate.php fails all the time when triggered from a module.
public function raw($ability, $arguments = [])
    {
        $arguments = Arr::wrap($arguments);

        $user = $this->resolveUser();

$this->resolveUser() always returns NULL instead of the user model. It think the authorisation does not even take place since the user is NULL.
When I use policies from inside the main laravel app folder, I can see the user model is returned properly from the Raw function. But never from the modules folder. What am I missing?
I am using an "admin" guard.
use Nwidart\Modules\Routing\Controller;
use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;

class TemplateController extends Controller
{
    use AuthorizesRequests, DispatchesJobs, ValidatesRequests;

    public function index(Request $request)
    {
        //checks policy
        $this->authorize('list', \Modules\Template\Entities\Template::class);

I used
Auth::shouldUse('admin');

and now it is working.


Oracle - Packages body not loaded

When you open an Oracle package from the DB Navigator panel by double-clicking it, the declaration part is correctly loaded, whereas the body part is not loaded.
See screnshot below for further details.

Is there anything in error log?

Hi Serge,
the error log is empty, but I observed that one of the metadata queries that are called to retrieve data gives 0 rows as result:

I don't know if this information can be helpful...

Thanks for info. At least I'll fix source loading text.
Could you please check that this package body is loaded in other Oracle tools?

Hi Serge,
I confirm that the package body is correctly loaded and parsed by TOAD using the same user.
TOAD retrieves also a slightly different package declaration in header.
TOAD:
CREATE OR REPLACE package SER_MIFID.PKG_CALCOLO_PATRIMONIALE as
    TYPE elencoCLI is varray(8) of char(11);

DBeaver:
package           PKG_CALCOLO_PATRIMONIALE as
    TYPE elencoCLI is varray(8) of char(11);

Yet another strange issue.
Are you working as SYSDBA or as a normal user? Could try it as SYSDBA (if you have such permissions)?
There is a difference in package declarations because TOAD makes additional source processing and inserts CREATE OR REPLACE operator. I think DBeaver also should do it..

I think that this problem has the same nature as  #37. Something about permissions.
But if you are using the same user then the only possible difference is a different connection role (SYSDBA, SYSOPER, Normal). Or maybe some additional initialization queries.
Did you try any other tools besides DBeaver and TOAD?

Hi Serge,
using TOAD I connect to the same DB using the same user and the same role (normal).
I tried using SqlDbx too, and I can see the package body without issues connecting as normal user.

I'm frustrated a bit.
Could you try this trick:
Execute query
SELECT TEXT FROM SYS.ALL_SOURCE WHERE TYPE='PACKAGE BODY' AND OWNER='SER_MIFID' AND NAME='PKG_CALCOLO_PATRIMONIALE'

in DBeaver and in TOAD. Is there any difference?

Hi Serge,
executing the query on DBeaver and TOAD gives no result on both of them :-(
Hoping to be more helpful, if I inpect the current user privileges using the following query:
select * from session_privs

I obtain the following privileges:

CREATE SESSION
ALTER SESSION
UNLIMITED TABLESPACE
CREATE TABLE
SELECT ANY TABLE
INSERT ANY TABLE
UPDATE ANY TABLE
DELETE ANY TABLE
CREATE CLUSTER
CREATE SEQUENCE
ALTER ANY SEQUENCE
SELECT ANY SEQUENCE
CREATE PROCEDURE
ALTER ANY PROCEDURE
EXECUTE ANY PROCEDURE
CREATE TRIGGER
CREATE TYPE
CREATE OPERATOR
CREATE INDEXTYPE
SELECT ANY DICTIONARY
ADVISOR

Hi Serge,
after some googlin I found the solution of the issue :-)
The correct schema to retrieve the package body if the current user has EXECUTE PACKAGE permissions and the package belongs to another schema is SYS.DBA_SOURCE and not SYS.ALL_SOURCE.
If you execute this query on DBeaver and on TOAD:
SELECT TEXT FROM SYS.DBA_SOURCE
WHERE TYPE='PACKAGE BODY'
	AND OWNER='SER_MIFID'
	AND NAME='PKG_CALCOLO_PATRIMONIALE'
ORDER BY line

I get the package body.
References: TOAD Wiki, Oracle Community

Hey,
Thanks a lot for information.
However I doubt that using DBA* views everywhere is the best choice. These views are usually not accessible to users without DBA permissions.
But I think I could try to read info from both DBA and ALL views. If DBA* view is missing/not accessible or contains no data then try to read from ALL* (or even USER*).
Will try to fix this in the next version.

Now DBeaver will read sources from DBA_SOURCE view (if it is available for current user).


ancient warfare tree growing

I have yellow 1 autumn sapling in spot one, 4 orange autumn sapling in spot two, and 52spruce wood saplings in spot three.. ancient warfare will only plant the first item listed. it will ignore the 2nd and 3rd tree saplings i Have

Will take a look and report it, may have to do with modded trees maybe?

Can we confirm if the latest update fixed this issue?


Where is whatinput

Where is whatinput.js? I see it gets compiled to all.js, but in order to recompile, do we need to include it ourselves?

Nevermind, I see it's a node module.


Backport "HBASE-26663 Upgrade Maven Enforcer Plugin" to branch-2.5

The upgrade is to get the fix in MENFORCER-336, making beanshell evaluation safe for use with mvn -T. Also upgrade extra-enforcer-rules to 1.5.1, as per experience with HBASE-26664.
Signed-off-by: Duo Zhang<EMAIL_ADDRESS>Signed-off-by: Sean Busbey<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 40s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
3m 19s
branch-2.5 passed

+1 :green_heart:
compile
9m 30s
branch-2.5 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  1s
the patch passed

+1 :green_heart:
compile
9m 21s
the patch passed

+1 :green_heart:
javac
9m 21s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
12m 28s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

44m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4132

Optional Tests
dupname asflicense javac hadoopcheck xml compile

uname
Linux b730ecc24042 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / d13cc385bf

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
87 (vs. ulimit of 12500)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
9m 56s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
4m  6s
branch-2.5 passed

+1 :green_heart:
compile
2m 54s
branch-2.5 passed

+1 :green_heart:
shadedjars
4m 59s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 35s
branch-2.5 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  2s
the patch passed

+1 :green_heart:
compile
2m 49s
the patch passed

+1 :green_heart:
javac
2m 49s
the patch passed

+1 :green_heart:
shadedjars
4m  6s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
421m 23s
root in the patch passed.

463m 45s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4132

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2acb329421cf 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / d13cc385bf

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/testReport/

Max. process+thread count
2456 (vs. ulimit of 12500)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 38s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
3m 30s
branch-2.5 passed

+1 :green_heart:
compile
1m 52s
branch-2.5 passed

+1 :green_heart:
shadedjars
3m 41s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 31s
branch-2.5 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 14s
the patch passed

+1 :green_heart:
compile
1m 32s
the patch passed

+1 :green_heart:
javac
1m 32s
the patch passed

+1 :green_heart:
shadedjars
3m  6s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 36s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
489m 12s
root in the patch passed.

510m 54s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4132

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux ff8be026cf93 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / d13cc385bf

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/testReport/

Max. process+thread count
2558 (vs. ulimit of 12500)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4132/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Vertical Stepper's StepConnector lines are misplaces in rtl direction

Problem description
Vertical Stepper's StepConnector lines are misplaces when muiTheme's isRtl is set to true and the direction is rtl.

Versions

Material-UI: 0.15.4
React: 15.3.0
Browser: Chrome 52.0.2743.116

Hi @mehdijjz.
i had the same issue with vertical stepper on right to left layout.
I currently putted stepper lines to the right by editing file located on node_modules/material-ui/Stepper/StepConnector.js replace line 54 and 55
54|  styles.line.borderLeftStyle = 'solid';
55|  styles.line.borderLeftWidth = 1;

with
    if(muiTheme.isRtl){
      styles.line.borderRightStyle = 'solid';
      styles.line.borderRightWidth = 1;
    }else{
      styles.line.borderLeftStyle = 'solid';
      styles.line.borderLeftWidth = 1;
    }

I hope they fix this on the next releases
material-ui v0.15.0

@ahmedNY Thanks ahmed !


Can't build with Xcode 13.3 beta

When I try and build with the Xcode 13.3 beta, I get the following error:

Library/Developer/Xcode/DerivedData/ApolloCodegen-bpkqdjekiwtugdbvjsrvflrfnjtd/SourcePackages/checkouts/apollo-ios/Sources/ApolloCodegenLib/Frontend/JavaScriptBridge.swift:9:14: error: 'Sendable' class 'JavaScriptError' cannot inherit from another class other than 'NSObject'
public class JavaScriptError: JavaScriptObject, Error {

The Sendable protocol appears to be part of Swift Concurrency; iOS 15+ and Xcode 13+. I wonder if the 13.3 beta has something misconfigured that is causing the compiler error - I get the same error. Could have something to do with concurrency back porting to iOS 13?
I'm going to close this issue for now, given that it's in the first beta, but if this persists in future beta releases please reopen and we'll take another look at it.

@calvincestari I'm still seeing this issue on the official Xcode 13.3 release

Thanks for checking.
I was just going to say that I use 13.3 (13E113) daily without issue. I'm putting up a PR now to update CircleCI to use 13.3 too so we can be sure it's working not just in our environments.

Yeah sorry about that. We had to fork your repo in order to strip null values for change streams


Adding lsn info to restorehistory

Putting this in as it'll make doing the continuing restore work easier as it's a shortcut to find out the last lsn in the recovering/standy db
Changes proposed in this pull request:

LSN information added to the output for Get-DbaRestoreHistory
Extra fields hidden by select-Defaultview
No other changes, fields were in already referenced tables.

How to test this code:

[ ]
[ ]

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

🤶


Update epic-games-launcher to 2.12.24-3321064

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Please don't merge, I need some casks to test https://github.com/vitorgalvao/tiny-scripts/issues/59