Configuration Management

Description

Implements configuration management.
course types, countries, languages, validation, current semester are now fetched from the backend
implements frontend validation

Dependency update

List all new dependencies (delete if no additions)

Type of change (remove all that don't apply)

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Manually tested all use cases this could alter

Test Configuration:

OS: Windows Mac Linux

Browser: Firefox Chrome Safari Chromium

Frontend: (remove all that don't apply)

[X] Development build
[ ] Production build
[ ] Dockerized production build

Backend: (remove all that don't apply)

[X] deployed development build

Checklist: (remove all that don't apply)

[X] I added corresponding E2E tests (especially for bugfixes)
[X] I added corresponding unit tests (if API code)
[X] I have performed a self-review of my own code
[ ] My changes generate no new linting warnings or console warnings
[ ] I have updated the CHANGELOG.md to include any changes made in this PR (add WIP to the top, if there is none already)

the merge introduced some ugly styling

the merge introduced some ugly styling

this should be fixed

Consider closing #354 on merge (or connecting it)

Overall that is great work.
For the E2E we should propagate the id from the parent component into the base-input component for properly getting them via Cypress. Otherwise, it looks like the following:

I would suggest something like
In the parent:
<base-input identifier="WhatEverIdYouWant" ..../>
and stting the input field's id via the prop in the BaseInput.vue
<input :id="identifier" .../>
Additionally we must not forget to rework the error message assertation in the E2E tests

should be fixed now


Reuse server renderer state on the client side

Obviously this doesn't provides any state. However it enables everybody to do this by their own.
With this change initially provided styles will not get overwritten.
How to achieve state sharing with it?
There are two ways to reuse/recreate the state on the client-side. The first one is a little hacky and could lead to performance issues in the browser, however it reduces network traffic:
1. Counting _emitChange calls
On the server-side I'm counting all _emitChange calls. When I have got the same styles rendered on the server and on the client, the number of _emitChange calls is the same. So after passing the state to the client-side renderer it ignores the first n _emitChange calls and does not apply the styles initially to the mount node.
So the styles do not get unnecessary rerendered in the users browser, which improves the performance and user-experience.
To achieve this you need to overwrite the fela ´renderer._emitChange´ function like this to count all it's calls:
let counter = 0
let renderer = createRenderer(config)
let emitChange = renderer._emitChange
renderer._emitChange =  () => {
   counter++
   emitChange()
}

Then you need to pass the counter number to the client somehow.
For example you can insert this into your html:
<script>var __GLOBAL__.felaState = ${counter};</script>

On the client side also need to overwrite the _emitChange method:
let offset = __GLOBAL__.fetaState
let renderer = createRenderer(config)
let emitChange = renderer._emitChange
renderer._emitChange =  () => {
   if(offset > 0){
      offset--
      return
   }
   emitChange()
}

This cancels the first n _emitChange calls.

Note: The mountNode has to be the same as you rendered the static css into at the server-side.

2. Transport the hole renderer
This causes a lot more network traffic. Basically the styles will be send 3 times. (Static styles 4 times)
From the server you just return the hole renderer somewhere like this:
<script>var __GLOBAL__.felaState = ${JSON.stringify(renderer)};</script>

At the client-side you have to overwrite some properties of the new renderer:
let renderer = createRenderer(config)
renderer.fontFaces = __GLOBAL__.felaState.fontFaces
renderer.keyframes = __GLOBAL__.felaState.keyframes
renderer.statics = __GLOBAL__.felaState.statics
renderer.rules = __GLOBAL__.felaState.rules
renderer.mediaRules = __GLOBAL__.felaState.mediaRules
renderer.rendered = __GLOBAL__.felaState.rendered
renderer.base = __GLOBAL__.felaState.base

No matter which way you choose this PR is necessary for both. Hope you understand and merge ;).
Cheers
Dustin

@dustin-H We discussed going with #2, but in a less hacky way, by providing a proper way of hydrating the renderer. I'm sure there is room for optimizing what gets sent down the wire, and we might not need to send everything if we can derive the complete state from several sub-states.
That said: gzip will most likely compress things a lot so I wouldn't worry about it too much for now.
I will leave the merge-decision for this one to @rofrischmann.


REFACTOR-#1938: avoid index access in a simple column reference

What do these changes do?
This patch allows column references without index access which is essential for lazy engine.


Save data corruption problem (The Outsider's Issue)

The Outsider does not separate its ship log data from original game's one.
So, the save data gets to be corrupted if The Outsider is disabled with unread its ship log.
Now, Vanilla Fix is dealing with it, but if you disable Vanilla Fix, the problem still happens.
We need the fundamental solution for it.
c.f.

So, I checked the problem really happens.

Install the latest The Outsider, Translation and Patches for The Outsider, and Vanilla Fix.

Make another profile, and make SHIP LOG UPDATED in The Outsider.

Quit the game without opening the computer in your ship.

Disable The Outsider and Vanilla Fix

Then, go to your ship. The computer is still blinking even though there are NOT newly found information in your base game.

Open the computer, and you will get a broken ship log, which does not have any entries. You cannot exit from viewing your computer unless quitting the game. After quitting the game, the title screen cannot appear...

We can get below error log:
NullReferenceException: Object reference not set to an instance of an object
Stacktrace: ShipLogController.BuildRevealQueue () (at <8585b6cb325949f692883e4747187a22>:0)
ShipLogController.EnterShipComputer () (at <8585b6cb325949f692883e4747187a22>:0)
ShipLogController.OnPressInteract () (at <8585b6cb325949f692883e4747187a22>:0)
SingleInteractionVolume.UpdateInput () (at <8585b6cb325949f692883e4747187a22>:0)
SingleInteractionVolume.UpdateInteractVolume () (at <8585b6cb325949f692883e4747187a22>:0)
FirstPersonManipulator.Update () (at <8585b6cb325949f692883e4747187a22>:0)

After quitting, we can get below error log:
NullReferenceException: Object reference not set to an instance of an object
Stacktrace: ShipLogManager.ClearNewlyRevealedFacts () (at <8585b6cb325949f692883e4747187a22>:0)
ShipLogController.OnDestroy () (at <8585b6cb325949f692883e4747187a22>:0)

This situation is not a rare case because The Outsider updates the ship log when arriving at The Eye.
Not a few players would disable The Outsider (and Vanilla Fix) without seeing updated information about the Eye of the universe after seeing the ending.
It leads to this issue, so we have to fix it.

I did it!!! I introduce NewHorizonsData.cs from New Horizons to save newly revealed fact mod ids in save.json separated from original save data.
So, I checked it works:

With v1.0.2 Translation and Patches for The Outsider, I got SHIP LOG UPDATED in The Outsider.

Quit the game without opening the computer.

Upgrade to v1.1.0 Translation and Patches for The Outsider and open the game.
The computer is still blinking, but the newly revealed fact mod ids are moved to save.json in orclecle.TranslationForTheOutsider. You got the log: newly revealed fact id: PS_POWER_STATION_X1 is found in the base game save file, so it is moved to the mod save file.

Quit the game without opening the computer.
Disable The Outsider, Translation and Patches for The Outsider, and Vanilla Fix. Then, open the game.
The computer is NOT blinking, and you can open the computer without any problems.

Quit the game.
Enable The Outsider, Translation and Patches for The Outsider, and Vanilla Fix (, but Vanilla Fix is not required now). Then, open the game.
The computer is BLINKING! You can check the newly revealed fact!

Of course, when you get SHIP LOG UPDATED with v1.1.0 Translation and Patches for The Outsider, the newly revealed fact mod ids are saved in save.json in orclecle.TranslationForTheOutsider.
So, the issue is FIXED!


Support Python 3.8

A bug tracker for people to follow up the initiative of this pull request draft[1] easily by searhing Issues.
[1] https://github.com/solvcon/solvcon/pull/235#issuecomment-828477913

This CI run by a draft codebase shows SOLVCON is built and run with python 3.8. https://github.com/tai271828/solvcon/actions/runs/1088314972
Unfortunately, the test for macOS failed. I have not figured out why. Need to have a closer look.

The numpy binary suggests to use AVX512, but it does not build on the CI machine: https://github.com/tai271828/solvcon/runs/3215888293?check_suite_focus=true#step:8:1048

I tested locally with Python 3.7 and self-built numpy and it passed, but I don't know when I will have time to build with newer Python.

I tested locally with Python 3.7 and self-built numpy and it passed, but I don't know when I will have time to build with newer Python.

Uha, this is indeed informative. Let me try to include self-built numpy in the process as well. It's really good to know Python 3.7 works.

Current status: it is ready to support python 3.8 with Ubuntu, but not macOS. Running SOLVCON with python 3.8 of devenv and macOS is currently blocked by https://github.com/solvcon/devenv/issues/83

Fixed. Now it's at the stage of SOVLCON build error macOS https://github.com/tai271828/solvcon/runs/3580950845?check_suite_focus=true#step:9:1176 . We moved forward \o/
My quick skim shows it may be libc compatibility. @yungyuc would you mind to have a look?

Current status: it is ready to support python 3.8 with Ubuntu, but not macOS. Running SOLVCON with python 3.8 of devenv and macOS is currently blocked by solvcon/devenv#83

@tai271828 at https://github.com/tai271828/solvcon/runs/3580950845?check_suite_focus=true#step:9:1106 , it seems a Cython module is not successfully built because of a deprecated declaration in Python:
creating build/temp.macosx-10.9-x86_64-3.8/solvcon/src
compile options: '-Isolvcon -Isolvcon -I/Users/runner/work/solvcon/solvcon/build/env/install/lib/python3.8/site-packages/numpy/core/include -Ibuild/src.macosx-10.9-x86_64-3.8/numpy/distutils/include -I/Users/runner/work/solvcon/solvcon/build/env/install/include/python3.8 -c'
extra options: '-Werror -Wno-#warnings -Wno-unused-function -msse -msse2 -msse3'
gcc: solvcon/mesh.c
gcc: solvcon/src/sc_mesh_build_rcells.c
gcc: solvcon/src/sc_mesh_extract_faces_from_cells.c
gcc: solvcon/src/sc_mesh_build_ghost.c
solvcon/mesh.c:6620:3: error: 'tp_print' is deprecated [-Werror,-Wdeprecated-declarations]
  0, /*tp_print*/
  ^

Next action:
Figure out why the python 3.8 running in Ubuntu did not complain the deprecated declaration.

@yungyuc I will keep working on this issue. May you assign the issue to me? I did not manage to assign myself : (  (no permission?)

Currently we are working on this known issue https://github.com/cython/cython/issues/3474

@yungyuc I will keep working on this issue. May you assign the issue to me? I did not manage to assign myself : ( (no permission?)

I have changed the permission for you.  Could you check if you can change the settings?

@yungyuc I will keep working on this issue. May you assign the issue to me? I did not manage to assign myself : ( (no permission?)

I have changed the permission for you. Could you check if you can change the settings?

Yes. I have assigned myself : D
btw I could now build cython for mac with devenv with a specific python interpreter. I believe I am on the way to close this bug... : )

@yungyuc I will keep working on this issue. May you assign the issue to me? I did not manage to assign myself : ( (no permission?)

I have changed the permission for you. Could you check if you can change the settings?

Yes. I have assigned myself : D
btw I could now build cython for mac with devenv with a specific python interpreter. I believe I am on the way to close this bug... : )

On the way to success.  Sounds so good :)

The first step of the house cleaning solvcon/devenv#86 I need to update the codebase of devenv because I would like to use devenv to build SOLVCON and make the approach a part of the CI.

Using devenv to build is a big step!  Super!

I create another devenv pull request to get devenv ready to support/resolve this issue. https://github.com/solvcon/devenv/pull/88


Link to sample downloads is incorrect

View or download sample code points to https://github.com/dotnet/AspNetCore.Docs/tree/master/aspnetcore/common, but that doesn't seem to be right.

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

ID: 81a32115-33f8-7d06-aaf0-b8ce918b5527
Version Independent ID: 31ad18df-1904-bbdd-cfed-8d0d0267c10e
Content: Create and use ASP.NET Core Razor components
Content Source: aspnetcore/blazor/components/index.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @serpent5 ... I'll fix it right now. It escaped my re-linking efforts when we did the ToC shakeup.


progress > text > percentage > wrong values

hi,
go to: image > progress > text > percentage > remove the "show total value". and apply.
watch the values, they are wrong, displaying in THOUSANDS.

Right, it's assuming the current installed culture of the UI.

sorry, v2.13, problems remain.
see attached animated gif:
top-left is: percentage > remove the "show total value"
bottom-right is: custom > $p%
plus - ux wise - when showing percentage there is no need to show fractions, e.g. 32.4... 76.9..
it is too much information. showing integers is better.

Hi @NickeManarin - just FYI, there's still the issue of the %progress being out by 100×.
However, I disagree with the suggestion that the value be truncated to zero decimal places, since the fraction is relevant if the project is >100 frames.  Perhaps the number format could be if (frames>100) {"0.0%"} else {"0%"}?  (The user could still use the custom format for a truncated value, or vice-versa.)

@Shemesh - In the meantime you can also display the progress percentage (without the total) like this:

[x] Alt+K (or click Image → Progress), and set:

Setting
Value

Type
Text

Precision
Custom

Format
$p%

[x] Then, hit Enter  to apply the change (or click Apply.)

[ ] Other custom formats can be viewed by hovering your mouse over the Format box.

In case you couldn't tell, I'm just learning (and practicing) GitHub-flavoured markdown...  Forgive me!  :nerd_face:

Ok, I was able to fix this.


Labels are getting hide after set outerRadius props

Problem
Hi! I was trying to fit the chart in my template because of the labels was overflowing the container and get cropped. So I try to set the outerRadius prop and the chart gets a little small like I want but the Labels get hide.
Code
import React, { useState } from "react";
import "./styles.css";
import ChordDiagram from "react-chord-diagram";

export default function App() {
  const [state, setState] = useState([
    [0, 9, 14, 16, 24, 18, 7, 21, 7, 6],
    [12, 0, 17, 16, 13, 23, 4, 16, 10, 8],
    [6, 13, 0, 6, 1, 8, 5, 8, 10, 14],
    [30, 15, 3, 0, 32, 13, 3, 53, 6, 8],
    [38, 28, 17, 54, 0, 27, 50, 138, 3, 4],
    [10, 7, 10, 13, 8, 0, 4, 8, 9, 8],
    [13, 12, 11, 16, 21, 23, 0, 31, 7, 8],
    [43, 22, 10, 45, 105, 47, 56, 0, 3, 8],
    [2, 3, 0, 0, 0, 0, 0, 0, 0, 6],
    [0, 0, 0, 2, 0, 0, 0, 0, 3, 0]
  ]);

  return (
    <div className="App">
      <ChordDiagram
        className="chartDiagram"
        matrix={state}
        componentId={1}
        groupLabels={[
          "Ignizione ON",
          "Movimento ON",
          "Movimento OFF",
          "LBL1",
          "LBL2",
          "Tagliando1",
          "Tagliando2",
          "Tagliando3",
          "Tagliando4",
          "Tagliando5"
        ]}
        groupColors={[
          "#2255A4",
          "#006629",
          "#15295E",
          "#790D1A",
          "#FFE000",
          "#060606",
          "#0F793B",
          "#15295E",
          "#B40224",
          "#13803F"
        ]}
        disableRibbonHover={false}
        disableGroupHover={false}
        outerRadius="150"
        innerRadius="140"
        blurOnHover={true}
        ribbonOpacity=".9"
        ribbonBlurOpacity="0.2"
        style={{
          fontFamily: "Sans serif",
          fontSize: "12px",
          color: "#333"
        }}
      />
    </div>
  );
}

Question
Am I doing something wrong or there is any solution for this?

Hello!
In the example above, looks like outerRadius and innerRadius are passed as strings. If you pass them as numbers, it should fix the labels:
outerRadius={150}
innerRadius={140}
Hope that helps!

Thank you! It works fine.


Fixed annoying warnings

Warnings used to pop up when the extension loads, this fixes that. Also updated the release date on the changelog to September 23.

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #14049 into release-2020.09 will decrease coverage by 0.02%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           release-2020.09   #14049      +/-   ##
===================================================
- Coverage            59.87%   59.85%   -0.03%
===================================================
  Files                  692      692
  Lines                38337    38337
  Branches              5513     5513
===================================================
- Hits                 22955    22946       -9
- Misses               14199    14205       +6
- Partials              1183     1186       +3

Impacted Files
Coverage Δ

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/common/utils/platform.ts
60.00% <0.00%> (-8.00%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

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 c1258f0...126ba52. Read the comment docs.


Split definition/lambda blocks into two parts

Currently the definition/lambda block is a mixup of conceptual different things.
The outer part is like a normal block with an output anchor.
And the inner part is a 'container' of blocks and has internal binder/result anchors.
With this split case blocks could easier reuse parts, and folding up a local definition will be simply hiding the inner part.

PR #262 makes a start with this, but needs to be generalized for case blocks, and how this fits in the ui/model split where blocks know their container.

Resolved in PR #286 / case-block branch.


Added React TDD examples with react-testing-library

What:
Added React TDD examples with react-testing-library

Why:
Shows how to use react-testing-library when doing TDD in React

Checklist:

[x] Documentation
[ ] Tests N/A
[x] Ready to be merged
[x] Added myself to contributors table

Your branch was outdated and had a conflict with the contributors list. You'll need to pull and run npx kcd-scripts contributors generate then commit and push the changes :+1: Thanks!

@aputhin must have pushed the code seconds before me ^_^
Should be fine now


freedv: <IP_ADDRESS> -> 1.8.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/drowe67/freedv-gui/releases.
meta.description for freedv is: Digital voice for HF radio
meta.homepage for freedv is: https://freedv.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8.11 with grep in /nix/store/gb7xgjd9icz94q56d38n5sry3fddgjsq-freedv-1.8.11
found 1.8.11 in filename of file in /nix/store/gb7xgjd9icz94q56d38n5sry3fddgjsq-freedv-1.8.11

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
freedv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gb7xgjd9icz94q56d38n5sry3fddgjsq-freedv-1.8.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A freedv https://github.com/r-ryantm/nixpkgs/archive/7cad643c59bcb347dadf10c52e2d8bbd911ed544.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gb7xgjd9icz94q56d38n5sry3fddgjsq-freedv-1.8.11
ls -la /nix/store/gb7xgjd9icz94q56d38n5sry3fddgjsq-freedv-1.8.11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

freedv

Maintainer pings
cc @illdefined @wegank for testing.

Result of nixpkgs-review pr 238213 run on aarch64-darwin 1

1 package built:

freedv


Data Table: Sortable don't work as expected

This is how I think sortable work:
columns = [       {         name: 'column1',         label: 'Column 1',         sortable: true       },       {         name: 'column2',         label: 'Column 2',         sortable: false       } ]
This isn't working, only works globally. If sortable false the arrows disappear and therefore it's no longer possible to sort the data.
Am I doing anything wrong? What is the expected behaviour? Is this supposed to work only globally or in the way I was expecting it to work, as I show in my code?

could u explain the situation by using a small plnkr? I dont really understand what u mean with "only works globally".

columns = [ {
name: 'column1', label: 'Column 1', sortable: true
},{
name: 'column2', label: 'Column 2', sortable: false
} ]

Hi @jotatoledo, I really don't think it is necessary to create a pnkr for this. As you can see in the line 2 I have sortable as true and in the line 4 I have sortable as false. This should work but it doesn't.
What I mean with globally is that is possible in the HTML to do like in this example:
<td-data-table
  #dataTable
  [data]="filteredData"
  [columns]="columns"
  [sortable]="true"
  [sortBy]="sortBy"
  [sortOrder]="sortOrder"
  (sortChange)="sort($event)">
</td-data-table>
I can specify that the all table is sorted or not.
My question is simple, can I individually say that one column is sortable and an other is not?

Be a good citizen and respect others' time when you're asking for help. Create the Plunker

@kyleledbetter I didn't mean to disrespect noone and I haven't done that. Isn't my question clear enough to avoid the Plunker?

Hey! Now I understand what u meant. Yes you can, in the documentation there is an example with a non sortable column. Take a look here. I cant say what would happen on that example if u still set the sortable attribute of the table. However, by reading the description of ur problem I guess the table attribute will override the individual column sortable property.

Yep, the table [sortable] overrides the column ones.
So if you need to remove the [sortable] input from the table or set it as false.

Thank you for all your help.
I shifted my table to use Data Table (Atomic) Components and it works properly now.


Fix to facilitate old LinkedIn URL

Some people still have their LinkedIn URL in the old fashion.
That is, instead of having a username-based URL i.e http://linkedin.com/in/username, their url is still http://linkedin.com/pub/firstname-lastname/some-random-character/etc
For example: https://www.linkedin.com/pub/greg-street/7/644/b35 or
my old linkedin url before I modify it: https://www.linkedin.com/pub/fadil-sutomo/10/8aa/a5b
In summary, the current format of simply supplying LinkedIn username might not work for some people

Thanks for the pull request @fadils, much appreciated.
This is pretty opinionated, but I think that anyone including a link to their LinkedIn probably uses the new format... and if they don't, they really should, as it's a lot nicer. I noticed you updated yours to the new format too, @fadils :)
Going to keep as it, but thanks for bringing this up.

I did change mine. =)
Thanks for the good work. Keep it up.
Best Regards,
Fadil Sutomo
On Jan 12, 2015 1:02 AM, "Barry Clark"<EMAIL_ADDRESS>wrote:

Thanks for the pull request @fadils https://github.com/fadils, much
appreciated.
This is pretty opinionated, but I think that anyone including a link to
their LinkedIn probably uses the new format... and if they don't, they
really should, as it's a lot nicer. I noticed you updated yours to the new
format too, @fadils https://github.com/fadils :)
Going to keep as it, but thanks for bringing this up.
--
Reply to this email directly or view it on GitHub
https://github.com/barryclark/jekyll-now/pull/63#issuecomment-69503893.


Get status code from LightCouch

Hi! How can I get status code from LightCouch in Android and use status code to send a message to  user about action he wants (if it is OK or not, if it is created or not and so on)?

The API throws specific exceptions to denote failure responses, for example org.lightcouch.NoDocumentException corresponds to 404 error code.


Fixed the build with the latest rust-nightly.

(rustc 1.0.0-nightly (92ff8ea52 2015-01-27 23:08:13 +0000))

There were the following issues with your Pull Request

Commit: afa8a1b314640b1e5d98d0d343aaa187153dc34c
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

There were the following issues with your Pull Request

Commit: f1523324c8dc45b859353c6b9d1498aa1771f68f
Your commit message body contains a line that is longer than 100 characters

Guidelines are available at https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

:+1:

Er, looks like travis is finding problems.

Yes, I only fixed the build. Not sure how to fix the tests, though :(

No problem, I'll pull your commit and fix the rest.

Cheers! :+1:

Filed all the fixes in #287 . Thanks!


textDecorationThickness error reported using @stylexjs/eslint-plugin

The problem
When using the following StyleX code;
textDecorationThickness: {
    default: '1px',
    ':hover': 'max(3px, 0.1875rem, 0.12em)'
}

@stylexjs/eslint-plugin reports This is not a key that is allowed by stylex  @stylexjs/valid-styles despite the styling being correct and working.

i have the same problem, also the queries variables dont work, had to manually set every property.

This is a known issue. Duplicate of #135.
Added textDecorationThickness to list of missing properties. Will fix this tomorrow.

While #135 fixed the main issue it still throws this eslint error
error  textDecorationThickness value must be one of:
a number ending in px, mm, in, pc, pt
a number ending in ch, em, ex, ic, rem, vh, vw, vmin, vmax, svh, dvh, lvh, svw, dvw, ldw, cqw, cqh, cqmin, cqmax
null
initial
inherit
unset
revert  @stylexjs/valid-styles

for the hover line.
textDecorationThickness: {
      default: '1px',
      ':hover': 'max(3px, 0.1875rem, 0.12em)'
    }

@millnut This is a known issue and we will eventually overhaul the ESLint plugin with a proper parser. I have been working on one

Great I'll turn off the eslint rule for now


Field view upload.blade.php : Fix "Next ErrorException: Undefined index: disk"

somebody fix the bug?

What exception are you seeing?

Hi @reeslo ,
Thanks for PR. Just got around to merging it, thanks. It'll be available in the next point update (today).
Cheers!

Nice !


GroupArray function with tuple not supported?

Describe the bug
Unable to pass query with groupArray
SELECT     t,     groupArray((reason, c)) as reasons FROM (         SELECT                 toStartOfFiveMinute(record_datetime) AS t,                 JSONExtractInt(data, 'reason') AS reason,                 count(*) AS c         FROM hepic_data.sip_transaction_call         WHERE         record_datetime >= toDateTime(1660711612)                 AND JSONExtractInt(data, 'reason') > 0                 AND JSONExtractInt(data, 'reason') != 16                 AND JSONExtractInt(data, 'reason') != 17                 AND JSONExtractInt(data, 'reason') != 31         GROUP BY t, reason         ORDER BY t, reason ) group by t order by t
returns
sql: Scan error on column index 1, name "isdnReasons": unsupported Scan, storing driver.Value type [][]interface {} into type *string: Could not process SQL results
If I wrap toString around group array I get the data I want but in a format I can't seen to wrangle grafana to time visualization to support
2022-08-17 06:30:00 | [(20,2),(21,2),(102,2)]
Versions
Grafana and Plugin versions.
Grafana 9.0.3
Clickhouse 1.1.2.
Expected behavior
When performing this with the vertmedia plugin and the clickhouse endpoint the group array is returned as follows:

Which allows rendering via the timeseries. The expected behaviour would be to be able to use the same query in the official clickhouse plugin.
Additional context
I'm pretty sure if I wrangled hard enough I could get this working straight from the clickhouse query, but it would be great if this worked out of the box.
Any help greatly appreciated :)

Hmm I had thought I'd of been able to work around it by doing
`SELECT     time,     toJSONString(groupArray((reason, c))) as data FROM
(
SELECT $__timeInterval(record_datetime) as time,
JSONExtractInt(data, 'reason') AS reason,
count(*) AS c
FROM hepic_data.sip_transaction_call
WHERE $__timeFilter(record_datetime)
AND JSONExtractInt(data, 'reason') > 0
AND JSONExtractInt(data, 'reason') != 16
AND JSONExtractInt(data, 'reason') != 17
AND JSONExtractInt(data, 'reason') != 31
)
GROUP BY time, reason ORDER BY time, reason
) group by time order by time`
and then telling grafana that field was json but that doesn't work either, although it feels like it should and I am missing something.

Should be fixed in 2.0 beta. Can you test @Alan01252 ?

@Alan01252 2.0 is out, please test

Will do mate. On me holidays at the moment but will let you know when I'm back!

This does work, sort of.. With the old vertmedia this could be rendered directly into a visualization, now although the query does not error you still get.

but it does render the data in the table without error

thanks @Alan01252 not quite sure how you want this data returned - you can use transformations in Grafana to unfold the array if needed. Are you looking for a row per array entry?

So with the old plugin the data renders like this

With the same query as above, I don't know what magic the plugin is doing that is allowing it.

Grafana to unfold the array if needed. Are you looking for a row per array entry?

Also, ( and I'm entirely sure it's me being thick here ) I really can't work out what transformation I should be using to achieve the above, any help there would be greatly appreciated :)

I do not want to highjack this thread but the very basic use case for the groupAray function is to get e.g. multiple values for the same timestamp - sounds quite bizarre at first but can be used to provide anti-aliasing. For technical application it is super important.
reduced example of a query that works with Altinity plugin but does not in the Grafana Labs one:

Hi @gingerwizard thanks for getting back to me!
Yep, that's what the plugin seems to do, so if we take that first timestamp
2022-08-30 reads as, there were 4 of status code 1, 1 of status code 19, 2 of status code 20.
Rendering a graph that looks like

Now fortunately/unfortunately depending on how you look at it, we have a tiny number of values usually so the graph above doesn't show why this is useful but more than once this graph has saved us as it quickly shows an increase in one or more status code showing a real problem in our stack :)

ok not i understand the intention i think we can do something. thank you @Alan01252 and sorry for the delay.

Absolutely no need to apologise my friend :) More than happy with you taking the time just to look at my particular special brand of edge case :)

@jkaflik would you mind taking a look? @gingerwizard can help you with the investigation


Is this plugin still active?

Hi, I would like to know if this plugin is still in active development or obsolete.
Any maintainer?
Any plan to upgrade to Capacitor 3?
Thank you in advance

And also updating the docs, especially with the deprecation notices showing up when copy-pasting the base code from the README.


Security vulnerability in the hoek module, please update the jsonwebtoken module

A security vulnerability advisory has been issued for the hoek module: https://nodesecurity.io/advisories/566
Hoek is an indirect dependency of nexmo via jsonwebtoken (^7.1.9) -> joi (^6.10.1).
Luckily, jsonwebtoken removed the joi dependency in 8.0.0, so it seems like it's just a matter of updating to jsonwebtoken 8.x. The list of breaking changes in version 8 doesn't look too worrying.

Theres also the used request API effected:
<EMAIL_ADDRESS>><EMAIL_ADDRESS>

@papandreou 2.2.1 has just been released via NPM. Thanks for reporting, and apologies for the delay


v-ref in dynamic component that has 'keep-alive'

http://jsfiddle.net/22eqsb38/
If you switch the dynamic component like this the v-ref isn't changed.  Is this a issue or a feature?
I'm using vue 1.0.4

Thanks.


All the links are 404

None of the links work at: https://zsiciarz.github.io/24daysofrust//

Fixed, thanks for the report!


TCP_Server send "1" and client receives "��1"

When I send a string("1") the receiver(client) gets 1 what are these strange symbols?
I used
https://github.com/Kermer/Godot/wiki/tut_data-transfer
https://github.com/Kermer/Godot/wiki/tut_connection
Is it a bug? Am I doing something wrong?
Tested with socat and browser both the same.

@AlexHolly I guess you are sending the data through PacketPeerStream and receiving it directly through StreamPeerTCP (or even directly through some other program), right?
Because PacketPeerStream is able to send things other than strings (like numbers, dictionaries, etc.), it adds some information disambiguating between them, as explained in the wiki.
If you want to, say, host an HTTP or TCP server in Godot for consumption from other programs, you should use StreamPeer::put_data() instead of PacketPeer::put_var()

Thanks pointing this out.
I'm still not able to get it running. How can I bind the client to a StreamPeer?
PacketPeerStream
peerstream[ index ].set_stream_peer( client )
There is no StreamPeerStream class... and StreamPeer has no set_stream_peer function.
In the wiki I can only see how to use PacketPeerStream but not StreamPeer(Stream)
Maybe some more doku on this topic would help others too.

I think StreamPeer has not many functions to get and send raw data, those
should be added eventually. If you are writing your own server on the other
hand, you can use PacketPeer and use the format documented for it in the
wiki:
https://github.com/godotengine/godot/wiki/binary_Serialization
On Sun, Nov 22, 2015 at 9:20 AM, AlexHolly<EMAIL_ADDRESS>wrote:

Thanks pointing this out.
I'm still not able to get it running. How can I bind the client to a
StreamPeer?
PacketPeerStream
peerstream[ index ].set_stream_peer( client )
There is no StreamPeerStream class... and StreamPeer has no
set_stream_peer function.
In the wiki I can only see how to use PacketPeerStream but not
StreamPeer(Stream)
Maybe some more doku on this topic would help others too.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/2863#issuecomment-158753814.

@AlexHolly you have to chose whether to communicate low level or high level. If you send high level and you receive low level, you get the behaviour described in the bug report.

Classes representing low level communication are: StreamPeer, StreamPeerTCP,

Classes representing high level communication are: PacketPeer, PacketPeerStream

If you have a low level class, you can use a PacketPeerStream to use it for high level communication.
If you have a high level class, you can't make it low level.
Yes, in the docs of PacketPeer it mentions the word "udp", so people might think this is low level. But it isn't. You can solely use it for high level communication (aka using the godot serialisation format).

I'm still not able to send a string I get "cannot convert argument 1 from string to rawArray"

To convert a string to a rawArray, you'll have to use the to_utf8() method of the String class. If you are getting that error, you most likely want to use the put_data function of StreamPeer, so the low level protocol. I don't think you want to do this. There are multiple problems connected to it. The largest one is that you have to know the size of the transmitted message at the receiving side. If you use the high level protocol, godot takes care of that for you by sending the size of every packet at the start of each packet.
So you should tunnel everything through PacketPeerStream. Just use its  set_stream_peer method to bind it to the StreamPeer on both sides.

I need a websocket in godot so that I can give the client(maybe a browser) a pure http text response without any godot overhead.
I made a function(put_text) for it.
The next step would be to implement the parsing of the incoming http requests, most of the parsing is already implemented inside http_client. Maybe someone could make the parsing more abstract to make it more easy implementing a websocket.
I will try to implement a quick solution in .gd to get things running I don't need a clean solution right now.
Maybe I miss something and there is already a solution for it.

Just use put_data("text".to_utf8()) then. put_text isn't needed.

That is what I was looking for, thanks!
I think there is no documentation for it.

I can't see to_utf8() in the string class.
-An example to to send non serialized data would be nice.

The method is very young, and the online docs only update every stable version or so. The docs from inside the editor are recent though (script tab then click "Classes" or "Search Help").

And #2898 has improved the docs for to_utf8() (from mention to additional text).

Actually the online API reference was never updated I believe, it's still from pre-open source Godot code.

So if I understand correctly, the initial issue has a solution already, but it is not well documented enough on the online wiki. The API reference being outdated is a more global issue so we probably don't need to keep this issue open for it, on the other hand is there a need to improve/write a doc wiki page to mention this to_utf8() method?

Yes I would like to see a documenation on how to communicate with another non godot system. It is basically a documentation of this issue.

need to add a low level API to streampeer
will work on this now

Nice, will it also support #3057?


Memory leak during build process after updating to Angular 15

Command
build
Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
14.2.8
Description
Hi,
After upgrading from Angular 14 to Angular 15, our library project has dramatically increased the amount of RAM needed to build the project. It never surpassed 8GB of RAM on build time with Angular 14, but after upgrading to Angular 15.1, it needs up to 25GB of RAM... it doesn't seem like a normal change. Furthermore, it takes far more time.
Minimal Reproduction
We are executing: ng build ng-ptp
Exception or Error
No response
Your Environment
Angular CLI: 15.1.0
Node: 18.12.0
Package Manager: npm 8.19.2
OS: darwin arm64

Angular: 15.1.0
... animations, cli, common, compiler, compiler-cli, core
... elements, forms, language-service, platform-browser
... platform-browser-dynamic, platform-server, router
... service-worker

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1501.0
@angular-devkit/build-angular     15.1.0
@angular-devkit/core              15.1.0
@angular-devkit/schematics        15.1.0
@angular/cdk                      15.0.4
@angular/material                 15.0.4
@angular/material-luxon-adapter   15.0.4
@nguniversal/builders             15.0.0
@nguniversal/express-engine       15.0.0
@schematics/angular               15.1.0
ng-packagr                        15.1.0
rxjs                              7.8.0
typescript                        4.9.4

Anything else relevant?
No response

Duplicate of https://github.com/ng-packagr/ng-packagr/issues/2530


Add a new homepage-like page

I want a new page that is more like a homepage. But I'm unable to do that by creating a _index.html file.
I want to have a section in the middle of the page (same as the homepage) but I want that to be on a different page and just a paragraph of text.

Are you wanting to override the current homepage? Or create a completely new page that looks like the homepage?
If you are looking for the former, you can simply override the default layout. See the README for more info.
If you are looking for the latter, you would just need to add a new page to your site following Hugo's guidelines. The lookup order will make sure to take your page into account.


Feature: BITFIELD command with all subcommands

Fixes: #639
Implemented BITFIELD key [SUBCOMMAND]. Have handled predominantly most test cases present in the redis tcl tests. implemented unit test cases.
Additionally, added *.rdb in .gitignore to avoid pushing rdb files to git.
Design:
Added utility functions in ByteArray struct to handle get set and incrby commands.
All test cases pass. Once merged #640 can be unblocked.
Interesting issue to work on, given that bit manipulations are last kind of questions I do on leetcode.
Thanks
Apoorv

Hi team, @JyotinderSingh , @lucifercr07 , @AshwinKul28 , PR is rebased with the master, Could you please review at your convenience?
Thanks,
Apoorv

@apoorvyadav1111 please fix the build once.

@apoorvyadav1111 please fix the build once.

Hi @lucifercr07, thanks for reruning the pipeline. It passed this time. I am trying to recreate the issue in the local meanwhile.
Thanks


An API to execute Ballerina stream next function in standard libraries

Description:
Ballerina gRPC library has a requirement to execute next function in a stream at native level. ATM, a stream comes to the native side as a StreamValue object. The StreamValue is an internal object, and hence it is not accessible to the stdlib native code.
Therefore, it would be better to have a direct API to execute the next function in a stream. Perhaps as a synchronous API and asynchronous API.
Related Discussions
[1] https://ballerina-platform.slack.com/archives/C9WMA6QDC/p1609926111194400
[2] https://ballerina-platform.slack.com/archives/C9WMA6QDC/p1609856798190700
Affected Versions:

SLP9

@manuranga @warunalakshitha Can you prioritize this and add support to read the next value from the stream object, please.

@manuranga @warunalakshitha Can you prioritize this and add support to read the next value from the stream object, please.

BStream already have a api to get iterator object and then you can call next method using Runtime.invokeMethodAsync method. ATM we do not have a synchronous api. @manuranga Shall we add sync api to so it will be direct call to object method without scheduler.  But we need to make sure object method doe not have any async code.

BStream already have a api to get iterator object and then you can call next method using Runtime.invokeMethodAsync method. ATM we do not have a synchronous api. @manuranga Shall we add sync api to so it will be direct call to object method without scheduler.  But we need to make sure object method doe not have any async code.

@BuddhiWathsala @daneshk is this still a high priority ? is Buddhi's current approach working ?

@BuddhiWathsala @daneshk is this still a high priority ? is Buddhi's current approach working ?

Yes, it is working as expected ATM. We can proceed with it.

Yes, it is working as expected ATM. We can proceed with it.

I am closing this as per the last comment.


[DRAFT] Trigger modals after transitioning to new route

Hoping this will solve #313.
Solution was to replace modalsService->applicationController dependency and instead use the router's didTransition event to track currentRoute.
Works in my project, need to add tests and ensure it's compatible across ember versions.

@mikegrassotti Thanks for looking into this Mike, it's the last pain point for my upgrade to 1.13

@ef4 ping, our project is on @mikegrassotti 's fork of this due to this issue, its impact is pretty major... any ETA on getting this reviewed/merged?

@davidgoli have you tried master pointing your project to liquid-fire master? Looks like this was fixed in a4d708e

L-F master does not fix this problem for me, but this patch does. Thanks @mikegrassotti

@mikegrassotti no, LF master seems to be pointing to canary, which is too unstable to adopt ATM

Thanks for this PR, I solved the problem slightly differently in afe50d28c4e14a0c8d9877feff1f6e59956eeb1e.
The root problem was that the CP wasn't actually accessing its dependency, so the chain observers were not initialized correctly.


Approval Notification Email

Should this send a notification email to the approvers?  I don't see a setting for that?

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

ID: b067a175-f640-7503-9c1e-f0130c6dbeda
Version Independent ID: ff743c7b-a103-eae6-4478-62ba995a4b36
Content: Pipeline deployment approvals - Azure Pipelines
Content Source: docs/pipelines/process/approvals.md
Product: devops
Technology: devops-cicd
GitHub Login: @shashankbansal
Microsoft Alias: shashban

This is being developed. Currently there is no notification for a pending approval.

Hi, is there an estimated time frame? Our team is really interested in this feature.

This is being tracked as one of the critical gaps with Multi-stage YAML pipelines. As per current state of the backlog, I expect it to be in place sometime in November this year.

Email notifications would be really useful to allow us to keep on top of approvals.

@shashban Is there any update on the timeline? Is this still on track to be in a November release?

This went through a bit of replanning and shifted from November to a January release.

Is this still on track for a January release?

Hi Guys.
Any updates on this one? Even when I configure the notifications with "An approval for a deployment is pending" I don't receive any email. It used to work with previous version of tfs, but with the current version(Azure dev ops server, on prem, 17.143.28912.1) it stopped working.
Email notification is needed for our business.
Thanks

any news on the Jan release?  likely to happen?  the shift from Nov to Jan concerns me and could be a game changer if another 2 month delay

This went through a bit of replanning and shifted from November to a January release.

@shashban is this still still on schedule? This is very important to us.

Our team is blocked by this functionality as well, please prioritize.

same here. our team is blocked by this for long time.

Please enable this. It's an incredible feature much needed for my team.

To be clear, the feature for approval notifications is currently under development and not available for preview. It is in active development that we expect to be available in 4-6 weeks.

Any update on this feature

You can do it!

Is it available now? I can't see this feature yet. Do you have any other workaround @papagala
Please help, this is much required for our team.

Is it available now? I can't see this feature yet. Do you have any other workaround @papagala
Please help, this is much required for our team.

Looks like “ To be clear, the feature for approval notifications is currently under development and not available for preview. It is in active development that we expect to be available in 4-6 weeks.”

Looks like this feature has finally been released!
https://docs.microsoft.com/en-us/azure/devops/release-notes/2020/sprint-167-update#approval-notification

Anyone have any idea how long the roll-out to Cloud can take?  It's been 4 days since the documented April 13th release with this feature and I still haven't seen any Approval notifications from my Environments.  I don't see any additional configuration details on the Approval config either.

Hi Folks checking in with the Product Group on this as some of the clients and myself have been looking for this.  All Azure Services follow a rind deployment model so not all regions are deployed to at the same time.... this sort of speaks to the concept if you're not familiar with it...https://docs.microsoft.com/en-us/azure/devops/migrate/phase-rollout-with-rings?view=azure-devops

Anyone have any idea how long the roll-out to Cloud can take? It's been 4 days since the documented April 13th release with this feature and I still haven't seen any Approval notifications from my Environments. I don't see any additional configuration details on the Approval config either.

Unofficially there was a small issue with the release so the rollout was paused.  Small fix should be done for the next release train 2-3 weeks.

Tried today and I also didn't receive any notification.

I'm receiving the pending approval mails since yesterday! ✅

I'm receiving the pending approval mails since yesterday! ✅

What region is your org. deployed in?  Did you have to make any config changes?

I've been seeing the notifications since last Friday.  Didn't want to mention it here until I'd seen them continue for several days, but it is up and appears to be working well since then. Our installation is in the Central US region.

This has been rolling out over the at week. We follow a ring based deployment model & that means depending upon the ring your organization belongs to, the feature might already be available or would be there by early next week.
Closing this issue.

I'm receiving the pending approval mails since yesterday! ✅

What region is your org. deployed in? Did you have to make any config changes?

West Europe. Did not make any changes.

I'm receiving the pending approval mails since yesterday! ✅

Me too

I'm receiving the pending approval mails since yesterday! ✅

Me too

It works for me too, tryed today and it's ok!

Is it available now? I can't see this feature yet. Do you have any other workaround @papagala
Please help, this is much required for our team.

Looks like “ To be clear, the feature for approval notifications is currently under development and not available for preview. It is in active development that we expect to be available in 4-6 weeks.”

It is available now. We are getting the alerts for pending stage approval.

question: can we make changes to the notification approval and approve the pipeline from the email without accessing the azure devops portal?

@mohammedhank91 -- thank you for your question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow


Porting assets modification plugin to V5

Hey.
I'm giving another shot to V5 upgrade in a more incremental way and one thing
I strugle with is porting of my plugin for assets modification.
The idea is that we are running multiple compilations of the project (one per color theme)
and I'm removing all non-css assets from all compilations except the first one.
The whole rather simple code that is working for us on V4 is here:
function PluginOnlyCss() {
    this.hooks.shouldEmit.tap('PreventEmit', compilation => {
        Object.keys(compilation.assets).forEach(assetFileName => {
            if (/\.css$/i.test(assetFileName)) return;

            // eslint-disable-next-line no-param-reassign
            delete compilation.assets[assetFileName];
        });
    });
}

This however causes the deprecation warning:
[DEP_WEBPACK_COMPILATION_ASSETS] DeprecationWarning: Compilation.assets will be frozen in future, all modifications are deprecated.
BREAKING CHANGE: No more changes should happen to Compilation.assets after sealing the Compilation.

I wasn't able to find good pointers that would aid me with the migration

Because you use invalid hook, no modification for assets on emit/beforeEmit hooks, please use this hook https://github.com/webpack/webpack/blob/master/lib/Compilation.js#L551:
compiler.hooks.thisCompilation.tap(name, (compilation) => {
  const Compilation = compier.webpack.Compilation;

  compilation.hooks.processAssets({ name: name, stage: Compilation.PROCESS_ASSETS_STAGE_PRE_PROCESS }, (assets) => {
   // Your logic...
  })
})


compile 'io.saeid:fab-loading:0.6.0' not valid adress

Gradle cant build this

Did you add `jcenter() into your build.gradle? it's available on jcenter
https://bintray.com/bintray/jcenter?filterByPkgName=fab-loading
If still you have problem you can add my repo
repositories { maven { url "http://dl.bintray.com/smasoumi/maven" } }

jcenter() fixed error thnx


Implement 380, 441, and 981

Three more solutions for ya. I both borrowed from online videos as well as made small changes to make them more readable

Nice PR! :)


Can we add new icons in this iconfont?

I'm looking mainly for WhatsApp, Telegram icons.

I don't know how to add it in.May be we should use other icon font source?

https://remixicon.com/, I found this icon pack


Refactor rules endpoint

Removes specific params
Improves RuleSet.FilterResource to accept multiple resources
Rework FindRules method in access-controller tpl

Please take a look.

Codecov Report

:exclamation: No coverage uploaded for pull request base (2022.9.x@aae7152). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             2022.9.x     #393   +/-   ##
===========================================
  Coverage            ?   38.91%
===========================================
  Files               ?      795
  Lines               ?    80669
  Branches            ?        0
===========================================
  Hits                ?    31389
  Misses              ?    44794
  Partials            ?     4486

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 aae7152...db56ad9. Read the comment docs.


Feature request: means to redirect to a system protoc

Hi! It'd be nice to have a way to make init() return a path to a user-managed installation of protoc. E.g. protoc-prebuilt could check if PROTOC is already set. Unless there's something that I missed, there's currently no way to achieve such behaviour
Thanks!

Now I understand. Yes, we can add check local installation before download binaries. But question about protoc version is still opened.
If another library use init with specified version, it expects that protoc-prebuilt return path to binary passed version.
Yes, one of solutions - it use environment variable, but it complicates work with library.
Another solution - it check locally installed binary exists, check it version and if it's not compare with required version, try to download required. In whis way third party library get that it expect, 3rd level dependency user can run it in sandbox, but should provide required version binary exists there. This solution not break current API, so this update will be minore and doesn't require any changes in third party libraries.

If another library use init with specified version, it expects that protoc-prebuilt return path to binary passed version.

If a user tries to override protoc-prebuilt's default behaviour using an environment variable, maybe they should just pay attention to what they're doing, and if they break something this way it's their fault, and not yours?

But if you'd really like to take extra care of the versions, here's a scheme I'd be most comfortable with:

If PROTOC_PREBUILT_PATH=/a/b/c/d/bin/protoc is set, proceed with this path. Otherwise download a binary
If using a prebuilt binary or the user sets e.g. PROTOC_PREBUILT_NO_VERSION_CHECK, skip the checks
If a check is required, obtain the binary's version, compare against the constraints, crash if the test fails

I'd prefer the deterministically failed build to an attempt at "fixing things", because if I asked to use a system-installed protoc and it's of the wrong version, then I almost certainly did so erroneously
But ofc that's just how I see it, you do what works best for you

Yes, I understand that it best solution for you. But I worried that described behavior can broke third party library and user may mistakenly think that problem in third party library, not in launched environment.
I think that compromise is force check locally installed protoc and require check passed version. If it not exists or versions not exual, try to download from github.

Sure, it's not a problem for me at all, I was just thinking that this would've been easier for you as well. And I don't see how you could break any third party libraries, since the user would have to explicitly set a protoc-prebuilt-specific environment variable to activate the new behaviour. They couldn't do that by accident, could they?

Sorry for very long feedback. Add required functional, see https://github.com/sergeiivankov/protoc-prebuilt/commit/fe88a63bf0efd95c4334945e2c15497050c5bfa7.
Thank you for taking part!

@SomeoneSerge, new version with this functional available https://crates.io/crates/protoc-prebuilt!

Thanks a lot @sergeiivankov! I had worked around the problem that originally motivated me to open the issue, but I'm pretty sure I'll eventually need this feature again and will test the changes!


can ng serve build files in a physical directory?

Please provide us with the following information:

OS?

Windows 2008 server

Versions.
angular-cli: 1.0.0-beta.24
node: 6.9.1
os: win32 x64
@angular/common: 2.4.1
@angular/compiler: 2.4.1
@angular/core: 2.4.1
@angular/forms: 2.4.1
@angular/http: 2.4.1
@angular/platform-browser: 2.4.1
@angular/platform-browser-dynamic: 2.4.1
@angular/router: 3.4.1
@angular/compiler-cli: 2.4.1
Repro steps.

run ng serve

The log given by the failure.

not an error

Mention any other details that might be useful.

ng serve builds and serves the files in memory. Is it possible to make ng serve build and serve from a physical directory instead? /dist/dev, for example?

I'm constrained by my environment's authentication to run within IIS and therefore need a location I can attach a virtual directory to .
I realize I could concurrently do an ng build but that seems wasteful.

First and foremost: Do not serve production builds using ng serve. This is a blanket warning because it's just not safe. You're trusting a generic, minimal configuration to secure your app.
IIS has a couple ways to serve these apps, and each has pros and cons. The method I prefer is to run a back end server (Hapi, Express) on a high number port, then use URL Rewrite to turn IIS into a reverse proxy. The back end then serves your dist folder just like any production Angular app. If you have a lot of services on various ports and need a lot of choice in your stack this might be a good, extendable option.
The other one is to literally serve the dist folder. Angular 2 is built into a static set of files and needs no real processing on the local server. Build it, set your IIS config, and away you go! More detailed info: (http://stackoverflow.com/questions/32719436/deploy-angular-application-on-iis)

{clarification: i'm not doing this in prod, it's on the local dev box for development purposes.}
Thank you, gelliott, I'm looking at your answer now.
my hope is to change a file, press save, and have the changes reflected in the IIS location, similar to using gulp watch to retrigger the build.
because of the authentication in this case (siteminder) I doubt URL rewrite will work; it's already being redirected in IIS to inject headers and cookies. Yes, it hurts my head too, but that's my constraint.
With Angular-seed i do literally serve the dist/dev folder. ng serve doesn't have an output directory though.  Therefore I was hoping for a way to configure ng serve to build to a physical directory.
I can think of a hackish way, such as adding a gulp watch to build on file changes, but trying to work within the cli.

ng build --watch will watch the build folder and automatically build on changes.

{doh}  thank you

Closing as @gelliott181 has answered it, cheers!


feat(util): added a gptme-util prompts git helper command

The idea is to move the helper scripts for prompt generation in gptme-bob into gptme.

Codecov Report
Attention: Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 71.18%. Comparing base (591b5be) to head (a3da224).

:white_check_mark: All tests successful. No failed tests found.

Files with missing lines
Patch %
Lines

gptme/util/cli.py
62.50%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
- Coverage   72.42%   71.18%   -1.25%
==========================================
  Files          64       64
  Lines        4323     4331       +8
==========================================
- Hits         3131     3083      -48
- Misses       1192     1248      +56

Flag
Coverage Δ

anthropic/claude-3-haiku-20240307
71.18% <62.50%> (+0.07%)
:arrow_up:

openai/gpt-4o-mini
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

JS Bundle Analysis - Avoid shipping oversized bundles


SNR chart blank

Actual Behavior:

`What is the issue? When loading the devices page, the SNR chart isn't loading. Instead it's showing a blank chart.

Additional Information:

Browser Type: affects all browsers etc.
Browser Version: *
OS: *
Page URL: *

Do not modify the titles or questions. Simply add your responses to the ends of the questions.
Add more lines if needed.

Add screenshots, gifs, illustrations as required to demonstrate your issue.

No duplicates, please use the search tool.

The adjustments in that commit don't resolve.


shader_prepass example crashes with opaque CustomMaterial

Bevy version
release-0.10.0 (also get the same crash on main).
Relevant system information
`AdapterInfo { name: "NVIDIA GeForce RTX 2060 with Max-Q Design", vendor: 4318, device: 7954, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "526.86", backend: Vulkan }`

What you did
I modified the shader_prepass example so that the CustomMaterial was AlphaMode::Opaque rather than AlphaMode::Blend.
diff --git a/examples/shader/shader_prepass.rs b/examples/shader/shader_prepass.rs
index 688152c5..ae85d681 100644
--- a/examples/shader/shader_prepass.rs
+++ b/examples/shader/shader_prepass.rs
@@ -109,7 +109,7 @@ fn setup(
         material: materials.add(CustomMaterial {
             color: Color::WHITE,
             color_texture: Some(asset_server.load("branding/icon.png")),
-            alpha_mode: AlphaMode::Blend,
+            alpha_mode: AlphaMode::Opaque,
         }),
         transform: Transform::from_xyz(1.0, 0.5, 0.0),
         ..default()

What went wrong
Bevy crashes. With other testing it appears the issue only happens when the camera is looking at the object with the custom material.
Additional information
2023-03-31T07:35:36.907647Z  INFO bevy_winit::system: Creating new window "Bevy App" (0v0)
2023-03-31T07:35:39.202777Z  INFO bevy_render::renderer: AdapterInfo { name: "NVIDIA GeForce RTX 2060 with Max-Q Design", vendor: 4318, device: 7954, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "526.86", backend: Vulkan }
2023-03-31T07:35:40.722993Z  INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "Windows 10 Home", kernel: "18363", cpu: "AMD Ryzen 9 4900HS with Radeon Graphics", core_count: "8", memory: "15.4 GiB" }
2023-03-31T07:35:41.512883Z  INFO naga::back::spv::writer: Skip function Some("calculate_view")
2023-03-31T07:35:41.553574Z ERROR wgpu::backend::direct: Handling wgpu errors as fatal by default
thread 'Compute Task Pool (7)' panicked at 'wgpu error: Validation Error

Caused by:
    In Device::create_render_pipeline
      note: label = `prepass_pipeline`
    error matching FRAGMENT shader requirements against the pipeline
    location[0] is provided by the previous stage output but is not consumed as input by this stage.

', C:\Users\Tesse\scoop\persist\rustup\.cargo\registry\src\github.com-1ecc6299db9ec823\wgpu-0.15.1\src\backend\direct.rs:3024:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread 'Compute Task Pool (7)' panicked at 'A system has panicked so the executor cannot continue.: RecvError', crates\bevy_ecs\src\schedule\executor\multi_threaded.rs:194:60
thread '<unnamed>' panicked at 'called `Option::unwrap()` on a `None` value', C:\Users\Tesse\Desktop\Bevy Games\bevy\crates\bevy_tasks\src\task_pool.rs:376:49
thread 'Compute Task Pool (7)' panicked at 'called `Result::unwrap()` on an `Err` value: RecvError', crates\bevy_render\src\pipelined_rendering.rs:136:45
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', C:\Users\Tesse\Desktop\Bevy Games\bevy\crates\bevy_tasks\src\task_pool.rs:376:49

I found this interaction on the discord: https://discord.com/channels/691052431525675048/866787577687310356/1077982198331867166. Not sure why this is necessary though.

Okay, I'm pretty sure the issue is that the default vertex prepass shader isn't consuming location[0] https://github.com/bevyengine/bevy/blob/f0f5d7991758514c1e301de327ba9dda212330f0/crates/bevy_pbr/src/prepass/prepass.wgsl#L86-L95, which is why this happens with custom materials. The pbr_prepass shader does consume it though, https://github.com/bevyengine/bevy/blob/f0f5d7991758514c1e301de327ba9dda212330f0/crates/bevy_pbr/src/render/pbr_prepass.wgsl#L7-L12. I assume the UV location isn't defined when AlphaMode is blend, which causes this problem only with opaque materials. Not sure if this is intentional.

Thanks for fixing this!


select_merge throws an exception when '...' is used in the bindings list

Elixir version (elixir -v): 1.6.5
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): PostgreSQL 10.4
Ecto version (mix deps): 2.2-3.0.0-dev
Database adapter and version (mix deps): 0.13.5-Postgrex from ecto master mix.lock
Operating system: macOS High Sierra

Current behavior
select_merge throws an exception when '...' is used in the bindings list.
  query =
        "posts"
        |> select_merge([..., p], %{title: p.title})

  ** (CompileError) test/ecto/query/builder/select_test.exs:121: undefined function escape_count/0
      (stdlib) lists.erl:1338: :lists.foreach/2
      (stdlib) erl_eval.erl:670: :erl_eval.do_apply/6
      (elixir) lib/code.ex:677: Code.require_file/2
      (elixir) lib/kernel/parallel_compiler.ex:201: anonymous fn/4 in   Kernel.ParallelCompiler.spawn_workers/6

Expected behavior
Should not throw exception

Sorry for this rather dumb question. How do I get this fix in Ecto 2.2? Is there any reason it wasn't included? @josevalim

Good question, it is a small fix, so we can definitely backport. I have done this and pushed to the v2.2 branch. But generally we only backport when asked, as we tend to focus mainly on the upcoming release (and v3.0-rc is out).

Thank you @josevalim Sorry for the trouble.

No trouble at all :)


