Use TS to avoid set the param() by hand

So working with strings within TS is weird, here's a sample what we can achieve there:
        var validator = require('fluent-validator');
        var validation = validator()
            .validate(this.Data, x => <TestPersonModel>x.LastName).isUppercase().isAlpha()
            .validate(this.Data, x => <TestPersonModel>x.SSN).isNotEmpty().and.matches("\\d{3}-\\d{2}-\\d{4}").and.isLength(11)
        ;

        // custom message
        validation.validate(this.Data, x => <TestPersonModel>x.FirstName).passes(x => x == "Abc", "Must be Abc");
        validation.validate(this.Data, x => <TestPersonModel>x.Age).passes(x => validator().isGreaterOrEql(x, 18), "Must be at least 18yrs old.");

        var errors = validation.getErrors();

Fortunately it's easy to extend/override fluent-validator default behavior and export it under a different node module. The code to override and to work like above is below. Also worth to notice that it does not break existing code, so might be worth to consider to add it to the library itself (not a big deal to compile it down to JS).
'use strict';

/*
SAMPLE:
var validator = require("fluent-validation-extension");
var errors = validator().validate(m, x => x.Name).isAlpha().getErrors();

 */

var fluentValidator = require('fluent-validator');

module.exports = function(value) {

    var inst = fluentValidator();
    return patchValidator(inst);
};

var originalValidate;

function patchValidator(validatorInst)
{
    originalValidate = validatorInst.validate;
    validatorInst.validate = extendedValidate.bind(null);
    return validatorInst;
}

// NOTE: see fluent-ts-validator for the original selector code
function extendedValidate<T, TProperty>(obj: T, expression: (instance: T) => TProperty) : any
{
    if (expression) {
        let propertySelector = getPropertySelector(expression);
        let value = getValue(obj, propertySelector)
        return originalValidate(value).param(propertySelector);
    }

    return originalValidate(obj);
}

var memberNamesExtractor = new RegExp("return (.*);?\\b");
var memberNamesExtractorArrowFunctions = new RegExp("=>(.*)");

function getPropertySelector<TResult>(name: (x?: TResult) => any) {
    let match = memberNamesExtractor.exec(name + "") || memberNamesExtractorArrowFunctions.exec(name + "");
    if (match == null)
        throw new Error("The function does not contain a statement matching 'return variableName;'");

    let expression = match[1].toString();

    let firstDotIndex = expression.indexOf('.');
    let propertySelector = expression.substring(firstDotIndex + 1);

    return propertySelector;
}

function getValue(obj: any, propertySelector: string) : any
{
    /*
    // NOTE: simple solution with prop1.prop2...propN - no support for arrays...
    var memberNames = propertySelector.split('.')
    var value = obj;
    for (let member of memberNames) {
        value = value[member]
    }
    */

    // NOTE: the don't do way
    // value = eval("obj." + propertySelector);

    // NOTE: alternative way to eval()
    let value = new Function('obj', "return obj." + propertySelector)(obj);

    return value;
}

Hi, thanks for code samples.
As you can see I did not update this project for a while. Could you create a PR?

Hi, No problem to create a PR. This sample is for TS, which means it should be type-safe and raise warnings at compile in case wrong type is used.
I can change toe code to pure JS and add modifications directly into the main lib\index.js - so no package override would be needed. Might take a time, i'd like also to check the custom user message #2.


Fixes bug in pipeToProcess

Signed-off-by: Thomas Fan<EMAIL_ADDRESS>Fixes bug in pipeToProcess when recording errors.
Description
Since there are only two go routines, a slice of length 2 is used to store errors. After wg.Wait() the errors are combined and returned.
Motivation and Context
In the previous implementation, when both the read and write go routines has an error, there were no guarantees that the errors slice gets populated with two errors. The go routine processing the errChannel may not have had the opportunity to process the second error when wg.Wait unblocks.
How Has This Been Tested?
Unit tests were added to test the different error cases.
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Closing this for now to adapt to openfaas's workflow.


Cannot find libcuda.so.1 when creating endpoint

Hi. I am struggling to make our object detection model available to use GPU instance on sagemaker.
We built our own pretrained model and packaged it in docker image. The problem occurs when creating endpoint using it. If i designate the EC2 instance as GPU(ml.p3.2xlarge), it produces following error.
tensorflow_model_server: error while loading shared libraries: libcuda.so.1
Everything works fine on CPU based EC2 instances(ml.m4.xlarge).
I accessed the endpoint on our local PC and got a proper response from endpoint.
We followed blog post to build a docker image ("https://medium.com/ml-bytes/how-to-create-a-tensorflow-serving-container-for-aws-sagemaker-4853842c9751").
We didn't use tf.estimator but converted our pre-trained model to tensorflow-serving compatible type as below.

We designated input and output in the code.
It is confirmed that the AWS endpoint receive input and return output as defined in the code above when using 'ml.m4.xlarge' instance.
We built the docker image using this create model and Dockerfile and nginx.conf (No python files using tf.estimator).
The aforementioned error doesn't occur in setting,
 'FROM tensorflow/serving' in  Dockerfile
and designating EC2 instance as 'ml.m4.xlarge'

But, if we change the setting to
'FROM tensorflow/serving:latest-gpu' or 'FROM tensorflow/serving:latest-devel-gpu' in dockerfile
and designate EC2 instance as 'ml.p3.2xlarge'

libcuda.so.1 occur when creating endpoint
Also, we found that when we run docker image in our PC using nvidia-docker, it works fine with GPU but running with conventional docker fails with same error(libcuda.so.1 cannot be found)
It is very confusing that why AWS sagemaker cannot assign libcuda.so.1 to our endpoint and fails.
Doesn't it run the ECR image using nvidia-docker? Then how could I fix this problem?
I hope there are a few experts who can answer this problem.
Thanks!

@yyc9268 It seems that the C shared object in the error is not in the environment while trying to use the GPU. It works fine for the usual - only CPU instances as the presence of the same object is not checked. Could you try using the docker files here:
https://github.com/aws/sagemaker-tensorflow-serving-container/tree/master/docker
They should work fine

First of all, you want to use tensorflow/serving:latest-gpu as base image, instead of the development image tensorflow/serving:latest-devel-gpu. The later includes the complete CUDA source and tool chain and might lead to unexpected dependency resolution problems when the model container runs on SageMaker. SageMaker public documentation explicitly calls out: "Don't bundle NVIDIA drivers with the image" when using BYO models with GPU instances.
In addition, depending on when your image was built, it might have resolved to a different release of TF serving container. The latest release uses CUDA 10, whereas the 1.12 release uses CUDA 9. You might run into issues if your inference code is compatible with one version of CUDA but not the other.

@nasehim7 Yes. This is exactly the right solution for me. Endpoint was successfully created with the Dockerfile in link. thanks.
@yijieqiu It is a new information for me that NVIDIA drivers shouldn't be included in image. So your guess is that tf-serving container version and CUDA version may not be compatible in our build. We are currently inspecting why ours work with @nasehim7 solution. Your answer would be great help!


Slices fails strong name validation on  Windows 7

@livarcocc
It throws

FileLoadException: Could not load file or assembly 'System.Slices, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. Strong name validation failed. (Exception from HRESULT: 0x8013141A)

Can it be caused by not having local admin rights?

And on Win10

Unhandled Exception: System.IO.FileLoadException: Could not load file or assembly 'System.Slices, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. Strong name validation failed. (Exception from HRESULT: 0x8013141A) ---> System.Security.SecurityException: Strong name validation failed. (Exception from HRESULT: 0x8013141A)

Hello,
This appears to be as a result of IL modifications after the dll is signed, it should just be a matter of resigning the binary after the IL modifications. I would love this fixed as it wreaks havoc with unattended deployments (azure)
Alternate workaround is to add a validation bypass via the registry for that pubkey token, or use sn.exe included with Windows SDK.

Just as a note, I stumbled upon this https://github.com/gluck/il-repack , which seems to offer a parameter to supply a keyfile to sign the resulting assembly, I am not sure if the current solution can do this, so it might be a nice move.

This appears to be as a result of IL modifications after the dll is signed

Yes, it started working after I commented out keyFile in projects.json

cc @davidfowl

I believe david fixed it.


Linking fails on ArchLinux

After installing it over nimble on ArchLinux, compiling it via "nim c example.nim" and then running it fails stating: "could not load: libcsfml-system.so".
The CSFML version on ArchLinux is "csfml 2.1-2" and locating it shows them installed correctly:
/usr/lib/libcsfml-system.so
/usr/lib/libcsfml-system.so.2
/usr/lib/libcsfml-system.so.2.1

CSFML itself is broken on Arch Linux. I've contacted the maintainer and they don't want to do anything about it.
From README.md:

CSFML 2.1, which requires SFML 2.1, must be installed to use it.

Thanks, I downgraded to SFML 2.1 in from the git-repo and it works fine now.


Common chunk order seems to be wrong

Do you want to request a feature or report a bug?
bug

What is the current behavior?
I'm probably doing something wrong but it seems common chunks order seems to be wrong.
If the current behavior is a bug, please provide the steps to reproduce.
Here is the config
const ExtractTextPlugin = require('extract-text-webpack-plugin');
const extractCSS = new ExtractTextPlugin({ filename: 'css.bundle.css' });
const extractLESS = new ExtractTextPlugin({ filename: 'less.bundle.css' });
const webpack = require('webpack');
const path = require('path');
const { TsConfigPathsPlugin } = require('awesome-typescript-loader');

const host = require('../src/main/resources/config/local/contentViewer.json').listen;
const isDevServer = path.basename(require.main.filename) === 'webpack-dev-server.js';
const entries =  {
    // Do not include babel-polyfill for embedded - should be included in external app (and having it twice throws an error)
    embedded: ['./src/app/viewer.tsx'],
    vendor: [
        'lodash',
        'react-dom',
        'react',
        'seamless-immutable',
        'seamless-immutable-cursor',
        'whatwg-fetch',
        'text-encoding'
    ]
};
module.exports = {
    entry: entries,
    output: {
        path: path.resolve(__dirname, 'dist'),
        filename: 'content-viewer.[name].js',
        library: ['wspt-doc-viewer', '[name]'],
        libraryTarget: 'umd'
    },
module: {
        rules: [
            // All files with a '.ts' or '.tsx' extension will be handled by 'awesome-typescript-loader'.
            // All output '.js' files will have any sourcemaps re-processed by 'source-map-loader'.
            {
                test: /(\.ts|\.tsx)$/,
                enforce: 'pre',
                exclude: /node_modules/,
                loader: 'tslint-loader',
                options: {
                    emitWarning: true
                }
            },
            {
                test: /\.tsx?$/,
                exclude: /node_modules/,
                use: ['babel-loader', 'awesome-typescript-loader']
            },
            {
                enforce: 'pre',
                test: /\.js$/,
                use: 'source-map-loader'
            },
            {
                test: /.js$/,
                exclude: /node_modules/,
                use: {
                    loader: 'babel-loader',
                    options: {
                        presets: ['babel-preset-es2015']
                    }
                }
            },
            {
                test: /.jsx$/,
                exclude: /node_modules/,
                use: {
                    loader: 'babel-loader',
                    options: {
                        presets: ['babel-preset-es2015', 'babel-preset-react']
                    }
                }
            }
]
},
 plugins: [
        new webpack.NoEmitOnErrorsPlugin(),
        new webpack.optimize.AggressiveMergingPlugin(),
        new webpack.optimize.ModuleConcatenationPlugin(),
        new webpack.optimize.CommonsChunkPlugin({name: 'vendor', minChunks: Infinity}),
        extractCSS,
        extractLESS
    ]
}

