fix telescope highlights regression from #386

the highlighting for the symbol type and the symbol name were lost with that change.
the telescope picker can be configured to display both symbol type and name and the line from the file.
#386 added highlighting for the line in the second part of the picker. however, doing so, it clobbered highlighting for the symbol type and name, which I am using thanks to the configurable get_highlight function to color differently public and private symbols.
this change makes sure the highlights for the first two columns are not clobbered by the line preview.
note, i'm not 100% sure about the offsets. i think there may be unicode widths shenanigans going on. but after this commit it's much better for sure.

I made some changes to hopefully account for multibyte characters. Can you check to see if it still works for you?

i think something is still off:

i guess too much bytes are given for the first column, which finishes too late, and one space is taken into account for the space between columns?
or maybe it's even more messed up, something is off with the preview here too:

without your changes i think the issue was mostly with the preview.

this is my commit without your changes btw:

and:

Hmmm...what version of Neovim are you on? And can you provide a small file that repros the weird highlights? This is what I'm seeing:

(with the icon and symbol name highlights overridden for testing, so they look bad but not incorrect)

hm, i didn't expect that you'd have issues reproducing the issue...
I'm using neovim 0.10.1. I'm on the telescope master branch, i wasn't at the latest commit but I now tried the latest commit and it's the same.
I also tried resetting my aerial configuration and it doesn't change anything.
This file with ft=typescriptreact shows the issue for me:
import React, { ReactChild } from 'react';

export default class BootstrapSelect extends React.Component<Props> {
  static defaultProps = {
    valueType: 'string'
  };
  selectElement: null | HTMLSelectElement;

  domValueToValue(val: string | string[]) {
    return val;
  }

  valueToDomValue(value: Props['value']) {
    let val: Props['value'] | null = value;
    return val;
  }

  dom() {
    return $(this.selectElement!) as any;
  }
}

i was also on a 3 weeks old nvim-treesitter, tried upgrading it to the latest, no change.

Hmmm...still can't repro. Can you try with this config? I did nvim -u repro.lua test.tsx and this is what I see.

-- save as repro.lua
-- run with nvim -u repro.lua
-- DO NOT change the paths
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "runtime", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

vim.g.mapleader = " "

local plugins = {
  "folke/tokyonight.nvim",
  {
    "nvim-telescope/telescope.nvim",
    dependencies = {
      "nvim-lua/plenary.nvim",
    },
    keys = {
      {
        "<leader>fs",
        function()
          require("telescope").load_extension("aerial")
          vim.cmd("Telescope aerial")
        end,
      },
    },
    opts = {},
  },
  {
    "nvim-treesitter/nvim-treesitter",
    config = function()
      require("nvim-treesitter.configs").setup({
        auto_install = true,
      })
    end,
  },
  {
    "emmanueltouzery/aerial.nvim",
    branch = "telescope_highlights",
    dependencies = {
      "nvim-treesitter/nvim-treesitter",
      "nvim-tree/nvim-web-devicons",
    },
    opts = {},
  },
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")

thank you for the help with troubleshooting, much appreciated!
i'll research this further, probably tomorrow. for now i can't make it work, it installs the tree-sitter parser but somehow aerial doesn't find it:

so I realized why I have the wrong offsets for the highlights.. As you guessed it was due to my config.
I didn't notice it because it was in the telescope part of my config, as opposed to the aerial part of my config... but i have this:
local entry_display = require("telescope.pickers.entry_display")
_G.aerial_displayer = entry_display.create({
    separator = " ",
    items = {
      { width = 2 },
      { width = 32 },
      { remaining = true },
    },
  })
  require('telescope').extensions.aerial.aerial({
    get_entry_text = aerial_elixir_get_entry_text,
    displayer = aerial_displayer,
    attach_mappings = telescope_center_mappings,
    sorter = sorter,
  })

in other words i customized the aerial telescope picker to have different column widths (4->2 and then 30->32). and that worked without issues until the syntax highlight was added.
Maybe it could be possible for aerial to allow the user to customize such values, but then again i don't want to make everything too complex in your code. So unless you see a reasonable way to achieve that, i guess you could just merge this PR to fix the highlights for the first two columns, and move on... If you do find a way to make this customizable, i would appreciate it... if you have an idea that'd suit you, you can explain to me, and i'll gladly implement it.

I have now pushed a third commit in the branch to allow the user to configure the telescope col1 and col2 widths, which completely fixes my issues.

Great! Looks good to me!


Split tests

This PR will split tests in log.spec.js into several test files to enable easier readability and maintainability.
Mor specifically, this PR includes the following changes:

Move CRDT tests to their own test file
Move heads and tails tests to their own file
Move log join tests to their own file
Move log loading (fromEntry) tests to their own file
Move log append tests to their own file
Make every test file use its own IPFS daemon
Use memstore in loading tests to make them run faster
Remove unused variables, fix linter errors

Closes https://github.com/orbitdb/ipfs-log/issues/153


Added an API which accepts a body of Content-Type: application/octet-…

Added Api as per assigned in issue number #20 which was assigned to me.
No core functionalities changed only added the api
https://github.com/foss42/api/issues/20
Contribution under GSSOC_24

The PR looks good. Just you need to remove
from fastapi.responses import JSONResponse
from fastapi import HTTPException

as we have pre-defined response utils (ok_200(),  internal_error_500()) already being used across the project.


Failed to create organization

Hi. I'm using the API to create a project and although the answer is 202 the project is not being created.
Is it necessary to have the organization created? Or it's created if the organization doesn't exist?
Thanks!

Hello @cmcheca, this requires an organization before you creating a project.


Nested p_map causing AssertionError in thread pool

Trying to nest p_map (e.g. to implement generating a square matrix) causes Python to barf out with a failed assertion.
Steps to reproduce:
from functools import partial

l = [1, 2, 3]

def add(a, b):
    return a + b

def gen_mat(x):
    nonlocal l
    return p_map(partial(add, x), l)

mat = p_map(gen_mat, l)
print(mat)

Expected Result:
[[2,3,4],[3,4,5],[4,5,6]]

Actual Result:
AssertionError: daemonic processes are not allowed to have children

same here


The new WSA version (2204) breaks compatibility

I tried to modify the new version (2204). While the Playstore gets installed, it blocks signing in, saying "This device is not Play Protect certified."

x2. tried registering product with no success

model name didn't recognized?

It gets recognized, but it is not validated

fix img size
fix build.prop location
login succeeded with open_gapps-x86_64-11.0-pico-20220215

extend_and_mount_images.sh

echo "Resizing system_ext.img"
#resize2fs $ImagesRoot/system_ext.img 108M
resize2fs $ImagesRoot/system_ext.img 256M

echo "chk vendor.img"
e2fsck -f $ImagesRoot/vendor.img

echo "Resizing vendor.img"
#resize2fs $ImagesRoot/vendor.img 300M
resize2fs $ImagesRoot/vendor.img 512M

apply.sh

#fix_prop product $MountPointProduct/build.prop
fix_prop product $MountPointProduct/etc/build.prop
#fix_prop system_ext $MountPointSystemExt/build.prop
fix_prop system_ext $MountPointSystemExt/etc/build.prop

apply.sh

#fix_prop product $MountPointProduct/build.prop
fix_prop product $MountPointProduct/etc/build.prop
#fix_prop system_ext $MountPointSystemExt/build.prop
fix_prop system_ext $MountPointSystemExt/etc/build.prop

You might also have to modify a couple other lines further down in the file

apply.sh

echo "Applying SELinux security contexts to props"
...
#chcon --reference=$MountPointSystemExt/etc $MountPointSystemExt/build.prop
chcon --reference=$MountPointSystemExt/etc $MountPointSystemExt/etc/build.prop
#chcon --reference=$MountPointProduct/etc $MountPointProduct/build.prop
chcon --reference=$MountPointProduct/etc $MountPointProduct/etc/build.prop
...

Following the steps from the past two comments fixed my issues.
I would suggest the README be updated to list the specific version(s) it has been tested on, as well as provide a method for downloading those specific version(s) (instead of the latest WSA). I wasn't aware I was getting this latest version when I followed the steps; I assumed it would be the older one that these scripts were originally developed for.

I use the installWSAGA.sh... Since that downloads the scripts directly from GitHub, I would need to either make the script modify the files after downloading or wait for me to manually edit them.
So I could either just put in a "pause" or make it rewrite the lines in the other script files (no idea on how to do that), right?

So, I modified the files once it asks for the architecture. However, it seems like the Gapps are completely missing....
(At least from start.)
Maybe I'm just stupid... Anyone can upload the modified files so I can simply drag them in?

OK I'm cloning repos and modified 2 files.
raw files
https://raw.githubusercontent.com/kensuke/WSAGAScript/0bf8ad4569e87322606dd6c9fadc38fd07d1cfd5/extend_and_mount_images.sh
https://raw.githubusercontent.com/kensuke/WSAGAScript/6c6da0c6d6b78ea178698f0f7e558f36a4fabd31/apply.sh
diffs
extend_and_mount_images.sh
https://github.com/kensuke/WSAGAScript/commit/0bf8ad4569e87322606dd6c9fadc38fd07d1cfd5
apply.sh
https://github.com/kensuke/WSAGAScript/commit/3eb26916bd4b1ad28494f066a8f88ccc2a1ad679 (1. mistakes!)
https://github.com/kensuke/WSAGAScript/commit/6c6da0c6d6b78ea178698f0f7e558f36a4fabd31 (2. repaired..)
repos
https://github.com/kensuke/WSAGAScript

I found a for Android 12L/12.1 x86_64 built OpenGApps pico here LSPosed/MagiskOnWSA#351, as the original link is not 24h available I uploaded it into my Onedrive https://1drv.ms/u/s!Aogy7BCP2WhegaNzD7R5di3_qJyqlw?e=SKgrmL, this may be better than using the android 11 OpenGApps

The default Android 11 GApps seem to work fine. The script modifications fix the certification issue, but the default "C:\WSA\AppxManifest.xml" Path doesn't work. It gives me an error "Wrong Parameter" and doesn't launch.
Copying it somewhere else (for me into my OneDrive) works fine though.

OK I'm cloning repos and modified 2 files.
raw files https://raw.githubusercontent.com/kensuke/WSAGAScript/0bf8ad4569e87322606dd6c9fadc38fd07d1cfd5/extend_and_mount_images.sh https://raw.githubusercontent.com/kensuke/WSAGAScript/6c6da0c6d6b78ea178698f0f7e558f36a4fabd31/apply.sh
diffs extend_and_mount_images.sh kensuke@0bf8ad4
apply.sh kensuke@3eb2691 (1. mistakes!) kensuke@6c6da0c (2. repaired..)
repos https://github.com/kensuke/WSAGAScript

I can confirm that now it works

I tried the installation twice. But i cant login into Google. After i start the Subsystem i get a lot of Messages (every 2 seconds) , that my device is not certified by Google Play Protect. I have to turn the Subsystem off to stop the messages.
I found this Files from kensuke but both don't work. I really don't know if im doing it right because there is no explanation. I downloaded the extend_and_mount_images.sh and the apply.sh and replace the originals in the folder. But at the point i have to use the "./extend_and_mount_images.sh" and the "./apply.sh" i got an error message. Curious.....the day before i got an other error with something with line 9. Now its 8.

You might need to uninstall any older version (yes, this implies loosing data) before converting and installing the new version.

I found this Files from kensuke

Try using only the files given in the repo

I dont have an older version. After it didn't work yesterday, i deleted WSA and startet from the beginning.
What do you mean with "only the files given in the repo"?
I first install the "normal" repo from https://github.com/WSA-Community/WSAGAScript.git
This didn't work because of Google Play Protect Certification. Then i tried to search for this problem and found this issue and the solution from kensuke.
Now i didn't know exactly what i have to do, so i downloaded the 2 files and replace the original ones. Then i start the installation from the beginning but get the error message if i try to "./" the extend.... and the apply file.
There was no error with the original files, but then i get thousands of messages, that my device is not certified.
I am new to this site. Sorry, if i dont understand all things inmediately.

I have explained myself wrong. Can you please try cloning kensuke's repo instead on these one?

That was the solution. I needed ALL files. Not only the 2 i downloaded. Now i dont need the adb kit to login to google. Many thanks for this quick help. I never expected this. ;)