Not to stick items

Hi
I am trying to use swipe mode but when items are sticking it is not possible to get a free flow feeling. Is there a way to make slider not have sticking items?
Thanks

Any one fixed this?

Any one fixed this?


Module not found: Can't resolve 'avataaars' in 'F:\github\avataaars-generator\src\components'

./src/components/Renderer.tsx
Module not found: Can't resolve 'avataaars' in 'F:\github\avataaars-generator\src\components'

npm install avataaars


[WIP] Update tekton results manifests

reference tekton results manifests from downstream repo
openshift-pipelines/tektoncd-results
update variable name for database secret and form literals
expose rest endpoint and remove grpc
remove initcontainer script from results api deployments

Signed-off-by: Avinal Kumar<EMAIL_ADDRESS>

I have removed the initcontainer script wrt to this discussion but the code should migrate somewhere, I am not sure where.

/retest

/retest

@avinal if you have removed the db deployment manifests without moving that somewhere else, it is not going to work.
You need a db for results to work.

/retest

@avinal You'll need to rebase.

@avinal When the PR is ready for review, please clean up the commit history and remove [WIP] from the PR title.

Closing in favor of #504


Add clandropswebhook plugin

Clan Drops Webhook
A plugin for RuneLite
A plugin which allows to send a message to your Discord server using a webhook whenever there is a clan broadcast in your name.
The webhook is triggered in the following instanced:

When you receive a drop which is broadcasted to the clan channel. Minimum value for the drop is set by the clan administrators.
When you receive a pet drop.

Config options:

Webhook URL.
Upload image or text.
Screenshot only the chatbox instead of the screen.
Automatically post a message in the chat with or without the current date (useful for bingo phrases).

TODO:

Add support for level ups.
Add support for multiple webhook links for specific channels

Can this be added to discord-loot-logger?

Can this be added to discord-loot-logger?

I will make a Pull Request to his repo and contact the developer.
I will close this PR.


RFC: New submission and merging process for PR's

I have a weird desire to get all these PR's merged as soon as possible so I am not being as thorough as I could be, so I want to propose some new ground rules for managing the submission and merging of PR's.
I want every PR to have a review request attached to it. I will not merge any PR's that did not have a review request and will not merge any PR's that have not passed a review.
Sometimes I have to put things in front of me to slow me down.

Sounds good to me, maybe this will stop me from introducing bugs all over the place

@chartjes
Problem is that contributors other than maintainers cannot request a review! 🤓

@localheinz Is there a setting I can change to allow non-maintainers to request a review?

@chartjes
I don't think so, that's only possible for collaborators!

Okay, I added in @localheinz and @BackEndTea as collaborators

@chartjes
Nice, thank you!
You should probably enforce an approved pull request review in https://github.com/opencfp/opencfp/settings/branches/master!