The vendor output is the common chunk. The other entry point is the application.
When building I get the following output the result is that embedded.js is emitted with number 0 and vendor.js with 1.
When this package is resolved in some other package that references it, only the embedded.js file is loaded and bundled. The vendor.js file is ignored.

What is the expected behavior?
Expected behavior is that vendor is emitted with number 0 and embedded.js with number 1, since embedded.js is dependent on vendor.js modules.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Webpack 3, mac osx

The numbers are internal and ain't related to the order.
Please submit support requests and questions to StackOverflow using the tag [webpack]. StackOverflow is better suited for this kind of support. The issue tracker is for bug reports and feature discussions. See also our CONTRIBUTING guidelines.


Y Sort breaks TileMap layer highlighting

Godot version
4.0.2.stable, 4.1.dev1 (db1302637)
System information
Windows 11, Forward+, NVIDIA GeForce GTX 1660 (531.61)
Issue description
Highlighting TileMap layers in the editor stops working when Y Sort is enabled.
Steps to reproduce
https://user-images.githubusercontent.com/10379487/234579030-145015a3-3f47-4efb-95e4-63f293cb86cc.mp4
Minimal reproduction project
TileMapLayers_Test.zip

Probably the same bug as 73266 and the more general 72519

Duplicate of #73266.


inconsistent #ifdef CSR_HDMI_IN0_FREQ_BASE

the status code is not the same for hdmi_in0 and hdmi_in1 noticed the status code is not the same for hdmi_in0 and hdmi_in1
The #endif is in a different place:
https://github.com/timvideos/HDMI2USB-litex-firmware/blob/master/firmware/ci.c#L465
https://github.com/timvideos/HDMI2USB-litex-firmware/blob/master/firmware/ci.c#L465

Hi Carl,
As far as I can tell, the code of the hdmi_in1 is ideally the correct one.
Reason: The highlighted part (wprint) depends on a peripheral (frequency detector) which may or may not be present in the gateware. So, we include the printf code only if that peripheral (frequency detector for HDMI_IN1) is present in the gateware.
#ifdef CSR_HDMI_IN1_FREQ_BASE
wprintf(" (@" REFRESH_RATE_PRINTF " MHz)",
REFRESH_RATE_PRINTF_ARGS(hdmi_in1_freq_value_read() / 10000));
#endif
Whereas the hdmi_in1_status() section of code is dependent on whether CSR_HDMI_IN1_BASE is defined or not.
Thanks!
Best regards,
Rohit


Pre-compiled Exe for convinience!

