Simulation with ResNet fails

During simulation with ResNet, a segmentation fault occurs at gem5.
I created ResNet pb and pbtxt file by running smaug/experiments/models/imagenet-resnet/resnet_network.py
All configuration files are the same with minerva example, but only model_files was modfied so that it points to generated pb and pbtxt file.
Input trace was generated by running trace.sh
Below is the stdout log at the end.
Scheduling data (Data).
Scheduling data_1 (Data).
Scheduling data_10 (Data).
Scheduling data_100 (Data).
Scheduling data_101 (Data).
Scheduling data_102 (Data).
Scheduling data_103 (Data).
Scheduling data_104 (Data).
Scheduling data_105 (Data).
Scheduling data_106 (Data).
Scheduling data_107 (Data).
Scheduling data_108 (Data).
Scheduling data_109 (Data).
stderr log before the backtrace shows the following message.
gem5 has encountered a segmentation fault!
Please, let me know if I configured something wrong.
Thanks.

Yuan, can you take a look at this?

Yes, will take a look at this week.


Refactoring: remove the validated tree?

I am not convinced the validated tree carries its weight for dada (I am still mildly convinced that HIR carries its weight for Rust, but only mildly). I'd be curious to see someone (maybe me, if I get to it...) try removing it entirely, which would mean:

The validator would just report errors but not create a new IR.
The BIR lowering would operate directly on the syntax tree.

Looking over the source, I'm still not convinced the validation step holds its weight, but it'd also be non-trivial to remove, and the resulting code might well be a mess. The validation step seems to handle introduction of temporaries, for example, and a few other similar rewrites. Hmmm. I'm not going to try it now, anyway.


nuttxgdb list function enhancement

Note: Please adhere to Contributing Guidelines.
Summary
This PR includes minor bug fixes to list iterator and introduced more powerful iteration commands.

