Bad redirect

I've 2 questions regarding the package:

I've tested it since 5.3 and it works well, except for strange isuue:
when I logout from one guard, it redirects to other guard login page,
for example: I logout from employee guard, it redirects to projectname/student/login, not to projectname/employee/login.
Some times when I go to projectname/employee/home (and the session expires or no login) it redirects to projectname/student/login.
strange redirect between guards. Is it related to function order in RouteServiceProvider file?

Does it support Laravel 6 ?
Hope I clearly explained the issue, and hope to hear from you soon
Regards

did you fixed it please tell me

with sorrow, Not yet


[CB-16643] Backup status poller timeout doesn't match backup timeout

Thunderhead timeout is currently 10 hours

@sxxgrc with a higher timeout we might see more false "stuck in progress" escalations. as a first step we will just need to surface that it could take as long as 10 hours to hit the timeout.


[type:fix] fix memory leak

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4486 (4fb718b) into master (d764f9d) will decrease coverage by 0.38%.
The diff coverage is 4.00%.

@@             Coverage Diff              @@
##             master    #4486      +/-   ##
============================================
- Coverage     68.40%   68.02%   -0.38%
+ Complexity     7591     7543      -48
============================================
  Files          1027     1027
  Lines         29246    29248       +2
  Branches       2633     2631       -2
============================================
- Hits          20006    19897     -109
- Misses         7667     7773     +106
- Partials       1573     1578       +5

Impacted Files
Coverage Δ

...g/apache/shenyu/common/cache/WindowTinyLFUMap.java
0.00% <0.00%> (ø)

...pache/shenyu/plugin/base/cache/MatchDataCache.java
100.00% <100.00%> (ø)

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


consider moving or deleting "curseforge" assets folder

When building your mod, the curseforge folder gets included in the final jar file which ends up being massive (138MB).
I'd consider moving the folder (or actually deleting it). Also you could reduce the file size by compressing the gif- and removing psd files ("crafting_table_cycling_recipes.psd" is 56Mb).

good call, I'll make sure to remove it from the release, considering it's only for my own use.


Changed the code to check for the first letter of the PR

Fixes https://github.com/Submitty/Submitty/issues/9893
The code to check if the first letter of the PR is capital has been added to the validate.ts file and the required changes in the index.js file in /dist directory have been made.

@williamjallen I am completely new to the field of open source. Can you please guide me what necessary changes need to be done in order to make my PR accepted. The build file shows that my code failed 'npm run lint' check but when I run this command on my local host I get multiple errors. How to deal with this?

The checks failed because your changes do not match our code style rules.  What do you see when you run the linter locally?

closing this due to lack of recent activity


Fix typo

python -> Python

Learn Build status updates of commit e06c8cc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/cognitive-services/use-key-vault.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@changeworld - thanks!


As a user I want to navigate to the process overview page

Synopsis
As a user
I want to navigate to the control report page
So that I'm able to find the page which will eventually supply me with the control report

Acceptance-Criteria

[ ] AC1: Users can access the control report page

Tech-Notes
Tasks

[ ]

Note: Two separate menu points have been created instead of one menu point with two child points, this is because the child menu point functionality has not been developed yet


Page menu / navigation buttons' focus outline cut off

Bug Description
There isn't enough space currently for displaying the focus outline for the buttons below the Page / Page navigation buttons.

Expected Behaviour
Focus outline should be fully visible

Steps to Reproduce
Tab into Page buttons / navigation

Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Verified in QA


Public Enum WebStatusCode - Ambiguous Name Detected: WebStatusCode

It's probably a first use problem. Downloaded VBA-Web - Example.xlsm and wanted to play with it. But whatever I do, I get an error message: Ambiguous Name Detected: WebStatusCode.
My environment: Windows10, Excel2010. No Excel projects besides VBA-Web - Example loaded.
Please let me know how to further debug the issue.
Thanks,
Chris

Hi Chris,
It sounds like I may have published a broken build, I check it out shortly. In the meantime, look for a backup_WebHelpers file (and any other with backup_) and remove those.

Thanks for the answer. I removed them this morning and was able to get a
bit further but I'm not yet there. It'll take me a few days before I can
get back to this, so take your time ... Something else: do you know of
anyone who has produced a Eurostat source based on your tool box?
Have a nice evening,
Chris

Hi Tim,
Thank you for sharing such a valuable tool.
I think this same issue above is occurring again:

I think you may have again left the backup_* files in the VBA-Web v4.1.6 release.

Darn, ok I'll get that fixed

Hi,
I am getting this too, on Win 10, Excel 2016 with VBA-Web4.1.6 fresh download trying to run the "VBA-Web - Example - Excel.xlsm" file.
Is this a problem only in the Example file or also in the installer version?
Thanks,
Owen

I'm also getting exactly the same thing on Excel for Mac 16.30 ==> Ambiguous name detected: WebStatusCode

Same issue. Is this project abandoned? This issue is two and a half years old. The installer file doesn't work on Mac, either, and the comments about fixing it are old.

The project is not abandoned but you just have to remove the duplicates yourself in the meantime.

You need to delete this "backup" WebHelpers module:


Can we add root/apex domain to AFD Std/Premium?

Can we add root/apex domain to AFD Std/Premium? We can achieve this in Azure Front Door if we are using Azure DNS.

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

ID: ed72df51-8f93-f2f3-6d66-fc4d085fe691
Version Independent ID: 2774ed11-a0a1-bc23-45f2-d128954650ba
Content: Configure HTTPS for your custom domain in an Azure Front Door Standard/Premium SKU configuration
Content Source: articles/frontdoor/standard-premium/how-to-configure-https-custom-domain.md
Service: frontdoor
GitHub Login: @duongau
Microsoft Alias: amsriva

@vikaskumawat80, Thanks for reaching out. We will investigate and update this thread.

@vikaskumawat80, AFDX will support it, but no ETA at this moment since we have dependencies with other Azure services. So, you can expect this by GA time.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


scrollIntoView Promise doesn't resolve on some elements

I discovered an apparent bug, where I have something like this:
scrollIntoView([HTMLElement], {
  scrollMode: 'if-needed',
  block: 'nearest',
  boundary: document
}).then(() => { console.log('done') })

It appears that if the element doesn't need to be scrolled into view (or some other reason?), then the Promise will never resolve. Even if I change .then() to .finally(), nothing happens. No console.log is output. Meaning I can't chain scrollIntoViews.
I didn't have this problem until the element I needed to check if it needed to be scrolled into view was sometimes the same height as the document.

Hey there!
Are you talking about smooth-scroll-into-view-if-needed?

@stipsan Yes, that's what I'm using.

Alright. Then it's for sure a bug. It's always returning a Promise.all:
https://github.com/stipsan/smooth-scroll-into-view-if-needed/blob/c8aa5a2d23541c8f24921f0196ece43843c58f0d/src/index.ts#L111-L125
If scrollMode: 'if-needed' is breaking the behavior it sounds like it's returning something other than an empty array. Gonna check it out, thanks for the report! 😄

I just released v1.1.13 with a fix, let me know if it works 😄

I'm open for feedback on what is returned when the promise resolved. It might make sense to include the same array of actions that it's mapping through?

Thanks for the fast turnaround! It works! ^_^

What it passes to the then() function is irrelevant for me. I just need to know when the Promise resolved.

Alright, good news! 😄👍

One thing that I'm seeing now. Although now an element that doesn't need scrolling returns a promise, it only resolves after the timeout, even though no scrolling happened. I would expect that if no scrolling needs to happen, then it resolves immediately.

Good catch! I'll take care of it right after https://github.com/stipsan/compute-scroll-into-view/issues/17 😄


Experiment with wasm-opt options when building wasm

We currently use the default optimizations and --strip when running wasm-opt in the build system. We should experiment with enabling additional optimizations to see how they change performance.

I believe we have done the experimenting here now, but please reopen if we have more to do on this.


Scheme is missing from static_configs

          Scheme is missing.

But we need to think about the topology we target here:

for in cluster scrape (by the same prom that is proxied by trfk?), we would need to use insecureSkipVerify, as the cert that would be served is for the external hostname.
for out-of-cluster scrape, we would need to give the external hostname and need prom/gagent to have a receive-ca-cert relation (relate to the external ca)

Originally posted by @sed-i in https://github.com/canonical/traefik-k8s-operator/pull/269#discussion_r1352548090

@PietroPasotti is this still relevant?


Problem with download

Hello,
I have a strange issue when i click on the export button from the BulkActionGroup is not working (i mean is not downloading without any error), but if i click on the button created by ExportBulkAction is working, but i need to customize the columns
->bulkActions([
                Tables\Actions\BulkActionGroup::make([
                    Tables\Actions\DeleteBulkAction::make(),
                    ExportAction::make()->exports([
                        ExcelExport::make()->withColumns([
                            Column::make('domain'),
                            Column::make('created_at'),
                        ]),
                    ])
                ]),
                ExportBulkAction::make()

            ]);

You are mixing up BulkExportAction and ExportAction. Just use the BulkExportAction. You can customize the columns the same way.


throttles api to archive 100k+ messages

I found that I needed to throttle API requests when archiving rooms with more than a year or so of messages. The app would silently die after receiving ~10,000 - 30,000 records. With the delay I was able to archive 100k+ messages from a single room.
Note: experiment with the delay if it is too conservative for you.

closing due to age


[.NET] Update FontStyle naming following Spec Update

Updates the FontStyle property to FontType in text blocks
Updates the FontStyles property to FontTypes in HostConfig

Fixes https://github.com/microsoft/AdaptiveCards/issues/2909
How verified
The project was compiled and the tests were run

iOS needs to be updated to fix build break.

@almedina-ms / @nesalang can you guys work to resolving the iOS break for the Android change, (ideally Android changes shouldnt cause iOS to break right?) similarly @RebeccaAnne Android specific renames shouldnt cause UWP failures yeah? Is there a better way to break this down without one pr ending up blocking another ? Does it make sense for one person to make the changes in one go across platforms?

In reply to: 493518311 [](ancestors = 493518311)

@shalinijoshi19 All platforms share the same set of sample cards and host configs, so renames that affect the json will affect all platforms. Looks like we've got all platforms except UWP in #2960. I'll try to get that in PR by end of day.

In reply to: 494556781 [](ancestors = 494556781,493518311)

