Z5: Merge duplicates citekey selection and change issue

Hi,
I was testing for error #707 and there was no problem, however then I tried to merge the duplicates and observed the following:

sometimes Zotero asks you which version you want to use but sometimes it does not. When Zotero does not ask you, it selects one of the citekeys, not necessarily the first one. When Zotero asks you which dulicate to use as a master. In the right reference panel it does not show the citekeys, so you need to assume that the first duplicate in the list has the citation key without the a,b,c or other extension. It will be good in the right reference panel also to display the citation key for the reference.
if after I create the duplicates I change the citation key format and I perform merge the old citation keys (based on old format pattern) are deleted and the merged reference have a completely new citation key based on the new pattern.
Error ID: E8UYMBK2 (before merging)
Error ID: 2CU9LPI6 (after merging)

Wow, those are big points. This worked the same in Z4 BTW, so I'm marking it as an enhancement. On these points:
Merging references
Zotero wants to edit these fields back into the target item; but the citekeys are not part of any of the items being merged, they only live in the BBT database. Patching around all this behaviour is super tricky and fragile, and I'd preferably wait until Zotero finally gets a proper field for the citekey, as it will make this process a lot more manageable.
I do display the citekey in the reference details but that takes a fair amount of trickery and the citekey is non-editable because editing that field yielded some nasty race conditions.
Changes to dynamic keys after merge
That would be correct, as references with dynamic keys will get a new citekey whenever they change (and the merge is a change); the only exception to this is if the item being saved by Zotero happens to generate the same base key, but that base key is taken, and the postfix it had before is not in use by another reference. So if you have gop2014 and gop2014b (presumably because you also had gop2014a but deleted it), and you change gop2014b in such a way that it would get gop2014<postfix>, BBT will try to retain gop2014b for that reference. In all other cases, it gets a fresh key. This is the nature of dynamic keys; any reference you change in any way after you change the pattern will get a new key according to that pattern, which a proviso that the postfix (but only the postfix) may be retained. This behaviour is here to stay, in principle, but once the real citekey field in Zotero emerges and they support it in the merge field I may make an exception to keep the selected key.

My workaround for this issue to right-click on the item in the library and select "Duplicate" item, it will generate a new item with the correct citation key. I then drag any associated files from the old merged item and deleted the merged copy. Tedious, but better than manually addressing all duplicates in the library without the built in duplicate items functionality.

Wouldn't just refreshing the key be easier?

For clarity, in my case I merged two references in the library that had citation keys of Dellacorte2009 and Dellacorte2009a. I choose the reference with the citation key of Dellacoete2009a as the master that would remain and the resulting reference had the Dellacorte2009a citation key despite there being no Dellacorte2009 citation key in the library.
Refreshing the citation key did not resolve this. I had this happen in two separate instances as well.

Ah I see. Correct, duplicating the item will get you the desired effect, but in this case I'd personally opt to just pin the citekey to Dellacorte2009a. The BBT keymanager aims to be conservative about removing postfixes, so duplicating (which gets you the postfix) and then removing the original will work, but if you want to be sure, pin it.

In #1221 I now have a solution where the merged keys are retained, regardless of which way the merge goes.

