Producing JAXPR code from a function containing sparse matrix

Description
Getting the JAXPR of the function is not possible.
import jax
from jax.experimental import sparse

M = jax.numpy.array([[0., 1., 0., 2.],
               [3., 0., 0., 0.],
               [0., 0., 4., 0.]])
def f(M):
    return sparse.COO.fromdense(M)

print( f(M) ) # f is working fine

out=jax.make_jaxpr(f)(M) # crash

It raises a jax._src.errors.ConcretizationTypeError:
jax._src.errors.ConcretizationTypeError: Abstract tracer value encountered where concrete value is expected: Traced<ShapedArray(int32[])>with<DynamicJaxprTrace(level=1/0)>
coo_fromdense nse argument
See https://jax.readthedocs.io/en/latest/errors.html#jax.errors.ConcretizationTypeError

I annotated the argument as static but still the same issue
What jax/jaxlib version are you using?
jax v0.2.17
Which accelerator(s) are you using?
CPU
Additional system info
Python 3.8 OS Linux
NVIDIA GPU info
No response

To elaborate: make_jaxpr is similar to jit, in that it evaluates the function for abstract inputs. Becuase your input array is abstract, there's no way to determine the number of specified elements in the output array, which means the function cannot be abstractly evaluated.
To address this, you can statically specify nse:
def f(M, nse=4):
    return sparse.COO.fromdense(M, nse=nse)

print(jax.make_jaxpr(f)(M))

    b:f64[4] c:i32[4] d:i32[4] = coo_fromdense[
      index_dtype=<class 'numpy.int32'>
      nse=4
    ] a
  in (b, c, d) }


W20869 doesn't create manifest correctly

Discovered by archive-ops-134 by @eroux.  When retried, got the same results as the OP.

This is because volume-manifest-tool failed to retrieve a BOM (fileList.json) from the S3 resource. It should have then just got the file list from an s3 list_entries call, but it wasn't doing that.

IIIF Viewer for W20869 shows the works

When I visit the link. click on the scan page image, scroll down a bit, then It seems the manifest is loaded for vol 1 - 4 and then loading is stalled indefinitely for me:

is this a manifest problem or something else?

I did some more testing and confirmed your findings. this bug is still
present. It’s a volume by volume thing. I’m looking at it some more today.
On Fri, Jul 31, 2020 at 9:01 PM Chris Tomlinson<EMAIL_ADDRESS>wrote:

When I visit the link. click on the scan page image, scroll down a bit,
then It seems the manifest is loaded for vol 1 - 4 and then loading is
stalled indefinitely for me:
[image: Screen Shot 2020-07-31 at 7 57 39 PM]
https://user-images.githubusercontent.com/3542294/89090431-79b22900-d368-11ea-85cf-fb71074e4e3f.png
is this a manifest problem or something else?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/buda-base/volume-manifest-tool/issues/28#issuecomment-667442761,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH7LSRNBVQF3EH2PHAVDHPLR6NSN3ANCNFSM4PQNLMNQ
.

--
Jim Katz
Archival Operations Manager
Buddhist Digital Resource Center
1430 Massachusetts Ave FL 5
Cambridge, MA 02138
Visit BDRC Web Library https://www.tbrc.org | Donate to BDRC
https://donate.tbrc.org
<EMAIL_ADDRESS>

@xristy @eroux
I regenerated W20869. All the dimensions.json files are fully populated. I still can't get beyond volume4, just like Chris can't.  I uploaded the dimensions.json of the fifth volume (assuming they're in order from W20869-0334)
0338dims.json.txt
It seems to be correct.
I'll regenerate all the missing volumes. Plz advise if you need more json samples.

Thanks! It seems to work fine now! I restarted the server, I'll look in to the bug that lead to the server being stalled on Monday. The person who reported the problem said this was relatively common, I'll tell you if more examples are reported... but it should all get detected by https://github.com/buda-base/archive-ops/issues/134 anyways.

W00EGS1016299  is having the same issue, even though its BOMS are all good, at least according to their size:
W00EGS1016922.dimensions.zip

Closed by Pull Request #29


Exception when calling an operation inside the preserve_loop_order context

Describe the bug
An exception is raised when putting an operation inside the preserve_loop_order context but outside the loop.
I do not have use cases where I need that but I think it is not supposed to be a problem.
To Reproduce
This is a example demonstrating the issue, simplified from a bigger task.
from pyinfra import state
from pyinfra.operations import server

def second_function(x):
    server.shell(f"echo second_function {x}")

def first_function():
    with state.preserve_loop_order([1, 2, 3]) as loop_items:
        server.shell("echo first_function")

        for x in loop_items():
            second_function(x)

first_function()

--> Preparing operations...
    Loading: tasks/share.py
    [pyinfra.api.operation] Adding operation, {'Server/Shell'}, opOrder=(0, 10, None, 17, 10, None, 11), opHash=5bcebad64f522f61edf549fd77de4ad7ad5ee70e
    [pyinfra.api.operation] Adding operation, {'Server/Shell'}, opOrder=(0, 10, 1, 17, 10, 1, 14, 10, 1, 6), opHash=ce85a058d5f262efd455825e43bb4bac7f4a3e53
    [pyinfra.api.operation] Adding operation, {'Server/Shell'}, opOrder=(0, 10, 2, 17, 10, 2, 14, 10, 2, 6), opHash=99aacc35c5d53e24a8892d8281c8a77ac4cc5bb3
    [pyinfra.api.operation] Adding operation, {'Server/Shell'}, opOrder=(0, 10, 3, 17, 10, 3, 14, 10, 3, 6), opHash=bfa31774ca5eed9d3816da00b4f114f5b8b0c2bb
    [localhost] Ready: tasks/share.py

--> Proposed changes:
    Ungrouped:
    [localhost]   Operations: 4   Commands: 4

--> Beginning operation run...
--> An unexpected internal exception occurred:

  File "/home/myuser/.local/lib/python3.9/site-packages/pyinfra/api/state.py", line 252, in get_op_order
    sorted_line_numbers = sorted(list(line_numbers_to_hash.keys()))
TypeError: '<' not supported between instances of 'int' and 'NoneType'

    [pyinfra_cli.exceptions]   File "/home/myuser/.local/lib/python3.9/site-packages/pyinfra_cli/main.py", line 223, in cli
    _main(*args, **kwargs)
  File "/home/myuser/.local/lib/python3.9/site-packages/pyinfra_cli/main.py", line 600, in _main
    run_ops(state, serial=serial, no_wait=no_wait)
  File "/home/myuser/.local/lib/python3.9/site-packages/pyinfra/api/operations.py", line 418, in run_ops
    for op_hash in state.get_op_order():
  File "/home/myuser/.local/lib/python3.9/site-packages/pyinfra/api/state.py", line 252, in get_op_order
    sorted_line_numbers = sorted(list(line_numbers_to_hash.keys()))

    [pyinfra_cli.exceptions] TypeError: '<' not supported between instances of 'int' and 'NoneType'

--> The full traceback has been written to pyinfra-debug.log
--> If this is unexpected please consider submitting a bug report on GitHub, for more information run `pyinfra --support`.

Expected behavior
no exception
Meta
v2.1

Now fixed as of v2.2!


Merge dev/route-generator into master

Should be the same as it was when I demo'd it to you, only made ember-get-config & ember-fetch dependencies instead of devDependencies since in centrale vindplaats it wouldnt work otherwise

Also would it be able to publish this on npm ? Since in centrale vindplaats get the package straight from the github repo instead of npm :)


Wetty docker container issue

Issue Description
After using docker compose to start the wetty and ssh containers accessing  with http://:3000/wetty and providing term/term as user name/pwd is resulting in
sh: -p not found
The docker compse logs show automatic disconnection message
wetty-ssh_1  | Accepted password for term from <IP_ADDRESS> port 54440 ssh2
wetty-ssh_1  | Received disconnect from <IP_ADDRESS> port 54440:11: disconnected by user
wetty-ssh_1  | Disconnected from user term <IP_ADDRESS> port 54440
wetty_1      | {"code":127,"pid":133,"level":"info","message":"Process exited","label":"Wetty","timestamp":"2020-12-27T09:59:06.572Z"}
Screenshot

Noticed same issue on non docker environment as well. I have an ansible script to setup emacs and wetty on an Ubuntu box. When ansible updated wetty I got the same error as OP with 2.0.3, downgrading to 2.0.2 fixes it.

Noticed same issue on non docker environment as well. I have an ansible script to setup emacs and wetty on an Ubuntu box. When ansible updated wetty I got the same error as OP with 2.0.3, downgrading to 2.0.2 fixes it.

Ive pushed a fix to master that should address the issue. if you can verify the fix ill be able to release it with version 2.0.4

Ive pushed a fix to master that should address the issue. if you can verify the fix ill be able to release it with version 2.0.4

Hi, the issue is also visible with the Yunohost package, and the latest commit did not solve the issue.

Hi, the issue is also visible with the Yunohost package, and the latest commit did not solve the issue.


Hard to understand

Please can we have more detail about points 1 and 2

Navigate to the pipeline in the web  ( well the web is a pretty big place ) could we have the url ?
How do I switch branches for a pipeline?

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

ID: ee4ec9d0-e0d5-4fb4-7c3e-b84abfa290c2
Version Independent ID: 3e2b80d9-30e5-0c48-49f0-4fcdfedf5eee
Content: Resources - Azure Pipelines
Content Source: docs/pipelines/process/resources.md
Product: devops
GitHub Login: @vtbassmatt
Microsoft Alias: alewis

https://stackoverflow.com/questions/52636120/how-do-i-switch-the-default-branch-in-my-azure-repository

For step 2
"Switch the default branch to he branch that includes the new service endpoint reference"
I had trouble finding where to do this.
Is the "default branch" the same thing as the repository default branch>
I updated the stack overflow link to show where I found the setting.

Gotcha! Hopefully my new wording clears things up that this is about the Pipelines branch, not the repo default branch:

Navigate to the pipeline in Azure Pipelines.
Switch the pipeline's default branch to the branch that includes the new service endpoint reference. For example, if you've made a change in a branch called features/add-resource, then switch the pipeline's default branch to features/add-resource.
Save the pipeline.
Revert back to the prior default branch and save the pipeline again.