Hi
Thanks for making this available to us. I've recently purchased a WIN10 laptop and it came with a rather annoying (fugly?) OEM boot logo. This NEED changing for something less dramatic, like a butterfly or similar ;)
Glad I've found your tool however my skills aren't polished enough to compile this by myself, wouldn;t know where to begin. Can I kindly ask you to upload a pre-compiled version of your tool (setup.exe) so I can give it a shot please? The image that is the UEFI is named "Splash.jpg" (found out using AMI's ChangeLogo_v5.exe utility).
That'd be awesome and quite helpful. Any other advice RE "how to compile this on Windows/x86" would also be appreciated.
All the best to you!

https://github.com/Metabolix/HackBGRT/releases

Indeed, the compiled program may be downloaded from the Releases page.

Sweet, thanks folks. I'll run this as soon as I can make it back home!!!


Implement API calls to retrieve the registered users and active planning groups from the backend

#7b68abd

Retrieve registered users: Done
Retrieve Groups: In progress


Error highlighting does not work for me

Describe the bug
Error highlighting does not work. For more info :
https://stackoverflow.com/questions/78328361/intellij-graphql-plugin-error-highlighting-not-working
To Reproduce
See SO question

Hi @oliverwatkins! From your question I see that you have a folder structure like this :

|-- project
|        |-- frontend
|        |    +-- myQueryDeclerations.ts
|        |    +-- schema.graphql
|       +-- graphql.config.yml

So, the path to the schema in your config should be schema: frontend/schema.graphql, instead of schema: schema.graphql.


Added Gravestone game objects

Duplicated gravestone game object
Repositioned all duplicates accordingly
Fixed scene issue

@joeperpetua @jsandinoDev Please review

@joeperpetua @jsandinoDev Please review

they are not maintainers

@jimenezz22 Please review


ERR: sdp: invalid syntax c=

Any idea on how to fix. ?
2020/08/28 23:26:27 [0/0/0] rtsp-simple-server v0.9.11
2020/08/28 23:26:27 [0/0/0] [UDP/RTP listener] opened on :8000
2020/08/28 23:26:27 [0/0/0] [UDP/RTCP listener] opened on :8001
2020/08/28 23:26:27 [0/0/0] [TCP listener] opened on :8554
2020/08/28 23:26:27 [0/0/0] [source proxied] started
2020/08/28 23:26:27 [0/0/0] [source proxied] connecting
2020/08/28 23:26:28 [0/0/0] [source proxied] ERR: sdp: invalid syntax c=
2020/08/28 23:26:33 [0/0/0] [source proxied] connecting
2020/08/28 23:26:34 [0/0/0] [source proxied] ERR: sdp: invalid syntax c=

Hello, please reopen the issue by using the "Communication issue with an external camera or software" template, and follow the instructions.


Fixes >>= not work on Execution

Changes RichExecutionOps to not pull in implicit conversions to MonadOps and BindOps.
RichExecutionOps was extending ToResultantMonadOps which also extends ToMonadOps and ToPlusOps. Since RichExecutionOps is not a companion object the implicits provided by ToMonadOps from ToResultantOps clashed with the ToMonadOps imported from Scalaz.
This changes RichExecutionOps to not extends ToResultantMonadOps and instead just provide an implicit conversion to ResultantMonadOps.
This fixes #372.

Looks good to me, nice to include a test.  :+1:


Task/APPENG-2738: NIM Cypress E2E tests

Description
Adding Cypress UI tests to cover the NIM Model use cases from all the supported pages and tabs:

Show/Hide NVIDIA NIM model serving platform
Deploy NVIDIA NIM models
List NVIDIA NIM models
Delete NVIDIA NIM model

How Has This Been Tested?
Running npm run test:cypress-ci command.
Test Impact
Increases code coverage of UI code for all the components impacted by the NIM models.
Request review criteria:
Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

This test is currently failing, but it seems to be passing in our local environment. It might be worth re-running the check to determine if it's a legitimate issue.
https://github.com/opendatahub-io/odh-dashboard/actions/runs/11182131290/job/31087701260?pr=3298
faling cypress file as per above check - applications/notebookServer.cy.ts
on local it is successful

/ok-to-test

@andrewballantyne , shall I rebase to resolve the new conflicts?

/unhold


Add server-side localization

Also I added base interfaces and classes: IPreference.cs, ServerPreference.cs, IPreferenceManager.cs, IServerPreferenceManager.cs, ServerPreferenceManager.cs, PreferencesController.cs to store and interract with the settings on the server side.
It's need to be implemented ILocalStorageService methods in CustomLocalStorageService.cs
This class can be based on:

existing local storage
database
cache
configuration
another source

Thanks :D

You are welcome!


Constructing the conv features from backbone

What is the meaning of this part when extracting features from the backbone? I cannot see any mention of this part in the paper.
conv_current = conv[:, 1:, :, :, :]
conv_prev = conv_current - conv[:, :time_step - 1, :, :, :].detach()
conv = torch.cat([conv_current, conv_prev], axis=1)

Hi there, we mention this in our paper:
“Then, the embeddings from the second layer of the pre-trained model are fused as input features to
the policy networks [64, 51].”
This is the commonly used flare fusing operation in many works. You can check [64, 51] to get further information.

Thank you for your response. Could you provide the code you use to visualize the feature map (from different layers of resnet) shown in the paper?

Hi, we directly visualize the feature map use the output of different layers of the ResNet without post-preprocessing.


[Edit Event] An authorized user can not change the order of pictures in the created event

Environment: Windows 10 x64, Chrome Version 109.0.5414.119 (Official Build) (64-bit)
Reproducible: always
Build found: 31.01.2023
Preconditions

Site https://www.testgreencity.ga/GreenCityClient/#/ should be opened.
User should be authorized.
Event should be created and saved.

Steps to reproduce

Click on "Events" tab in the header.
Find the created Event and Click on "More" button.
Click on "Edit" button at the right upper corner.
Drag and drop the images to a different position in order to change their order.

Actual result
The initial order of images has not been changed.
Expected result
The order of the pictures has been changed according to transfer.
User story and test case links
E.g.: "User story #3195
Test case"

Check this issue one more time. The bug is not completely fixed, the user should provide the page refreshing in addition that to see the changes which are made with the event Image.
https://user-images.githubusercontent.com/110603507/228564513-8f02c576-c16f-4044-865a-046595450099.mp4

The bug is reproduced on  Windows 10 x64, Chrome Version 113.0.5672.93 (Official Build) (64-bit) The image is not displayed in different position on the 'Edit Event' page after drag and drop it in other place


Take care of the error reported in cpp

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1513274&view=logs&j=a129effc-2dd1-54d1-fb5a-ad7bdc0e851d&t=9036d097-2774-5136-505e-b4dc0a37f42d&l=56

Error count went from 5 to 4. I think we need to update CredScanSuppression.json:
##[error]1. Credential Scanner Error CSCAN-GENERAL0020 - File: sdk/attestation/azure-security-attestation/test/ut/crypto_test.cpp:sdk/attestation/azure-security-attestation/test/ut/crypto_test.cpp. Line: 16. Column 46.
  Tool: Credential Scanner: Rule: CSCAN-GENERAL0020 (X.509 Certificate Private Key). https://aka.ms/credscan
  A potential secret was detected in 'crypto_test.cpp':(CSCAN-GENERAL0020 X.509 Certificate Private Key) Validate file contains secrets, remove, roll credential, and use approved store. For additional information on secret remediation see https://aka.ms/credscan.
##[error]2. Credential Scanner Error CSCAN-GENERAL0020 - File: sdk/attestation/azure-security-attestation/test/ut/crypto_test.cpp:sdk/attestation/azure-security-attestation/test/ut/crypto_test.cpp. Line: 28. Column 45.
  Tool: Credential Scanner: Rule: CSCAN-GENERAL0020 (X.509 Certificate Private Key). https://aka.ms/credscan
  A potential secret was detected in 'crypto_test.cpp':(CSCAN-GENERAL0020 X.509 Certificate Private Key) Validate file contains secrets, remove, roll credential, and use approved store. For additional information on secret remediation see https://aka.ms/credscan.
##[error]3. Credential Scanner Error CSCAN-GENERAL0020 - File: sdk/keyvault/azure-security-keyvault-certificates/test/ut/recordings/KeyVaultCertificateClientTest.DownloadImportPem.json:sdk/keyvault/azure-security-keyvault-certificates/test/ut/recordings/KeyVaultCertificateClientTest.DownloadImportPem.json. Line: 149. Column 30.
  Tool: Credential Scanner: Rule: CSCAN-GENERAL0020 (X.509 Certificate Private Key). https://aka.ms/credscan
  A potential secret was detected in 'KeyVaultCertificateClientTest.DownloadImportPem.json':(CSCAN-GENERAL0020 X.509 Certificate Private Key) Validate file contains secrets, remove, roll credential, and use approved store. For additional information on secret remediation see https://aka.ms/credscan.
##[error]4. Credential Scanner Error CSCAN-GENERAL0020 - File: sdk/keyvault/azure-security-keyvault-certificates/test/ut/recordings/KeyVaultCertificateClientTest.DownloadImportPkcs.json:sdk/keyvault/azure-security-keyvault-certificates/test/ut/recordings/KeyVaultCertificateClientTest.DownloadImportPkcs.json. Line: 149. Column 30.
  Tool: Credential Scanner: Rule: CSCAN-GENERAL0020 (X.509 Certificate Private Key). https://aka.ms/credscan
  A potential secret was detected in 'KeyVaultCertificateClientTest.DownloadImportPkcs.json':(CSCAN-GENERAL0020 X.509 Certificate Private Key) Validate file contains secrets, remove, roll credential, and use approved store. For additional information on secret remediation see https://aka.ms/credscan.
  Active results: 4

Unassigning, adding Client tag. These will need to be updated by C++ owners.

@gearama Please address the remaining cred scan issues so the CI gate can be enabled.

This was fixed and credscan was added in https://github.com/Azure/azure-sdk-for-cpp/pull/4328


Problems Nanopore Align

Hi!
I'm trying to use RASFlow for first time with ONT Minion (with master branch), after align step with HISAT2 no reads are aligned. I also tried bwa branch but there are some tokens errors so the pipeline fails (im using macOs Catalina). Any suggestion? maybe there are some hisat2 configuration for Nanopore??
Thank you in advance
AG
Alignment ressults:
38280 reads; of these:
38280 (100.00%) were unpaired; of these:
38280 (100.00%) aligned 0 times
0 (0.00%) aligned exactly 1 time
0 (0.00%) aligned >1 times
0.00% overall alignment rate

It seems that HISAT2 is not proper for long reads alignment, as this paper Evaluation of tools for long read RNA-seq splice-aware alignment  has a similar finding as you.


Refactor All Try Catch block to use a sinlge global error handler, ChatBot required, Email verification add at signup

Feature Request: Global Error Handler, Chatbot with Socket.IO, and Email Verification
Overview
This issue outlines the need to refactor the project to include the following features:

Global Error Handler

Refactor all try/catch blocks to utilize a single global error handler.

Chatbot Implementation

Implement a chatbot using Socket.IO for real-time communication.

Email Verification at Signup

Add email verification functionality during the user signup process.

Requirements
1. Global Error Handler

Create a centralized error handling middleware to manage errors across the application.
Remove individual try/catch blocks in routes and replace them with calls to the global error handler.

2. Chatbot with Socket.IO

Integrate Socket.IO to enable a real-time chatbot feature.
Ensure that users can send and receive messages through the chatbot interface.

3. Email Verification

Implement email verification during the signup process.
Send a verification email to users with a unique link upon registration.
Create a verification endpoint to handle user verification upon clicking the link.

Additional Notes

Ensure that proper testing is conducted for each feature after implementation.
Update the documentation to reflect any changes made during this process.

@Shub95-arch , thanks for creating this issue. you can start working on this issue Remember, this is a complex task, so start small and create a PR with a solid, small implementation. This will make management easier.

This issue may require other experienced contributors as well, so others are welcome to join

@Shub95-arch , thanks for creating this issue. you can start working on this issue Remember, this is a complex task, so start small and create a PR with a solid, small implementation. This will make management easier.

Thanks for your consideration on this issue , although i have already made the backend routes for the chatbot and email verification and all the try catch blocks are refactored. Please check the pull request #25 and if it helped contribute to this repo , do mark it has hacktoberfest-accepted.
Thanks

This issue may require other experienced contributors as well, so others are welcome to join

Thank you for telling me about some security measures, i have removed personal keys and the environment is safe now,
I've made another pull request #34


CocoaScrollView decelerationRate

Hello there. I want to make some changes in CocoaScrollView but I couldn't find where to do it. things i want to do:
    scrollView.bounces = false
    scrollView.decelerationRate = UIScrollView.DecelerationRate.fast

@nasru enabling/disabling bounce is accessible via scrollBounceDisabled(_:). As for deceleration rate, I'll add the ability to expose it.

I m used it scrollBounceDisabled. But did not working. Did not disabled bohncing

@nasru it only works with CocoaScrollView. Are you using it with SwiftUI's ScrollView or with CocoaScrollView?

@nasru I've added CocoaScrollView.decelerationRate(_:)


Intermittent slow outbound network in containers

What happened:
I'm running some Windows IIS containers on Azure VM scale set. For the most part, it seems to run fine. However, occasionally, especially after a high load, outbound connections become slow and even timeout. Restarting the container doesn't fix it, only restarting the host VM seems to resolve it. I cannot rely on this workaround of course. Connections to localhost inside the container work fine. And outbound connections on the host VM work fine too.
What you expected to happen:
The container outbound network should work consistently, and not fail intermittently. I'd expect the issue to resolve once a container is restarted
How to reproduce it:
I haven't been able to consistently reproduce, but one way would be to load test the container by sending many requests to it. I'm running an IIS container on Windows Server 2019 with Containers.
Orchestrator and Version (e.g. Kubernetes, Docker):
ServiceFabric, Docker
Operating System (Linux/Windows):
Windows
Kernel (e.g. uanme -a for Linux or $(Get-ItemProperty -Path "C:\windows\system32\hal.dll").VersionInfo.FileVersion for Windows):
10.0.17763.2867 (WinBuild.160101.0800)

Turned out to be port exhaustion


Video star

Describe the issue
Cracked ipa for video star not working when you try to open it it crash same with ESign but to resolve crash should enable remove mobileprovision before sign it but in live container don’t know how to resolve it

Instructions to reproduce
.
What version of LiveContainer are you using?
3.1
Other
Hope fix this issue soon

I downloaded a random cracked version and it just worked, not sure where you found the ipa. Please try another working ipa.


Add captions for easy searching

As said in issue #69, captions allow for searching a specific icon. I've implemented the captions in the README.md as well as in the radius.html. The latter uses the aria-label of the SVG, so icon builders do not need to do anything special for that.

Wow! That's fantastic. Thank you so much 😀

Dude, you're quick.
You're welcome, glad to help

There's a bug on the radius tool - it keeps adding the label multiple times. Happening on Firefox on Android, haven't checked other browsers yet.

Ah yes, you are right. I'm fixing that right away

Sorry, had to revert this for two reasons.

On mobile, it lead to inconsistent icon sizes on the main page.
The radius tool was still adding extra titles.

I think it is a good idea - just need to work out how to do it properly.

You are right, we have to do it correctly.

I don't fully understand what you mean. Are you talking about the radius.html or README.md?
I seem to have made a mistake in this PR. The other PR #83 was merged and should add the label only once on radius.html. That seems to work fine.

Here's a screenshot of the README.  As you can see, the length of the caption influences the size of the table cell.  This means some icons display smaller than others.

I see now what you mean. It looks like I'm not able to fix this, since GitHub does not render the "" I've added to my version of the markdown file. It looks like we have to live with the fact that we cannot display the labels in the markdown file and keep the layout nice. I think it's more important to display the icons in the same size, so no labels in the README.md file then.
I guess that's okay though, since the radius.html file is linked from the markdown file, which does contain the labels.


[RBAC] Specify type for AnsibleResourceField serializer

This fixes the tests failing on:
eda_api.exceptions.ApiTypeError: Invalid type for variable 'resource'. Required value type is str and passed type was dict at ['received_data']['resource']


opd package on the ohri clinical views

Requirements

[ ] This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes the ticket number in the format OHRI-123 My PR title.
[ ] My work includes tests or is validated by existing tests.

Summary

Screenshots

Related Issue

Other

@alaboso the build is failing. I think it will resolve when we run the build of esm-patient-chart-app with the updated package.json


fix: canRule false for juror who missed vote

If a juror misses their vote it should not show canRule in same session

Pull Request Test Coverage Report for Build 713

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
131 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-13.5%) to 39.636%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/contracts/implementations/arbitrator/KlerosPOC.js
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/contracts/ContractImplementation.js
1
81.43%

src/utils/Web3Wrapper.js
4
20.83%

src/utils/EventListener.js
5
57.5%

src/contracts/implementations/arbitrable/ArbitrableTransaction.js
20
15.0%

src/resources/Notifications.js
36
4.02%

src/contracts/implementations/arbitrator/KlerosPOC.js
65
28.64%

Totals

Change from base Build 711:
-13.5%

Covered Lines:
420

Relevant Lines:
923

💛  - Coveralls


I am sorry but my last commit did not work

Ok, I should have test it, or review it.
Perhaps can you add an example of its usage in the example1 app?
Or add a unit-test?
Also, Can you please rebase this above current 'master'? This pull request is unmergeable due to conflicts with the cleanups I did on top of your previous work.
Cheers

Thanks for your advice, Sébastien. I will handle it this weekend. Hope not too late.

Thanks @HighSparrow, I've integrated you sample code!
I've finally rework the method you provided, by using a std::map to cache the association of column name to index.
I've also added a basic unit test for it.


Removing a wallet causes the actions to disappear

To reproduce

load gist accd2826d83e7ce345f8417c06cb7cdd
In the "Simulation" tab, remove wallet 3
All actions disappear