foreach list command now can iterate any type of list. E.g

    (gdb) foreach list -n next &g_msgfree
    0: {prev = 0x40286200 <g_msgpool+280>, next = 0x4028628c <g_msgpool+420>}
    1: {prev = 0x40286e30 <g_msgfree>, next = 0x40286318 <g_msgpool+560>}

    (gdb) foreach list "(struct list_node *) 0x40286e30"
    0: {prev = 0x40286200 <g_msgpool+280>, next = 0x4028628c <g_msgpool+420>}
    1: {prev = 0x40286e30 <g_msgfree>, next = 0x40286318 <g_msgpool+560>}

    (gdb) foreach list g_active_tcp_connections.head -n flink -c "struct tcp_conn_s" -m "sconn"
                                                                                13:14:07 [1047/1047]
    0 @ *(struct tcp_conn_s *)0x40441510 {
      sconn = {
        node = {
          flink = 0x40441658 <g_tcp_connections+984>,
          blink = 0x0
        },
        list = 0x40443754 <g_cbprealloc+1488>,
        list_tail = 0x40443754 <g_cbprealloc+1488>,
        s_error = 0,
        s_options = 0,
        s_rcvtimeo = 0,
        s_sndtimeo = 0,
        s_boundto = 0 '\000',
        s_flags = 105 'i',
        s_tos = 0 '\000',
        s_ttl = 64 '@'
      },
      u = {
        ipv4 = {
          laddr = 16777343,
          raddr = 16777343
        },
        ipv6 = {
          laddr = {127, 256, 127, 256, 0, 0, 0, 0},
          raddr = {0, 0, 0, 0, 0, 0, 0, 0}
        }
      },
      rcvseq = "h\374\375\064",
      sndseq = "h\374\375\063",
      crefs = 1 '\001',
      domain = 2 '\002',
      sa = 0 '\000',
      sv = 12 '\f',
      rto = 12 '\f',
      tcpstateflags = 4 '\004',

Note that https://github.com/apache/nuttx/pull/14912 is needed to fix the issue of container_of.

Added foreach array, just like list.

    E.g.

    (gdb) foreach array g_mmheap->mm_nodelist
    0: {preceding = 0, size = 0, pid = 0, seqno = 0, backtrace = {0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0}, flink = 0x41692cb8, blink = 0x0}
    1: {preceding = 0, size = 0, pid = 0, seqno = 0, backtrace = {0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0}, flink = 0x40c378a0, blink = 0x40605f58}

    (gdb) foreach array g_mmheap->mm_nodelist -e "flink"
    0: 0x41692cb8
    1: 0x40c378a0

Impact
No impact.
Testing
Tested on qemu

compile
tools/configure.sh lm3s6965-ek:qemu-flat

Make sure +CONFIG_DEBUG_SYMBOLS_LEVEL="-g3"

Run

qemu-system-arm -semihosting \
        -M lm3s6965evb \
        -device loader,file=nuttx.bin,addr=0x00000000 \
        -nic user,id=user0,hostfwd=tcp::10023-:23 \
        -serial mon:stdio -nographic -s

Connect

gdb-multiarch nuttx -ex "source tools/gdb/gdbinit.py" -ex "tar rem:1234"

Run the example commands above.

[Experimental Bot, please feedback here]
This PR description is a good start but could be improved to better meet the NuttX requirements. Here's a revised version:
Summary
This PR enhances the GDB debugging experience by providing more powerful and flexible list and array iteration commands within the NuttX GDB helper scripts.  It fixes bugs in the existing foreach list command, enabling it to iterate over any type of linked list, and introduces a new foreach array command.
These improvements facilitate easier inspection of NuttX internal data structures, such as the heap memory allocation lists and TCP connection lists, directly from the GDB debugger.
Related NuttX Issue: None (Consider creating one if this addresses a previously reported issue or enhances existing functionality)
Related NuttX Apps Issue/PR: None
Impact

Is new feature added? YES, the foreach array command. Existing foreach list functionality is also significantly improved.
Impact on user: YES.  Users will have access to new and improved GDB commands, offering more flexible data structure inspection during debugging.  Documentation updates will be required to explain the new functionality and usage examples.
Impact on build: NO.  This change only affects GDB helper scripts, not the NuttX build system itself.
Impact on hardware: NO.  This is a debugging tool enhancement and doesn't impact hardware behavior.
Impact on documentation: YES. The GDB helper script documentation needs to be updated to describe the new foreach array command and the improved foreach list command, including usage examples like those provided in the summary.
Impact on security: NO.  This change does not introduce any new security risks.
Impact on compatibility:  Backward compatible with existing GDB scripts. Forward compatibility is not affected. No known interoperability issues.
Anything else to consider: The dependency on PR #14912 should be explicitly mentioned in the PR description for this change to function correctly.  Consider adding a check within the script to detect if #14912 has been applied and provide a helpful message if it hasn't.

Testing
I confirm that changes are verified on a local setup and work as intended:

Build Host: Linux (Distribution: [Specify Distribution], Version: [Specify Version]), CPU: [Specify CPU Architecture], Compiler: GCC (Version: [Specify Version])
Target:  arch: sim (qemu-lm3s6965-ek)

Testing logs before change:
(Include specific commands and their output demonstrating the limitations or bugs being addressed. For example, show the error encountered when trying to iterate a non-standard linked list or the lack of an array iteration command.)
(gdb) foreach list g_active_tcp_connections.head -n flink -c "struct tcp_conn_s" -m "sconn"
# Previous behavior, perhaps an error message or incorrect output

Testing logs after change:
(Demonstrate the corrected behavior and new functionality with the same commands used before the change, as well as examples of the new foreach array command)
(gdb) foreach list g_active_tcp_connections.head -n flink -c "struct tcp_conn_s" -m "sconn"
# Corrected output as shown in the summary

(gdb) foreach array g_mmheap->mm_nodelist
# Output as shown in the summary

(gdb) foreach array g_mmheap->mm_nodelist -e "flink"
# Output as shown in the summary

By adding more detail and addressing all aspects of the requirements, this revised PR description will provide reviewers with a clearer understanding of the changes and their impact, leading to a smoother review process.  Remember to replace the bracketed placeholders with your specific information.


added DiagnosticDataProviderImpl for android

Problem

DiagnosticDataProviderImpl not impled on android platform

Change overview

Added Mandatory Attributes for Diagnostic cluster on android

Testing

chip-tool generaldiagnostics read network-interfaces
chip-tool generaldiagnostics read reboot-count

Fast tracking platform changes.


Lokalise updates from Aug 28

Description:
Waiting on some missing Greek and es_PR translations, but this is the latest from August 28

Linked issues:

Screenshots:

How to test:

@aledustet, the run failed on this PR and I went through the logs to find this:
Execution failed for task ':app:mergeDebugResources'.

/home/runner/work/gaen-mobile/gaen-mobile/android/app/src/main/res/values-es-r419: Error: Invalid resource directory name

es-r419 was generated. Seems like this name doesn't comply or android doesn't like this name for some reason. The simplest solution to make this run work probably is to delete the es-r419 file from this PR. We're not using the Latin America version for now and we can just delete the file for now and resolve the issue. If you have any other suggestions, let me know.
Thank you.

@aledustet, the run failed on this PR and I went through the logs to find this:
Execution failed for task ':app:mergeDebugResources'.

/home/runner/work/gaen-mobile/gaen-mobile/android/app/src/main/res/values-es-r419: Error: Invalid resource directory name

es-r419 was generated. Seems like this name doesn't comply or android doesn't like this name for some reason. The simplest solution to make this run work probably is to delete the es-r419 file from this PR. We're not using the Latin America version for now and we can just delete the file for now and resolve the issue. If you have any other suggestions, let me know.
Thank you.

Sounds about right, for the moment can we delete it from the LOKALISE project so the automation tool keeps working? I think it would be difficult to communicate to the rest of the team to keep an eye on this. I can look into why this ISO is not a valid one. Thanks

done. deleted the es-r419 android file. it should build as expected now and should be ready for merge.
i remember our discussion on es-r419, let me know if you find out why android didn't like this ISO format :/ thanks @aledustet ❤


Parsing data from socketcluster to express

Hi Jonathan! I am using SocketCluster as stand-alone server and have attached the express httpServer on it. Express use  the express-Handlebars as render engine with raw html files. The client login via a "login" socket emit event to SCserver. On server-side the  "login" socket listener make a query to the mongo database with the username or email, find the user in Users collection and authenticate the client by setting the AuthToken object. Then send it back to the client with the "login" event response. In client-side when the socket status is authenticated, redirect the client to the "/home" page: location.pathname = "/home";
The problem: I need some of the user data to set in local variables in the "home" view before the handlebars engine render it and send it to the client but:
I had these data at setAuthToken() authentication time, but that was while the user was in "/login" page. Now the user is about to receive and load the "/home" page so need to send  a selector to the server to query the database to send again the user data to server but before express-handlebars engine render the "home" view. I have the selector in the socket.authToken but must send it to express before it render the html. The authentication became in a  socket.on("login", function(credentials, res){ ... }); block
and the rendering will take place in a app.get('/home', function(req, res){ ... }); block.
I look around about a solution on how to parse the user data from the socket block to the express block, but not found something relative. Can you help me please?

SOLVED: I followed your instructions here . I attached in the app the scServer.options.authKey and then created an jwt token using the app.authKey. After that i send it to the client and stored in localStorage under the key "socketCluster.authToken". Thank you.


Markdown Link is Incompatible with Blessed

markdown
- **[pica](https://nodeca.github.io/pica/demo/)** - high quality and fast image
  resize in browser.

output:
]8;;https://nodeca.github.io/pica/demo/pica]8;; - high quality and fast image

expected output:

pica (https://nodeca.github.io/pica/demo/) - high quality and fast image resize in browser.

code:
const blessed = require("blessed");
const marked = require("marked");
const TerminalRenderer = require("marked-terminal");

marked.setOptions({
      // Define custom renderer
      renderer: new TerminalRenderer()
    });

var screen = blessed.screen({
      smartCSR: true
    });

const markdown = blessed.box({
      top: 0,
      left: 0,
      width: "100%",
      height: "95%",
      border: {
        type: "line"
      },
      style: {
        fg: "white",
        bg: "black",
        border: {
          // fg: "#f0f0f0"
        },
        hover: {
          // bg: "green"
        }
      }
    });

markdown.setContent(
      marked(`- **[pica](https://nodeca.github.io/pica/demo/)** - high quality and fast image
    resize in browser.`)
    );

screen.append(markdown);

screen.render();

    screen.key(["escape", "q", "C-c"], function(ch, key) {
      return process.exit(0);
    });

please help
UPDATE, found the issue:
this is parsed markdown file from 1.5.0, with EXPECTED OUTPUT

    * pica (https://nodeca.github.io/pica/demo/) - high quality and fast image
     resize in browser.

and this is from 3.3.0 with UNEXPECTED OUTPUT:
    * ]8;;https://nodeca.github.io/pica/demo/pica]8;; - high quality and fast image
        resize in browser.

@tonywei92  -- I like to do it, please assign me.

Great, @IsAmrish. I've assigned you now. Please let me know if there's questions you have. I would start by just setting up locally and investigating/testing out the code. Run the examples and the test and see how it works.

@mikaelbr @tonywei92  --  I tried above code and I got this output. I am using PowerShell on windows

You can see my code here
https://github.com/IsAmrish/blessed-example
please let me know, how can I reproduce the above error?

I can reproduce the error doing the same steps @tonywei92  described above.


[Sample] http-client connectior sample throws a SSL Engine closed already error

Environment
Oracle jdk 1.8.0_121
ballerina-0.970.0-beta1-SNAPSHOT
Description:
Run the below command to execute the sample
./ballerina run /home/shanika/WSO2/ballerina-cmd4-BBEx/ballerina-examples/examples/http-client-connector/http-client-connector.bal
Observed the following output
GET request:
{"args":{"test":"123"},"headers":{"host":"postman-echo.com","user-agent":"ballerina/0.970.0-beta1-SNAPSHOT","x-forwarded-port":"443","x-forwarded-proto":"https"},"url":"https://postman-echo.com/get?test=123"}
POST request:
{"args":{},"data":"POST: Hello World","files":{},"form":{},"headers":{"host":"postman-echo.com","content-length":"17","content-type":"text/plain","user-agent":"ballerina/0.970.0-beta1-SNAPSHOT","x-forwarded-port":"443","x-forwarded-proto":"https"},"json":null,"url":"https://postman-echo.com/post"}
PUT request:
{"args":{},"data":"{"method":"PUT","payload":"Hello World"}","files":{},"form":{},"headers":{"host":"postman-echo.com","content-length":"40","content-type":"text/plain","user-agent":"ballerina/0.970.0-beta1-SNAPSHOT","x-forwarded-port":"443","x-forwarded-proto":"https"},"json":null,"url":"https://postman-echo.com/put"}
PATCH request:
{"args":{},"data":"{}","files":{},"form":{},"headers":{"host":"postman-echo.com","content-length":"2","content-type":"text/plain","user-agent":"ballerina/0.970.0-beta1-SNAPSHOT","x-forwarded-port":"443","x-forwarded-proto":"https"},"json":null,"url":"https://postman-echo.com/patch"}
DELETE request:
{"args":{},"data":"DE","files":{},"form":{},"headers":{"host":"postman-echo.com","content-length":"2","content-type":"text/plain","user-agent":"ballerina/0.970.0-beta1-SNAPSHOT","x-forwarded-port":"443","x-forwarded-proto":"https"},"json":null,"url":"https://postman-echo.com/delete"}
SSLEngine closed already

Not reproducible in the latest version.


LLToUTM and UTMToLL any reason why these functions are not exported?  Seems like they may useful as well

Any reason why LLToUTM and UTMToLL are not exported?

To transform between geographic coordinates and UTM, use https://github.com/proj4js/proj4js.


Upgrade sushi to 3.3.1

Having problems locally. Antlr fix needs webpack. Need to tune our docker image or wait longer until we get version without fixAntlr4.

#168 supersedes this.

Superseded by #168, right?

yep


Half of my ship goes down by one block when I deconstruct

I am currently using ValkiyrienSkies (forge-2.0.0-alpha6) and Eureka (1.0.0-alpha7) and sometimes when I deconstruct a ship, half of it goes down or up by one block(see picture below). After some testing I fugured out that it mostly happens when the ship faces y-axis.

The fix for this issue has already been pushed and will be part of the next Curseforge release.


Inclusion of react-addons-linked-state-mixin in package.json breaks compilation

When  "@rjsf/core": "5.18.3" or newer is installed the compilation on https://github.com/jupyterlab/jupyterlab_apod/pull/20 fails:
node_modules/@rjsf/core/lib/components/Form.d.ts:292:76 - error TS2694: Namespace '"/node_modules/@types/react/jsx-runtime"' has no exported member 'JSX'.

292     renderErrors(registry: Registry<T, S, F>): import("react/jsx-runtime").JSX.Element | null;
                                                                               ~~~

node_modules/@rjsf/core/lib/components/Form.d.ts:367:43 - error TS2694: Namespace '"/node_modules/@types/react/jsx-runtime"' has no exported member 'JSX'.

367     render(): import("react/jsx-runtime").JSX.Element;
                                              ~~~

downgrading to 5.18.2 fixes the issue.
react-addons-linked-state-mixin was added in https://github.com/jupyterlab/extension-template/pull/38. Why is it needed?

This appears related to React upgrade in https://github.com/rjsf-team/react-jsonschema-form/pull/4107 that was merged in @rjsf/core 5.18.3

Of note, deduplication does not fix it. It looks like a more fundamental resolution error. So there are three solutions for the users:

add resolutions: {"@rjsf/core": "5.18.2"} in package.json
remove react-addons-linked-state-mixin from package.json
set skipLibCheck: true

Nah, I think I was wrong about this. Back to debugging.


dirDisqus: prevent error when config object is undefined

I ran into this error TypeError: Cannot read property 'disqus_shortname' of undefined when using a config object that got their data asynchronously, in my WP REST API + AngularJS front-end blog.
Template:
<dir-disqus config="disqusConfig"></dir-disqus>
Controller:
afterSomeAjaxStuff(function(){
    $scope.disqusConfig = {
        disqus_shortname: 'myShortname',
        disqus_identifier: $stateParams.slug,
        disqus_url: $scope.currentURL
    };
});

This is just a simple flag to check that the config object actually has some data before continuing to execute the rest of configChanged().

:+1:


Create a page in doc repo about how we use the design system

@rroppolo @davidmferris @wenzeng4 @jojolee1996 @fausgaitan if you have ideas about what type of information this page should contain, add it here!
My quick thoughts are something like:

General philosophy

Talk about how we view "1 offs" etc

When to use design system components
When do add something to the design system
What to consider when updating an existing component


Fixed contents menu scroll

Fixed the contents menu on the left to scroll if the height exceeds the window.
Resolves #8

Yes, a scrolly bar thing is pretty much what I would expect to fix this. Look ok to me.


Master

Hacked Automatic-MCU Boot into Programm, using RTS and DTR pins (invert DTR Signal with a Transistor)

I use the RTS line to toggle the Boot signal so this might not be too good as a generic library patch. :)

I guess its not actively managed anyways, so I think I'll close it...
Forgot that i did this anyways.


ICON EUV Initial Support

Added experimental support for ICON EUV level-2 data. The preamble on variable names is automatically removed for brevity. Unit tests are included. A note was added to the change log. The instrument was added to the documentation.

Coverage decreased (-1.2%) to 79.955% when pulling bc3f85b405d73d1ad2a0c3e5d9c72f2c275ac320 on icon_euv into 5fc3c06b5a9376349ab53b0c1361f3f72e95ead0 on develop.

Coverage decreased (-1.2%) to 79.955% when pulling bc3f85b405d73d1ad2a0c3e5d9c72f2c275ac320 on icon_euv into 5fc3c06b5a9376349ab53b0c1361f3f72e95ead0 on develop.

Reduction in test coverage is a result of not being able to download and load a test day of ICON EUV data.


Prevent Safari 11 Error

fix #134
Safari 11 no longer supports createObjectURL

I suggest closing this in favour of https://github.com/bitpay/cordova-plugin-qrscanner/pull/214 if everyone is happy with that PR. This one unfortunately runs into memory issues after a short while due to the canvas-leak described in that PR


Errors when Terrain Mapper module active.

The latest version of Elevation Ruler (v0.10.0) seems to bug out when Terrain Mapper is active, even if there is no terrain on the map. A single Region placed on the map, even if it's not set as terrain, causes multiple console errors to fire off when you drag your token, and the ruler is not displayed during the token drag anymore.

libWrapper-wrapper.js:513 Uncaught TypeError: Cannot read properties of undefined (reading 'hasTerrain')
at MovePenalty.js:71:28
at Array.forEach ()
at new MovePenalty (MovePenalty.js:70:49)
at Ruler._computeSegmentDistances (segments.js:38:57)
at Ruler._computeDistance (Ruler.js:380:28)
at 🎁CONFIG.Canvas.rulerClass.prototype._computeDistance#0 (libWrapper-wrapper.js:187:52)
at Ruler.measure (foundry.js:36092:10)
at Ruler._addWaypoint (Ruler.js:226:8)
at 🎁CONFIG.Canvas.rulerClass.prototype._addWaypoint#0 (libWrapper-wrapper.js:187:52)
at Ruler._startMeasurement (foundry.js:36191:10)
at Ruler._onDragStart (foundry.js:36658:10)
at 🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:507:22)
at Ruler._onDragStart (Ruler.js:614:10)
at 🎁CONFIG.Canvas.rulerClass.prototype._onDragStart#0 (libWrapper-wrapper.js:187:52)
at Token5e._onDragLeftStart (Token.js:159:25)
at 🎁CONFIG.Token.objectClass.prototype._onDragLeftStart#0 (libWrapper-wrapper.js:187:52)
at MouseInteractionManager.callback (foundry.js:30823:17)
at #handleDragStart (foundry.js:31185:15)
at #handlePointerMove (foundry.js:31163:43)
at vh.notifyListeners (EventBoundary.ts:1464:33)
at vh.notifyTarget (EventBoundary.ts:658:14)
at vh.propagate (EventBoundary.ts:310:18)
at vh.dispatchEvent (EventBoundary.ts:210:14)
at vh.mapPointerMove (EventBoundary.ts:815:64)
at vh.mapEvent (EventBoundary.ts:231:28)
at ra.onPointerMove (EventSystem.ts:396:31)

