Fix IsStochasticWithThresholdAndInititator, add basic tests

Pull Request Test Coverage Report for Build 794730738

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 73.192%

Totals

Change from base Build 791948910:
1.5%

Covered Lines:
2135

Relevant Lines:
2917

💛  - Coveralls


display “Error” in recent days

Discussed in https://github.com/nhaouari/obsidian-textgenerator-plugin/discussions/95

Originally posted by cgofun February 11, 2023

Same on PC and mobile phone. How to solve it？

I come up with same problem.. waiting for solution...

I found the solution. You have to register with a Credit Card, even if you're not going to exceed the $18 initial allowance. Once I did that (and waited about 15 minutes for the servers to update), I was able to use the plugin as described.

I found the solution. You have to register with a Credit Card, even if you're not going to exceed the $18 initial allowance. Once I did that (and waited about 15 minutes for the servers to update), I was able to use the plugin as described.

When I registered, I only used the mailbox and phone number. Do you mean to buy Plus?


Android NDK transition to GCC (deprecated) to Clang (recommended)

http://developer.android.com/ndk/downloads/revision_history.html
http://developer.android.com/ndk/guides/application_mk.html
http://developer.android.com/ndk/guides/cpp-support.html#runtimes
https://github.com/readium/readium-sdk/blob/develop/Platform/Android/Application.mk
NDK_TOOLCHAIN_VERSION := 4.9
APP_STL := gnustl_static
=>
NDK_TOOLCHAIN_VERSION := clang3.4
APP_STL := libstdc++

NDK revision 12:
https://developer.android.com/ndk/downloads/revision_history.html
"The ndk-build command will default to using Clang in an upcoming release. GCC will be removed in a later release."

See feature branch: https://github.com/readium/readium-sdk/tree/feature/gcc-to-clang
Code diff: https://github.com/readium/readium-sdk/compare/develop...feature/gcc-to-clang

Pull Request: https://github.com/readium/readium-sdk/pull/240


Bug fixes

🚀

:tada: This PR is included in version 9.8.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Context menus make webview on the side steal focus

Does this issue occur when all extensions are disabled?: Yes

Version: 1.90.0-insider
Commit: bbc4ba1eff648ee542972589216f211b7a6a08f2
Date: 2024-05-14T05:47:50.971Z
Electron: 29.3.1
ElectronBuildId: 9464424
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Linux x64 6.8.8-300.fc40.x86_64

This bug has also been confirmed to occur on 1.88 and 1.89, but not 1.87.
Steps to Reproduce:

Open an untitled document and a webview panel next to it (for example, a markdown document and its markdown preview)
Right click in the untitled document, in the side bar or click an entry with a menu in the title bar
Observe that the webview panel becomes focused and the title bar entries jump over to the webview panel

Context:
In the VS Code extension for the Lean 4 programming language and theorem prover, we use a webview to provide additional information about the Lean document that the user is currently working on. We also use a title bar menu entry for all of our commands. The webview stealing the focus from the text document creates several very annoying issues while working on Lean documents:

Right clicking an identifier in the Lean document and selecting "Go to Definition", "Find References" or "Show Call Hierarchy" will open documents in the tab group of the webview, not that of the editor, covering the webview and displaying the contents of the opened file in a panel that is typically smaller than the editor panel and probably too small to read code
Right clicking to paste something in the search sidebar view and then clicking a search result entry will also shift the focus and open the document in the wrong pane
When clicking the title bar menu entry, the entry suddenly shifts away underneath the cursor to the webview panel, which is very disorientating

Original report at leanprover/vscode-lean4#438.
Potentially related issue: #210108

@mjbvz I would like to try fixing this bug. I know this issue has not made to "Good first issue" list, but this affects my extension which uses webview. I feel like i can manage this. Is there any chance that i could collaborate on this issue? If yes, should i sign a Contributor License Agreement?


Problem with rotated image server and getAtlasPixelTransform

the roi is imported correctly, but the rotated image server is not taken into account when getting the pixel transformation

Done in https://github.com/BIOP/qupath-extension-abba/commit/6c201f698408380738eea6c8c763d4e5b160e309


Compiler complains passing TrayStorage.Type.USER

The following causes the compiler to complain; TrayStorage.Type.USER needs to be an int.
private class MyPreference extends TrayPreferences {
    public MyPreference(@NonNull final Context context) {
        super(context, "myModule", VERSION, TrayStorage.Type.USER);
    }
}

This is correct. The third Parameter has to be an int: The Version Code.
You didn't define it. Add
private static final int VERSION = 1;

to your class and you should be fine. You can also remove the last Parameter since its the standard behavior (TrayStorage.Type.USER).

Hey, thanks for the quick reply. So the issue is not the VERSION param. I'm just saying that the constructors 4th param should be an enum of type "TrayStorage.Type". Or the "TrayStorage.Type" should be int and not enums. You guys should possible properly define that unless I'm not understanding something... :P

I'm not sure if I understand you correctly. You can pass 4 Parameters into the Constructor:

A context (Type: Context)
The name of the module (Type: String)
The version code (Type: int)
(optional) A storage type (Type: TrayStorage.Type)

Your posted Code should work if you define VERSION as an int.

Ah, you are correct. I was incorrectly using the constructor override. I was trying to pass the TrayStorage.Type as an int. My apologies. :/

You're welcome 😄


[Currency Bot] Bumped fastify from 4.28.1 to 5.0.0

Tada!

https://github.com/fastify/fastify/releases/tag/v5.0.0

Fastify (533):	startup failure TypeError: Cannot create property 'host' on string '4329'
@instana/collector:     at Object.listen (/artifacts/node_modules/fastify/lib/server.js:60:26)
@instana/collector:     at start (/artifacts/packages/collector/test/tracing/frameworks/fastify/app.js:136:15)
@instana/collector:     at Object.<anonymous> (/artifacts/packages/collector/test/tracing/frameworks/fastify/app.js:144:1)
@instana/collector:     at Module._compile (node:internal/modules/cjs/loader:1469:14)
@instana/collector:     at Module._extensions..js (node:internal/modules/cjs/loader:1548:10)
@instana/collector:     at Module.load (node:internal/modules/cjs/loader:1288:32)
@instana/collector:     at Module._load (node:internal/modules/cjs/loader:1104:12)
@instana/collector:     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12)
@instana/collector:     at node:internal/main/run_main_module:28:49

I have identified:

feat: emit diagnostics_channel events upon routing request by @tlhunter in https://github.com/fastify/fastify/pull/5252

Extended https://jsw.ibm.com/browse/INSTA-804

v5 no longer supports Node < 20. https://github.com/fastify/fastify/pull/5481/files
It throws an error with diagnostic channel.

TypeError: diagnostics.tracingChannel is not a function

app.listen changed. https://github.com/fastify/fastify/pull/4900

@aryamohanan @abhilash-sivan RFR

Useful guide: Fastify Migration Guide V5.


Update session status to include containers numbers and the steps's status when starting a session

Provide the following information about the session status when it is starting up.
{
   "status":{
      "state":"starting",
      "totalNumContainers":7,
      "readyNumContainers":3,
      "details":[
         {
            "step":"Setting up support containers",
            "status":"done"
         },
         {
            "step":"Cloning and configuring the repository",
            "status":"executing"
         },
         {
            "step":"Pulling main image",
            "status":"queued"
         },
         {
            "step":"Starting server",
            "status":"queued"
         }
      ]
   }
}

Status for steps:
done: step successfully completed
queued: queued for processing
executing: step in progress
failed: step completed with error

Related to https://github.com/SwissDataScienceCenter/renku-ui/issues/1879


Migrate to @solana/buffer-layout

The buffer-layout package used here as a dependency lacks proper typing, which causes issues with downstream libraries depending on the token program TypeScript code.
Solana Labs maintains a buffer-layout implementation with types already:
https://github.com/solana-labs/buffer-layout
I suggest to change the dependency to that implementation.

@jstarry alright. I'm picking this up then. Will submit a PR for your review ASAP.

@jstarry alright. I'm picking this up then. Will submit a PR for your review ASAP.

Awesome, thanks @israelidanny!

Reopening since swap and lending still use the old buffer-layout

@joncinque @jstarry I added this PR:
https://github.com/solana-labs/solana-program-library/pull/2634
It has a couple of caveats (generics missing in current typing e.t.c) but pushes us one step closer to making this tidy

+1, hope the PR lands soon :)

Not sure what this is being held back by! The PR is ready:
https://github.com/solana-labs/solana-program-library/pull/2634
And I'm more than happy to change anything if there are issues with the PR.

Just being held back by review at this point :) I'll try to get to it soon.

There is any way to quick fix it in the meanwhile the issue is still being fixed?

I am not proud of this, but the only way I could unblock myself is by setting "noImplicitAny": false in tscconfig.json:
➜  wonka git:(main) ✗ cat tsconfig.json
{
  "compilerOptions": {
    "target": "es5",
    "module": "commonjs",
    "esModuleInterop": true,
    "declaration": true,
    "outDir": "./lib",
    "strict": true,
    "noImplicitAny": false,
  },
  "include": ["src"],
  "exclude": ["node_modules", "**/__tests__/*"]
}

I reviewed #2634 and have one small question, but I plan to merge tomorrow.

I'm having the same problem. Is there anything I could help to fix this?

@jordansexton any luck? This has been stuck for ages ...

Still facing the issue on React Project, anyone, any suggestions?
Using following versions:
"@solana/spl-token": "^0.2.0",
<EMAIL_ADDRESS>"^1.36.0",

Yo, @jordansexton , what's going on with merging this?

Just been busy with other things 🙂 I'll take a look this weekend.

https://github.com/solana-labs/solana-program-library/pull/2634 has been merged. Thanks for your hard work on this, and sorry for the delay. We'll probably need to release some updated NPM packages, yeah?

when will the package be released this is breaking the metaplex/js lib as it uses the @solana/spl-token lib

@jordansexton yes, I would imagine so. I think this qualifies as a minor version bump (no API changes, just typing)?

when can we expect this to release we are stuck on this issue for days ?

Thanks for your patience. We are currently trying to figure out whether these packages need to be released as a major/breaking version, because it causes problems if someone has buffer-layout and @solana/buffer-layout in their dependencies. We'll sort this out soon! cc @steveluscher

Hey, @jordansexton, any idea when you guys have time to release this? Or is there a workaround (combination of older versions/..)? I'm currently stuck with a project due to this bug.

