Difficulties with building a library for distributing

I'm trying to compile a lib for distributing. I want my npm package with the library has the following structure:
├───dist
│     ├───lib/*.js + *.d.ts + *.js.map
│     └───modules/*.js + *.d.ts
├───src
│     ├───lib/*.ts
│     └───modules/*.ts
├───tsconfig.json
└───package.json

Inside dist I want runtime (.js) script, declaration files for compiling user apps (.d.ts) and source maps (*.js.map) for debugging that point to sources inside *.ts.
It seems legit structure, nothing fancy.
I staged all source files into dist/src folder. The folder dist contains package.json, tsconfig.json - it's root for npm package that will be published.
There're two strategies:

compile with -d inside dist/src then rename it to dist/dist and move *.ts from it into dist/src
compile with -d and --outDir dist/dist options

It turns out that both approaches have problems and don't allow to create a working package.

If I compile with -d inside dist/src then source maps (*.js.map) will have local links ("file":"file.js", "sourceRoot":"","sources":["file.ts"]). That means that if I move *.ts later from that folder it'll break source mapping for the lib in runtime. It seems that sourceRoot option could solve that issue - but it doesn't work as expected - see #23180 (TL;DR: I I set it that tsc put local absolute file paths inside *.js.map).
If I compile with -d and redirecting into other folder via --outDir then another issue arises: if a ts-file has a reference to a declaration besides it (.d.ts) then that reference will be changed in output folder.
For example, in dist/src I have file.ts with a reference to .stud.d.ts:

/// <reference path="./.stubs.d.ts" />

after compilation with -d --outDir ./dist/dist we'll have dist/dist/file.d.ts with the following reference:
/// <reference path="../src/lib/.stubs.d.ts" />

Oh my. This is a problem as I can't copy all *.d.ts from dist/src to dist/dist because such references become invalid. You may ask me why should I copy *.d.ts from src to dist. Well dist/dist should complete distributive and it should work without dist/src (which is optional and can be even removed in some cases)
So I can't understand how it suppose to work. How is it meant libraries should be built for distributing?

I do not think the first approach is ideal. as you said it messes up your source maps, though you should know about --sourceRoot for these cases.
The second is what i would recommend. to have a recommendation on the .d.ts issue, i need to know what are these .d.ts files represent? are they your code? external dependencies? are they from npm?

though you should know about --sourceRoot for these cases.

yes, but as I noted --sourceRoot puts local absolute path into js.map that make then useless

i need to know what are these .d.ts files represent? are they your code?

it's my code usually with declare module like
declare module "xhtmpl!*" {
	const m: HandlebarsTemplateDelegate;
	export = m;
}

or
declare module "core.ui" {
	export * from "ui/.ui";
	export import Dialog = require("lib/ui/Dialog");
}

that for example allows me to put Dialog class into core.ui.Dialog-
lib/ui/Dialog.ts:
...
core.ui.Dialog = Dialog;
export = Dialog;

also why are you using /// <references to start with?

if I don't add /// <references  then client app will have to include all *.d.ts explicitly from the lib.

external dependencies? are they from npm?

there're references to vendor's typings as well which distributed inside libraries (not from @types) like:
/// <reference path="../../../vendor/fancytree/jquery.fancytree.dnd.d.ts" />

yes, but as I noted --sourceRoot puts local absolute path into js.map that make then useless

that seems like a bug in tsconfig.json parsing. we should fix that. on the command-line it should be used verbatim, you can also use file:///./src to avoid it being resolved.

seems like the .d.ts file is something you are adding manually. then i would just not use ///<references at all. have a static file in your output call it entrypoint.d.ts this file all it has is:
/// <reference path="./mygloabls.d.ts" />
export * from "./index";

the other option is to have that .d.ts file in your dist folder directly, and add the /// <reference to it in the dist folder.

Do I understand correctly that you are suggesting remove all /// <reference> imports from all my .ts/.d.ts and put all of them into a single d.ts file, right?
It's far from ideal solution for me. As the lib is modular and every module contains its own references.
And anyway that "entrypoint" should be imported somewhere and at that place its path in reference will be changed on compiling.
It's not clear why tsc can't copy referenced .d.ts into outDir with other created d.ts. If a script imports d.ts via ///<reference> that means it's needed, so why not to copy it along?

So .d.ts files are not considered sources. the compiler does not generate output for them. think of all your @types packages for instance..
The compiler does not copy references either.. again think of all your @types\node package, you do not want its contents copied to your dist folder just cause you used require.
Triple-slash -reference tags are pointers to existing files to include in your compilation, that is all.. we had them before tsconfig.json was introduced. it is similar to adding the .d.ts file in the include section of your tsconfig.json..
The only difference is that the compiler traffics it to the output, and thus it ensures that it writes your triple-slash-reference tags correctly to output.., consider use case where you have multiple components with a shared folder with .d.ts files. you want the output to still point to the correct shared folder after it is moved.
Looks to me that the best option here is to have your .d.ts files in your dist folder.. e.g. dist\globals\fileA.d.ts and reference this in your sources, the compiler will rewrite them in the output to have the correct paths.

For the record.
I abandoned both approaches (--sourceRoot and --outDir). In the end I just compile everything in-place (in staged folder /dist/dist) where I get *.js, *.ts, *.d.ts, *.js.map then just move *.ts and *.js.map into sibling folder structure /dist/src. So the release package contains:
├───dist
│     ├───lib/*.js + *.d.ts
│     └───modules/*.js + *.d.ts
├───src
│     ├───lib/*.ts + *.d.ts + *.js.map
│     └───modules/*.ts + *.d.ts + *.js.map
├───tsconfig.json
└───package.json

Inside *.js I have local sourcemap references (like //# sourceMappingURL=file1.js.map). So if the lib's users want to debug TS they'll need to copy *.ts and *.js.map from /src subtree into server's "wwwroot".


Small modification of spech5 file structure

While I was working on a widget that can display a tree view of an HDF5 file or a specfile (using SpecH5), I found that the link measurement/mca_0/info/data was not working correctly due to a bug in SpecH5.
This caused the following error: KeyError: u'Name /94.1/measurement/mca_0/info/data does not match any known dataset.'
I could have just fixed it, but I don't think this link makes sense anyway. It was implemented because, at the time, I considered it was easier to make a simple link between groups (measurement/mca_0/info -> instrument/mca_0/) rather than implement a link for each dataset in instrument/mca_0/ except data.
This PR fixes the bug and changes the implementation by removing the group link and adding individual links for each dataset. The main API change that this causes is that the link measurement/mca_0/info/data no longer exists, but the data still exists as  instrument/mca_0/data and there is still the other link one level-up as measurement/mca_0/data.
@dnaudet : is this going to cause a problem for you? If it does, I can close this PR and just fix the problem without API changes.

The link effectively does not make sense but it is like that.
The link has to be made from measurement/mca_0/info ->
instrument/mca_0/ as originally made.

In that case, I will close this PR and open PR #204


VictoriaMetrics operator should be cgroup aware

Go is not cgroup aware. This means the VictoriaMetrics operator will be throttled hard in containerised environments with CPU limits.
The below screenshot shows the container with a CPU limit in Kubernetes. The first half of the time series is on a 13600k, with 20 threads. The second half is when the machine changed to use an EPYC 7763 with 128 threads.

automaxprocs can use cgroups automatically.

Hello @uhthomas ,
AFAIK the operator code itself doesn't use muti-thread to process resource objects, so I'm not sure the performance benefit here by setting GOMAXPROCS.  Did you see some good results with new GOMAXPROCS?

Hello, thanks for reporting the issue.
Currently, operator doesn't use GOMAXPROCS for paralleling  CPU bound tasks. Actually, it doesn't perform any heavy CPU bound operations. And unlikely it will change in the future.
Meanwhile the issue itself looks to me more like golang runtime problem, neither something that each application should fix on it's own.


Queue retry gets failed : Unable to find failed job with ID []

Laravel-mongodb Version: 10.0.2
PHP Version: 8.2.18
Database Driver & Version: 1.18.1

Description:
When queue:retry is run it returns  Unable to find failed job with ID [] error for each items in queue.
Steps to reproduce

Configure failed job with mongodb
Fail a job
retry failed job with php artisan queue:retry all

Expected behaviour
Failed jobs should be queued back to jobs table
Actual behaviour
Retrying fails.
in Queue:retry command
$ids = method_exists($failer, 'ids')             ? $failer->ids($queue)             : collect($failer->all())                 ->where('queue', $queue)                 ->pluck('id')                 ->toArray();
the failer provider is MongoDB\Laravel\Queue\Failed\MongoFailedJobProvider::class which extends Illuminate\Queue\Failed\DatabaseFailedJobProvider::class .
MongoFailedJobProvider::class does not override DatabaseFailedJobProvider::class's ids() function. The ids function is implemeted based on the myssql which  tries to get 'id' of the failed job instead of '_id' in the case of Mongodb

Hi @GromNaN, I worked on this issue for a couple of hours and found out the MongoDBQueueServiceProvider class and other related classes are removed in version 5,x. what's going on? currently, does the laravel-mongodb package support queues?
PS: the documentation needs an update.

Hi again, @GromNaN. I double-checked everything; It works correctly, but outdated documents may lead users to such an error.
I prepared the https://github.com/mongodb/laravel-mongodb/pull/3223  to fix that.


Handle events and still using the poise context

I'm trying to figure out how to handle events outside of commands, like leaving certain reactions on messages for example, and still use already built functions I have to post messages in response to said events. Plenty of those require the poise context but the current event handler is using the serenity context.
Am I missing something obvious? Is there an established way to do such a thing?

Poise context consists of serenity context plus information about the channel and author etc of the command invocation. Poise's reply functions are wrappers around serenity's message post functions, with the channel ID pre-populated. But you can still use serenity's own functions, mainly ChannelId.send_message, to send messages outside of a command invocation context, where poise Context cannot be available, like the event handler


Widgets are hard to find

From Drew Shimomura in private email:

also, a strong suggestion--it would be great if all 4 items of the profile (profile/giving/history/widgets) were displayed on the same page. Either that or the buttons to the four sections should be bigger and more prominent. I don't think I'm silly (even if i'm a bit vain) for having the first thing i do be looking for my own widget to put on my blog or whatever. (Me trying to advertise for you!!) and it seems silly that it takes so much work to get to that point.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Closing in light of our decision to shut down Gratipay.
Thank you all for a great run, and I'm sorry it didn't work out! 😞  💃


FinalizePrivateFields finalizes inner class fields when only assigned to from outer class

The FinalizePrivateFields recipe finalizes inner class fields even when there is an assignment to the field from an outer class. This breaks the code...
The following test to add to org.openrewrite.java.cleanup.FinalizePrivateFieldsTest.java fails:
    @Test
    void nestedClassFieldAssignedFromOutsideIgnored() {
        rewriteRun(
          java(
            """
              class OuterClass {
                  private final InnerClass inner = new InnerClass();

                  void assign() {
                      inner.b = 2;
                  }

                  class InnerClass {
                      private int b = 1;
                  }
              }
              """
          )
        );
    }

I've a problem which also relates to Inner Classes, perhaps there is a similar root cause. Recipe RemoveUnusedPrivateFields.
If I have an innerClassVariable variable in the inner class and it used like this in the outer class the innerClassVariable variable is removed in the inner class after running the recipe and the code no longer compiles.
slection = new InnerClass();
selection.innerVlassVariable  = 123;


Use latest tiller, fix name

As it is now, we run into server/client mismatches. I guess the other option is to use the 2.8.2 helm in helm-bundle-base.

I think since there are issues with 2.9.0 of helm, I'm going to close this for now and update the base to use 2.8.2, that way they are in sync.


Add jump keys parameter to TwoWay functionality

Alternative to #898 . What do you think @petarov ?
In order to specify custom keys you can do something like this:
Crafty.e("..., Multiway, Jumpway")
     ...
    .multiway(3, { RIGHT_ARROW: 0, LEFT_ARROW: 180 })
    .jumpway(6, [ 'UP_ARROW' ]);

Looks awesome! :beer:
I'm not a big fan of omitting params (jumpSpeed), but that's shouldn't be an issue at all.

I'm not a big fan of omitting params (jumpSpeed)
Yeah optional params are used throughout the library (they allow multiple functionalities within one method so that API doesn't explode). In this case, users can call the constructor multiple times to change the jump keys without the need for specifying/changing the jump speed.

No objections thus far, it's not a big change, follows the idea of 208c3789f171192a836408bb66625c164abc1561 of splitting components, merging.
More tests for control components to be added in future.


Add alpha to transparency detection

This changelist adds "alpha" to the list of shader inputs that are checked in transparency detection functions (e.g. mx::isTransparentSurface), making them inclusive of the conventions of the glTF PBR shading model.  Materials with non-zero or spatially-varying values for "alpha" will now be considered transparent by renderers that leverage these functions.

Hi @jstone-lucasfilm I'm was thinking to add in the semantic checking that @dbsmythe and @niklasharrysson and I were discussing in the spec comments. I think this might be a cleaner chang than using input names. What do you think?

@kwokcb Agreed that transparency metadata is the right path forward in MaterialX 1.39, but we'll need a solution in the current codebase as well, in order to address issues such as https://github.com/PixarAnimationStudios/USD/issues/1882.

I might be mistaken, but is it possible that the value should have been 1.0 instead of 0.0 since we want the fully opaque value, which is 1.0 for an alpha channel?

Agreed, @JGamache-autodesk, and I've updated this as part of @pablode's proposed fix to the glTF PBR default values (#971).


[BUG]Game crashes on player death.

Crash Report: https://pastebin.com/vfq8ffcF

Whoops it seems I used the immutable form to create a list for curios event which leading to this issue, will get a quick fix out

Should be fixed now


🐛 [Firebase Messaging] getInitialMessage returns Null in iOS Terminated mode

Bug report
Hello , i'm using firebase_messaging in my current project  in android the message is not null, but in iOS, the message is null, i was following flutterfire step-by-step with version
Firebase Messaging : ^8.0.0-dev.15
and flutter version 1.66.2
the problem was already there and i was in this issue facing same problem :
the closed issue
Closing based on https://github.com/FirebaseExtended/flutterfire/issues/5706#issuecomment-832163575
Closing based on https://github.com/FirebaseExtended/flutterfire/issues/5706#issuecomment-832163575
Originally posted by @markusaksli-nc in https://github.com/FirebaseExtended/flutterfire/issues/5706#issuecomment-832595512
after confirming the problem was fixed by flutter 2 and Firebase Messaging : ^9.1.3
i tried to convert my project and implement the new versions for all plugins , But sadly the problem still happens , and when dealing with iOS (real device - iphone 6s plus and iphone X) still facing the problem which it getInitialMessage  return null when open notification  while app terminated
my code
i call this in main.dart after iniat. firebase core
_firebaseMessaging.getInitialMessage()
        .then((RemoteMessage message) {
      print("init message");
      if (message != null) {
        print("init not null message");
      }
    });

please let me know if any one facing same problem , and I'm ready to give more information to fix this issue
Thanks alot

Hi @mustafa-707
What do you mean exactly here? Is it that when you open the terminated app with the notification you are not getting the print? Logs are not printed in a release build or after terminating the app on iOS so I'm not sure how exactly you are checking for this.
Could you try running the official example? It should navigate to a notification screen if the initial notification is not null (in initState).
https://github.com/FirebaseExtended/flutterfire/tree/master/packages/firebase_messaging/firebase_messaging/example/lib
I just checked this on a real iPhone 7 with firebase_messaging: ^9.1.3 and it works as expected.
Thank you
Thank you

still returning nulls getInitialMessage  after testing it on iOS 14 and 12,4 with new updates
  firebase_messaging: ^10.0.0

and flutter 2.2.0-stable

still returning nulls getInitialMessage  after testing it on iOS 14 and iOS 12 with new updates
  firebase_messaging: ^10.0.2

and flutter 2.2.2-stable

same issue here

Same issue here

finally solved by removing a plugin named flutter_apns also i checked removed an iOS  named FirebaseAppdelegateProxyEnable and now the getInitialMessage works as well as android on iOS


Split rubyTest into vanilla tests and tests with _strptime monkey patch

@muga For adding more tests of TimestampParser even in the Ruby world, let me split rubyTest into two parts at first.

test/vanilla is long-lived tests without any monkey patch.
test/monkey_strptime is recently added MRI's test imported with monkey patching Date._strptime.

Thanks! Merging.


Help maintaining repo

@JoelBesada I've noticed no new commits or merged pull requests in ~4 months. If you need help maintaining this repo, I will gladly help test & merge PRs if you add me as a contributor.
This is an awesome plugin and I'd love to see it progress. I think the rest of the community would as well.

@nathanchapman Hi, thank you for offering to do this! I've added you as a contributor to the repository now.
The main reason I haven't been actively working on this is because I haven't yet been able to transition to using Atom as my every day editor. This makes it harder to find the motivation to continue on it, since the plugin isn't something I'm able to use myself.
If anyone else is reading this and would also like to help out as a contributor, please let me know.

@JoelBesada Thanks Joel! I'm in midterms right now. I'll get started with some testing & merging in the next week or two

Hi! I would like to help too.
@nathanchapman do you have any plan regarding to the merging and testing in mind?

@lexcast A lot of the bugs are with newer versions of Atom and have to do with particles rending in the wrong spots or not in multiple panes. I have to finish up this last stretch of midterms and then I'll start on testing and merging. I'm sure @JoelBesada wouldn't mind adding you as a contributor. There's a considerable amount of work that needs to go into the current pull requests (plugin system, etc.) and fixing the rendering bugs. We just need to keep track of who's working on which PRs and issues with the github assignee feature

I definitely appreciate the help! I've added you as a contributor too @lexcast.

Thanks a lot.
When I was working on my PR I found some calls to deprecated methods, I will see if the rendering on wrong positions is because of that.


Editor and IPython console are sometines out of sync in debug mode

Description of your problem
In debug-mode, the shown implemented code in IPython Consule is different from the highlighted code in the Editor windows.
What steps will reproduce the problem?
Run code in debug mode and observe both editor windows and IPython Console window, they are sometimes different.(sometimes means: If there are comments, blank lines in one function. We use step into the function to see the procedure of this function) Actually, the line-number they pointed are the same. But if there are some comments in the code, which occupy some lines. The Pointer in Ipython windows may skip it while the Pointer in Edit window will continue on the next line.
What is the expected output? What do you see instead?
They should be the same.
Please provide any additional information below
Versions and main components

Spyder Version:Spyder 3.1.4
Python Version: Python 2.7.12, 64bits
Qt Version:4.8.6
PyQt Version: 4.10.4
Operating system: Linux

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
jedi >=0.9.0     :  0.10.2 (OK)
matplotlib >=1.0 :  2.0.2 (OK)
nbconvert >=4.0  :  5.1.1 (OK)
numpy >=1.7      :  1.12.1 (OK)
pandas >=0.13.1  :  0.20.1 (OK)
pep8 >=0.6       :  1.7.0 (OK)
psutil >=0.3     :  5.2.2 (OK)
pyflakes >=0.5.0 :  1.5.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.7.1 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
rope >=0.9.4     :  0.10.5 (OK)
sphinx >=0.6.6   :  1.6.1 (OK)
sympy >=0.7.3    :  None (NOK)

Thanks for reporting. We didn't know this was happening and we'll try to fix it after we release Spyder 3.2,


Regular sync v2

Still WIP since I didn't yet managed to reach altona head, but looks promising and stable

@mpetrunic @tuyennhv What's the status of this? Should it still be pursued?

@mpetrunic @tuyennhv What's the status of this? Should it still be pursued?

This was open just to provide some ideas, I think it's better to start from scratch at this point


Migrate to github actions

We should migrate from travis CI to github actions because it's better integrated in github.

closed by #140


Initial drop with new sdk dependency

Issue: There is a reference to sugar-crm connector in the readme
Otherwise it's good to go


Re-added ability to link manufacturer part to supplier part via API

@SchrodingersGat In #1836, the ability to link a manufacturer part to a supplier part using:

manufacturer ID
manufacturer MPN

was removed.
The idea was to create a shortcut in the creation of the supplier part, not having to find out which manufacturer parts are associated to the same part on the client side. In this case the validation of link is done on the server side and, if both information above are correct, the supplier is automatically connected to the corresponding manufacturer part.
Did you intentionally intend to force the client to retrieve the list of manufacturer parts, then get the correct manufacturer part id, then link it directly to the supplier part? Or are you open for shortcuts in this process? (if you have any better idea than the one presented here, let me know!)

The removal was intentional, as part of the migration to the new "API forms" approach.
While I was happy with the original implementation at the time, I don't like that it's "different" to how all other forms are managed - i.e. a single form could touch two different database tables in an opaque manner. In particular if the user makes a slight typo in the MPN, they will not know that it will result in a new manufacturer part being created.
The "ideal" shortcut here would be to re-implement the "secondary" modal forms in the API forms. i.e. if the required MPN data is not already in the available list, you can simply launch a secondary modal form on top of the current one to create the new manufacturer part, and then once that form is completed, the data is filled back into the original form.
Re-implementing the secondary forms has run into a road-block and is an outstanding task here - https://github.com/inventree/InvenTree/issues/1755
Once I've worked out how to implement that, it should be relatively simple to fix that here.

@SchrodingersGat I understand you are concerned about "API forms".
However my original question is related to external API calls using the inventree-python package.
I would have to review the code again but I don't think it was creating any new Manufacturer part on its own, it was just a way to identify a manufacturer part filling in both the name and part number.
When you don't have the UI form, it makes it easier to pass on this data (that you get from as spreadsheet or a supplier search) than having to fetch the manufacturer part ID, the fetching of the manufacturing part ID can be done on the server side.
Where do you think we can add this, if not in the serializer and save method?

Fair enough, I guess I did not fully understand the use case. If you want to submit a PR to re-introduce that feature into the API endpoint I'd be happy to merge back in.


Complete Whitespace cases

Following #185, I've completed the list of cases to match CharacterSet.whitespace and CharacterSet.newlines.
I've added 1 horizontal case (U+1680).
I've added 3 vertical cases (U+000B, U+000C, and U+0085).
For consistency, I've replaced .init(ascii:) occurrences with their corresponding hexadecimal literal value. I've also used hexadecimal literals instead of decimal when they match UTF-8 codepoints. I can switch to all decimal/hexadecimal if preferable.
I've also added a few comments to improve readability.
I can reverse these last changes if needed.
I didn't add new tests as the current implementation isn't exhaustive either. I can add a few ones if needed.

Ah, I ran the tests, but not the benchmarks. I'll look into it.


Minor changes to fix issues #117, #115, #114, #112 and #111

All good edits so far. I will wait to merge to main as James has said he might add more commits to the same PR.


Resolve animation if indexAngle is NaN

Resolves #10019
https://codepen.io/leelenaleee/pen/MWEEmXb

It saved more bits as I expected, interesting


Uses [[ for template, removes / for templates route

Shopping cart for the product listing is broken at the moment, this is a fix.

@sebastianbenz PTAL


Delete operation missing for contained target navigation properties to collections of entities.

For navigation properties we currently only seem to generate GET/PATCH/POST
  // navigation property (Get/Patch/Post)
            _handlers[ODataPathKind.NavigationProperty] = new Dictionary<OperationType, IOperationHandler>
            {
                {OperationType.Get, new NavigationPropertyGetOperationHandler() },
                {OperationType.Patch, new NavigationPropertyPatchOperationHandler() },
                {OperationType.Post, new NavigationPropertyPostOperationHandler() }
            };

https://github.com/microsoft/OpenAPI.NET.OData/blob/master/src/Microsoft.OpenApi.OData.Reader/Operation/OperationHandlerProvider.cs#L63
I think we also need to implement delete for these.  And deleting contained entities is going to have a different URL than non-contained entities.  Can you confirm @xuzhg  ?

? I think so.
See this line:
https://github.com/microsoft/OpenAPI.NET.OData/blob/master/src/Microsoft.OpenApi.OData.Reader/PathItem/NavigationPropertyPathItemHandler.cs#L76
I try to recall the discussion:

For Containment navigation property

Contained Navigation Property
Operation

Collection
Get, Post

Single
Get, Patch

For non-Containment navigation property

Non-Contained Navigation Property
Operation

Collection
Get

Single
Get

So, how about for the delete?

Contained Navigation Property
Delete Operation?

Contained Collection
delete❌

non-Contained - Collection
delete ❌

Contained - Single
delete ✔

non-Contained - Single
delete ✔

I'd like to involve @Michael Pizzo for his input.

@xuzhg yes, that looks like what we would want.  /cc @mikepizzo

I agree that DELETE should be supported to a single-valued contained nav prop provided that the property is non-nullable (i.e., cardinality 0..1)
I'm not sure I'm reading Sam's last table correctly; I would expect DELETE to be supported against a contained collection-valued navigation property.
Are the values for Contained-Collection and non-Contained-Single in the table reversed?

Synced with Mike offline, we have the following confirmed table.
Contained:

Single-Valued
DELETE ~/entityset/{key}/contained-single-valued-Nav

collection-valued
DELETE ~/entityset/{key}/contained-collection-valued-Nav/{navKey}

Non-contained:

Single-Valued
DELETE ~/entityset/{key}/single-valued-Nav/$ref

collection-valued
DELETE ~/entityset/{key}/collection-valued-Nav/$ref?$id={navKey}

Nested navigation property:

Single-Valued
DELETE ~/entityset/{key}/contained-single-valued-Nav/single-valued-Nav/$ref   DELETE ~/entityset/{key}/contained-collection-valued-Nav/{navKey}/Single-Valued-Nav/$ref

collection-valued
DELETE ~/entityset/{key}/contained-single-valued-Nav/{navKey}/collection-valued-Nav/$ref?$id={navKey}   DELETE ~/entityset/{key}/contained-collection-valued-Nav/{navKey}/collection-valued-Nav/$ref?$id={navKey}

@darrelmiller
I added the delete support at: https://github.com/microsoft/OpenAPI.NET.OData/commit/a7ee00dbf4925aeaadfc92c9dd13102ba80d1b98
Would you please try it and verify whether the implementation meets your requirement?


Update setup.py

Adding description string

@lowks Could you rebase from upstream? CI fixed after https://github.com/spoqa/nirum-python/pull/50 merged.


Continue reading resets to a different chapter then last read

This is how it is initially

After opening a different novel

The values at the history sections are right though.

Found the problem and fixed it. It should also fix sort/filters getting reset.


NPM release?

Hello, how far do you think we're from a NPM release?

Hello, I don't know well the npm ecosystem. I'll ask someone how to improve this.

I asked @yovanoc and we have a clean release on https://www.npmjs.com/package/seguid
What should I do with https://www.npmjs.com/package/seguid-javascript ? Delete it?

Thank you both.
I'm looking at the README; what does "esm" and "cjs" stand for?  I think we should spell them out and give some more information.

What should I do with https://www.npmjs.com/package/seguid-javascript ? Delete it?

I guess it can be deleted

I guess it can be deleted

done

I'm looking at the README; what does "esm" and "cjs" stand for? I think we should spell them out and give some more information.

https://dev.to/iggredible/what-the-heck-are-cjs-amd-umd-and-esm-ikm
it's part of the JS ecosystem mess. People who need them know them, others can just ignore them.

it's part of the JS ecosystem mess. People who need them know them, others can just ignore them.

I disagree; we should avoid adding to/boosting the "mess". They add confusion and ambiguity to people who don't know what they are - it's not much work to spell them out too and add some clarification.


Allow for variable substitution in simple argument parser strings

It didn't before because it was just meant to be "simple," but given that it's actually pretty useful I think simple argument parser would be lacking without it

Currently simple argument parser just uses JSON.parse so that fancy escape sequences work; the bashlike parser doesn't support escape sequences other than \$ \\ and escaping the quote


opendungeons: init at 0.6.0

Things done

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Dedicated PR for cegui: https://github.com/NixOS/nixpkgs/pull/14939


Support non-enumerable properties by deepMerge and deepClone

The current solution supports only enumerable properties by deepMerge and deepClone. The for-in bases are sources[i] and subject. I'd like to support cloning non-enumerable properties e.g. stack, id, etc... I would support it only manually.
deepCopy(subject, {nonEnumerable: function (){}});
deepCopy(subject, {"@property:non-enumerable": ...});

By using the subject as for-in base, it won't affect non-enumerable properties. I have to think about it. Maybe @property should check non-enumerable properties as well and @property:enumerable would be a selector specific to enumerable properties.

I could use
function getAllProperties(obj){
    var allProps = []
      , curr = obj
    do{
        var props = Object.getOwnPropertyNames(curr)
        props.forEach(function(prop){
            if (allProps.indexOf(prop) === -1)
                allProps.push(prop)
        })
    }while(curr = Object.getPrototypeOf(curr))
    return allProps
}

which recursively find non-enumerable properties. This is a crude workaround.

another possible solution to check each property in the config and if it was not in the for-in loop, check with getOwnPropertyDescriptor() or simply with o[p] !== undefined

I am not sure I need this. toString and id are cloned by Object.create. stack has a setter and a getter. I don't know of a non-enumerable property which is nested. I try to avoid to use them because enumerable:false is not implemented well in most of the environments.

I'll reopen when it is necessary to implement this.

The @native will be the keyword of this and #72 . It will be used for non-enumerable properties which are keywords or something like id or Object.toString, Error.stack, etc... they won't have a wildcard like @property by the enumerable properties... It will be harder to write deepClone and deepMerge options object by mergeOptions and buildOptions (if I really want those objects), but I can live with that.


rollback wallet selector version and add bitte wallet support

Doesn't work for me locally either. Rolled back master in #242
Added changes in #243


Additionally test on Python 3.6

Supporting Python 3.6 officially would be nice, given how awesome 3.6 is. :)

Thanks for the contribution! 🎉


Support data objects belong to a collection

Solve #394. See issue for details.
Summary, this PR:

Enables the collection_name argument to ExportData
When collection_name is given, add it to relations.collections in outgoing metadata
Add definition of relations.collections in schema

This allows for multiple objects to be exported with a common tag defining a collection they all belong to. The purpose of this is for clients to quickly identify other data objects related to a data object. The produced relations.collections will be uuid4 and identical within a case.
This is done by taking the provided collections_name argument and combining it with the current fmu.case.uuid.
Use case: When looking at a volume table, identify if the same volumes are represented as a 3D parameter or a surface.
Clients would do this through the following workflow: Given a data object, if "relations.collections" is present, find other data objects with the same reference.
There are some unanswered questions:

[ ] A requirement is that a data object can belong to several collections. Hence, relations.collections is a list. But does this work in reality? E.g. if a data object has 3 entries in relations.collections - how will the consumer know which one he wants to unravel?
[ ] What about pre-processed data that belongs to a collection? The fmu.case.uuid does not exist when these data are made, hence relations.collections must probably be remade when dealing with preprocessed data.

What about pre-processed data that belongs to a collection? The fmu.case.uuid does not exist when these data are made, hence relations.collections must probably be remade when dealing with preprocessed data.

I think that is solvable

I have problems understanding how we are actually going to solve the multicollection part. In theory I think the idea is good, but in practice I cannot really see how this will be done. Imagine that you have a specific grid property. This can be part of several collections:

As part of the export of a 3D grid and related properties
As part of properties used as input to inplace calculations
As part of input to seismic forward modelling
Not thinking properly about this one would set these exports as three different scripts utilizing ExportData, should we then allow for checking that the object is exported already and then just writing to the connected metadata file? Or do we imagine people thinking this through upfront, and then adding all these x number of collection names in one script. I foresee that this will not scale all that well...

The alternative is that one exports this grid property three times with different collection name, or something similar..


Bug in cs Utility.HashBytes

wrong:
        if ((len & 1) > 0)
        {
            char* pC = (char*)pwString;
            hashState = magicno * hashState + (ulong)*pC;
        }

correct:
        if ((len & 1) > 0)
        {
            byte* pC = (byte*)pwString;
            hashState = magicno * hashState + (ulong)*pC;
        }

Best regards & many thanks

This method wasn't actually used inside FASTER, but might have been a problem if applications were using it to compute hash codes of unsafe byte chunks. Fixed and merged, thanks!


Refactored Kafka src and ch code to reuse same consumer group handler code

Replaces #551
Fixes https://github.com/knative/eventing/issues/944
Fixes #552
Proposed Changes

Removed sarama-cluster lib
Refactored Kafka channel and source to share same code for handling incoming kafka messages
Now kafka source commit the offset when the message is malformed

Release Note
consumerMode property in kafka broker config-map is no longer supported

/ok-to-test

/retest

@slinkydeveloper
to the dispatcher_test.go file, please add:
func TestNewDispatcher(t *testing.T) {

	args := &KafkaDispatcherArgs{
		ClientID:     "kafka-ch-dispatcher",
		Brokers:      []string{"<IP_ADDRESS>:9092"},
		TopicFunc:    utils.TopicName,
		Logger:       nil,
	}
	_, err := NewDispatcher(args)
	if err == nil {
		t.Errorf("Expected error want %s, got %s", "message receiver is not set", err)
	}
}

/test pull-knative-eventing-contrib-go-coverage

/lgtm
/approve

/lgtm

/approve

/assign @n3wscott

/approve

/retest

Looks like Go is complaining about data races in ConsumerFactory tests

My bad, in sarama documentation it says that if kafkaConfig.Consumer.Return.Errors is true, then the consumer errors are notified directly through the channel. I suppose we can just comment the error when we call consumerGroup.Consume

/lgtm
/approve


Fixing e2e tests

Description
Please explain the changes you made here.
Which issue(s) does this PR fix

Fixes #?

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/lgtm
/approve

I guess we can link this PR to https://issues.redhat.com/browse/RHIDP-2793

/lgtm

/test e2e-tests

/test e2e-tests

/test e2e-tests

/lgtm


mysql warnings

hello after latest update mysql warnings show in console.
im using this in server.cfg
set mysql_debug "File"
and it doesnt work, i can still se it in console + creating new log file every restart.

Duplicate of #136


Fix CI

Update chromedriver
Update ubuntu version because latest is not compatible with the current Erlang/OTP versions. See: https://github.com/marketplace/actions/setup-erlang-otp-with-optional-elixir-and-mix-and-or-rebar3#compatibility-between-operating-system-and-erlangotp

It would be great to support latest versions of Elixir/Erlang and maybe deprecate others? I'll try to make a PR with that shortly.

Thanks for taking care of this ❤

It would be great to support latest versions of Elixir/Erlang and maybe deprecate others? I'll try to make a PR with that shortly.

@mcass19 Yes, I think we should deprecate the ones that are not longer supported


Order Gemfile alphabetically

Ordering the Gemfile in this way makes it easier to scan. It also avoids any confusion about where a new Gem entry should be placed.

Exclude Rails from ordering since it is the core of the app.

Thanks for all the contributions @andyw8 ! :+1:


Mark SMs as read when they are read

Do this via this API method:
GET /messages/read?messageid=12345

Or should we have a button to do it? Or an option? (since the shack doesn't provide a "mark as unread" feature)

Personally I'd like to mark it read immediately.


"flag" attributes

Hello !
The djot syntax reference does not allow flag attributes (attributes without a value, or empty attributes, as they are called in html).
I think those flag attributes are useful! Is there a reason for not supporting them?
(I would have the same question for pandoc's markdown, but since pandoc has so many more features, I though asking in this syntax-specific repo was better).
Anyway, thanks a lot for all your work on markup languages!

I'm open to reconsidering this.  My thinking originally was that if we allow flag attributes there is greater chance of capturing something that was meant as regular text, e.g. {this is just plain text in braces}.  But if we assume that braces are very rarely used in regular text, this isn't a big problem.
Semantically, how should flag attributes be represented?  In HTML, checked is semantically equivalent to checked="checked" and also, I think, to checked="".  So, we could parse a bare checked as a key-value attribute checked=checked.

A possible value for the representation of flag attributes in the AST would be true: element.checked would be true in boolean context but you could also do if ( element.checked == true ) to distinguish {checked} from {checked=checked} for better roundtripping.
In Pandoc Markdown I always use backslashes or verbatim for non-attribute braces but TBH I also always disable shortcut reference links so that I don’t have to remember to escape brackets around phonetic transcriptions! I do occasionally use braces around shorthand transliterations[^1] but I use verbatim with a class for those. I would believe that braces are even more unusal in text than brackets but there probably is some discipline which uses braces a lot! However I don’t think djot should disallow flag attributes because of that.
[^1]: Because shorthand is difficult to reproduce in electronic documents it is common to use a longhand transcription like {InTrODUKTion} where {In}, {Tr},  and {Tion} represent a single shorthand grapheme each, i.e. each shorthand grapheme is represented by an uppercase letter followed by zero or more lowercase letters. Some punctuation and symbols are used as well.


Initialise Firebase App

Basic Information

Setup firebase config
Make sure all key-values are stored in a .env file
Setup up a .env-template file with only the keys displayed (this should be added to github)

@ChantelleKerr I wanted to work on this
but I will need keys for that

@ChantelleKerr I wanted to work on this
but I will need keys for that

Hey, unfortunately the keys are only available for our project members


Add arm build target

Hi
I'd like to add arm v7 support to the problem detector builds and container image.
Could you let me know if those changes are sufficient?
I'm not particularly sure about CC=arm-linux-gnu-gcc, should it be gcc-arm-linux-gnueabi?
Thanks

/remove-lifecycle stale

/remove-lifecycle stale


fixed a bug where the component doesn't update when passing other props

fixed a bug where the component doesn't update when passing other props (onBlur for example it uses an old function which causes the state to go back to an old state )

The aim was not to rerender with prop changes, but I see your request, It would be better if I wrote it some other way, but your change will break the performance of the component. If you have specific prop which needs to reflect the change we should add just that property


in tauri worked failed

drop not working
Tauri
navigator.userAgent
"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko)" = $2

Looks to potentially be a tauri issue, not a react-dnd issue.
https://github.com/tauri-apps/tauri/issues/3277
Try setting "fileDropEnabled": false in tauri.conf.json. I managed to get this working with react-beautiful-dnd but not react-dnd. They must function differently under the hood.

Looks to potentially be a tauri issue, not a react-dnd issue.
tauri-apps/tauri#3277
Try setting "fileDropEnabled": false in tauri.conf.json. I managed to get this working with react-beautiful-dnd but not react-dnd. They must function differently under the hood.

I set "fileDropEnabled": false as you say, perfectly solved my problem, thank you very much！

Try setting "fileDropEnabled": false in tauri.conf.json. I managed to get this working with react-beautiful-dnd but not react-dnd. They must function differently under the hood.

This worked for me too, using react-dnd. The documentation says that this would fix dnd on Windows only, but it seems to be required on macOS too.

I had this problem when using react-dnd on my Mac, so I looked it up and arrived at this page and fixed the problem by setting fileDropEnabled to false.
I think it's okay to add notes to the document until you have a drastic solution.

邮件已收到，谢谢！

看起来可能是一个tauri问题，而不是一个react-dnd问题。
tauri-应用程序/tauri#3277
尝试设置"fileDropEnabled": false在tauri.conf.json. 我设法使它与react-beautiful-dndbut not一起工作react-dnd。它们在引擎盖下的功能必须不同。

Tanks ! solved my problem !

邮件已收到，谢谢！

Looks to potentially be a tauri issue, not a react-dnd issue.
tauri-apps/tauri#3277
Try setting "fileDropEnabled": false in tauri.conf.json. I managed to get this working with react-beautiful-dnd but not react-dnd. They must function differently under the hood.

Thank you! This works like a charm!

邮件已收到，谢谢！

@lukethacoder  Thank you for sharing, you comment fixed my problem as well.

Looks to potentially be a tauri issue, not a react-dnd issue.
tauri-apps/tauri#3277
Try setting "fileDropEnabled": false in tauri.conf.json. I managed to get this working with react-beautiful-dnd but not react-dnd. They must function differently under the hood.

I set "fileDropEnabled": false as you say, perfectly solved my problem, thank you very much！

This is woking perfectly!! Thank you!!! <3

邮件已收到，谢谢！


https://playground.talao.co/discount-demo

there "random_challenge" which is display at the bottom of the screen

It is a debug message for ensuring connectivity with the server. All debug messages will be removed after the project finishes.
I created a more general issue here #30 so as not to forget.


add odf instance example

Created an example instance of ODF intended to work with AWS (tested on RHPDS).
Additionally created a config-helpers folder that contains a job to help label the worker nodes.  I went back and forth on if this should be included in the aws overlay or a separate aws-node-labeler overlay and ultimately settled on creating the config-helpers folder.  The node-labeler should work with any future instance overlays and keeping it in the config-helpers will prevent the files from having to be copied to multiple instances in the future.  I also didn't want to include in the in the base because it may not be wanted for some ODF deployments where other methods are being used for labeling the nodes.
Since this folder is a new pattern in the gitops-catalog I welcome any feedback!

I like it.  Thanks for the addition.


Adds handling of miss-matching blank spaces in device names

Fixes bug that caused the calculation_k_tab function in corrections.py to crash when Allura Clarity was spelled with a blank space between the two words instead of in Pascal casing as used in the database. As discovered in #27
Also fixes a unit test of the calculate_k_tab function that previously did not actually check the output of the function.
Solves #41

Codecov Report

Merging #43 (1b35f56) into master (2dfa031) will increase coverage by 0.75%.
The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   64.82%   65.57%   +0.75%
==========================================
  Files          42       42
  Lines        1555     1560       +5
==========================================
+ Hits         1008     1023      +15
+ Misses        547      537      -10

Impacted Files
Coverage Δ

src/pyskindose/constants.py
100.00% <100.00%> (ø)

src/pyskindose/corrections.py
95.91% <100.00%> (+20.91%)
:arrow_up:

tests/unittests/test_corrections.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Removed invalid CBR option and channel layouts + added VBR option.

[MP3] Added LAME's highest quality VBR option ('-V 0') and renamed the MP3 VBR options. The maximum bitrate with MP3 is 320k, so I have removed the 448k option. Also, MP3 does not support more than two channels.

I wanted to say thank you. I am going to delete the last commit and accept this pull request first thing in the morning. Im very new to this and I didn't mean to do that.
I will definitely get this fixed!


extend header for 'packages' detailed list under Config List

extend header for 'packages' detailed list under Config List

done


Add membership api doc

In the Messaging API, you can now get membership information.

add /membership/v1/subscription/{userId}
add /membership/v1/list

News: https://developers.line.biz/en/news/2024/02/09/get-membership-plan-informations/

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Connect to ipv6 URI fails

Example: tcp://[2a03:2880:f02d:12:face:b00c:0:3]:443
It's interpreted as a host name which then is not valid of course

Thanks!


Explore Pombase spot, communty curation takeover

Although not strictly an "Explore PomBase"
Can we have something along the lines:
Community Curation: Most suitable  fission yeast papers are now author curated ~PomBase users now curate their own papers~. Find out how you can benefit in our new publication "Community curation in PomBase: enabling fission yeast experts to provide detailed, standardized, sharable annotation from research publications" (https://academic.oup.com/database/article/doi/10.1093/database/baaa028/5827230#.XqvRaBbbKsY.twitter)
and take the plunge here https://curation.pombase.org/pombe
pinned to this slot

@mah11 can you word this in such a way that it is true without it being as wordy.
It isn't  'most authors' its most suitable publications

pinned to this slot

does this mean take everything else out of the rotation?

does this mean take everything else out of the rotation?
yop, but I don't think it is rotting at the moment anyway  (the open biology paper is currently  'pinned')

rotating!

don't think it is rot[a]ting at the moment anyway (the open biology paper is currently 'pinned')

not true

I've put it in without the paper title, because that's what was making it so long & wordy. I can add it back if you think it's important enough to take up the space. (Closing this in the meantime tho.)

Yeah that was definitely too much text for that box!


Update readme

Updated requirements in the README.

Please rebase your commits to collapse them in to a single commit:
https://github.com/j2objc-contrib/j2objc-gradle/blob/master/CONTRIBUTING.md#rebasing-and-merging


Egui Drag an Drop shards

Opening a PR so I don't forget about this.

Added inline shard support for TryMany, StepMany & Expand + tests.


Add command to un-rewrite import paths

Add a command to switch from the vendored import paths to unvendored (pristine) paths. The reverse of 'godep save -r'. Being able to switch back and forth easily would allow for the benefits of the rewritten paths (go get-able, work with go tools) while still allow the benefits of the un-rewritten paths (easily switching between different versions, cleaner source, vendored packages in branch).

:+1:

:+1:
I'm missing this so badly when modifying dependent libraries.

Does godep save -r=false not accomplish this?

It does something like that, but for all vendored libraries, not a single one (at least for me it simply ignored the given package).

@gfrey I'm not sure what you mean by "given package".

I've vendored some large libraries and would like to be able to just "devendor" a single of them, like in godep save -r=false github.com/dynport/urknall. This would keep everything else fixed, keep the clutter in git diff minimal and allow for easy manipulation of a single library.
Not sure how dependent libraries of the dependency should be handled though.

So you don't really want to unrewrite the import you want to remove it? If you remove all of the import statements imports a dependency and re-run godep save -r any no longer used imports will be removed from the workspace. The arguments to godep save aren't the packages that you want to vendor, they are the package spec(s) that you want to analyze for vendoring.

In the example given (godep save -r=false github.com/dynport/urknall) I don't want to totally remove the usage of the urknall package (what I guess you understood), but un-rewrite the import paths, so that the version in my $GOPATH is used again, instead of the one in the Godeps/_workspace.
Benefit would be, that I'd not have to commit changes to the urknall library and use godep update github.com/dynport/urknall to test them in the respective binary.
When being finished with changing the urknall library, I'd just godep update github.com/dynport/urknall && godep save -r ./... to get everything back to normal in the binary.

@gfrey that would confuse the meaning of the package spec passed to godep save, which is the packages you want to analyze dependencies for, usually '.' (the default), './...', or './cmd/...'. To be honest rewrites are going away as we approach Go 1.6 and the use of the vendor/ directory, so I'm tempted to close as won't fix with the expectation being that you would use the vendor directory directly w/o rewrites anyway. At that time a godep rm command (which doesn't exist) would solve your problem (or just rm stuff manually from vendor/). ATM godep save -r=false <your local package spec> will un-rewrite everything so you can still do what you want, it will just take a few extra commands before or after.
Does that make sense?

If rewrite is going away anyway you are right (maybe I should put some focus on Go project politics), so I'm fine with issue being closed. Keep in mind I'm not the OP!


Clicking the menu icon doesn't toggle the menu but clicking the menu text does

Describe the bug:
Clicking the actual hamburger menu icon doesn't toggle the menu, clicking the text "Menu" does. When clicking the icon my graph cards seem to redraw but the menu doesn't slide in/out of view.
Config YAML:
- type: custom:compact-custom-header
  menu: overflow
  options: clock
  voice: hide

HA and/or Browser Error:
No errors in the HA logs or in the dev console.
HA, Card, and Browser Versions:
HA 0.89.1
Card 1.0.1b3
Browser Chrome 72.0.3626.81, same thing happens in Safari 12.0.3.

This is an odd one, I'm able to reproduce it. The icon works in mobile for me, but not on desktop.

Is this a mwc-button? I'm pretty sure that's an upstream bug.

Appears the HA header still uses paper-icon-button's for the header. CCH just moves it.

Pretty sure it's my problem though as it works fine before the move to overflow menu.


CSS from styled-components is not loading

Hi, I've noticed that SSR does not include generated styles from styled-components. I've added babel-plugin-styled-components, but it does not work. According to docs:
https://styled-components.com/docs/advanced#server-side-rendering , the style tag is being generated but is empty.
const sheet = new ServerStyleSheet()

function renderToString(el){
    return ReactDOMServer.renderToString(sheet.collectStyles(el));
}

createServer((page) => createInertiaApp({
    page,
    render: renderToString,
    resolve: (name) => require(`./Pages/${name}`),
    setup({App, props }) {

        return (
            <Provider store={store}>
                <App {...props} />
                {sheet.getStyleElement()}
            </Provider>
        )
    },

I'm sorry. The code I've posted finally worked. If you have similar problem feel free to use it :)


Can I apply filters to camera streams?

Thank you for developing such a great project.
Applying filters to files works perfectly.
I want to apply this filter to the camera stream as well.

I tried it and almost succeed.
When initializing the camera, set it to yuv420 for Android and bgra8888 for ios.
Create a MethodChannel and write a function in each native code that receives CameraImage and converts it into UInt8List data in jpeg format.
After executing the image conversion function through the method channel in the startImageStream function of the camera, provide this result as a factor for the TextureSource.fromMemory function.
After that, you can apply the configuration.

I tried it and almost succeed.
When initializing the camera, set it to yuv420 for Android and bgra8888 for ios.
Create a MethodChannel and write a function in each native code that receives CameraImage and converts it into UInt8List data in jpeg format.
After executing the image conversion function through the method channel in the startImageStream function of the camera, provide this result as a factor for the TextureSource.fromMemory function.
After that, you can apply the configuration.


needs test cases

before doing anything new. no regression!

mocha, chai, sinon, oh my! or jest? The former, because... nyan cat! :smile_cat:


Option to not focus live search when opened

Hello,
I would like to be able to include a search field but not have it focused when the control is opened. This is quite annoying on devices with onscreen keyboards where we do not want to go back to just using the native control.
I can't see if this is possible from reading the documentation or a quick look over the code.
Regards

+1

@caseyjhol can you please tell me where is solution for  this?

Hi Guys ,
After looking at the source code .
These lines gives focus to searchBox in the case of Live Search..
Source Lines for Focus
Either you can comment it out for permanent effect .you can pass additional option to recognize & then disable it .
Hope this will help you out .
Let me know

I have the same problem. When I'm on an IPAD and I select a value in the list the search field is focused and the onscreen keyboard pops open. This is very annoying.
can you remove this behavior?

Will this solution ever be merged into the project? Downloading and manually editing the source file isn't a good approach.

/* Hack for focus blur from liveSearch input */