Apologies, I was wrong. I freshly installed Terrain Mapper assuming that would be the most recent version, but it was not. Upon updating the module, it went to the correct version and I no longer ran into the problem.

Yeah, that one is on me. I had not yet updated the Foundry package listing, so it was giving you the older Terrain Mapper on fresh install.


.theiaStickySidebar showing ontop of YouTube full screen

Does anyone have advice on how to stop the sticky sidebar from showing when a YouTube video has been expanded to full screen? See example below...

Any ideas on solving this issue? I tried to do an overwrite on the YouTube styling but this doesn't seem to be the best idea.

Some ideas: Try setting a very high z-index on the YouTube iframe, or add position:relative to it, or both.


cartridges: init at 2.0.1

Description of changes
cartridges is a simple game launcher written in python using libadwaita. it supports managing games from platforms/programs such as steam, lutris, heroic, bottles, and itch.io
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build cartridges

Result of nixpkgs-review pr 234486 run on x86_64-linux 1

1 package built:

cartridges


{edencommon,fb303,fbthrift,fizz,folly,mvfst,wangle,watchman}: 2<IP_ADDRESS> -> 2<IP_ADDRESS>

Description of changes
Updating to the latest releases. Many of these need to be updated in tandem as there are interdependent changes without which the builds fail.

Successfully built all changed packages
Tested building some random dependent packages: fava, python3Packages.types-aiobotocore-opensearchserverless

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

On macos 12 x86_64, folly fails to build:
In file included from /tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/logging/example/lib.cpp:17:
In file included from /tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/logging/example/lib.h:21:
In file included from /tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/logging/xlog.h:26:
In file included from /tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/logging/LoggerDB.h:29:
In file included from /tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/detail/StaticSingletonManager.h:27:
In file included from /tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/lang/Thunk.h:22:
/tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/lang/New.h:66:12: error: no matching function for call to object of type 'const conditional_t<false, op_del_builtin_fn_, op_del_library_fn_>' (aka 'const folly::detail::op_del_library_fn_')
    return op_del_<Usual>(p, a...);
           ^~~~~~~~~~~~~~
/tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/lang/New.h:119:20: note: in instantiation of function template specialization 'folly::detail::do_op_del_sized_<false, std::align_val_t>' requested here
    return detail::do_op_del_sized_<detail::cpp_aligned_new_>(p, s, a);
                   ^
/tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/lang/New.h:52:1: note: candidate template ignored: substitution failure [with A = <void *const &, const std::align_val_t &>]
FOLLY_CREATE_QUAL_INVOKER(op_del_library_fn_, ::operator delete);
^
/tmp/nix-build-folly-2<IP_ADDRESS>.drv-0/source/folly/functional/Invoke.h:515:58: note: expanded from macro 'FOLLY_CREATE_QUAL_INVOKER'
    [[maybe_unused]] FOLLY_ERASE_HACK_GCC constexpr auto operator()(A&&... a) \

Awesome, all WFM now on macos 12 x86_64. Could you squash down the fixup commits so it's one commit per package and I'll approve?

Done


cmd/bosun: skiplast cmdline switch for development

I'm ok with the code. Wondering though...
The last stuff is all to support the host api for opserver. I imagine we are the only people who will use the opserver integration. Would it make sense to have an opt-in flag for that whole side of things?

I'm not sure it is only used in the host API? Maybe autocomplete uses it to see the age of things? If people are not indexing data in bosun the load won't take any time. So I'm okay keeping the featured tied to using indexing for now.


Viewing nothing when using Kong Ingress

I was trying to visualize my cluster. we are using KongIngress and the cluster was added propperly..
Even when klicking on edit (view mode only) the konfig gets loaded and all services are listed..
But I can only see the Kong Logo in the "view" part... No errors anywhere..

Hi, you have exposed the admin api of the controller/gateway? What version of Kong Ingress controller are you using?  Are you saying the config is showing properly in declarative viewer but not the map? If so, there is something in the map that is breaking the javascript?  What does the javascript console say in the browser? Can you share the resulting HTML of the map view (minus anything that would be private)

Hi, you have exposed the admin api of the controller/gateway? What version of Kong Ingress controller are you using?  Are you saying the config is showing properly in declarative viewer but not the map? If so, there is something in the map that is breaking the javascript?  What does the javascript console say in the browser? Can you share the resulting HTML of the map view (minus anything that would be private)

What exactly do you mean by controller/gateway?
KongIngress = kong:2.2.1-alpine
That is correct
There are no js errors in the console

The canvas element is there, but except the monkey.. There is nothing else:

What exactly do you mean by controller/gateway?
KongIngress = kong:2.2.1-alpine
That is correct
There are no js errors in the console

The canvas element is there, but except the monkey.. There is nothing else:

One last thing, can you share the kongmap cluster config json (minus sensitive data)?  Happy to jump on a zoom to help live.

One last thing, can you share the kongmap cluster config json (minus sensitive data)?  Happy to jump on a zoom to help live.

This could also be related to an issue w/ the new vis-network version that was released that broke some elements of the map. Will be updating the docker image in the next 24 hours to fix.

This could also be related to an issue w/ the new vis-network version that was released that broke some elements of the map. Will be updating the docker image in the next 24 hours to fix.

Hi @cronventis .  Can you pull the latest image from dockerhub and verify if the problem is now resolved related to https://github.com/yesinteractive/kong-map/issues/10?

Hi @cronventis .  Can you pull the latest image from dockerhub and verify if the problem is now resolved related to https://github.com/yesinteractive/kong-map/issues/10?

Nope, still not working.
how would you like to get in contact so you can take a closer look?

Nope, still not working.
how would you like to get in contact so you can take a closer look?

If you are not comfortable leaving your email here, please do so on the contact form on https://www.yes-interactive.com/ and reference this issue and I will get back to you via email. we can schedule a zoom call to figure out what's going on.
Is the admin API generating content? meaning, are there are the /routes and /services endpoints showing any data? It looks like nothing is being returned (ie, if there is nothing configured in Kong as far as ingress rules/routes and services then the map will just show the Kong logo).  A good test is to apply a sample ingress configuration  like https://github.com/yesinteractive/dadjokes/blob/master/examples/dadjokes_kong_for_k8s.yaml and then reload map)

If you are not comfortable leaving your email here, please do so on the contact form on https://www.yes-interactive.com/ and reference this issue and I will get back to you via email. we can schedule a zoom call to figure out what's going on.
Is the admin API generating content? meaning, are there are the /routes and /services endpoints showing any data? It looks like nothing is being returned (ie, if there is nothing configured in Kong as far as ingress rules/routes and services then the map will just show the Kong logo).  A good test is to apply a sample ingress configuration  like https://github.com/yesinteractive/dadjokes/blob/master/examples/dadjokes_kong_for_k8s.yaml and then reload map)

Issued was identified to be related to a trailing slash in the admin api configuration. Will update kongmap to trim trailing slashes. Re-labeled as a bug.

Issued was identified to be related to a trailing slash in the admin api configuration. Will update kongmap to trim trailing slashes. Re-labeled as a bug.

Didn't have time to get this in but updated docs to note that admin api can not have trailing slash in the config.


[BrokenLinksH2] Fixed the link path

Fixed the links

Docs Build status updates of commit f5ed213:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/CNTK-on-Azure.md
:warning:Warning
View
Details

articles/CNTK-on-Azure.md

Line 14, Column 22: [Warning: insecure-link - See documentation] Link 'http://azure.microsoft.com/' is insecure. Links to Microsoft sites must use 'https' instead of 'http'.
Line 22, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/machine-learning/data-science-virtual-machine/provision-vm' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 28, Column 3: [Warning: insecure-link - See documentation] Link 'http://aka.ms/dsvm/ubuntu/docs' is insecure. Links to Microsoft sites must use 'https' instead of 'http'.
Line 29, Column 153: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/services/batch/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 35, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/virtual-machines/windows/n-series-driver-setup' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 36, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/azure/virtual-machines/linux/n-series-driver-setup' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 22, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/machine-learning/data-science-virtual-machine/provision-vm' will be broken in isolated environments. Replace with a relative link.
Line 35, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/virtual-machines/windows/n-series-driver-setup' will be broken in isolated environments. Replace with a relative link.
Line 36, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/azure/virtual-machines/linux/n-series-driver-setup' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7f8150f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/CNTK-on-Azure.md
:warning:Warning
View
Details