Whilst this was deliberate....I have been persuaded it's not as great an idea as it seemed at the time. :-D
My thinking was that once you remove a wallet, you can invalidate the simulations, and there's no way in general we can know which actions are now invalid.
However, killing all the simulations is a bit of a, "sledgehammer to crack a walnut," solution, I admit.
I'm going to change this so that the user will have to delete/amend affected actions. They've always been able to submit invalid scenarios, and always been given appropriate feedback when they do.


direct local reader for raw read

Signed-off-by: fredchenbj<EMAIL_ADDRESS>What have you changed? (mandatory)
To reduce the latency of tikv's raw read, change the read flow "readpool->localreader->readpool" to "localreader->readpool", i.e. directly ask async_snapshot first, then pass the snapshot to readpool in the callback of async_snapshot.  Thus, reduce the overhead of context switch. Under a benchmark test, context switch reduced about 15% compared to the old implementation.
Changed four funtions about raw read: async_raw_get, async_raw_batch_get, async_raw_scan and async_raw_batch_scan.
What are the type of the changes? (mandatory)
Improvement
How has this PR been tested? (mandatory)
unit tests and manual tests
Does this PR affect documentation (docs) update? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

Thanks @fredchenbj
Can you give us the benchmark results compared to the previous implementation?
I guess the latency may be reduced.

@siddontang OK, I will test the latency of raw_get and raw_batch_get using the go-ycsb to justify the effect of this change.

Left is using this change, right is old.  From the image, grpc p999 duration decreases, and scheduler command duration decreases more effectively.
test command (only for raw_batch_get)

./bin/go-ycsb run tikv -P workloads/workloada -p tikv.pd="<IP_ADDRESS>:2379" -p tikv.type="raw" -p tikv.conncount=128 -p tikv.batchsize=128 -p batch.size=1000 --target=100000 --threads=32

Hi @fredchenbj
The scheduler command time looks zero, maybe there is something wrong for the metrics.

But through the gRPC metrics, we can see the latency reduces a lot, very cool.

seem you need to call _timer.observe_duration(); explicitly at the end of the future.

@siddontang OK, I have fixed this metric~

Thanks @fredchenbj
Please format your code and let CI passed.

how about the new fixed metrics?
what is the 999 read process duration?

@hicqu  @overvenus
seem reducing context switch can help us gain better performance, so I think it is worth to remove the scheduler and local read threads.

PTAL @breeswish

How is the performance difference? (e.g. with go-ycsb)

@breeswish There is a simple go-ycsb test, https://github.com/tikv/tikv/pull/4222#issuecomment-463966385. From the test, the raw_batch_get grpc p999 duration reduces, and the context switch reduced 15%.

Good job!

/run-integration-tests


chore: add "make deploy" helper command

The deploy command does a full ETL run on production using the analytics script ops/scripts/etl-prod.sh. The script helpfully acquires a global lock, so you don't have to worry about any race conditions with production runs.

@Marigold Should we move it across to the etl repo in an ops/ folder?


collapse search items in sidebar

Is there a way to collapse the search results in the sidebar (like in the explorer of the editor)? I work with several microservices and I have to search all the time and it is pretty confusing. I would love to use it furthermore but It depends on the good search possibilites. Please help :)

The search viewlet has the same "collapse all" button as the explorer. Is that what you mean?

Hi Rob
Sorry was my mistake. Im so glad for it, thanks!
Best, Michelle

On 12 Oct 2017, at 19:14, Rob Lourens<EMAIL_ADDRESS>wrote:
The search viewlet has the same "collapse all" button as the explorer. Is that what you mean?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Doesn't work with Homey 3.0 and appstore as of November2019

Would be great if the app could be updated to work with the new appstore. Thanks

Agreed, especially since athom removed the notice of app updates. So now you need to go through every app one by one to see if there is an update...


build error on xcode15.3

Describe the bug
after upgrade xcode15.3,  AWS SDK always happen build issue.
build error: Cannot convert value of type '(any Error) -> Void' to closure result type '()'
Steps To Reproduce
Steps to reproduce the behavior:
1. open xcode15.3
2. open existed AWS project
3. build all
4. See error(Cannot convert value of type '(any Error) -> Void' to closure result type '()' in Attempt.swift amplify-swift/AmplifyPlugins/Storage/Sources/AWSS3StoragePlugin/Support/Internal/Attempt.swift)

xcode15 and xcode15.2 do not have such issue.

Expected behavior
build correct.
Amplify Framework Version
2.17.2
Amplify Categories
Storage
Dependency manager
Swift PM
Swift version
4.5 or 5.0
CLI version
12.10.3
Xcode version
15.3
Relevant log output
Cannot convert value of type '(any Error) -> Void' to closure result type '()'

Is this a regression?
No
Regression additional context
this is compatibility with xcode15.3 issue
Platforms
iOS, macOS
OS Version
Mac OS 14.4.1
Device
do not need
Specific to simulators
No response
Additional context
No response

Same issue.

Does anyone have a solution?

This has been fixed in 2.26.1. Please update your dependencies to use the latest version.

Thanks for your feedback, I meet one strange thing, My AWS existed package upgrade in Xcode can not arrive at 2.26.1 or 2.29, only can see 2.18.  did we change path or rule?

Make sure you're using "Up to Next Major Version" as Dependency Rule in your project's settings.

Then just go to File -> Packages -> Update to Latest Package Versions

Thank for your help! already can build successfully.

Thanks for confirming! Let us know if you have any other issues or questions


Failed to install 'ms-dotnettools.csharp'.

Issue Type: Bug
I use
MacBook Pro (Retina, 13-inch, Early 2015)
macOS Big Sur version 11.2
I saw this error :
"Failed to install 'ms-dotnettools.csharp'."
then i clicked the button : "Download Manually"
file downloaded, and I saw following message :
Once downloaded, please manually install the downloaded VSIX of 'ms-dotnettools.csharp'.
and I clicked the button : "Install from VSIX..."
and here it is problem.
I chose downloaded file, and I clicked the button : "Install"
and there's no response.
I click again the button : "Install" from Extension search manu.
and this problem didn't solve
VS Code version: Code 1.27.1 (5944e81f3c46a3938a82c701f96d7a59b074cfdc, 2018-09-06T09:14:32.385Z)
OS version: Darwin x64 20.3.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz (4 x 2700)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 4, 4

Memory (System)
8.00GB (0.02GB free)

Process Argv
/private/var/folders/bx/d17zh1gj3q17sf9zmj7xstyr0000gn/T/AppTranslocation/C0BC7BF8-A841-4866-8A4B-C2A2049739A7/d/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

vscode-clang
mit
0.2.3

cpptools
ms-
0.21.0

Your VS Code version  is ancient. Please update to the latest, which is 1.53.2

Please try with latest and let us know if you still have this issue.


Search Bar Clipping "g" and "j" Characters

The search bar in the project's interface is currently clipping the lower portions of the "g" and "j" characters, making them resemble "q" and "i" respectively. To resolve this, we should consider adjusting the search bar's height to accommodate the full height of these characters.

@abishek0057 you must be referring to https://libraries.excalidraw.com ?
It looks fine for me

I am on chrome in Mac, which OS and browser you are using ?

@ad1992 Yes on https://libraries.excalidraw.com/ search input,
I am using Windows 11, and clipping occurs on both Chrome and Edge browser

yaa i writen github but its show like this

i am on chrome in win11

Thanks for reporting the issue. PR welcome for the fix :)

I would like to work on this can u assign this to me


fix: layout and no editor view

UI to shrink when the no editor view is active. Currently, the DB layout forms downwards which should be from left to right. The proposed solution fixes it, tho not completely but to some extent.

@Ovyerus

https://github.com/Ovyerus/prismaliser/assets/50935456/bfe81767-067c-4499-8616-ea89c00c64ad
Tried making the UI much more free in terms of focus.

Thanks for bringing the direction option to my knowledge! I've made the commit ab228c4 with only that change as I'm not fond of the header change personally.

Np, thanks.


added MediBloc

thanks @bauser -- and you please add the MediBloc logo to the logos folder in the repo? That way we'll make sure to update the Google side for the next update


Linux RT creates multiple listening socket for the same port

I noticed a couple of problem when running grpc-labview server on an RT machine.

The first time you run the server it runs properly but when you try to stop the server it does not work.
If you relaunch the server on the same port it creates another socket for the same port for listening. Due to which the client request get lost (probably because there are multiple sockets open). Attached netstat for the RT machine.

This same behavior is not seen in Windows.
Windows behavior is:

It kills the listening port when you stop the server.

Some ideas :

Close function is not called.
If the close function is the LabVIEW one, it never closed instantly a tcp connection...
... but a non-effective port can be reopened if the SO_REUSEADDR option is applied during this time.
If the close function is a Linux system call, a default behavior can be applied by the kernel.
The default behavior of Linux when closing a tcp port is to keep it unopenable during 60s with. But in that case the CLOSED flag or the TIME_WAIT flag is shown and the next connection cannot be opened, which is not the case here.

Conclusion : could it be an asynchrone issue between close and open while the SO_REUSEADDR option is used ?

#114 has resolved this issue. I am closing this


TLS/SSL Support

Curious how hard this would be to add, I think it would work well as a plugin.
Here's a guide for Nginx instead of Traefik: https://finnian.io/blog/ssl-with-docker-swarm-lets-encrypt-and-nginx/

This project already has a Traefik plugin which Traefik itself has full support for Let's Encrypt with HTTP and DNS challenges. So I don't think we need to build another way to do TLS/SSL.

Oh nice, I had yet to look into the traefik aspect of it.

Oh nice, I had yet to look into the traefik aspect of it.

Traefik itself is pretty nice. +1 on including it and making it a plugin. If done right, it would be what I'd call generally the "ingress" plugin 👌 Other pluggable ingress choices could be: nginx, caddy, etc.

Yup, I tried out Nginx initially, but Traefik was basically 0 config for docker swarm, so it fit right in. I'd be interested to know what else it needs to do before you'd consider it an ingress plugin.
I initially considered naming it the "ingress" plugin, but decided to go with the image name for now until I have a second plugin, and decide on a naming convention then. Right now, it's looking like a built-in docker registry will be the second plugin, so apps auto-update when you push images to it.

I think the main two things you want to be able to provide are:

Configuring the TLS / Let's Encrypt parts of Traefik
Adding domains (see above)
Adding custom routes

The rest (routes) should be handled automatically by Docker Swarm service labels.

Adding custom routes

What do you mean by different custom routes? Is that different than domain.com/path?

Adding custom routes

What do you mean by different custom routes? Is that different than domain.com/path?

Yeap! You just never know when you might want to add arbitrary custom routes to Treafik that aren't otherwise automatically picked up routes from Docker Swarm labels from running services.

The plugin already supports custom routes then! See how I've setup api-2 in the example:
miasma traefik:set -a api-2 --host web2.home.io --path /api