I was able to work around the type errors by creating a typescript file on my project with the following contents
declare module "buffer-layout";

// https://github.com/solana-labs/solana-program-library/issues/2534
// Remove this workaround when a new version of @solana/spl-token is available
declare interface Layout {}

I was able to work around the type errors by creating a typescript file on my project with the following contents
declare module "buffer-layout";

// https://github.com/solana-labs/solana-program-library/issues/2534
// Remove this workaround when a new version of @solana/spl-token is available
declare interface Layout {}

Could you elaborate on this? Where did you create the file? I've put this in<EMAIL_ADDRESS>but I get the same error.

I was able to work around the type errors by creating a typescript file on my project with the following contents
declare module "buffer-layout";

// https://github.com/solana-labs/solana-program-library/issues/2534
// Remove this workaround when a new version of @solana/spl-token is available
declare interface Layout {}

@felipecsl Could you elaborate on this? Where did you create the file? I've put this in<EMAIL_ADDRESS>but I get the same error.

Put this file in your project source tree, eg under src/, assuming that's where your code lives

I mean, since this is merged - all those quick hacks aren't necessary, are they? @jordansexton any ETA on when this lands on NPM?

Put this file in your project source tree, eg under src/, assuming that's where your code lives

I got that figured out, but it doesn't really fix my issue. Still having the following error. Any ideas?


Update npm dependencies

cc @rambo

d3.cartodb within shrinkwrap is pointing to https://github.com/cartodb/d3.cartodb/commit/2ff44bc39c42c4c450c6f05dd3b216af93e9950a, it should be :+1:

Passing frontend tests, merging.


Add new test option

Add an option to run new or replace with a --test option. This option will check a template file for errors (either .codetender or specified with --file) and if any errors are found, display them to the user and exit. If no errors are found, the test will find and display all files and content that will be replaced as follows for each file:
pattern -> replacement
  path\pattern.txt -> path\replacement.txt
  path\file.txt -> 5 occurences

TODO: Need to address nested replacements. So for example if FooBar is replaced first and Foo is then replaced, this replacement will only happen once but it will be found for each token. Perhaps simple math here will do the trick by finding prior patterns which contain the current pattern and reducing the current count by the parent count.

I think this was written before we always used a temp folder. I think this can run in the temp folder to get the number of replacements and then delete the temp folder.


Dark/black splash screen

https://www.reddit.com/r/xbox/comments/jzl5cb/i_built_a_white_noiserelaxing_sounds_app_for_xbox/gdek2rr?utm_source=share&utm_medium=web2x&context=3

Fixed


Dev

What type of pull request is this?

[x] Non-breaking code change
[ ] Breaking code change
[ ] Documentation change/addition
[ ] Tests change

Description

Merges dev into master
Checklist

[x] I've formatted my code with Black
[x] I've ensured my code works on Python 3.10.x
[x] I've tested my code

Have been on the sentry commit for 2 days. I think this pr is ready


Add append functionality for groups and update tests

For consistency, I stuck to the coding style already present in this cookbook as much as possible (rocket hashes, etc.)
Reasoning for this change is in updated README.md:
If you want some users on all roles, and some users applied to only some of those roles, but all users in the same group on the systems (in cases of shared filesystems, for instance), the JSON might look like this:
user_in_primary_role_group: {
  id: 'user_in_primary_role_group',
  groups: ['primarygroup', 'primaryrolegroup']
},
user_in_secondary_role_group: {
  id: 'user_in_secondary_role_group',
  groups: ['primarygroup', 'secondaryrolegroup'],
},
user_in_primary_group_but_not_roles: {
  id: 'user_in_primary_group_but_not_roles',
  groups: ['primarygroup', 'tertiaryrolegroup']
}

The recipe might then look like this, adding both users to 'primarygroup' by using the append attribute:
users_manage 'primaryrolegroup' do
  group_name 'primarygroup'
  group_id 3000
  append true
end

users_manage 'secondaryrolegroup' do
  group_name 'primarygroup'
  group_id 3000
  append true
end

In this case, 'user_in_primary_group_but_not_roles' is in 'primarygroup', but is not deployed to any servers based on the role groups.
Note: group_id does not need to be unique when using append true, but must match across all instances of users_manage for the group in the recipe. Additionally, the role groups are not created on the servers. Only 'primarygroup' is created with these users as members.

lgtm


[TE] Fix thirdeye-hadoop travis CI

recent pinot change broke thirdeye-hadoop. Compile fix for thirdeye-hadoop by following pinot changes in 3b888de6f61bb28f8bbb1f7906d9156fee1de930.

recent thirdeye change to jdk7 incomplete. Compile fix with compatible API.

update travis to build ThirdEye on oraclejdk7, openjdk7, and oraclejdk8. Only test on oraclejdk 7.

all comments addressed. build passes. please review and merge


About pretrained model

Could you please share the pretrained segmentation model on the Synthia4D dataset? Many thanks!

Hi,
Sorry for my late reply.  I will release the pre-trained model soon.
Best regards.

Thanks for your reply!

Hi @yzheng97,
Please find the pre-trained Synthia4D model via the link https://drive.google.com/file/d/1-hNUDrHNgX13vz7WOgd3y3zNTarvvgxj/view?usp=sharing
Best regards.

Hi @yzheng97,
Please find the pre-trained Synthia4D model via the link https://drive.google.com/file/d/1-hNUDrHNgX13vz7WOgd3y3zNTarvvgxj/view?usp=sharing
Best regards.

Many thanks!


Clean up build_html_pages.sh script

build_html_pages.sh
I started this as a series of shell scripts, so that different sections of the pages could be managed separately. But as I look at the whole series, it probably makes sense to pull them into one file, or a smaller series.
May also rewrite the entire series in Python, this was just what I already knew how to do in bash. I will rethink that as I start to play with the html files more. As of right now they are just a mirror of what nbconvert produces.
Regardless, the script should continue to work either by itself, or run from a commit hook.

Working on this currently, on the template_conversion branch.
This update will make it much easier to iterate on the site's styling, and on taking advantage of new developments in IPython/Jupyter. I'm excited about adding some new projects, and improving the more foundational parts of the project as well.

Closing this old issue in favor of a newer, more specific issue 113.


Add clean script and add clean to build

This helps make sure that you get a clean start when updating and building after changes.

Did you get a chance to look into how this worked @pablof7z ? seems to be helpful from my side but I'm not 100% sure it's any different.


DartType != Entity

https://github.com/dart-lang/sdk/blob/06a471ce86a89a7d712232381a664e6469afaab8/pkg/compiler/lib/src/js_backend/type_reference.dart#L665-L670
Found by enabling the unrelated_type_equality_checks lint
info: Equality operator == invocation with references of unrelated types. (unrelated_type_equality_checks at [compiler] lib/src/js_backend/type_reference.dart:666)
parent is DartType, type.element.typeDeclarationis an Entity. These will never be equal.

Quibble: in legacy mode both operands could be null.
https://dart-review.googlesource.com/c/sdk/+/212343


💡 Enhance Gitbook - add other governance tools

Area
Other
Is there new design needed?
No
What?

would be great to add links / summary of other governance tools via the gitbook

Why?

so people can see the other options for governance tooling

How?
I will do it
(Optional) User Story with acceptance criteria
n/a

@Ryun1 Is this really in progress?

With CC Portal being added to the docs, what other tools did you have in mind @Ryun1 ?

done! at least V1 is done


UnicodeDecodeError: 'charmap' codec can't decode byte 0x81

While testing Meltano I ran into this issue:
[info     ] UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 5719: character maps to <undefined>

Why is this surfacing as [info] instead of error?

I wondered if this might happen because in the Pandas based extractor/loader I wrote before trying Meltano I had to play with encodings because of non-UTF8 chars.

I think a reasonable fix would be to add a config setting of encoding: for tap-csv, and then if set use the specified encoding it in the call to open() in the get_rows() function in client.py. e.g:
with open(file_path, "r", encoding=encoding) as f:

Why is this surfacing as [info] instead of error?

@seajhawk Interesting, I'm not sure. Its hard to tell if its something the tap is doing or if Meltano is re-raising the log incorrectly.

I think a reasonable fix would be to add a config setting of encoding: for tap-csv, and then if set use the specified encoding it in the call to open() in the get_rows() function in client.py. e.g:
with open(file_path, "r", encoding=encoding) as f:

I agree, that sounds like a reasonable addition to the tap! Is this something you'd consider contributing a PR for?

@pnadolny13 - took me a while to find the time, but I did just submit a PR. https://github.com/MeltanoLabs/tap-csv/pull/58

Closing because https://github.com/MeltanoLabs/tap-csv/pull/58 merged. Thanks again @charris-msft

@pnadolny13 - What is the process for getting everything updated so I get the latest version of this tap on my local machine?
it seems like I should be able to run meltano upgrade, but that didn't pull the latest and when I look at Meltano Hub, it says the last update was '05 Mar 2022'.

@seajhawk the upgrade command doesnt upgrade plugin versions https://docs.meltano.com/reference/command-line-interface#upgrade.
If you dont have the tap version pinned then re-installing using --clean https://docs.meltano.com/reference/command-line-interface#install should do the trick.
If you've got the tap version pinned then bump it to v0.0.6 and run the --clean install.


Reward anti-whaling

Currently rewards suffer do not work as intended when applied to a range of markets, such as all markets with the same settlement assst.
Users on long tail markets receive almost no rewards compared to the most popular markets where there's a orders of magnitude difference in the volume between the most and least popular markets. This is at odds with the fact that the bigger markets generate more fees and profits for MMs, and so are less in need of incnetivisation. The result is rewards have to be manually targeted to smaller markets if the aim is to help them grow.
To fix this, an "anti-whaling" featujre similar in spirit (and likely in design, too) to the one that has helped to successfully keep VEGA delegation well distributed will be introduced to the protocol. This will put a cap on the share of rewards distributed for activity on each market relative to the equal fair share.
For example: if there are 10 markets, the "fair" share for each would be 10% of the rewards per market. Depending on the volume on each, with no anti-whaling, one market mayh get 90% of the rewards. Anti-whaling would allow the amount to be capped to some multiple of this fair share number, e.g. 2x would mean a larger market could get up to 20% of the rewards, or 1x would distribute the rewards eqaually between markets in all cases.
This must be optional, so that a reward can be distributed as they are now if desired.