var blurState = false;

$('.bs-searchbox .form-control').on('focus', function() {
    if (!blurState) {
        $(this).blur();
        blurState = true;
    }
});

$('.selectpicker').on('hide.bs.select', function() {
    blurState = false;
});

I tried fixed the js file directly!
File name(bootstrap-select.js)
function setFocus() {   that.$menuInner.trigger("focus"); }

I tried fixed the js file directly!
File name(bootstrap-select.js)
function setFocus() {   that.$menuInner.trigger("focus"); }

/* Hack for focus blur from liveSearch input */

var blurState = false;

$('.bs-searchbox .form-control').on('focus', function() {
    if (!blurState) {
        $(this).blur();
        blurState = true;
    }
});

$('.selectpicker').on('hide.bs.select', function() {
    blurState = false;
});

感谢这位老哥的想法,我需要移除在移动设备上自动聚焦弹出键盘的问题,以下是我修改的解答（如果是移动设备,除了直接点击输入框，一律将聚焦打散）,我的bootstrap-select版本为1.13.18（因为1.14还有未发布的问题）
Thanks to this brother's idea, I need to remove the question of automatically focusing on the pop-up keyboard on the mobile device. Here is my modified answer (if it is a mobile device, it will all be focused except by clicking on the input box directly). My bootstrap-select version is 1.13.18 (because 1.14 still has unpublished questions).
        //全局变量默认拦截聚焦//Global variable default intercept focus
        window.bssearchboxBlur = true;

        $('.bs-searchbox .form-control').on('focus', function() {
            if (bssearchboxBlur&&navigator.userAgent.match(/Android|BlackBerry|iPhone|iPad|iPod|IEMobile/i)) { //移动设备生效//Mobile devices take effect
                $(this).blur();//聚焦之后模糊//blur after focusing
            }
        });
        //只有点击事件能聚焦//Only click events can focus
        $('.bs-searchbox .form-control').on('click',function() {
            bssearchboxBlur=false;
            $('.bs-searchbox .form-control').focus();
            bssearchboxBlur=true;
        });

