Could not send quit message: 'MpvPlayer' object has no attribute '_listener'

Describe the bug
This is like #409 but this was not fixed by anything specified in the issue. When I hit "Run Syncplay" the application crashes due to the error above.
To Reproduce
Steps to reproduce the behavior:

Be on same OS and Kernel version
You can use either the syncplay package or compile from source or AppImage, all have the same error
Enter in server details and hit "Run Syncplay" or "Store configuration and run Syncplay"
See error "Could not send quit message: 'MpvPlayer' object has no attribute '_listener'"

Expected behavior
Syncplay opens and I can wait in the room until my friends join and then we start up a video and watch together through syncing.
Screenshots
If applicable, add screenshots to help explain your problem.
Version and platform:

OS: Manjaro Linux, Kernel: 5.15.21-1
Syncplay version and build type: Syncplay version 1.6.9 (Yoitsu)

Additional context
Versions:
MPV - 0.34.1-dirty (>= 0.23.0)
FFmpeg library versions:
   libavutil       56.70.100
   libavcodec      58.134.100
   libavformat     58.76.100
   libswscale      5.9.100
   libavfilter     7.110.100
   libswresample   3.9.100
FFmpeg version: n4.4.1

Python - 3.10.2 (>= 3.6)
Python-Twisted - 21.7.0-4 (>= 16.4.0)
PySide2 - <IP_ADDRESS>-1 (except for 5.15.0)
Python-Certifi - 2021.10.8-3 (>= 2018.11.29)
Python-PyOpenSSL - 21.0.0-5 (>= 16.0.0)
Python-Service_Identity - 21.1.0-4
Python-IDNA - 3.3-3 (>= 0.6 != 2.3)

GUI Logs (With debug enabled):
Attribute Qt::AA_EnableHighDpiScaling must be set before QCoreApplication is created.
[04:25:39 PM] Could not send quit message: 'MpvPlayer' object has no attribute '_listener'
Unhandled Error
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/twisted/python/context.py", line 118, in callWithContext
    return self.currentContext().callWithContext(ctx, func, *args, **kw)
  File "/usr/lib/python3.10/site-packages/twisted/python/context.py", line 83, in callWithContext
    return func(*args, **kw)
  File "/usr/lib/syncplay/syncplay/vendor/qt5reactor.py", line 164, in _read
    self.reactor._iterate(fromqt=True)
  File "/usr/lib/syncplay/syncplay/vendor/qt5reactor.py", line 270, in _iterate
    self.runUntilCurrent()
--- <exception caught here> ---
  File "/usr/lib/python3.10/site-packages/twisted/internet/base.py", line 967, in runUntilCurrent
    f(*a, **kw)
  File "/usr/lib/python3.10/site-packages/twisted/internet/defer.py", line 662, in callback
    self._startRunCallbacks(result)
  File "/usr/lib/python3.10/site-packages/twisted/internet/defer.py", line 757, in _startRunCallbacks
    raise AlreadyCalledError
twisted.internet.defer.AlreadyCalledError:

[04:25:40 PM] client/server >> {"TLS": {"startTLS": "send"}}
[04:25:40 PM] client/server << {"TLS": {"startTLS": "true"}}
[04:25:40 PM] client/server >> {"Hello": {"username": "jordan", "password": "cd1a2c444a8517fe1eb25bbf809fa613", "room": {"name": "DITF"}, "version": "1.2.255", "realversion": "1.6.9", "features": {"sharedPlaylists": true, "chat": true, "featureList": true, "readiness": true, "managedRooms": true}}}
[04:25:40 PM] client/server << {"Set": {"ready": {"username": "jordan", "isReady": null, "manuallyInitiated": false}}}
[04:25:40 PM] client/server << {"Set": {"playlistChange": {"user": null, "files": []}}}
[04:25:40 PM] client/server << {"Set": {"playlistIndex": {"user": null, "index": null}}}
[04:25:40 PM] client/server << {"Hello": {"username": "jordan", "room": {"name": "DITF"}, "version": "1.6.9", "realversion": "1.6.9", "motd": "", "features": {"isolateRooms": false, "readiness": true, "managedRooms": true, "chat": true, "maxChatMessageLength": 150, "maxUsernameLength": 150, "maxRoomNameLength": 35, "maxFilenameLength": 250}}}
[04:25:40 PM] client/server >> {"Set": {"ready": {"isReady": false, "manuallyInitiated": false}}}
[04:25:40 PM] client/server << {"Set": {"ready": {"username": "jordan", "isReady": false, "manuallyInitiated": false}}}

No-GUI Logs (With debug enabled):
[16:46:54] Attempting to connect to syncplay.jordanplayz158.xyz:8999
[16:46:55] Could not send quit message: 'MpvPlayer' object has no attribute '_listener'
ERROR:	Could not load mpv: '__Listener' object has no attribute 'mpvpipe'
ERROR:	Media player error: "[Errno 111] Connection refused" The reason mpv cannot start may be due to the use of unsupported command line arguments or an unsupported version of mpv.
[16:46:55] client/server >> {"TLS": {"startTLS": "send"}}
[16:46:55] Attempting secure connection
[16:46:55] Successfully reached syncplay.jordanplayz158.xyz (2603:7081:2a3e:99ac:34b2:e73b:83f8:f750)
[16:46:55] client/server << {"TLS": {"startTLS": "true"}}
[16:46:55] Secure connection established (TLSv1.3)
[16:46:55] client/server >> {"Hello": {"username": "jordan", "password": "cd1a2c444a8517fe1eb25bbf809fa613", "room": {"name": "DITF"}, "version": "1.2.255", "realversion": "1.6.9", "features": {"sharedPlaylists": true, "chat": true, "featureList": true, "readiness": true, "managedRooms": true}}}
[16:46:55] client/server << {"Set": {"ready": {"username": "jordan", "isReady": null, "manuallyInitiated": false}}}
[16:46:55] client/server << {"Set": {"playlistChange": {"user": null, "files": []}}}
[16:46:55] client/server << {"Set": {"playlistIndex": {"user": null, "index": null}}}
[16:46:55] client/server << {"Hello": {"username": "jordan", "room": {"name": "DITF"}, "version": "1.6.9", "realversion": "1.6.9", "motd": "", "features": {"isolateRooms": false, "readiness": true, "managedRooms": true, "chat": true, "maxChatMessageLength": 150, "maxUsernameLength": 150, "maxRoomNameLength": 35, "maxFilenameLength": 250}}}
[16:46:55] Successfully connected to server
[16:46:55] client/server >> {"Set": {"ready": {"isReady": false, "manuallyInitiated": false}}}
[16:46:55] client/server << {"Set": {"ready": {"username": "jordan", "isReady": false, "manuallyInitiated": false}}}
[16:46:55] Connection with server lost, attempting to reconnect

Looked into it and added debug messages to places in the code, it seems that the connection is refused to the custom made socket but.... the default mpvsocket can be connected to fine. Going to delve deeper into it, just reporting my findings

Very sorry, It appears one of my extensions for MPV were overriding the ipc socket displayed by MPV, so this isn't an issue with MPV or SyncPlay. But perhaps to prevent this in the future we should use the --no-config (Was told this isn't possible as you guys bundle scripts), or maybe we can change config directory to non-existent directory and use command line arguments to pass everything or if we do have scripts, change mpv's directory it checks for configs to SyncPlay's own config folder

Moved feature suggestion talk to #512


Nima WebClient Configuration for keepAlive

Should it be related to io.helidon.common.socket.SocketOptions.Builder#socketKeepAlive ?

Should it be related to io.helidon.common.socket.SocketOptions.Builder#socketKeepAlive ?

Yes


[s6.1] Line diagram with points (BLW)

Description
Add the capability for a line chart with data points displayed at each value for more visual clarity.
Examples

Possible Approaches

Add a toggle in chart settings to enable or disable data points.
Allow customization of data point size and color within the color palette settings.

Defintion of Done

[ ] A new line chart option with data points is available on the dashboard.
[ ] Data points are displayed at each value along the line, providing clearer data visualization.
[ ] Users can toggle the display of data points on or off within chart settings.
[ ] Data point size and color can be customized via settings.

@florelina Has it been a requirement by the client that the color can be adjusted separately? In my opinion the datapoint should have always the same color as the line. So whatever color the user chooses for the line, this will automatically be applied to the data point.

No this is not a request from the client as far as I know. @sosiology please correct me if I'm wrong (see 4th to do point)

@florelina correct. This is not a requirement (that is known at this time)


Making search table a little more responsive

Search results table was not properly displaying with screen width under ~1130px.
This PR tries to improve these design points:

Search results table works well with screen width 720px and above.
Moved the "Snapshot" link under the title of the project and next to "Github" link. It becomes then a conscious choice to either view the stats or visit the repo.
Changing the icon for "Snapshot". While, indeed, a snapshot is literally an instant image of a project, what the user will access are graphs, bar charts and tables.
The nowrap attribute is deprecated in HTML5. So it's replaced by the CSS equivalent.

Preview:

deployed http://dashboard.kuona.io

Awesome! :)


Can function callback be supported to call reactor reaction method?

Please do a quick search on GitHub issues first, the feature you are about to request might have already been requested.
Expected Behavior
The current FunctionCallback only supports blocking programming. If the bottom layer of the provided function callback method uses the streaming programming of reactor, it can only be blocked by block or asynchronous thread pool.
Current Behavior
I hope to provide a FunctionCallback that can be called by reactor streaming programming. For example, bi function < request, tool context, Mono >
Context
This is more friendly for applications built entirely by webflux, and using blocking FunctionCallback in webflux will bring many difficult problems.

@PeatBoy - Thanks for flagging this. Reactive FunctionCallback isn't currently supported.
While we (@chemicL) may consider adding such a feature in the future, for now you'll need to use a blocking call instead of .subscribe(res::add).

Hey, as @tzolov mentioned, reactive/asynchronous signatures for functions are currently not supported, but it looks like a desired functionality having in mind streaming scenarios.
I suggest closing the discussion #1757 and keeping this issue and related discussions here.
As you noticed:
        // ...
        .subscribe(res::add);
    return new Response(res);

this construct immediately returns an object which is incomplete and another Thread will mutate it (potentially not even making the result visible to the consuming Thread) with no synchronization whatsoever. That is undesired.
As a solution for the time being you don't need to introduce CompletableFuture at all:
List<Map<String, Object>> data = CompletableFuture.supplyAsync(
    () -> this.dataQueryRepository.query()
        // ...
        .collectList()
        .block())
    .join();

Instead,
this.dataQueryRepository.query()
    // ...
    .collectList()
    .block())

should get you the same result.
For the Spring AI's internal implementation, it's worth keeping in mind that in case when the user function is executed in an event loop, it's worth offloading the blocking call into a bounded-elastic Scheduler on which blocking is allowed, e.g. via
        return Mono.fromCallable(() -> userBlockingFunction())
            .subscribeOn(Schedulers.boundedElastic());

