Local edits lost if the file on the server is changed.

Steps:

Make some edits in TN and save in tCC

then go to DCS and make a change to my branch on the same file

Go back to TCC and make some new edits and try to save a new change.

tCC throws error like this:

After login back, the changes on the server are not seen but everytime user makes an edit the error message pops up, but the edits are not reflected or saved in the branch.

Notes from Rich: Under the hood, in the network response, I can see:
{"message":"sha does not match [given: dfb6a4bd57849cf8b06979bf35316cafd4d1a8af, expected: 90dd1e6d363343a06e5183350f36fe1a44a53340]","url":"https://git.door43.org/api/swagger"}
I worked with a content editor previously where I was the one who changed the file on the server. Yet then they kept logging in again, with their changes still there, but still wouldn't save. Apparently tCC still wasn't getting the new SHA. I'll ask him what is happening this time, but does it really let you clobber once you log in again? Before it wasn't.
The previous time, when I had made a commit, I had to undo my commit, then it worked. Maybe I can undo his last save if his system is still using the previous save's file SHA. Will try that. But doesn't seem to be updating the SHA in tCC.
I think this error message used to give us the two SHAs that don't compare, but was too technical so now doesn't show it. Wonder if still a way for them to easily get it without doing all the dev stuff in their browser.

@birchamp Instead of the login dialog box, there should be the modal warning that the "Save action would clobber the file on the server, do you want to keep or discard the server changes?" like how we had it when autosave feature was enabled.

I think this should be added to the Push Pull epic.  I believe it will need to be addressed then. Other than that, I believe this is a rare use case. @elsylambert did you mean for this issue to be in gatewayEdit.

I accidentally added it in GatewayEdit zenhub board instead of tCC. I just moved it to TCC here https://app.zenhub.com/workspaces/tc-create-workspace-5f049cdbfa88090016bf043e/issues/gh/unfoldingword/tc-create-app/1549


Consider delimiter length while right truncating directories

