Cannot find drive. A drive with the name 'F' does not exist.

I encountered the following issue installing SQL on a clean server. The weird thing is that it fails when i tried to install this for the first time but succeeds at the next attempt on the same server.
I'm trying to install SQL on the 2 environments at the same time on server A and B
VERBOSE: [LONSQL60B]: LCM:  [ Start  Resource ]  [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]]
VERBOSE: [LONSQL60B]: LCM:  [ Start  Test     ]  [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]]
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Loading module from path 'C:\Program Files\WindowsPowerShell\Modules\xSQLServer\1.
8.0.0\xPDT.psm1'.
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Importing function 'GetxPDTVariable'.
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Importing function 'NetUse'.
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Importing function 'ResolvePath'.
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Importing function 'StartWin32Process'.
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Importing function 'WaitForWin32ProcessEnd'.
Cannot find drive. A drive with the name 'F' does not exist.

+ CategoryInfo          : ObjectNotFound: (F:) [], CimException
+ FullyQualifiedErrorId : DriveNotFound,Microsoft.PowerShell.Commands.JoinPathCommand
+ PSComputerName        : LONSQL60B

Cannot bind argument to parameter 'Path' because it is null.
+ CategoryInfo          : InvalidData: (:) [], CimException
+ FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Microsoft.PowerShell.Commands.JoinPathCommand
+ PSComputerName        : LONSQL60B
Cannot validate argument on parameter 'Path'. The argument is null or empty. Provide an argument that is not null or empty, and then try the command again.
+ CategoryInfo          : InvalidData: (:) [], CimException
+ FullyQualifiedErrorId : ParameterArgumentValidationError,ResolvePath
+ PSComputerName        : LONSQL60B
VERBOSE: [LONSQL60B]:                            [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]] Path:
Cannot bind argument to parameter 'Path' because it is an empty string.
+ CategoryInfo          : InvalidData: (:) [], CimException
+ FullyQualifiedErrorId : ParameterArgumentValidationErrorEmptyStringNotAllowed,GetSQLVersion
+ PSComputerName        : LONSQL60B
VERBOSE: [LONSQL60B]: LCM:  [ End    Test     ]  [[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]]  in 2.3290 seconds.
The PowerShell DSC resource '[xSQLServerSetup]SqlInstaller::[TestScript]::[TestScript]' with SourceInfo '::265::1::xSqlServerSetup' threw one or more non-terminating errors while running
the Test-TargetResource functionality. These errors are logged to the ETW channel called Microsoft-Windows-DSC/Operational. Refer to this channel for more details.
+ CategoryInfo          : InvalidOperation: (:) [], CimException
+ FullyQualifiedErrorId : NonTerminatingErrorFromProvider
+ PSComputerName        : LONSQL60B
VERBOSE: [LONSQL60B]: LCM:  [ End    Set      ]
The SendConfigurationApply function did not succeed.
+ CategoryInfo          : NotSpecified: (root/Microsoft/...gurationManager:String) [], CimException
+ FullyQualifiedErrorId : MI RESULT 1
+ PSComputerName        : LONSQL60B
SQL Installer Script:
xSqlServerSetup SqlInstaller
{
Dependson = "[Script]DisableFirewall","[xMountImage]OSImage","[xMountImage]SQLImage","[File]DSCResourceFolder","[WindowsFeature]NetFramework35Core","[WindowsFeature]NetFramework45Core","[WindowsFeature]WindowsPowerShellISE","[WindowsFeature]WindowsPowerShell50","[WindowsFeature]ActiveDirectoryModuleForWindowsPowerShell","[WindowsFeature]FailoverClustering","[WindowsFeature]FailoverClusterTools","[WindowsFeature]FailoverClusteringPowerShell","[xADUser]$AGsvcAccount","[xADUser]$DEsvcAccount","[xADUser]$FTSvcAccount","[xADUser]$ISSvcAccount","[xADUser]$ASSvcAccount","[xADUser]$RSSvcAccount"
InstanceName = $InstanceName
Setupcredential = $Setupcredential
SApwd = $SACredential
SourcePath = "F:"
SourceFolder = ""
FEATURES="SQLENGINE,RS,IS,SSMS,ADV_SSMS,SQL" #FullText,SNAC_SDK,Conn,DQ,BC,DQC,Replication,MDS,Tools
InstallSharedDir = "C:\Program Files"
InstallSharedWOWDir = "C:\Program Files (x86)"
InstanceDir = "D:\Program Files\Microsoft SQL Server"
InstallSQLDataDir = "G:\SQLData"
SQLUserDBDir = "H:\SQLData"
SQLUserDBLogDir = "L:\SQLLogs"
SQLTempDBDir = "V:\SQLTemp$InstanceName"
SQLTempDBLogDir = "W:\SQLTempLog$InstanceName"
SQLBackupDir = "G:\SQLData"
ASDataDir = "C:\Program Files"
ASLogDir = "C:\Program Files"
ASBackupDir = "C:\Program Files"
ASTempDir = "C:\Program Files"
ASConfigDir = "C:\Program Files"
UpdateSource = "MU" # set this to windows update, otherwise it will use the default and cause the setup to crash
UpdateEnabled = 'True'
SuppressReboot = $False
ForceReboot = $False
PsDscRunAsCredential = $cred
FTSvcAccount = $cred
ASSvcAccount = $cred
ASSysAdminAccounts = $cred
ISSvcAccount = $cred
RSSvcAccount = $cred
#PID =
}

Or could this possibly be because the drives were mounted using the xstorade DSC resource? I'm beginning to wonder why. I've mounted the drives manually in the past and it worked.

xStorage have an issue with the drive not being visible to the system quick enough. Might be the same problem. See issue PowerShell/xStorage#58. Can that be your issue?

@johlju Thanks again. That was definitely my issue. That fixed it. I added the xWaitForVolume resource for the xmount image resource to depend on.

Same issue using DSC in Ansible. Even WaitFor doesn't help. Works only on second run.
- name: Mount ISO
  win_dsc:
    resource_name: MountImage
    Ensure: 'Present'
    ImagePath: "c:\\temp\\{{ mssql_iso_name }}"
    DriveLetter: 'S'
    StorageType: 'ISO'
  tags:
  - install_db

- name: Wait for ISO to be mounted
  win_dsc:
    resource_name: WaitForVolume
    DriveLetter: 'S'
    RetryIntervalSec: 5
    RetryCount: 10

- name: Configure SqlSetup
  win_dsc:
    resource_name: SqlSetup
    ForceReboot: False
    UpdateEnabled: False
    SourcePath: "S:\\"
    InstanceName: "{{ db_settings.instance_name|upper }}"
    Features: "SQLENGINE,AS"
...


Move kube-api-auth image builds to Rancher

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Other details that may be helpful:
In Rancher settings we are using an AuthImage published to an unofficial Dockerhub account:
https://github.com/rancher/rancher/blob/master/pkg/settings/setting.go#L17
This image should be generated & pushed to the Rancher hub account, and the following repo also moved to Rancher github:
https://github.com/erikwilson/rancher-kube-api-auth

repo and image are all under Rancher account now


A hyperlink got broken in the initial pull request

a hyperlink was broken and technically there is no requirement of a DSLR camera, just a video camera.

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

File
Status
Preview URL
Details

mixed-reality-docs/spectator-view.md
:white_check_mark:Succeeded
View

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


upgrade dependencies

Codecov Report

Merging #2057 (3149ef2) into master (6b8a40a) will decrease coverage by 4.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2057      +/-   ##
==========================================
- Coverage   94.54%   90.20%   -4.34%
==========================================
  Files         367      368       +1
  Lines        3428     3431       +3
  Branches      420      420
==========================================
- Hits         3241     3095     -146
- Misses        101      247     +146
- Partials       86       89       +3

Impacted Files
Coverage Δ

packages/client/lib/commands/HELLO.ts
18.18% <0.00%> (-81.82%)
:arrow_down:

packages/client/lib/commands/ACL_CAT.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/ACL_GENPASS.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/ZRANGESTORE.ts
20.00% <0.00%> (-80.00%)
:arrow_down:

packages/client/lib/commands/GETEX.ts
21.42% <0.00%> (-78.58%)
:arrow_down:

packages/client/lib/commands/LPOS_COUNT.ts
22.22% <0.00%> (-77.78%)
:arrow_down:

packages/client/lib/commands/ACL_LOG.ts
28.57% <0.00%> (-71.43%)
:arrow_down:

packages/client/lib/commands/COPY.ts
33.33% <0.00%> (-66.67%)
:arrow_down:

packages/client/lib/commands/LPOS.ts
33.33% <0.00%> (-66.67%)
:arrow_down:

packages/client/lib/commands/ZINTER.ts
36.36% <0.00%> (-63.64%)
:arrow_down:

... and 42 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 6b8a40a...3149ef2. Read the comment docs.


wrong React in remote subdependencies (require(react))

