Fix a font oversampling warning being printed when it shouldn't

The warning was being printed even when the stretch aspect mode was set to Keep Width, Keep Height or Expand.
Note that it still won't be printed when the stretch mode is set to Disabled (as it should be), due to the early return statement at the beginning of the function. The warning message was also slightly shortened.
Demo project for testing

Thanks!


Bug | Names can overflow size of application

Bug
Names that are too long will overflow the bounds of the application causing a scroll bar, and extending past the known bounds of the application frame.

Workaround
Scroll window to see full name.

This is fixed in https://github.com/TimothyMeadows/GUvrs/commit/90af81079f81d46eafcaa3f38a9c875c8dcb02ac


Combining gl branch option-c and -su should report an error

The instruction  gl branch -c origin/test2 -su origin/test2 doesn't report an error, but only execute the  gl branch -c origin/test2.
It would be handy to be able to create and set the upstream branch in one instruction like for instance gl branch -csu origin/test2.

Also when I tried gl branch -c test4 -c origin/test4 only the last -c origin/test4 was taken in account.
It should ideally report an error.

255b6d4fd2270429c666bc15598134f106e8f4d7


Source lua files (if any) from rtp when loading a plugin

Neovim 0.5.0 allows lua files to be used in runtime files (such as
plugin, ftdetect, etc.) as well as vimscript files. Indeed, some
plugins have plugin/*.lua scripts only, but not plugin/*.vim;
such plugins cannot be sourced and work properly if it is lazy-loaded.

I think this still would need more thoroughly tested, but not sure how I would cover all the possible scenarios.
It looks like that this has only in effect when it comes to lazy-loading (or manually calling plug#load). When I don't lazy-load the plugin, it seems that the plugin/*.lua runtime files are being sourced even without this patch (not sure how it happens...). But when lazy-loading such plugins, plugin/*.lua are currently not being sourced.

I don't use Neovim anymore, so I can't verify if it works as described. But if any of you can test it and confirm that it works, I'll merge the PR.

" it seems that the plugin/*.lua runtime files are being sourced even without this patch (not sure why and where it happens...)"
@wookayin
I thinks this is neovim's behavior, once you add plugin's directory into runtimepath, neovim will source lua files in plugin directory when starting neovim.
It is easy to test
Use following script as neovim's config, nvim -u mini_vimrc --startuptime nvim_start.log

set nocompatible

"telescope
execute 'set runtimepath+='.$HOME.'/.vim/bundle/plenary.nvim'
execute 'set runtimepath+='.$HOME.'/.vim/bundle/telescope.nvim'

filetype plugin indent on
syntax on
syntax enable
set background=dark
set termguicolors
colorscheme default
set t_Co=256

Here is the start log:
nvim_start.log
044.828  001.519  001.498: sourcing /Users/tracyone/.vim/bundle/telescope.nvim/plugin/telescope.lua

Why we need this PR?
I found some of neovim plugins can't be lazy loaded using vim-plug.
Following is an example code of lazy loading.
"Do not load plugin by default.
Plug 'nvim-treesitter/nvim-treesitter', {'do': ':TSUpdate', 'on': []}
Plug 'nvim-treesitter/nvim-treesitter-refactor', {'on': []}
Plug 'nvim-treesitter/nvim-treesitter-context', {'on': []}

function! LoadPlugins(timer) abort
      call plug#load(['nvim-treesitter', 'nvim-treesitter-refactor', 'nvim-treesitter-context'])
     "require setup module of treesittier(lua/treesittier_nvim.lua)
    call v:lua.require("treesittier_nvim")
endfunction

"run LoadPlugins function after 300ms
call timer_start(300, function('LoadPlugins'), {'repeat': 1})

Following plugins only have lua files in plugin directory, when call plug#load, lua files were skiped, so They are failing to lazy-loaded.

nvim-treesitter/nvim-treesitter and plugins that base on it
hrsh7th/nvim-cmp and plugins that base on it
nvim-telescope/telescope.nvim and plugins that base on it

After apply this PR, plugins that are mentioned above can be lazy-loaded, I think this PR is OK to merge.

But if any of you can test it and confirm that it works, I'll merge the PR.

It does not, unfortunately. I mean—it's not enough for the :TSUpdate post-install hook of https://github.com/nvim-treesitter/nvim-treesitter.
Their README recommends the following:
Plug 'nvim-treesitter/nvim-treesitter', {'do': ':TSUpdate'}

Note there's no on: nor for: so vim-plug adds it to runtimepath immediately (before it's installed) and then at https://github.com/junegunn/vim-plug/blob/154f60b7ecbfb158a497e751aacab590247c69d4/plug.vim#L1056-L1061 the runtimepath isn't updated as it doesn't need to change (s:loaded[name] == 1). Neovim, however, seems to cache the available lua modules (lua/*.lua, not plugin/*.lua) only when runtimepath is (re)set (more about this at https://neovim.io/doc/user/lua.html#lua-package-path). So when plugin/nvim-treesitter.lua tries to require("nvim-treesitter").setup(), it fails:
Vim(source):E5113: Error while calling lua chunk: .../nvim/plugged/nvim-treesitter/plugin/nvim-treesitter.lua:9: module 'nvim-treesitter' not found:
no field package.preload['nvim-treesitter']
no file './nvim-treesitter.lua'
no file '/__w/neovim/neovim/.deps/usr/share/luajit-2.1/nvim-treesitter.lua'
no file '/usr/local/share/lua/5.1/nvim-treesitter.lua'^@^Ino file '/usr/local/share/lua/5.1/nvim-treesitter/init.lua'
no file '/__w/neovim/neovim/.deps/usr/share/lua/5.1/nvim-treesitter.lua'
no file '/__w/neovim/neovim/.deps/usr/share/lua/5.1/nvim-treesitter/init.lua'
no file './nvim-treesitter.so'
no file '/usr/local/lib/lua/5.1/nvim-treesitter.so'
no file '/__w/neovim/neovim/.deps/usr/lib/lua/5.1/nvim-treesitter.so'
no file '/usr/local/lib/lua/5.1/loadall.so'
stack traceback:
[C]: in function 'require'
I.../nvim/plugged/nvim-treesitter/plugin/nvim-treesitter.lua:9: in main chunk

Adding let &rtp = &rtp before call s:load_plugin(spec) fixes this. (It probably belongs elsewhere though.)

If anyone wants to play in the same environment I'm doing this in, then:
FROM debian:unstable@sha256:0862a2d6f153cb5287d7a4e25869cb5a8852765e62ff370e970d9cfcc26658aa

RUN apt-get update -y
RUN apt-get install -q -y curl git build-essential
RUN curl -LSsf https://github.com/neovim/neovim/releases/download/v0.9.4/nvim-linux64.tar.gz | tar xz
RUN sh -c 'curl -fLo ~/.local/share/nvim/site/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/154f60b7ecbfb158a497e751aacab590247c69d4/plug.vim'
RUN mkdir -p ~/.config/nvim \
 && echo "call plug#begin()" >>~/.config/nvim/vimrc.vim \
 && echo "Plug 'nvim-treesitter/nvim-treesitter', #{do: ':TSUpdate', commit: '075a64addc33390028ea124a1046a43497f05cd1'}" >>~/.config/nvim/vimrc.vim \
 && echo "call plug#end()" >>~/.config/nvim/vimrc.vim \
 && echo "vim.cmd.source(vim.fn.stdpath('config') .. '/vimrc.vim')" >>~/.config/nvim/init.lua \
 && :

docker build -t nvim-treesitter-lua-bug .
docker run --rm -it nvim-treesitter-lua-bug:latest

Merged, thank you all for your help!


depending on mvn command illegal transitive dependencies are found and not found

If you mvn clean install you will see that no illegal transitive dependencies are found
but if you do mvn gwt:run-codeserver you will see a lot of illegal transitive dependencies.
He re is a link where you can download a web-app:
https://github.com/droolsjbpm/guvnor/tree/master/guvnor-webapp

After some struggle with GWT and a downgrade to JDK 7 I was able to build your project. As far as I understand the GWT build fetches a lot of dependencies at compile time (which are not more necessary during runtime). After adding these dependencies the check does not further complain:
org.drools:drools-workbench-models-datamodel-api
org.uberfire:uberfire-commons-editor-api
org.jboss.errai:errai-marshalling
org.dashbuilder:dashbuilder-dataset-api
org.uberfire:uberfire-runtime-plugins-api
org.kie.uberfire:kie-uberfire-social-activities-api
org.uberfire:uberfire-widgets-properties-editor-api
org.uberfire:uberfire-widgets-service-api
org.dashbuilder:dashbuilder-displayer-api
org.uberfire:uberfire-backend-api
org.uberfire:uberfire-apps-api

So far I see only to possible ways to cope with that problem: either disabling the check for that particular module or declaring all the dependencies above (which may be a bit verbose in your case)

thanX André,
We did the 2nd way.
Michael
----- Ursprüngliche Mail -----
Von: "André Schubert"<EMAIL_ADDRESS>An: "ImmobilienScout24/illegal-transitive-dependency-check"<EMAIL_ADDRESS>CC: "mbiarnes"<EMAIL_ADDRESS>Gesendet: Donnerstag, 29. Januar 2015 19:20:01
Betreff: Re: [illegal-transitive-dependency-check] depending on mvn command illegal transitive dependencies are found and not found  (#25)
After some struggle with GWT and a downgrade to JDK 7 I was able to build your project. As far as I understand the GWT build fetches a lot of dependencies at compile time (which are not more necessary during runtime). After adding these dependencies the check does not further complain:
org.drools:drools-workbench-models-datamodel-api
org.uberfire:uberfire-commons-editor-api
org.jboss.errai:errai-marshalling
org.dashbuilder:dashbuilder-dataset-api
org.uberfire:uberfire-runtime-plugins-api
org.kie.uberfire:kie-uberfire-social-activities-api
org.uberfire:uberfire-widgets-properties-editor-api
org.uberfire:uberfire-widgets-service-api
org.dashbuilder:dashbuilder-displayer-api
org.uberfire:uberfire-backend-api
org.uberfire:uberfire-apps-api

So far I see only to possible ways to cope with that problem: either disabling the check for that particular module or declaring all the dependencies above (which may be a bit verbose in your case)

Reply to this email directly or view it on GitHub:
https://github.com/ImmobilienScout24/illegal-transitive-dependency-check/issues/25#issuecomment-72075863

OK


tab background in 1.1.0

bug
sometimes the background will expand, but without the tabs
how to reproduce
swiping from right to left, and continuing past the browser window causes a background to expand but without the tabs expanding
video
I have increased the transition speed of the sidebar to showcase the bug, but this still occurs even when set to 0ms
https://github.com/user-attachments/assets/da937398-a922-4b08-a631-5c41ec8f113f
versions used
plugin version: 1.1.0
zen version: 1.0.2-b.2 (64-bit)
os: windows 11

hmmm can't reproduce this right now, do you have other mods installed?


Enable fetching basal energy

Hi,
I would like to fetch BasalEnergyBurned, and added method to do that.
Thanks.

@yamazaki-sensei 🎉


Flutter/Flutter.h file not found in Xcode

I am trying to build my Flutter iOS app in Xcode, but I am getting the following error:
'Flutter/Flutter.h' file not found
failed to emit precompiled header '/Users/user/Library/Developer/Xcode/DerivedData/Runner-bqwgaclmizdztbgwvyuydjgnmjad/Build/Intermediates.noindex/ArchiveIntermediates/Runner/PrecompiledHeaders/Runner-Bridging-Header-swift_3AO5QSACASUGH-clang_24K6MS22YMY0N.pch' for bridging header '/Users/user/development/projects/flutter/project/ios/Runner/Runner-Bridging-Header.h'
in file included from /Users/user/development/projects/flutter/project/ios/Runner/Runner-Bridging-Header.h:1:

I have tried the following steps to resolve the error:

I have removed the ios folder and created a new one.
I have run pod install in the ios folder.

I am still getting the error. What else can I do to resolve this error?
Additional information:

Flutter version: 3.13.7
Xcode version: 15.0
No error displayed in the output of flutter doctor -v command:

@mycar98765
Can you provide complete verbose log ?

@mycar98765  Make sure you are opening the Runner.xcworkspace not the Runner.xcodeproj

@israelglory , yes, I am sure that I am opening Runner.xcworkspace.

@darshankawar @israelglory any updates?

@mycar98765
Can you run flutter build ios -v and share the log here ?


module-setup.sh: Simplify the qemu_fw_cfg kernel module check

This fixes it on not supported arches(i.e. including module that doesn't exist). It will unblock the cosa on all arches that don't support the qemu_fw_cfg(related to the ignition #666). I haven't yet tested this on x86_64. I would much appreciate testing on x86_64 and comments on the general approach.

Can you speak to how your change fixes the problem? I don't quite understand.

for the record I kind of like the has_builtin_fw_cfg function. I think I would prefer something like:
diff --git a/dracut/30ignition/module-setup.sh b/dracut/30ignition/module-setup.sh
index ce0359e..d7945b7 100755
--- a/dracut/30ignition/module-setup.sh
+++ b/dracut/30ignition/module-setup.sh
@@ -62,7 +62,8 @@ has_builtin_fw_cfg() {

 installkernel() {
     # We definitely need this one in the initrd to support Ignition cfgs on qemu
-    if ! has_builtin_fw_cfg; then
+    # but only on x86_64
+    if ! has_builtin_fw_cfg && arch_is_x86_64; then
         instmods -c qemu_fw_cfg
     fi
 }
}

for the record I kind of like the has_builtin_fw_cfg function. I think I would prefer something like:
diff --git a/dracut/30ignition/module-setup.sh b/dracut/30ignition/module-setup.sh
index ce0359e..d7945b7 100755
--- a/dracut/30ignition/module-setup.sh
+++ b/dracut/30ignition/module-setup.sh
@@ -62,7 +62,8 @@ has_builtin_fw_cfg() {

 installkernel() {
     # We definitely need this one in the initrd to support Ignition cfgs on qemu
-    if ! has_builtin_fw_cfg; then
+    # but only on x86_64
+    if ! has_builtin_fw_cfg && arch_is_x86_64; then
         instmods -c qemu_fw_cfg
     fi
 }
}

I believe that the separate function has been culprit of it not functioning as designed. I'm no bash master, but this works, separate function doesn't(and I don't exactly know why).

Oh... I have forgotten that with the y it is built-in to the kernel(as opposed to the m). It needs different overhaul...

That said it actually should be something like
...
if grep -q CONFIG_FW_CFG_SYSFS=m; then
...

Have you tried the patch Dusty suggested? I think this is what you're looking for, no? I.e. on x86_64, we still require it to be either built in or as a module, while on other arches, we don't require it at all.

Have you tried the patch Dusty suggested? I think this is what you're looking for, no? I.e. on x86_64, we still require it to be either built in or as a module, while on other arches, we don't require it at all.
I believe that checking if it is module is most portable, if it is module let's include it, if not, it is either builtin or it is not supported. No need to explicitly check for arch.

I believe that checking if it is module is most portable, if it is module let's include it, if not, it is either builtin or it is not supported. No need to explicitly check for arch.

i'm ok with that assuming the kernel configs properly don't have CONFIG_FW_CFG_SYSFS=m on platforms where it doesn't exist.

if it is module let's include it, if not, it is either builtin or it is not supported

Right, but I wanted to catch the case where it is supported on the current architecture, but it just wasn't enabled in the kernel (i.e. CONFIG_FW_CFG_SYSFS=n). Anyway, good with that approach as well. Realistically, if the kernel config somehow regresses, we'd find out pretty quickly.


Go install fails on v0.6.10

This pr caused go install<EMAIL_ADDRESS>to no longer work. It returns the following error:
$ go install<EMAIL_ADDRESS>go<EMAIL_ADDRESS>(in sigs.k8s.io/aws-iam-authenticator@v0.6.10):
        The go.mod file for the module providing named packages contains one or
        more replace directives. It must not contain directives that would cause
        it to be interpreted differently than if it were the main module.

After some digging I found this issue which if I understand correctly means that replace directives cannot be used in go.mod files that have a main module.

Hah, reading that Go issue makes me mad!
Anyways for getting my go install of aws-iam-authenticator working, downgrading to v0.6.9 (diff) seems to build:
go install<EMAIL_ADDRESS>

This got broken by this commit: https://github.com/kubernetes-sigs/aws-iam-authenticator/commit/c98f1109162a3c58d9f45bb2ae7994413c8c389f#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6

Looks as though the go.mod replace directives have been cleaned up and the next release will resolve this, probably.

@nnmin-aws / @jyotimahapatra / @dims we can close this issue now

/close


Unable to create restoration in progress marker file

I get this warning on startup

The first two answers on StackExchange look helpful:
https://stackoverflow.com/questions/18932156/cryptic-warning-without-any-google-results-in-simulator


attr css function is not being turned into a dynamic value

emotion version: 5.1.0

Relevant code.
const PlaygroundWrapper = styled('div')`
  color: attr(color, #343a40);
`

What you did:
Ran latest code when using attr
What happened:
X.emotion.css will contain attr values

This is happening in the example


minimal gpt partition size is wrong

in partition/gpt/partition.go are multiple checks for end > start.
(e.g. https://github.com/diskfs/go-diskfs/blob/8ff8bc623cffbe3d41db4bcc902bca9823abfa22/partition/gpt/partiton.go#L253)
Both start and end represent sectors with start the first and end the last sector of the partition. so the minimal allowed sector size of a partition becomes 2 while it should be 1.
this is somewhat correctly reflected in
https://github.com/diskfs/go-diskfs/blob/8ff8bc623cffbe3d41db4bcc902bca9823abfa22/partition/gpt/partiton.go#L257
if size equals blocksize (but it would create an invalid result for 0 < size < blocksize, with end = start-1).
The code will fail later on in a second check for end > start.

Are you suggesting that this:
 case start >= 0 && end > start && size == calculatedSize:

should be
 case start >= 0 && end >= start && size == calculatedSize:

To be honest, this line is a bit in error in that it doesn't do anything, as it is a lone case statement.

yes.
https://github.com/diskfs/go-diskfs/blob/8ff8bc623cffbe3d41db4bcc902bca9823abfa22/partition/gpt/partiton.go#L150
https://github.com/diskfs/go-diskfs/blob/8ff8bc623cffbe3d41db4bcc902bca9823abfa22/partition/gpt/partiton.go#L253
https://github.com/diskfs/go-diskfs/blob/8ff8bc623cffbe3d41db4bcc902bca9823abfa22/partition/gpt/partiton.go#L254
should allow end == start
I assume
https://github.com/diskfs/go-diskfs/blob/8ff8bc623cffbe3d41db4bcc902bca9823abfa22/partition/gpt/partiton.go#L152
should be
case p.Size != 0 && p.End < p.Start:
or
case p.Size != 0 && p.End == 0
(btw p.Size <= 0 is the same same p.Size == 0 as it is unsigned but I guess that is more about preference)

Happy to take a PR on it.


Installed hightower unable to navigate to admin dashboard

Hi,
I used hightower gem and trying to check it functionalities but i am unable to view the dashboard.
i am running in localhost:3000
Thanks

did you define an authorization method on your User model (or whatever class represents a user in your app)
e.g.
class User
    def can_access_hightower?
        self.admin == true
    end
end

Then tell Hightower the name of the method in the hightower initializer
Hightower.authorization_method = :can_access_hightower?


Update sbt-ci-release to 1.5.11

Updates com.github.sbt:sbt-ci-release from 1.5.10 to 1.5.11.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.5.10).
You might want to review and update them manually.
project/src/main/scala/modules/javaSupport.scala

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.sbt", artifactId = "sbt-ci-release" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.github.sbt", artifactId = "sbt-ci-release" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #1738.


add conditional render for the relevant experience section

There might be times when people would prefer to just list total experience, and at the moment, if you delete the value for 'Relevant Experience', the label is still there.
This commit only shows that label if it has a value.
Changes
very small change to conditionally render the Relevant Experience label.
Fixes # (issue)
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

@lpmi-13 If the label is empty the respective section is not displayed. Did you try with that

I thought so, yes.
But lemme take another look and maybe raise a PR to update the docs, since it seemed a bit random which things were conditionally rendered based on what (eg, I worked out that the sections could be removed by deleting the label, but it didn't seem to work in this case.)

ah okay, so I finally found the place to delete labels, and it would be ideal if deleting that label also deleted it as a possibility in the Intro section, but I'm not at all sure how we would refactor this to make that more obvious, so I'll just work around it.


Adding basic documentation about GitLab

Adding basic documentation about GitLab. Solves https://coleridgeinitiative.atlassian.net/browse/TP-17

@grahamulator In my local environment test when accessing faq/monitorsettings.html some new pages disappear from the index. I don't know how to fix that or if it is just something cached.


Upgrade parser + printer to OCaml 4.04

Work in progress.
Builds upon our earlier work for 4.03: https://github.com/facebook/reason/pull/604
I'm already working on this.
A big challenge appears to be the refactored pretty printer in 4.04 which went from being an accessible class to a more closed module with functions. This will require much more code to be copied to our implementation.


CentOS 7 stemcells on Azure are reporting vhd size issue

Description:
We are running a Bosh director in Azure which is using Azure Blob storage. We have uploaded multiple newer versions (3763.55, 3763.57, 3763.59, 3763.61) of the CENTOS 7 Microsoft Azure stemcells. All stemcells upload correctly but when Bosh is preparing the deployment and compiling the packages we get the following error message:
Error: CPI error 'Bosh::Clouds::VMCreationFailed' with message '#<Bosh::Clouds::VMCreationFailed: Failed to get the user image information for the stemcell 'bosh-stemcell-d5dc2313-b659-47d1-a5ee-d4558559b517': #<Bosh::AzureCloud::AzureError: http_put - http code: 400
x-ms-client-request-id: 104e74a8-fa57-48c3-9589-4341000c3e4b
x-ms-request-id: bba799a9-a04f-4cea-aac2-b47ab824afb1
x-ms-correlation-request-id: 8834ca73-e0af-4474-a588-cb75a6e6adb0
x-ms-routing-request-id: SOUTHCENTRALUS:20200213T175103Z:8834ca73-e0af-4474-a588-cb75a6e6adb0
Error message: {
"error": {
"code": "InvalidParameter",
"message": "The VHD for disk 'bosh-stemcell-d5dc2313-b659-47d1-a5ee-d4558559b517.vhd' with blob https://5a90f885cf6060f56dd72585.blob.core.windows.net:8443/stemcell/bosh-stemcell-d5dc2313-b659-47d1-a5ee-d4558559b517.vhd has an unsupported virtual size of 3072.234375 MB. The size must be a whole number in (MBs).",
"target": "disks"
}
Expected Behavior:
We would expect to the see all of the packages complete compiling and continue on with the deployment.
Versions:
-Infrastructure: Azure
-BOSH version: 270.11.0
-BOSH CLI version: 3.0.1-712bfd7-2018-03-13T23:26:43Z
-Stemcell version: bosh-azure-hyperv-centos-7-go_agent/3763.57 (As mentioned above we have also tried versions 3763.55, 3763.59, 3763.61)
Please let me know if you have any further questions.

We've stopped shipping and support CentOS stemcells. We have fixed this in the linux-stemcell-builder repo for ubuntu stemcells however: https://github.com/cloudfoundry/bosh-linux-stemcell-builder/commit/e3bc0b618fbe77050a97fc624592721ed205a4ef . To pull this fix, you'd have to create your own stemcell.


Su_2016_Aug_3 → Su_YYYY_MMM_D

moment.parseFormat('Su_2016_Aug_3') parsed to Su_YYYY_MMM_D, I expected: dd_YYYY_MMM_D

\b doesn't really cut it in the regex you're using because it doesn't treat _ as a word boundary.  We should use ([^a-z]|^|$) instead as the boundary for names and abbreviations.

@gr2m why not just support it here though?  I already fixed it and added passing tests in my PR

@gr2m I get your point about #67, but for this issue, discriminating against _ seems very arbitrary to me, especially since it's so commonly used to replace spaces, for instance in filenames.  It's no harder to support than -.

@gr2m I should also point out that moment-parseformat tries to identify timestamps, which are almost never human-entered.

I know this is frustrating but please understand the perspective of the maintainers of this module and other Open Source libraries: for you it's a one time commitment, for us it's maintaining every edge feature until the end of the lifetime. I recommend you create issues first, discuss with the maintainers, then create a PR once everyone agrees it's a good idea, especially when it comes to adding features.
In years since this library exists you are the first to ask to add support for this feature, so I'm afraid I'm not willing to add it at this point. It's in the interest of everyone else depending on this library, and myself

Okay, I understand.  For the record, replacing _ in 2015_01_05 doesn't work:
parsing 2015 01 05 yields YYYY DD D instead of YYYY MM DD.  Replacing _ with - works though, even if I wind up with a string like 2015-03-05-4:00-PM.

The last sounds like a bug, can you make a test case for it? I'm mobile today, I can have a look over the weekend myself

okay


Update welly

Add older versions

Thank you for the contribution. It was merged directly as e5046fe045021b913affc6a2d221701e2e706509 to keep commit history cleaner. Your contribution is still credited to you.


Add support for 32 and 16 bit numpy floats

This commit adds support for 32 and 16 bit floats numpy as input types.
Since it doesn't look like there is a native symengine type for working
with single (or half) precision floats outside of using MPFR (which
seemed like the wrong approach here, it might make sense for np.float128
in a follow up though) this implicitly casts the np.float16 and
np.float32 types to doubles and just behaves as float/np.float64.
Fixes #351

Thanks for the PR. I think it is fine.
Only one comment: the different size floats now gets coerced into a RealDouble so when continuing using the number in symengine it will not behave the same numerically as it did in numpy. In the future it might be useful to have RealSingle, RealHalf and RealQuad in symengine to be able to retain the algebraic properties of the different sized floats and also to be able to evaluate expressions for these.


Copy-DbaCredential Maximum number of Dedicated Administrator Connections Exceeded

Version Information

Operating System (Name|Version): Windows 2012 R2
PowerShell Version: 4.0
SQL Server (Edition|Version): Sql Server 2016 Enterprise SP1 CU7

Steps to Reproduce
Everytime I execute Copy-DbaCredential -Source 'sqltest-1' -Destination 'sqltest-2' I get the following error in de SQL Server errorlog on the source:

Could not connect because the maximum number of '1' dedicated administrator connections already exists. Before a new connection can be made, the existing dedicated administrator connection must be dropped, either by logging off or ending the process. [CLIENT: <IP_ADDRESS>]

It's just like the cmdlet want to make a new connection without closing the previous one.

I managed to solve the problem by adjusting allcommands.ps1 and to make sure the DAC was made by IP address and not made by dns name.

Getting the same error if i run Copy-DbaAgentJob -Source "xxx" -Destination "yyy"   -WhatIf
from an external server. If i run locally there is no warning

Also getting same error when SQL job runs Sync-SqlInstanceObjects.ps1 to synch objects between nodes in AG..


[Issue]:

Do you need to file an issue?

[x] I have searched the existing issues and this bug is not already filed.
[x] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[x] I believe this is a legitimate bug, not just a question. If this is a question, please use the Discussions area.

Describe the issue
In the implementation, there is a conflict of versions between the packages 'graphrag' and 'json-repairs'. One does not support the other which is hindering the addition of graphrag to poetry.
Steps to reproduce
No response
GraphRAG Config Used
# Paste your config here

Logs and screenshots

Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

HI @anurag2506
It seems your graphrag-project is depending on json-repair ^0.30.0, but graphrag uses a lower version.
Please try reducing the target version in your graphrag-project dependencies file.

Graph tag uses a lower version of json-repair?
Or the version of graphrag should be lower than 0.3.6?
On Wed, 6 Nov 2024 at 09:14, Alonso Guevara @.***>
wrote:

HI @anurag2506 https://github.com/anurag2506
It seems your graphrag-project is depending on json-repair ^0.30.0, but
graphrag uses a lower version.
Please try reducing the target version in your graphrag-project
dependencies file.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/graphrag/issues/1324#issuecomment-2458668527,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCAOXYP7DNVZECTDUXB2B2TZ7GGCNAVCNFSM6AAAAABQTXVJO2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJYGY3DQNJSG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


added alternative way of solving tasks through docker

Additional changes in this PR:

Formatting and additional info in the README
Changes to the docker-compose file
Change the interpreter in python programs to be 'python3' explicitly


Problem with multi tier multitenant app consent

I'm Building an Angular 9 + Asp.Net Core WebApi App using Azure active directory for the authentication in a multitenant context. The WebApi Component have to call some MS Graph Api. So following the documentation, I created two apps on the portal (one for each component). In the web Api app i did the following operation:

added the authorizazion needed for the graph calls
added the ID of the client app in the knownClientApplications section of the manifest
3.Defined a scope in the "expose Api" Section

On the client app I added the scope definided at point three below.
On the Angular App I configured as follow the MSal library:
MsalModule.forRoot({
      auth: {
        clientId: 'CLIENT APP ID', // This is your client ID
        authority: 'https://login.microsoftonline.com/common', // This is your tenant info
        redirectUri: 'http://localhost:4200' // This is your redirect URI
      },
      cache: {
        cacheLocation: 'localStorage',
        storeAuthStateInCookie: true, // Set to true for Internet Explorer 11
      },
    },
    {
      popUp: true,
      consentScopes: [
        'user.read',
        'openid',
        'profile',
        'api://xxxxxxxxxxxxxxxxxx/.default'
      ],
      unprotectedResources: ['http://localhost:4200'],
      protectedResourceMap: [
        ['https://localhost:44351', ['api://xxxxxxxxxxxxxxx/api_usage']]
      ],
      extraQueryParameters: {}
    })

The problem is : When i login, the only consent prompted are like "user read information" and "user save information". The consent for the the web api are not reported, so the web api service principal in the customer tenant for web api is not created, lately the following calls from web api to graph fails for absence of permits.
For testing purpouse i tried to put other scopes (like calendar.read or place.read in the consentScopes of Msal configuration but the result is the same: the only consent requested is the two above.
Edited: Added doc metadata info

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 326f7173-a654-fda7-2438-6905da3fdbef
Version Independent ID: 7dbc5a10-9d61-26e1-f532-973b4bdfae9a
Content: Build apps that sign in Azure AD users - Microsoft identity platform
Content Source: articles/active-directory/develop/howto-convert-app-to-be-multi-tenant.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@simo-92,  Thanks for leveraging Azure docs feedback channel to raise this question. This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.
We recommend posting technical questions on the Microsoft Q&A and/or SO (I notice that you posted a thread on SO already) - where one of our specialized engineers in the subject or the Azure community members would provide the best possible answer to your scenario or question.
If the issue is related to Azure documentation, please do share the Azure doc for which this feedback is applicable to.

I followed this doc https://docs.microsoft.com/en-us/azure/active-directory/develop/howto-convert-app-to-be-multi-tenant. But for me writing the client id app in the Manifest of webapi app don't work as expeted. No consent for asked for the webapi and neither the service principal is created.

@simo-92, Thanks for sharing the documentation URL and additional details.  I have assigned the issue to a subject matter expert on this topic for further review, in the meanwhile you may checkout suggestions received on your SO  thread and see if it helps.

@simo-92, Thank you for reaching out. To clarify things and to make sure I understand the scenario correctly I am re-iterating the main points:

Front end application is developed in AngularJS
Front end application calls a backend WebAPI developed using ASP .net core.
The frontend app and backend API, is registered in AAD and the backend API is also exposed as an API in AAD.
The backend API has Graph API added under API Permissions blade

Based on this understanding if I have to share my views, I would say that when you call the backend API from the frontend app you are actually requesting for a token for the audience backend API from AAD, hence in the consent page you only get the permissions specified for that backend API and not for Graph API.
To get the consent for the Graph API and for the backend API to call Graph APIs, the backend app has to request for a token from AAD, with audience as Graph API and then you would get the consent page for the Graph API permissions. But since it being an API and if things has to happen silently, then you can get your global admin to provide consent for the graph api permissions and that would help the backend api to fetch a token silently for graph api using the client_credential flow of OAuth2.0 protocol.
This is a classic example of service-to-service call and for that the On-Behalf-Of flow is what suits the best. You can get more details about it here.
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-on-behalf-of-flow
Hope this helps. Do let us know if there are any more queries around this.

@simo-92, We are closing this thread for now since no response on this. Do update us if you would like to get this thread re-opened so that we can help you with that and help you further with your queries.

Thanks for the reply. I thought that simply accessing with an admin the whole consent was prompted. But i figured that is necessary access first to the specific link for the admin conenst.
Thanks


status page

something like the githubstatus.com
@nadirabbas

I believe that's planned based on my limited conversation with other contributors.


Update json-format extension

Description
Add support for escaped JSON strings. The validation would not allow this previously, even though the beautify() method can format it.
Screencast
https://github.com/user-attachments/assets/17809cd7-8131-4500-a8b2-39440a33339d

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Does anyone mind explaining what are those credits for? I've just linked my Github and don't seem to find any page related to it. Thanks


Check that pickadate and pickatime exist before merging

Loading a translation file could cause js errors in the case that you have not loaded both picker.date.js and picker.time.js. It should not be necessary to load a time picker if you only need a date picker and vice versa. By checking first, it is now safe to load a translation even if you are using only one or the other.

Related: https://github.com/amsul/pickadate.js/pull/744

In case it's not obvious why this solution works, the expression:
jQuery.fn.pickadate && jQuery.fn.pickadate.defaults
might look like it should return a boolean but it does not in javascript. If the left side is falsy, it returns the left side without evaluating the right. If the left side is truthy, it returns the right side. So what really happens here is, if pickadate is a defined key on jQuery.fn, then the defaults object is returned (and subsequently extended). If pickadate is not defined, it resolves as falsy and we don't try to do anything with the (obviously) undefined pickadate.defaults.

See https://api.jquery.com/jquery.fn.extend/ and https://github.com/amsul/pickadate.js/blob/master/lib/picker.js#L1088
These should be objects. What you have is a boolean check.

It's a floor wax AND a dessert topping!

@DanielRuf Do you see why it works?

As mentioned in the other PR, fixing the order of the imports is the correct fix. Otherwise your translations wouldn't work properly.


nix-query-tree-viewer: 0.2.0 -> 0.2.1

Motivation for this change
Bump nix-query-tree-viewer to latest version.
Things done

Built on platform(s)

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

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

ofborg succeeded, so I will go ahead and merge this in.


[FEATURE] Improve the video section #355

Working on possibility to embed multiple videos on the page.
Files included:

content/videos.json
components/GetVideos.js

Working:
Include multiple videoIds in content/videos.json in json format and all the videos will be available in the Video section of the site

Good job @AnvaySingh!!


Fix issue #318

Fixed the bug which the labour union modifier affect the entire country .
The syntax core_pop_xxx_modifier is only used in country scope therefore it would affect all the population in any core_province.（It has nothing to do with the pop culture types.）

yf22 :+1: Welcome to NWO modding :)


Conda Environment File

Now that we got rid of all the different conda environment files, how are users gonna setup an environment for Python 2 for quest? Are they going to have to just change the python version in the yaml file?

Yes. That is the plan. We need to add that to the documentation (as a deprecated but still supported installation). I see this as a step in phasing out Python 2.

Okay I just wanted to make sure that was the direction we wanted to take, because now if someone that does not know conda will not be able to setup a python2 environment.


pylab import replaced by 'py'

Hi,
The pylab import is seen as a py module
Create a file with just that import
[chaen]$ cat /tmp/test.py
from pylab import hist

Run findimports on it
[chaen]$ findimports /tmp/test.py
test:
  py

The installed version is 1.5.1
[chaen]$ pip freeze | grep findimports
DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
findimports==1.5.1

Any idea what is going wrong ?
Thanks !

Oh, this is interesting!
I cannot reproduce it:
$ virtualenv -p python2.7 /tmp/py27

$ /tmp/py27/bin/pip install findimports
Successfully installed findimports-1.5.1

$ cat /tmp/test.py
from pylab import hist

$ /tmp/py27/bin/findimports /tmp/test.py
/tmp/test.py: could not find pylab.hist
test:
  pylab.hist

$ /tmp/py27/bin/pip install matplotlib
Successfully installed backports.functools-lru-cache-1.5 cycler-0.10.0 kiwisolver-1.1.0 matplotlib-2.2.4 numpy-1.16.5 pyparsing-2.4.2 python-dateutil-2.8.0 pytz-2019.3 six-1.12.0 subprocess32-3.5.4

$ /tmp/py27/bin/findimports /tmp/test.py
test:
  pylab

Can you tell me what findimports /tmp/test.py -d would print?  For me it looks like
digraph ModuleDependencies {
  node[shape=box];
  mod0[label="test"];
  node[style=dotted];
  extmod0[label="pylab"];
  mod0 -> extmod0;
}

and the reason I ask is I want to make sure it's not the plaintext output getting truncated midword (although I've no idea why it would be truncated midword).
Another test would be to find out what Python interpreter is used for findimports with
$ head -n 1 $(which findimports)

and then run that interpreter and in it try
>>> import pylab
>>> pylab.__file__

which for me produces '/tmp/py27/local/lib/python2.7/site-packages/pylab.pyc'.
Actually, I have a Theory.  Bear with me a moment.

Reproduced!

virtualenv /tmp/py
/tmp/py/bin/pip install py findimports
/tmp/py/bin/findimports /tmp/test.py

produces the same output as your code.
And the reason is that in this loop:
https://github.com/mgedmin/findimports/blob/caa0e71f2d87fdfc09fb4999f02780d2f4649214/findimports.py#L502-L509
I'm trying to take off the last part of the dotted name with
        name = name[:name.rfind('.')]

and name.rfind('.') returns -1 when there are no dots in it, which cuts off the last character, one by one, until pylab becomes py, which exists in your site-packages!

(In my defense I originally wrote this code in 2003.  Python 2.3 was the modern version.  str.rindex ... existed, so I've no excuse!)

Released findimports 1.5.2 with the fix.

Woh, thanks a lot, impressive response speed :-)


Build & test AWS Managed Microsoft Active Directory

User Story
As a Modernisation Platform Engineer
I want to create a short-term deployment of AWS MMAD
So that I can identify any missing requirements prior to creating this for a Modernisation Platform customer
Value / Purpose
If we create an AWS MMAD in core-shared-services we can confirm that the SSO role created in this story is appropriate. We can also identify any potential missing pieces ahead of creating something for a customer at a later time.
Useful Contacts
@dms1981
Additional Information
https://github.com/ministryofjustice/modernisation-platform/issues/5736
We don't need to register a domain in Route53 as a resolver rule will do.
Proposal / Unknowns
No response
Definition of Done

[ ] AWS MMAD created
[ ] AWS MMAD access tested through SSO role-based login
[ ] AWS MMAD destroyed

Built a new Managed AD cluster in sprinkler for testing purposes

Created Managed AD management instance for end-user testing. Will place this in blocked until it's been evaluated, then remove the MMAD terraform and close this issue.

Removed resources through these PRs:

https://github.com/ministryofjustice/modernisation-platform-environments/pull/4514
https://github.com/ministryofjustice/modernisation-platform/pull/5959
https://github.com/ministryofjustice/aws-root-account/pull/855


try to get a separate swc compile with typecheck

I am not seeing a speed improvement from this.

Done as part of the move to remix.


Custom opening functionality bound to click event causes the popup picker to close instantly

With the following HTML:
<div id="emoji-wrapper">
  <button id="emoji-button">Toggle emojis</button>
</div>

The following initialization will cause the emoji picker popup to close instantly after clicking the "emoji-button":
import { createPopup } from "@picmo/popup-picker";

const wrapper = document.getElementById("emoji-wrapper");
const picker = createPopup({
  rootElement: wrapper
}, {
  position: { top: 10, left: 10 }
});
wrapper.querySelector("#emoji-button").addEventListener("click", () => {
  picker.toggle();
});

This is because the "click" event propagation is not prevented in the event listener which causes it to propagate to the document object which is listened here:
https://github.com/joeattardi/picmo/blob/9948a812ba13e99cb66722c8ec5c60812c643a44/packages/popup-picker/src/popupPicker.ts#L68
A workaround is to stop the event propagation at the listener but I just thought it might be a good thing if the document listening was omitted during the opening of the picker.
Example:
https://jsfiddle.net/5m4p0yw7/

You have two issues with the code:
(1) When creating a popup picker, you need to specify the triggerElement. Otherwise, when you click the button, PicMo thinks you clicked outside of it and closes, which is what you are seeing here. triggerElement should reference the button you click to toggle the picker, and should be passed with the popup options (second parameter to createPopup).
(2) Popup pickers should not have a rootElement defined - the popup picker creates a root element of its own to show in a popup.
See https://picmojs.com/docs/usage/creating-pickers#popup-elements for more details.

(1) When creating a popup picker, you need to specify the triggerElement. Otherwise, when you click the button, PicMo thinks you clicked outside of it and closes, which is what you are seeing here. triggerElement should reference the button you click to toggle the picker, and should be passed with the popup options (second parameter to createPopup).

Great thanks!
I did try triggerElement but I just thought it didn't work because I thought that passing the button element to it, it would automatically handle opening the picker when clicking it. Now that I pass it in the triggerElement it works fine.

(2) Popup pickers should not have a rootElement defined - the popup picker creates a root element of its own to show in a popup.

Well, that's the only way that I found to:

Position the picker absolutely
Not have the auto-positioning kick-in for messing up the position of the element

We want it to be stuck at the bottom of the <textarea> element because otherwise it will position completely weirdly in our use case
The element jumps at the top of the page on the first open when using the auto-positioning which makes it feel broken

I am basically using the "fixed" positioning but passing it { position: "absolute", right: 0, bottom: 0 } and it works perfectly for our use case and matches how we used to position emoji-picker.
Also, I found this the only way to force the picker element inside a specific container as this line is using this option:
https://github.com/joeattardi/picmo/blob/c435ece52fc4a05c2412b334a8dcccbb099b0673/packages/popup-picker/src/popupPicker.ts#L142
This is very much needed in our use case to be able to:

Force the element inside a specific container (with relative positioning)
Force the element at the bottom right corner of the container (using position: { position: "absolute", right: 0, bottom: 0 })

Otherwise it was not working how we want it to work.

Did you try using relative positioning and using your textarea as the referenceElement? That should work, if there is a bug with relative positioning and the element jumping around, we can open a new bug issue for that. Thanks!

There's a wrapper around the <textarea> that has relative positioning. I believe the <textarea> itself is not relatively positioned. It lives at the very top corner of the wrapper.
We'd rather have the auto-positioning skipped because it seems unnecessary + utilizes unnecessary extra CPU cycles at the client-side where as the relative+absolute positioning works totally fine.
I can try to re-create the situation at codepen later.

I don't think relative positioning is going to cost you anything appreciable in CPU time. It should be a one-time position calculation.
Here's an adaptation of your fiddle with relative positioning around the textarea: https://jsfiddle.net/oavzb5gn/17/

OK thanks, I guess the rootElement option was what was messing the positioning up. We want bottom-end but without the rootElement option it is now working fine without the "hacks".
But I believe there are still extra CPU cycles that I don't feel are necessary, as this line:
https://github.com/joeattardi/picmo/blob/c435ece52fc4a05c2412b334a8dcccbb099b0673/packages/popup-picker/src/positioning.ts#L44
Is calling all of this:
https://github.com/floating-ui/floating-ui/blob/fa9ae9acab663c124bb856144da6b5405fb5aea9/packages/dom/src/autoUpdate.ts
Which is not really necessary in case we just want the element to stick to the bottom corner of the element.
There's one more issue I need to figure out with the mobile styling compared to the old EmojiPicker we are trying to replace. The old version used to pop-up a dialog with a dimmed out background but the new version with the current configuration spans outside the screen with 375px wide screen (iPhone SE in Chrome developer tools) and about 40px gutters on both sides of the comment area.
But I'll continue investigation regarding this issue. Thank you for your outstanding support!

I just tried the suggestions and here is a situation close to what I get in the actual application:
https://jsfiddle.net/b32mLozh/
It may be better to look it at the full screen mode here:
https://jsfiddle.net/b32mLozh/show
There are two problems with this approach:

We would want the picker to be on top of the text area, at the very bottom-right corner
We would want the picker to stay at the bottom corner and not jump around when you scroll up and down

This was the initial problem that I wanted to solve by forcing the absolute positioning what I ended up using.
I figured the mobile view in the previous version is something that is not implemented in the current popup picker and we probably need to extend the popup picker to fix the mobile positioning. The current "jump around" functionality actually works quite well for mobile but it's not what is desired on desktop.


Fix setMute setting incorrect isMuted

#1966 incorrectly set wavesurfer object's isMuted to true even when it's supposed to be false.
This is not an issue in practice, as it gets correctly re-set by the volumechange handler, but this is wrong anyway, and as a result make the tests fail.

Coverage remained the same at 80.456% when pulling e3bae1332a26c1404e416091227e9f2b0e89d731 on ThibG:fixes/setMute into a5b488421643cc8c90c543377cd659ac7ab38155 on katspaugh:master.

thanks. did you also run the tests locally and see if they pass @ThibG?


Add support for 3440x1440 widescreen 21:9 monitors

I love the software, its great, but i usually use UTM with an external widescreen monitor.
Tried out the apple virtualization feature and found out that the widescreen resolution 3440x1440 aspect ratio 21:9 wasn't supported.
So creating this issue to raise awareness. Will also submit a PR that links to this issue.
Great software, keep up all the great work!

PR is submitted here #4669

Install SPICE Guest Tools on your VM. Go to VM settings>Display>Resize display to window size automatically.
Enter Full Screen

Install SPICE Guest Tools on your VM. Go to VM settings>Display>Resize display to window size automatically. Enter Full Screen

I already have SPICE guest tools installed, and the settings you are suggesting do not exist when running apple virtualization. They might exist when running QEMU, but im not running QEMU. Please look at my PR.


Can this be load balanced for redundancy?

If so, can a general high-level recommended route be posted here? I will post how I did it in detail after I have it working. I have access to F5 LTMs, just about anything else.
I am building a clustered demo of LogStash with redis clusters for my Company Cengage Learning in an effort to  offset or replace the extremely expensive Splunk tool we are currently using.
I am not looking for step by step how to, just a path that the developers think would work or better yet had tested.

@rhoml can you tell me more on how you are haproxy on twemproxy?

@pank7 what we do basically is to setup haproxy on TCP mode in front of twemproxy and we have a pool of twemproxy configured on each endpoint. This ensures that the requests are balanced and in case of any failure haproxy will take them out of the pool.
pretty basic straight forward.

what we do basically is to setup haproxy on TCP mode in front of twemproxy and we have a pool of twemproxy configured on each endpoint. This ensures that the requests are balanced and in case of any failure haproxy will take them out of the pool.
pretty basic straight forward.

Yes, it can be load balanced in any of the following ways

HAProxy in tcp mode to random instances works
If there are a lot of independent connections being made (e.g. php+apache) it's useful to choose a single random twemproxy instance (immediately before connecting) to connect to
Alternately, you can pass the full list of addresses of twemproxy instances to the application using twemproxy, if there is a single client (or fixed-size client pool) in an application uses long-lived connections (e.g. java)

Twemproxy is single threaded (the stats thread only processes requests on the stats port), so it will hit a bottleneck at 100% cpu - in applications that heavily use memcached the number of instances of twemproxy I've used is around 1/10th of the cpus (e.g. on a 96-core machine, use 10 nutcracker instances).


Can I get unbanned from the discord please?

I just want to be unbanned, I believe I spammed a few times so I didn't get kicked for being inactive by the purge that happened.
WolfoTheWolf#2764

spammed

Looks like you answered your own question


release: v1.0.0a16

Signed-off-by: Lars Holm Nielsen<EMAIL_ADDRESS>

Coverage remained the same at 97.994% when pulling a733ad91b01ccb7f09341020720b882ccf77842f on lnielsen:rel-v1.0.0a16 into 512c76b7b0e6c3734260bbea884d00fce44d8879 on inveniosoftware:master.


Estuary Selective Events to Persist

As Andy I would like to add stability and scalability to the system by only persisting events that I care about through Estuary.
AC:

[x] Add an environment variable that is a list of event types
[ ] Events in that list should be persisted.
[ ] Events that are not in that list should not be persisted
[ ] Events to persist:

Tech Note:
Something along these lines would be a good starting point
charts/deployment.yaml
...
          env:
          - name: EVENT_TYPES_TO_PERSIST
            value: {{ .Values.eventTypesToPersist }}
...

charts/values.yaml
eventTypesToPersist:
  - dataset:update
  - error:dataset:update
  - organization:update
  - error:organization:update
  - dataset:disable
  - dataset:delete
  - data:write:complete
  - dataset:harvest:start
  - dataset:harvest:end

https://github.com/Datastillery/smartcitiesdata/pull/1134
https://github.com/Datastillery/charts/pull/242
https://github.com/SmartColumbusOS/estuary-deploy/pull/40

This is incomplete - needs more work on the PR: https://github.com/Datastillery/smartcitiesdata/pull/1134


Optimize reading of aggregates

This PR fixes #60.
Recently, Storage.read(...) method was reworked. See details in the PR.
DsAggregateStorage was updated accordingly and now it uses a batch size from the newly introduced AggregateReadRequest to read a history of an aggregate by batches.
Spine version was increased to 0.9.75-SNAPSHOT.

@armiol PTAL.

@armiol PTAL.

@armiol PTAL.


test: update tests, add github actions reporter

Description

Coverage increased (+3.3%) to 54.098% when pulling 5dcf5c0a2cc121b155d77ebb14311540c7db8b56 on abrown/test-suite into 462807e43652b2063d3d5e0504526a30e25acd57 on main.


Bring back regionEdge option

Is your feature request related to a problem? Please describe.
Before v2.0.0, we had the ability to restrict the range of action of the rails and hedges commands to our selection. (This was previously knows as /cs rails regionEdge and /cs hedges regionEdge)
It was very handy for not drawing lines over large areas.
Describe the solution you'd like
Add an option in the JSON object of the different commands to do so: rails, railsnear, hedges, and hedgesnear.
Examples that might be suitable
/cs rails {"restrict":true}
/cs rails {"edges":true}

Additional context
The function that was responsible for this behavior can be found here  https://github.com/oganexon/BTE-tools/blob/9a02c4d232a0ba043f6c1ffb49a387c7bc7014d2/src/modules/drawLines.js#L48-L54
This should be easy for the rails and hedges commands as the region is already selected by the user.
For railsnear and hedgesnear it will be harder since a new region has to be created from code (cylindrical I think).

On It

Added in v2.1.0


[PE-D][Tester C] UI display Issue

i cant seem to scroll down to see the details

Labels: type.FunctionalityBug severity.Medium
original: Vanessamae23/ped#6

Invalid issue, user expanded application to beyond their screen resolution.


Fix bug header version

WHY

found a bug where I have implemented
when get the version from Accept-Header.

WHAT

fix bug

@shimastripe Could you show me the sample header?
I'm unsure whether is it okay to separate Accept header string by semicolon... It seems that RFC 2616 indicates the separator is comma.

@dtan4
First, I think this way.
Accept: application/vnd.wantedly+json; version=1.0.0

Then I write a code as follows.
ver = strings.Split(header, "=")[1]

But  "=", it is also used such as in the case of a charset.
Accept: application/vnd.wantedly+json; charset=utf-8; version=1.0.0

So I improve the code as follows.
ver = strings.Split(strings.SplitAfter(header, "version=")[1], ";")[0]

The separator of the accept-header's list seems to be only ";".

I got it. LGTM 👍 Thanks ❗


Move testInsertWithCoercion to engine-only tests

The coercion during insert is applied exclusively by the engine, so
there currently is no need to exercise the test against connectors.

Not the red maven checks.

CI https://github.com/prestosql/presto/issues/6412


Add documentation. #trivial

This adds some documentation around Endpoint, Image, Response, and MultiTarget.

Current coverage is 75.39% (diff: 100%)

Merging #899 into master will not change coverage

@@             master       #899   diff @@
==========================================
  Files            20         20
  Lines           699        699
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            527        527
  Misses          172        172
  Partials          0          0

Powered by Codecov. Last update 17e9d44...503edd2

We might want to unify the header docs at one point (only /// or your approach with \**), but lets leave that for another time.

Thus far my approach is to use /// for single line documentation comments and /** for multiline documentation comments, but I agree we should make sure it is consistent. I was having trouble getting the markdown (e.g. - parameters: to correctly register on multiline with ///.

You might have to use - parameter name: name does thing per parameter with ///.

Ah, that might be it then. In that case, I think:
/**
 function

 - parameters:
    - x: Is an x
    - y: Is a y
*/

