Exponential Backoff

This would be useful.

I was literally thinking this feature would be amazing! would be great to have, or even if anyone knows any other way to achieve this with another shell tool I'm all ears.


Proposal: New cargo-build mode

Current situation
So right now when we invoke rustic-cargo-build, it creates a new rustic-compilation-mode where it shows us various errors etc.
Now there are two problems with this behavior:

This is a global buffer. If you compile any other rust project, this will overwrite it. This makes it hard to work on two rust projects or even different crates within a single workspace.
Recompile doesn't work as expected if you pass any argument to it. If you press g on the buffer, it will run rustic-recompile which doesn't actually read rustic-cargo-build-arguments.

Proposal

Build a cargo-build mode on top of rustic-compilation
This would be crate specific with buffer local variables.

Any objections ? I'm willing to implement this in my spare cycles if there is no objections.

This is a global buffer. If you compile any other rust project, this will overwrite it. This makes it hard to work on two rust projects or even different crates within a single workspace.

I think this is the default compile.el behevior right ?

If you press g on the buffer, it will run rustic-recompile which doesn't actually read rustic-cargo-build-arguments.

recompile can also be used if not in the compilation buffer, which is an advantage of the global approach. Also rustic-recompile was supposed to be only used for commands that were run with rustic-compile. The other commands came later.
I think it's difficult to simply change this behavior after years and users have different preferences. And if we choose this approach we would also have to consider other cargo commands for consistency reasons. Do you think it would be a lot more work to let the user choose the behavior with an option. Maybe we even need an additional command like rustic-recompile-cargo-command that checks which cargo command was used so the right arguments will be used. If it's only about cargo-build it should also be enough to add a rerun command like rustic-cargo-run-rerun.

But I can understand that it can be convenient to have this buffer local and most users will also use it this way so it makes absolute sense for rust with its workspaces.

I think that makes the most sense :+1:
Sorry for the delay.


It is better to get search history on Search menu.

It is better to get search history on Search menu (like eclipse have).

/duplicate Add dropdown for search input history


ci(release_cli): add homebrew formula bumper

Summary
Solves #568
This PR extends the release_cli workflow to automatically create a version bump PR on homebrew when we publish a new stable version of the CLI.
@biomejs/core-contributors Because the GitHub Action creates a fork (in the biomejs org) and a pull request, we will need to use a personal access token with the following permissions: public_repo and workflow. Can any of you guys create such PAT and add it as a secret on this repository with the name HOMEBREW_BUMP_PR_TOKEN?

Done! I saved the secret in the environment package_manager


Split Angular section in Front-end.md into two sections

AngularJS and Angular 2+ have massive changes that break the library into two user basis. Most resources in the section were only for AngularJS.  This PR splits the section into shared, AngularJS, Angular 2+ to avoid losing those resources, but also keep them separate to avoid confusion.

@Ishaan28malik could this get a hacktoberfest-accepted label if possible? Need to know if I need more hacktoberfest specific PR's.


Update installation README instructions once bootc is out on podman desktop catalog

Is your enhancement related to a problem? Please describe
Installation instructions include copy-and-pasting the bootc extension link to Podman Desktop
Describe the solution you'd like
We should show the instructions on how to do it via podman desktop catalog instead.
Describe alternatives you've considered
No response
Additional context
No response

Closed via https://github.com/containers/podman-desktop-extension-bootc/pull/458


concurrent map writes when debug

It will  encounter concurrent map fatal when the Engine LoadDirectory with development .

`fatal error: concurrent map writes
goroutine 11488 [running]:
runtime.throw(0x1397be7, 0x15)
C:/GO/src/runtime/panic.go:566 +0x9c fp=0xc0690ceda0 sp=0xc0690ced80
runtime.mapassign1(0x11bca20, 0xc0422c50e0, 0xc0690cefd0, 0xc0690cef78)
C:/GO/src/runtime/hashmap.go:458 +0x8f6 fp=0xc0690cee88 sp=0xc0690ceda0
github.com/kataras/go-template/django.(*Engine).LoadDirectory.func1(0xc060a67ea0, 0x43, 0x1e18280, 0                                                                                xc068b82720, 0x0, 0x0, 0xc064cb9790, 0xc0690cf0f8)
D:/Framework/golang/src/github.com/kataras/go-template/django/django.go:94 +0x3e2 fp=0xc0690                                                                                cf030 sp=0xc0690cee88
`

This probaply happens only when you try to use goroutines to load your templates/assets, thing that you should never do, let the templates load before your server's run, in order to catch any panics before serving pages to the client. Anyway this is fixed now for any case :) thanks for the report


Wiki documentation needs to be overhauled

Quite a bit of documentation in the wiki is outdated or not up-to-date.
Things that have to be included/updated:

Everything regarding new config reading

how to update old configuration to the new one (probably download new templates project or update custom templates)
changes in config files (or a new guide to config file(s) altogether)

Acceptance Criteria Sprint 3:

[ ] Create list of needed changes within documentation

[ ] Create list of needed changes within documentation