I am not sure if this is issue with this library,  vite  or runtime module federation. But It works with webpack module federation.
So basically react in vite remote app works fine, react components render normally. But when I try to use Ag-grid-react I get infamous Invalid hook call. error.
    at Object.useRef (chunk-EFTEWGEA.js:1074:29)
    at AgGridReactUi (ag-grid-react.js:13873:36)
    at renderWithHooks (react-dom.development.js:16305:18)
    at mountIndeterminateComponent (react-dom.development.js:20074:13)
    at beginWork (react-dom.development.js:21587:16)
    at HTMLUnknownElement.callCallback2 (react-dom.development.js:4164:14)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:4213:16)
    at invokeGuardedCallback (react-dom.development.js:4277:31)
    at beginWork$1 (react-dom.development.js:27451:7)
    at performUnitOfWork (react-dom.development.js:26557:12)

Seems the issue with right version react obtained via require inside ag-grid-react (CJS library):
    var react_1 = __importStar(require_react());

@zhangHongEn you just fixed an issue for dynamic imports, maybe we also need fix for require

@husayt Good issue, I will fix it at noon

Also similar error when I use MUI (Material UI)
TypeError: Cannot read properties of null (reading 'useContext')
    at Object.useContext (chunk-EFTEWGEA.js:1062:29)
    at useTheme (chunk-JKYVMHHX.js:3290:32)
    at useTheme2 (chunk-JKYVMHHX.js:3298:10)
    at useThemeProps (chunk-JKYVMHHX.js:3839:15)
    at useThemeProps2 (chunk-JKYVMHHX.js:6377:10)
    at Paper2 (@mui_material.js:1304:17)
    at renderWithHooks (react-dom.development.js:16305:18)
    at updateForwardRef (react-dom.development.js:19226:20)
    at beginWork (react-dom.development.js:21636:16)
    at beginWork$1 (react-dom.development.js:27426:14)

@husayt Good issue, I will fix it at noon

I have other issues two, but this is the biggest and most serious issue affecting anyone who would try do any realistic use with 3rd party libraries.

@husayt Good issue, I will fix it at noon

I have other issues too, but this is the biggest and most serious issue affecting anyone who would try do any realistic use with 3rd party libraries.

Yes, this is a very important bug, but I didn't have time to look at it until 7 hours later.

Yes, this is a very important bug, but I didn't have time to look at it until 7 hours later.

that is fine, i posted some other issues as well. I just wanted to say that this one is the most pressing one among them.

@zhangHongEn you are doing great, take your time to avoid burnouts :rocket:

Vite pre-built subdependencies cannot be proxied in the usual way. I have found a solution, but I have been too busy recently and may need to wait for two days.

I made a simple repro to  test this case. Will try to commit that later. But gist of it I added another remote with
with these dependencies
  "dependencies": {
    "@module-federation/vite": "workspace:*",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "vue": "^3.4.29",
    "vue-router": "^4.4.0",
    "ag-grid-community": "^30.2.0",
    "ag-grid-react": "^30.2.0"
  },

and AgGridDemo.jsx
import 'ag-grid-community/styles/ag-grid.css';
import 'ag-grid-community/styles/ag-theme-alpine.css';
import { AgGridReact } from 'ag-grid-react';
import { useState } from 'react';

export const AgGridDemo = ({  }) => {
  const [rowData] = useState([
    { make: 'Toyota', model: 'Celica', price: 35000 },
    { make: 'Ford', model: 'Mondeo', price: 32000 },
    { make: 'Porsche', model: 'Boxter', price: 72000 },
  ]);

  const [columnDefs] = useState([{ field: 'make' }, { field: 'model' }, { field: 'price' }]);

  return (

          <AgGridReact rowData={rowData} columnDefs={columnDefs}></AgGridReact>

          <AgGridReact rowData={rowData} columnDefs={columnDefs}></AgGridReact>

  );
};

When running this remote on its own all works fine.
But when I try to run it from another host via MF I get the same error
Object.useRef (chunk-D5PPDA7K.js:1074:29)
    at AgGridReactUi (ag-grid-react.js:69725:36)
    at renderWithHooks (chunk-BISVECJP.js:11548:26)
    at mountIndeterminateComponent (chunk-BISVECJP.js:14926:21)
    at beginWork (chunk-BISVECJP.js:15914:22)
    at HTMLUnknownElement.callCallback2 (chunk-BISVECJP.js:3674:22)
    at Object.invokeGuardedCallbackDev (chunk-BISVECJP.js:3699:24)
    at invokeGuardedCallback (chunk-BISVECJP.js:3733:39)
    at beginWork$1 (chunk-BISVECJP.js:19765:15)
    at performUnitOfWork (chunk-BISVECJP.js:19198:20)

zhangHongEn@06dc35d All that remains is to make this step automatic.

This looks very promising. Tried to test the proposed solution on the example above. Seems I am missing something, so I have to wait for proper solution. Very excited to finally get this working. Thank you

@husayt Finally finished, thank you for finding this problem, I will test it again and issue a PR tomorrow night
https://github.com/zhangHongEn/vite/commit/d79fe1859c274ce08f55510497fcf8af5b085aa9

compeleted

Thank you @zhangHongEn . I tried to run latest and examples, but I get "Please call init first" error on both host and remote examples


[ios] fix hold and drag spacebar does not move cursor when obscureTex…

Fixes #122139 with engine pr flutter/engine#40216

(triage) @justinmc @LongCatIsLooong Could one of you take a look at this one?

LGTM. The tests are failing tho, could you rebase and let the ci run again?

Done, but the cla/google test is still failing.
@justinmc  Hi, your email is not associated with your CLA, could you please associate it ?

Fixed!  Sorry, it ended up with my personal email.

@cyanglaz Hi, Google testing is always pending, could you trigger a re-run? I think this should solve it.

cc @Renzo-Olivares from triage, run tests locally.

Tests pass locally so I set the testing status to passed manually.


Fix reference document

What this PR does / why we need it?

Let JS SDK APIs be documented.

Any background context you want to provide?
TL;DR
I added two things:

typescript-transform-paths
ts-patch

Why did this happen
It was a TypeScript compiler(tsc)'s problem.
We configured path aliases on #223 but TypeScript does not resolve path aliases when compile.
Hence, the import path in the type definition file(under lib/) was an unresolved. So, api-extractor referred to the source code, not the type definition file. And that's why document was not generated.
For exmaple:
/// lib/yorkie.d.ts

// expected
export { JSONArray } from "./document/json/array";
export { PlainText } from "./document/json/plain_text";
export { RichText } from "./document/json/rich_text";

// actual
export { JSONArray } from "@yorkie-js-sdk/src/document/json/array";
export { PlainText } from "@yorkie-js-sdk/src/document/json/plain_text";
export { RichText } from "@yorkie-js-sdk/src/document/json/rich_text";

How to solve it
There's a path transformer exists to solve this problem: typescript-transform-paths
But TypeScript doesn't support custom transformers in tsconfig.json, but supports it programmatically. To use custom transformers, we need to use ts-patch.
and.. that's all!
What are the relevant tickets?
Fixes #233
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

@hackerwins let me have a look after finishing work! :)


get toolkit_id and compare name instead of id

Description

the types where wrong so the query was failing and also the comparison was done between the id and name which was also changed
Type of change

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

Checklist

[x] My pull request is atomic and focuses on a single change.
[x] I have read the contributing guide and my code conforms to the guidelines.
[x] I have documented my changes clearly and comprehensively.
[x] I have added the required tests.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Question/Meta: What is the theoretically/mathematically best scaler to use with this?

Hi!
In your imagemagick comparison you stated that this antiring filter was designed to be used with: --scale=ewa_robidoux --scale-param1=0 --scale-param2=0.5 . However in your updated mpv scalers comparison the passages referencing this scaler as well as catmull_rom for downscaling are removed.
Is this filter now recommended to be used with lanczos instead for both up- and downscaling?
Please don't respond with "try and see which looks the best", since I'm curious about the theoretical/mathematical choice here :)

It was removed from the page because the point was evaluating AR itself and not the various combinations individually.
As for whether it's recommended to be used with Lanczos, the answer is no. Catrom + AR is still a better solution than Lanczos + AR for downsampling. You can use the shader with any filter, it wasn't "designed" with Lanczos in mind.
AR becomes more important as the filter becomes sharper, but AR itself can be used with anything to remove the overshoots and (hopefully) increase image quality. It works particularly well with polar Catrom (the filter you mentioned) because polar Catrom is exceptionally sharp, but it works even better with a 2-lobes Sinc for example.
My advice is, treat is as an alternative to the built-in AR functionality. It's sometimes better and sometimes worse, but generally speaking the solutions are close enough and the difference is imperceptible.


Add some contributor docs for debugging testsuite tests.

This adds some suggestions on how to debug testsuite tests.
Closes #9757

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit f437af3ec30830d0d9ab0391572378d7e3b6025c has been approved by alexcrichton

:hourglass: Testing commit f437af3ec30830d0d9ab0391572378d7e3b6025c with merge a2bed9380c909a67d9423c86388d40c36c8f7b36...

:broken_heart: Test failed - checks-actions

This can probably be retried now, since #9906 has been merged.

@bors: retry

:hourglass: Testing commit f437af3ec30830d0d9ab0391572378d7e3b6025c with merge 72ef2bd8ccc354926488164f013e0813f19349a8...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 72ef2bd8ccc354926488164f013e0813f19349a8 to master...


fixfix outer join bug when a subquery as nullable side

Proposed changes
Issue Number: close #xxx
Problem summary
When the subquery's root node is a agg node, the output slot may come from the child outer join node's nullable side. This nullable info is lost when creating the output tuple desc for agg node. This pr fixed this problem.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