and it also supports raw traefik rules:
miasma traefik:set --rule "(Host('hostname') && PathPrefix('/custom-path'))"

So all that's left would be TLS. I've got time this weekend, I'd like to get the project to the point where I can deploy it on EC2 instances or Digitial Ocean droplets so I can start using it for my personal, public stuff. And this is all that's left before I can do that.

That sounds good to me!
Let mw know if you'd like s hand testing things. Be more than happy to give this a whirl once things are in shape 👌

Well... actually I also need to add authentication... I'll start simple with a env variable token lol

https://doc.traefik.io/traefik/https/acme/#configuration-examples

https://stackoverflow.com/questions/60510232/how-can-i-redirect-http-to-https-using-traefik#60513369

@prologic, HTTPS has been merged and published to the aklinker1/miasma:nightly image. If you want to test it out, now is the time!
Let me know if you have any issues, I've probably forgotten to document something.

Will do! Probably this weekend if that's okay? I have a lot of my own do job work to do 🤦♂

No worries lol, I'm off today for 4th of July in the US.

No worries lol, I'm off today for 4th of July in the US.

Happy 4th of July! 🥳


"Cannot open Quick Item Search, dictionary translation has not yet finished."

Updated to version: 1.5.5
Been in game for an hour and still getting this message. Didn't have this issue on previous version and haven't added any new mods since.

Could you please send me your save file?

https://drive.google.com/file/d/1KDhtlgsOgaFQPxasyPyqRAL2LYI0km-e/view?usp=sharing
There is the save

Thanks.  It's fixed for the next version.

The patch has been released.


Updates the documentation.

Pull Request Test Coverage Report for Build 313

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.817%

Totals

Change from base Build 311:
0.0%

Covered Lines:
209

Relevant Lines:
245

💛  - Coveralls


Positioning of text is off when using hexes

fvtt 0.8.7
health estimate 2.5.8
the text positioning is off on both axes when the estimate is rendered on a hex map:

Thank you, it should be fixed on the next update.

tested now with 2.5.9. It's definitely much better, but I think it's still not fully centred for me:

I noticed my results weren't exactly like yours but I hoped it would work fine for you.
The odd thing is that it isn't that off-centered for me (maybe half a pixel or so).
Here's my results with the same settings as yours (Outline 0, x-large, Middle, -1):

It might be off by a pixel at most, but it's nothing like yours.

Could you also test it with the Hexagonal Columns grid type and tell me if they're also off-center? I haven't changed them at all on the update because they showed fine for me.

I think the issue is magnified at higher grid sizes.
Here is 100px for me:

And 150px:

I can confirm that hex columns grid renders the text centred at all grid sizes.


should examples/phy/pipe_phy.py work?, I got an import error

When I run examples/phy/pipe_phy.py I got this import error:
    from luna.gateware.interface.serdes_phy.backends.ecp5 import LunaECP5SerDes
ModuleNotFoundError: No module named 'luna.gateware.interface.serdes_phy.backends'

I tried searching for this files/modules and indeed they don't exist, is that a code that should be work in the future or do I miss something?

It looks like that got missed in some refactoring a while ago, I've just pushed a fix, thanks!
However, all of the USB3 parts of the codebase are currently incomplete/experimental, so if you're intending on using that you will find more things that are missing or broken.


[fix][broker] fix MetadataStoreException$NotFoundException while doing topic lookup

Motivation
#14839 fixes the HTTP lookup with multiple advertised listeners, and the added test
is trying to verify the advertised listeners with 2 brokers, but the config of the
the additional broker will not update, so the test gets passed.
The root cause is we are using the (advertised address + WebService port) to register
the broker under the loadbalance path, but while creating the lookup result, we will
use the WebService URL to get the data, so we will get MetadataStoreException$NotFoundException.
Modification

Added properties support for ResourceUnit, so that we can carry more information while getting the least loaded broker
For creating the lookup result, use the broker znode name to get the broker data if it presents in the ResourceUnit

Verification
No new tests are needed, just fix the test that was added in #14839
Documentation
Check the box below or label this PR directly.
Need to update docs?

[x] no-need-doc
(bug fix)

/pulsarbot run-failure-checks


⚠ ClamAV has degraded performance

In e75f05c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 2fd5497 after 12 minutes.


City and County of San Francisco source change

In reference to the data source: https://github.com/openaddresses/openaddresses/blob/f090841936bd20566a7a7498b2ac1aa0296e2ab9/sources/us/ca/san_francisco.json#L13
We have an internal ETL that extracts and syncs the shapefile to a table with point geometry on the portal. Over time we are looking to swap in an internal ETL that connects closer to the source to make the ETL more resilient and easier to monitor. The zipped shapefile isn't going away anytime soon, but I just wanted to see if it would make sense to hook this up to a new endpoint. The data is the same, the endpoints change.
The portal page is here: https://data.sfgov.org/Geographic-Locations-and-Boundaries/Addresses-with-Units-Enterprise-Addressing-System-/dxjs-vqsy
The CSV bulk link is here: https://data.sfgov.org/api/views/dxjs-vqsy/rows.csv?accessType=DOWNLOAD
The GeoJSON bulk link is here: https://data.sfgov.org/api/geospatial/c59h-mc8k?method=export&format=GeoJSON
The Shapefile bulk link is here: https://data.sfgov.org/api/geospatial/c59h-mc8k?method=export&format=Shapefile

Is there a preference on source type w/r/t the ETL engine for open addresses?
Does it make sense to swap in the new endpoint? There's no end of life yet on the zip shapefiles at the existing source, but by switching any changes to data will be documented and communicated at the portal level.
I'm happy to put in the PR if it makes sense

On second look, I would not use the shape file bulk link because of the truncation issue. The CSV and GeoJSON will retain the full names in the schema.


TweepError

hello:
i want to catch error code but get some error.
code:
try:
do something
except tweepy.TweepError as err:
print(err)
output:
[{'code': 88, 'message': 'Rate limit exceeded'}]
i want to get 88, so i use err[0]['code']  but error
or try to err.reason[0]['code'] bur TypeError: string indices must be integers

hello
err is a TweepError object which has 3 attributes (reason, response and api_code)
reason contain the str [{'code': 88, 'message': 'Rate limit exceeded'}]
so you must convert it to json or something similar to get the error code
try:
	do something
except tweepy.RateLimitError as err:
	tojson = json.loads(err.reason.replace("[", "").replace("]", "").replace("'", "\""))
	print(tojson["code"])

This is convoluted advice, although the code shows the better way to handle this. Catch RateLimitErrors separately and handle them - there's really no need to get the response code.
try:
    do_something()

except tweepy.RateLimitError as err:
    # err.api_code isn't assigned here, which is perhaps a little silly but you don't need it!
    print(88)

except tweepy.TweepError as err:
    print(err.api_code)

I met the same problem. Any update?

As @fitnr outlined, you can simply use RateLimitError instead.
Otherwise, you can simply use the TweepError.api_code attribute:
https://github.com/tweepy/tweepy/blob/a8e85263bd20e95322a874344a4776f2b9e7726c/tweepy/error.py#L8-L18  and check for 88.

so i use err[0]['code'] but error

err is a TweepError object/exception, so indexing it by 0 makes no sense, which is why an error is raised if you attempt to do so.

or try to err.reason[0]['code'] bur TypeError: string indices must be integers

TweepError.response is the payload response Twitter's API is returning as a string.
In this case, it would be "[{'code': 88, 'message': 'Rate limit exceeded'}]" as you see in the string representation of the TweepError.
You access the string via index 0, retrieving the first character, '[', and then you attempt to index that character by 'code', hence resulting in the TypeError telling you that you can't access a string/character with a string index.
For code block usage, see https://help.github.com/articles/creating-and-highlighting-code-blocks/.


[Question] Flux Scrubber

Will the Flux Scrubber return in the next update?
Because I am playing my own Modpack called TheZeroProject, and I love Thaumcraft. So the Problem is, that I don't really want to use Thaumcraft at the Moment because I already create Taint 2 times already. I don't want to move because I think Minecraft is a game where you should be able to build where ever you want. So I really would like to get rid of Flux.
Or is there already a way to remove Flux?

There's a sponge in Creative mode, but not in survival at the moment.

Yes I know that one, I already used that one, but I want to play Thaumcraft without cheating so I hope that a way to get rid of flux will be added to Thaumcraft in the next release. Because without it you can't really use Thaumcraft for a lot of stuff, my plan was to make an ore Automation only with Thaumcraft. But it would create much Flux so I have to do Technic instead.

Ok sorry maybe I don't really understand that but what do you mean with tear a rift, And also wich Item Should stabilize a rift ? there is an Infusion Stabilizer this is only for stabilizing Infusion Crafting.

What he means by "tear a rift" is let your flux build up to the point where it creates a tear in space-time; that void-looking tentacle-like thing that you can see through walls.  Then, you use the item that was added last update to stabilize the rift so it continually drains flux from your area without spawning taint related mobs and such.

I mean, isn't that the nature of volatile magic?  Also, I believe that the
rift always spawns as high up as you can get it, so you could always build
a platform high up in the sky before you try to summon a rift for
stabilization.

I've had a rift spawn at ~y=110 with several structures at y>130, nevermind Twilight Forest canopy trees reaching up all the way to 256 or something... not sure if that theory truly holds up.

I mean Thaumcraft 4 is still my favourite and I know Thaumcraft 6 has nothing to do with it. But I think there should be an easy way to get rid of flux without creating a rift.

Don't know if azanor adds an alternative option to clean flux, but Planar Artifice (Flux Scrubber) and Thaumic Additions (Puriflower) both are adding options to clean flux more easy... ;)

I mean I am a fan of Thaumcraft but it just doesn't feel right. I think Azanor will add something to remove Flux in a better way. So I don't want to install a Addons for Thaumcraft which would add an Item to remove Flux in a good way. Because if Azanor finally adds a way to remove Flux on his own, I then would have 2 Mods who would do so. So I personally will wait that Azanor adds this on his own. But my Hope is, that we will see something like that in the next Updates.

Yup stuff is coming soon to handle flux and the rifts


Variable feature request

First of all I just want to say that this is an excellent theme, and thanks for spending your time on it.
I simply just have a feature request: is it possible to get a variable for disabling the expanding of the sidebar when you hover over it?
Don't get me wrong, it looks very clean and it has a great animation- but I find it really hard to to click the search, mute, deafen, and user settings bottom at the bottom left. For example, I aim my cursor towards the user settings button, but when I get close to the button they all move to the right, and it really throws me off and I press the mute button/miss my click completely.
Again, thanks, and I hope you consider this addition.

check aurora/addons/disableExpansions.css


Rename git primary branch to "main"

It's 2023.
Steps:

[ ] Declare cutover day. Ideally not close to any deadlines :-)
[ ] Prepare a PR that updates all relevant workflows in .github, .travis.yml, .jenkinsfiles
[ ] Fix up any scripts in contrib
[ ] Change the settings in github
[ ] Fix any broken CI jobs :-)
[ ] Mirror main to master with https://github.com/google/mirror-branch-action

I might also add remove references to master from the docs

cc: @margamanterola

I'm wondering about the step Mirror main to master with https://github.com/google/mirror-branch-action, github will redirect web requests or return error 301 for master branch so do we need to mirror it? cf. https://github.com/github/renaming#renaming-existing-branches

We only need to mirror main to master if there is tooling out there that expects a git branch named master (that we don't feel like tracking down). Accesses to individual files via the github web UI will be redirected, but anything that uses git will not.


增加回收站功能，方便批量删除，删除不提示警告。

Is your feature request related to a problem?
有些信息不想归档长时间保存。单个删除又太麻烦。
Describe the solution you'd like
以上
Additional context
No response

Use Archive instead of Delete.


Add Code of Conduct link to CONTRIBUTING.md

PR created based on https://github.com/unitycatalog/unitycatalog-ui/pull/46#issuecomment-2262098041. Please review it @dennyglee 😎

Thanks very much @JuanPabloDiaz !


Git hub added Organization Projects and this seems to have broken the bot

None of the bot's features are working. It used to work. i cannot say exactly on when it stopped. maybe a week ago?
Is there a log or something that it outputs when it is having trouble?
We are in a private repo in an enterprise github subscription.

I woke up this morning and created a task, and now the bot is working, did nothing....

So now i have 1 repo that it is working in, and another repo that it is not working in.

Linked: https://github.com/probot/probot/issues/1639

This site i found out has a way to expose if the bot is up: https://github.com/wip/app/

Notice this new issue, no project bot entries: https://github.com/philschatz/project-bot/issues/120

Funny thing, it is working on pull requests:

So check this out:

So it half way worked here, however today it didn't move to the In Process automation, failed here:

https://github.com/philschatz/project-bot/issues/133


Invisible textarea has height of 1px

The textarea containing the plain HTML has a height of 1px when it is not visible (so actually it is visible but only 1px of it). This results in a small line between the scroll bar and the bottom border. Is there a reason why height is not set to 0? (I'm using Firefox 67.0.1)

It's for HTML5 validation purpose.
If you do not need that (no required attribute or so), you can add some CSS like so:
.trumbowyg-box.trumbowyg-editor-visible {
    .trumbowyg-textarea {
		display: none;
    }
}

I think it should work ^^

Thanks for the quick response. Yes, it works. Just for the record in plain CSS it is:
.trumbowyg-box.trumbowyg-editor-visible .trumbowyg-textarea {
    display: none;
}

Actually I can live with the 1px line, though I would suggest to add the following rule to avoid the move of following elements when switching the editor:
.trumbowyg-box.trumbowyg-editor-hidden .trumbowyg-textarea {
    margin-bottom: 1px;
}


Convert Test-RunAsAdmin to Test-ElevationRequirement

Is this:
Just place "x" to indicate selection (e.g. "[x]")

[X] feature
[X] bug:

System Details
The current implementation of Test-RunAsAdmin has some serious issues:

Illegal use of break
Design issues for a command with the Test- Verb

Example uses of the intended command:
if (Test-ElevationRequirement -ComputerName $computer -Not) { return }
if (Test-ElevationRequirement -ComputerName $computer -Not -Continue ) { continue }
if (Test-ElevationRequirement -ComputerName $computer -Not -Continue -ContinueLabel main ) { continue main }
if (Test-ElevationRequirement -ComputerName $computer -NoStop ) {
    # Whatever
}

Additional required ToDos:

Update all instances where it is currently used, as it changes the behavior of the function
Update Stop-Function so it can interrupt commands on a higher level instead of the one where it was called

@FriedrichWeinmann : I do not think this is still valid, can you please close this if it has already been solved ?

closing......


trigger to handle switch between two WiFi

Is there any way to detect the connectivity change if switching from one WiFi to another?

There is not, you can't access the ssid, so it is just if it change. Most likely you would get change events between the two.


Support conversion from hexadecimal number to int

I don't think we have a way of taking a string containing a number in hexadecimal form and convert it into our int. We should have something builtin, either like Python's int("1A", 16) or something else.
@sydow or have I missed something? Is there a way? :)

Hi,
I think the way to go is Python’s — but not until we have parameters with default values working. Otherwise, we’ll get more “Positional element not found”.
Björn

On 23 Sep 2023, at 14:16, Kristian Larsson @.***> wrote:
I don't think we have a way of taking a string containing a number in hexadecimal form and convert it into our int. We should have something builtin, either like Python's int("1A", 16) or something else.
@sydow or have I missed something? Is there a way? :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@sydow hah, yes. I agree completely, we'll wait for default args!  :)
It's hopefully not too long now... =)

Since we now have default arguments, we can proceed with this one! @sydow do you wanna put it at the end of your todo? :)

Yes, I will do it.
But perhaps one should think a bit about design issues first. Maybe Python doesn’t have the optimal solution.
First, we can note that Python, like many other languages, supports integer literals in decimal (123), hexadecimal (0x12ac), octal (0o765) and binary (0b1101) notation. For no good reason Acton does not support the binary version; this could easily be fixed, if found useful. For even less reason there is a bug in Acton’s support for octal digits (it transfers directly the literal 0o765 to C, which is wrong; the correct C version is 0765)... We can also note that in Python 0765 is a syntax error, while in Acton it is accepted and treated as a decimal integer. This is, I think, as it should be.
Now to translation from strings. We want to do this using the int (and i64,…)  constructor. In decimal notation, it is straightforward: int(“123”). For other bases, Python uses a second argument which defaults to 10. So int(“12ac”,16) converts from hex notation; here the ‘0x’ prefix can be used, so int(“0x12ac”,16) is the same, with redundant info that the string is hexadecimal. For octal and binary, the situation is analogous. This raises several questions:

Why not allow int(“0x12ac”) to convert from hex without the second parameter? In Python this raises

ValueError: invalid literal for int() with base 10: ‘0x12ac’
In the same way, we could allow for octal and binary strings with the corresponding prefixes.

Is the second parameter needed (I know that I suggested waiting for optional parameters to implement this)? There are both pro and cons arguments:

with the second parameter we can allow arbitrary base, at least up to 36, if we follow Python and use decimal digits and English letters as digits. But who uses translation from string literals in base 13 or 21?

In Python the second argument is only allowed to appear if the first argument is a string. This is a strange restriction, which is ugly to me.

So I tend to suggest that we don’t introduce any second parameter at all, but allow ‘0x’, ‘0o’ and ‘0b’ prefixes to do translation of hex, octal and binary strings to integers. What do you think?

On 9 Jan 2024, at 15:17, Kristian Larsson @.***> wrote:
Since we now have default arguments, we can proceed with this one! @sydow do you wanna put it at the end of your todo? :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.Message ID: @.***>

@sydow agree with most of what you say. While I support your proposal of allowing things like int("0x12ac") I still quite like the idea of keeping the second argument to allow for arbitrary base, as you mention, but also to make it more explicit. The input value might come from somwehere where we know it is in base 16 but it doesn't have a 0x prefix - it seems cleaner to do int(a, 16) than int("0x" + a). It could be argued it is then redundant, but I think that's OK, we'd just have to check so int("0x12ac", 9) would yield an error since 0x is not base 9.

This seems to be a not very important question, but let’s talk briefly about it on today’s meeting.
I prefer to avoid the second argument but am willing to implement it. But I would like to

get to know the existence of at least one application where one needs to convert strings in a base different from 10, 2, 8 or 16 to an integer. And here I don’t count the math exercise “convert the number 13423 in base 5 to base 10”. This is a worthwhile exercise for a student to understand representation in different bases, but to gain understanding they should either do calculations by hand or write the code needed to do the conversion.

As far as I can see there is no builtin Python function to convert an int to a string in arbitrary base, just 10, 2, 8 or 16 (str, hex, oct and bin)? Does this say anything?

understand why you think that int(a,16) is cleaner than int(“0x”+a) when a is a string of hexadecimal digits.

One can also note that in Python the functions bin, oct and hex do return strings with base prefix. I think this is a good, clean design; both literals and string versions of integers carry their base with them.
On 9 Jan 2024, at 22:21, Kristian Larsson @.***> wrote:

@sydow agree with most of what you say. While I support your proposal of allowing things like int("0x12ac") I still quite like the idea of keeping the second argument to allow for arbitrary base, as you mention, but also to make it more explicit. The input value might come from somwehere where we know it is in base 16 but it doesn't have a 0x prefix - it seems cleaner to do int(a, 16) than int("0x" + a). It could be argued it is then redundant, but I think that's OK, we'd just have to check so int("0x12ac", 9) would yield an error since 0x is not base 9.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Following today’s discussion I propose to do the following, which take the objections raised in the meeting into account. I just want to add one comment, which I didn’t think of immediately when it was raised today. The objection was that one would not for efficiency reasons want to create a new string “0x”+a if we have a very long string, i.e. from some file with thousands of hex digits. I don’t think this is plausible: you may very well have such a file and want to check it for correctness in some way, but you most certainly will NOT want to convert it to an int!  If the file has 10000 digits you would get a number of size around 10^12000 which you can’t do anything with. I still believe that for any practical uses of this constructor time or space efficiency is of no concern in choosing between int(‘0x’+a) and int(a,16).
Anyhow, let's add a second parameter base with default value 10 to the constructors for int types.
A user may or may not provide prefixes ‘0x’, ‘0o’ or ‘0b’ in the first (string) argument. If a prefix is provided and the second argument has the default value 10 (probably because the user omitted it), then the prefix takes precedence.
Examples:
int(“123”) gives 123
All the following give 0x123, i.e. 291:
int(“0x123”)
int(“123”, 16)
int(“0x123”, 16)
int(“0x123”, 10)
int(“0x123”, 8) gives a ValueError.
All values of base except 10, 2, 8, 16 gives a value error in the first implementation, but the  implementation will be dons so that there is no penalty if we want to add arbitrary bases later.

The one thing that sticks out is
int(“0x123”, 10) = 0x123 / 291

I think you are suggesting this because 10 would be the default value for the second argument, so def int(val: str, base: int=10), right? And thus, we cannot differentiate between when base is not set and when it is explicitly set to 10. Doesn't it make more sense to think of it as an explicit base, so we have def int(val: str, base: ?int). The implicit base is still 10, so when base arg is None, then we still presume the string is in base 10 (unless it has a prefix). But if base is set to anything, it will take precedence and if the value string has a prefix that differs from the explicit base argument, it will return an error?

Of course you are right. Thanks!