@localheinz Done -- all PR's should require a review now. I don't think I should turn on "require review from Code Owners" because not everyone who has written code still contributes to the project

@chartjes
One more question, though, how would you prefer to go forward with merging PRs?
Once approved, who can or should merge the PR?

@chartjes Thanks for the collaborator role.

@chartjes
Do you want to enforce something along the lines of 'A PR larger than X requires multiple reviewers'?

Do you want to enforce something along the lines of 'A PR larger than X requires multiple reviewers'?

I think we should use our best judgement rather than enforcing much of anything. If someone sends a 1 line diff (I'm looking at you @localheinz with all your check PRs :grin:), then that's pretty straight forward. I guess it's a good example too (e.g. @localheinz sends a check that might be "controversial". He's pretty good about recognizing that and asking for additional scrutiny. Maybe that wouldn't be a PR I would just merge without further conversation other than a "Thanks!").
If someone sends 3000 lines that crosses multiple boundaries of abstraction, it's probably best for the reviewer to reach out to another reviewer OR for the contributor to even realize what they've done :skull:  and apologetically ask for additional review.
All in all, I think the four of us can be responsible open-sourcers. :smile:

@BackEndTea @chartjes @mdwheele
Regarding https://github.com/opencfp/opencfp/issues/762#issuecomment-347574917, can we clarify?
For example, if I open a PR, someone else approves it, who should be allowed to merge it?

I would go with method 1 -- the reviewer merges it in, which is what I have been doing here

I think we've come to a decent resolution. Feel free to re-open; just grooming.


expand exclude and rid absent

i'm using 2.1-rc4.
When using expand (exclude) in a query, the rid is absent in the results
select expand(@this.exclude("fieldname")) from ClassName
ClassName extends V

fixed in develop branch.
ATTENTION: now records are returned with valid rid, so a doc.save() will remove excluded fields!!!
rid can be excluded too with the same syntax:
select expand(@this.exclude("fieldname", "@rid")) from ClassName


ChatGLM-6B finetune

Today, when I was doing the full parameter fine-tuning, my loss became 0.0 after the training of batch 17, and then it kept going to 0.0. I changed both the learning rate and batch, but I still failed to solve such problems

try to set float32, not fp16.
i run full parameter fine-tuning success with llama, not chatglm ever.


Can not read string array

Trying to read string array and only get first 2 elements. Writes work perfectly. Can always read array of 2. Code below produces following:
Values read:  { STRARRAY: [ 'String 00', 'String 01', '', '' ] }
[50475,547282909 <IP_ADDRESS>] Preparing to WRITE STR0,STR1,STR2,STR3,STR4
[50475,559641594] NST9:0 write completed with quality OK
[50475,559721341] NST9:1 write completed with quality OK
[50475,559756823] NST9:2 write completed with quality OK
[50475,559794130] NST9:3 write completed with quality OK
[50475,559815871] NST9:4 write completed with quality OK
Done writing.

function connected(err) {
if (typeof(err) !== "undefined") {
// We have an error.  Maybe the PLC is not reachable.
console.log(err);
//        process.exit();
}
conn.setTranslationCB(tagLookup);
conn.addItems('STRARRAY');

//    conn.addItems(['INT0','INT1','INT2', 'INT2']);
    async.forever(
        function(next) {
            //conn.readAllItems(valuesReady);
            if (doneReading && doneWriting) {
            doneReading = false;
            conn.readAllItems(function (err, values) {
                if (err) {
                  console.log("SOMETHING WENT WRONG READING VALUES!!!!");
                  plc_error = true;
                } else {
                  console.log('Values read: ',values)
                  plc_error = false;
                  ++sheet_length;
                  conn.writeItems(['STR0', 'STR1', 'STR2', 'STR3', 'STR4'],
                                  ['String 00', 'String 01', 'String 02', 'String 03', 'String 04'], valuesWritten);

//                      conn.writeItems(['INT0', 'INT1', 'INT2'],
//                                      [2, sheet_length, 500500], valuesWritten);
doneWriting = false;
}
doneReading = true;
setTimeout(function () {
next();
}, 1000);
//                  if (doneWriting) { process.exit(); }
});
}
},
function(err) {
console.log('READ LOOP ERROR: ' + err);
}
);
}
function valuesWritten(anythingBad) {
if (anythingBad) { console.log("SOMETHING WENT WRONG WRITING VALUES!!!!"); }
console.log("Done writing.");
doneWriting = true;
}
// This is a very simple "tag lookup" callback function that would eventually be replaced with either a database findOne(), or a large array in memory.
// Note that the return value is a controller absolute address and datatype specifier.
// If you want to use absolute addresses only, you can do that too.
function tagLookup(tag) {
switch (tag) {
case 'TEST1':
return 'Test_Int';              // Integer
case 'TEST2':
return 'B3:0/0';            // Bit
case 'TEST3':
return 'B3/17';             // Same as B3:1/1
case 'TEST4':
return 'F8:0,20';           // Yes this is an array...  20 real numbers.
case 'TEST5':
return 'F8:1';              // Single real.
case 'TEST6':
return 'F8:2';              // Another single real.
case 'TEST7':
return 'N7:1,2';            // A couple of integers in an array
case 'TEST8':
return 'NST9:2,4';             // Direct output
case 'TEST9':
return 'N7:0';             // Direct output
case 'STRARRAY':
return 'NST9:0,4';             // setup number
case 'STR0':
return 'NST9:0';             // setup number
case 'STR1':
return 'NST9:1';             // customer name
case 'STR2':
return 'NST9:2';             // order number
case 'STR3':
return 'NST9:3';             // next operation
case 'STR4':
return 'NST9:4';             // generic comment
case 'INT0':
return 'L7:0';              // discharge direction (flags?)
case 'INT1':
return 'L7:1';              // sheet width
case 'INT2':
return 'L7:2';              // sheet length
case 'INT3':
return 'L7:3';              // sheets in discharge
case 'INT4':
return 'L7:4';              // sheets in discharge
case 'INT5':
return 'L7:5';              // stacks to batch
default:
return undefined;
}
}

Did you verify the write in Studio 5000?
Because it's an N file, it needs 44 integers for a string, STR1 should be NST9:44, then 88 and so on.

That is why it works for me and not for you - I think it will be really difficult to support two-dimensional arrays with nodePCCC because the PCCC protocol is meant for simple data tables.
Reason: When we read an array of NSTRING values, we need to read 88x4 = 352 bytes.  The maximum amount of PCCC data we can request varies, but the latest nodePCCC figures it's about 200 in my case, but then shortens this to 176 to keep from "splitting a string" over multiple read requests.  So it comes up with two chunks of data - one starting at N9:0, and another one starting at - it calculates N9:88, because this is 176 bytes later, and the "88" number is part of the request for the second half of the data.  But in the case of a two dimensional array, how the ControlLogix/CompactLogix maps a two dimensional array to a SLC file internally probably doesn't agree with that calculation.  You could fill the array with a bunch of characters to see what you're getting, but then we would need additional syntax in the item definition to support this properly, and that could get complicated when extended to other data types as well.  Time would be better spent supporting CIP to get the tag in its native format.
When I tested this on the weekend, I set up a CompactLogix with the following variables:
N9 created as INT[484]
ST41 created as STRING[5]
And a single instruction in the program:
COP ST41[0] N9[0] 484
I typed in values in Logix for the strings and wanted to read them as NSTRINGS.
I then was able to read strings using either of the following methods:
TEST8 - NST9:0,4 to read or write ALL strings as an array.
STR0 - NST9:0
STR1 - NST9:44
STR2 - NST9:88
STR3 - NST9:132
Writing the array works too, I tested this as well, with the COP instruction disabled.
I would suggest you try the single-dimension arrays again with the COP done the way I suggest, it works much better with 0.1.11.
Alternatively, edit the code for theItem.plcnpratio = 44; on line 2412 and try different values to find what works for a two-dimensional array.

Ok, I see what I was doing wrong when writing to the single array. All works as advertised now.
I might continue using the two dimensional array for writing though. Works like so nice.
Thanks!

Good to hear it's working better now.  When I get a chance I will see if I can figure out what the PLC is doing with the 2D array and the PCCC request.

I did play around a bit with 2D arrays and at least I understand a little better how that works.  If you set the plcnpratio to 1 instead of 44 as in the latest code, it works reading and writing large arrays of strings.  If you want to have this as an option that you specify when connecting, I will add it, but I don't want to force it to be that way as I don't want to break any existing applications.  Either way I will close the issue, it works now with single-dimension arrays and we can support the 2D ones easily if anyone asks for the option.


WinAppDriver doesn't work with java sample

I am trying to run my test with java, but it is not work, so I tried to run with Visual Studio and it is work very well.
I am using WinAppDriver.exe

I receive this error when I run my test with Java and Intellij, my code is the same of microsoft's oficial repo.

My machine is windows 10
My machine is able to developer mode
My machine has WinAppDriver.exe instaled

-JAVA

-VISUAL STUDIO

can someone helps me with this error?
I very appreciate for some help.

Hi everyone,
I solved the problem with changing the dependency of appium, now I am using the

    <dependency>
        <groupId>io.appium</groupId>
        <artifactId>java-client</artifactId>
        <version>5.0.2</version>
    </dependency>


Port original pkg to ROS2

You can get it to ignore .dae files
On Tue, 12 Mar 2024, 16:33 Alejandro Bordallo, @.***>
wrote:

@.**** commented on this pull request.
In car_description/package.xml
https://github.com/ipab-rad/car_description/pull/1#discussion_r1521788538
:

<test_depend>ament_lint_auto</test_depend>

<test_depend>ament_lint_common</test_depend>

Thanks for the feedback!
Removed these couple of lines and brought in the pre-commit config file
you pointed to. I ran it and (Apart from over-zealously "correcting" a
"spelling-mistake" in the .dae file) did well!
I'll play with it for a bit before I tune it further.
—
Reply to this email directly, view it on GitHub
https://github.com/ipab-rad/car_description/pull/1#discussion_r1521788538,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA24PYPK7WI73OZI4IUFKNTYX4U6HAVCNFSM6AAAAABELNZL52VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMZRGU4DMOJQGQ
.
You are receiving this because you commented.Message ID:
@.***>

@GreatAlexander I have updated the meshes files to be able to visualise the vehicle on RVIZ without any loading errors. The vehicle now is shown in grey color

@GreatAlexander I think this branch can be merged :)

