[feat][cli] Support for transaction consistence test with PerformanceProducer.

Fixes #19780
Motivation
enhance PerformanceProducer to support for transaction consistence test.
Modifications

implement a message formatter to generate specific message format.
implement client-side persistence scheme.

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[x] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/thetumbled/pulsar/pull/18

BTW, this PR do not contain the code for reading content from topics and check for the data integrity, because i implement it out of pulsar project. the detailed code is as follows:
package org.example.utils;

public class PulsarCommonUtils {
    private static final Logger log = LoggerFactory.getLogger(PulsarCommonUtils.class);

    @Parameter(names = "-topic", description = "topic name")
    String topic = "persistent://test/test/testTxn10";

    @Parameter(names = "-sub", description = "subscription name")
    String sub = "sub";

    @Parameter(names = { "-u", "--service-url" }, description = "Pulsar Service URL")
    public String serviceURL;

    @Parameter(names = "-token", description ="token used to authenticate.")
    public String token = "";

    @Parameter(names = "--help", description = "help info", help = true)
    private boolean help = false;

    // check the content of topic whether constitute a continuous range
    public static boolean checkCorrectness(PulsarCommonUtils pulsarCommonUtils) throws PulsarClientException {
        boolean result = true;
        if (pulsarCommonUtils.serviceURL == null || pulsarCommonUtils.serviceURL.length() == 0) {
            log.error("please input Pulsar Service URL");
            return false;
        }
        PulsarClient client = PulsarClient.builder()
                .serviceUrl(pulsarCommonUtils.serviceURL)
                .authentication(AuthenticationFactory.token(pulsarCommonUtils.token))
                .build();
        Consumer<byte[]> consumer = client.newConsumer()
                .topic(pulsarCommonUtils.topic)
                .subscriptionName(pulsarCommonUtils.sub)
                .subscribe();
        RangeSet<Long> rangeSet = TreeRangeSet.create();
        Message msg;
        Long id;
        long msgCount = 0l;
        long duplicateCount = 0l;
        while (true) {
            msg = consumer.receive(10, TimeUnit.SECONDS);
            if (msg == null) {
                break;
            }
            msgCount++;
            id = CommonUtils.bytesToLong(msg.getData());
            if (rangeSet.contains(id)) {
                duplicateCount++;
            } else {
                // for n, we add range [n,n+1) into rangeSet.
                rangeSet.add(Range.closedOpen(id, id + 1));
            }
            consumer.acknowledge(msg);
        }
        // check for message loss
        Set<Range<Long>> set = rangeSet.asRanges();
        if (set.size() > 1) {
            long validCount = 0;
            for (Range<Long> range : set) {
                validCount += range.upperEndpoint() - range.lowerEndpoint();
            }
            log.info("Some messages are missing! more than one range exists." +
                    "Valid messages Count:{} RangeSet:{}", validCount, rangeSet);
            result = false;
        }else if (set.size() == 0) {
            log.info("there is no any message!");
            return true;
        }else{
            log.info("only one range exists. RangeSet:{}", rangeSet);
        }

        // check for message duplication
        if (duplicateCount != 0) {
            result = false;
        }
        log.info("duplicate message count:{}, total message count:{}, duplicate rate:{}%",
                duplicateCount, msgCount, 100.0 * duplicateCount / msgCount);

        client.close();
        return result;
    }

    public static void main(String[] args) {
        PulsarCommonUtils pulsarCommonUtils = new PulsarCommonUtils();
        JCommander jcommander = JCommander.newBuilder().addObject(pulsarCommonUtils).build();
        try {
            jcommander.parse(args);
            if (pulsarCommonUtils.help) {
                jcommander.usage();
                System.exit(1);
            } else {
                if (pulsarCommonUtils.checkCorrectness) {
                    checkCorrectness(pulsarCommonUtils);
                }
            }
        } catch (Exception e) {
            System.err.println(e);
            System.exit(1);
        }
    }
}


Support Writing CX Extensions for Reactions

Reference Issue