Can  you give more details about where to make the changes in the UI
For example
2a Navigate to the Get Sources Task in the Pipeline. You should be able to see "Default branch for manual and scheduled builds" in the right panel.


style: rename conflict check variable for clarity

Note: depends on matching change in UI

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


XML declaration in SAML Authentication Requests

Describe the issue:
I am using WSO2IS v.5.9.0 and it is configured with ADFS as federated IDP. When creating the authentication request to ADFS, WSO2IS adds the XML declaration in the request (). This can cause an error on ADFS discovery service, how can the XML declaration can be removed?
How to reproduce:
Configure WSO2IS v.5.9.0 with a SAML federated IdP and trace the authentication request from WSO2IS.
Expected behavior:
A clean authentication request without the XML declaration.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version:  IS 5.9.0
OS: CentOS
Database: PostgreSQL
Userstore: LDAP

Hi @hefesita
Would you mind providing reference that XML declaration cause issue (as you mentioned) in ADFS discovery.
Ideally any service consumes XML s supposed to able to handle XML declaration.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


grype: 0.24.0 -> 0.26.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/anchore/grype/releases.
meta.description for grype is: "Vulnerability scanner for container images and filesystems"
meta.homepage for grype is: "https://github.com/anchore/grype"
meta.changelog for grype is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/8hg267pzdkn1g1p8gs8mqvaymqhwzryr-grype-0.26.1/bin/grype passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 0.26.1 with grep in /nix/store/8hg267pzdkn1g1p8gs8mqvaymqhwzryr-grype-0.26.1
found 0.26.1 in filename of file in /nix/store/8hg267pzdkn1g1p8gs8mqvaymqhwzryr-grype-0.26.1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
grype

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8hg267pzdkn1g1p8gs8mqvaymqhwzryr-grype-0.26.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A grype https://github.com/r-ryantm/nixpkgs/archive/850180dcb2cdfd2c684abb0044ad19308e6cc00c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8hg267pzdkn1g1p8gs8mqvaymqhwzryr-grype-0.26.1
ls -la /nix/store/8hg267pzdkn1g1p8gs8mqvaymqhwzryr-grype-0.26.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

grype

Maintainer pings
cc @fabaff @kalbasit for testing.

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

1 package built:

grype


Radio question defaults for first option

Describe the bug
When families try to apply to the ORCA middle school opportunity program they are asked about their income. When families select the last option on the radio question for "Household income" >> "Not income eligible" the form saves it as the 1st option "Family size 2, annual income $32,000". I deleted the radio question options, saved, and re-published and the issues doesn't seem to be there anymore. But thought I'd add a bug in case this is something to look into a big more
To Reproduce
Steps to reproduce the behavior:

Go to "ORCA Middle School Opportunity"
Complete the question for household income "Not income eligible"
Select "Next"
Complete the rest of the form
Review the form as see that the answer to household income saved as "Family size 2, include $32,000"

Screenshots
Attached
Additional context
I deleted all answers and re-added them and don't see the issue any longer.

I am unable to reproduce this bug -- let's keep a lookout to see if this keeps happening. Going to close this for now.

