Added custom mapper support.

Description
Added support for user defined object mappers at fuel-jackson.
Motivation
I'm working on a layered spring boot web application following Domain Driven Design.
By default, our services communicate using snake case Json's, so at application startup, we create an object mapper and configure it to use snake case, along with other things. We use this mapper as a Bean, so we can inject it in other layers (like infra) and reuse the same behavior.
This change adds customMapper parameter to responseObject methods, so users (like us) can pass their own mappers.
Important:
I've renamed the property 'mapper' to 'defaultMapper', this can cause some applications to break if them are accessing this property. But I think this is very unlikely, and this property could even be internal or private. Please let me know your opinion.

Type of change
Check all that apply

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (a change which changes the current internal or external interface)
[ ] This change requires a documentation update

How Has This Been Tested?
Created the method jacksonTestResponseSyncObjectWithCustomMapper for custom mapper testing. I think the rest is pretty much covered. What do you think ?
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation, if necessary
[x] My changes generate no new compiler warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Inspect the bytecode viewer, including reasoning why

Can you added more Tests for this to make Codecov happy?


request: specific tags for text customization

Is your feature request related to a problem? Please describe.
I need to use text customizations like spoilers, quotes, code blocks and so on outside of embeds.
Describe the solution you'd like
Tags like <DiscordBold>, <DiscordSpoiler> or <DiscordQuote> for example.
Describe alternatives you've considered
Using my own CSS on span elements, but it feels wrong knowing I need to take into account the discord-message parents as well as the theme used.
Additional context
N.A.

Do you happen to already have your own CSS? If so, can you make a PR? Or otherwise you can share that.

My CSS is not fully styled like Discord and I have written it in the most easiest form so I am not sure it might be of good use. I mean anyone can set font-weight as bold...

I mean anyone can set font-weight as bold..

That is literally all that Discord does though. Wrap the text in <strong> tags and strong tags have font-weight: 700.
Anyway see commit above, they are added.

Published as v3.2.0


tcmalloc: fragmentation overhead instrumentation

This patch adds visibility into the overhead due to fragmentation for each size
class in the tcmalloc central free list, which is helpful when debugging
fragmentation issues.

Thanks for patch. Looks great. Can you please combine to single commit and rebase on top of master. We keep our history clean and it is unlikely somebody would care about your "fat fingered" fix few years from now. Much better if you squash those two commits together.

ok, for the sake of keeping this super clean, I made a whole new patch and PR, with a single commit here: https://github.com/gperftools/gperftools/pull/1173. I'll just shut this one down in favor of that


Updating servicefabric sdk package version

Related command

Description
Updating python sdk package
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

Please fix CI issues.

@mwesigwaguma Please address these CI issues

Please note that we are launching the release for this sprint this week. Please resolve all comments by tomorrow, otherwise the release of this PR will have to be postponed to the next sprint (on 09-26)


Fix shortcuts table formatting

Just a minor style fixes of shortcuts.md table. Seems like it was broken in 719bd8cfa135dcc2b315ff65cc58aad29e1f1dbf

Is this a result of ./run fmt? It should format all md files.


support negative timespans

fixes issue reported in #434

I don't think this fixes #434. That looks like a logic error. This fixes #436 though. Thanks for the fix.

Thanks for the contribution. This is now released to NuGet as v1.37.0.

sorry for the wrong tag. thanks for the pull.

On Jul 2, 2015, at 12:02 PM, Mehdi Khalili<EMAIL_ADDRESS>wrote:
I don't think this fixes #434. That looks like a logic error. This fixes #436 though. Thanks for the fix.
—
Reply to this email directly or view it on GitHub.


Strongly typed enumeration support

C++11 introduces enum classes which are not implicitly convertible to integers. This stops the standard CHECK_EQUAL etc macros from working.
To fix this you then have to write a streaming operator
std::ostream& operator<<(std::ostream& os, const EnumeratedType& val)
{
os << static_cast<std::underlying_type<EnumeratedType>::type>(val);
return os;
}

or cast to an int in the macro.
CHECK_EQUAL((int)EnumeratedType::OK, (int)actual);

This gets old pretty fast. Is there a possibility of writing some clever template trickery to support enum classes? It could be selectively enabled by detection of c++11.
std::enable_if, std::is_enum look like good starting points.

I closed #52 due to the lack of a response to comments; I may use it as a reference for implementing this myself later this week.


Updated copyright to 2016

@aqnouch thanks for the contribution, but we'll actually be updating the copyright internally across all of our repos :)


Question about Correspondence Mapping？

Hi，thank you for your work that I can know how to get U and V, I want to ask you some question about paer
papaer:"A correspondence map is a 2-channel image with values ranging from 0 to 255...... Once textured, we get a bijective mapping between the model’s vertices and pixels on the correspondence map"  " where C stands for the number of unique colors of the correspondence map, i.e., 256"
In your code, I can't get relation between color and UV, What is the meaning of UV?
Looking forward to your reply!