For every PR where you are a reviewer, please:

Start conversations if things need changing
Once happy, please approve before it can be merged (In our special case for these PRs, we should test them on the AV before they can actually be merged)

For every PR where you are an author, please:

Work on it and asking for feedback is optional if it is a draft PR
Ask for a review and work on feedback if it is Open (I have now set it to open)


explicit generators (feature request)

ripser had an experimental branch (which was never finished) to compute explicit generators.
it would be nice to have the following functionality:

return representatives of explicit generators
decompose an inputted cycle into generators

Thanks for the input! I have some semi-working code experiments for that, I hope I will soon find the time to make them publishable. Of course, I am also happy to accept pull requests.

Hi @luetge , I'm curious as to whether you have the code for representatives working yet. It would be awesome to have that functionality, and I would be willing to help finish off the functionality, especially if you can share what you have thus far to give me a starting point.


exhibitor: write JNA libraries to somewhere that isn't noexec

High Level Description
By default the JNA tmpdir is set to /tmp. This conflicts with some standard security policies that dictate that /tmp should be mounted as noexec.
@alberts's first thought is to use /run, but that is mounted noexec by default on Ubuntu 16.04 LTS, too.
We should set the JNA tmpdir to somewhere we control: /var/lib/dcos/exhibitor/tmp seems like a good option.
Related Issues