might be a touch more human readable than
/// function
///
/// parameter x: Is an x
/// parameter y: Is a y

But that's just personal preference. I could go either way.

I agree, but I think I like /// above /**. 😛


ValueError: No valid option generated in #select!

The bug
The "select" option with samples in Rusiian raises "ValueError: No valid option" exceptions. A similar construction with examples  in  Spanish or Dutch works fine.
To Reproduce
Russian sample (doesn't work)
import guidance
from guidance.llms import OpenAI
dvc = OpenAI("text-davinci-003")

program = guidance("""Imagine you are a sale assistant.
First step, determine the text's language.
Second step, you have a product name, determine which category it belongs.
"PRODUCT": "{{product}}"
"CATEGORY": "{{#select 'CATEGORY'}}Арматурный прокат{{or}}Сортовой прокат{{or}}Фасонный прокат{{or}}Unknown{{/select}}"
""", llm=dvc)

program(product="Арматура А-1 d8 4.55т")

>> ValueError: No valid option generated in #select!

Spanish sample
import guidance
from guidance.llms import OpenAI
dvc = OpenAI("text-davinci-003")

program = guidance("""Imagine you are a sale assistant.
First step, determine the text's language.
Second step, you have a product name, determine which category it belongs.

First step, determine the text's language.
Second step, you have a product name, determine which category it belongs.
"PRODUCT": "Racores A-1 d8 4.55t"
"CATEGORY": "Barras de refuerzo"

System info (please complete the following information):

OS Ubuntu: 22.04.2 LTS:
Guidance Version (guidance.__version__): '0.0.61'

Please try again with the new release :)
Sorry it took us so long to get to this


Two Oversights

Woke up and remembered a couple of things I forgot about.

[X] Clicking Select All or Deselect All with the Skybox CheckedListBox active will crash, forgot to implement it.
[ ] The Custom Music Textbox can end up line wrapping, noticed when adding a load of music, forgot to ever fix.

First one fixed in https://github.com/Knuxfan24/Sonic-06-Randomiser-Suite/commit/d10033a71c587b6c32be3fe862982c29906a1ee9
Second one... Turns out the TextBox already has TextWrapping set to NoWrap, but it does anyway after a very long value so...

Not sure if I can actually fix that second one, so I'm just throwing this out.


[EMH] Implement alternative versions of Optional[...]

Describe your issue
It seems that implementing generic union types (X | Y and Union[X, Y]) are difficult to implement and are rejected. We can try to implement reduced version containing None since it is often used instead of typing.Optional[]. We can transform it as follows:

X | None = None | X  -> typing.Optional[X]
X | Y | None = X | None | Y = None | X | Y -> typing.Optional[object]
typing.Union[X, None] = typing.Union[None, X] -> typing.Optional[X]
typing.Union[X, Y, None] = typing.Union[X, None, Y] = typing.Union[None, X, Y] -> typing.Optional[object]

This issue is specialised version of #4631

X | Y | None = X | None | Y = None | X | Y -> typing.Optional[object]
typing.Union[X, Y, None] = typing.Union[X, None, Y] = typing.Union[None, X, Y] -> typing.Optional[object]

These two aren't hugely worthwhile in Cython (because if you type something as object we never generate code that assumes it isn't None). They're harmless, but covered by the current behaviour of "annotation that Cython doesn't understand -> object"
X | None = None | X -> typing.Optional[X]
typing.Union[X, None] = typing.Union[None, X] -> typing.Optional[X]

These two are useful. Especially the one with | since I think that's what most people would choose to write. I have no idea how easy or hard they are to implement.


Control dynamic kubectl download

There are a few cases where dynamically downloading the kubectl client may not be what the bundle author desired and we should make it controllable by them:

kubectl is backwards compatible. When the bundle has a more recent version of kubectl, there isn't a reason to download an older version to match.
There may be bug fixes in newer patch versions of kubectl so matching down to the patch isn't always a great behavior when what we really need is API compatibility.
The bundle may be self-contained and not want to reach out to external resources, perhaps a thick bundle, and attempts to dynamically download the client should be strictly disabled.

Based on this (please add more in the comments below as you see them), I propose that we add the following configuration to the mixin:
mixins:
- kubernetes
     dynamicDownload:
       versionMatch: exact | minor
       useBackwardsCompatible: true
       disabled: true

dynamicDownload: Defines the behavior of the dynamic kubectl client download

versionMatch: Defines how the api server version is compared to the client version to determine whether or not a different client should be downloaded. Defaults to minor.
A download is triggered when:

exact - The versions do not match exactly.
minor - The Major.Minor versions do not match. The patch version is allowed to be different.

useBackwardsCompatible - When the client is newer than the server, take advantage of the kubectl backwards compatibility guarantee and do not download an older client. Defaults to true. (I tried to make a name that worked well for defaulting to false and couldn't come up with one)
disabled: Specifies if the dynamic download feature is disabled. Defaults to false.

With the defaults someone who doesn't customize the config settings will only download a client when the server is a newer Major.Minor version. What do you think?

This looks good to me.

minor - The Major.Minor versions do not match. The patch version is allowed to be different.

Should be  minor - The Major.Minor versions match. The patch version is allowed to be different., right?
My only other thought is to think about if this will prove to be generically useful across mixins enough to consider placing somewhere in a/the main Porter repo such that it's inherited by any basic mixin implementation (maybe clientVersion could be included at the same time).  However, it doesn't quite fit in exec's builder pkg, where we've previously placed similar interfaces... so this would probably be a separate/broader design/discussion.

Oh we can tweak the wording. The sentence above was "A download is triggered when: ..." so the condition for triggering a download is when the Major.Minor doesn't match. But we can flip it to talk about matching instead, whatever is less confusing.
Some of the logic we are discussing here is specific to kubectl (such as the backwards compatibility) so we would want to use interfaces again like we have in the builder package to opt into various behaviors. So yeah I think it would work well to be a common package in porter. We can find a spot for it, maybe a subpackage under pkg/mixins/ would be a way to do it going forward. But that can be worked out in an issue in the Porter repo.

I really love the idea of configurable dynamic download and the adaptation of this feature to other mixins. Moreover, it could resolve this question about downloading user-defined mixins see: https://github.com/getporter/porter/issues/975
I wonder when the download would happen. Is it at build-time or later at runtime? as a universal behavior

I'm glad you are interested in this path!
The dynamic installation of the kubectl client must occur at bundle execution. This is because it relies upon credentials to connect to the user's (not bundle author's) cluster to identity the targeted cluster version. Every time the bundle runs, it can be used against a different cluster with a different version of Kubernetes.
The mixin configuration allows the bundle author to specify the default kubectl client version that should be installed at buildtime. When it is not specified, it will fall back and install the version hard coded in the mixin. We can also improve the mixin to always install the latest version of kubectl which I think would be a welcome change.
This gives us a minimum of one and possibly two downloads. There is always a download that occurs at buildtime which guarantees that there will always be a kubectl client in the invocation image. This client may then be optionally used to detect the cluster version at runtime and then download a matching version dynamically.

For the other issue you referenced, getporter/porter#975, mixin installation always occurs at buildtime. Mixins are built into bundle during porter build and never needed to be installed on the client in order to execute a bundle (runtime). If you'd like to chat about that issue further, let's do it on that issue so that we keep the discussion in one place.


Remark Plugin for Headings

Motivation
To address issue #67 about the need to automatically link markdown headings for the convenience of users being able to share specific parts of blog posts.
Approach
I've looked at gatsby-remark-autolink-headers as an option but that plugin does not wrap the entire header as a link but only creates id and adds the share icon next to the heading (the icon would be the link in that case) so I had no choice but to create our own plugin for our use case.
Output
When gatsby-transformer-remark transforms markdown files into HTML, the plugin will catch all the markdown headings (with the exception of h1) and output the following:
<h# id={slugged-headline}>
  <a href="#${id}" class="header-link ${h2 ? '' : 'not-h2}">
    ${headline}
  </a>
</h#>

How does it work?

<h1>: the plugin doesn't do anything with it
<h2>: gets assigned an id and heading and icon are wrapped with an a tag but the share icon does not show up unless user hovers over the heading
<h3> && <h4>: same treatment as <h2> but icon visible as a different color when not :hover

You can alter the behavior output of the result of this plugin from ./plugins/gatsby-remark-header-link/index.js under node.value.
And ./templates/blog-post.css is where we can style these headings.

The only markdown files that are using headings in its content are blog posts so we can just focus on modifying the styles in blog-post.css for now.
We can also add default style attributes to the plugin itself if deemed necessary.
I just put placeholder svg files for demonstration purposes so those will need to be modified.

Preview
You can best see how it works on this blog post preview of this pull request as it contains various levels of headings.

This is great Min, thanks!
The only thing is that there is a package-lock.json, it seem that you're using npm for installing dependencies. However, we mostly use yarn and thus there's a yarn.lock already in the repo.

@jorgelainfiesta Removed package-lock.json. Do we need yarn.lock? Sometimes we include the lock files in .gitignore and sometimes we allow it to be pushed. 🤷♂

From my understanding, we do need yarn.lock to ensure we all run the same dependencies when developing and deploying. Many dependencies are specified in 'loose' terms, like ^2.4.0 which can resolve to slightly different versions.
I would always push the lock file on the repo, perhaps there are exceptions but I can't think of which circumstances lead to ignoring it.


Updated Security Groups.  Split outputs file.

Uncomment SSH ports in security group config
Add DB2 port (8100) to security group config
Split outputs into separate .tf file per best practice

Thanks for your contribution @jnewton03 !


fix typos

src/SymbolicControlSystems.jl

@baggepinnen
Do you want to approve the workflow?

Thanks for your PR, but I will choose to not merge this particular one. This PR changes two comments only, and those two lines of code are better left pointing to their original commits introducing them in the git blame to make it easier to figure out the background to why they were introduced in the first place.


Fixed link, input field and footer

Fixed issue #949
Hello PA,
I have fixed the link, input field and footer. Kindly check the screenshot below.
Browser-Firefox
Screenshot

Kindly merge the PR

@codervivek5 You can merge this.

Thankyou for your contribution :)


Unable to resolve path to module for .jsx files

Seems that currently files with .jsx extension are not supported by the resolver.
This list of extensions:
https://github.com/alexgorbatchev/eslint-import-resolver-typescript/blob/18eafa26db7cee5fcdbccfe211d1c412a1e899b9/index.js#L28-L32
does not cover .jsx. Adding it, fixed the issue for me. Would this be an acceptable fix? I can prepare a PR.

Are you using allowJs with .jsx?
You can create PR to https://github.com/rx-ts/eslint-import-resolver-ts/blob/master/src/index.ts#L17 which is a fork and enhacer for it.

@pekala PR welcome.


toolbar is wider than the screen

When in portrait mode on iPhone.

i believe this is fixed as i wrapped the toolbar in a scrollarea


[branch-2.10][cherry-pick] Close TransactionBuffer when MessageDeduplication#checkStatus failed

Motivation
Cherry-pick https://github.com/apache/pulsar/pull/19157 into branch-2.10
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@tjiuming Could you please help check the failed test? It looks like introduced by this PR.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


"r?" silently does nothing when the user is not a reviewer

In Miri, this is our configuration. If we now do r? @user, then nothing happens -- nobody gets assigned, but also no error is shown.

Tried to look at the triagebot logs but don't have enough visibility. I suspect something fails when the triagebot tries to pick someone from this this file but I don't see errors, this could be why the silent failure.

r? requires an assign.owners table to work. This was required to support the transition from highfive (so two bots didn't respond to every PR). Otherwise it is ignored.
It may be possible to do something like have an empty owners table to enable it and support something like #1806, but currently that is not supported. In the meantime, you can do the more explicit syntax of @rustbot assign @user


Firewall / NAT: support category filters

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I am convinced that mine is new.

Is your feature request related to a problem? Please describe.
Categories are currently only supported in the firewall section itself, when utilising large number of port forwards and/or outbound nat rules it could be practical to use the same concept there as well.
Describe the solution you like
Move the categories into a separate module to be used in other firewall sections as well and refactor the current legacy pages for outbound nat rules and port forwards to include categories. In a future version there's also a wish to support multiple category tags on a single rule, in which case we would replace the current type ahead input for the same tokeniser used in other components.
Describe alternatives you considered
none
Additional context
By moving the categories into its own model we also might consider options like color tags in the future and can reuse items in different sections. In case we would eventually replace the legacy input screens for api enabled ones these patterns should be easy to adopt as well.

sponsored by : Modirum (https://www.modirum.com/)

Colors would be nice, so when you have e.g. 3 categories selected you still see which rule is for which category

Colors would be nice, so when you have e.g. 3 categories selected you still see which rule is for which category

@AdSchellevis are you aiming for 21.1?

@AdSchellevis are you aiming for 21.1?

@fichtner no promises, but giving it a try

@fichtner no promises, but giving it a try

no worries, better flip milestones then :)

no worries, better flip milestones then :)

@AdSchellevis do I have to adapt this for the other themes (cicada, vicuna e.g)? What exactly has to be done? regards rené

@AdSchellevis do I have to adapt this for the other themes (cicada, vicuna e.g)? What exactly has to be done? regards rené

@opnsenseuser I don't expect you will need changes for the themes, the standard theme wasn't changed either for this and the category selector itself was already in the firewall page, only the backend code changed (the badge on the right will likely also work on the other themes, colours are selectable). I think the color selector is the only small item that might have a small risk of looking different, the layout of the component is sort of standard.

@opnsenseuser I don't expect you will need changes for the themes, the standard theme wasn't changed either for this and the category selector itself was already in the firewall page, only the backend code changed (the badge on the right will likely also work on the other themes, colours are selectable). I think the color selector is the only small item that might have a small risk of looking different, the layout of the component is sort of standard.

@AdSchellevis Just tested the new functionality, looks really like a big step head in organizing a big amount of rules. 👍🏻
Some minor UI topics I found:

Helptext of the "automatic" checkbox reads: Automatically added, will be removed when used I guess it should read "unused"?
Does it make sense to have the checkbox read-only?
Fields in the form are written in uppercase in all other forms, except the category add/edit form (Color instead of color). Also the tab in the view is written in lowercase currently. --> #4634

What's your opinion on following improvements:

Display mouse-over text when hoovering over a coloured bullet.
Add "(no category)" to the filter box at the top to allow identifying rules without category.

@AdSchellevis Just tested the new functionality, looks really like a big step head in organizing a big amount of rules. 👍🏻
Some minor UI topics I found:

Helptext of the "automatic" checkbox reads: Automatically added, will be removed when used I guess it should read "unused"?
Does it make sense to have the checkbox read-only?
Fields in the form are written in uppercase in all other forms, except the category add/edit form (Color instead of color). Also the tab in the view is written in lowercase currently. --> #4634

What's your opinion on following improvements:

Display mouse-over text when hoovering over a coloured bullet.
Add "(no category)" to the filter box at the top to allow identifying rules without category.

@8191 Thanks, I've corrected the help text. We don't need to change the property to read-only, since you may choose to persist the automatic ones as well (remove marker). The proposed additions look practical, if you open a ticket, I'll try to tale a look later.

@8191 Thanks, I've corrected the help text. We don't need to change the property to read-only, since you may choose to persist the automatic ones as well (remove marker). The proposed additions look practical, if you open a ticket, I'll try to tale a look later.

@AdSchellevis i need a few little color changes. can i put the modified pick-a-color-1.2.3.min.css into each theme folder? Like >> /build/css/pick-a-color-1.2.3.min.css. Does this work?
Regards René

@AdSchellevis i need a few little color changes. can i put the modified pick-a-color-1.2.3.min.css into each theme folder? Like >> /build/css/pick-a-color-1.2.3.min.css. Does this work?
Regards René

@opnsenseuser that should work indeed, it picks the stylesheet from the theme folder first.

@opnsenseuser that should work indeed, it picks the stylesheet from the theme folder first.

👍

👍

@AdSchellevis I just wanted to ask you what you think of adjusting the category color window to the width of the dropdown menu. i know that this is not important for a firewall, but a gui has to look tidy for me, then it is clearer and it is also more fun. what do you mean?

@AdSchellevis I just wanted to ask you what you think of adjusting the category color window to the width of the dropdown menu. i know that this is not important for a firewall, but a gui has to look tidy for me, then it is clearer and it is also more fun. what do you mean?

@opnsenseuser that looks neat indeed, if it's only a single diff we need to change with an additional style, I don't mind that much, for the standard theme I do prefer to keep the default stylesheets as much as possible to avoid future breakage when new versions are released.

@opnsenseuser that looks neat indeed, if it's only a single diff we need to change with an additional style, I don't mind that much, for the standard theme I do prefer to keep the default stylesheets as much as possible to avoid future breakage when new versions are released.


Use Moshi instead of Gson. Moshi is a Kotlin friendly lib

Why not kotlinx.serialization if the rationale is Kotlin friendliness? 🤔


[BUG]

What happened?
-With Discord Integration installed, my Forge 1.15.2 (31.2.45) server hangs when trying to restart. I try to restart the server, however it just hangs. I have singled it out to be specifically dcintegration-1.1.14-1.15, as without the plugin, the server successfuly restarts without having to force kill it.
What should happen?
-It prevents the server from succesfully restarting and stopping
Steps to reproduce

Have a Forge 1.15.2 31.2.45 Server
Install dcintegration-1.1.14-1.15
Try to restart server.

Demonstration of what happens WITH the mod installed in the mods folder: https://i.imgur.com/A9zvzjI.gif
Version

[ ] 1.12
[ ] 1.14
[X ] 1.15

Mod version 0.0.0
Config file
-https://paste.ubuntu.com/p/gbQtYFDvjm/
REMOVE YOUR BOT TOKEN BEFORE UPLOADING!!!
Crash report if available (use https://paste.dimdev.org)
-Here is a video that shows the server successfully restarts/stops without having to kill it (this is WITHOUT the mod installed):
https://streamable.com/pqo631
-Here is a video that shows the server does NOT successfully restart/stop without having to kill it (this is WITH the mod installed):
https://streamable.com/ily96j
Additional info

I just really want this to work

Duplicate of #58


Update the version of our Docker image

Updated some dependencies and integrated changes from sprint30.

Thanks, @zaneselvans!


Discrepancy between ZADD and ZRANGE score types

In rediscala 1.4.0 ZADD expects memeber scores to be of type Double, whereas ZRANGE expects start and stop to be Longs. Is that intended?
See commands/SortedSets.scala.

Yes it is
You want zrangebyscore http://redis.io/commands/zrangebyscore

http://redis.io/commands/zrange
Both start and stop are zero-based indexes, where 0 is the first element, 1 is the next element and so on.

@etaty thank you for a quick response! So yes, I mixed up the Redis commands. :-)


read files strictly in "Obelisk.Migration".

To avoid "too many open files" error.
This fixes #115 partly, I still get an error that is similar to the one on hydra:
No vertex found for HEAD (b0b562b83e8a198bb858deeb19d5868c)
✖ Checking existence of HEAD vertex

I still get an error that is similar to the one on hydra
No vertex found for HEAD (b0b562b83e8a198bb858deeb19d5868c)

Yea, see #116

thanks!

Yes; we should always be using Text anyway
On Thu, Jul 12, 2018, 11:27 Elliot Cameron<EMAIL_ADDRESS>wrote:

@3noch commented on this pull request.
In lib/migration/src/Obelisk/Migration.hs
https://github.com/obsidiansystems/obelisk/pull/117#discussion_r202078245
:

@@ -240,3 +240,8 @@ writeEdge dir actionFiles v1 v2 = do
forM_ actionFiles $ \fp -> liftIO $
writeFile (edgeDir </> T.unpack fp) ""
pure $ Just edgeDir

+readFileStrict :: FilePath -> IO String
+readFileStrict = makeStrict <=< readFile
Text.IO.readFile
https://hackage.haskell.org/package/text-<IP_ADDRESS>/docs/Data-Text-IO.html#v:readFile
is strict and seems more appropriate here?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/obsidiansystems/obelisk/pull/117#pullrequestreview-136704668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGlYMPHv65ex28VwPaZGLm6xBynWiORks5uF2rVgaJpZM4VNOU9
.

Requested improvements are incorporated now! :-)

Can we just use readFile from Data.Text.IO? It's explicitly strict and avoids manually packing.

Can we just use readFile from Data.Text.IO? It's explicitly strict and avoids manually packing.

Yeah, thanks for reviewing! The latest version of this PR already does this. Are you somehow reviewing an older version or is something messed up with my PR?

@eskimor Oops, I must have clicked on the first commit by mistake.


Add packages

The dependencies aren't consistently specified, but this is hard to do well until we actually have a bootable system working.

:+1:


Optimize pairings

This PR is to optimize the verification of signatures and proofs by exploiting a property of pairings.
Instead of doing 2 exponentiations, we can do only one final exponentiation after multiplying 2 points calculated by Miller loops.
In this way, we optimize the performance by avoiding one final exponentiation that would take about 14 ms, according to estimates from the noble-bls12-381 library.
The pairing function performs the final exponentiation by default but it's possible to change its behavior by introducing a third input withFinalExponent = false.

@microsoft-github-policy-service agree

FWIW, this is an important performance improvement.

Great, thanks @alessandroguggino.


Login Issue

Getting error message 404 not found after entering given user name and password.
Imported genhr.sql successfully.
need support.
deployed the package in CenOS with Nginx.

same problem I have, did anyone solve this problem please share. or recomment another solution. Thank you