@[kensuke]
Thank you very much it working!!!!!!
jeaaaaah now i can my own wsa gapps package

OK I'm cloning repos and modified 2 files.
raw files<EMAIL_ADDRESS>(raw<EMAIL_ADDRESS>(raw)
diffs extend_and_mount_images.sh kensuke@0bf8ad4
apply.sh kensuke@3eb2691 (1. mistakes!) kensuke@6c6da0c (2. repaired..)
repos kensuke/WSAGAScript

these steps worked for me, bypassed certified issue. thank you !

OK I'm cloning repos and modified 2 files.
raw files<EMAIL_ADDRESS>(raw<EMAIL_ADDRESS>(raw)
diffs extend_and_mount_images.sh kensuke@0bf8ad4
apply.sh kensuke@3eb2691 (1. mistakes!) kensuke@6c6da0c (2. repaired..)
repos kensuke/WSAGAScript

these steps worked for me, bypassed certified issue. thank you !

Hello! I don't understand how to apply those steps :( is it very difficult?


Add ModelCheckpoint.to_json method

What does this PR do
Add simple to_json() method to ModelCheckpoint.
In the current code, it requires three lines, and it could be automated by lightning.
Fixes #3047 (disclaimer: my issue with no answer yet)
Before submitting

[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @mpariente! Thanks for opening this PR.

In the file pytorch_lightning/callbacks/model_checkpoint.py:

Line 419:43: W292 no newline at end of file

Do see the Hitchhiker's guide to code style

Actually I'd prefer not to use json for this. If we really want this (which still needs to be considered), I'd prefer to go with YAML instead of JSON here, since we use yaml for most other human readable serialization and IMO should not mix file formats to much.

why not both or maybe more? to_some_format(format='YAML' or 'JSON' or ...)?
only if we really want this (which still needs to be considered)

If we really want this (which still needs to be considered)

I can make the required changes, but I'd like to know if you want it or not beforehand.

personally I would prefer YAML as it is readable and it can be used as an architecture draft, but we can have both...
any other thoughts on the API side? @PyTorchLightning/core-contributors

I would strongly recommend not to do both, since this is really not necessary here.  Doing both is a typical overkill and we still have to keep the effort to maintain in mind (probably not much here, but it increases with every little bit).

YAML only would be ok. It's just to have a one-liner to dump the saved models history.
I think there is a point to this feature because having this file is useful after training, for stochastic weight averaging or ensemble methods etc.. And if we don't dump this file, we have to open the checkpoint to have it's validation performance, which is not efficient.

yeah, YAML sounds good. Let's do YAML then :).

@mpariente shall we close this one and make another for YAML?

Sorry, I got absorbed by something else and forgot the PR.
Thanks for pushing this through @Borda !

@mpariente seems that with the update to YAML the test is not correct, mind have look at it...

This week end, I won't have time.
But I'll have a look on Monday.

This week end, I won't have time.
But I'll have a look on Monday.

It seems you did it, thank you :]

Yeah, finally ^^
What happening with the tests, you rebased master? The failures are not related right?

good to go now @Borda

Well, just forget to add changelog =)


Fix : forward and back button on browsers

Fix : forward and back button
While pressing back button and forward button , website does not redirect to that location.
Steps to reproduce the behavior:

Go to 'homepage'
Click on 'every section'
Try clicking  'forward and back button on browser'
See error

hello @c4coderandcreator can you assign this to me

hey can you assign this to me

@c4coderandcreator can you assign this issue to me

@Neel-07  Hey !! Work on it and follow the contributing guidelines !!

Hey @Ishsinghal  and @kashika13  thank you for will to contribute , I am opening 2 more bugs kindly check if you can resolve it !!


Update backplane-cli dependency for AWSV2Config

Currently, this error is returned when the url field is missing from a user's backplane config file:
❯ osdctl network verify-egress --cluster-id ${REDACTED}
2023/12/23 13:35:23 getting AWS credentials from backplane-api
2023/12/23 13:35:24 failed to get credentials automatically from backplane-api: can't find backplane url: %!w(<nil>). You can still try this command by exporting AWS credentials as environment variables and specifying the --subnet-id, --security-group, and other required flags manually. See osdctl network verify-egress -h for more details

Starting in v0.0.21, this url field was deprecated. This PR updates the dependency so that SREs can use the latest backplane-cli config file format.
OSD-20166

/lgtm
Thanks for updating this :)


Added Shortcuts Guide and PowerRename settings pages

Summary of the Pull Request

Added Shortcuts Guide and PowerRename settings pages
Stylized textstyles and margins
Added VisualState logic for sidepanel containing e.g. learn more buttons and contributors list

References

PR Checklist

[] Applies to #xxx
[] CLA signed. If not, go over here and sign the CLA
[] Tests added/passed
[] Requires documentation to be updated
[] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Good catch!

Ok. I went in and tweaked the branch


Adding missing properties to the swagger targeting 2021-05-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

Updating swagger to accurately reflect service

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@anuchandy Anything you need from me on this? Thanks!

These are not really breaking changes but to ensure that swagger for 2021-03-01 has the correct information to reflect already deployed service. [Confirmed with @joshdmcc]

/azp run unifiedPipeline

@anuchandy @JeffreyRichter are we able to close this out?


Omeka import

Pull request
Proposed changes

Fix #8

Types of changes

[x] New feature (non-breaking change which adds functionality).
[x] Enhancement (non-breaking change which enhances functionality)
[ ] Bug Fix (non-breaking change which fixes an issue).
[ ] Breaking change (fix or feature that would cause existing functionality to change).

Checklist

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Summary by CodeRabbit

New Features

Introduced a data processing step to enhance site deployment.
Added support for additional branch triggers in the deployment workflow.

Enhancements

Improved deployment workflow with environment variables for better integration with Omeka API.
Streamlined the build and deployment process by updating job dependencies and configurations.

Chores

Cleaned up workflow file by removing unnecessary comments.

@koilebeit Can you please take a look at the actions with https://github.com/nektos/act. My docker is on the frizz at the moment.

