Problem consuming from stream when moving from 1 shard to 2 shards.

We are using reactive-kinesis for the time to consume from a stream. With  1 shard, everything works fine. When we bump it up to 2 shards, the consumer shuts down soon after it starts. The log error is:
2018-05-09 12:03:48,991 [warn] c.w.r.k.c.ConsumerWorker - Worker for shard Some(shardId-000000000002): Skipped checkpointing on shutdown 2018-05-09 12:03:48,993 [info] c.w.r.k.s.KinesisSourceGraph - Consumer shutdown for shard shardId-000000000002 2018-05-09 12:03:48,995 [error] c.a.s.k.c.l.w.ShutdownTask - Application exception. java.lang.IllegalArgumentException: Application didn't checkpoint at end of shard shardId-000000000002 	at com.amazonaws.services.kinesis.clientlibrary.lib.worker.ShutdownTask.call(ShutdownTask.java:110)
Is there any configuration that must be done on the consumer? Is there something we have to do in code? Thanks for your time.

Duplicate of #37?

Is this a known issue? We are on a short deadline and we need to know if we can use this library in production where there are more than 1 shard.

We believe its because we switched from 1 to 2 shards. We consumed for some time with 1 before moving to two. This apparently closes the first shard and creates two new ones. Perhaps the closed shards have something to do with the problem? At any rate, it seems the the problem may lie in the changing of the number of shards.

@etspaceman As a temporary measure, we deleted the stream and recreated it with two shards, one ending in 0 and one ending in 1. When the consumer starts up, we get an error about a lease still existing for shard 2 (which I assumed got deleted when we deleted the stream). Have you seen this behavior?

Sorry for the late reply @jackmitchelwidman - I have not seen that behavior personally so I'm not of much help here :(

Are there any fix planned?


Pwa 744

Pull Request Test Coverage Report for Build 1597

0 of 7 (0.0%)  changed or added relevant lines in 5 files are covered.
2755 unchanged lines in 308 files lost coverage.
Overall coverage decreased (-38.3%) to 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pages/Product/subscriptions.js
0
1
0.0%

components/Reviews/mock.js
0
1
0.0%

components/Reviews/components/AllReviewsLink/index.jsx
0
1
0.0%

components/FilterBar/components/Content/components/FilterChips/connector.js
0
2
0.0%

components/FilterBar/components/Content/components/FilterButton/connector.js
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

components/SnackBar/mock.js
1
0.0%

components/FilterBar/components/Content/style.js
1
0.0%

pages/WriteReview/components/ReviewForm/components/FormButtons/mock.js
1
0.0%

components/Reviews/components/List/components/Review/components/Title/style.js
1
0.0%

pages/Product/components/VariantSelects/components/ProductVariants/helpers/StepByStep.mock.js
1
0.0%

pages/Reviews/constants.js
1
0.0%

pages/Favorites/constants.js
1
0.0%

components/FilterBar/components/Content/components/Sort/components/Item/style.js
1
0.0%

pages/Filter/components/ClearAllButton/style.js
1
0.0%

pages/FilterAttribute/components/ClearButton/style.js
1
0.0%

Totals

Change from base Build 1596:
-38.3%

Covered Lines:
0

Relevant Lines:
5123

💛  - Coveralls


worldToCanvas marks below ground surface

Describe the bug
worldToCanvas is currently marking below ground level.
To Reproduce

Enable Tile Indicators plugin
Head to any Y axis location greater than the games ground level
Click a destination to walk to
Marking won't be on the surface your character is on

Expected behavior
Indicator should be on the surface, not the sub level.
Screenshots
https://lambda.sx/PtL.gif
https://lambda.sx/YCk.png
Environment (please complete the following information):

OS: Win10
RuneLite version: Current Master

Updated

The changes made last night have now resolved the issue I reported on.

Then I guess it should be closed


Find/Search history is lost when (re)opening a workspace

I spent a while creating a fairly complex regex for searching for a pattern in a window. When I opened a new folder (window) I was expecting the find regex to still be available.  I thought maybe it remembered the history for each window which wouldn't have been so bad, but on reopening the original window again, the history is still missing.
This seems like a bug as Code seems to remember most things.

It is remembered per-folder/workspace, and you can navigate history using the arrow keys in the input box

But it's not. Steps to reproduce:

Open window.
Enter search term.
Open another folder (same window)
Re-open the original folder
Search history is gone.

I can reproduce this in Ubuntu and Macos

Sorry, are you talking about find in an editor "ctrl+F" or workspace search "ctrl+shift+F"?
I think the first is not currently saved, the second is.

Yes - ctrl+f.
That would explain it - are there any plans to save it?

Sorry, are you talking about find in an editor "ctrl+F" or workspace search "ctrl+shift+F"?
I think the first is not currently saved, the second is.

And it should be easily discoverable, so we can review search history in a list and select a previous search pattern to search a file again.
Becasue mostly when we use VSCode to read logs or other from a program, we always search for the same key word to find out problems.
This function should be essential, many other editors all implemete it, such as Notpad++, sublime text etc.

Sorry, are you talking about find in an editor "ctrl+F" or workspace search "ctrl+shift+F"?
I think the first is not currently saved, the second is.

And it should be easily discoverable, so we can review search history in a list and select a previous search pattern to search a file again.
Becasue mostly when we use VSCode to read logs or other from a program, we always search for the same key word to find out problems.
This function should be essential, many other editors all implemete it, such as Notpad++, sublime text etc.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/44127

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/44931

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/44981

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/48067

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/55787

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/55794

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/57292

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/70532

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/80256

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97692


[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)

Package
All packages
Existing issue?

[X] I checked the existing issues

What happened?
Expected vs actual result:
When creating a new project and adding the package it shows the error:
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)
Steps to reproduce:
Just create a new project, add the package and build it
When removing the package the error disappears!
Relevant log output
D/EGL_emulation( 8270): eglMakeCurrent: 0x7b4b934de0: ver 3 0 (tinfo 0x7b4b797560)
E/flutter ( 8270): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)
E/flutter ( 8270): #0      PathProviderApi.getApplicationSupportPath (package:path_provider_android/messages.g.dart:65:7)
E/flutter ( 8270): <asynchronous suspension>
E/flutter ( 8270): #1      getApplicationSupportDirectory (package:path_provider/path_provider.dart:78:24)
E/flutter ( 8270): <asynchronous suspension>
E/flutter ( 8270): #2      _localPath (package:google_fonts/src/file_io_desktop_and_mobile.dart:39:21)
E/flutter ( 8270): <asynchronous suspension>

I can't repro this, could you please share more details (package version, device, OS version, etc.)?


Error on end, Unhandled Promise

