[PERFORMANCE]: TS takes several seconds to validate types resulting in poor DX

Bug Report

🔎 Search Terms

🕗 Version & Regression Information

This is a slow type verification leading to bad DX.

⏯ Playground Link

Playground link with relevant code
Related issue: https://github.com/drizzle-team/drizzle-orm/issues/870
💻 Code

// We can quickly address your report if:
//  - The code sample is short. Nearly all TypeScript bugs can be demonstrated in 20-30 lines of code!
//  - It doesn't use external libraries. These are often issues with the type definitions rather than TypeScript bugs.
//  - The incorrectness of the behavior is readily apparent from reading the sample.
// Reports are slower to investigate if:
//  - We have to pare too much extraneous code.
//  - We have to clone a large repo and validate that the problem isn't elsewhere.
//  - The sample is confusing or doesn't clearly demonstrate what's wrong.

I understand that this issue uses an external library, drizzle which is the source of performance problems. However, I think it would be useful for the TS team to see how people use types in real world and to see if either the drizzle team can fix the types to be better performant or the TS team can do something to better handle the use cases used by drizzle.
import { integer, sqliteTable, text } from 'drizzle-orm/sqlite-core';

/**
 * This is a reproduction of slow TypeScript type validation with Drizzle.
 *
 * Steps to reproduce:
 * 1. Change "EDIT THIS" line and change `2` to `"2"` (add qoutes around the number)
 *    and save the file
 * 2. Because we have changed number to string the TS will now underline the line as
 *    being invalid.
 * 3. Go back and forth between the `2` and `"2"` and notice that there is a several
 *    second delay everytime you change source code and TS either has to add or remove
 *    the red underline. (TS Error)
 * 4. Once you establish a beseline feel to how long it takes TS to add/removo an error,
 *    comment out the `delayCount`/`latencyCount` block and repeat the steps
 * 5.  This interferes with code-completion and makes for a bad DX.
🙂 Expected behavior

Type validation should stay performant.

Please provide a self-contained repro; we do not have time to dig into large external typings to figure out if they're written performantly or not.

@mhevery #55224 will mitigate a good bit of this. Not all, but in your specific example's case about 50% overall of a tsc run, which should translate to a higher fraction during editing as tsc has to parse/bind anew (then emit), while the editor won't do any of that.
If you want to try it out before it's in a release, you can install the package from https://github.com/microsoft/TypeScript/pull/55224#issuecomment-1664729772 and then switch your editor to use it instead of what's built into VS Code (or wait for a nightly).

@jakebailey and I had further discussion around this tonight. Both his PR for recursiveTypeRelatedTo and drizzle's recent typing updates removing HKTs have made a significant improvement, however I'm still seeing situations where intellisense is slowing down (though, notably, NOT tsc). Moving from 5.1 to nightly in VS code on my "real" project I did see the reported ~50% speedup - it's possible the issue have may moved somewhere else now (and, if so, may be worth a separate issue).
The reproduction I have is similar to the original here, though larger: https://stackblitz.com/edit/stackblitz-starters-uaglzu?file=index.ts Try waiting at every . of db.query.edgeTable2.findFirst, and you'll see the slowdown.
I'm actively working on trying to build out a more minimal reproduction extracted from drizzle. So far a big thing I've noticed is that a big contributor is the extends in the key portion of the following mapped type (ExtractTablesWithRelations): [K in keyof TSchema as TSchema[K] extends Table ? K : never]: TSchema[K] extends Table ? {...} : never. Drizzle's extensive use of generics (including each column having a high likelihood of having a unique type due to the column name being embedded as part of the type) is assuredly not helping here


Static CSS Extraction (take 2)

What:
This is a continuation of some of the discussions and work that took place over in this PR: https://github.com/tkh44/emotion/pull/121. This adds another mode to the babel plugin that acts a bit like the extract mode and a bit like the inline mode.
{
  "plugins": [
    ["emotion/babel", {"extractStatic": true }]
  ]
}

Why:
I want the ability to cache static css and offload as much as possible from being done at run time, while also supporting older browsers than IE11.

How:
Clones the css root inside parser.js, removes dynamic content from it and removes static content from the original root, then returns two arrays (one dynamic, the other all static).
The static css then gets send to state.insertStaticRules to be inserted into the written out css file.

Checklist:

[ ] Documentation
[ ] Tests
[x] Code complete

Follow the original discussion about this here: https://github.com/tkh44/emotion/pull/121

Codecov Report

Merging #152 into master will decrease coverage by 1.63%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
- Coverage   88.88%   87.25%   -1.64%
==========================================
  Files          16       16
  Lines         594      612      +18
  Branches      132      143      +11
==========================================
+ Hits          528      534       +6
- Misses         53       60       +7
- Partials       13       18       +5

Impacted Files
Coverage Δ

src/inline.js
100% <100%> (ø)
:arrow_up:

src/parser.js
81.81% <33.33%> (-18.19%)
:arrow_down:

src/babel.js
98.42% <92.3%> (-1.58%)
:arrow_down:

Continue to review full report at Codecov.

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

We got this working in #154

That's awesome! This sort of felt like a long shot when looking at the initial work on the glamor re-write. Too excited right now.


Refactor xhgui launch command

DDEV introduced a way to reduce "launch" comand boilerplate using a port. https://github.com/ddev/ddev/pull/6024
This PR takes advantage of that, and reduces the laucnh down to a simple "port" implementation.
We'll need require DDEV 1.23 (?) during install.
This is considered a breaking change and will require a point release.

What is the recommended way to check for DDEV version for this PR?
The add-on-template uses ddev debug capabilities
$ ddev debug capabilities
multiple-dockerfiles
interactive-project-selection
ddev-get-yaml-interpolation
config-star-yaml-merging
pre-dockerfile-insertion
user-env-var
npm-yarn-caching
exposed-ports-configuration
daemon-run-configuration
get-volume-db-version
migrate-database
web-start-hooks
add-on-versioning
multiple-upload-dirs
debian-bookworm
corepack

But I'm not sure this is a "feature". Does bash has native support for parsing a string as a version number for comparison?

Why not just use ddev_version_constraint in config.xhgui.yaml? That might the best.
ddev/ddev-platformsh uses the capabilities technique, https://github.com/ddev/ddev-platformsh/blob/main/install.yaml#L23
SOMEWHERE I have a bash function for comparing version.
The only one I can find right now is:
https://github.com/ddev/ddev/blob/6e82ece9e371e9e4a54bcbe4495b1aab52eb39c3/.buildkite/sanetestbot.sh#L13

Seems like corepack was also introduced in 1.23 so I'm using that in the pre_install_actions.


Owner Permissions

added custom permissions so owner can run all commands
falls back to normal permission check if not owner

Codecov Report

Merging #9 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master      #9     +/-   ##
========================================
- Coverage    3.29%   3.19%   -0.1%
========================================
  Files          35      36      +1
  Lines         455     469     +14
  Branches       73      77      +4
========================================
  Hits           15      15
- Misses        368     378     +10
- Partials       72      76      +4

Impacted Files
Coverage Δ

src/commands/loot/remove.js
0% <ø> (ø)
:arrow_up:

src/commands/tier/add.js
0% <ø> (ø)
:arrow_up:

src/commands/message/update.js
0% <ø> (ø)
:arrow_up:

src/commands/message/remove.js
0% <ø> (ø)
:arrow_up:

src/commands/message/add.js
0% <ø> (ø)
:arrow_up:

src/commands/tier/list.js
0% <ø> (ø)
:arrow_up:

src/commands/tier/remove.js
0% <ø> (ø)
:arrow_up:

src/commands/loot/update.js
0% <ø> (ø)
:arrow_up:

src/commands/tier/update.js
0% <ø> (ø)
:arrow_up:

src/commands/loot/open.js
24.24% <ø> (ø)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Save a MaterialColor type directly.

Question
I have a class which as a property of type Color. I'd like to save the Color directly instead of converting to int and back.
Code sample
Category Class
part 'category_model.g.dart';

@HiveType(typeId: 2)
class Category {
  @HiveField(0)
  final String id;
  @HiveField(1)
  final String title;
  @HiveField(2)
  final Color color;

  const Category({
    @required this.id,
    @required this.title,
    this.color = Colors.green,
  });
}

 Generated Adapter Class
// GENERATED CODE - DO NOT MODIFY BY HAND

part of 'category_model.dart';

// **************************************************************************
// TypeAdapterGenerator
// **************************************************************************

class CategoryAdapter extends TypeAdapter<Category> {
  @override
  final int typeId = 2;

  @override
  Category read(BinaryReader reader) {
    final numOfFields = reader.readByte();
    final fields = <int, dynamic>{
      for (int i = 0; i < numOfFields; i++) reader.readByte(): reader.read(),
    };
    return Category(
      id: fields[0] as String,
      title: fields[1] as String,
      color: fields[2] as Color,
    );
  }

  @override
  void write(BinaryWriter writer, Category obj) {
    writer
      ..writeByte(3)
      ..writeByte(0)
      ..write(obj.id)
      ..writeByte(1)
      ..write(obj.title)
      ..writeByte(2)
      ..write(obj.color);
  }

  @override
  int get hashCode => typeId.hashCode;

  @override
  bool operator ==(Object other) =>
      identical(this, other) ||
      other is CategoryAdapter &&
          runtimeType == other.runtimeType &&
          typeId == other.typeId;
}

Version

Platform: Linux
Flutter version: 1.22.0-10.0.pre.455
Hive version: ^1.4.4+1
Hive Generator version: ^0.8.1
Build Runner version: ^1.10.3

You have to create and register adapter for Color type.
Check this file for adapter: color_adapter.dart


JSFiddle doesn't work

Hi, i use the code
source code
but gitbook doesn't display embedded result.
What's wrong here?

Can you show yours markdown, where are you insert jsfidller link?

Hi, i write markdown like this:
 [source code](https://jsfiddle.net/zalun/NmudS/embedded/result,js,html,css/)

You shouldn`t set 'embedded/result,js,html,css' those params in url. You can just paste  source code and plugins automaticly add reuires params in url.

I've tried but it's the same as before.
It's just render an anchor tag to that jsfiddle.
It's really strange here.

Can you provide gitbook config and package.json?

is it possible this plugin doesn't work direct at gitbook.com ? Is it only for local installation ?

Plugin work fine with gitbook. For exmaple our book is build direct in gitbook.com. You can see book.json  and package.json. Can you show your book.json and package.json?

Thanks, now it works. If was confusing the book configuration menu fires only errors. I copied only the example config. Adding title section and surrounding block was the solution :smile:

Thanks, now it works. If was confusing the book configuration menu fires only errors. I copied only the example config. Adding title section and surrounding block was the solution :smile:


[WIP] Stop copying metadata on disk from output repos

holding off on merging this because we don't actually have evidence of a speedup yet
Remove the unix-specific downloadTree implementation which wrote the block to disk. This was redundant because objClient now caches metadata blocks on disk, so we were going block storage -> disk -> disk -> merger.

I tested a few improvements:

removing the copy on disk from downloadTree
avoiding deserializing every proto in a block in nodes

Neither of them seemed to put a big dent in the total latency for GetFile. So the theory that the bulk of the time is streaming back data blocks seems feasible.


Benchmark on APPS dataset

This PR adds an ability to benchmark gpt-engineer on problems given in APPS dataset

How to run?
python3 gpt_engineer/benchmark gpt_engineer/core/default/simple_agent.py apps
Original Issue: https://github.com/gpt-engineer-org/gpt-engineer/issues/819

Looking forward to have apps merged!


Uses +[NSKeyedUnarchiver archivedDataWithRootObject:] which is deprecated when targeting iOS12

There is a simple replacement.

-unarchiveObjectWithData: addressed with #247

@bolsinga can this issue be closed now?

No, it can't. I'd mangled the title incorrectly. What remains is the use of+[NSKeyedUnarchiver unarchiveObjectWithData:]


All documentation for Core Library is missing

I guess this is our life now. Gonna start printing out docs on paper in case of emergency.
Repro steps

navigate to https://docs.microsoft.com/en-us/dotnet/fsharp/

Click "F# Core Library References

Attempt to google.

Expected behavior
I can find or read the documentation.
Actual behavior
I cannot find or read the documentation.
Known workarounds
None, I've tried crying a lot but it doesn't seem to help.
Related information
The error is a "500 error". I spoke to ~Reed~ @cartermp , he's already sent an email out about this so perhaps this issue is redundant. Please read the drama as humor and not any attempts to undermine people's hard work.

Correction - I think @cartermp sent out (or is involved in) the email chain about this...

@voronoipotato,
sobbing is good, and cursing.  They don't solve the underlying problem, but it makes me feel happier.
Kevin

Yes, Sorry @cartermp it was in a chatroom and I didn't carefully read who posted what. I'll try to be more careful about attribution in the future.

No worries - email is out, we'll see what's up relatively soon

Live again, looks like it was intermittent.


[question] difference between function foo(): A and function foo(): any

What is the difference between the function types
function foo<A>(): A
and
function foo(): any
?
It seems to me that both have the same effects - the first one seems more "sophisticated" and I see it sometimes in some projects, but it is the same as the second one, right?

I see it sometimes in some projects

function foo<A>(): A is not implementable, could you point to some examples?

Hm, you are right. But I have seen it in some declaration files. For example, here, immutable.js (again :))
https://github.com/facebook/immutable-js/blob/master/type-definitions/immutable.js.flow#L626
declare class Record<T: Object> {
  get<A>(key: $Keys<T>): A;
}

this is effectively get(key: $Keys<T>): any, right

declare class Record<T: Object> {
  get<A>(key: $Keys<T>): A;
}

In this case it's actually the same as get(key: $Keys<T>): empty -- but the difference between empty and any in this position is pretty nuanced and might even not be observable...so it may not be important here.
I suspect whoever did this just used a type param to state that the returned type isn't known because Flow doesn't really have a way to express the correct type -- but didn't want to suggest that it's any type, just some type that Flow doesn't have the facilities to express just yet.
Here's a strawman (that doesn't currently work) for how we might fill this out to be more complete:
declare class Record<T: Object> {
  get<Key: $Keys<T>>(key: Key): $PropertyType <T, Key>;
}

This doesn't currently work due to a limitation in the way $PropertyType works (and it may/may not be tractable to fix), but the idea is that we're using the type parameter to establish relationships between the types without actually saying what the specific type is.
Refresher on type parameters:
"function foo<T>(p: T): T { return p; }"
This makes a type parameter for the foo function. Type parameters are much like function parameters in that they are a variable that can be filled in later -- except they are a variable for a type rather than a variable for a value.
Here we create a type parameter, T, in order to say: Any time this function is called, the type of p and the type the function returns should be the same type. So if you call foo(..) with a string, then it should also return a string. If you call it with a number, it returns a number.
Type parameters let you establish type relationships without saying what the specific type is.
"function foo<T: {length: number}>(p: T): T { return p; }"
When we add the : {length: number} to the type parameter, we're saying that whatever the T type parameter is fulfilled with, it has to be compatible with the type on the right-hand side of the :.
In this case, we're saying that you can send any type into this function as long as it is an object that has a length: number property. The : is a way of setting a boundary on the possible types that can be used to fulfill the relationship between p and the return type.
So here, foo(42); is a type error (numbers don't have a length property). But foo("asdf") is ok (strings have a length property) and foo([]) is also ok since arrays have a length property.
"get<A>(key: $Keys<T>): A;"
Here we're creating a type parameter but we're not relating it to anything, so it's kind of useless. Type parameters basically start out as empty until they are filled in in some way. Since we're not filling it in here (and we're not relating it to anything), it just stays empty.
"get<KeyT: $Keys<T>>(key: KeyT): $PropertyType <T, KeyT>;"
Ok now we're getting pretty advanced. Here we create a type parameter called KeyT that we say must be compatible with $Keys<T> (AKA: The set of keys on the T object). Since KeyT is a type parameter, it doesn't say exactly what the type is...only that it fits within $Keys<T> (the set of keys on the T object).
Then we relate the function parameter, key with the return type by saying that the type of key is the same as the "property type of T's KeyT type". $PropertyType is a built-in type that looks up the type of the property on an object type. So we pass in T (the record object) and KeyT (the type variable that represents the key the caller requested).

Yeah, that's what I tried, but of course $PropertyType doesn't work like that, so the excercise is rather moot :))
Anyway, I found out Records are not really as useful as I thought. For example, you cannot capture state with more "options" (I think it's called "disjoined union" in docs); for example, you cannot have immutable Record with type definition like this (from docs)
type OptionalObject = {
    type: 'success',
    result: number
} | {
    type: 'error',
    message: string
};

you cannot create a Immutable Record with this type.

I recognize now that using Immutable Record for typed application state is not really an option (too many complications) and I will need to take care of application state immutability on my own.

it would be nice if at least Flow could enforce immutability somehow :)
....buuut it seems this feature request already exist, with issue number two orders of magnitude smaller than this one. OK :)
https://github.com/facebook/flow/issues/18

@runn1ng: One way you can enforce immutable "records" in Flow now using covariance + exact types:
https://flowtype.org/try/#0C4TwDgpgBAChBOBnA9gOygXigbwD4CgooBqAM1QEMBbCALikWHgEtUBzAGkJIBtKb6jFuy5FiAV0QJ+dBk1Zt8uAL4BufPh4RgUeONSoAjO3pwkaTDm7lqsgEQBpCvAg87oqH1v07AIQC3WgDW7tyS0t5QdnoGxmyhahoA9ElQAJLAAOSIUMhBUMDIuhAUACZQFFAAxsgAbs7MFKg6YPDIkPCg+AAUMUbsAHQ2AnLCbACU6vgpUACi8G3wAIRQAMJNmToA7izA0IUV1XUNTS1tHV0pfXFDMpZ2qO0QdqpAA

Very nice!!!
Doesn't this fix #18, at least for some cases?


DBZ-1727 Debezium Postgres connector does not apply keepalive lsn

@igabaydulin Applied, thanks!

Thx!

I have checked if Postgres shutdowns properly now GitHub Action (spoiler: it does!) if anyone is interested.
@gunnarmorling @jpechane thanks!

@igabaydulin Thanks for your contribution! As you seem to be quite familiar with Github Actions, is there any chance you could take a look at https://issues.redhat.com/browse/DBZ-1720 ?

@jpechane yes, I can. Where can I find the list of non-core repos? Or maybe there is one specific repo you are interested in the first place?

@igabaydulin Thanks for the offer, the repos in question are
https://github.com/debezium/debezium.github.io
https://github.com/debezium/debezium-incubator
https://github.com/debezium/docker-images
https://github.com/debezium/postgres-decoderbufs
I've ordered them by preference


MSP430 cleanup

Syntax highlighting can be improved.
Single-line ; comments should be removed with filtering.

Probably needs a new asm mode: https://godbolt.org/g/VPmgEg

I think all the things left are covered in #295 ... or else have been fixed along the way.


[Enhancement] Move to an agent-based architecture for controlling interfaces

Is your feature request related to a problem? Please describe.
Completely decoupling the server from the manager via an 'agent' daemon would make it easier to implement multi-server, as well as
running the control-plane on separate systems, by eliminating the concept of 'server' almost entirely; the only internal difference would be managed vs unmanaged peers.
Describe the solution you'd like
Split the interface management code off into a separate 'agent' binary, which can be deployed on several peers at once.
Describe alternatives you've considered
Only other 'real' option for multi-server is to make the daemon itself have an 'agent' mode on every server not hosting the webUI, or config deployment via SSH or similar.
Additional context
This is simply a possible implementation of a necessary part of multi-server support, and would also facilitate "managed" clients with truly automated config rollout possible.

...I misread the README; this is already supported, albeit not quite as generically as I'd like. I'll do some digging and possibly make some PRs.


consumer configs like "max_partition_fetch_bytes" are not allowed to set manually

Pipeline starry_scholar execution failed with error:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/mage_ai/server/websocket_server.py", line 112, in run_pipeline
pipeline.execute_sync(

File "/usr/local/lib/python3.10/site-packages/mage_ai/data_preparation/models/pipeline.py", line 434, in execute_sync
StreamingPipelineExecutor(self).execute(

File "/usr/local/lib/python3.10/site-packages/mage_ai/data_preparation/executors/streaming_pipeline_executor.py", line 97, in execute
raise e

File "/usr/local/lib/python3.10/site-packages/mage_ai/data_preparation/executors/streaming_pipeline_executor.py", line 87, in execute
self.__execute_in_python(

File "/usr/local/lib/python3.10/site-packages/mage_ai/data_preparation/executors/streaming_pipeline_executor.py", line 114, in __execute_in_python
source = SourceFactory.get_source(

File "/usr/local/lib/python3.10/site-packages/mage_ai/streaming/sources/source_factory.py", line 31, in get_source
return KafkaSource(config, **kwargs)

File "/usr/local/lib/python3.10/site-packages/mage_ai/streaming/sources/base.py", line 21, in init
self.config = self.config_class.load(config=config)

File "/usr/local/lib/python3.10/site-packages/mage_ai/shared/config.py", line 47, in load
return self(**config)

TypeError: KafkaConfig.init() got an unexpected keyword argument 'max_partition_fetch_bytes'

We'll add this option to consumer config


[BUG] - Problem with wifi Auto-Tunnel in flightmode with active wifi connection.

Describe the bug
After clarifying https://t.me/wgtunnel/1/84
I checked to enter the VPN--WLAN in two different tunnels (one is primary tunnel) configurations: none worked.
Auto-Tunnel is running and active for untrusted wifi in the general settings. Also in the main tab not paused but resumed.
Manually connecting with Auto-Tunnel deactivated works as well as Auto-Tunnel for mobile data.
Smartphone:

Device: [Pixel 4a 5G]
Android Version: [Android 14]
App Version [3.4.2]

To Reproduce
I can reproduce it now. It works when in mobile data (flightmode deactivated) with VPN-WiFi connected.
But doesn't work when in flightmode and connected to VPN-WIFI.
Location doesn't get turned off when going into flight mode. Location is still on.
Expected behavior
Also Auto tunnel with VPN-WIFI when in flightmode.

I was able to replicate this and I believe I have a fix for this.

Please try this prerelease build that I'm hoping fixes this issye: https://github.com/zaneschepke/wgtunnel/releases/tag/auto-tunnel-reliability-testing


feat: uppercase the strings.

Current dependencies on/for this PR:

main

PR #5   👈

This comment was auto-generated by Graphite.


Add dl options for series and films

As Addi7ed does not have real API, it would be a great to add some options on dl command like :

-e, --episode
-s, --season
-n, --name
-ver, --version

An other way would be to make sub-commands from apis instead of options.
In this way, each apis could have its optionsavailable using --help option like :

subify dl addi7ed -n "IT Crowd" -s 03 -e 04 -ver "PDTV-RiVER" -l french
subify dl addi7ed -n "chappie" -l english

Thank you Gwendal for your comment.
I just want Subify to take up the KISS principle. I think we should have only a video as an input parameter. I'd prefer parsing the video name or use the API of OpenSubtitles to get information (season, episode, etc.) about the video from the hash. Then create our own API for Addic7ed with this kind of information as an input.
What do you think ?

I agree with you but we need to define some pattern to parse correctly the input video. So there is some points to discuss  :

How to determine if the input video is a film or a tv show ?
According to the web conventions seasons and episodes should be parse using a pattern like this
/.*?(\d{1,2}).*?(\d{1,2}).*?/g with \1 as seasons et \2 as episodes
How to determine which version to choose (there is many versions on addic7ed and there is no ranking system)
I found an api in js here : https://github.com/same31/addic7ed-api

Maybe with OpenSubtitles API. We could get information from the hash of the video. Actually we don't really care because we could use the search function of Addic7ed. I tried "Shameless.US.S06E04.720p.HDTV.X264-DIMENSION" in their main page and I was redirected to the right page. With a film name, I don't get redirected to the page, but it displays a page of results.
Many patterns could be used indeed. We could use the search function of Addic7ed.
No rank needed, we "just" have to found the right subtitle from the Team that edited the video (It's in the page). To extract this information, I created in 2 minutes an API with import.io It could work well with a little tuning : https://api.import.io/store/connector/20268375-825f-44e7-bbe6-4b04d8a36145/query?input=webpage/url:http%3A%2F%2Fwww.addic7ed.com%2Fserie%2FShameless%2528US%2529%2F6%2F4%2FGoing_Once%252C_Going_Twice&&_apikey=91b43c5e88094addbd5dd95d619139a88556fdfbaf3b1998f011d2973d4fc363b3805f7d5d3cd1f908fc0dcabd5159fc5f4724feda33dc243209a6233feb571aa2e988822aed743e335be6d7ac4e3e2f
Other is solution is to search in the DOM like the JS API you found.
Could be a start.

Keep in mind that aim is to give the best suitable subtitle, transparently by using several APIs. It really is a "best effort mode". If Subify can't parse for Addic7ed or don't find exactly the right subtitle, don't worry it will probably succeed in downloading from OpenSubtitles or SubDB.

Bumping an old issue, but the Subtitleseeker API could be used just by providing a release title (filename).

@gwleclerc @Lomanic New version is out with Addic7ed support :)


error: '_bma_irq_cb' was not declared in this scope

Hi There,
I see the following error when I try to compile/upload the example "twatch rainbox".
/Users/USER/Documents/Arduino/libraries/TWatch_2021_Library/src/drive/bma423.cpp: In member function 'void TWatchClass::bma423_set_feature_event(uint8_t, irq_Fun_cb_t)': /Users/USER/Documents/Arduino/libraries/TWatch_2021_Library/src/drive/bma423.cpp:307:5: error: '_bma_irq_cb' was not declared in this scope      _bma_irq_cb.single_tap_cb = cb;      ^~~~~~~~~~~ /Users/USER/Documents/Arduino/libraries/TWatch_2021_Library/src/drive/bma423.cpp:307:5: note: suggested alternative: 'hal_aio_irq_cb'      _bma_irq_cb.single_tap_cb = cb;      ^~~~~~~~~~~      hal_aio_irq_cb Multiple libraries were found for "WiFiClient.h"  Used: /Users/USER/Library/Arduino15/packages/esp32/hardware/esp32/2.0.5/libraries/WiFi  Not used: /Applications/Arduino.app/Contents/Java/libraries/WiFi exit status 1 Error compiling for board ESP32 Dev Module.
Board Config:

Tried with both configs:
#define TWatch2021_V1
// #define TWatch2021_V1_2
Have I something forgotten to setup?
kind regards
Londi

Same error, pls help


What is the largest 3d volumetric image for the practical Neuroglancer viewing?

Hello,
I am interested to know what is the largest 3d volumetric image for practical Neuroglancer viewing.  What I mean by "practical" here is, chunks are fully loaded in the browser without hitting the RAM limit. (No black tiles in the display while navigation.)
The largest 3d volumetric image (TIFF) I've tried so far is about 245GB in size. Converted to the precomputed format with CloudVolume.  However, I have doubts on such a large volume to begin with.  Typically, tiles no longer load on the browser probably due to hitting the RAM limit from what the laptop is capable of.
https://github.com/seung-lab/cloud-volume/issues/505#issuecomment-1166190081
What is the largest 3d volumetric image size viewed on the Neuroglancer from your past experience?  It would be nice to know the upper bound for practical for viewing by the general user.  If I understand correctly, Chrome is capable of 4GB of memory.

You can increase them, e.g. the GPU memory limit up to 2GB, but Chrome does not allow the process to use more than 4GB of memory in total.
https://github.com/google/neuroglancer/issues/204#issuecomment-612193500

It would be great to know from your experience in handling large 3d volumetric images.
Thank you,
-m

If you produce sufficient downsampling levels, there is no limit.  Neuroglancer works well with 1PB+ volumes:
https://h01-release.storage.googleapis.com/landing.html


Requesting Support for wakanim.tv

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
Can you please support download video from :  https://www.wakanim.tv/fr/v2/catalogue/episode/754/silver-spoon-12

Is it a duplicate issue ? i can't find where is the old topic


make it compatible with Twitter Bootstrap js

Something I have been planning for a long time

Things to do

[ ] Set global jQuery variable to $dQuery
[ ] Add a dummy $dQuery.fn.jquery version
[ ] search parents if dQuery.fn.closest doesnt work for children
[ ] Rename $dQuery.event to $dQuery.Event
[ ] Support $dQuery.fn.data (even with no arguments)
[ ] Alias $dQuery.fn.matches to $dQuery.fn.is
[ ] Support $.proxy, maybe, alias for Function.prototype.bind
[ ] Support $dQuery.fn.prop (El.X vs EL.getAttribute(X))
[ ] Alias $dQuery.fn.once to $dQuery.fn.one
[ ] Support multiple classes in {add, toggle, remove}Class
[ ] Improve event namespacing
[ ] Support $dQuery.isFunction <-- seriously?!
[ ] Support $dQuery.fn.detach

No longer interested in maintaining this project


schema: simplify meterSigGrp model

This PR remodels meterSigGrp to use model.meterSigLike as proposed by @kepper
closes #970

Still an empty <meterSigGrp/> validates… Somehow an <empty/> element makes it to the parsed RNG:
<define name="mei_meterSigGrp">
  <element name="meterSigGrp">
     <a:documentation xmlns:a="http://relaxng.org/ns/compatibility/annotations/1.0">(meter signature group) Used to capture alternating, interchanging, mixed or other non-standard meter signatures.</a:documentation>
     <ref name="mei_model.meterSigLike"/>
     <oneOrMore>
        <ref name="mei_model.meterSigLike"/>
     </oneOrMore>
     <ref name="mei_att.common.attributes"/>
     <ref name="mei_att.facsimile.attributes"/>
     <ref name="mei_att.meterSigGrp.anl.attributes"/>
     <ref name="mei_att.meterSigGrp.ges.attributes"/>
     <ref name="mei_att.meterSigGrp.log.attributes"/>
     <ref name="mei_att.meterSigGrp.vis.attributes"/>
     <empty/>
  </element>
</define>

I am not sure I have a full understanding of the discussion here, but allowing an empty meterSigGrp might be needed in some cases. One typical case encountered elsewhere (beams) is when @sameas is given. This should be done by Schematron, but it might actually be simpler just to allow empty meterSigGrp. This might in fact also be useful when your encoding it just a high level one and used to represent what is on a page without detailed information about it.

@bwbohl You find this empty element at the end of every definition in the RNG. It has nothing to do with the allowed content.
How are you validating this?

@rettinghaus I had a sample file with varying configurations of meterSigGrp from empty to nested several times. All of those did validate. But I got curious when I deleted the fileDesc, and still everything did validate.
I have to apologise: after a couple of follow-up tests, I am sure there is something astray with RNG validation in my @oxygenxml… still have to figure it out!

As stated here there is some bug in my validation scenario with oXygen XML. Jing did the job.

@lpugin Could you please provide an example with @sameas? I'm trying to understand in which context this could happen.

I am thinking the case with a mid-measure meterSigGrp with two layers. You would encode the one in the second layer with meterSigGrp@sameas.


Adding google-api-core[grpc] dependency

Taking a direct dependency on google-api-core[grpc] as recommended in https://github.com/googleapis/google-cloud-python/issues/5023
Skipping grpc installation on PyPy
Upgraded tox version to latest
Removing redundant dependency declarations from tox.ini (tox installs them via setup.py anyway)

Resolves #184

4 years later and this solved the ModuleNotFoundError: No module named 'google.auth' problem for me.
Specifically, I ran pip3 install 'google-api-core[grpc]', after which the error no longer occurred.


CI against Rails 8.0

Description
Rails 8.0 has been released. I would like to confirm whether draper works well with it.

Need to fix rspec-activerecord-expectations first.
https://github.com/drapergem/draper/actions/runs/11792755485/job/32846939180?pr=938#step:4:105

minitest-rails doesn't support Rails 8.0 yet. Ref: https://github.com/minitest/minitest-rails/pull/258


Add PLEX_UID and PLEX_GID to support cross-fs (NFS) environments.

Is this ever gonna get merged?

+1

Hey folks. I just finished a move, and am in the process of standing my homelab back up to play with this some more. I'll see if I can't carve out some time to fix this in the near future, and find a way to get the travis checks to pass and get this merged.


Setting up the script gives a error.

When I run the script and get to the 'Choose if you want daily reboot section' it gives a error
Setting server file permissions...
tail: cannot open '' for reading: No such file or directory
Then:
Setup is complete.  Starting Minecraft server...
Must be connected to a terminal.
Must be connected to a terminal.
If I continue after this and run the start.sh script it doesn't turn the server on.
Is there any way I could fix that?
Thanks Squawkykaka

I'd need to see more.  I can fix that tail error but it doesn't tell us what is going wrong.  What do you get when you run start.sh?

This is what it says:
pi@raspberrypi:~/minecraft $ ./start.sh
Updating to most recent paperclip version ...
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   183  100   183    0     0    855      0 --:--:-- --:--:-- --:--:--   859
Latest paperclip build found: 138
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100 36.4M  100 36.4M    0     0  21.9M      0  0:00:01  0:00:01 --:--:-- 22.0M
Starting Minecraft server.  To view window type screen -r minecraft.
To minimize the window and let the server run in the background, press Ctrl+A then Ctrl+D
pi@raspberrypi:~/minecraft $ screen -r minecraft
There is no screen to be resumed matching minecraft.

No worries, everything looks in order there.  We probably need to look at the log files which are in the "logs" directory.  latest.log or one of the later date ones should tell us what it is here!

Yeah that definitely does tend to happen.  I'm sure the logs revealed the problem here but if not let me know and I can help further!

this is the latest.log file everything seems in check but i cant be sure:
[16:58:39] [ServerMain/INFO]: Building unoptimized datafixer
[16:58:44] [ServerMain/INFO]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD'
[16:58:44] [ServerMain/INFO]: Found new data pack file/bukkit, loading it automatically
[16:58:55] [ServerMain/INFO]: Loaded 7 recipes

I di update to a 64 Bit machine but the same error occurs:
Automatically reboot Pi and update server at 4am daily (y/n)?y
no crontab for pi
Daily reboot scheduled.  To change time or remove automatic reboot type crontab -e
Setting server file permissions...
Setup is complete.  Starting Minecraft server...
Must be connected to a terminal.
Must be connected to a terminal.

Oh apologies it seems to be running with the error still occurring?

No worries at all!  It should be running yes.  You can reattach with screen -r and it should pull up the server console for you!


Create Nth Fibonacci Term.php

As Required, i've made the Fibonacci algorithm ♾ in PHP programming language mentioned in the issue no. 50. Kindly merge my PR if you believe that it completes your requirement of the Fibonacci Algorithm.
Link to the issue : https://github.com/DevF3st/Learn-Programming/issues/50

hey @paradocx96 kindly check my PR out...


Use bambi for MMMs

Bambi + MMM? Interesting... Bambi + CLV would be ... heaven.

For CLV, I assume this is only relevant for models that allow predictors?

Exactly. Would love to be able to add predictors or allow for a correlation between the purchase and attrition process.

Okay that's a very good thing to keep in mind when we implement such CLV models!

Brainstorming how this might look
y ~ 1 + menten(geometric(Google + Facebook, max_lag=8))


DARK MODE TOGGLE NOT WORKING ON Login Page

On all other pages dark mode toogle works fine.

=========FIXED=======
OPENED NEW ISSUE


Enhancement Add a switch for users to force using alias name in group by and having clause

Proposed changes
Issue Number: close #15747
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.59 seconds
load time: 492 seconds
storage size:<PHONE_NUMBER>3 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230109152006_clickbench_pr_76290.html

add a regression test

Please update variables.md doc

Update docs and add regression test


Updated to 2017 dispensing fee guidelines"

I've added the latest 2017 dispensing fee guidelines as per http://www.gov.za/sites/www.gov.za/files/40577_gon48.pdf

Hi Adi
I hope you are well? I have been in contact with you a while ago and have been closely following your your mpr project for a while now (great initiative by the way). I am a pharmacist but am passionate about data administration and the the huge potential, especially in the medical field. I am currently working on a free medical professional authentication service (Oauth 1.0a). Essentially it is a free API which could be used by all current software developers and new players to design feature rich dispensing and prescribing software and web applications.
This is really the tip of the iceberg of iceberg in terms of what the system could potentially offer . Using the code for SA mpr api together with my api will allow front end developers to create exciting new medical applications and enhance existing software. I've taken a look at your site and I think we share a similar philosophy in terms of social enrichment. I want the core authentication and data repositories to remain free forever, for everyone.
I will be attending the AWS conference in Cape Town on Jul 5. I'm not sure if you will be attending as well? I thought it might be a good opportunity to get together and Network a bit.
(PS I have switched to a postgres - flask - gunicorn stack for my API's and I don't miss php or node express at all)
Kind Regards
Lance Moore
(Digital Solutions Expert)
Reg: 2017/140271/07
Tel: +27 (0) 728416306
---- On Mon, 30 Jan 2017 12:51:17 +0200 Adi Eyal<EMAIL_ADDRESS>wrote ----
@adieyal approved this pull request.
Great - thanks for the pull request
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Incorrect permissions for .config

MacOS 10.12.6:
Kap Version:2.0.0
Install it in Applications and run
Current behaviour: error message (see below)
Expected behaviour: a working program
Workaround: no idea

Try running sudo chown -R $USER:$GROUP ~/.config @kasperkamperman

That did the trick, however you can't expect from an average user to mess around in Terminal I think.

Absolutely agree @kasperkamperman. We'll keep this issue open and look into providing a better experience for the next user with this problem 🙏


GCC: aggressive estimated bitrate drop on seemingly healthy connection

Your environment.

Version:  a82b843
Browser: Firefox 103.0 can reproduce this issue with some consistency.

What did you do?
I am having Firefox subscribe to a simulcasted video stream originally coming from Chrome through a pion SFU.
What did you expect?
The estimated bitrate not to change so dramatically.
What happened?
The estimated bitrate sharply decreased.
 Of course I am also happy to provide further information if necessary.
I am not suggesting there's a bug or anything but I'd like to understand a bit better what's going on to potentially mitigate this behaviour if possible.

The delay-based estimator decreases to 0.85 * lastReceivedRate, which in this case looks like 0.85 * 537485 = 456862.25. Do you use a minimum rate of 500000? The logs look like the estimated bitrate is much higher than the actual rate at which the application outputs media. That should be fine if the actual rate is below the bandwidth limit. But when the rate approaches the link capacity, we want to reduce it to something close to the last known maximum instead of reducing the last estimate, which can be much higher than the link capacity.
I don't know why the received rate estimation only slowly ramps up in your logs even when the initial rate estimation is much higher. Can you confirm the rates coming from the browser are actually that low in the beginning? The delivery rate estimation is currently not very precise, but it should be somewhat close to what is acknowledged by the receiver.

Thanks @mengelbart, that's very insightful.
This is how I am configuring the estimator at the higher level:
congestionController, err := cc.NewInterceptor(func() (cc.BandwidthEstimator, error) {
    return gcc.NewSendSideBWE(gcc.SendSideBWEInitialBitrate(1_000_000), gcc.SendSideBWEMinBitrate(500_000), gcc.SendSideBWEMaxBitrate(10_000_000))
})

The source rate is configured with the following simulcasted tracks:

0.5Mb/s (lower quality)
1Mb/s (medium quality)
2Mb/s (higher quality)

I'd fully expect the incoming stream's bitrate to fluctuate quite a bit given the source of media is screen share and when it goes to static content it's easy to get very low rates. But maybe I am misinterpreting the output of this whole estimation process somehow. As a newcomer to the field, I'd expect to get an approximation on the client's available down-rate, no matter what the current bitrate is, although I understand it's something tricky to guess precisely.
Here's logs from another reproduction attempt (I can provide logs from the beginning if it helps, just let know):
delaystats: {343.157µs -1.07712ms 18.5ms 16ms normal increase 10000000}
delaystats: {-283.647µs -1.572ms 18.5ms 95ms normal increase 10000000}
delaystats: {-298.066µs -2.0781ms 18.5ms 19ms normal increase 10000000}
loss controller increasing; averageLoss: 0, decreaseLoss: 0, increaseLoss: 0
rateController.onReceivedRate: 1072762
rateController.onReceivedRate: 1080212
rateController.onReceivedRate: 1103171
rateController.onReceivedRate: 949658
rateController.onReceivedRate: 953296
delaystats: {768.476µs -578.46µs 18.5ms 31ms normal increase 10000000}
rateController.onReceivedRate: 935008
rateController.onReceivedRate: 953840
rateController.onReceivedRate: 939309
rateController.onReceivedRate: 866736
rateController.onReceivedRate: 885682
rateController.onReceivedRate: 882388
rateController.onReceivedRate: 845418
rateController.onReceivedRate: 809747
delaystats: {5.015546ms 8.8062ms 18.5ms 85ms normal increase 10000000}
rateController.onReceivedRate: 828617
rateController.onReceivedRate: 834338
delaystats: {4.889535ms 17.66232ms 18.5ms 10ms normal increase 10000000}
delaystats: {13.557862ms 42.4323ms 18.5ms 33ms normal increase 10000000}
rateController.onReceivedRate: 773424
rateController.onReceivedRate: 757010
rateController.onReceivedRate: 775835
delaystats: {2.613177ms 45.99006ms 18.5ms 13ms overuse decrease 659459}
SendSideBWE.onDelayUpdate: bitrate: 659459, lossStats.TargetBitrate: 659459, delayStats.TargetBitrate: 659459

This is from the source (Chrome browser producing the stream):

In this log you can see how the estimator detects overuse and then reduces the estimation to 0.85 * receivedRate. I assume the beginning of the log shows a ramp up from the initial rate (1_000_000) to the maximum rate (10_000_000)?

As a newcomer to the field, I'd expect to get an approximation on the client's available down-rate, no matter what the current bitrate is, although I understand it's something tricky to guess precisely.

That is the goal, but when the last estimation is higher than the rate at which data can be sent, we cannot guess the link capacity. The way this estimation works is by trying to increase the sending rate until the interarrival delay between two groups of packets as measured by the receiver increases, which indicates that we are approaching the bandwidth limit. When the application does not produce enough data to probe for higher bandwidths, we will always measure the same interarrival delays and cannot find the bandwidth limit. The actual problem in these logs seems to be that the estimation increases to the maximum, even though the actual delivery rate is application limited at a much lower rate. There are some optimizations possible, but they are currently not implemented in Pion:

To avoid very high target bitrates which are far from what is delivered, we could detect if the rate is application limited and stop increasing the target rate.
We could probe for higher rates using padding packets to fill the gap between the rate emitted by the application and the target rate estimation to probe for higher bandwidths.
There's a somewhat long delay between the time the delay-based estimator first detects a growing interarrival time and the time at which it decreases the rate. We might be able to reduce this in the future but need more tests and experimentation with different parameters first.

In this log you can see how the estimator detects overuse and then reduces the estimation to 0.85 * receivedRate. I assume the beginning of the log shows a ramp up from the initial rate (1_000_000) to the maximum rate (10_000_000)?

Yep, that's precisely what's happening.
I guess as first step I may try to set the max bitrate to the actual highest value I expect to send from the app level. Although that's not going to protect me from the rate suddenly going down for other reasons (e.g. encoder related).
It sounds like the probing solution you mention may be an interesting option but that raises a few more questions like how frequent should that happen to minimize bandwidth waste or even be detrimental and cause more congestion.
So overall I guess I just hit a potential limitation of the current implementation. Either way, your comments were very informative and helpful, thank you again @mengelbart :raised_hands:

You're welcome. More optimizations are on the roadmap but will take more time to implement.
I will close the issue but feel free to re-open it if necessary.


Add missing handler for removing font sizes in Themes

Fixes https://github.com/godotengine/godot/issues/74377.
Seems to be a regression from https://github.com/godotengine/godot/pull/62108, where this specific block was removed. I don't see any particular reason for that, and every other piece of code handling font sizes in the theme editor is intact. So probably just a silly mistake. cc @bruvzg though.

Thanks!

Cherry-picked for 4.0.1.


Abstract StateStore class methods do not need to be async

Expected Behaviour
Works as described.
Actual Behaviour
Works as described, with an extra Promise in the chain.
Reproduce Scenario (including but not limited to)
N/A
Steps to Reproduce
N/A
Platform and Version
N/A
Sample Code that illustrates the problem
N/A
Logs taken while reproducing problem
N/A

@moritzraho can you qualify?

@avoidwork , just chatted with @moritzraho as you guys were in touch yesterday. Your PR is welcome :)

I'm waiting for approval for inclusion in @adobe; once done I'll re-create the PR.

see #50


Should we go down the route of mocking the docker client to test this bit?
I've used mock before, mostly via its patch() context manager but it never felt "right" (always needed several mocks mocking each others and :-/). Maybe someone with experience of using it could create a minimal example of how we'd do this?

@betatim I don't think log messages need testing - or rather, I don't think repo2docker is at a testing place where adding tests for log messages is a useful user of our time.

@betatim I'm sorry, I thought you meant mocking the log object to do testing.
I don't entirely understand reference to docker client in this PR. Can you expand a little bit on what you meant?

Maybe meant for https://github.com/jupyter/repo2docker/pull/478#issuecomment-444942250? I think it's a great suggestion for that PR!


运行报错

执行  python local_infer.py
Please enter your sentence: (Press Enter twice to start processing)
输入 hello word.
报错
raise PipelineException(
transformers.pipelines.base.PipelineException: No mask_token ([MASK]) found on the input

python local_infer.py

I think this happens because I did not take in account chinese characters when I selected all text.
I will that this in to account in the future

Hi, thanks for your amazing app.
I have an issue. When  I input "hello world" in the cmd(win11), there are some errors:
Traceback (most recent call last):
File "C:\Users\XXX\Downloads\DetectGPT-main\DetectGPT-main\local_infer.py", line 24, in
model(sentence, 100, "v1.1")
File "C:\Users\XXX\Downloads\DetectGPT-main\DetectGPT-main\model.py", line 99, in call
return self.call_1_1(sentence, args[1])
File "C:\Users\XXX\Downloads\DetectGPT-main\DetectGPT-main\model.py", line 264, in call_1_1
mean_score = sum(scores)/len(scores)
ZeroDivisionError: division by zero


Create certificate failed

Issue description
cannot create certificates on 2 of our exchange servers
Steps to reproduce
letsencrypt.exe --plugin manual --manualhost olvevere.be,mail.olvevere.be,autodiscover.olvevere.be --validation selfhosting --centralsslstore d:\software\Central_SSL\
Client version:
win-acme.v<IP_ADDRESS>
Windows version:
Server 2012 standard
Relevant part of log file:
[EROR] Error authorizing [Manual] [3 bindings - olvevere.be, ...]
[DBUG] DependencyResolutionException: DependencyResolutionException {Message="A
n error occurred during the activation of a particular registration. See the inn
er exception for details. Registration: Autofac.Core.Registration.ComponentRegis
trationLifetimeDecorator ---> An exception was thrown while invoking the constru
ctor 'Void .ctor(PKISharp.WACS.Services.IInputService, PKISharp.WACS.Services.IO
ptionsService, PKISharp.WACS.Services.ILogService, PKISharp.WACS.SettingsService
, PKISharp.WACS.Services.ProxyService)' on type 'AcmeClientWrapper'. ---> The un
derlying connection was closed: Could not establish trust relationship for the S
SL/TLS secure channel. (See inner exception for details.) (See inner exception f
or details.)", Data=[], InnerException=DependencyResolutionException {Message="A
n exception was thrown while invoking the constructor 'Void .ctor(PKISharp.WACS.
Services.IInputService, PKISharp.WACS.Services.IOptionsService, PKISharp.WACS.Se
rvices.ILogService, PKISharp.WACS.SettingsService, PKISharp.WACS.Services.ProxyS
ervice)' on type 'AcmeClientWrapper'. ---> The underlying connection was closed:
Could not establish trust relationship for the SSL/TLS secure channel. (See inn
er exception for details.)", Data=[], InnerException=WebException {Status=TrustF
ailure, Response=null, Message="The underlying connection was closed: Could not
establish trust relationship for the SSL/TLS secure channel.", Data=[], InnerExc
eption=AuthenticationException {Message="The remote certificate is invalid accor
ding to the validation procedure.", Data=[], InnerException=null, TargetSite=Voi
d StartSendAuthResetSignal(System.Net.Security.ProtocolToken, System.Net.AsyncPr
otocolRequest, System.Exception), StackTrace="   at System.Net.Security.SslState
.StartSendAuthResetSignal(ProtocolToken message, AsyncProtocolRequest asyncReque
st, Exception exception)\r\n   at System.Net.Security.SslState.CheckCompletionBe
foreNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)\r\n
at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count,
AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Security.SslState.StartRe
ceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.
Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncP
rotocolRequest asyncRequest)\r\n   at System.Net.Security.SslState.ProcessReceiv
edBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)\r\n   at S
ystem.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest
asyncRequest)\r\n   at System.Net.Security.SslState.CheckCompletionBeforeNextRe
ceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)\r\n   at System.
Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProto
colRequest asyncRequest)\r\n   at System.Net.Security.SslState.StartReceiveBlob(
Byte[] buffer, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Security.S
slState.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolReq
uest asyncRequest)\r\n   at System.Net.Security.SslState.ProcessReceivedBlob(Byt
e[] buffer, Int32 count, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.
Security.SslState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequ
est)\r\n   at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(Prot
ocolToken message, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Securi
ty.SslState.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest
asyncRequest)\r\n   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buf
fer, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Security.SslState.Ch
eckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest async
Request)\r\n   at System.Net.Security.SslState.ProcessReceivedBlob(Byte[] buffer
, Int32 count, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Security.S
slState.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest)\r\n
at System.Net.Security.SslState.CheckCompletionBeforeNextReceive(ProtocolToken
message, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Security.SslStat
e.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequ
est)\r\n   at System.Net.Security.SslState.StartReceiveBlob(Byte[] buffer, Async
ProtocolRequest asyncRequest)\r\n   at System.Net.Security.SslState.CheckComplet
ionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest)\r
\n   at System.Net.Security.SslState.ForceAuthentication(Boolean receiveFirst, B
yte[] buffer, AsyncProtocolRequest asyncRequest)\r\n   at System.Net.Security.Ss
lState.ProcessAuthentication(LazyAsyncResult lazyResult)\r\n   at System.Threadi
ng.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallba
ck callback, Object state, Boolean preserveSyncCtx)\r\n   at System.Threading.Ex
ecutionContext.Run(ExecutionContext executionContext, ContextCallback callback,
Object state, Boolean preserveSyncCtx)\r\n   at System.Threading.ExecutionContex
t.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
\r\n   at System.Net.TlsStream.ProcessAuthentication(LazyAsyncResult result)\r\n
at System.Net.TlsStream.Write(Byte[] buffer, Int32 offset, Int32 size)\r\n
at System.Net.ConnectStream.WriteHeaders(Boolean async)", HelpLink=null, Source=
"System", HResult=-2146233087}, TargetSite=System.Net.WebResponse GetResponse(),
StackTrace="   at System.Net.HttpWebRequest.GetResponse()\r\n   at ACMESharp.Ac
meClient.RequestHttpGet(Uri uri)\r\n   at ACMESharp.AcmeClient.Init()\r\n   at P
KISharp.WACS.Clients.AcmeClientWrapper.ConfigureAcmeClient()\r\n   at PKISharp.W
ACS.Clients.AcmeClientWrapper..ctor(IInputService inputService, IOptionsService
optionsService, ILogService log, SettingsService settings, ProxyService proxy)\r
\n   at lambda_method(Closure , Object[] )\r\n   at Autofac.Core.Activators.Refl
ection.ConstructorParameterBinding.Instantiate()", HelpLink=null, Source="System
", HResult=-2146233079}, TargetSite=System.Object Instantiate(), StackTrace="
at Autofac.Core.Activators.Reflection.ConstructorParameterBinding.Instantiate()
r\n   at Autofac.Core.Activators.Reflection.ReflectionActivator.ActivateInstance
(IComponentContext context, IEnumerable1 parameters)\r\n   at Autofac.Core.Reso lving.InstanceLookup.Activate(IEnumerable1 parameters)", HelpLink=null, Source=
"Autofac", HResult=-2146233088}, TargetSite=System.Object Activate(System.Collec
tions.Generic.IEnumerable1[Autofac.Core.Parameter]), StackTrace="   at Autofac. Core.Resolving.InstanceLookup.Activate(IEnumerable1 parameters)\r\n   at Autofa
c.Core.Lifetime.LifetimeScope.GetOrCreateAndShare(Guid id, Func1 creator)\r\n  at Autofac.Core.Resolving.InstanceLookup.Execute()\r\n   at Autofac.Core.Resolv ing.ResolveOperation.GetOrCreateInstance(ISharingLifetimeScope currentOperationS cope, IComponentRegistration registration, IEnumerable1 parameters)\r\n   at Au
tofac.Core.Resolving.ResolveOperation.Execute(IComponentRegistration registratio
n, IEnumerable1 parameters)\r\n   at Autofac.ResolutionExtensions.TryResolveSer vice(IComponentContext context, Service service, IEnumerable1 parameters, Objec
t& instance)\r\n   at Autofac.ResolutionExtensions.ResolveService(IComponentCont
ext context, Service service, IEnumerable1 parameters)\r\n   at Autofac.Resolut ionExtensions.Resolve[TService](IComponentContext context, IEnumerable1 paramet
ers)\r\n   at PKISharp.WACS.Program.Authorize(ILifetimeScope renewalScope, Targe
t target)", HelpLink=null, Source="Autofac", HResult=-2146233088}
[EROR] DependencyResolutionException: An error occurred during the activation o
f a particular registration. See the inner exception for details. Registration:
Autofac.Core.Registration.ComponentRegistrationLifetimeDecorator ---> An excepti
on was thrown while invoking the constructor 'Void .ctor(PKISharp.WACS.Services.
IInputService, PKISharp.WACS.Services.IOptionsService, PKISharp.WACS.Services.IL
ogService, PKISharp.WACS.SettingsService, PKISharp.WACS.Services.ProxyService)'
on type 'AcmeClientWrapper'. ---> The underlying connection was closed: Could no
t establish trust relationship for the SSL/TLS secure channel. (See inner except
ion for details.) (See inner exception for details.)
[EROR] Create certificate failed

You probably went a bit too far tuning the TLS settings on your machine.
Could not establish trust relationship for the SSL/TLS secure channel. means that you don't have any combination of algorithms in common with https://acme-v01.api.letsencrypt.org/. Try allowing more until you can access that page from the browser on the server and then the program should work too.


Support READONLY to allow exporting data from replica nodes

Currently, if you run riot in non-cluster mode, and configure it to point to a replica node URI, you will get MOVED responses and the export/replication will fail.
However, it would be useful to be able to export/replicate data directly from replicas so as to avoid putting additional load on the primary nodes.  This would theoretically also allow greater throughput because we could run a separate parallel riot process for each shard in the cluster.
I could see this working in one of two ways:

a --read-only flag (or similar) that would tell riot to send a READONLY command before any scan/read operations; then, the source URI could point to a replica node without fear of MOVED responses.  In this case, it would be up to the caller of riot to understand the cluster topology and pass in the correct replica URIs, decide whether to run in parallel or in sequence, etc.
abstract this behavior behind some additional flag(s) when running in -c mode, such that riot itself could handle any potential parallelization (perhaps with tuning, e.g. --replica-read-threads N).  Conceptually, this would mean the caller wouldn't need to know the details of the cluster topology when invoking riot, but this is probably a much more complex change to implement.

Looking through the output for file-export --help, I see the --read-from REPLICA option.  This covers the ability to read from replicas to keep load off the primary nodes, but doesn't help with parallelization across shards.


GetOutput trait does not work on IResult<&[u8], &str>

After including use nom::GetOutput in my code I keep getting the following compile error when I try to call .output() on an IResult<&[u8], &str>:
no method named output found for type nom::internal::IResult<&[u8], &str> in the current scope
I tried to fix it myself to send a pull request but I think I don't know enough about rust and traits yet to fix it. =/

Fixed in 58b22d7

@aphistic I'm curious if 1.1.0 fixes the problem for you? I'm having the same issue in my parser here (using 1.1.0):
https://github.com/pjungwir/pjchess/blob/nom-error/src/board.rs#L579
Here is the error for me:
src/board.rs:579:58: 579:66 error: no method named `output` found for type `nom::internal::IResult<&[u8], board::Board>` in the current scope
src/board.rs:579     let b = fen(&b"1k6/8/8/8/4Q3/8/8/1K6 w - - 0 1"[..]).output().expect();

@pjungwir I would have to try it again.  I rewrote my code to work around that so I would have to double back and change it.

Okay thanks for chiming in! @Geal any chance you can follow my link and see if I'm doing something wrong or if it's a problem with nom? I am saying use nom::GetOutput but the method still isn't being found. This is my first rust project, but I think I'm doing everything right here. At least there isn't much code to look at. :-)

Btw feel free to add my project (a parser for chess FEN notation) to the list of example projects. It is text-based and very simple, so I think it'd be a great example for someone to learn from.
And if you have any suggestions to make the nom stuff more idiomatic, let me know! :-)


Move "default_units" to top-level yaml file

Often need to check and/or extend the "default_units" dictionary. Could we move this to a seperate YAML file and potentially store it somewhere easier to find?

Completed


Addition of book. (Sr. No 8)

Book at sr. No 8. has been added, which includes contents of the syllabus.

Book at sr. No 8 has been added, which included the syllabus content.


[r3-corda-ent] enable DLT deployment via GitHub Workflow & Action

Commit to be reviewed

feat(r3-corda-ent): enable DLT deployment via GitHub Workflow & Action
Changes:

Introduced a new GitHub Workflow enabling the deployment of Hyperledger Bevel's R3-CORDA-ENT DLT Platform to an EKS Cluster.
Introduced a new directory at path: platforms/r3-corda-ent/configuration/samples/workflow, containing two new sample network configuration files:

network-proxy-cordaent.yaml: Supports deployment with Ingress controller.
network-no-proxy-cordaent.yaml: Supports deployment without an Ingress controller, specifically designed for deployment on Minikube.

Additional Changes:

Resolved duplicacy in code related to ServiceAccount and Cluster-role-binding.
Replaced the Ambassador Ansible task with new Ambassador-Edge-Stack.

fixes #2416

@saurabhkumarkardam why do we have certain tasks removed from the roles? Are they handeled by other roles/charts?

Yes, we are achieving the same thing with the bevel-vault-mgmt chart available at the path platforms/shared/charts/bevel-vault-mgmt/templates/serviceAccount.yaml.


[GPU] Clustered Subgroup Reduction

Request description
Motivation
A pattern we notice in flash attention kernels is:
A: tensor<16x16xf16>
B: tensor<16x16xf16>
C: tensor<16x16xf16>

D : tensor<16x16xf16> = matmul(A, B, C)
E : tensor<16x1xf16>  = reduce(D, dim=1)
F : tensor<16x16xf16> = broadcast(E, dim=1)

When optimizing for matmul intrinsics, for performance, we
prefer to keep the computation here entierly in registers. So the distribution
of data on threads cannot change. We primarily optimize for matmul intrinsics
on GPUs, so the thread distribution follows that.
Accordingly, the thread distribution for tensor D follows that of output of a
matmul intrinsic. An example of such a thread distribution for a 16x16 shape is
each thread carrying a vector<1x4xf16>, distributed over a 16x4 thread grid.
The data can be though to be distributed on threads as (here the numbers
represent thread ids, and the matrix over which they are distributed is the
tensor D):
[0 0 0 0 1 1 1 1 2 2 2 2 3 3 3 3]
[4 4 4 4 5 5 5 5 6 6 6 6 7 7 7 7]
[8 8 8 8 9 9 9 9 ...            ]
.
.
.
[]

When reducing this along dimension = 1, we want to be doing multiple reductions
in parallel in "clusters" of 4 threads. (An element carried by multiple thread
ids is represented here by commas)
[0 0 0 0 1 1 1 1 2 2 2 2 3 3 3 3] --> [0, 1, 2, 3]
[4 4 4 4 5 5 5 5 6 6 6 6 7 7 7 7] --> [4, 5, 6, 7]
...

We will call such a reduction, a "clustered" reduction.
How we do it today
Today, we directly emit a bunch of gpu.shuffle ops and do the entire reduction
lowering in one shot:
https://github.com/iree-org/iree/blob/main/compiler/src/iree/compiler/Codegen/Common/GPU/GPUNestedLayoutDistributionPatterns.cpp#L402
Better lowering for clustered reductions
There is an existing operation in MLIR's gpu dialect, which can represent
a reduction across threads in a subgroup:
https://mlir.llvm.org/docs/Dialects/GPU/#gpusubgroup_reduce-gpusubgroupreduceop
The limitation of this operation is that it uses all available threads in a
subgroup to do the reduction, which means we cannot do a clustered reduction.
We would like to add support to this operation to do such clustered reductions.
Tasks

Add a new "cluster_size" attribute on the gpu.subgroup_reduce operation
Update the existing gpu.subgroup_reduce lowering to emit clustered subgroup
reductions.
Update the vector distribution pattern to use clustered reductions.

Useful Links
Current subgroup_reduce lowering: https://github.com/llvm/llvm-project/blob/main/mlir/lib/Dialect/GPU/Transforms/SubgroupReduceLowering.cpp
GPU Dialect documentation: https://mlir.llvm.org/docs/Dialects/GPU/#gpusubgroup_reduce-gpusubgroupreduceop
IR definition for gpu.subgroup_reduce : https://github.com/llvm/llvm-project/blob/main/mlir/include/mlir/Dialect/GPU/IR/GPUOps.td#L1196
What component(s) does this issue relate to?
MLIR, Compiler
Additional context
No response

I'm thinking of adding this part to the description of gpu.subgroup_reduce:

If a cluster_size is provided, the subgroup is divided into clusters of cluster_size lanes each, a reduction is done for all lanes of each cluster (in parallel), and the result is equal for all lanes in a cluster.

The operation of this is kind of difficult to succinctly explain. Maybe I would need to specify the particular way the clusters are laid out across the threads?

I'm thinking of adding this part to the description of gpu.subgroup_reduce:

If a cluster_size is provided, the subgroup is divided into clusters of cluster_size lanes each, a reduction is done for all lanes of each cluster (in parallel), and the result is equal for all lanes in a cluster.

The operation of this is kind of difficult to succinctly explain. Maybe I would need to specify the particular way the clusters are laid out across the threads?

I think the description sounds good for now. You can just mention that the clustered are divided in a contiguous manner.

I think I have a finished implementation now: https://github.com/llvm/llvm-project/pull/104851

When I got to trying to update GPUNestedLayoutDistributionPatterns.cpp, me and @Groverkss realised that being able to specify cluster sizes isn't enough, we also need to be able to specify cluster strides (so, non-contiguous clusters). So I guess we will have to add a second attribute to MLIR. I have a good idea for what this should look like so it shouldn't be a problem, just a shame this was missed earlier.

I made a patch to add "cluster strides" to MLIR (https://github.com/llvm/llvm-project/pull/107142), and I've successfully prototyped using this in GPUNestedLayoutDistributionPatterns.cpp downstream; I was able to get the existing test to pass unmodified when I hacked IREE to apply the upstream expansion. So I think this is going well. I've also made https://github.com/llvm/llvm-project/pull/107134 as I'll need that downstream.

Second round of MLIR changes was in https://github.com/llvm/llvm-project/pull/107142, now merged.
Using this in IREE will require a few different commits:

https://github.com/llvm/llvm-project/pull/107134 or https://github.com/iree-org/iree/pull/18443
To be continued

Main PR is up at https://github.com/iree-org/iree/pull/18515 and I made an issue for tracking potential follow-up cleanup work (https://github.com/iree-org/iree/issues/18516); I realised I shouldn't block this issue on it.

Third round of MLIR changes, to fix a problem encountered in https://github.com/iree-org/iree/pull/18515 (difficult to use the upstream patterns in the pipeline optimally because they're too general):

https://github.com/llvm/llvm-project/pull/109015
https://github.com/llvm/llvm-project/pull/109158


Migration lib logging

The logger provided to the create workflow function should be passed to the custom interpreter so it may use it instead of console. This will provide the same formatting, span, and log level support as the logger in the workflow function.

#21


refactoring documentation

Signed-off-by: Sally O'Malley<EMAIL_ADDRESS>Updating documentation file structure based on discussions.

/approve


Search memory leak

On nextra based sites you can write in search > space space and any letter and fast memory leak happens.
Affected only production builds. (Safari, Chrome based browsers, Mac)
https://github.com/shuding/nextra/assets/94939/63df3d3e-c2d7-40ad-b2fc-8a84d0d1ff67

Just some additional info about the problem:

Any input (as long its not empty) followed by two spaces and a single character will leak
You can also reproduce the same behaviour on development environment by building the project, and then running in development mode (because it maps all available contents in .next/static/chunks/pages)

https://github.com/shuding/nextra/blob/fix-memory-leak/packages/nextra-theme-docs/src/components/highlight-matches.tsx#L27-L29


Refactor CpanParser

There'll be breaking changes with newest CpanParser, which may break looking up Pods and their submodules.
TODO:

fix tests
add lookup method for modules;
do tests runs with real projects on github;

Check also version selectors.
Specifically check that the case "1.0.0" is handled as ">= 1.0.0", source: https://github.com/kraih/mojo/wiki/Installation-of-cpan-modules-by-cpanm-and-cpanfile

@timgluz That's the status on this? Is a PR for this ticket coming?

Isn't that already merged.? It addee product.fetch_cpan method

Yes. It's added.


JFXDIALOG :CLOSED event does not handle the code

I'm trying to handle differents methods when the JFXDialog is closing but ,CLOSE event type does not give me any response :
This is my code :
SharedRegister.getDialog().addEventHandler(JFXDialogEvent.CLOSED, eventHandler -> {
infoPopOver.hide();
System.out.println("closed");
});
SharedRegister.dialog.addEventHandler(JFXDialogEvent.CLOSED, eventHandler -> {
warningPopOver.hide();
System.out.println("closed");
});
SharedRegister.dialog.addEventHandler(JFXDialogEvent.CLOSED, eventHandler -> {
errorPopOver.hide();
System.out.println("closed");
});

Try using the onDialogClosedProperty instead. It defines a function that will be called whenever the dialog has been closed and will fire only after the close animation has finished. I believe it should give you the behavior you're looking for.
dialog.setOnDialogClosed(closeEvent -> {
     System.out.println("I am closed!");
});

@shadskii ,the problem I did not find onDialogClosedProperty for adding many listeners in different locations, dialog.setOnDialogClosed(closeEvent -> {      System.out.println("I am closed!"); }); With setOnDialogClosed() ,it is a methods allows to add only one event ,I mean if I want to change the event I need to remove the old and adding the new ,but it is not a good idea ,I'm looking for onClosedProperty() but it seems ,it is not available for Jar 8.1

Ah I understand your problem now. JFXDialog's onDialogClosedProperty does not exactly support what you are trying to do but you can achieve the desired behavior by using a BooleanProperty to represent the state of the dialog and adding listeners to it instead.
BooleanProperty dialogState = new SimpleBooleanProperty(false);
dialog.setOnDialogOpened(openEvent -> dialogState.set(true));
dialog.setOnDialogClosed(closeEvent -> dialogState.set(false));

dialogState.addListener((observable, oldValue, newValue) ->  {
     if(newValue){
          System.out.println("Dialog has been opened!");
     }else{
          System.out.println("Dialog has been closed!");
     }
});

Hope this helps!
Cheers

@XlintXms thanks for pointing out the issue 👍 , will update JFXDialog and push the fix. so it will be included in the next release.
Regards,


Update unclosed-quotation-mark.md

Pull request guidance
Thank you for submitting your contribution to our support content! Our team works closely with subject matter experts in CSS and PMs in the product group to review all content requests to ensure technical accuracy and the best customer experience. This process can sometimes take one or more days, so we greatly appreciate your patience.
We also need your help in order to process your request as soon as possible:

We won't act on your pull request (PR) until you type "#sign-off" in a new comment in your pull request (PR) to indicate that your changes are complete.

After you sign off in your PR, the article will be tech reviewed by the PM or SME if it has more than minor changes. Once the article is approved, it will undergo a final editing pass before being merged.

Confirmed with Terry via Teams chat. It's good for publishing.


Provider login/consent views do not read client_name correctly

The provider authz_request_view shows the client_organization_name to the user in the login page.  It reads the name from the resolved metadata, and falls back to client_id.
https://github.com/italia/spid-cie-oidc-django/blob/b7b28a8fbb59076dc8d7ba389383379282547315/spid_cie_oidc/provider/views/authz_request_view.py#L202-L204
Same goes for consent
https://github.com/italia/spid-cie-oidc-django/blob/b7b28a8fbb59076dc8d7ba389383379282547315/spid_cie_oidc/provider/views/consent_page_view.py#L59-L61
The client_name is always blank, because it is provided inside the openid_relying_party subsection.
something like this fixes the issue
        context = {
            "client_organization_name": tc.metadata.get(
                "openid_relying_party", {}).get("client_name", self.payload["client_id"]),

I'm also wondering whether displaying the client_name is the best approach here, given that we should have the proper organization_name inside the federation metadata, which is required for CIE/SPID RPs as per https://docs.italia.it/italia/spid/spid-cie-oidc-docs/it/versione-corrente/metadata_oidc_rp.html

it should be better to point to
metadata.federation_entity.organization_name
if any, or
metadata.openid_relying_party.organization_name
if any, or
metadata.openid_relying_party.client_name
if any, or
metadata.openid_relying_party.client_id
do you agree?
If yes and if you can, please provide a PR for this


chore(project): update dependencies

What does this PR do?
The package @babel/polyfill has been deprecated in favor of using core-js and regenerator-runtime directly, so this PRs removes the depenendency.
The replacement packages are not being installed on this repository because they'll be added by the build engines.
How should it be tested manually?
yarn test
# or
npm test

Pull Request Test Coverage Report for Build 278

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 276:
0.0%

Covered Lines:
2144

Relevant Lines:
2144

💛  - Coveralls


Add transaction ID to Cadence log entries

Issue To Be Solved
Let's say we have a transaction that uses the log() function to print something.
transaction {
  execute {
    log("Hello!")
  }
}

When that transaction is executed on the local emulator blockchain, the produced log outputs are displayed as emulator (Flow CLI) logs like so:
9:24PM INF LOG: "Hello!"
9:25PM INF LOG: "Hello!"

Problem: There is no way to tie a log entry to the transaction that emitted it. I believe this would be useful for many reasons:

users of Flow CLI can clearly see which logs were emitted within the same transaction (even if they can't remember the transaction ID)

this is especially useful in cases where each transaction emits multiple logs, which makes it harder to visually categorize them

developer tools using flow-emulator (e.g. Flowser, Flow playground,...) can provide a better UX

this would allow such tools to extract the transaction ID from each log and show it within the UI of the corresponding transaction

Suggest A Solution
We could solve this by including the transaction ID in each Cadence log entry, like so:
9:24PM INF transactionId=33d14437d6e5cb23b4d6de3b70ea754a815e2d4fac79ae85a29558da2b942aef LOG: "Hello!"
9:25PM INF transactionId=70d05c026c8c400dbd7a8627daffdd89cd3ca3d64c1fdf621275b8fd0f245833 LOG: "Hello!"

... or we could include only a subset ID to make it easier to view in the terminal:
9:24PM INF transactionId=33d14...422aef LOG: "Hello!"
9:25PM INF transactionId=70d05...45833 LOG: "Hello!"

Context
In the Flowser app, we have a logs view that shows all historic logs that ever happened (in the current emulator session). But when you execute a transaction in isolation, you usually only care about that transaction's logs and not all the historic ones.
This change would allow us to show currently executed transaction logs within that transaction's info view (related to this feature we just built: https://github.com/onflowser/flowser/pull/170).

I think this is a nice idea, can be implemented, but maybe we can put it behind a flag. txId is pretty long and takes a lot of space ( or maybe we can put it after the message ) what do you think @sideninja  ?

I would put it after the message. @chasefleming will be taking this on as his onboarding task.


API Language validation needs to pull from settings

i update my new language but it cant show how can i fix it ?

Hey @kurodenjiro sorry that we didn't see this problem coming. For the time being you'll need to update this validation line in the API. https://github.com/samtecspg/articulate/blob/master/api/modules/agent/config/agent.validator.js#L169
We'll work on making the validation pull from the settings.

@wrathagom  i updated . how to rebuild images by docker ? . it's stille like that

Try
docker-compose down
docker-compose -f docker-compose.yml -f build-compose-override.yml build api
docker-compose up


Close terminal pipe, when closing the pod panel

Moving deletePipe() in componentWillUnmount() ensures the pipe is deleted when:

the terminal is closed
the terminal is closed when closing the pod panel

The pipe still does not get deleted when popping out the terminal.
Fixes #3035.

when the popped out terminal is closed the pipe is not deleted

Is that a leak?

Is that a leak?

yes


[wip] implement cli cmd: node info

Info
This PR intends to implement the node info CMD to source a stored node config in the node state. More then likely a discussion is warranted since there are artifacts from other implementations to take into consideration here.
Considerations

We are not really using the NodeConfig in many places, we may want to start configuring the components should rely on.
Currently, the node ships with no config, and this implementation takes the RunOpts default values and writes them to the config file here: ./.vrrb/node/db/config.json. So a user must run the node to ensure that file gets written.
We may want to consider taking a step back and implementing a solution that solely runs on the node config and its implementing default values there. We can then iterate into the flag override that implements the RunOpts

stale but some may be worth looking back as I did have some ideas for how this all should work.


[Question]  /help (or any commands in discord) not working

Checks

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting.
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly.

Description
mcdiscordchat.txt
i put my .json file in .txt format to upload, not sure how else to do it as im new to github
Basically it wont let me use /help command in discord chat

Log?

You mean from the console itself? (if you mean console nothing shows up on console about me entering commands)

wait actually i was to fast, it does give me an error in the console: https://pastebin.com/raw/JLxUDc6Q

The errors u sent do not seems to be related to this issue. Did u follow the docs to (re-)setup the Discord Bot?

Yeah, i did, nothing happens in the console at all. It's only when i put these commands in the discord chat these error in the logs show up. Otherwise there's no errors showing up..
I'm kinda inexperienced and i was thinking maybe i did my webhook wrong(?) It's as simple as creating new webhook and copy url, and paste it into the .json webhook URL right?
Otherwise i genuinely have no idea why these errors show up when i input commands in Discord

i assumed it was the ID of the role i had to copy and put in there?

Yes

Any other ideas why it might not work?

Try execute /mcdc command in-game and from the console, see if there is error?

They both seem to work,
Also I'm OP and i wonder how come im lacking some 'level operator'? How do i assign this?

Also I'm OP and i wonder how come im lacking some 'level operator'? How do i assign this?

This is just a fixed help message send to everyone regardless of Op or not. So no need to care.

Then this should not be a problem with MCDC itself. You can try restart the server system, restart Discord and so on.

My server has been restarted multiple times, and so has my discord. Have also tried reloading using "/mcdc reload" but it still seems to be not working on discord.
Another issue what I'm having is that the server only announces when it's starting up, but not shutting down: (here in the image you see that it only says start, and before or after doesnt say stop,
(just as extra ive included also the part where it gives an error when trying to use a command through the bot incase you could pull some info out of it stil)

Yes that exception is what I need. You need at least one role to execute command.

omg that actually did it , thank you haha, giving myself a role seemed to fix that issue.
When i stop the server through the console , it doenst say that the server stopped though, any idea?

Exception?

No idea, it fixed itself now it seems im not sure how, I've got one more question though, when executing commands like "say hi" in through your bot through console, it doens't show up in console.
here as example i said hi, and also requested a list of backups that show up in console but not in the bot:

also i found out that most console logs just dont get logged through discord, is this intended?

For both questions, please reproduce the questions again and send the full log file (not an excerpt).

So the questions from your last two messages (https://github.com/Xujiayao/MCDiscordChat/issues/118#issuecomment-1426677194 & https://github.com/Xujiayao/MCDiscordChat/issues/118#issuecomment-1427116747) both are solved right?
If yes, this issue will be closed.

So the questions from your last two messages (https://github.com/Xujiayao/MCDiscordChat/issues/118#issuecomment-1426677194 & https://github.com/Xujiayao/MCDiscordChat/issues/118#issuecomment-1427116747) both are solved right?
If yes, this issue will be closed.

?

Yes this one has been solved, thank you

Nice, thanks 👍

You need at least one role to execute command.

Fixed in the latest release. A role is no longer required to execute Discord commands.


[feat-#1244][sybase] add sybase reader plugin

Purpose of this pull request

Which issue you fix
[Feature][sybase] add sybase reader plugin #1244
Checklist:

[ ] I have executed the 'mvn spotless:apply' command to format my code.
[ ] I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have checked my code and corrected any misspellings.
[ ] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

@PaddyMelody @FlechazoW please review


add aLRP-Loss

aLRP-Loss, accepted in NeurIPS2020, added to projects.md

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

Hi!
@cancam
First of all, we want to express our gratitude for your significant PR in the OpenMMLab project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


entity variant -> entity type

today on things no one but me cares about
notes:
in the registry, variant -> type refactor is done in #276

@EnnuiL mentioned in discord that there are actually more usages of variant that I missed in my initial enigma search, so I'm converting this to draft until I've done a more thorough search.

point to consider: FrogEntity uses getFrogType, which is a convention I followed instead of using getType everywhere. However, this led to things like getTropicalFishType, which is significantly longer than simply getVariant. Should we use getType, or getEntityNameType? I'm leaning towards the first.

I think variant for something that has a registry unless the registry types have behavior associated with them, and type for something that is just an enum.  I dont want to change BlockEntityType and EntityType, which is why i think the clause for registries is needed. If this could be added to the conventions file in this pr that would be great.

Also frogs should 100% be using variant. i just checked the code, and i messed up on that mapping

hm.
I'm thinking we use variant for something that's purely cosmetic, and type for anything with behaviour associated to it. As far as I know without checking, that would discard all this PR's content except for paintings, and would require frogs being refactored from type to variant.

I think so as well, but i also believe it makes better mappings and is more intuitive to new modders.

that way we have a simple explanation for BlockEntityType and EntityType keep their behaviour, as well as a very simple convention for deciding between the two. Since things like frogs and cats do have a registry, I think we should avoid the registry clause.

maybe something like:

Use Variant for cosmetic-only distinctions while withType is used when there is significant behavior associated with it.

looking into things further:
I was wrong about the registry bit not being needed.
The four entities that have variant registries, villagers, cats, frogs, and paintings, all have slightly different behaviour based on their variant. Frogs and cats have different spawning, villagers have their professions, and paintings change size.
With that in mind:

Use Variant for cosmetic-only distinctions while Type is used when there is significant any behavior associated with it. A general rule for this is to check if the different types are stored in a registry. If they are, you should be using Type over Variant.

setting this back to draft while we discuss

@OroArmor thoughts on that snippet as the convention?

Sounds good. I'm curious what @ByMartrixx and @LambdAurora think.

perfect!
I'm going to make the changes, and keep the pr as draft until their opinions are in.

mojang reworked entity variants; closing in favour of a new pull request


#904 dont fail in signup on gun issues

Description
Catching gun profile set exceptions
About #904
How Has This Been Tested?
Checklist:

[ ] PR title matches follow: (Feature|Bug|Chore) Task Name
[ ] My code follows the style guidelines of this project
[ ] I have followed all the instructions described in the initial task (check Definitions of Done)
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have added reference to a related issue in the repository
[ ] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[ ] @mentions of the person or team responsible for reviewing proposed changes

@AnastasiiaOdnoshevna this also needs to be tested in new release, though it is only part of the ticket


Can not find a file

Excuse me. I always got a tip  such as  FileNotFoundError : [ Error 2 ] File b ./poidata/Foursquare/sequence/Foursquare.txt  dose not  exist: b ./poidata/Foursquare/sequence/Foursquare.txt   recently when i running the code of prog_bpr_gru_spatial.py. I guess its maybe miss file that mentioned above.  I hope you can spend a few minutes browsing this question, I would greatil appreciate.

I think you probably miss the step to preprocess the dataset first.


Travis: Add mysql to the list of services [release/97]

Description
Travis is now in the process of changing the default Linux distribution it uses for builds from Ubuntu Trusty to Ubuntu Xenial. One of the differences between Travis Trusty and Xenial images is that the latter no longer implicitly start services such as the MySQL daemon. Update our travis configuration to explicitly start the MySQL service. See https://blog.travis-ci.com/2019-04-15-xenial-default-build-environment for details.
Use case
The vast majority of Infrastructure repositories both use MySQL and do not specify the distribution to use, meaning we might end up with our builds suddently going Xenial and being unable to connect to MySQL.
Benefits
Travis builds will hopefully continue to work once the default distro for our repositories has been changed from Trusty to Xenial.
Possible Drawbacks
None I can think of.
Testing
Have you added/modified unit tests to test the changes?
No, changes affect Travis only.
If so, do the tests pass/fail?
N/A
Have you run the entire test suite and no regression was detected?
No, changes affect Travis only. Travis builds for the merge branch continue to succeed.

Coverage remained the same at 81.444% when pulling a6ae7f553deaa2b8980e8c2e258fa92e0c422448 on travis_environment_rel97 into a4bee0de47a66f4899db76a544165790d3c5ebb8 on release/97.


Fix some doubled words in references

Signed-off-by: Misty Stanley-Jones<EMAIL_ADDRESS>

LGTM!  Thanks for fixing these @mstanleyjones!

jenkins, test this please


Update of the typer with shifts and assignments

Updates the Typer class to produce a result in the case of assignments
and the TyperHelper class to correctly type shift operations in the
case of left and right scalar types (the resulting type is the one
from the left operand).

Done!


Deeplinking (natively on iOS), passing the a promo code.

I am using Obj-C for my code. I want to set my UBSDKUpfrontFare to promotion for my UBSDKRideParametersBuilder instance. How do I do that ?

@edjiang Any updates on this?


add additional product ID to Aeotec Nano Shutter ZW141

Later firmware updates of the ZW141 have updated the product ID on the device - Needs to be reflected here for correct interview results.

Looks like we did have that device id in a different file (see error message). Can you verify that it is the same and merge them into one file?

Ok - looks like I stuffed up twice.

Aeon labs and aeotec - same company using two different manufacturer ID's. In the latest firmware update they changed the manufacturer ID. So I'll need to copy the file across to the right directory and clean up the Product ID's.
I used the product ID's for the Aeotec Zstick instead of the ZW 141 (facepalm).

I'll revert the file, do the copy etc and then resubmit the pull request.

Ok - looks like I stuffed up twice.

Aeon labs and aeotec - same company using two different manufacturer ID's. In the latest firmware update they changed the manufacturer ID. So I'll need to copy the file across to the right directory and clean up the Product ID's.
I used the product ID's for the Aeotec Zstick instead of the ZW 141 (facepalm).

I'll revert the file, do the copy etc and then resubmit the pull request.

I'm also learning how to use github and commits etc - Be gentle...

OK @AlCalzone - Let me know how this one goes...

Aeotec alerted us there would be a new manufactuerId and I already have a file for it, but a lot more changes than just adding the ID. They told us they did this because it was essentially a new product. Are you sure you installed the correct firmware? I assumed there was a hardware change the way it was described. Where did you get the fw from?

@Ledfoot775 thanks for the PR but after discussion with @blhoward2 we should fix this the correct way.


Eliminate unneeded overload

This overload incorrectly provides read/write access to an underlying type on the AutoPacket, and thus should be removed immediately without a deprecation schedule.

Oops.  This wasn't meant to be opened.


Add BQ External table connection param

Description & motivation
Add with connection to the template and params so it can be passed in for creating BL tables with a specified BQ connection
Checklist

[x] I have verified that these changes work locally
[ ] I have updated the README.md (if applicable)
[ ] I have added an integration test for my fix/feature (if applicable)

@cfyans I tried to run the integration tests for this PR but they failed with a very weird error. Did you manage to do the tests?

@cfyans I tried to run the integration tests for this PR but they failed with a very weird error. Did you manage to do the tests?

Think I've fixed. Works for us locally. How / where do I run the integration tests?

@jeremyyeo Can this be merged?

I like the #233 better yes, keeps the for loop of options clean.

closing in favor of #233


Leverage couldMatch in blockchain log listener

Adding an additional log listener overload in the blockchain that takes a logs query could let us filter out all the events in a block if none of the logs couldMatch on the logsBloomFilter in the event's block's header.

This has been stale for >1 year. Closing for now. Feel free to copy/paste in a new issue if relevant. cc: @sajz