@morrySnow


Fix issue 1033: access operator on vle edge lists (#1037)

This patch fixes an issue where the edge lists generated by the vle function couldn't be resolved by the agtype_access_operator.
Added regression tests.
A big thanks to Taha for showing that this function could be simplified.

Looks good


Issue with registering instance of type

Hello
I have a very specific use-case that need to use DryIoc for:

I have created some sample code that illustrates the issue I am having
I need to use the container in a custom wrapper and register a few types that depend on the same interface
I need to be able to register types by both name or by an instance of the type
One of the types, that haw parameter constructors, is passed on as an instance.
The Register() for the instance type passes by the Resolve() for the same type fails.
Thanks
Herald

using DryIoc;
using System;

namespace ConsoleApp1_TestDryIOC
{
    internal class Program
    {
        static void Main(string[] args)
        {
            var w = new Wrapper();
            w
                .Register<A>()
                .Register<B>()
                .Register<C>(new C("first", "second"));

            w.Resolve<A>().Execute("A1", "B1");
            w.Resolve<B>().Execute("A1", "B1");
            w.Resolve<C>().Execute(); //THIS THROWS AN ERROR
        }
    }
}

class Wrapper
{
    Container c;
    public Wrapper()
    {
        c = new Container();
    }

    public Wrapper Register<T>() where T : I
    {
        c.Register(typeof(I), typeof(T), reuse:Reuse.Singleton, serviceKey: typeof(T).Name);

        return this;
    }
    public Wrapper Register<T>(T instance) where T : I
    {
        c.Register<I>(made: Made.Of(() => instance), serviceKey: typeof(T).Name); //DON'T THINK THIS IS RIGHT, AS THE Resolve() THROWS ERROR

        return this;
    }

    public I Resolve<T>() where T : I
    {
        return c.Resolve<I>(serviceKey: typeof(T).Name);
    }
}
interface I
{
    void Execute(string a =null, string b=null);
}
class A : I
{
    public void Execute(string a, string b)
    {
        Console.WriteLine($"a-{a}:b-{b}");
    }
}

class B : I
{
    public void Execute(string a, string b)
    {
        Console.WriteLine($"a-{a}:b-{b}");
    }
}

class C : I
{
    readonly string a1;
    readonly string b1;
    public C(string a, string b)
    {
        a1= a;
        b1= b;
    }
    public void Execute(string a = null, string b = null)
    {
        Console.WriteLine($"a-{a1}:b-{b1}");
    }
}

Hi @hgjura
What version of DryIoc are you using?
If it is 4.x.x then you might use RegisterInstance instead of Register(Made.Of(..)) which is not supposed to be used for the instance anyway.

That was rather easy. Thanks. It works perfectly now.


feat(helm): Add ability to customize affinity per deployment

SUMMARY

Added ability to add affinity per deployment
Added ability to add topologySpreadConstraints https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Specify supersetNode.affinity
Affinity should be added to helm template result
Specify global affinity
Both affinity should be added to helm template result

ADDITIONAL INFORMATION

[ x ] Has associated issue: https://github.com/apache/superset/issues/19250
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

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

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

Please bump the version in Chart.yaml and re-gen the README using helm-docs

@craig-rueda Done

Hello, I want my helm-swapped superset to be evenly distributed among the nodes of my cluster and I write like this:
# -- TopologySpreadConstrains to be added to all deployments
topologySpreadConstraints:
    - maxSkew: 1
      topologyKey: kubernetes.io/hostname
      whenUnsatisfiable: ScheduleAnyway

But the pods are still not evenly distributed, does anyone have an example of how this works?
@sergeyshevch Maybe you have an example of even distribution between nodes?

@a-poluyanov You also need to add label selectors
Example:
    topologySpreadConstraints:
      - maxSkew: 1
        whenUnsatisfiable: DoNotSchedule
        topologyKey: kubernetes.io/hostname
        labelSelector:
          matchLabels:
            app: superset

Selectors are quite different across deployments
app: superset
app: superset-worker
etc


Add Enclaves DEX

Thanks, but do you have something more authoritative for this application? Few people are likely to trust a gmail address to report critical security information.

@dguido I added the domain specific email of<EMAIL_ADDRESS>Thanks,
Adam


Pyceres requires colmap == 3.9.1

I tried to install pyceres from source and it requires colmap == 3.9.1, which has conflicts with the requirements of pixel perfect sfm(colmap 3.8). I tried to install colmap 3.9.1 and adjust the head files and references under pixel perfect sfm but it seems failed. How can I solve this problem?
Thank you!

Use COLMAP 3.8 with the versions pinned here:
https://github.com/cvg/pixel-perfect-sfm/blob/40f7c1339328b2a0c7cf71f76623fb848e0c0357/requirements.txt#L1-L2

I use pip install pyceres==2.0 and it works with colmap 3.8


[5.1] IRGen: Work around JIT bug with GOT equivalents.

SR-10590 | rdar://problem/50968433 appears to be caused by an LLVM JIT bug where GOT-equivalent
globals get emitted with incorrect alignment. Not marking the GOT equivalent as unnamed_addr
(which prevents it from getting folded into the GOT, or other equivalent constants) appears
to work around the issue.
rdar://problem/50971519

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - d48146b5b11f476ea92c19be675c1cd67d0b3e59

Build failed
Swift Test OS X Platform
Git Sha - d48146b5b11f476ea92c19be675c1cd67d0b3e59

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - d48146b5b11f476ea92c19be675c1cd67d0b3e59

Build failed
Swift Test OS X Platform
Git Sha - d48146b5b11f476ea92c19be675c1cd67d0b3e59

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - c00bec85e3bbab16c6e839447584c2fb3ba18db0

Build failed
Swift Test OS X Platform
Git Sha - c00bec85e3bbab16c6e839447584c2fb3ba18db0

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 9cde02e93e250b96a266701db3a1725f4c3a5fd8

Build failed
Swift Test OS X Platform
Git Sha - 9cde02e93e250b96a266701db3a1725f4c3a5fd8


Allow class namespacing

Problem

I closed it because it is a work in progress that has not been thought through or used enough for me to feel confident it is a good solution.
We have come across two issues with Shale:  prepending namespaces (as with this PR) and setting the correct Shale type from real world XSD files (which do not use the ns1 namespacing).
https://github.com/kgiszczak/shale/commit/b7d8d8506945dbe93822bbfae2e1673f968c32ef
We are currently using the patched version of Shale with some production code.
Once we are done with our deadline, I am going to circle back around and clean this up and present a PR.
We are really happy to have Shale, we have been using a JAX parser in JRuby to convert XSD's to Ruby mapper objects – Shale is much nicer.

@jweir I'm really happy you like Shale and even more you use it in production. Waiting for the PR :)


GIT_EBUFS error being thrown on successful push the AWS CodeCommit repository.

After pushing to changes to a git repository on AWS CodeCommit using SSH based credentials, an error is thrown, however the push to AWS is successful.  The error message is
Error: Method push has thrown an error.
and the error number is {"errno":-6}, which I've tracked back to GIT_EBUFS found here https://github.com/nodegit/nodegit/blob/44779afd0d460011e13a5cba9a47a48119aa472e/vendor/libgit2/include/git2/errors.h#L28
I've got a reproduction case here : https://gist.github.com/Murcho/fcba744b2870fa38531f
Requirements for the case are:

Set up an AWS CodeCommit account.
Create an IAM user account.
Follow this guide for setting up the AWS IAM account with your SSH keys http://docs.aws.amazon.com/codecommit/latest/userguide/setting-up-without-cli.html
Add 2 permissions to the IAM user : AWSCodeCommitFullAccess and IAMUserSSHKeys.

@cbargren @Mr-Wallet this might be of use to you guys.


crash at showing progress dialog

android.view.WindowLeaked: Activity  has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView{360ddfca V.ED.... R......D 0,0-108,108} that was originally added here
code:
            hud = KProgressHUD.create(this)
                    .setStyle(KProgressHUD.Style.SPIN_INDETERMINATE)
                    .setCancellable(false)
                    .setAnimationSpeed(2)
                    .setDimAmount(0.5f)
                    .show();

Yes I have the same issue it happens when there is Screen rotation. I have spinning progress then rotate, and in the same time there should be success KProgressHUD ... .show()

Maybe here:
public KProgressHUD show() {
if (!isShowing()) {
mFinished = false;
if (mGraceTimeMs == 0) {
mProgressDialog.show();
} else {
mGraceTimer = new Handler();
mGraceTimer.postDelayed(new Runnable() {
@Override
public void run() {
if (mProgressDialog != null && !mFinished) {
mProgressDialog.show();
}
}
}, mGraceTimeMs);
}
}
return this;
}
Should be something like this
val activity = (context as? Activity)
if(activity != null && activity.window.decorView.isShown) {
//Show Your KProgressHUD
}
I tested outside as wrapper of hud.show() and it sometimes helps but not always as sometimes activity.window.decorView.isShown return true and as before detaching.

I encountered the same issue


Fix stable diffusion pipelines

What does this PR do?

Fixes #547.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


No instructions for how to use with bower

I see that there is a bower.json in this repo, but no instructions for how to install using bower.
Could some be included in the README?

There is example, where jsroot installed via bower:
https://github.com/linev/jsroot-example

Could some be included in the README?
I will do it.

Sounds good.
Those instructions do not work on my setup, but it might be my fault. I'll raise a new issue if I think it is a bug.

Those instructions do not work on my setup, but it might be my fault.

Did you try to clone jsroot-example and run "npm start"?
I test it shortly after last 5.0.0 release - it was working.

We use a gulpfile, which takes an array of values for javascript filenames, so passing an argument via URL is not so trivial. Here is the file we load using gulp:
$ cat vendor.base.json
[
  "bower_components/jquery/dist/jquery.js",
  "bower_components/lodash-compat/lodash.js",
  "bower_components/angular/angular.js",
  "bower_components/angular-route/angular-route.js",
  "bower_components/angular-cookies/angular-cookies.js",
  "bower_components/angular-animate/angular-animate.js",
  "bower_components/angular-ui-router/release/angular-ui-router.js",
  "bower_components/ngstorage/ngStorage.js",
  "bower_components/angular-ui-utils/ui-utils.js",
  "bower_components/angular-sanitize/angular-sanitize.js",
  "bower_components/angular-resource/angular-resource.js",
  "bower_components/angular-translate/angular-translate.js",
  "bower_components/angular-translate-loader-url/angular-translate-loader-url.js",
  "bower_components/angular-translate-loader-static-files/angular-translate-loader-static-files.js",
  "bower_components/angular-translate-storage-local/angular-translate-storage-local.js",
  "bower_components/angular-translate-storage-cookie/angular-translate-storage-cookie.js",
  "bower_components/oclazyload/dist/ocLazyLoad.js",
  "bower_components/angular-bootstrap/ui-bootstrap-tpls.js",
  "bower_components/angular-loading-bar/build/loading-bar.js",
  "bower_components/restangular/dist/restangular.js",
  "bower_components/checklist-model/checklist-model.js",
  "bower_components/jsroot/scripts/JSRootCore.js"
]

I'm not yet sure whether URL parameters can be passed using gulp. However, if bower was the default (so no URL parameters needed to be passed), it would save me a lot of time.
If this is something you'd like to discuss, I'd be happy to open a new issue about it.

I'm not yet sure whether URL parameters can be passed using gulp.

Can you check this? Script name should be:
"bower_components/jsroot/scripts/JSRootCore.js?bower"

However, if bower was the default (so no URL parameters needed to be passed), it would save me a lot of time.

JSROOT used very rarely via bower and main (default) use-case is definitely not a bower.
One could use some semantics  - like if path name contains "bower_components/jsroot/scripts", bower support will be automatically enabled

I tried that, it didn't work:
lhcbpr2fe_1     | [12:33:55] ✗ FAIL: Missing 1 expected files: bower_components/jsroot/scripts/JSRootCore.js?bower

Automatic support would work well.
Is there a reason bower/a package manager is not the default use case? Many other js libraries use multiple package managers without problems, although I'm not sure how they do it.

Many other js libraries use multiple package managers without problems

JSROOT supports require.js and bower. One can add some more (if necessary).

Is there a reason bower/a package manager is not the default use case?

Main reason - keep dependencies minimal.
At the moment default use-case - graphical interface for THttpServer in ROOT.
Here neither bower nor require.js are used.
I propose intermediate solution - enable bower support when "bower_components/jsroot/scripts" appears  in the scripts path. Are you agree?

Yes, I think that will work well for now.

Now in 5.1.0 release


npm audit issue associated with async lib <= 2.6.2

I upgraded to async 2.6.3 & the audit issue went away:
https://github.com/caolan/forms/pull/209

Fixed.


New package: DiscreteAxis v0.1.0

Registering package: DiscreteAxis
Repository: https://github.com/xtalax/DiscreteAxis.jl
Created by: @xtalax
Version: v0.1.0
Commit: 8ef742043980ea027413d3d6327366f56889cb59
Reviewed by: @xtalax
Reference: https://github.com/xtalax/DiscreteAxis.jl/issues/1

Please add (at least)
[compat]
julia = "1"

to your project file and retrigger the bot.

bump @xtalax


capture mutable fields changes

the current implementation does not capture any changes made to mutable data structures like dict  because  _snapshot_state does a shallow copy of the initial state, so any changes to mutable data structure will change both the initial state and the current state preventing any hook listening to changes from working.
This fix is to use a deep copy of the initial model.__dict__

@EnriqueSoria Yes for very big and complex models the performance will be affected by deepcopy, i think in this case we can use some tricks like
ujson.loads(ujson.loads(mode.__dict))

which is very fast compared to deepcopy
but for most of cases this will be just fine
Thanks for the review


oauth2 confusion

Hi,
I'm trying to run the 'oauth2.js' sample. I'm pretty sure I have put in the correct client ID/secret; I have also set REDIRECT_URL = 'urn:ietf:wg:oauth:2.0:oob'. No other change is made.
But I just keep getting this error:
An error occured Error: No access or refresh token is set.
at OAuth2Client.getRequestMetadata (/Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/google-auth-library/lib/auth/oauth2client.js:271:23)
at OAuth2Client.request (/Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/google-auth-library/lib/auth/oauth2client.js:358:15)
at createAPIRequest (/Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/googleapis/lib/apirequest.js:180:22)
at Object.Plus.self.people.get (/Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/googleapis/apis/plus/v1.js:227:14)
at /Users/weiwe/WebstormProjects/adxseller2fbmessenger/oauth2.js:60:15
at /Users/weiwe/WebstormProjects/adxseller2fbmessenger/oauth2.js:47:16
at /Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/google-auth-library/lib/auth/oauth2client.js:154:5
at Request._callback (/Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/google-auth-library/lib/transporters.js:67:30)
at self.callback (/Users/weiwe/WebstormProjects/adxseller2fbmessenger/node_modules/google-auth-library/node_modules/request/request.js:198:22)
at emitOne (events.js:96:13)
It seems it couldn't get the correct tokens; thus, when the code tries to retrieve Plus info, it throws the error. What am I doing wrong here?

The weird thing is it works fine now, although I don't think I have made any chances ....


artisan test hides PHPUnit deprecations

Laravel Version: 10.0.2
PHP Version: 8.2.1
Database Driver & Version: -

Steps To Reproduce::
(have a test with a PHPUnit deprecation)

Run tests with php artisan test
Run tests with ./vendor/bin/phpunit

Description:
Notice that deprecation are not thrown/visible until PHPUnit is invoked directly.
(php artisan test)
  Tests:    232 passed (626 assertions)
  Duration: 12.57s

(vendor/bin/phpunit)
➜  Leaf git:(master) ./vendor/bin/phpunit
PHPUnit 10.0.7 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.2.1
Configuration: xxx/PHP/Leaf/phpunit.xml

D..............................................................  63 / 232 ( 27%)
............................................................... 126 / 232 ( 54%)
............................................................... 189 / 232 ( 81%)
...........................................                     232 / 232 (100%)

Time: 00:11.870, Memory: 68.50 MB

There were 19 PHPUnit deprecations:
1) Tests\Unit\Models\CsrModelTest::testRankAttribute
Data Provider method Tests\Unit\Models\CsrModelTest::rankDataProvider() is not static
...