Maybe we don't need anti-whaling IF scaling https://github.com/vegaprotocol/specs/issues/2318 is up and running.


If-None-Match on non-GET APIs?

Do any Google APIs support If-None-Match on non-GET method calls?
e.g. should we include methods like:
func (c *ArchiveInsertCall) IfNoneMatch(entityTag string) *ArchiveInsertCall {

?
I think we won't for now, but add them if we find a supporting example, even if it's just a single API.
The spec (http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html) section 14.26 says it's also meant for PUT, etc. But does Google do that anywhere?
Related to issue #86 (ETag, etc)
/cc @gmlewis @mcgreevy @ImJasonH

The case that immediately came to mind was cloud storage, but they have custom in-band mechanisms for this kind of thing https://cloud.google.com/storage/docs/json_api/v1/objects/update#ifGenerationNotMatch

These clients are in maintenance mode. We don't plan to address this issue.
@pongad for the next generation of clients.


📝[Docs]: Add contributor in the image form

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
Adding the contributors to README.md
Suggested Change

Rationale
No response
Urgency
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

can u assign this to me @sanjay-kv


MomentJS "is a legacy project in maintenance mode"

It is nice to see this level of candor from a project:
https://momentjs.com/docs/?/displaying/format/

...
Another common argument against using Moment in modern applications is its size. Moment doesn't work well with modern "tree shaking" algorithms, so it tends to increase the size of web application bundles. If one needs internationalization or time zone support, Moment can get quite large. Modern web browsers (and Node.js) expose internationalization and time zone support via the Intl object, codified as ECMA-402. Libraries like Luxon (and others) take advantage of this, reducing or removing the need to ship your own data files.
Recently, Chrome Dev Tools started showing recommendations for replacing Moment for the size alone. We generally support this move.
You may also want to read:

You Probably Don't Need Moment.js Anymore
You don't (may not) need Moment.js
Why you shouldn't use Moment.js...
4 alternatives to moment.js for internationalizing dates

The Moment team has discussed these issues at length. We recognize that many existing projects may continue to use Moment, but we would like to discourage Moment from being used in new projects going forward. Instead, we would like to recommend alternatives that are excellent choices for use in modern applications today. We would also like to promote the Temporal addition to the JavaScript language, which is looking for feedback and contributors.
We now generally consider Moment to be a legacy project in maintenance mode. It is not dead, but it is indeed done.
...

Moment is imported from Obsidian core, it's not from this project.
I would hope they're using DayJS as it's feature compatible and exports a moment object.
Regardless, there's no action I can take within this plugin to affect the Obsidian devs decision.


Text inside  doesn't show up

@davidroyer, Reasking from issue #63 @Aoqidx
If content is '
Some initial content
', editor can show 'Some initial content' correctly, but when content is '
Some initial content
', editor throw a error "Cannot read property 'emit' of undefined"
This has still not been fixed and is a big show stopper for us.
Please help!!!

Really. Using TipTap till this gets fixed.

Really. Using TipTap till this gets fixed.

TipTap didn't work out of the box properly. Problem with icons. See https://github.com/scrumpy/tiptap/issues/125

@yocheved any luck finding a solution for this?

@t0milee no sorry we've just been using  tags instead. Looking for a new editor with no luck yet.

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


[BUG] : site logo in footer not functional

Description
The logo of "Collect your gaming tools" must redirect to the top of the page. Please let me work on it for gssoc-extd. Thanks!
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done

@swaraj-das Please assign this issue to me.

@swaraj-das Please assign ..

@swaraj-das Please assign brother, I'm requesting you..


Adaptive size of the documentation text box

Problem you would like to solve
Having a huge documentation text, I can increase the size of the textbox to see all at once.
When I select another element and coming back to the one with the huge documentation, the box shrinks again to the default size.
Proposed solution

The textbox for Documentation should remember its size after increasing for all elements.
We want to persist that information across modeler sessions (i.e. in persisted layout configuration)

Alternatives considered
Show everything by default.
But this could be distracting for users that don't care about the complete documentation and are only interested if a documentation exists or not.
Additional context
The diagram converter from https://github.com/camunda-community-hub/camunda-7-to-8-migration generates all migrations into the documentation element.
Depending on the Camunda 7 element configuration and element content, it could become many characters and sentences.

Not exactly.
Here is a screenshot:

I could not show everything, because it contains sensitive information. But the size of the scroll bar shows how long it can become.
I think it is OK to save the new size of the textbox and apply it to all other BPMN elements. They usually have different content and I have chosen the largest.

The textbox for Documentation should remember its size after increasing for all elements.

We cannot remember across sessions, but we may be able to remember within an individual session.

What about adding the size to config.json on disk?
I found this file in AppData\Roaming\camunda-modeler even touched today without using the modeler. It's still running in background.

What about adding the size to config.json on disk?

That is a away to persist it. Good idea.

@nikku can you assign this to me, want to give this a try!

Happy to take your contribution in the form of a PR.

We already have it in the name field of the BPMN editor.

Fixed upstream by https://github.com/bpmn-io/properties-panel/pull/377


Checklist for 4.0.0 (draft)

Implementations

[x] Fix Omega Strike to apply to touch attacks.
[x] Add Class variant features. (#69)
[x] Add Mod support for modded spells (#72)

Release Tasks

[x] Copy localization
[x] Update CHANGELOG.md
[x] Update README.md
[x] Update Info.json
[ ] Update Repository.json

Post-Release Tasks

Update mod description
Update mod version
New screenshots
Release description

yeah I really have to get started on that localization so we don't spam all the users log with the warning that Strings are automatically recreated, don't I?
Meanwhile preliminary Changelog and still open ToDo List:

internal logic switch to directly use TTCCore where possible rather than code "inspired" from them
merge spellbook logic to merge the spellbooks of the major classes into the isekai book rather than add each spell manually
patch for some other mod that turned deity selection into a recursive selection call onto itself which made it hard to select deities
Blessing of the Mythic Overwhelming Feature (grants abilities of a mythic class besides your own, possible selections include spells, trickster, Azata, Angel, Lich Feats, only Azata and Tickster Feats are what I would call well tested at this point though)
Major Class Progression rework, thief feats removed from all archetypes that had them, instead players can now select a base class from which their Isekai Hero Class descends, current choices are:

Barbarian Legacy - Ball of Rage, mostly grants Rage
Bard Legacy - Musical Prodige, for the hero that knows  they way to his harems heart is through a good song
Dread Knight Legacy - Dread Lord, only available with Expanded Content installed, the anti paladin
Paladin Legacy - Hero of Light, smite your enemies with the power of the light
Kineticist Legacy - Kinetic Knight, melee fighter Kineticist (while not required I advice installing Expandended Element as their energize weapon feat is what really makes this archetype even work)
Kineticist Legacy - Kinetic Lord, the default Kineticist
Magus Legacy - Spellblade, Attention Still requires any form of descent testing, so only use it if you are willing to essentially be the betatester for the Legacy at the moment
Oracle Legacy - Seeker of Truth, choose a curse, mystery, or revelation every few levels
Rogue Legacy - Supernatural Thief, all the talents I removed from the default template
Shaman Legacy - Spirit Beacons, hex hex grant me power my spirit
Sorcerer Legacy - Chimera, grants a bloodline or bloodline feat every few levels
Tactician Legacy - Isekai Tactician, not a baseclass by itself but rather a mixture of several like hunter and Monster Tactician to create a path focused around tactical feats

Paladin and Dread Knight Legacy are appropriatly Alignment Restricted.
ToDo restrict the two Kinetic Legacies to ensure you can only ever pick one, please do not try to combine these.
ToDo the archetypes might need to be balanced against each other, because being able to pick a bloodline or Mystery every few levels is truly worthy of a Cheater coughs Sorry I meant Protagonist ;) while Rogue and Paladin variants actually are less than their base class in terms of benefits.
Isekai Protagonist has access to all variants for their base feature.
Hero only has access to heroic ones like Paladin...
Edge Lord is limited to melee focused Legacies like the Kinetic Knight, Rogue, Barbarian...
Villain obviously does not get choices like Barbarian(because you are meant to be a planner not a brute that just hits everything), Paladin (this one should explain itself) or Bard(you are not a Disney Villain!)...
God Emperor does not get a default selection because it was missing Rogue Talents in their original Balancing against each other.
Every Version can Dualclass by picking the Overwhelming Ability - Legacy Class Feature, which currently imposes no restrictions on what archetype you pick so even the God Emperor can still get access to a legacy.
ToDo Maybe? Restrict some archetypes so a Villain can't pick Bard here either, this mostly just affects the villain as the other absolutely nonsensical choices like the Hero Dread Knight are already blocked by Alignment Restrictions...
**ToDo @JohN100x1 ** Decide if this really should be an Overwhelming feature or would be fairer for the archetypes as a special power
Known Issues:

Sorcerer and Oracle do not like retroactive selection of a Bloodline or Curse, they are fine with retroactivly applying the features of said Bloodline or Curse, to prevent the game from locking up due to invalid selections this means that if you pick those legacies at higher levels you only get the Bloodlines/Curses/etc. that you would get from that point on and not the past ones. Explanation by example: so if you theoretically select Sorcerer Legacy - Chimera at level 6 you miss out on the level 1 and 3 Bloodline but get the Level 6 one and all future ones and if you pick the arcana bloodline at level 6 you do still get all of its features including the arcane bond selection it had at level 1 at that step.
Neither the Curse Selection nor the Bloodline Selection count for the basic selection, so if you properly dual class (rather than just select their legacy as a dual class) into those classes later you still have to make those as choices.
If you use ToyBox to pick multiple classes at each Level you will run into the same issues that you would with Stigmatized Witch and Oracle for your progression calculation, meaning it sums it all up rather than caping it at character level, except since the Protagonist is now patched into most progressions you get it for almost any class that has a progression

I just leveled up in game and saw something I missed

capstones for the legacies if TT Alternate Capstones are enabled

"Neither the Curse Selection nor the Bloodline Selection count for the basic selection, so if you properly dual class (rather than just select their legacy as a dual class) into those classes later you still have to make those as choices."
I'd say this one doesn't sound like an issue (If it behaves the way I understand what you wrote).

"Magus Legacy - Spellblade, Attention Still requires any form of descent testing, so only use it if you are willing to essentially be the betatester for the Legacy at the moment"

Sorry if it's mentioned somewhere else, but is this related to why there isn't a Broad Study arcana option for the Magus to access the isekai class spellbook? I'm interested to learn what the expected issues are for this update, as well.

"Neither the Curse Selection nor the Bloodline Selection count for the basic selection, so if you properly dual class (rather than just select their legacy as a dual class) into those classes later you still have to make those as choices." I'd say this one doesn't sound like an issue (If it behaves the way I understand what you wrote).

It is an issue because if you multi class witch and oracle the curse  selection of the first will actually count for the second, same if you multi class magus and sorcerer and the bloodline.
So people should be aware that this is not the case if you go Protagonist first and then go for the second class, that means if you only wanted only one curse(for example you are role playing a dhampir from a different basespecies than humand and thus only wanted vampirism) then you should not pick it as a Protagonist feature because the Oracle will force you to pick another.

"Magus Legacy - Spellblade, Attention Still requires any form of descent testing, so only use it if you are willing to essentially be the betatester for the Legacy at the moment"

Sorry if it's mentioned somewhere else, but is this related to why there isn't a Broad Study arcana option for the Magus to access the isekai class spellbook? I'm interested to learn what the expected issues are for this update, as well.

There is no such option because the mod mostly focuses on the protagonist and what to give them. Adding this option for the arcana is possible if people want it. Though there is not much difference for the magus except that you don't have to think about if the spell was in the cleric, wizard, or druid spellist when picking this selection.
Expected issues are:

balancing issues amongst the archetypes(given this mod I am not sure this is an actual issue though ;) )
things not quite working as expected yet, for example during the last week of playtesting I found that shaman uses yet another mechanic to add the spells granted by its spirits, this method causes a level 0 shaman spellbook to appear on your sheet, ugly but not gamebreaking and any fix I could think of for it would mean we could no longer automatically support shaman spirits added by mods, and I am not sure if there are not some resources still missing that might cause issues in the form of I picked feature x but didn't get the expected bonus y
downward compatibility to the previous version, moving features into progressions means that you will have to respec from level 1(unless you are a God Emperor) to rebuild the character.

@JohN100x1
Updated Changelog and Readme, as for spells see my comment inthe original thread.
When you say update the info.json and repository.json you specifically mean the version number, correct?
So those would be the last things pre release

I also got myself a better tool for merging git repositories, the visual studio internal one is just horrible :D
so future merges should be easier and I expect to be done with the localization file this evening.

or earlier since I just noticed that it had updated that file in my game folder as well so I could just merge it over to the working directory.
I am also now officially rebased on the master branch and would consider TTCMerge closed making new mini branches for all future changes.

There is no such option because the mod mostly focuses on the protagonist and what to give them. Adding this option for the arcana is possible if people want it. Though there is not much difference for the magus except that you don't have to think about if the spell was in the cleric, wizard, or druid spellist when picking this selection.
That makes sense, I was just thinking about legend builds and throwing spellstrike onto a max-level villain for giggles. As you're already planning the legacy thing, though, I would say giving access via Broad Study isn't all that necessary beyond that particular use case. This is just a question I've had since noticing Broad Study didn't include the protagonist spellbook (i respec'd the levels after realizing the incompatibility, but the legacy would basically sidestep the process of multiclassing in the first place, which is good.)

* balancing issues amongst the archetypes(given this mod I am not sure this is an actual issue though ;) )

kermit_sip.png

* things not quite working as expected yet, ...and I am not sure if there are not some resources still missing that might cause issues in the form of I picked feature x but didn't get the expected bonus y

I sure hope spellstrike doesn't have something weird like this. I just wanna apply Harm and Heal with a scythe! And maybe put out a Polar Ray with a longbow idk.

* downward compatibility to the previous version, moving features into progressions means that you will have to respec from level 1(unless you are a God Emperor) to rebuild the character.

I... maybe will hold off on updating to the new system, at least until I finish the playthrough I'm on (or it corrodes under the weight of my poor mod management...)
Thanks for the continued work, y'all. I love this mod, it's great fun.

I will make certain to retest spellstrike specifically throughout the week.
Also, to clarify none of your features would actually be lost in the upgrade, you just wouldn't get additonal sneak attack dice at your following levelups without the respec.
So if you are already at max level the changes also won't mean anything to you.
But yes, finishing whatever run you are on at the moment and then using the new version for a fresh game is a good idea given the scope of the changes.
I would personally actually be interested in how many people switch to a non rogue legacy and what the most popular ones are, as well as feedback on what should still be added to existing templates and what else people would like to see.

I... maybe will hold off on updating to the new system, at least until I finish the playthrough I'm on (or it corrodes under the weight of my poor mod management...)

Can't wait to begin a new run with the new version!

I would personally actually be interested in how many people switch to a non rogue legacy and what the most popular ones are, as well as feedback on what should still be added to existing templates and what else people would like to see.

Kineticist forever here!
Does the main blast stat will be the one used for casting?
Would be more complicated if not (Truinto did something for letting us choose which stat we want for psychokineticist it's really great)
Oracle revelations and mysteries will be nice.
I guess for casters (I only play those...) bloodline will be less useful since the progression features are usually less useful than the arcanas themselves (even more for Protagonist regarding immunities , which can be acquired by Isekai feats)

You are patched into the kineticist as normal, so constitution would be the score for that.
Though honestly I would have to check if the hero or edge lord feat to change the damage bonus affect that.
When referencing that someone already did something for a specific archetype in a different mod please always clearly state the mod, preferably with a link, that makes checking on it easier.
Does only the Psychokineticist get that bonus or is it a feat that any kineticist might pick? If it is the second then just loading both mods should grant you that ability already because you count as a full kineticist for the sake of qualifying for feats.

@bundillion
So Broad Study is only necessary if Tabletop Tweaks Base is installed because that is what disables those features for everything but the magus spells in the first place.
I will update our compatibility patch so that if Broad Study is enabled the magus legacy also gets this Component as a hidden fact rather than as a Selection Option.

@kjk001
Sorry for the incomplete info :
Here you have the original topic on DarCodex repository and the way it was implemented (so not a feat, but a choice in the UMM settings)

Another point : the size bonus granted by burns would not be useful to any caster Protagonist (or redundant with spell buffs).
Complicated but would it be a way to select the kineticist archetype, since there is already a choice regarding kinetic knight?

I also got myself a better tool for merging git repositories, the visual studio internal one is just horrible :D so future merges should be easier and I expect to be done with the localization file this evening.

Nice, I personally use software called Sourcetree.
Anyways it is good to see that the mod is still worked on.

Undead template should switch any con based check over to charisma anyway, if it does not then that is an error in the template itself that needs patching by owlcat so make sure to send them a bug report.
I can add additional archetypes for separate subtypes, but making it a choice inside the progression would add another level of progression and that might cause issues if we add too many layers.
So if you have requests for specific subtypes feel free to ask but formulate them as their own legacy archetype and do provide its name and description as I am not very good at that.
Those Archetypes would then also be something for 4.1 as a version and not 4.0. Because if we keep adding to the current build then it will never be released.
@Nerin3 was it ever not? I mean the changes I just made might be rather massive and thus took a few weeks but John has continously worked on bugfixes for the existing features and done review and cleanup on my code.

Actually the undead template should not change blast main stat to charisma by itself since the burn system is based on non lethal damage, and undeads are immuned to those (that was the point of the Overwhelming Soul archetype).
I'll provide a name and description in a separate topic!

Let me rephrase my statement then, as far as I am aware the game should default to charisma for any con based requirement for undead to prevent bad Nullpointers or Null Divisions.
Regardless of the fact that some of those features should simply not be possible as an undead based on the more flexible ttrpg ruleset for which such pointers are not causes of a crash but simply open to interpretation.

Anyway here are some screens, bonus points to who can spot what horrible thing the edge lord in the last screen did before the update is released:

Fear my Edge Lord:

and I would say at least for version 4.0.0 I am done

Now for a trully OP Protagonist class: Legacy Master, every 4 levels they can pick one additional legacy.

Could be an overwhelming power only selectable at lvl1 :p

and no cookie for Nerin or mythalar

Horrible thing? Selecting dart as weapon of choice ? xD

I think it's the Arcane Armor Mastery, since the protagonist doesn't have Arcane Failure "Unless it was added back in."

I think it's the Arcane Armor Mastery, since the protagonist doesn't have Arcane Failure "Unless it was added back in."

You are right, I thought it was useless but the raging-darkshadows-edging lord of flying butter knifes would be a truly horrible sight to behold

@bundillion So Broad Study is only necessary if Tabletop Tweaks Base is installed because that is what disables those features for everything but the magus spells in the first place.

... I feel like I should have been able to figure this out. TT has such a long list of features, too.

either way patch that quietly adds the correct marker to the progression if TTB is enabled has been done and tested.
Arcane Armor Mastery is actually useful if you consider taking another arcane class because you then want to go mystic theurge on them. If not is it useless but not a horrible thing that is cause for fear

Except the fearsome darts I can only see the 10dex on an archetype based on Dex :p

either way patch that quietly adds the correct marker to the progression if TTB is enabled has been done and tested.

Actual Platinum-label Rockstar kjk, known icon

paladin abilities rely on charisma even if it is an anti paladin, also everything that was not a new for display feature was simply randomly picked to quickly get to the next point, something I now regret as you focused on everything but the obvious

Well, except that selecting Ball of Rage seems to have blocked him from selecting nearly half the proposed legacies I can't tell -_-

Edge Lord, Hero, and Villain have their own list of acceptable first choices for their legacy, that is the reason why the list is only half as long and not because of the choice of ball of rage

Yes I read this but that seemed to be a harsh cut !
But week, might be too obvious for me to find what's so terrifying;)

Not all lists are equally short, I tried to add to every list the choices that make sense for that archetype.
Also check the picked overwhelming feat, you can already pick a second legacy and there are no limits on it besides alignment.
So those apparent fever dreams of you and Nerin are already true.

Oh I thought I did not see that there was no limits.
An all-party man!

To be honest it did not strike me  so much that you could select both legacies from the start ^^

Hello, please tell me - what is TTCCore?

Hello, please tell me - what is TTCCore?

 It was made in response to Owlcat truncating a whole bunch of mechanics from the core system in order to make the videogame part of WotR (and Kingmaker) a bit more videogame-like rather than ttrpg-like.
(one of those options adds the Broad Study magus arcana, which is what caused the issue I brought up in the first place lol)

I think we need a standardised .csproj file because everytime I try to build I have to change it back to my setup

I'm puking blood, the 2.10u update broke scaling cantrips and, therefore, my progress. Guess I'll just wait for this to update for the next run. Gonna go get a mop for all this red bile.

patch for some other mod that turned deity selection into a recursive selection call onto itself which made it hard to select deities

@kjk001 Which mod is this exactly? The new code seems to give me a null reference exception when I try to load the game
NullReferenceException: Object reference not set to an instance of an object
  at Kingmaker.TextTools.NameTemplate.Generate (System.Boolean capitalized, System.Collections.Generic.List`1[T] parameters) [0x00027] in <7233d069edc947ad9245aca7f2d9f2b6>:0
  at Kingmaker.TextTools.TextTemplateEngine.Process (System.String text) [0x00242] in <7233d069edc947ad9245aca7f2d9f2b6>:0
  at Kingmaker.Localization.LocalizedString.op_Implicit (Kingmaker.Localization.LocalizedString localizedString) [0x00069] in <7233d069edc947ad9245aca7f2d9f2b6>:0
  at Kingmaker.Blueprints.Facts.BlueprintUnitFact.get_Name () [0x00000] in <7233d069edc947ad9245aca7f2d9f2b6>:0
  at TabletopTweaks.Core.Utilities.BlueprintExtentions+<>c.<AddFeatures>b__18_0 (Kingmaker.Blueprints.BlueprintFeatureReference feature) [0x00010] in <74e6287844a240d1a54a73c18234bf69>:0
  at System.Linq.EnumerableSorter`2[TElement,TKey].ComputeKeys (TElement[] elements, System.Int32 count) [0x00010] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at System.Linq.EnumerableSorter`1[TElement].ComputeMap (TElement[] elements, System.Int32 count) [0x00000] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at System.Linq.EnumerableSorter`1[TElement].Sort (TElement[] elements, System.Int32 count) [0x00000] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at System.Linq.OrderedEnumerable`1[TElement].SortedMap (System.Linq.Buffer`1[TElement] buffer) [0x00006] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at System.Linq.OrderedEnumerable`1[TElement].ToArray () [0x00025] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at System.Linq.Enumerable.ToArray[TSource] (System.Collections.Generic.IEnumerable`1[T] source) [0x0001f] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at TabletopTweaks.Core.Utilities.BlueprintExtentions.AddFeatures (Kingmaker.Blueprints.Classes.Selection.BlueprintFeatureSelection selection, Kingmaker.Blueprints.BlueprintFeatureReference[] features) [0x0007f] in <74e6287844a240d1a54a73c18234bf69>:0
  at TabletopTweaks.Core.Utilities.BlueprintExtentions.AddFeatures (Kingmaker.Blueprints.Classes.Selection.BlueprintFeatureSelection selection, Kingmaker.Blueprints.Classes.BlueprintFeature[] features) [0x0002b] in <74e6287844a240d1a54a73c18234bf69>:0
  at IsekaiMod.Content.Deities.IsekaiDeitySelection.Add () [0x0002f] in <9a50b11055804b4097dff00567a788dc>:0
  at IsekaiMod.Content.ContentAdder+BlueprintsCache_Init_Patch.AddIsekaiDeities () [0x00000] in <9a50b11055804b4097dff00567a788dc>:0
  at IsekaiMod.Content.ContentAdder+BlueprintsCache_Init_Patch.Postfix () [0x00073] in <9a50b11055804b4097dff00567a788dc>:0
  at (wrapper dynamic-method) Kingmaker.Blueprints.JsonSystem.BlueprintsCache.Kingmaker.Blueprints.JsonSystem.BlueprintsCache.Init_Patch5(Kingmaker.Blueprints.JsonSystem.BlueprintsCache)
  at (wrapper dynamic-method) Kingmaker.Blueprints.JsonSystem.StartGameLoader.Kingmaker.Blueprints.JsonSystem.StartGameLoader.LoadPackTOC_Patch2(Kingmaker.Blueprints.JsonSystem.StartGameLoader)
  at Kingmaker.GameStarter+<StartGameCoroutine>d__25.MoveNext () [0x00375] in <7233d069edc947ad9245aca7f2d9f2b6>:0
  at UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) [0x00026] in <25a85da7c6f04932b86e339dfd12957d>:0

