need a strategy for storing non-filetype objects

We need to develop a simple strategy for storing non-file types (email, ip addresses, domain names). Our current scheme could almost work. However, we cannot use SASI indexing on Cassandra due to the object name array. Additionally, we need a way to easily dedup entries.
@zhanif3 @cynexit what are your thoughts?
Current scheme for object as follows:
{
"_id": UUID,
"sha1: str,
"sha256": str,
"md5": str,
"mime": str,
"source": []str,
"obj_name": []str,
"submissions" []UUID,
}
Current scheme for submission as follows:
{
"_id": UUID
"object": UUID,
"user_id": str,
"source": str,
"date": ISO8601,
"obj_name": str,
"tags": []str,
"comment": str,
}

Cassandra does have the InetAddressType which should probably be used. we could then use a string for domain and email. However, we still need some more logic on Domain as we will probably want to capture TLD and subdomains.

Per discussions, we have the following options for the schemes.
Submissions remain as follows for either option.
submission
{
"_id": UUID,
"non_object": UUID,
"user_id": str,
"source": str,
"date": ISO8601,
"tags": []str,
"comment": str,
}

Single generic no_object type
non_object

{
"_id": UUID,
"value": str, // client performs cleanup for the values.
"sha1": str,
"sha256": str,
"md5": str,
"type": str, // ipv4, ipv6, domain, email
"source": []str,
"submissions": []UUID,
}

Unique objects for core types: ip, domain, email
domain
value is replaced by

"fqdn": str,
"tld": str,
"subdomain": str

ip
value is replaced by
"ip": InetAddressType,

email
value is replaced by
"email": str,
"local-part": str,
"domain-part": str,
"sub-addressing": str,

generic_non_object
same as the option 1

From Datastax:
create a generic table like CREATE TABLE generic(id uuid, data_type text, text_value text, long_value bigint, decimal_value double, uuid_value uuid, date_value timestamp, blob_value blob, inet_value inet, PRIMARY KEY(id));
so depending on the object, most columns will be empty except the column that has the correct type
and you specify the type in the "data_type" column
For example if "data_type" contains value "long", you just need to read column long_value etc ...
If an object has multiple values and their type cannot be known before hand, just modify the table to add clustering column, like this :  CREATE TABLE generic(id uuid, property_name text, property_data_type text, text_value text, long_value bigint, decimal_value double, uuid_value uuid, date_value timestamp, blob_value blob, inet_value inet, PRIMARY KEY((id), property_name));
For the same object "id", you can store many "property_name", each of the property has a "property_data_type" thats specifies the type of its value
the problem with this kind of schema is that for an object with 5 different fields, you'll need 5 inserts. But they can be batched together since the partition key is the same
to delete the whole object and all its values, just delete by partition key: DELETE FROM generic WHERE id = xxx;
and you can create UDT to store email fields
ANd add a column like email_value email
reason is that if we have multiple tables, reading the values of an object would require reading multiple tables, expensive

"_id": UUID,
"type": str<<[file, domain,ip,email,generic]>>,
"submissions" []UUID,
"source": []str,

"sha1: str,
"sha256": str,
"md5": str,

"file_mime": str,
"file_name": []str,

"domain_fqdn": str,
"domain_tld": str,
"domain_sub_domain": str

"ip_address": InetAddressType,
"ip_type": str<<ipv4, ipv6>>

"email_address": str,
"email_local_part": str,
"email_domain_part": str,
"email_sub_addressing": str,

added
first_submission_date_time timestamp
to the above. Allows for easier supporting on the database

added boolean for IP address type


Messaging issue

I have been testing the messaging feature by chatting with myself from PC and mobile. When I send a message from PC, it instantly comes on the mobile. But when I message from the mobile, it doesn't come on the PC unless I refresh the page or switch tabs. Can you tell me why is this happening?

hi @piyushx , i checked it. It's working both ways. When new message arrive it's not scrolling chat to bottom but it's receiving message instantly without refreshing. Check left message tab it will show last message without refreshing.

Hi @raj074 ! Thanks for the reply. It got solved. However now when I'm trying to register myself on the localhost, I'm getting an error from the backend. Here it goes:
node:internal/errors:464
ErrorCaptureStackTrace(err);
^
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
at new NodeError (node:internal/errors:371:5)
at ServerResponse.setHeader (node:_http_outgoing:576:11)
at ServerResponse.header (C:\Social\node_modules\express\lib\response.js:771:10)
at ServerResponse.send (C:\Social\node_modules\express\lib\response.js:170:12)
at ServerResponse.json (C:\Social\node_modules\express\lib\response.js:267:15)
at register (C:\Social\controllers\authCtrl.js:62:30)
at processTicksAndRejections (node:internal/process/task_queues:96:5) {
code: 'ERR_HTTP_HEADERS_SENT'
}
Can you let me know why is this happening?

Hi @raj074 ! Thanks for the reply. It got solved. However now when I'm trying to register myself on the localhost, I'm getting an error from the backend. Here it goes:
node:internal/errors:464 ErrorCaptureStackTrace(err); ^
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client at new NodeError (node:internal/errors:371:5) at ServerResponse.setHeader (node:_http_outgoing:576:11) at ServerResponse.header (C:\Social\node_modules\express\lib\response.js:771:10) at ServerResponse.send (C:\Social\node_modules\express\lib\response.js:170:12) at ServerResponse.json (C:\Social\node_modules\express\lib\response.js:267:15) at register (C:\Social\controllers\authCtrl.js:62:30) at processTicksAndRejections (node:internal/process/task_queues:96:5) { code: 'ERR_HTTP_HEADERS_SENT' }
Can you let me know why is this happening?

@piyushx , I am also contributor in above project.This error describes that res has been set more than once in your code.
remove line no. 60 res.json({ msg: "registered" });


CMake: misc

Set C_STANDARD locally
Split primac fortran sources in a separate library (not supported in some msvc configurations)

I'm not opposed to this, since I believe modern CMake prefers the user specify things like this on targets as opposed to globally, but I'm wondering why you're proposing this at this time? Is there some bug you're seeing that this solves or is it just a cleanup PR?

Looks like it's causing some build failures on intel classic on macos because the standard was not specified for the stress_c_exe target, can you specify C99 for that target and also for data_c_exe? Also if you rebase off the latest main the windows gcc11 build should work.

Thank you @jschueller for proposing this. Ping me when you and @nbelakovski think it is ready.

its ready from my side, @nbelakovski I added a commit to split the fortran/c sources into separate libs

@jschueller Can you make a comment (preferrably in the git commit but in the PR is fine too) as to why the C/F sources were split up? I know there was some discussion about it in a separate PR but I just want to make sure we have traceability as to the motivation for any changes.

done

@zaikunzhang I belive this PR is good to go.

Thank you @jschueller and @nbelakovski.
I have got two questions.

Split primac fortran sources in a separate library (not supported in some msvc configurations)

Does this mean that the previous setup was not supported in some msvc configurations, but the current one is supported?
Do we need to revise the CMake snippets in README?
Thank you.

Does this mean that the previous setup was not supported in some msvc configurations, but the current one is supported?

My understanding is yes.

Do we need to revise the CMake snippets in README?

I've reviewed the README and I don't think we need to update it, but maybe @jschueller would like to review it and come to his own conclusion.
I do think we should add some tests in order to validate this. Right now all tests override the default CMake generator by providing -G Ninja. I think we should add a test that looks similar to the one in which we override the C compiler for ifx. It could look something like this (these are additions to cmake.yml)
cmake-main:
    runs-on: ${{ matrix.os }}
    env:
      GENERATOR: "Ninja"

...

- os: windows-latest
            toolchain: {compiler: intel, version: '2023.2', cflags: '', fflags: '/warn:all /debug:extended /Z7 /fimplicit-none /standard-semantics /assume:recursion' generator: "Visual Studio 17 2022" }

...

- name: Override CMake generator
        if: ${{ matrix.toolchain.generator }}
        run: echo "GENERATOR=${{ matrix.toolchain.generator}}" >> $env:GITHUB_ENV

...
    cmake --version
    cmake -G $GENERATOR -DCMAKE_BUILD_TYPE=RelWithDebInfo ...

I haven't tested this, but this is the general idea.

I cannot make it work with the vs generator, but succeeded with the nmake generator

I also tried to get it to work with the vs generator on my machine and couldn't do it. It seems others have issues with this as well https://github.com/fortran-lang/setup-fortran/issues/45 (and the proposed solution in that thread, to use the install-intelfortran-action, doesn't work as it is being deprecated in favor of setup-fortran).
I'm not familiar with NMake. Is it part of Visual Studio? This also raises the question for me as to whether or not NMake requires the sources to be separated? I think it would be preferable to have fewer libraries so as to have fewer artifacts to move around for the various packaging systems we plan to support.

there is no additional files as its an object-only library
maybe the error is specific to github actions

Hi @jschueller and @nbelakovski, has your discussion reached a conclusion? What about the tests mentioned by @nbelakovski ? Thank you.

it seems we cannot actualy test it with the visual studio generator atm as we reach a problem to set it up

@zaikunzhang @jschueller Since NMake doesn't require sources to be separated I think we should undo the changes in this PR that separate the sources. My rationale is that the code was modified to achieve a specific goal (using Visual Studio CMake generator) and then the goal was changed (to supporting NMake), so the only changes that should be in this PR are those required to support NMake (as well as the change to move C standard from a global definition to a target-specific one). I think (I could be wrong here) that it already compiles with NMake, so the only changes would be the change to C standard and also to cmake.yml to ensure that NMake works and continues to work in the future.

yes I removed it because the goal was not nmake, and in fact we dont care about nmake
but the commit to split c/fortran sources is still valid

yes I removed it because the goal was not nmake, and in fact we dont care about nmake

I'm confused, what did you remove?

but the commit to split c/fortran sources is still a valid idea for visual, as reported by @TLCFEM

It may be a valid idea, but neither you or I were actually able to get it to work. Call me conservative, but I think that if we try an idea, and we can't get it to work, we should roll back the changes that were made in that direction. Otherwise we get these sections in the codebase whose history is "this was changed to try something which didn't work" and when someone is modifying or even just reading the code at a later date it's very confusing.
I'm not opposed to splitting up C/Fortran sources in principle, I just want to keep this PR focused and well-scoped.

I found other links claiming that VS/msbuild does not support more than a single language at once:

https://stackoverflow.com/questions/5434109/why-are-visual-studio-projects-restricted-to-a-single-language
https://cmake.cmake.narkive.com/0UQygsqi/mixed-fortran-and-c-projects-in-visual-studio-with-intel-fortran-compiler