To be fair this is a Collision issue. Going to work this week on this.


runas isn't running synchronously on macOS

runas on macOS Sierra in an Electron application is immediately returning with an exit status of -1.
You can still provide your fingerprint or credentials to run the command but you won't know if it succeeded or not.
Please let me know if you need any more details.

For what it's worth, it's synchronous when using child_process.fork().
However, when using fork() it doesn't use the the Electron app's icon, it appends "Helper" to the name, and it doesn't remember the authentication.

All repositories in the atom-archive organization are unmaintained and deprecated. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Check TableConfig.cdfEnabled when queryTable has version parameter

Check TableConfig.cdfEnabled when queryTable has version parameter

Should this be part of the implementation of supporting version parameter in QueryTable API?

Should this be part of the implementation of supporting version parameter in QueryTable API?

You are right, but that one is already merged...

Why is reading table be versions limited only to tables with CDF enabled ?
Isn't it possible to extract the info of the difference between a version and another by only knowing which files were added and which were removed ?
Supposedly replicating the metadata I am mentioning should lead to the same of effect of the CDF, correct ?


Add single-letter aliases for --dev and --port

It would be very handy to be able to write lume -sd or lume -sdp 80, for example.

I looks good to me. Thanks!


Peak data calculation

Checking data for Italy https://covid-gauss.site/country/italy it looks like the peak data are not well representing the gaussian estimation.
For istance, as for today, the peak is given at:

date: 2020-04-06
estimated new infections: 4162

When maybe:

date: 2020-03-28
estimated new infections: 5500

would be more meningful, as it is the peak of the gaussian curve too.
Not use that using the median on valuableData at this point is a good idea.
https://github.com/eatthatpie/covid-gauss/blob/6bb5cf53e4f723ac76ba06febc13be8e28ee645e/server/createEstimationReport.js#L109
What do you think?

Hi, thanks for catching it.
Strange as it sounds but I didn't want for the results to rely only on abstract maths so I decided that the timespan whose middle point will be treated as the peak will not be the Gaussian curve itself but it will start with actual data and end with estimated values for the future. In that approach the peak date can surely differ from the peak of the curve.
However it seems that people are misled by the difference between the shape of the curve and the calculated peak. So maybe adding another label (indicator?) on the country page like "Gaussian curve peak and estimated infections for that day" (or sth like that, you get the idea) would solve the issue?