regardless of whether the function is using reactive APIs or not, since just performing blocking calls in imperative code will degrade the performance and stall the event loop.

Ok, thank you for your reply @chemicL . I hope you can remind me under this issues if there is any progress in asynchronous/reactive function calling.


Ligma

Now that I have your attention, I'd like to propose that we remove RetroWrapper from the FAQ, as it's now been inactive for a year.

Wrong repo, sorry


⚠ Bofrost.fr has degraded performance

In 698c271, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 902 ms

Resolved: Bofrost.fr performance has improved in 14683c8 after 10 minutes.


[Elasticsearch DA] Add support to store service credentials in secrets manager

Once https://github.com/terraform-ibm-modules/terraform-ibm-secrets-manager/pull/157 is merged we should be able to call that module in the DA to add service credentials to a secrets manager secret

This should have been closed when https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/pull/247 was merged


address #89 - delay dashboard creation by 30s

Can we add all the DCGM profiling metrics? Why only these two? There are only six of them.
https://github.com/GoogleCloudPlatform/opentelemetry-operations-collector/blob/master/receiver/dcgmreceiver/metadata.yaml

LGTM after units fix.

Can we add all the DCGM profiling metrics? Why only these two? There are only six of them.
https://github.com/GoogleCloudPlatform/opentelemetry-operations-collector/blob/master/receiver/dcgmreceiver/metadata.yaml

@suffiank there are only these two here right now because they are the only two about which we have ever seen the dashboard module complain. I will add the remaining four now and update units/description with what is found in that metadata.yaml file you shared (thank you for that btw!)


add tests for MINLP

Codecov Report

Merging #12 into master will increase coverage by 4.6%.
The diff coverage is 75%.

@@            Coverage Diff            @@
##           master      #12     +/-   ##
=========================================
+ Coverage   78.53%   83.14%   +4.6%
=========================================
  Files          23       23
  Lines        1053     1056      +3
=========================================
+ Hits          827      878     +51
+ Misses        226      178     -48

Impacted Files
Coverage Δ

src/models/gndstruct_discdiam/minlp.jl
96.22% <75%> (+96.22%)
:arrow_up:

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 172

3 of 4 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.6%) to 83.144%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/models/gndstruct_discdiam/minlp.jl
3
4
75.0%

Totals

Change from base Build 170:
4.6%

Covered Lines:
878

Relevant Lines:
1056

💛  - Coveralls


Strange behaviour when navigating back between Cards

This is my code
    <Navigation navBarStyle={{ backgroundColor: 'purple' }} titleStyle={{ color: 'white' }}>
      <Card exact path="/watch" component={SeriesOverview} />
      <Card exact path="/watch/series/:seriesId" component={SeriesDetail} />
    </Navigation>

It seems that if you navigate back from SeriesDetail to SeriesOverview, the Component is rerendered without the match and history object. I wanted to ask why it behaves like that?

Hey @n1ru4l,
Strange behavior 😟 . Are you facing the same issue on this example ?

When I comment out the shouldComponentUpdate method in src/app/modules/Feed/components/Article/index.js and add console.log( this.props.match ) to the render method I can reproduce this issue.
Also I noticed that there are many dispatches when using the BottomBarTabs the redux store logs the current route name 8 times ( and the buttom also flickers 8 times), which does not seem wanted to me :/

Hi @LeoLeBras,
I found out if I clicked "See item X" button, then clicked "Feed" button on BottomNavigation, it can't be navigated back to the list, instead it just can't click anything.

@n1ru4l
When you navigate back from /app/feed/article/:id to /app/feed, the urls changes so match becomes null during the transition. You can use staticMatch to fix this issue :
<Card
  ...
  render={({ match, staticMatch }) => (
    <Text>{match ? match.url : staticMatch.url}</Text>
  )}
</Card>