It's not that I don't believe you that the VS generator doesn't support mixed sources, I just don't like the idea of doing part of the work to support the VS generator and not doing the rest of it, particularly when that work isn't just cleanup but more along the lines of re-arranging things.
That being said, if you insist on this then I guess it doesn't hurt, but in the unlikely event that this causes issues with some future changes I think it should be considered acceptable to re-merge the sources (assuming we're not actively supporting the VS generator at that point). I think such a scenario is extremely unlikely, it's just object files and definitions here, but I just don't want us to be in the position of having the accept the limitations of the VS generator while also not getting the benefits of actually having it as something we support. Does that make sense?

what do you mean the rest of it ? I think it has all the necessary bits @TLCFEM mentioned, except the exports macro problem which I adress by not changing the c lib name

I could not test of course

Why couldn't you test?

I dont have a proper windows machine, and it fails on ci.

If you have enough space (~80GB) you can obtain an evaluation copy of windows for free from Microsoft: https://developer.microsoft.com/en-us/windows/downloads/virtual-machines/
It has an expiration date (I'm assuming they update it every couple of months), so you need to download a new image for future testing but at least it's free.
But this is what I meant by doing part of the work to support a feature and not doing the rest of it. Claiming that it should work is not sufficient, there needs to be some kind of documented evidence that it did work, even if that's just a comment from a developer saying "it worked on my machine." (ideally more than that, i.e. CI testing).
You seem to be going back and forth in this PR between trying to support VS generator and giving up. What is your goal with this PR?

it is the best i can do


feat: useApolloInlineTrace plugin for Apollo's federated tracing

Closes #1489

[x] Manipulate the final result extensions field on parsing and validation errors (see failing tests)

Any update on this one?

@Arno500 this one cannot be merged before #1493 is handled (because parsing and validation errors cannot be manipulated currently).


Add functionality to wrap ERC721 tokens

Implements:

new deterministic unique wrapped token Id
a simple test
And re-ordered some test files. There is now a Test721PCOToken.sol, Test721Token.sol (without PCO functionality) and Test721Wrapper.sol

I hope you forgive some of the code style (variable name underscores, ordering, keeping it DRY) and missing tests, please let me know first if you think the architecture looks good! :)

Going to take a look at this shortly.  I hadn't touched the token-wrapper branch in awhile and there was some code in there that was removed in main.  Can you rebase your branch against token-wrapper?


Completion

Although I wrote my own adhoc completion for a previous iteration of Conjure, I think I'll end up depending on compliment for this.
This does mean that you'll need to add compliment to your project dependencies though, I won't go magically injecting it for you, that lets you control which version of compliment you want. It'll be entirely optional.
It looks like it might get ClojureScript support soon too which would be lovely.
If you already have nREPL / CIDER in your project dependencies then I think that pulls in compliment anyway, so it will probably Just WorkTM for most people.

Progress!

So you can try out the autocomplete by depending on c434e00d949c3e7b17b9d15cef472f8c1df8a287 and ensuring you have some version of compliment installed in your project. If you have CIDER / nREPL then I think it comes along with that anyway, so it may already be there.
It works but it can slow you down as you type, not ideal. I could make it entirely async by updating the completions for the buffer in the background when you make changes or the cursor idles then just filter down that list of namespace specific completions.
I took this approach in the Rust version and it worked really well but I'd love to use compliment's context features, that requires actual async completion. I can't think of a way to do that through omnifunc so I think the only way would be through an async completion framework of some kind.
I did however find this interesting gist: https://gist.github.com/prabirshrestha/0b3b7b4e2112c662979bfeeec4e1edb7
It's a tad hack-y but basically on cursor hold they send off some job to get the completions at the cursor. Once that's done it invokes the omnifunc and reads the results from the pre-populated list of completions, including the context.
This gets us the best of both worlds (snappy completion + context aware) but I wonder how that "trigger omnifunc when I'm done fetching" works in practice. I could see it causing all sorts of issues if you're moving the cursor fast enough. Like not needing omnifunc but having it popup at random times anyway.
Maybe I'll try it and see how it actually behaves in practice, I'd love to have actual async completions taking the context into account, it's just going to be tricky within the tools Neovim has to offer.

Anyone's thoughts on this are much appreciated :smile: are buffer local completions enough for you if they're super fast? Do you love having completions within let bindings? Do you use a Neovim completion framework that would allow me to hook into it easily? I use deoplete so I worry that I'd need to implement it for every framework under the sun... I hear there's one called CoC?
Maybe I could just not take a hacky route, do it all correctly but require that you have one of a handful of very popular completion frameworks installed. Could be a good tradeoff.

Decided to only provide the basic omnicomplete support, async autocompletion can come at a later date through deoplete, CoC or similar. It'll definitely require some work to get their Python processes talking to Conjure's JVM though :grimacing:
For now I'll go with what's stable, easy and predictable. Omnicomplete triggered by <c-x><c-o>, nothing fancy. So I'll add context support, some quick first pass docs and release this.

Spent a long time tinkering trying to get autocomplete and context working but ultimately it's a lot of changes for very little gain. I'm going for MVP right now where omnicompletion works and works well. I'll get this released so people can actually start using it.
Async completion should be solved through an async completion framework which will require a bunch of thought about how those Python processes will communicate with Conjure. The context support in compliment can be used but just requires changing some of the form reading stuff I think, I'll probably do that in the future once I have a test suite.


perf: reduce memory size of const dependency and dependency type

Summary
Const Dependency size 56 -> 40, DependencyType 32 -> 24

Test Plan

!bench

!bench


Would take a moment and update your example

Can please some time out of your busy life and Upate your demo Cause the setUp() showing me I need to override three function i.e. Url , Thumbnail and Title however the example just pass the two items
Also when I passIn three items it is force closing my app Because U have used ImageLoader somewhere and not Initialized anywhere  and its saying me to have a single Instance of ImageLoader

Ok example will update in 15h

What's the deal with the ImagLoader thing

Ok I fixed it look like u have to Initialize the ImageLoader and set the default configuration.
@lipangit  But why have you kept the seekToPosition() and all those stuff private why can't I access it from player Object

I will remove imageloader,because I believe every project will have it's own load image function, Is likely to be not different , So I make public thumbImage view for people can set thumb by theirselves.

Also can you make moving to a particular position on Video public that would be great also implement a wake lock on VideoPlayer Activity cause the app is going to sleep after some time

This library suit for less then 10min videos, I do not suggest seek position when play small video . if you want to play like TVplay more than 1 hour , you maybe save the position and seekto the position and go on see that video.

So can u add that  feature

Yes seek position many people want, I will add the feature (save position and go on play) to make this library good use

And you can join us if have some progress you can push and share you code

I have question , do you mind this project use eventbus ?  and Do you need to customize the UI

No I have no background in VideoResurface View so I don't think I can help much Yes Custom UI sounds aewsome.
Also add this to FullScreenVideoPlayerActivity.java
getWindow().addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);

good idea, you can push it add you to Collaborators

release v1.10

Dude Where is the function in which progressChanges

U can catch VideoEvents.VE_MEDIAPLAYER_UPDATE_BUFFER , this event is from mediaPlayer.setOnBufferingUpdateListener .
But there is no api to catch play progressChanges, JCMediaManager is private ,U can not get play progress.
What is your usage scenarios? Why you want to get progress progressChanges?

Dude can u remove all those Event Bus logs they are  spamming my Logcat and also is there any chances for implementing the Seek Thing in one-two days. I need it urgently

What log, library have just 3 line logs , eventbus logs are all in demo.

What is your application scenarios ? you want to get the video playback progress or you want to seek video progress in your code . Even I add the feature (save position and go on play) seek position API maybe not public , library will save position get position and seek position and go on play .
This library's concept is customers just config the library , customer don't need to write code about video .

Dude I want to Save the position of every video in database so next time I want to watch that video I directly Jump to that Position also If I rotate my device it restart my video from the start

Yes save position of every video is in my workline , but 2 or 3 days may not complete .

Why don't you keep the functionality open so other can modify it without relying on you
Sorry If I sound rude but I need that feature very badly just keep the doorway open so I can configure as I want

If you save video position for go on play video , I will say NO , because the concept is customer do not need to write code about video ,the library will do that for you.   If you get video position for other purpose has nothing to do with video , I will say YES

#50 will add this feature, next is custom ui


[1.0.0-dev.6] wrong read when override