Truncate the path from the right by taking in account the delimiter length, so that directories with names shorter than truncated name + delimiter aren't incorrectly truncated.
For example, if SHORTEN_DIR_LENGTH is 4 and the delimiter is ..``, ../tests/..and../custom/..are not incorrectly "truncated" to../test../..and../cust../..`, both of which are longer or the same length as the original untruncated directory names.

Great work, @eviltak, and thanks for updating the test case appropriately!

Thank you @bhilburn! I'll (or someone else can) check and see if a similar fix is applicable to the truncate middle configuration.
For those interested in knowing how I did this, I took advantage of the fact that sed replaces only when the regex is matched. The regex should be fairly self explanatory: (([^/]{$POWERLEVEL9K_SHORTEN_DIR_LENGTH})([^/]{$delim_len}))[^/]+/
The above regex matches only if the number of characters in the directory name is greater than POWERLEVEL9K_SHORTEN_DIR_LENGTH + the delimiter length. I then insert the second matched group (\2), that is the first inner group which matches the maximum number of characters.


don't use spaces in repo ids

@jamesnetherton @tdiesler can you guys merge? This was breaking prod builds.


OpenGUI: missing required input 'ODB  on Nix

Description
In Openlane I used to use ./flow.tcl -design spm -tag v1 -interactivecommand to visualize heatmap of the existing run with the tag v1 if the congestion is too high on global routing and flow has failed.
I want to do the same on openlane2. I run the following command to open up a gui to inspect a failed run.
python3 -m openlane ./openlane/examples/spm/config.json --run-tag v1 -f OpenInOpenROAD

https://github.com/efabless/openlane2/blob/499367593546b27c7583f6aaaf9bdfa4313ef56f/openlane/steps/openroad.py#L1979
Expected Behavior
Open up a GUI so I can inspect the heatmap.
Environment report
Failed to get Docker info: [Errno 2] No such file or directory: 'docker'
kernel: Linux
kernel_version: 6.7.1
supported: True
distro: nixos
distro_version: 24.05
python_version: 3.11.6
python_path:
  - /nix/store/xc12z7izn7psa0af9xqm6drzda7x5xq1-python3.11-openlane/bin
  - /nix/store/xanckpmyylk3d0ygpy11d223r1q77f96-python3-3.11.6-env/lib/python3.11/site-packages
  - /nix/store/qp5zys77biz7imbk6yy85q5pdv7qk84j-python3-3.11.6/lib/python311.zip
  - /nix/store/qp5zys77biz7imbk6yy85q5pdv7qk84j-python3-3.11.6/lib/python3.11
  - /nix/store/qp5zys77biz7imbk6yy85q5pdv7qk84j-python3-3.11.6/lib/python3.11/lib-dynload
  - /nix/store/qp5zys77biz7imbk6yy85q5pdv7qk84j-python3-3.11.6/lib/python3.11/site-packages
  - /nix/store/xc12z7izn7psa0af9xqm6drzda7x5xq1-python3.11-openlane/lib/python3.11/site-packages
  - /nix/store/2z0jalw27lp6zm642aqvnlydlqblhcdf-python3.11-click-8.1.7/lib/python3.11/site-packages
  - /nix/store/m1a3bk12cjgk6wq445pi7xch3v2gx8ql-python3.11-cloup-3.0.2/lib/python3.11/site-packages
  - /nix/store/a8z71acldic6hqdbd18sd81sqcqgygd3-python3.11-pyyaml-6.0.1/lib/python3.11/site-packages
  - /nix/store/jlb8fjb88bx2adivdz8pq8cx5aj5615h-python3.11-rich-13.5.2/lib/python3.11/site-packages
  - /nix/store/8x0qa1j4z2907a9pvvf9lwzs6n58i5m5-python3.11-markdown-it-py-3.0.0/lib/python3.11/site-packages
  - /nix/store/69l2fjl76rad6fz5n5v38px2kiwcz5x8-python3.11-mdurl-0.1.2/lib/python3.11/site-packages
  - /nix/store/ljd22jdzx6kj0hj7wsnhr71gqa4iqj88-python3.11-pygments-2.16.1/lib/python3.11/site-packages
  - /nix/store/y8yq448xka665bqyd5gvdhajqdbsnyys-python3.11-setuptools-68.2.2/lib/python3.11/site-packages
  - /nix/store/71js2p8zx17fqgq2m1b5f7q92ig0v7nn-python3.11-requests-2.31.0/lib/python3.11/site-packages
  - /nix/store/6jj18jqyik8k06plad2hm9fwcrnhpxf3-python3.11-brotlicffi-<IP_ADDRESS>/lib/python3.11/site-packages
  - /nix/store/qxsnv4322wqi9w629b572k898viw756h-python3.11-cffi-1.16.0/lib/python3.11/site-packages
  - /nix/store/r0mzj4k7gzx3bp4mbnr69x6if6lxxsb1-python3.11-pycparser-2.21/lib/python3.11/site-packages
  - /nix/store/hk5xz5kyhr1vaq2p65990c2fxw7mxlmp-python3.11-certifi-2023.07.22/lib/python3.11/site-packages
  - /nix/store/g3kzbrnirxky2w4g3n67p01745pa8xcc-python3.11-charset-normalizer-3.2.0/lib/python3.11/site-packages
  - /nix/store/g3w192qs0avwjaw776yiyaq5l0dcbwdp-python3.11-idna-3.4/lib/python3.11/site-packages
  - /nix/store/71236y06cag3yb637q5jj358ci6f20ah-python3.11-urllib3-2.0.7/lib/python3.11/site-packages
  - /nix/store/bnadfkpcdkmrzymnpyj8x2bdxq1f6kdw-python3.11-pcpp-1.30/lib/python3.11/site-packages
  - /nix/store/i41k9kc0s446fw5vl4k05vfq3prw06hv-python3.11-volare/lib/python3.11/site-packages
  - /nix/store/i46q5s4xj3700m2srcx4rd2bv2x6w51h-python3.11-httpx-0.25.0/lib/python3.11/site-packages
  - /nix/store/6ckbxrfj5q372da9svdh7hnh67vbwyhc-python3.11-httpcore-0.18.0/lib/python3.11/site-packages
  - /nix/store/43sv2g5lsl58md1lzidlx1g7xam2mpqp-python3.11-anyio-4.0.0/lib/python3.11/site-packages
  - /nix/store/4m6453i7as66sdmkck8fiwk793lx1b29-python3.11-sniffio-1.3.0/lib/python3.11/site-packages
  - /nix/store/1c9w7989rjr417rs50bxsrinjf63mz8n-python3.11-h11-0.14.0/lib/python3.11/site-packages
  - /nix/store/q9c0733drxk1zv5j60hq0lz61g395dic-python3.11-rfc3986-2.0.0/lib/python3.11/site-packages
  - /nix/store/i24kszh5ajb03gwhjlmb1df6sm0s0x72-python3.11-zstandard-0.21.0/lib/python3.11/site-packages
  - /nix/store/608w6rc3xa1mj44ppara7pnmr6z6kbpn-python3.11-tkinter-3.11.6/lib/python3.11/site-packages
  - /nix/store/k2xqs2rp3kyb0pbq1sn9v0sh05q0cmvc-python3.11-lxml-4.9.3-3/lib/python3.11/site-packages
  - /nix/store/vp0ivbg65mnsjvfgyran0x1bzsf3x66l-python3.11-deprecated-1.2.14/lib/python3.11/site-packages
  - /nix/store/bj6g1njs2z0ivrwlpkls9qr1ar1psjaw-python3.11-wrapt-1.14.1/lib/python3.11/site-packages
  - /nix/store/cghmw4316dgv9f86y6prdybi50v1f8sn-python3.11-immutabledict-3.0.0/lib/python3.11/site-packages
  - /nix/store/s0xm1cjrghaa6g11b8nivv64ga6bpwwd-python3.11-libparse/lib/python3.11/site-packages
  - /nix/store/a9v9j1mwxp5ij7yijfvahm30bfadxfr3-python3.11-ioplace_parser/lib/python3.11/site-packages
  - /nix/store/1z7vs2crryc63akgcyzyq9is33p7y230-python3.11-antlr4-python3-runtime-4.10.1/lib/python3.11/site-packages
  - /nix/store/8wkz5a9z1ppwjvr54375k69avhkmz4nc-python3.11-psutil-5.9.6/lib/python3.11/site-packages
  - /nix/store/6bxns8xhf27bimgb5sfzxmk89pjfmycd-klayout-pymod/lib/python3.11/site-packages
tkinter: True
container_info: None
nix_info:
  version: nix-env (Nix) 2.18.1
  multi_user: True
  sandbox: True
  channels: None
  nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Reproduction material
499367593546b27c7583f6aaaf9bdfa4313ef56f is the commit id.
I assume nix shell is reproducible. Otherwise something is missing in flake.nix
Relevant log output
[23:03:08] INFO     Loading the design configuration file has generated the following warnings:                                                    config.py:671
...
[23:03:08] WARNING  Key 'VERILOG_FILES' provided is unused by the current flow.                                                                    config.py:675
[23:03:08] INFO     Starting a new run of the 'Opening in OpenROAD' flow with the tag 'RUN_2024-01-29_23-03-08'.                                     flow.py:525
[23:03:08] INFO     Starting…                                                                                sequential.py:222
───────────────────────────────────────────────────────────────────────── Open In GUI ──────────────────────────────────────────────────────────────────────────
[23:03:08] VERBOSE  Running 'OpenROAD.OpenGUI'… (Log: ./openlane/examples/preg/runs/RUN_2024-01-29_23-03-08/1-openroad-opengui/openroad-opengui.log) step.py:878
[23:03:08] ERROR    The flow has encountered an unexpected error: OpenGUI: missing required input 'ODB'                                          __main__.py:146
Traceback (most recent call last):
  File "/home/emre/Desktop/openlane2/openlane/flows/sequential.py", line 266, in run
    current_state = step.start(
                    ^^^^^^^^^^^
  File "/home/emre/Desktop/openlane2/openlane/steps/step.py", line 900, in start
    raise StepException(
openlane.steps.step.StepException: OpenGUI: missing required input 'ODB'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/emre/Desktop/openlane2/openlane/__main__.py", line 135, in run
    flow.start(
  File "/home/emre/Desktop/openlane2/openlane/flows/flow.py", line 555, in start
    final_state, step_objects = self.run(
                                ^^^^^^^^^
  File "/home/emre/Desktop/openlane2/openlane/flows/sequential.py", line 271, in run
    raise FlowException(str(e))
openlane.flows.flow.FlowException: OpenGUI: missing required input 'ODB'

@hakan-demirli There should be a message similar to this:
15:19:14] VERBOSE  Using state at                                                                                flow.py:526
                    './openlane/examples/spm/runs/v1/61-misc-reportmanufacturability/stat
                    e_out.json'.

Indicating that the flow accepted --run-tag v1. Are you sure you were passing this argument in the output above?

I assume nix shell is reproducible. Otherwise something is missing in flake.nix

For future reference: that's different. For issues that involve bugs in OpenLane, we typically require the design files so we can isolate and fix the issue quickly. Nix eliminates variance in your environment, not in your design files. :)

As @kareefardi said, make sure you pass --run-tag v1 to both the initial run and OpenInOpenROAD, as the log output here implies that the run tag v1 was not found.

I have solved all superfluous errors. The real issue was Qt dynamic plugin loading.
Cannot mix incompatible Qt library (5.15.12) with this library (5.15.11)
I missed the error first time and performed other operations which evolved into the error given in the first message.
This was a nixpkgs bug which has fixed a long time ago with the introduction of wrapQtAppsHook. And, you guys are using wrapQtAppsHook. So, no problem on the openlane2 side.
I assume this is a Nixos bug. There are other peoples on nixos forums complaining about the same issue on different packages. I have changed the nixpkgs of the flake to unstable which resolved the issue.

flake.nix
...
    nixpkgs.url = "github:NixOS/nixpkgs/nixpkgs-unstable";
...

flake.lock
    {
  "nodes": {
    "flake-compat": {
      "locked": {
        "narHash": "sha256-kvjfFW7WAETZlt09AgDn1MrtKzP7t90Vf7vypd3OL1U=",
        "rev": "0f9255e01c2351cc7d116c072cb317785dd33b33",
        "revCount": 57,
        "type": "tarball",
        "url": "https://api.flakehub.com/f/pinned/edolstra/flake-compat/1.0.1/018afb31-abd1-7bff-a5e4-cff7e18efb7a/source.tar.gz"
      },
      "original": {
        "type": "tarball",
        "url": "https://flakehub.com/f/edolstra/flake-compat/1.tar.gz"
      }
    },
    "nixpkgs": {
      "locked": {
        "lastModified":<PHONE_NUMBER>,
        "narHash": "sha256-LE8lVX28MV2jWJsidW13D2qrHU/RUUONendL2Q/WlJg=",
        "owner": "NixOS",
        "repo": "nixpkgs",
        "rev": "90f456026d284c22b3e3497be980b2e47d0b28ac",
        "type": "github"
      },
      "original": {
        "owner": "NixOS",
        "ref": "nixpkgs-unstable",
        "repo": "nixpkgs",
        "type": "github"
      }
    },
    "root": {
      "inputs": {
        "flake-compat": "flake-compat",
        "nixpkgs": "nixpkgs"
      }
    }
  },
  "root": "root",
  "version": 7
}

Related: https://github.com/NixOS/nixpkgs/issues/86369


Fix bug

I had a nasty bug in the Cholesky code. This PR fixes it and adds a test which fails on master.

Essentially the implementation was just plain wrong, and was often terminating way before it should have.


UI - Progress widget

Is your feature request related to a problem? Please describe.
As someone learning how to code, I want to see how far I am generally across the languages/technologies I know.
Describe the solution you'd like
There should be a widget on the dashboard that displays all paths and the progress the user is through them.
Additional context
May want to use a canvas to draw the arc.
https://www.figma.com/file/MfOc0a8Ac9Zoh9RppoEuiD/CM-UI?node-id=2%3A0
Data
This will need dummy data for now. It should follow the format of:
interface PathProgress {
  path: Path // Still being worked on by @raysercast1
  progress: number; // 0 - 1
}

// Dummy interface until it's available from #5
interface Path {
  id: string;
  name: string;
  icon: string;
  description: string;
}

<ProgressWidget /> // fetches its own data (mocked for now)

Hi Tristan, I'd like to take a stab at this.
Should I create dummy data for this?

Hey @ahmedfaaid, shame you weren't part of the UI call we just had.
Yeah that's a good quetion. I'll update the specs for what properties it should query for as dummy data.

Hey @ahmedfaaid , how you doing with this? (Just cleaning up issues)a

Hi @tristanMatthias . So far mostly a static widget. My question is what kind of data can I seed into my local postgres database to help with more dynamic data?

Sorry for late response @ahmedfaaid, just saw this now. Seeders have been merged into develop, just waiting on #5 to have the paths added. Setting this as blocked till that's available.

Hey @ahmedfaaid, #68 was merged in now, so paths should be available for you to pull down. Have a chat with @suzynakayama to learn how to create a Container unstated-next for Paths in the front end.

That's great! Will do so.

Hey @tristanMatthias I don't think I got an invitation to the team. I can't seem to push my code.

Hey @ahmedfaaid, fixed now. Sorry about that!


Line Chart fills when colors are used

Bug report
Using apexcharts 3.27.3
When using the line chart, if I have only one line and I use a pre-defined pallete, then it shows the line just fine.
      @chart_options = {
        noData: {
          text: I18n.t('not_enough_data', scope: 'data_analyses.report'),
          align: 'center',
          verticalAlign: 'middle',
          offsetX: 0,
          offsetY: 0,
          style: {
            # color: undefined,
            fontSize: '14px',
            # fontFamily: undefined
          }
        },
        theme: 'palette1',
        fill: {
          opacity: 0.0
       }
      }

However, if I change the colors with parameters:
      @chart_options = {
        noData: {
          text: I18n.t('not_enough_data', scope: 'data_analyses.report'),
          align: 'center',
          verticalAlign: 'middle',
          offsetX: 0,
          offsetY: 0,
          style: {
            # color: undefined,
            fontSize: '14px',
            # fontFamily: undefined
          }
        },
        theme: 'palette1',
        colors: [DARK_BLUE, AJAX_CHARTS_GREEN, AJAX_CHARTS_ORANGE],
        fill: {
          opacity: 0.0
       }
      }

I get a fill pattern under the line.

I expect that if I add colors that override a template, that either the line chart doesn't create a fill below the line or I have some means of stopping this with a configuration parameter.

What is happening instead?
I am getting a fill pattern under the line on the chart.

What error message are you getting?
No error messages are received.

Please provide a reproduction app


Update to aws-crt-swift 0.33.0

Describe the feature
This version includes support for "string array" endpoint properties, and general updates to CRT C libraries.
Use Case
To enable use of latest CRT features & fixes, and to enable completion of https://github.com/awslabs/aws-sdk-swift/issues/1517.
Proposed Solution
No response
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Shipped in AWS SDK for Swift 0.57.0


ModuleNotFoundError: No module named 'libs.sepconv._ext.cunnex._cunnex'

Hi, I have tried to get the code running however I get this:
Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/rf/sepconv/src/main.py", line 16, in
from src.model import Net
File "/home/rf/sepconv/src/model.py", line 10, in
from libs.sepconv.SeparableConvolution import SeparableConvolution
File "/home/rf/sepconv/libs/sepconv/SeparableConvolution.py", line 8, in
import libs.sepconv._ext.cunnex
File "/home/rf/sepconv/libs/sepconv/_ext/cunnex/init.py", line 3, in
from ._cunnex import lib as _lib, ffi as _ffi
ModuleNotFoundError: No module named 'libs.sepconv._ext.cunnex._cunnex'
I have tested this on 3 different systems with the required software version including cuda installed, searching up this issue has yielded no useful answers
Any suggestions as to what's going wrong?
Thanks in advance

Hi, I have tried to get the code running however I get this:
Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/rf/sepconv/src/main.py", line 16, in
from src.model import Net
File "/home/rf/sepconv/src/model.py", line 10, in
from libs.sepconv.SeparableConvolution import SeparableConvolution
File "/home/rf/sepconv/libs/sepconv/SeparableConvolution.py", line 8, in
import libs.sepconv._ext.cunnex
File "/home/rf/sepconv/libs/sepconv/_ext/cunnex/init.py", line 3, in
from ._cunnex import lib as _lib, ffi as _ffi
ModuleNotFoundError: No module named 'libs.sepconv._ext.cunnex._cunnex'
I have tested this on 3 different systems with the required software version including cuda installed, searching up this issue has yielded no useful answers
Any suggestions as to what's going wrong?
Thanks in advance

you should install sepconv in ./lib/sepconv first.

Hi, I have tried to get the code running however I get this:
Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/rf/sepconv/src/main.py", line 16, in
from src.model import Net
File "/home/rf/sepconv/src/model.py", line 10, in
from libs.sepconv.SeparableConvolution import SeparableConvolution
File "/home/rf/sepconv/libs/sepconv/SeparableConvolution.py", line 8, in
import libs.sepconv._ext.cunnex
File "/home/rf/sepconv/libs/sepconv/_ext/cunnex/init.py", line 3, in
from ._cunnex import lib as _lib, ffi as _ffi
ModuleNotFoundError: No module named 'libs.sepconv._ext.cunnex._cunnex'
I have tested this on 3 different systems with the required software version including cuda installed, searching up this issue has yielded no useful answers
Any suggestions as to what's going wrong?
Thanks in advance

you should install sepconv in ./lib/sepconv first.

When I execute the bash file, there still seems to be some problems，


Pueue lambda promise

@pueue -- for some reason the CI build was concerned with the eslintrc, I took your branch, updated the .eslintrc.js then bumped the version.  Will merge when the CI build passes.
Great work!


Fix to encoding of vault name bug bug #3318 per issue #61

It would be nice if this worked.

Is the underlying bug in boto not fixed already? What's the reason the workaround has to be applied here, instead of the user using a fixed version of boto?

You are quite right, they have. The mistake I was making was using the version of boto specified in your requirements.txt (2.38) and I can confirm the bug is fixed in their latest (2.48)
I'll send you another pull request for updating that file.


Update post

Motivation
Closes #1617
Changes
This updates go-spacemesh to include https://github.com/spacemeshos/post/pull/34 and https://github.com/spacemeshos/post/pull/35.
https://github.com/spacemeshos/post/compare/3ade80883426...dfb8a5803e6d
Test Plan
Unit tests on both post and go-spacemesh pass. This is not a functional change, but a performance optimization, so no new tests are required. Automation tests also pass.

bors try

bors merge


Allow the recolor plugin to use grayscale images

This means "cutout" type images where a white shape is placed against a
black one now works correctly.

One minor request, could you update the readme to specify the updated behavior?


gather and expose metrics

https://github.com/choria-io/go-choria/issues/1007
metrics should age out so ones over 10 minutes are not shown, all metrics are gauges

done via local prom instead https://github.com/choria-io/go-choria/pull/1011


Cleanup dev dependencies

Remove celluloid-io: we don't use it anymore
Remove Ruby 1.9isms
Add more :require => false

:+1:


Testing lists holding moves

Added debug messages and a test list to see what went wrong with my last commit. I intend to revert this but hopefully I can get a better understanding of how the code works.

Marked as a draft untill im online again


Refactor NetworkController and PeerManager actors

NetworkController and PeerManager actors looks to be very tightly coupled and have complex mutual communications. Looks like they share same responsibility and this makes the code rather unclear.
I suggest to refactor these two actors. I suppose to make  PeerManager just an local storage component for NetworkController. We still will store peerDatabase and connection state there, but will access it not as an actor, but just using local reference. All external API messages for PeerManager will go to NetworkController. We can provide an abstract superclass for them if we will need to clean NetworkController from API logics.

I generally agree with reducing the number of actors in Scorex. Currently, it seem that everything is an actor in Scorex, but this doesn't need to be so.
I tend to agree in the particular case of PeerManager as well. One concern I have is that we wouldn't want NetworkController to be blocked for too long waiting for a non-actor Peer Manager to complete an operation. But I guess that, if this concern really becomes a problem, we could solve it by using Future when calling the methods of PeerManager from NetworkController.

Do we have heavy operations in  Peer Manager that could block NetworkController if we made them single actor?

As far as I know, we don't. But I don't know everything. That is why I wanted to raise this potential issue.
@kushti, what do you think?

Trying to get up to speed on the project so I wanted to see if any of these sentiments make sense to y'all.
The actor model is perfect for the boundaries of the system where concurrent events are occurring (NetworkManager). NetworkManager, however, doesn't need to have this asynchronous relationship with PeerManager.


add citation

Add a citation to the readme.

Author list should include https://github.com/victor-paltz https://github.com/nateagr https://github.com/hitchhicker
I think victor should be the first author, he did most of the work before release
Bo should be second author, he did most of the distributed implementation

what is the full name of nateagr?

It's Gaëtan Racist
Can you finish this PR please?


Update assignment code and a bit work on the projector

Updated AssignmentBallot class
use generic _create_ballot_common to reduce code
yield correct items for the projector
added dummy slide-assignment-poll on client
bind motion and motion poll for the delegate table

@FinnStutzenstein You are in high speed mode. I have a hard time keeping up with you ;-). I'm reviewing this now.


Fix some small issues in the xilem_core view macros (hardcoded identifiers and missing ViewMarker for Box)

Feel free to land yours first, I can rebase as needed.

I can merge this if need be, but I've also sent an invite to Philipp, as "author merges" is our usual procedure.


Vitest execution in a browser page

Clear and concise description of the problem
Currently, vitest is executing test in the classic way on the node.js engine.  I would like to execute my (non UI) vitest also on real browsers in their environment.
The benefit is to test for granular differences in different browsers like missing features or APIs. For example IndexedDB or Crypto related test.
Suggested solution
Reuse vitest environment to allow test execution in a web page. vite should be able to do the necessary updates quickly. Just provide a simple output like the one we get with the --ui option.
There is no need for automation regarding launching browsers etc. this could be achieved by integrating calls to the web pages by Cypress, Puppeteer, Playwright etc. if required.
Alternative
Cypress, Playwright, etc. provide their own test environment and focus on user interfaces.
Additional context
I made a tiny implementation for my own purposes: https://github.com/holtwick/zeed/blob/master/demos/tests/src/jest.ts
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Vitest is designed to run only inside Node. That's why we have vite-node. For example, mocking is relying on it.
Out general recommentation is to use Cypress for UI testing.

I see. Yeah, it is probably out of scope.

@holtwick If I understand you correctly, mocha-vite-puppeteer might give you what you want, as it runs code and tests inside Puppeteer, instead of inside Node. mocha-vite-puppeteer is powered by Vite and should be just as fast as vitest. You can find a somewhat realistic demo project using mocha-vite-puppeteer here: https://github.com/larsthorup/react-redux-typescript-vite-sandbox

I guess this comes close to what I intended: https://github.com/vitest-dev/vitest/pull/891
I created a kind of runtime to be loaded before running a regular behavioral test. Maybe this gist gives an idea of it: https://gist.github.com/holtwick/ec4219363808a0d066ef02fd6c405077

Now that #891 is on-going, I think this issue could be re-opened for discussion related to topic.
How should vitest handle coverage when tests are running in the browser? Currently vitest is using nodejs's V8 API + c8. In browsers there are similar API's. For example Puppeteer seems to be utilizing Chromium's Profiler:

https://github.com/puppeteer/puppeteer/blob/8ff9d598bf4afd10cbc61ca9579b7bd38edb8026/src/common/Coverage.ts#L237-L241
https://chromedevtools.github.io/devtools-protocol/tot/Profiler/

This would mean that coverage collection would be limited to Chromium based browsers only. Using something like nyc would be easy but not ideal now that c8 has been chosen on the nodejs side.

Introduced browser testing is very limited at the moment, and doesn't support many features (including module mocking and snapshots). And not even released :) Not sure if we should discuss it here, or open a proper roadmap issue

I'm interested when browser testing becomes more generally available.
My use case is for writing isomorphic JavaScript modules that can run on Node.js as well as in the browser.
I'm currently using Karma + Jasmine to test in browsers, while just using Jasmine for Node, which gives me the benefit of only having to write the tests once, but there's still a bit of overhead that I'd like to simplify.

If anyone is using Vite and wants browser tests, Cypress Component Testing works with your existing Vite config out of the box. https://docs.cypress.io/guides/component-testing/overview
It's aimed at component tests and UI development, but this might be useful for people in the "I am using Vite, but I want to run some tests in a browser, too".


🐛 Fixes missing copyright header

What this PR does / why we need it:
Adds missing copyright header to the suite_test.go file.

/test-integration

/close


DARK MODE

Want to add dark mode for good ux

want to add dark mode for better ui

want to add dark mode for better ui

@heyankit78 is already working on this issue. Please look after some other issues. 😀

Assign me this issue.


Updating is more difficult than it needs to

With most frameworks or apis you specify the identifier of the resource you want to change and then what you want to change.
But with this api you always have to specify everything.
Normal flow: send what to update to server.
Chirpstack flow: get current state. update state. send updated state to server

I'm not sure if this is (easily) fixable. With gRPC fields that are not specified fall back to their default values. E.g. a bool type field that is not specified will be interpreted as false. It would require an other mechanism (e.g.  specifying a list of fields that one would like to modify), this would require a considerable amount of code-changes which I don't think is worth the effort....

There actually is a common pattern for doing partial updates in gRPC using FieldMasks. However, it is almost always used in support of PATCH in grpc-gateway (https://grpc-ecosystem.github.io/grpc-gateway/docs/mapping/patch_feature/), and I can't remember having seen it as part of any vanilla gRPC API documentation.  The syntax is also kinda unfriendly to use outside of PATCH.
I do disagree with "most frameworks or apis" defaulting to partial updates, but that's not really important here.


Check if page components exist

Currently the testing framework allows you to test if a page component actually exists. This behaviour is configured in the /config/inertia.php file:
'testing' => [
    'ensure_pages_exist' => true,
    'page_paths' => [
        resource_path('js/Pages'),
    ],
    'page_extensions' => ['js', 'svelte', 'ts', 'vue'],
],

However, it would be great to do this all the time. Currently if you try to visit a page that has a component that doesn't exist, an exception isn't thrown. All you get is a console error. In production, this is especially problematic, because the user has no idea something went wrong.
Is there any reason why we can't just do this automatically all the time? Yes, there would be an extremely minor performance hit to checking if the file exists. That practically doesn't seem like a real issue.
I'm thinking that we remove the testing property from the config, and just have these options available top level. The ensure_pages_exist property could accept an array of environments to run this check in:
'ensure_pages_exist' => ['local', 'production', 'testing'], // or set to "true" for always
'page_paths' => [
    resource_path('js/Pages'),
],
'page_extensions' => ['js', 'svelte', 'ts', 'vue'],

Honestly, I don't see why you'd want to do this as part of every request, as once a deployment is live, all but the initial request (meaning, the 99.99%) will be unnecessary overhead that doesn't add any real value?
Instead, if this is a real concern, I'd probably use a bug-tracking library on the JS-side of things (which, with a large JS app, isn't a bad idea anyway) that automatically reports console errors/bugs to a bug tracker like https://docs.bugsnag.com/platforms/javascript/
I mean, the alternative is catching it on your back-end, and then sending it to the same bug tracker, but then it involves extra logic, and the whole extra filesystem check that just generally feels unnecessary.


Add accepted languages in how to contribute translation page

改善詳細 / Details of Improvement
The answer in https://github.com/tokyo-metropolitan-gov/covid19/issues/1696#issuecomment-650719771 should be explicitly written in How to contribute translations
page
Instead of stating that new languages might not be accepted, it should specify that only Japanese, English, Simplified Chinese, Traditional Chinese, and Korean can be provided in the website due to reviewing capacity.
スクリーンショット / Screenshot

期待する見せ方・挙動 / Expected behavior

Following messages should be added

Due to reviewing capacity, we allow only Japanese, English, Simplified Chinese, Traditional Chinese, and Korean can be provided in the website

動作環境・ブラウザ / Environment

@wiennat, thank you for your suggestion! I totally agree to add an explanation to the document.
Could you make changes to the document? If not, I'm willing to do that.


surround key name with single quotes for readability

This is a super, super, super-duper minor change that helps with readability. I find that user input fields are sometimes hard to read and a message like:
"Calling code requested setting named  but it was not in the config file. (if the appSettings key was blank)
or
Calling code requested setting named key but it was not in the config file. (in the case of a poorly named appSettings key)
Single quotes help in these cases.

No such thing as a too-small improvement :)
I'll get the packages up on NuGet later this evening.

Hey, thanks for a super-great and much needed package btw. I use this everywhere!


Fix link :active/:hover color accessibility in the examples

/cc @reznord

'preact create my-great-app' still doesn't have different color on the active route?


fix: covid academic years off

changes

I believe the default COVID academic years are off. Check my thinking here, but Spring 2020 is actually the academic year 2019-2020, and so on.
context

questions

even offset by a year, the cumnum covid terms was still a predictive feature in my prelim hacc model -- presumably because there's overlap with the actual covid terms. very curious to see how the proper alignment affects the feature's predictive power.


⚠ Bofrost.at has degraded performance

In 61e825b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1832 ms

Resolved: Bofrost.at performance has improved in 2e65e8e.


AIForged Add new dataset operations (Certified Connector)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @InusBot,
Could you please check this swagger validation error and fix the same?
The type of the property is incorrect. Expected type 'String', but value is of type 'Null'. ValuePath: /paths/~1api~1DataSet~1GetByDef/get/parameters/2/x-ms-summary

@vmanoharas
The request fix has been applied

@vmanoharas
All spelling mistakes have been rectified.

Hi @InusBot,
Thank you very much fixing all the comments.
There is another swagger validation issue "Loops of required schema properties are not allowed. ValuePath: /definitions/AIForged.ViewModels.CustomDataSet"
Could you please refer this document and fix the same, this might block you when you upload the files into ISV studio.
https://learn.microsoft.com/en-us/connectors/custom-connectors/certification-swagger-validator-rules#response

Good day @vmanoharas
I have added new commits that removed some operations that should fix the merge issues.

Hello @InusBot,
This PR is already approved, kindly upload your artifacts into Connector certification portal if you are not done already.


brednik.toml

Please provide some contact info provided. Not merged for now due to no contact info


Cow: Watch fastflashc.com

Cow requests the watch of the watch_keyword fastflashc\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by double-beep in Charcoal HQ

fastflashc\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Removing wrong letters in right-to-left texts

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Type rtl text to multiline textfield :
(DO NOT COPY AND PASTE)
بابا آب داد
با مهربانی
4طتانا جمعا 4واتا +یکسوییت 20متری
بنبابا 40ساله
Expected results
The letter to the right of the cursor should be deleted.
(It should be noted that when I copy the text from above and paste it in the text field, everything works correctly.)
https://github.com/flutter/flutter/assets/32570964/b3d09305-3b20-4388-b938-2ce451217a79
Actual results
But when I type the rtl text , the letter on the left side of the cursor is deleted.
https://github.com/flutter/flutter/assets/32570964/31f9ee2d-3aeb-441a-8568-97dcf7fdefe7
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:flutter_localizations/flutter_localizations.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      locale: const Locale("ar"),
      localizationsDelegates: const [
        GlobalMaterialLocalizations.delegate,
        GlobalWidgetsLocalizations.delegate,
        GlobalCupertinoLocalizations.delegate,
      ],
      supportedLocales: const [Locale("ar")],
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: const Center(
        child: Padding(
          padding: EdgeInsets.all(16),
          child: TextField(
            maxLines: 1000,
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/32570964/d06fc103-aca5-4df4-8242-fdc400247346

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.16.2, on Microsoft Windows [Version 10.0.19045.3636], locale en-US)
    • Flutter version 3.16.2 on channel stable at C:\Flutter\SDK
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9e1c857886 (3 days ago), 2023-11-30 11:51:18 -0600
    • Engine revision cf7a9d0800
    • Dart version 3.2.2
    • DevTools version 2.28.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Program Files\Android\Sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_SDK_ROOT = C:\Program Files\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.0)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.8.34309.116
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] VS Code, 64-bit edition (version 1.70.2)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.54.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.3636]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 113.0.5672.93
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 119.0.2151.58

[√] Network resources
    • All expected network resources are available.

• No issues found!

Thanks for the report @mohammadjalalis
I ran your code and observed below:
https://github.com/flutter/flutter/assets/67046386/9b1f8083-188e-4f24-84ae-13f4d7a2c9bd
When I tried deleting, it started to delete the letter / words that are on left side of the cursor, but the expected behavior is, the letter / words to the right of cursor should get deleted. Is this a correct interpretation of the issue you are reporting ?

Thanks for the update. Based on the findings and report, labeling for tracking.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.16.0, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.16.0 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (12 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.18.0-2.0.pre.22, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.18.0-2.0.pre.22 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision bfaacd5ef8 (81 minutes ago), 2023-12-04 22:48:25 -0500
    • Engine revision f43ef6cbdd
    • Dart version 3.3.0 (build 3.3.0-190.0.dev)
    • DevTools version 2.30.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I tried  20متری  in an RTL paragraph but couldn't reproduce the issue. @mohammadjalalis could you provide more detailed instructions to help us reproduce this?

In single-line mode, everything is OK.
But in multi-line mode, when we decide to delete a letter, usually the next letter is deleted. And in general, the placement of the cursor in multi-line mode in right-to-left texts is very problematic.
Video description:
From second 1 to 25, you can see that everything is correct in single-line mode, and when we try to remove the letter "ر", the same "ر" is removed.
From 25 to 1:40, we type a three-line RTL text(Persian language), when we decide to remove the letter "ر" in the second line, the letter "ی" is removed instead, which is incorrect.
From 1:40 to the end with the arrow keys on the keyboard, when we move the pointer over the text, the cursor is not in the right place, and sometimes it moves three times on some letters. For example, in the first line, the cursor moves three times on the word "آب" and the letter "ب".
But the interesting thing is that in the last line, the cursor moves completely correct and is placed at the end of each letter and spaces.
https://github.com/flutter/flutter/assets/32570964/12e18d3a-a5a0-42b2-8080-e566a11a4e5c
Tip:
I recently noticed an issue, that is, when we type the same text, for example, in the GitHub editor and copy it to the Flutter Textfield, everything is exactly as we expect.
The reason is the addition of "\r" before "\n".
I hope this tip helps.
In this video, I copied and pasted the following text into the Textfield.
بابا آب داد
20 متری
تا ابد
https://github.com/flutter/flutter/assets/32570964/6e563a8e-4fcc-4f6f-9d1c-1e45ee6cdfce

Unfortunately I still couldn't reproduce the issue. Could you add a listener to the controller:
  controller.addListener(() {
    print(controller.text.codeUnits);
    print(controller.value.selection);
    print(controller.value.composing);
  });

and copy paste the output before you press the backspace?

This is code :
import 'package:flutter/material.dart';
import 'package:flutter_localizations/flutter_localizations.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      locale: const Locale("fa"),
      localizationsDelegates: const [
        GlobalMaterialLocalizations.delegate,
        GlobalWidgetsLocalizations.delegate,
        GlobalCupertinoLocalizations.delegate,
      ],
      supportedLocales: const [Locale("fa")],
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final controller = TextEditingController();

  @override
  void initState() {
    controller.addListener(() {
      print(controller.text.codeUnits);
      print(controller.value.selection);
      print(controller.value.composing);
    });
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Padding(
          padding: const EdgeInsets.all(16),
          child: TextField(
            controller: controller,
            maxLines: 20,
            style: const TextStyle(
              fontSize: 100,
            ),
          ),
        ),
      ),
    );
  }
}

This is log:

flutter: [1576, 1575, 1576, 1575, 32, 1570, 1576, 32, 1583, 1575, 1583, 10, 50, 48, 32, 1605, 1578, 1585, 1740, 10, 1578, 1575, 32, 1575, 1576, 1583]
flutter: TextSelection.collapsed(offset: 19, affinity: TextAffinity.upstream, isDirectional: false)
flutter: TextRange(start: -1, end: -1)

Thanks for the detailed repro! When the caret offset is 19 the caret should be placed after "ی"

So this appears to be a caret placement issue.

Ah I reproduced this on Windows.

It's interesting that this is platform specific since the cursor placement logic in the framework is supposed to be platform-agnostic. I doubt this has to do with DirectWrite. Maybe this has something to do with CRLF.
Maybe related: https://github.com/flutter/flutter/issues/140739

When we use the built-in font on Android, everything works fine; however, there are issues when we use a custom font.
In this case, the caret is not positioned correctly and the selection of words is incorrect.
https://github.com/flutter/flutter/assets/32570964/166ef398-6999-4b61-9eaa-fe8c92583eaa

@LongCatIsLooong Did you check this out?

This is important and has a huge impact on the performance of the RTL applications. Please check and fix this issue.
@LongCatIsLooong @Renzo-Olivares @darshankawar

I solved my problem temporarily with this piece of code. But it is much better if this problem is fixed permanently by the Flutter team.
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: Text('RTL Text with \r\n on Enter'),
        ),
        body: MyForm(),
      ),
    );
  }
}

class MyForm extends StatefulWidget {
  @override
  _MyFormState createState() => _MyFormState();
}

class _MyFormState extends State<MyForm> {
  final TextEditingController _textEditingController = TextEditingController();

  @override
  void initState() {
    _textEditingController.addListener(() {
      print(_textEditingController.text.codeUnits);
      print(_textEditingController.selection);
      print(_textEditingController.value.composing);
    });

    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return Padding(
      padding: const EdgeInsets.all(16.0),
      child: TextField(
        controller: _textEditingController,
        keyboardType: TextInputType.multiline,
        maxLines: null, // Allow multiple lines
        textAlign: TextAlign.center,
        textDirection: TextDirection.rtl,
        style: const TextStyle(
          fontFamily: 'IranBakh',
          fontSize: 50,
        ),
        decoration: const InputDecoration(
          hintText: 'Type RTL text here',
        ),
        onChanged: (text) {
          // Handle cursor movement and backspace
          int cursorPosition = _textEditingController.selection.baseOffset;

          _textEditingController.value = TextEditingValue(
            text: text,
            selection: TextSelection.fromPosition(
              TextPosition(offset: cursorPosition),
            ),
          );

          if (text.isNotEmpty && text[cursorPosition - 1] == '\n') {
            print("Enter key pressed!");
            // Check if '\r' already exists before all '\n'
            bool existingCarriageReturn = _textEditingController.text.contains('\n');

            // Replace '\n' with '\r\n' only if '\r' doesn't already exist before all '\n'
            if (existingCarriageReturn) {
              _handleEnterKey(cursorPosition);
            }
          }
        },
      ),
    );
  }

  void _handleEnterKey(int cursorPosition) {
    String text = _textEditingController.text;

    String newText = '';

    for (int i = 0; i < text.length; i++) {
      if (text[i] == '\n' && (i == 0 || text[i - 1] != '\r')) {
        // Add '\r' before '\n' if '\r' doesn't exist before it
        newText += '\r\n';
        cursorPosition = cursorPosition + 1;
      } else {
        newText += text[i];
        cursorPosition = cursorPosition;
      }
    }

    newText = newText.replaceAll('\r\n\r\n', '\r\n \r\n');

    _textEditingController.value = TextEditingValue(
      text: newText,
      selection: TextSelection.fromPosition(
        TextPosition(offset: cursorPosition), // Restore cursor position
      ),
    );
  }

  @override
  void dispose() {
    _textEditingController.dispose();
    super.dispose();
  }
}


refactor: Update indexer tutorial to make it work and avoid dependency troubles

I've updated tutorial to work with recent master of nearcore and avoid troubles with dependencies. Thanks @amgando for good catch!
@thisisjoshford some weirdos with test-links as I took a working master and it still found broken links, don't have enough expertise in our docs world to decide how exactly it should be fixed. The same problems I have in my other PR #711

@thisisjoshford some weirdos with test-links as I took a working master and it still found broken links, don't have enough expertise in our docs world to decide how exactly it should be fixed. The same problems I have in my other PR #711

These two links were fixed in master branch. I merged it and everything looks good to go :) Try merging master into #711 and see if that resolves your issue.


Re-organise docs pages in a slightly better way

This PR re-organises the docs pages in a slightly better way.
Astropy and most affiliated packages call the tutorial intro "getting started" and the installation instructions page "installation", so this is just slightly more standard.
And by avoiding the second toctree the navigation tree on the side works lightly better IMO, it now looks like this:

@leejjoon - OK with you?
(I'll now start a new PR to add Sphinx autodoc API docs.)

Coverage remained the same at 50.662% when pulling 9ee0c65ba82ef5bd620366416cf699dfa4bf4b79 on cdeil:docs-org into 511d6da0a9034b67db14ddc6c5281648cf789957 on astropy:master.

I'll just merge this now so that I can continue with the API docs.
Comments / suggestions to improve the docs still welcome or course!


The gap between inference results and the reported reuslts in paper

hi, your work is great！But I wonder why there is a gap between inference results (about 28.4db on GOPRO) using the provided pretrained model and the reported reuslts (31.67db) in paper?

Thanks for your attention.
If you want to test on GOPRO dataset, please make sure that it uses the pretrained model named "CDVD_TSP_GOPRO.pt".

Thanks for your attention.
If you want to test on GOPRO dataset, please make sure that it uses the pretrained model named "CDVD_TSP_GOPRO.pt".

Thanks for your reply: )！For GOPRO dataset, I'm sure I use "CDVD_TSP_GOPRO.pt", and for DVD dataset, I use "CDVD_TSP_DVD_Convergent.pt" which also cannot obtain the reported results.

I just re-downloaded the GOPRO test dataset and the pretrained model from the given link "https://drive.google.com/drive/folders/1lw_1jITafEQ9DvMys_S6aYwtNApYKWsz?usp=sharing".
And, I ran the commands:
cd ./code
python inference.py --default_data GOPRO
Then, I got the same results reported in the paper.
So, please make sure that you followed the "https://github.com/csbhr/CDVD-TSP#quick-test" steps in the README.

I just re-downloaded the GOPRO test dataset and the pretrained model from the given link "https://drive.google.com/drive/folders/1lw_1jITafEQ9DvMys_S6aYwtNApYKWsz?usp=sharing".
And, I ran the commands:
cd ./code
python inference.py --default_data GOPRO
Then, I got the same results reported in the paper.
So, please make sure that you followed the "https://github.com/csbhr/CDVD-TSP#quick-test" steps in the README.

Thanks for your efforts!
I use the below command to test the performance previously：
python main.py --dir_data ../dataset/GOPRO/train --dir_data_test ../dataset/GOPRO/test --epochs 1 --batch_size 8 --test_only --pre_train ../pretrain_models/CDVD_TSP_GOPRO.pt
And following your last reply, I have retested the performance on the two datasets again using the quick-test commands you recommend. But I still can't obtain the reported performance (only about 28.+dB).

You can try to re-download the codes, test datasets and pretrained models.
Then run the commands again:
cd ./code
python inference.py --default_data GOPRO

Here is the inference log on GOPRO dataset, that I tested last night.
https://drive.google.com/file/d/1nMDERzlZRgkszPZcl8kIlFTf0CPQj824/view?usp=sharing

You can try to re-download the codes, test datasets and pretrained models.
Then run the commands again:
cd ./code
python inference.py --default_data GOPRO

I re-download the codes and re-test the performance again. Finally obtain the results:
Total AVG-PSNR=28.565, AVG-SSIM=0.8746
So it seems a little strange. Whether you modify the codes after releasing the work or use another pretrained model to test？

Again, I downloaded the codes, pre-trained models and test dataset.
I did not modify anything, just ran the commands:
cd ./code
python inference.py --default_data GOPRO
Then, I got the same results reported in the paper.
Please check whether there is something wrong with your operation.


The name 'Configuration' does not exist in the current context

When registering the database context, if I have the code like that I get the error "The name 'Configuration' does not exist in the current context". Can anyone help?

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

ID: b34c4871-adc3-0d13-6f20-6f6ac1f65e91
Version Independent ID: ea1742b3-808b-d449-3308-6847a9f339f9
Content: Add a model to an ASP.NET Core Razor Pages app with Visual Studio Code
Content Source: aspnetcore/tutorials/razor-pages-vsc/model.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Add
using Microsoft.Extensions.Configuration;

BTW, why are you using VS Code and not VS?

Thanks for that. I am using vscode for other projects and I prefer it. VS seems to do a lot more (like automatically adding dependencies like this I guess) and I prefer to see a bit more explicitly what's going on

VS doesn't automatically add dependencies, it just tells you what you need to add so you don't need to open an issue :)

@Rick-Anderson landed here from a google search - which helped me solve my problem. I would encourage MS to actually see this as a problem. Having problems understanding what's going on using the command line tools for a tool that supposedly supports the command line and other setups / OSs... "use Visual Studio" isn't ideal.
Thanks for all your hard work. I am having fun playing around with .net on linux :).

I agree with the above comment. Microsoft behaves as platform exclusive and act like they don't. I would like to use whatever editor I choose to use.
Plus, in ASP.NET core 6; you get the method from the builder object like so: builder.Configuration.GetConnectionString("BloggingDatabase")
And this solution should've been easy to search, but I had to visit a couple links to find it.

Add using Microsoft.Extensions.Configuration;

I am using Visual Studio 2022 and that doesn't work.
builder.Services.AddDbContext(options => options.UseSqlServer(configuration.GetConnectionString("ConnStr")));
Error	CS0103	The name 'configuration' does not exist in the current context


MacOS py-runtime-pkg snapshot build failing

Currently failing on a submodule problem (but others didn't):
2021-01-27T22:20:37.0340620Z ##[error]fatal: repository 'https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator.git/' not found
2021-01-27T22:20:37.0505120Z Fetched in submodule path 'third_party/vulkan_memory_allocator', but it did not contain 3528e2aed3e8808f33e1e7d63eeb1560456a605a. Direct fetching of that commit failed.
2021-01-27T22:20:37.1163540Z ##[error]The process '/usr/local/bin/git' failed with exit code 1

The prior build was failing on a python config problem, so there may be a stack of issues. Should check after next build in case this is a transient.

Obsolete (probably transient, appears relatively stable now)


install-mongdb fails on osx, bad architecture

It looks like its trying to fetch a nonexistent i386 version
dmacd$ mongoctl install 2.2.5
Running install for osx 32bit to mongoDBInstallationsDirectory (/Users/dmacd/mongodb)...
ERROR: Unable to download from url 'http://fastdl.mongodb.org/osx/mongodb-osx-i386-2.2.5.tgz' (response code '403'). It could be that version '2.2.5' you specified does not exist. Please double check the version you provide
Running the python 2.7.5 build from python.org.
Would love to use your tool, is there any way to provide a literal url?

Cannot reproduce. Closing. Please create a new issue if you are still seeing this. Thanks!


Changed configuration mechanism

Changed configuration mechanism: os.Env and configurable variables instead of hard coded data on the file system.

Already merged into dev.


Measure the impact of the ranking publication

Measure the impact of the ranking publication.
It could be something like this:

Propose some changes in the ranking. Something like "Add the word #GitHubRanking in your profile"
Follow the evolution of ranking, saving the data every week
Compare the evolution for people that add "#GitHubRanking" and other than don't

It's a good idea.
We can add "#GitHubRanking" to our location and create a ranking using this.

Good for the next version, this one's got to be ready this week... today or
tomorrow, if possible.


Getting exception when retrieving records from volatile memory

Describe the bug
Hi Im getting folwing error in Chat Copilot app with volatile memory on line 81 in webapi\CopilotChat\Skills\ChatSkills\DocumentMemorySkill.cs file.
inner exception is:
To Reproduce
Steps to reproduce the behavior:

Register a plugin.
Ask a question that plugin solves
Ask another question.'

Expected behavior
Retrieve similar records from volatile memory.
Screenshots

Platform

OS: [Windows]
IDE: [Visual Studio 2022]
Language: [C#]
Source: [e.g. NuGet package version 0.1.0, pip package version 0.1.0, main branch of repository]

Additional context
InnerException    {"The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.\r\nStatus: 404 (Not Found)\r\nErrorCode: DeploymentNotFound\r\n\r\nContent:\r\n{"error":{"code":"DeploymentNotFound", "message":"The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again."}}\r\n\r\nHeaders:\r\nOpenAI-Processing-Ms: REDACTED\r\nx-ms-client-request-id: 1652580f-151c-46c0-a75b-bc71e7b8a4b5\r\napim-request-id: REDACTED\r\nStrict-Transport-Security: REDACTED\r\nX-Content-Type-Options: REDACTED\r\nx-ms-region: REDACTED\r\nDate: Tue, 08 Aug 2023 09:19:46 GMT\r\nContent-Length: 198\r\nContent-Type: application/json\r\n"}    System.Exception {Azure.RequestFailedException}
Stack trace:
at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.d__251.MoveNext()    at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.<InternalGetEmbeddingsAsync>d__14.MoveNext()    at Microsoft.SemanticKernel.AI.Embeddings.EmbeddingGenerationExtensions.<GenerateEmbeddingAsync>d__02.MoveNext()
at Microsoft.SemanticKernel.Memory.SemanticTextMemory.d__7.MoveNext()
at Microsoft.SemanticKernel.Memory.SemanticTextMemory.d__7.System.Threading.Tasks.Sources.IValueTaskSource<System.Boolean>.GetResult(Int16 token)
at SemanticKernel.Service.CopilotChat.Skills.ChatSkills.DocumentMemorySkill.d__4.MoveNext() in C:\Work\chat-copilot-main\webapi\CopilotChat\Skills\ChatSkills\DocumentMemorySkill.cs:line 81

@alliscode I was using model name instead of deployment name for text embedding model. Sorry for being dumdum! :D


Switch from react-native-camera to react-native-vision-camera

Is your feature request related to a problem? Please describe.
react-native-camera is deprecated/archived and react-native-vision-camera is recommended instead.
Describe the solution you'd like
Migrate to react-native-vision-camera.
Describe alternatives you've considered
Look for another library or keep the current one.
Additional context

I realized that I didn't work on this ticket for more than a month and honestly also feel like I don't have the capacity to do this. I think the major blocker was react-native-camera. Therefore, I'll unassign myself.

Quote from https://github.com/digitalfabrik/lunes-app/issues/983#issuecomment-1752911517. Not sure what this blocked but probably the react-native upgrade (#1010).
Related: #983, #1010.

Duplicate. Closing in favor of #873.


Explorer: cannot copy text while renaming a file

VSCode Version:
1.19.0 Version 1.19.0-insider
Commit d8989c95964ce06c64ffebc2fb400a0327bd447d
Date 2017-12-08T10:45:40.870Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

OS Version: Windows

Steps to Reproduce:

right-click and on a file in file explorer and choose rename
press ctrl+c to copy selected text into the clipboard

What happens?
The selected text isn't copied to the clipboard.

The Insiders build for Windows just got out where this issue is fixed.


Allow specifying cacheControl attributes on Types and Fields

 Issue by kbrandwijk
Wednesday Oct 25, 2017 at 17:56 GMT
Originally opened as https://github.com/graphcool/prisma/issues/1032

What feature are you missing?
I am trying to configure caching on my Apollo API Gateway. For this, I need to add @cacheControl directives to my Types and fields. When I try to do that, the server doesn't throw any error, but the directives are silently removed. Unfortunately, mergeSchemas does not allow redefining fields, so it would be helpful if I could add them on the Graphcool side.
Also, the top-level queries that are generated are not accessible from the schema. It would be nice to be able to define cacheControl attributes for the generated query fields as well somehow.
Apollo Cache Control is an open standard, that will not only be implemented by Apollo Server, but also Sangria, etc. (according to their docs), so this is not a 'vendor-specific' thing.
How could this feature look like in detail? Tradeoffs?

Do not remove additional directives.
Proposal for generated query fields is to take the cacheControl attribute off the Type, so if I define:

type Post @model @cacheControl(maxAge: 30) {
 ...
}

Then the generated queries would also get that setting:
type Query {
   allPosts(...) @cacheControl(maxAge: 30)
   Post (...) @cacheControl(maxAge: 30)
}

Depends on:

[ ] Expose user-defined directives in API schema #1228

Any news on keeping those directives when stitching schemas together? Thanks!


how i put paymentGetwayUrl default_payment_profile_apple_pay

where should i set paymentGetwayUrl ?
i cannot find it in default_payment_profile_apple_pay.json
https://github.com/google-pay/flutter-plugin/blob/main/pay/example/assets/default_payment_profile_apple_pay.json

Hi @Alababdiy.
Where are you intending to that parameter?


Channel IDs starting with D

I see some channel ids starting with D. From what I deduced, each user has a corresponding channel ID that starts with D. Could you please give more info. Also, can you tell me how to get this association between user ids starting with U to channel ids starting with D

Channels starting with 'D' are direct message channels. For the three types, see https://api.slack.com/types (channel, group, im)


(#10739) Provide default subjectAltNames while bootstrapping master

Prior to #2848 (CVE-2011-3872), if Puppet[:certdnsnames] was not set,
puppet would add default subjectAltNames to any non-CA cert it
signed. The subjectAltNames were of the form:
DNS:puppet, DNS:, DNS:puppet.
The fix for #2848, prevented implicit subjectAltNames from ever being
added. However, this is desirable behavior when bootstrapping the
initial puppet master.
This commit re-enables this behavior but restricts it to the case
where all of the following are true:

We are a CA and master
We're signing the master's cert, not self-signing the CA
The CSR is for the current host

These should only ever be true when bootstrapping the initial
master. In particular, it should never be true for the CA's
self-signed cert, for remote agents, or for servers that are either
masters or CAs, but not both.
This commit also restores the previous behavior of only adding default
subject alt names if the fqdn of the host can be resolved.
Since the CSR for the initial master may contain subjectAltNames,
e.g. by setting Puppet[:dns_alt_names] prior to starting the master
for the first time, then the default subjectAltNames will not be
merged -- it's either one or the other, but not both.

On Tue, Nov 29, 2011 at 11:00, Josh Cooper
<EMAIL_ADDRESS>wrote:

Prior to #2848 (CVE-2011-3872), if Puppet[:certdnsnames] was not set,
puppet would add default subjectAltNames to any non-CA cert it
signed. The subjectAltNames were of the form:
DNS:puppet, DNS:, DNS:puppet.
The fix for #2848, prevented implicit subjectAltNames from ever being
added. However, this is desirable behavior when bootstrapping the
initial puppet master.

This is a necessary fix, but the approach isn't perfect.  Josh is
going to rework it to better match our desire to have explicit rather
than implicit behaviour around SSL certificates and requests.
Daniel
⎋ Puppet Labs Developer – http://puppetlabs.com
♲ Made with 100 percent post-consumer electrons


How to utilize META information for forecasting

It seems that chronos only supports uni-variate input, but if there is additional information (meta information like event) at a future point in time that we want to predict, can we utilize it?

@cbhaaa currently not: the models only take historical observations as input

Closing as related to #22


5.3.201 Major bug - The same OnChange method is triggered in all the open panels

Hi,
this is the more general case of my issue when opening the same panel twice and that I continue to investigate while developing my panels.
I'm re-using similar Called when modulator value change methods with the same name in all my panels.
I noticed that when you move one modulator in one panel, calling the OnChange method is done with source=4 in the current panel (OK) but then the same method is also launched in the other panel but with source=1
This leak looks like a major bug...
With the subcase of opening 2 times the same panel, I solve it by testing on source=6 and source=1 at the beginning of the method. I thought it was due to the presence of utils.xxx functions but in fact the utils.xxx functions are just making the issue visible.

@RomanKubiak coudl it be that this is indeed a feature. I don't recall the exact place but as far as I know, there are panels that should be separarted from each other and others that share some data.

well... not sure it is a feature because it is just a disaster when you open the same panel on different tracks in a VST.
Hopefully testing source=6 and =1 solves this completely.


Release 1-18 postsubmit

This isn't right exactly, but we need something like this to build the release

/retest

/hold


accordion toggle stopped working

Accordion toggle in 1.1.4 release fails with
runtime-core.esm-bundler.js:226 TypeError: Cannot read properties of undefined (reading 'toggle')
at Proxy.toggleAccordion (Ora2PgConfig.vue:174:20)
at onClick (Ora2PgConfig.vue:31:19)
at o (runtime-core.esm-bundler.js:155:22)
at l (runtime-core.esm-bundler.js:164:21)
at HTMLHeadingElement.s (runtime-dom.esm-bundler.js:366:13)

Possibly related:
runtime-core.esm-bundler.js?d2dd:38 [Vue warn]: Invalid prop: type check failed for prop "config". Expected Object, got Boolean with value false.
at <Ora2PgConfig ref="configComponent" project="default" config=false  ... >
at


Rails won´t start

Hi all,
my rails does weird stuff and does not start anymore. Don't know what I changed, I only tried to connect my farmbot which did not work. I get:
rails -s <IP_ADDRESS> WARNING: Use strings for Figaro configuration. 587 was converted to "587". rails aborted! Don't know how to build task '<IP_ADDRESS>' (see --tasks) /home/klimbim/.rvm/gems/ruby-2.3.3@farmbot-web-app/gems/railties-<IP_ADDRESS>/lib/rails/commands/rake_proxy.rb:13:in block in run_rake_task'
/home/klimbim/.rvm/gems/ruby-2.3.3@farmbot-web-app/gems/railties-<IP_ADDRESS>/lib/rails/commands/rake_proxy.rb:10:in run_rake_task' /home/klimbim/.rvm/gems/ruby-2.3.3@farmbot-web-app/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:51:in run_command!'
/home/klimbim/.rvm/gems/ruby-2.3.3@farmbot-web-app/gems/railties-<IP_ADDRESS>/lib/rails/commands.rb:18:in <top (required)>' bin/rails:4:in require'
bin/rails:4:in <main>' (See full trace by running task with --trace)
Anyone any idea?

Instead of rails -s try rails s --port=<IP_ADDRESS>.
Please let me know if there is documentation anywhere with the -s, as that would be a spelling error that needs to be corrected. Also, doing --port=<IP_ADDRESS> will bind to all ports, which will make debugging easier.

@RickCarlino I guess you meant rails s -b <IP_ADDRESS> being <IP_ADDRESS> the ip in the local LAN; this works for me!

@Klimbim1 Whoops! Yes, -b for "bind to IP".


Allow keras constraints that affect both the kernel and the bias at the same time.

System information.
TensorFlow version (you are using): R interface to Keras v2.4.0
Are you willing to contribute it (Yes/No) : No
Describe the feature and the current behavior/state.
With the current implementation of keras layers, weight constraints can be applied as kernel or bias constraints. This also means that when writing a custom constraint, it has an input parameter w that refers either to the kernel weights or the bias weights depeding on how it is used.
However, at least for research purposes, it would be useful to be able to include both the kernel and the bias together as input to a constraint. As an example, suppose that in a dense layer you want to constrain the L1 norm of the incident wegiths to a given neuron unit to be less than 1, but considering the bias at that unit as an extra element of the weight vector, ending up with a weight vector of length (input dimension + 1). Currently this simple constraint is not possible (as far as I know) because the input to a custom constraint is either bias or kernel weights, not both.
In this sense, even if the kernel_constraint and bias_constraint division is maintained, being able to use both kernel and bias weights as input to a custom constraint  would allow to use it with the two options kernel_constraint and bias_constraint and obtain the desired effect.
Will this change the current api? How?
Probably would be sufficient to add an extra option when creating custom constraints and how the constraints receive their inputs in the training process.
Who will benefit from this feature?
Anyone wanting to constraint the kernel and bias at the same time, for example computing the norm of the full weight vector incident to a neuron in a dense layer, including the bias.
Contributing

Do you want to contribute a PR? (yes/no): no
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

Is there any idea on how this change in the constraints behaviour could be approached?

@moralapablo Did you try to combine the weight and bias into a single tensor with a constraint. After that you may split apart the tensor in the layers call method. Please refer to this doc and let us know if it helps?
Thank you!


Unsecure Windows App

Hi, Windows 11 tells me the installation programm for jwlmanager is insecure. So it was stopped by the Microsoft Defender SmartScreen. The used asset was:
https://github.com/erykjj/jwlmanager/releases/download/v1.2.1/JWLManager_v1.2.1.exe
Any suggestions how to be sure the Defender is wrong?

Thank you for reporting.
How to be sure the Defender is wrong? Well, have a look here. These are the virus scan results for the file your Defender blocked: https://www.virustotal.com/gui/file/31c427784446669f2e09055ce8c4c338b02d6edeb5b4a55308223b986d77ae7e/detection
You'll notice the great majority of anti-virus products report is as clean. In particular, notice the important/big players: Microsoft, Google, Avast, Kaspersky, McAfee, etc. So why do some report trojans and such? Well...
The app is basically a Python 3 script, and (for convenience) I use PyInstaller to package it up into a Windows executable. It's this process of packaging up a script that some over-zealous anti-virus products flag as dangerous, since the script could be malicious.
So, if you prefer, you can install Python (and the required dependencies) and run the script directly. Other options include Virtualbox or Sandboxie-Plus. But, of course, that's only if you don't trust me :-)

@lexiconwriter , would you be willing to test a couple of "alternate" packages to see if they produce a similar alert from Windows Defender?

Hi Eryk,
thank you for the virus results. Funny that Microsoft alerts with one tool
but not with the other one.
If you need someone to test packages I can do it for u.
Kindly
Oliver
Am So., 18. Dez. 2022 um 01:58 Uhr schrieb Eryk J. @.***

:

@lexiconwriter https://github.com/lexiconwriter , would you be willing
to test a couple of "alternate" packages to see if they produce a similar
alert from Windows Defender?
—
Reply to this email directly, view it on GitHub
https://github.com/erykjj/jwlmanager/issues/1#issuecomment-1356595682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMRKE4CMZYEEDJKYDR5EPTWNZOT3ANCNFSM6AAAAAATCB7IZI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great. Here you will find three Windows exe files - two stand-alone packages, and one in a zipped folder with the required dependencies.
Please tell me which of these trigger your Windows Defender alert.
Thank you!

Defender detects all of them. I guess because the publisher is unknown.

That's a bit disappointing. Oh well, maybe down the road I'll try to find another method of packaging for Windows.
Thank you very much for testing!

Always welcome

Question, @lexiconwriter : so does Defender not allow you to continue at all? Meaning, you cannot open the app because it's flagged as malware and you can't get past it? Or is it just a warning that you can ignore?

Question, @lexiconwriter : so does Defender not allow you to continue at all? Meaning, you cannot open the app because it's flagged as malware and you can't get past it? Or is it just a warning that you can ignore?

Sorry for late reply: Defender gives a (martially) warning. But you will find a path to surround it in the dialog but not very explicit.

Well, I'm glad to know it's possible to get around it. It appears a lot of developers are unhappy with this behavior by Windows Defender, forcing them to pay big money for a signed certificate.

Yes, not funny. Maybe it is possible to drop a comment on this Windows behaviour in your Readme. Your decision.

Yes, thank you for the suggestion. I do mention it in a footnote.

I looked for the english defender text coming up when executing the installation file. Here it is under Windows 11:

In this case click on "More info". This will open this dialog:

For installation -> Run anyway

Do you have to take these steps every time you launch the app?

@erykjj No, it's only at installation time.

By "installation" are you referring to unzipping the folder? Because it's not an installer as such. It's either a stand-alone (complete) executable, or a zipped folder with the dependencies and executable.
I'd be happy if it only complained once (better still if it didn't complain at all :-))

I just used this linked exe: https://github.com/erykjj/jwlmanager/releases/download/v1.2.2/JWLManager_v1.2.2.exe It will be downloaded, the Windows downloader ask me to reject because is can be malicious, I tell "keep it" and then I doubleclick it in the download folder to start. Exe starts, and the defender comes up with the dialogs seen above.

Right. That's the stand-alone package. So, if you double-click it again from the downloads folder, it just starts up without any nagging?

second time without nagging - true.
That is all the tool "everything" is finding on my Windows system:

The prefetch files are simply cached shortcuts...
But it's a relief to know it only complains once and can be bypassed. That makes the app usable if you trust the source.
Thank you very much for all your testing and reports!!! It will help anyone who comes across this in the future.

Info added to SECURITY.md


Support onAction and isDisabled on item elements

Relates to #5058 #5459
This adds support for the onAction and isDisabled props directly on collection items in addition to the existing collection-level onAction and disabledKeys. In some cases it can be easier to set these props at an item level, for example with static collections (e.g. common in menus). If both are provided, an item is disabled if either the isDisabled prop is set or its key is in disabledKeys, and both onAction handlers are called.
Note that while this is implemented in the hooks, these props are only in the TS definitions for RAC and not RSP (or the older stately collections). Since RSP uses a single <Item> component across all components, we can't easily add props to some components but not others. In RAC we separated these out into separate components so this is possible.

yeah it shouldn't break because disabledKeys is still there, but just won't support the new feature until migrating.
what about isSelected?

what about isSelected?

did we want to support the same kind of thing? is that possible?

you mean isSelected on individual items? I think that's not possible because multiple items can be selected at once, so onSelectionChange really needs to be on the root level. Makes sense to keep selectedKeys at the same level.


compatible with enterprise github?

Hi,
Will this only work with https://github.com or could it be used with an on-premise github such as https://github.snakeoil.com ?
Thanks,
Ed

AFAIK it works with on-premise GitHub. If that's not the case, feel free to open issues.


chore: add Travis CI "build" badge

Just to display the master branch is in fact stable.

Coverage remained the same at 100.0% when pulling 0f695dc2e112d77bba2c53dd118acfd95ac0f422 on build-badge into b00e11eac614d8fb9427241fe6432f21a1d2930c on master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>6ad114d06ec9a4647c7cbf806f53a0 on build-badge into b00e11eac614d8fb9427241fe6432f21a1d2930c on master.


[FEATURE] Select PCI-E Passthrough during Rancher Cluster Configuration

Is your feature request related to a problem? Please describe.
When provisioning an cluster via Rancher into Harvester, the user is not able to select any PCI-E Passthrough options.
If desired, one must after deployment configure the VM within Harvester/Reboot.
Describe the solution you'd like

The PCI-E passthrough options are passedthrough(heh) to the Rancher cluster configuration when using Harvester as the cloud provider.
Describe alternatives you've considered

If there is yaml configuration which could be defined in the user/network data sections could be an alternative -- I have not pursued if this is possible.
Additional context

cc @ibrokethecloud @WuJun2016 @futuretea @guangbochen
Do we need to/have plan to extend the passthrough related features to upstream RKE2, Terraform ? thanks.


scrape mirrorlist for closest

I haven't tested this but it is a start.  For my Garuda PR I didn't have the option to do checksums but that's kinda redundant using zsync I think.
I think it's best to ensure the file you save has a constant name so zsync can update regardless of the incoming file. Hopefully this gets you close....

sorry for all the fat-finger behaviour.  Hopefully no more weird middle-button pastes while typing too quickly for a friday night ;-)

got there in the end :smile:

Thanks for your contribution! I decided to merge our work in one co-authored commit instead of merging your branch - looks cleaner imo.
This really helped me understand string manipulation and piping, thanks again :)

Glad it helped. Always happy to learn together.


Add login/signup page

Can u assign this issue to me under GSSOC'24 as a contributor I would lovw to work on this.
I will folow the guidelines and do this efficiently.
I have good knowledge on HTML,CSS and JavaScript and through my previous projects in this field I've gained a good command.

I want to work on this feature under GSSOC 24 please add a label to it and assign it to me.


Implement TreeSet Class - boolean containsValue(Object value)

Please assign it to me

@JyotiJauhari this is TreeMap issue


Create Netbeans.gitignore

Reasons for making this change:
Ignoring Netbeans IDE project folder and files
Links to documentation supporting these rule changes:
https://netbeans.org/
If this is a new template:

Link to application or project’s homepage: https://netbeans.org/

See Global/Netbeans.gitignore


Design update

The dynamic form must be easy to use and good looking.

Hi Blumster, good day to you, can you please let me know how to contact you to talk via voice  if possible ?


almin: Add Transaction id

Related #156
Currently, transaction has only name.
It will be duplicated name in an application.
We want to unique value between each transactions.

almin-logger need to use it.


Pythonic API

While the COM compatibility is a must-have, an alternate and more Pythonic API is also desirable.

@PMeira Great job on the package and having COM compatibility, it looks really great! I'm wondering if you have thoughts on making it compatible with OpenDSSDirect.py. And perhaps, if you are interested, we can even discuss merging these two efforts?

Hi @kdheepak

Great job on the package and having COM compatibility, it looks really great!

Thanks!

I'm wondering if you have thoughts on making it compatible with OpenDSSDirect.py.

They both share the same basic general idea, so it seems reasonable to reproduce the API from OpenDSSDirect.py. I also haven't started working on this issue (Pythonic API) so it'd be a good starting point). I'm working on other tasks right now, but give me a few weeks and I'll try to produce at least a proof-of-concept.

we can even discuss merging these two efforts?

Sure, that would be great!
Is it alright if I ping you back when I finish that proof-of-concept? When that is ready, we can better discuss merging and so on.
We also have a couple extra things that are not in the public repository yet. Maybe I'll be able to finish some of those by then.

Hey @kdheepak
Just a quick update. Since the PM version became the default on version 8, I'm waiting for a couple of official fixes so I can drop most of my custom patches to the Pascal code. From the official repository commits, it seems Davis is handling those recently.
As soon as the official code stabilizes, I'll work on this again, hopefully by next week. I'll send you an email then so we can better discuss merging.

Hey @PMeira I'm planning on working on some of these changes. Let me know if you want to discuss how we can work on this effort together.

@PMeira Thanks for tackling the OpenDSSDirect.py integration with dss_python. Are you thinking this issue is done or do you anticipate more work on this front?

@kdheepak I think it can be closed, I'll update the README.md to reflect this.
Right now I don't any thoughts for any other possible API. When I start exposing more in DSS C-API, since it's something new (doesn't need to mock the COM API, etc.), we could work on a better way to expose that in Python (might be simple enough to not warrant much discussion).


Integration Tests for New Roles created with limited Permissions

Signed-off-by: Aditya Jindal<EMAIL_ADDRESS>Issue #, if available: https://github.com/opensearch-project/alerting/issues/219
Description of changes:
CheckList:

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Tests Passing Locally: https://github.com/aditjind/alerting/pull/1

Codecov Report

Merging #302 (0a3f271) into main (ccbc3a1) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #302      +/-   ##
============================================
- Coverage     78.52%   78.40%   -0.12%
+ Complexity      218      217       -1
============================================
  Files           173      173
  Lines          6975     6975
  Branches        916      916
============================================
- Hits           5477     5469       -8
- Misses         1006     1014       +8
  Partials        492      492

Impacted Files
Coverage Δ

...h/alerting/model/destination/email/EmailAccount.kt
95.60% <100.00%> (ø)

...rch/alerting/model/action/ActionExecutionPolicy.kt
79.16% <0.00%> (-8.34%)
:arrow_down:

...arch/alerting/model/action/ActionExecutionScope.kt
61.90% <0.00%> (-7.94%)
:arrow_down:

...ing/destination/client/DestinationEmailClient.java
72.50% <0.00%> (-5.00%)
:arrow_down:

...ing/model/destination/DestinationContextFactory.kt
66.66% <0.00%> (ø)

.../kotlin/org/opensearch/alerting/util/IndexUtils.kt
74.41% <0.00%> (+2.32%)
:arrow_up:

Continue to review full report at Codecov.

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