DCOS-634 zookepeer should not write JNA libraries to /tmp
DCOS-11056 The requirment of mounting /tmp without noexec flag conflicts with security policies that some customers have

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: This is a config change - if it doesn't work zookeeper doesn't start and we'll notice.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@alberts, @lingmann please review

Merged in train 128 - #1225


fix: accept globs and array of globs for model option in config

fixes: https://github.com/aerogear/graphback/issues/977
For backwards compatibility specifying a dir like ./model will still work as it did before, but it also means you can supply globs or arrays. For example, the following are both valid now.
extensions:
  # Graphback configuration
  graphback:
    ##  Input schema
    model: ./model/**/*.graphql

extensions:
  # Graphback configuration
  graphback:
    ##  Input schema
    model:
      - ./model/comment.graphql
     - ./model/note.graphql

There is also a custom loadSchema function in graphql-serve which I am more than happy to update too if you think it's worth it

To not go into a discussion with details about I think that proposed changes are removing couple really troubling issues and will help us to maintain backward compatibility and better flexibility for model creation.

@craicoverflow thanks for the review, I've updated it now and I think it's a bit cleaner. I've also updated the docs and other places where we were using ./model as a value

@darahayes I have just noticed that the runtime template fails to run because it loads the model the old way:
https://github.com/aerogear/graphback/blob/4a10783e73dd1d30522289cb8831ebbcdc712226/templates/ts-apollo-runtime-backend/src/loadSchema.ts#L12-L24
This can probably be quick fixed for now - I was in the process I removing the custom loadSchema from the template here.
The model should not be loaded in the runtime template at all, so this can probably get resolved later in https://github.com/aerogear/graphback/issues/1071