/* Hack for focus blur from liveSearch input */

var blurState = false;

$('.bs-searchbox .form-control').on('focus', function() {
    if (!blurState) {
        $(this).blur();
        blurState = true;
    }
});

$('.selectpicker').on('hide.bs.select', function() {
    blurState = false;
});

感谢这位老哥的想法,我需要移除在移动设备上自动聚焦弹出键盘的问题,以下是我修改的解答（如果是移动设备,除了直接点击输入框，一律将聚焦打散）,我的bootstrap-select版本为1.13.18（因为1.14还有未发布的问题）
Thanks to this brother's idea, I need to remove the question of automatically focusing on the pop-up keyboard on the mobile device. Here is my modified answer (if it is a mobile device, it will all be focused except by clicking on the input box directly). My bootstrap-select version is 1.13.18 (because 1.14 still has unpublished questions).
        //全局变量默认拦截聚焦//Global variable default intercept focus
        window.bssearchboxBlur = true;

        $('.bs-searchbox .form-control').on('focus', function() {
            if (bssearchboxBlur&&navigator.userAgent.match(/Android|BlackBerry|iPhone|iPad|iPod|IEMobile/i)) { //移动设备生效//Mobile devices take effect
                $(this).blur();//聚焦之后模糊//blur after focusing
            }
        });
        //只有点击事件能聚焦//Only click events can focus
        $('.bs-searchbox .form-control').on('click',function() {
            bssearchboxBlur=false;
            $('.bs-searchbox .form-control').focus();
            bssearchboxBlur=true;
        });