articles/CNTK-on-Azure.md

Line 29, Column 153: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/services/batch/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 8d9f70a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/CNTK-on-Azure.md
:bulb:Suggestion
View
Details

articles/CNTK-on-Azure.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix broken links
@chrisbasoglu
The Content & Learning team is fixing broken links on docs.microsoft.com for the rest of H2. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes and does not change other content. I’d very much appreciate your timely review and feedback on the link changes I’ve suggested. Thanks!


[Feature] Manually bind execution plan

Codecov Report

Merging #1473 (f21fdcb) into master (19e178a) will decrease coverage by 0.15%.
The diff coverage is 16.29%.

:exclamation: Current head f21fdcb differs from pull request most recent head efd2e3e. Consider uploading reports for the commit efd2e3e to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1473      +/-   ##
==========================================
- Coverage   24.83%   24.69%   -0.15%
==========================================
  Files         167      167
  Lines       14853    14958     +105
==========================================
+ Hits         3689     3694       +5
- Misses      10893    10993     +100
  Partials      271      271

Flag
Coverage Δ

backend_integration
8.90% <14.50%> (-0.08%)
:arrow_down:

backend_ut
27.28% <3.14%> (-0.29%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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 19e178a...efd2e3e. Read the comment docs.


Fix/grpc ssl

What type of PR is this?
/kind bug
What this PR does / why we need it:
Adds a useSsl parameter to grpc node.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation e.g., usage docs, etc.:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Align distance filtering thresholds

What does this PR do?
Updates the code so that there is a single set of distance filtering thresholds, unifying the previously-separate distanceFilter and locationChangeThresholdMeters values.
Note: This PR changes the location update thresholds for the various accuracy values.
Any background context you want to provide?
There were two competing distance filtering thresholds in the code: one used to set the CLLocationManager.distanceFilter, and one used by LocationListener.shouldUpdateListenerForLocation() to filter location updates.
Since the existing values were different, I had to choose one set. I decided to go with the values for the distanceFilter, since those were chosen to combine well with the accuracy values (although the others probably were, too).
Where should the reviewer start?
To support this, I refactored the LocationManager a bit. Instead of calculating an accuracy and updateFrequency from the list of listeners and then computing distanceFilter and desiredAccuracy for the CLLocationManager from those, each LocationListener now calculates its own distanceFilter and desiredAccuracy, and the LocationManger just picks the appropriate (minimum) value.
After that, I removed the locationChangeThresholdMeters and changed shouldUpdateListenerForLocation() to use the distanceFilter instead.

@nonsensery Thanks!


augment default rules with ordering for App/PackageInstall

Describe the problem/challenge you have
want App/PackageInstall CR to work nicely (in context of ordering with service account/RBAC) out of the box in kapp deploy/delete.
Describe the solution you'd like
add default rules?
upsert:

upsert clusterrolebinding after service account and clusterrole
upsert PackageInstall/App should after service account and RBAC

deletion:

delete PackageInstall/App before service account and RBAC

Anything else you would like to add:
[Additional information that will assist in solving the issue.]

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

adding to un-prioritized backlog for now.


Change the page with left or right side

Hello, I open an issue to request a feature
Is it possible, as in mupdf, to add an option so that when you touch the left or right side of the screen, it changes the page?
Thank you

duplicate of #41


Will the lib parse terraform plan file?

Hi! Just curious, will the lib parse terraform plan file?
Thanks!

@renou I assume that you are talking about using [terraform plan -out](https://www.terraform.io/docs/commands/plan.html#out-path0, right? tfstate is intended to read the raw JSON state file and the plan -out file appears not to be JSON, so I expect it won't work.
I will say that I have not used the plan -out feature of Terraform and I am not familiar with the contents of the file, so it is possible that something could work. I would recommend testing it out and reporting back whether it works or not for you.

Thanks for answering! 👍


Uncaught Error: Cannot find module '/Users/nando/.atom/packages/proto-repl/lib/lein-runner.coffee'

[Enter steps to reproduce below:]

I simply started the repl using the Start REPL command from the Packages menu in the GUI.

Atom Version: 1.6.0
System: Mac OS X 10.10.5
Thrown From: proto-repl package, v1.1.3
Stack Trace
Uncaught Error: Cannot find module '/Users/nando/.atom/packages/proto-repl/lib/lein-runner.coffee'
At /Applications/Atom.app/Contents/Resources/app.asar/src/module-cache.js:279

Error: Cannot find module '/Users/nando/.atom/packages/proto-repl/lib/lein-runner.coffee'
    at Module._resolveFilename (module.js:336:15)
    at Function.Module._resolveFilename (/Applications/Atom.app/Contents/Resources/app.asar/src/module-cache.js:383:52)
    at Function.require.resolve (/Applications/Atom.app/Contents/Resources/app.asar/src/native-compile-cache.js:53:23)
    at new Task (/Applications/Atom.app/Contents/Resources/app.asar/src/task.js:35:26)
    at Function.module.exports.Task.once (/Applications/Atom.app/Contents/Resources/app.asar/src/task.js:17:14)
    at LocalReplProcess.module.exports.LocalReplProcess.start (/Users/nando/.atom/packages/proto-repl/lib/local-repl-process.coffee:63:27)
    at Repl.module.exports.Repl.startProcessIfNotRunning (/Users/nando/.atom/packages/proto-repl/lib/repl.coffee:87:16)
    at Object.module.exports.ProtoRepl.toggle (/Users/nando/.atom/packages/proto-repl/lib/proto-repl.coffee:177:17)
    at atom-workspace.subscriptions.add.atom.commands.add.proto-repl:toggle (/Users/nando/.atom/packages/proto-repl/lib/proto-repl.coffee:19:5)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:260:29)
    at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchApplicationMenuCommand (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1013:28)
    at emitOne (events.js:77:13)
    at EventEmitter.emit (events.js:169:7)

Commands
     -0:01.6.0 proto-repl:toggle (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "atom-light-ui",
      "subway-seoul"
    ],
    "disabledPackages": [
      "language-hyperlink"
    ]
  },
  "proto-repl": {}
}

Installed Packages
# User
Parinfer, v1.14.0
jsformat, v0.9.0
language-cfml, v0.12.0
language-lucee, v0.2.0
linter, v1.11.3
linter-clojure, v1.1.2
proto-repl, v1.1.3
subway-seoul, v1.0.9
tool-bar, v0.2.1

# Dev
No dev packages

I uninstalled and reinstalled proto-repl, and now it seems to be working again.

I've seen this happen when proto repl is upgraded and you don't restart atom. I don't see this issue with other packages so I wonder if I'm doing something wrong with the way dependencies are loaded. Note that lein-runner.coffee was moved to a subfolder during a recent update. It was still looking for lein runner in the old location.

I asked about this problem here https://discuss.atom.io/t/package-code-not-reloading-during-an-update/32411


Remove duplicate dependencies.

Is your feature request related to a problem? Please describe.
Currently the plugin does not notice or warn if you have two of the exact same dependency in your dependencies block.
Describe the solution you'd like
The plugin should remove duplicate lines from dependencies when using fixDependencies and maybe warn when running projectHealth

Thanks for the issue.


Vibrio parahaemolyticus strain CICESE-170 is missing PLFAMS

https://www.patricbrc.org/view/Genome/670.1042#view_tab=features

both plfam and pgfams are now available for this genome.


Spike for #814 - redirect to new spar screen from old

Describe the task
Need to understand the complexity of redirecting from old spar to new spar. We can also see what is needed to redirect from new spar to old (ie orchard management link/page)
Acceptance Criteria

[ ] document in confluence

Related to task #814

The menu tag implementation can be found here:
source/ear/src/main/webapp/sparMenuLinks.jsp
Each menu item is a jsMenuItem, which is defined here: source/ear/src/main/java/ca/bc/gov/mof/spar/web/taglib/MenuItemTag.java


Getting error while bootstraping vcpkg

Getting following error when I try to boot strap vcpkg for exe generation.
PS C:\vcpkg\scripts> .\bootstrap.ps1
Building vcpkg.exe ...
Process has exited, so the requested information is not available.
At C:\vcpkg\scripts\bootstrap.ps1:372 char:5

Wait-Process -InputObject $process

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], InvalidOperationException
FullyQualifiedErrorId : System.InvalidOperationException

C:\vcpkg\scripts\bootstrap.ps1 : Building vcpkg.exe failed. Please ensure you have installed Visual Studio with the Desktop C++ workload and the Windows SDK for Desktop C++.
At line:1 char:1

.\bootstrap.ps1

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,bootstrap.ps1

PS C:\vcpkg\scripts>
Also I have, Visual Studio 2017, CMake, Windows 10 64-Bit OS. I have confirmed both Visual Studio with the Desktop C++ workload and the Windows SDK for Desktop C++ installed through VS installer.
Please let me know how to resolve this issue.

This issue hasn’t been updated in a year; if it is still an issue, please reopen.


Book loader not displayed correctly

Book loader is not displayed correctly on landing page.
Happens on Firefox 85.0.2 and Chrome 88.0.4324.182.

@KnechtNoobrecht Thanks for noticing issue. Please raise PR if you know the solution.

@ashutosh1919 Is it fixed ?


Ignore known PowerShell constants

Maybe ignore hard-coded variables like null, true and false?
$null = $null is waste of space that can be better utilized.

@fflaten great idea! This is a really simple feature request. Wanna try contributing? I can walk you through it.
We already have the variable name here:
https://github.com/TylerLeonhardt/vscode-inline-values-powershell/blob/master/src/powerShellVariableInlineValuesProvider.ts#L19
So all you have to do is check to see if the variable name is $null or whatever and if it is, continue the loop.
I just added steps on how to contribute here:
https://github.com/TylerLeonhardt/vscode-inline-values-powershell#development

Released :) 0.0.5 should show up in a few minutes.


Fix deploy with problematic packages

Welp, I didn't get very far here. The Vercel suggestions are not that great.
Clearing the cache with yarn clean cache && yarn install just led to a typescript error for not having Node types. Which I don't understand because we have a dev dependency for that. Plus it takes a lot of extra time, so it is not a good solution in anycase.


Azure OpenAI API

Nice project! Do you plan to support the Azure OpenAI API? https://learn.microsoft.com/en-us/azure/ai-services/openai/reference