If it's made on purpose, nevermind, so my was just a personal opinion. I think the median is weighing too much the tail of the gaussian, pushing the peaks on the right side, resulting in a "bad" estimation.
South Korea  is another example where this approach tends to fail.
Not sure if adding another indicator would help or clarify, because I believe in that case, the "gaussian curve peak" would be more precise than the actual one, so..why having two different numbers if they are both estiamtions? But again, just personal thoughts.


Remove VisualStudio reattach deadcode

PR Details
Description
PR focuses on removing an unnecessary piece of code. The code attaches the Visual Studio debugger after every program reset to the GameStudio process in the NET Framework target only, making this code basically dead. I recommend attaching through VS directly. See article about reattaching to process or use ReAttach  plugin instead.
Types of changes

[x] Refactoring / dependency upgrade

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I think this was made specifically to catch issues that occur early in the application lifetime, going through the motion to reattach the debugger to the process may take too much time.
I'll merge this in since as you said the gamestudio already lost that feature but I think this could still be nice to bring back.


[Closed] add CheckPalindromeNumber.ts #2001

Added CheckPalindromeNumber.ts
Closes #2001

Hey, I am sorry -- but this issue had been assigned to me. I'll create a PR in some time, but just so you know - you can only open PRs for Issues you've been assigned.

Yep, got it!

Среда, 5 октября 2022, 15:47 +03:00 от rwlc @.***>:

Hey, I am sorry -- but this issue had been assigned to me. I'll create a PR in some time, but just so you know - you can only open PRs for Issues you've been assigned.
—
Reply to this email directly,  view it on GitHub , or  unsubscribe .
You are receiving this because you were mentioned. Message ID:  <codinasion/program/pull/2051/c1268390293 @ github . com>

--
Саша Сдобников
Отправлено из Почты  Mail.ru


Make XChange compatible with jakarta.ws.rs-api 3.x

As far as I can see, the jakarta.ws.rs-api is pulled transitively via com.github.mmazi:rescu. Once a com.github.mmazi:rescu release compatible with jakarta.ws.rs-api 3.x is available, XChange should be adjusted as well. This is currently blocked by https://github.com/mmazi/rescu/issues/142
It would be nice to have this to be able to use XChange on Quarkus 3.x.

@ppalaga Peter, use this if needed - https://github.com/martinformi/XChange/tree/feature/jakarta-namespace

@ppalaga Peter, use this if needed - https://github.com/martinformi/XChange/tree/feature/jakarta-namespace

Great, would you mind sending a pull request against https://github.com/knowm/XChange ?

Thanks for the PR. It is up to the project maintainers to decide how they deal with the incompatibility. I'd personally be fine with bumping the major version as having both xchange and xchange-jakarta dependencies in one project would not work anyway - it would not be possible to have transitives compatible with both of them.

@ppalaga looks like the MR https://github.com/knowm/XChange/pull/4735 resolves it. Feel free to test and eventually close the issue

is already fixed?

yes, issue can be closed


python311Packages.pyacaia-async: 0.0.7 -> 0.0.8

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.pyacaia-async
python310Packages.pyacaia-async.dist
python311Packages.pyacaia-async
python311Packages.pyacaia-async.dist


Admin: manage content page

Before (stock drupal):

After a little tweaking:

Questions:

Should we try a card-based view, or do we feel like the table-based view is best for content.

@lukefretwell @aschmoe thoughts, addl ideas for improvement?

Table-based is standard and looks clean, perfect.
Row color variations do a good job of separating each content item, so lines inside the table are overkill.
I'd use the same color across the whole row ('Updated' column looks odd in a different color.).
'new'/'updated' text would be fine in a grayer color.
The filter/selected is a little confusing and think text needs tweaking, so not sure if that applies to what you're doing here or if I need to email Dries directly.

Luke's points cover most everything.
"The filter/selected is a little confusing and think text needs tweaking, so not sure if that applies to what you're doing here or if I need to email Dries directly."
Think we could do some visual tying material design style.  Clicking on one of the selects pulses a shadow on the "With selected" box or something

I don't think we should (or are able to) imitate WordPress, but think they do a good job of simplifying/combining the filter/selection features.
With respect to bulk actions, I'd just borrow from WP:

remove 'With selected:' text.
change default menu text to 'Bulk actions'
button text: 'Apply'

Updated: http://pattern.getproudcity.com/?v=&page=admin-content

Filters look a lot more "tight" (D.O.P.E.) think this is ready for done.

I realized after inspecting the code that this might not be possible without some css trickery. There are two separate forms: the top filters (views exposed form), and the bulk operations form, which includes the views results table.
Do you think this is possible + worth the effort @aschmoe ?

Seems like we can definitely get them side by side floating at the very least.  Will try a treatment in drupal.

Here's a quick treatment in the stock drupal, I think we can get something similar without too much re-jiggering

nice


Separated class and struct constructor and destructor

So they can be styled differently.

It's really just some fancy taste of me. I like the .ctors to have the same color as their type, be class or struct.

@laicasaane, I moved the issue's part about a destructor to the separate issue #26


Lando usage

Hello,
Is there anybody who successfully use lando on a cloud server like DigitalOcean with code-server?
I am not a network expert and I don't know how to configure proxy and DNS stuff: for the moment I have an up and running code-server running on a DigitalOcean Droplet with a custom domain name and lando is starting by overriding proxyHttpsPort set to 444 but I cannot access the running web app I have launched via lando.
Have a good day.
Best Regards,

Also highly recommend accessing launched web apps via code-server.
See https://github.com/cdr/code-server/blob/master/doc/FAQ.md#how-do-i-securely-access-web-services

Thank you @nhooyr!
Have a good day.
PS : my current solution is to disable proxy in Lando and configuring Caddy properly.

Nice! Caddy is great :)

Guys, have you had an issue when lando not working properly with code-server?  I can execute lando via SSH to my server but if I want to use the vscode terminal it does nothing ...
Any clues? Any place to debug vscode to see what's going on?
Thank you!

@asmerkin might be better to ask as a GitHub Discussion!


Set Docker memory limit

Adds an 8GB memory limit to the updater step Docker container.
We opt for this limit since it's the amount we designate to our largest firecracker VM and certain ecosystems like Python can take a lot of memory to run an update.
Before this PR is deployed we should run the ghes-loadtest against it

Had to sign commits because of branch protection rules. Will merge after CI passes


Testing and Documenting of Each Ubuntu Box

As a user, I want my ubuntu box installations to work without issue.

Document: success of build, what had to be done to build, ease of build, time to build, any other comments. If build failed, document the output and exactly what you did.


Challenge: how_sum

Find a combination of numbers from numbers that add up to the target.

@adel-batal Thank you!


Added KiCad

Project URL
https://gitlab.com/kicad/code/kicad
Category
Development
Description
Add KiCAD EDA Suite
Also removed unnecessary trailing commas in the json
Why it should be included to Awesome macOS open source applications  (optional)
KiCAD is a really useful and well-known tool for designing and exporting electrical schematics and printed circuit boards.
Checklist

[x] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[ ] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

@Devnol Thank you 🚀

You're welcome!


Vm to docker/images

Description
This PR includes

documentation for the containers infrastructure
a little change in the novnc container, that now shows the novnc toolbar by default (and can be hidden through an ENV flag)

First, great for this commit!
I would like to suggest the following changes.

[ ] Add a small paragraph in file https://github.com/netgroup-polito/CrownLabs/tree/master/provisioning in order to introduce what's in the container folder.

[ ] Add a small README in the https://github.com/netgroup-polito/CrownLabs/tree/master/provisioning/containers folder in order to explain what's in there. Also say what the user has to do to create new applications in there (so, what it can be reused, what else has to be written from scratch).

[ ] Add a README in the https://github.com/netgroup-polito/CrownLabs/tree/master/provisioning/containers/gui-common folder, explaining the overall architecture of the solution (which containers are being created, what they are used for, how do they interact).

[ ] In general, add a README to all the folders that you created. I don't want to repeat myself, but who comes here may not know what is "fluxbox", what's in there, what is this used for. Documentation is the gold key to achieve maintainability.

[ ] Document more the Dockerfile you wrote. It would be very nice if all the commands in there are preceded by a small comment saying what you're doing and (even more important) why is this command needed, if this is meaningful (do not write obvious things, otherwise people would not read anymore).

In case some pictures are needed, you can consider uploading either in PNG or (if are supported by github; not sure, honestly) in SVG, so that it would be easier to modify and update in the future. Remember that pictures are often much more useful than a lot of text.
Thanks!