#LY

/* Hack for focus blur from liveSearch input */

var blurState = false;

$('.bs-searchbox .form-control').on('focus', function() {
    if (!blurState) {
        $(this).blur();
        blurState = true;
    }
});

$('.selectpicker').on('hide.bs.select', function() {
    blurState = false;
});

感谢这位老哥的想法,我需要移除在移动设备上自动聚焦弹出键盘的问题,以下是我修改的解答（如果是移动设备,除了直接点击输入框，一律将聚焦打散）,我的bootstrap-select版本为1.13.18（因为1.14还有未发布的问题）
Thanks to this brother's idea, I need to remove the question of automatically focusing on the pop-up keyboard on the mobile device. Here is my modified answer (if it is a mobile device, it will all be focused except by clicking on the input box directly). My bootstrap-select version is 1.13.18 (because 1.14 still has unpublished questions).
        //全局变量默认拦截聚焦//Global variable default intercept focus
        window.bssearchboxBlur = true;

        $('.bs-searchbox .form-control').on('focus', function() {
            if (bssearchboxBlur&&navigator.userAgent.match(/Android|BlackBerry|iPhone|iPad|iPod|IEMobile/i)) { //移动设备生效//Mobile devices take effect
                $(this).blur();//聚焦之后模糊//blur after focusing
            }
        });
        //只有点击事件能聚焦//Only click events can focus
        $('.bs-searchbox .form-control').on('click',function() {
            bssearchboxBlur=false;
            $('.bs-searchbox .form-control').focus();
            bssearchboxBlur=true;
        });

