Include net8.0 tools in StrawberryShake package

Adding StrawberryShake.Server to a dotnet 8 project will generate an error:
Could not execute because the specified command or file was not found.
  Possible reasons for this include:
    * You misspelled a built-in dotnet command.
    * You intended to execute a .NET program, but dotnet-/Users/antonio/.nuget/packages/strawberryshake.server/13.5.0-preview.7/build/../tools/net8/dotnet-graphql.dll does not exist.
    * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.

/Users/antonio/.nuget/packages/strawberryshake.server/13.5.0-preview.7/build/StrawberryShake.Server.targets(79,5): error MSB3073: The command "dotnet "/Users/antonio/.nuget/packages/strawberryshake.server/13.5.0-preview.7/build/../tools/net8/dotnet-graphql.dll" generate "/Users/antonio/Desktop/src/Demo" -o "/Users/antonio/Desktop/src/Demo/obj/Debug/net8.0/berry" -n Demo -a md5 -t" exited with code 1.

The tools directory does not have net8 subdirectory:
> ls /Users/antonio/.nuget/packages/strawberryshake.server/13.5.0-preview.7/tools
net6 net7

Tested on dotnet 8.0.100-preview.6.23330.14 with HC 13.4.0 and 13.5.0-preview.7
This PR adds the missing files to StrawberryShake package

 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.Antonio seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for contributing.


UADK multi-threaded performance issues

UADK multi-threaded performance cannot be increased to the hardware speed limit
8 threads：
algname:        length:         perf:           iops:           CPU_rate:
aes-128-cbc     1024Bytes       1254809.0KB/s   1225.2Kops      785.40%
16 threads：
algname:        length:         perf:           iops:           CPU_rate:
aes-128-cbc     1024Bytes       1420551.0KB/s   1387.2Kops      1564.60%
32 threads：
algname:        length:         perf:           iops:           CPU_rate:
aes-128-cbc     1024Bytes       539262.2KB/s    526.6Kops       3138.20%
In contrast, in the same synchronization mode, warpdrive can reach the hardware speed limit by adding threads:
8 threads：
algname:        length:         perf:           iops:           CPU_rate:
aes-128-cbc     1024Bytes       2970516.8KB/s   2900.8Kops      799.20%
16 threads：
algname:        length:         perf:           iops:           CPU_rate:
aes-128-cbc     1024Bytes       5664155.0KB/s   5531.4Kops      1597.80%
32 threads：
algname:        length:         perf:           iops:           CPU_rate:
aes-128-cbc     1024Bytes       8293766.6KB/s   8099.2Kops      3194.80%

With the debug branch (https://github.com/hzhuang1/uadk/tree/debug_sec_v0.1), the performance data is in below.
start UADK benchmark test.
[--algname]: aes-128-cbc
[--mode]:    1
[--optype]:  0
[--syncmode]:0
[--pktlen]:  1024
[--seconds]: 3
[--thread]:  8
[--multi]:   1
[--ctxnum]:  16
[--algclass]:cipher
[--acctype]: 0
[--engine]:
hisi sec init Kunpeng920!
algname:	length:		perf:		iops:		CPU_rate:
aes-128-cbc	1024Bytes	1569103.4KB/s	1532.1Kops 	799.21%
duty cycle:10.96%, 7.26%, 13.15%, 66.76%
start UADK benchmark test.
[--algname]: aes-128-cbc
[--mode]:    1
[--optype]:  0
[--syncmode]:0
[--pktlen]:  1024
[--seconds]: 3
[--thread]:  16
[--multi]:   1
[--ctxnum]:  32
[--algclass]:cipher
[--acctype]: 0
[--engine]:
hisi sec init Kunpeng920!
algname:	length:		perf:		iops:		CPU_rate:
aes-128-cbc	1024Bytes	2627257.0KB/s	2565.5Kops 	1598.67%
duty cycle:11.45%, 7.16%, 12.44%, 66.90%
start UADK benchmark test.
[--algname]: aes-128-cbc
[--mode]:    1
[--optype]:  0
[--syncmode]:0
[--pktlen]:  1024
[--seconds]: 3
[--thread]:  32
[--multi]:   1
[--ctxnum]:  64
[--algclass]:cipher
[--acctype]: 0
[--engine]:
hisi sec init Kunpeng920!
algname:	length:		perf:		iops:		CPU_rate:
aes-128-cbc	1024Bytes	2491805.3KB/s	2433.1Kops 	3198.24%
duty cycle:38.54%, 5.42%, 7.29%, 47.92%
All three kinds of performance data are increased. There're two reasons at here.

With the commit to remove lock in UADK scheduler.

commit e87ce6d78d6fb24a7cb85221c9b0c3038c9e0374 (HEAD -> debug_sec_v0.1, hzhuang1/debug_sec_v0.1)
Author: Haojian Zhuang<EMAIL_ADDRESS>Date:   Sun Sep 12 08:04:38 2021 +0800

    sched: remove the lock

    Use lock-free to select one ctx from region.

    Atomic operations are used to make sure each operation on region->last
    variable is atomic. Since other global variables are treated as contant
    value. And updating local variables won't change variables in other
    threads.

    Signed-off-by: Haojian Zhuang<EMAIL_ADDRESS>

Keep CTX numbers always greater than thread numbers.

Threads Number
CTX Number

8
16

16
32

32
64

OK


Start TaskSupervisor by default

The reason for this is that if two applications start bamboo and both of
them added Bamboo.TaskSupervisorStrategy.child_spec to their children,
then there would be an error because it was started twice.
This circumvents that by starting it just once when Bamboo starts. This
also simplifies the installation instructions.
Closes #131

an unrelated note: should Bamboo.SentEmail always be there?
You could leave this to the user to start in test cases..?

That is a good question. I think that it's ok to start it here though since the sent email adapter could be used on staging for example. It's also such little overhead to start a supervisor that it's probably the best option for making bamboo simple to get started with.
However if this ever becomes a problem then I would happily reconsider!

On Apr 25, 2016, at 11:43 PM, Po Chen<EMAIL_ADDRESS>wrote:
an unrelated note: should Bamboo.SentEmail always be there?
You could leave this to the user to start in test cases..?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I like it! LGTM

Thanks for the reviews and comments @drapergeek and @princemaple


Location ID and Reader ID is null

I've connected Stripe M2 reader to the sdk and the reader is discovered in onUpdateDiscoveredReaders method but its location ID and reader ID is null.
Has anyone faced this issue?

@abdul-rafeh that's expected if the reader hasn't been connected to the account yet, or if switching from test mode to live mode. The reader only gets an ID and a location once it's been connected to. The reader returned after a successful connect should have its id and location id filled in and then future discovery's will return the reader with its location and id filled in.
To assist with identifying what location to register a new reader to you can use getLocations to list the locations for the account and get at the location id.

What about device id? How can I retrieve that? Since location and device id are the two params required to connect with the device.

The device id shouldn't be required to connect. The two connect params for a bluetooth reader expected are the location id and the reader object itself that was returned during onUpdateDiscoveredReaders: https://stripe.dev/stripe-terminal-react-native/api-reference/index.html#ConnectBluetoothReaderParams

Thank you. It worked!


JAMES-4888 FILTER: forward loop issue

Tests should be way easier to read.
I did let a few comments on how to achieve this.

Needs a rebase though :)

I have pushed a new commit to resolve some issues

JMAPFilteringTest is failing on this branch.
I let you debug it...

Fixed JMAPFilteringTest issue
Simplified the overall handling by tracking only recipients that are converted to forward. This is easier to do if the localCopy is done with the forwarded mail.
Misc code style improvments based on earlier remarks.

Also please fix ticket number: JAMES-4448 is NOT a valid JIRA...

IMO could be closed in favor of https://github.com/apache/james-project/pull/1762


#1014 Fix DefaultViewLocator

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
#1014 bug in DefaultViewLocator
What is the current behavior? (You can also link to an open issue here)
Fails to resolve IViewFor i.e. A concrete class with an interface registered in Splat
MainViewModel as IMainViewModel register a view as IViewFor
the DefaultViewLocator fails to resolve the view
What is the new behavior (if this is a feature change)?
Resolves IVeiwFor
Does this PR introduce a breaking change?
No, unless desired operation is to fail when using interfaces as IViewFor registration
Please check if the PR fulfills these requirements

[ ] The commit follows our guidelines: https://github.com/reactiveui/reactiveui#contribute
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

@ChrisPulman If you merge latest upstream rxui7-master branch into your branch, you should get AppVeyor to build fine.

@ChrisPulman  If you merge latest upstream rxui7-master branch into your branch, you should get AppVeyor to build fine.

@ChrisPulman I know there are currently no tests for DefaultViewLocator, but I think it's likely an oversight. #1014 includes at least one test. Any chance you could add it and - if you're feeling really generous - maybe supplement with your own?

@kentcb I have updated the branch and all appveyor checks passed, I am going to get on the Plane now. I hope this is ok and what you had requested. Thanks.

@ChrisPulman can you please allow edits from maintainers on this PR:

That way, I can fix the build and add tests before merging.

@kentcb I have updated as requested.

Just tried pushing to this branch but didn't have much luck. Might just open a separate PR if I can't figure this out...

looking at the AppVeyor log it looks like #1259 merge is the root cause of the issues I am not sure exactly what the issue is though:
Error: NuGet: Process returned an error (exit code 1).
Command exited with code 1