I don't know what mod did it, but that place you changed back also has nothing to do with the actual fix for that recursive nightmare, because the patch function below is called much later than the adding of the deities.
If the sorting there crashes for you due to a null name then likely at least one mod in your collection also screws with deities in a way it shouldn't.
What is a bit weird is that I know that at least one of my mods also does this because I get a null value warning when I do the sorting at the end but for me the ttc way does not cause a crash.
Shelving and unshelving the csproj file with no further changes should work shouldn't it?

Ok the release has just been published. If there's any bugs that occur we add it to the 4.1.0 checklist (#81) to deal with.
I also created a new branch so anything being implemented should go in there rather than the master branch. That way I can see all the changes before merging to master.


nixos/no-x-libs: add more replacements

all-packages: change -nox overrides to callPackage's so there is no infinite recursion when replacing the package with a overlay
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@Artturin Is there an easy way to measure the delta in the closure size?

@Artturin Is there an easy way to measure the delta in the closure size?

this doesn't change the closure size much (atleast for a minimal-ish system assuming i did the comparison correctly) but instead reduces the dependency tree size ( reducing compilation times)
{
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/refs/pull/204287/head";
    # commit before the above pr
    nixpkgs-before.url = "github:NixOS/nixpkgs/7bfceb02093f2c7ea0ea7e353c99b26e9a4022fc";
  };

  outputs = inputs:
    let
      pkgs = import inputs.nixpkgs { system = "x86_64-linux"; };
      pkgs-before = import inputs.nixpkgs-before { system = "x86_64-linux"; };

      node = pkgs.nixos ({ modulesPath, ... }: {
        fileSystems."/".device = "ignore-root-device";
        boot.loader.grub.enable = false;
        imports = [
          (modulesPath + "/profiles/minimal.nix")
        ];
        networking.networkmanager = {
          enable = true;
          plugins = pkgs.lib.mkForce [];
        };
      });
      node-before = pkgs-before.nixos ({ modulesPath, ... }: {
        fileSystems."/".device = "ignore-root-device";
        boot.loader.grub.enable = false;
        imports = [
          (modulesPath + "/profiles/minimal.nix")
        ];
        networking.networkmanager = {
          enable = true;
          plugins = pkgs.lib.mkForce [];
        };
        nixpkgs.overlays = [
          (_: super: {
            stoken = super.stoken.override { withGTK3 = false; };
          })
        ];
      });

      closureInfo = pkgs.closureInfo { rootPaths = [ node.config.system.build.toplevel ]; };
      closureInfoBefore = pkgs.closureInfo { rootPaths = [ node-before.config.system.build.toplevel ]; };
    in
    {
      packages.x86_64-linux.default = pkgs.runCommand "compare"
        {
          inherit closureInfo closureInfoBefore;
          nativeBuildInputs = [ pkgs.nodePackages.insect ];
        } ''
        sizeBytes=$(cat $closureInfo/total-nar-size)
        sizeBeforeBytes=$(cat $closureInfoBefore/total-nar-size)
        insect "$((sizeBeforeBytes - sizeBytes)) bytes -> megabytes"
        touch $out
      '';
      packages.x86_64-linux.has-libX = closureInfoBefore;
      packages.x86_64-linux.no-libX = closureInfo;
    };
}