(but it won't fully help for dynamic keys)

This is fixed in latest BBT -- citekeys (dynamic or not) will be added in the ids field for the merged items.


[2.x] Replace webpatser/laravel-uuid with ramsey/uuid

Resolves #134

I first wanted to use namespace (DNS)-based uuids, but they return the same result given the same domain, which might be problematic, so I just went with a fully random UUID type 4.


Device image link broken

The AVD device image link in the  env_setup/README.md is broken.

It works on my end - would you please try it again?
https://drive.google.com/drive/folders/1ZGKrWiSoGqg8_NoIGT7rWmiZ8CXToaBF?usp=sharing

Sorry, It seems like a problem of my office network, now it's fine, thank you~

Closed as the issue is solved.


brave: 0.25.2 -> 0.55.22

Motivation for this change
This switches to a chromium-based browser from a muon-based one.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

The /nix/store/*-brave/share/applications/brave-browser.desktop file has the executable path (Exec) set to /usr/bin/brave-browser.

@emmanuelrosa thanks for testing for the blind spot (there are probably other issues with the dpkg /opt paths though); I had just fixed it.

Yeah, that happens with packages from other distros. Here's a list of what may need patching:
share/gnome-control-center/default-apps/brave-browser.xml
opt/brave.com/brave/xdg-settings
opt/brave.com/brave/xdg-mime
share/menu/brave-browser.menu
opt/brave.com/brave/default-app-block

I have patched brave-browser.xml, brave-browser.menu, default-app-block, but xdg-settings and xdg-mime have lots of assumptions on the path of gnom, /usr/share, /usr/bin/file, ... that it is impractical to patch every single path by hand.

Those scripts are currently within xdg_utils so you can remove them from $out and add a symlink to them instead.

Ok, fixed the scripts.

@rht Great, testing this locally :+1:

@rht I've just checked and the attribute name is brave so can you reformat the commit msg to be?
brave: 0.25.2 -> 0.55.22

Also I went ahead and switched your path corrections to use substituteInPlace.
substituteAll is for something else see https://github.com/NixOS/nixpkgs/blob/4d1abc44199c8957105f538119c2d19d67aee26f/pkgs/stdenv/generic/setup.sh#L749
quoted everything, and made the derivation not explicitly declare phases as per #28910.
(probably add this to the commit msg)
diff --git a/pkgs/applications/networking/browsers/brave/default.nix b/pkgs/applications/networking/browsers/brave/default.nix
index 24f606e5c3a..6a9726b3ec3 100644
--- a/pkgs/applications/networking/browsers/brave/default.nix
+++ b/pkgs/applications/networking/browsers/brave/default.nix
@@ -79,43 +79,44 @@ in stdenv.mkDerivation rec {
         sha256 = "0111wbzyp2n8s0yskzq7xp7ipv5l6nyqzjczhh23j8yagm084kki";
     };

-    phases = [ "unpackPhase" "installPhase" ];
+    dontConfigure = true;
+    dontBuild = true;

     nativeBuildInputs = [ dpkg ];

     unpackPhase = "dpkg-deb --fsys-tarfile $src | tar -x --no-same-permissions --no-same-owner";

     installPhase = ''
-        mkdir -p $out
+        mkdir -p "$out"

-        cp -R usr/* $out
-        cp -R opt/ $out/opt
+        cp -R "usr/"* "$out"
+        cp -R "opt" "$out/opt"

-        export BINARYWRAPPER=$out/opt/brave.com/brave/brave-browser
+        export BINARYWRAPPER="$out/opt/brave.com/brave/brave-browser"

         # Fix path to bash in $BINARYWRAPPER
-        substituteInPlace $BINARYWRAPPER \
-            --replace /bin/bash ${stdenv.shell}
+        substituteInPlace "$BINARYWRAPPER" \
+            --replace "/bin/bash" "${stdenv.shell}"

-        ln -sf $BINARYWRAPPER $out/bin/brave-browser
+        ln -sf "$BINARYWRAPPER" "$out/bin/brave-browser"

         patchelf \
             --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" \
-            --set-rpath "${rpath}" $out/opt/brave.com/brave/brave
+            --set-rpath "${rpath}" "$out/opt/brave.com/brave/brave"

         # Fix paths
-        substituteAllInPlace $out/share/applications/brave-browser.desktop \
-            --replace /usr/bin/brave-browser $out/bin/brave-browser
-        substituteAllInPlace $out/share/gnome-control-center/default-apps/brave-browser.xml \
-            --replace /opt/brave.com $out/opt/brave.com
-        substituteAllInPlace $out/share/menu/brave-browser.menu \
-            --replace /opt/brave.com $out/opt/brave.com
-        substituteAllInPlace $out/opt/brave.com/brave/default-app-block \
-            --replace /opt/brave.com $out/opt/brave.com
+        substituteInPlace "$out/share/applications/brave-browser.desktop" \
+            --replace "/usr/bin/brave-browser" "$out/bin/brave-browser"
+        substituteInPlace "$out/share/gnome-control-center/default-apps/brave-browser.xml" \
+            --replace "/opt/brave.com" "$out/opt/brave.com"
+        substituteInPlace "$out/share/menu/brave-browser.menu" \
+            --replace "/opt/brave.com" "$out/opt/brave.com"
+        substituteInPlace "$out/opt/brave.com/brave/default-app-block" \
+            --replace "/opt/brave.com" "$out/opt/brave.com"

         # Replace xdg-settings and xdg-mime
-        ln -sf ${xdg_utils}/bin/xdg-settings $out/opt/brave.com/brave/xdg-settings
-        ln -sf ${xdg_utils}/bin/xdg-mime $out/opt/brave.com/brave/xdg-mime
+        ln -sf "${xdg_utils}/bin/xdg-settings" "$out/opt/brave.com/brave/xdg-settings"
+        ln -sf "${xdg_utils}/bin/xdg-mime" "$out/opt/brave.com/brave/xdg-mime"
     '';

     meta = with stdenv.lib; {

Also I went ahead and switched your path corrections to use substituteInPlace.

Ok I thought substituteInPlace does the replacement only once instead on all occurences. The naming substituteAll is misleading where it should have been substituteEnv.
I will incorporate your changes except for the quoting.
As with the quoting, I don't think they are necessary (see e.g. https://github.com/NixOS/nixpkgs/blob/cf7e7127c56ab04efaf1459d09810735bae52ccd/pkgs/applications/networking/browsers/tor-browser-bundle/default.nix).

Still think that quoting anything that had a variable in would be a good idea.

All the $out in https://nixos.org/nixpkgs/manual are unquoted btw. The corner case that might lead to quoting having to be enforced was raised in #26951, but there was no resolution to it.

Unfortunately the browser no longer executes:
[emmanuel@~/]$ brave-browser -v
Fontconfig warning: "/etc/fonts/fonts.conf", line 86: unknown element "blank"
Trace/breakpoint trap
[emmanuel@~/]$ [12596:12596:1111/110757.228434:ERROR:sandbox_linux.cc(379)] InitializeSandbox() called with multiple threads in process gpu-process.
[12596:12596:1111/110757.228829:ERROR:broker_posix.cc(43)] Invalid node channel message

At that point it exits. An older version of the package works:
[emmanuel@~/]$ brave-browser -v
Fontconfig warning: "/etc/fonts/fonts.conf", line 86: unknown element "blank"
[13117:13117:1111/111010.229082:ERROR:sandbox_linux.cc(379)] InitializeSandbox() called with multiple threads in process gpu-process.
[13080:13191:1111/111010.468441:ERROR:object_proxy.cc(615)] Failed to call method: org.freedesktop.Notifications.GetCapabilities: object_path= /org/freedesktop/Notifications: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.Notifications was not provided by any .service files
[13080:13101:1111/111011.972065:ERROR:rewards_service_impl.cc(133)] Failed to read file: /home/emmanuel/.config/BraveSoftware/Brave-Browser/Default/ledger_state

Here's a diff from the package that works to the current one:
33c33,34
<   zlib
---
>   zlib,
>   xdg_utils
67a69
>     xdg_utils
80c82,83
<     phases = [ "unpackPhase" "installPhase" ];
---
>     dontConfigure = true;
>     dontBuild = true;
90c93
<         cp -R opt/ $out
---
>         cp -R opt/ $out/opt
92,93c95,98
<         # Fix path to bash in $out/opt/brave.com/brave/brave-browser
<         substituteInPlace $out/opt/brave.com/brave/brave-browser \
---
>         export BINARYWRAPPER=$out/opt/brave.com/brave/brave-browser
>
>         # Fix path to bash in $BINARYWRAPPER
>         substituteInPlace $BINARYWRAPPER \
96c101
<         ln -sf $out/opt/brave.com/brave/brave-browser $out/bin/brave-browser
---
>         ln -sf $BINARYWRAPPER $out/bin/brave-browser
101a107,119
>         # Fix paths
>         substituteInPlace $out/share/applications/brave-browser.desktop \
>             --replace /usr/bin/brave-browser $out/bin/brave-browser
>         substituteInPlace $out/share/gnome-control-center/default-apps/brave-browser.xml \
>             --replace /opt/brave.com $out/opt/brave.com
>         substituteInPlace $out/share/menu/brave-browser.menu \
>             --replace /opt/brave.com $out/opt/brave.com
>         substituteInPlace $out/opt/brave.com/brave/default-app-block \
>             --replace /opt/brave.com $out/opt/brave.com
>
>         # Replace xdg-settings and xdg-mime
>         ln -sf ${xdg_utils}/bin/xdg-settings $out/opt/brave.com/brave/xdg-settings
>         ln -sf ${xdg_utils}/bin/xdg-mime $out/opt/brave.com/brave/xdg-mime

Yep, adding dontPatchELF = true; to the package fixed it. Thanks, @worldofpeace

Added dontPatchELF = true;.

Additionally, I renamed the binary from brave-browser to brave.

@GrahamcOfBorg build brave
Just to check one more time FTR, should be good otherwise.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: brave
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: brave
Partial log (click to expand)
installing
warning: working around a Linux kernel bug by creating a hole of 2007040 bytes in ‘/nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave/opt/brave.com/brave/brave’
post-installation fixup
gzipping man pages under /nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave/bin
patching script interpreter paths in /nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave
/nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave/opt/brave.com/brave/cron/brave-browser: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave...
/nix/store/cigarm8w6nmpkl5skjrjkjlsfmjqxy3a-brave

Additionally, I renamed the binary from brave-browser to brave.

Then you need to change
substituteInPlace $out/share/applications/brave-browser.desktop \
            --replace /usr/bin/brave-browser $out/bin/brave-browser

Then you need to change

Ah, right, fixed.

Speaking of which, 0.56.12 was just released 2 days ago. So I am updating the version further.

@GrahamcOfBorg build brave

@worldofpeace I added your latest patch, which is a lot more succinct than how it is being done in chromium/google-chrome,
    for icon_file in chrome/app/theme/chromium/product_logo_*[0-9].png; do
      num_and_suffix="''${icon_file##*logo_}"
      icon_size="''${num_and_suffix%.*}"
      expr "$icon_size" : "^[0-9][0-9]*$" || continue
      logo_output_prefix="$out/share/icons/hicolor"
      logo_output_path="$logo_output_prefix/''${icon_size}x''${icon_size}/apps"
      mkdir -vp "$logo_output_path"
      cp -v "$icon_file" "$logo_output_path/$packageName.png"
    done

@rht Yeah that's a little complex. I'm assuming here that icon sizes will remain fixed though.

@GrahamcOfBorg build brave

Success on x86_64-linux (full log)
Attempted: brave
Partial log (click to expand)
installing
warning: working around a Linux kernel bug by creating a hole of 2007040 bytes in ‘/nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave/opt/brave.com/brave/brave’
post-installation fixup
gzipping man pages under /nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave/bin
patching script interpreter paths in /nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave
/nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave/opt/brave.com/brave/cron/brave-browser: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave...
/nix/store/mhw7141k0q659qx1a5f79fgk292w9gjn-brave

I've tested and:

the binary executes correctly
desktop file appears and has correct icons
clicking the desktop item launches brave

Everything works fine within the browser.
LGTM :+1:
cc @Mic92 @Infinisil (mergers)

@rht Thank you for your cooperation and contribution :sparkle:


Consider removing id, inserted_at, and updated_at from default form

By default, I would not expect id, inserted_at, and updated_at to be included in the form for creating a new record or updating an existing record.  I would consider removing these fields from the default form.

I think this is a sound suggestion. Users can always include them when they need to through form_fields/1.
This change is coming in the next version.

This is implemented in master. By default, the index and form pages will not show autogenerated fields. Those fields are by default id, inserted_at, and updated_at.


close tab 'x' button missing after upgrade to 0.175.0

After upgrading to atom 0.175.0 this morning, the x button is no longer present in the tab header (on hover or otherwise).
Looking in the inspector, the close-icon div is present, but nothing is inside it, nor is any CSS rule causing an 'x' to be rendered.

app-main.js

I am running Ubuntu 14.04.

I'm also noticing that file names that exceed the max tab width are incorrectly overflowing the tab container.  I'm not sure what caused this regression, but adding a CSS rule .tab-bar .tab .title { overflow: hidden; } seems to fix it.

This is a duplicate of #5198.

@sirianni what theme do you have? Does it happen in safe mode? Please quit atom completely and run atom --safe. Is the issue still there?

Thanks.  I was also using the webupd8 PPA.  Strangely though, I was seeing the issues with 0.175.0.  I removed 0.175.0 and installed the atom-amd64.deb manually.  This fixed it.


Load unconfigured schema meta data concurrently

Use multiple thread to load unconfigured schema meta data for different data sources to improve performance of loading.
The code is in RuleSchemaMetaDataLoader line 80, please search // TODO use multiple threads for different data sources

Hi, i want to do it

I want to solve this issue too!

@zjcnb @strongduanmu  you can discuss about it here.

@zjcnb @strongduanmu you can discuss about it here.

@kimmking @zjcnb First come first served, next time I will claim issue earlier. 😂 I'm so sorry for everyone!

@zjcnb @strongduanmu It seems you have good negotiation.
I just assign this issue to you, thank you. @zjcnb

@zjcnb @strongduanmu It seems you have good negotiation.
I just assign this issue to you, thank you. @zjcnb

thx

@zjcnb @strongduanmu you can discuss about it here.

i got it😁


new control to switch Alternate screen

new control to switch to the secondary buffer and perform an action and return to the primary buffer. options to configure bg and fg color and pads

Pads will not be implemented, because it is not possible to control the cursor position after changing


when issued by DAO, the issuer should be the original proposer

When an offset is issued by the DAO after a successful vote, the issuer should still be the original proposer.
We can store the original proposer as a field on the propsal or the proposed offset, and then set it when the contract executes.
The net result is that the token should show up on the original proposer's dashboard (ie Offsets Dealer's dashboard.)

Added in this commit: https://github.com/opentaps/blockchain-carbon-accounting/commit/2dd4280530fdd93b256183db92e0a690a6904513

NetEmissionsTokenNetwork.sol now has a function setTimelock() that is used to store the address of Timelock.sol
In NetEmissionsTokenNetwork.sol issue() has been split into three functions:

_issue(), which accepts an issuer and issuee parameter but can only be called internally
issueFromDAO(), which only Timelock can call and passes the issuer as the proposer
issue(), which calls the lower-level _issue() with msg.sender as the issuer and behaves as before (only registered dealers can call, etc.)

The deploy-all.js script automatically calls setTimelock() for testing on a local Hardhat Network, but after deploying to a testnet/main with deploy-net-emissions-token-network.js, it must be set manually by the admin/deployer.
Updated the signatures/calldata in the issue form on the React application to use issueFromDAO() instead of issue() as well as display them correctly in the governance dashboard


npm install fails because of missing dist files

Description
npm install fails because of missing dist files
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slack-events version: v1.0.0-beta.1
node version: v7
OS version(s): macOS v10.12.2
Steps to reproduce:

npm install --save @slack/events-api@beta

Expected result:
successful installation
Actual result:
ankur:test-node-events-api/ $ npm install --save @slack/events-api@beta
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm ERR! Darwin 16.3.0
npm ERR! argv "/Users/ankur/.nvm/versions/node/v7.2.0/bin/node" "/Users/ankur/.nvm/versions/node/v7.2.0/bin/npm" "install" "--save" "@slack/events-api@beta"
npm ERR! node v7.2.0
npm ERR! npm  v4.0.2
npm ERR! path<EMAIL_ADDRESS>npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod

npm ERR! enoent ENOENT: no such file or directory, chmod<EMAIL_ADDRESS>npm ERR! enoent ENOENT: no such file or directory, chmod<EMAIL_ADDRESS>npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/ankur/Developer/play/test-node-events-api/npm-debug.log

the issue is that the dist/ directory isn't in the distribution tarball, since its listed in .gitignore. the solution is to commit an empty .npmignore file to override the inheritence of .gitignore.


Style spinner

Is there anyway to make the spinner larger?

:+1:

Hello, I guess you can try
<GiftedSpinner
size={'large'} // ios size
style={{
  height: 20, // android size
}} />

@adrienbrault has that worked for you? I tried on ios and the spinner is the same size


Push BlueMapWeb

Just updates the submodule commit reference. See main repo for a description.

Closing this since it is now possible to add this using custom styles and css-classes for POI-markers :)


Error while installing overlay

Plugin's version
1.1.2
Playnite's version
10.20
Bug description
The error happened when using pip-installed Legendary. Enabling and configuring a custom launcher path fixed the issue when attempting to install again
Relevant log
[DLManager] INFO:  - Cache usage: 0.00 MiB, active tasks: 0
[DLManager] INFO:  + Download	- 0.00 MiB/s (raw) / 0.00 MiB/s (decompressed)
[DLManager] INFO:  + Disk	- 3.26 MiB/s (write) / 0.00 MiB/s (read)
[DLManager] INFO: Waiting for installation to finish...
[DLManager] INFO: All done! Download manager quitting...
[cli] INFO: Finished downloading, setting up overlay...
[cli] INFO: Updating overlay registry entries from "C:\Program Files (x86)\Epic Games\Epic Online Services\managedArtifacts\98bc04bc842e4906993fd6d6644ffb8d" to "G:\Games\PC\Legendary\.overlay"
Traceback (most recent call last):
  File "c:\python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Python39\Scripts\legendary.exe\__main__.py", line 7, in <module>
  File "c:\python39\lib\site-packages\legendary\cli.py", line 3086, in main
    cli.manage_eos_overlay(args)
  File "c:\python39\lib\site-packages\legendary\cli.py", line 2348, in manage_eos_overlay
    remove_registry_entries(prefix)
  File "c:\python39\lib\site-packages\legendary\lfs\eos.py", line 134, in remove_registry_entries
    remove_registry_value(HKEY_LOCAL_MACHINE, VULKAN_OVERLAY_KEY, value)
  File "c:\python39\lib\site-packages\legendary\lfs\windows_helpers.py", line 58, in remove_registry_value
    k = winreg.OpenKey(hive, key, reserved=0, access=access)
PermissionError: [WinError 5] Acces Denied

19-10 12:20:19.578|ERROR|LegendaryLibrary#LegendaryDownloadManager:[Legendary] exit code: 1

Screenshots
No response

I can't reproduce that. If Legendary really needs more permissions, then I'm afraid that I can't do much about it (https://github.com/Tyrrrz/CliWrap/issues/174#issuecomment-1311061879).

Ok, looks like I found a way to reproduce that. However as I said, I can't do nothing about that. If it EOS was previously saved in some path which only admin can modify or you want to download it to that path, then you need to relaunch Playnite as admin.


[BUG] Characters requiring AltGr / Right Alt cannot be typed on Windows with Crossterm backend

Describe the bug
On many keyboard layouts, some special characters need the AltGr / right Alt modifier.  For example, on the nordic keyboard layouts, the @ character is typed with AltGr + 2.
When running Cursive applications with the crossterm backend on Windows, these characters do not work at all. Crossterm itself has supported these keyboard events for a while now (https://github.com/crossterm-rs/crossterm/pull/488). The events are reported with the special character and Ctrl and Alt modifiers. (Crossterm's event_poll_read example shows the event as Event::Key(KeyEvent { code: Char('@'), modifiers: CONTROL | ALT }).)
Cursive's crossterm backend seems to convert any characters to Tab key events if both Ctrl and Alt are set (https://github.com/gyscos/cursive/blob/562f863852566b7bef185c9ab3fc8a73df33953d/cursive/src/backends/crossterm.rs#L114). Therefore these special characters are ignored in EditViews.
To Reproduce
On Windows, run:
cargo run --example key_codes --features crossterm-backend --no-default-features

With a nordic or any other keyboard layout with characters requiring the AltGr key, try to type these characters. The key_codes application will print CtrlAlt(Tab) instead of character events.
Expected behavior
Cursive should produce character events with the special characters so that they can be typed in EditViews.
Environment

Operating system used: Windows 10 20H2
Backend used: crossterm
Current locale (run locale in a terminal):

# In Git Bash
$ locale
LANG=en_GB.UTF-8
LC_CTYPE="en_GB.UTF-8"
LC_NUMERIC="en_GB.UTF-8"
LC_TIME="en_GB.UTF-8"
LC_COLLATE="en_GB.UTF-8"
LC_MONETARY="en_GB.UTF-8"
LC_MESSAGES="en_GB.UTF-8"

Cursive version (from crates.io, from git, ...): 0.16.3

Additional context
I temporarily fixed this locally by adding the following match arm before the existing CtrlAlt+key handler in crossterm.rs:
        CKeyEvent {
            modifiers: CTRL_ALT,
            code: KeyCode::Char(c),
        } => Event::Char(c),

It works perfectly now. Thanks for the quick fix!
When do you plan to release v0.17?

This works now with Cursive 0.17.


Need certification help (wont give certification bug)

its saying that i have not done the roman numeral converter but i have, and not only that it shows it as completed in my free code camp logs… i was wondering how to get this fixed so i could have that beautiful beautiful certification i have been working at. also let me just say… this is hands down the most amazing site ive ever used. Im actually mad there are not more places like this around (not even the free part, its the amount you guys have and the way you do the lessons, way better than learn by vid).
Profile Name: angelicarus

Browser Name: chrome
Operating System:  mac
(any info I'm missing for help?)

@syne612 If you go to the Settings page, scroll down to the JavaScript Algorithms and Data Structures Projects section, and click Claim Certification, is there any error message pops up on the screen? Please also check the browser console for error messages, if any.

its saying that i have not done the roman numeral converter but i have
I had the same issue with the Palindrome Checker. I assume it's a bug from when they made the beta live. I solved it by redoing the project. You'll notice that FCC saves your code from your projects. If it doesn't have your code saved then it doesn't know you completed the project.

Yea but id rather not redo that thing it was a pain.
Sent from my iPhone

On Jun 23, 2018, at 3:17 PM, Matthew Szczublewski<EMAIL_ADDRESS>wrote:
its saying that i have not done the roman numeral converter but i have
I had the same issue with the Palindrome Checker. I assume it's a bug from when they made the beta live. I solved it by redoing the project. You'll notice that FCC saves your code from your projects. If it doesn't have your code saved then it doesn't know you completed the project.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Did you get the most Recent email I sent you with the pictures?

On Jun 22, 2018, at 9:36 PM, Huyen Nguyen<EMAIL_ADDRESS>wrote:
@syne612 If you go to the Settings page, scroll down to the JavaScript Algorithms and Data Structures Projects section, and click Claim Certification, is there any error message pops up on the screen? Please also check the browser console for error messages, if any.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Jk web surfed and grabbed online answer nbd
Sent from my iPhone

On Jun 23, 2018, at 3:17 PM, Matthew Szczublewski<EMAIL_ADDRESS>wrote:
its saying that i have not done the roman numeral converter but i have
I had the same issue with the Palindrome Checker. I assume it's a bug from when they made the beta live. I solved it by redoing the project. You'll notice that FCC saves your code from your projects. If it doesn't have your code saved then it doesn't know you completed the project.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@syne612 / @Asjas please close this issue, seems fixed


Improve intersection type support

Purpose
Allow closed records to be intersected when possible. And improve the error message for intersection between types that we don't handle at the moment.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/30118
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/30403
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/30401
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

The build seems to have failed.


Trying to use DashBoard Feature

Hi!
i am tryiung to use the dash board feature but it gets in a infinity Loading screen so i thought that maybe it is because you are doing a request to a server to get de stats  and this dont responde back,

Hi @sbenitez10, the OSS version of the app returns data from network calls defined in MockApiClient.


Add Steps List to Workflow GraphQL Resource

In frontend V2, we display the process detail page when creating a workflow.
This page includes a progress bar with individual steps which atm is filled statically.
We need to enhance the Workflow GraphQL resource to include a step list (steps for consistency with Process) that provides information about the steps within a workflow. This enhancement will enable us to populate the progress bar with the relevant steps.

duplicate of https://github.com/workfloworchestrator/orchestrator-core/issues/380


Resource and asset merger: Can't determine type for tag ?attr/shapeAppearanceCornerSmall</macro

I searched the answer on Stack overflow but everyone says downgrade the version of gradle and downgrade is not a solution so I'm posting this issue on github
Hello, I'm using the latest version of Flutter, Here is the doctor summery

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.10, on Linux Mint 20 5.4.0-56-generic, locale
en_GB.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Chrome - develop for the web
[✗] Linux toolchain - develop for Linux desktop
✗ clang++ is required for Linux development.
It is likely available from your distribution (e.g.: apt install clang), or can be
downloaded from https://releases.llvm.org/
✗ CMake is required for Linux development.
It is likely available from your distribution (e.g.: apt install cmake), or can be
downloaded from https://cmake.org/download/
✗ ninja is required for Linux development.
It is likely available from your distribution (e.g.: apt install ninja-build), or
can be downloaded from https://github.com/ninja-build/ninja/releases
✗ GTK 3.0 development libraries are required for Linux development.
They are likely available from your distribution (e.g.: apt install libgtk-3-dev)
[✓] Android Studio (version 4.2)
[✓] VS Code (version 1.51.1)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
! Doctor found issues in 1 category.

My project is run well on the previous version. But after I update the Flutter SDK I got some errors Not sure why this error getting.  I'm using 7.4 gradle version URL.
distributionUrl=https://services.gradle.org/distributions/gradle-7.4-bin.zip

ERROR:/home/vltp001/.gradle/caches/transforms-3/2e96bf4ef56070ec71603a6b40541457/transformed/material-1.8.0/res/values/values.xml: Resource and asset merger: Can't determine type for tag '?attr/shapeAppearanceCornerSmall'
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDebugResources'.

/home/vltp001/.gradle/caches/transforms-3/2e96bf4ef56070ec71603a6b40541457/transformed/material-1.8.0/res/values/values.xml: Error: Can't determine type for tag '?attr/shapeAppearanceCornerSmall'

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 2m 37s
Exception: Gradle task assembleDebug failed with exit code 1

Hi @Shubham-Narkhede
Have you tried the solutions provided in this SO question? If they are not helpful, please share the content of /android/build.gradle.
Also, does this issue happen on the newly created Flutter project?

@huycozy this is solved now, Actually i have not updated the gradle version and kotline version in the app that's why the issue is getting


Merges develop branches and adds fixes from release/v2

Coverage increased (+0.03%) to 35.631% when pulling 7d98e73f1070bb28809f8b40cd3d61d82c9612ce on maxhq:develop-next into 758c90ec9ba6e2aed8c835788f5a1a18b1264f05 on openxpki:develop.


Add new Code actions

Purpose
Fixes #33563
Fixes #33565
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Changes look good. However the text edit is erroneous in the following case.
function func1() returns int? {}

And I have put some minor comments as well. Please address them if possible.

Fixed


[BUG] Cannot iterate over nested list of objects to apply text embeddings processor

What is the bug?
When a nested field contains a list of objects, and a foreach processor is used to generate the text embeddings for a text in each nested object, it does not work.
How can one reproduce the bug?

Create a pipeline with foreach processor that iterates over a nested list of object & applies text embedding processor to each nested object.  NOTE: The text embedding model also has to be present & loaded

This is way of using the foreach processor (to access internal nested objects with _ingest._value). Since text embedding processor does not current support . notation for fields, I had to write in the below way.
{
    "processors": [
        {
            "foreach": {
                "field": "nested-field",
                "processor": {
                    "text_embedding": {
                        "model_id": "7l8rQ4sB2f-9nv8R9Veh",
                        "field_map": {
                            "_ingest": {
                                "_value" :{
                                    "text-field": "vector-field"
                                }
                            }
                        }
                    }
                }
            }
        }
    ]
}

Test the pipeline with the following nested document.

{
    "docs": [
        {
            "_index": "test-index",
            "_id": "1",
            "_source": {
                "nested-field": [
                    {
                        "text-field": "This is a test"
                    },
                    {
                        "text-field": "This is another test"
                    }
                ]
            }
        }
    ]
}

The above does not work. But the thing is, neural search supports query nested fields (as nested query) and there is the possibility to only get the actual matches using inner_hits.
What is the expected behavior?
The expected behavior is that for the above example, text embedding should be generated (vector-field) for both the nested objects in the above example.
I understand that multi-value fields aren't currently supported for generating text embeddings, but since we are dealing here with nested objects & they are independant objects in lucene, this should work.
What is your host/environment?
I tested with the official docker image of Opensearch 2.9 running on Windows.

@zane-neo can you take a look into this.

So, on looking at the neural-search plugin code, this case (list of maps inside a nested map) wasn't supported. I could add this case & now the text embeddings processor works with a a nested field which has a list of objects.
curl --location 'http://localhost:9200/_ingest/pipeline/embeddings-nested/_simulate?verbose=true' \
--header 'Content-Type: application/json' \
--data '{
    "docs": [
        {
            "_id": "1",
            "_source": {
                "nested-field": [
                    {
                        "text-field": "This is another test"
                    },
                    {
                        "text-field": "This is a third field"
                    },
                    {
                        "text-field": "This is a fourth field"
                    }
                ]
            }
        }
    ]
}'

The corresponding output after my fix:
NOTE: I have removed the vectors to make the output shorter to post here
{
    "docs": [
        {
            "processor_results": [
                {
                    "processor_type": "text_embedding",
                    "status": "success",
                    "doc": {
                        "_index": "_index",
                        "_id": "1",
                        "_source": {
                            "nested-field": [
                                {
                                    "text-field": "This is another test",
                                    "vector-field": [
                                        0.029963804,
                                        ....
                                    ]
                                },
                                {
                                    "text-field": "This is a third field",
                                    "vector-field": [
                                        ...
                                    ]
                                },
                                {
                                    "text-field": "This is a fourth field",
                                    "vector-field": [
                                        ...
                                    ]
                                }
                            ]
                        },
                        "_ingest": {
                            "pipeline": "embeddings-nested",
                            "timestamp": "2023-10-26T12:24:00.218116911Z"
                        }
                    }
                }
            ]
        }
    ]
}

I would like to contribute this "fix" (or feature depending on how you would like to classify it). I can fork the project & make the commit there.
Can somehow look at my changes?
(I have made the changes & if its okay to commit, I will add tests etc.)

@krishy91 have you checked this issue?

@krishy91 Ok, do you have a PR regarding the fix? I haven't got time to reproduce this issue but would like to check your PR first.

@zane-neo I have a rough sketch of the solution which works. I have created a draft pull request here: https://github.com/opensearch-project/neural-search/pull/477
If you guys feel the solution is reasonable (I think so as it is adding just another case which needs to be handled), I will go ahead & fix the validateEmbeddingFieldsValue method (which checks if supplied input is valid) & add some tests for it. (before the pull request can be deemed fit for merging)

@krishy91 When we implemented this feature, we considered mostly the split documents case(which is a list of string), so not supporting the list of object case. But I do think your case is valid, I have no concern to provide this capability to user. cc: @navneet1v @model-collapse.

@zane-neo @navneet1v  @model-collapse PR is open and already in use. We would love to see it merged and available in 2.12.
WDYT?
https://github.com/opensearch-project/neural-search/pull/477

@krishy91 Recently did a test on the nested map type configuration, found some issues:
pipeline configuration:
{
  "description": "An example neural search pipeline",
  "processors": [
    {
      "text_embedding": {
        "model_id": "qhO5xY4BYwgbtrHt7KDf",
        "field_map": {
          "category": {
            "name": {
              "en": "category_name_vector"
            }
          }
        }
      }
    }
  ]
}

And simulate the pipeline processor:
{
    "docs": [
        {
            "_index": "neural-search-index-v2",
            "_id": "1",
            "_source": {
                "category": [
                    {
                        "name": {
                            "en": "this is a name"
                        }
                    },
                    {
                        "name": {
                            "en": "hello world"
                        }
                    }
                ]
            }
        }
    ]
}

Result:
{
    "docs": [
        {
            "doc": {
                "_index": "neural-search-index-v2",
                "_id": "1",
                "_source": {
                    "category": [
                        {
                            "name": [
                                -0.10758455,
                                0.07971476,
                                -0.04948872,
                                ...
                            ]
                        },
                        {
                            "name": [
                                -0.034477253,
                                0.031023245,
                                0.006734962,
                                ...
                            ]
                        }
                    ]
                },
                "_ingest": {
                    "timestamp": "2024-04-10T03:51:53.496385Z"
                }
            }
        }
    ]
}

The embedding results override the original values in nested maps which seems a bug needs to be fixed.

Created this issue to track: https://github.com/opensearch-project/neural-search/issues/686

@zane-neo can this be closed?

Closing this as it's been fixed.


Watchtower cannot create container on more than 1 network: Container cannot be connected to network endpoints

Description
I'm trying to patch watchtower to handle multiple networks on container update ( My attempt ).
Once I had a "working" environment, the watchtower initiated the update of the image and then tried to launch a new container.
Steps to reproduce the issue:

Create a container  connected to more than 1 network based on an image from the hub you have write access to.
Update the image on the hub
Launch tuxago/watchtower:net-bug container and check the logs

Describe the results you received:
watchtower_1  | time="2016-12-09T00:09:42Z" level=info msg="Stopping /projectweb_newimg_1 (e5821371721d080695b31c74694db15c2a200e1f0c35d92c6a6af7ebe052217f) with SIGTERM"
watchtower_1  | time="2016-12-09T00:09:43Z" level=debug msg="Removing container e5821371721d080695b31c74694db15c2a200e1f0c35d92c6a6af7ebe052217f"
watchtower_1  | time="2016-12-09T00:09:43Z" level=info msg="Starting /projectweb_newimg_1"
watchtower_1  | time="2016-12-09T00:09:43Z" level=error msg="Error response from daemon: Container cannot be connected to network endpoints: projectweb_back, gate_front-tier"
watchtower_1  | time="2016-12-09T00:14:43Z" level=info msg="Checking containers for updated images"

Describe the results you expected: I shouldn't have any errors, and the new version of the container should be connected to both networks
Additional information you deem important (e.g. issue happens only occasionally):
It seems the message comes from https://github.com/docker/docker/blob/master/daemon/create.go#L293 , but before that, the code seems to indicate that it doesn't handle multiple network connections at all.
But it puzzles me, because launching a docker-compose with containers on multiple networks works. But maybe it doesn't use the same code path.
 Subsequent networks can be attached after that (docker network attach, or the corresponding API call https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/#/connect-a-container-to-a-network)
If you check the daemon logs (in debug mode), you'll see that that is what docker-compose does as well.
 Will save me some time of figuring out stuff.
In the log, if I understand well, it connects to the last network by default (network2), then it disconnects from it, and then connect, to both networks.
Have I understood it well?
Can I ask why is that? Wouldn't it be easier if all of that logic could be handled by the daemon itself?
It seems it would be an intuitive way to do it. Because, I just got very lucky you gave me that detailed answer of the current logic, which doesn't look intuitive to me, and I think it could be unintuitive to other people as well.


Dead Project?

Thank you for this fantastic gem! I really appreciate your work.
There are no commits and absolutely no activity in the last 6 months. Is this project still alive?

Not dead, no. I work on other projects as well and open source projects, especially small ones can go for a while without many changes.
I am planning to rewrite the code to use ES2015, modular/plugin design, with a rollup build. Stay tuned.

Glad to hear that!

Massive rewrite complete. From scratch. We even have a new name, Europa. Check it out: https://github.com/NotNinja/europa

This is great news. I will give it a try in the future. Thanks for your work!


Fine tuning KnowBert for downstream tasks

Hi, I am trying to run the below code for fine-tuning KnowBert for Tcred dataset:
allennlp train --file-friendly-logging --include-package kb.include_all
training_config/downstream/tacred.jsonnet -s OUTPUT_DIRECTORY
I met several errors, I was able to resolve some of them. But not sure how to deal with the below error:
Traceback (most recent call last):
File "/home/xu_lu/.local/bin/allennlp", line 10, in
sys.exit(run())
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/run.py", line 18, in run
main(prog="allennlp")
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/commands/init.py", line 102, in main
args.func(args)
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/commands/train.py", line 116, in train_model_from_args
args.cache_prefix)
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/commands/train.py", line 160, in train_model_from_file
cache_directory, cache_prefix)
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/commands/train.py", line 226, in train_model
validation_iterator=pieces.validation_iterator)
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/training/trainer.py", line 725, in from_params
params.assert_empty(cls.name)
File "/home/xu_lu/.local/lib/python3.7/site-packages/allennlp/common/params.py", line 405, in assert_empty
raise ConfigurationError("Extra parameters passed to {}: {}".format(class_name, self.params))
allennlp.common.checks.ConfigurationError: "Extra parameters passed to Trainer: {'gradient_accumulation_batch_size': 32}"
2020-08-20 15:59:20,126 - INFO - allennlp.models.archival - removing temporary unarchived model dir at /tmp/tmprz5qhl43
My current environment has passed the tests and I am able to run the example to embed the sentences.

I think this might be the installed version of allennlp - can you check that this branch is installed? git+git://github.com/matt-peters/allennlp.git@2d7ba1cb108428aaffe2dce875648253b44cb5ba
Try
pip uninstall allennlp
pip install git+git://github.com/matt-peters/allennlp.git@2d7ba1cb108428aaffe2dce875648253b44cb5ba

I am also facing a similar issue, on trying the above-mentioned solution, I got,
Could not find a tag or branch '2d7ba1cb108428aaffe2dce875648253b44cb5ba', assuming commit.


add abs support to Dec

Really easy good first issue. Just need to call math.absInt on the underlying number and then wire up the zig builtin to get called.

This is a great first issue!


Support for nested tableColumns

I wanted to test Nested TableColumns when reading http://docs.oracle.com/javafx/2.0/ui_controls/table-view.htm#CJABHBEH. But this did not work. Therefore I created this small patch.

Thanks for the contribution!


Tracking fsspec related pending changes

Fsspec Compatibility

[x] Make RepoFileSystem fsspec-compatible. (@efiop)
[ ] Make LocalFileSystem fsspec-compatible. (@skshetry) (lower priority)
[x] Make GitFileSystem fsspec-compatible (@efiop)
[x] Get rid of FileSystem. (@skshetry)
[x] Get rid of CallbackMixin since callbacks are now generally supported. (@skshetry)
[x] Get rid of NoDirectoriesMixin, merge with HTTPFileSystem. (@skshetry)
[ ] use to_json/from_json instead of config (@skshetry)
[ ] use configs instead of creating filesystems right away (@skshetry)
[ ] Plugins (lower priority)
[ ] Reorganize/get-rid of fs.utils/dvc.utils.fs/System (low priority)

Post-fsspec changes

[ ] Handle OSErrors instead of DvcExceptions.  (low priority)
[ ] Rethink Cloud fixtures, so that they can work with any fsspec-compatible filesystems by default.
Extract Cloud.get_url() and Cloud.config out of it.  (low priority)

@skshetry @efiop Are you still tracking/updating this issue?

@skshetry @efiop Are you still tracking/updating this issue?

Yes but we have stopped working on this temporarily. No major changes are planned yet, small changes might happen outside of the points listed above.


Refactor AKSCluster.ps1 to support cilium network policy

This pull request includes a modification to the hasNetworkPoliciesEnabled method in the AKSCluster class in the AKSCluster.ps1 file. The change expands the network policy check to include "cilium" in addition to "azure" and "calico". This means that the method will now return true if the networkPolicy of the ClusterObject's networkProfile is either "azure", "calico", or "cilium".

Closes #6


docs: fix a typo

Description:

Testing done:

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?

Is this a breaking change?

Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e232369) 47.58% compared to head (2ae2113) 47.58%.
Report is 3 commits behind head on release/v1.2.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           release/v1.2     #170   +/-   ##
=============================================
  Coverage         47.58%   47.58%
=============================================
  Files               196      196
  Lines             14884    14884
=============================================
  Hits               7083     7083
  Misses             7757     7757
  Partials             44       44

Flag
Coverage Δ

unittests
47.58% <ø> (ø)

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

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


MATLAB codes are added and java code fixed

MATLAB codes are added
Issues at P002BasicNeuronLayer.java and P001BasicNeuron3Inputs.java have fixed

new file MATLAB Dot Product added


翻譯 RFC 2342 const control flow

Fix #1

有語意不順或是覺得怪怪的地方都歡迎留言討論！ 🙇

Rendered

Thanks for the review from both of you 🙇 I just addressed all the comments; if it looks fine, I will squash the commits!

I added the rendered link in PR description 🙇♂

hmm, I'm not so used to no CI tests 😂 I dunno what we should test though...
Anyway, I will merge directly this time and maybe we can discuss further if anyone has any ideas.


Add a starter project to TechTeam25 as a starter Next Project to get Devs started

@techteam25  could you create another blank repository and I'll throw up a small example in the near future?


Migrate DockerHub account for publishing and distributing container images

We started seeing docker login errors since our move to this github organization.

It turned out that the issue was not about summerwind's docker hub account being inaccessible, but rather we use the github account name as the dockerhub account name to login
https://github.com/actions-runner-controller/actions-runner-controller/blob/588872a31649b3021d93d031e99e722674d6399b/.github/workflows/build-and-release-runners.yml#L61
Probably changing this back to the previous value by any mean would fix inability to login anyway.


Issue: Version 0.4.1 creates Type Exception on getOrElse

I noticed when using 0.4.1 in contrast to 0.4.0 a type exception is thrown when using getOrElse on Option.
type '() => String' is not a subtype of type '() => Never' of 'orElse'

The code responsible for the error is:
model.activeProfile
           .map((final Profile profile) => profile.isActive)
           .getOrElse(() => false)

Is this a known issue? Downgrading to Version 0.4.0 from 0.4.1 works.

@SandroMaglione Can you elaborate what the issue exactly was? I tried to understand the PR but it just seems to change dependency versions.

@ju851zel you can read details in #92
The issue was with introducing a const factory constructor for None. When calling map, it caused the type to be wrong at runtime.

well, it's not fixed. i got
type '() => TaskEither<L, R>' is not a subtype of type '() => TaskEither<L, Never>' of 'orElse'<…>


Implement Currency API into currency to currency Feature

https://exchangeratesapi.io/ which works with ==> https://apilayer.com/


Get rid of P/Invoke SizeOf.

PR Details
Get rid of P/Invoke SizeOf.
Description
Eliminates all references to Stride.Core.Interop.SizeOf<T>().
Related Issue
#1461
Motivation and Context
See issue.
Types of changes

[x] ~Docs change~ / refactoring / ~dependency upgrade~
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

[x] Removed int Stride.Core.Utilities.SizeOf<T>(T[]) where T : struct; do the multiplication manually
[x] Removed int Stride.Core.Utilities.UnsafeSizeOf<T>(); duplicate of SizeOf<T>()

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

All tests that pass in master on my computer pass in this branch too. TestCopyPasteEntityAsChild didn't pass running all tests from this branch while it passed in master, but passed when stepping through.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Oh yes, I must've overlooked that Utilities is in fact public.

Maybe it can be left as Deprecated/Obsolete on Utilities class until 4.2 so we can release it in 4.1?

I left out internal static int UnsafeSizeOf<T>() because it is internal.


Playlist Not Showing

I have imported your project in my my project, also changed my api key...
Even playlist is being fetched but after fetching playlist name, nothing happens.
I am not getting the list of videos in particular playlist.
Kindly Help

uhmm,  I would love to help but you haven't provided enough information to go on.  What have you done to try to debug the issue?


Issues after updating libraries

Hi Matthias,
tried to update my esp32 today , simply to change one line of text on the OLED display, but now it appears that the code will not compile at all. I have installed all the latest for weathersensorreciever and weathersensor, but once i click upload the ide hangs at this point:

I have tried selectin both TTGO LoRA32 v2 and TTGO LoRa32 V2 (1.61) (the correct one) in the board revision, both with the same result!
I did managed some sucess earlier, but the screen wasn't working, nor was the board sending to TTN. this screenshot shows the serial output from such an attempt:

Any Advice would be greatly appreciated

Hi,
I'm pretty sure that the offending line with global_tbuf is part of your customization. Could you please upload it to your github? I'll have a look when time permits. Did you also update the IDE/compiler?
Cheers
Matthias

Your'e right, i hadnt declared global_tbuf , but have now.
I've uploaded my BresserWeatherSensorTTN3.ino and ...TTNCcfg3.h files to github for you to look at.
thye code compiles and uploads, but gives the guru meditation errors as shown in the previous post
My arduino ide is fully up to date, running version 2.1.1

Currently global_tbuf is never written to, but read in displayWeatherData(int ws). It seems you are lucky - global variables are initialized to zero (if I'm correct), so the string functions will find a terminating \0 immediately. But I would not rely on that and this is probably not what you intended to do.

Maybe a primer about the Adafruit_SSD1306 class helps.
Adafruit_SSD1306 inherits from  Adafruit_GFX
(see Arduino/libraries/Adafruit_SSD1306/Adafruit_SSD1306.h and Arduino/libraries\Adafruit_GFX_Library\Adafruit_GFX.h)
Adafruit_GFX inherits from Print (an Arduino class)
(see https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/Print.h)
That means, your display.print() method has the same interface as the Arduino print() method.
From Print.h:
    size_t print(const __FlashStringHelper *ifsh) { return print(reinterpret_cast<const char *>(ifsh)); }
    size_t print(const String &);
    size_t print(const char[]);
    size_t print(char);
    size_t print(unsigned char, int = DEC);
    size_t print(int, int = DEC);
    size_t print(unsigned int, int = DEC);
    size_t print(long, int = DEC);
    size_t print(unsigned long, int = DEC);
    size_t print(long long, int = DEC);
    size_t print(unsigned long long, int = DEC);
    size_t print(double, int = 2);
    size_t print(const Printable&);
    size_t print(struct tm * timeinfo, const char * format = NULL);

The first parameter shows which datatypes are supported - Arduino F("") strings, Arduino String class, C strings aka. character arrays, character, ...
The integer types of different width have a second parameter with default value DEC. This means without specifying anything else, the value will be printed as decimal. Other options are HEX, OCT and BIN.
The double type has a second parameter with default value 2. This is the number of fractional digits.

Now you only have to make sure that

Anything you want to print has a supported data type (or a typecast is possible, either automatically or explicitly)
The variable you want to access is within the scope of the function, i.e. is visible where the display.print() function is located

The compiler will help you if all warnings are enabled. (Please make sure they are!)

Another advice: If you save you code on GitHub, just overwrite your previous version with an update instead of using Sketch2.ino, Sketch3.ino etc. This allows to compare the different versions quite easily.

Thanks for all that Matthias, i guess what is confusing me is that the only thing i changed in the code was line 1756 from "White Lodge Weather" to "Cwm Hir Weather". Yes i updated the Libraries and had to comment out the lines about defining Bresser 7 in 1 and lightning , but other than that i changed nothing!
to me, the only things that have changed are the BresserweathersensorReciever  libraries
I may have found one issue, the wire hardwired to pin DIO1 on the board may have been contacting the metal cover as shown  by the green line in the picture:

Is it possible that i have fried the board?

Maybe. I don't remember the signal direction this wire is used for. If it is an output of the ESP32 and the port driver is affected, you could switch to another port.

BTW: Can you imagine any other heading for this issue? It seems the current one does not really fit any more.

I've ordered a new board, will try that and see if its a hardware issue :)
The board i have is not working with even basic code examples.
However it does seem to work somewhat with the weather sensor code, as it reports the correct weather values just before giving the guru meditation error!
I would argue that the heading does fit, after all the issues arose after i used the same code after updating the libraries?
If the new board works properly i will close the issue :)

Got the new board, but now I'm having issues with the complier https://forum.arduino.cc/t/issue-when-compiling/1151233/15 and here https://forum.arduino.cc/t/resolvelibray-arduinojson-h/1152357
Also, I keep getting errors causing the complier to crash that are just plain weird, as in they were not causing errors previously, but are now.
It just doesn't make sense!!

Well, I can only say that the latest CI run (compilation of the latest source code with the library versions defined in CI.yml using the current version of the Arduino command line tools) was just fine: https://github.com/matthias-bs/BresserWeatherSensorTTN/actions/runs/5678578038.
CI is done for several boards (currently 8), including esp32:esp32:ttgo-lora32:Revision=TTGO_LoRa32_v21new.

well i'm just about done with this. All I wanted to do was upfdate one line in the display and it has cost me over a weeks worth of data.
I have uploaded a folder to my github page which has the TTN .ino and Cfg.h files in, the ones from my windows machine, and those from my ubuntu. neither will compile. the compiler takes over 30 minutes on 'resolvelibrary<arduinoJson.h> and then the compiler either hangs forever (windows) or fails to compile with a variety of reasons (ubuntu) I have included screenshots of each machine at the end point of the compile process.
Both machines are running fresh installs of Arduino IDE, with all libraries as up to date as possible.
Niether the old board, or a brand new one (also with wirre from dpio2 to GPIO12) work on either machine.
I am really srtuggling to see where the issue is!!

Sorry about that! Never change a running system... If the data is precious, keep at least one last known good device.
Did you try to compile an ArduinoJson library example?
If you provide a compile log with all warnings enabled (text file, no screenshot!!!) I might have another look!

I never experienced any problems like those with the Arduino IDE. It's really strange that both the Linux and Windows installations fail.
You might consider to use the Arduino CLI tools. Or even set up the toolchain on GitHub.
I could change my CI workflow to keep the binary. You could copy the setup and just flash the binary from your local setup.

Don't know if this is related:
https://github.com/G6EJD/ESP32-e-Paper-Weather-Display/issues/222#issuecomment-1656693415

Interesting, I was going to try rolling back the arduinojson library to see when these compiler errors started

Fresh install on  my linux machine, all libraries installed (latest versions) have dumped the compiler output to .txt :
compile_log.txt
different error this time, 'multiple definitions of hal_init ???

Hi, any news on that?

Closed due to inactivity.


Denyanify: Links are broken after publishing a post

Summary
Links are broken after publishing a post
Expected Behavior
Links should be intact
Actual Behavior

Steps to Reproduce
This only happens with the letter combination na  in a link, and the plugin Denyanify enabled.

@volpeon I'm not sure if you're still maintaining this script... I think it was a known problem at the time with no easy way to fix, right?

Oh sorry. I saw this script here, and thought you are the maintainer.

Oh sorry. I saw this script here, and thought you are the maintainer.

Oh sorry. I saw this script here, and thought you are the maintainer.

I think it was a known problem at the time with no easy way to fix, right?

Yes, catching all of the edge cases would slow the script down to a crawl because the engine is very inefficient, so it's just a proof of concept.

@memo-567 No worries! This repo is a collection of stuff people have submitted... the only thing I maintain is the repo itself ^^
@volpeon Ah okay, in that case I'll close the issue :)


[BM-616] Disable onboarding after first app opening

Task

JIRA

Description

Add default FIRST_OPEN state set to UNDEFINED after onboarding

          1 Warning

:warning:
app/src/main/kotlin/co/netguru/baby/monitor/client/data/splash/AppStateHandler.kt#L8 - The class AppStateHandler defines no functionality and only holds data. Consider converting it to a data class.

          1 Message

:book:
:link: BM-616

JaCoCO Code Coverage 13.07% :white_check_mark:

Class
Covered
Meta
Status

co/netguru/baby/monitor/client/data/splash/AppStateHandler
0%
0%
:white_check_mark:

co/netguru/baby/monitor/client/feature/onboarding/FinishOnboardingUseCase
100%
0%
:white_check_mark:

co/netguru/baby/monitor/client/feature/onboarding/FeaturePresentationFragment
0%
0%
:white_check_mark:

co/netguru/baby/monitor/client/feature/splash/SplashFragment
0%
0%
:white_check_mark:

APK file size

size

30.4MB

Number of method references

Dex file
count

classes.dex
46099

  Generated by :no_entry_sign: Danger


Error resolving plugins.

Compiling the Documark example gives the following error:
Compiling..
Error!

/usr/local/lib/node_modules/documark/bin/documark-compile.js:39
	throw e;
	      ^
Error: Cannot find module 'documark-theme-default'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at nextPlugin (/usr/local/lib/node_modules/documark/lib/Documark.js:152:14)
    at Documark.applyPlugins (/usr/local/lib/node_modules/documark/lib/Documark.js:161:3)
    at Stream.<anonymous> (/usr/local/lib/node_modules/documark/lib/Documark.js:197:9)
    at Stream.stream.write (/usr/local/lib/node_modules/documark/node_modules/event-stream/node_modules/through/index.js:26:11)
    at Stream.ondata (stream.js:51:26)
    at Stream.emit (events.js:95:17)

This happens because Documark tries to resolve the package from its own location instead of the document root. I will have to rewrite it to take a Grunt CLI like approach.

Fixed. Use the documark-cli from now on.


Make polluting global scope optional

Hi,
We'd like to use adapter.js in our project but we don't agree with the decision to pollute the global scope. It's been mentioned in the past but not fixed. The current non-global approach only avoids exposing the adapter object, but still modifies the prototypes of native objects like RTCPeerConnection.
Our SDK is included on many different webpages so we do our best to avoid polluting the global scope because our clients are likely to be making decisions based on its state. Including adapter.js in its current form could invalidate their assumptions. It would be great if we could find a way to resolve this. I have a possible solution:
I propose that adapter and its shims be converted into factories that have the window object injected into them rather than always modifying the global scope. The existing adapter release files will still behave as they currently do by invoking the main factory with the global window object. An additional release file exposing the underlying factory will enable us to provide our own custom window-like object for the shims to modify.
The diff view of this PR is quite large but that's mostly indentation so please view it by ignoring whitespace changes.
Taking a look at just the changes to the safari shim (with whitespace ignored) should demonstrate the general pattern.
I tried to stick to the following when implementing this solution:

Don't break backwards compatibility with the existing adapter.js API
Don't change the general structure of the project (excluding whitespace changes)
Prefix global objects with window. where possible

If a global object is used often (eg. RTCPeerConnection) then declare it as a local variable and ensure its always pointing to the window property of the same name.

This PR unfortunately touches almost every line in the src folder so I made an alternative PR that makes as little change as possible in terms of lines, unfortunately the negative is that it's not backwards compatible with the existing adapter.js API: #548 It works in a similar way but instead of converting the shims into factories, an additional window argument is added to each shim function.
The most ideal solution is for the project to be restructured to accomodate these requirements (and others, such as properly modularising the shims) but I assume that's not likely to happen, mostly because it would break backwards compatibility.
The unit tests and integration tests all pass and it should be pretty seamless for both users and developers of adapter.js. Let me know if you have any questions, concerns or alternative solutions.
Note: There are quite a few conflicts because of all the PRs that have been merged in the past week but it should still illustrate the solution. If you agree with this approach then I can quickly work on resolving the conflicts.
Aiham

I have a slight preference for #548 but could live with either style

No problem, I'll close this and focus on #548

wait, lets ask for @KaptenJansson for an opinion too ;-)

I also prefer #548 since I think adapter has most value as a complete shim hence there is not really a need to make it more modular now. And I assume people do not want to inject different window objects in each shim?
I also wonder if it then makes sense to remove the extra build artifacts in https://github.com/webrtc/adapter/blob/master/Gruntfile.js#L18-L49 ? I think the MS edge should be standard now, we can do this in a follow up PR.

@KaptenJansson Sounds good. I'll stick with #548 then.


Set initial SNI hostname when creating an OpenSSL engine in client mode

Motivation:
When using the JDK SSL provider in client mode, the SNI host names (called serverNames in SslEngineImpl) is set to the peerHost (if available) that is used to initialize the SSL Engine:
http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/687fd7c7986d/src/share/classes/sun/security/ssl/SSLEngineImpl.java#l377
This allows one to call SslEngine.getSSLParameters() and inspect what is the SNI name to be sent. The same should be done in the OpenSSL provider as well. Currently even though the the SNI name is sent by the OpenSSL provider during handshake when the peerHost is specified, it is missing from the parameters.
Modification:
Set the sniHostNames field when SNI is to be used. Also verifies the peer is actually a hostname before setting it as the SNI name, which is consistent with JDK SSL provider's behavior.
Result:
SslEngine using the OpenSSL provider created in client mode with peerHost should initialize sniHostNames with the peerHost.
Calling SslEngine.getSSLParameters().getServerNames() should return a list that contains that name.
PS:
I'm not sure where to put tests. Please advise.

PTAL. All tests passed.

@jianglai please address the last one comment and squash. Will merge after this was done.

@jianglai also please sign the ICLA:
https://netty.io/s/icla

Removed reference to internal methods and squashed. PTAL.
Regarding the CLA, I am contributing as an employee of Google. Do I still need to sign an individual CLA? Please advise. Thank you!

Understood. ICLA signed.

@jianglai thanks!


[Feature] Quota Request pre Deployment

Is there an existing issue for this?

[X] I have searched the existing issues

Microhack
03-Azure/01-03-Infrastructure/06_Migration_Datacenter_Modernization
Feature Description
Add documentation or automation for quote requests based on the number of participants.
Anything else?
This is needed since the introduction of multiple deployments per subscription.

link #73 as the requirement results in the need for this


test(NODE-5852): Skip transaction unpin spec tests

Description
Skip transaction unpin tests related to a build failure.
What is changing?
Skip relevant tests.
Is there new documentation needed for these changes?
No.
What is the motivation for this change?
Build failure described in NODE-5852

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: type(NODE-xxxx)[!]: description

Example: feat(NODE-1234)!: rewriting everything in coffeescript

[x] Changes are covered by tests
[x] New TODOs have a related JIRA ticket

Just noticed 'Unpin when a non transaction user uses a session also fails' is also flaky.


Make npx tsc after npm install work again

Need
A dependency or transient dependency has changed, and doing an npm install then npx tsc no longer works.
There are compile errors.
Doing an npm ci with the checked-in package-lock.json file does work. You can run npx tsc after that ok.
If you delete the package-lock and run npm install, the compile will fail.

What is the upgrade that is breaking this?
What fixes are required to get the compile after npm install to work again?
What can we add to the build to test that the npm tsc runs with a new package json?

Complete


Question: Mztab score meanings

Hi I was wondering if you can help me with understanding the different columns with scores, the 'opt_ms_run[1]_aa_scores' and the 'search_engine_score[1]'.
I have ran my own files in and underneath the 'search engine score' column, there are negative values. I was wondering if you could help me understand this?

search_engine_score[1]: This is the score for the full peptide. Negative peptide scores arise from predictions that do not fit the precursor mass tolerance, in which case the score is penalized.
opt_ms_run[1]_aa_scores: These are the scores for the individual amino acids in the peptide sequence, separated by comma's.

We are still in the process of updating our documentation with this information (#27).

The model outputs a confidence score between 0 and 1. We subtract 1 if the predicted m/z is outside the specified m/z window.


ENH: Add MicroBadger Label Schema + update dockerfile version to support ARG in circleCI

Rebased on origin/centos of the previous PR https://github.com/thewtex/docker-opengl/pull/11
@thewtex Hope I have not forgotten anything this time :)

Excellent!


'npx nuxi info' command actual version not display

Environment

Operating System: Darwin
Node Version:     v18.13.0
Nuxt Version:     3.0.0-rc.14
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, target, routes, app, css, vite, build, components, imports, modules, buildModules, nitro, i18n, image, vueuse, typescript, facebook, runtimeConfig
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules<EMAIL_ADDRESS>unplugin-icons/nuxt

Reproduction
npx nuxi upgrade --force
and run
npx nuxi info
then,
The actual used version and the displayed version
are displayed differently like the screenshot.
(nuxt  version, nitro version)

Describe the bug
npx nuxi info actual using version and displayed version is difference.
(nuxt, nitro version)
Additional context
No response
Logs
No response

Hey, could you please paste here your package.json?
It seems to me that the nuxi info might be returning a content of package.json while the actual used version (from lock file) is the one that is updated from nuxi update --force.

@Baroshem
here it is my project 'package.json'
i also use 'npx nuxi upgrade --force'
🥲
'''
{
"private": true,
"engines": {
"node": ">=18.12.0"
},
"scripts": {
"dev": "nuxt dev --dotenv .env.dev",
"stage": "nuxt dev --dotenv .env.stage",
"build": "nuxt build --dotenv .env.stage",
"generate": "nuxt generate -dotenv .env.stage",
"preview": "nuxt preview",
"postinstall": "nuxt prepare"
},
"devDependencies": {
"@element-plus/nuxt": "^1.0.3",
"@intlify/unplugin-vue-i18n": "^0.8.1",
"@nuxt/image-edge": "^1.0.0-27919678.2f5b64b",
"@nuxt/typescript-build": "^2.1.0",
"@nuxtjs/device": "^3.0.0",
"@nuxtjs/eslint-config-typescript": "^12.0.0",
"@typescript-eslint/eslint-plugin": "^5.50.0",
"@typescript-eslint/parser": "^5.50.0",
"eslint": "^8.33.0",
"eslint-plugin-vue": "^9.9.0",
"firebase": "^9.16.0",
"nuxt": "^3.2.0",
"nuxt-simple-sitemap": "^1.0.6",
"typescript": "^4.9.5",
"vue-i18n": "^9.2.2"
},
"dependencies": {
"@element-plus/icons-vue": "^2.0.10",
"@nuxt/typescript": "^2.8.1",
"@nuxtjs/robots": "^3.0.0",
"@pinia/nuxt": "^0.4.6",
"@vueuse/core": "^9.12.0",
"@vueuse/nuxt": "^9.12.0",
"element-plus": "^2.2.29",
"json-hash": "^1.2.0",
"nuxt-facebook-pixel-module": "^1.5.0",
"nuxt-jsonld": "^2.0.7",
"pinia": "^2.0.30",
"qs": "^6.11.0",
"sass": "^1.58.0",
"sass-loader": "^13.2.0",
"unplugin-icons": "^0.15.2",
"unplugin-vue-components": "^0.23.0",
"vue-dragscroll": "^4.0.5",
"vue-gtag": "^2.0.1",
"vue-social-sharing": "^4.0.0-alpha4",
"vue3-touch-events": "^4.1.0"
}
}
'''

humm. i see i'ill try again!
i try reproduction, but, in stackbliz this phenomenon has not been reproduced.
(try re install nuxt:v3.0.0-rc.14) -> npx nuxi info -> but, not updated nuxi info below
i'll update my reproduction :) !!

@Baroshem  It might be cache problem, my office project is fine :)
I'll closed Issue! Thanks ❤


Messages not moved with Courier IMAP

Describe the bug
I have a server running Courier IMAP, deleting messages and sending an expunge works perfectly but because I'm required to keep the messages on the server, move them to a folder and let the customer delete them afterwards.
Code to Reproduce
            $folder->getClient()->getConnection()->enableDebug();
            $message = $folder->messages()->getMessage($msgid);
            if ($account->isImap()) {
                $message->move('INBOX'.$folder->delimiter.'procesados');
            } else {
                $message->delete();
            }

Expected behavior
Move the message to the "processed" folder.
Desktop / Server (please complete the following information):

OS: Debian 9
PHP: 7.4.29
Version [e.g. v2.3.1]
Provider: Courier IMAP

Here's the protocol exchange since the moment i'm trying to move the message:
>> TAG16 SELECT "INBOX.procesados"
<< * FLAGS (\Draft \Answered \Flagged \Deleted \Seen \Recent)
<< * OK [PERMANENTFLAGS (\* \Draft \Answered \Flagged \Deleted \Seen)] Limited
<< * 0 EXISTS
<< * 0 RECENT
<< * OK [UIDVALIDITY 664533991] Ok
<< * OK [MYRIGHTS "acdilrsw"] ACL
<< TAG16 OK [READ-WRITE] Ok
>> TAG17 EXAMINE "INBOX.procesados"
<< * FLAGS (\Draft \Answered \Flagged \Deleted \Seen \Recent)
<< * OK [PERMANENTFLAGS ()] No permanent flags permitted
<< * 0 EXISTS
<< * 0 RECENT
<< * OK [UIDVALIDITY 664533991] Ok
<< * OK [MYRIGHTS "acdilrsw"] ACL
<< TAG17 OK [READ-ONLY] Ok
>> TAG18 EXPUNGE
<< TAG18 NO Cannot expunge read-only mailbox.
>> TAG19 LOGOUT
<< * BYE Courier-IMAP server shutting down
<< TAG19 OK LOGOUT completed

Hi @rmontagud ,
many thanks for your detailed report.
This looks like the folder you are trying to move to is in a "read-only" mode: TAG18 NO Cannot expunge read-only mailbox..
At least that how I would read this error message. Can you move the message to a folder which isn't in "read-only" mode?
Best regards,

Hi @Webklex ,
I've tried to move the message using roundcube and i've been able to do it so it doesn't seem a read-only issue... i've changed the destination folder to "Drafts" in my code and the result is the same
>> TAG14 SELECT "INBOX.Drafts"
<< * FLAGS (\Draft \Answered \Flagged \Deleted \Seen \Recent)
<< * OK [PERMANENTFLAGS (\* \Draft \Answered \Flagged \Deleted \Seen)] Limited
<< * 0 EXISTS
<< * 0 RECENT
<< * OK [UIDVALIDITY 666948962] Ok
<< * OK [MYRIGHTS "acdilrsw"] ACL
<< TAG14 OK [READ-WRITE] Ok
>> TAG15 EXAMINE "INBOX.Drafts"
<< * FLAGS (\Draft \Answered \Flagged \Deleted \Seen \Recent)
<< * OK [PERMANENTFLAGS ()] No permanent flags permitted
<< * 0 EXISTS
<< * 0 RECENT
<< * OK [UIDVALIDITY 666948962] Ok
<< * OK [MYRIGHTS "acdilrsw"] ACL
<< TAG15 OK [READ-ONLY] Ok
>> TAG16 EXPUNGE
<< TAG16 NO Cannot expunge read-only mailbox.
>> TAG17 LOGOUT
<< * BYE Courier-IMAP server shutting down
<< TAG17 OK LOGOUT completed

After a google search I ended in a bugreport from MailKit where it seems that if you send the EXAMINE  command the folder stays as read-only (which might match the answer to TAG15?).
Also, I've noticed $message->move('INBOX'.$folder->delimiter.'procesados'); returns null, so  I suspect the actual MOVE command has not been sent at all because Message::move() cannot determine the "uidnext".


chore(pnpm): use catalog

https://pnpm.io/catalogs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.57%. Comparing base (ccdd930) to head (f6dbc37).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1229   +/-   ##
=======================================
  Coverage   70.57%   70.57%
=======================================
  Files          70       70
  Lines         656      656
  Branches      131      132    +1
=======================================
  Hits          463      463
  Misses        171      171
  Partials       22       22

Components
Coverage Δ

@suspensive/utils
47.16% <ø> (ø)

@suspensive/react
100.00% <ø> (ø)

@suspensive/react-dom
100.00% <ø> (ø)

@suspensive/react-native
100.00% <ø> (ø)

@suspensive/react-query
72.09% <ø> (ø)

@suspensive/react-query-4
0.00% <ø> (ø)

@suspensive/react-query-5
0.00% <ø> (ø)

@suspensive/react-query-next-experimental
∅ <ø> (∅)

@suspensive/react-query-next-experimental-4
6.15% <ø> (ø)

@suspensive/jotai
0.00% <ø> (ø)

@suspensive/cache
98.01% <ø> (ø)

@suspensive/react-image
80.39% <ø> (ø)

I agree with using the catalog protocol! The configuration has been set up correctly.
Optionally, what do you think about using named-catalogs?
With React 19 coming soon, it might be challenging for all of our packages to support React 19 simultaneously. So, perhaps we could use named-catalogs to differentiate between React versions in advance? (This is just my personal opinion. I agree with this PR and will approve it as everything looks good!)
Example:
catalogs:
  react17:
    react: ^17
    react-dom: ^17
    @types/react: ^17
# React 19 is just an example. I think it would be great to set it up like this for migration when ready-packages are compatible in the future.
  react19:
    react: ^19
    react-dom: ^19
    @types/react: ^19

Oh..! Could you do it after merging this Pull Request? in my opinion, React Query version could be matter for your comment's point!

Oh..! Could you do it after merging this Pull Request? in my opinion, React Query version could be matter for your comment's point!

Sure, I'll try it after this PR is merged


Schema Evolution in Parquet

Rebase Schema Evolution in Parquet

Now that #2258 is merged, can we close this one?

yes, the type conversion stuff is covered in https://github.com/facebook/presto/issues/2255


Generate deterministic Guids for tag helpers

#2449

Using the hash of file checksum along with the absolute position in the generated code to create the unique id.
Regenerated baselines

🆙 📅 @Eilon


fix: refactor and fix to_unixtime

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Main changes:

Fixed to_unixtime process the TimestampMillisecondVector and TimestampNanosecondVector not correctly, return the wrong value. It should return the unix timestamp in seconds.
Refactor to_unixtime function to avoid some unnecessary conversions.
Supports date type.

Checklist

[x]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@waynexia PTAL


Typo in WVD classic discovery URL

The "Windows Virtual Desktop (classic)" discovery URL is misspelled.  It says "webfedd" instead of "webfeed".

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

ID: ca8081a3-ec92-8aa5-2a51-1c10a298fdd1
Version Independent ID: 6c33184e-2ef8-0b7c-8666-50d4c4b16c7e
Content: Set up email discovery to subscribe to your RDS feed
Content Source: WindowsServerDocs/remote/remote-desktop-services/rds-email-discovery.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @ChristianMontoya
Microsoft Alias: chrimo

@rpstester https://rdweb.wvd.microsoft.com/api/feeddiscovery/webfeeddiscovery.aspx
https://rdweb.wvd.microsoft.com/api/arm/feeddiscovery
not miss spelled

The link was corrected in commit https://github.com/MicrosoftDocs/windowsserverdocs/commit/5f9957172568e0b443c785638da8ff0815a25a12 yesterday (March 12, 2021, at 03:03 AM GMT+1), and this ticket was opened on March 11, 2021 at 09:44 PM GMT+1, 5 hours and 19 minutes earlier.
It looks like this ticket could have been a source for the typo being corrected.

That's correct. Thanks for reporting this typo.
#please-close


Array as argument to match_response_schema

Closes #60
Add support for an Array matcher argument similar to Hash matcher
arguments.

Thanks for the amazingly speedy fix @seanpdoyle, this works nicely.


Hide DS4 is not working with new DS4 PID added to code

I made a pull request where I added the PID of the new DS4 to the code (#179), but with that added "Hide DS4" no longer works.
I get a warning that apps using the controller have to be closed. I've tried looking at the source, but can't see where the device is marked as not open. @Jays2Kings, you might know what's happening. 😄

Give this a try....  close DS4Windows completely (eg: not running in systray)
Open Device Manager
Navigate to Human Interface Devices
Disable all entries related to your DS4 controllers (Bluetooth HID Device)
Re-enable all entries related to your DS4 controllers
Run DS4Windows and try connecting a controller

Probably need to make the new PID to work along with the teokp fix https://github.com/teokp/DS4Windows which is already currently implemented for the old controller at least. Basically does what TristanYoung said but automatically when controller is first connected.

I wish I new C#.  Must make an effort to learn it.  I've looked at teokp's fix source, and while I see where it is disabled and then subsequently re-enabled, I don't quite understand how it doesn't just do it automatically like the v1 controller.  I don't see anywhere where it's referencing the PID data andersfischernielsen added.
Boy, as programming changed since I last did anything.  I am keenly interested in learning, would love to see the fix to understand.

I'm looking over the code.  Considering the pre-existing PID's - 0xBA0, 0x5C4 - shouldn't 0x09CC simply be 0x9CC?  Would that even make any difference?  Sorry, I'm unfamiliar with modern windows programming and C#; I would think they mean the same.
If teokp's fix is only checking the first one, then that means the v1 controller also won't work with teokp's fix, since it's not the first PID encountered.

Interesting.  Playing some more...

I restarted my system, and tried to use my new v2 controller - no go
Tried disabling then re-enabling the controller in device manger - only the one corresponding to my v2, appears third in the list - no go
Tried disabling then re-enabling the second controller in the list - no go
Tried disabling then re-enabling the first controller in the list - success!

I wont be able to get my hands on the newer version of the DS4 for a week or 2, but for now I've added the PID to DS4Windows. Hopefully once I get one, I can check for any issues firsthand

I've been using DS4Windows with this fix for a week now (playing The Witcher 3 - woo), and everything's working as it's supposed to. I still can't hide the DS4, but that seems to be a general issue on Win 10 that people are having.


Add more checks to pre-commit

Codecov Report

Merging #341 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #341      +/-   ##
==========================================
- Coverage   99.93%   99.93%   -0.01%
==========================================
  Files          23       23
  Lines        1620     1619       -1
  Branches      257      257
==========================================
- Hits         1619     1618       -1
  Partials        1        1

Flag
Coverage Δ

#unittests
99.93% <100%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/runner/__init__.py
100% <ø> (ø)
:arrow_up:

src/schemathesis/schemas.py
100% <100%> (ø)
: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 201ad2e...d6d767b. Read the comment docs.


Get to parity with JS SDK spec version v0.4.2

This PR is set up to track progress on the work to bring the Go SDK up to parity with JS SDK at spec v0.4.2. I'll merge changes into this branch as I make them, then this PR can be merged into main when the work is done. The final merge of this PR will close #9.
The required tasks are:

[x] Handle "null condition" evalCondition test cases.
[x] Implement new hashing algorithm and tests.
[x] Implement ranges and relevant feature and run tests.
[x] Add meta information, key and name everywhere they're needed (plus feature tests).
[x] Implement filters (and corresponding tests).
[x] Fix remaining spec tests.
[x] Add remaining feature evaluation logic from JS SDK not covered by JSON tests
[x] Port over all relevant tests from JS SDK
[x] Tidying and refactoring after logic changes
[x] Implement feature usage callback.
[x] Implement feature loading, repositories and caching.
[x] Update initialization API to make use of automatic feature loading.
[x] Encrypted features and experiments (and tests).

Remaining tasks:

[x] Version string comparison operators
[x] Pluggable cache API (as in Python SDK)
[x] Documentation updates

Sounds good re: this PR, just ping me when it's ready @ian-ross

Tests are failing just because I copied some tests for as-yet unimplemented features into the JSON test file.

@tinahollygb This is ready for a proper review now. I've written documentation changes in https://github.com/growthbook/growthbook/pull/1420
Let me know if there's anything you don't like here. I'm very happy to change things, add more tests, etc.  (although I think the test coverage is pretty good, the feature repository code has a little complexity to it, so it's hard to be 100% sure that it's perfect).

Also, do you have any idea what this failure in the docs build is? https://github.com/growthbook/growthbook/actions/runs/5415884987/jobs/9844819702

I can add an SSE example. I've actually only tested that code using a local test server, since I've not figured out how to switch the feature on in the GrowthBook UI. The testing with the local server is pretty extensive, but it would definitely be good to do something with real updates coming from the front-end.


It's not ticking in Safari

Hi, I've use this and it works in Chrome but not Safari.
Any idea how to fix this?
Thanks!

@pizza0502 I don't have a macOS available for testing with Safari. If you please could test it again this new 0.4.0 release it would be much appreciated.


fix: Always use bbox bounds for Controlnet Image (canvas)

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[ ] Yes
[ ] No, because:

Have you updated all relevant documentation?

[ ] Yes
[ ] No

Description
Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

Added/updated tests?

[ ] Yes
[ ] No : please replace this line with details on why tests
have not been included

[optional] Are there any post deployment tasks we need to perform?

Ah yeah. No cropping visible for ControlNet. Let me find it.


Incorrect access control or stale Natspec/docs on UbiquityDollarToken.setIncentiveContract()

Title
Incorrect access control or stale Natspec/docs on UbiquityDollarToken.setIncentiveContract()
Vulnerability details
UbiquityDollarToken.setIncentiveContract() is used to set incentive contracts for accounts.

    function setIncentiveContract(address account, address incentive) external {
        require(
            accessControl.hasRole(GOVERNANCE_TOKEN_MANAGER_ROLE, _msgSender()),
            "Dollar: must have admin role"
        );

        incentiveContract[account] = incentive;
        emit IncentiveContractUpdate(account, incentive);
    }

The issue here is with access control given to the above function. The revert message states "Dollar: must have admin role", it means the function is expected to have Admin role as only address who can access the UbiquityDollarToken.setIncentiveContract() function, However, the GOVERNANCE_TOKEN_MANAGER_ROLE is set which seems to be incorrect. On further study on this issue, the readme docs of incentive further confirms that the acces control should be given to Admin role.
Per interface.IIncentive.md,

Dollar admin can set an incentive contract . . .

Per IIncentive.sol

Dollar admin can set an incentive contract . . .

It can be said that the issue either relies with stale or incorrect readme documentation or Natspec or Incorrect function access implementation.
Recommendation
If the UbiquityDollarToken.setIncentiveContract() function is expected to be called by admin then do the following change,

    function setIncentiveContract(address account, address incentive) external {
-        require(
-            accessControl.hasRole(GOVERNANCE_TOKEN_MANAGER_ROLE, _msgSender()),
-            "Dollar: must have admin role"
-        );
+        require(
+            accessControl.hasRole(DEFAULT_ADMIN_ROLE, _msgSender()),
+            "Dollar: must have admin role"
+        );

        incentiveContract[account] = incentive;
        emit IncentiveContractUpdate(account, incentive);
    }

OR . . .
If function implementation is correct then correct the stale docs/Natspec mentioning tokenManager instead of Admin to avoid confusion to code readers.
cc- @pavlovcik @molecula451

This was purged as a part of the fix-review branch https://github.com/ubiquity/ubiquity-dollar/commit/bb6543eb61e8f328a9b12d20735d4785e9570c2b


Infer target version

It’s derived from requires-python in both Ruff:

target-version
[…]
If omitted, and Ruff is configured via a pyproject.toml file, the target version will be inferred from its project.requires-python field (e.g., requires-python = ">=3.8").

and Black (in https://github.com/psf/black/pull/3219):

-t, --target-version
[…] By default, Black will try to infer this from the project metadata in pyproject.toml. […]

Test failures are
 intersphinx inventory 'https://anndata.readthedocs.io/en/stable/objects.inv' not fetchable due to <class 'requests.exceptions.HTTPError'>: 403 Client Error: Forbidden for url: https://anndata.readthedocs.io/en/stable/objects.inv

wtf


bump cryptography version to latest

Description
Update cryptography dependency to latest patch version.

I think the rebase went sideways. Will fix.

The library is already at the latest version in main. Closing.