Hi, you forgot to update the [Obsolete] attributes in AdaptiveHostConfig.cs when you renamed FontStyles to FontTypes.  They're still telling people to use the old AdaptiveHostConfig.FontStyles, which no longer exists.
    [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
    [Obsolete("AdaptiveHostConfig.FontFamily has been deprecated.  Use AdaptiveHostConfig.FontStyles.Default.FontFamily", false)]
    public string FontFamily { get; set; }

    [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
    [Obsolete("AdaptiveHostConfig.FontSizes has been deprecated.  Use AdaptiveHostConfig.FontStyles.Default.FontSizes", false)]
    public FontSizesConfig FontSizes { get; set; } = new FontSizesConfig();

    [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
    [Obsolete("AdaptiveHostConfig.FontWeights has been deprecated.  Use AdaptiveHostConfig.FontStyles.Default.FontWeights", false)]
    public FontWeightsConfig FontWeights { get; set; } = new FontWeightsConfig();


access to test status in tearDown() and suiteTearDown()

in tearDown() and suiteTearDown() I am not seeing any way to access 1) list of passed tests 2) list of failed tests 3) list of skipped tests.
I would like my teardown logic to react to whether there were failures.

Closing as this has been feasible for a while using this.test.parent.tests, though it is using private APIs. A test which has not yet ran will not have a state or duration property.
To get total number of failures, if interested:
after(function() {
  var failures = this.test.parent.tests.map(function(test) {
    return test.state;
  }).reduce(function(prev, curr) {
    return (curr === 'failed') ? prev + 1 : prev;
  }, 0);
});

To get the state of the last test:
$ cat example.js
var util = require('util');

afterEach(function() {
  var test = this.test.parent.tests.reduce(function(prev, curr) {
    return (!curr.state) ? prev : curr;
  });

  console.log(test.title, test.state);
});

it('foo', function(done) {
  setTimeout(done, 100);
});

it('bar', function(done) {
  setTimeout(done, 100);
});

it('baz', function(done) {
  setTimeout(done, 250);
});

$ ./bin/mocha example.js

  ․foo passed
․bar passed
․baz failed

  2 passing (412ms)
  1 failing

  1)  baz:
     Error: timeout of 200ms exceeded. Ensure the done() callback is being called in this test.
      at null.<anonymous> (/Users/danielstjules/git/mocha/lib/runnable.js:170:19)
      at Timer.listOnTimeout [as ontimeout] (timers.js:110:15)

@travisjeffery We could expose a property in the runnable context like this.lastTest, though it's likely not warranted given the lack of interest.


Adaptive Layout

Adaptive Layout
User Story
As an employer,
I WANT to view a potential employee's deployed portfolio of work samples,
SO THAT I can review samples of their work and assess whether they're a good candidate for an open position.
Acceptance Criteria
GIVEN I need to sample a potential employee's  previous work
WHEN I resize the page or view the site on various screens and devices.
THEN I am presented with a responsive layout that adapts to my viewport.

work in progress


Update module to align with varlink API changes

Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

LGTM


world column too small in database