In the wiki sections Configuration, Eclipse Integration, Core Plug-ins, the following problems are currently there
core-Configuration
Configuration
- monolithic templates are assumed, examples show single context.xml in root folder
Context Configuration
- MENTION NEW CONFIGURATION AFTER COBIGEN VERSION X
- explains monolithic structure
- more than v1.0 xsd versions exist (first paragraph also confusing)
- trigger configuration mentions templateFolder
- add variableAssignment mandatory attribute
- remove incrementRef mentions
- rewrite java template logic
cobigen eclipse installation
Installation
- phrase the remark better (cobigen installed in eclipse)
- check necessary eclipse versions
- update installation screenshot to install every plugin so stupid mistakes don't get made
- Redo point 6, remove mini tutorial and reference a proper one instead
- Generate never greyed out
helpful links
- add explanations to the links
java plugin
- type 'expression' better formating for valid expressions
- do model restrictions still exist?
openapi plugin
- update plugin description ("until now" kind of outdated
- update note to add template developers
- rewrite to describe it as an existing plugin instead of a completely new one
- add mandatory extenstion explanation
- fix screenshot

Some further problems that need to be changed (list still not complete):
Overview section
Selection of current and past CobiGen applications

Generation of EXT JS 6 is not supported anymore
Generation of Angular client uses meanwhile Angular 13
Mention that generation of an Ionic Client is supported

Guide to the reader

Links behind “licensing of CobiGen” and “Shared Service” pointing to not existing documents

General CobiGen Use Cases

Link behind “Register Factory” not working
Link behind “devonfw wiki” not working
Mention that client application with devon4ng also supports Ionic clients

HowTo’s
Angular Client Generation

Change Angular versions (meanwhile devon4ng uses 13)
Change workspace image to show new template structure (context.xml in template folder)
Update last image that shows “ng serve -o” (Angular CLI version out of date)

Ionic Client generation

Link behind “You will find an example here” not working. Project with TableEto does not exist anymore, so we have to choose an other example
Image showing how to generate from Eclipse should be updated to match with the example

Enterprise Architect UML diagram Generation

Rework the “Prerequisites” section. Link behind “accumulative patch” not working (not sure why this link should be used). Steps to get the latest templates should be changed

CobiGen CLI Manual

Rework commands section, some of the parameters are out of date. Suggested structure:

generate,g: …

inputFiles: …
--increments, -i: …
--templates, -t: …
--out, -o: …

adapt-templates: …
update, u: …

--all: …

--version, -V: …
--help, -h: …
--verbose, -v: …
--templates-project, -tp: …

Update image for generate command

Option -cl in adapt-templates command not working anymore

Update CobiGen

Images show old versions. Use actual screenshots

.Net generation

The guide is the one from the devon4net repository. https://github.com/devonfw/devon4net/blob/develop/documentation/howto.asciidoc#devon4net-cobigen-guide. But the CobiGen documentation still have an old version of this guide. So that guide has to be replaced with the one from devon4net

Release creation:

Link behind “script” not working
Complete section needs to be updated. There is not python file anymore
Describe properly how to build a release with the release.sh file

External plugin creation

Section “templates creation” needs to be updated to match with the new folder structure (update images and paths)

CobiGen development
Dev IDE setup

Link behind “setup the devonfw IDE” not working
Quotation sign in step 8 not used correct
Mark text from buttons or checkboxes in step 9 properly
Link to contribution guide not working

Dependency and Licence tracking

All the dependencies and their versions have to be checked. Some of them are outdated
Versions of the current release have to be changed

Setup JRE for development

Images are not displayed.
There is no folder “/software/java/additionalJdk”

Implementation Docs
Eclipse-Plugin Development

Link behind “Plugin.xml” not working
Link behind “core development” not working
Plugin.xml is out of date. Now there are four commands and handlers
Introduction sentence in section “Handlers” says that the package workbenchcontrol contains handler for the two main use cases (generate and healthcheck). But after this all four handlers are explained. So we should mention all handlers in the introduction sentence
Explanations for “Update Template” and “Adapt Template” are very badly formulated
Section about GenerateHandler needs to be updated. XmlGeneratorWrapper does not exist anymore. Instead there are FileInputGeneratorWrapper and JavaInputGeneratorWrapper. UML diagram needs also to be updated
Section about HealthCheckHandler needs to be updated. The handler does not call the execute method from HealthCheck.java. It will call execute from HealthCheckDialog, which call perform() from HealthCheck.
Also, the source code snippets are out of date
Wizard Development:
First link does not work. Code snippets are out of date

Here are all the broken links in the documentation:
HowTo’s
Iconic client generation

Generation

Third and fourth link behind “here”

Enterprise Architect UML diagram Generation

Generation

Third and fourth link behind “here”.

Hi everyone,
this issue will be closed, because I have created an excel file with all the broken links in the CobiGen documentation that should be fixed. I also created another excel file with all above mentioned points with another new found point as well as some general notes that should also be fixed in the CobiGen documentation. The excel files have been shared in teams with the other colleages and new issues will be created to fix the mentioned points.


Create a PUT request to update a todo

please assign me
https://github.com/i-am-popo/CRUD-API/issues/1
https://github.com/i-am-popo/CRUD-API/issues/2
https://github.com/i-am-popo/CRUD-API/issues/3
https://github.com/i-am-popo/CRUD-API/issues/4

hey @i-am-popo can you assign this issue to me i can work on it


How to build it in Windows?

Hi this is a very helpful project, what changes needed to build this in windows too?

I do not use Windows at all.
As written here:
https://github.com/PatWie/tensorflow-cmake/blob/c5032890978b9bc26b9d5bdf2787e8552d17cebb/cmake/modules/FindTensorFlow.cmake#L49-L53
I wonder if this would work in Windows. But I guess it is not working without changes. Did you gave it a shot under windows?


terminating with uncaught exception of type NSException (lldb)

Hey Patchthecode,
Im new to coding so I may be creating a silly mistake but everyone and a while I get the posted error above. I started with your cocoa pod this morning and tried the posted file so I ran with that and as I slowly started to form it into what I needed I got this error. I then decided I was just going to delete it all and restart with your videos, thank you for those by the way, but once I got through it and started to add in secondary labels like in your second video and thats when the error came back. I think its something that I keep missing. If you have the time would you be able to look and see what you can find?
using the latest updated JTAppleCal
swift 3
latest cocoapods
calendar.txt
Thank you.

Hey, it would be nice if you can just zip a sample project with the error?
I will be able to tell you the problem instantly.

SchedCred.xcworkspace.zip

@oeufs yuou have send me one file by accident
I think you send me the entire folder?

thats the project file.

Hey, can you send me the project folder?
An XCode project is not just the one file alone.
There are also the other swift classes that go along with it.

SchedCred.zip
sorry thought you just wanted to project file. Thats the entire folder

Is your app crashing with this error?
2017-06-18 11:43:05.727617-0700 SchedCred[49527:22189303] Unknown class daysTasks in Interface Builder file.
2017-06-18 11:43:05.762211-0700 SchedCred[49527:22189303] *** Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<SchedCred.UserCalendarViewController 0x7fdd4bc05980> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key date.'
*** First throw call stack:

It actually says why its crashing. You can see it here ->
this class is not key value coding-compliant for the key date.'

Upon looking at your IB file you can see that there are two things you are not setting correctly.
They are shown with the exclamation marks. You have two of them.

It seems you have renamed month to currentMonth and date to currentDate but you forgot to delete the outlet on interface builder.

Your awesome!! Thanks, I figured it was a little mistake that I was making but it kept reoccurring.
Have a wonderful day!

Have a good one.
cheers 🍻


Discussion for ARC-0018: ASA Transfer Royalty Enforcement Specification

This is the new page of discussion for ARC-0018.
Moved from #70

I haven't fully read the new ARC but one comment I have first is the following.
The current ARC constrains the design of royalty in two ways:

It forces the keys used in local and global storage
It forces the way some functions (such as royalty_free_move) are permissioned: namely must be called by some specific administrator.

I think that removing these restrictions would allow more flexibility for the contract designer while still allowing clients (wallets, exchanges, explorers) to use the ARC.
Concretely:

Forcing the keys limit the number of ASA that can be handled while better schemes could be used compressing data in the value.
Forcing the existence of a single administrator account means that setting policy and royalty free move need to be performed by the same party. But we could imagine that we want to use two different parties. Or that "cheap" assets can be royalty-free move with a less secure account than "expensive" assets. In theory, we could allow all of the above by making the administrator an application account. But this looks to me like more complex than just not specifying the permissioning mechanism.

Another way to say it is that I think the way ARC-0020 is written allows more flexibility while providing similar value: it does not specify how permissioning works nor how storage works.
What do you think @barnjamin @cusma @wheresaddie?

@fabrice102 I agree, since ARC-0018 requires ARC-0020 I think is worth sharing the same approach here and allow for more flexibility, leaving those decision to the reference implementations (so not mandating them in the standard itself).

After reading the documentation and discussion regarding ARC-18 I am comfortable to say that it already seems to be the best solution to give NFT artists/creators the opportunity of choosing Royalties Percentages for their artworks without sacrificing marketplace interoperability. I understand that what is being proposed and developed (please correct me if I'm mistaken) is an ASA standard that forces a pre-established percentage of all resells of an NFT to be sent to pre-established wallet address(es), being those pre-establishments made exclusively in the moment of creation of said ASA by the creator, not being susceptible to changes in the future by anyone, including even the ASA creator.
Since the proposal is to implement an ASA standard that has much potential of being mass adopted by artists and creators to mint their NFTs, it is extremely important to keep the already existing conveniences that ARC-3 and ARC-69 brings, which are exactly what enables so much freedom for creative utility and marketing strategies in Algorand’s NFT ecosystem, while giving collectors a truly satisfying sense of ownership. I felt the necessity of pointing out the following scenarios in order to raise awareness to the possible limitations that the current ARC-18 could bring, which in my opinion would be enough for it not being used at all.

Direct ASA Transfers between wallets shouldn't configure a selling, thus it shouldn't be necessary to Enforce Royalties in this case. I'm pointing out three scenarios where needing to pay royalties would make ARC-18 very inconvenient:

1.1 Using a hot wallet to buy an NFT and then Transferring it to a safe wallet of your ownership;
1.2 Giveaways;
1.3 Gifting someone;
I couldn’t figure out the necessity of Enforcing Royalties when directly transferring assets, only if atomic-swaps are involved. I apologize if those scenarios were already thought out and I just couldn’t figure out by reading the doc and discussion. I’m not a code developer, only a community member which have much interest in Algorand’s NFT Ecosystem. With that said, finally I want to congratulate all developers involved in this discussion and development so far! Thanks for always bringing Algorand to a next level.

Hi @gbrgmanera good points and worth considering!
In this spec we've added an "escape valve" for doing transfers royalty free, namely the royalty_free_move method. This method will allow for transfers without enforcing a royalty payment.
It would be up to the application creator to determine how exactly to offer this method generally. It's hard to offer an API that takes into account the intent of the sender and the acceptability of a transfer sans royalty by the creator so I think it'd have to be on a case by case/application specific basis.
That said, it may make sense to demonstrate how an application may want to implement such behavior so that creators can approve (maybe automatically) certain types of transfers.

@barnjamin @gbrgmanera I think that royalty_free_move could be easily used to realize giveaways logics. What is harder, in my opinion, is adopting that same method to realize transfers between wallets belonging to the same owner, since it's more difficult to prove that two different public keys belong to the same user. This being said I agree this use case should be solved by specific ARC-18 implementations rather than by the ARC-18 standard directly.

Has any thought been given to the feasibility of using a single contract for multiple ASAs. For instance, let's say I mint 1000 assets that I all want to be controlled by the same royalty contract. It would be very convenient from a cost and administration perspective to be able to deploy the royalty contract once and then set the clawback for all 1000 assets to that contract. I get this is probably more a discussion for ARC-20 since this is associated with that though?

@robdmoore  With the current ARC18 spec the expectation is that any number of ASAs may be managed by a single application. While ARC18 is meant to be an implementation of ARC20, the current ARC20 reference implementation only allows for a single ASA to be managed so it will need to be tweaked to allow any number
Current ref for arc18
https://github.com/algorand-devrel/royalty
and some WIP references using beaker
https://github.com/algorand-devrel/beaker/tree/master/beaker/contracts/arcs

@robdmoore ARC-20 does not a single-Smart-ASA-per-app, but is open to a multi-SmartASA-per-App in principle. The current reference implementation of ARC-20 allows a single Smart ASA per App due to limitation in Global Storage that could be potentially overcome once Boxes are available. But again, ARC-20 is open to this possibility, is just a matter of implementation complexity.

Why basis points (uint64) as opposed to Arc4 ABI Type Fixed Point?

basis points seem good enough to convey the requirement and dont need any special codec

I have a point to raise on this.
The royalty Receiver can be any account ( App, LSig, wallets)

Royalty Receiver - The account that receives the royalty payment. It can be any valid Algorand account.

Usually, creators do not work alone.
If I want the whole team to be paid at once, there are 2 possibilities.

I create a Lsig/App that will be the royalty_receiver, and with a crontab, I call the Lsig/App to split the payment.
Pros:

We keep the arc as is. Easier for the marketplace to implement it
Cons:
We add friction to people who need this kind of feature

We add a new *OPTIONAL method after_transfer_call that needs to be called at the end of any transaction. This method might call a "Payment_Splitter" or any Application that needs to be called at the end of a transaction.
Pros:

Creators should not have to worry about setting up a server or running the application manually
Cons:
More work for the marketplace

I think 2 is better and should be considered to make Smart Asa ARC-20 Smarter.

As per Royalty Receiver description:

The Royalty Receiver is then responsible for any further royalty distribution logic, making the Royalty Enforcement Specification more general and composable.

I'm inclined to keep the relationship between the Royalty Enforcer and the Royalty Receiver modular.
There could be different "royalty distribution logic" depending on: distribution time schedules, minimum distribution amounts, different distribution shares among creators, etc. etc.
Since each project / NFT / marketplace could have its own distribution logic I would keep the implementation and the requirements of a Royalty Receiver (and eventually Distributor) separate, giving to the marketplaces, minting platforms, wallets, etc. the possibility of managing the distribution as they prefer (e.g. on a crontab time basis, etc.).
So for example: Marketplace XYZ will use a Royalty Enforcer to collect 3% on each trade in the Royalty Receiver. The Royalty Receiver is then programmed to redistribute 1% to the Marketplace XYZ and 2% to the Creator if and only if the Royalty Receiver balance is greater than a given minimum_distribution_amount (to save fees).


Support for /sys/class/leds/red/brightness instead of backlight

OpenWrt for Lumi gateway will migrate to new drivers of LED lights soon.
https://github.com/devbis/openwrt/commit/1a3adc95ccee91cad8766e413cc038cd5fb1b778
The new device paths will be:
/sys/class/leds/red/brightness
/sys/class/leds/green/brightness
/sys/class/leds/blue/brightness
And range 0-255 instead of 0-100.
Do you plan to add support for this schema in your application?
Proposal: you can check for the new paths and fallback to the old ones if they are missing

Отлично. Так даже проще с яркостью работать...

Отлично. Так даже проще с яркостью работать...


[zlib] build failure on wasm32-emscripten triplet

zlib build failure on wasm32-emscripten triplet:
try on linux ubuntu wsl2
Before change of today it was working.
Installing 4/7 zlib:wasm32-emscripten...
Building zlib[core]:wasm32-emscripten...
-- [OVERLAY] Loading triplet configuration from: /home/tal/boinc/3rdParty/vcpkg_ports/triplets/ci/wasm32-emscripten.cmake
-- Downloading https://github.com/madler/zlib/archive/v1.2.12.tar.gz -> madler-zlib-v1.2.12.tar.gz...
-- Extracting source /home/tal/boinc/3rdParty/wasm/vcpkg/downloads/madler-zlib-v1.2.12.tar.gz
-- Applying patch cmake_dont_build_more_than_needed.patch
-- Applying patch 0001-Prevent-invalid-inclusions-when-HAVE_-is-set-to-0.patch
-- Applying patch debug-postfix-mingw.patch
-- Applying patch 0002-android-build-mingw.patch
-- Using source at /home/tal/boinc/3rdParty/wasm/vcpkg/buildtrees/zlib/src/v1.2.12-f2d39b538c.clean
-- Configuring wasm32-emscripten-rel
-- Building wasm32-emscripten-rel
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:129 (message):
    Command failed: /home/tal/boinc/3rdParty/wasm/vcpkg/downloads/tools/cmake-3.22.2-linux/cmake-3.22.2-linux-x86_64/bin/cmake --build . --config Release --target install -- -v -j13
    Working Directory: /home/tal/boinc/3rdParty/wasm/vcpkg/buildtrees/zlib/wasm32-emscripten-rel
    See logs for more information:
      /home/tal/boinc/3rdParty/wasm/vcpkg/buildtrees/zlib/install-wasm32-emscripten-rel-out.log

Call Stack (most recent call first):
  installed/x64-linux/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
  installed/x64-linux/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
  ports/zlib/portfile.cmake:26 (vcpkg_cmake_install)
  scripts/ports.cmake:146 (include)

install-wasm32-emscripten-rel-out.log

Before change of today it was working.

zlib was not changed within the last month.

Before change of today it was working.

zlib was not changed within the last month.

I was compile it before and it was compiled.
Then today/yesterday there was a build world? Because all the packages compile again, then zlib is fail. So something is change in vcpkg in recent 24h or so.

So if you use an commit from 2 days ago it works again?

So if you use an commit from 2 days ago it works again?

I will try it and let you know. This is a good idea.

@autoantwort I compile on my local wsl with commit 2 days ago (91393faf123c4f1d22ef3dbfb4ec03531bac907a) and It fail.
Then tested on our CI on the latest vcpkg and it working. So I delete vcpkg and emsdk from my local wsl and compile from scratch.
So it something local on my wsl. So I am closing this issue, and if there is a problem, I will open it again. Thank you!


194 PathParam 잘못사용

Closes #194

Codecov Report

Merging #195 (a42bf80) into main (bf024a3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #195   +/-   ##
=========================================
  Coverage     49.01%   49.01%
  Complexity      426      426
=========================================
  Files           149      149
  Lines          1934     1934
  Branches         14       14
=========================================
  Hits            948      948
  Misses          980      980
  Partials          6        6

Impacted Files
Coverage Δ

...va/app/jg/og/zamong/controller/UserController.java
36.84% <ø> (ø)

...ontroller/dream/InterpretationDreamController.java
12.50% <ø> (ø)

...g/zamong/controller/dream/SellDreamController.java
5.00% <ø> (ø)

.../zamong/controller/dream/ShareDreamController.java
27.27% <ø> (ø)

Continue to review full report at Codecov.

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


Implement additional references to parent nodes for Type

Description
Implement Type feature: Additional references to parent nodes according to our documentation: https://capact.io/docs/feature/type-features/#additional-references-to-parent-nodes

Handle the relation in Populator

Support GraphQL filtering based on prefix
- Find Types based on prefix of parent nodes (consider refactoring Cypher query to TypeScript)

Update documentation to specify that this feature is implemented

Reason
With this feature, we are able to specify proper Implementation requirements (e.g. see the platform k8s/cloudfoundry example), as well as ability to define third-party Storage Backends in delegated storage.

Status:

I run into a problem with Node 17 and hub-js execution. investigate problem with not working node17 in hub-js
Currently in Hub logs we can see:
(node:18) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./public/" in the "exports" field module resolution of the package at /app/node_modules/extract-files/package.json.
Update this package.json to use a subpath pattern like "./public/*".
(Use `node --trace-deprecation ...` to show where the warning was created)

when using Node 17 this warning becomes error:
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './public/extractFiles' is not defined by "exports" in node_modules/extract-files/package.json

It is a problem with transitive extract-files dependency:
npm ls extract-files                                                                                ☸ gke_capact_europe-west1_capact-stage:capact-system via  v16.13.2 on ☁<EMAIL_ADDRESS>✗
<EMAIL_ADDRESS>/Users/mszostok/workspace/go/src/github.com/capactio/capact/hub-js
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>deduped
    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>
We should use extract-files >= 10.0.0 which fixes the issue, but the neo4j-graphql-js is already deprecated, and I didn't find an easy way to fix that problem.
As a fallback, I change my Node version to 16.

[x] update README.md development section for hub-js

[x] update Cypher to return related nodes (additionalRefs)

Found issues

I run into a problem with Node 17 and hub-js execution. investigate problem with not working node17 in hub-js
Currently in Hub logs we can see:
(node:18) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./public/" in the "exports" field module resolution of the package at /app/node_modules/extract-files/package.json.
Update this package.json to use a subpath pattern like "./public/*".
(Use `node --trace-deprecation ...` to show where the warning was created)

when using Node 17 this warning becomes error:
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './public/extractFiles' is not defined by "exports" in node_modules/extract-files/package.json

It is a problem with transitive extract-files dependency:
$ npm ls extract-files
<EMAIL_ADDRESS>/Users/mszostok/workspace/go/src/github.com/capactio/capact/hub-js
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>deduped
    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>
We should use extract-files >= 10.0.0 which fixes the issue, but the neo4j-graphql-js is already deprecated, and I didn't find an easy way to fix that problem.
As a fallback, I change my Node version to 16.

Fix bug that for current main version (https://github.com/capactio/capact/commit/43c74007d15f748f1210bb965eb2fd4c81fe44cf) where no entries are returned for empty filter:
It("all entries if there is no filter", func() {
	gotTypes, err := cli.ListTypeRefRevisionsJSONSchemas(ctx, gqlpublicapi.TypeFilter{
		// no path filter
	})

	Expect(err).ToNot(HaveOccurred())

	fmt.Println(gotTypes)
	Expect(len(gotTypes)).Should(BeNumerically(">=", 10))
})

Problem: The Go JSONMarshaler marshals nil pathPattern as null which was not handled properly in Cyper query.

Revisions are not supported as we as for Types and we use only pathPattern without revision info. Additionally, additionalRefs - has only NodePath, see:  additionalRefs:
  - cap.core.type.platform

We can do that on client site, but it doesn't go well with our API, so I will log a dedicated issue for it.


Increase in runtime

Hello, first for all, thank you so much for providing this great resource. We have been noticing that for the last couple of months or so the runs have been taking considerably longer, it used to be 6-7 hours, now it's more than 30. Is there a reason for that?  Thank you again very much.

Below is a graph of the average time a process takes. This one is from mid-September. The process that takes the longest to run per sample is Freyja.
There was an attempt to alleviate this problem in version 3.820230926 by requiring a minimum depth. This hasn't really made an impact, however.

I have thought about dropping Freyja boot.

Hi Erin,
Thank you. So I think that the issues started with version 3.8.20230915https://github.com/UPHL-BioNGS/Cecret/releases/tag/3.8.20230915 . Before that things were taking 6-7 hours and in this one they started to take more than 24 hours. It seems that it is the Freyja upgrade and there is not much we can do at the moment correct?
Thanks,
Ramiro
From: Young @.>
Reply-To: UPHL-BioNGS/Cecret @.>
Date: Wednesday, November 1, 2023 at 11:40 AM
To: UPHL-BioNGS/Cecret @.>
Cc: Ramiro @.>, Author @.>
Subject: Re: [UPHL-BioNGS/Cecret] Increase in runtime (Issue #249)
Resent-From: @.>
Resent-Date: Wednesday, November 1, 2023 at 11:40 AM
[This message was sent from an address outside the Larner College of Medicine. Please exercise caution when clicking links or opening attachments from this source. ]
Below is a graph of the average time a process takes. This one is from mid-September. The process that takes the longest to run per sample is Freyja.
There was an attempt to alleviate this problem in version 3.820230926https://github.com/UPHL-BioNGS/Cecret/releases/tag/3.8.20230926' by requiring a minimum depth. This hasn't really made an impact, however.
[newplot]https://user-images.githubusercontent.com/28793649/279720688-df648e36-b8a7-47ca-882f-7e133ed2711a.png
—
Reply to this email directly, view it on GitHubhttps://github.com/UPHL-BioNGS/Cecret/issues/249#issuecomment-1789181904, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADS2KRYTBPOH3PCHFKQTLGDYCJUNRAVCNFSM6AAAAAA6ZMWKOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGE4DCOJQGQ.
You are receiving this because you authored the thread.Message ID: @.***>

If you don't need the results from Freyja, you can change the freyja parameter to false.
In a config file add a params.freyja = false  line, and on the command line it's something like
nextflow run UPHL-BioNGS/Cecret <everything else you normally run> --freyja false

This should skip Freyja altogether?

Thank you very much!  This is very clear and it’s not a problem, we just need to make sure that we are not doing anything wrong and having the option to not do this is helpful. Just curious, do you know the reason why Freyja started to take so much longer after the update?
From: Young @.>
Reply-To: UPHL-BioNGS/Cecret @.>
Date: Wednesday, November 1, 2023 at 12:28 PM
To: UPHL-BioNGS/Cecret @.>
Cc: Ramiro @.>, Author @.>
Subject: Re: [UPHL-BioNGS/Cecret] Increase in runtime (Issue #249)
Resent-From: @.>
Resent-Date: Wednesday, November 1, 2023 at 12:28 PM
[This message was sent from an address outside the Larner College of Medicine. Please exercise caution when clicking links or opening attachments from this source. ]
If you don't need the results from Freyja, you can change the freyja parameter to false.
In a config file add a params.freyja = false line, and on the command line it's something like
nextflow run UPHL-BioNGS/Cecret  --freyja false
This should skip Freyja altogether?
—
Reply to this email directly, view it on GitHubhttps://github.com/UPHL-BioNGS/Cecret/issues/249#issuecomment-1789265186, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADS2KR4ZAXYGF6IWPYHBBDTYCJ2ATAVCNFSM6AAAAAA6ZMWKOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGI3DKMJYGY.
You are receiving this because you authored the thread.Message ID: @.***>

I'm not sure. I didn't even notice until this summer, so I'm thinking it's something with version 1.4.5+.

As an update to this issue, I have created a PR where freyja boot has been removed : https://github.com/UPHL-BioNGS/Cecret/pull/250
This makes the workflow much faster.
As soon as this PR passes testing, we'll merge it and it will be the default workflow.


ViewPagerAndroid lock scroll

ViewPagerAndroid should have a prop that allows us to disable or lock the scrolling ability.

@kevinejohn @christopherdro I have the same requirement, would love to submit a PR if it's alright

@AbilashK I started a pull request for this already. Would love improvements and feedback
https://github.com/facebook/react-native/pull/5968

@kevinejohn I had a look at your PR, can we discuss on discord when you are available?

@AbilashK yeah if you want. How does that work? I use the same username

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/viewpagerandroid-lock-scroll
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Update NN/NT F8BS/F8HS Performance

for Equality size, you set below restriction to get better perf AssertFree0ElementMultiple: 8 AssertFree1ElementMultiple: 8 AssertSummationElementMultiple: 32

Done


Custom Build Support.

I want to try to build a mini version and make the installation package within 30M,I'm not familiar with the current src-next build, would you like to support others' build?

After trying this, it is not possible, and it is not consistent with the main line of kiwi itself.


fix standalone setup errors

Resolves most of #284. Namely: 1, 2, 3. Fixes #254.
The remaining issues will be solved in #286 because they have to do with standalone launching.

@shivaram I tested my changes manually. It works as long as you have instance profile permissions. If you don't, it will error on the very last part of setup.
I also tested the automatic ec2 ssh key addition. Seems to work with all possible combinations of user inputs.

Now resolves #284


ENH: Refactor module

Closes #34.
This refactors the module as discussed in the above issue.

Requires https://github.com/scitran/scitran/pull/64 for scitran to work properly

Thanks, Eric. Very cool.
A couple of things:

Shouldn't we create the scitran.api namespace?
bin/centralclient.py should be executable, not api/centralclient.py.
Isn't a data directory for a single file a bit excessive? I, for one, find it plain annoying.
Please resolve the merge conflicts.

Shouldn't we create the scitran.api namespace?

Yeah, but unfortunately I don't think we can right now. IIRC there are a couple of things that need to import from both scitran.api and scitran.data, but they aren't actually both nested under a single scitran folder currently. So anything that tries to import both will only find one of them, and not the other (which it finds would be dependent upon sys.path order). But it shouldn't be difficult to nest api under scitran in the eventual "big merge" situation once this is in place.

bin/centralclient.py should be executable, not api/centralclient.py.

Whoops! Yeah I git mv'ed the latter, and created the former without changing perms. Fixed with latest commit.

Isn't a data directory for a single file a bit excessive? I, for one, find it plain annoying.

It's overkill for now. I usually do that in case the number of data files grows -- then it's convenient to provide some structure. But for now it's overkill and it can always be created later, so I've removed it.

Please resolve the merge conflicts.

Done.
I deleted my persistent folder and re-reaped some MEEG data, and it still works. So I don't think I broke anything with the changes :)

Cool. Makes sense about the namespace. Unfortunate, though.
If you guys are interested (@Eric89GXL, @kofalt), I'd like to do a Hangout next week to discuss the bigger refactoring plans.
I will be traveling to MSP tomorrow, but should have time to test this from scratch at the airport.
@kofalt Is apps.py still relevant?
@Eric89GXL Can we now safely rename collections_.py to collections.py?

Yeah I can do the renaming tomorrow morning.
Feel free to pick some times that work for you all for hangout. My schedule
is pretty flexible, but I'm on eastern time right now so before 2pm pacific
is better for me but not required.

collections_ moved.
@gsfr don't forget you need scitran/scitran#64 simultaneously when you go to test.

Works!

Not sure why we now have a Makefile to clean stuff.
Wondering if api.wsgi should be in bin. It's basically an executable python file. Is the currently layout typical for WSGI apps?
I don't think we actually have a need for bin/centralclient.py. We always use UWSGI.
@kofalt Is apps.py still relevant? (Restating for completeness, not pressure.)

Not sure why we now have a Makefile to clean stuff.

The Makefile is there because it's nice to be able to do make clean to ensure all the .pyc files have been cleared out, or their Python3 equivalents if eventually this moves to that platform. I can remove it if it seems unnecessary. Usually other commands end up in there, like make test or make flake (I guess make lint in this case probably), but maybe this will end up being redundant with live.sh somehow.

Wondering if api.wsgi should be in bin. It's basically an executable python file. Is the currently layout typical for WSGI apps?

I have never worked with it, so it doesn't really matter to me. If it's primarily used as an executable, which it sounds like it is, it's probably worth moving.

I don't think we actually have a need for bin/centralclient.py. We always use UWSGI.

I wasn't sure what that was used for, so I'm happy to remove it. It's just copy-pasted from some code that was already in there, perhaps it was outdated.

Not sure why we now have a Makefile to clean stuff.

The Makefile is there because it's nice to be able to do make clean to ensure all the .pyc files have been cleared out, or their Python3 equivalents if eventually this moves to that platform. I can remove it if it seems unnecessary. Usually other commands end up in there, like make test or make flake (I guess make lint in this case probably), but maybe this will end up being redundant with live.sh somehow.

I'd say we keep it lean until a real need arises.

Wondering if api.wsgi should be in bin. It's basically an executable python file. Is the currently layout typical for WSGI apps?

I have never worked with it, so it doesn't really matter to me. If it's primarily used as an executable, which it sounds like it is, it's probably worth moving.

Agreed. This will very likely need a matching change in scitran/scitran#64.

I don't think we actually have a need for bin/centralclient.py. We always use UWSGI.

I wasn't sure what that was used for, so I'm happy to remove it. It's just copy-pasted from some code that was already in there, perhaps it was outdated.

Yeah, it's legacy. Let's ditch it.

@gsfr comments addressed with changes here and in https://github.com/scitran/scitran/pull/64

I re-tested locally and things still appear to be okay

I also just retested. Everything still works. I'll be adding a few line comments shortly.

@gsfr comments addressed. You were right about the log, just setting log.setLevel(logging.INFO) made the messages appear in the uwsgi log.

On review this and scitran/scitran#64 make sense to me, what's the status here?
@gsfr Do we want to do a hangout discussing plans?
I'm ready to sanity check + merge these two PRs.

Ready from my end, but I don't think @gsfr has had a chance to look at the changes I made to address his comments.

Thanks for addressing these final things, @Eric89GXL. I'm good to go on this, bar the one line comment I just made.
@kofalt A final sanity check from scratch would be great.
@kofalt Verdict on relevance of apps.py?

@gsfr done

Nuke apps.py. But I wouldn't make that a requirement of this PR. Eric feel free to delete :)

@kofalt @gsfr done, it required removing the corresponding entry from the routes in api.py, I hope that's what you had in mind. If not, I can remove the last commit and it can be done properly in a subsequent PR.
@kofalt did the final sanity check from scratch work for you? It runs fine over here.

I did another test from scratch this morning, including scitran/data#28 and scitran/scitran#64. Everything seems to have worked.
Unfortunately, we have a merge conflict now. Please rebase or merge and then we are finally ready to go.

@gsfr done, I'll re-run it again just to make sure it still works

I'll look at this within the hour :)

Sample data loading and MEEG reaping both appear to work fine over here

Sanity checks passed; merging both PRs.


Update channels room lists

Possible enhancement:
Currently, master server is sending room list only if you join a channel, but if you are just hanging in channel and waiting for the games you won't see that new rooms being created. You will need to leave channel and enter it again.
Maybe need to implement some time interval - send every 10 seconds new room list to everyone who is hanging on channel.

The event of a new room could trigger a room list update to everyone in the room's channel

Please check out https://github.com/leang97/cso2-master-server/blob/master/src/channel/channelmanager.ts
I created a users list which will push update to the waiting users in room list every time someone makes changes to the room settings, it is not the best but it would work right now. You can port the functions to Ochii's original repo.


Do not override a user's font as set under Appearance

I just did a fresh install of the Solarized Theme and the font in edit mode used the mono spaced font instead of the font I selected.
I found a work around by changing line 219 in theme.css. I added the font I wanted to use (which I had also selected in the Obsidian Appearance Menu) before monospace.
font-family: var(--font-'MS Reference Sans Serif', monospace); }

From reading some of the other issues this doesn't seem to be the intended behavior. Other themes work as intended with the selected font showing in edit mode.
Let me know if you need any more details.
Thanks!

Thanks for submitting this issue.
I prefer to use a monospace in editing mode, so that's how that ended up in this theme. Since then, Obsidian have added the option for users to override fonts, so I've removed that part from the theme.
v1.0.9-beta should now respect the font you set in the appearance menu. Could you test this by uninstalling and reinstalling the theme?

I have uninstalled and reinstalled the theme and it looks to be working correctly.
Thanks for looking into the issue.


Update to Swift 3

waiting on ReduxSwift to update to Swift 3
https://github.com/lsunsi/ReduxSwift/issues/1


Sample: Add Bluetooth L2CAP Mesh sample

Hello everyone,
I have been working on several simultaneous Bluetooth Low Energy connections.
However, I encountered problems, mostly assertion errors, while doing a lot of parallel connections.
One example with 20 devices resulted in the following assertion fail:
d_14: @00:00:21.215881  ASSERTION FAIL [err == 0] @ WEST_TOPDIR/zephyr/subsys/bluetooth/host/hci_core.c:305
d_14: @00:00:21.215881  	k_sem_take failed with err -11
d_14: @00:00:21.215881  @ WEST_TOPDIR/zephyr/subsys/bluetooth/host/hci_core.c:305
d_14: @00:00:21.215881 WARNING: Exiting due to fatal error

I think that race conditions due to many connections result in undesired behavior of the BLE stack. Therefore, I built this specific, simplified example that bases on parallel connections: Devices create a sort of connection-oriented BLE mesh. They advertise and directly connect to other devices, sending dummy data over L2CAP channels.
I believe an example like this may aid in solving or even uncovering issues such as https://github.com/zephyrproject-rtos/zephyr/issues/30245 , https://github.com/zephyrproject-rtos/zephyr/issues/36381, or https://github.com/zephyrproject-rtos/zephyr/issues/25917  .
The multi-device environment is simulated using BabbleSim and should be easily tweaked for testing purposes as well.
What would you think of such a sample (or a respective test case)?

@prathje Thank you for this sample, I have not reviewed the implementation, but this sample should definitely help me analyse the cause of the deadlocks. I will post here if I have some results.

@prathje Please cherry-pick #37629 and verify if you still face issue (I have run bsim.sh 20 with no crashes)

@cvinayak Thank you very much for your support! And I hope that this sample could indeed help you.
Unfortunately, I tried to cherry-pick your commit and the other way around, but I could not get the simulation to run beyond the 21.215881 mark with 20 devices. As the BabbleSim runs are deterministic, it should not really be possible that we get different results. Did you run into the same assertion error before applying your fix?

Did you run into the same assertion error before applying your fix?

No, I was not able to reproduce your error on the PR branch, but I got assertions in ull.c and ull_conn.c files when running btsim.sh 11  11 connections.
Maybe, my toolchain makes a difference (archlinux with gcc version 11.1.0 on virtualbox)

Maybe, my toolchain makes a difference (archlinux with gcc version 11.1.0 on virtualbox)

I see. I will investigate that, thanks!

@prathje  Also, I had to remove -defmodem=BLE_simple -channel=Indoorv1 -argschannel -preset=Huge3 -speed=1.1 -at=50 because I got ERROR: (src/channel_2G4Indoorv1.c:380): Unknown modulation type 32 error after 1.41 seconds.

@prathje I had to pull in and rebuild my babblesim library, now I am able to reproduce the hci_core.c: 305 issue. I will analyse the issue further and  post here.

@prathje I had to pull in and rebuild my babblesim library, now I am able to reproduce the hci_core.c: 305 issue. I will analyse the issue further and post here.

Neat! I was just about to send you the closed issue in BabbleSim 👍
Your help is very appreciated!

log_20_5.txt
The above log with added printk indicates that the device d_14 is waiting to Tx a data packet (waiting for num cmplt) and then deadlocks trying to send HCI disconnect.
This appears similar to the issue #25917

Thank you for the investigation!
Once the sample works for a single connection (i.e.CONFIG_BT_MAX_CONN=2 results in L2CAP_MESH_MAX_CONN=1),
one may then try to connect to more than once device simultaneously with increased CONFIG_BT_MAX_CONN values.
Also, the amount of net_buffers can be increased with L2CAP_MESH_PARALLEL_BUFFERS.


Refactor rate storage

Moves rates out of the WALLET storage key to a new dedicated RATE key to ensure that only the minimum amount of data is stored in the WALLET key to avoid running into OS-specific per-key storage limits.

This looks great! Nice refactor.


Optimizing SQL server performance in Docker container

We are having an issue with a very slow start (and generally poor performance) of dockerized SQL server. Sometimes it takes 5+ minutes for a fresh container to start.
We are using SQL server in docker for integration tests during our CI build and everything is running in a Kubernetes cluster.
I think the root cause is that SQL server detects the "bare metal" CPU and does not respect the fact that it's running in a container with restricted resources:
2020-09-22 13:41:39.90 Server      SQL Server detected 12 sockets with 2 cores per socket and 2 logical processors per socket, 24 total logical processors; using 24 logical processors based on SQL Server licensing. This is an informational message; no user action is required

However, this is not correct. We allow only a small portion of CPU (currently 2.5 CPU). I read that number of logical processors has impact on number of worker threads.
I didn't find any param how to change the number of worker threads or the number of logical processors dedicated for the SQL server. The only param I found is for memory (MSSQL_MEMORY_LIMIT_MB). I'd like to do something similar for the CPU.
Does anyone have an idea how to run a SQL server in Docker with limited resources? Let's say 2 Gi of RAM and 2 CPUs at max (we need to scale our builds)? Is it even possible?
Thanks for any ideas!

Bump for notifications.

Bump, also for notifications

When running with CPU limits, any process running inside a container will see all of the CPUs but the underlying Linux kernel will ensure that the process is only consuming up to the number of CPUs limited. You will see this workload across all CPUs on the system...but usage will not exceed the limit. Even with CPU Sets, the process in the container will see all the CPUs, the workload will be scheduled to the CPUs defined in the CPU set. Yes, SQL Server will see all the CPUs.
In my experience with the SQL Server container, the primary culprit of a slow startup time of that magnitude is having larger database as part of the container image. Is that the case here?

Hi, we have discovered that the slow startup issue was in our case caused by poor disk IO performance (shared storage, time to time it was overloaded).
Detecting incorrect number of CPUs might still be an issue but not in this case.

Same here. We run a mssql linux 2019-cu9 container in aks and a simple sqlquery with 12'000 inserts takes 60-80 seconds to run. On an old onprem machine with standard 2019 sql installation it takes 2-3 seconds.

I am in sync with the comments from @nocentino. Using options like --cpuset-cpus , SQL still is able to see all the CPUs but will not be able to use them as Linux kernel will ensure that container is able to use only the number CPUs that is limited by the --cpuset-cpus option.  For the performance issue, can you collect data and troubleshoot this like any other performance issue to understand why a insert query is taking long, one obvious reason I can think of is the storage that is being used for containers. But before you go down that path, I would suggest troubleshooting the performance issue like you would for any SQL issue and identify the waits or any other bottleneck and then troubleshoot accordingly.

You can follow microsofts own sql aks tutorial and you will see the performance issues [1]. You will see the same performance issues when you run local docker desktop with kubernetes and use the same YAML [1]. But a windows vm on the same machine with standard sql 2019 installation ist much faster to execute the query.
[1] https://docs.microsoft.com/en-us/sql/linux/tutorial-sql-server-containers-kubernetes?view=sql-server-ver15
This simple query in a container needs 60-80 seconds to execute.
CREATE DATABASE performance_test

USE performance_test

CREATE Table tblAuthors
(
   Id int identity primary key,
   Author_name nvarchar(50),
   country nvarchar(50)
)
CREATE Table tblBooks
(
   Id int identity primary key,
   Auhthor_id int foreign key references tblAuthors(Id),
   Price int,
   Edition int
)

Declare @Id int
Set @Id = 1

While @Id <= 12000
Begin
   Insert Into tblAuthors values ('Author - ' + CAST(@Id as nvarchar(10)),
              'Country - ' + CAST(@Id as nvarchar(10)) + ' name')
   Print @Id
   Set @Id = @Id + 1
End

select * from tblAuthors

have you tried following the applicable performance guidelines for SQL on Linux as documented here: For SQL containers you can try a tuned profile of throughput and see if you see any improvement.

Not yet, but we tried the following, without any performance improvements.
/opt/mssql/bin/mssql-conf traceflag 3979 on
/opt/mssql/bin/mssql-conf set control.alternatewritethrough 0
/opt/mssql/bin/mssql-conf set control.writethrough 1

To use those settings, you need to ensure that you are not using ephemeral storage for SQL containers, and also the below conditions should be true :

Using SQL Server 2017 CU6 or newer, or SQL Server 2019
Using a Linux distribution and version that supports FUA capability (Red Hat Enterprise Linux 8.0 or higher, or SUSE Linux Enterprise Server 12 SP5)
On storage subsystem and/or hardware that supports and is configured for FUA capability

Else, the default option might be better. If you still do face issues with performance, please reach out to Microsoft support for further help as SQL containers and SQL on Linux is a support feature.

It's a shame that Microsoft can't deliver a good quality database.
Here's an image of resource usage of an instance of SQL Server inside docker:

And what it has? Nothing. Just newly created databases. No data at all. I mean 0 records. ZERO RECORDS and up to 80% of CPU usage? Are you kidding me?
Fix this Microsoft guys. Just fix this. If you can't simply create a new database engine from scratch.
And here's the docker-compose.yml behind this container:
version: "3.9"
services:
    database:
        image: mcr.microsoft.com/mssql/server
        user: root
        restart: always
        container_name: ContainerNameHere
        volumes:
            - /Organization/Databases:/var/opt/mssql/data
        environment:
            - ACCEPT_EULA=Y
            - SA_PASSWORD=password_here
            - MSSQL_PID=Express
        ports:
            - 1433:1433
        logging:
            driver: none
        networks:
            - NetworkNameHere
networks:
    NetworkNameHere:
        name: NetworkNameHere
        driver: bridge

How about now? I just created some empty databases (only schema, no data) and I deployed a couple of .NET APIs inside docker.

See how the load average is now at almost 6.
What specific information can I give you? Does it suffice to say that server crashed? A Debian 10 server only running docker and nginx crashed because of an empty SQL Server docker, with 4 GB of RAM (yes it's a small amount of RAM, but we're talking about choosing a technology for microservices to deploy SQL Server over many small VPS instances), with 2 CPU cores.
Please tell me what information do you need, and I will provide it for you.

I am writing SQL server CI testing for https://github.com/php/php-src. where I install SQL server using choco it takes about 5 minutes to startup as well.
Github Actions repro code:
    steps:
      - name: Setup SQL server
        run: |
          choco install sql-server-express -y --install-arguments="/SECURITYMODE=SQL /SAPWD=mssql_Pass11"
          sqlcmd -S "(local)\SQLEXPRESS" -U sa -P "mssql_Pass11" -Q "SELECT 1;"

I would expect the startup time to be no more than 30 seconds.

@mvorisek I'm glad SQL Server was slow. I was forced to migrate to another RDBMS, and now I'm completely happy. I chose MariaDB.
It was hard at first after 16 years of development with SQL Server. But it was worth it.

😆 PHP is language interpreter like python and every major DB vendor is supported, I personally use MySQL as well, but this issue is to make SQL Server startup faster as PHP CI pipeline will be then faster 🚀

There's no mention of the host OS from the original poster. the path for the volume would indicate a *nix environment...  Would point out that I/O to/from windows and mac host environments are sluggish to say the least... best to use an actual volume container if you aren't running from a Linux (or WSL) host environment.
As to limiting CPU resources, that would/should be part of your docker/kubernetes configuration in practice iirc.
Not a big fan of mySQL/MariaDB myself... I'd go for PostgreSQL as a first choice, it's popular, sane, portable and every cloud provider supports it in a hosted model (even most second tier clouds).  Every time I've used mysql, it pisses me off in one way or another, from utf8 not being utf8 for starters.  For even being able to have case-insensitive collation on an index for a BINARY field.  It just irks me.


Ready for testnet tracking issue

Epic for the tracking all that needs to be done before testnet
Config module issues:

#17
#19
#22
#23
#31
#18
#51
#52
#53
#54
#55

Light Client issues:

#20
#30
#25
#26
#36
#32
#34
#35

Vote Extensions / ABCI issues:

#40
#21
#24
#43
#46

Sidecar issues:

#5
#6
#7
#8
#10
#12
#9
#15
#13
#11
#38
#37
#42
#43
#51

General/others:

#14
#33
#48

Closed in favor of separate epics (parent issues with sub issues):

#37
#60
#61
#62
#63
#64


now ls does not list all deployments

now ls without additional args does not list all deployments:
now ls
...
compliments

  XXX      https://compliments-XXX.now.sh      41d ago

whereas with the respective package name it is complete:
now ls compliments
> 5 deployments found [936ms]

compliments

  XXX      https://compliments-xxxrcdds.now.sh      26s ago
  XXX      https://compliments-xxxsnidwqh.now.sh      51m ago
  XXX      https://compliments-xxxxpvfgof.now.sh      52m ago
  XXX      https://compliments-xxxyuapbxn.now.sh      57m ago
  XXX      https://compliments-xxxnxidbxz.now.sh      41d ago

now --version
𝚫 now 2.0.3

What's missing? 🤔

some of my deployments

Some additional information would make it easier to troubleshoot this :)

without additional args does not list all deployments:
now ls
...
compliments

  XXX      https://compliments-XXX.now.sh      41d ago

whereas with the respective package name it is complete:
now ls compliments
> 5 deployments found [936ms]

compliments

  XXX      https://compliments-xxxrcdds.now.sh      26s ago
  XXX      https://compliments-xxxsnidwqh.now.sh      51m ago
  XXX      https://compliments-xxxxpvfgof.now.sh      52m ago
  XXX      https://compliments-xxxyuapbxn.now.sh      57m ago
  XXX      https://compliments-xxxnxidbxz.now.sh      41d ago

now --version
𝚫 now 2.0.3

Ok. Narrowed it down: The shell with the freshly updated now does not work (reporting 2.0.3 as version). A "new" shell DOES work.

Is it mandatory to spawn a new shell after update? I cannot remember doing so with previous updates of now...

Indeed it seems that there was a regression in the api causing it to return incomplete lists for a few users... Thanks for reporting!
We've now applied a fix and the list should now show up in full.
Please verify it and reopen/comment in case the list still feels incomplete!

Afaik updates will essentially replace the existing binary; so it shouldn't be necessary to open a new shell for the update to take affect

THX!

Thanks :)


Initial congestion control API design

This is the initial proposal for the pluggable congestion control API to be reviewed by OTC.
Checklist

[x] documentation is added or updated

There seems to be a strange mix of functions.
We have calls that say when we transmit something and when it's acked, but at the same time we pass it an rtt. I would assume that it's perfectly capable of calculating that RTT itself.
It's unclear why we need to set an exception, and it can't do that itself.
When calling get_send_allowance(), why do we need to pass time_since_last_send, while it should know that itself?
Why does it have an on_data_lost / on_spurious_congestion_event, while it could just keep track of that itself?

There seems to be a strange mix of functions.
We have calls that say when we transmit something and when it's acked, but at the same time we pass it an rtt. I would assume that it's perfectly capable of calculating that RTT itself.

When calculating the smoothed_rtt the max_ack_delay as reported by the peer has to be accounted for. It would also unnecessarily complicate the pluggable algorithm implementation to compute it by itself when all the different implementations would have to reimplement the same computation.

It's unclear why we need to set an exception, and it can't do that itself.

This is just one of the two possible choices. I followed MSQUIC one.

When calling get_send_allowance(), why do we need to pass time_since_last_send, while it should know that itself?

The pluggable algorithm implementation should not require calls to gettimeofday() or similar syscalls.

Why does it have an on_data_lost / on_spurious_congestion_event, while it could just keep track of that itself?

The algorithm does not need to keep track of sent packets / lost packets or anything. All the various congestion control algorithms would have to reimplement it. That does not make sense when this functionality is always required by any CC algorithm. In other words the loss detection from RFC 9002 is outside of this pluggable API.

Have you proofed this design out against things like BBR and L4S?

Ping for review -- this should have gone in already.

I asked a question two months ago that has not been answered. https://github.com/openssl/openssl/pull/18018#issuecomment-1105193404

Have you proofed this design out against things like BBR and L4S?

No. However this design is not set in stone. Until the API is public we can change it. Also in my opinion the concept of changeables should allow for these modern CC methods but I did not look at it in depth.

the concept of changeables should allow for these modern CC methods

Okay, so what CC algorithms are supported by the basic API? Reno? Cubic?
"should" is weak. If the project is going to commit to CC, it should commit to supporting what will be ubiquitous at the time openssl is released.

"should" is weak. If the project is going to commit to CC, it should commit to supporting what will be ubiquitous at the time openssl is released.

The API is NOT set in stone. Full CC support is not even targeted at the MVP release (i.e., 3.1) but some later one. The API won't be public in the MVP release.

Okay, so what CC algorithms are supported by the basic API? Reno? Cubic?

New Reno and Cubic were looked at and should be supported. Again - should - because until you implement you really cannot know for 100%.

The API is not set in stone.

And yet #18676 would add API's (and code) that basically cement this design; that PR claims this has already been "workshopped" (whatever that means).
You should consider joining this effort: https://mailarchive.ietf.org/arch/msg/tsv-area/BqWfH1htnGmZvxAhZwfzsSzI4Pk/

And yet #18676 would add API's (and code) that basically cement this design

Nothing is cemented. Everything can be changed. Of course depending on the nature and amount of the necessary changes the costs of changing things can vary but there is no point in trying to design perfect things from start.

Sure, it's just code. But the design doesn't even understand the main congestion systems used in the short term, and you don't seem to be participating in the design of future ones.
I wasn't asking for a perfect design from the beginning. I was asking for accomodating reality.
Sigh.  Okay.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 61 days ago

The header file for this is already merged as part of ACKM. This should be merged or closed.

@hlandau please approve

This pull request is ready to merge

Merged to master. Thank you.


[BUG] generate.ps1 is behaving unexpectedly

Library name and version
https://www.nuget.org/packages/Microsoft.Azure.Management.RecoveryServices.Backup/4.3.0-preview
Describe the bug
We are seeing unexpcted behavior from the generate.ps1 in our recoveryservice.backup repo.
For multi-api packages the generate.ps1 is generating same clients for both packages which is leads to an compilation failure.

Expected behavior
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/recoveryservices-backup/Microsoft.Azure.Management.RecoveryServices.Backup/src/recoveryservicesbackup/Generated/SdkInfo_RecoveryServicesBackupClient.cs
Actual behavior

Generated client version if wrong.
Reproduction Steps
Fork the azure sdk for net repo and run the script
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/recoveryservices-backup/Microsoft.Azure.Management.RecoveryServices.Backup/src/generate.ps1
Environment
N/A

tagging @pratimaupadhyay02 for FYI

Hi @ArthurMa1978 can you please help with the issue

Turns out this was due to a typo in the path that we added for our passive stamp package.
Correcting the name help resolve the issue, closing this bug.


How do I use "--format" now?

Hello.
I used to do --format [prioritized formats] and get the first matching format, that was very useful - but now all given formats are downloaded. How do I set up preference order now (just one format needed) instead of requesting all formats?
My goal is to get the lowest possible quality of looong videos - without checking available formats of each video. Worked fine before :/ .

-f 36,5,43,18

Regards & respect

Have you read FAQ?


RFC: Coprocessor Cache

Signed-off-by: Breezewish<EMAIL_ADDRESS>

@breeswish Can we merge the RFC now?

Yes, as long as there are enough approvals. @andylokandy

/cc @tikv/maintainers


using some playlist link leading to error

Describe the bug
playlist link like this - https://www.youtube.com/playlist?list=PL9bw4S5ePsEEqCMJSiYZ-KTtEjzVy0YvK not working when pasting in the playlist area. most likely because of this symbol "-".

Oops, you are right!
I updated the regex and it should work now.
https://regex101.com/r/FTaJvI/1

@sanihaq above changes were deployed and live now.
Could you test it and let me know?

@iamsainikhil working as expected 👍


Current selection DOM element

I am trying to get the current selection element so that I can calculate the offset from the top of the document. I would like to position buttons on the left of the cursor like used in Facebook Note editor.

Is there an easy way to get the selected dom element?

Yes, I've wondered the same. Is there some kind of event on editor that gets fired if the selection changed with infos about position etc.?

Great question.
For the Notes editor, I reach directly into the Selection object, identify the block that contains the cursor, and calculate the position of the element.
This isn't exposed in the current API, though I think it might be useful to have a utility function available so that this doesn't need to be reimplemented.

One solution would be to add ref={blockKey} to DraftEditorBlock or its parent div so we could use findDOMNode to get the element.
Example:
ReactDOM.findDOMNode(editorState.getSelection().getStartKey())

Each top-level block element has data-block={true}, which you can use to find it if looking upward from the node with the current DOM Selection.

Thank you that helps a lot!

What would be most helpful here? A utility to provide the DOM element for the selected ContentBlock? I don't want people to have to go digging into the Selection API docs at MDN. :) So I can put something together if it would be useful.

A utility that provides a reference to the current block with cursor focus would be great.
How about the boundingClientRect of the current selection (to ease in positioning popups, widgets, etc)?

Great library! I was toying around with the Medium style formatting bar shown in the video (centering controls above selection), and, from quickly digging around MDN got something working in Chrome with just window.getSelection().getRangeAt(0).getBoundingClientRect(). But I have no clue if this will break in IE, etc.
It'd be amazing if this library itself could handle any cross-browser and edge cases involved in positioning relative to a selection!

I've had some success with https://github.com/bkniffler/draft-js-drag-drop/blob/master/js/components/draft.js#L65
Its just a mouseUp handler on the draft-js wrapping div.
mouseUp(e) {
      function getSelected() {
         var t = '';
         if (window.getSelection) {
            t = window.getSelection();
         } else if (document.getSelection) {
            t = document.getSelection();
         } else if (document.selection) {
            t = document.selection.createRange().text;
         }
         return t;
      }

      setTimeout(()=> {
         var selection = this.state.value.getSelection();
         var selected = getSelected();
         var rect = selected.getRangeAt(0).getBoundingClientRect();

         if (selection.isCollapsed()) {
            return this.setState({toolbox: null});
         }
         else {
            this.setState({toolbox: {left: rect.left, top: rect.top, width: rect.width}});
         }
      }, 1)
   }

The resulting position data can easily be used with something like react-portal to show a tooltip like here https://github.com/bkniffler/draft-js-drag-drop/blob/master/js/components/tooltip.js
The working example: https://draft-drag-drop.herokuapp.com/
I've only tested in Safari/Chrome though.

@bkniffler Nice! Thanks for sharing.

Here's my function for getting the selected block element
getSelectedBlockElement = () => {
  var selection = window.getSelection()
  if (selection.rangeCount == 0) return null
  var node = selection.getRangeAt(0).startContainer
  do {
    if (node.getAttribute && node.getAttribute('data-block') == 'true')
      return node
    node = node.parentNode
  } while (node != null)
  return null
};

I managed to get this working. To position the buttons on the left, I first retrieved the selected block element using something akin to the function above. Then I used the offsetTop property as it's parent node is the editor container, so it's safe to use this. This gave me the vertical offset for the buttons.
To then display the popover container which adds all the inline styles, like bold, italic, link, etc. I grabbed the position of the editor and the position of the current selection using the getBoundingClientRect method and computed the difference between the two to figure out where to place the inline buttons relative to the editor.
See it in action here

@AlastairTaft Getting the current block is straightforward using Draft:
const currentContent = editorState.getCurrentContent()
const selection = editorState.getSelection()
const currentBlock = currentContent.getBlockForKey(selection.getStartKey())

I prefer this technique over using the window object and digging through DOM nodes.

For retrieving information about the ContentBlock that contains selection, definitely rely on the model, not on the DOM.
For retrieving information about the actual node in order to obtain DOM position information, @AlastairTaft's solution is pretty close to what we use internally. :)

@gscottolson That's much nicer thanks

Hopefully this will be useful as well: https://github.com/facebook/draft-js/blob/master/src/component/selection/getVisibleSelectionRect.js
Going to go ahead and close this issue since we've got a useful thread here.

I'm trying to display a suggestions container and I'm trying to use the "new" getVisibleSelectionRect, but it returns null when the editor is empty. And the editor is empty when I'm typing the first character, because I'm using the function inside the onChange event.
So I tried to use @AlastairTaft's code to get the block and calculate the position but I'm having some difficulties to properly get the exact position of the cursor/selection, because the block is quite large. I cannot display the container at the beginning/end of the block. @hellendag do you mind sharing a bit how you do this at facebook?

Reading through this thread several months after it was closed, and still am confused. I too would like to display a popover block with autocomplete suggestions positioned close to the caret. Is there a good technique for finding out the position of the caret (its absolute top and left offsets or the offsets relative to the Editor)?

@azangru This is working for me:

// From MobX observables

  @observable isSelectionActive = false;
  @observable selectionCoords = {};

  @action modifySelection(value) {
    this.isSelectionActive = value;
  }

  @action setCoords({ editorBound, cursorBound }) {
    console.log('editorBound, cursorBound', editorBound, cursorBound);
    this.selectionCoords = {
      top: cursorBound.top - editorBound.top + 20,
      left: cursorBound.left - editorBound.left,
    };
  }

// The events of the class

  _handleBeforeInput(char) {

    switch(char) {

      case '@': {

        // The rectangle that bounds the editor
        const
          editorNode = document.getElementById(this.props.id),
          editorBound = editorNode.getBoundingClientRect();

        // The rectangle that bounds the cursor
        const
          s = window.getSelection(),
          oRange = s.getRangeAt(0),
          cursorBound = oRange.getBoundingClientRect();

        this.setCoords({ editorBound, cursorBound });
        this.modifySelection(true);
        return true;
      }

      case '#': {
        console.log('handled #!');
        break;
      }

    }

    return false;

  }

What is still needed is to control when the position of the caret is close to the edge to prevent the autosuggest div to overflow the parent div. In that case the property should be top & right instead of top & left.
Hope this helps!

@jmaguirrei Wow, thank you!
I eventually adopted the approach taken by the Draft-js-plugins team, where they use a draft-js decorator to wrap the word for which autosuggestion is offered in a span, and save the coordinates of that span. Once those coordinates are available, it is trivial to position the autocompletion box near the word that the cursor is in.
Thank you for offering another solution to this problem!

@azangru You are welcome, glad to help! I havent finished my editor component yet, so maybe I will be trying another solutions, like yours, in the future. Keep in touch. Regards

Using window.getSelection().getRangeAt(0).getBoundingClientRect() or document.getSelection().getRangeAt(0).getBoundingClientRect() to get the "cursor's" position does not work with Safari 10.0.3. It works with Chrome, Firefox and Edge, though.
Any idea how to get the position in another way?


Social Media Dashboard Implemented

Fixes
#564

Description

Implemented a Social media Dashboard using Web Dev (HTML, CSS, Javascript)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New project (non-breaking change which adds functionality)

How Has This Been Tested?

Tested it on local system

Tech Stack used:

HTML
CSS
Javascript

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have included a README.md file
[x] My project follows this structure: Domain Folder/ Project Name Folder/ Project files and Readme.md files
[x] I have added demonstration in the form of GIF/video file
[x] I am a GSSoC'22 Participant

Screenshots

Kindly review this PR, @manisha-27 @heet-kumar

Kindly review this PR, @manisha-27 @heet-kumar

Kindly review this PR, @manisha-27 @heet-kumar

@hariketsheth try to upload photographs using markdown language not link and don't forget to upload Demo Video till then I can't merge this PR.
How to upload video in markdown then watch this video : https://drive.google.com/file/d/12cjBwDaTCK4pu7cqNI_FNuRAZwamCywP/view?usp=sharing
Note : Screen Recorded video should include

code go-through
what are the files you have included
run your project in screen recording

@heet-kumar Done 👍🏻
Proceed with the review

https://user-images.githubusercontent.com/72455881/170635332-4df2d229-0b07-4654-a13e-ce23910561a1.mp4

@heet-kumar Please review it quick if possible since only few days left for GSSoC'22. Thank you so much and I request you to add Level 3 Label considering the contribution

@heet-kumar Please review it quick if possible since only few days left for GSSoC'22. Thank you so much and I request you to add Level 3 Label considering the contribution

@hariketsheth Buddy its depend on Work you have done, we can't do as per your demand.

@hariketsheth Hey buddy, your project is amazing, After the code go-through, In your Screen recording, just include it in addition to that run your project in Screen recording and show your project live run in Screen Recording then upload it here. That's it I want. Do it.

Its already done
Check https://github.com/abhijeet007rocks8/Dev-Scripts/pull/566/commits/2d584ef7564cf9215687a96979102d3d551b76b6


Discovery: Look into using images for page templates and text sets

Context
Now that we've got screenshots of every template page, we could try using them in the page templates panel of the editor to cut down on load time. On hover we could still render the full preview so that the user can see the animation - this would prevent us from loading any preview unnecessarily.
Just one idea, I am not sure if we want to mess with this yet or how this would impact the user created layouts. It's an option now though, so I wanted to start a conversation.
Description
Let's use the images we have from the work done on the dashboard's explore templates (screenshots of every page on the cdn by template slug and page number) and explore how we might do the same(ish) on page layouts and then text sets.
Questions to address:

how to track page type?
how to toggle between page templates that have template images vs the placeholder grid (we'll need to adapt the CLI for. generating template images to have an alternate to generate with grids)
should the preview load on hover/focus to allow user to see animations before selecting?
What's the perf gain?
Do we still need to virtualize the list?

Using findings here we'll figure out where to take this next.

How to track page type?

In defaultTemplates.js I added type: template.pages[index].pageTemplateType, to templatePosters while mapping through template.postersByPage)  in the pageTemplates.reduce this allows filtering as before.

How to toggle between page templates that have template images vs the placeholder grid (we'll need to adapt the CLI for generating template images to have an alternate to generate with grids)

Grabbing a second set of images for grids vs image so on toggle, we would load the alt set of images.

Should the preview load on hover/focus to allow user to see animations before selecting?

???

Do we still need to virtualize the list?

Navigation (keyboard) is much smoother without virtualization. The jank is still present if virtualized. It does allow for a faster initial load, but some images will be missing while scrolling until they are loaded.

What's the perf gain?
I took several renders and jotted down the avg time below.
Before:
no throttle -> 4213ms
fast 3g ->  4145ms
slow 3g -> 4539ms
Before (while trying to scroll to the very bottom):
no throttle -> 37169ms
fast 3g -> 42390ms
slow 3g -> 45025ms
After (with virtualization):
no throttle -> 3886ms
fast 3g ->  3763ms
slow 3g -> 4956ms
After (without virtualization):
no throttle -> 2838ms
fast 3g -> 3411ms
slow 3g ->  4201ms

Nice perf gains! Thanks for doing this research.

how this would impact the user created layouts

This part needs to be addressed still. Perhaps html-to-image could be used?

Todo list if we / when proceeding

Update CLI in templates to also include a version with images swapped for grids
Implement way to filter template type by meta-data story page type. See example wip pre-work on branch - try/static-page-layouts ->  defaultTemplates.js lines 117 - 131
Use images in page layouts for templates (leave existing story rendering for custom layouts saved by users) See example wip pre-work on branch - try/static-page-layouts ->  pageTemplate.js  lines 135-144
Create a second array similar to postersByPage for grid style images. When toggling between "grid" view and "image" view, grab corresponding images. Or store "grid" images in same postersByPage array, and only grab corresponding imaging based on toggle.
If keeping virtual scroll, fix jank / if removing virtual scroll implement standard grid keyboard nav, See example wip pre-work on branch - try/static-page-layout templateList.js
Figure out how to implement similar approach for savedTemplates

Re: Implement way to filter template type by meta-data story page type. See example wip pre-work on branch - try/static-page-layouts -> defaultTemplates.js lines 117 - 131
Not 100% sure if this will actually map out correctly. Might be safer to create an actual mapping based on id of the two.

Thanks, @BrookeGraham !
I think this gives us a solid launch point for an actual Epic to do this work efficiently and in a manner well communicated.
As for the custom templates saved by users, I think that can be handled separately. The text-sets have been turned to images, the templates are also static and should follow suite.
The only thing here that we need a thumbs up from Will, Andrew and George is if it's OK to lose the 'animate on hover/focus'. That's really the only "loss", but it comes at the gain of usability and speed so it feels worth it.
I'll leave a note in the design channel asking. The speed improvements are substantial though!

Ship it! 🚢

The only thing here that we need a thumbs up from Will, Andrew and George is if it's OK to lose the 'animate on hover/focus'.

I think that's fine and easily worth the performance gain. It also matches the current UX in the Explore Templates page.
2c: Eventually, it'd be nice to render the page with animations on mouse hover (like video autoplay in YouTube).

Ship it! 🚢

The only thing here that we need a thumbs up from Will, Andrew and George is if it's OK to lose the 'animate on hover/focus'.

I think that's fine and easily worth the performance gain. It also matches the current UX in the Explore Templates page.
2c: Eventually, it'd be nice to render the page with animations on mouse hover (like video autoplay in YouTube).

@choumx -
cool, i added a new epic and line item for the November release tentatively.

(Closing since discovery provided enough information to figure out what all would be needed to map out an epic, thanks @BrookeGraham )


Fix schema 1.2.4

evidence.variant2disease.provenance_type.database.version = force it to be a string and should be mandatory even if empty
evidence.variant2disease.provenance_type.database.dbxref,version = force it to be a string and should be mandatory even if empty
evidence.gene2variant.provenance_type.database.version = force it to be a string and should be mandatory even if empty
evidence.gene2variant.provenance_type.database.dbxref,version = force it to be a string and should be mandatory even if empty

SourceID: gwascatalog should be gwas_catalog (Olga)
Ensure in EvidenceValidation that it's of type gwas_catalog
SourceID for eva_somatic should be eva_somatic and not EVA. We don't allow overlapping sources acros data types.
Map genetic_literature to genetic_association for G2P and UniProt instead of creating another datatype.
Everything related to genetic association should be the same.
provenance type database version in evidence should be a string.
if genetic_association:
evidence.gene2variant.resource_score.value: should have a value
can be a problem for resources without a gene2variant mapping. So we should force to have this even if not used.
same for functional consequence

Remove identifiers.org from genes and map to ensembl ids (future)

Remove identifiers.org from OpenTargets specific terms and remove reference to CTTV

Keep only IRI that make sense for EFO, HP, MP, GO, ECO, SO, NBO (etc.)

Remove list for targets and diseases (so back to a triple).

@ElaineMcA we are at 1.2.8 now, if the above still an issue, re-open please


Handle different binary filename at runtime

Why
I want to use yt-dlp with youtube-dl-exec. While using https://github.com/microlinkhq/youtube-dl-exec#environment-variables can download the correct binary, but on runtime, we need to set YOUTUBE_DL_FILENAME manually in process.env
How
It should implement or document how to handle different binary filenames at runtime.
Either document how to do it with process.env or create something to handle these different binary filenames, either way, I'm fine

Oh, gotcha.
What if we expose a constructor way, being the default module.exports bind to YOUTUBE_DL_PATH constant?
So users can continue using the library as usual and power users like you can take advantage of the constructor for creating different way to consume the binary.

I agree with that

My issue may somewhat be unrelated but can i change the path to the binary that yt-dl ultimately uses to download ?.
I have tried setting YOUTUBE_DL_DIR env variable to the desired path but youtube-dl-exec still tries to read the default path
The default path to the binary is inside app.asar and so it cannot read the binary. What i need it to point to is /app,asar,unpacked
Environment

Production build of electron using electron-builder

<EMAIL_ADDRESS>added binary customization:
const { create: createYoutubeDl } = require('youtube-dl-exec')

const youtubedl = createYoutubeDl('/my/binary/path')

Also, added YOUTUBE_DL_SKIP_DOWNLOAD to avoid fetch youtube-dl remotely
YOUTUBE_DL_SKIP_DOWNLOAD=true npm install


residence function

Housing units are individuated by their residence functions.  We need to define this term.

Definition - A function that inheres in a material entity and is realized by protecting persons and their possessions from weather and by some person or group of persons habitually sleeping in at least one site that is contained by that material entity.

OMRSE_00000075
Definition Added


fix datahub-python module: remove trino - presto tweak

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

related issue: #10558


feature(ci): adding a CI pipeline that builds snapshot planner into f…

…abric8-ui and deploys it for testing fabric8-ui#63

Can one of the admins verify this patch?


New version: VTUFileHandler v0.2.2

Registering package: VTUFileHandler
Repository: https://github.com/baxmittens/VTUFileHandler.jl
Created by: @baxmittens
Version: v0.2.2
Commit: f0e7d0be526f1611692096e8041c7d0b394aa2e2
Reviewed by: @baxmittens
Reference: https://github.com/baxmittens/VTUFileHandler.jl/issues/2#issuecomment-1348476085
Description: A VTU library in the Julia language that implements an algebra for basic mathematical operations on VTU data.

I know the version numbering is messed up but I'm afraid I cannot go back since I used the other version numbers for internal numbering (this project is mirrored on our company's GitLab server). It would be way more convenient to just skip the missing version.
[noblock]


FW999 103 HW

FW 103 HW

Good work, just the one small thing. If you can push a quick fix then I will clear you for 103, and you are good to take on a task!

Pushed the change.


Replacing include subdirectory name, transaction --> collection

Hi!
From what I understood from this commit https://github.com/SpiderLabs/ModSecurity/commit/5643d2fa287ce51cf3041b0afe3b9e41daa49011, its seems like it require some minor change in /src/Makefile.am namely replacing transaction with collection.

Hi @manishmalik,
Thank you for the patch! merged!


Require at least cartopy 0.18

Require at least cartopy 0.18, as problems have been encountered when plotting, see https://github.com/SciTools/cartopy/issues/1615

this is a mess - can you retrigger the tests pls? cartopy 18 packages have been updated two days ago

I built the env and installed (w/o Julia and R tho) on my laptop - not sure where Circle is picking up those issues - you wanna turn on Github Actions testing for this branch?

How much memory is available in the GitHub actions machines? CircleCI is limited to 4 GB and solving the environment needs more sometimes, depending on the versions I used it required either 1.5 GB or 7 GB of RAM to build/install the conda package. Maybe if we just wait for a bit things will get better.

8 gigs on GA machines (99% sure) :+1:

with conda=4.9.1 it chugs about 1.5G max for env solving (not installation of the R packages, just env solving and creation) on me laptop - why is it needing so much more on a Circle machine?

ported this entirely in #1997 so closing this - @bouweandela feel free to reopen if you think I missed something, but pls merge master into it after :+1:

ported this entirely in #1997 so closing this - @bouweandela feel free to reopen if you think I missed something, but pls merge master into it after :+1:


look-up-config-in-current-dir-instead-of-home

Replace default configs with templates:
device-spec.json -> device-spec.json.template
config/stf-utils.ini -> stf-utils.ini.template

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   74.08%   74.08%
=======================================
  Files           5        5
  Lines         409      409
=======================================
  Hits          303      303
  Misses        106      106

Continue to review full report at Codecov.

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


xrdp 0.9.0+ IdleTimeLimit ?

I have several Debian 9 instances with xrdp 0.9.6 installed (from Debian repo) Problem is the rdesktop / xrdp connection is closed and killed on "inactivity"
I have older units (xrdp 0.9.0 manually built from source) and the sesman.ini has
IdleTimeLimit - when to disconnect idle sessions
Type: integer
Default: 0
if not zero, the seconds without mouse or keyboard input before disconnect
not complete yet
IdleTimeLimit=0

This option is not available with later xrdp releases?
If not, any suggestions on how to maintain the connections?

See #652.

I'm working on this.  Will be implemented next release.

K Great! it's a good feature.
Did alot of looking around ..I think my problem was due to the screensaver or power management energy star equipment.

Solved by #1227.


Handle 2d and 3d as word separators

I wanted to use this crate to convert some strings to snake_case, but the strings I wanted to convert need 2d or 3d to be together.
For example:
CollisionShape2D should convert to collision_shape_2d

Thanks for bringing this up. The algorithm for case conversion starts by determining word boundaries. This can be done in a variety of ways. Unfortunately the exact method is hidden behind the .from(Case::Snake). That is, you can't pick which patterns make word boundaries. This is a good point, and is a fault with its current implementation. Its been a year since I've updated this and I think exposing an API for determining word boundaries might make for a good 0.5.0. Thanks for bringing this up.
So in summary, current methods that determine word boundaries without a delimeter (such as camel case) always put a word boundary before and after numbers. To fix this exact case you'll have to do some manual string contruction (until I update something.)
Also, in an effort to remain language agnostic, I would rather expose word boundaries rather than adding special cases for 2d, 3d, and others.
The problem of what the definition of "cases" are is difficult. I have a side project at https://stringcase.org to try to classify these. It's a difficult problem. Feedback is welcome.

Interesting, in my case I realised after making this issue that I didn't need to generate the case automatically. Before realising this I simply made a simple snake_string.replace("2_d", "2d") after the case conversion.
I looked a bit into this and I'm not even sure how it would be possible to specify a word boundary for this. In the case of 2d and 3d these generally go together, but I can easily imagine having a string where numbers and letters need to be separated. Maybe it could be to split on the second letter after a number. I guess the generalized configuration would be to specify how many letters after a number you want to keep or even specify to only keep a specific letter. Either way, it obviously can't be the default case and as you said would need a way to configure it by the user.

This problem is finally solved.  I was able to create the entire case conversion modular, allowing you to specify exactly three things:

character boundaries to split a multiword identifier into "words"
a pattern that describes how to mutate the words
a delimiter to join the words into a single identifier

Thus, you can know do the following to fix your problem.
assert_eq!(
  "CollisionShape2D"
    .with_boundaries(&[Boundary::LowerUpper, Boundary::LowerDigit])
    .to_case(Case::Snake),
  "collision_shape_2d"
);

You can specify which boundaries you want to split the identifier on before converting into the new case.  Alternatively, you could create your own converter, which would store all the options for a case conversion.
let conv = Converter::new()
  .set_boundaries(&[Boundary::LowerUpper, Boundary::LowerDigit])
  .set_pattern(Pattern::LowerCase)
  .set_delim("_");
assert_eq!("collision_shape_2d", conv.convert("CollisionShape2D"));

You can see more examples of this in the docs for version 0.5 (now published).  Thank you for this issue.  This was a motivating use case that inspired me to overhaul the conversion process and expose the inner workings of the conversion.

This problem is finally solved.  I was able to create the entire case conversion modular, allowing you to specify exactly three things:

character boundaries to split a multiword identifier into "words"
a pattern that describes how to mutate the words
a delimiter to join the words into a single identifier

Thus, you can know do the following to fix your problem.
assert_eq!(
  "CollisionShape2D"
    .with_boundaries(&[Boundary::LowerUpper, Boundary::LowerDigit])
    .to_case(Case::Snake),
  "collision_shape_2d"
);

You can specify which boundaries you want to split the identifier on before converting into the new case.  Alternatively, you could create your own converter, which would store all the options for a case conversion.
let conv = Converter::new()
  .set_boundaries(&[Boundary::LowerUpper, Boundary::LowerDigit])
  .set_pattern(Pattern::LowerCase)
  .set_delim("_");
assert_eq!("collision_shape_2d", conv.convert("CollisionShape2D"));

You can see more examples of this in the docs for version 0.5 (now published).  Thank you for this issue.  This was a motivating use case that inspired me to overhaul the conversion process and expose the inner workings of the conversion.