UV Mapping - [https://en.wikipedia.org/wiki/UV_mapping]
I'm figuring that out for myself. This all a work in progress.
But what my understanding is that you're converting the point cloud data to coordinates in UV mapping but I still don't know how the number of colors come into picture.


doesn't work after v0.10.2

Describe the bug
running anything gives this error (`chatgpt, chatgpt help, chatgpt config, etc):
WARNING: backend 'chatgpt-api' has been renamed to 'api', and support for the old name will be removed in a future release. Please update your config file to
use 'api' for the 'backend:' setting.

SchemaUpdater - ERROR - Error during schema update process: Path doesn't exist: '/home/kira/.local/lib/python3.8/site-packages/chatgpt_wrapper/backends/api/schema/alembic'.  Please use the 'init' command to create a new scripts folder.
SchemaUpdater - ERROR - Stack trace: Traceback (most recent call last):
  File "/home/kira/.local/lib/python3.8/site-packages/chatgpt_wrapper/backends/api/schema/updater.py", line 84, in update_schema
    latest_version = self.get_latest_version()
  File "/home/kira/.local/lib/python3.8/site-packages/chatgpt_wrapper/backends/api/schema/updater.py", line 58, in get_latest_version
    script = ScriptDirectory.from_config(self.alembic_cfg)
  File "/home/kira/.local/lib/python3.8/site-packages/alembic/script/base.py", line 216, in from_config
    return ScriptDirectory(
  File "/home/kira/.local/lib/python3.8/site-packages/alembic/script/base.py", line 97, in __init__
    raise util.CommandError(
alembic.util.exc.CommandError: Path doesn't exist: '/home/kira/.local/lib/python3.8/site-packages/chatgpt_wrapper/backends/api/schema/alembic'.  Please use the 'init' command to create a new scripts folder.

An error occurred during the schema update process. Please check the logs

To Reproduce
already described
Expected behavior
it should open chatgpt
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS] ubuntu 20.04
Browser [e.g. chrome, safari] chrome
Version [e.g. 22] v0.10.2

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
log attached
chatgpt.log

have you find any solution about it?

Clearly the Python packaging system and I are not friends...
I believe the problems are now fixed in 0.10.3.


IMPORTANT Your shell script breaks all apps using MiDaS

Your install.sh script causes every single Python project using your module to break.
It is used by some Automatic1111 extensions and NONE of them work unless manually copying the externals/Next_ViT into the Web UI root folder.
Error loading script: depthmap_for_depth2img.py
Traceback (most recent call last):
File "X:\StableDiffusion\webui\modules\scripts.py", line 195, in load_scripts
module = script_loading.load_module(scriptfile.path)
File "X:\StableDiffusion\webui\modules\script_loading.py", line 13, in load_module
exec(compiled, module.dict)
File "X:\StableDiffusion\webui\extensions\depthmap2mask\scripts\depthmap_for_depth2img.py", line 11, in
from repositories.midas.midas.dpt_depth import DPTDepthModel
File "X:\StableDiffusion\webui\repositories\midas\midas\dpt_depth.py", line 5, in
from .blocks import (
File "X:\StableDiffusion\webui\repositories\midas\midas\blocks.py", line 21, in
from .backbones.next_vit import (
File "X:\StableDiffusion\webui\repositories\midas\midas\backbones\next_vit.py", line 8, in
file = open("./externals/Next_ViT/classification/nextvit.py", "r")
When an app uses your module, it will expect the externals/Next_ViT directory at the app root.
You cannot import a module in Python to a fixed location because that will break the dependency, since you're installing it in the wrong location. Your module won't find the module, because the main app installes it to it's own root, not to your module's root.
Anyone using your module will have to manually copy the externals folder to their own app's root folder, which is really unacceptable.
You should clone it to your own project's root using python git  and load it by instead of using
file = open("./externals/Next_ViT/classification/nextvit.py", "r")
in midas/backbones/next_vit.py
Use
file = open("../../externals/Next_ViT/classification/nextvit.py", "r")
Also, a shell script to run a git command? Really?? You think nobody uses another operating system?
This is why I say you MUST use Python's own git module.

This also breaks models loaded through torch hub.

@cooperdk you sound like you know what's up here. :) Do you have a pull request that would fix the issue? even if it's just a path fix to the actual location of the script?
This is the first time I've had to look under-the-hood on MiDaS - I'm just a new user (via disco) so I don't feel confident in offering a solution myself just yet.
Better still, maybe you can resolve the issue of needing to call the install_next_vit.sh shell script at all, with your suggestion of using pythons git module.

I'll take a look at it ASAP.

Oh dear, bit of a mess. The bytedance/Next-ViT repo isn't set up to be used as a module. Specifically, https://github.com/bytedance/Next-ViT/blob/main/classification/nextvit.py imports utils directly rather than relatively.
I've raised https://github.com/bytedance/Next-ViT/pull/11 - if this gets merged, you could then add https://github.com/bytedance/Next-ViT as a git submodule.
(The other option is just to copy nextvit.py and utils.py into the MiDaS project, but there's a license issue to doing that - bytedance/Next-ViT is Apache 2.0, and this project is MIT)

Here is a fork I've made to temporarily fix this issue (not tested yet).  Just for reference. I do not recommend using this fork. I make no guarantees about maintaining it.

Go to the file /content/MiDas/midas/backbones/next_vit.py and edit the line

file = open("./externals/Next_ViT/classification/nextvit.py","r")

to

file = open("/content/MiDas/midas/backbones/vit.py","r")

And it is solved

Hi guys. Thanks for pointing out this issue. We will work on a solution ASAP. @hafriedlander thanks for your suggestions. @shamnad-sherief your solution does not exactly solve the issue, it will just run the ViT backbone instead of the NextViT backbone.


go: 1.7.3 -> 1.7.4

Motivation for this change
This version includes a security fix for DoS vector
/cc @grahamc
Things done

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

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I'm taking care of this as part of https://github.com/NixOS/nixpkgs/issues/21142

Included in my branch. Thank you!


init.js:31 Error: multipleDefine(…) src: dojoLoader

hi, @tomwayson . i have try your example and it runs ok. but i when i try put arcgis api into my project, it shows an error shown like title, when add the   into index.html. do you know why?
ps: does it matter if the output.libraryTarget is set to umd in webpack.config.js?

I would expect umd to work. I assume you tried amd to resolve this issue, right?
Other than that, what changes did you make between this set up and that of your project? Does your index.html load other script tags besides the ArcGIS API for JavaScript? The multipleDefine error is commonly seen when other UMD libraries are loaded on the page before the ArcGIS API for JavaScript. Any of these suggestions (placement of script tags, using defer on the ArcGIS script tag) might work in your case.

@tomwayson thank you for your reply. yes, i have tried amd, but still failed. and i tried to using defer on the ArcGIS script tag, the multipleDefine error is not display, but it seems not load arcgis module success yet.
i have commit my test project to github(https://github.com/dyh333/ng2-arcgis-test). the project is forked from angular2-webpack-starter(https://github.com/AngularClass/angular2-webpack-starter). and what i changed is list below:

add arcgis script and css path in src/index.html
add externals config in config/webpack.common.js #304
change output.libraryTarget var to umd in config/webpack.dev.js #97
add typings.json file which is copy from your repo, and run typings install

after that, i tried to import esri/map module in src/about/about.component.ts #4, and in ngAfterViewInit function, load a simple basemap.
and run npm install;  npm start to debug by webpack-dev-server.
would you kindly help me have a look what is wrong. i have spent many days to solve it, but failed again and again. thank you very very much.

One thing, I do not see that you're index.html calls require() to load in the bundles the way this repo does here.
If that doesn't work, I would say the best approach is to start w/ a clone of this repo add then the parts of of the the angular2-webpack-starter that you need one by one.

@dyh333 let me know if you want me to try something as I tried various options and remained unlucky.

Did you try the two things I suggested in my previous comment?

@tomwayson yes I tried both. multipleDefine error solved by using defer in the script tag of arcgis-js-api. Using require in the index.html gives error " Uncaught ReferenceError: require is not defined ".
In webpack2, bundles are being loaded using script tag.

I've not tested this approach w/ webpack2, so that may not work at all. For now, I would advise against using that.
In order to use this approach, you must load the bundles via Dojo's require(), which will not be defined until the ArcGIS API for JavaScript is loaded on the page. You cannot load the bundles via script tags. Did you place your script with require() before or after the ArcGIS script tag?
I know that I need to do a better job of documenting how the hack that allows you use the ArcGIS API for JavaScript w/ Webpack works. In short, we are configuring Webpack to bundle your app code (and optionally vendor code) into a single AMD module that is ultimately loaded by the JSAPI's Dojo loader (via require()). When the Dojo loader parses the code in that bundle and comes across and external dependency (i.e. a module in the esri or dojo packages), it is able to resolve that dependency to the corresponding module w/in the built (CDN) ArcGIS API for JavaScript script.
Until you understand how your entire app is being loaded by Dojo that is bundled w/ the ArcGIS API for JavaScript, I would caution against trying to use this technique w/ other boilerplates.

@tomwayson you are our hero..... loaded modules with require and it worked. yayyyyyyyyyyyy
will write the steps shortly......
@dyh333 can I commit to your code?

@tomwayson I can update your angular2-esri-example with webpack2 ...... I already used that example in one of my project accessible with the link ... https://hassanqaiser.github.io/

That's great! I'm glad you got it working. Anything you can do to document the steps you needed to take to get it working w/ the angular class angular2 webpack starter would be appreciated. Perhaps you could leave a comment in #27?
Also, good to know that it works w/ webpack2! Your app looks great too.
I'd gladly accept a pull request to update to webpack2.

@hassanqaiser  great! please commit your code, i need it! would you tell me how you solved it?


fix active link

when we go to the About page the navbar "about" should have a darker color.

hi, can i work on this ?

Can I be assigned this task?

hi @Obikaviola 👋🏻. any updates

assign this to me under hacktoberfest accepted please

assigned @Blink29.
make sure it also works for the pricing and project section

assign this to me under hacktoberfest accepted please


Updates some tests and the tests.sh script for compatibility with Windows

This is part of the ongoing effort to make Nit work under Windows using msys2 / mingw64, this time we look at tests and support tools.

Tweak tests.sh to work correctly on Windows: diff ignore CR, don't use -s on hostname from coreutils and don't use the $tmp variable as it overwrites an environment variable.
Make the very badly named test a bit less bad so its path is valid on Windows.
Fix the pkg-config annotation behavior on multiple arguments and fix its broken tests. (This is not directly related to Windows support, but valid tests will help to debug.)
As a bonus, fix the repeated typo boostrap.

If I remember well, the “very bad name” test is here to test, among other things, that \ is correctly handled. With your renaming, it won’t test this. In another hand, I don’t know how we can test POSIX names while being compatible with Windows.

+1 !


[12.x] Switch models to UUID v7

Based on #52029, I'm proposing to switch the HasUuids trait to UUID v7.
This change is similar to the reverted #44210 but addresses its privacy concerns: While #44210 switched all UUIDs to v7, this PR only targets model UUIDs and they already contain a timestamp in the current implementation.
v7 is a standardized format, while the UUIDs generated by Str::orderedUuid() at the moment are only a custom variant of the v4 standard.
At least on PostgreSQL, v7 UUIDs also seem to perform much better.
Upgrade
This is a breaking change for users of the trait that rely on the sortability of UUIDs. The affected models need to continue generating UUIDs with Str::orderedUuid(). I added a legacy HasVersion4Uuids trait because that's the most convenient solution, IMO.
I removed the HasVersion7Uuids trait from #52029 because it's redundant now. We could also deprecate it and remove it later.

It seems like that Postgres tweet is referring to unsorted v4 UUIDs. I'm not sure sorted V4 to sorted V7 migration would show same performance gains?


Promise.done is deprecated.

Atom now uses ES6 Promises instead of Q. Call promise.then instead of promise.done
Promise.done (/Applications/Productivity/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1214:5)
addPreviewForEditor (/Users/foxd/.atom/packages/plantuml-preview/lib/plantuml-preview.coffee:45:48)
toggle (/Users/foxd/.atom/packages/plantuml-preview/lib/plantuml-preview.coffee:66:14)
atom-workspace.<anonymous> (/Users/foxd/.atom/packages/plantuml-preview/lib/plantuml-preview.coffee:137:16)
CommandRegistry.handleCommandEvent (/Applications/Productivity/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
handleCommandEvent (/Applications/Productivity/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59)

+1

+1

+1


Installation: 301 on cookbooks installation

Hi! I am trying to set up a development server of my own. I am now on step 5 and I get a following error:
[nina@leafblade tomatoes]$ librarian-chef install Could not cache bluepill/1.0.6 <http://community.opscode.com/api/v1> from http://community.opscode.com/api/v1/cookbooks/bluepill because 301 Moved Permanently!
Can I do something to bypass this error?

@matteodepalo is the one who setup that process in the first place, let's see if he can help you solving this issue.

@Liarra @potomak the Cheffile was updated 4 years ago so some of the libraries might have disappeared or they might have been moved to other places. To fix this we would need to look at the git urls in the Cheffile and update all of them to the latest versions. Also check if there are greater versions for site 'http://community.opscode.com/api/v1'. Try bumping that version to the latest one and see what happens.

@matteodepalo can you take a quick look at it and remove cheffile + doc if it's not usable anymore or update it please?

@potomak it would take a while to update the Cheffile and the recipes to the latest version. I would personally remove the files as I don't have the time to update them unfortunately.

Ok, I'll add a task to remove them. Thanks.


spec: auth/token scope should support/consider tags in resource names

Let's say we have the following situation:
docker.registry/namespace/image:latest
docker.registry/namespace/image:beta3

Some users should be able to access the image tagged as "beta3" but others must not.
The current definiton of "Docker Registry Token Scope and Access" does not include a tag, neither does the current "token"-implementation.
related: https://github.com/cesanta/docker_auth/issues/136

We have no plans to support per tag granularity. The issue with such a model is that a tag doesn't necessarily uniquely identify an image. An image could have several tags that represent it, such as 2, latest, or 2.1. The other problem is that a tag is mutable. For example, I could grant access to latest, but the value of latest changes in time that the access token was granted. There are other interesting edge cases that can come up with a tag-based access model. I'd recommend altering your policies and workflow to better match a repository-based model.
If that doesn't work for you, I'd recommend submitting a proposal for how this might be implemented and rolled out. It would have to address the issues identified above and provide some provision of helping with maintenance.


Cesium Earth: Is it possible to make the trailing path thinner?

I'm using the Cesium library to simulate the satellite motion. This code is used to show the path:
path:{
                leadTime:data.data_list[0].period/2,
                trailTime:data.data_list[0].period/2,
                width:1.5,
                material: color
              }

Is there a way to make the trailing path thinner?
As an example, the SpaceX video: http://youtu.be/rQEqKZ7CJlk?t=47m40s

@emackey gave a pretty comprehensive answer to this on Stack Overflow: https://stackoverflow.com/a/50914470/3191039


Support for more proxy protocols

Namely SOCKS and basic TCP/IP, in order to support more corporate environments.

@yeisonvargasf is this something that is in the road map. I think it can be helpful. Would like to contribute to this if this is in plan. Thanks!


make failed, uncaught signal 4 while building unf_ext

Machine and OS: Mac M1, Big Sur 11.1
Ruby version: 2.6.6
Rails version: <IP_ADDRESS>
Unable to run bundle install without getting this error:
Installing unf_ext <IP_ADDRESS> with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

    current directory: /Users/petrgazarov/.rbenv/versions/2.6.6/lib/ruby/gems/2.6.0/gems/unf_ext-<IP_ADDRESS>/ext/unf_ext
/Users/petrgazarov/.rbenv/versions/2.6.6/bin/ruby -I /Users/petrgazarov/.rbenv/versions/2.6.6/lib/ruby/2.6.0 -r
./siteconf20210118-57046-6o2mdv.rb extconf.rb
checking for -lstdc++... yes
creating Makefile

current directory: /Users/petrgazarov/.rbenv/versions/2.6.6/lib/ruby/gems/2.6.0/gems/unf_ext-<IP_ADDRESS>/ext/unf_ext
make "DESTDIR=" clean

current directory: /Users/petrgazarov/.rbenv/versions/2.6.6/lib/ruby/gems/2.6.0/gems/unf_ext-<IP_ADDRESS>/ext/unf_ext
make "DESTDIR="

make failed, uncaught signal 4

Gem files will remain installed in /Users/petrgazarov/.rbenv/versions/2.6.6/lib/ruby/gems/2.6.0/gems/unf_ext-<IP_ADDRESS> for inspection.
Results logged to
/Users/petrgazarov/.rbenv/versions/2.6.6/lib/ruby/gems/2.6.0/extensions/x86_64-darwin-19/2.6.0/unf_ext-<IP_ADDRESS>/gem_make.out

An error occurred while installing unf_ext (<IP_ADDRESS>), and Bundler cannot continue.
Make sure that `gem install unf_ext -v '<IP_ADDRESS>' --source 'https://rubygems.org/'` succeeds before bundling.

In Gemfile:
  pusher-push-notifications was resolved to 1.3.0, which depends on
    rest-client was resolved to 2.0.2, which depends on
      http-cookie was resolved to 1.0.3, which depends on
        domain_name was resolved to 0.5.20190701, which depends on
          unf was resolved to 0.1.4, which depends on
            unf_ext

Subsequently, running gem install unf_ext -v '<IP_ADDRESS>' --source 'https://rubygems.org/' results in the same error.
Any ideas?

Are you able to check that you have a compiler that matches your Ruby version and system architecture?

Are you able to check that you have a compiler that matches your Ruby version and system architecture?

I think the issue was with make binary. It works now, after I removed the wrong binary from the PATH. Thanks!


fix slot #0 texture

If some texture already exists in slot #0, and syncUniforms is trying to use it - mask wont be rendered at all.
Fiddle: https://jsfiddle.net/jdngw3xy/

Here's the fiddle using this PR:
https://jsfiddle.net/jdngw3xy/1/

Sorry, made wrong move :) its ok now


[djl-bench] Add missing tensorflow model zoo dependency

Change-Id: I8d854c0d0353767cae81d999bf4f7fd9616ff5fc
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1143 (e257cd6) into master (7b5bc04) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1143      +/-   ##
============================================
+ Coverage     70.10%   70.12%   +0.02%
- Complexity     5327     5329       +2
============================================
  Files           513      513
  Lines         23696    23696
  Branches       2545     2545
============================================
+ Hits          16613    16618       +5
+ Misses         5724     5720       -4
+ Partials       1359     1358       -1

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/repository/Artifact.java
88.33% <0.00%> (+1.66%)
:arrow_up:

api/src/main/java/ai/djl/inference/Predictor.java
90.38% <0.00%> (+2.88%)
:arrow_up:

Continue to review full report at Codecov.

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


Build error when using inline async methods from libraries built with another version of FSharp.Core

Repro steps
Option a)

Download the sample repo here: https://github.com/dkcsiltberg/FSharpCoreErrorExample
Try to build it with Visual Studio 2019

Option b)

Create a new FSharp library project in VS2019. This will have FSharp.Core 4.6.2.
Downgrade FSharp.Core to 4.5.2 in this project
Add the following method somewhere: let inline f x = async.Return x
Create another FSharp library project (keeping FSharp.Core at 4.6.2 in this one) in the same solution, and reference the previous one
Add the following method somewhere let f = <Module from first project>.f
Try to build the solution
(optional) Do the same again but with the versions of FSharp.Core reversed.

Expected behavior
It builds correctly
Actual behavior
Both referencing project fail with FS0193	The module/namespace 'Control' from compilation unit 'FSharp.Core' did not contain the val 'ValLinkagePartialKey(OnSuccess)'
Known workarounds
Either:

Make sure libraries and projects you use are either

all built with "Old Core" (see related information)
all built with "New Core"

Do not use inline async methods

Related information

Operating system: Windows 10
Editing Tools: Visual Studio 2019

First, let's establish some terminology. I want to divide the verison of FSharp.Core into two groups,
FSharp.Core <= 4.5.2 (henceforth "Old Core") and FSharp.Core >= 4.5.4 (henceforth "New Core"). The reason for this is that the "break" seemed to happen between those two versions (and possibly it happened between 4.5.0 and 4.5.1 as well, but was reversed in 4.5.2).
As can be seen in the example, this is a symmetric issue. It occurs both if the referencing project is built with New Core and the reference project is built with Old Core, and vice versa.
Granted, this is only a problem if you try to use inline async methods (and maybe some other cases we haven't noticed, of course), so this might not be as big of an issue as I fear. However, we've already run into it a couple of times, both shortly after FSharp.Core 4.5.4 was released. The first one was when using the somewhat popular https://suave.io/ which uses async signatures a lot. The second time was from an internal https://github.com/fsprojects/FSharpPlus like library, that uses inline to create generic map and bind functions for monadic types.
So at least in our experience it is a real problem. And if you use two or more libraries all built with different versions, there might be no way out of this, apart from keeping track of which versions of which nuget packages are built with which verison of FSharp.Core. And that is virtually impossible already with just a few packages.
When we first encountered this issue, some googling revealed the following issue that gave the same error and might thus be related: https://github.com/Microsoft/visualfsharp/issues/5378
It was this post that cemented our working theory of the problem. While no changes to signatures were made between Old Core and New Core, this post indicates that some functions were either inlined or deinlined (or something along those lines), and it was this that created a sort of binary incompatibility issue between the two versions that the F# compiler is unable to resolve. Now, the problem might of course be way more complex than that, and this is really not our forte, so we'll leave it at that.
In this post a sort of official answer was also provided, and it went something like this: "why would you ever use another version of FSharp.Core than the one you get when creating a new project from our official templates? That is the only officially supported version for the F# compiler".
While I belive that this position – that the latest version of FSharp.Core available on nuget is not an officially supported one – is not neccessarily a great one since this unspoken rule might cause all sorts of problems unless 100% of the F# community are both aware of it (we were not) and adheres to it (which I sincerely doubt), we said fair enough. Since we only ran into this problem a couple of months ago, we decided to use workarounds and just wait for VS 2019, which would bring with it a new version of the F# compiler and a new "officially supported" version of FSharp.Core according to the unspoken rule. And now this day has come.
But alas, the problem still remains, as can be seen in the example solution. And this is no longer when using unsupported versions of FSharp.Core. And regardless of your position on this unspoken rule, I think we can all agree that being able to use inline async methods from older libraries when using the latest "officially supported" version of FSharp.Core is a pretty decent feature to have.
I apologize in advance if I come off as passive aggressive about this, that is not my intent. Well, maybe a smidge. It DID cause us quite a bit of headache after all. And I really don't like this unspoken rule, in case that was unclear.

Thanks for the report and repro. I can also confirm this fails on my machine using .NET Core:

Library project explicitly using FSharp.Core 4.5.2
Console app referencing the library
Same code as your example with the same error

@dsyme can speak more towards the inlining of async members that was done for F# 4.5 and the ramifications of that. This one feels like a subtle breaking change, but I could be wrong.
I do want to clarify the support position here:

FSharp.Core is backwards compatible, meaning you should be able to reference an older FSharp.Core through some means and not have to change your code
FSharp.Core is binary compatible, meaning you could hot-swap your binaries and runtime and the application would still run
FSharp.Core is not forwards-compatible. That is, use of an arbitrary newer FSharp.Core from an arbitrary older F# compiler is not supported and not guaranteed to work. Although we take strides to make sure this doesn't generally happen, eventually we'll add new concepts to the binary that older compilers do not understand. Failing to take this position would mean we cannot feasibly evolve the F# language.

The linked issue was a binary compatibility issue where the F# binary format was inadvertently changed in a version of the package that is now delisted. You can view the full writeup here: https://github.com/Microsoft/visualfsharp/issues/5369#issuecomment-408205550
As far as I understand it, this issue is a binary breaking change, so it should be looked at.

I need to understand this issue - it is unexpected.  I'll look into it.

(If there is a regression here, as appears to be the case, then it is likely there's little we can do about it, except give good guidance)

If we determine it's a regression then I'll post a blog post on the .NET Blog, update release notes accordingly, file an issue in the fslang-designs repo, update the RFC, and keep this issue (even if closed) as a canonical source of the problem.. At the very least, we should have a fully traceable set of documentation around the problem.

In the repro, in any situation, project BuildErrorExample52.fsproj gives a warning because it references ExampleAsyncLib62, which is invalid, since that uses a higher FSharp.Core.
After changing that, this only repros with FSharp.Core 4.5.2.  I tried global replace 4.5.2 for all of these and the solution built clean:

4.5.1
4.5.0
4.4.0
4.3.4
4.5.4

Given this, I think it's just a glitch in 4.5.2 related to #5378.  Guidance should be that libraries using inline on async code should either target FSharp.Core 4.5.1 or 4.6.2

Ok, we've definitely had issues in more cases than that, so I did some more experimenting.

Referenced \ Referencing
4.5.0
4.5.1
4.5.2
4.5.4
4.6.x

4.3.4
Ok
Ok
Ok
Ok
Ok

4.5.0
Ok
Error
Error
Ok
Ok

4.5.1
Other
Wtf?
Error
Ok
Ok

4.5.2
Error
Error
Ok
Error
Error

4.5.4
Other
Error
Error
Ok
Ok

The most surprising thing here is that 4.5.1 seems to get the error with itself! So perhaps telling people to use 4.5.1 is not the best option.
I also included some "invalid" examples where we use a higher version for the referenced project than the referencing. When it says "Other", it means we get a slightly different error message. The reason for including this is twofold:

The warning you mentioned does not appear if it is only between minor versions, e.g. 4.5.1 references 4.5.2.
Many nuget packages do not specify the specific version of FSharp.Core in their dependecies, they might just state >= <IP_ADDRESS>, when they are in fact using 4.5.0 or something, and it is quite a pain to figure out which specific version it uses.

Yes, I am all for unlisting 4.5.0-4.5.2, that is a great start. It won't make the problem go away immediately tho.
Take a look at this package for instance, which is one of those that caused us problems: https://www.nuget.org/packages/Suave/2.5.3
The dependency is listed as FSharp.Core (>= 4.0.0 && < 5.0.0), so it will be perfectly happy to continue existing unaltered and be resolved properly by package managers even after that unlisting. However, it is actually built with 4.5.0, and there is no way of knowing that until you actually run into the problem, or use ildasm to inspect the dll in the package. Or look at the source, I guess.
But I guess it is the responsibility of library authors to read the guidance and update the packages accordingly. I just don't trust everyone to do that in a timely fashion. Or ever, in some cases.

I have spent two days working through the Suave Music Store demo, and now I got stuck on this issue and landed here. I got the exact error reported here. Using FSharp.Core 4.5.2 from NuGet solved the issue, but I am not sure if that is a workaround I can accept when using Suave for my business projects. If I understand correctly, a solution would be to have Suave compiled with a later FSharp.Core, so I will consider bringing attention to this issue in the Suave repo on GitHub.
It looks like you don't need another repro for this issue, but you can have my demo if it helps - just let me know. (Note to self: revision 382.) It does not depend on a DB like the original demo, so it's just compile and run. VS 2019. Project is a .NET Core 2.2 console application.

I believe we can close this as things are fine with more recent FSHarp.Core.


Vue wrapped planned?

This looks really cool!!
I see you've already got one for React.. any chance you'll show Vue some wrapper love too?

is not planned, but you can contribute


Add TargetPlatform to SeedImageSpec

TargetPlatform is used when building raw disk-images for other platforms. An example being building rpi images on a cluster running on x86_64 hardware.

target platform is ambiguous.
It could be x86_64 vs aarch64. Or is it amd64 vs arm ? Or RaspberryPi vs OrangePi ?
I'd assume you mean target architecture but even then amd64 or x86_64 needs to be clarified.

In the toolkit we use the same platform spec as used in docker and containerd: https://pkg.go.dev/github.com/containerd/containerd/platforms
It can be specified as linux/arm64, linux/amd64, etc (it will also accept linux/aarch64 and linux/x86_64 etc)
I will add some validation and documentation to the CRD to clarify what the accepted values are.


zf-open - cursor hand

Example: http://foundation.zurb.com/apps/app-templates/productivity/#!/
Open profile for off canvas panel, no hand cursor on Profile or Projects.

We fixed this in 1.0.3 (53d5952802e7d7da667c3fb19f73e0a39bc0591e), so now all <a> elements, not just ones with an href attribute, get the pointer cursor. Since there are no hrefs in Angular apps, browsers weren't applying the proper cursor to elements that are technically links.
The templates are all being updated right now, and they'll use the latest version of the framework when they're done.


Nginx cannot connect to pipedproxy

Hello,
I am trying to self host, I ran the configure-instance.sh and chose nginx, I use nginx proxy manager as the reverse proxy.
I followed all of the steps in the self host docs, my instance can show video titles but doesn't show thumbnails or play videos. I looked at the docker logs for the piped containers, they all look working, except the nginx logs which showed the error:
2024/06/21 04:01:05 [crit] 22#22: *53 connect() to unix:/var/run/ytproxy/actix.sock failed (13: Permission denied) while connecting to upstream,
client: <IP_ADDRESS>, server: pipedproxy.example.org, request: "GET /sw.js HTTP/1.1", upstream: "http://unix:/var/run/ytproxy/actix.sock:/sw.js", host: "youtube.example.org"

All of the containers run on the same user so I don't understand why there are permission problems.
I kept all of the configuration files the same and I am running this on a nixos 23.11 server.

Hello, same procedure and I have a 400 Bad Request on a generic Ubuntu 22.04 server


No module named 'elasticapm'

Describe the bug: ...
When I install the package using pip install elastic-apm, the elasticapm module is not found in the library. My pip is pointing to /usr/local/lib/python3.8/site-packages/pip. When I change this to point at Macpython /Library/Frameworks/Python.framework, it works fine. I need to why It is failing since the same is happening inside my Kubernetes pod as well.
To Reproduce

...install the package using pip install elastic-apm
...try importing like this from elasticapm.contrib.django import *
...you would encounter something like this: No module named 'elasticapm'

Environment (please complete the following information)

OS: Mac OS
Python version: 3.8.6
Pip version: 20.2.1
Framework and version [e.g. Django 2.1]:2.2.6

If you want to use Elastic APM agent under the framework Django, follow these instructions https://www.elastic.co/guide/en/apm/agent/python/current/django-support.html
If you want to use the agent out of a framework, follow these instructions https://www.elastic.co/guide/en/apm/agent/python/current/instrumenting-custom-code.html#instrumenting-custom-code-transactions

Hey @abiral-sijaria, I'm going to close this since we never heard back from you. Please comment if you still need help!


External profiles followed by local users show no statuses

Currently, when an external profile is followed by a local user, its content appears empty—none of its statuses are initially visible. A helpful feature is the link that allows users to view the statuses on the profile’s original server. This behavior is similar to how Mastodon and Pixelfed servers function.
However, a more user-friendly option would be to include a button that allows users to fetch statuses (along with reactions and comments) to the local server on demand. This would synchronize the external profile with its local copy, making it easier to engage with content directly from the local instance.
This issue has been described in detail at the following link:
https://github.com/w3c/activitypub/issues/450

This is certainly a functionality worth considering 🤔.
I assume that retrieving statuses would be feasible; however, obtaining comments and likes seems unlikely. These elements are typically not retrievable directly through the ActivityPub API. Another challenge would be maintaining the correct order of statuses. It is not possible to „insert” a new status into an arbitrary position in the database. Status IDs are generated according to the ‘snowflake’ algorithm and are sorted based on their ID numbers.

Actually, the first 41 most significant bits of a Snowflake ID represent a timestamp, which means you can substitute it with a past date from the status being retrieved from the remote server. This should solve the issue of inserting a status into any arbitrary position in the database.
Unless we're thinking about different Snowflake algorithms, the one commonly referred to as Snowflake ID should address the challenge of placing statuses in the correct order.

Yes. But we can try to download statues as "unlisted". Thus they will appear on local timeline but not on the user's timelines. Still not perfect but I don't have other ideas for now.


Bug Report: Okay I don't know if it's a bug but when you go into the charting window you can remove a note whenever I try it just doesn't work.

The charting window

The charting window

The charting window


[Content]: duplicate proxy entry?

What article on segment.com/docs is affected?
https://segment.com/docs/connections/sources/catalog/libraries/website/javascript/upgrade-to-ajs2/
What part(s) of the article would you like to see updated?
In the "Using a Custom Proxy" section, Option 1. It says:

Option 1: Update the proxy so that:
https://cdn.yourdomain.com/analytics.js/* maps to https://cdn.segment.com/analytics.js/*
https://cdn.yourdomain.com/analytics.js/* maps to https://cdn.segment.com/v1/*
https://cdn.yourdomain.com/analytics-next/* maps to https://cdn.segment.com/analytics-next/*
https://cdn.yourdomain.com/next-integrations/* maps to https://cdn.segment.com/next-integrations/*

However, the first two URLs are identical. They are both: https://cdn.yourdomain.com/analytics.js/*. Which of the two destinations is this URL supposed to be mapped to?
Additional information
No response

Hi @onlywei, thanks for pointing this typo out. This has been fixed in our docs.


BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue?

Hi,
I have several times ran aka.ms/getbc (both no and w1) During week 40 and 41.
Full O365 integration, copy AAD users, letsencrypt.

Running With Internet Explorer, both directly on the VM and "Remote" on my desktop (W10 1809, Edge and IE).
1.1 - Getting "Sorry, Your session exppired. Please refresh the page to continue" on initial run of the app.
1.2 - Have to log in to the web Client, og back and try over, then the app is running for a short time before the same issue appeares.
1.3 - The routine wich is giving the match on a contact does not run at all. No "Do you want to make a New contact" ore matching the contact i have manually made.
1.4 - When trying to run the BC App in a "extended window" i always get: "Sign in failed due.....Security token signature.

Running With Chorme (wich always has been the "walkaround" if IE-issues appeares).
2.1 - Not possible to run the App in Chrome, on run the "Sign in failed due.....Security token signature" appeares. No "walkaround"..

I have ran this setup an ran the "flow" regarding the O365/AAD/PowerBI/Excel etc. based on aka.ms/getnav 10-15 times and are therefore quite shore that "my" setups are correct...

BTW: Have also tested With an updated dev.license

Have also tested this With BC onprem CU1 - still the same issue.

This is a Bug related to Docker. Have ran a supportcase on this. Not related to BC, Works fine when running in a standard installation.

Could you send me a step by step description of how you set this up. Might save me some time in doing a repro.
Thanks

Hi, Of course I can 👍

Create new O365 demotenant

Running aka.ms/getbc with the O365 and create AAD-users. Letsencrypt..

When finished go into the web-client or the rtc-client
*   Run Assisted setup
                                                     i.      Set up your Business Inbox in Outlook

 *   Next
 *   Set up for: Choose «My Organization» - Next
 *   Is your email hosted in Office 365 – Yes – Next
 *   The Email/pw is the admin<EMAIL_ADDRESS>demo-tenant – Next & Finish

Send a e-mail to the admin<EMAIL_ADDRESS>

Log into portal.office.com

Goto Outlook Inbox

Find your test-email – Open this

Now you will se the «Dynamics 365 Business Central……» and «Document Wiew»

Click on the first – and you now get «Select a contact» this is the first Issue – should have asked if you want to make a new contact. If you make this manually the next time you run you will get the same issue
*   In Dyn NAV 2018 and previous versions there was a «log on» window – it is possible that i have read that this is changed i BC…

Try to «Pop Out» Then you see the second Issue and further on…

Good Luck 😊
Vennlig hilsen / Kind regards
Dag Fr. Salvesen
Løsningsarkitekt og Salg/Marked
M: +47 900 99 399
Fra: Freddy Kristiansen<EMAIL_ADDRESS>Sendt: onsdag 5. desember 2018 14:43
Til: Microsoft/nav-docker<EMAIL_ADDRESS>Kopi: Dag Salvesen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Could you send me a step by step description of how you set this up. Might save me some time in doing a repro.
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nav-docker/issues/275#issuecomment-444489068, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ap4CN5QKK3pyLU51E3sqwc70kBY99biLks5u181TgaJpZM4XZDJg.

Dag - could you open ISE, run
Enter-NavContainer navserver
cd \inetpub\wwwroot\nav
psedit navsettings.json
add https://*.office365.com to AllowedFrameAncestors
Exit the pssession
use Restart-NavContainer and remove cookies etc. from your browser
test again

Hi,
Did but still the same issue ☹
This is how the string now are:
"AllowedFrameAncestors":  "https://.office365.com,https://.dynamics-tie.com,https://.dynamics.com,https://.sharepoint.com,https://.office.com,https://.officeppe.com,https://.dynamics-ppe.com,https://.dynamics-servicestie.com",
Vennlig hilsen / Kind regards
Dag Fr. Salvesen
Løsningsarkitekt og Salg/Marked
M: +47 900 99 399
Fra: Freddy Kristiansen<EMAIL_ADDRESS>Sendt: torsdag 6. desember 2018 14:44
Til: Microsoft/nav-docker<EMAIL_ADDRESS>Kopi: Dag Salvesen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Dag - could you open ISE, run
Enter-NavContainer navserver
cd \inetpub\wwwroot\nav
psedit navsettings.json
add https://*.office365.com to AllowedFrameAncestors
Exit the pssession
use Restart-NavContainer and remove cookies etc. from your browser
test again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nav-docker/issues/275#issuecomment-444874897, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ap4CN3HCKfteyvTpN6AZAHOGJoWau0kNks5u2R81gaJpZM4XZDJg.

Looks correct – and you did restart the container?
This fixed it on my VM – I don’t get the token or session expired anymore.
/Freddy
From: DagPilaro<EMAIL_ADDRESS>Sent: Thursday, December 6, 2018 3:07 PM
To: Microsoft/nav-docker<EMAIL_ADDRESS>Cc: Freddy Kristiansen<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Hi,
Did but still the same issue ☹
This is how the string now are:
"AllowedFrameAncestors": "https://.office365.com,https://.dynamics-tie.com,https://.dynamics.com,https://.sharepoint.com,https://.office.com,https://.officeppe.com,https://.dynamics-ppe.com,https://.dynamics-servicestie.comhttps://*.office365.com,https:/*.dynamics-tie.com,https:/*.dynamics.com,https:/*.sharepoint.com,https:/*.office.com,https:/*.officeppe.com,https:/*.dynamics-ppe.com,https:/*.dynamics-servicestie.com",
Vennlig hilsen / Kind regards
Dag Fr. Salvesen
Løsningsarkitekt og Salg/Marked
M: +47 900 99 399
Fra: Freddy Kristiansen <notifications<EMAIL_ADDRESS>Sendt: torsdag 6. desember 2018 14:44
Til: Microsoft/nav-docker <nav-docker<EMAIL_ADDRESS>Kopi: Dag Salvesen <dag.salvesen<EMAIL_ADDRESS>Author <author<EMAIL_ADDRESS>Emne: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Dag - could you open ISE, run
Enter-NavContainer navserver
cd \inetpub\wwwroot\nav
psedit navsettings.json
add https://*.office365.com to AllowedFrameAncestors
Exit the pssession
use Restart-NavContainer and remove cookies etc. from your browser
test again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nav-docker/issues/275#issuecomment-444874897, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ap4CN3HCKfteyvTpN6AZAHOGJoWau0kNks5u2R81gaJpZM4XZDJg.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fnav-docker%2Fissues%2F275%23issuecomment-444881674&data=02|01|freddy.kristiansen%40microsoft.com|4ef72a0214744ea7726708d65b841092|72f988bf86f141af91ab2d7cd011db47|1|0|636797020099373582&sdata=h4jZb%2Fg2GAfQnuiEbyTtOuWYWoMjHhLD3dXfF7LWNc0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKRqAwOd_INUCzsPp60FQUpTgGhBBfWEks5u2SR4gaJpZM4XZDJg&data=02|01|freddy.kristiansen%40microsoft.com|4ef72a0214744ea7726708d65b841092|72f988bf86f141af91ab2d7cd011db47|1|0|636797020099383573&sdata=50b0NE1UODD7M5CiWrkc2g8bBxByNuU%2FvujVt3Nl2OQ%3D&reserved=0.

I actually also added it to hosting.json in the same folder:
{
"urls": "http://:48900,https://.office365.com"
}
/Freddy
From: DagPilaro<EMAIL_ADDRESS>Sent: Thursday, December 6, 2018 3:07 PM
To: Microsoft/nav-docker<EMAIL_ADDRESS>Cc: Freddy Kristiansen<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Hi,
Did but still the same issue ☹
This is how the string now are:
"AllowedFrameAncestors": "https://.office365.com,https://.dynamics-tie.com,https://.dynamics.com,https://.sharepoint.com,https://.office.com,https://.officeppe.com,https://.dynamics-ppe.com,https://.dynamics-servicestie.comhttps://*.office365.com,https:/*.dynamics-tie.com,https:/*.dynamics.com,https:/*.sharepoint.com,https:/*.office.com,https:/*.officeppe.com,https:/*.dynamics-ppe.com,https:/*.dynamics-servicestie.com",
Vennlig hilsen / Kind regards
Dag Fr. Salvesen
Løsningsarkitekt og Salg/Marked
M: +47 900 99 399
Fra: Freddy Kristiansen <notifications<EMAIL_ADDRESS>Sendt: torsdag 6. desember 2018 14:44
Til: Microsoft/nav-docker <nav-docker<EMAIL_ADDRESS>Kopi: Dag Salvesen <dag.salvesen<EMAIL_ADDRESS>Author <author<EMAIL_ADDRESS>Emne: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Dag - could you open ISE, run
Enter-NavContainer navserver
cd \inetpub\wwwroot\nav
psedit navsettings.json
add https://*.office365.com to AllowedFrameAncestors
Exit the pssession
use Restart-NavContainer and remove cookies etc. from your browser
test again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nav-docker/issues/275#issuecomment-444874897, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ap4CN3HCKfteyvTpN6AZAHOGJoWau0kNks5u2R81gaJpZM4XZDJg.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fnav-docker%2Fissues%2F275%23issuecomment-444881674&data=02|01|freddy.kristiansen%40microsoft.com|4ef72a0214744ea7726708d65b841092|72f988bf86f141af91ab2d7cd011db47|1|0|636797020099373582&sdata=h4jZb%2Fg2GAfQnuiEbyTtOuWYWoMjHhLD3dXfF7LWNc0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKRqAwOd_INUCzsPp60FQUpTgGhBBfWEks5u2SR4gaJpZM4XZDJg&data=02|01|freddy.kristiansen%40microsoft.com|4ef72a0214744ea7726708d65b841092|72f988bf86f141af91ab2d7cd011db47|1|0|636797020099383573&sdata=50b0NE1UODD7M5CiWrkc2g8bBxByNuU%2FvujVt3Nl2OQ%3D&reserved=0.

Hi,
Did that too… but still issues. Have cleaned «all» including everything in «crediential managaer»
First time i log on (using IE) it opens the Cantact list (shold have been the question – do you want to….)
After some minutes – the session exp. Appears.
Not possible to run Chrome…
Ill send you the rdp to the vm.
Vennlig hilsen / Kind regards
Dag Fr. Salvesen
Løsningsarkitekt og Salg/Marked
M: +47 900 99 399
Fra: Freddy Kristiansen<EMAIL_ADDRESS>Sendt: torsdag 6. desember 2018 15:18
Til: Microsoft/nav-docker<EMAIL_ADDRESS>Kopi: Dag Salvesen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
I actually also added it to hosting.json in the same folder:
{
"urls": "http://:48900,https://.office365.comhttp://*:48900,https:/*.office365.com"
}
/Freddy
From: DagPilaro <notifications<EMAIL_ADDRESS>Sent: Thursday, December 6, 2018 3:07 PM
To: Microsoft/nav-docker <nav-docker<EMAIL_ADDRESS>Cc: Freddy Kristiansen <Freddy.Kristiansen<EMAIL_ADDRESS>Comment <comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Hi,
Did but still the same issue ☹
This is how the string now are:
"AllowedFrameAncestors": "https://.office365.com,https://.dynamics-tie.com,https://.dynamics.com,https://.sharepoint.com,https://.office.com,https://.officeppe.com,https://.dynamics-ppe.com,https://.dynamics-servicestie.comhttps://*.office365.com,https:/*.dynamics-tie.com,https:/*.dynamics.com,https:/*.sharepoint.com,https:/*.office.com,https:/*.officeppe.com,https:/*.dynamics-ppe.com,https:/*.dynamics-servicestie.comhttps://*.office365.com,https:/*.dynamics-tie.com,https:/*.dynamics.com,https:/*.sharepoint.com,https:/*.office.com,https:/*.officeppe.com,https:/*.dynamics-ppe.com,https:/*.dynamics-servicestie.com<https:/*.office365.com,https:/*.dynamics-tie.com,https:/*.dynamics.com,https:/*.sharepoint.com,https:/*.office.com,https:/*.officeppe.com,https:/*.dynamics-ppe.com,https:/*.dynamics-servicestie.com>",
Vennlig hilsen / Kind regards
Dag Fr. Salvesen
Løsningsarkitekt og Salg/Marked
M: +47 900 99 399
Fra: Freddy Kristiansen <notifications@github.com<mailto:notifications@github.commailto:notifications<EMAIL_ADDRESS>Sendt: torsdag 6. desember 2018 14:44
Til: Microsoft/nav-docker <nav-docker@noreply.github.com<mailto:nav-docker@noreply.github.commailto:nav-docker<EMAIL_ADDRESS>Kopi: Dag Salvesen <dag.salvesen@pilaro.no<mailto:dag.salvesen@pilaro.nomailto:dag.salvesen<EMAIL_ADDRESS>Author <author@noreply.github.com<mailto:author@noreply.github.commailto:author<EMAIL_ADDRESS>Emne: Re: [Microsoft/nav-docker] BC-App in Outlook has several issues. BC on prem NO, W1, Docker issue? (#275)
Dag - could you open ISE, run
Enter-NavContainer navserver
cd \inetpub\wwwroot\nav
psedit navsettings.json
add https://*.office365.com to AllowedFrameAncestors
Exit the pssession
use Restart-NavContainer and remove cookies etc. from your browser
test again
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nav-docker/issues/275#issuecomment-444874897, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ap4CN3HCKfteyvTpN6AZAHOGJoWau0kNks5u2R81gaJpZM4XZDJg.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fnav-docker%2Fissues%2F275%23issuecomment-444881674&data=02|01|freddy.kristiansen%40microsoft.com|4ef72a0214744ea7726708d65b841092|72f988bf86f141af91ab2d7cd011db47|1|0|636797020099373582&sdata=h4jZb%2Fg2GAfQnuiEbyTtOuWYWoMjHhLD3dXfF7LWNc0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKRqAwOd_INUCzsPp60FQUpTgGhBBfWEks5u2SR4gaJpZM4XZDJg&data=02|01|freddy.kristiansen%40microsoft.com|4ef72a0214744ea7726708d65b841092|72f988bf86f141af91ab2d7cd011db47|1|0|636797020099383573&sdata=50b0NE1UODD7M5CiWrkc2g8bBxByNuU%2FvujVt3Nl2OQ%3D&reserved=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nav-docker/issues/275#issuecomment-444885193, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ap4CN5O2mvv0fz857JyHXe14HIZ_4S1hks5u2ScpgaJpZM4XZDJg.


chore(master): release 0.40.0

:robot: I have created a release beep boop
0.40.0 (2024-09-28)
Features

implement entitymatching (dfadb4c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.40.0
:sunflower:


Various location fixes

Optional locations now store NULL in their start and end to indicate that they were not provided.
When they are serialized, they always serialize a single byte to indicate whether or not they are present.
The deserializers now always read at least one byte to determine if they are present, and then read the location if it's non-zero.

Fixes #589.

As discussed on Slack this is also an issue for optional nodes. And maybe for optional tokens (not if EOF Token can never be the value of an optional token field, but can we guarantee that)?.

Yeah may as well do tokens as well. https://github.com/Shopify/yarp/issues/665


Invalid synchronous instrument name with spaces

Create a synchronous instrument with a string containing spaces as input name will give an error in the cpp-opentelemetry end.
For example,
counter = meter.createCounter("my counter");
A temporary fix has been applied to replace spaces with underscores. But this issue may be further discussed.

We need a more general solution. Spaces are not the only character disallowed. See sdk/src/metrics/instrument_metadata_validator.cc For names, it is only allowing letters, numbers, underscore, minus, dot, slash. All other symbols are disallowed. Units also have its own rules.
Consider something simple such as if a name fails, just use an empty name

The current behavior is if instrument name invalid, the instrument will still be created with the invalid name. But no values will be recorded and the instrument becomes like a no-op instrument.
Also, the naming rules is part of the OpenTelemetry spec

There is also this "error" message: [Error] File: otel-cpp\src\opentelemetry-cpp\sdk\src\metrics\meter.cc:67 Meter::CreateDoubleCounter - failed. Invalid parameters.__5352153afdsaf  . Measurements won't be recorded.
This seems to me perfectly reasonable. Although it's an error message, it doesn't stop anything, so is effectively a warning. And it exactly describe the no-op behavior.
I think it is reasonable to just stick to the C++ behavior and not do anything extra, which means we should back out the change to convert empty spaces to underscore. Any invalid names will get the warning message and becomes a no-op instrument.

Backed out the previous change to replace spaces with underscores.


auctioneer uses hardcoded metron port

The auctioneer hardcodes the metron port to be 3457
The metron port can be configured in CloudFoundry to be something other than 3457, so a change to the metron port will break auctioneer metrics.

Hi, @roxtar,
The Diego team will address this across the release in https://www.pivotaltracker.com/story/show/103689460, so I'l close this issue out in favor of that.
Thanks,
Eric


Feature: add warnings

adds warnings to usePost, usePatch
references issue: https://github.com/bornfight/aardvark/issues/27

Codecov Report

Merging #32 into master will decrease coverage by 0.07%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   85.41%   85.33%   -0.08%
==========================================
  Files          49       49
  Lines        1049     1064      +15
  Branches      161      168       +7
==========================================
+ Hits          896      908      +12
- Misses        151      154       +3
  Partials        2        2

Impacted Files	Coverage Δ
src/hooks/usePatch.ts 88.23% <77.77%> (-3.77%)	⬇
src/hooks/usePost.ts 89.65% <83.33%> (-1.65%)	⬇
Continue to review full report at Codecov.

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

@JuraJuki I added codecov action to the CI pipeline. Now, after merging master into this branch we got a drop in code coverage. You can see the detailed report here (look for the red squares).
I suggest we increase the coverage for this particular feature and then we merge?


Add processEvent to ConsoleConnector

At the moment the only events that can be processed with the
ConsoleConnector are messages with very restricted configuration. By
adding a processEvent method any event can be tested including messages
with custom sourceEvent setups for real client dependent (Slack, MS
Teams, etc) tests.

@bilby91,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@bilby91, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Seems reasonable...

Are releases scheduled? When can we expect a new version? Thanks!
Martín Fernández
On Wed, Jul 05, 2017 at 17:44 Steven Ickman
<
mailto:Steven Ickman<EMAIL_ADDRESS>
wrote:

a, pre, code, a:link, body { word-wrap: break-word !important; }
Merged
https://github.com/Microsoft/BotBuilder/pull/3074
.
—
You are receiving this because you were mentioned.
Reply to this email directly,
https://github.com/Microsoft/BotBuilder/pull/3074#event-1151645349
, or
https://github.com/notifications/unsubscribe-auth/ACGV9-Mai-q0bBBEzbV6Y0fH-VkWbdgUks5sK_WQgaJpZM4OLzS0
.


Update install instructions to point to community operatorhub releases.

Warning on building your own OLM bundle (nobody uses this afaik, but I don't want to remove the docs on how to do it just incase)
/assign @gregsheremeta

/lgtm

/lgtm


darwin.stdenv: use minimal python to bootstrap clang

Motivation for this change
By using a minimal version of python in the stdenv we can remove a bunch of dependencies like db, readline and sqlite.
I'm testing a branch WIP with #21078, #21099 and #21101.
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

+bootstrap-python-2.7.12.drv
+bzip2-<IP_ADDRESS>.1.drv
-bzip2-<IP_ADDRESS>.1.drv
-clang-5.3.patch
+cmake-3.6.2.drv
-cmake-3.6.2.drv
-curl-7.51.0.drv
-curl-7.51.0.tar.bz2.drv
-db-5.3.28.drv
-db-5.3.28.tar.gz.drv
-e0ec3471cb09.drv
-gdbm-1.12.drv
-gdbm-1.12.drv
-gdbm-1.12.tar.gz.drv
-hook.drv
+libarchive-3.2.2.drv
-libarchive-3.2.2.drv
-libssh2-1.7.0.drv
-libssh2-1.7.0.tar.gz.drv
-libxml2-2.9.4.drv
-link-against-ncurses.patch
+lzo-2.09.drv
-lzo-2.09.drv
-no-arch_only-6.3.patch
-openssl-1.0.2j.drv
-openssl-1.0.2j.drv
-openssl-1.0.2j.tar.gz.drv
+pkg-config-0.29.drv
-pkg-config-0.29.drv
-properly-detect-curses.patch
-python-2.7-distutils-C++.patch
-python-2.7.12.drv
-python-2.7.12.drv
-readline-6.3.tar.gz.drv
-readline-6.3p08.drv
-readline-6.3p08.drv
-readline63-001.drv
-readline63-002.drv
-readline63-003.drv
-readline63-004.drv
-readline63-005.drv
-readline63-006.drv
-readline63-007.drv
-readline63-008.drv
-setup-hook.sh
-sqlite-3.15.0.drv
-sqlite-autoconf-3150000.tar.gz.drv
-use-etc-ssl-certs.patch

Why do we want this? Are those dependencies a problem? Is it for the closure size? While it is possible with this change to reduce the closure size, it will typically require more building.
Earlier we got rid of the 'small' Python with the modules because it was causing a lot of confusion. It will need to be clear then that this derivation should typically not be used or we end up with the previous scenario.
Is it necessary to copy the 2.7 expression?

The main reason why I would like this is to reduce the set of packages that only cause rebuilds on darwin.

Can we name it "pythonMinimal" instead of "pythonBoot"? "pythonMinimal" seems more descriptive.

It's not really just a minimal python, things might not work if you do more then run a python script that uses the standard library. Might be better not to expose it in all-packages.nix
I'm working on some other changes that, together with this change, will get rid of curl and openssl

The cmake changes look pretty good, that will make this a bit simpler.

Closing in favor of #21596


Operator precedence climbing parser

Consider what peg (https://docs.rs/peg/latest/peg/#precedence-climbing) was doing, I think it is suitable for us to implement precedence climbing for easier design of expression grammar

From my understanding, precedence climbing is a way to fix the inefficiency of pure recursive-descent parsers for grammars that describe expressions with operators. This incidentally allows writing simpler grammars rules.
Hime use bottom-up parsing algorithms (LR and GLR) that do not suffer from this inefficiency. So precedence climbing is not really required per-se. It is true that we still have to write grammars that explicitly encode the operators precedence, as shown on Wikipedia.
My conclusion is then that the support for precedence climbing would then be only a way to alleviate the tediousness of writing grammar rules for this case, as shown in the peg crate doc. Writing grammar rules this way for expressions leads to ambiguous grammars for LR parsing techniques.
LR parsing techniques are very different from PEG; the implementation of precedence climbing would be very different. I'll have to do some research. Maybe we can have some higher-level syntax for grammar rules with operator precedence that translates to usual grammar rules that can be used by LR techniques.

https://inst.eecs.berkeley.edu/~cs164/fa20/lectures/lecture8.pdf
but it seems like bison have it...


[7.x] Only return strings with __ helper

This PR modifies the __ helper to only return strings. This allows for it to be safely used, even if it collides with translation files. The trans helper remains unaffected.

I think I will just avoid changing this for now since it doesn't fix the originally reported issue of translating reserved words like "validation", etc. and could actually lead to confusion when they get a string and don't understand why it isn't translating.

I've been thinking about this problem with the __()-helper returning an array when the key accidentally is also an existing php language file. I think it's worth revisiting this PR!

I think I will just avoid changing this for now since it doesn't fix the originally reported issue of translating reserved words like "validation", etc. and could actually lead to confusion when they get a string and don't understand why it isn't translating.

...but it seems the json-translations is actually checked for matches first, before looking in the php language-files here:
https://github.com/laravel/framework/blob/dc2721a8460db7d06f381dbe22d6da90187fa89e/src/Illuminate/Translation/Translator.php#L113
So, when the string is present in the json file, there has never been a problem! An early match is found and that translation (string) is returned. So there was never a conflict with "reserved" words not being translatable!
However, when the string is not in the json file and there is also a php-file named like the string, only then do we get the array back and the template breaks.
In this case, returning the original (however untranslated) input from __() is just what we want, and this PR does just that!

We probably won't revisit this any time soon sorry.

PS: I quit my (mostly) Node.js + React job some days ago, so now I'm back in the Laravel ecosystem that I love! That's why you see me commenting on issues and PRs all over the place today 😄 bear with me!

\o/ 😄


Getting crash on long length video.

Apps getting crash while recording long length session. Its happen while your session is approx. more then 30 min  or more.

Which device did you use? I streamed for more than 40 hours without interruptions/errors with an iPad 2.

its happen in iPhone 6s.

Have you checked all segment upload perfect? Please see index(.m3u8) file for that.

45 minutes recording on iPad mini 2.
index.m3u8: http://pastebin.com/ELfzTivX
vod.m3u8:    http://pastebin.com/g4439K5C

Have you checked #39 ? Maybe it's somehow related. I don't have a 6s to test yet.

Was this ever solved? Having an issue with long videos being uploaded on Android. Anything other ~30 mins can't be viewed after the live stream. Not in the app and the video is missing altogether from the Web link.

@dyoungjr08 This is iOS not Android.
Do you have errors in logcat? I found a solution for ~30 mins limit in Android, you can find the issue here [(https://github.com/Kickflip/kickflip-android-sdk/issues/35#issuecomment-149854977)]

Thank you @Arlem . I read that solution but We're unable to get it to run. Is there anyway you can share the updated and compiled .so files? Thank you.


virtcontainers: Use only supported namespaces

Depending of the linux versions some namespaces are supported or not.
To avoid failures running containers and unit tests in old linux kernels,
only the supported namespaces should be used.
Depends-on: github.com/kata-containers/tests#1414
fixes #228
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

/test

@devimc , I see that centos did not run the unit tests

@devimc , I enabled sles to run this PR and verify your commit is working

/test-sles

/test-sles

/test-sles

/test

/retest

/retest


Long Processing Time

Hi,
More a query than an issue. Any idea why a sample could be hanged at the analyzer step (--preset hla) for more than 24hrs with -t 30. I have only witness it with some fastqs and I have yet to further investigate the files themselves but I can say there is no significance in # of reads or size of the file.
Thanks for any possible insights.

The analyzer step will call the SNPs, which involves enumerating all the alleles to which the multiple-aligned reads in that region are aligned. I guess in your case, region happens to involve too many alleles and will take a long time to complete. If you can share the _aligned and *allele.tsv files with me, I can take a look at them and improve the time efficiency later.
If you only need the genotyping information, you can run T1K with the option "--skipPostAnaysis" to just conduct genotype. In your case, since the genotyping has already finished, you can safely kill the job and directly use the genoptying results in the *_genotype.tsv file.

Ah okay! Thanks so much, I'll see if i can provide you with the _aligned & alle.tsv.

Also you should correct in your parameters description on the github, "--skipPostAnaysis" to "--skipPostAnalysis"

Nice catch! It is fixed now. Thank you.


[BUG] - Build Failing Everytime

Describe the bug
Trying default Splunk Security COntent to build app, after validation it fails everytime.
Expected behavior
Expecting a app generated that can be imported to SPlunk.
Screenshots
~/de# python contentctl.py -p . build -o build1/escu -pr ESCU
Running Splunk Security Content Control Tool (contentctl)
starting program loaded for TIE Fighter...
 T~T[_.-Y  / \  / \  Y-._]T~T _[_].|| |     [|-+[  ___]| [__  |-=[--()--]=-|  __] |[___  ]+-|]      | ]|"[_]  l_j[_"-l  \ /  \ /  !-"_]l_j  [_]~|| |      | |-' "~"---.,"\  "o--o"  /"/,.---"~" `-'| |
| |             ~~"^-.___.-^"~~             | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
l_i                                          l_j -Row
Building Splunk App from source ./dist/escu
Copying Splunk App Source to ./dist/escu in preparation for building...done
Validating Splunk App...
/usr/local/lib/node_modules/slim/index.js:25
const file = _file.endsWith('.graphml') ? _file : ${file}.graphml;
^
ReferenceError: Cannot access 'file' before initialization
at Command.compileYedFile (/usr/local/lib/node_modules/slim/index.js:25:58)
at Command.listener [as _actionHandler] (/usr/local/lib/node_modules/slim/node_modules/commander/index.js:413:31)
at Command._parseCommand (/usr/local/lib/node_modules/slim/node_modules/commander/index.js:914:14)
at Command.parse (/usr/local/lib/node_modules/slim/node_modules/commander/index.js:717:10)
at slim (/usr/local/lib/node_modules/slim/index.js:18:11)
at Object. (/usr/local/lib/node_modules/slim/sli:3:19)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
error: Command '['slim', 'validate', 'build1/escu/DA-ESS-ContentUpdate']' returned non-zero exit status 1.
Traceback (most recent call last):
File "/root/de/bin/contentctl_project/contentctl_core/application/use_cases/build.py", line 60, in validate_splunk_app
nothing = subprocess.check_output(["slim", "validate", self.output_dir_source])
File "/usr/lib/python3.10/subprocess.py", line 421, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "/usr/lib/python3.10/subprocess.py", line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['slim', 'validate', 'build1/escu/DA-ESS-ContentUpdate']' returned non-zero exit status 1.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/root/de/contentctl.py", line 461, in
main(sys.argv[1:])
File "/root/de/contentctl.py", line 455, in main
return args.func(args)
File "/root/de/contentctl.py", line 302, in build
Build(args)
File "/root/de/bin/contentctl_project/contentctl_core/application/use_cases/build.py", line 31, in init
self.validate_splunk_app()
File "/root/de/bin/contentctl_project/contentctl_core/application/use_cases/build.py", line 65, in validate_splunk_app
raise(Exception(f"Error building Splunk App: {str(e)}"))
Exception: Error building Splunk App: Command '['slim', 'validate', 'build1/escu/DA-ESS-ContentUpdate']' returned non-zero exit status 1.
App Version:
latest

(de) root@ip-172-31-1-19:~/de# python contentctl.py -p . validate -pr ESCU
Running Splunk Security Content Control Tool (contentctl)
starting program loaded for TIE Fighter...
  This may take some time...
Client: [100%]...Done!
Enterprise: [100%]...Done!
Relationships: [100%]...Done!
Groups: [100%]...Done!
MITRE Technique Progress: [100%]...Done!
Creating Security Content - ESCU. This may take some time...
Lookups Progress: [100%]...Done!
Macros Progress: [100%]...Done!
Deployments Progress: [100%]...Done!
Baselines Progress: [100%]...Done!
Investigations Progress: [100%]...Done!
Playbooks Progress: [100%]...Done!
Detections Progress: [100%]...Done!
Stories Progress: [100%]...Done!
Validation of security content successful.
(de) root@ip-172-31-1-19:~/de# python contentctl.py -p . generate -o dist/escu -pr ESCU
Running Splunk Security Content Control Tool (contentctl)
starting program loaded for TIE Fighter...
  This may take some time...
Client: [100%]...Done!
Enterprise: [100%]...Done!
Relationships: [100%]...Done!
Groups: [100%]...Done!
MITRE Technique Progress: [100%]...Done!
Creating Security Content - ESCU. This may take some time...
Lookups Progress: [100%]...Done!
Macros Progress: [100%]...Done!
Deployments Progress: [100%]...Done!
Baselines Progress: [100%]...Done!
Investigations Progress: [100%]...Done!
Playbooks Progress: [100%]...Done!
Detections Progress: [100%]...Done!
Stories Progress: [100%]...Done!
Generate of security content successful.
(de) root@ip-172-31-1-19:~/de# python contentctl.py -p . build -o build1/escu -pr ESCU
Running Splunk Security Content Control Tool (contentctl)
starting program loaded for TIE Fighter...
 T~T[_.-Y  / \  / \  Y-._]T~T _[_].|| |     [|-+[  ___]| [__  |-=[--()--]=-|  __] |[___  ]+-|]      | ]|"[_]  l_j[_"-l  \ /  \ /  !-"_]l_j  [_]~|| |      | |-' "~"---.,"\  "o--o"  /"/,.---"~" `-'| |
| |             ~~"^-.___.-^"~~             | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
| |                                          | |
l_i                                          l_j -Row
Building Splunk App from source ./dist/escu
Copying Splunk App Source to ./dist/escu in preparation for building...done
Validating Splunk App...
/usr/local/lib/node_modules/slim/index.js:25
const file = _file.endsWith('.graphml') ? _file : ${file}.graphml;
^
ReferenceError: Cannot access 'file' before initialization
at Command.compileYedFile (/usr/local/lib/node_modules/slim/index.js:25:58)
at Command.listener [as _actionHandler] (/usr/local/lib/node_modules/slim/node_modules/commander/index.js:413:31)
at Command._parseCommand (/usr/local/lib/node_modules/slim/node_modules/commander/index.js:914:14)
at Command.parse (/usr/local/lib/node_modules/slim/node_modules/commander/index.js:717:10)
at slim (/usr/local/lib/node_modules/slim/index.js:18:11)
at Object. (/usr/local/lib/node_modules/slim/sli:3:19)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
error: Command '['slim', 'validate', 'build1/escu/DA-ESS-ContentUpdate']' returned non-zero exit status 1.
Traceback (most recent call last):
File "/root/de/bin/contentctl_project/contentctl_core/application/use_cases/build.py", line 60, in validate_splunk_app
nothing = subprocess.check_output(["slim", "validate", self.output_dir_source])
File "/usr/lib/python3.10/subprocess.py", line 421, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "/usr/lib/python3.10/subprocess.py", line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['slim', 'validate', 'build1/escu/DA-ESS-ContentUpdate']' returned non-zero exit status 1.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/root/de/contentctl.py", line 461, in
main(sys.argv[1:])
File "/root/de/contentctl.py", line 455, in main
return args.func(args)
File "/root/de/contentctl.py", line 302, in build
Build(args)
File "/root/de/bin/contentctl_project/contentctl_core/application/use_cases/build.py", line 31, in init
self.validate_splunk_app()
File "/root/de/bin/contentctl_project/contentctl_core/application/use_cases/build.py", line 65, in validate_splunk_app
raise(Exception(f"Error building Splunk App: {str(e)}"))
Exception: Error building Splunk App: Command '['slim', 'validate', 'build1/escu/DA-ESS-ContentUpdate']' returned non-zero exit status 1.

Hey @abhinavkakku, what is it that you're trying to accomplish? If you take a look at the CI pipeline used for building ESCU (.github/workflows/build-and-validate.yml), the build command isn't used. I don't believe it was finished in this version of contentctl.
Depending on what you're trying to do, you may want to look into using the new dedicated contentctl codebase

@ljstella  sure I will try that new contentctl codebase.
I am only trying to use this Security Content as documented, and failing to build app that can be deployed to Splunk.
Possibly I need to understand more about this whole thing.


I do not do twitter.

"I'll be publishing a few articles on this topic. Follow me on twitter if you are interested and want to be notified when they are out."
I'm interested in any articles you have on this topic.
thanks for your work.

Hello! The new Blazor 3.2.0 preview 2 adds support for token based authentication
https://devblogs.microsoft.com/aspnet/blazor-webassembly-3-2-0-preview-2-release-now-available/
Therefore I think there's very little point in updating this project :)
Thanks a lot for having used it!


Ubuntu 16.04. compatibility

Found two issues when setting this up for Ubuntu 16.04.
The packed client dev package is called libmariadb-client-lgpl-dev.
It should be possible to modify the hard-coded libmariadbclient-dev through configuration.
Ubuntu 16.04. comes with native mariadb 10.0 packages.
It will install but there are warnings from the cookbook stating the OS does not have the packages.

Hi goatherd !
Is it possible for you, to prepare a PR for that ? I'm really out of bandwith to work on your issue for now... And i know it can take months before i found time to work on it !
Thx in advance :)