The world column is only VARCHAR(36) but that is too small for my use case.
A plugin I am using (World System Plugin) uses the players' uuid plus an id for the world name which ends up longer than 36 letters.
I wanted to use the context system in luckperms to give players perms in only their world but I can't because of this error:
13.05 12:55:07 [Server] WARN Caused by: org.h2.jdbc.JdbcBatchUpdateException: Value too long for column """WORLD"" VARCHAR(36) NOT NULL": "'id1-6e05911c-d04a-4cba-ab04-fe0bf50c8388' (40)"; SQL statement:
13.05 12:55:07 [Server] WARN INSERT INTO `luckperms_user_permissions` (uuid, permission, value, server, world, expiry, contexts) VALUES(?, ?, ?, ?, ?, ?, ?) [22001-199]

Is the only place that needs to be updated these files?
common/src/main/resources/me/lucko/luckperms/schema

Yes, that's the only place.
Sorry I haven't replied, I'm not really sure the best way to resolve this, the 32 character limit seemed reasonable to me at the time, I could bump it but but where do you set the bar? 🙃

I personally think a 64 limit would be good but I am not sure if there would be a better way to go about it.


Separated ras tool tests for hw and mock platforms.

Separated the valid test cases for ras to only run on MCP.
Separated the catastrophic and fatal error injection cases into mock only test cases.
Created a new test case for MCP hw only to run a valid test case but without catastropic and fatal error injection.

Yeah, the fix is here in 885.  It hasn't made its way to master yet.

@tswhison Ok. Can we get this in prior to #885 and update the ras tests in that PR? It still has a few fixes left in enum before that can be merged anyways.


Error occured at: src/parser.c - read_log - 2669

Hi,
I have the following error when we invoke goaccess:

GoAccess - version 1.2 - Mar  7 2017 06:40:58
Config file: /etc/goaccess.conf
Fatal error has occurred
Error occured at: src/parser.c - read_log - 2669
Unable to open the specified log file. No such file or directory

My config file has the following:
time-format %H:%M:%S
date-format %d/%b/%Y
log-format %h %^[%d:%t %^] "%r" %s %b "%R" "%u"

And some example traces of my log are:
<IP_ADDRESS> - - [17/Feb/2018:20:15:00 +0100] "GET /escudos.php HTTP/1.1" 200 195 "-" "Dalvik/2.1.0 (Linux; U; Android 7.0; SM-G935F Build/NRD90M)"
<IP_ADDRESS> - - [17/Feb/2018:20:15:00 +0100] "GET /escudos.php HTTP/1.1" 302 249 "-" "Dalvik/2.1.0 (Linux; U; Android 5.1.1; SM-J500FN Build/LMY48B)"

Is something wrong in my log-format?
Thanks in advance!

How did you invoke goaccess? The following should work:
goaccess /path/access.log --log-format=COMBINED

I usually try with
goaccess -f ${LOGFILE} -a > /var/www/html/report/report.html
Using your command, it returns:
:~$ sudo goaccess /var/log/apache2/access.log.1518652800 --log-format=COMBINED

GoAccess - version 1.2 - Mar  7 2017 06:40:58
Config file: /etc/goaccess.conf

Fatal error has occurred
Error occured at: src/parser.c - read_log - 2669
Unable to open the specified log file. Device or resource busy

You must be using the same log from a different process and thus goaccess can't open it. Please make sure no other process is using it and try again. This may help:

lsof +D /path

That will recurse through the filesystem under /path, so beware doing it on large directory trees.
Once you know which processes have files open, you can exit those apps, or kill them with the kill(1) or pkill command.

Hi,
It's solved, i had a rookie error (in my config file I had uncommented log-file parameter)
Thanks for your help, i close the issue

@kete1987 Glad to hear that :)

For me this was caused by goaccess not being able to create the report file in the folder.  I had to manually create the report file first with touch


RMC incorrectly reports an assertion failure on a two-dimensional vector

I tried this code:

fn main() {
    let a: Vec<Vec<i32>> = vec![vec![0; 2]; 1];
    assert!(a.len() == 1);
}

using the following command line invocation:
rmc test.rs

with RMC version: main-154-2021-08-06
I expected the assertion to pass, but it didn't:
[main.assertion.1] line 3 assertion failed: a.len() == 1: FAILURE

If I compile the test and run it, the assertion is not hit:
$ rustc test.rs
$ ./test
$

Also, if I modify the first line in the function to
    let a: Vec<Vec<i32>> = vec![vec![0; 2]];

i.e. leave out the length for the outer vector, the assertion passes.

The callstack on using the vec! initialization macro looks like:
[+] extend_with
[+] from_elem
[+] from_elem
extend_with uses the SetLenOnDrop optimization as described in #191. Since rmc does not currently handle drop behavior, the length of the Vec does not hold the correct value after initialization. This behavior should be fixed by a combination #402 and #336.

Added as a unit test in #402


Warp to start text not present in other langauges / PAL

Saving at a save station still prints the vanilla text

It was added back when multiple language support wasn't a thing or partially a thing. I would need some help for the translations if it requires to have other languages supported unless we're  just defaulting to english.

Forcing english is good enough.

I'll look into it

Relevant line in patcher:
https://github.com/randovania/randomprime/blob/329f5e0d4436314d12dfe0994fab5638b85f94c4/src/patches.rs#L9179
Use the edit_strings function in strg.rs


OAuth redirect url is invalid

Describe the issue
The generated redirect url from swagger.js during OAuth2 workflow in swagger ui is invalid: "http://localhost:7071http://localhost:7071/api/oauth2-redirect.html"
To Reproduce

Use Microsoft.Azure.WebJobs.Extensions.OpenApi.FunctionApp.InProc example
(not necessary but for better traceability): change "AuthorizationUrl" in PetStoreAuth.cs to any invalid url, so that you can see the full auth url without redirect
Open http://localhost:7071/api/swagger/ui
Start  oauth2 authorization
Auth url contains "http://localhost:7071http://localhost:7071/api/oauth2-redirect.html" as redirect_uri parameter

Expected behavior
Auth url should contain "http://localhost:7071/api/oauth2-redirect.html" as redirect_uri parameter
Environment (please complete the following information, if applicable):

OS: Windows
Browser chrome
Version 98

I have a similar issue but for me the redirect URI seems to be: 'http://localhost:7071api/oauth2-redirect.html' and MSAL shows a AADSTS90102: 'redirect_uri' value must be a valid absolute URI. error message. I hope the hotfix #62 will bring the fix.

Need this fix, I'm seeing the same issue!

I see that a fix has been made for this. Are there plans to bring this change in and release it?

I can confirm that downgrading to 1.0.0 fixes the issue.
Here are my test results:

1.3.0 - broken
1.2.0 - broken
1.1.0 - broken
1.0.0 - works

@justinyoo I see the release of this bug fix in version 1.0.4 is being held up by enhancement #369. I can work around this bug by manually editing the authentication url, but it is quite a pain.  Is there any chance of getting this bug fix released soon by bumping #369 to a later release.  I can't downgrade to an earlier version as others have suggested because I am using the excellent new Document Filters feature.


[question] sqlite extension alignment problem in exqlite, but not in sqlite3

👋
I understand that it's unlikely to be a problem in exqlite, but just in case you have any idea as to what I'm doing wrong.

Repo: https://github.com/ruslandoga/sqlite-zig-problem
I'm trying to write an extension for sqlite in zig and I'm getting an alignment panic when fetching sqlite's aggregate context. But, I'm only getting it when executing the query from elixir, not from sqlite3 cli.
Meaning,
sqlite> .load dist/extc
sqlite> .load dist/extzig

-- this uses c extension
sqlite> with recursive ten(x) as (select 1 union all select x+1 from ten where x<10) select sumc(x) from ten;
55.0

-- this uses zig extension
sqlite> with recursive ten(x) as (select 1 union all select x+1 from ten where x<10) select sumzig(x) from ten;
55.0

but in elixir:
iex(1)> E.sumc
# [debug] QUERY OK db=0.6ms decode=1.3ms idle=738.5ms
# with recursive ten(x) as (select 1 union all select x+1 from ten where x<10) select sumc(x) from ten []
%Exqlite.Result{
  columns: ["sumc(x)"],
  command: :execute,
  num_rows: 1,
  rows: [[55.0]]
}

iex(2)> E.sumzig
thread 8760720 panic: incorrect alignment

@warmwaffles here's the zig file: https://github.com/ruslandoga/sqlite-zig-problem/blob/master/src/ext.zig
Here's how I'm compiling it on mac: https://github.com/ruslandoga/sqlite-zig-problem/blob/0c206a50001139ff633a74ce0b909790588200c6/Makefile#L10-L12
Zig docs suggest downloading "nightly" master builds so I'm using
> zig version
0.10.0-dev.2473+e498fb155

@ruslandoga are you also on an M1 mac?

@warmwaffles ah, yes.

@warmwaffles small update: I printed the pointer and it was 102a7acdc when called from Elixir and 120016530 when called from sqlite3-cli. Converting these to base10 yields<PHONE_NUMBER> and<PHONE_NUMBER>. And<PHONE_NUMBER>/8 = 603991206 but<PHONE_NUMBER>/8 = 542438811.5 and that's what @alignCast checks.

std.mem.alignForward workaround works

I 100% trust enif_alloc to do the right alignment based on the architecture OTP was compiled for.

@ruslandoga are you still having alignment issues?

@warmwaffles hi! I wasn't able to find the cause of this issue at the time so I moved away from using extensions in my app. I still plan to learn more about low-level programming and find out what's going on, though :)

Alright

👋
I think it was due to shifting the address by four bytes sizeof(int)

Hmmm, I wonder if the ssize_t type would be a better type to use since on a 64 bit system it should be up to 64 bits and on a 32 bit system it should be up to 32 bits.


DRILL-7514: Update Apache POI to Latest Version

DRILL-7514: Update Apache POI to Latest Version
Description
Drill's Excel Format Plugin uses Apache POI to parse Excel files. While this reader is effective in that it parses formulae and data types, it uses memory inefficiently and will struggle to read very large Excel files.
The latest version of POI addresses some of the memory issues and hopefully Drill will be able to query larger Excel files without running out of memory.
This PR updates Drill to use the latest version of Apache POI and also updates the User Agent Parser to a more recent version.
There was a minor change to the POI's behavior with respect to empty sheets, so I had to enclose a line in a try/catch block.
Documentation
No user visible changes.
Testing
All relevant unit tests run and passed.

@vvysotskyi
Addressed review comments and squashed commits.

@cgivre, please address this comment: https://github.com/apache/drill/pull/1991#discussion_r382168457

@vvysotskyi Thanks for the review.  Commits squashed.


Crash occurs when using it with Next.JS 13 (+App Directory)?

Describe the bug
Hi, First thanks for your amazing work! I've been searching through the editor library exactly like this.
I'm setting up a project using Next.JS v13 with App Directory feature enabled. However, when I try to use Yopta, it works at the first time but when I try to add a block or change order it crashes with the below error:
https://user-images.githubusercontent.com/1618736/224556042-45eebf44-2aaf-48ae-b2b6-7c93eb78220d.mov
The stacktrace is:
Unhandled Runtime Error
TypeError: Cannot read properties of null (reading 'type')

Call Stack
e.isVoid
<EMAIL_ADDRESS>(21:20240)
Object.isVoid
<EMAIL_ADDRESS>(1246:0)
Jo
<EMAIL_ADDRESS>(13:68781)

It seems like on withVoidNodes function on Editor, a null is being injected on element: Element argument to Editor.isVoid handler.
https://github.com/Darginec05/Yopta-Editor/blob/8bc4f3c5a76a0e7a19223fb987e4f6a63c82f855/src/components/Editor/plugins.ts#L9-L17
I cannot give the full code, but it's currently in a vanilla setup like:
'use client';

import * as React from 'react';
import { Descendant } from 'slate';
import { YoptaEditor } from 'yopta-editor';
import 'yopta-editor/dist/index.css';

export default function NewGravityPage() {
  const [value, setValue] = useState<Descendant[]>([]);
  return (
    <div ...>
      ...
      <YoptaEditor value={value} onChange={(data) => setValue(data)} />
    </div>
  );
}

One suspect is Next.JS v13 App Directory— it changed the default rendering method from client-side to SSR.
I've noticed that after adding yopta-editor, I get multiple "Hydration failed" warnings— which can indicate that it's related to
problem between yopta-editor and Next.JS, but I'm not sure of that since I'm a newbie to Next.JS.
Any ideas on how can it be fixed? or can it be solved on v2 you're currently working on?
Environment Information
Desktop:

OS: macOS Ventura 13.0.1
Browser: Chrome
Version: 110.0.5481.177 (arm64)

Dependency Versions

"slate": "^0.91.4"
"slate-react": "^0.91.10"
"yopta-editor": "^1.1.4"
"next": "13.2.3"
"react": "18.2.0"
"react-dom": "18.2.0"

Hm, strange behaviour, because I provided <NoSSR /> wrapper for Editor🤔
But thanks for the report!
I'll check it now

I checked locally and it works as expected.
Can you try please with incognito mode?
About warnings.
I'm getting warnings from browser extenstions, so when I use Yopta-Editor in incognito it works without any warnings.

@Darginec05 Tried on incognito but no luck 😢
I've created a sample project which reproduces the issue: could you look at it?
https://github.com/therne/yopta-next-test/
This replicated the same project base from my code:

Next.JS v13 + App Directory
Same UI Framework (Tailwind, PostCSS, https://github.com/shadcn/ui )

You can fork and run pnpm dev to reproduce the issue.
The code applied YoptaEditor is on this commit: https://github.com/therne/yopta-next-test/commit/550a3f5f82ec3f02059d8f37705a6121aacafb8a

@therne bug fixed in @1.1.5.


More DocTableInfo tweaks

See commits

LGTM Can we do the same for numberOfShards?

Yes, turns out we can:
https://github.com/crate/crate/blob/00e4bcfc6e77988a28c81fc14129439ddc26bc43/server/src/main/java/org/elasticsearch/cluster/metadata/IndexMetadata.java#L1047-L1054
Added another commit to the PR


IGNITE-8801 Remove IGNITE_ALLOW_ATOMIC_OPS_IN_TX, set default as false

Need to change default behaviour of atomic operations to fail inside transactions.

Remove IGNITE_ALLOW_ATOMIC_OPS_IN_TX system property.
Set default value to restrict atomic operations in CacheOperationContext constructor without arguments and arguments for calls of another constructor.
Fix javadocs.

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

@timoninmaxim Hi Maxim, could you please review my changes?

@timoninmaxim could you please review?


Impossible to use IBT4ZWAVE (Unable to write true on 38-0-Open)

The device is included / pingeable / Inclusion was done in secure S2 mode
When i click on START in the device screen, i have two messages. Nothing is moving (no open, no close) for the portal
Any idea ?
API WRITEVALUE CALLED
VALUE UPDATE FAILED
Zwave JS UI log
2023-06-27 14:49:14.038 INFO Z-WAVE: Calling api writeValue with args: [
{ nodeId: 61, commandClass: 38, endpoint: 0, property: 'Open' },
true,
null,
[length]: 3
]
2023-06-27 14:49:14.040 INFO Z-WAVE: Writing true to 61-38-0-Open
2023-06-27 14:49:14.217 ERROR Z-WAVE: Unable to write true on 38-0-Open
2023-06-27 14:49:14.218 INFO Z-WAVE: Success zwave api call writeValue false
`
IBT4ZWAVE
par NICE Spa
Connecté via [700 Series-based Controller]
Firmware: 7.0
Informations Z-Wave
ID: 61
État: Actif
Prêt: Oui
Sécurité maximale: S2 Authentifié
Z-Wave Plus: Version 2

Hi!
Could you explain me what you did ? :D Did you fix it ? Must wait the next version ?


[Question] Install from tarball

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Hi, Im install yt-dlp from tarball releases
is it ok to ignore this message?
/home/rex/.cache/build/yt-dlp/src/yt-dlp/setup.py:41: UserWarning: Skipping file README.txt since it is not present. Try running `make pypi-files` first
  warnings.warn('Skipping file %s since it is not present. Try running `make pypi-files` first' % fn)
/usr/lib/python3.9/site-packages/setuptools/dist.py:484: UserWarning: Normalizing '2021.09.25' to '2021.9.25'
  warnings.warn(tmpl.format(**locals()))

Version
$ yt-dlp -v --version
2021.09.25
$ python -v --version
Python 3.9.7

local/python 3.9.7-1
local/python-pip 20.3.4-1
local/python-setuptools 1:57.4.0-2

I running use Arch Linux rm -Rdf pkg src && makepkg -s
makepkg
$ curl -fsL 'https://github.com/yt-dlp/yt-dlp/releases/download/2021.09.25/SHA2-256SUMS' | sha256sum -c --ignore-missing
yt-dlp.tar.gz: OK

$ bsdtar -xzf ../yt-dlp.tar.gz
$ ls; cd yt-dlp
yt-dlp/

$ sed -i 's|etc/bash_completion.d|share/bash-completion/completions|' setup.py
$ sed -i 's|etc/fish/completions|share/fish/vendor_completions.d|' setup.py
$ export PYTHONHASHSEED=0
$ python setup.py build
==> Starting build()...
/home/rex/.cache/build/yt-dlp/src/yt-dlp/setup.py:41: UserWarning: Skipping file README.txt since it is not present. Try running `make pypi-files` first
  warnings.warn('Skipping file %s since it is not present. Try running `make pypi-files` first' % fn)
/usr/lib/python3.9/site-packages/setuptools/dist.py:484: UserWarning: Normalizing '2021.09.25' to '2021.9.25'
  warnings.warn(tmpl.format(**locals()))
running build
running build_py
creating build
creating build/lib
creating build/lib/yt_dlp
copying yt_dlp/YoutubeDL.py -> build/lib/yt_dlp
...

# package()
$ python setup.py install --root="${pkgdir}/" --optimize=1 --skip-build
==> Entering fakeroot environment...
==> Starting package()...
/home/rex/.cache/build/yt-dlp/src/yt-dlp/setup.py:41: UserWarning: Skipping file README.txt since it is not present. Try running `make pypi-files` first
  warnings.warn('Skipping file %s since it is not present. Try running `make pypi-files` first' % fn)
/usr/lib/python3.9/site-packages/setuptools/dist.py:484: UserWarning: Normalizing '2021.09.25' to '2021.9.25'
  warnings.warn(tmpl.format(**locals()))
running install
running install_lib
...
writing byte-compilation script '/tmp/tmp7v19xt_j.py'
/sbin/python /tmp/tmp7v19xt_j.py
removing /tmp/tmp7v19xt_j.py
running install_data
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/bash-completion
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/bash-completion/completions
copying completions/bash/yt-dlp -> /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/bash-completion/completions
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/zsh
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/zsh/site-functions
copying completions/zsh/_yt-dlp -> /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/zsh/site-functions
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/fish
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/fish/vendor_completions.d
copying completions/fish/yt-dlp.fish -> /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/fish/vendor_completions.d
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/doc
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/doc/yt_dlp
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/man
creating /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/man/man1
copying yt-dlp.1 -> /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/share/man/man1
running install_egg_info
running egg_info
creating yt_dlp.egg-info
writing yt_dlp.egg-info/PKG-INFO
writing dependency_links to yt_dlp.egg-info/dependency_links.txt
writing entry points to yt_dlp.egg-info/entry_points.txt
writing requirements to yt_dlp.egg-info/requires.txt
writing top-level names to yt_dlp.egg-info/top_level.txt
writing manifest file 'yt_dlp.egg-info/SOURCES.txt'
reading manifest file 'yt_dlp.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
adding license file 'LICENSE'
adding license file 'AUTHORS'
writing manifest file 'yt_dlp.egg-info/SOURCES.txt'
Copying yt_dlp.egg-info to /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/lib/python3.9/site-packages/yt_dlp-2021.9.25-py3.9.egg-info
running install_scripts
Installing yt-dlp script to /home/rex/.cache/build/yt-dlp/pkg/yt-dlp/usr/bin

#129

It means the README.txt will not be copied to share/docs. Otherwise, you won't have any issues. The file is however supposed to be in the tar. So I am marking this as a bug

Thank you,
yes README.txt is not in the share/doc folder yt-dlp-2021.09.25-1-any.pkg.tar.zst.txt
sorry I don't know if it will cause any problem, I used it with mpv, will it work if I add README.txt like this PKGBUILD

I don't know if it will cause any problem

No, it wont. It is just convention to put the documentation there

can README.txt be added to the released tarball, I see README.txt on PyPI Wheel exists
I will close this issue, Thank you.

The file is however supposed to be in the tar. So I am marking this as a bug


Enable all h2spec test

#3645 was closed, and all tests passed on my laptop. We should be able to enable all the tests now.

FreeBSD build failed but the error is not because of the change. Rerunning.
gmake[3]: Warning: File '/usr/local/lib/perl5/5.30/mach/Config.pm' has modification time<PHONE_NUMBER> s in the future
Makefile out-of-date with respect to /usr/local/lib/perl5/5.30/mach/Config.pm /usr/local/lib/perl5/5.30/mach/CORE/config.h
Cleaning current config before rebuilding Makefile...
make -f Makefile-pl.old clean > /dev/null 2>&1
gmake[3]: [Makefile-pl:802: Makefile-pl] Error 1 (ignored)
"/usr/local/bin/perl" Makefile.PL "INSTALLDIRS=perl" "INSTALL_BASE=/var/jenkins/workspace/freebsd-github/14166/install" "PREFIX="
Generating a Unix-style Makefile-pl
Writing Makefile-pl for Apache::TS
Writing MYMETA.yml and MYMETA.json
==> Your Makefile has been rebuilt. <==
==> Please rerun the make command.  <==
false
gmake[3]: *** [Makefile-pl:806: Makefile-pl] Error 1
gmake[3]: Leaving directory '/var/jenkins/workspace/freebsd-github/src/lib/perl'
gmake[2]: *** [Makefile:679: all-local] Error 2
gmake[2]: Leaving directory '/var/jenkins/workspace/freebsd-github/src/lib/perl'
gmake[1]: *** [Makefile:585: all-recursive] Error 1
gmake[1]: Leaving directory '/var/jenkins/workspace/freebsd-github/src/lib'
gmake: *** [Makefile:854: all-recursive] Error 1
++ exit 1
Build step 'Execute shell' marked build as failure

[approve ci freebsd]

[approve ci freebsd]


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1376 ms

Resolved: Bofrost.de performance has improved in 87dc6db.


Add RefreshMode and RenderMode support to mobile navigator.

Addresses issue #1720
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[x] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

This is great, and looks a lot like TabModel.  Would we consider pushing that analog even further in the naming?  Specifically, would we consider renaming NavigatorPageModel to just PageModel and have the routes argument to NavigatorModels ctor renamed  to pages;

Thanks for taking a look. Great idea about the naming - will make those updates. My main concern there is that this makes it a breaking change, but it will be quite obviously broken and fairly trivial to update apps.

could potentially maintain "routes" as a deprecated name.   But I think
could also sell this as an enhancement worth retyping that one key.  Don't
think anything else likely to be breaking.
On Mon, Mar 2, 2020 at 5:29 AM Tom Tirapani<EMAIL_ADDRESS>wrote:

Thanks for taking a look. Great idea about the naming - will make those
updates. My main concern there is that this makes it a breaking change, but
it will be quite obviously broken and fairly trivial to update apps.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xh/hoist-react/pull/1732?email_source=notifications&email_token=AARTL2JUMUO2I4LPI734M63RFODBTA5CNFSM4K5RIK22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENOYWTI#issuecomment-593333069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARTL2ME5XN3Q7PXOTRRGMLRFODBTANCNFSM4K5RIK2Q
.

Note that navigator already exposes an observable pages[], which is distinct from routes. (routes being the "possible" pages and how they map to Router5, and pages being the instances of the pages currently in the stack)

I've implemented the refactor as discussed yesterday with Lee. Please see new Toolbox PR required for this branch: https://github.com/xh/toolbox/pull/375


[OSX] Can't get out of mini-player without quitting

OS: macOS 10.11.5 El Capitan
Issue Descriptions:
After clicking the mini player icon in the bottom right i can no longer get to the full sized player without quitting.
Steps to Reproduce:
Press mini player icon in lower right and try and go back.
I've tried going back by clicking mini player again (expected result)
Also i've gone to window and slected mini player and then full player and neither are allowing the window to go back to full sized.

After re-sizing the window now it's functioning properly.
Both Window menu and button function now. This is an odd bug.

Sounds like a one off IPC socket glitch, unless you can reproduce consistantly 👍


IdotMaster1

its literally so faster i tested

coconut

average bunkercoin moment
literally

i made it 1.15.5-dev now. pretty please with gumdrops

im gonna stop protesting now this whole idea was stupid lol. but i am still going to make a brand around #coconutting


Allow Mist to Listen on Unix Domain Socket

It'd be nice to allow Mist to listen on a Unix Domain Socket.

I think this has come up before... I'll need to add support to glisten first, and then figure out a good way to expose that in mist.
I have something roughly working locally for glisten, but I'm not sure the API is really ideal...
I'll noodle on this a bit, and maybe try to get some feedback in the Discord.


Map a custom struct while making join.

create table firms_users(
  firm_id integer references firms(id),
  user_id integer references users(id),
  primary key (firm_id, user_id)
);

How would I join on this table note the intended result would be some of users attributes + some of firm attributes not the whole model.
pub struct UserWithFirm {
    pub(crate) user: SlimUser,
    pub(crate) firm: SlimFirm,
}

Thank you.

Great question.
Right now there's only support for many-to-1 joins, though support for other types is planned.
If I'm understanding right, you're asking about a many-to-many join. You can work around the limitation mentioned by operating on the many to many table, rather than the User table or the Firm table. It looks something like this:
#[derive(Model)]
pub struct UserWithFirm {
id: Uuid,
user_id: Uuid,
#[ormlite(many_to_one_key = user_id)]
pub(crate) user: Join,
firm_id: Uuid,
#[ormlite(many_to_one_key = firm_id)]
pub(crate) firm: Join,
}
Once you have that struct, you should be able to apply select filters on the joined objects. Something like:
let results = UserWithFirm::select()
.join(UserWithFirm::organization())
.join(UserWithFirm::user())
.where_bind("user.active = ?", true)
.fetch_all(&pool)
.await?;

for uf in results {
    println!("user: {:?}", uf.user);
    println!("firm: {:?}", uf.firm);
}

Thank you for taking a look into it.
Will the result be Vec, I noticed the loop. Well the intended result is only UserWithFirm struct, no Vec, becasue each user belongs only to one firm. Thus user: will be some of users attributes, firm: will be some of firms attributes.

Loop is because of the fetch_all. For one, try something like:
let uf = UserWithFirm::select()
.join(UserWithFirm::organization())
.join(UserWithFirm::user())
.where_bind("user.id= ?", 100)
.fetch_one(&pool)
.await?;

println!("user: {:?}", uf.user);
println!("firm: {:?}", uf.firm);

Waht about runtime, I notice only runtime-tokio-rustls , Is there any plans for actix?

Actix runs on tokio. The runtime-actix is deprecated.
See this thread: https://www.reddit.com/r/rust/comments/qc3wc2/sqlx_exist_a_problem_if_pick_runtimeactix_vs/

This issue seems to be resolved. If there's a separate issue about the runtime options, let's put that in its own issue.


UGDG Update

UG update:

Added FAQ for java installation guide and executing the app through the terminal. Resolves #272.
Updated find to clarify on its use. Resolves #302.

DG update:

Added generic sequence diagram for XYZCommandParser to reduce unnecessary duplication on other sequence diagrams. Resolves #205
Sequence diagrams on Command-s now start from the Command itself, rather than going from the LogicManager which passes to a Parser before going to Command.
AddsCommandSequenceDiagram has been updated to reflect the current implementation.
FocusCommandSequenceDiagram has been slightly modified to better reflect the sequence of operations being performed.
Updated AssignCommandSequenceDiagram in accordance to the new design. Resolves #211.
Updated appending on manual testing with some more tests (could always use more)
Deleted more to be added from use case and user story sections

Codecov Report
Base: 61.60% // Head: 64.66% // Increases project coverage by +3.06% :tada:

Coverage data is based on head (d1058be) compared to base (ed52582).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #328      +/-   ##
============================================
+ Coverage     61.60%   64.66%   +3.06%
- Complexity      641      669      +28
============================================
  Files           112      111       -1
  Lines          2297     2281      -16
  Branches        243      241       -2
============================================
+ Hits           1415     1475      +60
+ Misses          810      730      -80
- Partials         72       76       +4

Impacted Files
Coverage Δ

src/main/java/seedu/taassist/model/TaAssist.java
61.42% <0.00%> (ø)

...va/seedu/taassist/logic/commands/ListsCommand.java

...va/seedu/taassist/logic/parser/TaAssistParser.java
90.00% <0.00%> (+2.90%)
:arrow_up:

...ain/java/seedu/taassist/model/session/Session.java
80.00% <0.00%> (+4.00%)
:arrow_up:

...c/main/java/seedu/taassist/model/session/Date.java
81.81% <0.00%> (+9.09%)
:arrow_up:

...ain/java/seedu/taassist/model/student/Student.java
66.19% <0.00%> (+9.85%)
:arrow_up:

.../seedu/taassist/model/moduleclass/ModuleClass.java
79.41% <0.00%> (+14.70%)
:arrow_up:

.../taassist/model/moduleclass/StudentModuleData.java
73.33% <0.00%> (+30.00%)
:arrow_up:

.../taassist/logic/parser/ParserStudentIndexUtil.java
93.33% <0.00%> (+33.33%)
:arrow_up:

...java/seedu/taassist/model/session/SessionData.java
42.85% <0.00%> (+42.85%)
:arrow_up:

... and 1 more

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.


Codecove integration

LGTM.
We are like a really serious open source project now.

Current coverage is 63.13% (diff: 100%)

No coverage report found for master at 9226ec5.

Powered by Codecov. Last update 9226ec5...137d0e4

@cpojer  we should probably collect coverage for only some specific packages.
most of the things are now tested through integration tests and collecting coverage for those is not really possible

yeah, feel free to improve it. I just wanted to get it in, because why not? :)


Permission denied

Hi P'@katopz , I've got a problem that I can't push new branch name solana(content in solana page) to create merge request and review before merge to main, and I gonna assign to P'@katopz as reviewer and merge it krub. 🥺🙏🏻
I've use SSH with new key generate also HTTPS too but it's still can't push krub, that why i think it come from permission on repo
it's seem like i don't have permission for this book repo krub 🥺
or I need to fork first and create merge request from my fork repo krub ? 🥺
ERROR: Permission to gist-rs/book.git denied to supamongkonR.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights

remote: Permission to gist-rs/book.git denied to supamongkonR.
fatal: unable to access 'https://github.com/gist-rs/book.git/': The requested URL returned error: 403

This is content that I've to gonna push from my local repo krub.

🥺🥺🙏🏻

OK krub now I can create merge request by fork fist 😅😅 this issue is closed krub.

Yes, you have to fork and make a PR for review.
Better create new branch (e.g. consensus or dev) instead of main on your repos because you will end up to rebase (and get mess up) when original has been changed and that will lead to code conflict later. see: https://opensource.com/article/19/7/create-pull-request-github

HTH 🤗

@katopz Thanks you so much krub, now I've create PR and waiting to review krub.
P'@katopz krub, Solana is so progressive!!! make us feel like living in a future, i never seen consensus like this before,
they system design, how they make it fast. it so geniusssss!!!!
I've spent all my holiday all day all night with fun and feeling good when know more about this technology,
I'll rise more content about solana and rust research krub. Thanks you so much again to help me and bring me to this world krub🥺🙏🏻


PORT FOR LOGS

what port do i have to set in order to make logs work? currently it just says website unreachable or something like that? nice bot by the way ;)

port 8890 as mentioned inside the configuration option

Doesnt work obviously @Girtablulu

Have you set port forwarding?

How do i do that? :thinking: @Girtablulu

Inside your router

same question

Port forwarding is outside the scope of these GitHub issues. Please look into e.g. https://portforward.com/ for details and guides on port forwarding.


ObjectStore API to read from remote storage systems

Which issue does this PR close?
Closes #616 .
Rationale for this change
Currently, we can only read files from LocalFS since we use std::fs in ParquetExec. It would be nice to add support to read files that reside on storage sources such as HDFS, Amazon S3, etc.
This PR substitute #811 by only adding in the ObjectStore interfaces, leave hooking these APIs up into the rest of the system / rewrite the existing data sources (like Parquet, etc) as separate follow-ups.

What changes are included in this PR?

Introduce ObjectStore API as an abstraction of the underlying storage systems, such as local filesystem, HDFS, S3, etc. And make the ObjectStore implementation pluggable through the ObjectStoreRegistery in Datafusion's ExecutionContext.
Are there any user-facing changes?

Users can provide implementations for the ObjectStore trait, register it into ExecutionContext's registry, and run queries against data that resides in remote storage systems as well as local fs (the only default implementation of ObjectStore).

@rdettai on the topics of prefix semantic and filter pushdown, in my mind, these are higher level logic/abstractions that are not natively supported by the object store. For example, I am under the assumption that TableProvider is responsible for parsing the partition parameters from the URI bucket/prefix?partition=year&partition=month and constructing the list object API calls with appropriate prefixes. Similar for URIs like bucket/prefix?versionAsOf=v2, the delta table TableProvider should be responsible to parse out versionAsOf=v2 and issue corresponding object list calls to gather the right list of commit/snapshot objects to fetch from the object store.
I am also having a hard time imaging how will filter push down expressions be used to help the object store build a better or more efficient API call. It seems like these kind of higher level logic should be handled at layers above ObjectStore? They seem to be object store agnostics. As a result, it would be better to just implement them once in abstraction layers like TableProvider instead of adding separate implementation in each type of object store.
Does this make sense to you? Or am I missing specific features from some object stores that can support native filter push down optimizations at the API level?

@rdettai @houqp After checking DataFusion and delta-rs code, I think it is more natural to have the TableProvider dealing with filters; it's the abstraction over a table, therefore also a suitable entity for table partitions (inferred or user-provided). The laziness of file listing could be achieved by TableProvider and rely on ObjectStore to only listing related files.

I am under the assumption that TableProvider is responsible for parsing the partition parameters from the URI bucket/prefix?partition=year&partition=month and constructing the list object API calls with appropriate prefixes

You need an extra capability of the object store to do that: list the folders. Otherwise you cannot now what partitions are there and build the appropriate prefixes. Maybe this example helps getting more context: https://docs.google.com/document/d/1nKXPvolft1_DuVjVK7ceOx37arCdQLKc_Im-TfZh9dk/edit?usp=sharing

Thanks @rdettai for the detailed google doc write up. I left some comments there :) I think the object store interface proposed in this PR does support listing object keys with partition prefixes, for example: ObjectStore::lsit("bucket/path/date=2018"). But it's missing an important argument delimiter, which is required to make "folder" hierarchy discovery more efficient in object stores.

@yjshen @houqp you are right, I think that maybe with the delimiter argument added to ObjectStore.list(), the TableProvider would be able to get all the information it needs to efficiently prune the partitions in a raw hive partitioning setup! This way we leave the ObjectStore abstraction with only core object store capabilities 😄
I am still wondering how we could replicate the hive partitioning support for TableProviders relative to different file formats (ParquetTable, CsvFile, NdJsonFile). Actually, same would apply for a HiveCatalogTable that could actually store any file format. It makes me feel that something is wrong with how the different abstractions are organized. This is being discussed in #133.

Thanks @rdettai @houqp  for the optional delimiter.
I was thinking list API could deal with multiple partition columns by listing one directory at a time for filesystems (instead of listing leaf files recursively for non-partitioned tables), therefore could achieve a lazy listing when implementing ObjectStore.
After taking both partitioned and non-partitioned tables in one remote storage into consideration, this original list signature would lead to two ObjectStore implementations for one storage system, one for partitioned and one for non-partitioned. It's wired. I like the idea of an optional delimiter because it brings a lot of flexibility.

I am still wondering how we could replicate the hive partitioning support for TableProviders relative to different file formats (ParquetTable, CsvFile, NdJsonFile).

For raw file tables like a folder of parquet, csv and ndjson files, I think the partition discovery and pruning logic should be agnostic to both file formats and object storages. We could implement this logic as a shared module, which gets invoked in ParquetTable, CsvFile and NdJsonFile table providers to resolve the filtered down object list. Then these table providers will pass down the object paths to corresponding format specific physical execution plans.

Actually, same would apply for a HiveCatalogTable that could actually reference various file formats. It makes me feel that something is wrong with how the different abstractions are organized. This is being discussed in #133.

Given that hive catalog manages a collection of tables with their corresponding metadata, I think CatalogProvider is probably the  right abstraction for it. The Hive catalog provider could query hive metastore for a given schema and table name. It can then construct a format specific table provider based on hive metastore response.

Python test failure seems unrelated, could you please retrigger this single GitHub action @houqp ? Thanks!

looks like python test is broken on master

@alamb What's your opinion on the current list_file and list_dir API?

@alamb is on vacation this week :P

Sorry for the delay @yjshen  -- I will review this carefully today.

@alamb What's your opinion on the current list_file and list_dir API?

I think it looks good 👍

We have resolved the python CI failure on master so I am going to merge this PR in as is. Thanks again @yjshen

Thanks again for all the guidance and help @houqp @rdettai @alamb   🎉 .

Amazing work on laying out a solid foundation for IO abstraction in datafusion @yjshen !


Cannot login, gunicorn[xx] trap invalid opcode

Deployed using Docker Compose on a x86_64 Ubuntu 20.04.1 LTS system (5.4.0-58-generic).
Docker Server Version: 20.10.1
Librephotos deploys without any errors
librephotos-db is up-to-date
librephotos-redis is up-to-date
librephotos-backend is up-to-date
librephotos-frontend is up-to-date
librephotos-proxy is up-to-date

Dmesg out logs following, other than that, no errors in the application logs
[23250.331125] traps: gunicorn[319934] trap invalid opcode ip:7fd207d3a817 sp:7fff262843c0 error:0 in _dlib_pybind11.cpython-38-x86_64-linux-gnu.so[7fd2074b8000+91e000]
[23251.048550] traps: gunicorn[319939] trap invalid opcode ip:7fd207d3a817 sp:7fff262843c0 error:0 in _dlib_pybind11.cpython-38-x86_64-linux-gnu.so[7fd2074b8000+91e000]
[23251.513296] traps: gunicorn[319964] trap invalid opcode ip:7fd207d3a817 sp:7fff262843c0 error:0 in _dlib_pybind11.cpython-38-x86_64-linux-gnu.so[7fd2074b8000+91e000]
[23252.211993] traps: gunicorn[319969] trap invalid opcode ip:7fd207d3a817 sp:7fff262843c0 error:0 in _dlib_pybind11.cpython-38-x86_64-linux-gnu.so[7fd2074b8000+91e000]
[23252.675719] traps: gunicorn[319974] trap invalid opcode ip:7fd207d3a817 sp:7fff262843c0 error:0 in _dlib_pybind11.cpython-38-x86_64-linux-gnu.so[7fd2074b8000+91e000]
[23253.383585] traps: gunicorn[319987] trap invalid opcode ip:7fd207d3a817 sp:7fff262843c0 error:0 in _dlib_pybind11.cpython-38-x86_64-linux-gnu.so[7fd2074b8000+91e000]

I can access the Librephotos Frontend on port 3000, I see the login screen, but nothing happens when I enter the User/Pass and click on "Log In". Page doesn't even refreshes, User/pass fields remain populated with the entries I made.
Here are the logs from librephotos-proxy
librephotos-proxy    | <IP_ADDRESS> - - [01/Jan/2021:19:36:35 +0000] "POST /api/auth/token/obtain/ HTTP/1.1" 499 0 "http://jkumar-server.local.lan:3000/login" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.66 Safari/537.36"

EDIT: Adding my cpuinfo
processor	: 0
vendor_id	: AuthenticAMD
cpu family	: 18
model		: 1
model name	: AMD A6-3650 APU with Radeon(tm) HD Graphics
stepping	: 0
microcode	: 0x3000027
cpu MHz		: 2595.092
cache size	: 1024 KB
physical id	: 0
siblings	: 4
core id		: 0
cpu cores	: 4
apicid		: 0
initial apicid	: 0
fpu		: yes
fpu_exception	: yes
cpuid level	: 6
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt hw_pstate vmmcall arat npt lbrv svm_lock nrip_save pausefilter
bugs		: fxsave_leak sysret_ss_attrs null_seg spectre_v1 spectre_v2
bogomips	: 5190.18
TLB size	: 1536 4K pages
clflush size	: 64
cache_alignment	: 64
address sizes	: 40 bits physical, 48 bits virtual
power management: ts ttp tm stc 100mhzsteps hwpstate

Can you share the output from the librephotos-backend container?

Can you share the output from the librephotos-backend container?

Pulled the latest release, and it seems to have resolved the issue.

Pulled the latest release, and it seems to have resolved the issue.


Accessing .toml keys from bridge/helpers/helper.go

As a newbie to golang I have to little experience to figure out how to solve this. I've tried a few different approaches but none of them lets me read out a value from .toml. I've set a bool value in mattermost part of .toml "Slack2MMAttachments".
Most of the other code is done, link renameing the key title for the attachement from Slack titlelink to Mattermost title_link, and links in attachments are stored in Slack format and mattermost seems to be unabled to translate them.
If i can solve this last bit a mattermost-link to slack-link conversion in attachments should be as easy.

You probably want to pass in the boolean value into the helper! Do you have a branch or something I can look at?

Thank you for your reply., Well since I'm not that used to github, I haven't looked into branching and stuff, I've just committed changes from my local repository to my fork of matterbridge.
What I've done now to try to get it to work, is faking a a new config. and logger, to be able to access config.IsKeySet(). I just committed the last few hours work, to my fork.  Haven't even tested the code yet.
Work has been done on bridge/helper/helper.go, bridge/mattermost/mattermost.go, bridge/slack/slack.go bridge/config/config.go, matterclient/messages.go and matterbridge.toml.sample.
Still code all over the place since i''m still working on to get it as i want it, the link handling uis right now directly writen for a special github-bot on slacks messages that gets malformated on MM
My fork

Just implemented your suggestion, sending over a book to the func in helper/helper.go. time to compile and test.

Seems resolved, going to close


missing pylint next fix in qa

API change check
API changes are not detected in this pull request.


Exception causes CORS Error

Whenever an exception is thrown by any of my classes, response loses cors headers. Setting CORS headers manualy within the slim 'errorHandler', solves the issue, but does not feel right.

Middleware can only set headers if it is called. The error in your classes most likely causes middleware stack not to run.

The exception is thrown after cors middleware has been invoked. Somehow response object falls back to its initial state whichs leads to the cors error.

Same issue here. Response headers are losing CORS headers when there's an exception being thrown that should be handled by errorHandler.

Apparently it's a slim issue (App.php method process()):
        // Traverse middleware stack
        try {
            $response = $this->callMiddlewareStack($request, $response);
        } catch (Exception $e) {
            $response = $this->handleException($e, $request, $response);
        } catch (Throwable $e) {
            $response = $this->handlePhpError($e, $request, $response);
        }

It does either one or the other (handle errors or middleware), not both.

@edsonmedina You are correct. Currently only way I know how to handle this is to set the CORS headers again in the error handler. This is framework issue, not middleware issue.

A typical solution for this issue would be saving CORS headers before handling next middleware/Controller. When an unexpected exception occurs it will be handled by an Exception Handler. This handler has to take the saved headers and add them to response.

Also, there is another error handling strategy: final handler (controller) should not throw any exceptions (it just catches them all) so middleware can expect to always have a chance to add the headers. If that's the case then no 'special' storage for CORS headers is needed and the exception handler is not needed to know about CORS existence.

This is for sure not an issue of this middleware.
I had the same issue with ZF expressive as the error handler middleware is the first one and catches all exceptions happened before = means the CORS middleware does not respond anything as a thrown exception stops executing the current program flow until it gets catched.
As a solution you need to handle exceptions twice and the middleware pipeline looks like this:

exception handler (needs to be the first one to handle really all exceptions)
required middlewars for CORS like routing
CORS middleware
second exception handler
all the rest

Now your CORS middleware can act an most exception based error responses except the once hapend before the CORS middleware

As a sidenote. I know some people use exceptions for program flow, for example to return a 404 response. I do not see a 404 as an exceptional situation and tend to use special NotFoundResponse classes instead. This also works as a solution to above described problem with middlewares and exceptions.

@tuupola A developer will have to deal with exceptions in this case or another anyway.
IMHO, a controller should handle all errors from itself and below and returns only responses (2XX, 4XX or 5XX). Otherwise, the controller must know about cors middleware and its needs to add headers, which is odd.

If a response is generated before the CORS middleware is invoked, the headers are added to the response. When an exception is thrown in a middleware stack, it's possible that the exception breaks out of the stack, depending on your setup. As long as you can catch that exception and generate a response before the CORS middleware is invoked, you're all set. So, generally speaking, you can include an exception handler middleware that generates a response before the CORS middleware in the stack. Here's a basic example pipeline:

exception handler (most outer middleware to handle any exception still unhandled)
routing
CORS
exception handler (handle any exceptions from "upstream")
request hander

In a normal request without any exception, the response is generated by (5), (4) is bypassed, (3) adds relevant CORS headers to the response and ultimately the response is rendered.
If an exception is thrown in (5), it's handled by (4) which generates a response based on the exception, then (3) adds the headers.

@marcguyer Could we just pipe CorsMiddleware before ProblemDetailsMiddleware or in your example 4. before 3.
This is example from Zend Expressive project.
$app->pipe(ErrorHandler::class);
$app->pipe(RequestIdMiddleware::class);
$app->pipe(ServerUrlMiddleware::class);
$app->pipe(CorsMiddleware::class);

// Always use Problem Details format for calls to the API.
$app->pipe('/api', ProblemDetailsMiddleware::class);

$app->pipe(RouteMiddleware::class);
...

Yes since ProblemDetailsMiddleware behaves like an exception handler. As long as the response is already produced before it's passed back through CorsMiddleware then it can do it's work on that response.

@marcguyer I just wanted to confirm that there are no any drawbacks for this approach since I'm using it.
It might also be use-full for others since this is quite common problem when building API-s and solution is simple.


docs(showcase): Add miradorliving.com to pages.json

Assisted living + senior care search website: https://www.miradorliving.com/
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

@gioboa and @blakeley  😣

@blakeley do you have third party scripts or useVisibleTask$ in your website?


Bump docker version to support the overlay2 storage driver

https://jira.mesosphere.com/browse/DCOS_OSS-1202 - Support the overlay2 storage driver on DC/OS Docker
Given that the aim of this is to get overlay2, I chose to upgrade to version 1.13.1 for multiple reasons:

It was simple to change the version number and this is the latest version with that property I believe
This matches the currently documented minimum version of the Docker client. I'm not sure if there are conflicts with using newer daemons.

I have tested this by running {{make}} on a Mac with the overlay2 storage driver. {{make}} and {{make postflight}} both complete.

cc @karlkfi

There's a known bug with Docker 1.13 and DC/OS that is fixed on master, but not released yet. It has to do with iptables usage changes in Docker. So I suspect this will break VIPs and/or Overlay networks.

Plan after discussing with @karlkfi on Slack:

Allow a version of Docker to be given
If the version is not in (1.11, 1.13), error
If the version is 1.11 - support aufs, overlay
If the version is 1.13 - support aufs, overlay, overlay2

Default to 1.11

Closing in favour of https://github.com/dcos/dcos-docker/pull/46


Update doc: from single box deployment to "quick deployment"

The current single box deployment is for a single node topology, which is actually not a recommend topology for PAI deployment. We should update this doc to a Mini set recommend topology, i.e. 10 GPU on 2 Master Node and 5 Work nodes.

will address this in #2130


WIP: Testing Farm: enable powertools repo on CentOS

Should fix the 'SUT installation failed' error

OK, @thrix said that this is not possible to do on our/user side and he'll enable the repo by default. Closing


Computational expression docs should mention applicative "and!"

As of F# 5, computational expressions now support applicative behavior via and!.  However, this is not mentioned in the documentation for computational expressions.

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

ID: 247c440d-b855-0df1-f2db-076cd3393a0b
Version Independent ID: 1cbccd4d-1721-e14f-7546-8fa868c965b1
Content: Computation Expressions - F#
Content Source: docs/fsharp/language-reference/computation-expressions.md
Product: dotnet-fsharp
GitHub Login: @cartermp
Microsoft Alias: dotnetcontent

Contributions likely welcome https://github.com/dotnet/docs/issues/21272


Revert "fix(coordinator): recover prover_task when the related object is verified."

Reverts scroll-tech/scroll#1466

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

Project coverage is 53.29%. Comparing base (d163abe) to head (329901f).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1468      +/-   ##
===========================================
+ Coverage    53.27%   53.29%   +0.01%
===========================================
  Files          155      155
  Lines        11774    11770       -4
===========================================
  Hits          6273     6273
+ Misses        5006     5002       -4
  Partials       495      495

Flag
Coverage Δ

database
42.85% <ø> (ø)

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

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


FDG-9546 scrolling  closing the drop down

https://federal-spending-transparency.atlassian.net/browse/FDG-9546
scrolling  closing the drop down
no change in test

Blur handling looks great! I am not quite sure what is causing this, but I noticed some weird behavior was introduced with the search bar. If you type something in and then click the x button, the list of results does not reappear

Nevermind, search issue is unrelated to this ticket!


delete-user: Fix confirmation output

When deleting a single user which is a cluster-admin as well as a dedicated-admin, the confirmation message displays different format of comma-separated users (",travi" and "travi,") for either categories.
While deleting dedicated-admin user:
$ moactl delete user -c rbt-moa-test --dedicated-admins travi
 ? Are you sure you want to delete users ,travi from cluster rbt-moa-test? Yes

While deleting cluster-admin user:
$ moactl delete user -c rbt-moa-test --cluster-admins travi
 ? Are you sure you want to delete users travi, from cluster rbt-moa-test? Yes

This patch fixes it so that commas are trimmed on output.

rebased and updated commit log and PR description


[FLINK-13360][documentation] Add documentation for HBase connector for Table API & SQL

What is the purpose of the change
Add documentation for HBase connector for Table
Verifying this change
without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Thanks @sjwiesman for your review~ Updated.

Thanks @wuchong for your review, updated.


Quit by 'q' / 'Q' doesn't work when redirection is in place

sh metracer.sh -v  class > /tmp/ooo
In this case quit by q won't happen

Fixed in rev. a47bc57bb3a123b967a68d0ab2785165825db95b


Compile error when pass string literal with ':' in it

Compiler finds : symbol without ignoring string literals. Need to replace find method to own method which will support guarding literals

Still exists in function parameters and e.t.c.


[UPDATE ALGORITHM] Add Checks for Infix to Prefix conversion

Description:
I would like to add test cases for the Infix to Prefix conversion algorithm that I previously contributed to. It initially lacks proper testing cases, to ensure that the algorithm is good.
Checklist:

[X] Contributor in GSSoC-ext
[X] Want to work on it

Affected Algorithm:
The algorithm affected is Infix to Prefix/conversion/program.c
Additional Information:
Can this issue be considered a level 2.

@pankaj-bind can you assign me this issue.


Adding basePath to post to ruleset definition

And apparently my editor removed some extra whitespace.

Hi Rich, I have integrated the change into the main branch.

Thanks!


Zemst A is listed as OIS 7,but was published as Eemian (=OIS 5)

Activity
Show details
Write a comment…
Diana Pushkina Mar 18, 2022 12:54 PM
Of course, I agree with you but its not always that the latest publications have the best data, so usually I have kept all the data, esp. since in Pangaea exvel its still given for Z A.


Creating bundles for slides does not work

I tried to create slides as content/slides/talkname/_index.md, just like how posts are structured, but that did not work.

In the above case, it should be content/slides/talkname/index.md rather than content/slides/talkname/_index.md. Then reference slides = "talkname" in the front matter of a content file such as a talk or publication. Refer to Academic and Hugo docs.


natural-id + not-found

https://hibernate.atlassian.net/browse/HHH-17197 - Add check for illegal combo of to-one + natural-id + not-found https://hibernate.atlassian.net/browse/HHH-17196 - Documentation for @NaturalId should be more explicit about non-nullability

Oracle failure has nothing to do with this work.  Merging


Adding user-defined Fresnel wavefront error

cf #1: Is it possible to add wavefront error to a Fresnel wavefront if it is not one of the standard wavefront error classes?  The following code crashes:
import poppy
import numpy as np
import astropy.units as u

if __name__ == "__main__":
    wl = 500e-9
    #create a wavefront
    wf  = poppy.FresnelWavefront(0.01*u.m,
                                 wavelength=wl,
                                 npix=1024,
                                 oversample = 2)
    #add 1 nm surface error through a "screen"
    surf_error = np.random.normal(loc = 0, scale = 1e-9, size = wf.shape)
    #convert to phase
    error = np.exp(2*np.pi*1j/wl*surf_error)
    #next line crashes
    wf *= error

A scan through the API documentation did not suggest a solution.

Never mind, you can do this with the ArrayOpticalElement class, which I did not have with version 0.6.  Here is an implementation below showing that it is doing the right thing
import poppy
import numpy as np
import astropy.units as u

if __name__ == "__main__":
    wl = 500e-9
    #create a wavefront
    wf  = poppy.FresnelWavefront(0.01*u.m,
                                 wavelength=wl,
                                 npix=1024,
                                 oversample = 2)
    #add 1 nm surface error through a "screen"
    surf_error = np.random.normal(loc = 0, scale = 1e-9, size = wf.shape)
    #convert to phase
    error = np.exp(2*np.pi*1j/wl*surf_error)
    #Use the poppy version
    error_p= poppy.ArrayOpticalElement(opd = surf_error)

    print np.allclose(error, error_p.get_phasor(wl*u.m))
    wf *= error_p

Great, glad you figured this out, Michael.
The problem with the first version is that the *= invokes the Wavefront.__imul__ function, which knows how to "multiply" a wavefront by a scalar, an OpticalElement instance, or a coordinate transform, but not a plain ndarray. Wrapping the error into an ArrayOpticalElement fixes that, as you figured out.
The other way you could have done this in your first example is by directly accessing the wavefront data as an ndarray rather than via the Wavefront class:
wf.wavefront *= error

It would be simple enough to extend the Wavefront.__imul__ function to do the right thing if it's handed a complex ndarray with the right dimensions. Do you think that would be a useful enough thing to add in here?
Cleaning up and improving the documentation is on our to-do list for the next release this fall. I'll make sure we include an example along these lines!


Cake.Parallel does not successfully install when using bootstrap

What You Are Seeing?
Cake.Parallel module does not install successfully when using the bootstrap argument.
What is Expected?
Module should install successfully.
What version of Cake are you using?
0.25.0
Are you running on a 32 or 64 bit system?
64
What environment are you running on? Windows? Linux? Mac?
Windows
Are you running on a CI Server? If so, which one?
Yes. TFBuid
How Did You Get This To Happen? (Steps to Reproduce)
In build.cake added the module Preprocessor Directives
#module "nuget:?package=Cake.Parallel.Module&prerelease=true"

Using the bootstrap parameter to install the module fails with an error
build.ps1 --bootstrap

Output Log
2018-02-21T14:13:55.1024598Z Preparing to run build script...
2018-02-21T14:13:57.6963592Z Running build script...
2018-02-21T14:14:11.0563213Z Error: Failed to install module 'Cake.Parallel.Module'.
2018-02-21T14:14:11.1813253Z ##[error]Process completed with exit code 1.

Is that with the verbose flag enabled?

Sorry, below with diagnostic verbosity
PS C:\Repos\Cake> .\build.ps1 --bootstrap -Verbosity Diagnostic
Preparing to run build script...
Running build script...
Verifying assembly 'Cake.Parallel.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Registering module Cake.Parallel.Module.ParallelCakeModule...
NuGet.config not found.
Analyzing C:/Repos/Cake/build.cake...
Installing modules...
  GET https://api.nuget.org/v3/registration3-gz-semver2/cake.parallel.module/index.json
  OK https://api.nuget.org/v3/registration3-gz-semver2/cake.parallel.module/index.json 1129ms
Found package 'Cake.Parallel.Module 0.20.0' in 'C:/Repos/Cake/tools/Modules'.
Package Cake.Parallel.Module.0.20.0 has already been installed.
Successfully installed 'Cake.Parallel.Module 0.20.0' to C:/Repos/Cake/tools/Modules
Executing nuget actions took 29.49 ms
Error: Cake.Core.CakeException: Failed to install module 'Cake.Parallel.Module'.
   at Cake.Core.Scripting.ScriptProcessor.InstallPackages(IReadOnlyCollection`1 modules, DirectoryPath installPath, PackageType packageType)
   at Cake.Commands.BootstrapCommand.Execute(CakeOptions options)
   at Cake.CakeApplication.Run(CakeOptions options)
   at Cake.Program.Main()

Thank you @stephan7878, I'll see what I can dig up. At a glance it's like the NuGetPackageInstaller doesn't handle my package very well.

@stephan7878 Try again now? You the #module directive you provided you should be pulling down a working v0.20.1-rc3

Yes the new version is working for me:
PS C:\Repos\Cake> .\build.ps1 --bootstrap -Verbosity Diagnostic
Preparing to run build script...
Running build script...
Verifying assembly 'Cake.Parallel.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Registering module Cake.Parallel.Module.ParallelCakeModule...
NuGet.config not found.
Analyzing C:/Repos/Cake/build.cake...
Installing modules...
Missing C:\Users\stephanb\.nuget\packages\cake.parallel.module\0.20.1-rc3\cake.parallel.module.0.20.1-rc3.nupkg
  GET https://api.nuget.org/v3/registration3-gz-semver2/cake.parallel.module/index.json
  OK https://api.nuget.org/v3/registration3-gz-semver2/cake.parallel.module/index.json 991ms
Retrieving package 'Cake.Parallel.Module 0.20.1-rc3' from 'nuget.org'.
  GET https://api.nuget.org/v3-flatcontainer/cake.parallel.module/0.20.1-rc3/cake.parallel.module.0.20.1-rc3.nupkg
  OK https://api.nuget.org/v3-flatcontainer/cake.parallel.module/0.20.1-rc3/cake.parallel.module.0.20.1-rc3.nupkg 882ms
Acquiring lock for the installation of Cake.Parallel.Module 0.20.1-rc3
Acquired lock for the installation of Cake.Parallel.Module 0.20.1-rc3
Installing Cake.Parallel.Module 0.20.1-rc3.
Completed installation of Cake.Parallel.Module 0.20.1-rc3
Adding package 'Cake.Parallel.Module.0.20.1-rc3' to folder 'C:\Repos\Cake\tools\Modules'
Added package 'Cake.Parallel.Module.0.20.1-rc3' to folder 'C:\Repos\Cake\tools\Modules'
Added package 'Cake.Parallel.Module.0.20.1-rc3' to folder 'C:\Repos\Cake\tools\Modules' from source 'https://api.nuget.org/v3/index.json'
Successfully installed 'Cake.Parallel.Module 0.20.1-rc3' to C:/Repos/Cake/tools/Modules
Executing nuget actions took 1.2 sec
PS C:\Repos\Cake>

Awesome. I'll release a full 0.20.1 once I resolve some minor build script issues I found while fixing this. Thanks you for the logs.

For the sake of documenting the root cause at the time of this issue being reported the project was built with net462. This may not seem like the issue, especially when 0.20.0 works perfectly fine when pulled down by other means such as paket, but using the --bootstrap command exposed an issue in how the module gets "installed."
The primary culprit was the NuGetContentResolver, primarily where we attempt to find the nearest compatible version available to use. During execution on my windows VM the current version was net46. With this project being compiled against net462 FrameworkReducer.GetNearest('net46', ['net462']) would naturally return null, which naturally results in the failed to install error being returned just down the stack.
On an interesting note, the error provided by @stephan7878 was a false negative. Yes, it did kill the bootstrap process, but the module was successfully pulled down by nuget all the same. Subsequently running the build results in cake correctly picking up the parallization module and executing it.
At the end of the day the problem has been resolved simply by setting the target framework for this project to netstandard1.6. A larger discussion can happen if we want to handle modules a little differently in the ScriptProcessor. The biggest hurdle was having to painfully dig deep into the guts of the NuGetContentResolver to find that the inability to find a "nearest" framework and we could probably use some better logging in that area to expose this.
cc @gep13


fix(contextmenu): uncaught TypeError

SUMMARY
This PR resolves a uncaught TypeError in MixedTimeseries charts with saved metrics.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before
Screencast from 04-24-2024 05:16:25 PM.webm
After
Screencast from 04-24-2024 08:09:58 PM.webm
TESTING INSTRUCTIONS
Create a MixedTimeseries chart with using saved metrics. Without the PR, the context menu (right click) will throw an uncaught TypeError exception. With the PR, the context menu works as expected.
ADDITIONAL INFORMATION

[X] Has associated issue: #28199
[ ] Required feature flags:
[X] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report
Attention: Patch coverage is 25.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 59.23%. Comparing base (a1983e4) to head (da81ff2).
Report is 167 commits behind head on master.

Files
Patch %
Lines

superset/common/query_context_processor.py
0.00%
2 Missing :warning:

superset/connectors/sqla/models.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #28203      +/-   ##
==========================================
- Coverage   67.42%   59.23%   -8.19%
==========================================
  Files        1923     1928       +5
  Lines       75591    75928     +337
  Branches     8443     8443
==========================================
- Hits        50967    44977    -5990
- Misses      22566    28893    +6327
  Partials     2058     2058

Flag
Coverage Δ

hive
49.21% <0.00%> (?)

postgres
?

python
60.99% <25.00%> (-16.95%)
:arrow_down:

sqlite
?

unit
57.73% <25.00%> (?)

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

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

Hey @sowo - sorry this seems to have slipped through the cracks, but if you're able to rebase this to resolve conflicts, I think we can get some more eyes on it and get it merged. Thanks!

Thanks for the quick rebase! Looks like there are some test failures on CI, but I'm not sure if they're related to the change or not.

as far as I can tell, all test failures complain about missing strftime.
(either "AttributeError: 'numpy.float64' object has no attribute
'strftime'" or "AttributeError: 'str' object has no attribute 'strftime'").
I don't think the changes in PR are causing those.
On Tue, Dec 10, 2024 at 10:06 PM Evan Rusackas @.***>
wrote:

Thanks for the quick rebase! Looks like there are some test failures on
CI, but I'm not sure if they're related to the change or not.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/superset/pull/28203#issuecomment-2532891880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVDI5L5ZIMNB7ULZLOUWQD2E5JWPAVCNFSM6AAAAABTL2O7FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZSHA4TCOBYGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

turns out there was an issue in query_context_processor. Tests are passing
now.
On Tue, Dec 10, 2024 at 10:12 PM Wolfgang Sommerer <
@.***> wrote:

as far as I can tell, all test failures complain about missing strftime.
(either "AttributeError: 'numpy.float64' object has no attribute
'strftime'" or "AttributeError: 'str' object has no attribute 'strftime'").
I don't think the changes in PR are causing those.
On Tue, Dec 10, 2024 at 10:06 PM Evan Rusackas @.***>
wrote:

Thanks for the quick rebase! Looks like there are some test failures on
CI, but I'm not sure if they're related to the change or not.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/superset/pull/28203#issuecomment-2532891880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVDI5L5ZIMNB7ULZLOUWQD2E5JWPAVCNFSM6AAAAABTL2O7FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZSHA4TCOBYGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Homepage layout

This branch's purpose was to fulfill:

Create the main part of the homepage that keeps the layout.
Fix #15

Hi @evillaz  ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights
📌 No Linters error ✅
📌 Descriptive pull request✅
📌 Good commit messages ✅
📌 Professional README file ✅
📌 Correct Setup with webpack ✅
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Removed fake_rule_label from precompiled_apple_resource_bundle_impl and using swift_module instead

Why this is needed?
fake_rule_label was as a workaround and I'd like to remove it and use swift_module.
This pr is a continuation of https://github.com/bazelbuild/rules_apple/pull/2335.
If we try to build 2 different targets with same module name - plist generation will fail due to rule name collision as fake_rule_label is used, which was created from swift_module or bundle_name
What has changed?
Using swift_module for compiling assets, plists, storyboards, etc. instead of fake_rule_label. Now, each plist generation action has it's own rule name which has no name collision
NOTE
This PR Requires rules_apple to be at least 3.2.0, so I'll fix tests once it's updated

Thanks, this is a nice cleanup! If this requires a new rules_apple version you'll have to rebase after I merge #864

Thanks, this is a nice cleanup! If this requires a new rules_apple version you'll have to rebase after I merge #864

Sure, I'll keep an eye on it and rebase/fix unit tests once it's ready 🙏

@luispadron how can I trigger CI checks? 🙏 It seems waiting for status for a long time


New solver in coherence

update: see https://lcnr.de/update/2023-12-01, main blocker is non-fatal overflow handling in the new solver


Graphics dimensions not being set correctly in Word

```
.Set of logos
[cols="^2a,^2a,^3a"]
|===
|
image::iso.png[height=100]
|
image::iec.png[width=200]
|
image::itu.png[height=100,width=200]
|===

Fine here's the document:
= SAMPLE: MN Study Report
:docnumber: MN/1001/1
:edition: 1
:copyright-year: 2021
:revdate: 2021-01-10
:published-date: 2021-01-10
:title: SAMPLE: MN Report
:doctype: report
:status: published
:security: Confidential
:docfile: sample-sample.adoc
:mn-document-class: rsd
:mn-output-extensions: xml,html,doc,rxl
:authors: Ronald Tse
:imagesdir: images
:local-cache-only:
:data-uri-image:

.Foreword

Foo

== Introduction

.Set of logos
[cols="^2a,^2a,^3a"]
|===

|
image::iso.png[height=100]
|
image::iec.png[width=200]
|
image::itu.png[height=100,width=200]

|===

[bibliography]
== Bibliography

Archive.zip


Only try to bind if late registration not failed.

Motivation:
We should not try to call bind if registration failed.
Modifications:
Only call doBind0(...) when the registration not failed.
Result:
Not try to to bind if the registration failed.

Cherry-picked into 4.0 (1836f833d5c96df3a4ff6adbe3fb5f5830e5edca) and 4.1 (5ffa3b1c466f8a4c22b1cb96312d2809c653fc1b)

lgtm


Refactor example to be SpringBoot idiomatic

Thanks @AlexeySoshin. I'll get this merged and update the blog post this week.


ErrNoRows not returned on INSERT ON CONFLICT DO UPDATE WHERE...

Describe the bug
ErrNoRows error is not returned when update is skipped during an upsert with PostgreSQL.
Code snippet
Using the following query:
Classes.INSERT(
		Classes.Code,
		Classes.Year,
		Classes.Semester,
		Classes.Programme,
	).MODELS(
		inserts,
	).ON_CONFLICT().ON_CONSTRAINT(
		"ux_code_year_semester",
	).DO_UPDATE(
		SET(
			Classes.Programme.SET(Classes.EXCLUDED.Programme),
		).WHERE(
			Classes.Programme.NOT_EQ(Classes.EXCLUDED.Programme)
		),
	).RETURNING(
		Classes.AllColumns,
	)

where classes has the following schema:
CREATE TABLE classes
(
    id         BIGSERIAL PRIMARY KEY,
    code       TEXT        NOT NULL,
    year       INTEGER     NOT NULL,
    semester   TEXT        NOT NULL,
    programme  TEXT        NOT NULL,
    CONSTRAINT ux_code_year_semester
        UNIQUE (code, year, semester)
);

When a row with the same code, year, semester, and programme fields is going to be inserted:

ON CONFLICT will attempt to DO UPDATE
DO UPDATE will not set the new field because the programme field has the same value according to WHERE clause.
RETURNING returns no rows (destination is nil).

However, when doing QueryContext on the query, no ErrNoRows error is returned even though there is no result.
Expected behavior
Since the update is skipped and no rows are returned from the resulting query, I expect ErrNoRows to be returned but this is not the case. Not sure if this is expected or a bug.

https://github.com/go-jet/jet/blob/33333585e9571ee4375c1a6ee99acff129c801ba/internal/jet/statement.go#L21-L24
Only if the destination is a pointer to a struct ErrNoRows is returned.
If the destination is a slice, you can check if there were any updates, by checking the slice length:
if len(dest) == 0 {
   // no updates
}

Thanks for the reply!
Is this a deliberate design choice for compatibility with multiple database implementations (Postgres, MySQL, etc...)? Because I personally think the behaviour of errors should be type-agnostic for more predictable behaviour.

Yeah, it's a mistake made at the begging and has remained until now. There is a plan to deprecate the current Query and QueryContext and create two new ones, QueryScanOne and QueryScanMany, but there is always some higher priority features. Maybe we can squeeze it into the next release.


Update the string parameter to accept None

The following ops accept "None" instead of None as in PyTorch

[x] div         : https://github.com/tenstorrent/tt-metal/pull/15258
[x] div_bw  : https://github.com/tenstorrent/tt-metal/pull/15173
[x] rdiv        : https://github.com/tenstorrent/tt-metal/pull/15295
[x] rdiv_bw : https://github.com/tenstorrent/tt-metal/pull/15298

Merged , hence closing issue


Add clarity for Private Link on DNS requirement on ESLZ

In ESLZ there is a lack of guidance regarding the creation on Private DNS zone in the connectivity subscriptions.
I also like to add guidance on Azure Firewall provided DNS as proxy setting
As discussed with @victorar, this issue will track the updating of the documentation to detail those points mentioned above

PR created to update CAF documentation.

pr merged into master. Doc publicly available, closing this issue


Open-API Schemas: Client Generation

After recent work by @BeniVF, Skeumorph is able to parse an API specification in the OpenAPI 3.0 format (a.k.a. Swagger), and of generating an http4s client library for it. However, HTTP being a somewhat complicated protocol, not all features are supported from the beginning.
The goal of this over-issue is to keep track of the features that are pending to be supported by Skeumorph / OpenAPI.

[ ] Support for query parameters: elements in query parameters should be included as input parameters of the client method.
[ ] Support for getting a URI from the Location Header of a 201 Created response, usually for  a POST or PUT request, that has created a new entity

Closed by https://github.com/higherkindness/skeuomorph/pulls?q=is%3Apr+is%3Aclosed+label%3A"Open+API+(aka.+Swagger)"


How to enable Test Management in our project?

I see from the closed issue that this should be available -  Is it available upon request? What steps do we need to see it in our Azure DevOps Services project?  (Today is October 31 2019 and it appears that it is available to some accounts.)

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

ID: a8dfb3bf-8cce-7ad8-b2c6-40d6c2b9d441
Version Independent ID: 53e2d39d-6808-70d5-9c26-bc4c5bb109e3
Content: Flaky test management - Azure Pipelines
Content Source: docs/pipelines/test/flaky-test-management.md
Product: devops
Technology: devops-cicd
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow

Azure DevOps Support Bot

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


updates all samples for google/cloud-tools and adds delay to endpoints

see #259

Also adds back the commit for php-cs-fixer:2.0 because that release is finally stable :-/

This is waiting on #263 which fixes a bunch of tests