This is an iteration of an older pull request (https://github.com/rdkit/rdkit/pull/7340)
There are still a few things that are still unsupported.

Link nodes
Coordinate bonds

What does this implement/fix? Explain your changes.
This pull request seeks to add support for utilizing CX extensions for reactions.
Core functionality of this add-on is in regards to supporting CXSmiles and allowing for atomLabel's and basic properties of the extension.
Any other comments?
Issues brought up by @rachelnwalker still stand regarding substance group writing with regards to extra commas and colons written compared to the input
Input SMARTS: [CH3:6][O:5][CH:3](-*)[O:2]-*>>[CH3:6][NH:5][CH:3](-*)[O:2]-* |$;;;star_e;;star_e;;;;star_e;;star_e$,SgD:1,0:foo:bar::::,SgD:7,6:foo:baz::::,Sg:n:4,2,1,0::ht,Sg:n:10,8,7,6::ht,SgH:2:0,3:1|

Roundtrip output SMARTS: [C&H3:6][O:5][C&H1:3](-*)[O:2]-*>>[C&H3:6][N&H1:5][C&H1:3](-*)[O:2]-* |$;;;star_e;;star_e;;;;star_e;;star_e$,SgD:1,0:foo:bar::::,,SgD:7,6:foo:baz::::,,,Sg:n:4,2,1,0::ht:::,,Sg:n:10,8,7,6::ht:::,SgH:3:1.1|

Code Examples
Some code examples can be seen here:
Examples for writing to reactions to CXSmiles
from rdkit.Chem import AllChem, rdChemReactions
CXreaction_string = "[CH3:1][CH:2]([CH3:3])[*:4].[OH:5][CH2:6][*:7]>>[CH3:1][CH:2]([CH3:3])[CH2:6][OH:5] |$;;;_R1;;;_R2;;;;;;;;$|"
reaction = rdChemReactions.ReactionFromSmarts(CXreaction_string)
cxsmiles_reaction_string = Chem.rdChemReactions.ReactionToCXSmiles(reaction)
>>> print(cxsmiles_reaction_string)
[CH3:1][CH:2]([CH3:3])[*:4].[OH:5][CH2:6][*:7]>>[CH3:1][CH:2]([CH3:3])[CH2:6][OH:5] |$;;;_R1;;;_R2;;;;;$|

and also CXSmarts
cxsmarts_reaction = "[CH3:1][CH:2]([CH3:3])[*:4].[OH:5][CH2:6][*:7]>O=C=O>[CH3:1][CH:2]([CH3:3])[CH2:6][OH:5] |$;;;_R1;;;_R1;;;;;;;;$|"
rxnCXSmarts = rdChemReactions.ReactionFromSmarts(cxsmarts_reaction)
rxnCXSmarts_string = rdChemReactions.ReactionToCXSmarts(rxnCXSmarts)
>>> print(rxnCXSmarts_string)
[C&H3:1][C&H1:2]([C&H3:3])[*:4].[O&H1:5][C&H2:6][*:7]>O=C=O>[C&H3:1][C&H1:2]([C&H3:3])[C&H2:6][O&H1:5] |$;;;_R1;;;_R1;;;;;;;;$|

@KollinRT I'm wondering why the last substance group is different in the example (SgH:2:0,3:1 vs SgH:3:1.1)? I'm also not sure why there are additional commas in the roundtrip SMARTS

@KollinRT I'm wondering why the last substance group is different in the example (SgH:2:0,3:1 vs SgH:3:1.1)? I'm also not sure why there are additional commas in the roundtrip SMARTS

It looks like the PARENT relationship between the substance groups in the reactant is being stripped. What happens when you write out the CXSmarts for just the reactant?

@KollinRT I'm wondering why the last substance group is different in the example (SgH:2:0,3:1 vs SgH:3:1.1)? I'm also not sure why there are additional commas in the roundtrip SMARTS

It looks like the PARENT relationship between the substance groups in the reactant is being stripped. What happens when you write out the CXSmarts for just the reactant?

[C&H3:6][O:5][C&H1:3](-*)[O:2]-* |$;;;star_e;;star_e$,atomProp:0.molAtomMapNumber.6:1.molAtomMapNumber.5:2.molAtomMapNumber.3:4.molAtomMapNumber.2,SgD:1,0:foo:bar::::,,,Sg:n:4,2,1,0::ht:::,SgH:1:0|

It appears the substance groups are still preserved? I wonder if the substance group hierarchy is impacted by the reactant/product group hierarchy or is this more of a parsing bug than other things?

It appears the substance groups are still preserved? I wonder if the substance group hierarchy is impacted by the reactant/product group hierarchy or is this more of a parsing bug than other things?

I think you're right. It looks like this happens with insertMol anyway:
smarts = "[CH3:6][O:5][CH:3](-*)[O:2]-*>>[CH3:6][NH:5][CH:3](-*)[O:2]-* |$;;;star_e;;star_e;;;;star_e;;star_e$,SgD:1,0:foo:bar::::,SgD:7,6:foo:baz::::,Sg:n:4,2,1,0::ht,Sg:n:10,8,7,6::ht,SgH:2:0,3:1|"
rxn = rdChemReactions.ReactionFromSmarts(smarts)

reactant = rxn.GetReactants()[0]
product = rxn.GetProducts()[0]

combined_mols = Chem.RWMol(reactant)
combined_mols.InsertMol(product)

print(Chem.MolToCXSmarts(combined_mols))

gives
[C&H3:6][O:5][C&H1:3](-*)[O:2]-*.[C&H3:6][N&H1:5][C&H1:3](-*)[O:2]-* |$;;;star_e;;star_e;;;;star_e;;star_e$,atomProp:0.molAtomMapNumber.6:1.molAtomMapNumber.5:2.molAtomMapNumber.3:4.molAtomMapNumber.2:6.molAtomMapNumber.6:7.molAtomMapNumber.5:8.molAtomMapNumber.3:10.molAtomMapNumber.2,SgD:1,0:foo:bar::::,,SgD:7,6:foo:baz::::,,,Sg:n:4,2,1,0::ht:::,,Sg:n:10,8,7,6::ht:::,SgH:3:1.1|
I'm not familiar with the details of how substance group hierarchy works, so I'm not sure whether this is a bug or not.
Re: the link nodes (and coordinate bonds): I believe the issue is that link nodes and coordinate bonds are not copied over in insertMol:
m = Chem.MolFromSmiles('OC1CCCC1 |LN:1:<IP_ADDRESS>|')
new_m = Chem.RWMol()
new_m.InsertMol(m)
print(f"{Chem.MolToCXSmiles(m)} vs {Chem.MolToCXSmiles(new_m)}")

gives
OC1CCCC1 |LN:1:<IP_ADDRESS>| vs OC1CCCC1

This is looking good to me! @greglandrum I think this is ready for you to take another look at

I just tried resolving the merge conflicts. If the CI builds pass, I will go ahead and merge this


VS Code not responding after hitting a key stroke

Type: Bug
It doesn't repro everytime, but when it happens, it it like the screen is locked, both keyboard and mouse clicks are no longer accpted. Before hitting a keystroke, mouse still works. So It looks more like the issue is triggered by keyboard key stroke.
Also, I've experienced similar issue on Microsoft Edge browesr, so I wonder if it is related to bugs on the chromium engine that both apps based off.
There is no issue on Google Chrome, so the chromium engine must have been diverged, and it is microsoft apps specific now.
VS Code version: Code 1.95.2 (e8653663e8840adaf45af01eab5c627a5af81807, 2024-11-07T11:07:22.054Z)
OS version: Windows_NT x64 10.0.26100
Modes:

System Info

Item
Value

CPUs
Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz (16 x 2793)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: unavailable_softwarewebnn: unavailable_software

Load (avg)
undefined

Memory (System)
63.95GB (36.72GB free)

Process Argv
. --crash-reporter-id 33f9c5ed-6bdd-411a-946e-94e2347c382e

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

azure-dev
ms-
0.8.4

vscode-azureappservice
ms-
0.25.4

vscode-azurefunctions
ms-
1.15.4

vscode-azureresourcegroups
ms-
0.9.9

vscode-azurestaticwebapps
ms-
0.12.2

vscode-azurestorage
ms-
0.16.1

vscode-azurevirtualmachines
ms-
0.6.6

cpptools
ms-
1.22.11

cpptools-extension-pack
ms-
1.3.0

vscode-node-azure-pack
ms-
1.2.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
g316j359:31013175
dvdeprecation:31068756
dwnewjupyter:31046869
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013

Please share the profile from the unresponsive window following the steps at https://github.com/microsoft/vscode/wiki/Performance-Issues#profiling-the-renderer-process


Bug: application doesn't terminate on receipt of signal interrupt ctrl+c

running as normal unprivileged user on linux.
steps to reproduce:

issue an invalid command e.g. qme --help
observe that error message received and then application begins idling
introduce signal interrupt key sequence ctrl+c
application acknowledges interrupt signal received and states shutdown is beginning
application hangs indefinitely and doesn't shut down

2022/01/26 21:42:13 failed to execute command --help: exec: "--help": executable file not found in $PATH
2022/01/26 21:42:13 idling...
^C2022/01/26 22:04:11 interrupt signal received
2022/01/26 22:04:11 shutting down

Thanks for raising this issue @c-nv-s, I'll report back once I replicate the bug.

any update on this getting fixed? or is the project now on ice?

@c-nv-s Sorry, I forgot this issue even existed, and no, the project is alive and kicking.
The issue is that there's no argparser used in the app. It considers every argument passed to it as part of the command that will be run.
$ qme cmd arg1 arg2
# runs `cmd arg1 arg2`
qme --help
# runs `--help`, which doesn't work

I'll add a proper flag parser and push a new update


Create web app design

We need a design for the web app and preferable we should use Adobe XD.
In case you didn't know you can download Adobe XD for free here.
At the moment I am still very open to any new suggestions on how this app will work and how we will differentiate the users. Maybe through a abstract JSON database that gets updated with Node.js? Please also comment any suggestions here or join the Discord to discuss changes and get help.
Tips for the design:

Use plants as a background whenever possible
Keep in mind that it needs to be responsive
Keep it modern, sleek and elegant
Use the picture in the README.md as inspiration
Please use pictures from Unsplash or similar sites
Max width for the wrapper is 1280 pixels

What I think we need (may change):

A landing page where the user has to enter his name at least (I don't know yet if we will make accounts)
A quick tutorial where the user gets to see the core features
A overview for the workouts like in the picture mentioned above
A page where you add your workouts
A page where you can see a workout day you select

Submit all your designs and ideas here.

This is the current status of the design:

Proposed change to accommodate for cardio change

[ ] Design a "Add a workout" page for cardio training

Here is the cardio workout page. But as @Flusinerd mentioned in our Discord the "Workout time" part may not make any sense since nobody ever hits a preset time usually.


how to connect phoenix with user and password?

like following java code:
DriverManager.getConnection("jdbc:phoenix:thin:url=http://<IP_ADDRESS>:8765;serialization=PROTOBUF", "user", "password");

@ivanjoy42
Thanks for opening the issue.
I have not implemented authentication yet. I think the implementation will be part of the DSN:
http://[username:password[|comma-separated-roles]]@address:port[/schema][?parameter1=value&...parameterN=value]

I think it will only be possible to implement HTTP Basic and HTTP digest auth at the moment, as there is no pure-go kerberos client yet.
Would you be able to implement this and open a PR?

@ivanjoy42 After investigating, I cannot find any information where Phoenix/Hbase supports username/password pairs. The only option is to use kerberos.
Have you tried the code you posted in a Java environment?
DriverManager.getConnection("jdbc:phoenix:thin:url=http://<IP_ADDRESS>:8765;serialization=PROTOBUF", "user", "password");

I connect to phoenix through Phoenix Query Server
I did not use kerberos. Just modify Phoenix Query Server server side java code to verify user and password.
For java use Phoenix Query Server client, phoenix-version-thin-client.jar
DriverManager.getConnection("jdbc:phoenix:thin:url=http://<IP_ADDRESS>:8765;serialization=PROTOBUF", "user", "password");
For python use phoenixdb
phoenixdb.connect('http://<IP_ADDRESS>:8765', user='user', password='password')
For go I can also connect to Phoenix Query Server.
But how to add user/password param?

@ivanjoy42, the Go driver currently does not support support using a username and password pair yet.
I am using the pre-built phoenix query server by running: bin/queryserver.py start &.
I tried grepping the python phoenixdb project, but cannot see how it is using the username and password.
Can you show the code modifications to the Phoenix Query Server?

hey @ivanjoy42,
After some research, here are the possible authentication methods:

Pass username and password directly through to the backend database (like your example).
Pass username and password to avatica (authentication is for avatica only) using either HTTP Basic or Digest auth.
Use SPNEGO/Kerberos

I think the easiest to implement would be your use-case:
http://username:password@your-phoenix-queryserver:1234/schema

For Keberos (in the future when there is a good pure-go kerberos library, or maybe using cgo):
http://your-phoenix-queryserver:123456/schema?keytab=path/to/keytab&principal=someprincial

For Avatica auth, it is more complicated. We need to account for the possibility where the driver authenticates with avatica, then passes a separate username/password down to the database. Maybe something like this:
http://username:password@your-phoenix-queryserver:1234/schema?avatica_user=avatica&avatica_password=avatica-password&authentication=DIGEST

Let me know what your thoughts are.

Thanks for your replay!
I've been tried above DSN methods, but failed.
I modify phoenix queryserver by this way: open following file, add 3 lines between 431-432 line, like this:
phoenix-core-4.10.0-HBase-1.1-sources/org/apache/phoenix/query/ConnectionQueryServicesImpl.java
    private void openConnection() throws SQLException {
        try {
            boolean transactionsEnabled = props.getBoolean(
                    QueryServices.TRANSACTIONS_ENABLED,
                    QueryServicesOptions.DEFAULT_TRANSACTIONS_ENABLED);

            String user = props.get("user");
            String password = props.get("password");
            logger.info("Test user: " + user + "password: " + password);

            this.connection = HBaseFactoryProvider.getHConnectionFactory().createConnection(this.config);
            GLOBAL_HCONNECTIONS_COUNTER.increment();

@ivanjoy42 73eeb51cf5aab4c0a10454c5365b10dd5653fc97 supports passing the JDBC username and password to the backing database. Please give it a go and let me know how it works for you.


removed yo generator.

It wouldn’t install trailpack - it will generate a new one.

No this one will add the trailpack into the trails project :) to create a trailpack it's simply yo trailpack

Yep. it's early morning =D


Thanks for this - I think HomeState is unnecessary bcoz (HomeState.loggedOut() can fit in to AuthState

Thanks for this Boilerplate ,i am currently using it on app that will soon go to production .
I think HomeState is unnecessary bcoz (HomeState.loggedOut() can fit in to AuthState.
@freezed
class AuthState with _$AuthState {
const factory AuthState.initial() = _Initial;
const factory AuthState.loading() = _Loading;
const factory AuthState.loggedIn() = _LoggedIn;
const factory AuthState.loggedOut() = _LoggedOut;
const factory AuthState.error(AppException error) = _Error;
}
Cheers

in current situation it's unnecessary but when you will add more feature on home then it will come handy.


I can't  use this with cocoapods.

platform :ios, '8.0'
pod 'DrawerController', '~> 1.0.1'
var drawer =  DrawerController(centerViewController: main, leftDrawerViewController: left)
xcode can't recognize  DrawerController Class！

Did you import the generated module in the file where you want to use it like this?
import DrawerController

Cannot load underlying module for 'DrawerController'。。。。QAQ

Do you use CocoaPods 0.36 and did you add use_frameworks! to your Podfile?

e.
pod --version
0.36.3
Podfile:
use_frameworks!
platform :ios, '8.0'
pod 'DrawerController', '~> 1.0.1'

Sorry, I can't reproduce this. Could you send me a sample project?

I haved the same problem:
pod version is 0.37.1
use DrawerController (1.0.4)
when import DrawerController ,xcode show a error:
"cannot load undelying module for ‘DrawerController’"

This is because the project did not clean，select product->Clean，then rebuild the project，solve this problem


Restrict admin access to Pub/Sub push handler.

Thanks, I'll incorporate this into #44

Closing in favor of #44


Fix #55: make I and A work in Visual mode

Here's a potential fix for issue #55.
A more general solution might be possible by changing the <Plug>(multiple-cursors-apply) mappings to expression mappings and having apply_user_input_next() return an empty string - trying to resolve the issue of text being inserted into the buffer when I and A are pressed in Visual mode.
inoremap <expr> <Plug>(multiple-cursors-apply) <SID>apply_user_input_next('i')
nnoremap <expr> <Plug>(multiple-cursors-apply) <SID>apply_user_input_next('n')
xnoremap <expr> <Plug>(multiple-cursors-apply) <SID>apply_user_input_next('v')

@djjcast Thank you for your contribution! Would you mind adding test cases to make sure it fixes the problem?

@balta2ar No problem, tests have been added.

@faceleg @eapache Guys, could you take a look, please? Also, travis build is failing, vim-gtk is missing. According to logs, tests are running on Ubuntu 12.04. I think vim there is pretty old. Shouldn't we use something more recent?

CI fixed in #183 so this should pass once rebased.
Code looks good to me, though I'd like to take a bit of time to play with it in the next couple of days first - I tried a fix like this a while back and couldn't get it to work... I'd like to know what I was doing wrong :)

The latest commit supports patterns without word boundaries and variable-length patterns which should cover all use cases.
Instead of using b to reposition the cursors, the idea is to save the start positions of all of the visual regions and use them to reposition the cursors before entering Insert mode.

Except of the my last comment-related remark, LGTM :+1:. Thank you for you work, @djjcast! Over to you, @eapache.

LGTM. The trick I was missing I think was to do it in two steps by spoofing the v character.


[W5][W16-3]Zhang Jiayu

Update the User Guide; Add a new command findPhoneNumber to find a person's information through input his telephone number.

It's really a good job to find a person by some other details.


Run tests periodically

The test suite runs against httpwg/structured-header-tests, so new tests may cause new failures which wouldn't be detected until new commits are pushed (and they may be unrelated to the pushed changes).  Running the test suite periodically will help quickly detecting issues, and isolate errors in new tests from errors introduced in changes.

Actions' cache archives are immutable, and only expire if not accessed for 7 days, so running tests daily would cause the composer cache to hold onto the results from its first run and be less effective over time as dependencies are updated.
This could possibly be mitigated by adding a date to the cache key (e.g. month or week number) so that caches are forced to rebuild periodically.


Fix mouseEnter/Leave deprecations

Fix: https://github.com/mansona/ember-cli-notifications/issues/247
Make some changes to accommodate the new {{on}} helper in the message component.
Remove component tag, class name bindings and attr bindings and add a wrapper div tag with all the class names, data test attr, mouse enter/leave events.

I'm merging this without Travis passing. It's just ember-beta failing because of emberjs/ember.js#18693

This is released in v6.1.0 👍


[flashv2] NaNs in bw pass for some inputs

Thanks for pushing Flash-Attention v2! The speedups are really huge and this will make so many workloads much faster!
I would like to update to Flashv2 in xformers, however we have a few tests failing due to nans in the BW pass, and I have managed to isolate a minimum repro here. Is it possible to have a look?
Thanks a lot!
Repro code
# Tested with
# flash-attn-2.0.0.post1 (d1a3b52f17b914c93bf740654387b566a7330687)
# torch==2.0.0
# NVIDIA A100-SXM4-80GB
# cuda 11.8
import torch
import flash_attn

torch.manual_seed(352)
q = torch.randn([1, 1, 1, 16], dtype=torch.float16, device="cuda") * 3
k, v = [torch.randn([1, 1, 1, 16], dtype=torch.float16, device="cuda") * 3 for _ in range(2)]
q.requires_grad_(True)
k.requires_grad_(True)
v.requires_grad_(True)

grad = torch.full_like(q, 1.0)

out = flash_attn.flash_attn_func(q, k, v)
out.backward(grad)

print("Q gradient:", "NaNs!" if q.grad.isnan().any().item() else "OK")
print("K gradient:", "NaNs!" if k.grad.isnan().any().item() else "OK")
print("V gradient:", "NaNs!" if v.grad.isnan().any().item() else "OK")
print("Q: ", q.tolist())
print("K: ", k.tolist())
print("V: ", v.tolist())

Output
Q gradient: NaNs!
K gradient: OK
V gradient: OK
Q:  [[[[-2.705078125, 2.4375, 4.84375, 1.771484375, 1.064453125, 6.9296875, -3.859375, -4.0625, 2.37890625, -4.2578125, -1.05078125, 0.8740234375, 0.45166015625, 0.9140625, 9.0546875, -0.66259765625]]]]
K:  [[[[6.296875, 3.10546875, -1.3349609375, -0.03778076171875, 2.873046875, 0.081787109375, 1.984375, -0.85986328125, 0.028167724609375, 6.8984375, 4.2421875, -2.48828125, 0.2978515625, 3.55078125, -2.533203125, -3.24609375]]]]
V:  [[[[3.48046875, -3.859375, 2.8203125, -1.25, -2.10546875, 3.3984375, 1.28515625, -4.29296875, -1.08984375, 1.294921875, 1.2265625, 0.81005859375, -1.076171875, 0.45361328125, 1.958984375, 0.1907958984375]]]]

Thanks for the bug report, I'll take a look today.

Hi, I meet the same problem today. it seems like only q will get NaN gradients

Same here when using flash attnetion2 in training ViT.

Yes this is an issue when the sequence length is not divisible by 128. I figured out the problem and I'll try to push the fix soon (traveling right now).

v2.0.3 should hopefully fix this problem. Can you guys try the latest version (v2.0.3)?

Tried and it works now.

Fixed by https://github.com/Dao-AILab/flash-attention/commit/a4f148b6abe077cd387e69e5caae45d63a76bb24 and can be closed.


(BOLT-1227) Document OpenSSH configuration file options

Add documentation about where OpenSSH config files are read from and what values are respected. Also document dependency on ssh-agent for authentication with password protected private keys.

CLA signed by all contributors.

Ping @melamos


GIP/gip/GeoResource.h:32:28: fatal error: gdal/gdal_priv.h: No such file or directory

I am using GDAL version 1.11.2 and  gdal_priv.h is located at ./gdal1.11.2/gcore/gdal_priv.h.  When I compile gippy, I get this compiler error:
~/dev/gis/gippy$ python setup.py build
running build
running build_py
creating build
creating build/lib.linux-x86_64-2.7
creating build/lib.linux-x86_64-2.7/gippy
copying gippy/version.py -> build/lib.linux-x86_64-2.7/gippy
copying gippy/init.py -> build/lib.linux-x86_64-2.7/gippy
running build_ext
building 'gippy/libgip' extension
creating build/temp.linux-x86_64-2.7
creating build/temp.linux-x86_64-2.7/GIP
x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -IGIP -I/usr/include/python2.7 -c GIP/GeoImage.cpp -o build/temp.linux-x86_64-2.7/GIP/GeoImage.o -std=c++11 -O3 -DBOOST_LOG_DYN_LINK
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++ [enabled by default]
In file included from GIP/gip/GeoImage.h:25:0,
from GIP/GeoImage.cpp:22:
GIP/gip/GeoResource.h:32:28: fatal error: gdal/gdal_priv.h: No such file or directory
#include <gdal/gdal_priv.h>
^
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Hi, this looks like python-gdal is not installed.
If you were installing this as part of lidar2dems using the easy-install script there were some missing dependencies (it's been updated now).
Here's the new list of libs to install via apt-get:
sudo apt-get install git cmake g++ libboost-all-dev libgdal1-dev libeigen3-dev libflann-dev libopenni-dev libvtk5.8-qt4 libqhull-dev qt-sdk libvtk5-qt4-dev libpcap-dev python-vtk libvtk-java python-numpy libgeotiff-dev python-setuptools swig swig2.0 python-gdal python-scipy libxslt1-dev
matt

Hi Matt, I installed GDAL from source since python-gdal under Ubuntu 14.04.2 only has GDAL 1.10.1 and I need 1.11.2.  For some reason /usr/local/include/gdal_priv.h is not being used.  I see python-gdal has gdal_priv.h located at /usr/include/gdal/gdal_priv.h.  So I don't know why it is not being picked up.

The include statement for gdal_priv.h in GIPPY is
#include <gdal/gdal_priv.h>

because the standard header install location (and how I've seen it included in other programs) is under a gdal subdirectory in the include path.   Two simple solutions:
1 - if you manually installed the files, move the gdal headers into a gdal subdir in /usr/local/include (or /usr/include).
2 - make a gdal symbolic link in the include directory that points to the same directory:
 /usr/include $ sudo ln -s ./ gdal

So when it looks for gdal/gdal_priv.h it will find it as the gdal symbolic link points to the right directory.


S40 thermostat doesn't work

Hello,
I just got a s40 thermostat installed and I guess they removed HomeKit support from it that the s30 had for some reason. When I tried this I get "unauthorized" loop. The app for the s40 is using lennoxicomfort.com not myicomfort.com. I am wondering if this is the issue. Please let me know what info you need to hopefully implement the new s40 thermostat. Thank you!

Hi, apologies for the delay in getting back to you.  That is precisely the reason why that is happening.  Like I wrote to other user who left a comment, I don't have those thermostats so it is a little difficult to test.
Additionally, it will be a little trial and error to make sure that the APIs at the different urls are the same and function the same as the ones for the wifi thermostat.

I am happy to help if you want to implement this as I think Lennox has migrated to these thermostats for new installs. Please let me know what I can do to help if you would like. If not I totally understand. Thank you!

I have both an S30 and an S40 and would be happy to help as well because I’d really like the S40 to be in the Home app!

Any update on the S40? I really don't want to use home assistant

Same here!
ᐧ
On Thu, Apr 11, 2024 at 4:01 PM BioTech2000 @.***>
wrote:

Any update on the S40? I really don't want to use home assistant
—
Reply to this email directly, view it on GitHub
https://github.com/akgoode/homebridge-lennox-icomfort/issues/2#issuecomment-2050426296,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD3RCGQSXPTFV75J2C4X6S3Y43TZVAVCNFSM6AAAAAAZBXAKM6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQGQZDMMRZGY
.
You are receiving this because you commented.Message ID:
@.***>

Any update on the S40? I really don't want to use home assistant

Unfortunately it won't be possible for me to make any other kind of thermostats work.  The package that this app depends on doesn't know the format of the API for other thermostats so I can't connect it.  If that existed, I could.  Its also harder to test without those thermostats in my house.


Grep command

While creating containers over LXD, through a script, machine went out of IP addresses. So new containers were getting created but without any assigned IP address.
lxc list | grep RUNNING | wc -l
This works fine.
but
lxc list | grep -c RUNNING
throws an error:
goroutine 815 [IO wait]: net.runtime_pollWait(0x7f5b34d14b58, 0x72, 0xc8203f2000)  /usr/lib/go-1.6/src/runtime/netpoll.go:160 +0x60 net.(*pollDesc).Wait(0xc8203259c0, 0x72, 0x0, 0x0)  /usr/lib/go-1.6/src/net/fd_poll_runtime.go:73 +0x3a net.(*pollDesc).WaitRead(0xc8203259c0, 0x0, 0x0)  /usr/lib/go-1.6/src/net/fd_poll_runtime.go:78 +0x36 net.(*netFD).Read(0xc820325960, 0xc8203f2000, 0x1000, 0x1000, 0x0, 0x7f5b34d0b028, 0xc8200900a0)  /usr/lib/go-1.6/src/net/fd_unix.go:250 +0x23a net.(*conn).Read(0xc8203081b0, 0xc8203f2000, 0x1000, 0x1000, 0x0, 0x0, 0x0)  /usr/lib/go-1.6/src/net/net.go:172 +0xe4 net/http.noteEOFReader.Read(0x7f5b34d15128, 0xc8203081b0, 0xc8203418c8, 0xc8203f2000, 0x1000, 0x1000, 0x444ea3, 0x0, 0x0)  /usr/lib/go-1.6/src/net/http/transport.go:1683 +0x67 net/http.(*noteEOFReader).Read(0xc8201a8800, 0xc8203f2000, 0x1000, 0x1000, 0xc82058dd1d, 0x0, 0x0)  <autogenerated>:284 +0xd0 bufio.(*Reader).fill(0xc8203cd920)  /usr/lib/go-1.6/src/bufio/bufio.go:97 +0x1e9 bufio.(*Reader).Peek(0xc8203cd920, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)  /usr/lib/go-1.6/src/bufio/bufio.go:132 +0xcc net/http.(*persistConn).readLoop(0xc820341860)  /usr/lib/go-1.6/src/net/http/transport.go:1069 +0x177 created by net/http.(*Transport).dialConn  /usr/lib/go-1.6/src/net/http/transport.go:853 +0x10a6

Hmm. Can you paste the full traceback for all goroutines? This seems like a race, I'm not sure that the -c makes any difference.

Its hard to reproduce.

Ok, well if you can reproduce it please post the full backtrace. afcffb844b0f4cbcf558800a636ee17ba51a3772 fixes something like this on master, so it may be fixed if it is the same issue.

oh! the error looks pretty much same. It was due to concurrent reads too.

Ok, I'll close this for now. If you find a different error, feel free to file another bug with the full stack trace. Thanks!


snapTo method only works good if called two times

Hey! I'm currently having some issues with snapTo method.
If I call it to present the bottom sheet like this
// Present bottom sheet on button press
const onButtonPress = () => {
    bottomSheetRef.current.snapTo(1)
}

// Hide bottom sheet on inner button press
const onInnerButtonPress = () => {
    bottomSheetRef.current.snapTo(0)
}

// sheet setup
<View>
    <Button onPress={onButtonPress} />
    <BottomSheet
        snapPoints = {[0, 250]}
        renderContent = {this.renderContent} // inner button lies here
    />
</View>

So this works only if I press both buttons twice or call snapTo twice in each function..
However snapTo works when i hide the sheet with gesture.
Looks like you can't use method multiple times in a row without gesture interaction.
By the way, waiting time between presses doesn't change anything

i am also having this issue
@lukebars Did you able to solve this or any workaround?

@amilaDulanjana Currently I'm resorting to double calling the method.. dirty, but works.

Same here

should be fixed on master

You also I think resolved another issue where "hide" was also opening other bottom sheets.   👏


Compilation error when using CEREAL_REGISTER_TYPE

I have the following code:
derived.h:
#include "base.h"

#include "cereal/archives/json.hpp"
#include "cereal/access.hpp"
#include "cereal/types/polymorphic.hpp"

class derived : base
{
public:

    template <class Archive>
    void serialize(Archive& archive)
    {
        archive(cereal::virtual_base_class<base>(this), CEREAL_NVP(x));
    }

    template <class Archive>
    static void load_and_construct(Archive& archive, cereal::construct<derived>& construct)
    {
        some_other_class* s;
        archive(CEREAL_NVP(s));
        construct(s);
    }

    friend class cereal::access;

private:
    int x;
};

CEREAL_REGISTER_TYPE(derived);
CEREAL_REGISTER_POLYMORPHIC_RELATION(base, derived);

When compiling I get a lot of errors:
shared/include/cereal/types/polymorphic.hpp:86:10: error: 'binding_name' is not a class template
   struct binding_name<__VA_ARGS__>                                       \
          ^
src/hld/system/derived.h:95:1: note: in expansion of macro â€˜CEREAL_REGISTER_TYPEâ€™
 CEREAL_REGISTER_TYPE(derived);
 ^
shared/include/cereal/types/polymorphic.hpp:87:3: error: explicit specialization of non-template cereal::detail::binding_name
   {                                                                      \
   ^
src/hld/system/derived.h:95:1: note: in expansion of macro â€˜CEREAL_REGISTER_TYPEâ€™
 CEREAL_REGISTER_TYPE(derived);
 ^
In file included from shared/include/cereal/types/polymorphic.hpp:39:0,

shared/include/cereal/details/polymorphic_impl.hpp:76:12: error: 'init_binding' is not a class template
     struct init_binding<__VA_ARGS__> {                                   \
            ^
shared/include/cereal/types/polymorphic.hpp:91:3: note: in expansion of macro CEREAL_BIND_TO_ARCHIVES
   CEREAL_BIND_TO_ARCHIVES(__VA_ARGS__)
   ^
src/hld/system/pacific_pvt_handler.h:95:1: note: in expansion of macro CEREAL_REGISTER_TYPE
 CEREAL_REGISTER_TYPE(derived);
 ^
shared/include/cereal/details/polymorphic_impl.hpp:76:38: error: explicit specialization of non-template cereal::detail::init_binding
     struct init_binding<__VA_ARGS__> {                                   \
                                      ^
shared/include/cereal/types/polymorphic.hpp:91:3: note: in expansion of macro CEREAL_BIND_TO_ARCHIVES
   CEREAL_BIND_TO_ARCHIVES(__VA_ARGS__)
   ^
src/hld/system/pacific_pvt_handler.h:95:1: note: in expansion of macro CEREAL_REGISTER_TYPE
 CEREAL_REGISTER_TYPE(derived);
 ^
shared/include/cereal/details/polymorphic_impl.hpp:77:16: error: bind_to_archives does not name a type
         static bind_to_archives<__VA_ARGS__> const & b;                  \
                ^
shared/include/cereal/types/polymorphic.hpp:91:3: note: in expansion of macro CEREAL_BIND_TO_ARCHIVES
   CEREAL_BIND_TO_ARCHIVES(__VA_ARGS__)
   ^

You're not using a virtual base class, try using cereal::base_class instead of cereal::virtual_base_class.


LTA steering feels very nice and has a 95 degree steering limit stock, but I was able to get even higher angles with OP. Torque bottoms out at some point.
You can try it out here:
https://github.com/zorrobyte/openpilot/tree/zp-lta

I'm cleaning up my branch and may attempt a PR at some point. Getting away from torque control and the tuning that comes along with it is incredibly nice. We can also use a steerRateCost of 0.1, which makes OP very responsive. Differences between laneless and laneful are apparent with angle based controls!
Dashcam LTA route:
4aa5d874f367adbc|2022-02-20--22-59-18
“Stock” LTA OP
4aa5d874f367adbc|2022-02-21--18-22-18
"Broken" OP LTA route (would not steer):
4aa5d874f367adbc|2022-02-21--18-37-54
Dashcam with disablements (LTA OEM steer overrides)
4aa5d874f367adbc|2022-02-22--14-47-21
Dadcam mode TSS2.5 stock
f15e3c37c118e841|2022-02-18--13-58-56

This form of documentation is better as a GitHub Discussion, rather than inline comments.


Update how-to-use-schema-compare-to-compare-different-database-defini…

…tions.md
fixing right New Schema Comparison parent menu name

@zhangmx : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Note that publishing in this repository is suspended until Tuesday, May 19. Merging will resume at 7:00 AM PT Tuesday. Authors can still review and sign off on PRs during this time. Thanks.

@zhangmx
approved.
@ktoliver
please merge.
Thank you.


Add option for random delay

Allow requesting a random delay time. Could be useful in testing...

https://github.com/siwalikm/Flash handles this.


Agregar Jugador, Vehiculo, Ciudad, Pozo y primera prueba

Resolves #1

Codecov Report

Merging #3 (78a53df) into master (6d29b09) will decrease coverage by 13.20%.
The diff coverage is 83.72%.

@@              Coverage Diff               @@
##              master       #3       +/-   ##
==============================================
- Coverage     100.00%   86.79%   -13.21%
- Complexity         5       22       +17
==============================================
  Files              2        8        +6
  Lines             10       53       +43
  Branches           0        1        +1
==============================================
+ Hits              10       46       +36
- Misses             0        5        +5
- Partials           0        2        +2

Impacted Files
Coverage Δ

src/main/java/edu/fiuba/algo3/modelo/Auto.java
77.77% <77.77%> (ø)

src/main/java/edu/fiuba/algo3/modelo/Jugador.java
80.95% <80.95%> (ø)

src/main/java/edu/fiuba/algo3/modelo/Ciudad.java
88.88% <88.88%> (ø)

...rc/main/java/edu/fiuba/algo3/modelo/Obstaculo.java
100.00% <100.00%> (ø)

src/main/java/edu/fiuba/algo3/modelo/Pozo.java
100.00% <100.00%> (ø)

src/main/java/edu/fiuba/algo3/modelo/Vehiculo.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Finalmente nos quedamos con #6


The platform in editable.dart should probably be obtained from the BuildContext

The platform in editable.dart should probably be obtained from the BuildContext.
https://github.com/flutter/flutter/blob/b9817b4f595c9ee83bc0625efa60a19599565d63/packages/flutter/lib/src/rendering/editable.dart#L1607
https://github.com/flutter/flutter/blob/b9817b4f595c9ee83bc0625efa60a19599565d63/packages/flutter/lib/src/rendering/editable.dart#L1665

Why do you think so? This code lives in the rendering layer and doesn't really have access to the BuildContext.

I think it needs to accept the user specified platform.
MaterialApp(
  theme: ThemeData.light().copyWith(platform: TargetPlatform.iOS),
);


Fix timeout error when connecting with client to server.

In the old version of the README, there would be a timeout error for no reason when trying to run the example in the README. It turns out that is not an issue in the echo.rs example, so I tested it, and found the fix!

Hey, thanks for the PR
I already added a sleep in the README in b873925ebcf65d5ddb21b3adac9e8bff8de2e83f that should fix this when people copy the code.


Bad link in docs

https://github.com/ongr-io/ongr-sandbox readme.rst link Router bundle, points to ElasticSearch Bundles docs.

These links should be removed. Documentation lists them anyway.


godot_4: 4.0-beta16 -> 4.0-beta17

Description of changes

Changelog: https://godotengine.org/article/dev-snapshot-godot-4-0-beta-17/
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Shortly after this PR was submitted, Godot 4.0 RC1 was released. #215379 is a new PR that updates godot_4 to 4.0 RC 1.


Zoxide incapable on symlinks

-.-
On my system the Windows Temp folders are on a ramdisk (via symbolic links) so Zoxide never manages to install.
Some variant of the issue already has entries in the mod's github, that being said, the point is that every time I start pwsh, the script tries to install Zoxide because it does not detect the command mod, so before I get involved on my own, I came in case anyone here had a similar question (without luck, of course).
I hope that disabling Zoxide patching when starting pwsh will fix my problem (although unfortunately, losing a lot of the packmod's functionality)

Well, the brute force solution worked quickly, at least for now. I leave the images of the 'arrangement' in case anyone else needs it:

(after making a backup copy of the original version) here the 'muted' (#) lines (337 to 352) in the 'main' config script.

after a restart requested by the modpack, powershell acquired the desired behavior without traces of the Zoxide installation attempt, maintaining it until now
I hope it helps if someone even less experienced than me needs it, and of course thank you, Titus, for your always top-notch tools and shared knowledge!!!


Upgrades plugin for IJ 2024.1

upgrades clj-kondo
upgrades gradle IJ plugin

Something went wrong with this release. Somehow the compatibility range got lowered

Something went wrong with this release. Somehow the compatibility range got lowered

Addressed (hopefully) in #58

according to official document，plugins must use IntelliJ Platform Gradle Plugin 2.x from IDEA2024.2，this is migration document they provide.I hope this is helpful

#58

Thanks, @zjjfly. I'll take a look at fixing the gradle plugin shortly.


add links and bio: Ricciardi

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
6fa35f6953a5831d3ce450e09d10b38efbae7b19

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/63dd8b71d53ac80008c7a01c


Add cross-layer energy paper

@mosharaf The arXiv replacement submission was released from hold. I think it'll be out in two days.
The problem is that we actually had a separate entry on our pub list pointing to the arXiv entry. When the arXiv replacement submission gets announced, the two will go out of sync (both abstract and title).
I think we can (1) update the older entry on our homepage pub list with the newer abstract and title, and (2) keep the changes proposed by this PR, creating a new pub list entry containing the final PDF file.
WDYT?

Historically, we never went back to update an arXiv enrty in bib file even though a newer version has replaced it.

In that case, one way is to just implement (2) above. Another way is to withdraw the arXiv replacement submission and implement (2).

The downside of the former is that we need to let Nishil know that we're not changing the arXiv entry

You mean the latter, no? Former seems the best course of action: keep old arXiv entry in our website as is; update arXiv as you've had; create new entry => just merge this PR.

Ah yeah I meant the latter. Going with the former. The PR is ready for your review and once it's merged, there's nothing more to do other than me keeping track of the arXiv replacement submission.


Add social media in the profile.

Add a option in the profile to: Anilist, Twitter, MyAnimeList and etc.

maybe in V3?

maybe in V3?

i'm so lazy to do this

i'm so lazy to do this


add default extensions

rust binary do not have default extensions, extensions need auto install and load, but auto install or manual install both get error:
for manual install:
"Catalog Error: Copy Function with name "parquet" is not in the catalog, but it exists in the parquet extension.\n\nPlease try installing and loading the parquet extension by running:\nINSTALL parquet;\nLOAD parquet;\n\nAlternatively, consider enabling auto-install and auto-load by running:\nSET autoinstall_known_extensions=1;\nSET autoload_known_extensions=1;"
for auto install:
DuckDBFailure(Error { code: Unknown, extended_code: 1 }, Some("Invalid Input Error: Initialization function "parquet_init" from file "/root/.duckdb/extensions/v1.0.0/linux_amd64_gcc4/parquet.duckdb_extension" threw an exception: "Attempted to dereference unique_ptr that is NULL!""
platform
CentOS Linux release 7.9.2009 (Core)
duckdb version
1.0.0
need default extensions
parquet, json, icu

I'm hitting a related issue:
Error: IO Error: Extension "//.duckdb/extensions/v1.1.1/linux_amd64/h3.duckdb_extension" not found.

I'm using the bundled build, and it works on macOS, but not in my Linux docker image 🤔


Extension doesn't appear in the Experimental Plugin List

I'm using Ghidra 10.1.3 and I successfully installed the most recent version of Ghidra-Cpp-Class-Analyzer.
I have done the following:

I checked that it shows up in "Install Extensions..."
I checked that it's in the Extensions folder and has the name "Ghidra-Cpp-Class-Analyzer" (which should avoid something like #44)
The directory that it lists for the extension in "Install Extensions..." is correct
I tried uninstalling and reinstalling
I have quit and reopened Ghidra a number of times
I have made sure to open the plugin configure window from the code editor.

One thing to note is that I did have to add a gradle.properties to fix a problem I was having with Java, and I did have to edit extension.properties to set the version number to 10.1.3 manually, but I don't see how that could cause a problem here.

May you attach the application log file please? You can find it the same way as mentioned in the other linked issue. Just replace the Ghidra version in the path as necessary. It's probably something silly like in the other issue or a duplicate installation of the plugin.
I'm sorry you're having such a hard time with this. It's supposed to be a very simple process. Unfortunately when something minor goes wrong it throws a wrench in things and is difficult to diagnose. The entire extension build and installation process is dictated by Ghidra and I have minimal control over it.

Sorry for the delay. I did notice the following:
2022-05-25 12:43:01 ERROR (ExtensionTableProvider$1) Installation Error: Selected file: [ghidra_10.1.2_PUBLIC_20220311_Ghidra-Cpp-Class-Analyzer] is not a valid Ghidra Extension
2022-05-25 12:43:24 ERROR (ExtensionTableProvider$1) Installation Error: Extension version for [Ghidra-Cpp-Class-Analyzer] is incompatible with Ghidra.
2022-05-25 12:43:36 ERROR (ExtensionTableProvider$1) Installation Error: Extension version for [Ghidra-Cpp-Class-Analyzer] is incompatible with Ghidra.
2022-05-25 13:18:55 ERROR (ExtensionTableProvider$1) Installation Error: Extension version for [Ghidra-Cpp-Class-Analyzer-master] is incompatible with Ghidra.
2022-05-25 13:19:39 ERROR (ExtensionTableProvider$1) Installation Error: Selected file: [Ghidra-Cpp-Class-Analyzer-master] is not a valid Ghidra Extension
2022-05-25 13:21:04 ERROR (ExtensionTableProvider$1) Installation Error: Extension version for [Ghidra-Cpp-Class-Analyzer-master] is incompatible with Ghidra.
2022-05-25 13:36:57 ERROR (ExtensionTableProvider$1) Installation Error: Extension version for [Ghidra-Cpp-Class-Analyzer-master] is incompatible with Ghidra.

Check and see if there is a duplicate installation. A collision of this sort would normally be logged but other than that I don't have any ideas.
Extensions are automatically loaded from the two following paths:
C:\Users\jtint\.ghidra\.ghidra_10.1.3_PUBLIC\Extensions
C:\Users\jtint\Downloads\ghidra_10.1.3_PUBLIC_20220421\ghidra_10.1.3_PUBLIC\Ghidra\Extensions

I'm also having this issue on 10.1.4.
I installed everything (to my knowledge and according to application.log) correctly, but the plugin itself doesn't show up at all in the Experimental Plugin List.

What version of Java are you using? I'm wondering if there is an underlying issue from a newer jdk that I'm not seeing on jdk 11.

What version of Java are you using? I'm wondering if there is an underlying issue from a newer jdk that I'm not seeing on jdk 11.
I also just noticed that I didn't make a release for 10.1.4, which is ironic because I made a comment about exactly this earlier. I will hold off on doing so until I figure out what is going on here anyway.

Unfortunately, I'm also using JDK 11.
As for the 10.1.4 release, no worries! It was easy enough to bump the version myself.

Hmmm. I wasn't able to reproduce it when I built it and installed it with a fresh Ghidra 10.1.4. May you try this and attach the log please? I probably won't get back to this until tomorrow night.
https://github.com/astrelsky/Ghidra-Cpp-Class-Analyzer/issues/44#issuecomment-919581841

Unfortunately, I'm also using JDK 11.
As for the 10.1.4 release, no worries! It was easy enough to bump the version myself.

Hmmm. I wasn't able to reproduce it when I built it and installed it with a fresh Ghidra 10.1.4. May you try this and attach the log please? I probably won't get back to this until tomorrow night.
#44 (comment)

Upon using a fresh Ghidra, rebuilding and installing the extension again, I was unfortunately met with the same result.
The contents of application.log seem unremarkable to me, but, hopefully, there's just something I'm missing.

With a quick look through on my phone I did see it load some of the classes. With the log settings the same may you create or open a project and then the code browser? I don't recall seeing any project name but may have missed it on my phone. If the classes were found and loaded then one of two things are happening. Either an error is occurring when a tool is opened and the plugin package is loaded, or an error is occurring elsewhere and it is caught but not logged. I really hope it is the former or I'm at a loss for what to do.

May you check the lib folder in the zip file of the built plugin for extra jar files? I noticed the other day that some jars for other extensions were being pulled in during the build that shouldn't be. It's due to a bug in my gradle script modifications that set the runtime classpath for debugging. I'm not sure why it is affecting the build. However if there is a duplicate plugin jar then Ghidra might silently blacklist the lib folder to prevent the collision and this would prevent the classes from being loaded.

I'm having the same issue, and I'm trying to build in Releases.


issue #7: implement mapping function + tests

Closes #7

[x] configurable dictionary field mapping
[x] mapping function
[x] update transform function
[x] tests

isn't there a library that does this kind of mapping?

I've refactored the code to use dpath. Please take a look.


Missing info on data frame structure/contents for iPSC analysis

Hello,
Thanks for this great resource!
I've been trying to reproduce your iPSC analysis (hipsci_analysis.py), but I'm having trouble running the code because it isn't clear what the columns are for the input data frames (i.e. ipsc_df). Specifically:

You refer to 'MapCov' and 'TotRead' columns, but TotRead isn't a listed column in any of the PAC output files...so how has this obtained? Are MapCov and TotRead the same?
For ASE selection, the function ASE_selection filters based on a 'p' column in the data frame. Since the binomial test is not run within PAC, was the binomial test run for each row of the PAC txt file (results_2genomes_ID.RSEM.STAR.SOFT.NOTRIM_baq.txt), creating this 'p' column?
Is the Chr_Pos column created after the PAC run?

Thanks again!

Hi jdee3,
Thanks for your interest in the pipeline.  To answer your questions:

PAC will produce 2 types of output file, a haplotype (gene) level one, and a site-level one.  In the gene level output there is a column called 'totalCount' which is the total coverage. In the site level one, there is 'Mapcov' which is the total coverage. The main page has instructions on each column in the two types of output file.
Yes, we ran a binomial test for every row in the output file for this analysis.
We don't create 'Chr_pos' column from the output of PAC, we just generated this in later analyses.

I hope that helps, but lease get in touch if something is still unclear.

Thanks for the prompt reply AJ.
Just to clarify- the code defines the following function:
def columns_to_numeric_and_filter_coverage(df):
    """
    Changes relevant columns to numeric types.
    """
    df['Chr'] = df['Chr'].astype(int)
    df['Pos'] = df['Pos'].astype(int)
    df['MapCov'] = df['MapCov'].astype(float)
    df['TotRead'] = df['TotRead'].astype(int)
    df = df.loc[df.TotRead >= 10]

How does this data frame, df, contain both MapCov (from the site-level output) and total counts (from the gene-level output) when the rows in these text files don't match? How was this df created, is basically what I'm trying to disentangle. Seems to be a fusion of info from the output files? How did you extract total counts for each site, given that you only have total counts per gene? Of course, going the other way is easy, as you can sum the map coverage for all sites within a gene, but how do you divide the gene total counts across all the sites?
Thanks a lot for your help!

Hi jdee,
Sorry for the delay - this script was written by Anna and she has now left the lab.  I believe that the script you are referring to is from Anna's thesis (if it is in the thesis directory), rather than the manuscript. Which analysis are you trying to reproduce?  Is it the GxE analysis of cells treated to different environmental conditions (Findley data)?

Yes, the file 'hipsci_analysis.py' is in the thesis_scripts folder. Wanted to generate the same kinds of plots with the data, but the data frame structure isn't quite clear, so I was trying to figure that out.

Hello,
Apologies for late reply, I haven't kept in touch as well after leaving the lab. I changed the column names and forgot to clarify on the GitHub. All the information needed to reproduce the df in the hipsci_analysis.py is in the PAC output. MapCov refers to Ref allele coverage, TotRead to total number of reads covering the site, Chr refers to chromosome and Pos to position along the chromosome. They do not refer to different output files, but within file (eg. site level will contain all these columns just renamed and some columns that I didn't need were deleted) I renamed for old convention we used for unpublished data as previous PAC version had different output. Apologies for the confusion.


Tuesday 25-07-2023 roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

present

Good morning :)

Present

hi

GOOD MORNING

present

present

Good morning

Morning!

MOrning

good morning

Good Morning

Present

Was there


Cutomize curve without p value

Hi!
I saw a similar question was asked, but I'm trying to generate a custom Curve() with my own parameters and I can't find how to exclude p value since I'm using y^2 = x^3 + 7 (NIST256P) and I don't need the modulus. I looked through the official documentation but didn't see anything that answered this question. My only thought was to make p a huge value, but I'm sure there's a better way.

What are you trying to do? If you’re working with numbers that are comparable in size to what one normally would use in a cryptographic context your x and y values are going to get REALLY big very quickly given what you propose. Meaning so big you can do efficient computations with them.

I want to create my own curve to test against data I'm given for a CTF challenge. I'm given the curve, the generator points x & y, the group order, another point x & y, the public key, & two messages with their sigantures one which does not have the first part of the signature. I want to create a NIST256P curve using the init function to test outputs but I don't want to use p in the function. I'm still learning ecc so I apologize if my explanation isn't very clear.

Yes, the mod p is sometimes left out of the equation, many places just assume all operations are done in the field with characteristic p, which can lead to confusion. Glad you were able to get clarification on your initial issue.


Test changes to netlify CI

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for modest-edison-79d82e ready!
Built with commit a533846dece2ec60ef61745c424bdf9f47b4338b
https://deploy-preview-7043--modest-edison-79d82e.netlify.com


App crash on remove node from tree after loading ad

Godot version
4.1.2
Plugin version
v3.0.0
Phone information
OPPO A78 with Android 13
Issue description
App crash
Steps to reproduce

... do like the doc and try add ad format Rewarded
in runtime after the app call this RewardedAdLoader.new().load(unit_id, AdRequest.new(), rewarded_ad_load_callback)
immediately remove the node from tree before any of rewarded_ad_load_callback callback was called
app crash

Additional context
note:
i did use this on the node
func _exit_tree():
	if _rewarded_ad:
		_rewarded_ad.destroy()
		_rewarded_ad = null

but app still crash
I thing it happen because the plugin was trying to call the callback after finishing loading ads but the callback was not there because the node already removed from tree.
My workaround was like this
class_name RewardedAdLoader
#...
func load(
	owner_node : Node, # add this
	ad_unit_id : String,
	ad_request : AdRequest,
	rewarded_ad_load_callback := RewardedAdLoadCallback.new()) -> void:

	if _plugin:
		_plugin.load(ad_unit_id, ad_request.convert_to_dictionary(), ad_request.keywords, _uid)
		_plugin.connect("on_rewarded_ad_loaded", func(uid : int):
			#if uid == _uid:
			if uid == _uid and owner_node: # check if owner_node still exist
				rewarded_ad_load_callback.on_ad_loaded.call(RewardedAd.new(uid))
		)
		_plugin.connect("on_rewarded_ad_failed_to_load", func(uid : int, load_ad_error_dictionary : Dictionary):
			#if uid == _uid:
			if uid == _uid and owner_node:
				rewarded_ad_load_callback.on_ad_failed_to_load.call(LoadAdError.create(load_ad_error_dictionary))
		)

then in the node call this
func _on_load_pressed():
    #...
    #RewardedAdLoader.new().load(unit_id, AdRequest.new(), rewarded_ad_load_callback)
    RewardedAdLoader.new().load(self, unit_id, AdRequest.new(), rewarded_ad_load_callback)

The the issue was resolved.
I did not test this on ios or other ad format but i think the issue was there too

Hi, thank you very much for reporting this bug, it seems like a critical bug and I'm looking for it!

without breaking compatibility 🤔
class_name RewardedAdLoader
#...
func load(
#...
    _plugin.connect("on_rewarded_ad_loaded", func(uid : int):
        #if uid == _uid:
        if uid == _uid and rewarded_ad_load_callback.on_ad_loaded.is_valid():
            rewarded_ad_load_callback.on_ad_loaded.call(RewardedAd.new(uid))
        )


:sparkles: Add sensor interfaces

Bump version to 0.3.3
Add the following units: - [angle] degrees - [magnetic flux] gauss

Resolves #17
Resolves #18
Resolves #19
Resolves #137
Resolves #172
Resolves #202

Codacy is complaining that the fields within the struct are never used, which is true but not a problem for an interface.


Invalid State on Ubuntu 14.04 Server

Ubuntu Version:
Linux ubuntu-14 3.13.0-24-generic #46-Ubuntu SMP Thu Apr 10 19:11:08 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
Lots of layers, but this is on a windows machine with an Ubuntu VM. I tried the Windows install but there was some strangeness when it came to the formatting of things in some of the pages, so I was trying the ubuntu install method, but my issue may be I'm using ubuntu server.
The error I see is this:
Enter memory (MB) to be allocated to Panamax VM(1536):
Enter CPUs to be allocated to Panamax VM(2):
Enter version you want to use(dev/stable, defaults to:stable)
Would you like Panamax to create the alias (panamax.local)? (You will be prompted for sudo creds) [y/N]:y

Setting local alias to panamax.local
/home/vagrant/.panamax/desktop: line 103: bc: command not found
**Same thing about 100 times**

Reinstalling Panamax...
Would you like to re-download coreos base box[y/N]: n
UUID changed to: 1d39386f-7c2a-4994-b1b4-c11b117a785d
0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%

Creating a new CoreOS VM...
UUID changed to: a6939527-4751-41d0-bded-b5764cd0ebc2
Bringing machine 'panamax-vm' up with 'virtualbox' provider...
==> panamax-vm: Importing base box 'panamax-coreos-box-522.6.0'...
==> panamax-vm: Matching MAC address for NAT networking...
==> panamax-vm: Setting the name of the VM: panamax-vm
==> panamax-vm: Clearing any previously set network interfaces...
==> panamax-vm: Preparing network interfaces based on configuration...
    panamax-vm: Adapter 1: nat
    panamax-vm: Adapter 2: hostonly
==> panamax-vm: Forwarding ports...
    panamax-vm: 22 => 2222 (adapter 1)
==> panamax-vm: Running 'pre-boot' VM customizations...
==> panamax-vm: Booting VM...
==> panamax-vm: Waiting for machine to boot. This may take a few minutes...
The guest machine entered an invalid state while waiting for it
to boot. Valid states are 'starting, running'. The machine is in the
'poweroff' state. Please verify everything is configured
properly and try again.

If the provider you're using has a GUI that comes with it,
it is often helpful to open that and watch the machine, since the
GUI often has more helpful error messages than Vagrant can retrieve.
For example, if you're using VirtualBox, run `vagrant up` while the
VirtualBox GUI is open.
VM Creation failed. Exiting.

I installed bc in ubuntu. Then re-installed, and ended up with the same issue at the end. I tried starting the VMs manually and got this:
vagrant@ubuntu-14:~$ VBoxManage list vms
"coreos_production_vagrant-522-6-0_1425918030074_20201" {bed27967-6cc5-4387-a5e8-342b9dacc195}
"panamax-vm" {60e08b91-d8c8-4875-b74c-7a4dba9a1282}
vagrant@ubuntu-14:~$ VBoxManage startvm panamax-vm
Waiting for VM "panamax-vm" to power on...
VBoxManage: error: The virtual machine 'panamax-vm' has terminated unexpectedly during startup with exit code 1

VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component Machine, interface IMachine
vagrant@ubuntu-14:~$ VBoxManage startvm coreos_production_vagrant-522-6-0_1425918030074_20201
Waiting for VM "coreos_production_vagrant-522-6-0_1425918030074_20201" to power on...
VBoxManage: error: The virtual machine 'coreos_production_vagrant-522-6-0_1425918030074_20201' has terminated unexpectedly during startup with exit code 1
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component Machine, interface IMachine

@rothnic
Thats quite a try :) ... yeah..thats way too many indirections happening in there...  Windows -> ubuntu -> coreos -> docker... . We tried a setup like this initially, the only one thing we noticed was,

We had to enable hardware assisted virtualization flag on the base VM (inside virtualbox).
We had to use ubuntu desktop.

So when you mention strangeness in formatting of pages, what were you seeing?
Also, another route you can possibly take is create a coreos vm and then install panamax into it, you'll need to follow cloud provider instructions for that.

I have the same issue on a virgin Ubuntu 14.x server using Vbox 4.3.28 and vagrant 1.7.2 - spent about an hour chasing it down, then gave up.  It seems to be a problem with vagrant communicating to virtualbox about hostonly network adapter setup.
Here's a copy of my error if it helps:
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["hostonlyif", "create"]

Stderr: 0%...
Progress state: NS_ERROR_FAILURE
VBoxManage: error: Failed to create the host-only adapter
VBoxManage: error: VBoxNetAdpCtl: Error while adding new interface: VBoxNetAdpCtl: ioctl failed for /dev/vboxnetctl: Inappropriate ioctl for devic
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component HostNetworkInterface, interface IHostNetworkInterface
VBoxManage: error: Context: "int handleCreate(HandlerArg*, int, int*)" at line 66 of file VBoxManageHostonly.cpp

Happens when you call panamax init or panamax reinstall or vagrant up [id] or try to start the vbox image.
This is on a new install of Ubuntu Server 14.04.2 LTS (up to date as of this post) on a bare metal machine (not a cloud VM).  Note that using 'Ubuntu Desktop' is not a solution as this machine has no video card at all (well, I suppose I could export an X session, but this is supposed to be server software....).  Also, not sure how to set hardware assisted virtualization in virtualbox from the command line.

i have the same issue with ubuntu 14.04 server. i installed ubuntu desktop packages but didnt help.
bosh-lite@bosh-lite:~/workspace/bosh-lite$ vagrant --version
Vagrant 1.7.2
bosh-lite@bosh-lite:~/workspace/bosh-lite$ VBoxManage --version
4.3.28r100309
I doubt it a vagrant or Vbox issue since i was able to bring up hashicorp/precise32 successfully.

seems like need to have VT-x enabled. I tried in a server where vt-x is enabled. it worked.

Closing as the issue is resolved.


DO NOT MERGE! Fast Forward stable to 270cd0eae17224e3dd40ea851471d43814792b3e.

This PR will be merged automatically once checks are successful.

Jenkins job manager/postsubmit passed


FR: analytics to accept param type of array of strings

It would be great if the Analytics SDK supported sending an array of strings as a parameter in the method logEvent. If I send  { teachers :["Teacher Name 1", "Teacher Name 2"] }, I see an error in the console log "Invalid extra param structure. Must be a dictionary. Parameters: Teacher Name 1, Teacher Name 2" and the parameter is not logged in the DebugView.

Firebase SDK version: 5.5.0

Unfortunately we do not support this currently. It may not be exactly what you want, but as an alternative I would suggest turning the array into a comma separated string and logging that as the parameter.

Going to close since there is an alternative and no plans to add the feature.


ReferenceError: NEXT_DATA is not defined

Bug report
Describe the bug
I don't know what this error is. Happens on app start and on every reload. Should I define that var somewhere or am I missing something? I didn't find anything about it on your site.
System information

OS: all
Browser (if applies): all
Version of Next.js: 4.2.1

But can you please point me somewhere where I can read what it is and what am I missing to declare?

It's defined as part of the Next.js codebase, you don't have to define it, but I'd be happy to look at a reproduction of the issue 👍

I've got some router passed to props in my index.js page file to redirect from index page to another which probable caused Next not properly define that variable. I ended up using next/router and using this method https://github.com/zeit/next.js/wiki/Redirecting-in-`getInitialProps` and it's no longer throwing that exception!


Got the button in the center and did some styling

So what I did is I first got the button and the generated password at the center. Then I added the "Times New Roman" font to the button so as to make it attractive by importing  tkinter.font package as Font.

Nice commit @Anoop01234 !


PR: Remove regex escape characters from replacement string on findreplace widget

Fixes #4934
Preview

Is it possible to add a test for this?

I guess that it is possible to add a test

Please add a test then :-)

Hello @andfoy! Thanks for updating the PR.

In the file spyder/widgets/tests/test_editor.py, following are the PEP8 issues :

Line 290:1: E302 expected 2 blank lines, found 1


Add DABcoin

Website: https://dabco.in

Support email please

Hi,
sorry<EMAIL_ADDRESS>Best,
Bostjan

Our support email is<EMAIL_ADDRESS>


Add a Certifications section

Description:
Adds a certifications section to the profile page to display industry certifications.
Issue: https://github.com/arifszn/gitprofile/issues/233

Thanks for the PR. Will look into it soon.

Bumping this. Should I add anything else or is this good?

Sorry, I was on vacation. Will review it tonight. :raised_hands:


Vertical graph

Is it possible to make to graph go vertical in stead of horizontal? If not integrated, where do I have to look to make it work myself?

What about putting it in an svg transform for rotation? Or would you like the labels to be unrotated too? (You may try to fiddle with css to do so, but it's hard to guarentee the labels won't overlap).

Yes that's the best solution for now I think, thanks!


Fix more warnings

This fixes a couple of more warnings that cropped up in Xcode 9.1, particularly in regards to the deprecation of String.characters.

Thanks!


M> Translate style_guide_assets/README.md to Greek

Completed: 100%
Tasks:

[x] Create the file README_GR.md
[x] Translate the content
[x] Check the spelling
[x] Review each other's work
[x] Have it reviewed by others with knowledge of English and Greek

Collaborators Work:

@ICA3DaR5: 100%
@zero-andreou: 0%
@thiefinshadows: 0%

Suggested Budget: $5

This has been completed as well.

@ICA3DaR5 I don't know if you observed this issue  #288 is duplicated? Or should there be a reason for that which we don't understand?

@ICA3DaR5 We don't know yet if everything in style_guide_assets will go into production. Before doing any more translating let's wait on movement.

@kitblake Alright. No problem :)

@BelovedAquila Sure, you are right. I just thought it would be better to create an issue on a per file basis.


Fix pathname issue and add JSONP support

What's this PR do?
Resolves two issues:

Pathname issue

when the widget.js file is called from a third-party server, calls to legislators.php and bills.php break - their pathnames (./app) load on the calling server (pathnames are relative to the URL in the browser bar)

Cross-domain REST issues

Since legislators.php and bills.php are hosted on an external server, regular REST calls fail due to a cross-domain security issue.
Changed calls in app.js to use JSONP, and added JSONP support to the JSON output in legislators.php and billing.php

Where should the reviewer start?
here is how the changes work, in sequence

widget.js - before initializing appWidget, add the assets variable (containing the pathname) into the appWidget object
app.js - when legislators.php or bills.php is called, is now using JSONP
legislators.php / bills.php - at the end of each, JSONP support has been added

How should this be manually tested?
call this script from an outside server to verify everything works
also call from the same server to verify everything works locally as well (ie nothing that worked before has broken)
Any background context you want to provide?
Hopefully this is all self-explanatory. I work with a lot of REST-based Javascript apps at my regular job, but don't have a lot of experience in git just yet. So apologies if anything is 'off'.
What are the relevant github issue?
submission button broken with embedded script on other sites
https://github.com/campaignzero/campaign-zero-widget/issues/5
Screenshots (if appropriate)
there are some screenshots in the original issue report (from another user).
What gif best describes this PR or how it makes you feel?

Definition of Done:

[DONE ] You have actually run this locally and can verify it works
[DONE ] You have added code comments to all code being submitted
[N/A] You have updated the README file (if appropriate)

This has been resolved in 1.0.13


Request for Pre-extracted EGTEA Features and Additional Dataset Configuration Files

Hi Sakib Reza,
First of all, I would like to extend my sincere thanks for your outstanding work on this project. The tools and resources you've provided have been incredibly valuable, and your contributions to the community are greatly appreciated.
I wanted to ask if it would be possible for you to open-source the pre-extracted features of the EGTEA dataset? Having access to these would be extremely helpful for those of us looking to dive deeper into your work without the initial overhead of feature extraction.
Additionally, I was wondering if you have any plans to open-source configuration files or settings for other datasets? This would be immensely beneficial in helping others replicate your results and adapt your methods to different datasets.
Thank you again for your fantastic work and for considering this request. I look forward to your response.
Best regards,
Matt

Hi @maojiaqi2324, Thanks for your kind words! I've added the links to the pre-extracted features of EGTEA and config files of the other datasets in the README file. Please let me know if you need any further support. If you find this repository and my response helpful, please feel free to star ⭐ the repo. Thanks!


Reform update strategy types

It's convenient to be able to leave out the update strategy, since
there is only one possible value at present; and if there were
alternatives, the present choice would still be a reasonable
default. However, with the format as it is, this doesn't work with
OpenAPIv3 schema, so you have to supply a value, even though there are
no parameters:
spec:
  update:
    setters: {}

A more self-explanatory format which does work with defaulting is to
name the strategy rather than relying on the presence of a field:
spec:
  update:
    strategy: Setters

The whole update field can be elided and left to default. This
doesn't preclude having other strategies later, even those with
parameters, e.g.,
spec:
  update:
    strategy: Foo
    fooParam: 5

This PR changes the API types and code that uses them, and the CRD
manifest, and adds a test that checks the defaulting actually works.
Fixes #66.

Also needs:

[ ] doc updates

Also needs:

[ ] doc updates

If setters was optional there is no need for a conversion webhook, Kubernetes API will drop that field since it's no longer in the CRD spec.

If setters was optional there is no need for a conversion webhook, Kubernetes API will drop that field since it's no longer in the CRD spec.

If setters was optional there is no need for a conversion webhook, Kubernetes API will drop that field since it's no longer in the CRD spec.

That's handy! People will still (effectively) have an update: {} field; I will test how that affects things when updating the CRDs and controller.

If setters was optional there is no need for a conversion webhook, Kubernetes API will drop that field since it's no longer in the CRD spec.

That's handy! People will still (effectively) have an update: {} field; I will test how that affects things when updating the CRDs and controller.

OK, the controller will barf on existing objects once it's updated. To avoid that, I've put a default on the .spec.update.strategy field as well. Existing objects will look like .spec.update: {} (setters field now ignored), which won't default according to the rule for .spec.update, but will default according to the rule for .spec.update.strategy, giving the same result.

OK, the controller will barf on existing objects once it's updated. To avoid that, I've put a default on the .spec.update.strategy field as well. Existing objects will look like .spec.update: {} (setters field now ignored), which won't default according to the rule for .spec.update, but will default according to the rule for .spec.update.strategy, giving the same result.


Add Terralith

Terralith seems to work with the Continents datapack and because it uses Terrablender it is highly compatible with both Regions Unexplored and Promenade #7. I have all 3 loaded atm in a world and it's running great. Structures + biomes from each mod generate fine as well.

Might be a bit much tbf thinking about it. Explored the world and it generates fine but it's a bit derpy in places. Either Terralith or #7 is better


deps: Bump unplugin to 1.5.0

Fixes https://github.com/getsentry/sentry-javascript-bundler-plugins/issues/435

Totally forgot that it is a dep in all packages.

I cannot be bothered to fix this. I am gonna remove unplugin as a dependency.


Compiling Error missing Files

Hi,
I get multiple compiling errors about missing files like this one:
/src/dmx/types.h:10:10: fatal error: esp_intr_alloc.h: No such file or directory
   10 | #include "esp_intr_alloc.h"
      |          ^~~~~~~~~~~~~~~~~~
compilation terminated.

I tried to fix these by manually finding the missing files in other repositories, but as one after another missing file error pops up, at some point I gave up. The board I am trying to compile for is an ESP 8266 D1.
I reinstalled your project and various versions of it, as well as the Arduino-ESP32 Framework with no success.
Do you have any idea what I'm doing wrong?

Thanks for reaching out. Unfortunately, this library is not compatible with the ESP8266. It is only compatible with the ESP32 and the 32 family of Espressif microcontrollers. This is because the ESP8266 does not use Arduino-ESP32, but instead uses Arduino-ESP8266.
Sorry for the confusion, and hopefully this helps! Let me know if you have any other questions.

Thanks for your reply! I should have payed more attention reading the closed issues. 😅
I will have a look at DMXSerial.


assignment 1 added

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
added assignment 1
What did you learn from the changes you have made?
Data structures, hashing
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Yes, there were greedy method to solve the same thing.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
By running the code
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

Hi Chetan,
Your submission has NOT been approved. I previously requested changes because the output of your function doesn't match the expected output you described in your example 2. Please re-open your PR and provide a short explanation. Once your submission has been approved, you can merge into your main branch.
Thanks,
Tianyi [LS]


BUNDLE: reduce bundle size of quadstore-comunica by working on immutable.js and decimal.js

There are two potential avenues that might lead to shaving significantly more bytes out of quadstore-comunica's bundle:

using something other than Immutable.js and/or enabling tree-shaking so that only Immutable's Map ends up in the bundle

PR for shared implementation of bindings factory (OPEN)

using one of the alternatives to decimal.js

PR for swapping decimal.js with bignumber.js (MERGED!)

Both of these require upstream work.
Number 1. in particular is a bit more complex as the Bindings type and factory function are declared separately in both comunica/comunica and comunica/sparqlee.

Potential alternatives to Immutable.js' Map:

https://www.npmjs.com/package/hamt
https://www.npmjs.com/package/@typed/hashmap


Start filling out the whitelist plugin

@briandealwis @akerekes

After resolving my license feature comment, LGTM

Since you merged in the license branch here, close #53?


[Feature Request] Add support for disabling signature validation

jsonwebtoken exposes the insecure_disable_signature_validation method, that allows for decoding JWTs without validating their signature. This is useful is some use-cases where the service can assume that the token was already validated (in our case, it is done by the service mesh) and only needs to access the encoded claims.
However, from what I can tell from the code here, the Validation is never exposed outside the layer and there is no way to configure it.

yes, I'd like to expose some validation configuration, I hope I will be able to make a pull request in a few days or so.

This is great, thank you @cduvray!


Format type from % to f-string

lol, just some typing change

actully. this PR was not finish

Thank you for working on this. Please comment again when you're ready, and I'll give it another look.

Thank you for working on this. Please comment again when you're ready, and I'll give it another look.

Thanks♪(･ω･)ﾉ

thanks @Zhuruoling for helping me

Hi,
Is there still more work you want to do, or do you think it's ready to merge in?

Hi,
Is there still more work you want to do, or do you think it's ready to merge in?

sorry, I just saw it
I will commit some today, If it passes the test, merge it please


ifconfig resource does not work with bootproto DHCP (or does not work at all)

Configuration: CentOS 6, chef-client 12.4.1
It seems to not be possible to use the ifconfig resource to generate a DHCP network configuration. The documentation is also contradictory - among the examples, there are two that show identical resource configurations, yet claim two different generated configuration files.
It also generates ifcfg files with an incorrect name.
The specific issue I found:
Using the following resource:
ifconfig "eth1" do
device "eth1"
onboot "yes"
bootproto "dhcp"
end
Expected result (on CentOS 6):
It should generate a file /etc/sysconfig/network-scripts/ifcfg-eth1 with (at least) the following content:
DEVICE="eth1"
BOOTPROTO=dhcp
NM_CONTROLLED=no
ONBOOT=yes
TYPE=Ethernet
IPV4_FAILURE_FATAL=yes
Preferably, the hardware address and, for CentOS 7, also a UUID should be included.
Actual result:
The file name misses the device name suffix (i.e., it is called ifcfg- instead of ifcfg-eth1) and contains an IP address.
ONBOOT=yes
BOOTPROTO=dhcp
IPADDR=eth1
It appears that there are architectural issues with this resource; a non-breaking fix may be difficult. My recommendation is to deprecate this resource (and provide either a new resource, or a cookbook)
Architectural issues:

The resource seems to assume that the name is an IP address. It should be a device name (or an arbitrary resource name) instead.
The relationship between specifying an IP address in the resource name, and through the inet_addr attribute is not clear.
The relationship between device and target is not clear.
Interfaces can have either multiple IP addresses, or may not have any IP address at all (such as with DHCP).
Many essential configuration options are apparently not supported. For instance, you cannot specify a default gateway or DNS Servers.
No support for IPv6.
No support for bridging, vlans etc.

As a workaround, it appears that there is a third-party cookbook on the supermarket: network_interfaces_v2 . I have not yet tested it.

The ifconfig provider is somewhat unloved.


Error 133 error LNK1181: cannot open input file 'dxguid.lib' FreeRDP\server\Windows\LINK wfreerdp-server

Hi all, I try to compile using MS Visual C++ Express 2010, but I am receiving this error:
Error 133 error LNK1181: cannot open input file 'dxguid.lib' FreeRDP\server\Windows\LINK wfreerdp-server
Anyone can help-me, please?
Other question:
If I compile under Windows 7 x64, may can I use the dlls under Windows 10 x64 or I need to compile a Windows 10 x64 version and also, if I use under Windows 10 x86, can I use tha same dlls or I need to compile each specific versions?
Thanks in advance.
Best regards.

Support questions please via irc or the mailing list.
Thx


Disable auto-adding brackets/parentheses on completion

I am using Spyder 4.2.0 in Python 3.7.8 on Windows installed via pip without Kite and such the default completion configuration (upgraded to most recent versions) with the python language server (0.36.1) and Rope (0.18.0) enabled (options "Enable code completion", "Show completion details", "Show completions on the fly", "Enable code snippets" enabled in the Completion and linting section in the Preferences.
When I choose any suggestion from the auto-completion list, the editor always adds brackets and the required parameters after the completed term independent on in if it is a variable, a function or a class. This is annoying since I kind of always delete the brackets and parameters afterwards due to various reasons (brackets after variables are not allowed, for function calls and class constructor calls I like to type the opening bracket myself and then add the various arguments myself following the displayed calltips from the Introspection feature and for type hints (": ") brackets are not allowed, too).
Can you please either disable auto-adding the brackets and required parameters for term completion or add an option to disable/enable it like it was before in Spyder 3.
This ticket seems to be related to what we had in issue #2602, but this was for the old version and it seems the code has changed a lot since then.

Hi @TBSenseo I think to prevent that you will need to uncheck the Enable code snippets option. Just in case pinging @andfoy and @ccordoba12 (maybe they have more info about the current config possibilities).

Thanks @dalthviz. This seems to have solved my problem. I understand "snippets" as being prepared code blocks that can be inserted quickly and then filled with in-situ variable names, etc, like the insertion of if   else  in Visual Studio. Maybe rename the configuration item to "Append function prototype parameters to completed terms".
What is your thought of my suggestion?

Hi guys! Happy to join the discussion!
I stepped into the same issue here (spyder 4.1.5) and disabling code snippets is a good workaround for parenthesis added after classes and variables.
However, it spoils the autocompletion of methods and functions.
Is there a way to achieve one (not having parenthesis added after classes) without loosing the other?
Thanks!

Hey @la-foglia, yes, there is a way to do that (and we consider the current behavior a bug). We'll try to do it in a future release.


WIP: Use upstream DOMAIN_INTERFACE_ADDRESSES_SRC_AGENT and libvirt events

What does this PR:
In past we used  domain.QemuAgentCommand for getting the Interfaces via qemu-agent.

Now we are using domain.ListAllInterfaceAddresses(libvirt.DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE

additionally as you can see, we get more simpler codebase with the new function from upstream.

for domains which are down we just skip. This was also the behaviour from libvirt-api

We still need a loop function with timeouts because otherwise we might endup in a situation where the QEMU-AGENT is installed but not already setup. So we need to wait and retry for a delay of time.
If we don't have the timeout with the loop we will get at 1st try error: Guest agent is not responding: QEMU guest agent is not connected

@dmacvicar i was thinking to split the biggeest function domainGetIfacesInfo in 2 Parts.
domainGetIfacesInfoViaQemuAgent and domainGetIfacesInfoViaLibvirtApi and in future domainGetIfacesInfoViaArpCache instead of having 1 big function

i set it again to WIP because i see a strange behaviour in sumaform with the new patch.
I can get the IP adresses from the qemu-domain with the patch but then the provisioner terrafrom exec ssh fail to login.
I can login via ssh with the IP retrieved. I think this patch is ok but i might check in the sumaform side if we have some trivial things to be fixed..

@dmacvicar  actually i experienced something interesting.
I needed to pass  the host var when using terraform connection.
https://github.com/moio/sumaform/pull/435/files
So having this works in sumaform with this patch.
 connection {
    user = "root"
    host = "${libvirt_domain.domain.network_interface.0.addresses.0}"
    password = "linux"
    type = "ssh"
  }

Without this patch we don't need to set the host (which is used for connection for ssh).
If you have any idea why, feel free to give some hints :thinking:

I don't want to rush with this PR, as I was not a fan of the previous timeout loop either.
I see that using SRC_AGENT without the agent connected fails immediately if no timeout is set, so that is good. It won't get stuck.
virsh # domifaddr 1
 Name       MAC address          Protocol     Address
-------------------------------------------------------------------------------
 vnet0      7e:ba:be:47:7d:d8    ipv4         <IP_ADDRESS>/24

virsh # domifaddr --source agent 1
error: Failed to query for interfaces addresses
error: Guest agent is not responding: QEMU guest agent is not connected

As using the agent at all is not a property of the domain, may be we could do the following:

If using the agent, at creation register virConnectDomainEventRegisterAny with VIR_DOMAIN_EVENT_ID_AGENT_LIFECYCLE and create a simpl for VIR_CONNECT_DOMAIN_EVENT_AGENT_LIFECYCLE_STATE_CONNECTED event, before issuing the command

Testing this with virsh produces the event as I expect:
virsh # event --domain 1 --all

(starts qemu-guest-agent on the machine)... and I get:
event 'agent-lifecycle' for domain doman_ubuntu_resized: state: 'connected' reason: 'channel event'
events received: 1

Would this be a better approach than timeouts?

Sounds intersting i will have a look.

@dmacvicar i am not finished yet with the events code, but we need either tk wait forever for thea event with (EventRunDefaultImpl) or we add possible timeout for listening. Other

@dmacvicar so i played a bit with libvirt events ( was funny) :smile: . I have added some usefull examples in my last comment (https://github.com/dmacvicar/terraform-provider-libvirt/pull/432#issuecomment-427175738).
Here my facit:
i think for our current usecase libvirt event is overkill and will add more complexity. ( event handling etc).
Furthermore and i think the most important thing is :
VIR_CONNECT_DOMAIN_EVENT_AGENT_LIFECYCLE_STATE_CONNECTED this event is sent** only 1 time**( if the domain has started). But what if the domain already exist?
In our use cases in terraform, we need to consider the situation where a domain already exists and we do for example apply or refresh or plan, so updates operation. In this case we endup in weird situation because the event was already sent and we cannot really count on it.
Imho for solving this problem, the best and simple solution is to have the approach i did in https://github.com/dmacvicar/terraform-provider-libvirt/pull/432/commits/ec86c07288a78f2d4fc0a97f3e6f2d6ad6b74813
but without a refresh function from terraform. Additionaly to that patch, we don't check the QEMU_AGENT erorrs. ( otherwise we fail), and we use the loop timeout already present from wait_for_lease option.
( in this way we avoid to have 2 timeouts). In this way, we return empty interfaces in errors, or we use already the loop wait_for_lease =true. ( this mean that also for qemu-agent we need this var).
I think in that case is not worth to check error handling, we just need to return empty interfaces.
Anyway with my current poc i tested already the limits of this events.
018-10-06T16:13:46.769+0200 [DEBUG] plugin.terraform-provider-libvirt: 2018/10/06 16:13:46 TeneroTango: INSIDE EVENT!
2018-10-06T16:13:46.795+0200 [DEBUG] plugin.terraform-provider-libvirt: 2018/10/06 16:13:46 TeneroTango: INSIDE EVENT!
2018-10-06T16:13:51.769+0200 [DEBUG] plugin.terraform-provider-libvirt: 2018/10/06 16:13:51 ERROR: unable to get ifaces via qemu-agent virError(Code=86, Domain=10, Message='Guest agent is not responding: Guest agent not available for now')
2018-10-06T16:13:51.795+0200 [DEBUG] plugin.terraform-provider-libvirt: 2018/10/06 16:13:51 ERROR: unable to get ifaces via qemu-agent virError(Code=86, Domain=10, Message='Guest agent is not responding: Guest agent not available for now')

I got the error with this code
	gaCallback := func(c *libvirt.Connect, d *libvirt.Domain, eva *libvirt.DomainEventAgentLifecycle) {
		log.Printf("TeneroTango: INSIDE EVENT!")
		allInterfaces, err = domain.ListAllInterfaceAddresses(libvirt.DOMAIN_INTERFACE_ADDRESSES_SRC_AGENT)
		if err != nil {
			log.Printf("ERROR: unable to get ifaces via qemu-agent %s", err)
			...
	}

	gaCallbackID, err := virConn.DomainEventAgentLifecycleRegister(&domain, gaCallback)

imho we should use events for the autoyast feature but not for this one, because we don't gain much.
THe main problem are the hidden one, like when we shutdown/reboot or change the status of domain this will trigger events, and this we have complexity +  we need to consider the cases of broken connection/etc.

feel free to play expiement with this POC https://github.com/dmacvicar/terraform-provider-libvirt/pull/432/commits/bc95e0ebd20078de0f41d85ce6f96b47d025cc75.
It shows more or less the limit and complexity we can face with it.
If you  have a better approach feel free to share :cupid:  enjoy the weekend

@dmacvicar what do you think?

@MalloZup can you please post an example of the configuration that would fail? (no need for it to be real, just an example, to understand it better)

I will merge this in the middle of next week, all tests were fine I might check some things in case

@dmacvicar  the example i tested was when using sumaform. https://github.com/moio/sumaform
sumaform has always the qemu_agent var set to true ( https://github.com/moio/sumaform/blob/0d7fd02f9789104104e54fa8d42561a8ea213148/modules/libvirt/host/main.tf#L24)
Imho this variable should be set to true when we have a bridge as networking.
Anyway the example was this one:

provider "libvirt" {
  uri = "qemu:///system"
}

module "base" {
  source = "./modules/libvirt/base"
  // fake
  cc_username = "Pino"
  cc_password =   "Daniele"
  pool = "default"
  network_name  = "default"
  name_prefix = "dam-"
  testsuite = true
  images = ["sles12sp3"]
}

module "minssh-sles12sp3" {
  source = "./modules/libvirt/host"
  base_configuration = "${module.base.configuration}"
  name = "minssh-sles12sp3"
  image = "sles12sp3"
  gpg_keys = ["default/gpg_keys/galaxy.key"]

}

``
The sles12sp3 image has the qemu-agent enabled inside.

In the example the typo i did was `network_name` to default.

The problem was that we were getting the wrong iface and IPs from domain.  ( the solution in sumaform side is to use `br`).

With this we were getting the `iface` `default` instead of the domain one. ( i need to investigate if the pb was sumaform specific or upstram one..

The pb is that is not easy to get VMs  upstream with qemu-agent inside enabled by default unless you build your own ..

The only thing is that i think this was working previously with the flavio custom call of qemu-command, but imho that command was doing more then just using the MACRO.

FACIT: i could not reproduce it anymore with a  normal sles12sp3  image with qemu-agent.
we merge it

i prefer to concetrate on other issues which will bring more value.
I think this feature would be a nice refactor but i am unsure about the qemu-agent, since at moment we don't tests it in testacc so we cannot assume this bring a regression.  ( all the tests i did were ok  but i am maybe missing some complexity). So in this case i prefer to keep the old code that works instead of this refactor