Hi nicolas,
only dumping notices for now testing a new deployment. Having those notes often helps me determine issues faster so maybe someone else will profit from it until a fix is demanded/ provided.
As there still are several weird issues with Ubuntu 16.04. and some discussion with the MySQL 5.7 incompatibility it is not to certain that I will rely on the cookbook for now. If the team chooses MariaDB I'll provide a pull request afterwards.
Somewhat short on time as well.

Also trying to converge on Ubuntu 16.04 and unfortunately stucked with a repository key that is apparently missing. Without that key, packages are not installed. But the following command allow to continue:
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys  F1656F24C74CD1D8

The thing is I don't understand where this key is coming from. I thought the mariadb repository key was CBCB082A1BB943DB ?
@goatherd did you not encounter such error ?

@PowerKiKi the only issue with the keyserver I know of is proxy-related: some old VM installs rejected the keyserver request. Not sure that was a host routing issue or port forwarding related.
Never had any such issues with server deployments nor with MariaDB.
Not being able to update or install from the base repository is rather weird. Are you sure the network is working and unrestricted?
Also unrelated to the issue.

I didn't really have much of a problem with this cookbook on 16.04 but I ran into the issue with 10.0 being shipped; just made my wrapper call the repository to be added and .. it worked fairly painlessly