First, I apologize... I pushed on the "update branch" button by mistake :-(
Second, this PR looks rather good to me . Please:

review the text that I wrote, just to make sure that I understood everything correctly
please document file nginx.conf.template, whose directive do not look obvious to me
I wonder if we have to document the 'USER' and 'UID' which are set in all Dockerfiles

Finally, I would suggest to transform the websockify Docker into a multi-stage build:

you create a first docker that downloads the code, compiles it
then, you create, from scratch, a second docker that takes the result from the former (i.e., the executable), copies into it, and terminates. So, the user can simply use the second Docker, discarding the first.
Thanks!

First, I apologize... I pushed on the "update branch" button by mistake :-(
Second, this PR looks rather good to me . Please:

review the text that I wrote, just to make sure that I understood everything correctly
please document file nginx.conf.template, whose directive do not look obvious to me
I wonder if we have to document the 'USER' and 'UID' which are set in all Dockerfiles

Finally, I would suggest to transform the websockify Docker into a multi-stage build:

you create a first docker that downloads the code, compiles it
then, you create, from scratch, a second docker that takes the result from the former (i.e., the executable), copies into it, and terminates. So, the user can simply use the second Docker, discarding the first.
Thanks!

/rebase

/rebase

/rebase

/rebase

/merge

/merge


Update nvidia packages

This makes gpu work out of the box at least on ubuntu 20.04

Thank you for this update. Your commit is merged in the PR: https://github.com/ubuntu/microk8s/pull/1299


[SPARK-28031][PySpark][TEST] Improve doctest on over function of Column

What changes were proposed in this pull request?
Just found the doctest on over function of Column is commented. The window spec is also not for the window function used there.
We should either remove the doctest, or improve it.
Because other functions of Column have doctest generally, so this PR tries to improve it.
How was this patch tested?
Added doctest.

cc @HyukjinKwon

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11672/
Test PASSed.

Test build #106429 has started for PR 24854 at commit 6afd936.

Test build #106429 has finished for PR 24854 at commit 6afd936.

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

Merged build finished. Test PASSed.

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

Merged to master and branch-2.4.


Add scala-native-p, a NIR deserialization program

It is based on the dumping feature of Scala Native, but slightly redesigned to allow directly passing regular files and printing to stdout instead of saving to a file.
It features --help, --version and --cp options. The last one allows to pass a classpath where the NIR files are meant to be found. If not passed, the files are looked for in cwd.

Closing, as this functionality was merged in #8


Make synchronized files easier to recognize

Thank you so much for creating such a wonderful and useful tool!
What
Make synchronized files easier to recognize.
Why
When synchronizing for the first time, a lot number of articles will be imported who have already written many articles, however the id is not readable and understandable by people.
Suggestion
Add created date to the head of the file.
e.g. 20220120_d054b95f68810f70b136.md

Hi kyntk,
thank you for your suggestion!
I think it's a great idea and I would like to implement your idea in future release.
I will come back to you when it gets ready.

Thank you so much for your very quick response and implementation!


Depicting a Point in an UI using DearPyGUI

@AAPPHH
There is the code you asked me for about showing how to depict a point in DearPyGUI.
I also depicted a line, a text and an arrow because why not. I can.
Code is in "gui.py".

Nice to know it works that easily


Workspace - The default room opening after creating a workspace is #announce

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4278685&group_by=cases:section_id&group_id=229065&group_order=asc
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to staging.chat.expensify.com
Click on the "Choose a workspace" button and click on the + button next to "Workspaces" to create a new one
Navigate to "Chats" by using the bottom nav

Expected Result:
It should be #admins
Actual Result:
The default room opening after creating a workspace is #announce
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/0ad77a54-00ea-43e7-aac6-dcfded003304
View all open jobs on GitHub

We think that this bug might be related to #wave6
CC @greg-schroeder

I can take this over since its related to ideal nav

Proposal
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
The sortReportsByLastRead function is used to get the last read report for ReportScreenIDSetter to set a report ID.
https://github.com/Expensify/App/blob/b7a3fcaf353cae13f7d685a1f26145ad1554bf3a/src/libs/ReportUtils.ts#L1003
However, the value of lastReadTime in optimistic adminsChat and announceChat are equal hence, the ReportUtils.findLastAccessedReport function will return the announceChatReportID for the ReportScreenIDSetter.
https://github.com/Expensify/App/blob/b7a3fcaf353cae13f7d685a1f26145ad1554bf3a/src/libs/ReportUtils.ts#L3290
https://github.com/Expensify/App/blob/b7a3fcaf353cae13f7d685a1f26145ad1554bf3a/src/libs/ReportUtils.ts#L904-L916
Furthermore, I noticed, in the backend response, that the value of lastReadTime of the announceChat is greater than that of adminsChat and is a contributing factor to the issue due to the nature of the sortReportsByLastRead function.
What changes do you think we should make in order to solve the problem?

At the backend, we should create the admin chat before the announce chat. H
ence, the response must return the value of adminsChatData.lastReadTime to be greater than announceChatData.lastReadTime.
For the offline fixes, we should set adminsChatData.lastReadTime a couple of milliseconds from the rest of the chats by using a function such as DateUtils.subtractMillisecondsFromDateTime just like the backend.
What alternative solutions did you explore? (Optional)
Option A:
If the backend fix is not feasible, we could update the lastVistedTime after the Policy.createWorkspace function has finished executing in the savePolicyDraftByNewWorkspace.
This will force the sortReportsByLastRead function to use lastVistedTime instead of lastReadTime.
Option B:
Apart from navigating to ROUTES.HOME, in the BottomTabBar component, we can modify the route taken by the Navigation.navigate function to the policy's admin chat.
Only if it is a newly created policy.
https://github.com/Expensify/App/blob/b7a3fcaf353cae13f7d685a1f26145ad1554bf3a/src/libs/Navigation/AppNavigator/createCustomBottomTabNavigator/BottomTabBar.tsx#L42-L47


question about the experimental results

Dear author, when I use the pretrained model of CRNN, the accuracy of each evaluation dataset cannot reach the result reported in the paper. I wonder what is wrong with that? Thank you.

Hello,
First of all, it should be slightly different because the reported value is the average value of three different trials.
(See the "evaluation metric" part of subsection 5.1 Implementation Detail in our paper.)
If there is a huge difference between the reported value and the accuracy of the pretrained model, it seems to be something wrong.
In this case, can you upload your evaluation log (the results of the pretrained model)?
To check something wrong or not, seeing how much difference between the results can be helpful.
Hope it helps.
Best

Sorry to bother you again. I have evaluated the performance of TRBA on six benchmarks with the pre-trained model "TPS-ResNet-BiLSTM-Attn.pth" and the accuracy of each dataset is as follows:
accuracy: IIIT5k_3000: 87.367   SVT: 87.326     IC13_1015: 92.217       IC15_2077: 75.390       SVTP: 80.155    CUTE80: 74.216  total_accuracy: 83.834  averaged_infer_time: 1.379     # parameters: 49.555
but I cannot find the corresponding results in the published paper. Did I get anything wrong?

Hello,
"TPS-ResNet-BiLSTM-Attn.pth" is a different model.
"TPS-ResNet-BiLSTM-Attn.pth"
https://www.dropbox.com/sh/j3xmli4di1zuv3s/AAArdcPgz7UFxIHUuKNOeKv_a?dl=0
is the pretrained model of the ICCV2019 paper "What Is Wrong With Scene
Text Recognition Model Comparisons? Dataset and Model Analysis."
In our CVPR2021 paper "What If We Only Use Real Datasets for Scene Text
Recognition? Toward Scene Text Recognition With Fewer Labels,"
we used the real data for training, and you can download pretrained models
of the paper in here
https://www.dropbox.com/sh/23adceu2i85c4x1/AACLmaiL43Jy8eYIVVUkZ344a?dl=0.
Check "TRBA-Baseline-real.pth"
Hope it helps :)
Best
2022년 4월 24일 (일) 오전 12:19, hyangyu @.***>님이 작성:

Sorry to bother you again. I have evaluated the performance of TRBA on six
benchmarks with the pre-trained model "TPS-ResNet-BiLSTM-Attn.pth" and the
accuracy of each dataset is as follows:
accuracy: IIIT5k_3000: 87.367 SVT: 87.326 IC13_1015: 92.217 IC15_2077:
75.390 SVTP: 80.155 CUTE80: 74.216 total_accuracy: 83.834
averaged_infer_time: 1.379 # parameters: 49.555
but I cannot find the corresponding results in the published paper. Did I
get anything wrong?
—
Reply to this email directly, view it on GitHub
https://github.com/ku21fan/STR-Fewer-Labels/issues/6#issuecomment-1107519767,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB4V4M2C2UPZOWHXJAVBCATVGQIJNANCNFSM5UEKQPJA
.
You are receiving this because you commented.Message ID:
@.***>


The problem with UrlEncode/UrlDecode！

Hi，Arut ！
I use the module to build the webdav server.when I upload a file with Chinese character name,the filename is  你好 ,But it dispaly like "%E4%BD%A0%E5%A5%BD" ,I can't solve the case! please help,thank you!

the 0.0.3 version currently used in debian still shows the issue codywon described. #17 looks like a good patch for it.

This seems to be fine in the v2 branch, which will eventually become a new release.

v2.0.0


add submodule revision change helper script

This PR adds the sub-rev-change.sh script, that eases the update of the submodules to a desired revision.
A branch or a tag can be used, and optionally a remote can be provided.
Also, remote(s) can optionally be fetched before looking for the desired revision.
If the requested revision cannot be found, nothing is done, to allow the change on the remaining submodules.

I still do not understand how to use git submodule system properly, so I assume it should be @zoedberg reviewing this

Please briefly document this on the README, then I'll merge this

commit updated to include a new Contribute section that briefly documents its presence and usage


go: read in log time order with overlapping chunks

Public-Facing Changes
Adds a reverse argument to McapReader.Messages(), letting a user read in reverse order.
Indexed MCAP reads will now return messages in log time order, even if there are chunks with overlapping time ranges.
Description
This PR changes the approach to reading messages using the indexed reader.
Before
The previous implementation had one active chunk, and all messages were read out of it in ascending log time order before moving on to the next chunk. If two chunks are present in an MCAP with overlapping time ranges, this would cause messages to be read out-of-order.
After
This PR changes to a scheme where there is one heap containing chunk indexes and message index entries that match the search criteria.
When a chunk index reaches the top of the heap, the corresponding chunk is decompressed and all of its message index entries are pushed to the heap.
Message index entries in the heap are stored alongside a pointer to the underlying decompressed chunk. There is no "owning" pointer to the decompressed chunk data, so we now rely on the garbage collector to clean up the decompressed chunk once all message index entries are consumed.
Additionally, since it's easy enough, this PR adds a flag to iterate through the MCAP in reverse time order. This can be used to search for the first message on a topic before a given timestamp.
Fixes #533.
TODO

[x] Add reader test with overlapping chunks
[x] Add test to read in reverse order

Rather than increasing the arity of Messages, would it make sense to switch to a "functional options" pattern as described in here? https://golang.cafe/blog/golang-functional-options-pattern.html
it seems like it would be nice for users not to care about reversing unless the require it


Add note to documentation of how to relate to ProcrastinateJob model from another Django model

When creating a relation from another Django model to the ProcrastinateJob model like this
queued_job = models.OneToOneField(
  ProcrastinateJob, null=True, on_delete=models.SET_NULL, related_name="+"
)

I get an error when jobs are deleted from the database (e.g., by app.run_worker(delete_jobs="always")). This is because Django sets the on_delete trigger only at the application level but not at the database level.
I solved it by altering my table itself with some custom migration. I even created a function for that
from string import Template

def procrastinate_on_delete_sql(app_name: str, model_name: str, reverse=False):
    template = """
        ALTER TABLE ${app_name}_${model_name}
        DROP CONSTRAINT ${app_name}_${model_name}_queued_job_id_key,
        ADD CONSTRAINT ${app_name}_${model_name}_queued_job_id_key
        FOREIGN KEY (queued_job_id)
        REFERENCES procrastinate_jobs(id)
        """

    if not reverse:
        template += "\nON DELETE SET NULL;"

    return Template(template).substitute(app_name=app_name, model_name=model_name)

I think we should add a note to the documentation (for example, in the FAQ).

If we document it, I think it makes more sense to document it as a migration and let people factor it differently if relevant for them. For the vast majority of people, the relevant thing to do will be to create a migration.
Also, if you create the FK (or o2o) with db_constraint=False, you don't even need to remove the constraint.

BTW, here's how Django generates the constraint name


fix(router) pop with data now work

voila

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@baa51cd). Click here to learn what that means.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             master     #15   +/-   ##
========================================
  Coverage          ?   53.9%