Convert from_type to parent logical type before attempting a cast.

This is part of larger PR for polars https://github.com/pola-rs/polars/issues/9373.
Though it does seem to be the right thing when doing a cast and dealing with extension types.

Closing PR as per https://github.com/pola-rs/polars/issues/9373#issuecomment-1602265718


[HW] Manuel Aguilera

Name: Manuel Aguilera
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/manuelaguileraa/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:
I got it to work until step 5 , When updating dependencies in the pom.xml to include Cassandra dependencies , All of the dependencies include a version except for "com.google.code.gson" . Therefore, it gives me the following error message.

Congratulations . ! Awesome workshop!

Hey Manuel Aguilera, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/-SBrNHykTA6SYUBooz7I-Q


fix: fix passing multiple audio files as args

imagine the arguments ./encore something.png something.mp3 this won't work, because encore originally intreprets something.png as a non-audio file, so it tries to parse it as a playlist, and dies because &str/String expects to be utf8.
whereas... ./encore something.mp3 something.png works right, because encore treats it correctly (audio files instead of playlist)
this commit pr fixes that behaviour and makes it right, but due to how it was written originally, it had to be refactored. its probably not the best way to fix it, or rewrite it in this case, but it works.

been dogfooding a bit, it works


Agenda - DevOps Mutualization SIG Meeting - Thursday 16 September 2021