Describe the bug
======== Exception caught by widgets library =======================================================
The following ProviderException was thrown building _Home(dirty, dependencies: [UncontrolledProviderScope], state: _ConsumerState#117da):
An exception was thrown while building second:AutoDisposeProvider<int>#e15a2.

Thrown exception:
An exception was thrown while building first:AutoDisposeProvider<int>#bdd14.

Thrown exception:
Bad state: It shouldn't have happened

Stack trace:
#0      firstProvider.<anonymous closure> (package:bug/main.dart:7:12)
#1      AutoDisposeProvider.create (package:riverpod/src/provider/auto_dispose.dart:29:61)

To Reproduce
import 'package:flutter/widgets.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';

void main() => runApp(ProviderScope(child: _Root()));

final firstProvider = Provider.autoDispose<int>(
  (ref) => throw StateError('It shouldn\'t have happened'),
  name: 'first',
);

final secondProvider = Provider.autoDispose<int>(
  (ref) => ref.watch(firstProvider),
  name: 'second',
);

class _Root extends ConsumerWidget {
  const _Root({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context, WidgetRef ref) {
    return ProviderScope(
      overrides: [
        firstProvider.overrideWithValue(1),
      ],
      child: _Home(),
    );
  }
}

class _Home extends ConsumerWidget {
  const _Home({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context, WidgetRef ref) {
    print(ref.watch(secondProvider));

    return Container();
  }
}

Expected behavior
I expected the second provider to read the value from the overridden provider
Perhaps this is not a bug, but it should be?

@nikitadol This is expected. You should override the second provider as well because it depends on the first. There are other issues and discussions about this situation.
You should change your code to:
overrides: [
    firstProvider.overrideWithValue(1),
    secondProvider,
],

Hi @davidmartos96

This is expected

Is it described somewhere?

There are other issues and discussions about this situation.

Could you show it?

@nikitadol Here is one: https://github.com/rrousselGit/river_pod/discussions/573
I know there is an issue out there, but can't find it.
I don't think it's described anywhere. I answered here to let you know. It was a bit confusing to me as well at first.

@davidmartos96
Thanks!

@nikitadol @davidmartos96 Here you are: https://github.com/rrousselGit/river_pod/issues/575, https://github.com/rrousselGit/river_pod/issues/669

BTW, I personally think this design decision is terrible, and more and more bugs will be created here (until it is documented, but even then not every one will read it). It's not maintainable to be forced to hunt down all dependent providers and override them as well, and then the ones that depend on them as well, etc.

@wujek-srujek As Remi pointed out it works like in regular provider, you need to override there as well. Maybe with Static metaprogramming is not as cumbersome.
Here is a diagram on how I see providers interacting.
Riverpod:

Equivalent in Provider:

As I see it, a provider in riverpod is like a reusable configuration on how to create a piece of state that it's stored in a ProviderContainer (ProviderScope).
Overriding a provider requires overriding the dependencies as well because they hold state relative to a different scope.
I'm sure this is a nontrivial problem.

BTW, I personally think this design decision is terrible, and more and more bugs will be created here (until it is documented, but even then not every one will read it). It's not maintainable to be forced to hunt down all dependent providers and override them as well, and then the ones that depend on them as well, etc.

@wujek-srujek
As I mentioned before, that's not a "decision". There is no other way.
I wish I could automatically override a provider, but this is not possible to implement.
The problem is, a provider can depend on new providers at any time (especially with Future/StreamProvider or StateNotifierProvider). Yet to be able to automatically overrride a provider, we need to know all the dependencies of a provider.
I have one improvement in mind, which is to allow users to define the list of dependencies:
final provider = ...

final scoped = Provider((ref) {
  ref.watch(provider);
}, dependencies: {provider});

in which case when overriding provider, I will be able to override scoped too.
But this is not implemented yet.
The goal is also to have this list be automatically generated once we have static-metaprogramming.

In any case, I will close this since this is expected.


[ Future feature request ] Consider adding randomization to break scoring ties

This could be good, as in we have another try
Or bad, because it introduces non-predictability
I think it should certainly be an option, but not default
Only to be turned on in tricky cases
And even then we ought to add it only if we have tested and
confirmed this is a useful way out of tricky cases

Closing because it's for the future


[web]: OffsetLayer.toImage support in Canvaskit

Currently OffsetLayer.toImage is not supported in Cavanskit. https://github.com/flutter/engine/blob/c712e4d1957078b19f5a128d3a5d701fca829089/lib/web_ui/lib/src/engine/canvaskit/layer_scene_builder.dart#L20

We use this in desktop Rive to provide a color loupe in our color picker. This is preventing us from supporting this feature on the web!

Assigning to @hterkelsen who has something almost working already

Fixed by https://github.com/flutter/engine/pull/22085


gulp-util version deprecated

Hi,
when installing this wonderfull tools, npm states that the version of gulp-util is now deprecated.
We should use V4.0.*
Regards.

Fixed in current master :-)


Remove ebucore.rdf file from repository

The file ebucoreplus/ontology/EBUCorePlus/ebucore.rdf is outdated and not maintained any further. After the merge of CCDM and ebucore, we only continue development based on ebucoreplus.owl. I propose to remove ebucore.rdf from the repository.

@JuergenGrupp you may close the issue as it's merged.


Improve situation with build flags

Is the version updated from unset to tagged version when we do a release ? @geoand

Yes, the build flags come into play in that case as well.
the VERSION env var is set to the version we release and is used as a build flag


How to use loader with draco compression

I tried to load this model: https://market.pmnd.rs/model/wall
but it won't show up.
I tried to enable the draco decoder via <Scene gltfModel="dracoDecoderPath: https://www.gstatic.com/draco/versioned/decoders/1.4.1/"/> but with no luck.

I am also having issues here. When I do:
<Scene gltf-model='dracoDecoderPath: https://www.unpkg.com/three@0.123.0/examples/js/libs/draco/;' ...

then it looks like we are creating a network request which accidentally includes dracodecoderpath: :
Request URL: dracodecoderpath<EMAIL_ADDRESS>
GET dracodecoderpath<EMAIL_ADDRESS>net::ERR_FAILED

If I instead do:
<Scene<EMAIL_ADDRESS>...

I get a 302 back:
Request URL<EMAIL_ADDRESS>Request Method: GET
Status Code: 302

and no error from the network request. But draco loading does not work:
components:gltf-model:warn Cannot read properties of undefined (reading 'byteStride')
aframe-master.js?b048:3026 components:gltf-model:warn Unexpected token < in JSON at position 0

So I believe we may need some more parsing of gltf-model on Scene?


Can not load local model by --model-id

try two methods to create container (ghcr.io/huggingface/text-generation-inference:sha-7de8a37) but no luck:
docker run --gpus all --shm-size 1g -p 8080:80 ghcr.io/huggingface/text-generation-inference:latest --model-id ./output/checkpoint-400 --num-shard 2
docker run ghcr.io/huggingface/text-generation-inference  --model-id checkpoint-400 --num-shard 2

It seems trying download the model rather than use local model there.

another reported issue #99

Thanks for your reply. I did try your approch but it does not work?

You can try mounting a directory from the host which contains the model, and then using --model-id with the complete path within the container. This worked for me:
$ pwd
/mnt/huggingface
$ ls
opt-125m  version.txt
$ ls opt-125m/
config.json  generation_config.json  merges.txt  pytorch_model.bin  special_tokens_map.json  tokenizer_config.json  vocab.json

$ docker run --gpus all --shm-size 1g -p 8080:80 --volume /mnt/huggingface:/data ghcr.io/huggingface/text-generation-inference:latest --model-id /data/opt-125m --num-shard 1

After several attempts, I found that the problem lies in the mapping path in docker run.
Now I can confirm that your approach works!
docker run --gpus all --shm-size 1g -p 8080:80 -v /root/huggingface:/data text-generation-inference:latest --model-id /data/checkpoint-400 --num-shard 2

However, the another error came out:

It works if I set --num-shard to 1. Then the question is what does the num-shard param mean here?

other issues I have found:

when I use ab tool to test with c=2:

ab -n 10 -c 2 -p post_data.json -T application/json -H 'accept: application/json' http://localhost:8080/generate

a decode error has been rasied when batch_size = 2?

num-shard means GPU parallelization using tensor parallelism. This is needed for larger models as they don't fit inside a single device. However, for tensor parallelism to work, you need your embeddings and other linear layers inside the models to be dividable by the number of shards.

--num-shard and other arguments for TGI ("Text Generation Interface") are documented here:
https://huggingface.co/docs/text-generation-inference/en/basic_tutorials/launcher#numshard


Launch app maximized on first load

Just another super small thing, but I noticed that on first load SnippetStore doesn't launch fully maximized. I know other Electron apps used to have that issue, but don't seem to any more. Would this part of the API fix that? https://electronjs.org/docs/api/browser-window#winmaximize
PS, thank you for the big donation credit. I really appreciate your work on this app though, especially the VS Code integration, which is key! All around an awesome productivity tool!

Thanks, @onetrev I'll fix it soon :+1:

Fixed in the latest release:
https://github.com/ZeroX-DG/SnippetStore/releases/tag/v0.2.10

Hello! Sorry to report that for me at least the app still doesn't open full screen on first load. Anything I can troubleshoot to assist?
PS, hopefully you can add auto-update or definitely at least a notice when a new version is out. Lots of people probably missing out on the new features and bug fixes being added.

Thanks, @onetrev somehow the bug re-appear again :cry: I'll implement something to notify users when there's a new version. I'm still learning how to create auto-update :smile:


Primitive Library Additions

We could sure use:

3 terminal Resistor
3 terminal Cap
3 terminal Inductor
Controlled voltage source (VCVS/ CCVS)
Controlled current source (VCCS/ CCCS)

The latter are also #28

Outlook: before #53


Updated handlebars and underscore.

The handlebars dependency, or (as a more abstract idea) the template adapter depedency, was recently made part of the app bundle (https://github.com/rendrjs/rendr/pull/467).
And there's this PR not yet merged that was aiming to remove handlebars from this package.json so that this module could take any version injected by the app: https://github.com/rendrjs/rendr-handlebars/pull/52
Not sure if @saponifi3d wants to keep a sort of "default" version from the adapter side though, so this upgrade would still be helpful.

Yeah, this should be done as a dev dependency now instead.  Your app itself can keep track of the handlebars version you want to use instead of this repo deciding for you.  If you want to re-open as dev dependency :+1:


Fix error when accessing HttpContext.Request

HttpContext.Current.Request throws an HttpException when request is not available (for instance during App_Start.
We have a helper HttpContextCurrent.Request that catches this exception so it does not impact callers, but it was not used in the UserAgent enricher.
Fixes #1

I don't have access to a Windows machine lately, so that's a change I did directly in the GitHub UI ... 🤞

LGTM! Likewise, haven't touched ASP.NET in quite a while :-)


Japanese date format of the day part is ambiguous at Announce screen

Describe the bug
The title of  announce screen's item, like 2022/09/7 is no zero padding at day part.
To Reproduce
Steps to reproduce the behavior:

Set in Japanese from drawer menu Setting.
Show drawer.
Tap Announcement.
If there are anoucements, you can see the date text.

Expected behavior
It might be better add zero padding at the day part or no zero padding at all.
Screenshots
If applicable, add screenshots to help explain your problem.

Thanks!

The URL is not a link either, so it is most likely an older version of the app.

Thanks! I thought it was different screen🙏

I'm sorry and thank @Corvus400 san to fix it, and @takahirom san for response 🙏


Setup does not recognize compatible external drive

When refreshing an image, the previously used external drive should be recognized, as it could have service data.
Unsure if FE or BE issue presently. Needs further investigation.
Expected behavior:

if a GUID comes back from external drive, notify user, ask if want to reattach
If select no, and opt to create with a selected drive that has a GUID, we should warn the user that data will be deleted

Confirmed this is a frontend issue. Assigning to @DrewAns

Has this been tested @DrewAns?


Optimized home page identifying

Fixes #796

Can you confirm how you tested this? Thank you 🙂

@yaichenbaum Tested cases: Home page <-> browser and back,, recent files on the home page ->browser, changing tabs Home page -> browser and back, Settings page(just to be sure. All tested cases successful.


Not working with manually created validators

Hi, I am using manually created validator in my code. As per documentation it is supporting only basic form request validation. When it comes to big projects this basic validation may not be used so please extend your support to Manually Created Validator as well. Thank-you.

@HimakarPV about custom validation rules, see https://github.com/mpociot/laravel-apidoc-generator/issues/247#issuecomment-353553829

I see that alpha_spaces and imei rules are custom.
For such, I'd use a syntax like
'legalName' => [
  'required',
  new AlphaSpacesRule()
],
'imei' => [
  'required',
  'unique:users',
  new ImeiRule()
]

See an example where I use a custom AbsentRule here and see its definition here.
You should take a look at custom form requests too.

@HimakarPV this package currently only supports FormRequest, but support for manual validators is on the map. We'd welcome any PRs.

This code is OK:
class CarsOptionsUpdate extends FormRequest
{
    public function rules()
    {
        $uuid      = $this->request->get('uuid');
        $is_serial = $this->request->get('is_serial');

        return [
            'uuid'      => ['nullable', 'uuid', Rule::unique('cars_options', 'uuid')->ignore($uuid, 'uuid')],
            'code'      => ['required', 'string', 'max:255'],
        ];
    }
}

But this is FAIL:
class CarsOptionsUpdate extends FormRequest
{
    public function rules()
    {
        $uuid      = $this->request->get('uuid');
        $is_serial = $this->request->get('is_serial');

        return [
            'uuid'      => ['nullable', 'uuid', Rule::unique('cars_options', 'uuid')->ignore($uuid, 'uuid')],
            'code'      => ['required', 'string', 'max:255', new CarsOptionCodeExist($is_serial)],
        ];
    }
}

No longer valid because of #343


[Question / Bug] dependencyCheckAggregate task is stuck

Problem: My Gradle project times out (1 hour execution time) whereas some of my other Gradle projects, with same/similar setup, execute within 5 minutes or so. There's no obvious difference in configuration.
The build is stuck in this phase in one of the projects:
- Analyzing main:debugCompileClasspath
WARNING: step_script could not run to completion because the timeout was exceeded. For more control over job and script timeouts see: https://docs.gitlab.com/ee/ci/runners/configure_runners.html#set-script-and-after_script-timeouts
ERROR: Job failed: execution took longer than 1h0m0s seconds

What could I do to troubleshoot this? I don't think changing the timeout will help as I've run without timeout and waited 2 hours and counting...
Top level declaration (Gradle Kotlin DSL):
    extensions.findByType(DependencyCheckExtension::class)?.let {
        it.formats = listOf(HTML.name, XML.name)
        it.nvd.maxRetryCount = 0
        it.nvd.apiKey = <HIDDEN>
    }

The task that I run:
gradle dependencyCheckAggregate --info

(I've also tried with --stacktrace but with no additional information...)
My setup:

Multi-module project
Gradle 8.8
JDK 21
Dependency check version 10.0.2

Possibly related: https://github.com/jeremylong/DependencyCheck/issues/3897

You might be stuck in a garbage collection loop due to low memory in Gradle Daemons perhaps? You could look at memory/cpunusage and see if it's stuck (doing nothing) or stuck (doing something).
As a side note, do you happen to have JavaScript dependencies in your projects?

You might be stuck in a garbage collection loop due to low memory in Gradle Daemons perhaps? You could look at memory/cpunusage and see if it's stuck (doing nothing) or stuck (doing something).
As a side note, do you happen to have JavaScript dependencies in your projects?

Memory usage doesn't change. CPU doesn't change.
JavaScript dependencies, not really - although I am using Open API generator (https://github.com/OpenAPITools/openapi-generator) for example in my build script to generate source code which could in turn depend on JavaScript

Looks like a deadlock of some sort to me.
Worker threads are waiting for a lock held by included builds which in term is waiting for something else which is not happening when trying to resolve dependencies across modules.
The thread that has the lock that everything else is waiting on is at
https://github.com/dependency-check/dependency-check-gradle/blob/aa6e9d078370378167c2d2a921d4fdab0229cdb4/src/main/groovy/org/owasp/dependencycheck/gradle/tasks/AbstractAnalyze.groovy#L575
Could also be to do with issues with Gradle and/or the Gradle plugins you are using along with parallel execution more than anything within OWASP Dependency check (but I don't really know).
All I can suggest is

Since this is while analyzing artifact types, you could try narrowing it down by excluding various artifact types from the default set https://jeremylong.github.io/DependencyCheck/dependency-check-gradle/configuration-aggregate.html
disable parallel execution as a workaround, perhaps only when running ODC (--max-workers=1)
or reduce parallelism/# workers if more of them increases the chance of it getting stuck
try upgrading Gradle and/or various plugins to see if fixed there?
experiment with disabling/skipping certain Gradle configurations to see if you can find one that causes problems
experiment to see whether you get the same problem on Linux? Seems you are on Windows.

Looks like a deadlock of some sort to me.
Worker threads are waiting for a lock held by included builds which in turn is waiting for something else which is not happening when trying to resolve dependencies across modules.
The thread that has the lock that everything else is waiting on is at
https://github.com/dependency-check/dependency-check-gradle/blob/aa6e9d078370378167c2d2a921d4fdab0229cdb4/src/main/groovy/org/owasp/dependencycheck/gradle/tasks/AbstractAnalyze.groovy#L575
Could also be to do with issues with Gradle and/or the Gradle plugins you are using along with parallel execution more than anything within OWASP Dependency check (but I don't really know).
All I can suggest is

Since this is while analyzing artifact types, you could try narrowing it down by excluding various artifact types from the default set https://jeremylong.github.io/DependencyCheck/dependency-check-gradle/configuration-aggregate.html
disable parallel execution as a workaround, perhaps only when running ODC (--max-workers=1)
or reduce parallelism/# workers if more of them increases the chance of it getting stuck
try upgrading Gradle and/or various plugins to see if fixed there?
experiment with disabling/skipping certain Gradle configurations to see if you can find one that causes problems
experiment to see whether you get the same problem on Linux? Seems you are on Windows.

Thanks for the suggestions. I'll update this reply with new information as I collect it:

Since this is while analyzing artifact types, you could try narrowing it down by excluding various artifact types from the default set https://jeremylong.github.io/DependencyCheck/dependency-check-gradle/configuration-aggregate.html
This does not seem to help

disable parallel execution as a workaround, perhaps only when running ODC (--max-workers=1)
Will try this

or reduce parallelism/# workers if more of them increases the chance of it getting stuck
I'm not sure exactly what this means, but I am guessing that it is not feasible for my project where parallelism is a must to reduce build time

try upgrading Gradle and/or various plugins to see if fixed there?
I am on the latest stable versions of Gradle and other plugins (problem was reproducible in previous versions as well)

experiment with disabling/skipping certain Gradle configurations to see if you can find one that causes problems
I'm not sure exactly what this means

experiment to see whether you get the same problem on Linux? Seems you are on Windows.
This is a problem both on Windows and MacOS (M1) at least

Api request failures have nothing to do with deadlocks. Plenty of other tickets and guidance on this one.

Api request failures have nothing to do with deadlocks. Plenty of other tickets and guidance on this one.

Sure, but that is just a small finding while I attempted to troubleshoot. I have other projects where the plugin works well with the same setup


No video file

Hi, can you share the video file again, I clicked on the file and it's not working right now

Updated the link.


Deleting article from smart folder or filtered folder does not work

See http://forums.cocoaforge.com/viewtopic.php?t=26908&p=139682
and https://forums.cocoaforge.com/viewtopic.php?t=26932&p=139807

@barijaona do you think this has been fixed with your recent commits?

Yes. I just forgot to close the issue.


IPROTO-253 ProtoBuf 3.21.10

https://issues.redhat.com/browse/IPROTO-253

merged! thanks @tristantarrant !


Allow zoom using scroll wheel on macOS

I realised that there is no way to zoom in/out when using a traditional mouse with scroll wheel on macOS. This PR enables the scroll wheel to zoom in and out of the image.

Thanks for sharing. I would like to point out one thing.
I think you should  call removeMonitor(_:) somewhere to stop monitoring as Apple said in their document.

Thanks @infinitepower18 and @hjsupernova !
I've merged manually, added removeMonitor(_:) as suggested, did some cleaning and support for Swift 6, and plan to release this change today.


No index names in interactive migrations

I have to go blindfolded when creating a migration. Index names should be more descriptive than this:
did you drop index '__::idx' of object type 'default::Shift'? [y,n,l,c,b,s,q,?]
> y
did you drop index '__::idx' of object type 'default::Shift'? [y,n,l,c,b,s,q,?]
> y
did you create index '__::idx' of object type 'default::Shift'? [y,n,l,c,b,s,q,?]
> y
did you create index '__::idx' of object type 'default::Shift'? [y,n,l,c,b,s,q,?]
> y
did you drop index '__::idx' of object type 'default::Shift'? [y,n,l,c,b,s,q,?]
> y

EdgeDB Version: 5.2
EdgeDB CLI Version: 5.1.0+7c5764f
OS Version: Linux

Steps to Reproduce:

edgedb migration create

Schema:
type Shift {
    ...
    property date -> datetime;
    index on (.date);
    ...
}

@dnwpark See get_friendly_description and the stuff it calls; you'll probably want to overload those in IndexCommand

Probably get_friendly_object_name_for_description is the main thing you care about here


Possible to use a remote temperature sensor to reduce short cycle?

In MEL-AC library, there is a command to set external temperature value. mgos_mel_ac_set_ext_temp(float temp); Is it possible to read data from Homekit (ie homepod in same room) and use value to set above command?
Mistubishi units are known to short-cycle, where the unit turns on/off many times and uses more power. A common fix is to use the MHK2 thermostat or other remote thermostat, placed far from the unit. But MHK2 is expensive and does not allow daisy-chain to other CN105 connectors.

External in this conext means extarnal device can update temperature value on MEL-AC device. Unfortunately, interaction with another homekit accessory is not a part of homekit spec.

Would it be possible to add a temperature sensor to the esp8266 board and use that to replace the temp sensor?
Or poll an http address or something?
As an example, see what this project is doing:
https://github.com/geoffdavis/esphome-mitsubishiheatpump?tab=readme-ov-file#remote-temperature

I see that Eve has released a matter-protocol device that directly communicates between controller and sensor (albeit, for radiators). So sounds like a project like "mel-ac-matter" would theoretically allow a remote sensor to work. Some idea for the future!
https://www.evehome.com/en-us/eve-thermo-control

fyi, the temp sensor is locating inside incoming air chamber (in ducted internal unit)
so the temperature measured by incoming airflow


AzNamingTool - Connect to Azure Government

When deploying the solution, I get an error stating the subscription is not found. I know I need to connect to Azure Government but I cannot figure out where to change this value.
Run azure/login@v1
/usr/bin/az cloud set -n azurecloud
Done setting cloud: "azurecloud"
Note: Azure/login action also supports OIDC login mechanism. Refer https://github.com/azure/login#configure-a-service-principal-with-a-federated-credential-to-use-oidc-based-authentication for more details.
Error: : No subscriptions found for ***.

Error: Az CLI Login failed. Please check the credentials and make sure az is installed on the runner. For more information refer https://aka.ms/create-secrets-for-GitHub-workflows

I figured it out. You just have to add the line below to the login step of the workflow.
environment: 'AzureUSGovernment'


Networking Primitives to get ready for Sentry Nodes

Sentry nodes are special nodes set up by a validator operator that are intended to be the points-of-contact for the validator node, which is shielded from public connections.
Typically, sentry nodes will function as normal nodes. But for consensus networking in particular, they will have to behave as though they are the validator node. That means that it needs to become aware of what knowledge the validator node has.
This will typically be handled on the level of particular networking protocols (knowing how to proxy certain requests to the underlying validator).

Basics
What needs to be made available to the networking protocol:

Whether the local node has sentry nodes and what their PeerIds are.
Whether the local node is a sentry node and what its PeerId is.

This information needs to be made available to the core-level networking, any NetworkSpecialization and any consensus_gossip::GossipValidator
Then we'll also need CLI parameters for configuring and setting up sentry nodes.

cc @twittner : do you have some input on how best to integrate into the substrate networking to attach info onto PeerIds when connecting?

Whether the local node is a sentry node and what its PeerId is.

What the PeerId of its validator node is?

do you have some input on how best to integrate into the substrate networking to attach info onto PeerIds when connecting?

I am not sure what you mean. I think information about PeerIds would go into network::config::NetworkConfiguration.

What the PeerId of its validator node is?

yes, sorry.

Done as part of https://github.com/paritytech/substrate/pull/5520 https://github.com/paritytech/substrate/pull/5665


Be smarter about StringBuilders

/cc @Eilon
This saves about 40mb of allocations by moving a declaration out of the loop.
Before:

After:

:shipit:

:shipit:

This will probably lower your allocation rate even further - https://github.com/dotnet/roslyn/blob/master/src/Compilers/Core/Portable/Collections/PooledStringBuilder.cs
Maybe as a part of another change :)

This will probably lower your allocation rate even further

We already have this actually :+1: : https://github.com/aspnet/Common/blob/dev/src/Microsoft.Extensions.ObjectPool/ObjectPoolProviderExtensions.cs#L15
Going to hold off on integrating it for now since this is only allocating one per request.


Added support to download annotated data in CoNLL format

Please add some test cases about CoNLLRenderer and CoNLLPainter.

will create a fresh pull request


strange caracters and no arrows

Here's a screenshot : http://i.imgur.com/8pWYwzL.png
Why I do not have beautiful arrows ?
Thanks for your support,
David

This usually happens when you aren't using a font patched to support Powerline. Generally speaking, the Unicode characters representing those arrows are in the so-called private-use Unicode space, so they aren't usually bundled with fonts by default — hence the patches.
See here for compatible fonts.

Hello,
I've installed theses fonts but no changes.
$ ./install.sh
All Powerline fonts installed to /home/david/.fonts

But same result (after closing and after reboot).

You've installed them -- but have you configured your terminal to use them?

Houra !
I was thinking that the gnome-terminal select them automaticaly.

:+1:

I have exactly the same screenshot. It's not entirely clear however, how you get your terminal to use them.
@clicproxy can you briefly mention or point to how you achieved this?

@stonier I think it was probably in profile > profile preferences > general > font

Oh ho...with all the command line documentation creating so much noise over in the powerline docs, I didn't even think about the trivially obvious.
Works fine now - thanks :neckbeard:

@stonier heh you're welcome :)


New Website Logo: Luxe

Please fill the following out:

Name: Luxe
Homepage url: http://luxe.com/
Brand Guidelines/Licensing: n/a
Logo: https://www.dropbox.com/sh/ogf8qmhfau9to01/AABZopk4SIQbn_FFVfLnjEyaa/Luxe Photos/Luxe Logo?dl=0&preview=Luxe-Logo.png

Great meeting you last month, thanks for sending this out!


feat: add ali oss adapter

Hi there,
This is an Ali OSS adapter. This adapter functions somewhat like the S3 service, although there are notable distinctions between the two.
Notably, it operates seamlessly both on the online service and simulator. For the latter, a slight workaround is required to activate the sldEnable feature.
if you want to use the simulator to mock the environment, the following steps are necessary:
import oss2
auth = oss2.Auth('AccessKeySecret', 'AccessKeySecret')
bucket = oss2.Bucket(auth, 'http//<IP_ADDRESS>:8880', 'payload-bucket')
bucket.create_bucket()

I look forward to the review of the code and let me know how to improve the current work.

This plugin is now being maintained in the Packages Directory of the Payload Monorepo. This repo will soon be archived and all open PRs including this one will be closed. Would you be willing to re-open this same PR but within the Payload monorepo?


feat: jose/robusta nightly

Hello!
This PR contains the resources for running tests against Robusta
It has been tested and it works, we can connect to robusta-nightly 👌
Also, it contains the changes pulled from the PR: https://github.com/celestiaorg/test-infra/pull/208 (maybe we'll have to wait until merge it)
Best!
Jose Ramon Mañes

keep it open until we merge: https://github.com/celestiaorg/test-infra/pull/208

hello @Bidon15
I've resolved some conflicts that we had after merging the other PR.
Let me know when you'll check it if you find anything to modify :)
cheers!


Revert sub-classing of Promise

This reverts commit d9b4de79ef23c57fa7ce6ace971c3a429d8f039c and 1618234d538cd36e8c167a495820147577369866.
Fixes #121

I made this as a PR to easily test that the tests still pass

@watson do you have access to publish?

@hayes yes, I'll publish now


Cannot create Kubernetes discovery

My k8s apiserver without any verification，so my prometheus.yaml is:
- job_name: 'kubernetes'
  kubernetes_sd_configs:
  - masters:
    - 'http://<IP_ADDRESS>:8080'
    in_cluster: false

When I start, I get
prometheus, version 0.16.1 (branch: release-0.16.1, revision: 968ee35)
  build user:       fabianreinartz@macpro
  build date:       20151016-13:19:57
  go version:       1.5
INFO[0000] Loading configuration file prometheus.yml     source=main.go:196
ERRO[0000] Cannot create Kubernetes discovery: open : no such file or directory  source=targetmanager.go:424
INFO[0000] Loading series map and head chunks...         source=storage.go:262
INFO[0000] 309 series loaded.                            source=storage.go:267
INFO[0000] Listening on :9090                            source=web.go:223

Can you help me，Which file I lack?

Looks like a problem reading the CA certificate. You might want to try setting a tls_config with insecure_skip_verify: true.

That's a terrible error message - hope to get some time to clean it up soon.

Ah looks like I've already done that in HEAD so should be in next release.

Fixed in 7ff5cc6

This was actually a bug reading the service account token file IIRC.

OK, closing this then. Please re-open if it turns out to be a different issue.

@beorn7   setting a is_config with insecure_skip_verify: true not effective， So I may have to wait for the next release or setting cadvisor address replace kubernetes_sd_configs ?
I try setting is_config with ca_file、cert_file、key_file  error still exists.

As I said above this is due to bearer token file handling when running in a pod, fixed in master. If you're running a Docker container then you can use prom/prometheus:master image that contains the fix, although be aware there are some changes to Prometheus config that you would need to do.


Reloading CameraTest will give an error and leave the network in a non-workable state.

Describe the bug
Reloading CameraTest either through File => Open or File => Library will generate an error every time.
Application: Brain Simulator
Module/Network: CameraTest
To Reproduce
Steps to reproduce the behavior:

Open Network CameraTest either through File => Open or File => Library or File => Recent Files.
Try the same action a second time.
Notice Graph Builder error presented.
Brainsimulator II will leave camera window empty.
Restart of the program is required to clear this error!

Expected behavior
The CameraTest network should open without error messages, no matter what network has been opened before.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows 10 Pro   20H2
Version v1.0

Add any other context about the problem here.

I can replicate this bug.

Current Master has this problem fixed.


Fixing typo in LineJoin enum

There should be Bevel not nBevel. Reference: https://www.cairographics.org/manual/cairo-cairo-t.html#cairo-line-join-t

Thanks!
Whereas the tests fail, I'll still merge it since it doesn't seem to be related to your code. The last cargo version has changed the handling of Cargo.lock apparently...

cc @gkoz


Module parse failed

I have same issue https://github.com/stockulus/pouchdb-react-native/issues/73
Which babel presets I must use??

Help me please

??????????

I have no clue about this, @reutopiaer properly has.
for me react-native works


Integrate Open AI TTS

Open AI Text-To-Speech
With the release of Open AI's realistic text-to-speech API we should be able to upgrade the voice of the program.

Integrate Open AI TTS
Allow the user to change the model via the options

Integrated with previous update


POLIO-1516: Calendar to TSX

In order to apply some changes in the future on the calendar it can be very useful to have it working with tsx
Related JIRA tickets : POLIO-1516
Self proofreading checklist

[x] Did I use eslint and black formatters
[x] Is my code clear enough and well documented
[x] Are my typescript files well typed
[ ] New translations have been added or updated if new strings have been introduced in the frontend
[ ] My migrations file are included
[ ] Are there enough tests
[ ] Documentation has been included (for new feature)

Changes
Pass all js files to JSX, create /adapt calendar types
How to test
Go to to calendar page and check everything is working nice !

Additonal comment: Maybe not related to the PR, but the map doesn't fit to bounds when changing the country or country group selection. But maybe it's on purpose, as recentering the map at every filter change  maybe tedious for users
https://github.com/BLSQ/iaso/assets/38907762/3e0caf68-a524-466a-bb96-bc6799c36d84


[Android]:Why not use a class annotation in the Java  ViewManager?

In the code of javascript components, we use '@providesModule' to export the component, but in java we define a unimplemented method getName() for mapping. I don't understand this design and I think using a class annotation named '@providesModule' in java would be better！

@facebook-github-bot label android

@providesModule is an implementation detail in React Native. You shouldn't use it outside of the core React Native code.


Example should have a space

The help text at http://freecodecamp.com/challenges/waypoint-target-a-specific-child-of-an-element-using-jquery gives an example that says:
$(".target:nth-child(3)").addClass("animated bounce");
This should say
$(".target :nth-child(3)").addClass("animated bounce");
per :nth-child's syntax.

duplicate #1927


TypeError: ERROR ❌ segment dataset incorrectly formatted or not a segment dataset.

Hi, everyone!
I'm trying to train a segmentation model using the FracAtlas dataset, but I'm encountering an error indicating that the dataset isn't correctly formatted for this type of task/model.
I'm splitting my data using the utility notebook (FracAtlas/notebooks/dataset/Utilities/coco2yolo.ipynb) provided for conversion of COCO annotation to YOLO format, and then splitting of images and labels as it's needed. While this works wonderfully for object detection, it doesn't seem to work for segmentation.
Any advice or assistance would be greatly appreciated!
Thanks in advance.

I've used this code here https://github.com/z00bean/coco2yolo-seg/blob/main/COCO2YOLO-seg.py to convert annotations and worked now!


Unknown pending operation error

Environment
Knex version: 3.1.0
Database + version: Postgres 16.2
OS: Bookworm
Bug

Explain what kind of behaviour you are getting and how you think it should do

I am seeing a thrown error from tarn.js for an aborted pending operation that Knex re-raises. I am unable to diagnose what operation is pending or why there is an operation pending (since from my code, I don't see what "remains" to be done, everything has completed).
I am not sure how to introspect the aborted pending operation to see what is causing this. If it's some internal-to-knex query, then I would expect it should be safe for me to call knex.destroy without anticipating it. If it's something on my end, I can't tell at all.

Error message

2024-06-02T04:00:05.387Z | /app/node_modules/.pnpm/tarn@3.0.2/node_modules/tarn/dist/PendingOperation.js:25
-- | --
  | 2024-06-02T04:00:05.387Z | this.reject(new Error('aborted'));
  | 2024-06-02T04:00:05.387Z | ^
  | 2024-06-02T04:00:05.387Z | Error: aborted
  | 2024-06-02T04:00:05.387Z | at PendingOperation.abort (/app/node_modules/.pnpm/tarn@3.0.2/node_modules/tarn/dist/PendingOperation.js:25:21)
  | 2024-06-02T04:00:05.387Z | at /app/node_modules/.pnpm/tarn@3.0.2/node_modules/tarn/dist/Pool.js:208:25
  | 2024-06-02T04:00:05.387Z | at Array.map (<anonymous>)
  | 2024-06-02T04:00:05.387Z | at /app/node_modules/.pnpm/tarn@3.0.2/node_modules/tarn/dist/Pool.js:207:53
  | 2024-06-02T04:00:05.387Z | at processTicksAndRejections (node:internal/process/task_queues:95:5)
  | 2024-06-02T04:00:05.387Z    at Client_PG.destroy (/app/node_modules/.pnpm/knex@3.1.0_pg@8.11.3/node_modules/knex/lib/client.js:358:9)

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to MySQL or PostgreSQL, then single file example which initializes
needed data and demonstrates the problem.

// just a snippet to give the gist
class DbImpl {
    constructor(knex) {
        this.knex = knex;
    }

    async destroy() {
        await this.knex.destroy();
    }
}

const config = ...;
const k = knex(config);
const db = new DbImpl(k);

// do stuff - everything is awaited

await db.destroy(); // error occurs

Missing / erroneus documentation
All I can find on the knex.destroy method in the docs are this comment:

If you ever need to explicitly teardown the connection pool, you may use knex.destroy([callback]). You may use knex.destroy by passing a callback, or by chaining as a promise, just not both. To manually initialize a destroyed connection pool, you may use knex.initialize([config]), if no config is passed, it will use the first knex configuration used.

So I could be wrong on how to use it, but the intent of me calling destroy is that I am done with the connection and want it to be GC-ed since my DB wrapper object is going to be GC-ed.

Finally figured this out by enabling asyncStackTraces and seeing the operation that was being cancelled emit a log asynchronously.
There was a bug in that code that was easily rectified. Will make a PR to the docs repo to make it a bit easier to find.


fix(deps): update required_provider to latest for the landing zone das

This PR contains the following updates:

Package
Type
Update
Change

external (source)
required_provider
minor
2.2.3 -> 2.3.2

ibm (source)
required_provider
minor
1.60.0 -> 1.61.0

Release Notes

hashicorp/terraform-provider-external (external)
v2.3.2
Compare Source
BUG FIXES:

data-source/external: Prevent regression since v2.3.1 where null query element values would be sent to the program as an empty string (#208)

v2.3.1
Compare Source
BUG FIXES:

data-source/external: Remove query validation to allow null and empty string values to be passed to the external program (#193)

v2.3.0
Compare Source
NOTES:

provider: Rewritten to use the terraform-plugin-framework (#184)

IBM-Cloud/terraform-provider-ibm (ibm)
v1.61.0
Compare Source
Features

Support for MQ on Cloud

Datasources

ibm_mqcloud_queue_manager
ibm_mqcloud_queue_manager_status
ibm_mqcloud_application
ibm_mqcloud_user
ibm_mqcloud_keystore_certificate
ibm_mqcloud_truststore_certificate

Resources

ibm_mqcloud_queue_manager
ibm_mqcloud_application
ibm_mqcloud_user
ibm_mqcloud_keystore_certificate
ibm_mqcloud_truststore_certificate

Support for Secret Manager

Datasources

ibm_sm_service_credentials_secret_metadata
ibm_sm_service_credentials_secret

Resources

ibm_sm_service_credentials_secret

Support for VPC

Datasources

ibm_is_snapshot_consistency_group
ibm_is_snapshot_consistency_groups

Resources

ibm_is_snapshot_consistency_group

Enhancements

feat(Cloud Databases): Database user password complexity validation (4931)
Update pi_user_data to accept string input (4974)
support host_link_agent_endpoint for Satellite host (4970)
Add mtu and accessConfig flags to subnet create commands for terraform (4690)
feat(Cloud Databases): Redis Database User RBAC support (4982)
fix(Cloud Databases): fix Unwrap return value for go 1.18 compat (4991)
update issue fixed ibm_is_subnet_reserved_ip (4988)
Adding Flexible IOPS (4992)
Removing Support For Power VPN Create (4993)
Feature(share-crr): Share cross region replication (4995)
Enhancement: Added operating system attributes to is images datasources (4998)
added enhancement to one step delegate resolver in is_vpc (5000)
resolved delete issue for the floated nics on bm server (5001)
Regenerate projects provider based off the latest go sdk (5003)
Support route advertising in vpc (5005)
Add a nil check for boottarget of bms (5014)
Delete wait logic changes (5017)

BugFixes

Fix IBM pi documentation bug (4969)
Incorrect key_algorithm handling forces delete & replace of ibm_sm_private_certificate on every apply (4978)
ibm_sm_private_certificate_configuration_template arguments ttl and max_ttl are not documented (4977)
ibm_sm_private_certificate unsupported argument: rotation.rotate_keys (4976)
data ibm_schematics_workspace bug (4990)
Secret Manager docs bug fix (5018)

v1.60.1
Compare Source
BugFixes

Regenerate Projects TF to fix generated doc and samples
(4961)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 5.7.3 :tada:
The release is available on:

GitHub release
v5.7.3

Your semantic-release bot :package::rocket:


Removed sentence referring to non-existent code

Removed: "Note that the output, an object representing the container, is stored in a variable $con. This variable is used in subsequent commands."
The variable $con is not used in the PowerShell code below, also the variable is not used in this part or the next one.

Thanks RuudBorst.


[bug] AppImages made with tauri do not run on all still-supported Ubuntu releases

Describe the bug
AppImages made with tauri do not run on all still-supported Ubuntu releases in case they contain binaries built on systems newer than the oldest still-supported Ubuntu release.
Reproduction

Run the oldest still-supported Ubuntu LTS release
Download the AppImage from https://github.com/nymtech/nym/ (made with tauri)
Try to run it; you will get libc.so.6: version 'GLIBC_2.29' not found (required by nym-wallet)

Expected behavior
Runs
Platform and versions
I don't have this information, as neither npm nor cargo are installed on the system I am trying to run the AppImage on

Stack trace
https://github.com/AppImage/appimage.github.io/runs/6744305757?check_suite_focus=true#step:4:402

Additional context
To avoid this kind of issue, tauri could

Ensure that all ingredients that go into the AppImage are built on sufficiently old build machines (i.e., no newer than the oldest still supported LTS Linux distributions) - this is what major projects like Firefox, LibreOffice, Electron, etc. are doing
As a last resort, bundle everything privately, down to and including glibc (or something more bundling friendly like musl libc)

Note that this is not AppImage specific. The same considerations would need to be made if one would want to distribute executables for Linux in e.g., zip files.

This is not about any "ingredients" of the appimage, but mainly about the binary itself. Generally the mantra is: "build on the oldest system you want to support", for reference our docs on this: https://tauri.studio/v1/guides/distribution/linux#limitations.
Taking nym-wallet as an example they use "ubuntu-latest" in their workflow instead of "ubuntu-1804".
And as you said, it's not just appimage specific, .deb has the same problem and we can't really do anything about that one for example.

To avoid this kind of issue, tauri could
Ensure that all ingredients that go into the AppImage are built on sufficiently old build machines (i.e., no newer than the oldest still supported LTS Linux distributions) - this is what major projects like Firefox, LibreOffice, Electron, etc. are doing

There is absolutely no way for us to ensure that.

As a last resort, bundle everything privately, down to and including glibc (or something more bundling friendly like musl libc)

"private" bundling: kinda possible, but the tooling that does that is not cross-distro yet (blocker for us) and vastly increases appimage size.
musl: not really possible atm because a) harder to use for the average user and more importantly b) (still) not 100% compatible.
That said we're keep exploring in all directions especially because of webkit versions.

This is not about any "ingredients" of the appimage

That's what I meant by "ingredients", the binaries inside the AppImage, or .zip, or .deb, or whatever ormat.

And as you said, it's not just appimage specific, .deb has the same problem

Exactly!

There is absolutely no way for us to ensure that.

Well, here is how I ensure it in a similar tool of mine, linuxdeployqt:
https://github.com/probonopd/linuxdeployqt/blob/66d6eeea28efb1a6a03e8c69670b634b27025260/tools/linuxdeployqt/main.cpp#L190-L212

Just as an indication, out of the handful of applications using tauri that I tested, none ran on my system. Either they need too new libraries, or they are missing libraries like libwebkit2gtk that cannot reasonably be assumed to be installed on all target systems. So it would be great if the tauri tooling could help application developers to produce higher-quality AppImages.
I'd he happy to help with ideas and suggestions as I have accumulated quite some experience with AppImages over the years.

That's my mantra too - but in my tools I add some checks to make sure that developers using my tools are aware of this mantra (surprisingly many aren't). Maybe you'd consider doing that as well?

Everytime we add a new warning or similar, we get new issues and tons of discord messages about it (even with something simple like a message about an outdated tauri-cli) 😅
And yeah, really nobody reads the docs so i could agree on a warning (with a link to the docs?) as long as it doesn't stop the bundler, because it's legit and because of webkitgtk really common to build on newer systems, especially if you just want to quickly share your development progress with a friend for example.

Either they need too new libraries

That's why i wanted to change our tauri-action docs/readme to use ubuntu-1804 but never got around doing it :/

or they are missing libraries like libwebkit2gtk

This means they built they app >1 year ago (or with dependencies that old), can't do anything about that 😅

That's why i wanted to change our tauri-action docs/readme to use ubuntu-1804 but never got around doing it :/

:+1:


env COREFILE to setup coredns on docker

wrapper script to run coredns, pipe COREFILE if set.

Note we already have COREDNSPATH. Not really used for anything. Maybe prefix everything with CORE_ ? So CORE_FILE and CORE_PATH?

oh, let's do this properly in Go and document it


Merge1

Merge commit: merged text from "Publishing Packages to Mobile Stores" into "Publishing a Hybrid Mobile App to Mobile App Stores"
Delete commit: deleted "Publishing Packages to Mobile Stores"

Closing, send a new PR when ready please


Fix Trace Status Calculation

What?
Status for some traces were being incorrectly calculated, causing there to be missing contracts in the contracts table.
Why?
The function calculate_trace_statuses_for_single_transaction was designed to only process a single transaction at a time but was instead processing transactions from multiple blocks due to it being grouped by transaction_index rather than block_number + transaction_index.
How?
Update code to group by block_number + transaction_index.

LGTM


No less code

Fixed description.
Less code:
sub = "Some value"
raise RuntimeError(f"{sub!r} is incorrect")

More code:
sub = "Some value"
msg = f"{sub!r} is incorrect"
raise RunetimeError(msg)

I meant less code displayed in the traceback. But I can see how it’s unclear.


Terrain interaction

What are sequences based on terrain interaction?

I don't have an exact list of sequences on hand. In the data processing script these sequences are detected heuristically and discarded. You can run this processing script and track the discarded sequences to see which ones are detected to have terrain interaction.


[docs] Fix about page flags

Preview: https://deploy-preview-29314--material-ui.netlify.app/about/

Nice to see that we have consolidated on a single flag provider (same as https://deploy-preview-29314--material-ui.netlify.app/components/autocomplete/#country-select). The downside of this one is that the flag aren't squared, it doesn't provide the same visual output, but it's fine 👍


Request for additional word highlights

Hello I'm using the plug-in so well.
I'd like to make a request for adding one function.
I've been using the word highlights
There are so many things that can't be done.
If you include the -,numbers, "",'',etc., you can't highlight it. Please do it

I already started working on it. If you want just take the main.js and styles.css and overwrites yours with it (after you created a backup). The new one would work as descrived in #92 and #93. I don't have a solution for specifying the " yet.

Please read the updated line and text highlight in version 1.2.8. Should be fixed now.


Use devstats.PID.json as devstate

What type of PR is this:
/kind feature
What does this PR do / why we need it:
State of odo dev if now saved in a file .odo/devstate.${PID}.json, to be able to run multiple instances and state for each of them.
For compatibility, the file .odo/devstate.json is also used for the first instance of odo dev.
One drawback is that if an instance of odo dev is killed, the devstate.json file won't be overwritten. One possibility would be to delete it when running odo delete component and the process with registered PID into devstate.json is not existing.
Which issue(s) this PR fixes:
Fixes #6494
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

Thanks @rm3l for your review

/hold I need to add some integration tests

Investigating this error:
On Windows, it seems that the second process detects correctly that a first process exists.
But at the time of stopping the first process at the end of the test, an error occurs.
[FAILED] [66.184 seconds]
odo dev command tests when a component with multiple endpoints is run [It] should fail running a second session on the same platform
C:/Users/Administrator.ANSIBLE-TEST-VS/3707/tests/integration/cmd_dev_test.go:3081

[...]

 [odo]   __
  [odo]  /  \__     Developing using the "ijoreq" Devfile
  [odo]  \__/  \    Namespace: cmd-dev-test3081fac
  [odo]  /  \__/    odo version: v3.9.0
  [odo]  \__/
  [odo]
  [odo] - Running on the cluster in Dev mode
  [odo]  X  unable to save state file: a session with PID 11436 is already running on platform "cluster"

[...]

[FAILED] Unexpected error:
      <*fmt.wrapError | 0xc001fd6260>: {
          msg: "calling GenerateConsoleCtrlEvent: The parameter is incorrect.",
          err: <syscall.Errno>0x57,
      }
      calling GenerateConsoleCtrlEvent: The parameter is incorrect.
  occurred
  In [It] at: C:/Users/Administrator.ANSIBLE-TEST-VS/3707/tests/helper/helper_dev.go:195 @ 04/17/23 03:58:31.703

/unhold

@feloy Can you rebase your branch onto main? I was about to re-lgtm this PR, but noticed it has a conflict.

/override windows-integration-test/Windows-test
Flaky tests


Set a budget by default

Apps can break or misbehave or be compromised. Without a budget all funds on the wallet can be drained.

Duplicate of https://github.com/getAlby/nostr-wallet-connect/issues/163


Implement Scheduler for Services - FrontEnd

Once the backend API endpoints for scheduling services to run work (see #193), the Services page on the frontend should get a pointy-clicky user interface for managing scheduled services. See #181 for discussion.


Got git stuck on "please clean your repository before pulling"

Issue Type: Bug
Working from the train, the internet is not always most reliable. I committed and tried to sync changes to remote repo. Immediately after pressing sync I realized that there was still changes to be committed, quickly committed the remaining files as Code was still trying to connect to remote. Eventually it managed to connect only to greet me with "please clean your repository.."  Refresh button in Git view didn't help either. Only way I got it back to working order was to restart Code.
 Let me know if you can repro, I'll reopen if you do so.


Function to convert graphql plan data to rest plan data

Reason for change

We need a function to convert between these data structures in order to ease the transition from REST to GraphQL. Here is the first pass at the function. There are still a few tasks left to do though.
TODO

[x] Handle metered features
[ ] Handle Configurable features
[x] Add tests

Testing

Checklist

[ ] CHANGELOG: The Unreleased section of CHANGELOG was updated
[ ] Prop changes: docs were updated
[ ] Prop changes: E2E tests were updated, testing from the highest level possible
[ ] Platform testing: If this change should be tested against a platform, has it been?

Gary asked me to figure out what’s still undone for GraphQL in preparation for the squad change next week, and I used your PR to see what’s undone for GraphQL for configurable features. Anyway to get a feel for the work myself, I wanted to see how much work it’d be to convert configurable features.
I think I have a quick add—may be imperfect—but it should add configurable features to your PR by pasting in a couple functions and some test data:
https://gist.github.com/dangodev/797d841dc34045eda2fa3f9218399ae5
Again, once you fix up the current tests (whatever that means), let’s merge this and keep going! 🚀

Not sure what's going on here with VRT 🤔 I reran the job a couple times and it's still hanging


Feature flag generator

table with id check constraint = single num
Utility to Add bit flag column or take it away
sproc to setup
Sproc to update flag, all other flags unchanged
Sproc to get flag for conditional
Sproc to error if flag is = 0 for feature not enabled

Spawn more issues if this is a good fit for this project

I spun off a separate project for this: https://github.com/sdoubleday/featureFlagSqlGenerator


Create whats-new/v3.x

Summary

Migrated from https://github.com/TheThingsIndustries/lorawan-stack/issues/1681#event-3988842893
Add a whats-new-in-minor section to docs, which we copy over from Changelog.
Why do we need this ?

Share what's new in docs
Can you do this yourself and submit a Pull Request?

I can create, but product owners need to populate it

@johanstokking do we want to update these files as part of new feature PRs, or once per release?
Also, how do we display them? Maybe the latest what's new on the homepage? A separate what's new page with all of the version updates?

It would be per release.
Note that November 2020 is frozen, moving to December.


⚠ API - Lens has degraded performance

In 2bc7e4c, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 350 ms

Resolved: API - Lens performance has improved in 32cfbbb after 7 minutes.


Rust compiler-explorer produce error with -g or no output with -O

When I use the below valid code written in Rust, I see assembly generated - it's all good. I use the default GUI options of compiler-explorer and no compiler options. However, if I use the compiler option:

-g, the result is <Compilation failed>.
-O, the result is 7 empty lines.

In both cases, I expect to see assembly code.
fn main() {
    let mut sum: u32 = 0;
    let (from, to) = (0, 500);

    for i in from..to {
        sum += i;
    }

    println!("{}", sum) // 124750
}

https://godbolt.org/g/QgcCMQ

WIth -g the error is:
error: -g and -C debuginfo both provided
Compiler exited with result code 101

(CE already passes -debuginfo to get the line colouring)
CE compiles rust as a crate; so with optimizations on, any non-public symbols are dropped. Make your function pub fn main as a workaround for now: https://godbolt.org/g/OXJGA9

(CE already passes -debuginfo to get the line colouring)

Is it not miss leading for the user? Naively, I though the compiler options field is empty, so no option are provided: equivalent to run the command rustc --emit=asm test.rs. Do you think it will make sens to enable coloring only if -g is provided?
To continue about the defaulted -g, the produced asm code is very different between (in term of line number) between -O and -O -g. I am not expert enough in asm, but are the optimizations the same in both cases? In my example, it seems correct, since the for loop is computed a compile time: movq	$124750, (%rsp).

I understand why it might be confusing! However several options are provided to the compiler:

The -g is required to get the colourisation, so I auto-provide it (well, actually -C debuglevel=1). I subsequently learnt that for no good reason that I can see, asking rust to generate debug info causes code generation differences (see #79).
--emit=asm is supplied unless the "binary mode" option is selected.
-o <filename> is supplied to set the output to a temporary file.
--crate-type rlib is provided. This is the best we could come up with to solve #139 but I confess I don't fully understand it.

Further thoughts welcomed, but my personal thoughts are CE should work with all its features "out of the box", and if those features need compiler options then it should provide them. I agree though that when this causes compilation differences, it becomes more of a problem. I've been hoping the rust folks might fix the underlying problem.

Do you think showing this information on the web page (rust.godbolt.org) would give more transparency? Or show a small line with the actual run command. Example:

However, I have the feeling the UI is generic across supported compilers.
I would be happy to give a hand, on the Rust side. However, I very little knowledge in JavaScript (I guess this the main language in the project). Tell me if you think there are easy tasks :-)

That seems like a nice idea: thanks! Indeed, JS is pretty much the only language in this. The only rust is in the demangler

I forked your repository. I'll try to add a bottom line with the actual command run. I let you know when I have some results.

I played with the code. In the bottom line, I was able to display compiler options.
I continued to think on how to display nicely the required options to give more transparency. Instead of using the bottom line, would it be better to put them as the picture below:

To give a better user experience, this text could be replaced by "..." to hide options on a click. A tool tip can be displayed to explain these options are required. Normally, it should be generic for all compilers.
What do you think?

That sounds like an idea! My only concern is it taking up too much screen space. I'll have a think (and would accept patches too) - thanks!

I agree. To save place on screen, these hard coded options required for compiler-explorer could be toggled between a "..." and the hidden options.
An ASCII drawing of the compiler and its options. Default view will hide the hidden options. Example with rust:
|------------------------------------------|
| rustc 1.15.1 | ... | compiler options... |
|------------------------------------------|
|                 <asm code>               |
|------------------------------------------|

If I clic on "...", the compiler bar would become for example a read only text showing hidden options:
|------------------------------------------------------------------------------------|
| rustc 1.15.1 | -g --emit=asm --crate-type rlib -o <filename> | compiler options... |
|------------------------------------------------------------------------------------|
|                                     <asm code>                                     |
|------------------------------------------------------------------------------------|

Tell me if my ASCII drawings are not clear enough.

That's very clear, good thought (and nice use of ASCII art!)

It's not quite what you wanted but if you click on the compilation results indicator (between the compiler selection dropdown and the flags) you see all the options we pass. I hope this is good enough: If you feel strongly that this doesn't quite cover your idea then please reply and we'll consider alternatives. Thanks!


Support for .H265 Codec?

No Support for .H265 Codec
Current Behavior
When playing .H265 video codec the player throws an error
Environment
URL attempting to play: http://tivix.eu:8000/movie/demo4/1SVHifcnbO/88814.mkv
Error
bubbles: false
cancelBubble: false
cancelable: true
composed: false
currentTarget: null
defaultPrevented: false
eventPhase: 0
isTrusted: true
path: (8) [video, div, div.App, div#root, body, html, document, Window]
returnValue: true
srcElement: video
target: video
timeStamp: 1427.930000001652
type: "error"

I am also getting the same issue?

Any update?

+1

Any update on this? Facing same issue.

Unless you are playing a HLS or DASH stream, ReactPlayer simply renders a <video> element with some event bindings when you pass a file path. Nothing special. If the browser doesn't natively support it, ReactPlayer won't support it.
More info here (namely in the resources tab): https://caniuse.com/#feat=hevc


RTDB persistence layer triggers a bug with number parsing

UPDATED After digging around a bit more, I think that I have gotten closer to the actual issue. My findings are added in the bottom.
[READ] Step 1: Are you in the right place?
Yup, most definitely.
[REQUIRED] Step 2: Describe your environment

Xcode version: 10.3
Firebase SDK version: 6.2.0 (also present in 6.10.0)
Firebase Component: Database
Component version: 6.2.0

[REQUIRED] Step 3: Describe the problem
From the user perspective we wrote the value 1000 to the RTDB and when we read it back in we got -844.
What actually happened is that we wrote the value<PHONE_NUMBER>999999487.
Reading back this value through the persistence layer triggers an issue with NSDecimalNumber conversion.
When disabling persistence and reading/writing directly to the RTDB, this problem does not occur.
I started a discussion about the issue on the Swift forums:
https://forums.swift.org/t/nsdecimalnumber-conversion-error/29862/3
And I was directed to previous descriptions of the issue. Apparently it has been around for a long time.
http://www.openradar.me/radar?id=5007005597040640
Steps to reproduce:
Use Firebase RTDB with persistence enabled.
Write an NSDecimalNumber with many decimals of precision.
Read it back and check that the returned value is completely bonkers.
Turn off persistence and repeat to verify that it only happens with persistence enabled.
Add the following code to a Swift project to test.
Run twice to trigger the error. Then disable persistence and run again to see that the error is gone:
import UIKit
import FirebaseDatabase
import Firebase

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()

        let db = Database.database()
        db.isPersistenceEnabled = true
        let ref = db.reference(withPath: "a")

        let decimal = Decimal(_exponent: -16, _length: 4, _isNegative: 0, _isCompact: 0, _reserved: 0, _mantissa: (65023, 35303, 8964, 35527, 0, 0, 0, 0))

        _ = ref.observeSingleEvent(of: .value) { snap in
            print("SNAP", snap)
        }

        ref.setValue(decimal as NSDecimalNumber)
    }
}

Relevant Code:
The code that triggers the issue is in FLevelDBStorageEngine.m:
- (id)fixDoubleParsing:(id)value
    __attribute__((no_sanitize("float-cast-overflow"))) {
    // The parser for double values in JSONSerialization at the root takes some
    // short-cuts and delivers wrong results (wrong rounding) for some double
    // values, including 2.47. Because we use the exact bytes for hashing on the
    // server this will lead to hash mismatches. The parser of NSNumber seems to
    // be more in line with what the server expects, so we use that here
    if ([value isKindOfClass:[NSNumber class]]) {
        CFNumberType type = CFNumberGetType((CFNumberRef)value);
        if (type == kCFNumberDoubleType || type == kCFNumberFloatType) {
            // The NSJSON parser returns all numbers as double values, even
            // those that contain no exponent. To make sure that the String
            // conversion below doesn't unexpectedly reduce precision, we make
            // sure that our number is indeed not an integer.
            if ((double)(int64_t)[value doubleValue] != [value doubleValue]) {
                NSString *doubleString = [value stringValue];
                return [NSNumber numberWithDouble:[doubleString doubleValue]];
            } else {
                return [NSNumber numberWithLongLong:[value longLongValue]];
            }
        }
    }
    return value;
}

The value is checked to see if it can be represented as an integer. Since the doubleValue is 1000, it can be represented as an integer.
But then a new number is created using the longLongValue of the same number, and this triggers the error and gives -844.
Update
I think that the core of the issue is, that if I try to save a double value like 2.47 (which cannot be represented directly in a double), then the RTDB stores it precisely as 2.47. At least that is what I see when I retrieve the value again.
BUT: If I wrap the double in an NSDecimalNumber, then the double value is represented 'as is' with all decimals: 2.470000000000000512
So if the persistence layer tries to fix up any values when reading them, basically it has no knowledge about whether 2.47 or 2.470000000000000512 is how the RTDB actually stores the value. And as such I guess that it cannot be certain to calculate the same hashes or produce the same results when fetching data.
If those suspicions are correct, then any 'fixing' of the data needs to be performed when writing the data into the persisence layer - and not when reading out the values.

Another solution is to disallow serializing values of the NSDecimalNumber subclass, but the RTDB documentation states that you can write NSNumbers, and that of course includes NSDecimalNumbers...

I will close this for now as #4109 was merged. If we need to improve our parsing further to match the hash codes the backend uses, then we can open a follow-up issue.

@mortenbekditlevsen Thank you for your contribution.


[Feat.Req] Add Persistence

Problem statement
Currently, all state for the Radio is kept in-memory. While this is all that is essential for the POI Radio to operate (i.e. detect real-time divergences), persisting state across process restarts results in slightly better quality data/performance, and also unlocks other functionality like smarter notifications (e.g. subgraph divergence state changes, rather than individual POI mismatch notifications) and better dashboards and reporting.
Expectation proposal
Move to a persisted state model. Consider a SQL database backend. Consider using https://github.com/launchbadge/sqlx, with SQLite as the (simple) recommended backend for Indexers.
Alternative considerations
Alternatively we can keep the Radio stateless, and somehow export the data to an external sink for persistence, however this added complexity likely doesn't make sense. We could also leave the Radio stateless and accept the constraints that come with that.

What do you think about something like

create a serializable data structure that contain all the states
periodically save cache (to .json since serializable), and/or upon panic use std::panic::catch_unwind
load cache from json at start of program if file exists

We can get away from using a DB, but the process is similar either way


Plaid linktoken and expiration are null

Hi, I tried running this locally, and it works perfectly fine, but on the  AWS Environment live server, there's no exception thrown and it returns linktoken and exception null.
Here's the code snippet:
public async Task<CreateLinkTokenResult> GetPlaidLinkToken() {     var linkToken = await _client.CreateLinkToken(new CreateLinkTokenRequest()     {         ClientId = _clientId,         Secret = _secret,         User = new CreateLinkTokenRequest.UserInfo         {             ClientUserId = _userContext.CurrentUser.UserId,         },         CountryCodes = new[] { "US" },         ClientName = "TestClient",         Products = new[] { "transactions" }     });     return new CreateLinkTokenResult     {         Expiration = linkToken.Expiration,         LinkToken = linkToken.LinkToken     }; }
And we always get this null result:
{   "data": {     "linkToken": null,     "expiration": null   },   "exception": null,   "errorMessage": "",   "statusCode": 200 }
The result should not be nullable. Hope you guys have an idea on this.

If you switch to Going.Plaid, you can turn on ShowRawJson to see the full response you're getting from Plaid. Maybe the json is getting manipulated in the AWS Environment.

Does it only support .NET 5 up?

It's designed for .NET 5, but I've tested it with .NET Core 3.1 and it's worked well for me.

Oh I see. I got this error while trying to set value on ClientUserId for creating a linktoken, init-only setter is not available in C# 8.0. I think it only supports C# 9 up which is .NET5

You can use C# 9 or C# 10 with .NET Core 3.1. The only restriction with a higher language and a lower core version is that you can't use certain features that depend on objects in the later frameworks. You may need to set the <LangVersion /> tag in the .csproj to 9, however.
I had not considered the init-only setters being a problem w/ C# 8. If you use a C# 8 compiler, it won't complain. Only a C# 9 compiler set to language version 8 will complain. I may make some changes for that, though. Thanks for letting me know!

Yeah right! I'll try it. Thanks for helping!

Going.Plaid fixed our issue. Thanks! Will close this ticket now.


Samesite is not being respected

Describe the bug

To Reproduce Steps to reproduce the behavior:

Set a cookie as follows:

await request.cookieStore.set({
  name: 'your_name',
  secure: true,
  sameSite: 'none',
  expires: Date.now() + expiresIn,
  domain: process.env.DOMAIN || 'localhost',
  value: 'your_value',
})

Expected behavior
The cookie should be set with SameSite=None; Secure. But instead, when "Secure" is being applied, it's always forced to be "Lax" instead (https://github.com/ardatan/whatwg-node/blob/master/packages/cookie-store/src/getCookieString.ts#L22), this invalidates many uses cases like hosting the backend on any other domain than the frontend.

Environment:

OS: MacOS
package-name...: @whatwg-node/server-plugin-cookies
NodeJS: v18.15.0

Additional context

Merged and released! Thanks for your PR!


[refactoring] Avoid producing empty string/comment categorized edit ranges when renaming init. Also disallow renaming inits with no arguments.

When renaming an initializer we don't try to pick up argument labels and also don't record the base name location, since you can't actually rename it. This means there are no ranges to rename, but we still return categorized edit or type string/comment with no location information. This patch changes the category return to unmatched, rather than string/comment, to indicate they shouldn't be handled.
Also we currently show rename as available on initializers that take no arguments, even though nothing can be renamed.

Resolves rdar://problem/37905156.

@swift-ci please smoke test

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - b9c507d11c8d02d0078388657596b7ae07baaca2

Build failed
Swift Test Linux Platform
Git Sha - b9c507d11c8d02d0078388657596b7ae07baaca2

@swift-ci please test osx

@swift-ci test os x

@swift-ci please smoke test osx


Asset compression support with nuxi generate

Nitro 0.5.x added new option to enable assets pre-compression (https://github.com/unjs/nitro/pull/449) and can be enabled using this:
import { defineNuxtConfig } from 'nuxt'

export default defineNuxtConfig({
  nitro: {
    compressPublicAssets: true,
  },
})

Currently, it only works with nuxi build that runs the public assets plugin but nuxi generate seems not working (thanks @fedeweb for pointing it out!)

Hey guys is there's a workaround for this now? I'm having the same issue as @timb-103. Compressed files are generated correctly but not served to the browser