Thanks, Erin! I haven’t received any more emails about the issue.
[Title: City of Seattle logo]Elise Kalstad
She/Her/Hers
City of Seattle, Innovation & Performance
206-256-5515 |<EMAIL_ADDRESS>Learn more about Innovation & Performance: seattle.gov/innovation-performancehttps://www.seattle.gov/innovation-performance
From: Erin Ettenger @.>
Sent: Tuesday, November 09, 2021 9:54 AM
To: seattle-uat/civiform @.>
Cc: Kalstad, Elise @.>; Author @.>
Subject: Re: [seattle-uat/civiform] Radio question defaults for first option (Issue #1675)
CAUTION: External Email
I am unable to reproduce this bug -- let's keep a lookout to see if this keeps happening. Going to close this for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://protect2.fireeye.com/v1/url?k=95d52773-ca4e1e38-95d50fc3-867c6b071c6f-4474c729159a93cd&q=1&e=77ff3d95-ddf7-446e-a369-8b61b5e1f2a2&u=https%3A%2F%2Fgithub.com%2Fseattle-uat%2Fciviform%2Fissues%2F1675%23issuecomment-964439038, or unsubscribehttps://protect2.fireeye.com/v1/url?k=f148ff69-aed3c622-f148d7d9-867c6b071c6f-83c49837839a3810&q=1&e=77ff3d95-ddf7-446e-a369-8b61b5e1f2a2&u=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUQYIY7SGEDVUULZI4Q5ED3ULFU5FANCNFSM5HO75KCQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://protect2.fireeye.com/v1/url?k=0634eefd-59afd7b6-0634c64d-867c6b071c6f-15bf47424a063b72&q=1&e=77ff3d95-ddf7-446e-a369-8b61b5e1f2a2&u=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub.


Update TDS error handling to support special characters

Description

This commit modifies TDS error message handling to add support for special characters such as emojis
An earlier commit introduced error handling when users attempt to create a procedure / function with the same name as one that is already defined. In recent testing, we found that if a user attempts to recreate an existing procedure that contains certain special characters, there will be a TDS error.

Issues Resolved
Task: BABEL-4972
Test Scenarios Covered

Use case based -

-- Initialize Procedure
create proc [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##] @in INT AS SELECT @in;
go

-- Expect error for duplicate procedure
create proc [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##] @in INT AS SELECT @in;
go
~~ERROR (Code: 2714)~~

~~ERROR (Message: Function '%%#%@$^$姓氏すず🤬🤯86d3db6066be0016789ff9cd274595b8' already exists with the same name)~~

-- Expect error for duplicate function
create function [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##] (@input int)
returns varchar(250)
as begin
    return "test"
end
go
~~ERROR (Code: 2714)~~

~~ERROR (Message: Function '%%#%@$^$姓氏すず🤬🤯86d3db6066be0016789ff9cd274595b8' already exists with the same name)~~

-- Initialize View
create view [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##]
as
    select 1
go

-- Expect error for duplicate view
create view [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##]
as
    select 1
go
~~ERROR (Code: 2714)~~

~~ERROR (Message: relation "%%#%@$^$姓氏すず🤬🤯86d3db6066be0016789ff9cd274595b8" already exists)~~

create table [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##] (
    col1 int
);
go
~~ERROR (Code: 2714)~~

~~ERROR (Message: relation "%%#%@$^$姓氏すず🤬🤯86d3db6066be0016789ff9cd274595b8" already exists)~~

-- Cleanup
drop view [%%#%@$^$姓氏すず🤬🤯🫣🤗🫡🤔🫢🤭き,😀 鈴木##]
go

Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 72.819%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
41858

Relevant Lines:
57482

💛  - Coveralls


[ISSUE #2061] Init websocket integration test

About #2061 .

Codecov Report

Merging #2187 (dac43db) into master (02b2999) will decrease coverage by 0.62%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2187      +/-   ##
============================================
- Coverage     60.82%   60.20%   -0.63%
+ Complexity     4079     4059      -20
============================================
  Files           607      607
  Lines         18097    18099       +2
  Branches       1734     1733       -1
============================================
- Hits          11008    10896     -112
- Misses         6191     6312     +121
+ Partials        898      891       -7

Impacted Files
Coverage Δ

...rg/apache/shenyu/admin/utils/ThreadLocalUtils.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...apache/shenyu/admin/transfer/MetaDataTransfer.java
34.56% <0.00%> (-65.44%)
:arrow_down:

...ache/shenyu/client/tars/common/dto/TarsRpcExt.java
0.00% <0.00%> (-30.77%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
64.70% <0.00%> (-29.42%)
:arrow_down:

...enyu/client/tars/TarsServiceBeanPostProcessor.java
33.82% <0.00%> (-22.06%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-18.45%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...in/response/strategy/NettyClientMessageWriter.java
69.56% <0.00%> (-4.35%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
58.94% <0.00%> (-4.22%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
20.43% <0.00%> (-3.65%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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


Token tab balances too precise

Token tab on the /account page should only show 3 decimal places (possibly including more precision on hover)
https://www.teloscan.io/address/0x160505F3dfD1cb58B91e322c828Ae0F74c043c3C#tokens -> STLOS

Acceptance criteria

The tokens precision should be capped at 4 decimals
The tokens full number should be displayed as a tooltip
If its zero, we'll continue to show 0.0
Trim zeros whenever possible

Fixed in #227


Default the server's JAVA_HOME to java.home of the harness, if it isn't set

I can't create an environment where I can observe the problem you were seeing but I know that you had a test which exposed this, in your IDE.  Can you verify that this resolves the problem and then merge if it does?

For reasons I don't understand my IDE is now setting JAVA_HOME for me, and so it's not reproducible for me any more.  I think the patch is 'right' in principle still, and I don't see harm in it.  Up to you if you want to merge.


Timer

Topline changes:

Added basic timer to PageRevision state during Revision (potential todo - I think this can be modularised with a jazzy pure function & made slightly more accurate with Date.now() but this currently works)
Added ResultsContainer component to display results of Revision.
Styling :tada:
Renamed various files to match updated component architecture (e.g. HistoryCaseRevision.functions.test.js now PageRevision.functions.test.js)

Codecov Report

Merging #120 into master will increase coverage by 4.69%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
+ Coverage   94.25%   98.94%   +4.69%
==========================================
  Files          13       15       +2
  Lines          87       95       +8
  Branches        7        8       +1
==========================================
+ Hits           82       94      +12
+ Misses          5        1       -4

Impacted Files
Coverage Δ

src/components/TopBar/TopBar.js
100% <ø> (ø)
:arrow_up:

src/components/Pages/Revision/RevisionContainer.js
100% <ø> (+20%)
:arrow_up:

src/utils/RevisionPage.functions.js
100% <100%> (ø)

src/utils/timerFormat.js
100% <100%> (ø)

src/components/Pages/Revision/ResultsContainer.js
100% <100%> (ø)

src/components/TopBar/CompleteButton.js
100% <100%> (+33.33%)
:arrow_up:

src/app.js
100% <100%> (+2.63%)
:arrow_up:

src/components/Pages/Revision/RevisionPage.js
92.85% <80%> (+12.85%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Customizing Opro::Oauth::TokenController

Hello!
Currently I am writing an article about opro (probably you've seen it on Sitepoint's Trello board). It seems that TokenController is not meant to be customizable, however what I wanted to do is to return a bit more information when presenting a token, for example, some info about a user who is authenticating. https://github.com/opro/opro/blob/ecffe22e777bb9cacfedd55d059a2ca3cb163a05/app/controllers/opro/oauth/token_controller.rb#L15 Like, you know, when authenticating via Twitter we get a pretty big hash of various data about a user apart from token (name, avatar, followers count etc). I could've created some custom controller action and after receiving a token just make individual request there, but that seems a bit cumbersome to me.
Maybe you could recommend something? I'd be very grateful. :)

Can't we just do something like this https://github.com/opro/opro/blob/master/lib/opro/rails/routes.rb#L10 to allow customizing
mount_opro_oauth controllers: {oauth_token: 'oauth/token'}

Sorry for radio silence, i've been pretty busy.
You can see an example of a controller being overwritten https://github.com/opro/opro_rails_demo/blob/master/app/controllers/oauth/client_apps_controller.rb and https://github.com/opro/opro_rails_demo/blob/41d7795bbd9d4d3604bc084d040ee3e32b8562b4/config/routes.rb#L5
So, yes you would need to over-write the controller action that returns data, to extend it with some custom json.
Here is where we are building the json and returning it after a successful auth https://github.com/opro/opro/blob/ecffe22e777bb9cacfedd55d059a2ca3cb163a05/app/controllers/opro/oauth/token_controller.rb#L9-L23 (I think).
It would be hard to change that hash without re-implementing all of the logic. If you wanted to modify opro to make this easier, the easiest way would be to put that hash generation in another method like generate_access_token_hash and then you could easily extend it.
class MyCustomTokenController < Opro::Oauth::TokenController
  def generate_access_token_hash(auth_grant)
    (super auth_grant).merge(email: "whatever", username: "whatever")
  end
end

Is that kinda what you're looking for? That would require a change and cutting a new release, but it seems pretty easy.
I'm a bit curious why you picked opro/ Most people tend to favor https://github.com/doorkeeper-gem/doorkeeper.

@schneems Thank you for detailed explanations :) Yeah, I understand how this all can be redefined, but currently I haven't found a way to actually redefine the route to point to a new TokenController. Therefore I submitted this PR https://github.com/opro/opro/pull/54 The basic idea is that anyone can point route to a new token controller and then redefine the .json view as needed. For me the befit of using a separate .json view is that I can only change it without touching controller's method at all (and without introducing any additional methods) for example like this:
class TokenController < Opro::Oauth::TokenController
  def create
    super
    render 'auth/create' # points to my custom view
  end
end

Tomorrow I am going to update my source code for the article and demonstrate that idea a bit more clearly.
Of course I've heard about Doorkeeper but as long as my goal is to cover various solutions, I am writing about everything that seems viable to me :) Probably the article will be composed of two parts, I'll send a link then.

Okay, so here is how can it work.

I create a new "server" app and redefine a route to point to my new TokenController https://github.com/bodrovis/Sitepoint-source/blob/master/Auth_Provider_with_Opro/OproServer/config/routes.rb#L4
I create a new TokenController. No methods have to be redefined because in my case I only need a custom view https://github.com/bodrovis/Sitepoint-source/blob/master/Auth_Provider_with_Opro/OproServer/app/controllers/oauth/token_controller.rb#L1
I create a custom view and add any fields as necessary https://github.com/bodrovis/Sitepoint-source/blob/master/Auth_Provider_with_Opro/OproServer/app/views/oauth/token/create.json.jbuilder#L5

The whole set up (server and client) is here https://github.com/bodrovis/Sitepoint-source/tree/master/Auth_Provider_with_Opro. To me this seems pretty convenient, however I might've missed something.


Tcandc- Wrong job locations

ENVIRONMENT: production
URL: https://scrapers.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
1.Open the URL in the browser.
2. Search for "tcandc" Company
3. Click on "Run Scraper" button
4. Click on the "tcandc" careers button.
5. Check the job locations.
Expected results
Job locations should be the same on Scrapers-UI as on the company website.
Actual results
Job locations is displaying in Romania, however on the careers page location is in Budapest.
Attachments:

Retested, all is ok.


Add styling for collection/object content

Is your feature request related to a problem? Please describe.
Additional styling is needed for the collection and object content.
Describe the solution you'd like
Add styles to:

Show lines linking child components to parents
wrap objects in border with parent

Can we also throw in left-aligning the title with the other content as part of this issue?


Crash on mobile sites

I have implemented baguette box on a one of our sites http://www.lbar.com, and it works for some of the smaller profiles, but when we start to use more pictures baguette box will crash on mobile devices.  An example to look at is http://www.lbar.com/property/2001-Winchester-Road-Paris-KY-40361_1519425.html.  I have tried lowering the preloader to 0, but I am still crashing.

Another example is http://www.lbar.com/property/2374-Walnut-Grove-Lane-Lexington-KY-40509_1502572.html t looks like safari can't handle the memory load all at once.

Thanks for reporting that. I'll look into it soon. Could you just provide more info about the issue? On which mobile devices? Which OS, browser? How the crash looks like?

Sure thing.  It works fine on android, but will crash on iOS in both safari and chrome.  In chrome the browser completely crashes, and in safari it to reloads the page with the error "A problem occured with this webpage so it was reloaded."

I have an issue on iOS as well. There is no crash, to the page or browser. The plugin displays the gallery overlay, but the images don't ever load. The spinner is all I can see. Currently I have about 15 galleries on a single page. If I cut that number dramatically (to 2 or 3) everything works as it should.
N.B. works fine on Android/Desktop - only iOS is the problem

Hi, sorry I neglected this issue, but I wasn't able to reproduce the crash, as I don't have access to any iOS device.
One of the possible solutions (at least for @snoogans235) would be removing image nodes while the user moves far enough from them. This could reduce memory consumption.
As for @dmcaodha problem, I'm not quite sure. Can you perform remote debugging and see whether the console outputs any errors?

@snoogans235 does this issue still happen with the latest release?

we had to remove it from our site, so I am not sure.

Ok, then I'm closing the issue as not reproducible.

It is perfectly reproducible see there (this is not spam)

Ok, I'll look into that.

Same here on Safari (macOs/iOs): https://immobilien-schrammek.de/immobilien-detail/neu-und-hochwertig-top-ausgestattetes-einfamilienhaus-in-idyllischer-lage.html


DocumentSession.ForTenant(...).Events should return EventStore instead of QueryEventStore

When you want to query documents for all tenants, you can do so by using .AnyTenant() in your query. If you then want to store events for the respective tenants, you currently have to open a new session for every tenant because it is not possible to store events in the event store returned by ITenantOperations.
var documents = await session.Query<Document>().Where(x => x.AnyTenant()).ToListAsync();

foreach(var document in documents)
{
    session.ForTenant(document.TenantId).Events.Append(new Event()) // This is not possible since session.ForTenant(...).Events returns an IQueryEventStore instead of an IEventStore
}

While it is possible to downcast the IQueryEventStore to IEventStore, it is not safe to do so since Marten will query mt_streams using the tenant id from the parent session when calling session.SaveChanges().

Gonna rephrase this a bit, 'cause you're in tactical "how do I solve my immediate problem" and we need to think strategically about the API. Can't introduce a breaking change to the API until 6.0.


Namespace should be captured as a column in the database

Currently capture pipeline ID as tablename (e.g 'test_pipe') but the namespace (e.g. 'namespace: test') should also be captured into the database.
Example yaml:
Sample output:
pipeline_id: test_pipe
project:
  number_of_things:
    type: integer
    description: "Number of things"
samples:
  percentage_of_things:
    type: number
    description: "Percentage of things"

Sample config.yaml:
namespace: test
record_identifier: sample4
schema_path: drc_sample_output_schema.yaml
database:
  dialect: postgresql
  driver: psycopg2
  name: pipestat-test
  user: postgres
  password: pipestat-password
  host: <IP_ADDRESS>
  port: 5432

To Do:
pipeline_id from output config is stored in  psm.namespace
However, we also store the name space from the db config file here: psm["_config"]["namespace"]} which may be different than the pipeline_id. This value is placed in a separate column in the sample table under 'namespace'.
Recommend refactoring either of these to be less confusing for future maintenance.

the namespace doesn't correspond to the pipeline id, it corresponds to the project name

Confirm that namespace/projectname is also being placed inside the project table


React app no longer works

The React application no longer functions, seems to be related on the surface to the schema for the authorization object changing (scope is not passed correctly, because it moved in the schema). There may be problems related to no more third-party cookies, and refresh tokens for multiple access tokens do not work.

My bad, it seems what I was looking is ahead of the master but not updated for two years and not in the master.

Oh yeah, and maybe stop linking to this blind branch from the auth0-react documentation :)


Added dblog viewer

merging this branch since it's approved (as discussed in this morning's standup)


Fix a small typo in an error check

Just a tiny fix - wrong dtStatus variable was used here.

Closing until I can update this with feedback from @grahamboree.


Use a vector to indicate the output links for ExecutionStage

Which issue does this PR close?

Closes #144.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Hi @thinkharderdev, could you help review this PR?

Hi @andygrove, could you help review this PR?

We can close this, since it's already been merged to the main branch.


cannot install using home-brew on MacOS10.15.7 with Xcode 12.4

I get the message:
Traceback (most recent call last):
brew extract or brew create and brew tap-new to create a formula file in a tap on GitHub instead.: Invalid usage: Non-checksummed download of swiftplantuml formula file from an arbitrary URL is unsupported!  (UsageError)

Damn, a couple of days ago this was working :( might be homebew 3.0.0 or I have the check-sum wrong. I will check
Please try to build from source in the meantime. Clone the repo, cd into cloned folder and then run brew install --build-from-source swiftplantuml.rb. This works for me on MacOS 11.1


How to access an entry of a Tensor in mshadow when using GPU mode?

I want to create a new operator to implement hierarchical softmax by writing C++ code. At the moment, the operator works fine in CPU mode, but will crash in GPU mode. One cause of this issue is that I need to access an entry in a Tensor, i.e.,
Tensor<xpu, 2, DType> label = in_data[hsoft::kLabel].get<xpu, 2, DType>(s);
...
if(label[m][n]==2.0){//crashes here
....
}
....
So what is the correct way to access one entry in GPU mode? Thanks.

You can't directly access memory on GPU.

Yes. Finally solved by copying data to host.


*: specify how to pass runbook link in alert

A bit of clarification on how to pass link in alerts.
Closes MON-1738

/lgtm


Update needs-sme-stale-check.yaml

Why:
The workflow has this error

What's being changed:
Removing extra ' in L17, col9

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Opened in internal repo as this file cant be modified in the docs repo 😭


Missing information or link

Team, why does this page not link to information about the cost of adding additional storage to SharePoint?  This is fundamental to what the page is discussing.

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

ID: b31aa550-3d84-95b8-6a74-b3e7ad1cbfe9
Version Independent ID: 795391d3-aced-e2f3-f52d-034e14fe1aee
Content: Add storage space for your subscription
Content Source: microsoft-365/commerce/add-storage-space.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@Astartes74 Thank you for your feedback. This page is accessed globally and the price differs for every region. If the Admin navigates as per the steps provided in the document, they will be able to view the exact price for their tenant/region.
Hope this helps!
@yogkumgit There are no changes required in the document and the issue can be closed.
Thanks
Sri


The supabase storage api does not parse the RANGE http header

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
Supabase supports RESTful API to access its storage.
For upper layer applications, which require range read for inner bytes, they typically attach a RANGE header in the GET request for the object in supabase storage.
However, currently the supabase storage API does not parse the RANGE header, working as if the RANGE header is not attached.
We found out that although this uses the RANGE header, but the inside of it does not.
getAsset(request: FastifyRequest, options: RenderOptions) {
    return this.backend.getObject(options.bucket, options.key, options.version, {
      ifModifiedSince: request.headers['if-modified-since'],
      ifNoneMatch: request.headers['if-none-match'],
      range: request.headers.range,
    })
  }

  async getObject(
    bucketName: string,
    key: string,
    version: string | undefined
  ): Promise<ObjectResponse> {
    const file = path.resolve(this.filePath, withOptionalVersion(`${bucketName}/${key}`, version))
    const body = fs.createReadStream(file)
    const data = await fs.stat(file)
    const { cacheControl, contentType } = await this.getFileMetadata(file)
    const lastModified = new Date(0)
    lastModified.setUTCMilliseconds(data.mtimeMs)

    const checksum = await fileChecksum(file)

    return {
      metadata: {
        cacheControl: cacheControl || 'no-cache',
        mimetype: contentType || 'application/octet-stream',
        lastModified: lastModified,
        // contentRange: data.ContentRange, @todo: support range requests
        httpStatusCode: 200,
        size: data.size,
        eTag: checksum,
        contentLength: data.size,
      },
      body,
    }
  }

To Reproduce
Previous discussion on this issue is here.
To reproduce this, you can simply read an object from supabase with RANGE header, and the RANGE header will not work as expected.
Expected behavior
The supabase storage API should parse the RANGE header and hence support range read.

More specifically, Supabase Storage SaaS does not support range reading (I don't know what's the underlying storage service underhood).

I believe that the range header is not supported on the File System Storage just yet. The S3 Backend adapter should support it
Open for contributions on this

This is now fixed by https://github.com/supabase/storage/pull/358


ASK: "#" symbol make the rest of handlebars code goes white blank

Hello, I am sorry if I ask a very simple thing, but I couldn't find the solution. I tried to use handlebars for the template of generating PDF along with Puppeteer. Everythings went fine until I have to display some special characters in my template. If I directly use this code in my template.hbs:

      My Special Characters: #`~!@$%^&*()_-+={}[]\|;:'",<.>?/

   {{!-- other codes --}}

everythings after # not rendered. Only white blank. But, if I remove the #, everythings displayed. Thats the case if the # is static.
Another case is if # symbol presents in a string variable. If I have a variable:
const myString = '#`~!@$%^&*()_-+={}[]\|;:'",<.>?/';
and passed to template.hbs and displayed like this:

{{{myString}}}
 -- using triple-stash to avoid myString being escaped if I use only double-stash. It will also display blank spash after the # symbol. But everythings fine if I remove the #.
I've read https://handlebarsjs.com/expressions.html which mention that # is whitespace. Then, any work around for my problem, please? I need to display this # symbol.
Thank you in advance.

If you refer to the section

Identifiers may be any unicode character except for the following: Whitespace ! " # % & ' ( ) * + , . / ; < = > @ [ \ ] ^ ` { | } ~

Then this should not be your problem: If means that "whitespace"  and the other mentioned characters (including #) are not allowed as part of an expression (i.e. {{prop#ertry}} is not allowed).
The code
const myString = '#`~!@$%^&*()_-+={}[]\|;:'",<.>?/';

does not work, because you have to escape the ' character in your string.
const myString = '#`~!@$%^&*()_-+={}[]\|;:\'",<.>?/';

in any case, I could not reproduce this behavior: Please create a jsfiddle using the following template:
https://jsfiddle.net/0L92ufpv/

I've open and try the snippet based on your template. Everythings fine on the jsfiddle. So, I want to try to share some of my code of the template and how I pass the parameter.
<!DOCTYPE html>

  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <meta http-equiv="X-UA-Compatible" content="ie=edge">

TITLE

              header1

              header2

              header3

            {{#each sampleData as |dataVariable|}}

                {{dataVariable.value1}}

                {{dataVariable.value2}}

                {{dataVariable.value3}}

            {{/each}}

Above code, is lies in a file named myTemplate.hbs (in .hbs format, not in .html. One of my reason is to get the for loop feature). And here, is how I pass the parameter:
export const pdfMaker = async (): Promise<any> => {
  try {
    const browser = await puppeteer.launch({
      headless: true,
      // another config
    });

    const compile = async (data: any) => {
      const filePath = path.resolve(`${process.cwd()}/path/to/template/myTemplate.hbs`);
      const html = fs.readFileSync(filePath, 'utf8');
      return handlebars.compile(html)(data);
    };

    const page = await browser.newPage();

    const sampleData = [{
      value1: '1-1',
      value2: '1-2',
      value3: '1-3',
    }, {
      value1: '2-1',
      value2: '#2-2',
      value3: '2-3',
    }, {
      value1: '3-1',
      value2: '3-2',
      value3: '3-3',
    }];

    const data = {
      sampleData,
    };

    const content = await compile(data);

    await page.goto(`data:text/html;charset=UTF-8,${content}`, {
      waitUntil: ['domcontentloaded', 'networkidle0', 'load', 'networkidle2'],
    });

    await page.emulateMedia('print');

    const pdf = await page.pdf({
      format: 'A4',
    });

    await browser.close();

    // code
  } catch (e) {
    // code
  }
};

above code, with # presence in #2-2 will produce:

if I remove the #, or even change it to other special character, like this:
const sampleData = [{
      value1: '1-1',
      value2: '1-2',
      value3: '1-3',
    }, {
      value1: '2-1',
      value2: `~!@$%^&*()_-+={}[]|\:;"'<,>.?/2-2`,
      value3: '2-3',
    }, {
      value1: '3-1',
      value2: '3-2',
      value3: '3-3',
    }];

it will produce:

or, if I change:

TITLE

into

#TITLE

the result is only a blank page. Is this problem with handlebars file that I create? or, whats wrong here? Btw, thank you for your fast response.

I think your problem is with this part
await page.goto(`data:text/html;charset=UTF-8,${content}`, {
      waitUntil: ['domcontentloaded', 'networkidle0', 'load', 'networkidle2'],
    });

You should use encodeURIComponent to encode you content into a data-url
await page.goto(`data:text/html;charset=UTF-8,${encodeURIComponent(content)}`, {
      waitUntil: ['domcontentloaded', 'networkidle0', 'load', 'networkidle2'],
    });

Ah, thank you so much!

thank you!!


Fix many tests

Remove use of Task.Wait() and Task.Result to use await or GetAwaiter().GetResult() to ensure exception stacks are propagated.
Fix Shake-128 and Shake-256 digest sizes and tests to align with new BouncyCastle behavior of 32 and 64 bytes per digest.
Fix DAG API call path matching to an underlying object's properties to account for the CBOR library's name canonicalization algorithm by using a case-insensitive comparison on the property name.

This does not resolve all test errors. 'Tar' querying tests are still broken as are almost all integration-style tests that connect over a local port.


Use copy-pr-bot

This PR replaces the copy_prs functionality from the ops-bot with the new dedicated copy-pr-bot GitHub application.
Thorough documentation for the new copy-pr-bot application can be viewed below.

https://docs.gha-runners.nvidia.com/apps/copy-pr-bot/

Important: copy-pr-bot enforces signed commits. If an organization member opens a PR that contains unsigned commits, it will be deemed untrusted and therefore require an /ok to test comment. See the GitHub docs here for information on how to set up commit signing.
Any time a PR is deemed untrusted, it will receive a comment that looks like this: https://github.com/rapidsai/ci-imgs/pull/63#issuecomment-1688973208.
Every subsequent commit on an untrusted PR will require an additional /ok to test comment.
Any existing PRs that have unsigned commits after this change is merged will require an /ok to test comment for each subsequent commit or the PR can be rebased to sign the unsigned commits as mentioned in the docs here: https://docs.gha-runners.nvidia.com/cpr/contributors.
This information is all included on the documentation page linked above.
I've skipped CI on this PR since it's not a change that is tested.
[skip ci]

Keeping this PR as a draft until I've had a chance to coordinate with Julio or Adam to get this new application installed on the NVIDIA-Merlin organization.


imap display bug

I have a huge problem with insert mode mappings in Oni: there are phantom characters all over the place. See gif below, taken on a fresh build from master, comparing with neovim:

This looks to be an artifact of the predictive typing.
In Oni, when you are typing to make the latency as low as possible, we show the typed character instantly, instead of waiting for nvim to respond and only then showing. That works fine 99% of the time, but in this case, typing ,t  then collapses down to a single char and it gets confused. The phantom characters are left over from the predction, when really they should be being reolved and removed.
If I turn the debug mode on and try your same mappings, it can be seen more easily (predicted but not yet confirmed chars are shown in red). Oni will stay in this state until the next char is typed, when it should be resolved quickly.

I'll have a think about a fix, and perhaps @bryphe will already have some ideas since this was somewhat spoken about when the predictive side was added.

Thanks @CrossR for the helpful description, I'll add that setting to the minimal recommended settings and remove when it's stable.

Thank you very much for this speedy answer. I can confirm "editor.typingPrediction": false, fixes this.

Great find, @CrossR ! Awesome troubleshooting - glad it resolved the issue 💯

I'll have a think about a fix, and perhaps @bryphe will already have some ideas since this was somewhat spoken about when the predictive side was added.

Yes, unfortunately this looks like a bug in the implementation - right now, we assume that the character will eventually move the cursor position, to hit the clearing logic here:
https://github.com/onivim/oni/blob/ecb0a767c395883db0702ea918a0a5757d23c393/browser/src/Services/TypingPredictionManager.ts#L78
In this imap case unfortunately that assumption is invalid. It'd be better for this prediction logic to have a deterministic way to know that Neovim has processed the input, as opposed to trying to derive it from cursor position changes. For example, if we could know when Neovim sends us a redraw back that it has processed inputs up to N, we could clear all the predicted characters up to N. We use the cursor unreliable proxy for this today.
I'd like to keep this open to track that - implementing a more deterministic strategy for resolving when a  prediction completes. Ideally, the prediction logic should be completely seamless and not cause these problems 😄 It reduces typing latency significantly in some environments so it'd be great to fix this issue!


Error with gzip

hi guys i have a problem with apickli, when me make a report . The problem is rare characters with gzip.. im modified the apickli.js for gzip. the error persist.
https://stackoverflow.com/questions/70021552/problem-in-response-in-atomatization-with-gherking

Hi @Hyuden
In your Before function, just add:
  this.apickli.httpRequestOptions = {
    gzip: true,
  };

Hi @seymen Your refer to Apickli.js? what line in specific?

No, not apickli.js. It is a file in your project (not in apickli) so I don't know what you called it. In the README, it is referred to as init.js. It has a function called Before. Could you search your project for a function called Before as per my original comment?

Yes, i modified before from init.js:

I've added a test to https://github.com/apickli/apickli/pull/187 to show you how it is done. Please see the changes introduced in that PR and repeat in your implementation. Otherwise, modify that gzip scenario to demonstrate failure.

Hi i make the new proyect with your version i have one proble, the test cases doesnot iniciate.

I don't know what that means, but in the PR https://github.com/apickli/apickli/pull/187, you can see that the tests are passing. I will close this for now, as you have an example of how it is working. I will merge this sometime this week.

Ok


Fix top level logical assignment leak

Fixes #40494
Fixes #41153

Ci failed but not my fault.

@typescript-bot pack this.

Emmm.... It's better to merge before 4.1 IMO
CC:/ @sandersn

UUUP

Emmmm.. Friendly Ping again

Sorry. Are there anything else todo for this fix?....

@typescript-bot perf test this

Sorry, I'm a little unsure.
If I'm correct, Logical assignment operator will always create an control flow for above code, but logical operator not.
So The function isTopLevelLogicalExpression might not an approximate name. I think check should called something like shouldCreateNewFlow.

That sounds like a more accurate name from my reading of the code too. But I still don't understand why the deleted code checks that node.parent and not node is a logical assignment operator. Can you try to find out why that is the case?

We only check a node is a "top level logical expression" if the node is a logical expression.
If a node is "top level", the parent node should not be an logical expression. Otherwise the parent node is the top level.
Seems reasonable doesn't it?.😂


question: how do I access the actual message id?

First off,  I'm loving this bot
So far I've been using the right-click -> copy message link to get a URL to a message, but how do I get the message ID itself?
Anyway, I just installed this bot and moved a LOT of messsages and considered submitting a PR to add support for moving a range of messages, but saw that it's already in the works on your 'dev' branch.
Still, if specifying a set of non-consecutive messages (where you can't use -n )   -m csv,of,msg,urls. -d  might get a bit unwieldy.
So I'm wondering, how can I get the message ids themselves?

You must enable developer mode under App Settings -> Advanced. After enabling this setting, you may right click a message and select Copy ID. The message ID is also the last segment in the URI path obtained by copying the message link.
The next release will allow bulk movement of messages. However, I do not currently have an ETA for the next release.


本地部署出现问题

您好。我尝试本地部署模型，
app.py中加入如下代码才能正常运行，
model args.model name or path = '/home/xfwl/huggingface/galactica-1.3b
部署完成后输入问题，机器人回答的是输入的内容，且有时还会回复代码。
http://lmflow.com 网页中微信群和discord链接全部都失效了，想进入社群学习。且这个网页的机器人不能对话。

下面是我下载的权重，引用，以及回复截图。

Thanks for your interest in LMFlow! Yeah this file has some hard-wired parameters, maybe @SHUMKASHUN can look into this issue and improve the interface? As for the QR code, we will update it soon, please wait for a while 😄

The QR code is updated accordingly (https://imgbb.com/ZhM4hhn), which is valid until Nov. 5. Thanks 😄


⚠ Bofrost.be has degraded performance

In cddb8dd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2017 ms

Resolved: Bofrost.be performance has improved in c62b7c3 after 26 minutes.


How to change variables

There are two themes in my project and I want to switch between them.
Themes are implemented as 2 scss styles with variables

how can i implement this?

I don't think your question is specific to this package.
I think you either will have to switch at build time, via some Webpack magic, or at runtime by using different variable names.


[BUG] OS 2.2.0 release missing from maven gradle build-tools repos

The new release 2.2.0 doesn't appear to be available here: https://repo.maven.apache.org/maven2/org/opensearch/gradle/build-tools/
Is it a question of just waiting a bit and it will appear there or is there an additional step needed?

https://repo.maven.apache.org/maven2/org/opensearch/gradle/build-tools/2.2.0/ has appeared now, i think i needed to be more patient.
Closing :-)


SQLAlchemy 2 DeclarativeBase is not supported

When using render_er with a class extending DeclarativeBase an error is raised, because switch_input_class_to_method cannot handle DeclarativeAttributeIntercept. I'm happy to give a PR a try

Do you have a short example which uses this?
Would be cool, if you could submit it along to a PR for it :+1:
But just a small example would help too!

Sure. You can find an offical example here: https://docs.sqlalchemy.org/en/20/orm/mapping_styles.html#declarative-mapping. I solved it local with this code
from eralchemy2 import main as eralchemy2_main
eralchemy2_main.switch_input_class_to_method[
        "DeclarativeAttributeIntercept"
] = eralchemy2_main.declarative_to_intermediary


build error in macos

Compiling libscreenshot v0.3.1 (https://github.com/MadrigalStreetCartel/libscreenshot?branch=area-capture#2bd0a19c)
error[E0046]: not all trait items implemented, missing: capture_window_area, capture_focused_window_area
--> /Users/chy/.cargo/git/checkouts/libscreenshot-7b3b756c80d81841/2bd0a19/src/platform/macos/cg_provider.rs:46:1
|
46 | impl WindowCaptureProvider for CGProvider {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ missing capture_window_area, capture_focused_window_area in implementation
|
::: /Users/chy/.cargo/git/checkouts/libscreenshot-7b3b756c80d81841/2bd0a19/src/traits/window_capture_provider.rs:5:5
|
5  |     fn capture_window_area(&self, window_id: WindowId, area: Area) -> Result;
|     -------------------------------------------------------------------------------------- capture_window_area from trait
6  |     fn capture_focused_window(&self) -> Result;
7  |     fn capture_focused_window_area(&self, area: Area) -> Result;
|     ------------------------------------------------------------------------- capture_focused_window_area from trait
For more information about this error, try rustc --explain E0046.
error: could not compile libscreenshot due to previous error
warning: build failed, waiting for other jobs to finish...
Error failed to build app: failed to build app
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
this happens in macbookpro m1

Please create issue on corespondent branch


Dlib master & 19.19 and later - build failed on MacOS/clang 10/cmake 3.17.1

$ cmake -DCMAKE_BUILD_TYPE=Release ../
-- The C compiler identification is Clang 10.0.0
-- The CXX compiler identification is Clang 10.0.0
-- Check for working C compiler: /usr/local/opt/llvm/bin/clang
-- Check for working C compiler: /usr/local/opt/llvm/bin/clang - works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/local/opt/llvm/bin/clang++
-- Check for working CXX compiler: /usr/local/opt/llvm/bin/clang++ - works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Using CMake version: 3.17.1
-- Compiling dlib version: 19.19.99
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
 *****************************************************************************
 *** DLIB GUI SUPPORT DISABLED BECAUSE X11 DEVELOPMENT LIBRARIES NOT FOUND ***
 *** Make sure XQuartz is installed if you want GUI support.               ***
 *** You can download XQuartz from: https://www.xquartz.org/               ***
 *****************************************************************************
-- Found system copy of libpng: /usr/local/lib/libpng.dylib;/usr/lib/libz.dylib
-- Found system copy of libjpeg: /usr/local/lib/libjpeg.dylib
-- Searching for BLAS and LAPACK
-- Searching for BLAS and LAPACK
-- Found PkgConfig: /usr/local/bin/pkg-config (found version "0.29.2")
-- Checking for module 'cblas'
--   No package 'cblas' found
-- Checking for module 'lapack'
--   No package 'lapack' found
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of void*
-- Check size of void* - done
-- Found LAPACK library
-- Found CBLAS library
-- Looking for cblas_ddot
-- Looking for cblas_ddot - found
-- Looking for sgesv
-- Looking for sgesv - found
-- Looking for sgesv_
-- Looking for sgesv_ - found
CUDA_TOOLKIT_ROOT_DIR not found or specified
-- Could NOT find CUDA (missing: CUDA_TOOLKIT_ROOT_DIR CUDA_NVCC_EXECUTABLE CUDA_INCLUDE_DIRS CUDA_CUDART_LIBRARY) (Required is at least version "7.5")
-- DID NOT FIND CUDA
-- Disabling CUDA support for dlib.  DLIB WILL NOT USE CUDA
CMake Error at dlib/cmake_utils/use_cpp_11.cmake:74 (target_compile_features):
  target_compile_features no known features for CXX compiler

  "Clang"

  version 10.0.0.
Call Stack (most recent call first):
  dlib/CMakeLists.txt:855 (enable_cpp11_for_target)

-- Building a C++11 test project to see if your compiler supports C++11
-- C++11 activated.
-- Configuring incomplete, errors occurred!
See also "/Users/max/Documents/devel/libs/dlib/build-release/CMakeFiles/CMakeOutput.log".

Does doing what #2009 suggests fix it?

Unfortunately not.
$ head -20 ../dlib/CMakeLists.txt
#
# This is a CMake makefile.  You can find the cmake utility and
# information about it at http://www.cmake.org
#

cmake_minimum_required(VERSION 2.8.12)

if(POLICY CMP0025)
  cmake_policy(SET CMP0025 NEW)
endif()
if(POLICY CMP0048)
   cmake_policy(SET CMP0048 NEW)
endif()
if(POLICY CMP0077)
   cmake_policy(SET CMP0077 NEW)
endif()

project(dlib)

$ cmake -DCMAKE_BUILD_TYPE=Release ../
-- The C compiler identification is Clang 10.0.0
-- The CXX compiler identification is Clang 10.0.0
-- Check for working C compiler: /usr/local/opt/llvm/bin/clang
-- Check for working C compiler: /usr/local/opt/llvm/bin/clang - works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/local/opt/llvm/bin/clang++
-- Check for working CXX compiler: /usr/local/opt/llvm/bin/clang++ - works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Using CMake version: 3.17.1
-- Compiling dlib version: 19.19.0
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
 *****************************************************************************
 *** DLIB GUI SUPPORT DISABLED BECAUSE X11 DEVELOPMENT LIBRARIES NOT FOUND ***
 *** Make sure XQuartz is installed if you want GUI support.               ***
 *** You can download XQuartz from: https://www.xquartz.org/               ***
 *****************************************************************************
-- Found system copy of libpng: /usr/local/lib/libpng.dylib;/usr/lib/libz.dylib
-- Found system copy of libjpeg: /usr/local/lib/libjpeg.dylib
-- Searching for BLAS and LAPACK
-- Searching for BLAS and LAPACK
-- Found PkgConfig: /usr/local/bin/pkg-config (found version "0.29.2")
-- Checking for module 'cblas'
--   No package 'cblas' found
-- Checking for module 'lapack'
--   No package 'lapack' found
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of void*
-- Check size of void* - done
-- Found LAPACK library
-- Found CBLAS library
-- Looking for cblas_ddot
-- Looking for cblas_ddot - found
-- Looking for sgesv
-- Looking for sgesv - found
-- Looking for sgesv_
-- Looking for sgesv_ - found
CUDA_TOOLKIT_ROOT_DIR not found or specified
-- Could NOT find CUDA (missing: CUDA_TOOLKIT_ROOT_DIR CUDA_NVCC_EXECUTABLE CUDA_INCLUDE_DIRS CUDA_CUDART_LIBRARY) (Required is at least version "7.5")
-- DID NOT FIND CUDA
-- Disabling CUDA support for dlib.  DLIB WILL NOT USE CUDA
CMake Error at dlib/cmake_utils/use_cpp_11.cmake:74 (target_compile_features):
  target_compile_features no known features for CXX compiler

  "Clang"

  version 10.0.0.
Call Stack (most recent call first):
  dlib/CMakeLists.txt:861 (enable_cpp11_for_target)

-- Building a C++11 test project to see if your compiler supports C++11
-- C++11 activated.
-- Configuring incomplete, errors occurred!
See also "/Users/max/Documents/devel/libs/1/dlib/build-release/CMakeFiles/CMakeOutput.log".

¯_(ツ)_/¯ Seems like a bug in cmake or something. I don't have a way to reproduce this, so you will need to figure out a workaround.  And post a PR when you figure it out :)

Yes, it seems like a bug in Cmake. My workaround is quite simple but it's not a solution - comment out lines 74-85 of dlib/cmake_utils/use_cpp_11.cmake

Huh.  It seems like there should be a better way.  It would be a bit surprising if target_compile_features was just totally broken.


Support fetching node details through node label instead of ID

If we could call /api/topology/{name}/{id} with node labels instead of IDs, we would be able to load node details directly if only node labels were provided in the URL.
As far as I can see, this is the only backend blocker for resolving #2030 and it would only require us to do a bit more sophisticated search through nodes in handleNode in api_topology.go.
cc @rade @dlespiau

What exactly do you mean by "node labels"? The label field in the json? For k8s that does contain the k8s name, but don't we also need the namespace? Note that the rank field combines namespace and name.

What exactly do you mean by "node labels"? The label field in the json?

Yeah, I did mean the label field originally, but it makes more sense to include namespace in the name. I guess rank is exactly what we want since for controllers it has the same format as our Deploy/Monitor workload IDs, it's just the name rank that I find confusing :P


code 400, message Bad request syntax

Info:
Server version: 1.7
Client version: 3.1.2
Platform: iOS
Server Platform: Android on Termux
How do I fix this code 400 error, I typed https on the binary file instead of http only.

I think it needs certificate?

I tried putting https because http don't work/doesn't have a response to the server

Server is not running under HTTPS. You need a domain name and SSL certificate if you want to use HTTPS.

Server is not running under HTTPS. You need a domain name and SSL certificate if you want to use HTTPS.

I got a SSL certificate, but how do I get a domain name?

One SSL certificate must bind one domain, and it can be only used in that domain. If you have no domain, why you think you got a SSL certificate? It is obviously useless. The HTTPS connection will be untrusted.
Most of domains requires fees. Normally, a device or a server with public IP, a domain, an SSL certificate, are all required for building an HTTPS server.

The iOS platform allows the utilization of HTTP by enabling the NSAppTransportSecurity-NSAllowsArbitraryLoads key to true in info.plist.


Unable to resolve action actions/add-to-project@v1, unable to find version v1

After we merged a dependabot PR that upgrades this this action, the action runner fails with:

Unable to resolve action actions/add-to-project@v1, unable to find version v1

Log of failed run: https://github.com/zitadel/oidc/actions/runs/8535894709/job/23383365622?pr=577
Did dependabot do the wrong thing here, or is there something wrong with release publishing to the marketplace?

Apologies, just found out that my colleague changed the version in the dependabot PR. But as there is no dedicated v1 release branch, v1 won't work. Will revert to v1.0.0 as proposed by dependabot initially.

@skw super annoying and breaks the convention of other GitHub Actions, please fix this


Santosh

Good

Good

Tnx
On Tue, 14 Nov, 2023, 8:59 am Jsjsusjwh, @.***> wrote:

Closed #268 https://github.com/TeamVanced/VancedMicroG/issues/268 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/TeamVanced/VancedMicroG/issues/268#event-10949365370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAQZGBD4MJCLA3BMTLZWSWDYELQSPAVCNFSM6AAAAAA7KEKEKCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHE2DSMZWGUZTOMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hello

Hii


feat(captcha): captcha guard and module

#581
#578
#577
#576  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ABCxFF and the rest of your teammates on  Graphite

#581
#578
#577
#576  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ABCxFF and the rest of your teammates on  Graphite


Fixed broken link to Rob Sewell's blog

Link to Rob Sewell's blog article called "How to fork a GitHub repository and contribute to an open source project" was broken. I updated the link to the correct location and added a mirror link to the article on archive.org, in case the URL changes again in the future.

@way0utwest Thanks for merging my PR! For some reason, it seems that my changes didn't make it onto this PR page, which I navigated to by opening the SQL Saturday homepage, scrolling to the Events section, clicking the running a new event link, then using the Resources menu at the top of the page to select SUBMITTING A PR TO CHANGE AN EVENT.

It's. Fine. I'm out and assumed the link was good. Approved on my phone


Translation strings for autocompleter placeholders

The first string is for the existing placeholder attribute of the "help identify" filter input.
The other is preparation for the upcoming refactor of autocomplete inputs.
Printing a placeholder will fix a current issue where Safari tries to autofill name and location autocomplete inputs with contacts from the user's address book.

coverage: 93.644%. remained the same when pulling 2ca120a875cf7ab3b6998abebee391b1f78a5f68 on nimmo-placeholders-translations into 259f8157dd4e501d0299b79b5a298480e8334914 on main.


Ressource pack 1.20.4 build missing some textures o not rendering them

1.20.4:

1.20:

I just updated the RP with the missing items, please let me know if you find any more :)
Also, your original issue was fine. I think I get notifications for the Magic repo but not this one, so it may be better there anyway.
Thank you!


chore: use Azure Functions runtime v4

https://github.com/OfficeDev/TeamsFx-Samples/blob/204f1948b1eb67b68b3dc5d771023badff351774/todo-list-with-Azure-backend/templates/azure/provision/function.bicep#L39

FUNCTIONS_EXTENSION_VERSION
Added.
Good catch, I searched all "// Use Azure Functions runtime" o(╥﹏╥)o


[Compute] az vm extension set: Add examples and suggestion to use VM extension

Description

Added information about using the Azure Key Vault virtual machine extension to install certificates on the VM instead of add az vm secret as using the key vault extension provides a more secure experience. Also added a sample on how to add the secret using the extension.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

vm

@mimckitt Please fix the linter and style issue~

Please solve the CI issue in time today, otherwise this PR will miss the release of this sprint and can only be released in the next sprint (about 08-06)

@zhoxing-ms any idea what the actual formatting error is? I can't seem to figure it out.

Hi, @mimckitt
For the error from flake8

/home/vsts/work/1/s/src/azure-cli/azure/cli/command_modules/vm/_help.py:2141:417: W291 trailing whitespace
/home/vsts/work/1/s/src/azure-cli/azure/cli/command_modules/vm/_help.py:2141:417: W291 trailing whitespace
Please remove the trailing whitespace

For the error from linter

FAIL - HIGH severity: faulty_help_example_rule
Help-Entry: vm secret add - The following example entry indices do not include the command: 0 | 1
For this problem, you need to modify the comments above

@zhoxing-ms thanks! I took the sample from our doc on the VM extension
https://docs.microsoft.com/en-us/azure/virtual-machines/extensions/key-vault-linux
I will check to make sure those docs are correct as well.

@smotwani FYI

@zhoxing-ms I believe I addressed the issues. Not sure what the problem with the checks are now. Could you assist?

@mimckitt  There are still some CI issues, could you please fix them?

FAIL - HIGH severity: faulty_help_example_parameters_rule
Help-Entry: vm secret add -
There are 2 violations:
"az vm extension add -n "KeyVaultForLinux" --publisher Microsoft.Azure.KeyVault -g "" --vm-name "" --version 2.0 --settings '{"secretsManagementSettings": { "pollingIntervalInS": "", "certificateStoreName": "", "certificateStoreLocation": "", "observedCertificates": ["  ", "  "] }}'" is not a valid command.
argument _subcommand: unknown parser 'add' (choices: delete, show, set, list, wait, image)
"az vm extension add --name "KeyVaultForWindows" --publisher Microsoft.Azure.KeyVault --resource-group "" --vm-name "" --settings '{"secretsManagementSettings": { "pollingIntervalInS": "", "certificateStoreName": "", "certificateStoreLocation": "", "observedCertificates": ["  ", "  "] }}'" is not a valid command.
argument _subcommand: unknown parser 'add' (choices: delete, show, set, list, wait, image)
FAIL - HIGH severity: faulty_help_example_rule
Help-Entry: vm secret add - The following example entry indices do not include the command: 0 | 1

@mimckitt Hi, please solve these comments and conflicts in time today, otherwise this PR will miss the release of this sprint and can only be released in the next sprint (09-07)

I am looking at the best way to address pushing users to use the extension. So it's okay if we miss the release and go in sept.

@zhoxing-ms how do I just add a note under the command?
Similar to this:

That way I can add a note saying we suggest to use the extension instead but I don't have to modify the examples

how do I just add a note under the command?

@dbradish-microsoft Could you please take a look at this question?


Option to drop faulty rows during import

I am importing a lot of rows from my crawler and it's very inconvenient that one single faulty row causes my entire import to be halted.
Is there anyway to specify django-import-export to just skip a row if it contains any error?
(Perhaps I can override something at for_delete?)

Hi, @vincentwhales can solve this by using try and except in the method for_delete or as in my case in after_import_row.


Only Allow Declaring Partition Columns in PARTITIONED BY Clause

Is your feature request related to a problem or challenge?
DataFusion implicity reorders columns in table definitions so that PARTITION BY columns are stored at the end of the underlying parquet files. This leads to very confusing behavior when selecting directly out of the parquet file as the parquet schema has a different column order than the  order of the columns in the CREATE TABLE statement.
Datafusions SQL dialect declares partitioned tables like this:
CREATE EXTERNAL TABLE(partition varchar, trace_id varchar)
STORED AS parquet
PARTITIONED BY (partition)
LOCATION '/tmp/test/';

Note that the partition column is declared with the other columns and again later in the PARTITIONED BY clause. Internally, Datafusion reorders table schemas so that partition columns come at the end, which is a common convention. This leads to confusing examples like #7892 and the following
DataFusion CLI v36.0.0
❯ create external table test(partition varchar, trace_id varchar) stored as parquet partitioned by (partition) location '/tmp/test/';
0 rows in set. Query took 0.001 seconds.

❯ insert into test values ('a','x'),('b','y'),('c','z');
+-------+
| count |
+-------+
| 3     |
+-------+
1 row in set. Query took 0.016 seconds.

❯ select * from test;
+----------+-----------+
| trace_id | partition |
+----------+-----------+
| a        | x         |
| c        | z         |
| b        | y         |
+----------+-----------+
3 rows in set. Query took 0.002 seconds.

Since you declared the order as (partition varchar, trace_id varchar) you would expect this order to be respected when inserting data, but instead it is silently reordered so that the partition column comes at the end.
Describe the solution you'd like
Rework CREATE EXTERNAL TABLE syntax to only allow partition by columns to be declared in the partitioned by clause. The above example then becomes:
CREATE EXTERNAL TABLE(trace_id varchar)
STORED AS parquet
PARTITIONED BY (partition varchar)
LOCATION '/tmp/test/';

This leaves much less room for confusion about the ordering of the columns when inserting values. This also follows the syntax of HiveQL, see: https://cwiki.apache.org/confluence/display/hive/languagemanual+ddl#LanguageManualDDL-PartitionedTables
Describe alternatives you've considered
We could instead drop the convention of moving the partitioned by columns to the end of the schema and respect the ordering of columns that the user declares.
Additional context
No response

take

Thanks for the interest in picking this up @Lordworms!
Since this would be a breaking change and possibly overlap/conflict with #9369, we should make sure that we have consensus on the plan before getting too deep into it. cc @alamb and @metesynnada if you have any thoughts on this.

If we can keep both the current behavior as well I think this is a good idea (aka if this is backwards compatible)
So specifically, if both the following SQL statements result in the same table schema (trace_id varchar, partition varchar)
CREATE EXTERNAL TABLE  test(partition varchar, trace_id varchar)
STORED AS parquet
PARTITIONED BY (partition) -- no type specified here
LOCATION '/tmp/test/';

CREATE EXTERNAL TABLE test(trace_id varchar)
STORED AS parquet
PARTITIONED BY (partition varchar)
LOCATION '/tmp/test/';

If we need to break backwards compatibility, I think it should be discussed more widely

If we need to break backwards compatibility, I think it should be discussed more widely

I think that we either should

break backwards compatibility and throw an error when a column is declared as both a regular column and a partition column
rework internal logic of partitioned ListingTables so they respect the original order the columns were declared in by the user, even if some are partitioned columns

I favor the first option and believe it would be much easier to implement. We could perhaps go for a phased approach where we deprecate the existing syntax with a warning of why it is not recommended, though maintaining both syntaxes would be more complex.
With all that said if we are against a breaking change, we could simply update documentation to increase visibility into the existing behavior and especially clarify that partition columns must be moved to the end when inserting data.

break backwards compatibility and throw an error when a column is declared as both a regular column and a partition column

This seems reasonable to me
Here is what I suggest to get some sort of consensus about this potentially breaking change:

Send a note to the mailing list  ("we are considering a breaking change to CREATE EXTERNAL TABLE ... please comment on the ticket if you have an opinion")
Maybe also cross post to discord / slack

The idea is to get all the feedback into github but try and make sure as many people have a chance to weigh in as possible

If we can keep both the current behavior as well I think this is a good idea (aka if this is backwards compatible)

I think this is very much possible with minimal changes to the parser

I think this is very much possible with minimal changes to the parser

@MohamedAbdeen21 if you are interested in working on a PR to implement this, that would be much appreciated :pray: . I think that would be a great step to take prior to gathering feedback/consensus on making the change breaking.

I'll try to get a draft PR up before EoD


Nzbhydra2

Motivation for this change
Closes #76878
Use the AUR package and jackett.nix as guidance on how to package NZBHydra2.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/345

@JamieMagee Please resolve the merge conflcit.

@SuperSandro2000 Ready for review

Pushed an update from 2.29.1 to 3.8.0.
@aanderse I had to remove the MemoryDenyWriteExecute = true; in order to get OpenJDK to run. Otherwise it would fail with
OpenJDK 64-Bit Server VM warning: INFO: os::commit_memory(0x00007f9e8b2c3000, 2555904, 1) failed; error='Operation not permitted' (errno=1)

@ofborg eval

@SuperSandro2000 I propose merge - any outstanding issues from your point of view?

@SuperSandro2000 I propose merge - any outstanding issues from your point of view?

@aanderse It appears there are no concerns?

@aanderse It appears there are no concerns?

@JamieMagee sure. I'll assume @SuperSandro2000 is good with the package and merge once the bot runs your test.
Thanks for contributing to nixpkgs!
@GrahamcOfBorg test nzbhydra2

@JamieMagee sure. I'll assume @SuperSandro2000 is good with the package and merge once the bot runs your test.
Thanks for contributing to nixpkgs!
@GrahamcOfBorg test nzbhydra2


Dont see .svg icons in the gutter

Hi,
I don't know if you saw this, since I originally posted it on https://github.com/pgsandstrom/vscode-navigate-edit-history/issues/21
Thanks for the reply.
I've tested your fork, and I successfully see marks in the overview and the border.
The gutter still won't show either of the .svg icons when I choose them in navigateEditHistory.markerStyle, though.
The relative path looks correct, but I also tried the absolute path in navigateEditHistory.markerJSONCustom.
Is there anything you know of that would force my gutter not to display things?
Regards

the default svg files are triangle.svg and rectangle.svg, in user's folder.vscode\extensions\austinyoung.navigate-mark-edit-history-1.0.2\svg\
I am sorry that I don't know why if the default svg not to display
if you use your svg , make sure the size of svg, such as (width="5" height="20" ) ,too big maybe not show in gutter


invalid bower.json

Hi, when installing tex.s2cms.ru, doing bower install, it complain about unexpected } in bower.json. The error is at line 18. Just deleting the ending , make it work.

Fixed, thanks!


Rich text asset field properties have the wrong types

Brief bug description
The rich text asset field properties have the wrong types.
images: [kontent_item_rich_text_element_link]
links: [kontent_item_rich_text_element_image]

Expected behaviour
Should be
images: [kontent_item_rich_text_element_image]
links: [kontent_item_rich_text_element_link]

Screenshots

Additional comments
Will require an update to @rshackleton/gatsby-transformer-kontent-image when the type names are updated.

Thanks @rshackleton,
the structure of the generated type was correct, but the names are wrong!
Going to fix it.

Fixed in 6.0.1


Delete misc/sexuallymutilatedchild.org directory

bro i think it's obvious why

Oh, we made 2 pull requests for the same thing!

It seems to be fixed now :
https://github.com/Alex313031/Alex313031.github.io/commit/15f9d5bd9a50f85f0019ec9bcc2d4183f868a6b5


Kubelet config/file should read Pod

Currently it assumes it can read ContainerManifest - it should try both

Why not hold on this until we have Kubelet-read-Pod in?

Also the HTTP source has the same problem.

It should read Pod. Changed the title.

Ideally, I'd like all Pods created directly via Kubelet to be added to etcd and surfaced via the API. If updates are made via the API, I'd like those to propagate back to the node. We can discuss whether the config file should be updated or ignored. At some point, I'd like Kubelet to persistently cache pods received from the apiserver, perhaps by just writing them all out as config files.

#4090 is the issue to sync local pods back to apiserver.

@wojtek-t this is somewhat related to other issues we talked about.

Detail: We want locally specified pods to be able to use YAML config files, but we also want to rip YAML out of the API server #1004.
The config-file behavior should be as similar to the kubectl experience. Kubelet could use parts of the kubectl library to translate YAML to JSON.
Vision:
Eventually, I'd like Kubelet to use the apiserver library and support exactly the same Pod API as the master, at which time I'd like kubectl be able to create pods directly on Kubelets, as well as reading from config files.
I also expect to have a per-node controller #2491. I'd like to be able to use the per-node controller to update pods started via config files (or direct kubectl->kubelet pods), overriding those original files.
For availability and self-hosting, we'll eventually need local checkpointing #489, ideally just using serialized JSON for each object. If Kubelet used apiserver, this could be done using a custom registry implementation.

I think that #5392 is a prerequisite for it.

If noone picks it up, I'm going to start working on it tomorrow.

thanks, wojtek-t

#5598 fixed it for http channel


Remove focus border when loaded in run window

Describe the bug

When loaded the app, a focus border always shows.

Steps To Reproduce

Press win + R to open run window.
type calc, and press enter key.
See error

Expected behavior

Loaded with no focus border

Screenshots

Device and Application Information

OS Build:22000.613
Architecture:x64
Application Version:11.2201.4.0
Region:zh-cn
Dev Version Installed:

Additional context

Requested Assignment

I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.


toCSS: commented out @import statements appear in CSS output

When SCSS files contain lines with commented out @import statements, theses import statements appear (as real statements, not within a comment!) in the generated CSS files.
With Hugo version 0.67 this still worked as expected.

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.68.3-157669A0/extended linux/amd64 BuildDate: 2020-03-24T12:13:38Z

Does this issue reproduce with the latest release?
yes.

I created an example to demonstrate this at  https://github.com/backmes/website-hugo-7111

LibSASS doesn't really support import of regular CSS files, which is why I created a (temporary) workaround for these some time ago. I'm not going to spend time on improving that fix, as I expect this will be fixed upstream.
In the meantime, I suggest you remove the import statements that you do not want/neex.

LibSASS doesn't really support import of regular CSS files, which is why I created a (temporary) workaround for these some time ago. I'm not going to spend time on improving that fix, as I expect this will be fixed upstream.

I just was surprised that something in a comment would appear as actual statement in the output, therefore I thought this was a "real" bug, not the result of some workaround...

In the meantime, I suggest you remove the import statements that you do not want/neex.
I already did.


feat:add referenceMany for beforeCreate/beforeUpdate and e2e test

Resolves: #223
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows conventional commit format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #223
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade" workflow

Codecov Report

Merging #50 (9003839) into main (4958138) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #50   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           27        26    -1
=========================================
- Hits            27        26    -1

Impacted Files
Coverage Δ

src/backend-stack.ts
100.00% <100.00%> (ø)

src/lambda/process-metrics-function.ts
100.00% <100.00%> (ø)

src/stateful-stack.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[Playground][Backend][Task]: Dataset processing with python scripts to deploy examples

What needs to happen?
To run emulator-enabled examples python scripts have to process examples with specific tags
Issue Priority
Priority: 2
Issue Component
Component: beam-playground

.take-issue


How to create the elevation tiles from Esri?

I want to host streets-gl on my local machine. How can i create the elevation tiles from Esri for self-hosting? I mean the tiles which are configured with the ElevationEndpointTemplate:
https://github.com/StrandedKitty/streets-gl/blob/dev/src/app/Config.ts#L145

The current ElevationEndpointTemplate is redirecting to https://osm-tiles.ams3.cdn.digitaloceanspaces.com/elevation/
Who is providing these tiles?

New here, but found this https://github.com/StrandedKitty/streets-gl/issues/46#issuecomment-1540936524
not sure what level ids are, but guessing they downloaded the start and end for each level then put it into a digitalocean s3 server

I wonder how it compares to https://github.com/Jorl17/open-elevation/tree/master

@StrandedKitty Any more insights we can get on this?


js error fixed in admin when selects from the drop down fields in model admin search

Error was uncaught exception: cannot call methods on tabs prior to initialization
Tabs must be initialized before you can work on them

Duplicate of https://github.com/silverstripe/silverstripe-framework/pull/5920, sorry @priyashantha 😅


add DesafioLatam to domains

@dbarrientos Please provide us with an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school. Thank you!

@dbarrientos You might miss my previous comment. Please react.

@dbarrientos I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the abovementioned issue. Thank you for the understanding!


Add Babel code frame to stylishFormatter output

Babel code frame makes it easy to print line/col numbers with a nice pointer for lint errors.

looks pretty cool, I'm always down for better error messages!

This is done

This is done


results on HOI recognition on CAD-120 dataset

Through experimental training, the sub-activity and object affordance recognition results  are 88.35 and 89.89 on visualModel ,78.95 and 80.1 on semanticModel. Running eval_120.py, the final results are 88.43 and 90.05. So the result is differrent from the paper. Can you tell me why?

Thank you for your attention. Please adjust the super parameter step_size = 10 to try. In addition, because the CAD-120 dataset is relatively small, it is difficult to train. You can get stable results on something-else dataset.


WebSocketManagerMiddleware class throws Websocket Exception when the socket is not properly closed

I've been struggling to find this problem for the past 24 hours.
In the WebSocketManagerMiddleware class, the awaitable Receive Task will throw an unhandled Websocket Exception that will break the application in the case of a socket not being properly closed. The ExceptionHandlerMiddleware was unable to catch it as well, not sure why.
The exact message is something like "The remote party closed the WebSocket connection without completing the close handshake."
This can probably be replicated using the mvc sample, then using a cellphone to connect to the websocket and then switching back to another process so the browser will be in background.

I read something about this problem and I modified the method of closing the socket. I put the changes in a new branch at this link.
@v1n1akabozo, please test it and tell me if the bug still persists.

@mihai-burduselu
I'll give it a try soon. What i did this morning was to remove the nuget dependency, use the original source and then handle all possible exceptions and now my application is stable.

Unfortunately, the NuGet is based on a lower version of the repo (that in the meantime has been updated).
Take a look at the current version, at the fix @mihai-burduselu made and if you still handle additional exceptions, please submit a pull request.
Thanks!


Support Swagger-UI version >= 3.27.0

Thanks for all the hard work that goes into this project. I know that support for Swagger-UI 3.26.0 was just recently added but our team is wanting to take advantage of the click-to-copy feature that was added in 3.27.0.
Thanks again.
VERSION:
5.5.X

Sorry, opened this in the wrong place. Closing.

Issue 1784 created for Swashbuckle.AspNetCore


[backlog table] Update as of 25/11

What type of PR is this?

documentation
subproject management

What this PR does / why we need it:

Update Capability and Runtime Restrictions as Approved
Update High-throughput Elastic Network as Approved
Update CarrierBillingCheckOut API as new version released
Update Quality by Design as renamed to Session Insights
Include new backlog entries for:

Facial Recognition
Network Health Assessment
Network Traffic Analysis

What this PR does / why we need it:
Including #126 #127 #128 and updating #116
No issue fixes in this PR, as issues #126 #127 #128 will be closed with their respective template merge.

@caubut-charter can you please validate for merging?


add customOptions option to have better control over custom templates

We're not open to extending sw-precache in this direction.
For this use case, I'd recommend going with the new Workbox library instead. It supports similar precaching to sw-precache, while giving you full control over the top-level service worker file.
See, for example, https://workboxjs.org/examples/workbox-sw/


Discord Bot Trouble

Hello, I'm trying to set up a bot with your discord-whitelister plugin. But having trouble getting my bot to come online, can you help me with this?

I've accepted your request on Discord, we can discuss it on there.


Probe-modules: IPIP

Add a probe module to scan for hosts with IPIP protocol (4) supported.
These hosts may be used for IP spoofing, NAT traversal or just lengthening the path which an IP traverses along the network.
Usage example:
zmap  --target-port=53  --probe-module=ipip -i eth0.2
For uses see: PoC||GTFO 0x21:3
This issue was responsibly disclosed two years ago to Cert CC

@phillip-stephens Looked at this briefly and the probe module itself looks reasonable. Would you be willing to help fix up types/tests issues, and merge?

Validated that after fixing up merge conflicts from the past 2 years of changes, the scanning behavior was not changed.
Specifically, ran sudo ./src/zmap -p 53 --probe-module=ipip -i enp0s8 -B 100M -c 2 <IP_ADDRESS>/13 on commit 0285d19 and eab1103 and the hit hosts were identical (3 found hosts). The scan did not return anything when run locally on my MacBook, but when run on an Ubuntu VM in the lab, it did. I'm chocking this up to something strange with my Mac's networking (valid?).
Let me just validate the packets generated look fine in Wireshark and then I'm good to merge.


Update all rust-bitcoin crates

Bumps bitcoin,  bitcoincore-rpc,  and miniscript crates to latest versions.

اريد السحب
Le jeu. 9 juin 2022 à 18:25, andrewtoth @.***> a écrit :

@.**** approved this pull request.
ACK 5a1e578
https://github.com/bwt-dev/bwt/commit/5a1e578b04f454b544aaec815c5c947948ffb2be
—
Reply to this email directly, view it on GitHub
https://github.com/bwt-dev/bwt/pull/103#pullrequestreview-1001729742,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATE6EI3TSPFHMIGOVFFEC7LVOISHZANCNFSM5YKBAHFQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Refactoring + fix: Proper status checks

Description
Proper status checks
Type of change

[x] Refactor
[ ] New feature
[x] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

/lgtm

/approve

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

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

Project coverage is 94.15%. Comparing base (3bc1fb5) to head (6ed4a81).
Report is 24 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1583      +/-   ##
==========================================
- Coverage   94.83%   94.15%   -0.69%
==========================================
  Files          60       60
  Lines        2671     2702      +31
==========================================
+ Hits         2533     2544      +11
- Misses        138      158      +20

see 1 file with indirect coverage changes

/retest

/override "Red Hat Konflux / lightspeed-service-on-pull-request"
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/override "Red Hat Konflux / lightspeed-service-on-pull-request"
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/override "Red Hat Konflux / lightspeed-service-on-pull-request"


Merge all sources into a single header library

Single header libraries are much easier for distribution and use

That is well spotted.
Also.
Having a single header library that can be used on Windows, using clang-cl and VisualStudio will win you even more friends.

The single header library version is now available.
The experimental sbrk backing is still separate - once stabilized it will also be available via a define.