Date
Thursday 16 September 2021 - 12pm EST / 5pm UK
Untracked attendees

Name
Firm
Comment

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[ ] Convene, roll call, welcome new people
[ ] Display FINOS Antitrust Policy summary slide
[ ] Approve previous meeting minutes
[ ] DevOps SIG events at FINOS Member Meetings in London and New York // Promo code available for FINOS member registration - https://events.linuxfoundation.org/open-source-strategy-forum-london/register/

[ ]  London  - Monday 4th October from 12pm BST - Automated Change Deployment Process, Tooling, and Regulatory Response // cc @guspaul
[ ]  New York  - Tuesday 9th November - DevOps Self Service tooling and other potential open source project ideas

[ ] Automated Change Deployment Working Group Update - https://github.com/finos-labs/devops-mutualization-automated-change-deployment

[ ] Question : When should the WG next meet?
[ ] Question : Who else should be recruited in order to help move forward?

[ ] What other topics should DevOps SIG focus on in 2022?
[ ] AOB, Q&A & Adjourn (5mins)

Sept 28th to Sept 30th : DevOps World - FINOS DevOps Mutualization Special Interest Group Panel Discussion

James McLeod, Director of Community at FINOS will lead this panel discussion with Amol Shukla from Morgan Stanley and Eduardo Ferreira Dos Santos from Itaú Unibanco. How DevOps mutualization unites subject matter experts from across Engineering, IT, Compliance, Security and Audit to solve common financial services DevOps problems.

Decisions Made

[ ] ...

Action Items

[ ] ...

WebEx info
Webex:
https://finos.webex.com/finos/j.php?MTID=md62056638ba05fbea86b9582df94f807
Dial-in

US +1-415-655-0003 US Toll
UK +44-20319-88141 UK Toll
Access code:<PHONE_NUMBER>

Github Repo: https://github.com/finos/devops-mutualization/
Mailing List: Email<EMAIL_ADDRESS>to subscribe to our mailing list

Hello 👋🏻

o/

here

Hello

👋  🌊  Hi everyone!


Set element's max-height to avoid reflow

Changing the original element's height would cause the browser to reflow, in a very very long page,  this might be a performance issue, what about setting element's max-height before the binary search, and using scrollHeight or creating a detached element with an absolute position?

Hey @zaaack! Sorry, I didn't see this message until now. I'll look asap!

Hei, never mind, thanks for your awesome project!


Can not do inference using simple_inference_waymo.py

Fist of all,i want to know what does this scripts do?detecion or track.
and what checkpoint should i use?
Then when i download waymo test data,the data is below:
{'scene_name': '10504764403039842352_460_000_480_000', 'frame_name': '10504764403039842352_460_000_480_000_location_other_Dawn/Dusk_1558060923374527', 'frame_id': 0, 'lidars': {'points_xyz': array([[-37.286133  ,   3.137207  ,   3.7067337 ],
       [-37.28418   ,   3.2282715 ,   3.7067184 ],
       [-37.375     ,   3.3269043 ,   3.7104187 ],
       ...,
       [ -6.729186  ,   8.910209  ,  -0.27966785],
       [  0.15275276,  14.4378395 ,  -0.3565687 ],
       [ -0.6661949 ,   6.959557  ,  -0.15457088]], dtype=float32), 'points_feature': array([[0.25195312, 0.05269789],
       [0.37890625, 0.03513193],
       [0.33203125, 0.05269789],
       ...,
       [0.02099609, 0.63823   ],
       [0.05126953, 0.32204267],
       [0.01080322, 0.15809368]], dtype=float32)}}

get error below:
Traceback (most recent call last):
  File "tools/simple_inference_waymo_custom.py", line 154, in <module>
    detections = process_example(points, args.fp16)
  File "tools/simple_inference_waymo_custom.py", line 100, in process_example
    output = run_model(points, fp16)
  File "tools/simple_inference_waymo_custom.py", line 92, in run_model
    outputs = model(data_dict, return_loss=False)[0]
  File "/root/miniconda3/envs/centerpoint/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in __call__
    result = self.forward(*input, **kwargs)
  File "/centerpoint/det3d/models/detectors/two_stage.py", line 156, in forward
    return_loss, **kwargs)
  File "/centerpoint/det3d/models/detectors/voxelnet.py", line 73, in forward_two_stage
    x, voxel_feature = self.extract_feat(data)
  File "/centerpoint/det3d/models/detectors/voxelnet.py", line 24, in extract_feat
    input_features = self.reader(data["features"], data["num_voxels"])
  File "/root/miniconda3/envs/centerpoint/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in __call__
    result = self.forward(*input, **kwargs)
  File "/centerpoint/det3d/models/readers/voxel_encoder.py", line 20, in forward
    assert self.num_input_features == features.shape[-1]
AssertionError

My point only contain x,y,z,Where can i get data include 6 input features?quite confused.

@tianweiy, I got this error when trying to use simple_inference_waymo.py. I am using the latest spconv version.
Did you met such an issue? Thanks!
x, voxel_feature = self.backbone(
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/code/fmf/FMF/det3d/models/backbones/scn.py", line 156, in forward
    x = self.conv_input(ret)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/spconv/modules.py", line 134, in forward
    input = module(input)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/spconv/conv.py", line 196, in forward
    out_features = Fsp.indice_subm_conv(features, self.weight,
  File "/usr/local/lib/python3.8/dist-packages/spconv/functional.py", line 87, in forward
    return ops.indice_conv(features,
  File "/usr/local/lib/python3.8/dist-packages/spconv/ops.py", line 118, in indice_conv
    return torch.ops.spconv.indice_conv(features, filters, indice_pairs,
RuntimeError: expected scalar type Float but found Half

Both model and input need to be converted to fp16

I convert only voxels : voxels = voxels.half(). I think that is all, right?

No, also need model = model.half

Thanks. It worked!


Multi diff editor: implement focus and hasFocus

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.86.0
OS Version: MacOs

Steps to reproduce:

Split window to 2 tab groups
Open normal editor in 1st tab group.
Open multi diff editor in 2nd tab group.
Focus normal editor.
Run View: Focus Next Editor Group

Expected: multi diff editor is focused.
Current: normal editor stays focused.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@hediet the multi diff editor needs to make sure to implement some methods from EditorPane to better integrate into the workbench. For this specifically:

focus
hasFocus

I think you already have some experience in this area, given the merge editor:
https://github.com/microsoft/vscode/blob/983a3bc0aecfc0772dd7b37919d7da36c89a6f2d/src/vs/workbench/contrib/mergeEditor/browser/view/mergeEditor.ts#L455-L468
Let me know if you need help.

Started a PR here https://github.com/microsoft/vscode/pull/204970 but need some help.

Fixed in #204970

https://github.com/microsoft/vscode/assets/29464607/2b1cb8c2-bd9c-40f5-9178-5ebf9f3f78fd


Update password and Eye button

Is this also completely done ? @va24nsh

Eye button functionality is implemented. Will start working on update password.


SteamVR_Input._default.inActions no longer generated in Beta 08 and 09

SteamVR_Input._default no longer has inActions reference? Am I missing a change in the documentation? I haven't been able to upgrade to latest beta because of the missing generated inActions code.
https://steamcommunity.com/sharedfiles/filedetails/?id=1416820276
I was able to reproduce the missing inActions reference by downloading Beta 09, installing it in a new Unity 2018.2.0f2 project and clicking "Save and generate".
using the following code after adding it in the SteamVR Input UI in unity:
SteamVR_Input._default.inActions.TrackpadTouch.GetStateDown(SteamVR_Input_Input_Sources.LeftHand);
I get a null reference for inActions

Thanks! I'll fix this in the next RC