On 10 Jan 2024, at 14:08, Kristian Larsson @.***> wrote:
The one thing that sticks out is
int(“0x123”, 10) = 0x123 / 291
I think you are suggesting this because 10 would be the default value for the second argument, so def int(val: str, base: int=10), right? And thus, we cannot differentiate between when base is not set and when it is explicitly set to 10. Doesn't it make more sense to think of it as an explicit base, so we have def int(val: str, base: ?int). The implicit base is still 10, so when base arg is None, then we still presume the string is in base 10 (unless it has a prefix). But if base is set to anything, it will take precedence and if the value string has a prefix that differs from the explicit base argument, it will return an error?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

It's possible to search on github for uses of int with a weird base. This searches for base 9: https://github.com/search?q=%2F+int\([^%2C()]%2B%2C+*9\)%2F+language%3APython&type=code - update the query to search for others (there are so many cases of 2, 8, 10 & 16 that I find it easier to explicitly search multiple times for other numbers). From some quick skimming there seems to be use of bases other than 2, 8, 10, 16 although many of them are related to coding competitions, or weird hacks... but I think some of them are valid use cases. Perhaps we should spend a little more time browsing this to gain a better understanding...

This is fixed by #1644.

Cool, that was quick! Nice @sydow =)


Local IP address selection

Is it possible to use a specific IP address if multiple addresses are assigned to a Windows network interface on server?

@starkdm how would you do that on Chrome?

Don't know. I want to understand how it can be done

@starkdm Puppeteer-Sharp automates Chrome, so it would be a matter to know what to automate.


motd: cut v0.2.1

This patch cuts a new version for the motd
crate. The dump-motd wrapper binary remains
unchanged.

Tagging reviewers as an FYI. Once checks pass I'm going to merge since there are no code changes, just a new version.


fix: fix bug in rest_api_query

Summary of Changes
Potential bugs in rest api query:
The yield record_dict for example {'product': 'mode'} may be transformed in an extractor to a dict like {'product': 'mode', 'new_field': 'new_field'}. When the next yield record is extracted, it still should be {'product': 'mode'} instead of {'product': 'mode', 'new_field': 'new_field'} since two records should be unrelevant.
See this extractor as an example that changes record_dict https://github.com/amundsen-io/amundsen/blob/1cece5b4461c8623414ccfb635e495e794a55988/databuilder/databuilder/extractor/dashboard/mode_analytics/mode_dashboard_queries_extractor.py#L74
This is not causing a problem on our end right now. It is a proactive change to avoid possible problems.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

@allisonsuarez @feng-tao @dorianj


[BUG] Coverage plugin finds non-existent files in Cython 3.0.0

Describe the bug
With the release of Cython 3.0.0, my Cython-dependent project is now giving this error when coverage xml is run on GitHub Actions:
For py39, py310, py311 (all OS, Windows error shown):
No source for code: 'D:\a\pycalphad\pycalphad\__init__.cython-30.pxd'.

For py37 and py38 (all OS, Windows error shown):
No source for code: 'D:\a\pycalphad\pycalphad\__init__.pxd'.

Prototypical example here: https://github.com/richardotis/pycalphad/actions/runs/5708548169/job/15466332182
This pxd file does not exist anywhere in the build pipeline, neither in the built wheels nor the source working directory, which I have confirmed with list-dir operations.
Downgrading to 0.29.x makes the error go away, and (according to debug logs) coverage does not detect this spurious file.
Code to reproduce the behaviour:
None available unfortunately as I cannot reproduce locally
Expected behaviour
This spurious file should not be detected.
OS
Linux, Windows, and Mac OS
Python version
3.7, 3.8, 3.9, 3.10, 3.11
Cython version
3.0.0
Additional context
I am struggling to reproduce this bug in my local environment, but I'm seeing it on all platforms for CI ( https://github.com/pycalphad/pycalphad/pull/484 ). I am admittedly not sure if this is a bug in Cython's Coverage plugin, or in Coverage.py itself.
Interestingly, if I adjust to use Cython<3 as a build dependency in my pyproject.toml, so PEP517 wheels are built in an isolated environment with 0.29.x, but leave Cython 3.0.0 in my main environment where pytest and coverage xml runs (so the 3.0.0 coverage plugin is used for the report generation), the spurious file is not detected and everything works.
This is speculation, but I suspect that something in the coverage metadata for the Cython-generated source files (.cpp, .h, etc) changed between 0.29.x and 3.0.0, and this is causing the Cython coverage plugin to attribute some change in coverage to a non-existent pxd file.
Again, everything is working for me in my local development environment, even when I try to follow all steps. If there's a suggestion on how I might be able to piece together a reproducer for GHA, it would be greatly appreciated.

As a workaround, you can tell Coverage.py to turn the error into a warning by adding this configuration to pyproject.toml:
[tool.coverage.report]
ignore_errors = true


docs: fix a typo in the Request/reply pattern tutorial document

Description

There is a little typo in the document.
pong-pong to ping-pong

@TRohit20 @alequetzalli please have a look

@derberg please merge it.

@TaegyunWoo @TRohit20 sorry but this can be merged only if @alequetzalli approves it

@alequetzalli need your approval here

This is a duplicate and already fixed on live website :)


MTG JSON 4 support

The script currently fails because of the breaking changes in MTG JSON version 4. I see two feasible ways of fixing this:

Update the mtgjson dependency with mbr/pymtgjson#7 or replace it.
Rewrite the script in Rust, which is planned anyway for integration with the Lore Seeker desktop app.

I'm probably going to go ahead with option 2, but a pull request with option 1 would be welcome. Update: work on option 2 is in progress on the riir branch.

Since this has seen a lot of progress already, I'm closing this in favor of #25.


Typescript typings