Thanks @didalgolab. It's not in my near future plans, maybe in the next major release, however I would like to know why you think I should support it? I mean, wouldn't it be kind of redundant to support Azure OpenAI if they are calling OpenAI services behind the scenes?

One reason to use Azure OpenAI is that currently it's one of the few methods available for accessing gpt-4-32k model. Other considerations that might be important for some users or companies include: usage policies, GDPR compliance, co-location with other Azure services.

@didalgolab did you try it with Azure OpenAI? It may just work. I tried it with Anyscale and it works like a charm including function calling.

@the-gigi Yes, and unfortunately, it's not compatible. While the Anyscale API has been built with compatibility in mind, Azure OpenAI hasn't. It seems almost as if Microsoft did this on purpose:

On Azure, there is a mandatory query parameter "api-version" which must be added to each request. This can't be passed in the baseUrl to this library because it renders the URL invalid further down the road.
The "model" property is not allowed in the request.
Finally, there is a different header for authentication.

But anyway, it's a low priority for me now.

@sashirestela I looked into it and I think with some modifications simple-openai can handle Azure OpenAI too. I'm willing to work on a PR with your guidance.
Here are the sticking points:

Endpoint URL.

Azure OpenAI has a dedicated URL per deployment. A deployment already has a specific model configured. The format of the URL is:
https://YOUR_RESOURCE_NAME.openai.azure.com/openai/deployments/YOUR_DEPLOYMENT_NAME/chat/completions?api-version=2023-05-15