I refactored <BottomNavigation /> and added a new <TabBarBottom /> to fix many bugs. It reduces number of redux logs, fixes flicker effect ...
Let me know if this solves your issue !
@chuyik
Thanks for your feedback. This should be fixed in 1.0.0-beta.3 (https://github.com/LeoLeBras/react-router-navigation/commit/4eccc23e0fe0556c9bdfab6b7013d8798994bc21).

@n1ru4l I will checkout staticMatch in a few minutes. I checked the performance of the Huge sample app and noticed that you have to change the dependency if react-router-navigation from to file:./../../ to v1.0.0-beta.3, otherwise I would get lots of @providesModule naming collision errors when running yarn install. Switching on the BottomNavigation seems much better now, however the Tabs animations are laggy (I am using android and atm I am unable to test on ios). I recorded a short screencast - I will tyr to figure out how I can attach it here :D

Hey @LeoLeBras: was this specific bug fixed in ß3 ?
@n1ru4l
Thanks for your time, appreciate the feedback! Did you try the last beta and still needed staticMatch?
About your screencast: if I'm not mistaken, you were in __DEV__ = true mode there. Try to turn it off and see if you notice any changes.

@CharlesMangwa Yes !!


[QUESTION] Unable to autocomplete commands

First check

[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Typer documentation, with the integrated search.
[X] I already searched in Google "How to X in Typer" and didn't find any information.
[X] I already searched in Google "How to X in Click" and didn't find any information.

Description
How can I have an autocomplete for commands?
Example
I have this simple app:
from typer import Typer

app = Typer()

@app.command(help="Check the code style.")
def lint(folders: List[str]):
    Linter.run(folders)

@app.command(help="Format according to the code style.")
def format(folders: List[str]):
    Formatter.run(folders)

Also I have an entry point in the setup.py:
    entry_points={"console_scripts": ["my_app=my_app.main:app"]}

But I'm unable to have autocompletion when I have something like this:
my_app li<tab here should add `nt`>

Looks like you need this page.
https://typer.tiangolo.com/tutorial/options/autocompletion/

I don't see how it is related... The problem is not the "option"

Oh, sorry. Sleepy in the morning. In my case, adding the command name in the decorator gave me command completions.
@app.command("lint")
def lint(...)
    pass

Hope it works for you.

Not working yet :worried:

For autocompletion to work you have to install it, with:
$ my_app --install-completion

Then you have to restart the terminal.
Check more about it here: https://typer.tiangolo.com/tutorial/package/#test-your-wheel-package
Also, it works for Zsh, Bash, Fish, and PowerShell. It won't work on cmd.exe in Windows, there's no way to support completion in it, that's why they deprecated it in favor of PowerShell.

Now, there's the additional package typer-cli (apart from typer). It creates a command typer that you can use with python scripts that use Typer, to run them, without having to install them as an app (very much similar to running with python directly).
But if you install completion for Typer-CLI, then when you use it to run your own scripts, e.g.:
$ typer run my_app.py

...it will have completion for those scripts, magically.

I'm almost sure it didn't work with the --install-completion and I just forgot to mention. But I'll check later. For now, I'm closing the issue, and once I have time I'll confirm.

Thanks! :rocket: :coffee:


When using cloudflare R2 hosting URLs incorrect

See response from image endpoint has https://https://
{
  "page_size": 50,
  "page_number": 1,
  "page_results_count": 1,
  "total_results_count": 1,
  "images": [
    {
      "name": "0_image_0.png",
      "url": "https://https://pub-c76bbc9f936b4cd3929aadc9c2afc70a.r2.dev/84a8ff1c-c463-5a97-b0c4-93daf7102b5f/files/0_image_0.png"
    }
  ]
}

it needs host name, not url... remove https:// from the env.
should be
R2_CUSTOM_DOMAIN=pub-c76bbc9f936b4cd3929aadc9c2afc70a.r2.dev
NOT
R2_CUSTOM_DOMAIN=https://pub-c76bbc9f936b4cd3929aadc9c2afc70a.r2.dev


502 errors caused by invalid timemap

Presumably this is caused by an invalid timemap, causes the response to 502
panic: runtime error: index out of range

goroutine 6 [running]:
main.extractMementos(0xc820504f00, 0xc820508e10)
        /go/src/github.com/oduwsdl/memgator/main.go:187 +0xd4c
main.fetchTimemap(0xc8200112c0, 0x13, 0xc820017b10, 0x7, 0xc820010f20, 0x13, 0xc82000b3e0, 0x2a, 0xc820010f00, 0x20, ...)
        /go/src/github.com/oduwsdl/memgator/main.go:273 +0x1357
created by main.aggregateTimemap
        /go/src/github.com/oduwsdl/memgator/main.go:374 +0xa0a

goroutine 1 [IO wait]:

This is not necessarily due to an invalid TimeMap, rather it's because of a case that was not taken care of. The panic is raised due to the following lines of the code:
kv := regs["kvaldlm"].Split(attr, 2)
linkmap[kv[0]] = kv[1]

In the above lines, a link attribute is split in name and value (0 and 1 indexes of an array respectively), then the value is added to a dictionary with the name as the key. It is possible that there is an attribute that does not have a value. This style of attributes is common in HTML attributes such as async, readonly, selected, and disabled etc. In such a case, existing code will have anything for the value or kv[0] which will cause this panic. Here is a test case that illustrates this issue.
However, the fix is simple. We just need to check if the kv slice has at least two elements before assigning it, otherwise ignore the attribute.


Custom Data Factory

It should be possible at this point to provide a class factory that will create given a certain set of parameters (or simply using a manipulator generator blueprint of sorts) to have Sponge generate a custom class with some unique identification that gives minimal code base requirements from plugin developers. Effectively, this can eliminate the need altogether for plugin developers to need to create at least 3 classes for each custom data themselves.
What it could look like:

public class MyDataStuff {

  public static final Key<Value<String>> FOO = KeyFactory.create(String.class, Value.class, DataQuery.of("myData"));

  public static Class<? extends DataManipulator<?, ?>> MUTABLE_FOO = CustomDataGenerator.builder()
    .id("FooData")
    .withString(FOO)
    .version(1)
    .create(PluginClass.getInstance());
  public static Class<? extends ImmutableDataManipulator<?, ?>> IMMUTABLE_FOO = CustomDataGenerator.immutableOf(MUTABLE_FOO);

}

The only issue of course is that usability of the manipulator as a class itself is completely useless since it's type erased to a wild carded class. The advantage is that what takes 3 lines of code from a developer is all it would take to get started using custom data relatively easily.
The disadvantage of this all is that of course the plugin would not be able to see the data class itself, unless in the implementation.
My idea is that the class name generated itself would utilize the plugins' id as a root and the ending being retrieved from the id provided though the generator.
I'm hoping that the community could see this for what it is as an addition and not a replacement of some serious custom data.
A note: The generator would only be able to accept some default serializable/supported objects like primitives, strings, and potentially other DataSerializables.

Pinging @Aaron1011 and @viveleroi as I've seen both to have some potential interest in such a thing being discussed.

I think this would be great. It's still a bit lengthy for situations where we're only storing a boolean or a single primitive value but it's quite a lot better than the three class files I had to a) learn and b) implement just to store a single boolean. Not only is this example more compact but it's a lot more understandable.

@viveleroi The only loss here though is that the factory will not be able to provide you with a nice generic class to use with data manipulators themselves. The fortunate part is that you'll still be able to generate the generic keys to interact with that way.

@gabizou withString? Why not just string? No other builders use with, and it'll be weird to start doing so

@kashike leave the nit picking of method naming when it's actually being PR'ed ;)

Yes. This is a good idea. I was thinking about something like generating the classes when I was doing a bit of refactoring to the data processors.

@simon816 It's something that can be done for simple custom data, but for supported vanilla data, class generation is definitely not the way we want to go about it due to tracking potential issues.


Add html tag

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
#173

@dood- Thanks!


Add methods to generate notable move categories

Fixes #6

The other lists don't contain fields that I can use to filter the correct moves; I have hypothesized that I can try and filter based on the shortDesc of moves, items, and abilities but will leave that for other PRs.


Add support for redshift connector

Add support for redshift connector  - depends on  https://github.com/finos/legend-pure/pull/483

We need to include integration tests :
https://github.com/finos/legend-engine/blob/master/legend-engine-executionPlan-execution-store-relational-connection-tests/src/test/java/org/finos/legend/engine/plan/execution/stores/relational/connection/test/ExternalIntegration_TestConnectionAcquisitionWithFlowProvider_Snowflake.java
https://github.com/finos/legend-engine/blob/master/.github/workflows/database-snowflake-integration-test.yml

Those will come in another version once we create the flow.


LWM2M / CoAP

Hello
Are there plans for a LWM2M and Coap client?

Project is discontinued

Project is discontinued


Access user collection

Hi,
I'm having a strange issue. I'm publishing my posts via Meteor.publishComposite. In my React Native app I can access every published data for that post (f.e. Comments) without problems, but when I try to access users:
   //After subscription is ready - works fine with Comments collection
   console.log(Meteor.collection("users").find());

I only get an empty array. This is our publish method:
Meteor.publishComposite("posting", function(id) {

    return {

        find: function() {
            return Postings.find({_id:id, isVisible:true,fileReady:true},{limit:1});
        },

        children: [{

            find: function(post) {
                return Meteor.users.find({_id:post.userId});
            }
        },

            {
                find: function(post) {
                    return Comments.find({postingId:post._id});
                },

                children: [{
                    find: function(comment) {
                        return Meteor.users.find({_id:comment.userId});
                    }
                }]
            }
        ]

    }
});

Do I miss something here?

This is weird.
I have never used publishComposite so I don't really know how it is going with this.
I will be looking into this soon.

Can you please post a fork of the example provided in this repo with this bug ? Thanks !

@Mokto Sorry for not answering this, but it seems fixed or was the mistake on our side. In our latest app, everything is working fine.

Hello,
In my app i can access all the collection but the users. Always returns null.
Client code:
export default createContainer(() => {
let terminals;
let subTerminals = Meteor.subscribe('terminals', Meteor.userId() );
if(subTerminals.ready()){
console.log("HIS READY"),
   terminals = Meteor.collection('terminals').findOne( Meteor.userId() );
}

return {
    status: Meteor.status(),
    user: Meteor.user(),
    loggingIn: Meteor.loggingIn(),
    terminals: terminals,
    tests: Meteor.collection('tests').findOne('6XJNau7tF5jZwnnTk')
  };
}, App);

Server code:
export default() => {
    Meteor.publish('terminals', (userId) => {
        console.log("userId", userId);
        return Meteor.users.find({_id:userId})
    })
}

My Meteor.user() in the client returns the user object but when I try to get a user through a publication. The test subscription (exactly the same just pointing to a "tests" collection) works fine.
Any help will be appreciated!
Thanks

I have a similar problem.
My published collection users only gives me access to the current logged in user, which seems alright to me according the following publish
Meteor.publish("users", function users() {
	return [
		Meteor.users.find(
			this.userId,
			{
				fields: {
					school: 1,
					emails: 1
				}
			}
		),
		Plannings.find({ ownerId: this.userId }),
		CounsellingProfiles.find({ ownerId: this.userId })
	];
});

Now the real trouble is, to test things I have a publish allUsers publishing all users like this
Meteor.publish("allUsers", () => Meteor.users.find());

But whenever in my react-native app I try to access allUsers, like this
Meteor.subscribe(
     'allUsers',
     error => console.log(error),
     () =>
       this.setState({
         users: Meteor.collection('allUsers', { cursoredFind: true })
           .find()
           .fetch(),
       }),
   )

I always get an empty array,
Any idea what could cause this ?


pysc2.bin.play dosen't work

When I run the command python -m pysc2.bin.play --map Simple64, I receive the following stack-trace.
Traceback (most recent call last):
  File "/home/rishav/anaconda3/envs/osrl/lib/python3.5/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/rishav/anaconda3/envs/osrl/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/rishav/Desktop/repo/pysc2/pysc2/bin/play.py", line 218, in <module>
    app.run(main)
  File "/home/rishav/anaconda3/envs/osrl/lib/python3.5/site-packages/absl/app.py", line 300, in run
    _run_main(main, args)
  File "/home/rishav/anaconda3/envs/osrl/lib/python3.5/site-packages/absl/app.py", line 251, in _run_main
    sys.exit(main(argv))
  File "/home/rishav/Desktop/repo/pysc2/pysc2/bin/play.py", line 154, in main
    controller.join_game(join)
  File "/home/rishav/Desktop/repo/pysc2/pysc2/lib/remote_controller.py", line 96, in _valid_status
    return func(self, *args, **kwargs)
  File "/home/rishav/Desktop/repo/pysc2/pysc2/lib/remote_controller.py", line 71, in _check_error
    return check_error(func(*args, **kwargs), error_enum)
  File "/home/rishav/Desktop/repo/pysc2/pysc2/lib/remote_controller.py", line 62, in check_error
    raise RequestError("%s.%s: '%s'" % (enum_name, error_name, details), res)
pysc2.lib.remote_controller.RequestError: SC2APIProtocol.ResponseJoinGame.Error.MissingOptions: 'Render interface not implemented.'

It seems as though the sc2client-proto dosen't implement render interface as I found it in one of the comments. If the blizzard API doesn't support it, I am wondering how to get real game output through pysc2.

Workaround: Pass the flag --rgb_screen_size 0 to pysc2.bin.play.
For example:
python -m pysc2.bin.play --map Simple64 --rgb_screen_size 0

Yeah but the real game interface stream is being bypassed by that. Dosen't solve the issue if you want to access the rgb rendering.

The game does support it, at least new versions. You may need to install libmesa.


Append fail, when Prometheus server is trying collect metric

I think this issue appear due to sanitising.

Link to source

Tool to check prometheus formatting [this blog post] (https://www.robustperception.io/invalid-is-not-a-valid-start-token-and-other-scrape-errors)

p1mac078:prometheus-2.12.0.darwin-amd64 k01136$ curl http://localhost:8080/metrics | ./promtool check metrics
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 27537  100 27537    0     0  1372k      0 --:--:-- --:--:-- --:--:-- 1415k
error while linting: text format parsing error in line 206: second HELP line for metric name "producer_node_metrics_request_size_max"

KafkaJmxExporter exports several metrics with tagKey node-id with single dash - and double dash--. These metrics registered twice in KafkaJmxExporter , and also twice when exporting. Current solution is to ignore metrics with double dash --.

#13 Fix in PR


Output file time fields are missing trailing numbers?

Using either dockerized or Windows version of BirdNet, and saving output in CSV format, the "Start (s)" and "End (s)" fields seems to sometimes miss trailing zero (or other number?)
Example, rows 2, 16 and 17:
1018.0,1021.0,Buteo lineatus,Red-shouldered Hawk,0.1006
102.0,105.0,Turdus viscivorus,Mistle Thrush,0.2857
1020.0,1023.0,Turdus merula,Eurasian Blackbird,0.1264
1020.0,1023.0,Fulica atra,Eurasian Coot,0.1195
1022.0,1025.0,Fulica atra,Eurasian Coot,0.5702
1026.0,1029.0,Drepanis coccinea,Iiwi,0.1028
1030.0,1033.0,Dog,Dog,0.1531
1030.0,1033.0,Turdus viscivorus,Mistle Thrush,0.1335
1030.0,1033.0,Corvus corax,Common Raven,0.1332
1032.0,1035.0,Anas crecca,Green-winged Teal,0.3920
1032.0,1035.0,Anser anser,Graylag Goose,0.1338
1034.0,1037.0,Anas crecca,Green-winged Teal,0.1263
1036.0,1039.0,Turdus merula,Eurasian Blackbird,0.4798
1038.0,1041.0,Anser anser,Graylag Goose,0.4178
1038.0,1041.0,Cygnus cygnus,Whooper Swan,0.1124
104.0,107.0,Turdus migratorius,American Robin,0.2337
104.0,107.0,Turdus philomelos,Song Thrush,0.1144
1040.0,1043.0,Anser anser,Graylag Goose,0.2485
1040.0,1043.0,Cygnus cygnus,Whooper Swan,0.1544

Problematic fields seem to systematically have larger value on the end field than the following row: e.g. on row 2 end value should maybe be 1050.0, which is larger than the end value 1023.0 on row 3.
Occasionally the time value is even shorter, like on row 2 here:
1198.0,1201.0,Anser anser,Graylag Goose,0.8534
12.0,15.0,Haemorhous mexicanus,House Finch,0.1176
1200.0,1203.0,Anser anser,Graylag Goose,0.9340

It seems that this issue is related to the way how Python sorts timestamps. My guess is, the timestamps are correct and refer to seconds 102 to 105 in the first example, and simply appear in the "wrong" line. Not sure how Python does the sorting, though. Maybe this behavior is different on Windows and Linux?

Not seen since, reopen if encountered again.


DayDecorator with multiple dots

My goal is to to display multiple coloured dots for different types of events that occur on a particular day, like in the image below:
https://www.dropbox.com/s/yflsbttl0slte4a/calendar.png?dl=0
Even though I can create my own EventDecorator to handle the drawing (that implements DayViewDecorator) there seems to be a flaw in the api design:
//this only allows you to return a boolean to decide whether the decorator view should be drawn or not
@Override
public boolean shouldDecorate(CalendarDay day) {
return dates.contains(day);
}
//when this is eventually called, there is no way for you to know what type/colour of dot to draw
@Override
public void decorate(DayViewFacade view) {
view.addSpan(new DotSpan(7, color));
}
Btw, this may be related to https://github.com/prolificinteractive/material-calendarview/issues/122
Is there a way to solve this ?
Perhaps, if the api also passed back the calendar day to public void decorate(DayViewFacade view) then we could use that to perform our logic?

Hey @damianflannery, early on the decorator API allowed you to decorate each day differently, like you are asking for, but it was rather slow and the only way to make it faster was to make it possible to cache decorations. That meant that we couldn't allow decorates to know which dates they were applying decoration to.
What you will have to do is create a new instance of a decorator for each date, or for each combination of dots. You would also need to create your own span to allow for multiple dots. This should still be efficient as long as you don't invalidate decorators frequently, but the burden is on you to make it work.

Someone solved?

I have a task to show text in calendar dates and dots also, but I couldn't find the way to resolve this problem
I don't want multiple dots on one date but I need to show a different color in different Dates with text. if the text is not possible then at least I want to show Span color in different Dates.
How can I deal with this library
https://github.com/prolificinteractive/materialcalendarview/blob/master/docs/DECORATORS.md
I got Dates From server and store in Bean classes and I am converting dates to according library Formate. But the problem is below code.
Please See Below
if(bookingList.get(i).getTimeperiod().equalsIgnoreCase("AM"))
{
String date[] = bookingList.get(i).getDate().split("-");
int year = Integer.parseInt(date[0]);
int monthOfYear = Integer.parseInt(date[1]);
int dayOfMonth = Integer.parseInt(date[2]);
widget.setSelectedDate(CalendarDay.from(year, monthOfYear, dayOfMonth));
widget.addDecorator(new EventDecorator(Color.RED,dates));
}
if(bookingList.get(i).getTimeperiod().equalsIgnoreCase("PM"))
{
String date[] = bookingList.get(i).getDate().split("-");
int year = Integer.parseInt(date[0]);
int monthOfYear = Integer.parseInt(date[1]);
int dayOfMonth = Integer.parseInt(date[2]);
widget.addDecorator(new EventDecorator(Color.BLUE,dates));
}
if(bookingList.get(i).getTimeperiod().equalsIgnoreCase("Full day"))
{
String date[] = bookingList.get(i).getDate().split("-");
int year = Integer.parseInt(date[0]);
int monthOfYear = Integer.parseInt(date[1]);
int dayOfMonth = Integer.parseInt(date[2]);
widget.setSelectedDate(CalendarDay.from(year, monthOfYear, dayOfMonth));
widget.addDecorator(new EventDecorator(Color.YELLOW,dates));
}
According to Code, I got AM, PM, FullDay Strings in JSON and I am changing span color with conditions but At last result of JSON convert my all widget Span color.

mayanksugandhi5 I also wants to bind same values, Any help would be appreciated.


SIP-4: DWN Routes

New DWN route for publishing a manifest, with dwn config for dwn endpoint
Input:
{{baseUrl}}/v1/dwn/manifests
{
  "manifestId": "WA-DL-CLASS-A"
}

Output
credentialManifest: {...}
published: true

Codecov Report

Merging #113 (dd6ae21) into main (0232fb7) will decrease coverage by 0.06%.
The diff coverage is 26.56%.

:exclamation: Current head dd6ae21 differs from pull request most recent head 1d4984e. Consider uploading reports for the commit 1d4984e to get more accurate results

@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
- Coverage   27.76%   27.69%   -0.07%
==========================================
  Files          16       17       +1
  Lines        1059     1123      +64
==========================================
+ Hits          294      311      +17
- Misses        723      769      +46
- Partials       42       43       +1

Impacted Files
Coverage Δ

config/config.go
11.76% <0.00%> (-0.74%)
:arrow_down:

pkg/server/router/dwn.go
16.66% <16.66%> (ø)

pkg/server/server.go
72.09% <75.00%> (+0.29%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


chore: beanstalk cleanup

Reviewers
Functional:
@MDunitz
Readability:

Changes

add
remove
modify

Should we also remove or rename the eb directory? (in a new branch/PR)

The EB AWS resource hasn't been destroyed. Is that future work?


docs: migrate wiki to gitbook

This migrates wiki documentation into a GitBook. It's in a sub-directory with its own package.json for future monorepo setup. A follow up PR will introduce a mechanism to publish. Then we can update README links and decommission the wiki.
A cool thing I learned that made this much easier than otherwise: you can grab the Markdown files and sidebar content from any GitHub wiki by cloning a secret repo with a .wiki suffix, e.g., git clone git@github.com:ryanricard/mockyeah.wiki.git.
Fixes #51, related to #70.

@ryanricard I can't setup publishing to GitHub pages myself without admin permissions. Up to you at this point.

It would be nice keep the url similar to the repo url, whatever that takes.

@AndersDJohnson your saying you can setup the site with your collaboration rights, correct

@ryanricard I don't know if I can - you need to setup GitHub pages in the repo settings to use a gh-pages branch.


Upgrade dependencies

native-tls 0.2
websocket 0.21

This resolves an OpenSSL compilation error when using the latest OpenSSL
version.

Looks stale. Still relevant? :slightly_smiling_face:


I can not find 'signnames.csv'.

First, I try to run inference.py
$ python inference.py -m demo
.
.
Traceback (most recent call last):
File "inference.py", line 271, in
generate_output(input_files, mode)
File "inference.py", line 188, in generate_output
with open('signnames.csv', 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'signnames.csv'
I can not find 'signnames.csv' in your git.
Is that in 'LISA Traffic Sign Dataset' ?

I download 'LISA Traffic Sign Dataset'.
But I can not find signnames.csv in that.

Thanks for the catch, I forgot to include signnames.csv in this repo. I have just added it. It is basically a hand-generated file (or can be generated with simple script) that maps sign integer ID to sign name string, but reserving ID 0 for background (no need to explicitly map 0->background though). Let me know if inference.py works now. Thanks.

Thanks, you added signnames.csv file.
I can run "inference.py -m demo". It is very nice!.
But my GPU card is poor, so tensorflow says 'Ran out of memory trying to allocate 2.14GiB.'.
ssd-caffe (by Weiliu) could not train on my system. ( Out of memory?).
mxnet-ssd( by Zhreshold) could but training time is very huge on my system ( Poor GPU?).
ssd_keras(by rykov8) could not train for lost training data set(framexxx.png).
After I will try training to use your system.
Thank you for sharing your software!

Cool, sounds good! I haven't documented the data pre-processing steps, so you might have to look through the code to figure it out :S But my base network is AlexNet so training should not take so much resources compared to the original VGG16 base network. Also if you run out of memory a work-around is to reduce your batch size.

Very very thanks!
I try to run inference.py not only png files but also mp4 moview files.
It works fine!
So, now I tried to run data_prep.py for training.
I think it make pickle data for train.py
But, this python program need pickle data too. It it true?
if __name__ == '__main__':
        with open('data_raw_%sx%s.p' % (IMG_W, IMG_H), 'rb') as f:
                data_raw = pickle.load(f)

        print('Preparing data (i.e. matching boxes)')
        data_prep = do_data_prep(data_raw)

        with open('data_prep_%sx%s.p' % (IMG_W, IMG_H), 'wb') as f:
                pickle.dump(data_prep, f)

        print('Done. Saved prepared data to data_prep_%sx%s.p' % (IMG_W, IMG_H))
        print('Total images with >=1 matching box: %d' % len(data_prep.keys()))

I think this code need data_raw_400x260.p file.
(....I wonder it was generated by AlexNet traing time?)

Yup, I haven't documented that part in detail, I'll work on the documentation this weekend~

Oh!. don't worry!
My ubuntu PC 's tensorflow environment is broken and needs to be repaired.
So, if only inference can be on my windows PC.
And, thank you for your quick response!

I added some instructions to the README, about how to train from scratch. Let me know if that helps!

Thank you.
But, my tensorflow environment is not good....
And Google Release tensorflow version 1.0.
Your inference.py is not work under my environment now.(Last week, it worked last week..)
So, I revise the environment of tensorflow to 0.12 and try your training python script.
Regards,


Allow model feedbacks into the application

Is your feature request related to a problem? Please describe.
I have some trained sklearn model, which I use in production. I want to feed its predictions back into refinery, as I would love to use this information to find mismatches between production inference and weakly supervised labels, to sort by its confidence etc. (Basically apply the same methods as for weak supervision)
Describe the solution you'd like
I want an extension of the SDK, so I can program any external model into my workflow.
Describe alternatives you've considered
Adding the model predictions as attributes to my records, which would be a workaround.
Additional context
-

Solved for classification tasks, added new issue #91 to keep this for extraction tasks.


New dockerfile and start script

Description
This PR implements a new Dockerfile and docker init script.
Instead of pulling the repository, the new dockerfile uses COPY to generate the context for the build.
The new script allows for starting different parts of licode as well as starting the whole thing.
You can for instance run it with --nuve to only start nuve. For now, no further customization of the configuration is provided.
For users of the older Dockerfile
This new Dockerfile is located in the root of the repository to be able to generate the context via COPY
Also, CMD has been changed to ENTRYPOINT to make it easier to pass parameters to the init script. Remember you have to use --entrypoint to change the entrypoint instead of just passing the script as a parameter.
[] It needs and includes Unit Tests
Changes in Client or Server public APIs
Add a detailed description of any change in the public APIs.
[] It includes documentation for these changes in /doc.

@jcague @aalonsog
I've gone ahead and added the doc changes here so everything stays synchronized.


WP-API 2.x

http://v2.wp-api.org/

Hi,
are you going to add support for wp rest api v2?
i want to know if i can pull data from posts,page,custom post types,users,menus,metadata and etc..?
Thanks

Yes support for v2 is scheduled. I have already prepared the client lib for angular https://github.com/shprink/wp-api-angularjs. Just waiting for wp-api v2 to be released.


Fixing #15

This change solves mostly everything from issue #15.
It removes unnecessary copy-paste code from source-map-support. Instead of that it uses it as a package, so we no longer need source-map package.
To achieve this I used retrieveFile and retrieveSourceMap options of source-map-support with our rewritten handlers that are based on original ones but without unnecessary checks (like isInBrowser or is source map inlined) and with extra logic for transpiling TypeScript.
The only thing I'm unsure is using of cache - it requires extra investigation, but it would be tracked in another issue.

Further discussion about cache is in issue #30.


IRGen: Map Int and UInt to Clang types via NS[U]Integer.

If NSInteger and NSUInteger typedefs are available in the current Clang context, use them when mapping Swift's Int and UInt to Clang types. This gives us consistent method and property type encodings for @objc methods with Clang, fixing rdar://problem/17506068.

@swift-ci Please test and merge


update packages

:white_check_mark: Build TheIdServer 7.4.4-fix-dependencies1-0076 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/2dbce933cc by @invalid-email-address)


Invoking karate.range throws a TypeError exception

The issue is reproducible using the example from the documentation:
Feature: Use karate.range

Scenario: test karate.range

    * def foo = karate.range(4,9)
    * match foo == [4, 5, 6, 7, 8, 9]

It produces the following exception:
* def foo = karate.range(4,9)
>>>> js failed:
01: karate.range(4,9)
<<<<
org.graalvm.polyglot.PolyglotException: TypeError: invokeMember (range) on com.intuit.karate.core.ScenarioBridge@907f2b7 failed due to: Unknown identifier: range
- <js>.:program(Unnamed:1)

@treboriwa works for me. you must be on an old version. follow this process please: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue

@ptrthomas My bad. I was running with an experimental version I built from source. It works as expected with 1.0.1. Thanks!


SAK-30711 - Update Forums grading to use the new getAssignmentByNameOrId methods

Update from #4450 to use separate methods to avoid any conflicts in tools not yet migrated and call the numeric lookups after the name lookups.

I'm closing this for a second again, this patch has a bug with regrading that the other didn't.


Error on RPi during withdrawal from Bitvavo

Hello Jorijn,
First of all, thanks for making this script available! And I used your affiliate link on Bitvavo.
Unfortunately, I am getting an error running this command on RPi:
$(command -v docker) run --rm -it --env-file=/home/pi/Docker/bitcoin-dca/.bitcoin-dca ghcr.io/jorijn/bitcoin-dca:latest withdraw --all --yes --no-ansi
Error:
PHP Fatal error:  Uncaught TypeError: Argument 2 passed to Jorijn\Bitcoin\Dca\EventListener\XPubAddressUsedListener::__construct() must be an instance of Jorijn\Bitcoin\Dca\Component\AddressFromMasterPublicKeyComponent, instance of Jorijn\Bitcoin\Dca\Component\ExternalAddressFromMasterPublicKeyComponent given, called in /app/var/cache/container.php on line 509 and defined in /app/src/EventListener/XPubAddressUsedListener.php:28
Stack trace:
#0 /app/var/cache/container.php(509): Jorijn\Bitcoin\Dca\EventListener\XPubAddressUsedListener->__construct()
#1 /app/var/cache/container.php(174): BitcoinDcaContainer->getEventListener_XpubAddressUsedService()
#2 /app/vendor/symfony/event-dispatcher/EventDispatcher.php(267): BitcoinDcaContainer->{closure}()
#3 /app/vendor/symfony/event-dispatcher/EventDispatcher.php(230): Symfony\Component\EventDispatcher\EventDispatcher::Symfony\Component\EventDispatcher{closure}()
#4 /app/vendor/symfony/event-dispatcher/EventDispatcher.php(59): Symfony\Component\EventDispatcher\EventDispatcher->callList in /app/src/EventListener/XPubAddressUsedListener.php on line 28
I already found an issue with a similar error but it was related to Kraken.
Thanks for helping me!

Hey @orosseel
Thanks for using my affiliate link!
I am very sorry, as it seems a minor bug crept into the code, which went unnoticed for quite some time. I just pushed a patch that should solve your problem. I will create a new version later today. If you follow the upgrade instructions to bump your local image to the latest version, you shouldn't experience this error.
Best of luck, DCA'ing!
Jorijn

v1.6.1 contains the fix :-)

What a quick response and fix! Thanks, @Jorijn!


New version: Tencent.QQ.NT version <IP_ADDRESS>909 (<IP_ADDRESS>808)

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate of #167508;


$add silently ignores action if record is also in another collection.

Hello,
I ran into the situation where there were two models: "House" and "HouseGroup".
I fetched a list of houses using House.$search();, then took one of the records in that collection and $add-ed it to a HouseGroup (which has a hasMany relation to House).
The action fails silently because that house was already in the House collection.
This is the code in the library that causes it:
$add:` function(_obj, _idx) {
  Utils.assert(_obj.$type && _obj.$type === this.$type, 'Collection $add expects record of the same $type');

  return this.$action(function() {
    if(_obj.$position === undefined) { // <--- Here's the problem.
      if(_idx !== undefined) {
        this.splice(_idx, 0, _obj);
      } else {
        this.push(_obj);
      }
      _obj.$position = true; // use true for now, keeping position updated can be expensive
      this.$dispatch('after-add', [_obj]);
    }
  });
},

Proposed solution: allow records to be in multiple collections, or document why this is problematic and throw an exception instead of failing silently.

Sent in a pull request to fix it, but I'm not sure if those changes are correct within the overall design of the library.
I would argue that it is, since I think records should be allowed to be in multiple collections.

Suggestion about why its' probably done this way:
Objects currently have a reference to the collection they're in. If you would make it possible to add it to multiple collections, the object would have references to multiple collections.
If one of those collections is no longer needed, it would normally be destroyed. However, there would still be a reference to that collection through the object that is in multiple collections, therefore resulting in that collection persisting, thus leading to a memory leak until refrences to all collections and all objects in those collections are dropped.
Proposed solution: use a WeakSet or some polyfill for it to store the back-reference to the collection.

I'll get to work on a PR.


#371 Integrate Circuit Breaker with Http Client

Thanks for your pull request.  Please review the following guidelines.

[x] Title includes issue id.
[x] Description of the change added.
[ ] Commits are squashed.
[x] Tests added.
[x] Documentation added/updated.
[x] Also please review CONTRIBUTING.md.

Addressed the review comment for the first batch.. The only discussion point is FiniteDuration.apply.  I have changed the code to use FiniteDuration.create, this is the Java API.  I prefer this to align with other standard Akka Stream stages.

I'm OK with using FiniteDuration.apply as a conscious decision based on the given rationale. Don't worry.

This PR actually addresses https://github.com/paypal/squbs/issues/373 as well.


Conflict build process when used with build_runner

I was trying to do JSON parsing and the recommended way uses build_runner. But after build_runner ran, the asset.dart file is deleted.
Running flutter generate again will not regenerate asset.dart.
Currently just doing JSON parsing by hand instead of the recommended way but would be great if we can solve this.

I think the manual method of generation still works:
flutter packages pub run r_flutter:generate --intl-file="lib/resources/i18n/de.arb"

it would be nice to document it. Perhaps it could also use config from pubspec instead of command line arguments (intl-file, etc.).

Seems to work when you add it to dev_dependencies instead of builders.
dev_dependencies:
  flutter_test:
    sdk: flutter
  build_runner: 1.7.2
  r_flutter: 0.4.1

@kuhnroyal I just tried that and flutter generate no longer generate the correct files.

@hiaw correct, if you want to run from command line (the old method), then there is no point in calling flutter generate.

@hiaw correct, if you want to run from command line (the old method), then there is no point in calling flutter generate.

Sorry I must have missed something. There's a new method? Or do you mean the manual method you commented before? I only see flutter generate in the ReadMe.

There are multiple ways of doing code generation in flutter:

Adding package to builders:  - r_flutter currently uses it.
build_runner, used by json_serializable.
Command line utility - before r_flutter used method 1, it used this.

As you have noticed, methods 1 and 2 do not play nicely with each other, so you can switch to method 3 for r_flutter. Readme does not mention it, but it still works.

Sorry for the late response.
For now I had no luck making the generation work with build_runner as well.
https://github.com/timfreiheit/r_flutter/pull/15 will document this and align the command line usage to the normal generation process to have the same configuration step using the pubspec.yaml
I hope this helps for now.

One problem with manually generating the file is that people will want to commit it, and then absolute paths in asset docs do not look great.
I tried to find a solution for VSCode but found nothing. I created a feature request to support relative paths, but it may be never implemented: https://github.com/microsoft/vscode/issues/86564
The same problem probably exists in Android Studio.

I agree with commiting the generated file, but actually I dont see the need to have absolute paths in the comments of the generated file. For image paths VSC's Dartcode plugin will give you a gutter preview of the image using only a relative file path in the actual static's value without needing the code comment at all.
In regards to using builders need to note that that functionality is currently broken and the breakage has made it all the way into current stable channel release: https://github.com/flutter/flutter/issues/42003

Sorry I stand corrected, looks like the general recommendation is not to commit generated files. I do wonder then about requiring a build step for then running the code generator when building in a CI system, but perhaps thats taken care of by Flutters new builders: once the current bug is fixed.


Add RFC 5080 message authenticator support to the authenticator (RADIUS client) in core

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.

Is your feature request related to a problem? Please describe.
In #4306 we just added RFC 5080 message authenticator support to the FreeRADIUS plugin. It would be nice if the client implementation in core had the matching feature.
It's out of my league to implement this change in the core code. I have learned to find my way around most of the MVC based modules, but this file - no idea, sorry.
Describe the solution you'd like
A single boolean option to enable message authenticator in src/opnsense/mvc/app/library/OPNsense/Auth/Radius.php
Describe alternatives you've considered
None.
Additional context
https://forum.opnsense.org/index.php?topic=42094.0
Kind regards,
Patrick

I think the Message-Authenticator extension is actually described in RFC2869 (https://datatracker.ietf.org/doc/html/rfc2869#section-5.14), looking at our implementation, which uses pecl-radius (https://pecl.php.net/package/radius), it doesn't look feasible to add it as in order to calculate the HMAC-MD5 you need to be able to read the entire Access-Request packet (as far as I could find, there are no handles for that).
If someone does want to spend time on this subject, the easiest option is likely to add a new RADIUS authenticator using some pure php implementation, inspiration might be found at the following sources, which may or may not have their own limitations (the ones I ran into):

https://github.com/dapphp/radius/tree/master
https://codeberg.org/fkooman/php-radius/src/branch/main

Is this related to this "enhancement":  https://support.microsoft.com/help/5040268 ???

@stumbaumr Yes, that's the motivation.
There's at least one user on the forum who uses OPNsense as the RADIUS client/NAS ("network access server") with AD as the RADIUS server and who would greatly appreciate the feature.
@AdSchellevis Thanks for the follow up.
According to Microsoft's document linked by @stumbaumr it's not critical if it's missing as long as you control the infrastructure/network between the NAS and the RADIUS server. That might not always be the case but in an enterprise or service provider data centre generally should be.
No wonder I could not find how to build it myself by browsing the docs and the source code ;)
Kind regards,
Patrick


Avoid duplicate OBJECT_IDs after MVU

Signed-off-by: Jungkook Lee<EMAIL_ADDRESS>Description
PG allows duplicated OIDs in different catalog tables. However, in Babelfish, we use PG's OID as OBJECT_ID in all system catalog views and it should be unique within a logical database (https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/relational-databases/system-catalog-views/sys-all-objects-transact-sql.md). Duplicate OIDs can break SQL server tools, e.g., SSMS, as well as customer applications.
In PG/Babelfish, duplicate OIDs can occur in the following two cases:

After MVU (Major Version Upgrade)
After hitting OID wraparound

This commit is to prevent duplicate OIDs during MVU and consists of two parts:

First of all, from the old cluster, it finds the max OID in the 5 catalog tables: pg_extension, pg_authid, pg_enum, pg_class and pg_type. And then pass the max OID to the new cluster as a form of GUC, babelfishpg_tsql.dump_restore_min_oid. In the new cluster, set the nextOid greater than the max OID from the old cluster using GetNewObjectId_hook.

Secondly, in pg_upgrade.c, it skips the step of transferring the next OID from the old cluster to the new cluster.

Also note that the type of babelfishpg_tsql.dump_restore_min_oid is defined by DefineCustomStringVariable() to support UINT_MAX.

Also, this commit contains minor improvements on the pg-upgrade Github action and dev-tools.sh:

pg-upgrade Github action: add retain option to see pg_upgrade log even if pg_upgrade succeeds and jdbc tests fails.
dev-tools.sh: developers can choose the migration mode, e.g., ./dev-tools.sh test input multi-db.

Engine PR: https://github.com/babelfish-for-postgresql/postgresql_modified_for_babelfish/pull/54
Issues Resolved
BABEL-3613
Test Scenarios Covered

Use case based - Yes
Boundary conditions - Yes (babelfishpg_tsql.dump_restore_min_oid)
Arbitrary inputs - N/A
Negative test cases - N/A
Minor version upgrade tests - Yes
Major version upgrade tests - Yes
**Performance tests -**N/A
Tooling impact - N/A
Client tests - N/A

Check List

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

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

Let's include new test template in the PR description.

Looks good to me!


Hello-microverse-project | Pull Request

Milestone - Project Completed
In this pull request I implement the following features:

Added a .html file that includes a header with the text "Hello Microverse!"
Added a .css file that change the color of the header to dark cyan

Hi @iambenkis, thanks for your review!

I have a question, should I request a review again?
I already do the changes on the PR summary, I think that now it's a lot better, thank you for letting me know what to improve!

Yes
On Wed, 11 Jan 2023 at 12:16 AM Nicolas Olaya @.***>
wrote:

I have a question, should I request a review again?
I already do the changes on the PR summary, I think that now it's a lot
better, thank you for letting me know what to improve!
—
Reply to this email directly, view it on GitHub
https://github.com/NicolasAndrehh/hello-microverse-project/pull/1#issuecomment-1377966267,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATF3DV347JPM4EGHAQXLXRTWRXNUNANCNFSM6AAAAAATXGE4I4
.
You are receiving this because you were mentioned.Message ID:
@.***>


[tests] test additional_context passed to get_context

[tests] test additional_context passed to get_context
Now that we've added support for additional_context in gptty.context:get_context(), we should add some tests where we bootstrap additional context to the three different cases and validate the structure of the returned context, as well as its length.

Originally posted by @signebedi in https://github.com/signebedi/gptty/issues/37#issuecomment-1496220128

Added in 5891bb7.


Command to open raw user settings.json, not in split view

We have a setting to disable the settings split view, but it might be useful to have a command to open the user settings.json outside of the split view for a one-off.
From https://github.com/Microsoft/vscode/issues/41040#issuecomment-387479618

@usernamehw I am reverting the PR, it actually doesn't work. We intercept the editor open here: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/preferences/common/preferencesContribution.ts#L83

Damn, it's already a part of my workflow 😢. What's wrong with it?

So it opens the raw settings.json outside of the split view for you? For me, it actually just acts the same as the normal open settings command and opens the settings split view.

And you don't have "workbench.settings.openDefaultSettings": false?

I don't have "workbench.settings.openDefaultSettings": false. The command itself works for me. Does it break some other functionality?

OS version: Windows_NT x64 10.0.17134

I see now. Opens a split view on linux VM. Was it ever working before on non-Windows environments?

Yeah it only works on Windows, because the == check with fspath can fail due to drive letters being different. I guess that's a separate bug... I'll revert this tomorrow.

We got a good bug out of this: #50760

Now that the settings UI exists, I don't plan on investigating much more on the JSON editor. The split JSON UI will probably go away at some point (the JSON files themselves will always remain)

We will always allow users to see the json file. Hopefully you can still open the raw json file with "workbench.settings.openDefaultSettings": false.

I know about that option, it's not great.

I'll close this just because I am cleaning up issues that are specific to the JSON split editor. But to be clear, editing the JSON file will never go away :)


Docz with TSX and React JS

Hey everyone. i'm trying to use docz and have my props display. Currently its working with all my JS components. However, i just added my first TSX component which means in my doczrc.js i added   typescript: true into the file. This made my TSX File display correctly along with the props. However, all my other JS components don't display the correct Props anymore. I'm assuming this is because the props are in different places between a JS component and a TSX component and once i do typescript: true, it searches for the props in the interface of a typescript file (which isn't in the normal react JS components). Is it possible to have both TSX and react JS components in the same Docz environemnt? and if so, does anyone know how to set it up? Thanks!

Hey @yehbrian, thanks for reporting this.
Support for both TSX and JSX React components at the same time is not currently possible. However we're planning on updating react-docgen to the latest version that supports both TSX and JSX parsing.
Will keep this in mind when we update to make sure your use-case will be solved 👍
Happy to help if someone would like to submit a PR.


[FEA] profileUI: Add RAPIDS Tab to SHS

Part of the integration between RAPIDS plugin and Spark History Server (#4808)
Requires adding new packages to Rapids-tool jar and modifications to profiling tool in order to integrate initialization of the tool with SHS.

Due to change of priorities, this will be on hold for now.
Instead, the UI for qualification tool will get the main focus for this quarter.
I investigated SHS and the possibility of triggering RAPIDS profiler from the UI.
I wrapped up the code changes in a PR on my fork https://github.com/amahussein/spark-rapids/pull/1
Known issues:

I found a bug in processing logs that are in the format of zstd inside a zip. The code was assuming that a decompressed file would be available once the zipstream is opened. this will cause an exception reading the lines from the zipstream.
It will be good to save the ApplicationInfoSummary to the KVStore. This will be an optimization issue.
Code needed some refactoring to reduce redundancy.

Summary of the process implemented in https://github.com/amahussein/spark-rapids/pull/1
The approach relies on using SHS to report the output of the profiling tool.

first step is to implement a SHS plugin that extends AppHistoryServerPlugin
implement new classes to load a SparkUI tab and pages.
To SHS with support of the new plugin: Add rapids-tools jar to the SHS classPath, then restart the SHS. The SHS should start with a new Tab that has "RAPIDS".
The new tab shows summary of the Rapids-profiling output of the current app.
Navigating to the "Compare" tab will list all the applications through the SHS. Note that this is done through REST call since a SparkUI tab doesn't have access to the full list of the applications.
In this tab, clicking "compare" will navigate to a new page that shows the output of comparison between the current App and the selected one.


Better structure of website

I created better structure of the website. Visually is almost the same, but it now contains more valid HTML.
Some changes are:

Changing "chrome extension" to "browser extension"
Adding quotation marks when they are missing
Replacing RawGit URLs with relative ones
Better indentation

Did you ran it through a prettifier or did all that manually?
I don't use quotes wherever I can do the page is smaller and cooler (and also valid as per html5), but I guess the world it's not ready for that yet.

@fiatjaf For some things, I used prettifier and some things manually.


Parsing error: unknown token

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.57.0
What version of eslint-plugin-astro are you using?
0.33.1
What did you do?

Configuration
import eslintPluginAstro from "eslint-plugin-astro";

export default [
  // add more generic rule sets here, such as:
  // js.configs.recommended,
  ...eslintPluginAstro.configs["flat/recommended"],
  {
    rules: {
      // override/add rules settings here, such as:
      // "astro/no-set-html-directive": "error"
    },
  },
];

<template name="submission-success">

      Thank you, your download is ready.

    <button
      class="h-10 p-2 mr-auto rounded-md btn btn-primary min-h-10"
      onclick="window.location.href = `/some-other-location`">Download</button>
</template>

What did you expect to happen?
Parse with or without errors
What actually happened?
  39:9  error  Parsing error: Unknown token at 1260, expected: "name", actual: "e>\n<templa"

Link to Minimal Reproducible Example
https://github.com/endymion1818/test-astro-plugin-eslint-bug/tree/main
Additional comments
I appreciate your time, if there's anything I can do to make your debugging easier please let me know.

Thank you for this issue!
Hmm. It seems to be a problem with the AST returned by the astro compiler. Parsing the component below returns an incorrect AST.
---
---
<template name="foo">

</template>
<template name="bar">
</template>

The resulting AST returns an AST with the following structure:
---
---
<template name="foo">

  <template name="bar">
  </template>
</template>

The workaround is to put
.
---
---
<template name="foo">

</template>
<template name="bar">
</template>

Ah darnit. Thanks @ota-meshi, my bad.


sortType: "datetime" doesn't work

Describe the bug
When adding sortType: "datetime" to a datetime field (initial data comes from mongodb, formatted in the accessor using dayjs), the sorting crashes
Your minimal, reproducible example
https://codesandbox.io/s/hopeful-snowflake-m7985k
Steps to reproduce

Click on the date header column
crash happens

Expected behavior
Should filter the column accordingly
How often does this bug happen?
Every time
Screenshots or Videos

Platform
Codesandbox
react-table version
7.7.0
TypeScript version
NA
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

It must be a date object, not a string

Thanks @tannerlinsley for the light !
One (last) question :
How can I, at the same time, return a primitive to the accessor and a date object to the sorting plugin through the sortType: "datetime"

The accessed value is what is passed to the sorting interface.

yes that was my issue, I had to dig into the object return by each rows. Thank yo 🙏


BUG: Dark mode not working properly.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When the dark mode is enabled, the icons in navbar menu are not being visible properly. Also the arrows in carousel part will look good if they turn white in dark mode.
What happened?
Some content is not changing according to theme toggling.
Expected behavior
Changing the code such that dark mode works properly.
Screenshots

Please assign me this task, I would like to contribute to this project under GSSOC'24.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hi @arghadipmanna101 , because of new PRs,  the complete UI has been changed and there is no dark mode option available in the website. Should I raise a new issue and add the dark mode again?


Print worktree log to stderr

What does this PR do?
Prints worktree info log to stderr to avoid making this workflow fail.
Motivation
Describe how you validated your changes

Possible Drawbacks / Trade-offs
Additional Notes

/merge


Is there any option for Supplier management/ Vendor assessment through Nautobot

As we are using the Nautobot tool.
We have a requirement for Vendor risk assessment questionnaires / Vendor management. Is there any option available in Nautobot to achieve this requirement?

Good morning @arunqa!
I see a related discussion opened around vendor management: https://github.com/nautobot/nautobot/discussions/1974
At this time there is no planned feature in core to support this information but this could be a great candidate for a plugin.


Incorrect CRPIX Keyword in UV Weight FITS Output

It seems that size of UV weight images is always half of the primary beam and intensity images, but the reference pixels of the former are set to be the same as the reference pixels of the later. This makes the reference pixels fall outside of the images as supposed to be at the center.
Please see /data4/piyanat/fhd_images/test on ASU enterprise.

The UV weight FITS image header has been modified to include parameters relevant to u-v space.


[Question / Suggestion] Case sensitive placeholders

Hi there !
I'd like to know if there is a way to handle upcased placeholders. I currently run into an error for unrecognised command.
e.g. {{FOO.BAR}} isn't understood as {{foo.bar}}
Context: Customers can generate template themselves. Some of them mess around and send documents with upcased placeholders.
I didn't find an option like: caseSensitive: false which could be useful imho 🤷♂ .
Any idea ? Thanks !

Hah, I have the same problem... Why can't those pesky customers just do it right?? 😂 Haven't gotten around to implementing a solution yet, though.
Core of the problem is that everything between the command delimiters is essentially javascript syntax interpreted by a javascript VM. It is case sensitive to the extent that javascript is...
One solution would be to lowercase both the provided data object and each command before interpretation. This is not as trivial as it sounds, though, because commands can contain arbitrary javascript that cannot just be lowercased and expected to work correctly.
If you're just worried about ALL CAPS or all lowercase commands (and don't care about mixed-case commands) you could just create copies for all keys in your data object to have both lowercase and uppercase versions.
Let me know if you think of something, though.

I didn't really wanted to duplicate each of my values since it represent A LOT of values and it can be deep so I'm currently trying a workaround using a custom error handler :
createReport({
  ...,
  errorHandler: (err, command_code) => {
    if (err instanceof ReferenceError) {
      const args = command_code?.toLowerCase().split('.')
      const value = args?.reduce((acc, curr) => acc?.[curr], data)

      if (value) return value
        else throw err
      } else {
        throw err
      }
  }
)


No read QR with logo

http://prntscr.com/180es9p no logo
http://prntscr.com/180evpw with logo

Hi @andrtechno unfortunately when using a logo I can not guarantee readability. However, you can adjust the size, error correction level, logo size etc to make it more readable as described on https://github.com/endroid/qr-code#readability and you can use the built-in validation reader to check if the QR code is at least readable by the library itself. However that does not guarantee any other reader (they are all somewhat different) will be able to read the QR code.

@endroid https://www.qrcode-monkey.com/
reads normally. generates different QR with and without logo

it generates QR outside the block with the logo, and you seem to overlap the top


Fall back to .gitignore if no .cfignore exists

Command
cf push
What occurred
If no .cfignore file is present then entire directory (minus the exceptions) are pushed to CF.
What you expected to occur
If no .cfignore file is present then the cli falls back to use the .gitignore as a substitute meaning developers only need to maintain one ignore file.
CLI Version
All
CC API Endpoint Version
All
CF Trace

Platform & Shell Details

Any other relevant information
This practice is widely adopted in other platforms that have their own ignore file (Heroku, AWS etc...)

Hey there @edshee,
Thanks for the request, however, we explicitly decided to have the .cfignore separable from the .gitignore and want to continue doing that going forward. While I understand the convenience, we always want the user to be explicit when doing a CF push, especially since there is no switch to ignore the .cfignore file. By falling back to the .gitignore file when the .cfignore file does not exist, the user might have issues pushing files that they don't necessarily want to commit. For these reasons, we're not going to fall back to using the .gitignore.
Let us know if you have further feedback, but we don't believe that a fall back is the right approach.


Postgres timezone without microseconds

If I want to work with DateTime? (not DateTimeTz).
The difference are in the "microseconds":
DateTime:
2016-08-04 14:34:34
DateTimeTz:
2016-08-04 14:34:34.367606
If I use:
$table->addColumn('due_date', 'timestamp', array('timezone' => true));

I get:
ADD COLUMN due_date timestamp without time zone

But, If I want DateTime (without microseconds) I must to do:
ADD COLUMN due_date timestamp(0) without time zone

But now way to do that with Phinx.
Maybe if add the option "limit" in type "timestamp", this way I can add the numeric option in timestamp.
eg:
$table->addColumn('due_date', 'timestamp', array('timezone' => true, 'limit' => 0));

Postgres has a variable return format for the datatype TIMESTAMP(n) and TIME(n) if microseconds are allowed (n > 0). Whenever you save a value with microseconds = 0. PostgreSQL will return this value in the format:
2010-10-10 10:10:10 (Y-m-d H:i:s)

However if you save a value with microseconds it will return the full representation:
2010-10-10 10:10:10.123456 (Y-m-d H:i:s.u)

Using the DateTime, DateTimeTz or Time type with microseconds enabled columns can lead to errors because internally types expect the exact format ‘Y-m-d H:i:s’ in combination with DateTime::createFromFormat(). This method is twice a fast as passing the date to the constructor of DateTime.

#955
https://github.com/celorodovalho/phinx/blob/cfaf146651622bec82258727a94c452bd2ca01e4/src/Phinx/Db/Adapter/PostgresAdapter.php#L915-1105


Update README.md

Which branch your pull-request should merge into?

dev: Current 2.x BugFixs or Features
1.x: 1.x BugFixs

Describe your changes
see #1602
Issue ticket number and link
#1602
Checklist before requesting a review
For each PR

[ ] Add Copyright if it missed:
- "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

[x] I have performed a self-review of my code.
Optional:

[x] I have checked readme and add important infos to this PR.
[ ] I have added/adapted some tests too.

For core/new feature PR

[ ] I have checked readme and add important infos to this PR.
[ ] I have added thorough tests.

cloesed it (make one PR for all)


[BEAM-12386] [DO NOT MERGE] Revert "[BEAM-3713] Move validatesRunnerBatchTests and validatesRunne…

…rStreaming tests from nose to pytest (#14788)"
This reverts commit 02a2222e6cbdd3fa84217ed1c6b98af416cf2eda.
Just want to test if this fixes anything.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

run seed job

Run Python Dataflow ValidatesRunner

Run Python Dataflow V2 ValidatesRunner

Run Python 3.6 PostCommit


Add an example body to the configuration

Seeing as we already have randomized data that should not a risk for privacy;
It would be nice if one can have a more readable response and body. Meaning instead of strings showing up as {"foo": "string"} it could show up as {"foo": "bar"}

Data provided since v2.x, applied in the PathData since v2.3.x and actually applied in later release using commit: 3009afa2e76e38803bd08b4e08ad815059fcda58


Error in in zonal_stats.ipynb

When running the example proposed in the Aggregation options section:
aggregated_custom = ds.xvec.zonal_stats(
    world.geometry,
    x_coords="longitude",
    y_coords="latitude",
    stats=[
        "mean",
        "sum",
        ("quantile", "quantile", dict(q=[0.1, 0.2, 0.3])),
        ("numpymean", np.nanmean),
        np.nanstd,
    ],
)
aggregated_custom

I'm getting this error:
ValueError: cannot reduce over dimensions ('stacked_latitude_longitude',).
I'm using xvec==0.3.0 and xarray==2024.7.0 on Python 3.10 in a conda environment.

Can you show the full Traceback and full conda list? I can't reproduce that locally...

In an clean environment the error disappeared, thanks anyway!


Change NSExceptions to NSErrors for ZXOneDimensionalCodeWriter subcla…

This solves #261 for all the 1D writers, but there are more exceptions that i feel should be errors.
The code could maybe do with some refactoring, and maybe use different error codes.

please merge this. try-catch should be avoided in production code, and If I get wrong barcode value string from the backend I must do error handling. And this is a problem to make properly, if we get an exception. Check bbum's (Senior Apple employed dev) comments for more info.


Map UID to User Aliases in Robinhood

How to map user ID to usernames in Robinhood so the owner page will be more clear ?

Lustre in IaaS mode has been removed.


Logging Tool

Purpose

To produce simple logging tool.
To unify logging message format.
To unify logging method.

Usage
I expect to use this in the following ways.

>>> from logger import *
>>> setLogger(__name__) # name logger （required）
>>> info(message) # The general log that tell us the progress of the process.
message

I'm going to format the logger output message in the following.
The output format of the logger will be in the folowing form.
YYYY-MM-dd hh:mm:ss,fff [LoggerName] Level: msg
>>> setLogger("test")
>>> info("message")
2022-02-13 17:09:03,257 [test] INFO: message

We can set the logger level to add JARVIS_LOG=LEVEL_NAME  to the environment variable without changing the file.
There are five types of LEVEL_NAME according to the standard logging module: DEBUG, INFO, WARNING, ERROR, CRITICAL .

Testing method
The testing method of this is to run the test file and  to check their console output. No aumated testing is performed.
Test items are,

Whether the logger's level and name are what it is expected  to be at runtime.
Whether the logger's outputs are what  is expected.
Check if an error occurs when setLogger is not called.

I created LoggingServer module.
https://github.com/Geson-anko/logging_server

全ての関数を実行する時にtry-except構文によって自動的にロガーにエラーキャッチをさせたいです。
実装するとしたらこのような感じです。デコレータにする案もあります。
logger = getMainLogger()
def catch_exception(func, *args, **kwds) -> Any:
    try:
         return func(*args,**kwds)
    except Exception as e:
        logger.exception(e)
        raise e


Revert "Add simple mechanism to automatically convert old configs"

Fixes #239
This reverts commit 461aca3617567d2c5ce4a1c822b34e72eefbd4bd.

CI:

Linux
Linux-aarch64
macOS
Windows [](http://ci.ros2.org/job/ci_windows/4927/


Cleanup: SPIRVInstruction constructor counts its own words

Remove instruction constructor that takes a word count

Self-tested on Release and Debug builds.
LGTM


No such extension com.alibaba.dubbo.common.serialize.Serialization by name json

There is an error show up while I upgrade my dubbo from v2.5.3 to v2.5.7.
com.alibaba.dubbo.remoting.RemotingException: Failed to send message Request [id=407, version=2.0.0, twoway=true, event=true, broken=false, data=null] to /<IP_ADDRESS>:55161, cause: No such extension com.alibaba.dubbo.common.serialize.Serialization by name json
I use protocol setting below:
 <dubbo:protocol name="dubbo" port="20400" serialization="json"/>
Before upgrading version, every things run very well.
But After that, this error I mention above is almost killing me.
Please, help me to resolve this issue. thank you.

'json' extension has been removed in v2.5.7, because this inner implementation turns out to have some bugs, and also we won't have some resources maintaining.
It is recommend that you use fastjson instead:
<dubbo:protocol name="dubbo" port="20400" serialization="fastjson"/>

If you have to use this json extension, you can add a com.alibaba.dubbo.common.serialize.Serialization file with this content under you classpath:
json=com.alibaba.dubbo.common.serialize.support.dubbo.JsonSerialization

Well, it works for me.
Thank you bro @chickenlj


[finetuning] Improvements and experimenting

hey @drewrip , I think the new role should be merged with the old role, WDYT?


Remove the StopEngine method from the Kurtosis Backend

Background & motivation
Remove this endpoint because now it's equivalent to DestroyEngines.
This method left some backend resources (like the engine's container for the Docker version) in the previous version for debugging purposes (get the engine logs after the engine was stopped) but this feat wasn't used and it's not relevant, so was it better to remove it and remove the resources.
Desired behaviour
Remove the method from the backend, in order to clean up the code, and replace the calls with the "DestroyEngines" method because now are doing the same.
Refactor the StopEngineIdempotently method on the CLI Engine Manager in order to wait until not engine is returned from the backend instead of checking the status
How important is this to you?
Nice to have; this feature would make using Kurtosis more enjoyable.

We can't eliminate it finally because it's necessary to gracefully stop the Engine first (we need to stop some internal actions like closing the enclave pool) before destroying the engine's resources, like the container.


Redirect Unauthorized to /Account/Login

Explicitly define route for [Authorize] Attribute
Use Case:
Prior to Logging in, HomeController currently displays blank screen until you navigate to ~/account/login manually

.Net Core Web Template (with Individual Auth):
 "Microsoft.AspNetCore.Authentication.Cookies": "1.0.0",

JabbR-Core:
 "Microsoft.AspNetCore.Authentication": "1.1.0-alpha1-22453",
    "Microsoft.AspNetCore.Authentication.Cookies": "1.1.0-alpha1-22453",

I removed'"Microsoft.AspNetCore.Authentication": "1.1.0-alpha1-22453” from JabbR-Core with no effect (Barry’s git repo shows to have both these Nuget but not sure where it is being used)
I updated the dependency version of the out of the box Web Template project to use "Microsoft.AspNetCore.Authentication.Cookies": "1.1.0-alpha1-22453" and the automatic redirect no longer works. If we navigate to “~/Home” it shows a blank screen with no redirect. This is the same situation as our JabbR-Core. When changing back to use "Microsoft.AspNetCore.Authentication.Cookies": "1.0.0", it works again.

I was under the assumption that 1.1.0-* would have the same Authentication functionality as 1.0.0.

Emailed Hao Kung who works on Identity

Close Issue pending PR #278


The gan loss is negative？

Hello, I clone your code and try the demo training, however, I found the gan loss of generator and discriminator without GP is negative. Is that normal? And the total aeu-loss doesn't seem to converge.
Did I make a mistake somewhere？
I appreciate for any help.

Hi @Hans1984 , according to the definition, the loss is negative. It indeed takes some time to converge at the beginning, you are suggested to track it over a few thousand iterations.


Integrate and further elaborate the general idea of the game's look and feel

General game's look and feel idea was previously conceived in #30.

The general idea of the game's look and feel was integrated and further elaborated in the "Look and feel" section of our GDD.
GDD: https://docs.google.com/document/d/1iHbk6SjNvEBSYVCmc0KMylksYg6bwNcP/edit#heading=h.iq6is1wkc2sn


fix(@schematics/angular): added alt tag to img

get that 100/100 in Lighthouse Accessibility

current status


Default chart does not always match first series label

For example, see https://metriq.info/Task/141.

To clarify, for unitaryHACK: when tasks have multiple "chart metric" options, the front end app is supposed to start the view on the metric with the most data points. However, while this happens, the "chart metric" option doesn't match the metric plotted, sometimes. Please make the "chart metric" field match the plotted task chart, when landing on the view.


Installed by Plugins Manger But not see any request option

Hi, i am using JMeter 5.6.3 and install Plugins Manger 1.10 then install 1.2.0 ssh test lib, but i can not see any  request option on menu, could you anyone give me advice?

At list I found  Plugins Manger  not installed  jsch-0.1.55.jar to $JMETER_HOME/lib directory, so do it by myself, now it works!


finetune.py segmentation fault

I am trying to run the finetune.py and getting a seg. fault. Can anyone help. I am on Apple M2 mac mini with 24G memory.
% python finetune.py
loc("mps_transpose"("(mpsFileLoc): /AppleInternal/Library/BuildRoots/75428952-3aa4-11ee-8b65-46d450270006/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShadersGraph/mpsgraph/MetalPerformanceShadersGraph/Core/Files/MPSGraphUtilities.mm":206:0)): error: 'anec.transpose' op Invalid configuration for the following reasons: Tensor dimensions N1D1C4096H1W32000 are not within supported range, N[1-65536]D[1-16384]C[1-65536]H[1-16384]W[1-16384].
loc("mps_matmul"("(mpsFileLoc): /AppleInternal/Library/BuildRoots/75428952-3aa4-11ee-8b65-46d450270006/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShadersGraph/mpsgraph/MetalPerformanceShadersGraph/Core/Files/MPSGraphUtilities.mm":39:0)): error: 'anec.matmul' op Invalid configuration for the following reasons: Tensor dimensions N1D1C4096H1W32000 are not within supported range, N[1-65536]D[1-16384]C[1-65536]H[1-16384]W[1-16384].
zsh: segmentation fault  python finetune.py

Which version of pytorch do you use? I saw some issues with pytorch 2.0.1 (https://github.com/pytorch/pytorch/issues/110975)
did you update the slowllama repo recently?

Thank you!

Thanks. In that case I should upgrade?

Yes.

% pip freeze | egrep -i 'torch|numpy|sentence|fewlines'
fewlines==0.0.9
numpy==1.25.2
sentence-transformers==2.2.2
sentencepiece==0.1.99
torch==2.0.1
torchvision==0.15.2

Just couple of hours ago.

Yes, try 2.10 please. Also please make sure you ran prepare_model and finetune at the same version of slowllama - as it is pretty early/experimental there's no backwards compatibility.

Thanks, I did and it works now! This is super. Great work! Some points I noted:

Not all snapshots were written to disk. Some were skipped. I will re run and check.
GPU usage during finetune was optimum. However during inferencing, it is using fairly low, <10%. Is this expected?
Will try more things in coming days. Thanks again.

The logic to save is to only save if loss is lower. We can change that (https://github.com/okuvshynov/slowllama/blob/main/finetune.py#L53-L58).
Yes, I put close to no effort to inference optimization. I think there are other libraries focusing on that specifically (e.g. llama.cpp)

Thanks for the clarifications, I will dive into the code too. Though I am not an expert at this but I will give it my best shot. llama.cpp works great for me, however I am unable to get the finetune to work on it on my mac.


Import: all instances of the same mesh share the same morph weights

For example, only one cube should be animated in this example: MorphCube.zip
This is caused by the fact that morph weights are per mesh instance in glTF, but there is one shape key per mesh (not per mesh instance) in Blender. I solved this by duplicating the mesh.

I confirm.
We have to detect if some nodes are animated in order to know if we have to instance the mesh or create a new one

IMO you should always create a new one, because the user can always change the weights by hand, so they would still experience "false sharing" of unanimated instances.

I'm not agreed with that.
I know that there is no perfect import in that case because animation is at node level in glTF, and at mesh level in Blender, but it's really important to keep mesh instancing for performance.
If user want to change weights by hand in a specific object, he can easily make a single-user copy of the mesh.
I think we have to be as close as possible to the glTF file, so keeping mesh instancing everytime we can

Okay. Also of course, even unanimated instances can have different weights just because their nodes have different "weights" properties.

Yes, of course in this case we need to create a new mesh

"weights" at node level is not implemented in importer for now. In this issue, I will solve animation issue only.
I will create a new issue to manage weights at node level


Update enum values to be upper case and underscored

Description
As title

@yiliuTo  @saragluna Please help review this PR.


Question - How do you make platform aware modules

Modules are exported for both Darwin and Linux, so how do you go about ensuring Linux modules are not exported in Darwin and vice versa?
 Definition values:
       - In `/nix/store/z5ywvdwj1wrpaxikdw10gqdhmqb8dssd-source/modules/services/networking/my-wg-mesh.nix':

Now if I make the attribute optional and check if isLinux, you end up with:
❯ nix flake check
warning: Git tree '/etc/nixos' is dirty
warning: Using saved setting for 'extra-experimental-features = nix-command flakes' from ~/.local/share/nix/trusted-settings.json.
warning: Using saved setting for 'extra-substituters = https://nrdxp.cachix.org https://nix-community.cachix.org' from ~/.local/share/nix/trusted-settings.json.
warning: Using saved setting for 'extra-trusted-public-keys = nrdxp.cachix.org-1:Fc5PSqY2Jm1TrWfm88l6cvGWwz3s93c6IOifQWnhNW4= nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=' from ~/.local/share/nix/trusted-settings.json.
warning: unknown flake output 'deploy'
warning: unknown flake output 'lib'
warning: unknown flake output 'darwinModules'
error: infinite recursion encountered

       at /nix/store/1vqdd7m7wjbsgandpr1zrm56hvkqh5gx-source/lib/modules.nix:496:28:

          495|         builtins.addErrorContext (context name)
          496|           (args.${name} or config._module.args.${name})
             |                            ^
          497|       ) (lib.functionArgs f);
(use '--show-trace' to show detailed location information)

For now I have settled on optional attribute set base on options? Not sure if that is the best strategy or not, i.e.
{ config, lib, pkgs, options, ... }:

with lib;

let
  cfg = config.services.MyWgMesh;
in
{
  options.services.MyWgMesh = {
    enable = mkEnableOption "Enable mesh VPN service tailscale.";

    environmentFile = mkOption {
      type = types.str;
      description = ''
        File containing the TS_SERVICE_AUTH_KEY (Auth key) in the format of an EnvironmentFile as described by
        systemd.exec(5)
      '';
    };
  };

  config = mkIf cfg.enable (mkMerge [
    (optionalAttrs (options ? services.tailscale.enable) {
      # see https://tailscale.com/blog/nixos-minecraft/
      # make the tailscale command usable to users
      environment.systemPackages = [ pkgs.tailscale ];

      # enable the tailscale service
      services.tailscale.enable = true;
      # create a oneshot job to authenticate to Tailscale
      systemd.services.tailscale-autoconnect = {
        description = "Automatic connection to Tailscale";

        # make sure tailscale is running before trying to connect to tailscale
        after = [ "network-pre.target" "tailscale.service" ];
        wants = [ "network-pre.target" "tailscale.service" ];
        wantedBy = [ "multi-user.target" ];

        # set this service as a oneshot job
        serviceConfig.Type = "oneshot";

        unitConfig.ConditionPathExists = "${cfg.environmentFile}";
        serviceConfig.EnvironmentFile = "${cfg.environmentFile}";

        # have the job run this shell script
        script = with pkgs; ''
          # wait for tailscaled to settle
          sleep 2

          # check if we are already authenticated to tailscale
          status="$(${tailscale}/bin/tailscale status -json | ${jq}/bin/jq -r .BackendState)"
          if [ $status = "Running" ]; then # if so, then do nothing
            exit 0
          fi

          # otherwise authenticate with tailscale
          ${tailscale}/bin/tailscale up -authkey "''${TS_SERVICE_AUTH_KEY}"
        '';
      };

      networking.firewall = {
        # enable the firewall
        enable = true;

        # always allow traffic from your Tailscale network
        trustedInterfaces = [ "tailscale0" ];

        # allow the Tailscale UDP port through the firewall
        allowedUDPPorts = [ config.services.tailscale.port ];
      };
    })
  ]);

}

Check the devos example: https://github.com/divnix/digga/blob/29bbac0173859812893d777fdce204af83a231ea/examples/devos/flake.nix#L103-L159
Basically you have two different hostDefaults attribute sets, one for NixOS and another for Darwin hosts. Then maybe you can have something like:
imports = [ (digga.lib.importExportableModules ./modules/nixos) ];

and
imports = [ (digga.lib.importExportableModules ./modules/darwin) ];