I'm using v1.2.0 to have chromless log into spotify and get the user auth token to manage a playlist (they have to be associated to a user not an application).
After chromeless.end I get an unhandeled promise rejection. I do briefly see chrome asking to save the password? Maybe not related.
Error: No such target id: undefined
    at IncomingMessage.<anonymous> (/Users/scoope7/xm-playlist-backend/node_modules/chrome-remote-interface/lib/external-request.js:16:26)
    at emitNone (events.js:110:20)
    at IncomingMessage.emit (events.js:207:7)
    at endReadableNT (_stream_readable.js:1045:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickDomainCallback (internal/process/next_tick.js:218:9)

the code requires working username and password
import { Chromeless } from 'chromeless';

async function run() {
      const chromeless = new Chromeless();
      const screenshot = await chromeless
        .goto('https://accounts.spotify.com/en/authorize?client_id=bcfe5f4d02244abfb7e38a3052f02a76&response_type=code&redirect_uri=http:%2F%2Flocalhost:5000%2FupdatePlaylist&scope=playlist-modify-public&state=xmplaylist')
        .click('.btn,.btn-sm')
        .wait('#login-username')
        .type(config.spotifyUsername, 'input#login-username')
        .type(config.spotifyPassword, 'input#login-password')
        .click('.btn-green');

      await chromeless.end();
}

Edit: To be clear. Even with a catch on both awaits the error is not caught and throws a UnhandledPromiseRejectionWarning.

Same issue to me

I have same issue.
Node 6.2.2
Chrome 60
OSX 10.12.6

Same issue, but in my case I am playing one video asset on browser. It looks like node process is getting terminated but not chrome browser with end() call.

same issue with me.
hopefully ASAP fix.

Seeing it here too

Curious if any of the above cases trigger a pageload of some kind. Right now that's a feature we're missing in chromeless is multi-page interactions

Seems to occur even without doing anything for me:
const { Chromeless } = require('chromeless')

async function run() {
  const chromeless = new Chromeless()
  await chromeless.end()
}
run().catch(console.error.bind(console))

I'm still experiencing this issue in version 1.2 using the same codeblock that @mx781 had posted.  Any updates?

A new version hasn't been cut yet with the fix.

Still having the problem, with the given example -
const { Chromeless } = require('chromeless')

async function run() {
  const chromeless = new Chromeless()

  const screenshot = await chromeless
    .goto('https://www.google.com')
    .type('chromeless', 'input[name="q"]')
    .press(13)
    .wait('#resultStats')
    .screenshot()

  console.log(screenshot) // prints local file path or S3 url

  await chromeless.end()
}

run().catch(console.error.bind(console))

Idem, same error with the given example.

Node v8.3.0
Chrome 60.0.3112.113 (Build officiel) (64 bits)
Windows 10

@adieuadieu Why don't you just release new version?

I'm sorry for the delay. Just published v1.3.0 to NPM. Please do let us know if it resolves these issues!

It's working perfectly :+1:

Itou. Nickel 👍


Changing the port does not seem to work

The default port for this addon is 8080. In the Home Assistant GUI there is an option to change the port number I have another HA Addon that is using (forwarding) port 8080. So, when i tried to start this addon it would not start as it said that port 8080 was already in use. When I change the port number it starts but the log shows  that port 8080 is being used:
Setting trap PID 1
Chunked transfer encoding: False
Threads: 1
 * Serving Flask app 'app'
 * Debug mode: off
WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
 * Running on all addresses (<IP_ADDRESS>)
 * Running on http://<IP_ADDRESS>:8080
 * Running on http://<IP_ADDRESS>:8080
Press CTRL+C to quit

I have also tried to set the port number using the configuration instructions for the "RHVoice component for Home Assistant" (definitio/ ha-rhvoice), but this too does not seem to set the port. With some settings I get a 404 error with others a 500 error..

Logs will always show 8080 but you can change forwarding port.
Example:

Set addon port to 9999.
Set rhvoice integration port to 9999 (it must match)
Restart Home assistant
I tested and it works.

Thanks @definitio I tried this and it does work. I am guessing that the port set in the configuration is being forwarded to 8080 in the container which is why it will always have that in the log.

Yes, you're right.
On Sat, 23 Sept 2023, 09:50 Frederick Henderson, @.***>
wrote:

Thanks @definitio https://github.com/definitio I tried this and it does
work. I am guessing that the port set in the configuration is being
forwarded to 8080 in the container which is why it will always have that in
the log.
—
Reply to this email directly, view it on GitHub
https://github.com/definitio/ha-rhvoice-addon/issues/1#issuecomment-1732234288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI4KKJ5HKKMLQEF74BEA5A3X32BD3ANCNFSM6AAAAAA5BFG6UU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Unable to reuse module

I'm trying to use the same instance of a module in two different views (in React Native 0.17.0). The module works fine in the first view, but when I navigate from the first view to the second view, all of the module's properties and functions are suddenly undefined.
Here's a reproducible example, starting with a FirstView.js file:
'use strict';

import React from 'react-native';
import SharedModule from './SharedModule';
import SecondView from './SecondView';

let {
    Text,
    TouchableHighlight,
    View
} = React;

export default class FirstView extends React.Component {

    buttonPressed() {
        SharedModule.foo();
        this.props.navigator.push({component: SecondView});
    }

    render() {
        return (
            <View>
                <TouchableHighlight onPress={() => this.buttonPressed()}>
                    <Text>Next</Text>
                </TouchableHighlight>
            </View>
        );

}

Here's a SharedModule.js file:
'use strict';

class SharedModule {

    foo() {
        console.log('foo...');
    }

}

module.exports = new SharedModule();

And here's a SecondView.js file:
'use strict';

import React from 'react-native';
import SharedModule from './SharedModule';

let {
    Text,
    TouchableHighlight,
    View
} = React;

export default class SecondView extends React.Component {

    buttonPressed() {
        SharedModule.foo(); // Error: _SharedModule2.default.foo is not a function
    }

    render() {
        return (
            <View>
                <TouchableHighlight onPress={() => this.buttonPressed()}>
                    <Text>Submit</Text>
                </TouchableHighlight>
            </View>
        );

}

What am I doing wrong here?

This turned out to be user error. I had a circular dependency in my code.


Modeladmin search backend

I was using modeladmin for a model that searched several fields, including related lookups. The search was timing out because the orm query that it built up was pretty big.
So I thought it would be a good idea if modeladmin could use the built in wagtail searching, since Wagtail allows the indexing of custom models.
This doesn't include is a way to turn off searching for an Indexed model, but I couldn't think of a use case for that. You could just override the show_search property.

Hi @seb-b. This looks like a really useful addition, thank you! Utilising the search backend where available definitely make more sense than relying on the ORM (and a sensible default). However, I do have some slight concerns regarding backwards compatibility; particularly where developers may be using filters for fields that aren't included in their model's search_fields list.
I think we should offer those developers a way to turn the ORM-based behaviour, in case they have issues.
I think we might also want to include a note in the documentation for ModelAdmin.list_filter that makes these limitations clearer when adding filters for an indexed model.
Does this sound okay to you @seb-b?

@ababic sounds good to me :smile:
The other approach I started with was to add another option to ModelAdmin class like search_backend = 'default' to hide this behaviour behind a switch, also giving users the option to change to a different search backend if there was more than one. Would that make more sense? It would make this a non-breaking change but would mean that the search backend is not the default

The other approach I started with was to add another option to ModelAdmin class like search_backend = 'default' to hide this behaviour behind a switch, also giving users the option to change to a different search backend if there was more than one. Would that make more sense? It would make this a non-breaking change but would mean that the search backend is not the default

Yes, I think this would be a better approach. Another thing I hadn't thought about before is: if the developer is still using the default Wagtail search backend, automatically using the search backend here would create a poor experience. I don't think we really have a choice but to stick with the ORM approach by default, and have this behaviour swappable somehow.
An attribute like search_backend would do nicely. Though, I think I might prefer search_helper_class  (to follow the modeladmin terminology), and instead of introducing some arbitrary string values, we could make it reference a class that would handle the searching, e.g. wagtail.contrib.modeladmin.helpers.DjangoORMSearchHandler (or accept a path string and import lazily when needed). This would make the behaviour easier to customise.
Does this sound okay to you?

Yep sounds good.

@ababic I've just encountered a problem when using this. The results returned from WagtailBackendSearchHandler would be some sort of subclass of SearchResults and not a querset, that means that if the filters want to use distinct it'll blow up. So I'm thinking either the WagtailBackendSearchHandler.search_queryset should turn the the results back into a queryset or the IndexView.get_queryset is reworked to account for this (perhaps distinct is called after filtering and searching)

Hey @seb-b, good spot!
I would propose that  IndexView.get_queryset() be updated to apply distinct() beforehand if necessary (based on filters_use_distinct). Then pass an additional distinct_applied boolean to get_search_results(), and then on to search_handler.search_queryset(). The search_queryset() method can then use this to figure out whether or not to apply distinct() again, and then just return a queryset.
The WagtailBackendSearchHandler can probably just ignore distinct_applied completely, as I think most search backends only ever return unique results.
Does that sound okay to you?

@seb-b. Really appreciate your time and effort on this, thank you :) It's looking really good. I especially like that you've been updating the docs as you go. I always find that a great way to sanity check my implementation.
From a testing perspective, I think it would be good to establish some separate unit tests for these handler classes, now that they are nicely abstracted? Simple input/output tests are so much more concise and easier to maintain (more fun to write too!). I know I've asked a lot already, but are you okay to make a start on this?

@ababic Thanks for all your feedback. I've added some test cases in the latest commit that cover most of the gotchas I think. But if there's any other functionality you think they should cover let me know :smile:

@ababic I reverted the commits with the full test cases and added in some more specific unit tests

@seb-b Sorry for the radio silence... It's been a busy few weeks! Things are looking much better on the testing front, thank you. I still need find some time to review this all properly in my local environment. If I have not done this and got back to you in the next 7 days, please shout at me! :)

Hi @seb-b, I'm afraid I'm noticing a bit of an issue locally in relation to ordering. IndexView.get_ordering() automatically adds -pk to the ordering fields when pk isn't already present, which tends to result in an OrderByFieldError when searching with the WagtailBackendSearchHandler activated, because index.FilterField('pk') itself is rarely added to a model's search_fields. I think it's  more common to add index.FilterField('id') or index.FilterField('pk') (or whatever the pk field is).
To make this work, I think IndexView.get_ordering() needs some tweaking to identify the name of the field being used as the primary key, and add ordering for that instead. I also think we need to provide some extra warnings in the docs about using WagtailBackendSearchHandler, as literally every field/relationship used in ordering (any column that can be mapped to a field), filters or search_fields must be indexed for the model in order for search to work.
Do you have any thoughts on this?

I ran across this while testing, it didn't seem like a big problem though. Each time it happened the warnings told me exactly what to fix, what field to add and to what model. I feel a warning in the docs would be enough, since this isn't the default search handler.
Note:
While using WagtailSearchHandler ensure that any fields specified in the modeladmins ordering attribute are indexed using FilterFields. If no ordering is set then the models primary key must be indexed.

e.g. #code goes here

@seb-b I see your point, but I don't feel like like user's having to index pk specifically is something they should have to do just to make use of this functionality. Indexing the same field twice with different names certainly feels like a strange thing to force on developers, and I don't know enough about the inner workings of the various supported backends to know if it could cause other problems somewhere down the line.
Your example warning, though definitely helpful, doesn't really explain this away, because the developer isn't specifying that 'pk' be added to ordering... it's something that view is applying without their knowledge.
Don't worry about changing anything more now. I can look at this myself when I come to preparing this to be merged :)

Hi @seb-b. How would you prefer to be named in CONTRIBUTORS.rst?

Scratch that! I just realised you are already in there :)

Merged in b839bd6. Thanks @seb-b!

Unfortunately it looks like this has broken our Elasticsearch tests (https://travis-ci.org/wagtail/wagtail/builds/543188481 - we've had to put them as 'allow failures' because they had a tendency to fall over due to intermittent server issues). @seb-b @ababic, please could you investigate?

Unfortunately it looks like this has broken our Elasticsearch tests (https://travis-ci.org/wagtail/wagtail/builds/543188481 - we've had to put them as 'allow failures' because they had a tendency to fall over due to intermittent server issues)

@gasman I will look into this. Probably best to fix in a separate PR as this one is done with now.


bug: stuck when dropping sink

Describe the bug
Create sink (nexmark_q4), wait for a while, and then drop it. The client was stuck then.
CREATE SINK nexmark_q4
AS
SELECT Q.category,
       AVG(Q.final) as avg
FROM (SELECT MAX(B.price) AS final,
             A.category
      FROM auction A,
           bid B
      WHERE A.id = B.auction
        AND B.date_time BETWEEN A.date_time AND A.expires
      GROUP BY A.id, A.category) Q
GROUP BY Q.category
WITH ( connector = 'blackhole' );

Observations:

The CPU usage is zero.
async stack trace:

Barriers:

To Reproduce
Failed to reproduce it in my environment...
Expected behavior
No response
Additional context

Looks like we're stuck in sync_epoch. Does it mean that there's a deadlock in the Hummock event loop? Note that dropping streaming jobs triggers the de-registration of the compaction groups and bloom filter catalog for internal tables, and it's easy to get it wrong.
Will #7637 be related?

Will https://github.com/risingwavelabs/risingwave/issues/7637 be related?

I guess not? #7637 was a deterministic bug while this one looks accidental.

Does it mean that there's a deadlock in the Hummock event loop? Note that dropping streaming jobs triggers the de-registration of the compaction groups and bloom filter catalog for internal tables, and it's easy to get it wrong.

Can you give some hint on how to investigate on this? I am trying to enable verbose_async_trace now.

Will #7637 be related?

Guess not +1. #7637 only fix the sink catalog metadata in catalog manager. Compaction group related registration and un-registration are already handled properly via TableFragments.

I am trying to enable verbose_async_trace now.

async_stack_trace seems to be set to Verbose by default on the testing cluster as "store_await_sync" is already a verbose span. 🤔
I've quickly checked the code of the HummockEventHandler and there's even no await while processing an event. I've also run the gdb -batch -ex 'thread apply all bt' -p 1 to check the backtrace of all threads and find nothing interesting like parking-lot lock parking. 🤔

async_stack_trace seems to be set to Verbose by default on the testing cluster as "store_await_sync" is already a verbose span. 🤔

Haha, that screenshot was updated by me after I set aync_stack_trace to "verbose"

I've tried to reproduce the bug both in our local environment and in the CI environment, but failed.
The CI environment used to be able to reproduce the bug, but now we have run under the same benchmark setting for many times, but still cannot reproduce it.
I suspect that the bug has been fixed by some PR by the way, and I have added necessary logs to trace the progress of sync. If there is some pending sync task, we can see from the log.
I think we can hold this issue and observe whether the bug happens again in the future.

It seems that this bug has not been seen for several months?

It seems that this bug has not been seen for several months?

I think so. Maybe we can close this issue?

Agree. Let's close it now.


Feature Request - Zoomable and Clickable Highlighted Screenshots in Video Search Results

I hope this message finds you well. I am a user of your Cyte Screenshot app and appreciate its unique features and capabilities for knowledge management and easy history search. I am writing to propose a new feature that I believe would significantly enhance the user experience and the app's functionality.
Feature Description:
I would like to request a feature that allows users to zoom in on highlighted screenshots taken from videos and/or have the ability to click on the video result to open it at the specific time of the search result. Additionally, the searched words should be highlighted within the video or transcript, making it easier for users to visually identify the information they are looking for.
It is really hard to find the result from the video, users have to watch the whole video and try to find the result.
Best regards,

Thank you for raising this, you are absolutely right that Cyte should take you right to the specific time of the result and it should be highlighted when you press the expand button, this seems to be a regression I introduced in https://github.com/shaunnarayan/cytev2/commit/263c69dc4bed7cc877510df54c8850edaf3fceea
I'll work on getting this fixed ASAP

Fixed in https://github.com/shaunnarayan/cytev2/commit/a1ecd8d0836681033c7195b6f56a91277c7333c4
Here's a demo of the expected functionality; the result goes to the exact timestamp with the target highlighted, and continues to highlight the search term even when scrolling the timeline


This should be a workflow

so we can add the concurrency controls

Is this the time to migrate to the semantic-release-action organization?


Versions not aligned or Maven Central publishing issues? Could not find com.netflix.graphql.dgs.codegen:graphql-dgs-codegen-client-core:5.2.4

My CI/CD pipeline failed with the error: Could not find com.netflix.graphql.dgs.codegen:graphql-dgs-codegen-client-core:5.2.4
I did some digging, and found that version 5.2.4 does not show up on https://search.maven.org/search?q=com.netflix.graphql.dgs.codegen. The last version that shows up there is 5.1.17, see screenshot:

On https://mvnrepository.com/artifact/com.netflix.graphql.dgs.codegen , however, I found that 3/4 of com.netflix.graphql.dgs.codegen artifacts are available at version 5.2.4, while graphql-dgs-codegen-client-core is still at 5.1.17. It says that graphql-dgs-codegen-client-core had its Last Release on Mar 1, 2022 (see screenshot).

Is it intended that these versions are not aligned, or did something go wrong with the publishing here?
Thanks!

Nevermind 🙂
I just found this PR: https://github.com/Netflix/dgs-codegen/pull/411 , where graphql-dgs-codegen-client-core was renamed to graphql-dgs-codegen-shared-core


Benchmarking methodology

Hi team!
Could you please describe the methodology used to measure the 20k orders per second throughput?

How many validators,s entries and full nodes were created?
Where were they created?
What machines were used?
What was the network topology?
How many clients were used to broadcast the transactions?

Also I would like to understand what do you mean exactly by saying that 20k orders per second are processed?

Do you mean that single transaction is executed per order?
Or maybe many messages are processed in single transaction?
Does one transaction/message contain a single order or there are many orders in single transaction/message?
Let's say that there is a logic which matches two orders (buy & sell). Do you count it as two orders in the "20k" result or as a single action?

Thanks in advance!

Hey @outofforest, thanks for the question!
How many validators,s entries and full nodes were created?
We did benchmark with 35 validator nodes. No full nodes were created as part of the benchmark since they were
Where were they created?
They were dispersed among various regions. An example breakdown would be:
Ohio - 5
Sydney - 6
Seoul - 6
Frankfurt - 6
Sao Paulo - 6
Cape Town - 6
What machines were used?
16CPU, 64GB mem
What was the network topology?
We had all nodes set each other as persistent peers (connected graph)
How many clients were used to broadcast the transactions?
we used up to 1000 clients (the number changed based on how large batch size was)
Do you mean that single transaction is executed per order?
We did tests on both single and batched txs
Or maybe many messages are processed in single transaction?
Answered above
Does one transaction/message contain a single order or there are many orders in single transaction/message?
Answered above
Let's say that there is a logic which matches two orders (buy & sell). Do you count it as two orders in the "20k" result or as a single action?
This would be counted as multiple txs since it does result in 2 txs committed to the chain

Feel free to reopen if you have more questions - happy to chat!


Slurm headless/burst topology.conf when "scale" cluster

When the cluster is scaled (ie. cyclecloud_slurm.sh scale) the topology.conf file is overwritten.  That also overwrites the onprem switch definitions.  Is it possible to prevent this or separate the onprem topology from the Cycle topology?

This would be really helpful. Please.


PowSetCtor (Expand) pointers

[ ] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entries added to ./unreleased/ for any new functionality

closes #2378

Codecov Report

Merging #2399 (d173558) into main (ad8be0a) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2399      +/-   ##
==========================================
- Coverage   78.44%   78.43%   -0.02%
==========================================
  Files         441      441
  Lines       15115    15116       +1
  Branches     2088     2093       +5
==========================================
- Hits        11857    11856       -1
- Misses       3258     3260       +2

Impacted Files
Coverage Δ

...syte/apalache/tla/bmcmt/rules/aux/PowSetCtor.scala
83.33% <100.00%> (+0.57%)
:arrow_up:

.../at/forsyte/apalache/tla/bmcmt/arena/ElemPtr.scala
83.33% <0.00%> (-16.67%)
:arrow_down:

...he/io/annotations/parser/CommentPreprocessor.scala
96.70% <0.00%> (-1.10%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Planned changes for 4.0.0

This is a tracking issue for any breaking changes that we plan to include in the next major version.

Rename kRing and remove hexRange, hexRanges, and maybe hexRing. (#42)
Change all functions that allocate blocks of memory or that could receive invalid input to return a success/error code (uber/h3-java#10, uber/h3-java#21)
** Error codes should have meaning, e.g. invalid was not a valid H3 index, etc. uber/h3-java#22
Consider accepting an allocator/free function for functions that must allocate memory (#100)

Do you all have an idea of what kRing and kRingDistances will get renamed to, or when you expect to decide (I see a range of options in #42)? We're working on adding H3 to a geospatial library in Clojure (Factual/geo#22), and before that integration gets released it'd be ideal to use whatever names you expect to use in the next major version so we don't immediately have to start aliasing functions there too.

@willcohen can't you just reuse the Java API. I played with Clojure ~2 years ago and JVM integration was pretty straightforward.

thoughts on changing H3ToParent and H3ToChildren to use relative resolution instead of absolute?
getting parent:
H3ToParent(h, GetResolution(h)-1) -> H3ToParent(h, 1)
H3ToChildren(h, GetResolution(h)+1) -> H3ToChildren(h, 1)
the name of the functions already imply direction, so a resolution of 0 is the identity and negative offsets are invalid.

thoughts on changing H3ToParent and H3ToChildren to use relative resolution instead of absolute?

Not sold - I think the relative use case is no more common than the absolute, and not worth introducing confusion. But if we wanted to introduce h3ToRelativeParent as a convenience I'd support.

I agree with @nrabinowitz, I think in general both use cases are roughly as common.

I think another major-version change to consider is whether we can replace GeoPolygon and friends with LinkedGeoPolygon and friends. I really don't like having two different sets of data structures for this; the challenge is that GeoPolygon, while replaceable, is probably much faster than the linked version.

@nrabinowitz + @sahrk : i cannot anticipate the use case popularity for either relative or absolute and am not suggesting one is more popular than another.  I do believe that h3To[Parent|Children] inherently suggests relative operations, though.  while using h3, I found using absolute resolutions for each function counter-intuitive.  if there isn't general agreement on this being counter-intuitive then I'm all-in for not changing the meaning of the function for my sake :)

@joegilley I can see framing problems using either relative or absolute resolution; though I personally would also be more inclined to use relative, I don't think it's counter-intuitive to use absolute. Coincidently I was just using this function when you posted and it didn't feel odd. So I guess I'd still vote for both options. The question then is whether it warrants breaking the api to make the default the relative case.

Looking at the planned changes it would be great to have defines for the version number in the h3api.h header file. For example something like
#define H3_VERSION_MAJOR 4
#define H3_VERSION_MINOR 0
#define H3_VERSION_PATCH 0

would really help projects depending on this library to deal with the API-change.

@nmandery I think we could include that in the current version, since it's not changing an existing API.

@isaacbrodsky That would also be great. For me it would just be important to have a method to check the H3 version when the API break happens.

Per offline discussion: We should have a consistent, documented approach to input validation in V4, including what the criteria are to add input validation to a function, how errors are returned, and what kind of benchmark changes would lead us to avoid validation in favor of performance.

I've created a branch 4.x which can be used for developing the 4.0.0 release. PRs can be made against that branch for breaking changes, and when ready it can be merged into master of uber/h3.

We'll keep track of this using the project, https://github.com/uber/h3/projects/1 instead of this issue. Keep an eye out for H3 v4 in the future :)


Fix css for alignment

Small fix for alignment.
https://github.com/HTMLGhozt/Thomas-dillard-studios/issues/1

Fixed in #2.


Bug:  Missing Fonts on clean bower install of Ionic 1.3

I came across this when upgrading to 1.3
When my bower.json file is
{
"name": "HelloIonic",
"private": "true",
"devDependencies": {
"ionic": "^1.3.0"
}
}
The folder www\lib\ionic\fonts  is missing.
if my bower.json file is
{
"name": "HelloIonic",
"private": "true",
"devDependencies": {
"ionic": "driftyco/ionic-bower#1.3.0"
}
}
everything is installed correctly.

If you download the zip file from the CDN  everything is there.  Seems to be bower related.

Hi @Wombatpm,
Thanks for letting us know.  We'll check it out further.  I do have the fonts in my bower_components/ionic/release/fonts directory but I will be the first to admit I'm a bower n00b.
Any thoughts on why this might be happening?
Thanks,
Dan

I think I figured it out.  I had written my index.html file against ionic-bower#1.3.0 then cleared everything and installed ionic.  The path change to ionic/release/fonts  from ionic/fonts messed things up in my code because I was referencing the old css file. I'm a bower n00b as well. So you can close this issue as user error.
One Question though:  Is there a reason that the angular dependencies are 1.4.3  for one file and 1.5.3  for another?
bower install ionic  uses this bower.json file
https://github.com/driftyco/ionic/blob/master/bower.json
{
  "name": "ionic",
  "version": "1.3.0",
  "codename": "delhi",
  "homepage": "https://github.com/driftyco/ionic",
  "authors": [
    "Max Lynch<EMAIL_ADDRESS>    "Adam Bradley<EMAIL_ADDRESS>    "Ben Sperry<EMAIL_ADDRESS>  ],
  "description": "Advanced HTML5 hybrid mobile app development framework.",
  "main": [
    "css/ionic.css",
    "fonts/*",
    "js/ionic.js",
    "js/ionic-angular.js"
  ],
  "keywords": [
    "mobile",
    "html5",
    "ionic",
    "cordova",
    "phonegap",
    "trigger",
    "triggerio",
    "angularjs",
    "angular"
  ],
  "license": "MIT",
  "private": false,
  "dependencies": {
    "angular": "1.4.3",
    "angular-animate": "1.4.3",
    "angular-sanitize": "1.4.3",
    "angular-ui-router": "0.2.13"
  }
}

bower install driftyco/ionic-bower#1.3.0 uses this file instead
https://github.com/driftyco/ionic-bower/blob/v1.3.0/bower.json
{
  "name": "ionic",
  "version": "1.3.0",
  "codename": "delhi",
  "homepage": "https://github.com/driftyco/ionic",
  "authors": [
    "Max Lynch<EMAIL_ADDRESS>    "Adam Bradley<EMAIL_ADDRESS>    "Ben Sperry<EMAIL_ADDRESS>  ],
  "description": "Advanced HTML5 hybrid mobile app development framework.",
  "main": [
    "release/css/ionic.css",
    "release/fonts/*",
    "release/js/ionic.js",
    "release/js/ionic-angular.js"
  ],
  "keywords": [
    "mobile",
    "html5",
    "ionic",
    "cordova",
    "phonegap",
    "trigger",
    "triggerio",
    "angularjs",
    "angular"
  ],
  "license": "MIT",
  "private": false,
  "ignore": [
    "**/.*",
    "CONTRIBUTING.md",
    "Gruntfile.js",
    "component.json",
    "config",
    "examples",
    "package.json",
    "release/js/angular",
    "release/js/angular-ui",
    "scripts",
    "test",
    "js/controllers",
    "js/ext",
    "js/utils",
    "js/views",
    "js/_license.js",
    "js/ionic.js"
  ],
  "dependencies": {
    "angular": "1.5.3",
    "angular-animate": "1.5.3",
    "angular-sanitize": "1.5.3",
    "angular-ui-router": "0.2.13"
  }
}

Cool, thanks for contributing @Wombatpm.  Let me know if there is anything else I can help you with.
Thanks,
Dan


extension elements in aap_metadata.xml

There is no way to specify AAP extensions from plugins i.e. they are currently always provided by host. The features should be selectable by plugins too.
The simplest way to express this is <extension> elements in aap_metadata.xml, within <plugin> elements. When specified on XML, extensions have to be parsed and passed to instantiate().
It would be the simplest solution to https://github.com/atsushieno/aap-lv2-sfizz/issues/3 too.

It does not have to be an extension. It should rather be a property. Then it is a duplicate of https://github.com/atsushieno/android-audio-plugin-framework/issues/80.


Added Book hub android project

I uploaded my android development project  which is called book hub .   Plz accept my PR .
Thankyou ✨

Yeah sure  !!!


Error: BrowserModule has already been loaded

Hi
Was trying to use this module but faced this problem Error: BrowserModule has already been loaded
I was checking and your module is indeed importing the BrowserModule, it shouldn't do that https://angular.io/docs/ts/latest/cookbook/ngmodule-faq.html#!#q-browser-vs-common-module

Hi.
I would be really glad to help but unfortunately, I do not use Angular 2 anymore at all.
If you would like to contribute to this repository or fork it and maintain another version on your own, you are most welcome.

@dpinheiro I have the same issue also.

i have the same issue also

This is already fixed with the lastest version. The lib is now importing CommonModule instead of BrowserModule so it should be fine.


Question - A help on API -14?

This is not an issue since Android 2.3 is a pretty old platform and trying to support is on our own. This is more of a friendly need for help.
I managed to port the time picker dialog back compatible to API 10 simply by using AppCompat and NineOldAndroids. (And also some Build Version checks of course.) It is working fine but after choosing hour, RadialPickerLayout doesn't enter the minute chooser although animation takes place.
More precisely, after choosing hour, RadialTextsView disappear but same one appear again, not the minute ones.
Do you think it might be related to the current code?

I don't think I can help you here a lot.
This library started as a fork from the open source google date and timepicker. I have never touched that part of the code.
Both the time pickerviews are instances of the same class. The picker figures out what exact values to draw based on an index. Most likely you messed up the index somewhere, passing in HOUR_INDEX instead of TIME_INDEX.
The best you can do is set a breakpoint in setCurrentItemShowing() in the RadialPickerLayout and step through the code to see what exactly is going on.

It is only happening on API 10.
As you said, I'll have to do more debugging.
Thanks for the tip.

The problem is with the getDisappearAnimator & getReappearAnimator in RadialTextsView class.
You'll need to wrap your view with an AnimatorProxy if the running platform needs to be proxied.
so use:
mDisappearAnimator = ObjectAnimator.ofPropertyValuesHolder(                 AnimatorProxy.NEEDS_PROXY ? AnimatorProxy.wrap(this) : this, radiusDisappear, fadeOut)                 .setDuration(duration);
instead of:
mDisappearAnimator = ObjectAnimator.ofPropertyValuesHolder(                 this, radiusDisappear, fadeOut).setDuration(duration);

Thank's @Thore1954
it worked!


Publish on Open VSX

Hello!
Would you mind publishing your extension on Open VSX registry? Packages such as code or vscodium can't use Visual Studio Marketplace due to the fact that their Terms of Use restricts it only to Microsoft's products.
Here are the publishing instructions. It comes down to adding ovsx publish command in addition to your normal vsce publish.
Thanks for making this extension!
Note: the text is copied from vscode-beancount issue.

https://open-vsx.org/extension/dongfg/vscode-beancount-formatter

[x] Published


use CA_ADDR to align with the env variable CA_ADDR used in sds_agent.go

use CA_ADDR to align with the env variable CA_ADDR used in sds_agent.go
https://github.com/istio/istio/blob/master/pkg/istio-agent/sds-agent.go#L94

@irisdingbj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
0d5eb77e59f66f84afb2e6ee3a6b062d92012c5f
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


List MACs on power up

Idea:  When doing a power up, it would be useful to list the MAC addresses in a human readable form
This could be used by anybody, or a manufacturing machine, from the terminal when the system powers up.
Currently 1.4.0 the MACs are only listed out following the configuration.
If somebody needed to find the MACs before configuration, they can't.
LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.Completed scan for access points
WiFi Network scan done
10 network(s) found
1: AzondeNetSsid (-44)*
....
ESP32 WiFi interface ready
Found no matches for saved networks
ESP32 WiFi started
ESP32 WiFi interface ready
WiFi Stopped
Waiting for SmartConfig.
LED_WIFI_OFF - White
ESP32 soft-AP stop
ESP32 soft-AP stop
ESP32 soft-AP stop
ESP32 soft-AP stop
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
then when configured through the App
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
ESP32 WiFi connected to AP
LED_CONNECT_WIFI - Green
LED_CONNECT_WIFI - Green
LED_CONNECT_WIFI - Green
ESP32 station got IP from connected AP
Obtained IP address: <IP_ADDRESS>
LED_CONNECT_WIFI - Green
SmartConfig received.
Waiting for WiFi
Smart Config done, connected to: AzondeNetSsid with psswd: xxxxxx
Writing network to NVM: AzondeNetSsid,xxxx
Stored networks in NVM: 0
Device has 1 networks stored in NVM
LED_CONNECT_WIFI - Green
WiFi Connected
WiFi MAC: A8:03:2A:4D:CC:B4
ETH  MAC: A8:03:2A:4D:CC:B7
A8032A4DCCB4
ETH Started

The firmware does list the mac addresses (eventually and if you troll through the console log).
Very few (non-geeky) people are going to connect a serial monitor to the board.
This request is to print them sooner?
Or would you prefer to have the mac address as part of the hostname?
See this ticket: https://github.com/openeew/openeew-firmware/issues/16

Yes the request is to print them sooner if possible.
My idea is that is to be able to uniquely ID the device easily, possibly before they have a serial number printed on them, and could be used in the unit assembly process by @andygrillo manufacturing program.
Once @andygrillo manufacturing program has loaded the production firmware, at the factory, it won't get configured in the factory, so listing them earlier would allow the ID to be visible.
@andygrillo this is what I was thinking of. Production firmware download for unit assembly by your rPi test jig, that would list the MACs.  Would it be useful  for your rPi manufacturing program in the unit assembly to be able to parse the MAC at this point,  and then print them to  any label that is created for the openEEW sensor unit before packaging.
Is it easier to have a JSON like ID to be able to pull them off in your manufacturing program?
It would also be a type of manufacturing test, as if able to parse a valid MAC it probably indicates a lot of the openEEW sensor hardware is working.

I left this GitHub issue open because there are other requests here.

For the latest download from openeew-firmware-main (main)  version 1.5.0 "git pull;   Updating 0884667..9336cd6"
the MAC numbers don't show before it gets a configuration.
For no WiFi networks active (but ethernet plugged in),  I can't see what the MACs are, -
I was thinking power up is before configuration.
OpenEEW Sensor Application
ESP32 WiFi interface ready
Stored networks : 2
ESP32 WiFi started
Completed scan for access points
WiFi Network scan done
8 network(s) found
1: Kitchen speaker.o (-66)
2: PurpleAir-42fc (-67)
3: LazySky (-67)*
4: ATT6HzT3GE (-81)*
5: AzondeNetSsid (-90)*
6: DIRECT-57-HP OfficeJet 3830 (-90)*
7: HOME-B202 (-92)*
8: HomeBaseWifi104 (-94)*
ESP32 WiFi interface ready
Reading stored networks from NVM
ESP32 WiFi started
ESP32 WiFi started
Found network Arthur2004Sid , xxxxxx
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Reading stored networks from NVM
Found network ArthurGuestSsid , xxxxxx
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Found no matches for saved networks
ESP32 WiFi interface ready
WiFi Stopped
Waiting for SmartConfig.
LED_WIFI_OFF - White
ESP32 soft-AP stop
ESP32 soft-AP stop
ESP32 soft-AP stop
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.LED_LISTEN_WIFI - Blue
.....
FOR a WIFI network ArthurGuestSsid present and Ethernet plugged in

You have some other build.  The log above isn't the from the final 1.5.0 release.
In the scenario of a registered device but that WiFi network not available, with an Ethernet cable connected, the log should look like this:
OpenEEW Sensor Application
ETH Connected
ETH MAC: A8:03:2A:4D:D5:F3, IPv4: <IP_ADDRESS>, FULL_DUPLEX, 100Mbps
ESP32 WiFi interface ready
Stored networks : 1
ESP32 WiFi started
Completed scan for access points
WiFi Network scan done
5 network(s) found
1: wynd565o (-45)*
2: wynd565o (-55)*
3: wynd565o (-81)*
4: Fios-Gk6Rn (-90)*
5: oliver (-93)*
ESP32 WiFi interface ready
Reading stored networks from NVM
ESP32 WiFi started
ESP32 WiFi started
Found network 565guest , xxxxxx
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Got no match for network
Found no matches for saved networks
Stored networks : 1
Previously registered device, use hardwired Ethernet connection.
WiFi MAC: A8:03:2A:4D:D5:F0

Many thanks. Don't know what happened with my build and download- driver error I guess - still learning it.
Now showing
OpenEEW Sensor Application
ETH Started
ETH Connected
ETH MAC: A8:03:2A:4D:CC:CF, IPv4: <IP_ADDRESS>, FULL_DUPLEX, 100Mbps
ESP32 WiFi interface ready
Stored networks : 2
ESP32 WiFi started
ETH Disconnected
I'm closing this as that was the purpose to have a unique ID.


[DisplayList] Provide a method to transfer save stack state from one DisplayListBuilder to another

Currently, for our SkCanvas rendering, we apply rendering calls to either or both of a leaf_nodes_canvas (which is performing the actual rendering for the current under-over-inter-layer of our embedding stack) and/or an internal_nodes_canvas (which has all of the canvas objects for all of the embedding stack tied together in a multi-dispatch adapter). The latter allows us to track the transform/clip state for all of these canvas objects in parallel to avoid issues when we switch from one embedding layer to another.
A simpler implementation would be to provide a way to transfer the transform/clip/save stack from one graphics object to another, such as:
DisplayListBuilder::copyState(DisplayListBuilder& other) {
  // copy all entries from save_stack_ to other.save_stack_;
  other.transform_ = this.transform_;
  other.clip(_bounds)_ = this.clip(_bounds)_;
}

Note that since the outgoing graphics object will be retired when switching between embedding stack elements, this could also be a transfer operation that ended up with effectively restoring the source builder back to its initial state as it mirrors all of the items into the new builder. Perhaps offering both "copyState" and "move/transferState" depending on whether the caller wants to move or copy the state.

Advantages:

engine layers no longer have to deal with "which canvas do I execute my call on?"
reduced amount of tracking of transform/clip state in multiple redundant objects (note that even after we retire a canvas as we progress through the embedding stack, that canvas will still have to deal with save/restore calls even though it is no longer going to be doing any rendering)


Using the ovirt_vm_infra.yml example with -C option, fails to find the 'ovirt_vms' variable

ovirt_vm_infra.txt
As per the attached dump file ovirt_vm_infra.txt, when running the ovirt_vm_infra.yml example with the -C option, not all libraries seems to be brought in so the 'ovirt_vms' variable is not defined. Removing the -C option resolves this error, but -C should still work for checking.
Command line example:
ansible-playbook -C -vvv ovirt_vm_infra.yml
Error:
fatal: [localhost]: FAILED! => {
"msg": "The task includes an option with an undefined variable. The error was: 'ovirt_vms' is undefined\n\nThe error appears to have been in '/etc/ansible/roles/oVirt.vm-infra/tasks/create_vms.yml': line 6, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: "Create VM {{current_vm.name }}"\n  ^ here\nWe could be wrong, but this one looks like it might be an issue with\nmissing quotes.  Always quote template expression brackets when they\nstart a value. For instance:\n\n    with_items:\n      - {{ foo }}\n\nShould be written as:\n\n    with_items:\n      - "{{ foo }}"\n\nexception type: <class 'ansible.errors.AnsibleUndefinedVariable'>\nexception: 'ovirt_vms' is undefined"
Compressed file with yml example:
ovirt_vm_infra.tar.gz
Note: use "tar xvzf  ovirt_vm_infra.tar.gz"

@stvnrsnbrg Hey, i tried to reproduce locally with master and I can't. Instead I am getting the following:
(ansible) [kkoukiou@katerina ovirt-ansible-infra]$ ansible-playbook -C -i inventory test.yaml
[DEPRECATION WARNING]: The use of 'include' for tasks has been deprecated. Use 'import_tasks' for static inclusions or 'include_tasks' for dynamic inclusions. This feature will be removed
in a future release. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
[DEPRECATION WARNING]: include is kept for backwards compatibility but usage is discouraged. The module documentation details page may explain more about this rationale.. This feature will
be removed in a future release. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.

PLAY [oVirt infra] ***************************************************************************************************************************************************************************

TASK [oVirt.vm-infra : Login to oVirt] *******************************************************************************************************************************************************
skipping: [localhost]

TASK [oVirt.vm-infra : Create VMs] ***********************************************************************************************************************************************************
included: /home/kkoukiou/repos/oVirt/ovirt-ansible-vm-infra/tasks/create_vms.yml for localhost
included: /home/kkoukiou/repos/oVirt/ovirt-ansible-vm-infra/tasks/create_vms.yml for localhost
included: /home/kkoukiou/repos/oVirt/ovirt-ansible-vm-infra/tasks/create_vms.yml for localhost

TASK [oVirt.vm-infra : Create VM postgresql-vm-0] ********************************************************************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "check mode and async cannot be used on same task."}

TASK [oVirt.vm-infra : Logout from oVirt] ****************************************************************************************************************************************************
skipping: [localhost]
	to retry, use: --limit @/home/kkoukiou/repos/oVirt/ovirt-ansible-infra/test.retry

PLAY RECAP ***********************************************************************************************************************************************************************************
localhost                  : ok=3    changed=0    unreachable=0    failed=1

So I guess your issue does not reproduce any more, however you can not use this role in check mode, because of check mode and async cannot be used on same task.

Since this role is using async mechanism, it's not possible to support dry mode.


Address confusion about meaning of the properties that affect modules consumption

The compiler ignores the properties 'Additional BMI Directories' and 'Public C++ Module Directories' for no apparent reason. - Visual Studio Feedback
It would be good to add more info and maybe samples in the documentation for those properties:
VC++ Directories – Public C++ Module Directories, All modules are public
C/C++ - General – Additional BMI directories, Additional Module Dependencies
Output Files – Module Output File Name

Right now, the F1 target for the All Header Files are Public, Public C++ Module Directories, and All Modules are Public properties is the VC++ Directories property page doc. This location isn't ideal for examples. The same is true for Additional BMI Directories where F1 takes you to the C/C++ Property Pages doc. There's no /ifcSearchDir[path] compiler option article; this option isn't even advertised by the compiler, and I've seen no other mention of its existence.

Thank you for your patience as we work through our backlog. We have received more issues than we can manage at this time. We are closing issues based on their severity, how broadly applicable they are, and whether there is enough information provided to act on the issue. If you believe that this issue should not have been closed, please add a comment explaining why by 3/1/2024. We appreciate your contribution and encourage you to propose changes using public PRs, or submit feedback through our updated control at the top of the page, as we phase out Git Issues for this repo. For more information on our feedback systems, see https://aka.ms/ContentUserFeedback.


External access configuration when using QNAP Dockerstation

Is there a configuration file to map  -p 81:1025 \  for external access?
I start the docker on QNAP Dockerstation and not using the command line as suggested;
docker run -d
--name="blueiris"
--privileged
--init
--restart=always
-e TZ=America/Los_Angeles
-p 8080:8080
-p 5900:5900
-p 81:81
-v /path/to/data:/home/wineuser/prefix:rw
--log-opt max-size=5m --log-opt max-file=2
jshridha/blueiris

Change the -p 81:81 to -p 81:1025 and once you get into Blueiris settings, change the WebUI port to 1025 so it matches. Second number is for the port inside the container, first is the port for outside the container.


DataTable sort feature

Description
Adds ability to define sort in DataTable code using sort, where you can pass one column and a sort direction ("asc" or "desc")
For groups (using groupBy), groups will be sorted according to subtotals where applicable.
Basic example
<DataTable data={flights} sort="fare desc"/>

Simple group example
	<DataTable {data} groupBy="category" sort="sales" subtotals=true>
		<Column id="category" />
		<Column id="item" />
		<Column id="sales" fmt="usd" />
	</DataTable>

Checklist

[x] For UI or styling changes, I have added a screenshot or gif showing before & after
[x] I have added a changeset
[x] I have added to the docs where applicable

Last remaining issue on this:

Initial sort affects the data shown in the query viewer. Subsequent sorts via clicking columns do not affect it. This happens even with an explicit order by in the sql query


CBL-2783 : Add more C++ Database tests

Added a script for generating code coverage for CPP API (Note: This report is not push to the PR and can be enhanced later).
Added more tests for Database.hh.

Before:

Filename
Function
Line
Region
Branch

Database.hh
77.08% (37/48)
74.87% (149/199)
72.13% (44/61)
41.67% (5/12)

Totals
84.62% (198/234)
83.55% (721/863)
82.44% (277/336)
68.89% (62/90)

After:

Filename
Function
Line
Region
Branch

Database.hh
93.75% (45/48)
91.46% (182/199)
83.33% (55/66)
37.50% (6/16)

Totals
88.46% (207/234)
87.83% (758/863)
84.75% (289/341)
67.02% (63/94)

Before

Filename
Function
Line
Region
Branch

Database.hh
77.08% (37/48)
74.87% (149/199)
72.13% (44/61)
41.67% (5/12)

Totals
84.62% (198/234)
83.55% (721/863)
82.44% (277/336)
68.89% (62/90)

After

Filename
Function
Line
Region
Branch

Database.hh
93.75% (45/48)
91.46% (182/199)
83.33% (55/66)
37.50% (6/16)

Totals
88.46% (207/234)
87.83% (758/863)
84.75% (289/341)
67.02% (63/94)


Level magic numbers

Overview
Magic numbers were used around level ids. I created 3 constants to be used instead as well as simplifying the way Level subclasses are instantiated.

Thanks! 🔢
It became easier to read!

Please fix this 🙏
e: /home/circleci/project/app/src/test/java/io/github/droidkaigi/confsched2018/presentation/sessions/feedback/SessionsFeedbackViewModelTest.kt: (47, 25): Too many arguments for public fun createDummySessionFeedback(sessionId: String = ...): SessionFeedback defined in io.github.droidkaigi.confsched2018 in file DummyDataCreator.kt

LGTM 👍

すみませんでした、対応しました。


Output COCO-style JSON Annotation for single images in eval.py

Hi @dbolya,
thank you for this great piece of software. I already used it, to generate images with drawn masks using the inference script eval.py on single images and folders. This worked fine. Now after getting the images, i wanted to programmatically use the inference results, which is why i would need a coco style json file instead of an png as an output.
I already saw there is the args.output_coco_json parameter, but looking at the code, it seems to only work when evaluating whole datasets (with ground truth available?) and not for single images or folders of single images, without annotations.
What i would like to do, is calling eval.py similar to this:
--config=yolact_base_config
--trained_model=weights/yolact_base_54_800000.pth
--output_coco_json
--bbox_det_file=myimage.bbox.json
--mask_det_file=myimage.mask.json
--score_threshold=0.2 --top_k=15
--image=myimage.png:myimage_out.png

(with only myimage.png, yolact_base_config and weights/yolact_base_54_800000.pth already existing.)
i also tried putting the following additional parameters into the arguments:
--mask_proto_debug
--dataset=coco2017_testdev_dataset

but they did not help, as the coco style dumping function in eval.py (https://github.com/dbolya/yolact/blob/02bde378e98cd7e2831639ff782c1f5a5046771a/eval.py#L960) does not get executed when --image is set (see https://github.com/dbolya/yolact/blob/02bde378e98cd7e2831639ff782c1f5a5046771a/eval.py#L855).
Is there any other way i can use arguments to reveice a coco style json file with annotations for a single image or a folder of images?
If not, please regard this as a feature request :)
Kind Regards
Alexander

The way --output_coco_json works right now is it operates on a dataset (which doesn't need to have ground truth). So you need to give a --dataset parameter instead of an --image parameter. The dataset you use doesn't need annotations, simply create a COCO dataset with just images and an empty annotation array and follow the dataset definition of coco2017_testdev_dataset.
For single images, this is kind of cumbersome, but for large batches it probably works fine? This feature was intended for submission to the COCO eval servers so that's why it's so roundabout. Let me know if this is indeed too cumbersome, and I can add --output_coco_json support to --image.

Thank you very much!
I will try and get back to you.
Will take a while though as i now have found a workaround for my usecase, so i will close this for now :)

Thank you very much!
I will try and get back to you.
Will take a while though as i now have found a workaround for my usecase, so i will close this for now :)

Hi,
Did you tried --dataset parameter. As I'm getting the same error as you have mentioned. If yes, can you share the code?

Thank you very much!
I will try and get back to you.
Will take a while though as i now have found a workaround for my usecase, so i will close this for now :)

Hi,
Did you tried --dataset parameter. As I'm getting the same error as you have mentioned. If yes, can you share the code?

Hi Ritesh,
i have not tried it yet, there was a change of priorities in our project and i think we won't need this anymore. Sorry I can't help you.
Kind Regards
Alexander

The way --output_coco_json works right now is it operates on a dataset (which doesn't need to have ground truth). So you need to give a --dataset parameter instead of an --image parameter. The dataset you use doesn't need annotations, simply create a COCO dataset with just images and an empty annotation array and follow the dataset definition of coco2017_testdev_dataset.
For single images, this is kind of cumbersome, but for large batches it probably works fine? This feature was intended for submission to the COCO eval servers so that's why it's so roundabout. Let me know if this is indeed too cumbersome, and I can add --output_coco_json support to --image.

Sorry, could you clarify how should define the dataset in this case?
I am trying the following:
my_dataset = coco2017_testdev_dataset.copy({
    'name': 'My  Dataset',
    'train_images':  'folder_to_coco',
    'train_info': None,

    'valid_images': 'folder_to_coco',
    'valid_info': None,

    'has_gt': True,
    'class_names': my_cat_array,
    'label_map': None
})

But I get error:
Loading model... Done.
Traceback (most recent call last):
  File "eval.py", line 1131, in <module>
    evaluate(net, dataset)
  File "eval.py", line 923, in evaluate
    progress_bar = ProgressBar(30, dataset_size)
  File "./utils/functions.py", line 60, in __init__
    self._update_str()
  File "./utils/functions.py", line 76, in _update_str
    num_bars = int(self.length * (self.cur_val / self.max_val))
ZeroDivisionError: division by zero

The way --output_coco_json works right now is it operates on a dataset (which doesn't need to have ground truth). So you need to give a --dataset parameter instead of an --image parameter. The dataset you use doesn't need annotations, simply create a COCO dataset with just images and an empty annotation array and follow the dataset definition of coco2017_testdev_dataset.
For single images, this is kind of cumbersome, but for large batches it probably works fine? This feature was intended for submission to the COCO eval servers so that's why it's so roundabout. Let me know if this is indeed too cumbersome, and I can add --output_coco_json support to --image.

It's too cumbersome, please add json output to --image use ;)
(I've created my own dataset, but cv2.imgread seems to be failing for some reason.  I've verified that the path is correct, yet I get 'NoneType' object has no attribute 'shape' :-(

The way --output_coco_json works right now is it operates on a dataset (which doesn't need to have ground truth). So you need to give a --dataset parameter instead of an --image parameter. The dataset you use doesn't need annotations, simply create a COCO dataset with just images and an empty annotation array and follow the dataset definition of coco2017_testdev_dataset.
For single images, this is kind of cumbersome, but for large batches it probably works fine? This feature was intended for submission to the COCO eval servers so that's why it's so roundabout. Let me know if this is indeed too cumbersome, and I can add --output_coco_json support to --image.

It's too cumbersome, please add json output to --image use ;)
(I've created my own dataset, but  if target.shape[0] == 0: is failing, as target = []. What am I doing wrong?)

hey, I also failed in 'ig target.shape[0]==0', how can fix? thanks


fix: input focus error in accordion

📝 Description
Fix: input focus error when using in accordion with selected/default expand keys on page loaded.
Mini-reproduction: nextui-input-focus-error
⛳ Current behavior (updates)
Change input wrapper onClick property, removed condition e.currentTarget === e.target
🚀 New behavior
Input component focus behavior normal in accordion component with selected/default expand keys
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
Nothing!

Summary by CodeRabbit

Bug Fixes

Improved click event handling for input components, streamlining focus logic.

Refactor

Simplified internal code structure for clarity and conciseness while maintaining backward compatibility.

Fixed focus, and still error with validator trigger. When page loaded, focus field trigger validator bug execute normal without accordion
With accordion component and default selected key:

Without accordion component:


vivipary

New term request for
term: vivipary process
definition: A process promoting the
germination of embryos while still attached to the
mother plant.
REf: PMID: 7599651
Ref: PMID: 11244106
Ref: Annual Review of Ecology and Systematics
November 2000, Vol. 31, pp. 107-138
Pankaj
Reported by: jaiswalp
Original Ticket: "geneontology/ontology-requests/1641":https://sourceforge.net/p/geneontology/ontology-requests/1641

thanks www.movied.org


Does not work with the new version of Obsidian

I'm submitting a ...
[X] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
Fantastic plug-in. I'm using the 2.1.1 version on Mac (macOS Monterey Version 12.6.1), and unfortunately since the new version of Obsidian (1.3.0 - Installer 1.2.8), it does not work anymore.

Antidote version : Antidote bilingue v4.0.1 (Français 11.4.5767 | Anglais 11.4.5767)
macOS version : Monterey Version 12.6.1
Obsidian version : 1.3.0 - Installer 1.2.8
Plugin version : 2.1.1

Hi,
I cannot reproduce it.
It works great on my side with:
Antidote version: Français 11.3.5077 | Anglais 11.3.5077 with Connectix version Version <IP_ADDRESS> (1104.59.27)
macOS: Ventura 13.3.1
Obsidian version: 1.2.8 (Installer 0.15.9)
Plugin version: 2.1.1
The only differences I see are the OS version (which is higher in my case), and the Antidote version, where you French and English version are higher than me, and I do not find anywhere how to get the 11.4. The website and Connectix updater only allows me to get the latest 11.3.

Bonjour Quentin,
Everything had been working fine for weeks. The reported problem happened immediately after the last Obsidian update.
My computer:
iMac (Retina 5K, 27-inch, 2020)
Processor 3.6 GHz Intel Core i9 10 cores
Memory 128 GB 2667 MHz DDR4

I deleted all other plugins → no change
I reinstalled Obsidian → no change

Strange thing, I don't have this problem with the Macbook Pro (which is under Ventura) with the same obsidian configuration!
For the moment I'm doing the corrections with the Macbook, waiting to understand why it doesn't work anymore on the iMac

I do not have a Monterey to test. But it seems more a Connectix issue with "older" mac version than an issue of this plugin.

I'm having the same issue on Windows 11, clicking any of the corrector or dictionary buttons on the toolbar don't do anything. It works fine in other applications like Microsoft Word and Mozilla Firefox
What I tried:

Reinstalled Antidote and Connectix
Restart my PC
Delete and reinstall the plugin, both manually and through the community browser plugin in
Removing all other plugins
Creating a new notebook

Razer Blade 16 (13th Gen Intel(R) Core(TM) i9-13950HX, 64 GB RAM, 3 TB SSD, RTX 4080)
Microsoft Windows 11 Pro Version	10.0.22621 Build 22621
Antidote English 11.3.4470
Antidote French 11.3.4470
Connectix 11.3.447

I think it's the new version of Antidote that's the problem, because having updated Antidote on the Macbook Pro, now it doesn't work on this machine either :(

Hum… maybe we should contact the support of Antidote.
Maybe @Acibi or @alaroucheDruide have some insights about this issue?

For the moment the only "solution" I found is to use the new Antidote 11 clipboard correction feature:

I copy all the text on the Obsidian page ( CMD+C )
I use the Antidote clipboard correction function ( OPT+CMD+K )
I make the corrections
I paste the corrected text on the Obsidian page ( CM+V )

I do not have a Monterey to test. But it seems more a Connectix issue with "older" mac version than an issue of this plugin.

I've just tried it with a new-generation Mac, freshly installed with the latest version of Obsidian and Antidote, and ... same problem.

I also have this issue with the latest version of Obsidian.
I will investigate.

Okey, it is fixed in https://github.com/Heziode/obsidian-antidote/releases/tag/2.1.2
Sorry for the delay

Thanks @Heziode
I can confirm that it works now on my system. Cheers!


multiple threads -w not work

Even set -w to a different number, the program consist running in 3 cores...

How did you know that didn't work? Did you check the thread number in tools like task manager?  3 worker threads mean 6 thread in total.
If you set -w 5, you may see 8 threads.
Be noted that fastp is so fast that if you set too many threads, it will have bottlenecks in I/O, so the CPU usage will be not increased.

Closing


Fix formaldehyde scale

Resolved #1127
It works fine with my device, but I'm not sure whether this is the correct way.

Thanks!

https://github.com/AlexxIT/XiaomiGateway3/releases/tag/v3.3.3


feat: configuration file support

It should be possible to provide configuration options via a config file, for options which are largely invariant within a project / environment. We should follow general recommendations (clig.dev, 12 factor).

NOTE: adding a -c/--config flag will require renaming the rules specification flag (maybe -r/--rules?)

Objective: support layered config via cli options or config flags
Requirements:

[ ] decide on implementation (possible solutions: config-rs, clap+serde)
[ ] rename existing options as needed
[ ] implement according to above recommendations
[ ] demo config + tests

Started something with clap+serde in the branch feat/config-file, but this would require more work because:

optional variables do not work: as missing arguments cause clap to fail
merging config with args is not possible as-is
This would require a more advanced crate (clap-serde or something simlar), which requires defining a separate config struct.
Maybe overkill for now


SOLR-8393: Component for resource usage planning

https://issues.apache.org/jira/browse/SOLR-8393
Description
New component that attempts to extrapolate resources needed in the future by looking at resources currently used.
Original idea by Steve Molloy, with additional parameter based on comment from Shawn Heisey.
Documentation copied from the Jira ticket.
Solution
New component: SizeComponent.java.  Component can be set on the /select handler to provide sizing for a single core.
New collection operation: ClusterSizing.java.  Action 'clustersizing' is added to CollectionsHandler.  Class ClusterSizing calls the size component for each core.
DRAFT : Skeleton implementation of V2 API.  What would be the best approach ?  Going through OverseerCollectionMessageHandler and implementation of CollectionApiCommand ?  Cluster sizing is not really a collection operation, it visits the whole cluster.  And there's the NamedList built in the SizeComponent, per replica: how can that tie into the cluster sizing V2 response ?  Should it be a more generic structure (Map), and formatted... later...?
Tests
The size component is tested in SizeComponentTest.java
Cluster sizing response is tested in ClusterSizingTest.java
Full test on a running instance of Solr.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[x] I have added documentation for the Reference Guide

I'd prefer to see a V2 api added instead of a V1 api.   Adding more V1 api's is just adding to the backlog of work on our V2 migration, so I'd love to see that instead added...!

The use of the hphenated (kebab style?) total-disk-size pattern I think should be changed to camelCase totalDiskSize, that is the pattern we use in the rest of our JSON output.

This looks very helpful, though I can't speak to if it's accurate or not...    I'd love to see somethign replace the old excel spreadsheet that we recently removed as it was no longer useful/accurate.  Maybe @janhoy you have some thoughts on this....

I'd prefer to see a V2 api added instead of a V1 api. Adding more V1 api's is just adding to the backlog of work on our V2 migration, so I'd love to see that instead added...!

Agreed, but, as mentioned, this is from a pre-existing patch.
I come back to Solr only about once a year, and usually to apply some old patch on a more recent version. That means I have a limited understanding of what ties into what and why.  So, implementing everything needed for clustersizing v2 would be a long and difficult process for me.
Participation is welcomed!

About to take a look at the code and see if I can help with the v2 side of things, but before I dive into that I figured it was worth asking:
Does size-estimator-lucene-solr.xls actually work for folks?  Do you use it regularly @igiguere ?  Have you found it to be pretty accurate?  Any other folks have experience with it?
I'm happy to be wrong if we have several groups of folks out there in the wild that are using it, but my initial reaction is to be a little skeptical that it's reliable enough to incorporate into Solr.
Primarily because, well, modeling resource-usage is a really really tough problem.  There's a reason that the community's only response to sizing questions has always been pretty much "You'll have to Guess-and-Check".
And secondarily, because the spreadsheet this is all based off of was added in 2011 and hasn't really seen much iteration in the decade since.  There's an absolute ton that's changed in both Lucene and Solr since then.

@gerlowskija

Does size-estimator-lucene-solr.xls actually work for folks? Do you use it regularly @igiguere ? Have you found it to be pretty accurate? Any other folks have experience with it?

Me, personally, no, I don't use it ;).  I'll try to find out from client-facing people in the company.  I doubt anyone has compiled usage vs success statistics.

... the community's only response to sizing questions has always been pretty much "You'll have to Guess-and-Check".

The cluster sizing feature is documented to estimate (i.e.: guess) resource usage.  We could make the documentation clearer that it's not a fool-proof measure.  But, at least it beats holding a finger to the wind.  And it's a bit less complicated that the xls and a calculator.

And secondarily, because the spreadsheet this is all based off of was added in 2011 and hasn't really seen much iteration in the decade since. There's an absolute ton that's changed in both Lucene and Solr since then.

We're only calculating RAM, disk size, document size.  Whatever has changed in Solr and Lucene, if it has an effect on RAM, disk space, doc size, then it should be reflected on the results... No?
Note that this feature is meant to be used on a current "staging" deployment, to evaluate the eventual size of a "production" environment, for the same version of Solr.  No one is expected to draw conclusions from a previous version, so changes from one version to another are not a concern in that way.
As a more general note, I should add that I'm a linguist converted to Java dev.  Not a mathematician ;)  If there's an error in the math, I will never see it.


Cleanup

Reformatted according to .editorconfig and removed unused imports

As always, thanks you very much for the PR!
Would it be possible to change the .editorconfig from spaces to tabs and reformat? I changed this for the dprint formatting script I use, but I forgot to do this for the .editorconfig it seems. Or let me know if you'd prefer spaces instead (either is fine for me!)

@Fevol yes, we can change to tabs if it is your personal preference.
Spaces were chosen because the original obsidian.d.tsuses spaces.
See the comment https://github.com/Fevol/obsidian-typings/issues/22#issue-2071578922

Ah right, I'm sorry - I forgot about that.
I'll update the dprint config to match the obsidian.d.ts styling.


Update generate.ps1 for AutoRest version

It seems that AutoRest version restriction for generating the Resources SDK changed. Using the latest master branch of SDK repo, the AutoRest "V2" produced schema errors. We were able to generate the SDK without errors by using the latest version of AutoRest.
Also found the linked commit which seems to be related to AutoRest version requirement change: https://github.com/Azure/azure-sdk-for-net/commit/82b3e736e66151a529260cf3b996e92dee3cd698#diff-d9cb33753189ab29d54760a2ac998bc6
Based on that updating the generate.ps1 AutoRest version parameter.

@isra-fel FYI

Abandoning the PR.


LDAP protocol support

Is it in your roadmap?

It's never been a priority, mainly due to lack of demand.  Can you tell me
a bit about what you're looking for?
On Fri, Feb 12, 2016 at 6:42 AM, Pál Szák<EMAIL_ADDRESS>wrote:

Is it in your roadmap?
—
Reply to this email directly or view it on GitHub
https://github.com/bbyars/mountebank/issues/85#issuecomment-183307175.

@bbyars I want this feature so much. There are some apps in my company that use the LDAP server as an user authentication server. Unfortunately, it's not stable sometimes, so I want to mock them to make sure that I can execute my tests at anytime.
This is my rough idea for Mountebank to support for LDAP protocol.
Mountebank should accepts/decline bind request, allow user to set the predicates for search request and configure response message with simple format e.g. JSON.
I googled it and found the ldapjs which provides API that can make a LDAP server.

Thanks for the nudge; that's enough for me to move it out of the "wishlist" and into the roadmap. That said:

I'd like to do https://github.com/bbyars/mountebank/issues/174 before implementing any new protocols
I'm well over capacity at the moment, just keeping up with basic functionality and writing the book, so if you're depending on me it's several months out at least.

For anyone who still needs this, here is a really basic one: https://gitlab.com/dAnjou/mountebank-ldap

Closed as community created through protocol extensions.


BUG: eval throws error on var (replace eval)

@SamLewis0602
i am using typescript and wanted to use the transpiled js files with your awesome plugin, but noticed that the ts output
var CoolStringer = class {
  coolify(s) {
    return `\u{1F60E} ${s} \u{1F60E}`;
  }
};

does not work.
On the other hand, "regular" js classes work.
class CoolStringer {
  coolify(s) {
    return `\u{1F60E} ${s} \u{1F60E}`;
  }
};

Basically, whenever there is a var, the files does not get loaded.
Error message from Obsidian Dev Console:
main.ts:65 SyntaxError: Unexpected token 'var'

also, eval should probably not be used. i propose using a npm module with better & safer evaluation

would love to hear your feedback and to merge.
furthermore, i like to work on this extension with you if you would like and grant me dev access to manage this repo together 🙂.

Hi @alexn-s thanks for the PR. I'm happy to look at PRs but I won't be granting you access to manage the repo.
See here for why eval is required to work on Obsidian mobile: https://github.com/SamLewis0602/obsidian-custom-js/issues/3#issuecomment-912801366
What OSes have you tested your PR on?

@SamLewis0602  ok understandable.
OS -> windows, but code is expected to be platform independent.
i think i will probably for your repo, to put my own "spin" on it and drop mobile support,
or would you be willing to create an obsidian-custom-js-desktop repo with features that do not work on mobile,
so that #3 could be solved in an "easier" way?
may i suggest a few ideas for this repo (would love to send pull request for them also)?
of course the suggestions are opinionated, so let know what do you think

refactor code and put into src folder (maintainability for the future for creating multiple small files)
tsconfig use paths so relative imports dont have to be used (easier to digest)
quicker bundler (looking at esbuild/ tsup)
types into types/ @types folder

looking forward to your input and hearing about the pr

Rather than creating a plugin for that, you can just require() files directly in code blocks in Obsidian if you don't care about mobile support. This plugin only exists because that doesn't work on mobile. So a plugin for desktop-only would be unnecessary. If you do decide to create it, all I ask is that you call it something else to avoid confusion.
For your suggestions, I agree those things are useful in a large javascript project such as a web app, but I don't believe they're necessary (or even applicable in the case of 4) for this small plugin that's basically feature complete as it is.


webpack 5 - hmr check api no longer returns updated modules

Bug report
https://github.com/webpack-contrib/webpack-hot-middleware/issues/390
I've been investigating the cause.
as per the documentation https://webpack.js.org/api/hot-module-replacement/#check
it expects check to yield an array of updated modules. It uses this to compare to the updated modules in order to decide whether to reload.
when I debug I see:
https://github.com/webpack/webpack/blob/master/lib/hmr/HotModuleReplacement.runtime.js#L257
check doesn't add any modules, it passes the array to the runtime to update.
https://github.com/webpack/webpack/blob/master/lib/hmr/JavascriptHotModuleReplacement.runtime.js#L413
The runtime doesn't add any modules, but it passes the array on to the loader
https://github.com/webpack/webpack/blob/80c5041fe1f909d34975abd33e03bdb069876ea9/lib/web/JsonpChunkLoadingRuntimeModule.js#L280
The loader doesn't accept the second argument so it doesn't add any modules to the array
It looks like a bug because the documentation wasn't changed and the code still references the updatedModules array.
Seems it was introduced in webpack 5 alpha. 0 here:
https://github.com/webpack/webpack/commit/9d918615922f8e7fb8ab1c9712135908cf1cadd7#diff-99b22d1932803b7795bbe843d01cdf63d32e52a247610bfd9cffb95170227cc6
it seems that the node/ version had the updated module id's, but the web version does not.
Other relevant information:
webpack version:  5.3.1
Node.js version: 12
Operating System: windows 10
Additional tools:

/cc @sokra

/cc @sokra

I created an issue with webpack-hot-middleware that details an eventual side-effect of this: https://github.com/webpack-contrib/webpack-hot-middleware/issues/398. I believe that it's the exact same thing going on as in your linked issue (https://github.com/webpack-contrib/webpack-hot-middleware/issues/390)

bump

Hi! I'm facing the same problem, there is any update ?

Facing the same issue, by stepping into it a little further, I find this code in node_modules/webpack/lib/hmr/JavascriptHotModuleReplacement.runtime.js at around line 439：
``js
chunkIds.forEach(function (chunkId) {
if (
$hasOwnProperty$($installedChunks$, chunkId) &&
$installedChunks$[chunkId] !== undefined
) {
// HERE
promises.push($loadUpdateChunk$(chunkId, updatedModulesList));
currentUpdateChunks[chunkId] = true;
}
});

This calls `loadUpdateChunk` with 2 arguments, however inside `node_modules/webpack/lib/web/JsonpChunkLoadingRuntimeModule.js` around line 273, `loadUpdateChunk` is defined as:

```js
? Template.asString([
		"var currentUpdatedModulesList;",
		"var waitingUpdateResolves = {};",
		// HERE
		"function loadUpdateChunk(chunkId) {",
		Template.indent([
			`return new Promise(${runtimeTemplate.basicFunction(
				"resolve, reject",
				[

It only defines one parameter, it never pushes any entry into updatedModulesList array, causing hot.check fails to report updated modules.

bump

any news on this ?

is there any progress?

Send a PR

fixed

I don't know if this is related, but unless I use runtimeChunk: 'single' (can't use this because multi entrypoints app and if I do use it, code evaluated only once, which is problematic with js code embedded in modals), hmr doesn't work in dev ever since moving to webpack 5
Relevant part where error occurs:
self["webpackHotUpdatedevmyproject"] = function(chunkId, moreModules, runtime) {
	for(var moduleId in moreModules) {
		if(__webpack_require__.o(moreModules, moduleId)) {
			currentUpdate[moduleId] = moreModules[moduleId]; // Fails here with: Cannot set properties of undefined (setting '45461')
			if(currentUpdatedModulesList) currentUpdatedModulesList.push(moduleId);
		}
	}
	if(runtime) currentUpdateRuntime.push(runtime);
	if(waitingUpdateResolves[chunkId]) {
		waitingUpdateResolves[chunkId]();
		waitingUpdateResolves[chunkId] = undefined;
	}
};

Because you have several runtimes and only the first one processes, but the module is in another, I recommed to use single only for dev in such cases

I understand, however In webpack 4, I didn't have runtimeChunk: 'single' and HMR was working fine

can't use 'single' because then modals's js code is only evaluated first time

Sorry, but this shouldn't be related

So my only solution is to go back to webpack 4 where it was working I guess...rip http2
Thanks for replies

@Herz3h Sorry I don't understand you, if you show me a reproducible repository I'll show you how to solve it, multiple entries and HMR works without any  problems

Here is a repro: https://github.com/Herz3h/webpack-runtime-single-reproduction
When you click in fetch code, alert is shown first time, but not anymore if you click on the button multiple times

Because you are using the wrong configuration.

module.exports = {
  entry: {
    index: './public/js/index.js',
    homepage: {
      dependOn: 'index',
      import: './public/js/homepage.js',
    },
  },
};

Each un dependOn entry is a separate page and can not be used on one page because multiple runtimes are generated to make it work properly.

When you click on fetch code button, alert is shown first time, but not anymore if you click on the button multiple times

your code being executed (or not being executed) does not change in any way when changing the configuration, if this does not happen, there is probably a bug somewhere in the global application settings, not in webpack

Wasn't aware of the dependOn option. It seems to solve my issue. Thank you !! @alexander-akait


how to use mixup_criterion()  for mixup augmentation?

I defined the augmentation transforms and clf as following:
transforms = [Mixup(alpha=0.4, beta_per_sample=False),
              FTSurrogate(probability=1, phase_noise_magnitude=1, channel_indep=True)]

clf = EEGClassifier(model,
                    lr = learning_rate,
                    max_epochs=n_epochs,
                    iterabraintor_train=AugmentedDataLoader,
                    iterator_train__transforms=transforms,
                    criterion=mixup_criterion,
                    optimizer=torch.optim.AdamW,
                    train_split=predefined_split(Dataset(valid_cat_X,valid_cat_y)),
                    optimizer__weight_decay=weight_decay,
                    batch_size=batch_size,
                    callbacks=["accuracy",
                               ('training', EarlyStopping(patience=n_epochs)),
                               ("lr_scheduler", LRScheduler('CosineAnnealingLR', T_max=n_epochs - 1)),
                               ],
                    device=device,
                    verbose=0
                    )

It gives me TypeError: mixup_criterion() missing 2 required positional arguments: 'preds' and 'target'
Could someone please provide me with examples of how to mixup_criterion() ? I would really appreciate it.

There is a typo in  iterabraintor_train=AugmentedDataLoader,  @yiyuchen-lab

Yes, the typo could fix this. Through the AugmentedDataLoader, the transforms are loaded into the collate_fn of torch's DataLoader. Here is an example of mixup_criterion.

great thanks @sbbrandt


the src folder maybe needed in published package

hi, thanks for this nice package, I find that the src folder does not exists in the published package which will cause the sourcemap cannot work as they're expected
can we add src folder into the files field here

Thank you for your contribution and sorry for the late reply... I was hesitant to add src to the package but I reckon is not a bad idea. Merged and shipped in v2.16.1


Async setup

For ext.commands allow setup to be async

actually not possible =((

Nah it's possible, you just don't use asyncio.run for it


Document the OAuth2 sequence for static sites and Edge Proxy Server secure access to the preferences server

The Problem
The user is interacting with a web site that has a preferences editor embedded into its pages, for example, UIO+.  The user wants to retrieve and/or save their UIO+ preferences to the preferences server.  Access to the preferences server is secure.
For a more complete description of the issues, see "GPII Preferences Edge Proxy Refinement I".
What is needed is documentation of the OAuth2 sequence of requests, responses, their payloads, and what is stored in various databases to realize the interaction outlined there.

Resolved with the merge of https://github.com/fluid-project/personal-data-server/pull/8.


Debugging Unit Tests in context gone.

Does this issue occur when all extensions are disabled?: No there is no 'language support' if disabled

VS Code Version: 1.81.0
OS Version: ubuntu 22.04.1

.NET SDK:
Version:   7.0.306
Commit:    f500069cb7
Runtime Environment:
OS Name:     ubuntu
OS Version:  22.04
OS Platform: Linux
RID:         ubuntu.22.04-x64
Base Path:   /home/user/dotnet/sdk/7.0.306/
Host:
Version:      7.0.9
Architecture: x64
Commit:       8e9a17b221
.NET SDKs installed:
6.0.412 [/home/user/dotnet/sdk]
7.0.306 [/home/user/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.20 [/home/userdotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.9 [/home/userdotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.20 [/home/user/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.9 [/home/user/dotnet/shared/Microsoft.NETCore.App]
packages in test project

runtime; build; native; contentfiles; analyzers; buildtransitive
all

Steps to Reproduce:

Open folder having csproj with unit tests.
Put cursor on a [Fact] decorated function.

Right mouse click

Result no detectable tests
Schermopname van 12-08-23 10:09:40.webm

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.81.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Please file on the C# extension repo, thanks


[Tabs] Add icon and text

Added tab with icon
Added tab with icon and text
Added one example for each

Support FontIconand SvgIcon.

Shouldn't we put this above the {label}?

(https://www.google.com/design/spec/components/tabs.html#tabs-usage)

Wow how can I miss that ? Fixed ;)

@pradel That looks good :+1:. Do you want to try a squash down of the commits :grin:? Thanks.

Okay squashed this time working :)

@pradel Awesome. It's almost good. Can you add a bit of spacing between the label and the icon to follow more closely the specification?

@oliviertassinari like that is better ?

@pradel That's much better :+1:! Thanks. Can you squash down again :grimacing:?

Squashed :)

@pradel Thanks!


[Question] Connecting to VRS

I've managed to load in a domain the Virtual Radar Server (VRS), which has a url:port to fixed to recive data, the question is how do I configure the RPi to sent the data into the VRS? I've found a setting in  /etc/default/dump1090-mutability which says "NET_BIND_ADDRESS" and is fixed as default to "<IP_ADDRESS>", could it be I need to add another argument with the url:port of the VRS, and if so, how?, I've try to 'play' around with it but nothing has worked, any help is greatly appreaciated :D

Comment the line starting with NET_BIND_ADDRESS with a # and restart dump1090. You can then read data from dump1090 via VRS on any IP address assigned to the RPi.


New brackets page fails brackets test.

We went through some cleanup years ago to ensure that "brackets" and "braces" were never used as is, but always as:

curly braces
square brackets
angle brackets
lenticular brackets

And now we have a page, "Brackets", that has just that as a title without the modifier.
Do we fudge the test for this one instance, or do we fix the generated Brackets page to comply?

I'm up for a rename of the page, but what name would you use? The two-word expressions are great for the usual cases, but how would you describe the large set of sometimes unfamiliar characters that may not easily fit those definitions.
My suggestion for a possible better name for the table is "Bracketing character pairs."

This case is special enough, updated the test.


Update signalr-quickstart-rest-api.md for error code

Not restricted to 503, but all 5xx code indicates service error.

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