2167
$ nix path-info -rsSh ".#packages.x86_64-linux.no-libX" --derivation | wc -l
1866

FYI you motivated to try this again especially for my simd branch and I created the following PRs:

#204545
#204564
#204573

Related

#204977

I'm not doing this through the NixOS no-x-libs overlay, because this should improve many packages that have no business having graphical libraries in their build closure, saving many rebuilds, regardless of whether the user wants to optimize for a non-graphical system configuration.

wait until https://nixpk.gs/pr-tracker.html?pr=204426 is in master

Why not merge this into staging?

Needed to create a new PR because force pushing that one almost deadlocked my computer. #211428

This PR causes this error:
error: infinite recursion encountered

       at /home/lafiel/works/src-nix/nixpkgs/pkgs/top-level/all-packages.nix:20695:3:

        20694|
        20695|   libdevil-nox = libdevil.override {
             |   ^
        20696|     inherit (darwin.apple_sdk.frameworks) OpenGL;
(use '--show-trace' to show detailed location information)


properties: Can't use the macro for a custom glib::Boxed type property

This is a minimal reproducer of what I'm trying to do: lib.zip.
The compilation fails with:
error[E0277]: the trait bound `RefCell<Option<StrBoxed>>: Property` is not satisfied
  --> src/lib.rs:16:21
   |
16 |     #[derive(Debug, Properties, Default)]
   |                     ^^^^^^^^^^ the trait `Property` is not implemented for `RefCell<Option<StrBoxed>>`
   |
   = help: the trait `Property` is implemented for `RefCell<T>`
   = note: this error originates in the derive macro `Properties` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0277`.

Am I just doing something wrong?

We are probably missing a HasParamSpec trait impl, could you try adding a manual implementation to your type and see if it helps?

@bilelmoussaoui That gives me:
error[E0119]: conflicting implementations of trait `glib::HasParamSpec` for type `StrBoxed`
 --> src/lib.rs:4:17
  |
4 | #[derive(Clone, glib::Boxed)]
  |                 ^^^^^^^^^^^ conflicting implementation for `StrBoxed`
...
8 | impl glib::HasParamSpec for StrBoxed {
  | ------------------------------------ first implementation here
  |
  = note: this error originates in the derive macro `glib::Boxed` (in Nightly builds, run with -Z macro-backtrace for more info)

I think what we're missing is the Option<T> part here, not StrBoxed. That would need a generic impl in glib

I think what we're missing is the Option<T> part here, not StrBoxed. That would need a generic impl in glib

Isn't it expected that ParamSpecBoxed doesn't accept nullable values? as in, I would expect the user to have StrBoxed(Option<Str>)  instead

Most boxed/object properties are nullable, and you can derive boxed/shared as nullable too.

HasParamSpec is implemented on every type Option<T> where T: glib::value::ToValueOptional.
If StrBoxed doesn't implement ToValueOptional, the macro can't handle it. Option<StrBoxed> can't be converted to a glib value at all, so it can't be treated as a property.
Workarounds without implementing ToValueOptional:

Adding type = StrBoxed and manually adding a getter and a setter.
Using glib::ConstructCell, if you plan on setting the value during construction, and then keeping it always Some(x). This type didn't get much love/testing yet.

StrBoxed would have to use
#[derive(Clone, Debug, PartialEq, Eq, glib::Boxed)]
#[boxed_type(name = "MyStrBoxed", nullable)]
pub struct StrBoxed { ... }

Most boxed/object properties are nullable, and you can derive boxed/shared as nullable too.

Oh, yeah, you're right, I completely forgot about deriving nullable. Now it works, thanks :)

Using glib::ConstructRefCell, if you plan on setting the value during construction, and then keeping it always Some(x). This type didn't get much love/testing yet.

That's super nice, I'll definitely try it soon in my projects :)
I'll consider this solved then.


Error loading css styles from URLs

VSCode Version: 1.39.2
OS Version: Arch Linux x86_64 with 5.3.7 Linux kernel

Steps to Reproduce:

Modify markdown.styles property with a remote css
Try the preview of your .md file
You'll get a warning.

Does this issue occur when all extensions are disabled?: Yes/No
I need Markdown All In One to preview .md files but I've already reported in there, In fact I'm coming from here
Thank you all.

This is not a VS Code issue. Github explicitly serves content from raw with the content-type text/plain. Most modern browsers will refuse to treat this as a stylesheet
If you use an https style, make sure it is served up with the correct content-type


Bash completion

We should have that.

This was (partially) implemented in #27. I'm going to let people kick it around and see what happens.


performance degradation after 60 volumes

Hi all,
I'm testing different solution of kubernetes storage and I came across Longhorn.
In order to do some stress test I created 100 deployments with replica 1 and nginx with one volume attached.
After 55-60 volumes, longhorn system hung and the api became really slow.
Pod creation went in timeout due to missing volume.
At 100 volumes the system became completely unusable.
current hardware setup:
3 nodes Kubernetes cluster on Centos 7, one master, two worker, tainting disabled on 3 HP DL380 G10, 64 core, 512 GB RAM, 20Gb ethernet between nodes. 2 disks pool 5TB
The operating system was pretty in idle. Longhorn-manager Pods had 30% cpu load (docker stats)
There are some limitations I need to know or some tweaks need to be applied on order to resolve this situation?
I've downloaded the support bundle in case of need.
thank you for your help.

@marcomezzaro Thanks for the reporting. And it's really nice you've saved the support bundle. Can you send the support bundle to<EMAIL_ADDRESS>? We will take a look. I have some suspects in mind.

Mark as a duplicate of https://github.com/longhorn/longhorn/issues/1058 . It will be fixed in upcoming v0.8.0 release.


Add [KuCoin kucoin.com] #memo-required GAJ4BS…BQJW2P

Address: GAJ4BSGJE6UQHZAZ5U5IUOABPDCYPKPS3RFS2NVNGFGFXGVQDLBQJW2P
Tags: memo-required
Title: KuCoin  Domain: kucoin.com
Requested by @rexboy14

Memo1929829082

@rexboy14
Combined with https://github.com/stellar-expert/public-directory/issues/457


Provide a template for data models aligned with NGSI-LD

current idea is to have a single specification that serves for all the NGSI flavours

fixed #487


Update specs2-core, specs2-scalacheck to 4.9.2

Updates

org.specs2:specs2-core
org.specs2:specs2-scalacheck

from 4.8.3 to 4.9.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update, semver-minor

Superseded by #145.

Superseded by #145.


Issue #641 log if any issuer in TVP.ValidIssuers is null or empty

@lovemaths,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:shipit:


Can no longer export

Current Behaviour (for problems)
When testing 1.2.1-beta, I can no longer export successfully. I tried a very simple example, and filled in all the usual required information. However, I end up with the Export package error of There is a problem with at least 1 package property. Please check and try again.
Steps to Reproduce

Open -> Comma Seperated
Fill in Table, Provenance, Package info
Click Export

Your Environment

Data Curator version: 1.2.1-beta
Operating System and version: Microsoft Windows [Version 10.0.18363.900]

Thanks for logging this @JDziurlaj
I'll have a look at this today.

I have installed version <IP_ADDRESS>_x64 from the Microsoft Store, and while some initial trials appeared to work, I just experienced this issue after completing full metadata for my CSV data table with no export being produced.  It generates the "Export package error" of 'There is a problem with at least 1 package property. Please check and try again.', but there is no indication what this error is.  This sounds like it could be the same issue as reported in this closed issue, albeit using the updated release.

Prior to attempting export, I experienced some instability when attempting to add sources (it wouldn't add multiple, with the option to add a source ceasing to work in the Package view; and in the table view, when adding a path for a source it complained of an invalid URL when the URL was valid, and when adding an additional source, the entries for both sources appeared corrupted (all blank except title, which for both replaced with a single character of last key pressed).  I removed all attempts to provide a source, and there were not specific errors marked for this upon attempt to export -- but it didn't work.  So I am not sure if the instability related to sources has resulted in the mysterious error preventing export.
Perhaps this is the same issue, still being an issue?  A possible work around would be if I could export the JSON on its own regardless of errors --- then I wouldn't lose my work, and could perhaps modify this manually.  Currently, I am concerned that if I close the program I will lose the metadata entries already added.
Here are my windows version details:

Cheers, Carl


effective rate limiting

I'm finding that I'm exceeding rate limits occasionally.  Has anyone got an effective way of dealing with them?
I see there s a nice little RateLimiter class in guava.  But i think to use it properly it has to be baked in at the lowest level where the RestProxyFactory is invoked.  It needs to be applied at the request level, and shared across the AccountService, TradeService and MarketDataService.    Wrapping the services in another rate limiting proxy might work. Something like this:
  protected TheRockAccountServiceRaw(Exchange exchange) {
    super(exchange);

    ExchangeSpecification spec = exchange.getExchangeSpecification();
    this.apiKey = spec.getApiKey();
    this.signatureCreator = new TheRockDigest(spec.getSecretKey());

    TheRockAuthenticated theApi = RestProxyFactory.createProxy(TheRockAuthenticated.class, spec.getSslUri());

    RateLimiter rateLimiter = spec.rateLimiter();

    if(rateLimiter != null) {
      this.theRockAuthenticated = rateLimit(theApi, rateLimiter, TheRockAuthenticated.class);
    } else {
      this.theRockAuthenticated = theApi;
    }
  }

  private static <API> API rateLimit(API theApi, RateLimiter rateLimiter, Class<API> theClass) {
    return (API) Proxy.newProxyInstance(ClassLoader.getSystemClassLoader(), new Class[]{theClass}, new InvocationHandler() {

      @Override
      public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
        rateLimiter.acquire();

        return method.invoke(theApi, args);
      }
    });
  }
    ...

@timmolter thoughts?

That's what I am currently doing in a custom wrapper everytime I need to call the api.

That's stretching the scope of this lib, I'd say. Especially since we still don't have a robust solution to the rate limit metadata yet, across all exchanges.

A reason to do it at this base level (once an appropriate model is in place) is that some implementations make multiple API calls for a single high level method call.  So putting your rate limiter in your application won't work as effectively.

I do agree, and I can think of specifying this rate at this level for example, not sure if this would be in a third parameter or part of ExchangeSpecification:
public static ExchangeWorkerItf create(Class exchangeName, ExchangeSpecification spec)

Yeah, you're right. I can see integrating something like this into RESCU in a way similar to the nonce factory, where different exchanges have different implementations of rate limiting logic. @mmazi what do you think?

Perhaps a start would be to move the responsibility of creating the api into the ExchangeSpecification? So instead of:
ExchangeSpecification spec = exchange.getExchangeSpecification();
TheRockAuthenticated theApi = RestProxyFactory.createProxy(TheRockAuthenticated.class, spec.getSslUri());

You could have:
ExchangeSpecification spec = exchange.getExchangeSpecification();
TheRockAuthenticated theApi = spec.createProxy(TheRockAuthenticated.class);

That would allow developers to provide their own wrappers around the low level API, which would allow them to do things like rate limiting.  They'd just need to implement their own ExchangeSpecification (or override the default implementation of the createProxy method).

Perhaps XChange could include a RateLimitingInterceptor; see the docs:
https://github.com/mmazi/rescu/wiki/Interceptors
Then this interceptor could be registered when the proxy is created as suggested above.

Hi,
I have implemented this in cointrader, a fully automated java based algo
trading platform. Take a look at this class for an implementation or just
fire up the cointrader and it is all taken care of for you.
https://github.com/timolson/cointrader/blob/master/src/main/java/org/cryptocoinpartners/module/xchange/XchangeData.java
On Tue, Jul 25, 2017 at 8:38 AM, Matija Mazi<EMAIL_ADDRESS>wrote:

Perhaps XChange could include a RateLimitingInterceptor; see the docs:
https://github.com/mmazi/rescu/wiki/Interceptors
Then this interceptor could be registered when the proxy is created as
suggested above.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/timmolter/XChange/issues/1632#issuecomment-317655947,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AENHav3yvSwxoyi7n6XQjj0bl_LhEbhAks5sRZtngaJpZM4OhE5Y
.

Here's a a possible solution it may be a bit dirty but you could always take a variable like a Boolean and have a separate thread that sets it to true every three or four hundred milliseconds and if that bool is true have it do your query or if it's false have it sleep like I said it's not the best solution it's not even a great solution but it would work

@K1ltu It would not work. The proposed solution is open to race conditions.

here is an example of rate limiting
https://github.com/timolson/cointrader/blob/master/src/main/java/org/cryptocoinpartners/module/xchange/XchangeData.java
it takes queries and a period. for example queries=10 and periods =5 would
mean 10 queries every 5 seconds.
the rate at which to poll is limited by the exchange but also by the number
or markets, usage of price data, bandwidth and storage so  seems quite
implementation specific rather than a generic feature.
On Sun, 24 Dec 2017, 11:32 Gili Tzabari<EMAIL_ADDRESS>wrote:

@K1ltu https://github.com/k1ltu It would not work. The proposed
solution is open to race conditions.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/timmolter/XChange/issues/1632#issuecomment-353779329,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AENHauOvMB-iT5boE0PwJNHx70frzy9Qks5tDjZPgaJpZM4OhE5Y
.

a new proposition in this regard: https://github.com/knowm/XChange/issues/3211


Using other buttons went wrong!

I tried to change the buttons in the code by JUST changing the CASE numbers (because I have another NEC remote control which doesn't have the same buttons which in the code buttons), but, it didn't work!
Any ideas?

Hi, thanks for raising this issue.  I guess that you are trying to use incorrect address/command from you remote. It can be quite difficult when it comes to check the code and address without debugging it. This weekend I hope to post an Attiny13 NEC code analyzer that would help you debug it using Software UART.

Hi, first thank u very much for replying fast to this issue :)
I'm new to this stuffs but as I understood that this code is for attiny13 @9.6 MHz and for NEC reomte controls, is it right or I missed anything else?
OBS! I used TSOP1738 as an IR receiver!

Thats right. The code is for ATtiny13 internal @9.6 MHz and for NEC proto.
$ make
avr-gcc -std=c99 -Wall -g -Os -mmcu=attiny13 -DF_CPU=9600000 -I. -o main.o main.c
avr-ld -o main.elf main.o
avr-objcopy -j .text -j .data -O ihex main.o main.hex
avr-size -C --mcu=attiny13 main.elf
AVR Memory Usage
----------------
Device: attiny13

Program:     766 bytes (74.8% Full)
(.text + .data + .bootloader)

Data:         15 bytes (23.4% Full)
(.data + .bss + .noinit)

$ make fuse
avrdude -p attiny13 -c usbasp -U hfuse:w:0xFF:m -U lfuse:w:0x7A:m
(...)
$ make flash
avrdude -p attiny13 -c usbasp -U flash:w:main.hex:i -F -P usb
(...)

I suggest you to test your project with simpler main function that will catch any signal from your remote (NEC) and indicate a received command/address by toggled led. Example code below.
(...)

int
main(void)
{
        uint8_t addr, cmd;

        /* setup */
        DDRB |= _BV(LED_PIN); // set LED pins as OUTPUT
        IR_init();

        /* loop */
        while (1) {
                if (IR_read(&addr, &cmd) == IR_SUCCESS) {
                        PORTB ^= _BV(LED_PIN); // toggle LED1
                }
        }
}

Hope it will help. L

Hi, if you still have a problem with that please take a look at http://blog.podkalicki.com/attiny13-ir-receiver-nec-proto-analyzer/
/L

Hi, thank u very much for all those informations. Actually, I'm using Arduino as an ISP programmer, but, great job thx !

Hi again, I tried a lot but it didn't work. I burnt the fuses as you explained. I guess my problem is with the buttons which I defined. My remote control is the Adafruit NEC protocol 44 buttons.
The buttons are like this:
0xFF02FD (0x40)
0xFF1AE5 (0x58)
0xFF3AC5 (0x5C)
........................... and so on, so can you help me please and tell me what I have to change in you code so it works with my remote control, I'm sure that I have to change the cmd cases in the end of the code.
I tried like this, but it didn't work:
while (1) {
if (IR_read(&addr, &cmd) == IR_SUCCESS) {
if (addr != 0xff)
continue;
switch(cmd) {
case 0x40:
PORTB &= ~(_BV(LED1_PIN)|_BV(LED2_PIN)|_BV(LED3_PIN)|_BV(LED4_PIN));
break;
case 0x58:
PORTB ^= _BV(LED1_PIN);
break;
case 0x59:
PORTB ^= _BV(LED2_PIN);
break;
case 0x45:
PORTB ^= _BV(LED3_PIN);
break;
case 0x44:
PORTB ^= _BV(LED4_PIN);
break;
default:
break;
}
}
}

Hi there. Sorry for late response. Did you try my snippet code from my last post ? I recommend it as a first step of debugging. It will test that your circuit is correct and that library work fine. Expected result is blinking LED when you push any button of your remote.
`(...)
#define LED_PIN PB0
int
main(void)
{
uint8_t addr, cmd;
    /* setup */
    DDRB |= _BV(LED_PIN); // set LED pins as OUTPUT
    IR_init();

    /* loop */
    while (1) {
            if (IR_read(&addr, &cmd) == IR_SUCCESS) {
                    PORTB ^= _BV(LED_PIN); // toggle LED
            }
    }

}
`


Add min period to worker sleep methods.

Found the pyTAK workers can use 100% CPU when not handling any message traffic. It seems this is likely the RX Worker spinning while looping.
Add a member to Worker class to hold a consistent minimum period. Currently set to 0.1 sec. This reduces CPU from 100% to 0.7%.

I just ran a highly unscientific test. Unpatched pytak, with a sleep of 0, sits above 90% CPU most of the time. Changing to 0.1 keeps me well below 15% most of the time. Lowering even more to 0.01 has me between 10 and 20%.

Thanks @PeterQFR !


how I can use flowbite in react without flowbite-react library

I did not find any documentation out there which help me use flowbite in reactjs project without flowbite-react library. Just wondering if someone can guide me on how we can do that.
Thanks

If you're using create-react-app you should install tailwindcss by following tailwindcss guide
then install flowbite following the official docs
now start the react server and use any component you want :D
Dont's forget to configure the tailwind.config.js file :)


High fluctuation in resource usage, especially with webserver pods

What is the bug?
After deploying the Airflow Helm chart, I see a high fluctuation in resource usage from some pods. Especially, the web server pod will go from as low as 3m CPU and 100 Mi to as high as 550m CPU and 1000 Mi. Attempting to place a smaller resource limit of 100m CPU/500 Mi on the webserver pod causes it to be OOM killed.

What are your Helm values?
Only default values.
What is your Kubernetes Version?:
1.20
What is your Helm version?:
3.50

@AlexDHoffer did you ever figure out why this is happening?

@thesuperzapper I didn't, but no matter what I did with any form of Airflow in Kubernetes it chewed up resources like crazy. Eventually I may return to try it out again but for now the issue is stale because I don't have time to look at it.


maxHashLength doesn't work

Here is build.sbt
...
scalaVersion := "2.11.11"
libraryDependencies ++= Seq(
 ...
  "com.typesafe.scala-logging" %% "scala-logging" % "3.7.2"
)

assemblyOption in assembly := (assemblyOption in assembly).value.copy( appendContentHash = true, maxHashLength = Option(5))

The result jar looks like this:
leaddeduplication-assembly-1.0-4a26b5b1a5150bcca3724d76d9d7e0b79f2d3955.jar
Expectation:
leaddeduplication-assembly-1.0-4a26b.jar
Version:
addSbtPlugin("com.eed3si9n" % "sbt-assembly" % "0.14.5")

Now it says
[error] Not a valid command: assembly
[error] Not a valid project ID: assembly
[error] Expected ':' (if selecting a configuration)
[error] Not a valid key: assembly
[error] assembly
[error]         ^
both for


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2021-04-16T18:08:36+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio_release-1.8
a07bc6a02c499a14860e9c5103febc3566e911d2
link
/test integ-telemetry-k8s-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


[texlive] Adding Gregorio

Issue description
I’d like to contribute a Nix expression for Gregorio — tools for beautifully typesetting Gregorian chant.
It needs to plug itself into a TeX Live installation.
I already have a working expression here — gregorio.nix in michalrus/dotfiles@2183df5 — but plugging into our pkgs.texlive.combine seems a bit… hacky (see the part in my notebook.nix and the last lines of gregorio.nix above).
@vcunat, how should I improve that and where should gregorio.tex be put inside the <nixpkgs> tree?
Technical details

System: 16.09pre84674.fbde3a7 (Flounder)
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: "16.09pre84674.fbde3a7"

I see gregoriotex-4.1.4 in the latest texlive upstream, so this is just one piece of the bigger problem: our texlive state is around a year old, and quite a few users mind that.
IIRC there is at least one PR trying to (partially) resolve that. Lately I haven't been motivated to invest much time into TeX and others with merge privileges don't seem to step in a lot, so the state is lagging a bit. I see texlive-2016 was released about a week ago, so within a couple weeks I might get to updating to that version.

Oh, great to hear that! :heart_eyes_cat:  Thank you!
And then, if I wanted a newer version, I’d just override one package’s version & hash? Pretty cool!

I don't know of any easy overriding way except for modifying the nixpkgs tree.

Oh… :disappointed_relieved:

so within a couple weeks I might get to updating to that version.

Now it doesn't seem too likely I'll manage it that soon.

I was wrong: https://github.com/NixOS/nixpkgs/pull/16391

Awesome! :heart_eyes_cat:

Nice, fixed & closed.


Circle is broken

I made a View end set the Custom Class to CircleProgressView. Unfortunately something went wrong it looks like this.

I got it right on one screen. But when I change screen sizes I get hard issues. it looks like the picture above or even worse.
How can achieve the circle looking good on all screens?

after doing aspect ratio 1:1, you can't center the circle correctly anymore.

its alright now. thank you for your nice circleProgressView


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.16 to before v4.14.241",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=2d53139f31626bad6f8983d8e519ddde2cbba921",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=edddc79c4391f8001095320d3ca423214b9aa4bf"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "2d53139f31626bad6f8983d8e519ddde2cbba921",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "edddc79c4391f8001095320d3ca423214b9aa4bf",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "usb: max-3421: Prevent corruption of freed memory\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.241 by commit edddc79c4391f8001095320d3ca423214b9aa4bf, it was introduced in version v3.16 by commit 2d53139f31626bad6f8983d8e519ddde2cbba921. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001401


Show callout with secondary actions

Hi danielsaidi,
First of all I thank to you for great repository.
I try to use your repo and in tests i didn't find a feature. I upload an image about this feature. In Default keyboard when i press to letter is growing as in the uploaded picture. But in your repo I didnt find this. Has your repo include this feature?. Also yours select letter when touchupinside but i think in default touchupOutside. Thanks for answer and for all.

Hi @aliihsanural
This feature is not included at the moment, but it's on top of my list to do whenever I get time to work on this project again.

Hi @danielsaidi
if you want using Objective-C framework inside the KeyboardKit, I can recommended for aforementioned feature in this link: https://github.com/illyabusigin/CYRKeyboardButton
All the best!

Thank you! I will have to port this to SwiftUI though, since that is the future of this library.

Hi danielsaidi,
I really appreciate to hear that our demand is in your plan. I hope this feature will come as soon as possible 🙏.
Thank you so much again.

Hi @aliihsanural - yes, I will try to find a nice way to implement this. One question, do you use UIKit or SwiftUI?

Unfortunately I use UIKit 😔.

There is another issue for this, where a UIKit implementation is discussed. Look here: #93

@mustafaysil I had a look at CYRKeyboardButton and while it looks great, there are a bunch of drawbacks:

It's written in obj-c and hasn't been updated in 6+ years
It's based on a TurtleBezierPath lib that hasn't been updated in 7+ years
It has quite a lot of code that would have to be ported

However, the end result is great, so it's a shame that it hasn't been updated and isn't a standalone lib.


refactor(invoker): implement HttpInvoker

Issue
https://gravitee.atlassian.net/browse/APIM-7417

📚  View the storybook of this branch here

The Integration Tests step is failing but it is perfectly normal since we modify a method from DefaultApiReactor preventing the message reactor to inherit from it -> the plugins relying of this override from reactor are not taking benefits from it and so tests are failing.
Those tests should be fixed when the message-reactor next PR will be merged


Update Package.toml

Changed URL from user to organization.

In case it helps, here is the package with the new url: https://github.com/JuliaWaveScattering/EffectiveWaves.jl
and I am sadly the only contributor


Import Modular SCSS and $themeColor variable

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 5.2.0

Platform/Target and Browser Versions: macOS 10.15.1

Live Link or JSFiddle/Codepen or website with isssue: https://codesandbox.io/embed/great-heisenberg-f7uyy

What you did
Importing all SCSS node_modules/swiper/components components to load only what i want. See _swiper.scss : https://codesandbox.io/embed/great-heisenberg-f7uyy
Also, i wrote (& commented) a dirty fix, i would avoid using it :D
Expected Behavior
Navigation arrows should get the $themeColor color.
Actual Behavior
Navigation arrows stays black.
Thanks 🙌

This seems more of a styling issue with your code, rather than a swiper problem?

Ive fixed your code here @proov
add this to your swiper.scss file
// This is the fix
.swiper-button-prev,
.swiper-button-next {
  &:after {
    color: $themeColor;
  }
}

@proov I'm with you. I feel like the line in components/core/core.scss should be interpolated:
--swiper-theme-color: #{$themeColor};

If I make that change in my src (or use your "dirty" fix) then it works exactly as expected.

@proov I'm with you. I feel like the line in components/core/core.scss should be interpolated:
--swiper-theme-color: #{$themeColor};

If I make that change in my src (or use your "dirty" fix) then it works exactly as expected.

Yeah that's what i think too 😅

Fixed by #3334


Sub menus are full-width

Hello.
The dropdown menu now has full width sublinks. Is this correct?

Hello Mentolnet,
In WBST there is an optional mega-menu -- if you don't like this, simply delete the meganav code in the bst.js and the bst.css. (Similarly, the menus drop down on hover -- this is optional too -- you can delete hovernav from the bst.js and the bst.css). I did not modify Bootstrap itself.
If you have navbar 'children' dropdowns, then the dropdown happens in the normal Bootstrap way:
... Parent 1 ...
    Child 1a
    Child 1b
    Child 1c

Remember, Bootstrap navbar does not allow 'grandchildren' links. But meganav does allow -- by creating a full-width menu (a mega-menu). The 'child' links are rendered bold.
... Parent 1 ...
    Child 1a             Child 1b             Child 1c
    Grandchild 1ai       Grandchild 1bi       Grandchild 1ci
    Grandchild 1aii      Grandchild 1bii      Grandchild 1cii
    Grandchild 1aiii     Grandchild 1biii     Grandchild 1ciii

I added this meganav because some people need to create big WooCommerce shops with many items in different 'departments' or categories. Example
https://econsultancy.com/blog/65357-mega-menu-design-trends-in-ecommerce-2011-vs-2014/
If you don't need this, delete it, as I describe above.
Enjoy!
Simon

Hi,
No I don't know how to do this, because (a) I can't know/calculate where the left edge of your dropdown will be (this depends on position of its parent link), and (b) I can't know/calculate where the right edge or your dropdown will be (this depends on the number of columns in the mega dropdown; plus you will need to stop it going off the right edge of the screen). Probably this can be done from a zero-start (not starting with Bootstrap .navbar). Buy my meganav is only a simple override of Bootstrap. It is useful of you have lots of child 'columns', but not if you only have a few.
Maybe you can forget the Bootstrap .navbar and use one of these instead:
http://www.maxmegamenu.com
http://www.designchemical.com/blog/index.php/tutorials/wordpress-tutorial-create-your-own-custom-mega-menu-styles/
http://www.wpfreeware.com/tutorial/wordpress-mega-menu-plugins-for-better-navigation/

Hello Simon and thank you again for your answer and examples.
I´ll give them some attention and see if i can manage this.
All the best!

Thanks. Sorry I couldn't be of more help.

On 23 Oct 2015, at 12:36, mentolnet<EMAIL_ADDRESS>wrote:
Hello Simon and thank you again for your answer and examples.
I´ll give them some attention and see if i can manage this.
All the best!
—
Reply to this email directly or view it on GitHub.


Add nix flake and rust toolchain

This PR provides rust-toolchain.toml to fix the toolchain used for builds, which is a nice thing to have for replicable builds. Moreover, it adds flake.nix for nix users that can be used to use the project directly, and to contribute.

@wcampbell0x2a Hello! I would like to contribute by adding nix support then no_std, but I do not understand why the check fails. Isn't this because of some credential?

@RatCornu I'm sure it's because of some permissions problem. Thanks!

Thanks!


AddMessage with no animation

According to the code, I can't find a way to add and present a new message without animation.
On class NMessengerViewController I would like to override addMessageToMessenger method, but it is specified to don't override.
I evenly tried in this way:
override func addMessageToMessenger(_ message: GeneralMessengerCell) {
    message.currentViewController = self
    super.messengerView.addMessage(message, scrollsToMessage: true, withAnimation: .none)
}

In this way rows are presented with no animation, but some times cells have a broken UI

@velthune What you mean by broken UI? Post some screens


Preview  is larger than image on higher resolutions

Originally posted at gohugoio/hugoThemes#695.

That is what is happening. It gets larger as the screen gets wider.

I confirm this bug and I will look into it in the coming days.

Commit 0c477f6 has fixed this issue on my end @pacollins
https://themes.gohugo.io/
Let me know if the issue remains on your end.


Adjusted display offset

There was an empty line on the side of the screen so I copied the offset values from the M5Cardputer-UserDemo

Thanks for the pull request.
Do you have an image of the display as it appeared when you identified the problem, and as it will appear after the fix, that I can compare?

Yeah sure
Before:

After:

Thanks for testing.
However, when I apply the patch offset to (40,52), there is an area on the bottom line that is not drawn as shown below.
This is a problem I noticed myself when I implemented the patch with (40,52) before. So I finally adjusted it to the current offset.

Ah I see. I played around with the values a little bit and for me (40, 53) works the best.

For 240x135, it appears that the offset should be centered at 320x240.
Horizontal offset value :  (320-240)/2=40
Vertical offset value : (240-135)/2 = 52 or 53
When scanning from bottom to top in Landscape, the offset is 52, but when scanning from top to bottom, the offset is 53. It looks like the correct offset for Landscape(true) is (40,53).
Can you fixup the PR with this value?


Make Path receive input only on the path bounds

We discussed adding a test case for this. Not sure if you still plan on doing that.

Done


No search option while loged out

We are not able to search through the pictures if we are not loged on. Search is only visible after logging in. We use the latest version.

Hello :)
I would like to have a search function for all my users, too.
Perhaps now is the time? ;)
Thanks a lot for your great gallery. :+1:
Julius

Hi, this would be an interesting feature. I would like to see it as an opt in :-)
Cold we speed up the implementation with a donation?
Thanks anyway for this nice software :thumbsup:

I needed the same feature. This fix below worked for me. Just add an extra input element into the public header section:

        <svg class="iconic"><use xlink:href="#account-login"></use></svg>

      <input class="header__search" type="text" name="search" placeholder="Search ~@�">

Yeah i ran into the same question, is it possible to search without being logged in?

The demo is modified version where the user is always logged in.

This is a real shame as the rest of it is pretty great. I spent several hours setting this all up and testing it out only to find out that demo feature doesn't work as presented. Please make it clearer so nobody else who needs this wastes their time. Thanks.

This is literally the only thing preventing me from using this otherwise great software. It would be nice to have an admin user and then everyone else just goes to the site and searches.

I wonder if you would consider some sort of fund raising for features?
I'd pledge some money to get this for example. You could set a goal amount and maybe 4 or 5 people could chip in for a few hours of your time?

This issue will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community. Important issues have been migrated. Feel free to re-add your issue(s) when necessary.


XKCD style passwords

Implement XKCD style password in the generator.
http://xkcd.com/936/

Decided this is going to grow the project too far in terms of size on disk to enable what is basically a\ joke feature


XML based Spring configuration

It is not clear whether spring-vault supports configuration via Spring XML.  If it does, the documentation should be enhanced to make this explicit and provide examples on how to configure via XML.  If it does not, I propose that XML support be added.
I know Java config is all the rage these days, but basic support via XML should be provided as well.

Spring Vault has no XML configuration component and I don't think that XML config is going to happen unless we find very good reasons. A couple years ago, things were a bit different. We provide EnvironmentVaultConfiguration for a minimalistic and convenient configuration approach.
Care to elaborate why you prefer (require) XML over JavaConfig?

We have a large number of applications built with Spring, which predate the Java config capability.  The time and risk involved in completely rewriting the bootstrap/config code is substantial.  Lack of XML config support makes this project a non-viable option for would be adopters who must support existing codebases.

You're not required to rewrite your config to JavaConfig but you can keep parts as Java config and pick it up from the XML config with <context:component-scan base-package="com.example.config" />.

Oh, I didn't realize you could integrate like that.  I think that should do.
Thanks!

We decided to not provide XML-based configuration hence closing the ticket.


fix broken build when cmake/rostest cannot find the gmock library

Description of changes:
Mimic the way the CMakeLists.txt is written at https://github.com/aws-robotics/health-metrics-collector-ros1/blob/master/health_metric_collector/CMakeLists.txt#L66.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks good to me, tests are running on that build
I see this fix avoids failing when google mock it's not installed. On Travis we get gmock installed with the current ros-industrial build, and the new travis-scripts will also install it, as gmock is part of gtest now. However I think we should consider adding instructions for running the tests on a dev machine, as with this change tests would be skipped. An option is refactoring travis-scripts to make them easy to use locally, but we can see that later