The idea here is correct but it may not work for some people as it might happen that the select picker is not initialized and thus the event handlers will not be attached. So I used the above in the following way (Just a small optimizations also - The events are only attached if the user is on mobile) -
`if(/Android|webOS|iPhone|iPad|iPod|BlackBerry/i.test(navigator.userAgent)){//Mobile devices take effect.
window.bssearchboxBlur = true;
            $('#country-dropdown').on('loaded.bs.select', function () {

                const liveSearchField = document.getElementsByClassName('bs-searchbox')[0].getElementsByClassName('form-control')[0];
                liveSearchField.addEventListener('focus', function() {
                    if (bssearchboxBlur) {
                        liveSearchField.blur();//blur after focusing
                    }
                });

                //Only click events can focus
                liveSearchField.addEventListener('click',function() {
                    bssearchboxBlur=false;
                    liveSearchField.focus();
                    bssearchboxBlur=true;
                });

            });
        }`

Hi! I provide my solution.
Adding an attribute to control blur or focus event when ONLY user clicks on input.
In the tag add "nofocus" class
<select name="" id="" class="selectpicker nofocus" multiple data-live-search="true" data-live-search-normalize="true" data-size="10" title=""></select>
Next with setTimeout if it is needed,
setTimeout(() => {                                 console.log('events added to seletpicker no focus');                 $('.bootstrap-select.nofocus').find('.bs-searchbox .form-control').on('focus', function() {                     if( $(this).attr('autoBlur') == undefined ){                         $(this).blur();                     }                     $(this).removeAttr('autoBlur');                 });                 $('.bootstrap-select.nofocus').find('.bs-searchbox .form-control').on('click', function() {                     $(this).attr('autoBlur',1);                     $(this).focus();                 });             }, 500)
I hope its helps