========================================
  Files             ?       3
  Lines             ?     128
  Branches          ?       0
========================================
  Hits              ?      69
  Misses            ?      59
  Partials          ?       0

Impacted Files
Coverage Δ

lib/src/router.dart
4.76% <0%> (ø)

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 baa51cd...0e47671. Read the comment docs.

Merged


ui: create step 2 DEX onboarding screens

Closes #121
The add server form on the third step is just a placeholder and the back button on the design is pending feedback from designer @joyce_ofoche
https://github.com/crypto-power/cryptopower/assets/57448127/35f5a887-25bd-4844-8356-da74fa0bd0b5

i also noticed some some comments from wisdom in the design room regarding the dex onboarding page, such as the cert being multi line, having a choose from files option etc.
not sure any conclusion was reached

such as the cert being multi line, having a choose from files option etc.

Yes, the cert input is multi-line but the file input will wait, I think. More like a v2 design?

such as the cert being multi line, having a choose from files option etc.

Yes, the cert input is multi-line but the file input will wait, I think. More like a v2 design?

okay sounds good. so other concerns raised would be a v2 too?

so other concerns raised would be a v2 too?

I think the file input is the only thing left for this step(2). The rest would be on the Post Bond page.

UI changes in https://github.com/crypto-power/cryptopower/pull/135/commits/42e2a99e80f6837e7e7bba2be2ad0acd84b6a44d

Closes #121
The add server form on the third step is just a placeholder and the back button on the design is pending feedback from designer @joyce_ofoche
EDIT: Got feedback from @joyce_ofoche and have implemented the back button in 42e2a99. https://github.com/crypto-power/cryptopower/assets/57448127/35f5a887-25bd-4844-8356-da74fa0bd0b5

Please add screenshots to the description on the UI that was implemented.


Updates to docs crawler

Changed docusaurus crawler to use playwright and crawling of docs instead of using the repo to avoid edge cases
Improved index_url to support additional file types

This all looks good to me. Although I don't have any experience with playwright, so for that part of it my review isn't worth all that much.


Add request for feedback via 👍

This change encourages users to give a 👍 if they really like a particular feature.

Sadly there's no way to sort by reactions. I'm not sure this will be easy to do or fulfill the promise

Believe or not, there is a way to filter by reactions: https://docs.github.com/en/search-github/searching-on-github/searching-issues-and-pull-requests#search-by-number-of-reactions

Believe or not, there is a way to filter by reactions: https://docs.github.com/en/search-github/searching-on-github/searching-issues-and-pull-requests#search-by-number-of-reactions

That's sorting by total quantity, not number of :+1:. So any reaction counts towards that including :-1:

Believe or not, there is a way to filter by reactions: https://docs.github.com/en/search-github/searching-on-github/searching-issues-and-pull-requests#search-by-number-of-reactions

That's sorting by total quantity, not number of 👍. So any reaction counts towards that including 👎

Good point. Would you prefer I revert the PR?

I just want to be clear that this will be a manual process of sorting and then checking the reactions manually which may make this more onerous for us and thus break expectations for users who are trying to vote as we indicate. I think we should try it out, but I don't have high expectations for success personally.
GitHub isn't designed for us as Open Source projects - it's designed and built almost exclusively for corporate users at this point.  It's why pull request reviews are prime vectors for spam. It's why trying to manage feature requests doesn't work. I almost wonder if discussions are a better venue for feature requests than issues, even though they're completely a separate flow and notifications seem unreliable there.


Add license

This looks like a cool project! You might want to add a license in case anyone wants to reuse or adapt your code.

Thanks! Glad you think so. Good point about the license, I just added one.


sequence_length in pad_sequences for nlevels=2

I noticed that the sequence_length is not collected properly in the pad_sequence function:
https://github.com/guillaumegenthial/sequence_tagging/blob/master/model/data_utils.py#L335
Shouldn't it be:
_, sequence_length = _pad_sequences(sequence_length, 0,  max_length_sentence)

I am slightly surprised that this hasn't caused any issues so far.

Actually the purpose of this line is to pad the sequence_length parameter, in the case where we are computing the length of each word of a sentence. For instance
First sentence: "Hello" -> length = [5]
Second sentence: "Hi John" -> length = [2, 4]
what we want for the batch of this two sentences is to have sequence_length equal to
[[5, 0],
[2, 4]]
so I think I am making the right call!


Allow content-type "application/json"

Hello, thank you for this repo!
I would like to propose a small change which makes the check on content-type less strict.
Even though "application/importmap+json" is maybe more correct for importmaps, I don't see a really good reason why we couldn't allow "application/json" as well?
This PR introduces the changes to accept both these content-types. What do you think about it?
Issue: #11

The import-maps specification specifically forbids the application/json MIME type because of security reasons. See the following links:
https://github.com/WICG/import-maps?tab=readme-ov-file#installation
https://github.com/WICG/import-maps/issues/105


OpenSSL 3.4.0 Support

I went to compile Kyanos on the latest version of Arch Linux and noticed it is missing support for OpenSSL 3.4.0. This causes a lot of functionality to be lost, and many errors when starting the app.
WARN[0004] versionKey openssl 3.4.0 found but bpfFunc not found

Describe the solution you'd like
Support for the latest OpenSSL.

Thank you for your reply. OpenSSL 3.4.0 will be supported in the next release. Additionally, can you use uname -ar to see what your os version is?

Linux seras 6.11.6-arch1-1 #1 SMP PREEMPT_DYNAMIC Fri, 01 Nov 2024 03:30:41 +0000 x86_64 GNU/Linux

I don't mind building from the source, I was planning on packaging it up on the AUR as soon as it is working correctly for Arch. =)


add py.test support

generate py.test based unit tests

Will gladly accept pull requests for this. :)

can you show me path on how can I add py.test support for unit test generate

Currently there are two types of generated output supported: unittest and nose-style tests.
To add another you need to define two classes: a subclass of TestGenerator, and a subclass of Template. See UnittestTestGenerator and UnittestTemplate as examples.
Switching between templates happens in TestGenerator class initializer. Add your new case there and you will be able to use it from the command line -t option.

thanks I will try to implement and push the changes

Hey that would be great!

Hey, do we have update on this?


Expain differences between items and allItems

When should I use items and when allItems?