Thank you all for your feedback and sorry for hijacking this thread. I'll give another try in the coming weeks and open a separate issue if I can better pinpoint what's going on.

@PowerKiKi no problem I had to modify the cookbook to handle the different apt key.  We seem to be using a hardcoded key :(
diff --git a/cookbooks/mariadb/attributes/apt.rb b/cookbooks/mariadb/attributes/apt.rb
new file mode 100644
index 0000000..2b85044
--- /dev/null
+++ b/cookbooks/mariadb/attributes/apt.rb
@@ -0,0 +1,3 @@
+node.default['mariadb']['apt']['keyserver']="hkp://keyserver.ubuntu.com:80"
+node.default['mariadb']['apt']['key']="F1656F24C74CD1D8"
+
diff --git a/cookbooks/mariadb/recipes/repository.rb b/cookbooks/mariadb/recipes/repository.rb
index e8081eb..25ffe4b 100644
--- a/cookbooks/mariadb/recipes/repository.rb
+++ b/cookbooks/mariadb/recipes/repository.rb
@@ -15,8 +15,8 @@ if node['mariadb']['use_default_repository']
         node['mariadb']['install']['version'] + '/' + node['platform']
       distribution node['lsb']['codename']
       components ['main']
-      keyserver 'hkp://keyserver.ubuntu.com:80'
-      key '0xcbcb082a1bb943db'
+      keyserver node['mariadb']['apt']['keyserver']
+      key node['mariadb']['apt']['key']
     end
   when 'yum'
     include_recipe 'yum::default'

Doesn't look like the above made it into a PR. May I?

@joraff if it's not already done feel free.

Can confirm that I cannot install on Ubuntu 16.04 without first running this command in the bento vagrant box:
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys  F1656F24C74CD1D8

Any progress? I'm relying on this cookbook transitively through the boulder server for letsencrypt integration testing.

I had to mess around with keys to get an installation happening, was having issues with this cookbook as a transitive dependency. I think the winner was this one:
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys C74CD1D8

but I haven't confirmed yet.

I never could pinpoint exactly what was going on since last September, but my solution also involved using the key F1656F24C74CD1D8 for a manual install via apt_repository resource. This was the easiest I could come up with without messing too much with this cookbook. In the end that seems to be the same as @liamdawson.

The key changes based on what distro you run.  :( If that helps


Update TX committees

@NewAgeAirbender  Check this out if, it is missing anything.

So far I have two notes:

Committees that don't exist any longer can be deleted instead of moved to retired
Is there a way to strip the jurisdiction & chamber from the Committee name? Most jurisdictions don't list either in the name & will lead to issues matching committees to actions on Plural's end but also issues matching them to Events/Votes/Sponsorship on OS if they're not similar. I think we can discuss & plan to edit or update either side

So far I have three notes:

Committees that don't exist any longer can be deleted instead of moved to retired
Is there a way to strip the jurisdiction & chamber from the Committee name? Most jurisdictions don't list either in the name & will lead to issues matching committees to actions on Plural's end but also issues matching them to Events/Votes/Sponsorship on OS if they're not similar. Could add it as an other_name to the yaml file if it can't be edited. I think we can discuss & plan to edit or update either side.
If a member exists on a committee without being a legislator, we just add them with the role: member and don't set the person_id at all so doesn't need to add a null value.

So far I have three notes:

Committees that don't exist any longer can be deleted instead of moved to retired
Is there a way to strip the jurisdiction & chamber from the Committee name? Most jurisdictions don't list either in the name & will lead to issues matching committees to actions on Plural's end but also issues matching them to Events/Votes/Sponsorship on OS if they're not similar. Could add it as an other_name to the yaml file if it can't be edited. I think we can discuss & plan to edit or update either side.
If a member exists on a committee without being a legislator, we just add them with the role: member and don't set the person_id at all so doesn't need to add a null value.

So far I have three notes:

Committees that don't exist any longer can be deleted instead of moved to retired
Is there a way to strip the jurisdiction & chamber from the Committee name? Most jurisdictions don't list either in the name & will lead to issues matching committees to actions on Plural's end but also issues matching them to Events/Votes/Sponsorship on OS if they're not similar. Could add it as an other_name to the yaml file if it can't be edited. I think we can discuss & plan to edit or update either side.
If a member exists on a committee without being a legislator, we just add them with the role: member and don't set the person_id at all so doesn't need to add a null value.

Made these changes, please take a look if they make sense.


Add support for node suspension due to runtime liveness failures

TODO

[x] Basic structures for tracking node liveness faults.
[x] Update the scheduler service to ignore suspended nodes when electing committees.
[x] Update the roothash service to track node liveness failure in the previous epoch.
[x] Support for slashing.
[x] Tests.

My thinking was to leave it up to a runtime descriptor update.


feat: define CRD RAG under group evaluations

What type of PR is this?

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #507
Special notes for your reviewer
Changelogs:

move arcadia base CRD's controllers under controllers/base
add new group evaluations
add new CRD RAG under group evaluations  to evaluate arcadia Application/Agent

@Abirdcfly @0xff-dev any comments if I move controllers under controllers/base/ ?

I guess also need update apiserver/pkg/common/schema.go:122 init function, but apiserver do not use it.So we can add this to next pr.

Good thought. This is only a definition which may change.


⚠ Invidious has degraded performance

In 26666b0, Invidious (https://invidious.perennialte.ch) experienced degraded performance:

HTTP code: 200
Response time: 6665 ms

Resolved: Invidious performance has improved in 5379c4b after 7 minutes.


Add choco package to README

just package for Chocolatey. Package definition
Command: choco install just

Very nice, thank you!