How to access pooler and/or FPN features after forward?

❓ Questions and Help
What is the best way to access pooler and/or FPN features after forward, without inferencing the whole image again?
Thanks!

Found a solution (not sure if it's the best):
  ```
  roi_pools = []
  def get_pooled_rois(module, input, output):
        roi_pools.append(output)
 predictor.model.roi_heads.box.feature_extractor.pooler.register_forward_hook(get_pooled_rois)


Change "Add Item" to "Create Item" in the all items list

Is your feature request related to a problem? Please describe.
It's not super clear this "Add Item" is a different button than the one on the grocery list which brings you to this screen
Describe the solution you'd like
Change wording to create item
Describe alternatives you've considered
Other wording could be used, but hasn't been considered

Good afternoon,
If this issue is still open, please assign it to me?


[Incompatibility] Toil-Player doesn't have turret shown during TME emote

Hey there,
Admittedly this is a weird/funny one.  But I was testing some stuff this morning, and I noticed that when Toilhead mod (https://thunderstore.io/c/lethal-company/p/Zehs/ToilHead/) makes a Toil-Player (spawns a player with a turret -- lol), they don't observe the turret when performing a TME emote.
radar showing turret attached to player

turret attached from observing player's normal view

no turret visible to emoting player

I'm not 100% sure on this, but it's likely because they're rendering the turret on layer 23, which is layer that the game uses to intentionally to hide objects, such as the little girl.
My mod cannot magically render objects on this layer without rendering other unintended objects. For objects and such that do show/hide while emoting/stop emoting, those work because I had to specifically add support for those.
The best way to handle this would be for the creator of this mod to add support, and maybe change the layer of the local player's turret when it detects that the third person camera from the TME mod is active or something, but there could be a few different ways to go about doing this.

Gotcha, I'll pass this along to the Toilhead developer.
I thought about doing that initially, but OpenBodyCams seems to handle the turret already so I thought it might be better addressed from your end.
Cheers

Another thought I had was that I could try to be smart, and when you emote, I would check every object that is a child/grand-child of the local player, and if it's on layer 23, I would show it, but I can see cases where an object like this aren't meant to be shown, so I wouldn't want to implement it in this way either.
I'll continue thinking about this just in case there is a better solution that I could implement that would be future-proof, and would be compatible with most everything.


Feature request: be more careful about removing default modules in other itemchanger modes

Removing default modules can cause issues in other itemchanger modes. For example, in Archipelago, if Transcendence is enabled accidentally, Elder Baldurs will not spit without VS and make greenpath unreachable even though it may be in logic via another baldur killer. There are various ways to do this, including but not limited to:

Add a MenuChanger mode menu for transcendence.

Drawback: loses compatibility with other similar mods in the future

Treat archipelago as an additional special case, that adds the items at their vanilla locations but doesn't do other things

Drawback: requires keeping a list of all the edge cases which may not scale in the future

Make the mod globally togglable (see #3)

Drawback: this is probably clunky (as mentioned on #3)

A specific mode menu for Transcendence doesn't make sense, since the mod is meant to work with all the standard game modes as well as randomizer. Making the mod togglable isn't doable right now as I explained in #3. Not removing the modules makes the vanilla experience, well, not so vanilla. So I think the best solution for the time being is in fact to add a special case to not disable the modules for Archipelago.
If ItemChanger didn't enable any modules by default we'd also have no problems, since Transcendence doesn't want to add any itself.

Turns out ItemChanger actually had a solution for this all along - when creating a settings profile I can make it not create the default modules if another mod hasn't created the profile already. That way if some other mod's using ItemChanger (like rando or Archipelago) we don't mess with modules at all.

this signature for CreateSettingsProfile may be of interest. You'd need to make sure you do this after everything else though, as it seems only the first requester gets priority for default modules

That's exactly what I was talking about. It works out fine for rando at least, because at the point where I call CreateSettingsProfile has created its own already. I don't know about Archipelago; I can't find source code for the mod anywhere.

Ah, you found it while I was typing a comment. Here's AP source, I assume it's probably similar to how rando does it as I believe it's used much of the code as reference but haven't verified
https://github.com/Ijwu/Archipelago.HollowKnight

I had a look and yep, it too, like rando, sets up ItemChanger's settings before calling UIManager.StartNewGame. So our trick works fine for both.

This is (or at least should be) resolved in 1.1.


BeamDyn may not converge WARNING

Hello, everyone.
I'm running simulations of the new IEA 22MW WT using OpenFAST and the following message pops up at the beginning of each of my simulations.
FAST_InitializeAll:BD_Init:BD_ValidateInputData:Damping values in blade file are not of similar
order of magnitude. BeamDyn may not converge!
FAST_InitializeAll:BD_Init:BD_ValidateInputData:Damping values in blade file are not of similar
order of magnitude. BeamDyn may not converge!
FAST_InitializeAll:BD_Init:BD_ValidateInputData:Damping values in blade file are not of similar
order of magnitude. BeamDyn may not converge!
I didn't change anything in BeamDyn, i.e. I'm using the reference BeamDyn input files from the repository.
Could you help me understand and solve this issue, please?
Thank you!
Best.

The same question was posted and addressed here https://forums.nrel.gov/t/iea-22-280-rwt-first-set-of-simulations/7178/7


XLM Stellar Lumens

Ok, with its official stellar-base and other dependencies the resulting .js got pretty big (1M) and I know that's not really appealing. Anyways, here it is...
Specs => https://github.com/stellar/stellar-protocol/blob/master/ecosystem/sep-0005.md
(Main difference is, path usually only contains purpose, coin and account#)
Live => http://bip39.sui.li/src/
Worked with the example from specs and a mnemonic from my ledger.

Thanks for this.
Not sure why there's some json files, can you please help me understand why they're included?
Also what process should be used audit the stellar-util.js file?

Sure.
libs/stellar-util/stellar-util.js is the actual source which contains nothing than deriving the key and converting it to a stellar keypair. It uses the ed25519-hd-key (implements stellar key derivation, don't know much about the lib but lgtm) and stellar-base (part of "official" js-sdk from stellar dev foundation) libraries. Those dependencies as well as the build command are defined in the package.json. You can build the resulting browserified+compressed src/stellar-util.js on your own as stated in libs/stellar-util/readme.md:
rm src/js/stellar-util.js
cd libs/stellar-util
npm install

# (will run `browserify stellar-util.js | uglifyjs -c -m > ../../src/js/stellar-util.js`)
npm run build

Thanks for clarifying.
bitcoinjs-lib, ethereumjs-util etc are not minified / uglified (actually all libraries should not be but some seem to have slipped through!)
Can you please update with the unminified version?
Sorry to keep pushing work back to you but I am not able to directly do work supporting non-bitcoin code because it would become too much.

Shouldn't the MASTER_SECRET be "ed25519 seed" instead of "Bitcoin seed" for Stellar according to here?https://galactictalk.org/d/549-secret-key-seed-from-bip39/15

Shouldn't the HDNode.MASTER_SECRET be "ed25519 seed" instead of "Bitcoin seed" for Stellar according to here? https://galactictalk.org/d/549-secret-key-seed-from-bip39/15

@heri16 Yeah, but most params somehow don't apply to stellar, I'm actually not even using that at all since key derivation is completely done in stellar-util.js. Now I've set a dummy entry to not break things.
@iancoleman Removed uglify.