So, from simple-openai point of view the base URL will be https://YOUR_RESOURCE_NAME.openai.azure.com/openai/deployments/YOUR_DEPLOYMENT_NAME (note, no /v1) and then it needs in the end to add the query params.
This can be accomplished by checking if the base url contains ? and then strip these and in the end before sending the actual HTTP request append the query params back.
simple-openai uses cleverclient, which provides an abstraction over URLs with annotations. so, I'm not sure how to accomplish it exactly. I see that for GET requests there is a @Query annotation. maybe it work for POST requests too.
The other related issue is that simple-openai automatically adds the /v1 before each endpoint. This can be handled simply by removing it and having the endpoint be just /chat/completions (for chat, similar to other endpoints)
instead of :
@Resource("/v1/chat/completions")
    interface ChatCompletions {

just:
@Resource("/chat/completions")
    interface ChatCompletions {

Two additional minor issues as @didalgolab pointed out:

the authorization header is just api-key: <API Key>
model is not used (as mentioned the endpoint you're hitting is pre-configured for a specific model).

I believe supporting Azure OpenAI can be done without making simple-openai aware of the differences between providers. But, if going for multi-provider support it may be better to introduce the concept of an optional provider. The default will be OpenAI (which works out of the box for Anyscale too). For other providers like Azure OpenAI that currently are incompatible with some assumptions, simple-openai can internally adjust its behavior (e.g. drop the /v1 only for Azure OpenAI and add the api-key: <API Key> header only for Azure Open AI).
Let me know what you think.

@the-gigi @didalgolab Do you know if the api-version url parameter can be different by endpoint or is it unique for all the API?

@sashirestela yes. they can even be different for the same endpoint. Each endpoint corresponds to a Deployment, which has its own model and has its own subdomain and supports different API versions with the query parameter.
See https://learn.microsoft.com/en-us/azure/ai-services/openai/reference#rest-api-versioning

@sashirestela yes. they can even be different for the same endpoint. Each endpoint corresponds to a Deployment, which has its own model and has its own subdomain and supports different API versions with the query parameter.
See https://learn.microsoft.com/en-us/azure/ai-services/openai/reference#rest-api-versioning

In that case, one option could be adding a Query parameter to all the http methods and doing one of the following:

Create default methods by each http method to set to null that api-version, so no impacting OpenAI users. The code is going to be pretty messy.
Don't create default methods and the OpenAI users will need to add a null value for the api-version parameter. I don't think this is a good option.

Let me try to think of other options, but if you say that the parameter could change in every call, it is a very challenging requirement.

@sashir my thinking is that simple-openai should support the latest version by default for each API method. so, users normally don't have to pass it. But, if they want a different API version then there will be an optional apiVersion in their request and it will replace the default when specified. All this applies only if its targeting Azure OPenAI of course.

@the-gigi
The scenario is the same, because Java doesn't support optional arguments for methods neither default values for arguments, so we will need to overload all the http methods with new default methods in order to support that requirement of varieable api-version for Azure. The code of the OpenAI.java interface is going to grow a lot just for that new parameter.
I think that trying to use the same interface is very hard. Perhaps another option could be to create duplicated versions, I know that it is not ideal for your goal, but Microsoft is imposing a non standard way to handle this.

Standard: (Existing)

Interface: OpenAI.java
Class: SimpleOpenAI

Microsoft: (New)

Interface: AzureOpenAI.java
Class: SimpleAzureOpenAI.java

My idea is not to pass the query parameters as part of the method, but as part of the request object. so, for example ChatRequest will have one new nullable field
public class ChatRequest {

    @NonNull private String model;
    @NonNull private List<ChatMsg> messages;
    private ChatRespFmt responseFormat;
   ...
   private String apiVersion; // new optional field for Azure OpenAPI version support.

If the field is not present then the default apiVersion for each Azure OpenAI API endpoint is appended to the URL.
So, the method signature remains (except for the /v1 that will be added to OpenAI only, or the relevant version if not /v1):
Instead of
@Resource("/v1/chat/completions")
    interface ChatCompletions {

We will have
@Resource("/chat/completions")
    interface ChatCompletions {

It will require some dynamic setting of the base URL before calling celverclient.
pretty the @Resource annotation specifies just part of the path part of the URL. Different API methods and providers may have different prefixes (e.g. /v1 in case of OpenAI for /chat/completions) and different query params (api-version in case of Azure OpenAI)
Regarding the model. it shouldn't be required if we go for the multi-provider approach because not all providers require it. We can still implement per-provider logic and verify that for OpenAI it is provided.

My idea is not to pass the query parameters as part of the method, but as part of the request object. so, for example ChatRequest will have one new nullable field

If you pass it as part of the request object, it'll be on the payload body of the request, not in the url as a query parameter, so have you verified that Azure is going to recognize it in that way?

Instead of
@Resource("/v1/chat/completions")
interface ChatCompletions {
We will have
@Resource("/chat/completions")
interface ChatCompletions {
It will require some dynamic setting of the base URL before calling celverclient.

Remember that you need to do this by method (/v1 could be different by method). If you change baseUrl you will change it for all the methods and in a multithread scenario this could be an issue.

Regarding the model. it shouldn't be required if we go for the multi-provider approach because not all providers require it. We can still implement per-provider logic and verify that for OpenAI it is provided.

If we are going to support per-provider logic, please, take in account the Open-Close principle: open for extension and closed for modifications.

Yes. There is a lot of stuff that happens automatically with the annotations and serialization, so I need to verify my idea works. I'll follow up after I tried to implement my approach. It will require some provider-specific logic for sure. I hope it doesn't require duplicating a lot of classes and interfaces with different signatures per provider. Instead, I aim for a general-purpose implementation with some per-provider map of method -> prefix + query params. But, until I actually implement it, it is speculation only.

In my opinion api-version should be provided just in baseUrl (and correctly handled by the cleverclient while making requests). I think there is little to no reason for a user to use different api-versions for different endpoints. More so, because single baseUrl in Azure represents just a one specific deployment that works with just a single api method (e.g. it works for chat completion and nothing else; or embeddings and nothing else - if that makes sense for you).
The user will have to create separate clients with different baseUrls for different endpoints anyway. I also agree with @sashirestela to add separate interface AzureOpenAI but preferably extending OpenAI so that both have some common part which could be used in the user's code exchangeably.

Technically the base URL is the part of the URL before the the path. This is how it is treated also by simple-openai/cleverclient. cleverclient constructs the full URL by appending the Path (e.g. /v1/chat/completions) to the base Url. It also checks if there are query params defined in the method signature and adds those. What we need is to provide the queryParams dynamically to clever client, so it can construct the URL as usual and then in the end add the provided query params  (if any).
This is where it happens:
https://github.com/sashirestela/cleverclient/blob/main/src/main/java/io/github/sashirestela/cleverclient/http/URLBuilder.java#L29
@sashirestela the current design of cleverclient hard-codes the query params in the interface annotations as in:
@Resource("/posts")
public interface PostService {

  @GET
  List<Post> readPosts(@Query("_page") Integer page, @Query("_limit") Integer limit);

If you prefer to keep it strict (all query params must be specified at the interface level) then I think supporting Azure OpenAI will require a split.
But, if we add dynamic query params to cleverclient then we simpleopen-ai can add api-version query param based on the provider and we can remain with one version of the interfaces.

@sashirestela See https://github.com/sashirestela/cleverclient/pull/39. This PR will allow simple-openai to inject the Azure OpenAI api-version query parameter into cleverclient without touching the interfaces.

@the-gigi Many things to comment:

Finally, you are discarding to set the api-version parameter per http method and you are going with the option to set a unique value for all the endpoints at the moment that you instance the SimpleOpenAI class. That's fine, we have an agreement here.

Cleverclient was developed to be an independent library to ease Java's HttpClient usage, it's not tied to simple-openai exclusively , so we need to think in general approaches when we want to introduce new functionalities.

I saw your PR to cleverclient. Please, don't misunderstand my next comment: it could resolve the specific requirement to adapt us to the Azure behavior regarding api-version, however we need a more general approach here, imho.

For some time now, before this Azure issue, I was thinking to update cleverclient to support request post processing to change the url, for example to add query params (eg: api-version) or to remove parts of the url (eg: /v1) or whatever change in the url that we could need to do. For this, I was planning to use the Interceptor concept to pass a function as a parameter, so the logic to change the url will be on the cleverclient's client side (simple-openai for our case). I was thinking of adding an Interceptor at cleverclient instantiation level and in the near future, to be more flexible, other Interceptors at endpoints creation level (eg: you could have different api-version per endpoint).

I think the time has come to develop that Interceptor functionality. So, I'll be working on that change. I'll keep you informed.

@the-gigi
I've just developed the new feature UrlInterceptor. You can see an example of usage in my working branch (it is not merged yet):
https://github.com/sashirestela/cleverclient/blob/c848691ab80d6e79b77181f749a602b0fcf81372/src/test/java/io/github/sashirestela/cleverclient/CleverClientTest.java#L92-L119
In that example, I'm creating a functional Function with a logic to add a query param to url and remove '/vX' from url. Then, it is passed to the Cleverclient builder.
On the other hand, in previous comments you said:

Regarding the model. it shouldn't be required if we go for the multi-provider approach because not all providers require it. We can still implement per-provider logic and verify that for OpenAI it is provided.

So, where in the code do you think that we could introduce a per-provider logic to validate if the model in the request objects are required or not?

@sashirestela that looks great. I didn't verify if it's OK to send a redundant model. I wonder if we cleverclient should provide just a url provider or a complete request interceptor. This way consumers can intercept and modify any part of the request, including the body, which can be used to remove the model if necessary.

@sashirestela that looks great. I didn't verify if it's OK to send a redundant model. I wonder if cleverclient should provide just a url interceptor or a complete request interceptor. This way consumers can intercept and modify any part of the request, including the body, which can be used to remove the model if necessary.

Guys, the cleverclient library was extended to handle the full request (url, body, headers). So, simple-openai can now support to use Azure OpenAI, and you just need to provide a requestInterceptor function where you can make all the changes to adapt the standard request to the particularities of the Microsoft library.
See an example of a requestInterceptor here: https://github.com/sashirestela/cleverclient/blob/main/src/test/java/io/github/sashirestela/cleverclient/CleverClientTest.java#L71-L139


Merge go1.5+ assembly

See commit message.

The goal was just to reduce the mental overhead of reading this code. Regarding the added function call: I see no difference on local benchmarks:
Before:
$ go test -v -bench .
=== RUN   TestGet
--- PASS: TestGet (0.00s)
goos: linux
goarch: amd64
pkg: github.com/petermattis/goid
cpu: AMD Ryzen 9 7950X 16-Core Processor
BenchmarkGet
BenchmarkGet-32<PHONE_NUMBER>               0.9726 ns/op
PASS
ok      github.com/petermattis/goid     1.073s

After:
$ go test -v -bench .
=== RUN   TestGet
--- PASS: TestGet (0.00s)
goos: linux
goarch: amd64
pkg: github.com/petermattis/goid
cpu: AMD Ryzen 9 7950X 16-Core Processor
BenchmarkGet
BenchmarkGet-32<PHONE_NUMBER>               0.9563 ns/op
PASS
ok      github.com/petermattis/goid     1.076s

I suspect that's because Get itself is always inlined, so the only real function call is into getg.

Added CI coverage for inlining as well.


Use invoke in async trait, thread safety error

Hi,
thanks for this great library!
I was trying to use the invoke function in a trait implementation but received an error regarding thread safety.
Is this behavior intentionally?
See example and error below.
Cargo.toml
[package]
name = "tauri-sys-trait"
version = "0.1.0"
edition = "2021"

[dependencies]
async-trait = "0.1"
anyhow = "1"
serde = { version = "1", features = ["derive"] }

[dependencies.tauri-sys]
git = "https://github.com/JonasKruckenberg/tauri-sys"
branch = "main"
features = [
  "all",
]

main.rs
use async_trait::async_trait;
use serde::{Deserialize, Serialize};

#[derive(Serialize, Deserialize)]
struct Model;

#[async_trait]
trait Adapter {
    async fn model_list(&self) -> anyhow::Result<Vec<Model>>;
}

struct TauriAdapter;

#[async_trait]
impl Adapter for TauriAdapter {
    async fn model_list(&self) -> anyhow::Result<Vec<Model>> {
        tauri_sys::tauri::invoke::<_, Vec<Model>>("get_model_list", &())
            .await
            .map_err(|e| anyhow::anyhow!(e))
    }
}

fn main() {
    println!("Hello, world!");
}

Output:
[Running 'cargo lcheck --all-features']
    Checking tauri-sys-trait v0.1.0 (/tmp/tauri-sys-trait)
error: could not compile `tauri-sys-trait` (bin "tauri-sys-trait") due to 3 previous errors
error[E0277]: `Rc<RefCell<wasm_bindgen_futures::Inner>>` cannot be sent between threads safely
   --> src/main.rs:16:62
    |
16  |       async fn model_list(&self) -> anyhow::Result<Vec<Model>> {
    |  ______________________________________________________________^
17  | |         tauri_sys::tauri::invoke::<_, Vec<Model>>("get_model_list", &())
18  | |             .await
19  | |             .map_err(|e| anyhow::anyhow!(e))
20  | |     }
    | |     ^
    | |     |
    | |_____`Rc<RefCell<wasm_bindgen_futures::Inner>>` cannot be sent between threads safely
    |       within this `{async block@src/main.rs:16:62: 20:6}`
    |
    = help: within `{async block@src/main.rs:16:62: 20:6}`, the trait `Send` is not implemented for `Rc<RefCell<wasm_bindgen_futures::Inner>>`
note: required because it appears within the type `JsFuture`
   --> /.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-futures-0.4.39/src/lib.rs:101:12
    |
101 | pub struct JsFuture {
    |            ^^^^^^^^
    = note: required because it captures the following types: `wasm_bindgen_futures::JsFuture`
note: required because it's used within this `async` fn body
   --> /.cargo/git/checkouts/tauri-sys-af581fbb8e6f4ef1/55fe1d1/src/tauri.rs:95:5
    |
95  |     #[wasm_bindgen(module = "/src/tauri.js")]
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = note: required because it captures the following types: `ControlFlow<Result<Infallible, serde_wasm_bindgen::error::Error>, wasm_bindgen::JsValue>`, `impl Future<Output = Result<wasm_bindgen::JsValue, wasm_bindgen::JsValue>>`
note: required because it's used within this `async` fn body
   --> /.cargo/git/checkouts/tauri-sys-af581fbb8e6f4ef1/55fe1d1/src/tauri.rs:67:97
    |
67  |   pub async fn invoke<A: Serialize, R: DeserializeOwned>(cmd: &str, args: &A) -> crate::Result<R> {
    |  _________________________________________________________________________________________________^
68  | |     let raw = inner::invoke(cmd, serde_wasm_bindgen::to_value(args)?).await?;
69  | |
70  | |     serde_wasm_bindgen::from_value(raw).map_err(Into::into)
71  | | }
    | |_^
    = note: required because it captures the following types: `impl Future<Output = Result<Vec<Model>, tauri_sys::error::Error>>`
note: required because it's used within this `async` block
   --> src/main.rs:16:62
    |
16  |       async fn model_list(&self) -> anyhow::Result<Vec<Model>> {
    |  ______________________________________________________________^
17  | |         tauri_sys::tauri::invoke::<_, Vec<Model>>("get_model_list", &())
18  | |             .await
19  | |             .map_err(|e| anyhow::anyhow!(e))
20  | |     }
    | |_____^
    = note: required for the cast from `Pin<Box<{async block@src/main.rs:16:62: 20:6}>>` to `Pin<Box<dyn Future<Output = Result<Vec<Model>, anyhow::Error>> + Send>>`
    = note: this error originates in the attribute macro `wasm_bindgen` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: `*mut u8` cannot be sent between threads safely
   --> src/main.rs:16:62
    |
16  |       async fn model_list(&self) -> anyhow::Result<Vec<Model>> {
    |  ______________________________________________________________^
17  | |         tauri_sys::tauri::invoke::<_, Vec<Model>>("get_model_list", &())
18  | |             .await
19  | |             .map_err(|e| anyhow::anyhow!(e))
20  | |     }
    | |     ^
    | |     |
    | |_____`*mut u8` cannot be sent between threads safely
    |       within this `{async block@src/main.rs:16:62: 20:6}`
    |
    = help: within `{async block@src/main.rs:16:62: 20:6}`, the trait `Send` is not implemented for `*mut u8`
note: required because it appears within the type `PhantomData<*mut u8>`
   --> /.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/marker.rs:809:12
    |
809 | pub struct PhantomData<T: ?Sized>;
    |            ^^^^^^^^^^^
note: required because it appears within the type `JsValue`
   --> /.cargo/registry/src/index.crates.io-6f17d22bba15001f/wasm-bindgen-0.2.89/src/lib.rs:91:12
    |
91  | pub struct JsValue {
    |            ^^^^^^^
note: required because it appears within the type `ControlFlow<Result<Infallible, Error>, JsValue>`
   --> /.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/ops/control_flow.rs:85:10
    |
85  | pub enum ControlFlow<B, C = ()> {
    |          ^^^^^^^^^^^
    = note: required because it captures the following types: `ControlFlow<Result<Infallible, serde_wasm_bindgen::error::Error>, wasm_bindgen::JsValue>`, `impl Future<Output = Result<wasm_bindgen::JsValue, wasm_bindgen::JsValue>>`
note: required because it's used within this `async` fn body
   --> /.cargo/git/checkouts/tauri-sys-af581fbb8e6f4ef1/55fe1d1/src/tauri.rs:67:97
    |
67  |   pub async fn invoke<A: Serialize, R: DeserializeOwned>(cmd: &str, args: &A) -> crate::Result<R> {
    |  _________________________________________________________________________________________________^
68  | |     let raw = inner::invoke(cmd, serde_wasm_bindgen::to_value(args)?).await?;
69  | |
70  | |     serde_wasm_bindgen::from_value(raw).map_err(Into::into)
71  | | }
    | |_^
    = note: required because it captures the following types: `impl Future<Output = Result<Vec<Model>, tauri_sys::error::Error>>`
note: required because it's used within this `async` block
   --> src/main.rs:16:62
    |
16  |       async fn model_list(&self) -> anyhow::Result<Vec<Model>> {
    |  ______________________________________________________________^
17  | |         tauri_sys::tauri::invoke::<_, Vec<Model>>("get_model_list", &())
18  | |             .await
19  | |             .map_err(|e| anyhow::anyhow!(e))
20  | |     }
    | |_____^
    = note: required for the cast from `Pin<Box<{async block@src/main.rs:16:62: 20:6}>>` to `Pin<Box<dyn Future<Output = Result<Vec<Model>, anyhow::Error>> + Send>>`

[Finished running. Exit status: 101]

Yeah so apparently JsFuture (exported by wasm_bindgen_futures) which is the underpinning type of this crate does not implement Send which async_trait imposes on the methods.
As WASM (at least right now) is not multithreaded anyway having a Send requirement doesn't really make sense to begin with.
To solve this you can either use the "proper" async trait that's currently in nightly but will be stable with 1.75.0 since that doesn't require the Send bound,
or you disable the Send bound in the async_trait macro like shown here https://docs.rs/async-trait/0.1.74/async_trait/#non-threadsafe-futures

Ah awesome! Thanks for this detailed explanation!


Changes to accommodate for custom benchmarks in bayesian analysis

This will prevent MAP assignments to fall over due to NaNs in the benchmark series

You can probably fix this with git rm bayesian.py.

Not idea why it fails Travis CI build. The file display changes I made on my local copies to handle the problem

pyfolio/bayesian.py:96:54: E231 missing whitespace after ','
Note that we enforce pep8 in the code-base, you can run flake8 to test.
Not sure this is the right fix, however. Can you remove the find_MAP() call and just try to call pm.sample() (without anything else) with the current master of pymc3?

I've fixed the error you pointed out. As I said, I'm not a good coder at all and I'm only prototyping things but not ready for prime time at all.
As for your point about using the current code on master and just remove the MAP step when before sampling, the answer is that the whole model_return_t_alpha_beta() function simply won't work at all without my hack. If I didn't handle NaNs coming from index discrepancies in the returns series versus the chosen benchmark, the whole model would simply fall over even in calling np.linalg.lstsq(X, y)[0]. This is because X would include NaNs as the index in data would not be the same as the one in bmark. This isn't a problem with Quantopian supplied benchmark data (which if I'm not mistaken is coming from yahoo via pandas_datareader) but if I supplied my own index or had a series with a different holiday schedule than that of the benchmark I would end up with NaNs. The problem is evident when calling pm.sample() (with or without MAP) assignment but the reason why I mentioned MAP is because unless I fixed the data going into the optimiser I would get weird results when calling MAP. In other words, this isn't about MAP but a data issue that has repercussions throughout the code. I totally get the fact that there might be better ways to fix this but this is what I came up with and I haven't noticed weird alpha or beta values from running my version of the code.

Can't you call .ffill() before passing your data in?

I suppose the problem is that you have data for which no benchmark data exists. Do you have weekends in there or an odd start-time?
I think a better fix would be to do the intersection of benchmark and algo returns' index and use that.

Most of the problem has to do with US holidays not being in the FF and benchmark factors used by pyfolio but in my return series. Your suggestion makes sense but I thought that forward filling (and back fillling in case the first day is a NaN too) was a way of not losing those days when we have a missing benchmark data since I care more about performance statistics of my series being correct than slightly messing up those of the benchmarks.
At the end of the day, we should probably close the pull request since I realise how this solution is just a hack and I will let you know if I manage to figure out something better instead but I do think that I'm not the only one who has encountered a problem like this since not everybody only trades SPY and US based ETFs or similar. For example, I'm using futures traded in the US, Asia and Europe across multiple asset classes so I'm always going to have problems when matching against local benchmarks.

@bpajusco closing PR as requested. Please let us know if you figure out a better solution!
@gusgordon @twiecki making pyfolio support multiple asset classes across multiple countries is too much to do at once, but we should make sure pyfolio supports US futures, since that sounds like the next big internal use case.


test: Fix Karma debug.html support

To debug Karma tests, you can load debug.html in a browser.  However, this context has no back-channel to Karma and tries to load scripts by direct insertion into the document.  For compatibility, we need to have Closure inject scripts with the same methodology, to avoid having scripts load in the wrong order.  We also have to define the dump() method that Karma would normally inject into the testing context.

@gkatsev FYI

thanks! I'll give a try next week, at this point

Sadly, singleRun: false is incompatible with the useIframe: false and runInParent: true options.
If you can live with this for now, we are investigating alternatives to the now-dead Karma for the long term.
If you can't live with this for now... please send patches to my fork of Karma: https://github.com/joeyparrish/karma/tree/shaka-fixes


Bootstrap with docker-compose

Project bootstrap creation using Docker containing the following services from official images:

[x] nginx
[x] PHP 7.3 fpm
[x] MySQL

Requirements

[x] Volume to DB data folder
[x] DB dumps folder volume
[x] Database access data defined in .env file
[x] If need Dockerfile or other files for a service, put it in the .docker folder on root of project, for example: .docker/php7/Dockerfifle, .docker/mysql/Dockerfile
[x] docker-compose.yml file should be at project root

I made changes to the project infrastructure, temporarily removed the environment variables from docker-compose and installed via phl the data structure module of php. I also noticed that the ngnix container did not talk to the php container, linked both containers and created a www directory to point the application, separating infrastructure from the rest of the project. I updated the composer version because it is not compatible with the php version installed in the project (dockerfile)


Expectation refactor

I tried out the expectation stuff. It's really sweet.
This is where I mostly ended up with it.. what do you think?

I love it 😍  Looks really fancy! Thanks @paulirish


[Tabs] Remove deprecated fullWidth and scrollable props

[x] I have followed (at least) the PR section of the contributing guide.

How does this fit in with #15324? If this was actually useful behavior that can't be replicated with the current API we should probably revert the deprecation entirely?

AFAICT, only a couple of styles need to be applied to get the behaviour equivalent to both fixed and scrollable being set at the same time. However I can see the merit in consolidating the API changes to one change in v5, rather than one now, and another one later.


Test failure on Debian system

Hi,
I'm one of the Debian maintainers of scikit-bio.  There was a failure in the test suite reported.  Do you have any hint how to fix this?
Kind regards, Andreas.

Hi @tillea .  that looks like that issue is stemming from the deepcopy issues that we've seen in the newest release in pandas (see this PR thread).  The best way proceed with this is to only rely on the previous release of pandas ( before 0.20.3 ).


ChantModelTest.test_get_concordances behaving strangely

While working on #1422, I discovered some unusual behavior with ChantModelTest.test_get_concordances:
If I run the entire test suite, this test errors:
# python manage.py test main_app.tests
Found 328 test(s).
Creating test database for alias 'default'...
System check identified no issues (0 silenced).

...quite a lot of output trimmed...

======================================================================
ERROR: test_get_concordances (main_app.tests.test_models.ChantModelTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/code/django/cantusdb_project/main_app/tests/test_models.py", line 93, in test_get_concordances
    chant = Chant.objects.get(id=1)
  File "/usr/local/lib/python3.9/site-packages/django/db/models/manager.py", line 87, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/db/models/query.py", line 637, in get
    raise self.model.DoesNotExist(
main_app.models.chant.Chant.DoesNotExist: Chant matching query does not exist.

----------------------------------------------------------------------
Ran 328 tests in 27.896s

FAILED (errors=1, skipped=3)
Destroying test database for alias 'default'...

But if I run just test_models, the test passes:
# python manage.py test main_app.tests.test_models
Found 37 test(s).
Creating test database for alias 'default'...
System check identified no issues (0 silenced).
.....................................
----------------------------------------------------------------------
Ran 37 tests in 0.642s

OK
Destroying test database for alias 'default'...

It also passes when I run python manage.py test main_app.tests.test_models.ChantModelTest and python manage.py test main_app.tests.test_models.ChantModelTest.test_get_concordances.
fwiw, I'm observing this on the develop branch at d8a631bd2ca6107536191f6fbd568475309f41a2

Having noted the commit I was at, I realize that something weird has been going on with my git history. I'm confident I've been committing to my branch 1388-suggested-chants, yet these commits, as well as several made directly on GitHub, are appearing in my git log while I'm on my develop branch. Going to need to figure out what's going on here...


[add] Instructions to use the docker appbuilder in README

@xchapron-ledger I think this goes beyond this PR and maybe this repo even.


MODLD-583: LCCN Validation of Pattern

Hi @PBobylev, @askhat-abishev, @AndreiBordak,
In order to make the DTO validation approach consistent, I think it will be good to refactor the current PrimaryTitleDtoValidator to follow the approach in this PR. What do you think?

Hi @PBobylev, @askhat-abishev, @AndreiBordak, In order to make the DTO validation approach consistent, I think it will be good to refactor the current PrimaryTitleDtoValidator to follow the approach in this PR. What do you think?

Hi @pkjacob
I agree but, in my opinion, it should be done in the scope of another ticket


Passthrough USB HDD

Hi, I would like to know if it is possible to passthrough a physical USB external HDD connected to the Host to the docker via
volumes

/dev/sdb:/xxx

Thanks Kroese for your help.
Whenever I add the device to the docker-compose config file, docker does not work. The log says:
❯ Starting Windows for Docker v1.03... ❯ For support visit https://github.com/dockur/windows ❯ ERROR: KVM acceleration not detected (device file missing), this will cause a major loss of performance. ❯ ERROR: See the FAQ on how to enable it, or continue without KVM by setting KVM=N (not recommended).
This is my configuration:
  windows:     image: dockurr/windows     container_name: windows     devices:       - /dev/kvm     cap_add:       - NET_ADMIN     networks:       - proxy     ports:       - 8006:8006     stop_grace_period: 2m     restart: unless-stopped     volumes:       - /opt/containers/windows:/storage     devices:       - /dev/sdb     environment:       RAM_SIZE: "8G"       DEVICE2: "/dev/sdb"

Yes, it was devined devices twice, grouping then in only one, now it is working and HDD is detected and accessed in Windows.
Thanks!


Proxy-936

Related MR: https://github.com/Lumerin-protocol/WalletCore/pull/20

Pull changes from dev
Increase version of wallet-core to 1.0.15 (package.json > line 41)


Same dapr components in multiple apps

consider a scenario, where I want to publish one message from one app to other using dapr pub/sub, from one microservice app I am publishing the message and in other microservice I am consuming same message using Topic sub, now first when I try to deploy subscription app using az containerapp create with enable-dapr true and dapr-components same as of publisher, azure container apps wont allow me to create another app with same components, now when I remove dapr-components part, and link my subscription client to publishing client using AddDapr(usgrpc and usehttpclient), it gives an error that there are no dapr- components, now I am confused how to architect my application if other application cant have its own components

Hi @SarangRapid thanks for trying Container Apps + Dapr and reporting this.
If I understand this right you'd like to use the same Pubsub component in both a publisher containerapp and a subscriber containerapp.  And these containerapps are in the same containerapp environment?
This should work so long as both container apps both enable-dapr and also get created with the component config for the same Pub-sub.  component.  Can you share exactly the command you used to deploy both container apps?
Note a new feature is coming which allows shared Dapr components at the environment level of container apps.
@greenie-msft  could you please have a look at this?  Maybe we should move to a better repo too?

@paulyuk and @greenie-msft Should we close this if we already create an issue in the other document?

Yes @amulyavarote we should close.  I believe this is resolved in the GA version of container apps.  Dapr components are global to the whole environment by default unless you further scope them.  This is in essence "fixed"!  FYI @SarangRapid

Fixed!


Make sure to support all of the terms in ConstructorPattern in Expr

The reason for 3-4 days here is, it may end up in shuffling a bit of code, some cleanups and writing tests. I may have over estimated. But I am keeping it to be 5

Expecting only 2 days, but giving it estimate of 5 making it 3 or 4 days, to give a space for cleanup and necessary refactoring and writing proper tests


No ECID when opt-in after opt-out in Identity shared state

Expected Behaviour
If app opt-in after opt-out, app should receive ECID in identity shared state
Actual Behaviour
If app opt-in after opt-out, Identity shared state doesn't contains the ECID. When opt-in is done second time then Identity shared state contains ECID.
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

@shtomar-adb I ran our sample app and when I opt-in after being opt-out I see a new ECID included in the Identity shared state.
Here is the shared state after changing privacy to opt-out, which does not contain an ECID as expected.
2021-05-28 10:55:45.724025-0600 AEPSampleApp[21587:11571142] [AEP SDK DEBUG - <EventHub>] Standard shared state created for com.adobe.module.identity with version 91 and data:
{
  "lastsync" :<PHONE_NUMBER>.3747649
}

Here is the log from Identity after changing to opt-in, which contains the ECID:
2021-05-28 10:56:49.766038-0600 AEPSampleApp[21587:11571227] [AEP SDK DEBUG - <EventHub>] Standard shared state created for com.adobe.module.identity with version 96 and data:
{
  "blob" : "j8Odv6LonN4r3an7LhD3WZrU1bUpAkFkkiY1ncBR96t2PTI",
  "lastsync" :<PHONE_NUMBER>.7656798,
  "mid" : "03236312170745520675786750585880425528",
  "locationhint" : "9"
}

I'm not able to reproduce this issue. Please provide some steps to reproduce this issue or create a sample app that demonstrates it. Logs for this issue would also be helpful.


Open links in New Tab by default instead of same tab.

Checklist

[x] I have read the contributing guidelines.
[x] There is no similar request on existing issues (including closed ones).
[x] I have read the project progress and know the current progress of the project.
[x] I was in the master branch of the latest code.

Is your feature request related to a problem? Please describe
Let say I am reading a blog and I click a link mentioned there. The link is opened in the same tab, but the user was reading the content. So, it would a great if the link is opened in a new tab by default.

You can use IAL to achieve this.
For an example:
Click [here](https://github.com){:target="_blank"} to open Github.

You can use IAL to achieve this.
For an example:
Click [here](https://github.com){:target="_blank"} to open Github.

I am creating the blog from some of my markdown notes. It would be a lot of work to change all the links like that.

There doesn't seem to be such an option inside Jekyll to configure. You can do this with the plugin jekyll-target-blank.


wslg crashed when I want to take a screen shot and window flashed for a while

Environment
Windows build number: Microsoft Windows [Version 10.0.21390.2025]
Your Distribution version: Release:        20.04
Your WSLg version: 1.0.22

Steps to reproduce

display xxx.ppm
take a screen shot by a third-party software (such as,QQ)
the imagemagick crashed, and the window flashed for a moment (I can see the background of windows terminal become transparent and not transparent, and I will lose the focus when I type if the window flashed)

WSL logs:

Attach WSLg logs from  /mnt/wslg

You can access the wslg logs using explorer at: \\wsl$\<Distro-Name>\mnt\wslg (e.g.: \\wsl$\Ubuntu-20.04\mnt\wslg)

puseaudio.log
weston.log
versions.txt
pulseaudio.log
versions.txt
weston.log

Expected behavior

I used imagemagick to display this window and I want to take a screenshot.
Actual behavior

I took the screenshot and found wslg crashed.

Your WSLg version: 1.0.22

It could have been fixed already so try installing latest WSLg version 1.0.24 from https://github.com/microsoft/wslg/releases/download/v1.0.24/wsl_graphics_support_x64.msi

I installed the latest version and it still crashed.
when@When:/mnt/e/project/C/opengl/smallpt$ display image.ppm
X connection to :0 broken (explicit kill or server shutdown).

@WhenMelancholy, do you see any core dumps at /mnt/wslg/dumps? thanks!

@WhenMelancholy, do you see any core dumps at /mnt/wslg/dumps? thanks!

core.weston.zip

@WhenMelancholy, thank you for core dumps, but due to our side of issue, we haven't enabled debug info in past release of WSLg, would you please collect dump again with WSLg 1.0.26 at https://github.com/microsoft/wslg/releases/tag/v1.0.26 ? thanks you very much for trying out WSLg!

@WhenMelancholy, do you see any core dumps at /mnt/wslg/dumps? thanks!

new-core-weston.zip
By the way, I found my issue was the same as #348 , since I was using QQ to take screenshot.


Getting access to the iframe right after instantiation of socket

In my app I'm using easyXDM.Socket to communicate between iframes on different domains. And the iframe generated by easyXDM.Socket is the iframe the user is navigating in. One problem that has annoyed me is that the iframe is not generated right away, and thus I can't add events to it, manipulate it, etc. So I can't write stuff like:
this.socket = new easyXDM.Socket({
	remote: this.url,
	container: this.container,
	onMessage: function() {
		console.log("got message");
	}
});

var iframe = this.container.getElementsByTagName('iframe')[0];
// do stuff with iframe

That is because I have to wait for the onReady event which is called when the iframe is created. Wouldn't an option be that you can manually specify an iframe element yourself?
var iframe = document.createElement('iframe');
var socket = new easyXDM.Socket({
	iframe: iframe,
	remote: "http://foo/bar",
	onReady: function() { /* iframe is prepped and ready for communication */ }
});

Sorry, but the iframe requires special properties, eg the name, and due to
issues doing this cross-browser, I simply do not trust users to get it right
:-)
But is there an issue waiting for the onReady event? Also, what kind of
events are you planning on listening for on the iframe? The only one that
could be of any use is probably 'load', and you can easily wait for onReady
to attach this.
On Oct 13, 2011 2:47 AM, "Erkie" <
<EMAIL_ADDRESS>wrote:

In my app I'm using easyXDM.Socket to communicate between iframes on
different domains. And the iframe generated by easyXDM.Socket is the iframe
the user is navigating in. One problem that has annoyed me is that the
iframe is not generated right away, and thus I can't add events to it,
manipulate it, etc. So I can't write stuff like:
   this.socket = new easyXDM.Socket({
           remote: this.url,
           container: this.container,
           onMessage: function() {
                   console.log("got message");
           }
   });

   var iframe = this.container.getElementsByTagName('iframe')[0];
   // do stuff with iframe

That is because I have to wait for the onReady event which is called when
the iframe is created. Wouldn't an option be that you can manually specify
an iframe element yourself?
   var iframe = document.createElement('iframe');
   var socket = new easyXDM.Socket({
           iframe: iframe,
           remote: "http://foo/bar",
           onReady: function() { /* iframe is prepped and ready for

communication */ }
});
--
Reply to this email directly or view it on GitHub:
https://github.com/oyvindkinsey/easyXDM/issues/130

That is understandable! So you can't just change the name when it's passed along? Or is that an attribute that you can't change once created?
It's just that it makes the API a lot simpler from my point of view. It's just that if any code is dependent on an iframe existing, it has to be moved into a callback, which is possible, but I see it as unnecessarily complicated. I have used easyXDM in two projects, and in both cases the iframe-not-existing-but-existing-after-a-while had to be worked around.

'name' is a special property in IE and cannot be changed externally by doing iframe.name = ....

I understand. But why isn't the iframe created in the constructor directly? Why does it wait for onload?

Because the iframe is created by a specific transport behavior, and this can also be lazy-loaded.
But I really don't see an issue with having to wait for onReady...


Issue with IE < 11

In IE 10 and lower I get a blank screen, it looks like IE is changing

to

Is there an easy fix for this?
Thanks

Is it IE doing that or something in your build/setup? You could always name your views but it shouldn't be required.

I think the issue was unrelated to ui-router

doesn't seem to be a problem.

@stlouisweb did you ever find out the cause for this?

Please reply if you ever solved this problem

No, I moved on to React


Inference.lagda.md doesn't load

I got the following error when trying to load "src/plfa/part2/Inference.lagda.md":
/home/shreckye/GitProjects/plfa/src/plfa/part2/Inference.lagda.md:583,5-26
Could not parse the application ¬ ∃[ A ] (Γ ∋ x ⦂ A)
Operators used in the grammar:
  ¬    (prefix operator, level 3) [¬_ (/home/shreckye/GitProjects/agda-stdlib/src/Relation/Nullary.agda:26,1-3)]
  ∃[_] (prefix notation, level 2) [∃-syntax (/home/shreckye/GitProjects/agda-stdlib/src/Data/Product.agda:80,1-9)]
when scope checking ¬ ∃[ A ] (Γ ∋ x ⦂ A)

It seems to be casused by ¬_ having higher precedence than ∃-syntax and parentheses should be added.

I don't get the problem you report when loading on my machine. Do you have the version of Agda specified in "Getting Started"?

Sorry about this. I just checked and I do have a different version of Agda. I have the latest Agda version 2.6.2, which is different from <IP_ADDRESS> specified in "Getting Started". My stdlib version is v1.6 as specified.

@ShreckYe Can you confirm this is resolved after switching to the correct Agda version?

OK. I switched back to <IP_ADDRESS> and the problem is still there. I installed Agda using Cabal. Can this be why it happens?

I think I remember fixing a bunch of problems just like this a long time ago, when trying to upgrade to the most recent version of Agda. I think something did change in the standard library.
Apologies, the book is in a slight state of disrepair because I’ve been dealing with carpal tunnel for the past half year. In the meantime you can fix it locally by simply changing the parentheses.