Apart from the github action, the objectid must be lowercase (see https://collectionbuilder.github.io/cb-docs/docs/metadata/csv_metadata/#objectid).
The object_location field is also missing, otherwise the images and data cannot be downloaded (https://collectionbuilder.github.io/cb-docs/docs/metadata/csv_metadata/#object_location)

Apart from the github action, the objectid must be lowercase (see https://collectionbuilder.github.io/cb-docs/docs/metadata/csv_metadata/#objectid). The object_location field is also missing, otherwise the images and data cannot be downloaded (https://collectionbuilder.github.io/cb-docs/docs/metadata/csv_metadata/#object_location)

I get it to work with uppercase objectid (perhaps the sanitization of cb does the trick). Leaving out object_location is by design (or choice). As long as we cannot access these files directly from Omeka we cannot integrate the links.

When I try it with act I get following Error-Message: [Deploy Jekyll site with Data Processing/data-processing] ⭐ Run Main Process Data [Deploy Jekyll site with Data Processing/data-processing] 🐳 docker exec cmd=[python /var/run/act/workflow/3.py] user= workdir= | Traceback (most recent call last): | File "/var/run/act/workflow/3.py", line 72, in <module> | item_set = get_items_from_collection(ITEM_SET_ID) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | File "/var/run/act/workflow/3.py", line 18, in get_items_from_collection | response = requests.get(url, params=params) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/requests/api.py", line 73, in get | return request("get", url, params=params, **kwargs) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/requests/api.py", line 59, in request | return session.request(method=method, url=url, **kwargs) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/requests/sessions.py", line 575, in request | prep = self.prepare_request(req) | ^^^^^^^^^^^^^^^^^^^^^^^^^ | File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/requests/sessions.py", line 486, in prepare_request | p.prepare( | File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/requests/models.py", line 368, in prepare | self.prepare_url(url, params) | File "/opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/requests/models.py", line 439, in prepare_url | raise MissingSchema( | requests.exceptions.MissingSchema: Invalid URL 'items': No scheme supplied. Perhaps you meant https://items? [Deploy Jekyll site with Data Processing/data-processing] ❌ Failure - Main Process Data [Deploy Jekyll site with Data Processing/data-processing] exitcode '1': failure [Deploy Jekyll site with Data Processing/data-processing] 🏁 Job failed Error: Job 'data-processing' failed
edit: probably I do not have set the OS environment variables..

I added the secrets also to my fork and it seems to work https://github.com/maehr/collectionbuilder-csv/actions/runs/8173244290

@koilebeit the script turns all multiple/compound_items into compound_items https://github.com/maehr/collectionbuilder-csv/actions/runs/8207996344
haven't had the time to fix that.

@koilebeit I added multiples as well. Now the only caveat is that PDF files are not shown nor downloadable. See the build files https://github.com/maehr/collectionbuilder-csv/actions/runs/8208244993

@koilebeit I added multiples as well. Now the only caveat is that PDF files are not shown nor downloadable. See the build files https://github.com/maehr/collectionbuilder-csv/actions/runs/8208244993

The PDF files cannot be downloaded because the column object_location is empty. And they are not shown because image_small and image_thumb are empty (see CB display_template)
Then I noticed that the display_template is still filled out incorrectly. As I understand it from the documentation CB Compound Objects the parent object gets the display_template "compound_object", the sub-objects each get their format (e.g. "pdf" or "image") like this:

objectid
parentid
display_template

abb001

compound_object

abb001_a
abb001
pdf

abb001_b
abb001
pdf

@koilebeit I added multiples as well. Now the only caveat is that PDF files are not shown nor downloadable. See the build files https://github.com/maehr/collectionbuilder-csv/actions/runs/8208244993

The PDF files cannot be downloaded because the column object_location is empty. And they are not shown because image_small and image_thumb are empty (see CB display_template)
Then I noticed that the display_template is still filled out incorrectly. As I understand it from the documentation CB Compound Objects the parent object gets the display_template "compound_object", the sub-objects each get their format (e.g. "pdf" or "image") like this:
objectid	parentid	display_template
abb001		compound_object
abb001_a	abb001	pdf
abb001_b	abb001	pdf

I'll check again and fix it. I guess we can add the object_location by checking for the media to be private or not first.


Could not acquire lock to ...\paket.locked

Description
I've experienced very strange paket restore behaviour today. I was constantly getting this error message on restoring packages: Could not acquire lock to C:\Gitlab-Runner\builds\cc2bad82\0\***\obj\paket.locked. Finally I found out it was caused by blocking the creation of *.locked files by my antivirus (McAfee) because of some virus which encrypts files and creates *.locked copy of it I guess.
Since our organization policy doesn't allow us to change any security settings it becomes unreal to run my build pipeline.
Please advice. Is it possibly to make paket.locked constant configurable or rename it to smth else? Thanks
Known workarounds
I made a custom build with changed to paket.locked-whatever constant (located in Constants.fs) instead of paket.locked.

I think the correct fix is to tell McAfee about it. It's a bug in their
scanner.
Ivan Bianko<EMAIL_ADDRESS>schrieb am Do., 28. März 2019, 22:32:

Description
I've experienced very strange paket restore behaviour today. I was
constantly getting this error message on restoring packages: Could not
acquire lock to C:\Gitlab-Runner\builds\cc2bad82\0***\obj\paket.locked.
Finally I found out it was caused by blocking the creation of *.locked
files by my antivirus (McAfee) because of some virus which encrypts files
and creates *.locked copy of it I guess.
Since our organization policy doesn't allow us to change any security
settings it becomes unreal to run my build pipeline.
Please advice. Is it possibly to make paket.locked constant configurable
or rename it to smth else? Thanks
Known workarounds
I made a custom build with changed to paket.locked-whatever constant
(located in Constants.fs) instead of paket.locked.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3535, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNK7vqKzbpts4L2J9vqwp8Psn80bZks5vbTTlgaJpZM4cRLic
.

We've got the same behaviour in my company and it's pretty annoying. It's not directly related to Paket but we planned to remove Paket because of that :(

all I can say: tell your antivirus vendor. They usually whitelist pretty fast

@forki Files ending with *.locked extension are treated by McAfee as ransomware. It may be stupid, but that's how McAfee works. I issued a ticket to whitelist this file in our company and after a week we only got temporary approval. I agree that it McAfee problem, but many companies use it and security experts are just afraid. We are basically forced to come with some alternative because our company does not want to allow it permanently. The alternative may be to not use Paket or have a custom fork. None of these options are good for us. It would be much better to just change the single line of code in Paket to change the filename. Is there some specific reason the file must have the name paket.locket?

I'm fine with changing the name
Michal Jirouš<EMAIL_ADDRESS>schrieb am Fr., 24. Mai 2019,
16:15:

@forki https://github.com/forki Files ending with *.locked extension
are treated by McAfee as ransomware. It may be stupid, but that's how
McAfee works. I issued a ticket to whitelist this file in our company and
after a week we only got temporary approval. I agree that it McAfee
problem, but many companies use it and security experts are just afraid. We
are basically forced to come with some alternative because our company does
not want to allow it permanently. The alternative may be to not use Paket
or have a custom fork. None of these options are good for us. It would be
much better to just change the single line of code in Paket to change the
filename. Is there some specific reason the file must have the name
paket.locket?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/3535?email_source=notifications&email_token=AAAOANGGZQS53QUTZZLYSFTPW7Z6PA5CNFSM4HCEXCOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWFP37A#issuecomment-495648252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAOANBN6SDKUXIOFLUXDW3PW7Z6PANCNFSM4HCEXCOA
.

@forki would you mine to name it paket.lockfile or smth else? I can create a PR for it.

I'm fine with renaming the file as long as the change is backwards compatible on restore.

I think it should be something like .processlock

The HP Sure Click software does the same thing.  Just lost an hour tracing down locking problems on a  machine till we found this thread.   Whitelisting the project folder fixed it, but not super smart strategy on their part.


Add example for let_* functions

Add both C++ code and bikeshed wording.
More examples with the server theme will follow.

These look good to me. Thanks, @lucteo. @kirkshoop, any thoughts?


feat: add eslint 9 support

BREAKING CHANGE: needs @eslint/js and @estlint/eslintrc dependencies to work

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update mobile-feature-comparison.md

-added &#x2778  (number3)

Added: Viewing embedded video in PowerPoint presentations is not supported.
#1789
See UserVoice:
The ios Skype for business client (Phone and ipad) does not play videos embedded in PowerPoint. This has been confirmed by Microsoft support

https://www.skypefeedback.com/forums/299913-generally-available/suggestions/31537219-the-ios-skype-for-business-client-phone-and-ipad
https://www.skypefeedback.com/forums/299913-generally-available/suggestions/35314720-powerpoint-sharing-error-in-android-app

@jambirk Hi James, changes look correct. Could you please check and merge? Thanks!

@PhillipGarding do you approve of the changes here?

Yes, I approve.
From: James Birkinshaw<EMAIL_ADDRESS>Sent: Monday, April 8, 2019 1:11 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Phillip Garding<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Update mobile-feature-comparison.md (#1977)
@PhillipGardinghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPhillipGarding&data=02|01|phillipg%40microsoft.com|cdbbbf42528b47750d6608d6bc5e4657|72f988bf86f141af91ab2d7cd011db47|1|0|636903510430172993&sdata=OTZ8%2FsLsg5CWM88bASroJ9eSvqdCUJ2%2BM2LQlUGAEQw%3D&reserved=0 do you approve of the changes here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F1977%23issuecomment-480988179&data=02|01|phillipg%40microsoft.com|cdbbbf42528b47750d6608d6bc5e4657|72f988bf86f141af91ab2d7cd011db47|1|0|636903510430182994&sdata=RNerkMInaP3sufAaSzCIfI%2B%2BOzJpA%2Bd1lvd9UmQ8mxs%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAkYGQ4s0M94a16iI_P7lil2nWtRGDo4Rks5ve6JAgaJpZM4cepV-&data=02|01|phillipg%40microsoft.com|cdbbbf42528b47750d6608d6bc5e4657|72f988bf86f141af91ab2d7cd011db47|1|0|636903510430182994&sdata=XZMxUeD1Z1Tm6MK6d6h64O7OoZWRi6UVz7QGbPxV5Jo%3D&reserved=0.


Julia test error

Hi @findmyway I am having a problem in the Julia tests on Travis that I cannot reproduce locally, details here: https://travis-ci.org/github/deepmind/open_spiel/jobs/705469596
Any ideas?
159/164 Test #164: julia_test ........................................***Failed   45.49 sec

   Testing OpenSpiel

 Resolving package versions...

 Installed Missings ─────────── v0.4.3
 Installed SortingAlgorithms ── v0.3.1
 Installed DataAPI ──────────── v1.3.0
 Installed OrderedCollections ─ v1.3.0
 Installed StatsBase ────────── v0.33.0
 Installed DataStructures ───── v0.17.19
signal (11): Segmentation fault
in expression starting at /home/travis/build/deepmind/open_spiel/open_spiel/julia/src/OpenSpiel.jl:5
gc_get_stack_addr at /buildworker/worker/package_linux64/build/src/gc.c:1484 [inlined]
gc_read_stack at /buildworker/worker/package_linux64/build/src/gc.c:1491 [inlined]
gc_mark_loop at /buildworker/worker/package_linux64/build/src/gc.c:2015
_jl_gc_collect at /buildworker/worker/package_linux64/build/src/gc.c:2703
jl_gc_collect at /buildworker/worker/package_linux64/build/src/gc.c:2903
maybe_collect at /buildworker/worker/package_linux64/build/src/gc.c:781 [inlined]
jl_gc_pool_alloc at /buildworker/worker/package_linux64/build/src/gc.c:1096
jl_gc_alloc_ at /buildworker/worker/package_linux64/build/src/julia_internal.h:233 [inlined]
jl_gc_alloc at /buildworker/worker/package_linux64/build/src/gc.c:2940
jl_new_bits at /buildworker/worker/package_linux64/build/src/datatype.c:611
get_module_functions at /home/travis/.julia/artifacts/d54a2909808f0f05123ef98f8eef4fcde1073e52/lib/libcxxwrap_julia.so.0 (unknown line)
get_module_functions at /home/travis/.julia/packages/CxxWrap/94t40/src/CxxWrap.jl:419 [inlined]
initialize_julia_module at /home/travis/.julia/packages/CxxWrap/94t40/src/CxxWrap.jl:408
__init__ at /home/travis/.julia/packages/CxxWrap/94t40/src/StdLib.jl:20
unknown function (ip: 0x7f13fa970d0d)
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2141 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2305
jl_apply at /buildworker/worker/package_linux64/build/src/julia.h:1631 [inlined]
jl_module_run_initializer at /buildworker/worker/package_linux64/build/src/toplevel.c:74
jl_init_restored_modules at /buildworker/worker/package_linux64/build/src/dump.c:2477
_include_from_serialized at ./loading.jl:692
_require_search_from_serialized at ./loading.jl:776
_require at ./loading.jl:1001
require at ./loading.jl:922
require at ./loading.jl:917
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2135 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2305
jl_apply at /buildworker/worker/package_linux64/build/src/julia.h:1631 [inlined]
call_require at /buildworker/worker/package_linux64/build/src/toplevel.c:399 [inlined]
eval_import_path at /buildworker/worker/package_linux64/build/src/toplevel.c:436
jl_toplevel_eval_flex at /buildworker/worker/package_linux64/build/src/toplevel.c:656
jl_eval_module_expr at /buildworker/worker/package_linux64/build/src/toplevel.c:181
jl_toplevel_eval_flex at /buildworker/worker/package_linux64/build/src/toplevel.c:640
jl_parse_eval_all at /buildworker/worker/package_linux64/build/src/ast.c:873
jl_load at /buildworker/worker/package_linux64/build/src/toplevel.c:878
include at ./boot.jl:328 [inlined]
include_relative at ./loading.jl:1105
include at ./Base.jl:31
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2135 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2305
jl_apply at /buildworker/worker/package_linux64/build/src/julia.h:1631 [inlined]
do_call at /buildworker/worker/package_linux64/build/src/interpreter.c:328
eval_value at /buildworker/worker/package_linux64/build/src/interpreter.c:417
eval_stmt_value at /buildworker/worker/package_linux64/build/src/interpreter.c:368 [inlined]
eval_body at /buildworker/worker/package_linux64/build/src/interpreter.c:778
eval_body at /buildworker/worker/package_linux64/build/src/interpreter.c:705
jl_interpret_toplevel_thunk_callback at /buildworker/worker/package_linux64/build/src/interpreter.c:888
unknown function (ip: 0xfffffffffffffffe)
unknown function (ip: 0x7f13fcadb80f)
unknown function (ip: 0x2)
jl_interpret_toplevel_thunk at /buildworker/worker/package_linux64/build/src/interpreter.c:897
jl_toplevel_eval_flex at /buildworker/worker/package_linux64/build/src/toplevel.c:814
jl_toplevel_eval_in at /buildworker/worker/package_linux64/build/src/toplevel.c:843
eval at ./boot.jl:330 [inlined]
eval at ./client.jl:425
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2135 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2305
top-level scope at ./none:3
jl_toplevel_eval_flex at /buildworker/worker/package_linux64/build/src/toplevel.c:808
jl_toplevel_eval_flex at /buildworker/worker/package_linux64/build/src/toplevel.c:764
jl_toplevel_eval_in at /buildworker/worker/package_linux64/build/src/toplevel.c:843
eval at ./boot.jl:330
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2135 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2305
exec_options at ./client.jl:263
_start at ./client.jl:460
jfptr__start_2084.clone_1 at /home/travis/packages/julias/julia-1.3.1/lib/julia/sys.so (unknown line)
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2135 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2305
jl_apply at /buildworker/worker/package_linux64/build/ui/../src/julia.h:1631 [inlined]
true_main at /buildworker/worker/package_linux64/build/ui/repl.c:96
main at /buildworker/worker/package_linux64/build/ui/repl.c:217
__libc_start_main at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
_start at /home/travis/packages/julias/julia-1.3.1/bin/julia (unknown line)
Allocations: 2434104 (Pool: 2433732; Big: 372); GC:

ERROR: LoadError: Failed to precompile OpenSpiel [7e5d8dc8-489f-4be6-9273-f4c3010e6ec1] to /home/travis/.julia/compiled/v1.3/OpenSpiel/pEQx0_VvzXr.ji.

Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1283
 [3] _require(::Base.PkgId) at ./loading.jl:1024
 [4] require(::Base.PkgId) at ./loading.jl:922
 [5] require(::Module, ::Symbol) at ./loading.jl:917
 [6] include at ./boot.jl:328 [inlined]
 [7] include_relative(::Module, ::String) at ./loading.jl:1105
 [8] include(::Module, ::String) at ./Base.jl:31
 [9] include(::String) at ./client.jl:424
 [10] top-level scope at none:6
in expression starting at /home/travis/build/deepmind/open_spiel/open_spiel/julia/test/runtests.jl:1
ERROR: Package OpenSpiel errored during testing
Stacktrace:

 [1] pkgerror(::String, ::Vararg{String,N} where N) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.3/Pkg/src/Types.jl:113
 [2] #test#131(::Bool, ::Nothing, ::Cmd, ::Cmd, ::typeof(Pkg.Operations.test), ::Pkg.Types.Context, ::Array{Pkg.Types.PackageSpec,1}) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.3/Pkg/src/Operations.jl:1372
 [3] #test at ./none:0 [inlined]
 [4] #test#62(::Bool, ::Nothing, ::Cmd, ::Cmd, ::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::typeof(Pkg.API.test), ::Pkg.Types.Context, ::Array{Pkg.Types.PackageSpec,1}) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.3/Pkg/src/API.jl:253
 [5] test(::Pkg.Types.Context, ::Array{Pkg.Types.PackageSpec,1}) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.3/Pkg/src/API.jl:239
 [6] #test#58 at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.3/Pkg/src/API.jl:233 [inlined]
 [7] test() at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.3/Pkg/src/API.jl:230
 [8] top-level scope at none:1

160/164 Test #144: python_pyspiel_test ...............................   Passed  152.81 sec
161/164 Test #124: python_value_iteration_test .......................   Passed  266.55 sec
162/164 Test #119: python_generalized_psro_test ......................   Passed  324.65 sec
163/164 Test #143: python_policy_test ................................   Passed  199.97 sec
164/164 Test #140: python_games_bridge_test ..........................   Passed  363.55 sec

99% tests passed, 1 tests failed out of 164

Total Test time (real) = 1249.07 sec

The following tests FAILED:

	164 - julia_test (Failed)

Errors while running CTest

At least one test failed.

If this is the first time you have run these tests, try:

pip3 install -r requirements.txt

Note that outside a virtualenv, you will need to install the system

wide matplotlib: sudo apt-get install python-matplotlib

The command "./open_spiel/scripts/travis_script.sh" exited with 1.

Done.
Do you think it's a memory issue? I will try running some memory checks on our side.

Hey Jun,

I just triggered a new build and it passed.
I just discovered that we have a few memory errors. So this could very well be on our side, sorry. I will continue to investigate and close this if indeed it's no longer a problem.

I pushed an update that included a number of memory fixes and the Travis tests passed. I believe this was likely not the fault of the Julia API, so I am closing it.


License

Hi,
I kind of had the same Idea about starting 2022 with static-site-generation in go for a small personal website thingy.
Since I'd like to use giraffe as a base to start from (and of cause contribute things back to) I wanted to ask, how you plan on licensing your code.

Hi @bomb20
License added https://github.com/tatthien/giraffe/blob/master/LICENSE
I appreciate your contributions.

Closing this issue due to no longer responding.


Using the image make our engine slow by 8 times

When I turn on the LETSENCRYPT_HOST on my image it makes the connections to my engine almost 8 times slower, here is the configuration that I have set up:
version: "3.9"

services:
  tracker:
    image: username/image
    deploy:
      replicas: 3
      update_config:
        parallelism: 1
        delay: 30s
        order: start-first
    ports:
      - "3000"
      - "9100:3000"
    environment:
      - VIRTUAL_HOST=domain.com
      - VIRTUAL_PORT=3000

      - LETSENCRYPT_HOST=domain.com

  nginx-proxy:
    image: jwilder/nginx-proxy
    ports:
      - "80:80"
      - "443:443"
    labels:
      com.github.jrcs.letsencrypt_nginx_proxy_companion.nginx_proxy: "true"
    volumes:
      - certs:/etc/nginx/certs:ro
      - vhostd:/etc/nginx/vhost.d
      - html:/usr/share/nginx/html
      - /var/run/docker.sock:/tmp/docker.sock:ro

  letsencrypt:
    image: nginxproxy/acme-companion
    depends_on:
      - nginx-proxy
    environment:
      - NGINX_PROXY_CONTAINER=nginx-proxy
      -<EMAIL_ADDRESS>    volumes:
      - acme:/etc/acme.sh
      - certs:/etc/nginx/certs
      - vhostd:/etc/nginx/vhost.d
      - html:/usr/share/nginx/html
      - /var/run/docker.sock:/var/run/docker.sock:ro

volumes:
  acme:
  certs:
  html:
  vhostd:

With LETSENCRYPT_HOST enabled on the image our benchmark is as follow:
ab -n 1000 -c 200 'https://domain.com'
This is ApacheBench, Version 2.3 <$Revision: 1843412 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking domain.com (be patient)
Completed 100 requests
Completed 200 requests
Completed 300 requests
Completed 400 requests
Completed 500 requests
Completed 600 requests
Completed 700 requests
Completed 800 requests
Completed 900 requests
Completed 1000 requests
Finished 1000 requests

Server Software:        nginx/1.21.4
Server Hostname:        domain.com
Server Port:            443
SSL/TLS Protocol:       TLSv1.2,ECDHE-RSA-AES256-GCM-SHA384,4096,256
Server Temp Key:        X25519 253 bits
TLS Server Name:        domain.com

Document Path:          /
Document Length:        0 bytes

Concurrency Level:      200
Time taken for tests:   4.058 seconds
Complete requests:      1000
Failed requests:        0
Non-2xx responses:      1000
Total transferred:      289000 bytes
HTML transferred:       0 bytes
Requests per second:    246.43 [#/sec] (mean)
Time per request:       811.582 [ms] (mean)
Time per request:       4.058 [ms] (mean, across all concurrent requests)
Transfer rate:          69.55 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:       10  509 987.6     60    2751
Processing:     3  199 354.7     64    1301
Waiting:        2  199 354.8     64    1301
Total:         21  708 1299.6    125    4035

Percentage of the requests served within a certain time (ms)
  50%    125
  66%    161
  75%    201
  80%    268
  90%   3789
  95%   3842
  98%   3864
  99%   3875
 100%   4035 (longest request)

But when we comment the LETSENCRYPT_HOST in the image we get the following benchmark:
ab -n 10000 -c 200 'http://domain.com'
This is ApacheBench, Version 2.3 <$Revision: 1843412 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking domain.com (be patient)
Completed 1000 requests
Completed 2000 requests
Completed 3000 requests
Completed 4000 requests
Completed 5000 requests
Completed 6000 requests
Completed 7000 requests
Completed 8000 requests
Completed 9000 requests
Completed 10000 requests
Finished 10000 requests

Server Software:
Server Hostname:        domain.com
Server Port:            80

Document Path:          /
Document Length:        0 bytes

Concurrency Level:      200
Time taken for tests:   4.895 seconds
Complete requests:      10000
Failed requests:        0
Non-2xx responses:      10000
Total transferred:      1960000 bytes
HTML transferred:       0 bytes
Requests per second:    2042.72 [#/sec] (mean)
Time per request:       97.909 [ms] (mean)
Time per request:       0.490 [ms] (mean, across all concurrent requests)
Transfer rate:          390.99 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0    4  62.0      0    1037
Processing:     1   91  65.3     77     298
Waiting:        1   91  65.3     77     298
Total:          1   96  88.6     77    1152

Percentage of the requests served within a certain time (ms)
  50%     77
  66%    106
  75%    153
  80%    167
  90%    189
  95%    209
  98%    227
  99%    239
 100%   1152 (longest request)

TL;DR When we use the LETSENCRYPT_HOST the RPS (Request per second) is 246.43 but when we comment it; it's 2042.72 which is almost 8 times bigger than when we don't use the LETSENCRYPT_HOST.
Question: Why is this happening, and can we boost up the speed?

I don't really think there is an issues there, that's expected with TLS v1.2, see https://serverfault.com/a/934309
Bottom line is :

the TLS handshake has a significant added cost when doing benchmarks with ab
TLSv1.3 should be better in this regard but ab does not support TLSv1.3 at the moment
ab testing is not representative of real world usage

You can see in your own benchmark that the connect time goes from 4s on HTTP to 500s on HTTPS.  Testing on google.com does not yeld significantly different number, with the mean connect time being 5 times higher for HTTPS.
By the way nginxproxy/acme-companion is not responsible for proxy settings that could influence this, nginxproxy/nginx-proxy is. Do not hesitate to open a discussion there to see if we have room for improvements.

Inactive issue, closing.


Bump firebase-admin version to 8.4.0

In order to remove the warning:
The behavior for Date objects stored in Firestore is going to change
AND YOUR APP MAY BREAK.
To hide this warning and ensure your app does not break, you need to add the
following code to your app before calling any other Cloud Firestore methods:

  const firestore = new Firestore();
  const settings = {/* your settings... */ timestampsInSnapshots: true};
  firestore.settings(settings);

which is fixed in the 8.4.x version

@googlebot I signed it!


No autoload for traad-install-server, had to require it

Hi,
this is a great package !
A detail is that right after installation I couldn't find traad-install-server, as says the readme, even if it's an interactive function, because it is not #autoloaded.
There is no "add argument" function ?

Try out the autoload-install-function branch. If that works for you then I'll merge it in.

There is no "add argument" function ?

I think that must have been added to rope since I last worked on traad. To add support for this we'd need to:

Add support for it to traad
Then add support to this project

I don't think either would be too difficult, but I really don't have time for it right now. If you want to take a try at adding it, I'll be more than happy to help out if necessary, and of course I'd be delighted to have to additions.

that works for me thanks !
I'll have a look and open another issue for discussion if necessary.

Great! I'll merge in the PR and close this out. Thanks for taking the time to report this.

Fixed in de76278e77d26ab8c400c03cef7bc3326be1e257

btw I can't find a mention of adding a method parameter on rope overview. Might as well ask.

I can't find a mention of adding a method parameter

This ArgumentAdder class sounds like it should do the trick. I think they just haven't updated their docs yet. If you looks in traad.rope.change_signature I think it should be pretty clear how to add support to the traad core.


Check HubConnection state before running invoke logic

Before calling methods that require the HubConnection to be connected we run a simple connection state check like
https://github.com/aspnet/AspNetCore/blob/e310ccac7b30e901f657f81c86c4048fecfedf7b/src/SignalR/clients/java/signalr/src/main/java/com/microsoft/signalr/HubConnection.java#L464-L466
Addresses #4393

@mikaelm12 can you also add the shiproom template? (You can find a copy from https://github.com/aspnet/AspNetCore/pull/4403)

Approved for 2.2.2

The failure was for an IIS Functional test. An unrelated test failure.
Starting test execution, please wait...
2018-12-04T22:55:50.8997274Z   [xUnit.net 00:00:17.97]     Microsoft.AspNetCore.Server.IIS.FunctionalTests.AppOfflineTests.AppOfflineDroppedWhileSiteFailedToStartInRequestHandler_SiteStops_InProcess [SKIP]
2018-12-04T22:55:51.8627524Z   Skipped  Microsoft.AspNetCore.Server.IIS.FunctionalTests.AppOfflineTests.AppOfflineDroppedWhileSiteFailedToStartInRequestHandler_SiteStops_InProcess
2018-12-04T22:57:20.4801986Z   [xUnit.net 00:01:47.51]     Microsoft.AspNetCore.Server.IIS.FunctionalTests.AppOfflineTests.AppOfflineDroppedWhileSiteStarting_SiteShutsDown_InProcess [FAIL]
2018-12-04T22:57:20.4848481Z   Failed   Microsoft.AspNetCore.Server.IIS.FunctionalTests.AppOfflineTests.AppOfflineDroppedWhileSiteStarting_SiteShutsDown_InProcess
2018-12-04T22:57:20.4848649Z   Error Message:
2018-12-04T22:57:20.4848762Z    Assert.Equal() Failure
2018-12-04T22:57:20.4848871Z   Expected: 0
2018-12-04T22:57:20.4848956Z   Actual:   -1073740767


Invalid model

Error code 403, response -> {"error":{"type":"permission_error","message":"Invalid model","code":"model_not_allowed"}}

Please add 'model_claude' to 'client.send_message'
res: SendMessageResponse = client.send_message(
        chat_id=chat_id, prompt="Hello! Tell me about yourself", model_claude="claude-2.0"
)

Hi, which version of the api are you currently using?
I can't actually reproduce this error, as this is not an intended behaviour...
As I remember the new version upgraded to Claude's model 2.1, please try removing the library and installing the latest version...closing this as not planned.

@ermaxinc Ok I guess i'll make a PR for this...

@ermaxinc Waiting for your confirmation in order to merge pull#10


Drone

sorry, this PR should have gone to RoboStack master :) I am trying to use a Drone instance to see if we can build continously on a private machine.


DragLayer opacity 1

Is it possible to have the default draglayer image be fully opaque upon dragging?
Thanks for an awesome framework, still getting into it.

Hi @kasperpihl. Unfortunately there's no way to modify the appearance of the native drag layer (it's controlled completely by the browser), but you can implement a custom drag layer if you want to do more advanced tweaking. The linked example also has more info about why native tweaking isn't possible.

Aha I see. Thanks a lot for the help. (y)
On Wed, Mar 16, 2016 at 9:59 PM Billy Janitsch<EMAIL_ADDRESS>wrote:

Hi @kasperpihl https://github.com/kasperpihl. Unfortunately there's no
way to modify the appearance of the native drag layer (it's controlled
completely by the browser), but you can implement a custom drag layer
http://gaearon.github.io/react-dnd/examples-drag-around-custom-drag-layer.html
if you want to do more advanced tweaking. The linked example also has more
info about why native tweaking isn't possible.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/gaearon/react-dnd/issues/405#issuecomment-197703862

Updated link for implementing a custom drag layer: https://react-dnd.github.io/react-dnd/examples/drag-around/custom-drag-layer


No implementation found

I was working before, I don't know why it dosen't now?
I get this error on console, the problem is only Anroid related:
No implementation found for long com.shockwave.pdfium.PdfiumCore.nativeOpenDocument(int, java.lang.String) (tried Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument and Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument__ILjava_lang_String_2)
Thank so much!

I also encounter the same issue. Perhaps, the title should be Native libraries failed to load: couldn't find "libmodpng.so".
The library was working fine on dev channel 1.16.4-pre83.
Now it fails on later dev versions as 1.16.4-pre83 is no longer there, and also fails on 1.17.1 stable channel:
Flutter 1.17.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision f7a6a7906b (6 days ago) • 2020-05-12 18:39:00 -0700
Engine • revision 6bc433c6b6
Tools • Dart 2.8.2

E/com.shockwave.pdfium.PdfiumCore( 8159): Native libraries failed to load - java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/[appPackageNameCode]/base.apk"],nativeLibraryDirectories=[/data/app/[appPackageNameCode]/lib/arm64, /data/app/[appPackageNameCode]/base.apk!/lib/arm64-v8a, /system/lib64, /system/vendor/lib64]]] couldn't find "libmodpng.so"
W/ps.app_name( 8159): Accessing hidden field Ljava/io/FileDescriptor;->descriptor:I (light greylist, reflection)
E/ps.app_name( 8159): No implementation found for long com.shockwave.pdfium.PdfiumCore.nativeOpenDocument(int, java.lang.String) (tried Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument and Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument__ILjava_lang_String_2)
E/PDFView ( 8159): load pdf error
E/PDFView ( 8159): java.lang.UnsatisfiedLinkError: No implementation found for long com.shockwave.pdfium.PdfiumCore.nativeOpenDocument(int, java.lang.String) (tried Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument and Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument__ILjava_lang_String_2)
E/PDFView ( 8159): 	at com.shockwave.pdfium.PdfiumCore.nativeOpenDocument(Native Method)
E/PDFView ( 8159): 	at com.shockwave.pdfium.PdfiumCore.newDocument(PdfiumCore.java:113)
E/PDFView ( 8159): 	at com.github.barteksc.pdfviewer.source.FileSource.createDocument(FileSource.java:38)
E/PDFView ( 8159): 	at com.github.barteksc.pdfviewer.DecodingAsyncTask.doInBackground(DecodingAsyncTask.java:53)
E/PDFView ( 8159): 	at com.github.barteksc.pdfviewer.DecodingAsyncTask.doInBackground(DecodingAsyncTask.java:25)
E/PDFView ( 8159): 	at android.os.AsyncTask$2.call(AsyncTask.java:333)
E/PDFView ( 8159): 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
E/PDFView ( 8159): 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
E/PDFView ( 8159): 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
E/PDFView ( 8159): 	at java.lang.Thread.run(Thread.java:764)
W/System  ( 8159): A resource failed to call close.

Same in my app.

Sadly still with the same problem.

For me solved as following:

Removed the app from the device.
flutter clean
Re-installed the app.

I have this problem Too.
Flutter:
1.17.5
Runing in debug mode or release mode.
Both doesnt show the pdf file.
E/com.shockwave.pdfium.PdfiumCore(28515): Native libraries failed to load - java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/****-TaiWevzo7au7sddeIqZynw==/base.apk"],nativeLibraryDirectories=[/data/app/****-TaiWevzo7au7sddeIqZynw==/lib/arm64, /data/app/****-TaiWevzo7au7sddeIqZynw==/base.apk!/lib/arm64-v8a, /system/lib64, /system/vendor/lib64]]] couldn't find "**libmodpng**.so"

I was working before, I don't know why it dosen't now?
I get this error on console, the problem is only Anroid related:
No implementation found for long com.shockwave.pdfium.PdfiumCore.nativeOpenDocument(int, java.lang.String) (tried Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument and Java_com_shockwave_pdfium_PdfiumCore_nativeOpenDocument__ILjava_lang_String_2)
Thank so much!

Any news about this? ...

Run:
flutter pub add pdfx
The issue will be resolved as it adds all of the missing library and build correctly.
reference:
https://github.com/syncfusion/flutter-widgets/issues/677#issuecomment-1099531436


Beartype 0.19.0 bad interaction (AttributeError) with Click

Congratulations on the release @leycec . I've found an issue, and a workaround for it.
This works with beartype==0.18.5 but does not work with beartype==0.19.0.
# /// script
# dependencies = ["click==8.1.7", "beartype==0.19.0"]
# ///

import click
from beartype import beartype
from click.testing import CliRunner

@beartype
@click.command()
def main() -> None:
    pass

runner = CliRunner()
result = runner.invoke(cli=main)

With beartype==0.19.0 I get:
Traceback (most recent call last):
  File ["/Users/adam/Library/Caches/uv/archive-v0/Mj31znXpw5UdBGD5z-rgT/lib/python3.12/site-packages/click/testing.py", line 403](coderunner:///Users/adam/Library/Caches/uv/archive-v0/Mj31znXpw5UdBGD5z-rgT/lib/python3.12/site-packages/click/testing.py#403:0!), in invoke
    prog_name = extra.pop("prog_name")
                ^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'prog_name'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File ["/Users/adam/Desktop/bt_example.py", line 17](coderunner:///Users/adam/Desktop/bt_example.py#17:0!), in <module>
    result = runner.invoke(cli=main)
             ^^^^^^^^^^^^^^^^^^^^^^^
  File ["/Users/adam/Library/Caches/uv/archive-v0/Mj31znXpw5UdBGD5z-rgT/lib/python3.12/site-packages/click/testing.py", line 405](coderunner:///Users/adam/Library/Caches/uv/archive-v0/Mj31znXpw5UdBGD5z-rgT/lib/python3.12/site-packages/click/testing.py#405:0!), in invoke
    prog_name = self.get_default_prog_name(cli)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ["/Users/adam/Library/Caches/uv/archive-v0/Mj31znXpw5UdBGD5z-rgT/lib/python3.12/site-packages/click/testing.py", line 195](coderunner:///Users/adam/Library/Caches/uv/archive-v0/Mj31znXpw5UdBGD5z-rgT/lib/python3.12/site-packages/click/testing.py#195:0!), in get_default_prog_name
    return cli.name or "root"
           ^^^^^^^^
AttributeError: 'function' object has no attribute 'name'

As well as with click testing, I've seen similar issues with sphinx-click.
My workaround is to move from:
@beartype
@click.command()
def main() -> None:
    pass

to
@click.command()
@beartype
def main() -> None:
    pass

I confirm this same error with beartype claw in a click program

Guh! Technically speaking, Click is almost certainly probably to blame here. Click appears to be intolerant of decorator chaining (i.e., of other third-party decorators like @beartype preceding the @click.command() decorator). But nobody cares about technicalities and neither do I.
Pragmatically speaking, it is genuinely awful that @beartype used to support this and no longer does. I fully agree! Previously, @beartype just kinda ignored Click and hoped for the best. Now, I'll add official support for Click in an upcoming @beartype 0.19.1 patch release in a day or two.
So sorry about this, CLI warriors. @beartype promises to do better for Click shortly. :face_exhaling:
Help @Beartype Help Itself
Relatedly, consider installing @beartype 0.20.0 release candidates when the time comes in a few months. I tried really hard to avoid breaking backward compatibility by releasing three @beartype 0.19.0 release candidates over the last several months. But that only works if everybody installs these release candidates. I guess nobody installed these release candidates, huh? I mean, I get it. There's only so much time in the day. But we can avoid pain like this in the future!
Please help @beartype help itself. I beg this of you, fam. :bow:

We'll do this somehow! This is probably the last action item for our upcoming @beartype 0.19.1 patch release. Although praying for a speedy resolution only gets us so far, prior @beartype releases did support this. Let us reflect on the past with fondness in our hearts. :face_holding_back_tears:

Resolved by 17e121e33c707dba. Click and @beartype are now BFFLs once more. Now that @beartype officially supports Click, I'm fairly certain that @beartype previously just ignored Click commands; @beartype wasn't actually type-checking Click commands. Now, @beartype type-checks Click commands.
Thanks again for that super-awesome minimal working example (MWE), @adamtheturtle. Your clever (and probably intentional) use of the pytest-friendly click.testing.CliRunner scaffold enabled me to immediately turn this into a working unit test without knowing anything about Click. You're awesome! Turtle power forever. :turtle: :fist_raised: :turtle:

Thank you for the great work @leycec ! 🐢 ❤ 🐻


Change OMIM to MIM

There are a couple of definition xrefs in the file that use OMIM:### these should be changed to MIM:###
Waiting a couple of days to do this in case this would cause issues for any users

OMIM:### has been changed to MIM:###


Running Routinator behind reverse proxy with Docker

Hello,
I'm having a very difficult time getting routinator working.
I'm trying to use a reverse proxy, as recommend in the documentation:

Please note that this server is intended to run on your internal network and doesn’t offer HTTPS natively. If this is a requirement, you can for example run Routinator behind an NGINX reverse proxy.

But I think that the way that Routinator binds to ports might be confusing the reverse proxy (Traefik in this case). The reverse proxy logs are filled with:
traefik    | time="2020-05-12T02:34:57Z" level=error msg="service \"routinator-routinator\" error: unable to find the IP address for the container \"/routinator\": the server is ignored" container=routinator-routinator-501f908608811495a27bf2cc19982ef29f79388aa6273506d59d749006d4f34f providerName=docker

And 500s are returned from the reverse proxy. No requests seem to make it to Routinator.
I have tried overriding the CMD
https://github.com/NLnetLabs/routinator/blob/be1e1c5949c90bd9dbcf450b006e9808ae707962/Dockerfile#L52
with:
-v server --http <IP_ADDRESS>:8323 --rtr <IP_ADDRESS>:3323

-v server --http <IP_ADDRESS>:8323 --rtr <IP_ADDRESS>:3323

As well as the docker container's ip address, all to no avail. I also was very confused by the documentation's references to HTTP running on 8323, but in Docker using 9556.
I suspect that this is not necessarily a problem with Routinator itself, or Traefik either, but how they interact with one another. I'm mostly curious if there is anything non-standard with how Routinator binds to addresses.
The Docker configuration is as follows:
version: '3'

services:
  routinator:
    image: nlnetlabs/routinator
    container_name: routinator
    restart: unless-stopped
    command: '-v server --http <IP_ADDRESS>:8323 --rtr <IP_ADDRESS>:3323'
    expose:
      - 8323
    ports:
      - '3323:3323'
    networks:
      - 'traefik'
    volumes:
      - tals:/home/routinator/.rpki-cache/tals
    labels:
      - 'traefik.enable=true'
      - 'traefik.docker.network=traefik'
      - 'traefik.http.routers.routinator.rule=Host(`rpki.tld`)'
      - 'traefik.http.routers.routinator.entrypoints=websecure'
      - 'traefik.http.routers.routinator.tls.certresolver=letsencrypt'

volumes:
  tals:

networks:
  traefik:
    external: true

Looking deeper into Traefik, it appears that it only sees a server listening at :3323, not :9556 (or :8323 when configured like above). Depsite this, I can see the bindings inside the container:
$ netstat -a
Active Internet connections (servers and established)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:9556            <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:36791        <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:3323            <IP_ADDRESS>:*               LISTEN

I was finally able to figure out a workaround, though I'm not sure why Traefik wasn't able to determine the port detection on its own.
For anyone trying to set up Routinator in the future, you may need to explicitly set the port of the service with a label like:
    labels:
      - 'traefik.enable=true'
      - 'traefik.docker.network=traefik'
      - 'traefik.http.routers.routinator.rule=Host(`rpki.tld`)'
      - 'traefik.http.routers.routinator.entrypoints=websecure'
      - 'traefik.http.routers.routinator.tls.certresolver=letsencrypt'
      - 'traefik.http.services.routinator-routinator.loadbalancer.server.port=9556'

(specifically the last line)

Thanks a lot for the instructions @bswinnerton. Would you be willing to contribute this to the docs on this page?

This seems relevant from the Traefik docs:

If a container exposes multiple ports, or does not expose any port, then you must manually specify which port Traefik should use for communication by using the label traefik.http.services.<service_name>.loadbalancer.server.port (Read more on this label in the dedicated section in routing).


Infinite $digest Loop

I have a large Angularjs app, where I want to use ui-grid.
I started from the test example - http://ui-grid.info/docs/#/tutorial/101_intro. Added data to controller and added row to view. The grid works, but I have a lot of error about Infinite loop in my console.
New errors are generating when I try to scroll grid or when invoked $apply.
You can see error details  here
Thanks.

I tried to use trick for grid in modal ( http://ui-grid.info/docs/#/tutorial/110_grid_in_modal), but I still get Infinite loop errors while scroll table.

I'd suggest starting at 101 (or one of the slightly more complex tutorials), and then clicking 'edit in plunkr'.  Add your logic there, and see if you still get the problem.  If not, then work out what's different between the plunker and your real code.  If you do, then you could post the plunker here for us to look at.
What angular does with the binding/digest loop stuff is that it just iterates over all your bindings re-evaluating them until they become stable.  As a general rule, an infinite digest will occur when you have some data that angular thinks changes every time it runs the digest loop.  When it's happened to me before the usual cause is that I've modified some of my data in a way that means it changes every time it is evaluated, by for example creating a new object in a function, rather than returning a pointer to an existing object if it's already correct.  I see that your data has a function buried within each row, so that's probably the most likely cause.

Possible solution offered, no response for some time, assuming raiser has resolved their issue and closing.


Mourner Clue Scroll

On the cryptic clue whose message is "Stop crying! Talk to the head." the arrow for the client will point to the head mourner on the surface in West Ardougne.  (The one in the building next to the entrance to the city).  It is not possible to complete the clue with this NPC however - the head mourner the clue refers to is the one underground by the jail cell, which is also pointed to by the arrow.

Would be fixed by #2202.

The methods for fixing this are in #2202, but that clue in particular doesn't look like it's locked. I can add it after I finish working out the rebase, but since I'm on vacation it might not be for a while.

Duplicate of #5231


Warning fix using MemberNullAttribute causes accessibility level error.

          > I think a better solution is to generate

 public string BaseUrl
 {
     get { return _baseUrl; }
     [System.Diagnostics.CodeAnalysis.MemberNotNull(nameof(_baseUrl))]
     set
     {
         _baseUrl = value;
         if (!string.IsNullOrEmpty(_baseUrl) && !_baseUrl.EndsWith("/"))
             _baseUrl += '/';
     }
 }

To let the compiler null analysis know that the property setter sets the field _baseUrl.

In my netstandard2.0 project, this gives: 'MemberNotNullAttribute' is inaccessible due to its protection level.
Originally posted by @kvansaders in https://github.com/RicoSuter/NSwag/issues/4704#issuecomment-1927882073

I'm seeing the same issue as the other commenter on that issue, where the build completely breaks with the following error message:
Error	CS0234	The type or namespace name 'MemberNotNullAttribute' does not exist in the namespace 'System.Diagnostics.CodeAnalysis' (are you missing an assembly reference?)

as it is, 14.0.3 breaks netstandard compat from what I can tell

Same issue on my side, MemberNotNull shouldn't be emitted when targeting netstandard2.0

The same issue for targeting WinX86 runtime for net framework 4.5.2

The feature System.Diagnostics.CodeAnalysis.MemberNotNull introduced with .NET 5 as MS documentation declares at https://learn.microsoft.com/en-us/dotnet/api/system.diagnostics.codeanalysis.membernotnullattribute?view=net-8.0
So, the code line
            [System.Diagnostics.CodeAnalysis.MemberNotNull(nameof(_baseUrl))]

should be as follows to solve the compilation error
#if NET5_0_OR_GREATER
            [System.Diagnostics.CodeAnalysis.MemberNotNull(nameof(_baseUrl))]
#endif

I need to multi-target net40 and this makes 14 unusable, have to use 13.

Forward duplicated by #4833, fixed since 14.0.6


top-level/release-haskell.nix: also support aarch64-linux and x86_64-darwin

top-level/release.nix builds haskellPackages on those systems as well,
so we should test on them as well. Currently we have virtually no
insight into the state on there.

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I'll check if we this is feasible in terms of build resources. Also we'd need to clean up some eval errors by restricting the supported platforms in the configuration for hackage2nix.

I have no opinion on this. We just need to make sure that no one has an issue with us doing this.
I mean yeah, most of those builds would happen sometime after the merge. But it would certainly not lead to less builds alltogether.


Builder enhacement request

Would be possible to have a builder accept a current instance of an object in order to initialise itself? This would let you keep your class immutable but go through the builder to tweak it.
Thanks for some great code!

I use the toBuilder field in @Builder annotaion for this now. Try this out

Perfect, thanks!!


Potential for memory leaks.

For my app, I have LeakCanary set up and it found that an activity was leaked because a dialog I had created implemented the AuthenticationListener interface which was passed to Reprint.authenticate(). According to the LeakCanary Dump, Reprint looks to be holding onto the listener it's given which in my case kept the dialog and activity that launched it in memory.
I have created a workaround for my app, but ideally Reprint would drop its references once authentication is complete.
If it matters, I'm running a Huawei Nexus 6P with Android 6.0.1.

Hmm, interesting. Reprint doesn't keep a reference to the listener that you pass in. Looking at the dump, it it looks like it's actually android.hardware.fingerprint.FingerprintManager that's holding the reference. Right now Reprint keeps a global FingerprintManager to avoid reinitializing it on every call, but if the manager is leaking the listener, than I suppose that I could create a new manager on every reprint call as a workaround. I don't think that will solve the problem, though, since the FingerprintManager is a static system service.
What is the workaround that you use that works?

It's not really a workaround so much as just mitigation. I have a static class that holds a weak reference to my dialog and forwards the calls so that the dialog and activity can still be garbage collected. It's not ideal, but it seems to work.

Looking at it more, Reprint definitely doesn't hold any references to activity contexts. This leak would happen if you used the FingerprintManager directly, so there's nothing Reprint can do, unfortunately.

I don't have the original code nor do I remember the exact situation but it boils down to this:
/*
 * SomeActivity is some heavyweight object that is very costly if leaked.
 */
public class SomeActivity extends Activity implements SomeListenerInterface {

    /*
     * MyListener is a minimal implementation of the necessary interfaces that forwards all of its
     * method calls to the outer class if the parent has not been GC'd.
     *
     * Crucially, because this listener only holds a weak reference, it won't prevent SomeActivity from being GC'd. And because MyListener is a small object, it may still be leaked but it's a matter of bytes being leaked rather than megabytes if an Activity were leaked.
     *
     * This class must be static. A non-static inner class will hold a strong reference to the outer class which we can't have.
     */
    static class MyListener implements SomeListenerInterface {
        private WeakReference<SomeActivity> ref;
        public MyListener(WeakReference<SomeActivity> ref) {
            this.ref = ref;
        }

        @Override
        public void someMethod(String arg) {
            // grab a temporary strong reference and check to make sure SomeActivity hasn't been GC'd.
            SomeActivity a = ref.get();
            if (a != null) {
                a.someMethod(arg);
            }
        }
    }

    @Override
    public void onCreate(Bundle savedState) {
        // instead of directly registering SomeActivity, register the custom listener.
        WhateverManager.addListener(new MyListener(new WeakReference(this)));
        ...
    }

    @Override
    public void someMethod(String arg) {
        // do whatever here
        System.out.println(arg);
    }
}


output folder of the OMTD components

We must have a flat structure for the folders containing the input/output files. That is required for the OMTD Galaxy and workflow-service to handle the results.
this issue is related to issue #51
see also the comments from here https://github.com/openminted/omtd-docker-specification/issues/2#issuecomment-375707398

Just to mention that the required parameters are currently only available in DKPro Core 1.9.1-SNAPSHOT. I am planning to do a 1.9.1 release soonish - need to check that the metadata of DKPro Core meets the latest OMTD requirements and that the OMTD Maven Plugin are released first.

@reckart wrong thread?

@rbossy not sure. I had the impression that the discussion from the docker spec branched out to here seeing this issue being linked there when we discussed that Alvis components should be reading the type system as well.

@mandiayba ist this working properly now?

yes   @rbossy


how to deploy yurita on production

gr* talk on Reversim 2019
there are no guidelines on how to deploy yurita on production env.
how does it interact with spark?
is it only using the standalone mode? or can it be integrated with a spark cluster ? or EMR?

Thanks!
Yurita is a framework that enhances spark by adding anomaly detection capabilities. Meaning that essentially it is a spark (structured streaming) application that needs to get deployed, so you can use the guidelines that your organization has for deploying spark applications.
And it supports all spark modes also yarn and not only standalone.


Update for Bootstrap 3.3.0?

@tjwudi there is a workaround feature ('overrides'), to getting the LESS of bootstrap relying only on the original/official bootstrap package dependency. (which has always the latest)
Add the following to your bower.json file after including the bootstrap dependency.
"overrides": {
"bootstrap" : {
"main": [
"less/bootstrap.less",
"dist/fonts/glyphicons-halflings-regular.eot",
"dist/fonts/glyphicons-halflings-regular.svg",
"dist/fonts/glyphicons-halflings-regular.ttf",
"dist/fonts/glyphicons-halflings-regular.woff",
"dist/fonts/glyphicons-halflings-regular.woff2"
],
}
}

Thanks, this helps! @igorlino


Improve CheckValidFile and CheckValidDir

We have generic helper functions like CheckValidFile() and CheckValidDir(). In most cases they work fine. However, there are also concerns that these helpers are potentially buggy, especially regarding permissions or security. We should improve those helpers or fix potential risks.
See also https://github.com/kinvolk/kube-spawn/pull/200#issuecomment-343917333

Not valid at the moment. Closing.


pylint doesn't work

Issue Type
Bug
Description
I'm reporting this from vscode.  If it is in the wrong place, please let me know where the python extension forum is.
It use to work, what I mean by that I would see the results of pylint on the current file.  Now I have to run it in terminal and get the results.  So, I don't get to click on it and go to the line.


Extend naming system

Naming system could become more sophisticated:
Introduce ability for council to verify names. Deposit FEE+BOND; council verifies your information (by checking your website on which the account ID should be published, and querying you on riot); if good, BOND is returned, FEE goes to treasury. If bad, BOND+FEE goes to treasury.
In the future, more identification fields can be added.
#[derive(Encode, Decode)]
pub enum Field {
  Web,
  Riot,
}
/// Information that has been verified by the `VerifierOrigin` (which will initially be configured
/// to be the council, but could be a community-led oracle, too).
Verified: map T::AccountId => Vec<(Field, Vec<u8>)>;

Secondly, introduce basic vouching system, so that outfits who control a lot of public accounts (e.g. Cryptium) are able to vouch for each individually without bothering the council for each one.
/// Ordered account IDs of accounts that have been vouched for by the key. The balance
/// is how much has been deposited so far for registration. Except for fee changes, It should
/// be proportional to the number of entries. The size of the `Vec` should be capped at
/// something sensible (e.g. 100). If additional entries are needed, then multiple levels of
/// indirection can be used.
Vouched: map T::AccountId => (T::Balance, Vec<T::AccountId>);

It's very good idea.


Draft: Workflow execution

This PR adds requirements to allow executing workflows instead of single applications only.  This relates to #35 and #36.

We'll have to merge this with #34 somehow 😂

nice! Let's have some fun! :D

Update with some of the changes from #34. @mocsharp If you can review that'd be great.
Some notes/comments:
General
My wording in the requirements is slightly more generic ("conditional statements" instead of "evaluators") because I think it's important we try not to write the requirements based on a specific design - it's the wrong way round. Let me know what you think.
Data Ingestion

"REQ-DI-001 MWM SHALL allow users to upload data": Do we really need an upload API, given that we have a requirement to allow users to notify of data arrival? When will it be useful to send payloads using the API, rather than save it to storage & send a notification? I did add it as an R3 requirement for now.

Workflow

"[REQ-WF-002] MWM SHALL be able to process multiple workflows for a given payload" - I have it down as a requirement for R3, not R1 - I believe most use cases can be accommodated by having one registered workflow or at the very least mutually exclusive workflows.
"[REQ-WF-003] MWM SHALL be able to define evaluators before and after executing a task" - as I said above, I phrased this in a more neutral way, "MWM SHALL allow workflows to contain conditional statements" - I think it works better as a requirement since it's not tied to a specific design.
"[REQ-WF-004] MWM SHALL be able to execute a task with any external orchestration engine or service" - I have this down under Task Management requirements, as it's not really a workflow requirement per se.
"[REQ-WF-005] MWM SHALL allow tasks to join and feed results into a downstream task" - I think this is a fairly advanced use case which can be worked around at first, so think this should be R3, not R2.

Data Export

I think you missed a requirement of registering export sinks & sending results to them - you only had one about sending to multiple destinations, and that was for R1. I wrote down two requirements - a single destination (but multiple available!) for R1 and then sending to multiple destinations in R3.

Task Management

I think this is an important section to document the requirements around how we execute tasks. I included requirements for asynchronous execution & callbacks, supporting plugins (you had this one under WF), and tracking task status (was under FR).

Functional Requirements

"[REQ-FR-001] MWM SHALL provide a mechanism for clients to subscribe to notifications" - this was down for R3, but I think it's a must have for R1, because this is how we send data back to the MIG (otherwise clients have to poll for results, introducing latency).

Non Functional Requirements

I've added a new requirement (and section) here, around allowing users to retrieve logs for executions by a unique correlation ID for that execution. This is really important when execution a workflow, so that you can see the end-to-end logs for that workflow execution.


Calculate cardinal direction distances between two coordinates

Given you have two Coordinates:

Point1 at (51.049531, 13.720699)
Point2 at (51.069706, 13.775459)

you'd like to know the shortest possible right angle to reach the Point2 from Point1

This would be possible by determining the point in the NorthWest corner (Point3) and calculate the distances Point1<->Point3 and Point2<->Point3.
As the sortest possible distances should be calculated, there has to be a check before: If Point1 is in NorthWest of Point2, the shortest directions are to go south and east.

Point1 in reference to Point2
Directions

North
South

East
West

South
North

West
East

NorthWest
South, East

NorthEast
South, West

SouthWest
North, East

SouthEast
North, West

The only real new logic would be this determination.
The determining of Point3 could be done via the class Bounds, the calculation of the distances via the class Coordinate.
What do you think about this feature? If something is unclear, please ask me. :)

That sounds like it's a useful feature which should be relatively easy to implement :-)

Nice, I'll implement it in the next days.


ci: setup pull request workflow

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


standardizing script formatting

Do we want to enforce Google Style Guidelines and typing with mypy? If so, I am happy to make a pass through the existing scripts and update them.

Okay with me


Existing use of org_name, org_unit, org_uid should be converted to use the Organization object

We consolidated all organization attributes into the Organization object, which should be used in place of the former standalone attributes in:
Device Config State
Device Inventory Info
Cloud
These would become org.name, and org.ou_name for Device Config State and Device Inventory Info
and
org.uid for Cloud

Merged.


test(connector-besu): migrate v21-get-past-logs-endpoint to Jest

test(connector-besu): migrate v21-get-past-logs-endpoint to Jest
Primary Changes:
packages/cactus-test-plugin-ledger-connector-besu/src/test/typescript/
integration/plugin-validator-besu/v21-get-past-logs-endpoint.test.ts

Fixes: #3572
Signed-off-by: Udhayakumari<EMAIL_ADDRESS>Pull Request Requirements

[ ] Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
[ ] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[ ] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[ ] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[ ] Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

Hello @outSH @petermetz, I have made changes you suggested. Please re-review and let me know!

@petermetz I have fixed the linter error. Please re-review.

@petermetz , I have fixed the linter error. Please re-review.


Update NAB object status appropriately if VeleroBackup already exists for the NAB object

Are we able to reproduce this error constantly :question: I think we talked about this being a corner case
If it is a corner case, would not prioritize it now

Agree this is corner case as we generate names based on the NAB name, namespace and hash, so it's really corner case to expect VeleroBackup to be there beforehand.

Right now I get the following status on NAB object if the VeleroBackup object already exists:

@shubham-pampattiwar now I am confused
is not the bug you are worried, but only a feedback message that NAB created Velero Backup :question:

We want to keep NAB : VeleroBackup :: 1:1

I will try to reproduce this and comment again

@shubham-pampattiwar can you test this in https://github.com/migtools/oadp-non-admin/pull/73 ?
I think the error was related to the race condition I fixed there
On your example https://github.com/migtools/oadp-non-admin/issues/59#issuecomment-2096833461
it failed here
https://github.com/migtools/oadp-non-admin/blob/4b2f57f3ad905456240e230895148a6d8f2a4be1/internal/controller/nonadminbackup_controller.go#L306-L315
and on the next reconcile it went here
https://github.com/migtools/oadp-non-admin/blob/4b2f57f3ad905456240e230895148a6d8f2a4be1/internal/controller/nonadminbackup_controller.go#L265


MovingPlanes are not correct

Most notably with e5021, MovingPlanes do not rotate correctly with non-default parent orientations causing a number of bugs e.g. in the cloud realm (1030, 1032, 1040)
Additionally the texture animation does not loop seamlessly

7cd37e10cc1af15dc9975bcd1aa40b9327babdbb fixed some general parts of MovingPlanes but not all. Good thing that useDirection is only used in e5021, but I need more time (and probably dynamic analysis at this point) to figure out what horrible math is causing Zanzarah to render these two quads.


After 3 hours of training on raccoon dataset, accuracy not good.

So my question how long does it take to train model from scratch? I was using SSD512 before this, and it gives me good result after 14hours.

I haven't check raccoon but kangaroo working well.

How long does it take to train it?

what should be the minimum configuration of laptop  to train model ?

I was training model on CARPRK(aerial parked cars) dataset 14hours on GTX 960M 4GB
Loss 5.05 and result is nothing. I tested it on racoon for 3 hours to see if the model converges, but also not get any results(even on trained data).
So that's why asking, cause i don't know is this normal behavior of Yolo3, or there is a bug somewhere.

I don't know. I get the best result with raccoon. Check out the readme page.


Table view names gets reset to default name "Table" when adding fields to table after changing the view name

What happened?
The table view name gets reset to the default name "Table" when adding new fields after changing the table view name.
What did you expect to happen?
The view name should be persisted even though other changes, such as adding fields, are performed.
How can we reproduce it (as minimally and precisely as possible)?

Create a project based on Folder
Open the project in Project view (Table)
Add a new note
Change the view name to something else than "Table"
Add a field
Check the view name which now have returned to the default name "Table".

Anything else we need to know?
No response
Plugin version
1.17.1
Obsidian version
1.5.2
OS
Windows

Thanks for the report! As the new field creation is coupled with previous apis in an ugly way, some features are affected and some dirty values will return after the new field creation. I've recorded a similar problem in #733. There will definitely be a fix but it  will take some time.
A temporary workaround is, you can close the Projects tab after renaming a field, then reopen and create a new one. Then your rename will persists.


Implement data and index kv sort in lightning

Feature Request
Is your feature request related to a problem? Please describe:

To remove importer component. we need to split & scatter region in lightning before generate sst, so the sortkv need to confirm first.
Describe the feature you'd like:

add new abstract backend(local)
writeRows to local and sort (maybe use goleveldb or something)

Describe alternatives you've considered:

data kvs can sort partly but index kvs need sort globally

Teachability, Documentation, Adoption, Optimization:

Note: This is part of #300. Close?

Local backend has been implemented, considered fixed.


Support for other IoC containers

It appears there is a hard dependency on TinyIoC -- it would be great if there was an easier way to support other containers.  Possible create a DefaulftNancyBootstrapper class that takes a generic type that has to be derived from a specific interface (IDependencyResolver or something like that).
Thoughts?

Errrrrrr... http://www.nuget.org/packages?q=nancy.bootstrappers

Seems like a lot of work to add support for a new one :)  If you look at what we did in SQLite, in XLabs, or what was done in Json.NET - there was a simply interface that has 2 methods (CanResolve, and Resolve) that needed to be implemented.

Container abstractions nerf the power of containers to point of any differentiation being pointless. Our system is easy to implement and yet allows users the full power of their chosen container. CanResolve and Resolve is a slippery slope towards the CSL.


[BUG] arc sync keeps asking for git credentials

Describe the bug
All steps to authenticate PC against datahub were done:
git config --global credential.helper cache
git config --global credential.helper store
arc remote accesstoken get -s https://git.nfdi4plants.org

Still, during arc sync git credentials manager pops up and asks for credentials.
OS and framework information (please complete the following information):

OS: Windows 10 Pro 22H2
git version 2.40.0
Standard user, not admin.

Hey, first of all the naive question:
Does the gitlab user account you log into when getting the access token have the necessary rights to sync?
If so, could you check whether the token gets correctly stored?
You can find this information by opening the Credential Manager application (you can find it via windows search). Then in the Windows Credentials section under Generic Credentials there should be a credential stored for git:https://git.nfdi4plants.org.

Does the gitlab user account you log into when getting the access token have the necessary rights to sync?

Yes. ARC owner.
Thanks, I'll forward that.


URL decode URL form encoded parameter key. Add spec.

Parsing of parameters with encoded characters in keys (such as key[1]=value) was broken between Beta1 and RC1.

OK, thanks!