allItems includes items which are marked as deleting. Normally you should use items, to benefit from the optimistic deleting behaviour (i.e. the deleted item already vanishes from items while the DELETE request is still ongoing).
https://github.com/ecamp/hal-json-vuex/blob/e4d893b7427595fe907e83f993ffe911c9cb8384/src/CanHaveItems.js#L20:L21


fix [OSSLifecycle] service test with new active target

Resolves #3873

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against bf1b6438be593ab37ae973f2b2756abe4e59eaac


Suggest to use singular names for directory naming

This change is

I assume someone who has write access needs to merge this PR. Thanks.


refactor(all): remove shadowing

:heavy_check_mark: Checklist:

[x] Have only one commit (if not, squash them into one commit)
[x] Tests for the changes have been added (for features / bug fixes / refactoring)

Codecov Report

Merging #228 (1233202) into main (da87702) will increase coverage by 5.67%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #228      +/-   ##
==========================================
+ Coverage   66.84%   72.52%   +5.67%
==========================================
  Files          10       12       +2
  Lines         368      535     +167
==========================================
+ Hits          246      388     +142
- Misses        111      115       +4
- Partials       11       32      +21

Impacted Files
Coverage Δ

internal/apischema/find.go
0.00% <0.00%> (ø)

internal/openapi3/parse.go
45.69% <48.09%> (+11.07%)
:arrow_up:

internal/openapi3/openapi3.go
60.00% <60.00%> (ø)

internal/apischema/apischema.go
72.41% <72.41%> (ø)

internal/openapi3/schema.go
75.00% <75.00%> (ø)

internal/faker/faker.go
96.92% <100.00%> (+1.64%)
:arrow_up:

internal/openapi3/example.go
100.00% <100.00%> (ø)

internal/openapi3/mediatype.go
100.00% <100.00%> (+14.28%)
:arrow_up:

internal/faker/person.go
95.65% <0.00%> (ø)

internal/server/handler.go

... 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 48e5e86...1233202. Read the comment docs.


better CI/CD

Since your Travis build has some errors, it can be awesome if persian-tools switch to CircleCI or GitHub actions or even more accurate continuous integrated flow with Travis. Codcov can be added for better detection in code coverage as well as.

I'm absolutely agreed with your idea, I implemented the Travis CI for about 3 years ago and Github Actions is more useful and easy to work and very comfortable with the Github repositories.
So, Do you have enough time to do that?

Yes, of course, but I don't have the right access to persian-tools Github Actions now

I sent you an invitation as a collaborator.

Seems the problem is resolved after b16c9fe530344de5327766ae6a5588b7909be728.
I'm going to close the issue.


GLM-2B-Chinese Pretrained Model please

I think it is suitable for Apple silicon M2 24G memory

@lockmatrix Hi,
Thanks for your interest in GLM. Unfortunately, we didn't pre-train a 2B Chinese version. Maybe a feasible solution is to use quantization (e.g., 8 bit) for model compression from tools like BMInf, which can compress a 10B model's memory usage from about 36G to 18G and accelerate the inference as well.
We are also working on the quantization of GLM-10B, and if your time permits please wait for our news. Thanks!


make view unresponsive while hud visible

hello, i would like to prevent users from interacting with the app while the hud is visible. how can i achieve this?

That's the default behaviour. See the interactionType property: https://github.com/JonasGessner/JGProgressHUD/blob/master/JGProgressHUD/JGProgressHUD/JGProgressHUD.h#L105


refactor: unify Zero / Empty to Default + refactor away from std::unsafe::zeroed() (noir part only)

Docs Preview
Hey there! 👋 You can check your preview at https://66151a5a3fbea79f9b0bdb1f--aztec-docs-dev.netlify.app

Closed in favor of https://github.com/AztecProtocol/aztec-packages/pull/5685


[Bug] review-assign-bot assigns reviewer on draft PR

🐛 Bug
The bot assigns reviewers on draft PRs, which aren't ready for review.
To reproduce
Create a draft PR.
Expected Behavior
Reviewer is only assigned when draft PR is actually published.

This is now fixed and deployed.


Add metadata support for title, author, description, etc. to rantpkg format

Since there was some discussion of a package manager sometime down the road, I'd like to add support for package metadata to the .rantpkg format.
Best way would be with JSON:
{
  id: "Rantionary",
  title: "Rantionaray - Standard Rant Dictionary",
  authors: "Nicholas Fleck, Andrew Rogers",
  description: "The official dictionary files for Rant",
  version: "2.0.0"
}

See #81


A promise was created in a handler error

This following error is consistently showing up when my team and I run npm install on one of our projects. It wasn't happening a couple weeks ago.
(node:58234) Warning: a promise was created in a handler at /Users/miles_johnson/.nvm/versions/node/v8.9.1/lib/node_modules/npm/node_modules/pacote/lib/fetchers/registry/tarball.js:39:25 but was not returned from it, see http://goo.gl/rRqMUw
    at ret (eval at makeNodePromisifiedEval (/Users/miles_johnson/.nvm/versions/node/v8.9.1/lib/node_modules/npm/node_modules/bluebird/js/release/promisify.js:184:12), <anonymous>:8:21)

I'm guessing because this line (https://github.com/zkat/pacote/blob/latest/lib/fetchers/registry/tarball.js#L43) is not returning. Seems that was last touched 2 months ago, which sort of aligns with this showing up for us.
Node: 8.9.1
NPM 5.8.0

@fernan256 I haven't really dug into it further, but it's still happening.

It's really annoying, hope we have some answer from the maintainers.

I don't get why this is happening tbh

@zkat so you're looking why this is happening?

I did, it just doesn't make sense. All the promises I can see getting created have handlers attacked to them. foo.pipe() doesn't create a promise (that I know of?!), so I've no clue why Node is warning about this.
/cc @mylesborins Do you have any clue about this?

pinging @benjamingr since this is using bluebird

When you create a promise in a then you should generally return it because otherwise it is possible it's not being listened to and can cause an unhandled rejection.
If you explicitly want to opt out of it you have two options

explicitly return null
turn off warnings

I recommend fixing the core issue (that there is a promise no one is listening to) though :)

@zkat fromManifest creates a promise in https://github.com/zkat/pacote/blob/latest/lib/fetchers/registry/tarball.js#L55 (when it does fetch) but does not return it which causes the warning.
In all fairness, that code is fine and the warning is probably a bit too eager since a stream whose continuation we can listen to is returned in that case, just return null from the then handler calling fromManifest would fix it.

Thanks! That seems like an easy enough fix.
@milesj @fernan256 do either of you want to PR that?

@zkat where is the handler that calls fromManifest?

@zkat @fernan256 I took a stab at it: https://github.com/zkat/pacote/pull/148

@zkat Question when you think the fix is gonna be merged and how can we get the change, is gonna be a new release of the library in npm? Thanks.

@fernan256 it'll be done when it's done.

you can thumbs down all you want, I'm still not going to ship it faster than it's gonna ship ;)

Lol you’re the author you can do whatever you want. I just thought your comment was funny. It’s just a warning so it’s not critical.

go answer my question in the PR :P

It’ll be done when it’s done

this error showed up on me a few hours ago, is done, or still waiting to be done?

I'm waiting for extra confirmation on the patch in the pr. I don't consider this urgent because it's just a warning only some folks receive, and there's no actual bug: this is just an overly eager automatic warning mashing unnecessary noise with valid code.

Sorry @zkat , I think I really underrated your valuable work regarding pacote! Since most of the people don't have any clue what is behind the scenes of npm. Thanks!

cool. In any case, this is fixed in<EMAIL_ADDRESS>and will be included with the next npm release (which should be next week, I think).


Hover Option should be provided to the license Icons to get better user experience.

Describe the bug
Hover option is not there, when we hover over the license icons!
To Reproduce

Go to https://search.creativecommons.org/
Search any Image
See error

Expected behavior
Hover Option should be provided to the license Icons to get better user experience.
Screenshots

Desktop (please complete the following information)

OS: Ubutu
Browser Firefox

Additional context
If this issue is considered, then I would like to work on it :)

I'm unsure what you mean by "hover", what do you think should happen when the icons are hovered over?
I've added ticket work required to give you the opportunity to elaborate :)

I think @abhisheknaiidu wanted to say that, It would be better if we get the info about the license when we hover over it
When I looked into it, each icon had a separate alt attribute so there would be no accessibility error and since the main image along with the provider image and the license icons are in the same a tag it's better to give the main image's title to the title attribute of the tag

@akmadian I just wanted to say what @hemanth-hk  has said, that,It would be better if we get the info about the licenses when we hover over it!

It is a good proposal, and it definitely improves accessibility. However, we are only considering, as hover interaction, showing the license icons and image's name in future implementations. But I do agree with showing the license name ( "BY NC SA") in the alt information. Hover inside hover is confusing and not suitable for responsiveness.

I think the present text in alt is enough to describe the icon
Screenshots

@hemanth-hk No it needs to have the exact license name like "BY NC SA", which @panchovm suggested, for better accessibilty! I mean that it should be attached to figcaption's section.

Right, I think that you've got the right idea @abhisheknaiidu. Go ahead and start work on this :)


Support for INSERT OR REPLACE?

Would be really helpful to have this.

Duplicate of https://github.com/vapor/sql-kit/issues/31