Hmm, I don't think that's merged yet though (it's also failing, but completely different reason). And there's a couple of other branches failing for other reasons. This CI is great when it works, but I feel completely helpless/out of my depth when it doesn't.

I have synced the repo and at this moment I have 9 projects that wont build, I am trying to find out why, it looks like missing references, this could possibly be the cause for failing especially if you are using VS and have had a successful build before, closed your solution and then opened it again, it will automatically find the dll from your builds, its worth checking the paths or delete your obj and bin dir's see if it builds then.

To be clear, are you trying to build this branch or another?

I am using this branch, after executing sync in Git for Windows

Weird. I just switched to this branch, did a git clean -xdf, a git pull upstream, and then tried building in VS. It worked fine (some warnings, but that was it).

Thnaks for doing that, I will keep trying to get a clean build, can I suggest adding the nuget package System.Windows.Interactivity.WPF to the ReactiveUI.Blend_Net45 project to ensure that the reference is common for all, I have had so many issues with the blend libs being different or even missing on some machines, I guess mine is one of those, mine being a different version :(
The other issues seam to be related to the Andriod and WinRT projects I am thinking my VS install has issues or the wrong versions of android

C:\Program Files (x86)\Microsoft SDKs\Windows\v8.1\ExtensionSDKs\BehaviorsXamlSDKManaged
is missing on my machine hence issues with WinRT. Other than this all non Android projects build.
What SDK should be being used for Android?

Superseded by #1282.


Stubs out fetching language suggestions.

refs #3416
Why was this change made?
Less work for catalogers.
How was this change tested?
Which documentation and/or configurations were updated?


Allow separate persistence control for native window position and size

I'll link to #591 here, where the topic was discussed already.
In conclusion, it should be possible to have the window position persistent, so the user can move the window whereever it should be and have it remembered on next launch. At the same time, the window size should be able to kept at a fixed size determined by the application at startup.

Agreed. I'm also interested in having control over the new fullscreen state.


airborne feylights

it might be nicer to use feylight if it didnt fizzle if there's no blocks for it to place the lights against and simply placed them midair at the end of its range instead, or having an imbuement that lets them be placed without the need of a block being targetted

This should definitely be possible.

If we just find the end-point of the actual ray-trace and check if it's an air block or not, we can place there instead of relying on a hit. I think a miss actually guarantees that it's an air block, so it's a matter of getting what blockpos it is?


Clickhouse ingestion  column type Map schema failure

Describe the bug
clickhouse tables that have the following rows  does not work vs works .... or is this not supported?
column type of Map(String, Map(String, Nullable(String)))  fails to produce schema
column type Map(String, Nullable(String)) produces schema as expected (and is boring).
To Reproduce
in clickhouse - make a bad table
CREATE TABLE IF NOT EXISTS bugtable on cluster 'local' (
	id Int,
	metadata Map(String, Map(String, Nullable(String)))
) ENGINE = MergeTree()
order by id

simple recipe
source:
    type: clickhouse
    config:
        env: PROD
        platform_instance: clickhouse
        host_port: '${chi_cluster01_host}:8123'
        username: '${chi_cluster01_user}'
        password: '${chi_cluster01_pass}'
        profiling:
            enabled: false
        stateful_ingestion:
            enabled: true
            remove_stale_metadata: true

Expected behavior
Expected behavior is to produce schema like other formats.
some logs
/tmp/datahub/ingest/venv-clickhouse-v0.11.0/lib/python3.10/site-packages/clickhouse_sqlalchemy/drivers/base.py:273: SAWarning: Did not recognize type 'name String' of column 'changes'
  warn("Did not recognize type '%s' of column '%s'" %
/tmp/datahub/ingest/venv-clickhouse-v0.11.0/lib/python3.10/site-packages/clickhouse_sqlalchemy/drivers/base.py:273: SAWarning: Did not recognize type 'previous_value String' of column 'changes'
  warn("Did not recognize type '%s' of column '%s'" %
/tmp/datahub/ingest/venv-clickhouse-v0.11.0/lib/python3.10/site-packages/clickhouse_sqlalchemy/drivers/base.py:273: SAWarning: Did not recognize type 'new_value String' of column 'changes'
  warn("Did not recognize type '%s' of column '%s'" %
/tmp/datahub/ingest/venv-clickhouse-v0.11.0/lib/python3.10/site-packages/clickhouse_sqlalchemy/drivers/base.py:273: SAWarning: Did not recognize type 'reason String' of column 'changes'
  warn("Did not recognize type '%s' of column '%s'" %
...
  ... /tmp/datahub/ingest/venv-clickhouse-v0.11.0/lib/python3.10/site-packages/clickhouse_sqlalchemy/drivers/base.py:273: SAWarning: Did not recognize type 'Strin' of column 'metadata'

Warnings about the inability to produce schema.
 'at.audit_log_local': ["unable to get column information due to an error -> Map.__init__() missing 1 required positional argument: 'value_type'"]},

If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
cli_version': '0.11.0',
 'cli_entry_location': '/tmp/datahub/ingest/venv-clickhouse-v0.11.0/lib/python3.10/site-packages/datahub/__init__.py',
 'py_version': '3.10.10 (main, Mar 14 2023, 02:37:11) [GCC 10.2.1 20210110]',
 'py_exec_path': '/tmp/datahub/ingest/venv-clickhouse-v0.11.0/bin/python3',

Additional context
this seems more like a sqlalchemy issue than a datahub issue :/

Depends on https://github.com/xzkostyan/clickhouse-sqlalchemy/issues/269

Leaving this open for tracking, but we can't really do much until https://github.com/xzkostyan/clickhouse-sqlalchemy/issues/269 is fixed.


Repeated typo: Destory -> Destroy

Original issue: https://charm.cs.illinois.edu/redmine/issues/164

The word 'destroy' is misspelled as 'destory' in several places in the codebase. I'm fixing some of them, but leaving others for the moment to avoid conflict with other ongoing work.

Original date: 2013-04-05 17:37:39

Fixed the instances in persistent in commit:eed4862. Leaving net/machine.c and arch/util/machine-smp.c until #163 is fixed, and ckmemcheckpoint.C until other work on it (replica_ft, etc) gets merged.

Original date: 2013-05-30 21:21:04

The machine layer instances were fixed in code but not comments with the net-lrts work, so I've fixed the comments too and cut the blockage on other machine-layer changes. Still want the ckmemcheckpoint.C fixes, too.

Original date: 2013-10-07 19:34:02

Once the outstanding branches that heavily modify ckmemcheckpoint.C get integrated, these typos should be fixed.

Original date: 2014-01-06 04:55:39

A grep on src yield 0 such instances.

Original date: 2015-01-29 03:03:44

$ git grep -i -n destory
src/ck-core/ckmemcheckpoint.C:828:class ElementDestoryer : public CkLocIterator {
src/ck-core/ckmemcheckpoint.C:832:        ElementDestoryer(CkLocMgr* mgr_):locMgr(mgr_){};
src/ck-core/ckmemcheckpoint.C:868:// destory all array elements
src/ck-core/ckmemcheckpoint.C:921:  // including destorying all array elements
src/ck-core/ckmemcheckpoint.C:927://  CKLOCMGR_LOOP(ElementDestoryer chk(mgr); mgr->iterate(chk););
src/ck-core/ckmemcheckpoint.C:963:  // cannot be CkMemCheckPT::restart because destory will modify states

This hasn't been fixed.

Original date: 2016-04-14 19:41:47

~~https://charm.cs.illinois.edu/gerrit/#/c/1107/~~ https://github.com/UIUC-PPL/charm/commit/8efeeb6fd8986ae8acc6999c580666992fc1c6b4


Cannot write more than once

I have specified the option of append mode, but fruits.parquet has only the first 2 row (apples & Oranges). What am I missing?
Thanks!
var opts = {flags: 'a'};
var writer = await parquet.ParquetWriter.openFile(schema, 'fruits.parquet', opts);
// append a few rows to the file
await writer.appendRow({name: 'apples', quantity: 10, price: 2.5, date: new Date(), in_stock: true});
await writer.appendRow({name: 'oranges', quantity: 10, price: 2.5, date: new Date(), in_stock: true});
write.close();
writer = await parquet.ParquetWriter.openFile(schema, 'fruits.parquet', opts);
// append a few rows to the file
await writer.appendRow({name: 'banana', quantity: 10, price: 2.5, date: new Date(), in_stock: true});
await writer.appendRow({name: 'peaches', quantity: 10, price: 2.5, date: new Date(), in_stock: true});
write.close();

Appending to a parquet file is a little more complicated than specifying append flag on the file, as the file has a metadata and footer at the end of the file.
One way to do a pure append, is first read the metadata and then append manually to the file, and finalize by writing the updated metadata and the footer at the end.  The old metadata would be essentially orphaned off.

FWIW: I just grouped all the rows I needed for a particular parquet file into a custom data structure. Once built, I looped through that structure and appended to the parquet file within a single open/close block. Solved the problem of having to worry about appending via the parquetJS api.


lags on chromebook

Chromebooks are underpowered and definitely not meant for emulation. This isn't really an issue.


questions about rapids folder

I saw that some packages in the requirements.txt are installed via local .whl files, especially in the folder named rapids. However, I have not found this rapids folder. How should I solve this problem?

I will modify them.
Currently I suggest you install transformers and GEARLM at the newest version. And then it may run.

I will modify them. Currently I suggest you install transformers and GEARLM at the newest version. And then it may run.

thankyou！


feat(core): add container { defaultArgs, withDefaultArgs }

What needs to happen before this can be merged?

Some update need to happen because of https://github.com/dagger/dagger/issues/3284


Add annotations block for airflow-env ConfigMap

What is your feature request?
Please add a way for us to add annotations directly to the airflow.cfg ConfigMap.  We deploy with Spinnaker, which naturally versions Secrets and ConfigMaps when they change.  As a result, worker pods search the ConfigMap to see what the name of the ConfigMap for the Env is; the problem is that it's a static value ( airflow-env ).
To get around this, Spinnaker allows the annotation on Secrets / ConfigMaps:
strategy.spinnaker.io/versioned: "false"
This would allow a working configuration for anyone else that deploys via Spinnaker.
What alternatives have you considered?
We had previously created a subchart or additional Manifests that mirror the env configmap, but this would be the easiest way to get an official implementation with an alternative block for the configuration.
Happy to contribute or make the change.
Thank you!

@kencieszykowski we always welcome contributions!
If you are going to contribute, please also take a look at Issue #8, but if you simply want to add annotation support to the ConfigMap, that shouldn't be too hard.


Is every model stampable by default?

It seems that every model is stampable by default. This would be odd as the whole point of the stampable method is to indicate which models should be stamped.

Yes, every model is stampable by default. I'm making it optional here because my app would rather stamp every single model.
It will not create the stampable callbacks if there is no created_by/updated_by columns, hence the need for the connection when the model is loaded.

Do reopen this if this is a problem.


unable to connect to the goal

Hi,i get this when i run the code.Has someone meet the same question?
Checking if can connect to goal at 9 samples
Could not connect to goal
Checking if can connect to goal at 38 samples
Can connect to goal

I'm not certain I understand what the question is.
The output you are seeing is a status of the sampling (for a better understanding of the underlying algorithms, please see Steven LaValle's site.
Every time a new edge is added, there is a probability to check whether the goal can be reached - shown by Checking if can connect to goal at $n$ samples. Often towards the beginning of the sampling, it will not be possible, as indicated by Could not connect to goal. If a connection is possible, then you will see the final line Can connect to goal.

I meet the same question and the final line is 'can connect to goal'.But why python  only give this answer?


Tweak formatting in changelog.md.

Coverage remained the same at 90.859% when pulling 453d71a72776659f0e8c86008281def5286e3484 on fix-changelog into a17a44a00f447857bd51350d1b041aaf0c6f05c4 on master.


Enforce CPU time limit in SqlQueryManager

logic is very similar to existing ClusterMemoryManager. adding necessary config hooks

Looks like you're missing some tests. See Travis

Please don't use passive voice for commit messages.

@yangyangyyy Please address my comments, and also rebase on latest trunk. Let me know when you finish.

@cberner, can you do a final pass on this one?

Merged. Thanks!


UX Change - Font to Montserrat for improved readability.

Current
Currently the docs site uses Roboto for the body text. This decision was made when we were using Google Material Design System to align with the font on other open source/mission apps.
There are some issues with this font impacting user readability on the docs site, which is text heavy

Font weight too heavy at 400
Font height too short
dense letter spacing

Update

[ ] Change docs template typface default to Montserrat - with sans sarif fall back.

The transition to Carbon has made the use of roboto no longer aligned with the design system. And with the docs site falling more under our company branding it would be best to align the body font with the new website branding as well.
The new company website uses Montserrat for it's body text.  https://fonts.google.com/specimen/Montserrat?query=Montserrat
This typeface achieves high legibility even in small sizes.

wider letter spacing
Featuring a large x-height
short descenders and wide apertures,

Teams need to update their dependencies to get the update
https://main--starlit-valkyrie-7f1dd9.netlify.app/docs/


Handle WS CLOSE properly

Codecov Report

Merging #1819 into master will increase coverage by 73.32%.
The diff coverage is 47.36%.

@@             Coverage Diff             @@
##           master    #1819       +/-   ##
===========================================
+ Coverage   24.88%   98.21%   +73.32%
===========================================
  Files          59       59
  Lines        2933     2912       -21
===========================================
+ Hits          730     2860     +2130
+ Misses       2203       52     -2151

@vrde Could you add a description to this pull request so we understand the reason for it?

Did you test this against what @TimDaub described in the original issue #1787 ?

Did you test this against what @TimDaub described in the original issue #1787 ?

Yes, but it's not in a unit test.

Okay, that's reassuring.

@vrde wrote in an email:

You can merge it even if coverage complains.

because unit tests were not written to test all the code paths in this PR. Normally we wouldn't do that, but I think @vrde is considering replacing all this ascyncio stuff with something else.


V86 unable to be resized

The "Virtual x86" application is currently unable to be resized, regardless of whether there is an image loaded or not. Unsure if this is by design or unintentional, so I wanted to mention it in case you were not aware.

Thanks. It is indeed by design because of lack of resizing after the fact within v86. I think it would not be trivial to have resize work in a good way when its running a linux gui. Several apps cant be resized that rely on canvas and have non-ideal api's for dynamically changing their sizes. If you can find a way to do it with v86 I'd be happy to integrate it, but I dont think its easy. Same issue with a few apps that have hardcoded libs for doing some of the pointer event listeners which makes scaling also not work. Its just one of the limits I have accepted as I use various 3rd party scripts with different levels of flexibility.


interface with "new" member not working

Bug Report
declaring an interface with a method named "new" is not working properly
🔎 Search Terms
interface new member
interface membername new
🕗 Version & Regression Information
Seams to effect all versions including nightly.
⏯ Playground Link
(https://www.typescriptlang.org/play?ts=3.3.3#code/JYOwLgpgTgZghgYwgAgJIBUIGczIN4CwAUMqciBAO4AUAlAFzkCuAtgEbQDcxAvscQgA2cLFmSQcyYCwAOgiCwjgxGbLkIkyFGrXzEyB5FAhgmUEMgCs3TaT5F7QA)
💻 Code
interface ITest {
    new(): number;
}

class test implements ITest {
    new() {
        return 5;
    }
}

🙁 Actual behavior
Error:
Class 'test' incorrectly implements interface 'ITest'.
Type 'test' provides no match for the signature 'new (): number'.
🙂 Expected behavior
Interfaces should allow the (in JS) non-reserved "new" to be used, so one can create interfaces for already existing classes.

This is working as intended.
new() is TypeScript reserved syntax that declares a needed constructor. You have to write ['new']() instead.

thx that works for me, but I still don't get the point of calling a constructor "new" in interfaces and "constructor" in classes? why does it need two different keywords for the same thing?

Because constructor is not a keyword, and actually creates a property called that (even in classes).  JS just happens to treat it specially in classes, but not, e.g. in object literals, so TS needs a general way to express “this is a type you can call with new”


ValueError: Unknown initializer: f

E1111 14:35:28.563664<PHONE_NUMBER>45280 keras_to_tensorflow.py:85] Input file specified only holds the weights, and not the model definition. Save the model using model.save(filename.h5) which will contain the network architecture as well as its weights. If the model is saved using model.save_weights(filename), the flag input_model_json should also be set to the architecture which is exported separately in a json format. Check the keras documentation for more details (https://keras.io/getting-started/faq/)
Traceback (most recent call last):
File "keras_to_tensorflow.py", line 172, in
app.run(main)
File "/scratch/workspace/anaconda2/lib/python2.7/site-packages/absl/app.py", line 299, in run
_run_main(main, args)
File "/scratch/workspace/anaconda2/lib/python2.7/site-packages/absl/app.py", line 250, in _run_main
sys.exit(main(argv))
File "keras_to_tensorflow.py", line 118, in main
model = load_models(FLAGS.input_model, FLAGS.input_model_json)
File "keras_to_tensorflow.py", line 94, in load_models
raise wrong_file_err
ValueError: Unknown initializer: f
My .h5 file contains the model structure and weights. Can you help me see what is wrong?

I just meet this problem. Do you fix it？ @zyyhl


"Please Wait" dialog after the "Repository Connection" dialog won't close until it is manually moved or closed.

Resolved by 297677b3fb526e311864e55bcc72408a9945dca1


UNIQUE constraint should not enforce on NULL values

From Balint:
Dolt uses different policy for NULL values in the case of UNIQUE constraints then MySQL.
MySQL enables multiple NULL-s for a column if it has UNIQUE constraint on it, does not consider these violation.
Dolt follows here the solution like I think postgreSQL which only enables one NULL in a UNIQUE constraint field.
We designed or db with the MySQL logic. What is the reason Dolt does not follow this like in MySQL?

Fixed in https://github.com/liquidata-inc/dolt/pull/894


[6494] Changed user droplist (now takes data from xml). Cron user update

Story / Bug id: https://bugzilla.stxnext.pl/show_bug.cgi?id=6494
Project / App: Onboarding - Presence Analyzer
Reference person: @ADubielSTX
Description: Changing users droplist (now displays user name and last name instead of User ). Taking user info from xml file and auto updating that file (cron).
Migrations:
Rollback:
New imports / dependencies:
Unit Tests:
What tests do I need to run to validate this change:
Scripts to be run:
./bin/buildout -vN
(to update xml file cofing and update crontab)
New or Changed JS and / or HTML Files:
changed:
src/presence_analyzer/layout.html
src/presence_analyzer/static/mean_time_weekday.js
src/presence_analyzer/static/presence_start_end.js
src/presence_analyzer/static/presence_weekday.js
new:
update_avatar.js

Gist with pep8, pylint, unittests etc.:
https://gist.github.com/ADubielSTX/a2f246a7ca6da3f04651

Review finished


Uncaught Error: RegExp is not valid

[Enter steps to reproduce below:]

...
...

Atom Version: 0.180.0
System: Mac OS X 10.10.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: RegExp is not valid
At /Applications/Atom.app/Contents/Resources/app/node_modules/text-buffer/lib/history.js:124

Error: RegExp is not valid
    at Error (native)
    at OnigScanner.findNextMatchSync (/Applications/Atom.app/Contents/Resources/app/node_modules/oniguruma/lib/oniguruma.js:35:18)
    at Scanner.module.exports.Scanner.findNextMatch (/Applications/Atom.app/Contents/Resources/app/node_modules/first-mate/lib/scanner.js:57:23)
    at Rule.module.exports.Rule.findNextMatch (/Applications/Atom.app/Contents/Resources/app/node_modules/first-mate/lib/rule.js:101:28)
    at Rule.module.exports.Rule.getNextTokens (/Applications/Atom.app/Contents/Resources/app/node_modules/first-mate/lib/rule.js:143:21)
    at Grammar.module.exports.Grammar.tokenizeLine (/Applications/Atom.app/Contents/Resources/app/node_modules/first-mate/lib/grammar.js:125:39)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLineForRowWithText (/Applications/Atom.app/Contents/Resources/app/src/tokenized-buffer.js:488:28)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLineForRow (/Applications/Atom.app/Contents/Resources/app/src/tokenized-buffer.js:477:19)
    at TokenizedBuffer.<anonymous> (/Applications/Atom.app/Contents/Resources/app/src/tokenized-buffer.js:430:31)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLinesForRows (/Applications/Atom.app/Contents/Resources/app/src/tokenized-buffer.js:438:10)

Commands
     -6:49.6 core:copy (atom-text-editor.editor)
     -6:48.8 core:paste (atom-text-editor.editor)
     -6:48.5 core:save (atom-text-editor.editor)
     -5:56.9 core:copy (atom-text-editor.editor)
     -5:56.3 core:paste (atom-text-editor.editor)
     -5:55.5 core:backspace (atom-text-editor.editor)
  2x -5:51.4 core:undo (atom-text-editor.editor)
     -5:48.0 core:copy (atom-text-editor.editor)
     -5:47.1 core:move-right (atom-text-editor.editor)
     -5:46.7 core:paste (atom-text-editor.editor)
     -5:41.2 core:copy (atom-text-editor.editor)
     -5:40.1 core:paste (atom-text-editor.editor)
     -5:39.8 core:save (atom-text-editor.editor)
     -5:37.7 core:close (atom-text-editor.editor)
     -0:11.3 core:paste (atom-text-editor.editor.is-focused)
  2x -0:07.0 core:backspace (atom-text-editor.editor.is-focused)

Config
{
  "core": {},
  "editor": {
    "fontSize": 12,
    "showIndentGuide": true,
    "invisibles": {}
  }
}

Installed Packages
# User
autocomplete-plus, v1.1.0
language-ember-script, v0.1.0

# Dev
No dev packages

Did you get this error while editing an EmberScript file?

No, while editing a rails erb file.
Now getting it constantly :(

After restarting atom the issue seems to be gone for now


Pre review

Authentication design:
Backend using passport middleware and gunjs
Frontend built on react + redux
Warnings:
JWT to be migrated to auth-header from cookie storage
Random hash key to replaced in oauth

very nice work @DecFox.


feat: add bracketed-paste mode in the REPL

Resolves #2068
Description

What is the purpose of this pull request?

This pull request:

adds bracketed paste mode in the REPL
The default is enabled in TTY environments and is disabled back upon closing the REPL.
Pasted content is not executed and can be edited.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #2068

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Aside: this PR is quite nice, as it is a big win to be able to copy and paste multi-line content into the REPL. So thank you for working on this.

@kgryte I was thinking if it would make sense to disable auto close pairs during the pasting process? I noticed that pasting feels slow and it might also interfere if the user is not pasting syntactically correct code?

@Snehil-Shah That makes sense.

Whenever a user pastes content, what gets pasted should match verbatim the original content.


Newly creating uninstall string using the product code value retrieved from the uninstall_string of the Windows registry to avoid using invalid uninstall_string.

w.r.t. https://github.com/chef/chef/issues/5274
In-progress.

Hmm. It sounds to me like the issue in #5274 is a case of true vendor error and I wonder if we alter/harden the string here, if that could just cause problems with otherwise valid uninstall strings. Might be best just to let this be.

@mwrock I have tested these changes with 30 MSI installers (most of the installers had /I in their uninstall_strings while some other MSI installers had /X in their uninstall_strings) and it worked successfully for each of them.
I have attached the test_results document on the Card-399 (refer the doc with the latest date appended) which explains the overall behavior.


Play video from web interface

Instead of always having to download a video now you can play it directly from the web interface.

@roleoroleo what do you think of this?

I will try it next monday.
Thank you for your contribution.

@roleoroleo

@BenjaminFaal
Thank you for your work.


Add MAF to VCF functionality

VCF to MAF is already supported (e.g. maf_vcf2maf and pymaf.MafFrame.from_vcf). Now add MAF to VCF functionality.
https://bioinformatics.stackexchange.com/questions/3421/tools-to-do-vcf-to-maf-and-maf-to-vcf-conversion

implemented in 0.13.0-dev


Mount verb succeeds when ProjFS is not enabled

It is possible to get into a state where the ProjFS driver is not enabled but the mount verb succeeds (the mount process then has trouble initializing placeholders). This may only happen for new clones, but requires investigation.

A few updates on what happened (from log files):

prjflt service was running, but the native ProjectedFSLib.dll was missing
Somehow ProjectedFSLib.dll was later present in C:\Windows\System32, but it did not appear that GVFS.Service did this.
Writing placeholders failed with 80070057(InvalidArg), suggesting that the ProjectedFSLib.dll that was copied into System32 was incompatible with the prjflt.sys present on the system.

Next step is to follow up with the ProjFS team and see if they have any recommendations on checking if the .sys and .dll are compatible.

Closing this bug as per the logs the ProjFS service was running which allowed the mount verb to succeed.


Customize number of day cells in a month else default it to 42 cells(6*7)

https://github.com/Adphorus/react-date-range/blob/4bab53471f58d9b68838bad54610e0018d7c2b5b/src/Calendar.js#L208

It would be helpful to be able to customize the number of days that appear in a month instead of showing 42 cells for every month

This issue is resolved in the latest release. (1.1.0)
Now you can set fixedHeight prop to true if you want all the months to appear in 6 rows.
Thank you for pointing it out. I'm closing this.


[2024 Symposium] OHDSI Analytics

Items to be completed in support of OHDSI Analytics and related tools in advance of the OHDSI Symposium

Cover how feature extraction and other OHDSI tools would be used in this use case

TODO

[ ] #354

Enable participants to use the tools and these kinds of data in a cooking-show style:
Two participant groups:

Atlas

Allude to Atlas/Circe functionality IF the Exposure Occurrence was referenceable
Possible we could demonstrate a tweak to Circe to enable Exposure Occurrence

Notebook interface (probably using R)

TODO:

[ ] Schedule meeting with Jenna & A.S.

#Integration of GIS Data into OHDSI Model Building
Motivation
Although the proposed GIS extension tables have been approved by the broader OHDSI community, they are not yet integrated natively into the OHDSI tooling. These tables and the data they capture, however, can still be referenced in standard model building and analytics workflows. The purpose of this analytics demonstration is to show how  data in the EXPOSURE_OCCURRENCE table can be utilized in the training and evaluation of an OMOP-specific patient-level-prediction (PLP) model. The analytic process we executed and describe below relies on a GIS-version of the Tufts Synthetic Dataset that has both EHR and geospatial data integrated; see the description of that dataset for more details about contents or access. Much of the work is based on the detailed vignette that describes custom feature engineering in PLP
Step-by-Step Process
Step 1: Create Target & Outcome Cohorts
We defined our 'target' cohort within the sampling procedures when creating a subset of the Tufts Synthetic Data, and we described this process at length elsewhere. For the purposes of using the PLP package, we formalized this group of individuals in the cohort table using a simple cohort definition including all individuals with "any visit", and include that atlas-compatible json definition here for reference.
We also created our outcome cohort - in this case, those patients with COPD or a conceptual descendant thereof - in Atlas and have shared the json definition (we also included an equivalent SQL script).
Step 2: Create Generic PLP Lasso Logistic Regression Model in R
It is possible to create an R package that serves as a basis for a PLP model using Atlas, but given Atlas does not yet support the GIS extension, we have created thid demo model directly using the PLP package.
After configuring the environment (very important!) appropriately, we imported necessary packages and defined the relevant parameters about the datasource:
library(PatientLevelPrediction)
library(dplyr)
outputFolder <- "/ohdsi-gis/copdResultsPM25_NEW"
saveDirectory <- outputFolder
ExecutionDateTime <- Sys.time()
logSettings = createLogSettings(verbosity = "DEBUG", timeStamp = T, logName =
                                  "runPlp Log")
analysisName = 'Generic PLP'

# Details for connecting to the server:
connectionDetails <- DatabaseConnector::createConnectionDetails(
        dbms = 'spark',
        server = '/default',
        connectionString = '<REDACTED>'
    )
# Add the database containing the OMOP CDM data
cdmDatabaseSchema <- 'gis_syn_dataset_5_4'
# Add a sharebale name for the database containing the OMOP CDM data
cdmDatabaseName <- 'TSD-GIS'
# Add a database with read/write access as this is where the cohorts will be generated
cohortDatabaseSchema <- 'gis_syn_dataset_5_4'
tempEmulationSchema <- NULL
# table name where the cohorts will be generated
cohortTable <- 'cohort'

After defining parameters, we created a database settings object and launched the runMultiplePLP function to create a base plpData object:
databaseDetails <- PatientLevelPrediction::createDatabaseDetails(
        connectionDetails = connectionDetails,
        cdmDatabaseSchema = cdmDatabaseSchema,
        cdmDatabaseName = cdmDatabaseName,
        tempEmulationSchema = tempEmulationSchema,
        cohortDatabaseSchema = cohortDatabaseSchema,
        cohortTable = cohortTable,
        outcomeDatabaseSchema = cohortDatabaseSchema,
        outcomeTable = cohortTable,
        cdmVersion = 5
)

# Run very simple LR model against two cohorts created in Atlas. Use model
# as basis for augmented model with pollutants below
runMultiplePlp(
   databaseDetails = databaseDetails,
   modelDesignList = list(createModelDesign(targetId = 9, outcomeId = 8, modelSettings =
                                              setLassoLogisticRegression())),
   onlyFetchData = F,
   cohortDefinitions = NULL,
   logSettings = createLogSettings(verbosity = "DEBUG", timeStamp = T, logName =
                                     "runPlp Log"),
   saveDirectory = outputFolder,
   sqliteLocation = file.path(saveDirectory, "sqlite")
 )

Step 3: Split plpData object to train/test, augment labels with EXPOSURE_OCCURRENCE values
The labels sub-object within plpData contains per-individual data elements like gender and age; we added an additional data element to this object derived from the EXPOSURE_OCCURRENCE table:

cohortDefinitions <- NULL
modelDesign <- createModelDesign(targetId = 9, outcomeId = 8, modelSettings = setLassoLogisticRegression())
populationSettings <- modelDesign$populationSettings
splitSettings <- modelDesign$splitSettings

plpData <- loadPlpData("/ohdsi-gis/copdResultsPM25_B/targetId_9_L1")

mySplit <- splitData (plpData = plpData,
                      population = createStudyPopulation(plpData, 8, populationSettings),
                      splitSettings = splitSettings)

labelTrain <- mySplit$Train$labels
conn <- DatabaseConnector::connect(connectionDetails)
pollutants <- DatabaseConnector::querySql(conn, "SELECT person_id as subjectID, CAST(MEAN(value_as_number) AS DOUBLE) AS pmValue FROM gis_syn_dataset_5_4.exposure_occurrence WHERE value_as_number IS NOT NULL GROUP BY person_id;")
labelTrainPol <- merge(x=labelTrain, y=pollutants, by.x = "subjectId", by.y = "SUBJECTID")

mySplit$Train$labels <- labelTrainPol

labelTest <- mySplit$Test$labels
labelTestPol <- merge(x=labelTest, y=pollutants, by.x = "subjectId", by.y = "SUBJECTID")

mySplit$Test$labels <- labelTestPol

trainData <- mySplit$Train

testData <- mySplit$Test

Step 4: Reference augmented label objects in custom feature engineering function
We would like to convert our per-patient labels into the covariate data structure referenced by the PLP workflow. To do this, we were able to follow the feature engineering vignette and create two functions, createPollutants and implementPollutants:

createPollutants <- function(
                     method = 'QNCV'
                     ){

  # create list of inputs to implement function
  featureEngineeringSettings <- list(
    method = method
    )

  # specify the function that will implement the sampling
  attr(featureEngineeringSettings, "fun") <- "implementPollutants"

  # make sure the object returned is of class "sampleSettings"
  class(featureEngineeringSettings) <- "featureEngineeringSettings"
  return(featureEngineeringSettings)

}

implementPollutants <- function(trainData, featureEngineeringSettings, model=NULL) {
  if (is.null(model)) {
    method <- featureEngineeringSettings$method
    gisData <- trainData$labels
    y <- gisData$outcomeCount
    X <- gisData$PMVALUE
    model <- mgcv::gam(
      y ~ s(X, bs='cr', k=5, m=2)
    )
    newData <- data.frame(
      rowId = gisData$rowId,
      covariateId =<PHONE_NUMBER>,
      covariateValue = model$fitted.values
    )
    #outData <- data.frame(
    #  x = gisData$pollutants,
    #  y1 = 9 - gisData$outcomeCount,
    #  y2 = 9 - model$fitted.values
    #)
    #write.csv(outData, "fitplot.csv")
  }
  else {
    gisData <- trainData$labels
    X <- gisData$PMVALUE
    y <- gisData$outcomeCount
    newData <- data.frame(y=y, X=X)
    yHat <- predict(model, newData)
    newData <- data.frame(
      rowId = gisData$rowId,
      covariateId =<PHONE_NUMBER>,
      covariateValue = yHat
    )
  }
  # update covRef
  Andromeda::appendToTable(trainData$covariateData$covariateRef,
                           data.frame(covariateId=2052499839,
                                      covariateName='Average PM2.5 Concentrations',
                                      analysisId=1,
                                      conceptId=2052499839))

  # update covariates
  Andromeda::appendToTable(trainData$covariateData$covariates, newData)

  featureEngineering <- list(
    funct = 'implementPollutants',
    settings = list(
      featureEngineeringSettings = featureEngineeringSettings,
      model = model
    )
  )

  attr(trainData$covariateData, 'metaData')$featureEngineering = listAppend(
    attr(trainData$covariateData, 'metaData')$featureEngineering,
    featureEngineering
  )

  trainData$model <- model

  return(trainData)
}

We can then execute these functions to create training and test data objects that contain our extended covariates:
featureEngineeringSettingsPol <- createPollutants('QNCV')
trainDataPol <- implementPollutants(trainData, featureEngineeringSettings)
testDataPol <- implementPollutants(testData, featureEngineeringSettings, trainDataPol$model)

Note that if we plot the output model of the gam fitting in the implementPollutants function, we end up with a plot that aligns well with our underlying relationship between Odds Ratio and PM2.5 concentration that we used to distribute our synthetic data by location:

Step 5: Apply new train and test datasets to runPlp and evaluate output
analysisId <- '1'
analysisPath = file.path(saveDirectory, analysisId)

settings <- list(
  trainData = trainDataPol,
  modelSettings = setLassoLogisticRegression(),
  analysisId = analysisId,
  analysisPath = analysisPath
)

ParallelLogger::logInfo(sprintf('Training %s model',settings$modelSettings$name))
model <- tryCatch(
  {
    do.call(fitPlp, settings)
  },
  error = function(e) { ParallelLogger::logError(e); return(NULL)}
)

prediction <- model$prediction
# remove prediction from model
model$prediction <- NULL

#apply to test data if exists:
if('Test' %in% names(data)){
predictionTest <- tryCatch(
  {
	predictPlp(
	  plpModel = model,
	  plpData = testDataPol,
	  population = testDataPol$labels
	)
  },
  error = function(e) { ParallelLogger::logError(e); return(NULL)}
)

predictionTest$evaluationType <- 'Test'

if(!is.null(predictionTest)){
  prediction <- rbind(predictionTest, prediction[, colnames(prediction)!='index'])
}

}

More description to come...


[BUG] Alert 2022-06-06

node1 error_logs.txt:
...
express-rate-limit: req.ip is undefined - you can avoid this by providing a custom keyGenerator function, but it may be indicative of a larger issue.
express-rate-limit: req.ip is undefined - you can avoid this by providing a custom keyGenerator function, but it may be indicative of a larger issue.
express-rate-limit: req.ip is undefined - you can avoid this by providing a custom keyGenerator function, but it may be indicative of a larger issue.
express-rate-limit: req.ip is undefined - you can avoid this by providing a custom keyGenerator function, but it may be indicative of a larger issue.
express-rate-limit: req.ip is undefined - you can avoid this by providing a custom keyGenerator function, but it may be indicative of a larger issue.
express-rate-limit: req.ip is undefined - you can avoid this by providing a custom keyGenerator function, but it may be indicative of a larger issue.

node1 node-8080-error-05-Jun-22.log (full):
{"name":"mainnet-prod","hostname":"mainnet-node-1-oregon","pid":3151,"level":50,"msg":"[P2P_CLIENT] Address confirmation hasn't sent back. Close the socket(ws://<IP_ADDRESS>:5000/) connection","time":"2022-06-05T03:26:05.965Z","v":0}
{"name":"mainnet-prod","hostname":"mainnet-node-1-oregon","pid":3151,"level":50,"msg":"[P2P_CLIENT] Address confirmation hasn't sent back. Close the socket(ws://<IP_ADDRESS>:5000/) connection","time":"2022-06-05T07:41:24.907Z","v":0}
{"name":"mainnet-prod","hostname":"mainnet-node-1-oregon","pid":3151,"level":50,"msg":"[P2P_CLIENT] Address confirmation hasn't sent back. Close the socket(ws://<IP_ADDRESS>:5000/) connection","time":"2022-06-05T08:56:09.423Z","v":0}

restart trial 1:
bash deploy_blockchain_incremental_gcp.sh mainnet <username> 0 1 1 --keystore --keep-code --keep-data --fast-sync

restart trial 2:
/home/ain_blockchain_data/chains/8080/n2b/6$ mv 670732.json.gz 670731.json.gz 670730.json.gz 670729.json.gz ..

bash deploy_blockchain_incremental_gcp.sh mainnet <username> 0 1 1 --keystore --keep-code --keep-data --fast-sync

Will check the diffs on block 670729.

Block 670732 from node 1 is empty:

https://github.com/ainblockchain/ain-blockchain/issues/1061

Let me mark this issue closed.


Optimise setState dayPickerContainerStyles in responsivizePickerPosition

May be fix https://github.com/airbnb/react-dates/issues/1699
Main idea to not setState dayPickerContainerStyles, if it is already empty.
This will prevent unnecessary renders on resizing window for example

Coverage decreased (-0.04%) to 84.871% when pulling fd1595d18036b83212d7a61f5ea2d54c01a88848 on mmarkelov:Optimize_setState_dayPickerContainerStyles_ into 8e81a8b734440088c9ee0a2e79642bf4aebcb792 on airbnb:master.


Diff view showing an older version of notebook file

Testing #105770
Did a git pull and made sure my notebooks file is up-to-date, make a change in a cell and the left side of the diff view has an older version of my file and not the one I made the most recent change in.

Did a git pull and made sure my notebooks file is up-to-date, make a change in a cell and the left side of the diff view has an older version of my file and not the one I made the most recent change in.

Did you happen to have the diff editor opened once before you do git pull? if so, I think it's a duplicate of https://github.com/microsoft/vscode/issues/105917


refactoring news to use recoil

https://trello.com/c/QLJyDq4y/1640-3-refactor-news-and-events-to-use-recoil

LGTM. But test coverage it's still low.


Funded a channel - txid for funding does not exists, causes issue when trying to close "channel"

I have funded a channel, after not seeing any confirmations for quite a while, i tried to look at the funding tx on the main chain (or mempool) and it does not exist. Now the problem is that I cannot seem to close this "channel" and I doubt it will auto close after 144 blocks as there is no actual transaction.
Using rebroadcast did not make any difference, not sure what would happen when trying to rebroadcast a create channel tx when that same channel is now attempting to close itself.
Feels like if this could be resolved then the process would actually be straight forward, I reckon my app has now gotten itself into a weird state, should I re-install and use the previous seed?

Can you paste the txid here?
Indeed the case where the tx never confirms isn't well handled currently, and channel will stay in CLOSING forever (but no funds are lost, your balance should be correct and your wallet should continue working ). Managing this is not trivial since the only way to be sure that it really will never confirm is to actually double spend it.

The channel ID: 0192d3d468d68fa3f30763ddbe108183d0bbf8fcf59e8e6227ab6c6e2
Funding txid: 1f638e296e6cab27628e9ef5fcf8bbd0838110bedd6307f3a38fd668d4d39201
Makes sense that there would be some trickyness to dealing with this scenario.
I think the main issue for my install right now is the validation rule that fires when I try create a new channel to the same node as the failed closing channel.
Is there a rule that removes that problematic channel after a set amount of time?
Thanks for your hard work, this is looking really cool. I am not too worried about the funds, just excited to get to play :)

In my case I don't think the tx ever went out onto the network, can't be found in mempool or on chain.
Seems like all records related to a funding tx whose txid never appears in the mempool need to be flushed, maybe there is a threshold of time without the txid appearing in mempool where this can happen? (sorry if my suggestion is over simplified, assume there is a lot of complexity going on under the hood)

I've done the dns update and restarted as per the instructions in the thread relating to the non closing of a channel and the non return of funds.
My issue is a bit different, the channel funding transaction never hit the mempool, but the channel record was visible in the app. I tried to close the channel (Force) which changed the state to CLOSING (uncoop).
The problem for me I believe is that this channel will never close because  it was never opened in the first place. My balance is correct, but I cannot create a new channel to the same node due to validation saying no duplicate channels allowed.
I just need to be able to flush these non channel records from the app, or do I need to re-install and re-initialize the seed?
Thanks


Epic S-3: As a user, I want to be able to get outdoor directions.

Once the starting point and destination are entered, the directions are displayed.

Reopened to reflect that PO requested some changes. view #84


Wowza Live stream and Clappr

Hi,
I'm using Wowza Streaming Engine 4.5 to stream hls to the clients, but wowza doesn't send ext-x-endlist tag when the stream ends.
The player still make requests but the server return the same chunklist file for some period and after that it starts to return 404.
I'm looking to fix the problem at the server side, but is there any clientside solution? Maybe to stop requesting when the clappr receive 3-4 consecutive times the same index?
Thanks in advance.

hi @IlianYotov
Clappr is using internally the library HLS.JS to play .m3u8 playlist. I may be wrong, but you may want to signal this behaviour to the author. I have not looked deeper the library yet, i dunno if this can be implemented in Clappr playback wrapper.

@kslimani is right this would be a hlsjs issue. However I don't think what you're requesting is really feasible as it goes against the hls spec, and just because no chunks are added for 20 seconds might not mean they aren't delayed and would appear after 30.
You using the wowza dvr feature?

Might be relevant
https://www.wowza.com/forums/showthread.php?47755-EXT-X-ENDLIST-this-tag-is-not-appended-to-m3u8-playlist

@IlianYotov I think you can do that https://github.com/dailymotion/hls.js/blob/master/API.md#fragloadingmaxretry--manifestloadingmaxretry--levelloadingmaxretry
{
      playback: {
        hlsjsConfig: {
         manifestLoadingTimeOut: 10000,
         manifestLoadingMaxRetry: 3,
         manifestLoadingRetryDelay: 500,
         manifestLoadingMaxRetryTimeout : 64000
      }
    }
}

@IlianYotov can we close this or this isn't enough yet? (I mean setup max retries)

Yes we can close it. I try with hls config and max retries.


Added onDismiss callback

The onDismiss() callback can be useful when actions need to be done when the sheet is being dismissed.

Can’t you just use the SwiftUI binding for this? Since bottom sheet requires a binding to a boolean or item you should just be able to monitor that with an onChange modifier.


Migrated cheermotes to Helix API

Pull request checklist:

[ ] CHANGELOG.md was updated, if applicable

Description
WIP KKona

yes, I forgot about this stream content PR, will get to work on it


Check for existing stats instance before inserting in the global stats registry

Adds a check for a statistics instance in g_stats_registry with the same address as the one being added (previously duplicates were being rejected only if the name is the same).
Multiple sensors may use the same driver but some drivers only implement one instance of statistics. This can lead to a loop in the g_stats_registry linked list, because currently stats_register_internal() assumes that statistics with different names will use unique statistics.
The problem arises when a statistics instance is registered twice under different names. The actual instance is the same, so the second time it is registered the linked list insertion will cause s_next of the entry to point to itself.

I think this fix is needed since a loop in the linked list needs to be detected. But the real problem here is now only one stat would be enabled. So, it would be better to define stats in the driver structure instead of the source file as a global. That way there are two separate globals for stats since there would be two different driver structures.


chore(deps): Bump wpt from 104b1fd to f0abdbb

based on https://github.com/GoogleChromeLabs/chromium-bidi/pull/2387
Bumps wpt from 104b1fd to f0abdbb.

Commits

f0abdbb Digital Credentials: Update META.yml to add tplooker (#46997)
61a0206 IDB WPTs: Extend idbdatabase transaction tests to worker environments.
8607a1d IDB WPTs: Extend IDBCursor continue() ObjectStore WPTs to run on workers
80ff968 [FedCM] Share more code between OnResolve and CompleteTokenRequest
929aad6 Increased total pixels for the pixel tolerance to account for devices with la...
5dae272 part 5) Provoke a CSP violation for object-src 'none' with "video/mp4" and ...
60939fe Move digital-credentials transient user activation to wpt_internal
a2486bb [css-anchor-position] Fixed serialization of position-try
4dfc1cf Map media element pseudo-class tests to web-features (#46982)
41da941 Handle button box in intrinsic text field sizing.
Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

@dependabot rebase will rebase this PR
@dependabot recreate will recreate this PR, overwriting any edits that have been made to it
@dependabot merge will merge this PR after your CI passes on it
@dependabot squash and merge will squash and merge this PR after your CI passes on it
@dependabot cancel merge will cancel a previously requested merge and block automerging
@dependabot reopen will reopen this PR if it is closed
@dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
@dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
@dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
@dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Merging with updated WPT expectations


Custom helper functions

Gets rid of two dependencies, lowers the lib size a bit.

Current coverage is 99.55% (diff: 100%)

Merging #10 into master will increase coverage by 0.02%

@@             master        #10   diff @@
==========================================
  Files            14         14
  Lines           430        451    +21
  Methods          78         82     +4
  Messages          0          0
  Branches         83         88     +5
==========================================
+ Hits            428        449    +21
  Misses            2          2
  Partials          0          0

Powered by Codecov. Last update fd26a8e...564450a


print server REST endpoint only once

@chrismattmann
The following is printed to stdout for every document parse request to Tika REST server.
('server endpoint:', 'http://localhost:9998')

https://github.com/chrismattmann/tika-python/blob/master/tika/tika.py#L276

can you submit a PR to remove this @harsham05 ?

@chrismattmann
The print statement is indicative of debugging/sanity reasons. It was not present in previous tika-python Releases.  I will submit a PR removing the same.
('server endpoint:', 'http://localhost:9998')

Tika REST Server runs on port 9998 by default & the same is already mentioned in Readme.

Fixed in https://github.com/chrismattmann/tika-python/pull/92

fixed in #92


Return value not checked

In nc4file.c, function nc4_create_file() we have this code (comments in my branch, not yet on master):
   /* Get the HDF5 superblock and read and parse the special
    * _NCProperties attribute. */
   NC4_get_fileinfo(nc4_info, &globalpropinfo);

   /* Write the _NCProperties attribute. */
   NC4_put_propattr(nc4_info);

At the bottom of nc4_open_file() we have:
   /* Get the HDF5 superblock and read and parse the special
    * _NCProperties attribute. */
   NC4_get_fileinfo(nc4_info, NULL);

Both NC4_get_fileinfo() and NC4_put_propattr() can return error codes, but we are not checking for that.
When I put in checking, I get a failure. @DennisHeimbigner is it expected that either of these functions will return non-zero in correct operation?

the question is what to  do if we cannot wrire this special attribute. ideally we generate a
warning and pcontinue.

Do we know what conditions we will not be able to create it or read it? That is, under what circumstances we expect them to fail?
I have not studied why or where an error was returned. Just wondering if it is expected. If so, I could document that.

good idea. i will do that.

OK, whatever was causing the failure was cleaned up in some other way. I just put in return code checking for these two calls and tests all pass, which is good and as expected.
I have merged this into open PR #968.

These changes have made it into master, so I will close this ticket.


[ALLUXIO-2402] Make MAX_AVAILABLE in ClientRWLock configurable.

https://alluxio.atlassian.net/browse/ALLUXIO-2402
Make MAX_AVAILABLE in ClientRWLock configurable.
There is a exist property key as:
WORKER_TIERED_STORE_BLOCK_LOCKS = "alluxio.worker.tieredstore.block.locks";

As the code shows, this key is used to set how much block locks a Worker can have. And the MAX_AVAILABLE in ClientRWLock is to control a block lock's permit. These two config keys seems not exactly the same. So I add another key.
Please confirm if the new key's below name is acceptable:
WORKER_TIERED_STORE_BLOCK_PARALLEL = "alluxio.worker.tieredstore.block.parallel"

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12054/
Test PASSed.

Merged build finished. Test PASSed.

the patch looks good to me. regarding the naming, how about
alluxio.worker.tieredstore.block.lock.readers?

@apc999  address bin's suggestion

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12107/
Test PASSed.

Merged build finished. Test PASSed.

LGTM, thanks for the patch!


Can't select which audio/subtitle track to use from my .mkv files.

Tried using --tomp4 and --ffmpeg-map, but it does not seens to be working. 😞
Anyone have made it work?

+1


Tg 20 fix attention bugs

LGTM, I see that we are not using Conv2D in our current code and replaced it with conv1D itself. I checked the dimensions and they seem to match out and its fine, so I guess it should not matter right if we use conv1D or conv2D in this case as our kernel_size = 1.


output Encoding is not UTF-8

Now generates output in UCS-2 LE BOM.
I need UTF-8

It uses standard JS encoding. Use Buffer to convert, that's not this package's task.

Adding a command line flag to specify a target encoding isn't hard, but being rude is easier.


Question: import not available kubeflow/core/all.libsonnet and other problems with master

@bryanl Are there any known issues with the code @ e0e68f8f1542b4775933cf0e2ddcec28b96bb409
I'm encountering a problem when running ks show. Here's the full stack trace
Looks like a problem with the vendor directory not being picked up.
These are the steps I followed.
Also is there a bug here; should there be a ";" at the end of the line
local k = import "k.libsonnet"

Same, https://github.com/kubeflow/kubeflow/issues/566#issuecomment-378470994


Add Sonata Hybrid port

route shows the hybrid models have issues with DBC. @adeebshihadeh  see this route which I was talking about earlier where I recommended DBC change for EV messages. The OP engaged but never steers because the vehicle speed is coming in as gas pedal info.
8ad3857087181f80|2020-08-06--19-01-03

Thanks for helping with the port of SONATA HYBRID 2020
@realistic69

It does look like the signal is different for this car. Before merging, we'll need a new signal. Ideally, the new one should also work on the currently supported and EVs.

It does look like the signal is different for this car. Before merging, we'll need a new signal that ideally works with the currently supported EVs.

https://github.com/commaai/openpilot/pull/1990


KDD doesn’t pass node IP information to Felix during resync

Specific to KDD, when calico-node is restarted, we see a data plane connectivity issue that lasts 5-12 seconds. This issue is seen in both v2.3 and v2.4.
Expected Behavior
Restarting calico-node should not result in data plane disruption.
Current Behavior
Running intra-node iperf session, I observe a 5-12 second break in connectivity.

Possible Solution

Steps to Reproduce (for bugs)

deploy k8s cluster with KDD backend
spin up iperf on two different nodes: kubectl run iperf --image=phlak/iperf --replicas=2
run iperf: kubectl exec -it iperf-X -n default sh, then: # iperf -c 192.168.X.X -t 3600 -i 1
kill one of the calico-nodes where iperf is running: kubectl delete pod -n kube-system calico-node-XX

See https://github.com/projectcalico/calicoctl/issues/1419 for related debugging info.
Context

Your Environment

libcalico-go version:
Operating System and version:
Link to your project (optional):

Tracked this down to the KDD syncer not sending through Node IPs to Felix during its initial snapshot - see here.
Felix thus removes the node IPs from the "all-ips" ipset and re-populates them as it receives subsequent watch updates.

A DROP rule was getting hit during calico-node restart, observed via (on each node):
sudo watch -d -c -n 0.1 "iptables-save -c -t filter | grep DROP"
Subsequently watched the associated ipset change for a brief moment (on each node):
sudo watch -d -n 0.1 "ipset list cali4-all-hosts"


PE-5011: fix: missing u tags in direct to network bundles

We've identified an issue where $U tags are not being included in D2N uploads. This update ensures that each D2N transaction is accompanied by the necessary $U tags.
That’s an example of a bundle with the $U tags:

Tests
This PR introduces tests for metadata_generator.dart, focusing on:

Comprehensive testing of the ARFSTagGenerator class.
ARFSUploadMetadataGenerator tests to validate metadata creation for files.

Additional tests for other aspects will follow in a subsequent PR.
Workflow Update:
We've integrated unit tests for each internal package into the testing workflow. See the ticket: https://ardrive.atlassian.net/browse/PE-5018
--- Releases ---
Android release: https://appdistribution.firebase.google.com/testerapps/1:305132849030:android:6cf0cd5ec064fad3ffce07/releases/4g62vs948rm08

PR is good but I don't think it make sense to create the dumb tests for arfs_test, ardrive_crypto_test and arconnect_test

I created so it won't break the script I implemented.


[Feature Request] Example: Roll 4d6, keep the highest 3, re-roll 1s and 2s

This is a pretty popular setup for d20 character creation and I don't believe it is supported by Sidekick.
I would suggest the syntax:
Roll four hexahedrons, keep the highest three and re-roll ones and twos
/r 4d6k3r<=2
Roll four hexahedrons, keep the highest three and re-roll ones
/r 4d6k3r1
Roll four hexahedrons, keep the highest three and re-roll twos
/r 4d6k3r2
Roll four hexahedrons, keep the highest three and re-roll fives and sixes
/r 4d6k3r>=5

Hi !
have you heard about DiceParser ? It is a really powerful bot to roll dice on discord.
It can do what you want:
These commands work with DiceParser.

4d6k3r[<=2] or 4d6k3r[<3]
4d6k3r1
4d6k3r2
4d6k3r[>=5]

but the order is important here, as if you want to keep the highest after the reroll or before.
Actually with your command on DiceParser, reroll is done after the keep.
To do it before, commands are:

4d6r[<=2]k3 or 4d6r[<3]k3
4d6r1k3
4d6r2k3
4d6r[>=5]k3

In diceparser r is the reroll operator, the value is reroll once.
If you want to reroll until it is different of 1 : it is 4d6k3R1.
If you want to reroll and add the new value to the previous one: 4d6k3a1
If you want to make some exploding dice : 4d6e6k3
To install it on your server: https://discordapp.com/oauth2/authorize?&client_id=279722369260453888&scope=bot&permissions=0
full documentation: https://github.com/Rolisteam/DiceParser/blob/master/HelpMe.md


Fix response.request accessible from PrometheusBackend

We had a problem in our codebase that we reported example.com to prometheus because we took it from response and not request.
Im not sure if we want to merge it like that. The whole existence of ExampleGet exposed in the core looks quite dangerous to me, but I don't feel adventurous enough to try deeper refactor.

True, this should be probably more clearly marked as testing utilities to avoid bugs as here. Can you create an issue to fix this in sttp4?
Meanwhile, let's see where else this apply is used :)


Connections' context menu needs improvements

As of 4.3.1 (but also in 4.2.6 and before) there are 2 things I find not intuitive:

When using Postgres, "Create New Database" can be found in context menu for... other database inside connection, not for connection itself (and it doesn't make sense since you create database within connection, not inside other database). On the other hand, creating new database for MySQL connection is intuitive since there is "Databases" node and context menu for that node allows creating new database.
"Creating New Connection" can be found in any connections' context menu and it does not make sense too. There is "New Connection" button at the top of the "Database Navigator" view, also there is item in context menu when clicking on the view's background (not connection) and it's all user needs. When user right-clicks on connection, menu should represent connection's context and creating connection from other connection is, in my opinion, out of that context.

I hope this is clear enough :smile:

behaviour of the Create menu is changed.
Seems that ticket is obsolete


Creation of a new jupyter notebook tutorial

This issue is meant as a unitaryHACK bounty issue.
Bounty: $75
We would like to make scqubits easier to use by providing more involved tutorials that describe its various features. Some of these are already present in this repository and can be explored interactively here.
The new jupyter notebook tutorial should present examples of interfacing scqubits with qutip.
This could include:

building a composite system (e.g., a fluxonium qubit coupled to resonator)
calculating some relevant drive operators in the dressed basis of the composite system (e.g. charge operator, for the resonator and/or the fluxonium)
using the effective Hamiltonian and relevant drive operator in qutip's mesolve or sesolve to study the evolution of the system being subject to some reasonable time-dependent pulses (that should, for example, result in a gate).

A more involved example could include doing some evolution as part of a custom ParameterSweep (see "Custom Sweeps" section of this notebook for ideas).
The final jupyter notebook tutorial should be added to this examples/tutorials repository.

Hi @RamAIbot
I would guess a good starting point would be the tutorials we already have (see top post for links). That could give you a sense what's this one might look like.
There is a bit of freedom with what you choose to do, but the list of items I outlined might represent an easy way to get things going (i.e. see point form items in my top post).
Let me know if there is something specific that's not clear and we can go from there.

This is from an old hackathon. Not needed anymore, closing.


[WIP] Remove checkpanic from slbeta1 open-api tool

Purpose

$subject
Fix for request payload record name mismatch in method signature
Fix for parameter doc comment mismatch from actual parameter name
Fix for api key + OAuth authentication code generation issue

Resolves https://github.com/wso2-enterprise/choreo/issues/6008
Goals

Update slbeta1 version of open-api tool

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

Java 11
Ballerina slbeta1

Fixes are added along with the code revamping https://github.com/ballerina-platform/ballerina-openapi/pull/365


Efiling additions to the Appeals Court package

removes questions that are already defined in AL question library. Not sure if it's included by default by including this package though.
adds a few e-filing specific questions that would be needed in multiple Appeals forms.

Haven't tested yet though, and would likely want to before.
Do you think that just going through the existing appeals forms would work well here?

Tested on the packages that were including MassAppealsCourt in interviews (motion to stay general, appeals misc motion, late docketing statement, civil docketing, motion to reconsider, and motion to stay eviction), they all look fine and nothing seems to break, so pushing now.


Fix assoc type shorthand from method bounds

In code like this:
impl<T> Option<T> {
    fn as_deref(&self) -> T::Target where T: Deref {}
}

when trying to resolve the associated type T::Target, we were only
looking at the bounds on the impl (where the type parameter is defined),
but the method can add additional bounds that can also be used to refer
to associated types. Hence, when resolving such an associated type, it's
not enough to just know the type parameter T, we also need to know
exactly where we are currently.
This fixes #11364 (beta apparently switched some bounds around).

bors r+


opendaylight latest image not found

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Go to the OpenDaylight repo.
Observe no latest tags were created for that image
Select a different tag in Kitematic
🎉  enjoy container running.

Ideally you open an issue with the repo owner to let them know how frustrating it is that they don't have a latest image tagged


Use local faucet in CI

this is how far I got. Feel free to continue this work.

I can take a swing at the pk file if you wish... at least make the output match current faucet style.

Okay, there is the idea of just reimplementing the faucet in ts on top of iov-core. This came from backend guys.
If that is done, we can easily test in one process in CI, no docker needed. I'll just leave this pending until then.

superseded by #472


Flaky Network -> RemoteProtocolError

Description
Transfered from Bug report
Describe the bug
When I am running my Pytest modules on a Github hosted runner, there are some flaky tests.
This occurs because I get a RemoteProtocolError when trying to connect with an external API Castor EDC.
This happens for a different test every time, and as such is not really reliably reproducable.
However, when tests are run on my local machine with the same settings, this never happens.
Already tried:

Checking logs at remote API: they see no failed requests
Turning of TCP/UDP offloading with sudo ethtool -k eth0 tx off rx off
Changed all fixtures to be ran for each function so a new connection is made every time

To Reproduce
Run the pytest.yml action for my repository:
https://github.com/reiniervlinschoten/castoredc_api/actions/workflows/pytest.yml
See for example:
https://github.com/reiniervlinschoten/castoredc_api/runs/7089556459?check_suite_focus=true
https://github.com/reiniervlinschoten/castoredc_api/runs/7074334780?check_suite_focus=true
https://github.com/reiniervlinschoten/castoredc_api/runs/7074535607?check_suite_focus=true
Expected behavior
That all testing steps pass.
What's not working?
The network connection to the remote API.
The logs of the remote API have been checked and they don't see anything out of the ordinary, and it works on my machine.
Error: /opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/httpx/_transports/default.py:78: RemoteProtocolError
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions

Virtual environments affected

[ ] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
2.294.0
See also the build
Is it regression?
No
Expected behavior
Describe the bug
When I am running my Pytest modules on a Github hosted runner, there are some flaky tests.
This occurs because I get a RemoteProtocolError when trying to connect with an external API Castor EDC.
This happens for a different test every time, and as such is not really reliably reproducable.
However, when tests are run on my local machine with the same settings, this never happens.
Already tried:

Checking logs at remote API: they see no failed requests
Turning of TCP/UDP offloading with sudo ethtool -k eth0 tx off rx off
Changed all fixtures to be ran for each function so a new connection is made every time

What's not working?
The network connection to the remote API.
The logs of the remote API have been checked and they don't see anything out of the ordinary, and it works on my machine.
Error: /opt/hostedtoolcache/Python/3.8.12/x64/lib/python3.8/site-packages/httpx/_transports/default.py:78: RemoteProtocolError
Actual behavior
I expect that all testing steps pass, as they do all run correctly on my local machine and sometimes on the runner.
Repro steps
Run the pytest.yml action for my repository:
https://github.com/reiniervlinschoten/castoredc_api/actions/workflows/pytest.yml
See for example:
https://github.com/reiniervlinschoten/castoredc_api/runs/7089556459?check_suite_focus=true
https://github.com/reiniervlinschoten/castoredc_api/runs/7074334780?check_suite_focus=true
https://github.com/reiniervlinschoten/castoredc_api/runs/7074535607?check_suite_focus=true

Hey @reiniervlinschoten.
Please attach tcpdump trace like in - https://github.com/actions/runner-images/issues/5615#issuecomment-1138741887

@al-cheb, see below.
sntp capture.zip

@al-cheb, see below (zip because I can't upload .cap files)
sntp capture.zip

The file is empty. Please pay attention at the port configuration  sudo tcpdump -nn -i any port 123 -w sntp.cap &, 123 - replace to API port.

I will do this, but I see that some API specifications changed, so I'll need to adjust my testing suite/code first. I'll get back to you with the proper capture file.

Yes, it seems that with a change in backend architecture for the API, the tests are less flaky. Closing this issue for now.


BeReferralPrescription requester field

Hello,
As discussed in a meeting on the 07/04/2023, in the BeReferralPrescription, the requester field only allows a reference to a BePractitioner :

But it should only allow a reference to a BePractitionerRole.

WG: we agree on this.


Cannot use Intellij Diagrams

I don't fully understand how this works, but after changing theme i cannot use Diagrams
I'm not receiving any errors, diagrams just simply don't show. After switching back to default theme everything works as expected.

I don't have any issues here... which OS are you on? Which version of IDEA?

Ach, maybe that is the case... I'm stuck with 2016.1.2 under Windows 7


Add TruthFinder as easy

added TruthFinder

Entry needs to be sorted within the JSON, see the error on the CI

Thanks for your contribution!


Error: No provider for Title!

Hi,
everything is working perfectly when I don't use server-side rendering but when I use it I get this exception
Exception: Call to Node module failed with error: Error: Uncaught (in promise): Error: No provider for Title!
any ideas?

Fixed in v3.0.0 and above.


Declarative JPMS support

The current version of Netty supports automatic module names.
This should bring declarative JPMS support in the new Netty 4.2 branch with a set of module-info descriptors that provides a better JPMS  support for Netty consumers.
Motivation
Although Netty remains compatible with old version of Java, the Java runtime can often be a recent JVM. The JEP Integrity by default will restrict how Netty operates in the future (JNI, Unsafe). Supporting JPMS will help to mitigate this by making Netty a clear identified consumer of these features, letting users configure Netty to use them (e.g. --enable-native-access=io.netty.xxx).
JPMS is a multi-year / multi-project effort, this is a second step toward JPMS support.
Goals

Netty continue to operate as before when Netty jars are on the classpath
No breaking changes with classpath, module path might have behavioural changes which should be considered as acceptable
Give the opportunity to JPMS consumers (libraries / frameworks / applications) of Netty (e.g. Eclipse Vert.x) to use Netty on the module path instead of the classpath

Non goals

Encapsulation is not a goal as it requires impacting changes in Netty

Implementation
Since Netty 4.2 aims to use Java 8, the considered approach is the generation of module descriptors in Netty jars using a tool like moditect or module-info.
Module descriptors is generated at META-INF/versions/11/module-info.class (multi release jar) which makes it invisible as a class to Java 8 class loading.
Descriptor will export all Netty packages by default, it might be restricted in later versions, e.g. having logging internal packages could be attempted.
Maven integration tests should be provided to test the behaviour, e.g. creating a jlink image (jlink does not accept automatic modules), etc...
Known issues and open concerns

for most modules, jar and test-jar exhibit split packages

prevents test jar reuse
not blocker, but might be in the future

netty-common coupling to io.projectreactor.tools:blockhound

this currently works because the io.netty.util.internal.Hidden and META-INF/services/reactor.blockhound.integration.BlockHoundIntegration are ignored by Java
Java requires to publish this as a provides…with module declaration which couple the module with this jar which might not be desirable for most users
not blocker, but might be in the future
this could be offloaded to a third party jar and does not require to be in netty-common

there is a difference between native transport and openssl native engine with respect to the naming of module with a classifier (containing the native libraries to load)

openssl assumes only a single dependency can be present at runtime and jars with classifier use the same module name io.netty.internal.tcnative
native transports declare module names with the classifier, e.g. io.netty.transport.kqueue.osx.aarch_64
we need to judge pros and cons for both approach and settle on the best approach

integration test for BouncyCastle is not trivial since there are fallbacks when it is missing
[x] split package issue between io.netty.codec and io.netty.codec.xml

Testing
Testing module descriptors is achieve through a dedicated test suite that requires Java 11 or above.
There are several level of testing of a module

compilation: the tested module is used as a dependency
execution: there is a test that is executed with unit and use it
jlink creation: a jlink image can be create with the module
jlink runtime: the feature can be tested in a jlink generated image

Upstream declarative JPMS support
Netty dependencies should also provide declarative JPMS descriptors, some dependencies already support it.
Most of the dependencies below are optional and the lack of declarative JPMS support means taking the appropriate measure and might not be a blocker issue.
Dependencies already supporting declarative JPMS

[x] brotli4j
[x] bouncy castle
[x] compress-lzf
[ ] zstd-jini

issue

[x] FasterXML/aalto-xml and FasterXML/stax2-api

issue

Dependencies that shall support declarative JPMS in a reasonable time frame

[ ] tcnative

[ ] contrib
[ ] issue to fix

[ ] prototobuf

issue
contrib by @sgammon

[ ] jboss-marshalling

Dependencies under investigation

[ ] lzma-java
[ ] rxtx
[ ] conscrypt

Dependencies considered inactive or deprecated
Does not mean it is hopeless.

[x] Jetty/alpn-api: dropped since OpenJDK8 supports ALPN
[ ] Jetty/npn: Jetty NPN repository is archived as they consider it deprecated in favour of ALPN. Netty 4.2 still provides support for SPDY which is the main motivation for keeping NPN support
[ ] barchart-udt: the project looks inactive
[ ] jzlib: project maintenance seems discontinued
[ ] lz4: project maintenance is discontinued

Contribution
I am volunteering to implement JPMS support and supporting it in the long run. I welcome any help for this effort of course.

Sounds good :) Looking forward to a PR.

FWIW here is a PR on my own repo of the work in progress : https://github.com/vietj/netty/pull/2

Please let me know how I can help, for example by rebasing the protobuf PR upstream. Excited to see energy around JPMS! 😄

cc / @graemerocher

(You should find the path clear for errorprone and Guava dependencies as well, although Guava itself is still pending and also needs a rebase.)

@sgammon anything that can get protobuf get declarative JPMS support is an awesome progress

@vietj We have a repo called the "JPMS Attic" which can hold upstream artifacts before they release: https://github.com/elide-dev/jpms
I am happy to push additional artifacts there if at all helpful. That is where we perform integration tests with downstream libraries. I'll probably add Netty once this PR is fully formed. I'm also happy to help with downstream testing as we're big users of Netty in several places.

sounds great @sgammon I noticed this repo as well.
do you think Protobuf Java will eventually accept your JPMS patch ?

@vietj I think it is likely but it may take awhile. In the meantime, the JPMS Attic already ships Protobuf with JPMS, so you can test against that. It will stay identical with my PR

Keep in mind that anything that has Guava in the dependency graph also is precluded from full JPMS support. So Guava and Protobuf will probably both be needed as I imagine Netty depends on Guava somewhere.

protobuf is a static dependency for the protobuf codec
fortunately Netty does not use Guava at all

so overall it is possible to support JPMS with jlink out of the box and I think we are looking quite solid

@vietj I think it is likely but it may take awhile. In the meantime, the JPMS Attic already ships Protobuf with JPMS, so you can test against that. It will stay identical with my PR

we might be using this dependency in the testsuite-jpms for testing purpose

we could maybe also cleanup the netty-codec module and extract some modules (marshalling / protobuf) like the codec-xml, I'll open an issue for this

Updated the issue with a coverage section, the effort is looking good, still requires a few items to resolve before completion.

I have just released zstd-jni-1.5.6-5 with the fix for the Java module declaration https://github.com/luben/zstd-jni/issues/322

that's great @luben I'll update the PR with this new dependency and add testing for it

@vietj is there anything we need for 4.2 that is left ?

there are a few items we haven't yet discussed in the issue but I consider the JPMS support as fine, besides known limitations due to some dependencies that we should seek to address in the future

so we can close this issue from my perspective, this was an umbrella issue and I'll reopen issues for the possible improvements we can do in the future


Add Windows Clipboard Shortcut Key

Hi,
I would like to add a very handy shortcut key for Windows to view the Clipboard.
Win + V
This will show all the copied text and used it multiple times in clipboard app.

It's really a good feature, that I missed mentioning. Sure, You can add it! Just make sure to add it to the right section and incase I missed few more, you can add them too :)

It's really a good feature, that I missed mentioning. Sure, You can add it! Just make sure to add it to the right section and incase I missed few more, you can add them too :)

hi aryan,
I just add it.. please check if possible to merge? Thanks


Fix various theming issues

When removing preem-ui several font and background colors did not get updated. Need to take a pass and ensure the telestrator supports all theming options.

Fixed 0.0.4


Pagination is not supported error given in the response when startIndex is set to a number other than 1 in Filtering groups with SCIM2

Suggested Labels
Affected : 5.8.0- beta
Severity: Major
Priority: Highest
Component: SCIM2
Type-Bug
Steps to reproduce

Create 2 roles as manager and mana via management console
Try to filter the created groups with the below request

`curl -v -k --user admin:admin 'https://is580cluster2.qa.com:443/scim2/Groups?startIndex=3&count=2&filter=displayName+sw+m'
Response Received{"schemas":["urn:ietf:params:scim:api:messages:2.0:Error"],"detail":"Pagination is not supported","status":"501"}`
When the startIndex=1 will get the response as below without error
Request
curl -v -k --user admin:admin 'https://is580cluster2.qa.com:443/scim2/Groups?startIndex=1&count=2&filter=displayName+sw+m'
Response
{"totalResults":2,"startIndex":1,"itemsPerPage":2,"schemas":["urn:ietf:params:scim:api:messages:2.0:ListResponse"],"Resources":[{"displayName":"PRIMARY/manager","meta":{"created":"2019-04-17T06:39:23Z","location":"https://is580cluster2.qa.com/scim2/Groups/8cdaedd1-3f51-4144-be49-f25dde786e40","lastModified":"2019-04-17T06:39:23Z"},"id":"8cdaedd1-3f51-4144-be49-f25dde786e40"},{"displayName":"PRIMARY/mana","meta":{"created":"2019-04-17T10:31:39Z","location":"https://is580cluster2.qa.com/scim2/Groups/af4220e9-6739-4705-acbe-ccd220f8f31d","lastModified":"2019-04-17T10:31:39Z"},"id":"af4220e9-6739-4705-acbe-ccd220f8f31d"}]}
`

This is the current behaviour since pagination is not implemented in groups endpoint. Only count param in request will not give this error since the count param is omitted when processing the request.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


pure-maps: 2.9.2 -> 3.0.0

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Should we make mapbox-gl-native an alias of maplibre-gl-native?

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 169702 run on x86_64-linux 1

4 packages failed to build and are new build failures:

libsForQt512.mapbox-gl-qml: log was empty
libsForQt512.maplibre-gl-native: log was empty
libsForQt514.mapbox-gl-qml: log was empty
libsForQt514.maplibre-gl-native: log was empty

3 packages built:

libsForQt5.mapbox-gl-qml (libsForQt515.mapbox-gl-qml ,plasma5Packages.mapbox-gl-qml)
libsForQt5.maplibre-gl-native (libsForQt515.maplibre-gl-native ,plasma5Packages.maplibre-gl-native)
pure-maps

4 packages failed to build and are new build failures

I marked them as broken for Qt < 5.15

Result of nixpkgs-review pr 169702 run on x86_64-linux 1

4 packages marked as broken and skipped:

libsForQt512.mapbox-gl-qml
libsForQt512.maplibre-gl-native
libsForQt514.mapbox-gl-qml
libsForQt514.maplibre-gl-native

3 packages built:

libsForQt5.mapbox-gl-qml (libsForQt515.mapbox-gl-qml ,plasma5Packages.mapbox-gl-qml)
libsForQt5.maplibre-gl-native (libsForQt515.maplibre-gl-native ,plasma5Packages.maplibre-gl-native)
pure-maps


About the Forward Transfer

Hi,
Thanks for your excellent work!
I'm curious about how to calculate the "Forward Transfer" after training. For example, I have successfully re-produced the class-il results for Fine-tuning and CaSSLe (with BYOL) on Cifar-100 but don't know how to directly check the FT results. Does it need a seperate run to obtain the "linear evaluation accuracy of a random network" as the paper stated?
BTW, just to be sure, is it right to directly check the "val_acc1" results of wandb board as the final linear evaluation accuracy?

Hi. Thanks for the interest in our work.
Yes, if I remember correctly I ran linear evaluation on a random network.
Yes, it is fine to use the val_acc1 for cifar, while for in100 and DomainNet you should run linear evaluation (for all intermediate tasks as well).

Got it, Thanks again!