I realize this is quite a long shot, but I'm wondering if we can use the typings from @types/dc to create typings for the functions in this library. I'm willing to help out, but I'm relatively new to creating type files and I'm having trouble with some of the code (I already started trying this on my end).
For example:
export function baseMixin(chart, props) {...

seems like it should become approximately
export function baseMixin(chart: DCGraphicType, props: BaseMixin<any>) {...

where
export type DCGraphicType =
  | dc.PieChart
  | dc.SunburstChart
  | dc.BarChart
  | dc.LineChart
  | dc.DataCountWidget
  | dc.DataTableWidget
  | dc.DataGridWidget
  | dc.BubbleChart
  | dc.CompositeChart
  | dc.SeriesChart
  | dc.GeoChoroplethChart
  | dc.BubbleOverlayChart
  | dc.RowChart
  | dc.ScatterPlot
  | dc.NumberDisplayWidget
  | dc.HeatMap
  | dc.BoxPlot
  | dc.SelectMenu
  | dc.TextFilterWidget
  | dc.CBoxMenu;

however, it seems like showControls, id are not on the BaseMixin, amongst other hurdles. I'd be open to hearing how best to approach this issue, as I tried debugging the playground but it doesn't seem there are source maps...

Hey @parksj10, thanks for contributing!
That seems like the right approach, nearly all of the typings should apply here, as the methods are the same.
There are corner cases, as you mentioned, such as showControls (among others being developed). This one is essentially a shorthand for turnOnControls and turnOffControls. As these actions are declarative in dc.js, I took the opportunity to convert them to a more idiomatic react approach based on a single value showControls=true (means turnOnControls), otherwise, I couldn't represent them correctly.
id refers to an identifier to keep track of the chart in an internal registry (I'm strongly considering to keep it as prop but refactor it out of baseMixin)
As for the source maps, the playground is indeed a bit messy in terms of source maps. I'll have a look over an alternative and come back to this.

Added source maps in 4b024b7c51fd0331f0c2097458cbeecb8baa4945


[Question] Data types supported by the Connector

How does the Connector support connections or inputs of different data types (such as double, int, bool, string, and even custom classes)?

The connector's view model can store information about the data type and the value. Then the XAML can use a different input control for each data type.

Feel free to reopen the issue if you need more information.


Status of build server can't be seen because of permissioning

The status of builds from the readme file doesn't show. I imagine it might for you as you might have a cookie enabling it. See the screenshot below:

Should be fixed, can you confirm?

This is fixed. Thanks for fast turn around :D


[BUG] Node removed from Rancher during node reboot

Rancher Server Setup

Rancher version: 2.7.5-rc6 (started with 2.7.4, same issue)

Information about the Cluster

Kubernetes version: v1.25.10+rke2r1
Cluster Type: Custom

3 nodes - controlplane, etcd roles
3 nodes - worker role
OpenSuSE MicroOS with RKE2 Provisioner.
Cluster nodes are provisioned with system-agent-install.sh which is fetched from Rancher itself
https://rancher.example/system-agent-install.sh --server ${SERVER} --label 'cattle.io/os=linux' --token ${TOKEN} ${ROLES}

Describe the bug
When you reboot any node in cluster, it becomes Unavailable in Rancher, then immediately disappears, and never comes back, despite server is back from reboot.
To Reproduce

Run Rancher 2.7.4 or higher on OpenSuSE MicroOS with RKE2 any Kubernetes version.
Reboot any node in cluster

Result

Node disappears in Rancher and never comes back, despite server is up and running after a few minutes.

Expected Result
Node is not removed from Rancher, and when server comes back, node becomes Active.
Additional context

When you run kubectl get node in a loop, you see that node registers in Kubernetes after reboot, then disappears. Though, you don't see it in Rancher UI.
apply-system-agent-upgrader job crashes with You must select at least one role error. Job script checks for "node-role.kubernetes.io" labels in order to detect role, but there's no such labels after node is removed from Rancher, and then registered back (The moment whe node appears in k8s for a moment before disappears)
OpenSUSE MicroOS uses transcational-update for atomic OS upgrades which requires reboot.
Full wipe and reinstall of a node brings it back to Rancher.
nodeDeletionTimeout change in Machine object before server reboot has not effect.

It appeared that was missing one detail.
We use VCD as a cloud provider, and we run vmware-cloud-director-ccm in Kubernetes.
When node is not available, this CCM sends an event which triggers k8s controller manager which deletes node.
Closing issue as not relevant to Rancher.


Fix typo in log message

Found this typo doing some debug.

@microsoft-github-policy-service agree

Thanks @elarrieux


Wrong 'reveal' config for Linux

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
kitty 0.30.1
Did you try the latest code to see if this problem got fixed?
Tried, but the problem still
yazi --debug output

$ yazi --debug
Yazi
yazi 0.2.4 (VERGEN_IDEMPOTENT_OUTPUT 2024-03-10)

Environment
OS: linux-x86_64 (unix)
Debug: false

Emulator
Emulator.via_env: ("xterm-kitty", "")
Emulator.via_csi: Ok(Kitty)
Emulator.detect: Kitty

Adaptor
Adaptor.matches: Kitty

tmux
TMUX: false

Zellij
ZELLIJ_SESSION_NAME: None

Desktop
XDG_SESSION_TYPE: Some("x11")
WAYLAND_DISPLAY: None
DISPLAY: Some(":0")

Ueberzug
Version: Ok(Output { status: ExitStatus(unix_wait_status(256)), stdout: "", stderr: "Usage:\n    ueberzug layer [options]\n    ueberzug library\n    ueberzug version\n    ueberzug query_windows PIDS ...\n" })

WSL
/proc/sys/fs/binfmt_misc/WSLInterop: false

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/jan/.local/state/yazi" directory.

Describe the bug
The last default reveal action at https://github.com/sxyazi/yazi/blob/latest/yazi-config/preset/yazi.toml#L41 seems to be incorrect, it tries to show the EXIF info instead of revealing the file in a GUI file manager. The EXIF action would probably only make sense for images.
The best "reveal" action I've found for Linux is this:
    { run = 'gtk-launch $(xdg-mime query default inode/directory) "$1"', desc = "Reveal", for = "linux" }

Unfortunately this only works if gtk-launch is installed, which probably isn't always the case. There are some alternatives listed here but it would probably require a dedicated shell script or plugin to pick the right one for the current system.
Expected Behavior
List a "Reveal" action for Linux
To Reproduce
Try to reveal a file on Linux
Configuration
No response
Anything else?
No response

Hi, I think this is expected behavior, since I didn't find a universal way to add "Reveal" for Linux, so I decided not to add it. Instead, I want to encourage users to customize it according to their needs and installed tools.
I set the description for this exiftool command as "Show EXIF" instead of "Reveal" to avoid displaying misleading text when the user presses O. However, it may not be appropriate to include it in reveal = [], perhaps it's worth a new category, but I want to keep the default configuration as simple as possible. Curious about your thoughts!

I also found "Show EXIF" confusing until I went source diving and read the macos and windows defaults for "Reveal"
In the absence of a script to intelligently handle various GUI file managers, I'd suggest xdg-open . or nothing would be better defaults on Linux.

If it's at all useful, some quick research:

Dolphin: dolphin --select foo, or dolphin --select foo --select bar
Nautilus: nautilus --select foo, opens multiple windows if specifying more than one path (probably not desired)
Thunar: thunar foo, opens multiple windows if specifying more than one path (probably not desired), passing a directory opens that directory instead of selecting it

I agree with @callahad, without a better general solution just using xdg-open . would probably be good enough as the default for reveal on Linux. It would probably still make sense to move the "EXIF" action to a separate, image-specific opener though.

IIRC, xdg-open /a/b opens the directory b, i.e. CWD is /a/b, while the behavior of "reveal" should open directory a and select directory b, i.e. CWD is /a.
I think these are different, and it only works for directories. If it's a file, it won't open the GUI file manager, because the file will be opened directly in the corresponding application, such as mpv for video files.

IIRC, xdg-open /a/b opens the directory b, i.e. CWD is /a/b, while the behavior of "reveal" should open directory a and select directory b, i.e. CWD is /a.

Yes, this is correct. That's why I think it would make sense to explicitly use xdg-open . to open the current directory. This should still open the directory the currently "hovered" file is in, it just won't select it. That's what I meant with a "good enough" solution for the general case. This is only a suggestion of course, I just think it's better than not having any kind of action similar to "reveal" by default.

Ah I see, will create a PR to add it


Using next export

Discussed in https://github.com/shuding/nextra/discussions/1197

Originally posted by RPDeshaies January  6, 2023
Hi, I'm currently testing using Nextra in replacement for a Vuepress project I had and I was wondering if it was possible to use the command next build && next export with Nextra?
I've tried running the command and looking at the exported assets, but I'm getting a bunch of 404 of the static assets (js/css/etc)

I initially had an error about using next/images which I fixed by overriding a config in the next config, but I'm not sure at all that's the proper way of doing things.
info  - No "exportPathMap" found in "/next.config.js". Generating map from "./pages"
Error: Image Optimization using Next.js' default loader is not compatible with `next export`.
  Possible solutions:
    - Use `next start` to run a server, which includes the Image Optimization API.
    - Configure `images.unoptimized = true` in `next.config.js` to disable the Image Optimization API.
  Read more: https://nextjs.org/docs/messages/export-image-api

// next.config.js
const withNextra = require("nextra")({
  theme: "nextra-theme-docs",
  themeConfig: "./theme.config.jsx",
});

module.exports = {
  ...withNextra(),
  images: {
    unoptimized: true,
  },
};

Thanks in advance for the help. And also, Nextra looks absolutely fantastic, I'm really eager to make it work!

`const withNextra = require('nextra')({
theme: 'nextra-theme-docs',
themeConfig: './theme.config.tsx',
staticImage: false,
images: {
unoptimized: true,
},
})
module.exports = withNextra()`
Might solve this issue

'staticImage: false' in Config might solve this

I got the same error and switched image optimization off.

I got the same error and switched image optimization off.

Same error as OP. Playing with next config and various solutions, but I suspect something isn't parsing between nextConfig and withNextra() on export.
@pplonski can you share your working config? Thanks!

I just used:
// next.config.js
const withNextra = require("nextra")({
  theme: "nextra-theme-docs",
  themeConfig: "./theme.config.jsx",
});

module.exports = {
  ...withNextra(),
  images: {
    unoptimized: true,
  },
};

From what I read the yarn next export doesnt work with optimized images.

Interesting! You're deploying to Netlify? What are your build & deploy commands if so? I keep getting the same image problem when deploying to Fleek, but it builds fine. Also to note, this config breaks the Vercel demo build entirely via logs, which was working as expected out of the box.

Working for me on non-Vercel & Vercel deploys:

updated build command on line 6 in package.json to:
"build": "node ./scripts/gen-rss.js && next build && next export"
updated next.config.js to:

const withNextra = require('nextra')(https://github.com/shuding/nextratheme-blog', './theme.config.js')

const nextConfig = {
  images: {
    loader: 'akamai',
    path: '/'
  }
}
module.exports = withNextra(nextConfig)

set build commands in deploy to:
yarn install && yarn run build

closed by https://github.com/shuding/nextra/pull/2563


Feature/fix filter and search

Removed this.greyOutMarkers() from filterActiveProviders because its implementation was deleted a while ago. Zipcode search is still not fixed

This contains all of master code, which has some changes that develop doesn't want. I copy and pasted the good parts and now the search and filter work again. I'll make that a seperate commit into develop. I'll look into the zip code issue too.


Remove creation of admin account during setup

@cpauya & @aronasorman this fixes #238
I attached a sample screenshot of the KA_Lite preferences dialog.

@amodia Some code clean-ups are needed.

@cpauya that is not a copy paste code, it's an sent action of the start, stop button in the preferences menu.

They are the same codes, only different function names, example:
- (IBAction)stop:(id)sender {
    showNotification(@"Stopping...");
    if (self.processCounter != 0) {
        alert(@"KA Lite is still processing, please wait until it is finished.");
        return;
    }
    [self runKalite:@"stop"];
}

- (IBAction)stopButton:(id)sender {
    showNotification(@"Stopping...");
    if (self.processCounter != 0) {
        alert(@"KA Lite is still processing, please wait until it is finished.");
        return;
    }
    [self runKalite:@"stop"];
}

I tested your PR and its working fine.

@amodia merge conflicts

@cpauya I already merged develop and run it on this branch. I attached a sample screenshot.

Nice one @amodia - merging this now.


Question: why does hlint complaint about Redundant lambda?

hlint complaints about redundant lambda.
Yet on https://downloads.haskell.org/~ghc/7.0.3/docs/html/users_guide/pragmas.html
it is stated:
Moreover, GHC will only inline the function if it is fully applied, where "fully applied" means applied to as many arguments as appear (syntactically) on the LHS of the function definition. For example:
comp1 :: (b -> c) -> (a -> b) -> a -> c
{-# INLINE comp1 #-}
comp1 f g = \x -> f (g x)

comp2 :: (b -> c) -> (a -> b) -> a -> c
{-# INLINE comp2 #-}
comp2 f g x = f (g x)

The two functions comp1 and comp2 have the same semantics, but comp1 will be inlined when applied to two arguments, while comp2 requires three. This might make a big difference if you say
map (not `comp1` not) xs

which will optimise better than the corresponding use of `comp2`.

So there is a good reason to use lambda expressions in definitions and I had not expected this warning.
But probably I miss something...

HLint has the goal of making Haskell code cleaner - not faster. Some of the changes may make it faster. Some of the changes may inhibit GHC's optimiser to a small extent. If you've hyper-optimised your code to the extent you think it will matter, you shouldn't even be breathing on it. In general it probably makes no difference to the optimiser, but if the code is clearer, humans will be grateful.


fix-typo in EquationsPlugins

i found typo in EquationsPlugin.js.

@alajillo I see there's a merge conflict, if you could please resolve that, we can merge your PR :)

@trueadm thanks :)

@trueadm  i've resolved merge conflicts but there are some problem in e2e-test. how can i fix it?

@trueadm i've resolved merge conflicts but there are some problem in e2e-test. how can i fix it?

Seems like a bad run (on our side). Thank you!


错误: 写入（文件名） (拒绝访问。)    这个错误经常出现是什么问题，我把这个文件强制删除后运行正常，再运行就又出现了这个错误

Look at here

→ Did you conform to the Rules?
→ Did you read the Wiki Carefully?
→ Did you search in Google?
→ Did you search in openned&closed Issues?
→ Receive only bugs and suggestions

Environment

[x] MVPArms Version:
[x] AndroidStudio Version:
[x] Gradle Plugin Version:
[x] Target Android Version:
[x] Device Model:

Bug Description:

Related Code:

Bug Log:

Others:

不清楚，你问的是说明问题


Issue #371, plus other enhancements/cleanup to PersonaCard

Updates to PersonaCard.  Bug #371 plus:

Order of actions <li> and detail sections in the HTML are now irrelevant.  Action has an attribute 'data-datailsTargetId' which must match the id of the target <ul>
Whichever action has the 'is-active' attribute will be set to visible on initial load
JS no longer tied to specific class names in CSS for which action/details to show (no need to change CSS if changing actions)
Made sure it works with multiple cards on a page (so long as IDs are unique)
Cleaned up CSS, JS and HTML
Added support for actions to navigate to new pages via data-actionUrl attribute (i.e. for "View Profile" action.)

Hi @manndave, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@manndave, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Crap.  I deleted my forked repo from the wrong account so this PR is probably lost.  I still have a local copy so let me know if you need me to resubmit the PR.  :-(

Could you resubmit it? I was finally going to test your changes today (I swear!) but I don't think I can without manually making them based on GitHub's files changed view.

Done: https://github.com/OfficeDev/Office-UI-Fabric/pull/413.
Sorry for the hassle...

A new pull request has been opened for this, closing this one.