Order of "Recently opened" and "File (and symbol) results" should be reversed by default and/or configurable

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

VS Code Version: 1.74.1
OS Version: Windows_NT x64 10.0.19044

Steps to Reproduce:

CTRL+P or use the Search box at the top of the window if enabled
Search for a very common filename that you would find in lots of different repositories/workspaces but typically only once in any given one, such as package.json

Expected results:
The files from the current workspace are autocompleted first, for example  CTRL+P package.json ENTER should open the package.json from the root of the current project.
Actual results:
The most recently opened files are autocompleted first even if they are from a completely different project, for example CTRL+P package.json ENTER opens a package.json from some other project I opened yesterday rather than the project I am working on right now.
You can turn off search.quickOpen.includeHistory but it's a useful feature, I often want to copy and paste from or look and what I did in another project.  And it's probably worth making an option since some people do work in huge monorepos where it useful to surface the most recently used files at the top, but then again those people could just set search.quickOpen.history.filterSortOrder to recency too.

I think this is basically a duplicate of https://github.com/microsoft/vscode/issues/5806 and is covered in https://github.com/microsoft/vscode/issues/27317
but I'll leave this open as a feature request.


fix: improve typing of flatten to infer return iterator type

Also add a test for flatten, based on flatten.test.ts

@felixfbecker is this PR waiting for something else?

@cojack please read the discussion above.

@cojack thanks for the reminder! i forgot i had this one open. I'll get to it soon, and I can also set up dtslint or https://www.npmjs.com/package/tsd in a new PR.


Python: use fixed-point combinator, add pkgs to interpreter

Motivation for this change
Use a fixed-point combinator for the Python package set to allow easier overriding of its contents.
Earlier implementations were proposed in #16784 and #17428. This commit is by comparison much smaller and changes only what is needed.
A package set is constructed for a specific interpreter. Therefore, we add the
possibility to override the package set to the interpreter. This should make it
easier to override the interpreter and the package set at the same time.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

This is a smaller change than I proposed in https://github.com/NixOS/nixpkgs/pull/16784.
All packages work just fine with this change. With this change I don't think it will be possible anymore to have overrides of self inside the package set. This was done in the past to make chains of packages use a different version. While that sounds useful it is also very fragile since Python can handle only one version of a package at a time.

You rock :)


Make list of cards in final pdf customizable

By adding some \filter command, that uses somewhat weird pgfkeys logic.
We swallow the content of cards, that should not be visible, by looking for some \end{cards} (so this will break whenever we'll do recursive cards, but we should not do that). We only read ahead token by token (or group by group).
This allows to display only some cards by doing things like
\showCards{all} % show all cards
\showCards{title=Heal} % show all cards with Heal title
\showCards{title=Heal, level=1} % show all cards named "Heal" or that have level=1
\showCards{type=Spell} % show only spells

You can't really express everything, as the conditions are "OR"-ed together.

We can merge this if it helps with @fkz's workflow.

This could help with

would be great if one could preview multiple cards at once…
This could even be a pre-defined set to show the full range of card types and features we use…
/show-preview or /preview-deck or something like that

I want to have faster compile times, and also be able to just work on a specific card without having to find it in the increasingly bigger pdf.
I don't particularly like the "filter language" currently, I wonder if pgfkeys is really a good fit here anyway.
But let's merge this and improve later :)

Not sure how much this helps with the "/show" command.
It currently works by matching the bookmarks in the generated pdf, as that is faster than calling latex (which is slow mostly because it first needs to download the whole of TeXLive).


Tests backward compatibility

Run latest Pelican binary in the container built for Pelican v7.8 release and run tests
Run v7.8 release in the latest container and run tests

tl;dr: No, they don't work. If the tag mismatched binary and image are put together, they crash badly.

  |   | pelican binary version
-- | -- | --
  |   | main (latest) | v7.8 release (v7.8.9)
pelican container image version | v7.11.7 (latest) | \ | all tests passed; goreleaser build succeeded; Pelican running failed*
v7.8 release (v7.8.9) | unit test (TestHttpOriginConfig) failed** | \

note: latest = as of Dec 3; all the tests happen in the same container (which means no cross-container test)
*error message: http: TLS handshake error from <IP_ADDRESS>:45812: remote error: tls: bad certificate
**error message: Xrootd initialization failed. So there's no need for goreleaser and Pelican running tests
These images doesn't contain many necessary packages, including Go, Goreleaser, make, Node.js, npm

Running a container built from the images hosted on Harbor could be challenging. Here is my solution for macOS users:

error message: http: TLS handshake error from <IP_ADDRESS>:45812: remote error: tls: bad certificate

That suggests a simple TLS configuration issue, not anything systematic.  What's causing it?

error message: http: TLS handshake error from <IP_ADDRESS>:45812: remote error: tls: bad certificate

That suggests a simple TLS configuration issue, not anything systematic. What's causing it?

It fails at the beginning of registry initialization. I double checked that TLSSkipVerify is set to true. I also tried to delete /dist and /run/pelican and build again, but it doesn't work. Here is the full log

[root@87f35ac5cf5f pelican_linux_arm64_v8.0]# ./pelican registry serve -p 8446
Version: 7.8.10-next
Build Date: 2024-12-04T15:56:36Z
Build Commit: b93a59097d088aee28b84279efc2d9e7ba555e78
Built By: goreleaser
2024/12/04 16:05:59 OK<PHONE_NUMBER>2712_create_db_tables.sql (2.29ms)
2024/12/04 16:05:59 OK<PHONE_NUMBER>4951_change_admin_metadata_default.sql (3.13ms)
2024/12/04 16:05:59 goose: successfully migrated database to version:<PHONE_NUMBER>4951
2024/12/04 16:05:59 http: TLS handshake error from <IP_ADDRESS>:39092: remote error: tls: bad certificate
2024/12/04 16:05:59 http: TLS handshake error from <IP_ADDRESS>:39100: remote error: tls: bad certificate
2024/12/04 16:05:59 http: TLS handshake error from <IP_ADDRESS>:39116: remote error: tls: bad certificate
...
^CERROR[2024-12-04T16:06:03Z] Web engine check failed:  context canceled
Error: context canceled
ERROR[2024-12-04T16:06:03Z] Fatal error occurred at the start of the program. Cleanup started: context canceled
Pelican is safely exited

Thanks for your pro-tip @turetske! I just updated the result above. Now running the v7.8.9 code/binary in latest (v7.11.7) container is all good, but the error persists in the other way. I also add a "controlled group" as the above shown.
Troubleshooting note: When I run multiple pelican containers, the https://localhost:8446/view/registry/ website could not be reached maybe due to some proxy conflicts. To fix it, just stop the container I'm not currently using.

@h2zh Most likely because @bbockelm added some configuration variables that are in the newer cache.config and not recognized with older xrootd.

More specifically, the problematic lines are probably line 30 and lines 61-64 in xrootd/resource/xrootd-cache.cfg

Why do we care about older xrootd with newer Pelican?
The project is to run an older Pelican in the CI pipeline - which would imply a new xrootd, no?

@bbockelm Yes, but I think you mentioned you wanted to test both ways, so that was done. Possibly as misunderstanding on our part. I agree with you.


Unified way to execute Hive and MR-based compaction jobs

Currently the execution of the Hive and MR-based compactors is not out of the box, needs extra work:

there's no implementation for MR compaction for CLI execution
Hive compactor is a standalone package that needs to be built separately

This PR tries to fix these issues. A new shell script bin/gobblin-compaction.sh was added with which one can execute both Hive and MR-based compaction jobs from the CLI.
Usage: gobblin-compaction.sh [OPTION] --type <compaction type: hive or mr> --conf <compaction configuration file>
Where OPTION can be:
  --projectversion <version>    Gobblin version to be used. If set, overrides the distribution build version
  --logdir <log dir>            Gobblin's log directory: if not set, taken from ${GOBBLIN_LOG_DIR} if present.
  --help                        Display this help and exit

I assumed that the executable gobblin-compaction.jar and tarball creation are no longer needed.

Coverage decreased (-0.5%) to 43.779% when pulling 4c528255a3d31a36542cd2623a14677e6d5a4285 on swisscom-bigdata:swc_compact into 911ab2eb57a36a6cacc373f4f3f409fba5adb257 on linkedin:master.

@zliu41 what do you think about this add-on? Can you please review?

Sure, I'll take a look.

LGTM. Thanks for the contribution.

@zliu41 thanks for the review. I'll create a PR for the documentation.


Hash password

I think it is a good practice to save a user's password to the database encrypted instead of in plain text.

Oh, isn't the password encrypted? I haven't checked myself, but I consider this very important. Maybe bcrypt or something similar could be used for that :-)

Hmm, perhaps read up a bit on best practices for that. I'd guess a
user-specific salt is also needed. Also, I don't think they need to be
encrypted, just hashed. With a user-specific salt. You could create the
salt by hashing some metadata about the user (username, first password,
time of creation) together with some random (to avoid problems with bad
PRNGs) and just store the salt next to the password.
On Thu, Oct 25, 2018, 10:20 Thomas Leister<EMAIL_ADDRESS>wrote:

Oh, isn't the password encrypted? I haven't checked myself, but I consider
this very important. Maybe bcrypt or something similar could be used for
that :-)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ortuman/jackal/issues/62#issuecomment-432978197, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0F3OLvaQiOxfmPOTX3S1uvqFR2Fn3-ks5uoYJ5gaJpZM4X5hvw
.

Of course, hashing the password is what I meant, sorry for the confusion.
I am not very used to the best practices there, but I guess just normally hashing md5- or SHA-1-hashing the password would already be sufficient.

Please don't use md5 or sha1. MD5 has been broken a long time ago and sha1 was broken in 2016 or 2017 ... sooo... better choose something serious ;-)

Even if SHA-1 weren't broken, the SHA series of hashes is designed to be fast, making them entirely inappropriate for protecting passwords in a database for which you really want a slow key-derivation function such as argon2 (the current OWASP recommendation, IIRC) or PBKDF.2 (like SCRAM uses). For XMPP it is common to store SCRAM bits as if the user had logged in with SCRAM-SHA-1 (that way if you support that auth mechanism you can use them directly, and if you're doing PLAIN you can treat it just like any other hash.

Since no one has so far, I would propose SCrypt or it's brother BCrypt.

+1 for Bcrypt, implementing it seems straightforward too: https://godoc.org/golang.org/x/crypto/bcrypt

👋 Hi everyone! Just wanted to share with you that since latest release we only store user passwords in hashed form into database (no more plain texts). Also, and additional security layer was added by providing support for salted+pepper password hashes.
If there's any other suggestion in this regard feel free to open an issue. 🙇


Guess My Number

PR Description 📜
Added new Game Guess My Number
Fixes #455

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[ ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Thank you @khushbumaheshwarii ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Follow proper convention make changes in your code as per contributing guideline also make your entry in main README and add your image 😃 in assets/images

Thank you  @khushbumaheshwarii , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Feat: Add authentik provider

Adding Authentik provider https://docs.goauthentik.io/docs/providers/oauth2/

@pilcrowOnPaper Hello! Just letting you know that I processed all the requested changes and should be ready for review. Just let me know if you want me to make any changes. Thanks for your time, and take care!


client.publish() crash on ESP32 related to string length

Hello,
When sending a JSON string larger than 128 chars, client.publish() causes heep overflow on the ESP32.
I have changed the buffer size with client.setBufferSize(512); but maybe it's not working or I'm not using it correctly.
here's the min reproducable code:
#include <WiFi.h>
#include <PubSubClient.h>
#include <LinkedList.h>
const char* ssid = "*****";
const char* password = "******";
class MqttPublisher{
  public:
  /// using <PubSubClient.h>
    #define _OWNER "fflori4n"
    #define _CLUSTER "ESP0"
    #define path "******"
    #define retryNo 5
    //const char* mqttToppic = (owner path cluster);

    const char* mqttToppic = "*****";
    const char* mqttUser = "*****";
    const char* mqttPassword = "******";
    const char* mqttServer = "<IP_ADDRESS>";
    const int mqttPort = 1883;
    String mqttMsg = "";
    PubSubClient client;

    MqttPublisher(PubSubClient client){
        this->client = client;
        client.setBufferSize(512);      /// buffersize is set here.
    }
    void init(){
      client.setServer(mqttServer, mqttPort);
      }
    int connect(){
      if(client.connected()){ /// TODO: check if network is available
        return 0;
      }
      Serial.print("Connecting to MQTT...\t\t");
      for(int i=0; i < retryNo; i++){
        if (client.connect("ESP32Client", mqttUser, mqttPassword )) {
          delay(1000);
          Serial.println("[ OK ]");
          return 0;
        }
        else{
          delay(15000);
        }
      }
      Serial.print("[ ER ] Can't connect to MQTT server: ");
      Serial.print(client.state());
      return -1;
    }
    int publishMqtt(){

      if(!client.connected()){
        if(this->connect() == -1){
          return -1;
        }
      }
      Serial.print("publishing... \t\t\t");

      //client.setBufferSize(512);   /// issue No 2 : if this is uncommented, ESP will also crash. But might be unrelated or I'm not using setBufferSize() corretly.

      String testMsg = "{ \"Temp0_ESP0_fflori4n\": 22.0, \"Temp1_ESP0_fflori4n\": 22.0, \"Humidity0_ESP0_fflori4n\": 35.0, \"Dust0_ESP0_fflori4n\": 0.6, \"Motion0_ESP0_fflori4n\": 1.0 }"; /// bad message, probably to long? - over default 128 chars
      //String testMsg = "{\"Temp0_ESP0_fflori4n\": 22.0, \"Temp1_ESP0_fflori4n\": 22.0, \"Humidity0_ESP0_fflori4n\": 35.0, \"Dust0_ESP0_fflori4n\": 0.6}"; /// good message, less than 128 chars. Sent without problem.

      Serial.println(testMsg.length());

      if (!client.publish(mqttToppic,  testMsg.c_str())) { ///crash, heep overflow somehow...
        Serial.println("[ ER ] publish failed");
        client.loop();
        return -1;
      }

      Serial.println("[ OK ]");
      client.loop();
      return 0;
    }
};
WiFiClient espClient;
PubSubClient client(espClient);
MqttPublisher publisher(client);

void setup() {
  Serial.begin(115200);
  Serial.println(F("Serial OK"));

  WiFi.begin(ssid, password);
  Serial.print("Connecting to WiFi..");
  while (WiFi.status() != WL_CONNECTED) {
    delay(500);
    Serial.print(".");
  }
  Serial.println("Connected to the WiFi network");
  publisher.init();
}
int tick = 2000;
void loop() {
 if(tick > 15000/50){
  tick = 0;
  publisher.publishMqtt();
 }
 delay(50);
 tick++;
}

and the serial output:
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0018,len:4
load:0x3fff001c,len:1216
ho 0 tail 12 room 4
load:0x40078000,len:10944
load:0x40080400,len:6388
entry 0x400806b4
Serial OK
Connecting to WiFi.......Connected to the WiFi network
Connecting to MQTT...		[ OK ]
publishing... 			151
Guru Meditation Error: Core  0 panic'ed (LoadProhibited). Exception was unhandled.
 With buffer you can do something like : ESP.getMaxFreeBlockSize() - 4096

Serial.println("[WSc] SENT: Broadcast message!!");
const size_t capacity = 1024;
DynamicJsonDocument doc(capacity);
doc["temp"] = temp;
doc["humi"] = humi;
doc["humi"] = humi;
doc["humi"] = humi;
doc["humi"] = humi;
doc["humi"] = humi;
String buf;
serializeJson(doc, buf);

The problem was with the publishing function, managed to fix it with a workaround splitting up the string, suggested in the large msg example.
client.publish() was replaced with client.beginPublish(), client.print(), and client.endPublish().
As in: https://github.com/knolleary/pubsubclient/blob/master/examples/mqtt_large_message/mqtt_large_message.ino
Thanks for the suggestions, especially the JSON buffer stuff is much appreciated and will be part of the code in the future.


Update test case so that checking errored case

h/t Josh Vermette

Looks great @insop !

Thank you @SebastianHurubaru , merging.


Missing SPDX LicenseDeclared and LicenseConcluded in Document

I was just trying a straight read-write of a sample SPDX document in this way:
package main

import (
        "os"

        "github.com/bom-squad/protobom/pkg/formats"
        "github.com/bom-squad/protobom/pkg/reader"
        "github.com/bom-squad/protobom/pkg/writer"
)

func main() {
        reader := reader.New()
        document, err := reader.ParseFile(os.Args[1])
        if err != nil {
                panic(err)
        }

        w := writer.New()

        w.WriteStreamWithOptions(
                document, os.Stdout, &writer.Options{Format: formats.SPDX23JSON},
        )

}

then
$ go run main.go package_json.spdx.json
{
    "spdxVersion": "SPDX-2.3",
    "dataLicense": "CC0-1.0",
    "SPDXID": "SPDXRef-DOCUMENT",
    "name": "spdx_with_json",
    "documentNamespace": "https://spdx.org/spdxdocs/",
    "creationInfo": {
        "licenseListVersion": "3.20",
        "creators": [
            "Tool: protobom-devel",
            "Tool: spdx-maven-plugin"
        ],
        "created": "2024-02-14T16:53:19Z"
    },
    "packages": [
        {
            "name": "org.json:json",
            "SPDXID": "SPDXRef-gnrtd144",
            "versionInfo": "20220924",
            "downloadLocation": "NOASSERTION",
            "filesAnalyzed": false,
            "homepage": "https://github.com/douglascrockford/JSON-java",
            "copyrightText": "UNSPECIFIED",
            "summary": "JSON is a light-weight, language independent, data interchange format.\n        See http://www.JSON.org/\n\n        The files in this package implement JSON encoders/decoders in Java.\n        It also includes the capability to convert between JSON and XML, HTTP\n        headers, Cookies, and CDL.\n\n        This is a reference implementation. There is a large number of JSON packages\n        in Java. Perhaps someday the Java community will standardize on one. Until\n        then, choose carefully.",
            "description": "JSON is a light-weight, language independent, data interchange format.\n        See http://www.JSON.org/\n\n        The files in this package implement JSON encoders/decoders in Java.\n        It also includes the capability to convert between JSON and XML, HTTP\n        headers, Cookies, and CDL.\n\n        This is a reference implementation. There is a large number of JSON packages\n        in Java. Perhaps someday the Java community will standardize on one. Until\n        then, choose carefully."
        }
    ]
}

The original document has licenseDeclared and licenseConcluded which are mandatory for SPDX 2.3:
$ jq . package_json.spdx.json
{
  "SPDXID": "SPDXRef-DOCUMENT",
  "spdxVersion": "SPDX-2.3",
  "creationInfo": {
    "created": "2024-01-22T09:38:52Z",
    "creators": [
      "Tool: spdx-maven-plugin"
    ],
    "licenseListVersion": "3.22"
  },
  "name": "spdx_with_json",
  "dataLicense": "CC0-1.0",
  "documentNamespace": "http://spdx.org/spdxpackages/spdx_with_json",
  "packages": [
    {
      "SPDXID": "SPDXRef-gnrtd144",
      "copyrightText": "UNSPECIFIED",
      "description": "JSON is a light-weight, language independent, data interchange format.\n        See http://www.JSON.org/\n\n        The files in this package implement JSON encoders/decoders in Java.\n        It also includes the capability to convert between JSON and XML, HTTP\n        headers, Cookies, and CDL.\n\n        This is a reference implementation. There is a large number of JSON packages\n        in Java. Perhaps someday the Java community will standardize on one. Until\n        then, choose carefully.",
      "downloadLocation": "NOASSERTION",
      "filesAnalyzed": false,
      "homepage": "https://github.com/douglascrockford/JSON-java",
      "licenseConcluded": "NOASSERTION",
      "licenseDeclared": "NOASSERTION",
      "name": "org.json:json",
      "summary": "JSON is a light-weight, language independent, data interchange format.\n        See http://www.JSON.org/\n\n        The files in this package implement JSON encoders/decoders in Java.\n        It also includes the capability to convert between JSON and XML, HTTP\n        headers, Cookies, and CDL.\n\n        This is a reference implementation. There is a large number of JSON packages\n        in Java. Perhaps someday the Java community will standardize on one. Until\n        then, choose carefully.",
      "versionInfo": "20220924"
    }
  ]
}

I've attached the json document.
package_json.spdx.json

I'm thinking that as far as reading SPDX, I don't think protobom needs to store licenseConcluded. However, when writing SPDX, it should be creating a licenseConcluded=NOASSERTION. I would expect there to be some data loss here.


can't start the game

I'm sure I paste the correct api-key but I can't start the game. It's always loading.

Please make sure either you choose "client mode" with a network that can reach openai, or you choose server mode (can also get blocked by openai).


Update the tools listed within the Imaging and Recovery Guides landing page

Update/add the following text from Working in the BitCurator Environment, QS Guide
"The Imaging and Recovery submenu lists tools to assist in device imaging and recovery. These include:

Brasero: A GNOME application to burn CDs and DVDs, and create 1:1 copies of CDs and DVDs.
cdrdao: An application to record audio or data CD-Rs in disk-at-once (DAO) mode based on a textual description of the CD contents (toc-file).
Clonezilla: A partition and disk imaging/cloning program.
dcfldd: An enhanced version of GNU dd with features useful for forensics and security, including hashing on-the-fly and split outputs.
dd: dd copies input to output with a changeable I/O block size, while optionally performing conversions on the data. Can be used to create raw images of devices.
ddrescue: A data recovery tool with advanced features to assist in rescue of good data from devices with read errors.
dumpfloppy: A tool to read floppy disks in arbitrary formats supported by the PC floppy controller, and work with the resulting image files.
ewfacquire: A utility to acquire media data from a source and store it in EWF format (Expert Witness Compression Format).
Guymager: A GUI tool to create raw and EWF-packaged images from devices."

to Imaging and Recovery Guides

Thanks @bparnin !


Added support for logging

Added support for logging and replaced all print statements with log statements. I also tested the distributed method and now the results generated by the distributed_align and piecewise_align are identical so the code is ready to be unified

This is very useful @GFleishman maybe you can have a look, thanks./


Rbac CRDs not deleted after un-install

$ k get crd --all-namespaces
NAME                                    AGE
rbacconfigs.config.istio.io             2d

This was with release of July 15. I will try with image from 17.


Querying for files to watch... )) was unexpected at this time. Bazel query failed: exit status 255

Reproduction:
npx @angular/cli@next new bazel-angular
cd bazel-angular
ng add @angular/bazel@next
npm i @bazel/ibazel@latest --save-dev // ensure using the updated ibazel version since it's outdate in the schematics
ng serve

ng serve
iBazel [9:00AM]: Querying for files to watch...
)) was unexpected at this time.
iBazel [9:00AM]: Bazel query failed: exit status 255
Loading: 0 packages loaded
Loading: 109 packages loaded
    currently loading: @nodejs_windows_amd64// ... (3 packages)
Loading: 218 packages loaded

Loading: 218 packages loaded
iBazel [9:00AM]: Running //src:devserver
Loading: 0 packages loaded
Loading: 0 packages loaded

Loading: 0 packages loaded
iBazel [9:00AM]: Launching with notifications
listening on :35729

Versions:
OS: Windows
@bazel/bazel: 2.0.0,
@bazel/ibazel: 0.11.1

Workaround:
npm i -g @bazel/bazel

To me this looks like an issue with the angular cli. If there is something specific iBazel needs to do, please reopen.

I don’t think this is a cli issue. The cli command just spawns the ibazel with a target.

That said, I’ll take another look at this in the coming days.

To me this looks like an issue with the angular cli. If there is something specific iBazel needs to do, please reopen.

@achew22  if bazel knows the filename it would be helpful to say where

This is a nightmare! I finally figured out I can add a .bazelrc file with the following:
startup --output_user_root=bazel_stuff,--verbose_failures

This contained the very revealing error that:

WARNING: Download from https://mirror.bazel.build/nodejs.org/dist/v12.14.1/node-v12.14.1-win-x64.zip failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException GET returned 404 Not Found

Sure enough - that file does not actually exist.
So I updated node myself and now I get something like this:
npm ERR<EMAIL_ADDRESS>postinstall: `ngcc --properties es2015 browser module main`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.


Cleric giving random enchantments on 1.18.1

https://github.com/itsdinkd/AQM2/issues/32
As reported by a player of my modpack, the Cleric is not running as intended

Running this modpack from the same creator. Can confirm that we are having the same issue.


Possibility of TPU runtime

How likely do you think that it is possible to support TPU? If it is not possible, what are the reasons (need for more research, lack of man-power)?
I want to investigate the problem in my spare time since it is an interesting problem.

It would be possible, but we don't have enough man-power to lead a project in that direction. If you are interested, you could create a MLIR Runtime: https://mlir.llvm.org/ using the TPU dialect.


A way to choose a different youtube account under same email address

I have 3 Youtube Accounts under the same email address.
Is there a way to choose which youtube account I am uploading to, or is it not possible?

I would expect that when you first authenticate through your web browser it'll ask you which account, but I can't remember tbh. If an improvement to the code would help with this, feel free to submit a PR!
I don't have any personal motivation to work on this project but will merge PRs/release if others want to keep developing it :)

This should now be solved in version 0.5.1 with the new "Reset YouTube Auth" button (which just deletes the auth token created when you log in).
So, let's say you need to bulk upload some videos to one YouTube channel and then some other videos to a different channel, you can just click the button to reset your auth after the first set of videos are uploaded, and when you click Run again it'll open your browser to repeat the YouTube authentication flow, during which you can select a different channel.


Changing enum array type loses mapped enum names

Version and Platform (required):

Binary Ninja Version: 3.6.4709-dev Personal, 715fcd79
OS: windows
OS Version: 11
CPU Architecture: x86_64

Bug Description:
Initial analysis:

After fixing the array size, the initial enum name is lost:

Expected output: (I had to map each enum val to name manually)

Also, if it's possible to default select the last used enum in the Select Enum dialogue box it will be useful in similar scenarios.

Related to: https://github.com/Vector35/binaryninja-api/issues/4431

I have created a simpler binary to reproduce this:
test_enum_arrary.zip

Looking at the expression type, it seems that we correctly deduce the type of arr[0] is the enum, we just fail to propagate that to the right side of the comparison

If I look at the MLIL, if explains:
>>> current_il_instruction
<MediumLevelILSetVarField: arr[0] = 1>
>>> current_il_instruction.dest
<var enum foobar arr[0x3]>
>>> current_il_instruction.dest.type
<type: immutable:ArrayTypeClass 'enum foobar[0x3]'>

So this issue a bit trickier than I have expected. While at HLIL we correctly deduce the type of arr[1] is an enum and we should be able to just do the type propagation easily, we are actually only doing the type propagation at MLIL. And unfortunately, the MLIL instruction is a MediumLevelILSetVarField, which means the operand arr[1] does not even exists:

>>> current_il_instruction
<MediumLevelILSetVarField: var_34.b = 2>
>>> current_il_instruction.operands
[<var struct enum_struct var_34>, 4, <MediumLevelILConst: 2>]


Suggestion: when bot is generating answer, would be nice to be able to scroll in the live generated answer

In the original Chatgpt chat ui, when the bot is writing the answer, you can scroll every line that has been generated. On chatbot-ui it looks like scrolling is blocked until the full answer.

You can close this issue it is already fixed here #174

Fixed in #174


terraform-aws-multi-az-subnets

what

terraform-aws-multi-az-subnets implementation

why

We need a module to provision private or public subnets in each Availability Zone, and then be able to find the subnet IDs by the AZ names

test
Given the following configuration
locals {
  public_cidr_block  = "${cidrsubnet(var.vpc_cidr, 1, 0)}"
  private_cidr_block = "${cidrsubnet(var.vpc_cidr, 1, 1)}"
}

module "public_subnets" {
  source              = "git::https://github.com/cloudposse/terraform-aws-multi-az-subnets.git?ref=master"
  namespace           = "${var.namespace}"
  stage               = "${var.stage}"
  name                = "${var.name}"
  availability_zones  = ["us-east-1a", "us-east-1b", "us-east-1c"]
  vpc_id              = "${module.vpc.vpc_id}"
  cidr_block          = "${local.public_cidr_block}"
  type                = "public"
  igw_id              = "${module.vpc.igw_id}"
  nat_gateway_enabled = "true"
}

module "private_subnets" {
  source              = "git::https://github.com/cloudposse/terraform-aws-multi-az-subnets.git?ref=master"
  namespace           = "${var.namespace}"
  stage               = "${var.stage}"
  name                = "${var.name}"
  vpc_id              = "${module.vpc.vpc_id}"
  cidr_block          = "${local.private_cidr_block}"
  type                = "private"
  az_ngw_ids          = "${module.public_subnets.az_ngw_ids}"
  nat_gateway_enabled = "true"
}

output "private_az_subnet_ids" {
  value = "${module.private_subnets.az_subnet_ids}"
}

output "public_az_subnet_ids" {
  value = "${module.public_subnets.az_subnet_ids}"
}

the output Maps of AZ names to subnet IDs look like these
public_az_subnet_ids = {
  us-east-1a = subnet-ea58d78e
  us-east-1b = subnet-556ee131
  us-east-1c = subnet-6f54db0b
}
private_az_subnet_ids = {
  us-east-1a = subnet-376de253
  us-east-1b = subnet-9e53dcfa
  us-east-1c = subnet-a86fe0cc
}

and the created subnet IDs could be found by the AZ names using map["key"] or lookup(map, key, [default]),
for example:
public_az_subnet_ids["us-east-1a"]
lookup(private_az_subnet_ids, "us-east-1b")

How can we use this module, if we want to create several subnets in each AZ?
How NGW will be selected in the following cases?

3 public subnets, 2 private subnets
2 public subnets, 3 private subnets

I think, https://github.com/cloudposse/terraform-aws-named-subnets resolves all subnets distribution tasks:

It can be instantiated per-AZ if we need Multi-AZ
It provides a single NGW per AZ and we are free to select which NGW we want to use
It provides capability of running several subnets in a single AZ

On the other hand, https://github.com/cloudposse/terraform-aws-dynamic-subnets provides out-of-the-box solution for creating subnets topology based on amount of AZs you want to utilize. Only option to set  amount of subnets per AZ may be required.
IMHO  https://github.com/cloudposse/terraform-aws-named-subnets provides more flexibility than https://github.com/cloudposse/terraform-aws-multi-az-subnets and https://github.com/cloudposse/terraform-aws-dynamic-subnets together.
https://github.com/cloudposse/terraform-aws-dynamic-subnets calculates subnet CIDRs automatically and saves time.
I see no benefits from https://github.com/cloudposse/terraform-aws-multi-az-subnets
@aknysh @osterman @SweetOps any thought on this?

use this and you'll get multi-named-multi-az :)
locals {
subnets_count = "${length(compact(var.subnets_names) * length(compact(var.availability_zones))}"
}

resource "aws_subnet" "private" {
count             = "${local.subnets_count}"
}

@comeanother , mb that module need in some project... IMO we should think about implementation  functional of named-subnets in dynamic-subnets.

@SweetOps It appears to me that now dynamic-subnets can be entierly replaced with named-subnets.
But for now only adding routes to particular subnets may be an issue.

@comeanother  you reviewed an outdated PR, I was testing the code in my test solution and did not update this PR. Now it has the latest code and it "does work", with the exception of one thing related to multiple module invocation (see Note below).
I agree with you that terraform-aws-named-subnets could be used in many cases and it's more flexible and many use-cases could be covered by it. We have too many subnet modules now (three), maybe it's a good idea to write down all use-cases and combine them into one module (if possible).
NOTE: current issue with terraform-aws-multi-az-subnets invocation.
Terraform does not want to accept an input map for this module as an output from another module.
For example, this code
locals {
  public_cidr_block  = "${cidrsubnet(var.vpc_cidr, 1, 0)}"
  private_cidr_block = "${cidrsubnet(var.vpc_cidr, 1, 1)}"
}

module "public_subnets" {
  source              = "git::https://github.com/cloudposse/terraform-aws-multi-az-subnets.git?ref=init"
  namespace           = "${var.namespace}"
  stage               = "${var.stage}"
  name                = "${var.name}"
  availability_zones  = ["us-east-1a", "us-east-1b", "us-east-1c"]
  vpc_id              = "${var.vpc_id}"
  cidr_block          = "${local.public_cidr_block}"
  type                = "public"
  igw_id              = "${var.igw_id}"
  nat_gateway_enabled = "true"
}

module "private_subnets" {
  source             = "git::https://github.com/cloudposse/terraform-aws-multi-az-subnets.git?ref=init"
  namespace          = "${var.namespace}"
  stage              = "${var.stage}"
  name               = "${var.name}"
  availability_zones = ["us-east-1a", "us-east-1b", "us-east-1c"]
  vpc_id             = "${var.vpc_id}"
  cidr_block         = "${local.private_cidr_block}"
  type               = "private"

  # Map of AZ names to NAT Gateway IDs was created in "public_subnets" module. Assign it to `az_ngw_ids` input
  az_ngw_ids         = "${module.public_subnets.az_ngw_ids}"
}

gives an error

module.private_subnets.aws_route.default: aws_route.default: value of 'count' cannot be computed

because of this line
az_ngw_ids         = "${module.public_subnets.az_ngw_ids}"

looks like TF can't calculate dynamic count using a Map variable from another module .
(I have not found a solution yet).
On the other hand, this code works OK:
locals {
  public_cidr_block  = "${cidrsubnet(var.vpc_cidr, 1, 0)}"
  private_cidr_block = "${cidrsubnet(var.vpc_cidr, 1, 1)}"
}

module "public_subnets" {
  source              = "git::https://github.com/cloudposse/terraform-aws-multi-az-subnets.git?ref=init"
  namespace           = "${var.namespace}"
  stage               = "${var.stage}"
  name                = "${var.name}"
  availability_zones  = ["us-east-1a", "us-east-1b", "us-east-1c"]
  vpc_id              = "${var.vpc_id}"
  cidr_block          = "${local.public_cidr_block}"
  type                = "public"
  igw_id              = "${var.igw_id}"
  nat_gateway_enabled = "true"
}

module "private_subnets" {
  source             = "git::https://github.com/cloudposse/terraform-aws-multi-az-subnets.git?ref=init"
   namespace          = "${var.namespace}"
  stage              = "${var.stage}"
  name               = "${var.name}"
  availability_zones = ["us-east-1a", "us-east-1b", "us-east-1c"]
  vpc_id             = "${var.vpc_id}"
  cidr_block         = "${local.private_cidr_block}"
  type               = "private"

  # Map of AZ names to NAT Gateway IDs was created in "public_subnets" module. Assign it to `az_ngw_ids` input
  az_ngw_ids = {
    us-east-1a = "ngw-3213232"
    us-east-1b = "ngw-4234343"
    us-east-1c = "ngw-8676764"
  }
}

but it's not very useful because we need to know the NAT Gateways IDs since we are not getting them from the public subnets.
@osterman @comeanother @SweetOps
What are your thoughts on all the above?
Should we try to find a fix for terraform-aws-multi-az-subnets invocation?
Should we just use terraform-aws-named-subnets and maybe improve it to cover more use-cases?
Should we improve terraform-aws-dynamic-subnets to cover more use-cases?

@aknysh I think, we should use terraform-aws-named-subnets as a basis for further subnet modules.
And extend its capabilities.
I think, that terraform-aws-dynamic-subnets and terraform-aws-multi-az-subnets have too narrow range of use cases. As we're limited by one subnet per AZ.
If we add availability_zones list capability to terraform-aws-named-subnets, we will get one general module for subnets.
Imagine, if we're using terraform-aws-dynamic-subnets or terraform-aws-multi-az-subnets and we would need to add one (or a buch) more subnet to a specific AZ.
We would need to select another CIDR for that and perform additional calculation, when the first CIDR is not fully utilized.

I think the calculus of supporting named subnets + multi-az + nat gateways is way to complicated. The mulit-az subnet module is ideal for embedding in modules. For example, rds/redis/beanstalk/etc can define their own subnets with this.
dynamic-subnets optmizes for a simple flat network with public and private subnets.
named-subnets optimizes for lots of subnets in a single AZ (less-common use-case).
multi-az-subnets optimizes for a single subnet in many AZs (very common use-case).


get rid of the dummy crates

https://github.com/rust-lang/cargo/pull/4953 suggests that it should be possible to support importing different versions of the same crate.

I would recommend against this. Renaming dependencies in the manifest is the "correct" way to import multiple versions of a crate, but it does not work on versions of Rust before 1.31.0 (2018-12-06).
Using a renamed dependency in your library's Cargo.toml will force any user to upgrade to 1.31+ (or they choose to not use that crate, instead).
To me, the point of this legacy crate is to allow older versions of Rust to use multiple versions of serde. This crate could encourage people that already rely on newer Rust versions to use renaming instead of this crate, however.


Question: How to concat keywords?

 I searched the web site, but as I don't speak French, it's a little hard to understand. ;-)
I'd like to use a command like "Licht an" or "Licht aus" to put on/put off my lights.
I added this command:
*LICHT (*)== /home/pi/licht.sh (1)
licht.sh takes "AN" or "AUS" as an argument.
As keywords I registered "licht", "an", and "aus" with snowboy.
Now after saying the magic word ("computer") and saying "Licht an" snowboy recognizes both keywords "licht" (no. 7) and "an" (no. 2) which is quite fine. Unfortunately after that only the second keyword "an" is interpreted as the command. This of course fails, as there is no "AN" statement in the command file. I would expect that both keywords are concatenated to one command "LICHT AN" which would than be matched in the command file. What am I missing or doing wrong?
Output of "Troubleshooting mode":
Logs
pi: (listening...)
DEBUG: models=computer
INFO:snowboy:Ticks: [2, 20, 5, -1]
INFO:snowboy:Keyword 1 detected at time: 2019-03-03 14:26:33
INFO:snowboy:Ticks status: 2 2 1 1
DEBUG: modelid=0
computer
DEBUG: entering_cmd hook
pi: (listening...)
DEBUG: models=alexa,an,aus,computer,datum,gutenacht,licht,snowboy,temperatur,test,zeit
INFO:snowboy:Ticks: [2, 20, 5, -1]
INFO:snowboy:Keyword 7 detected at time: 2019-03-03 14:26:38
INFO:snowboy:Keyword 2 detected at time: 2019-03-03 14:26:39
INFO:snowboy:Ticks status: 2 2 0 1
DEBUG: modelid=1
an
DEBUG: start_speaking hook "I did not understand: an"
computer: I did not understand: an
DEBUG: stop_speaking hook
DEBUG: exiting_cmd hook

Unfortunately I have the same issue.
Any solutions?


Package name

Hi all,
we discussed the fact that drf-forms was a temporary name and that we should find a better name, here are a few proposals:

drf_admin_toolkit
drf_fe_toolkit
drf_fe_adapters

What about drf_schema_generator? drf_schema_adapters?

I like drf_shema_adapters

Unless somebody opposes to this name in the next 48h, I'll publish the package under that name on pypi.

drf_schema_adapters is fine for me


bot requesting feedback more than once per day

unable to replicate.  think it was a timezone issue?


[backport] Fix split and array_split with indices overrun

Backport of #2814

[automatic post] Jenkins, test this please.

Just tested the backport script with --branch option.

Jenkins CI test (for commit c4d636d01a934269c927524bc485148d1effe975, target branch v7) succeeded!


MB-7736 failed gex connection should not mark payment reqeust status as edi error

Description
Currently when the job runner processes a Payment Request and fails to send it to GEX due to connection issue the status of the request is changed. This should not be the case as the only option for us is to retry later. This PR fixes that so the status stays REVIEWED so that it will be retried later.
Reviewer Notes
I moved some tests from the payment request reviewed processor into a file for send to syncada helper, since that's what was  really being tested.
Setup
Make sure GEX_URL is not set in your environment
change line 601 of cmd/milmove/serve.go to return a non 200 status
			w.WriteHeader(http.StatusForbidden)

setup the server
make db_dev_e2e_populate
make server_run
make office_client_run

Mark existing payment requests as sent
update payment_requests SET status = 'SENT_TO_GEX';

Create a payment request
rm -f bin/prime-api-client
make bin/prime-api-client
prime-api-demo RDY4PY

Approve the payment request
Run the job runner and see that the status does not change
go run ./cmd/milmove-tasks process-edis --send-to-syncada

select * from payment_requests where id = '<id of pr>';

Code Review Verification Steps

[X] Code follows the guidelines for Logging
[X] Request review from a member of a different team.
[X] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Messages

:book:
:link: MB-7736

  Generated by :no_entry_sign: dangerJS against 101f92bc07ca4f5cd798b73d65a8557e682f9156


What GPU was used for the benchmarks?

Hi, super awesome work on nboost!

I was just curious as to what GPU was used for the benchmark on the readme? Thanks! :)

Sorry, I just found and read https://github.com/koursaros-ai/nboost/issues/68#issuecomment-635514944
Please feel free to close :)


Import/Export Testing

This PR contains simple testing functionality for import/export code. I currently cannot import exported JSON dumps and am trying to find the cause, which has led me to write unit tests.
Since those tests test the CLI, i added a fourth subproject cottontaildb-test which contains test-utils. This is the preferred way to share testing code among modules in Kotlin/Gradle.
@ppanopticon could you have a look at why the ExportImportCommandTest.exportImport test is failing? I am unable to truncate an entity using the exact same code as in the CLI. This means i am unable to test the import functionality currently.

I have fixed several issues on master. Can you please try to merge and run your tests again?
I'm not quite sure what you mean by

I am unable to truncate an entity using the exact same code as in the CLI. This means i am unable to test the import functionality currently.

I just added the CLI functionality to truncate an entity, i.e., I'm not quite sure what "code" you're referring to. However, in my tests, truncating and entity worked fine.

Thanks for the fixes!
I was using the Clear command, since the description sounded like it would truncate the entity. In that case, it's unclear to me what the difference between clear and truncate is.
The tests now successfully run, but the performance is not good at all, which is why i've opened #99

Thanks for the fixes! I was using the Clear command, since the description sounded like it would truncate the entity. In that case, it's unclear to me what the difference between clear and truncate is.
The tests now successfully run, but the performance is not good at all, which is why i've opened #99

Good point: entity clear should indeed have the same effect but stems from a time where there was no TRUNCATE primitive. I'll remove the command but if it doesn't work, there must still be some bug.

@ppanopticon - as an update: you can find a discussion on the issue here, with the accepted answer relying on a java plugin which does not work well when i tested it, and the answer after that suggesting the solution of this PR

@ppanopticon - as an update: you can find a discussion on the issue here, with the accepted answer relying on a java plugin which does not work well when i tested it, and the answer after that suggesting the solution of this PR

I resolved the issue with the dedicated cottontaildb-test module and replaced it by the approach described under the link you posted. Please review. If you're happy with it and if all tests succeed we can merge this.
Sorry for the delay... ^^


Change browser title

What
This PR changes the formatting of the browser title from "Page data: <content_title>" to "<content_title>: Page data"
Why
To allow user to differentiate tabs based on which content item they represent.
Screenshots

Review Checklist

[ ] Changes in scope.
[ ] Added/updated unit tests.
[ ] Added/updated feature tests.
[ ] Added/updated relevant documentation.
[ ] Added to Trello card.

 (2) Make page title appear first in browser tab titles


Github account keeps signing out every time I open Code.

Everytime I open code, my Github account is signed out and I have to sign in again and again. Once I've signed in and I close Code, to then open it again, my account is signed out and I have to sign in again.
Tried extension bisect and nothing helps.
Tried disabling all extensions and not fixed.
Its been like this for a couple of months, it seems fixed on the Insider build but just doesn't work on the release build.

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

VS Code Version: 1.69.2 (Universal)
OS Version: MacOS 12 and 13

Steps to Reproduce:

Open VSCode
VSCode says "You are not signed into any accounts" and "Sign in to sync settings (1)"
I sign into my github account successfully
Then close VS Code
and open again for it to tell me I'm not signed in again.

On MacOS 13 this is #152964

/duplicate #152964
This fix will go out this week and then feel free to give it a try. If you still have issues after that, don't hesitate to reach out!

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


CodeSmell: high complexity in orderRawData-Method (OrderByCondition Trait)

Line 24 - 50
Problems:

Refactor this function to reduce its Cognitive Complexity from 23 to the 15 allowed!
This function has 5 returns, which is more than the 3 allowed.

Won't fix!


React Version Conflict

Your component is the perfect match with what I needed. But it doesn't support react version less than 17. I'm working on a project of react 16.3.1 version. Can you please tell me a way to make it work on this version?

You may have some luck updating your project to React 16.4.0 (if that's a possibility). The thing that is most likely causing issues is the new react-jsx JSX transform, and it was back-ported to React 16.4.

Is there something I can do to make it backwards compatible?


Failed to compile with BUILD_CUTTER_PLUGIN=ON on Linux: fatal error: rz_core.h: No such file or directory

When built for Cutter, the build fails, without the BUILD_CUTTER_PLUGIN=ON build flag, it works.
this was tested on archlinux and all the dependencies are installed. also encountered this issue on trisquel(ubuntu).
Requirements, versions:
gcc (GCC) 10.2.0
cmake version 3.19.2
radare2 4.5.1
Installation steps:
git clone https://github.com/avast/retdec-r2plugin
cd retdec-r2plugin
mkdir build
cd build/
cmake .. -DCMAKE_INSTALL_PREFIX=~/.local -DBUILD_CUTTER_PLUGIN=ON
make -j 13

Cmake output:
$ cmake .. -DCMAKE_INSTALL_PREFIX=~/.local -DBUILD_CUTTER_PLUGIN=ON
-- The C compiler identification is GNU 10.2.0
-- The CXX compiler identification is GNU 10.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Setting build type to 'Release' as none was specified.
-- Found PkgConfig: /usr/bin/pkg-config (found version "1.7.3")
-- Checking for one of the modules 'r_core'
-- Found Radare2: Radare2::libr
-- Found PythonInterp: /usr/bin/python3 (found suitable version "3.9.1", minimum required is "3.4")
-- Performing Test SUPPORTS_FVISIBILITY_INLINES_HIDDEN_FLAG
-- Performing Test SUPPORTS_FVISIBILITY_INLINES_HIDDEN_FLAG - Success
-- Capstone: using remote Capstone revision.
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found ZLIB: /usr/lib/libz.so (found version "1.2.11")
-- LLVM: using remote LLVM revision.
-- YARA: using remote YARA revision.
-- YaraMod: using remote YaraMod revision.
-- Found OpenSSL: /usr/lib/libcrypto.so (found suitable version "1.1.1i", minimum required is "1.0.1")
-- -- Linking with /usr/lib/libstdc++fs.a library
-- Found Cutter: /home/XXX/XXX/retdec-r2plugin/build/_deps/cutter-src/src
-- Linking with /usr/lib/libstdc++fs.a library
-- Configuring done
-- Generating done
-- Build files have been written to: /home/XXX/XXX/retdec-r2plugin/build

Make build output(error):
$make -j 13
[  2%] Built target utils
[  3%] Built target yara
[  3%] Built target pdbparser
[  4%] Built target yaramod-project
[  6%] Built target pelib
[  7%] Built target capstone-project
[  7%] Built target tinyxml2
[  8%] Built target llvm-project
[ 10%] Built target common
[ 13%] Built target ctypes
[ 13%] Built target idr2pat
[ 13%] Built target ar-extractor
[ 14%] Built target yaracpp
[ 14%] Built target macho-extractor
[ 15%] Built target ctypesparser
[ 17%] Built target serdes
[ 17%] Built target ar-extractortool
[ 18%] Built target capstone2llvmir
[ 18%] Built target macho-extractortool
[ 22%] Built target demangler
[ 23%] Built target config
[ 31%] Built target fileformat
[ 31%] Built target getsig
[ 32%] Built target patterngen
[ 34%] Built target loader
[ 35%] Built target cpdetect
[ 35%] Built target pat2yara
[ 35%] Built target bin2pat
[ 36%] Built target unpacker
[ 36%] Built target stacofin
[ 37%] Built target rtti-finder
[ 37%] Built target debugformat
[ 37%] Built target stacofintool
[ 39%] Built target unpackertool
[ 72%] Built target llvmir2hll
[ 73%] Built target main
[ 85%] Built target fileinfo
[ 98%] Built target bin2llvmir
[ 98%] Built target retdec
[ 98%] Automatic MOC and UIC for target r2retdec_cutter
[ 98%] Built target retdec-decompiler
[ 99%] Built target core_retdec
[ 99%] Built target r2retdec_cutter_autogen
[ 99%] Building CXX object src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/r2retdec_cutter_autogen/mocs_compilation.cpp.o
[ 99%] Building CXX object src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/core_plugin.cpp.o
In file included from /home/user/XXX/retdec-r2plugin/build/_deps/cutter-src/src/widgets/CutterDockWidget.h:4,
                 from /home/XXX/XXX/retdec-r2plugin/build/_deps/cutter-src/src/plugins/CutterPlugin.h:6,
                 from /home/XXX/XXX/retdec-r2plugin/build/src/cutter-plugin/r2retdec_cutter_autogen/4NLJWMO7SH/../../../../../include/cutter-plugin/core_plugin.h:12,
                 from /home/XXX/XXX/retdec-r2plugin/build/src/cutter-plugin/r2retdec_cutter_autogen/4NLJWMO7SH/moc_core_plugin.cpp:10,
                 from /home/XXX/XXX/retdec-r2plugin/build/src/cutter-plugin/r2retdec_cutter_autogen/mocs_compilation.cpp:2:
/home/user/Apps/retdec-r2plugin/build/_deps/cutter-src/src/core/CutterCommon.h:8:10: fatal error: rz_core.h: No such file or directory
    8 | #include "rz_core.h"
      |          ^~~~~~~~~~~
compilation terminated.
make[2]: *** [src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/build.make:82: src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/r2retdec_cutter_autogen/mocs_compilation.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from /home/XXX/XXX/retdec-r2plugin/build/_deps/cutter-src/src/widgets/CutterDockWidget.h:4,
                 from /home/XXX/XXX/retdec-r2plugin/build/_deps/cutter-src/src/plugins/CutterPlugin.h:6,
                 from /home/XXX/XXX/retdec-r2plugin/include/cutter-plugin/core_plugin.h:12,
                 from /home/XXX/XXX/retdec-r2plugin/src/cutter-plugin/core_plugin.cpp:9:
/home/user/Apps/retdec-r2plugin/build/_deps/cutter-src/src/core/CutterCommon.h:8:10: fatal error: rz_core.h: No such file or directory
    8 | #include "rz_core.h"
      |          ^~~~~~~~~~~
compilation terminated.
make[2]: *** [src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/build.make:95: src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/core_plugin.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:2542: src/cutter-plugin/CMakeFiles/r2retdec_cutter.dir/all] Error 2
make: *** [Makefile:149: all] Error 2

Afterwards rizin-git was installed, and it still did not work, same error.
Package: https://aur.archlinux.org/packages/rizin-git/
Rizin version, on second attempt:
rizin 0.1.0-git 25276
commit: 40a29cba29ebb58a15ca372a765be56e5d19d5ac

Same outcome for me. Rizin-git 0.2.0. You ever get this working @FreedomFrappa?

When compiling under iaito i get this error.
Of course i changed the flags to reflect the iaito plugin build.
/retdec-r2plugin/include/r2plugin/r2retdec.h:10:10: fatal error: r_util/r_annotated_code.h: No such file or directory

r2retdec is outdated, unusable now as r2 plugin too

r2retdec is outdated, unusable now as r2 plugin too

Might have something to do with this:
https://github.com/radareorg/r2retdec/pull/40
But my issue may be seperate i am not sure.
if r_util/r_annotated_code.h is new, then its likely due to being outdated.

The last time i tried to get r2retdec to work with r2 retdec was abandoned, and it was requiring some work in their side to make it work again because of incompatibilities with the version of capstone (cs4 vs cs5) and their linking was done wrong.
iaito doesnt have plugins, because it just uses r2 plugins, the code involved is useless and will be removed.
Their authors stopped responding me a year ago, but seems like there are new commits these days and i assume they fixed. so maybe if im bored i'll pick it up, but i'm already maintaining over 9000 projects alone and it would be great if someone could help on maintaining or contributing to get it working again.


Errors when running train_mnist.py on CPU and GPU with python 2.7 and 3.6

Hi, I am unable to run train_mnist after following the related tutorial. In fact, after running the following command:
python gop_tutorial1.py -m hemlgop -c cpu
I get the following errors/outputs:
Using TensorFlow backend.
Start progressive learning
-------------Layer 0 ---------------Block 0 ------------------
Iterative Randomized Search
Using TensorFlow backend.
Traceback (most recent call last):
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/GOP/utility/gop_search.py", line 93, in
main(sys.argv[1:])
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/GOP/utility/gop_search.py", line 42, in main
params = dill.load(f)
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/dill/_dill.py", line 278, in load
return Unpickler(file, ignore=ignore, **kwds).load()
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/dill/_dill.py", line 481, in load
obj = StockUnpickler.load(self)
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/pickle.py", line 864, in load
dispatchkey
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/pickle.py", line 892, in load_proto
raise ValueError, "unsupported pickle protocol: %d" % proto
ValueError: unsupported pickle protocol: 3
Traceback (most recent call last):
File "train_mnist.py", line 171, in
main(sys.argv[1:])
File "train_mnist.py", line 157, in main
verbose=True)
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/GOP/models/hemlgop.py", line 152, in fit
verbose)
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/GOP/models/hemlgop.py", line 232, in progressive_learn
test_data)
File "/home/retina/dembysj/.conda/envs/op-ann-2/lib/python2.7/site-packages/GOP/utility/gop_utils.py", line 639, in search_cpu
raise RuntimeError('Search process using CPU terminated but no result exists!')
RuntimeError: Search process using CPU terminated but no result exists!
I get the similar errors using CPU or GPU with Python 2.7 and 3.6. Please, do you have any guidance?
I am using Ubuntu 16.04, TensorFlow 1.4.1 and Keras 2.2.4

Hi,
Same problem with following versions:

Ubuntu 20.04
python 2.7 / 3.7 (tested on both)
tensorflow 1.4.1 /  Keras 2.2.4
Execution on CPU

hi,
If you look into the error message, you will see the line unsupported pickle protocol. This suggests that the code has been run on python3 and the intermediate result was created (using pickle from python 3) and when you run the code again, you run with python 2, without clearing the temporary directory.
I would suggest that if you run into trouble, try clearing the tmp_dir before running again the code. Please let me know if this fixes the issues

@viebboy I had the same error running the code in python 3.6 ou 3.7
Should I run in python 2?

@gilsoneng you should run the code using at least python 3.7

Yes but I get the same error running on python 3.7


Broken link

http://aka.ms/hrcsync - link is broken

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

ID: a610f434-e238-d32a-9589-e2b9cbe184c3
Version Independent ID: e9a881b7-8fa2-1574-391d-7be1a6e74443
Content: How to troubleshoot password synchronization when using an Azure AD sync appliance - Active Directory
Content Source: support/azure/active-directory/troubleshoot-pwd-sync.md
Service: active-directory
GitHub Login: @genlin
Microsoft Alias: genli

@CMC-Spilman  Thanks for the feedback. I will removed the link. The troubleshoot tool is integrated with Azure AD Connect. For more information, see
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/tshoot-connect-objectsync


Typeahead feature

Want something like in queries, but for icons

https://github.com/grafana/grafana/blob/83e0f69cdeff9122bd0d970414eceb8ae6dd9b65/public/app/core/components/query_part/query_part_editor.ts


Remove 2 Outline button section

Issue:

Thank you for your work but someone else is assigned to this issue.


Elastic 7.0+ changes (+ Grafana)

With this updated template, logstash and elasticsearch seem to work nicely. This should fix #50, but there's something going on.
The reason this is merely a draft PR is because I am not entirely sure if this is fully correct or if something is missing. That's because Grafana shows this:

category and accounting_id don't have any values. However! If I manually enter values, it appears to at least partially show up in Grafana (until the auto-refresh kicks in and clears all data again).

Is there perhaps something I'm missing?

@Lax Do you have any idea if my changes to the elasticsearch template for logstash are complete or if whether there's changes needed for newer versions of Grafana to work witht the provided dashboard file?

:) works as intended now


Sniff to discourage use of number_format() and date() in output

WP has the number_format_i18n() and the date_i18n() functions which allow for localization of numbers and dates.
While the "normal" number_format(), date_format(), date(), gmdate(), gmstrftime() and strftime() all have their place in code, I believe that when used in combination with an output generating function, the WP functions should be encouraged (and the PHP ones discouraged).
Opinions ?
On that note, there is also a PHP money_format() function. I'm not aware of a WP equivalent. Anyone know of one ?

I think it is a good idea . What we would also need to check that the date format is not hardcoded. This would be the correct way: date_i18n( get_option( 'date_format' ) )
I don't think WordPress has a currency function as it is only needed for ecommerce sites. WooCommerce has one though wc_price() and EDD has edd_format_amount()


Only paid users can use this?

Applying the Thinking Protocol

Copy the latest version in model_instructions folder
Start a new Project in Claude.ai
Paste the instructions to the Custom Instructions section
Claude will now follow the thinking protocol for all subsequent interactions

I can not find "Start a new Project" on claude.ai. Is this only for paid users?

yes, it;s a paid feature


Help text for Form components always shows (like Parcel Ids on submission intake)

Description
Help text on form components like InputText now always displays
so that the user can see the help text without needing to clicking into field, saving confusion
PADS-245
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Approved. Please push an update to remove the unused imports if you can. Always a good idea to run npm run lint before commits.

done


feat: support OAuth2 client credentials flow

Problem
Currently, the client credentials flow for oauth is not supported.
Explanation
This happens because support has not yet been built out.
Solution
With this PR the client credentials flow is supported.
Sample in UI

Sample in UI after successful auth

hey @mpminardi do you know of an example spec/server so we can test this? How were you testing?

hey @mpminardi do you know of an example spec/server so we can test this? How were you testing?

Hey @amritk! Had been testing this with tailscale OAuth clients which use the client credentials flow.
Have a sandbox link here which should have a minimal reproducer for being able to get a token / hit an endpoint with it.
Are authorization code / client credentials supported now as of https://github.com/scalar/scalar/pull/2215?

Not in this client, but in the new one! Coming soon 😁

Alright @mpminardi its out now! Give it a try on the new client. Authcode and client credentials are available on the client only for now so you won't see them on the api reference side (yet)

Alright @mpminardi its out now! Give it a try on the new client. Authcode and client credentials are available on the client only for now so you won't see them on the api reference side (yet)

@amritk any idea when we might see client credentials on the api reference? 🙏
https://docs.scalar.com/ & "@scalar/api-reference": "^1.24.60", don't render those inputs as far as I can tell:
clientCredentials ❌

implicit ✅

password ✅


enhance: #28 long time to load bg image

Summary
Changed the jpeg bg image to webp file eliminating 5-6 seconds and crucial resources that was being taken to load a bg image.
Description
Hosted the converted jpeg to webp file on imgbb to save on the overall build site keeping in mind that we will never the chanage the bg image, the home page now loads much faster and looks better.
Images
OLD AND NEW GIFS
Issue(s) Addressed
Enter the issue number of the bug(s) that this PR fixes

Template should be strictly **Closes #28 **
Example: Closes #1

Review Checklist
Please make sure to go through this checklist before submitting your review.

[ x] Make sure the PR title is of format tag: Issue Title

tag should be one of fix,feat, docs, chore, refactor
Ex: feat: Added responsive design for the home page

[ x] Ensure the code follows our coding standards and guidelines

Ex:  naming conventions should be of form file-name.ts not fileName.ts

[ x] Ensure that you have staged only the required commits and raised the PR and not included all the files which would be otherwise required for local setup
[ x] Check if the PR is rebased to the latest main/development branch.
[ x] Make sure there are no unnecessary files or debugging artifacts in the PR.

Hey @krishnaacharyaa ,tried to create a new pr following the given pr template, and for the previous commits , im a little new at git for now, i have tried my best to do as you mentioned but if it is still not as it should be, please inform me for the same ill try to do more research about it and create  a new pr..... Thanks, any kind of feedback will be greatly appreciated

Thank you @jjinendra3, for your sincere efforts I really appreciate the time and efforts put into storing the gif into google drive and sharing the link, which was not expected, thank you for the efforts.
Just some points to share if you already didn't know,

You can minimize all the efforts by directly embedding gif in the md file like https://stackoverflow.com/questions/34341808/is-there-a-way-to-add-an-animated-gif-to-a-markdown-file this will greatly save your time and efforts for coming future.
And coming to the git part sure, I'll be more than happy to help you.

Some observations

Why does it say it closes 1st issue ?
And small spacing issues in the [ x] it should have been [x]

To answer the git part
I have tried to explain the present scenario to the best according to my understanding about git, please feel free to go through you might have to do some research on top of it, I have just tried to give you overall understanding, hope it helps.
If there is still more questions to ask, we can have a call.
Please go through and if you further find nice articles about git feel free to share, there's always more things to learn :)

Just one thing to keep in mind you can use this as reference dont take my word for it, sometimes some part even I might have missed, feel free to let me know if any. Happy to correct myself and learn :)
And you might have to close this PR and raise new PR for this as per my understanding, feel free to explore what is the best solution :) You know what I would do now :)

Hey @krishnaacharyaa ,
I wanted to take a moment to express my sincere thanks your efforts that have helped me grow and learn, not only about Git/GitHub but also about the MERN stack.
The guidance and knowledge you've shared is something I genuinely appreciate.
Now, I plan to create a new pull request and will try incorporate all the changes you've mentioned.
Thank you once again for your guidance and the opportunity to contribute to this project.

@jjinendra3 Thank you for your kind words 😇 Always happy to share and I am glad to hear that it is greatly helping you :)


How to create a custom mapping for classes

Hi,
I would like to use this package to handle the persistence of some entities in DynamoDB. In my case I have some properties in those entities that are not scalar values themselves and I wonder how can I create a mapper that has custom instructions when saving or restoring that property.
Example:
class Id {
   private value: string;

   private constructor(value: string) {
          this.value = value;
   }

    public static create(value: string) {
         return new Id(value);
    }

   public toString(): string {
     return this.value;
  }
}

class Foo {
      constructor(private readonly id: Id, private readonly name: string) {}
}

So I want to inform the data mapper that it should use the toString to get the representation of Id when persisting any instance of the Foo class, and at the same time when retrieving the a Foo instance it should take the string representation of the Id from Dynamo and call the Id::create.

I have the same problem now :)

@mariobittencourt I found solution, you can use 'Custom' type. I checked it with annotations
@hashKey({
    type: 'Custom',
    attributeType: 'S',
    marshall: (input) => {
      return { S: input.get() };
    },
    unmarshall: (input) => UserId.createFromSafe(input.S as string),
  })

If you are using it without annotations check it:
https://github.com/awslabs/dynamodb-data-mapper-js/tree/master/packages/dynamodb-data-marshaller#custom

Thank you @Mararok. I will try to use it and see if it can work with the use cases I am considering.


MAINT: Bump SurfaceTopography to 1.2.5

This fixes import of acquistion time for some file formats.
Closes #936

Already done in other commits. The current version 0.92.0 of topobank is using SurfaceTopography 1.2.5. Closing.


Quick Fix dialog covers line

VSCode Version: 1.52.1
OS Version: 17763.1637

Steps to Reproduce:

Have a line that is offered a Quick Fix by VS Code.  For me this happens with an unused variable for react (filetype .jsx).  In this case I'm looking at the "setReadinessValid" method:

Hover over line and the Quick Fix dialog covers the line.  In this case I would be likely trying to copy the method name to paste it where I need it to be called:

This is extremely frustrating because so often in my current workflow this is for an unused variable/reference and I go back to the line to copy it so I can paste the variable/reference but then I can't because the dialog keeps covering the variable so I can't select it to copy it.   If the dialog appeared just a lines height above (or below depending on where in the window the line is of course) the line with the issue it would be a lot easier for me to fix in this case the reference issue.

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

/duplicate of #112391


Swap vector for unordered_set

Change FileContent API param from vector to unordered_set to avoid
having to iterate to find (the only purpose the input arg are used)

Why do the logs have to be updated ? If this is due to the hashing generated a different order, then you might want to use std::set<> instead of unordered_set<>
In many security-hardened environments, the hash function is randomized with each start of a binary so this will result in unpredictable tests in case this is due to the order.

Why do the logs have to be updated ? If this is due to the hashing generated a different order, then you might want to use std::set<> instead of unordered_set<>
In many security-hardened environments, the hash function is randomized with each start of a binary so this will result in unpredictable tests in case this is due to the order.

Google test is reporting one additional error and the error itself seems very legitimate. Specifically, this particular error is reported with this error which isn't with current master.
From file generic/typedef/typedef_test_28__bad.sv.

I am wondering why the error is reported now and not before. Did you fix a bug with this change ?

As a general remark, I'd probably typedef/using using VObjectTypeSet = std::unordered_set<VObjectType>; somewhere and use VObjectSet  everywhere where you now manually use std::unordered_set<VObjectType>.
Not only is it shorter to read, but it also makes it easier to play with different implementations.


Not getting notification's title and body with getInitialPushPayload()

Hello @andrehtissot! Thanks a lot for creating and maintaining this plugin.
The issue that I am facing is, when my android application is killed and a notification arrives, after tapping it, the fcm.getInitialPushPayload()method  gives only the data payload that was sent along with the notification. For some reason it doesn't give me the notification's title and body. Here is the notification structure that I'm sending from my node backend using FCM SDK:
const notification = {
    tokens,
    notification: {
      title: category,
      body: message,
    },
    data: {
      device: 'my-device',
      type: 'my-type',
      color: 'primary',
      date: dayjs(date).fromNow(),
    },
    android: {
      notification: {
        click_action: 'FCM_PLUGIN_ACTIVITY',
      },
    },
  };

Seems to be expected behavior.
I worked around it by adding "notification_title" and "notification_body" to my data payload and falling back to these values if "title" and/or "body" are not available directly from the notification payload

Seems to be expected behavior.
I worked around it by adding "notification_title" and "notification_body" to my data payload and falling back to these values if "title" and/or "body" are not available directly from the notification payload

Thanks @icemycoffee, that's what I'm doing too.
Btw I don't quite get how is this an expected behaviour?

Thanks @icemycoffee, that's what I'm doing too.
Btw I don't quite get how is this an expected behaviour?

Yes, @chiragsrvstv. Unfortunately this is a native Android limitation. the "notification" part of the payload is not given to the application, and, therefore, not given by getInitialPushPayload().


memory leak

i have found a problem that about memory leak,when i use WkWebVieW ,you shoule use _ _weak decorator the _webViewDelegate in WKWebViewJavascriptBridge class but in face not ,this lead to memory lead when i set delegate.

I believe this has been fixed. Please update to latest version and check. Feel free to reopen.


.

.

There's a short time where you don't know if the user is logged in, needs to re-login, etc. I might hit your site, and have logged in before, but that's not "known" instantly. After firebase resolves whether I'm logged in, logged out, or have never logged in, I'm now "known". It's mostly so you can have a loading animation while auth negotiates.

That's correct.
As an alternative, instead of using a flag, you could also have the store contain a promise which would be unresolved until the status is known, and then resolved with a user object if authenticated or null of not.
That can play nicely with the {#await $auth} block in Svelte templates but ultimately provides the same functionality template-wise.
But a promise can make it easier if you need to wait for auth status to settle elsewhere before doing something, perhaps in another derived store that fetches data for example.

Thank you.

@moutafatin1 FYI: deleting the original question is bad etiquette.


Add suppressing output (verbose level) to the launch function

📝 Description of the feature
I would like to add an argument to the launching function of Fluent that enables suppressing the output of Fluent that is being printed to the console during meshing or solving steps. Some kind of a verbose level type argument.
I am not sure if this is already implemented or not, but I could not find anything related in the documentation.
💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

After trying a couple of stuff, using start_transcript=False in the pyfluent.launch_fluent() function did the trick.


Events always post on HipChat

Despite setting notifications to failed only, all build messages are passing to HipChat. It appears that the "on" setting isn't working.
Configuration:
    - hipchat-notify:
        token: $HIPCHAT_TOKEN
        room-id: 7xxxx0
        from-name: wercker
        on: failed

Yup. Confirmed. :(


Check the semantics of VerifyStateTransition function

VerifyStateTransition function returns a set of calls. These calls are the valid transactions from the block. It needs to be investigated what happens if the calls returned by the VST function are different than the transactions in the passed block.

Add this to the gotcha

I think that this should quite clearly resolve into an all or nothing execution. Transactions are already verified singularly, therefore if the VerifyStateTransition fails, it is either because the block received was faulty or, in the case of a block generator, there is an abnormality that should make the node panic.
Alternatively, the generator could use the VSTF as a way to attempt at batch-verifying transactions concurrently. However, in this case, I would prefer to use a dedicated, explicit RPC exposed by RUSK (such as BatchVerifyTx


fmt Integration is Broken

Hello!
I don't think fmt integration currently works. You define a target, libassert::assert, and specify that users can add a compile definition LIBASSERT_USE_FMT to enable fmt integration on the target. However, you never link the fmt::fmt library to libassert::assert, so enabling LIBASSERT_USE_FMT results in a failure to compile. Specifically:
libassert/include/libassert/assert.hpp:50:11: fatal error: 'fmt/format.h' file not found

I think the fix is pretty straightforward. I think you just need to add a CMake option for fmt integration that will conditionally link libassert::assert to fmt::fmt.
Best,
Rob

Hi Rob,
This is intentional. LIBASSERT_USE_FMT is turns on the relevant code in the assert.hpp header and it's relying on the user having already appropriately installed fmt or linked against fmt::fmt. The thought was if a user is using this flag they should already be using fmt in their codebase. I'm happy to revisit this however presently I figured it was best to not try to have libassert manage this dependency.

Thanks for the help with this Jeremy. I finally figured out what I was doing wrong. I was including both fmt and libassert as subdirectories of my own project. Now that I am properly building and installing them as dependencies and using find_package in my CMakeLists.txt, fmt integration appears to work as intended.

Excellent! I’m glad you got it working!


Move isnan out of contrib_ops and add float16 support for it as per the spec.

/AzurePipelines run all


DO NOT MERGE

just my little experiment

Merged build triggered.

Merged build started.

Test build #36250 has started for   PR 7156 at commit 5860767.

Test build #36250 has finished for   PR 7156 at commit 5860767.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Cast(child: Expression, dataType: DataType) extends UnaryExpression with Logging
    // compiled class file for the closure here will conflict with the one in callUDF (upper case).

Merged build finished. Test FAILed.

sigh ... scala compiler crashes when trying to run the mima checks ...

Merged build triggered.

Can probably close this now.

@squito is your experiment done? Can we close this patch?

yup, sorry forgot about this one ...


proposal: io: io.MultiReader should read until the buffer is full or there are no Readers left

It's an example
func (mr *multiReader) Read(p []byte) (n int, err error) {
	bufSize := len(p)
	for len(mr.readers) > 0 {
		// Optimization to flatten nested multiReaders (Issue 13558).
		if len(mr.readers) == 1 {
			if r, ok := mr.readers[0].(*multiReader); ok {
				mr.readers = r.readers
				continue
			}
		}
		var nr int
		nr, err = mr.readers[0].Read(p[n:])
		n += nr
		if err == io.EOF {
			// Use eofReader instead of nil to avoid nil panic
			// after performing flatten (Issue 18232).
			mr.readers[0] = eofReader{} // permit earlier GC
			mr.readers = mr.readers[1:]
		}
		if n >= bufSize {
			if err == io.EOF && len(mr.readers) > 0 {
				// Don't return EOF yet. More readers remain.
				err = nil
			}
			return
		}
	}
	return n, io.EOF
}

The current code seems OK and implements the documented semantics for io.Reader.  Why should it be changed?

Thank you for your reply. When using io.Copy or io.CopyBuffer, it is often desirable to fill the buffer as much as possible before writing (except for the last write) when reading, the reason being to align memory and disk (probably when using direct-io) for more efficient write performance.
When using direct-io, in order to align 4KB, it may be necessary to first read a portion of the file to form a multiple of 4KB with the newly added data. In this scenario, the two parts can form a MultiReader, and io.Copy does not care if the buffer is full when writing.

sounds like a case where should wrap an io.MultiReader in a bufio.Reader?

This is a part of bufio source code.
// Read reads data into p.
// It returns the number of bytes read into p.
// The bytes are taken from at most one Read on the underlying Reader,
// hence n may be less than len(p).
// To read exactly len(p) bytes, use io.ReadFull(b, p).
// If the underlying Reader can return a non-zero count with io.EOF,
// then this Read method can do so as well; see the [io.Reader] docs.
func (b *Reader) Read(p []byte) (n int, err error) {
	n = len(p)
	if n == 0 {
		if b.Buffered() > 0 {
			return 0, nil
		}
		return 0, b.readErr()
	}
	if b.r == b.w {
		if b.err != nil {
			return 0, b.readErr()
		}
		if len(p) >= len(b.buf) {
			// Large read, empty buffer.
			// Read directly into p to avoid copy.
			n, b.err = b.rd.Read(p)
			if n < 0 {
				panic(errNegativeRead)
			}
			if n > 0 {
				b.lastByte = int(p[n-1])
				b.lastRuneSize = -1
			}
			return n, b.readErr()
		}
		// One read.
		// Do not use b.fill, which will loop.
		b.r = 0
		b.w = 0
		n, b.err = b.rd.Read(b.buf)
		if n < 0 {
			panic(errNegativeRead)
		}
		if n == 0 {
			return 0, b.readErr()
		}
		b.w += n
	}

	// copy as much as we can
	// Note: if the slice panics here, it is probably because
	// the underlying reader returned a bad count. See issue 49795.
	n = copy(p, b.buf[b.r:b.w])
	b.r += n
	b.lastByte = int(b.buf[b.r-1])
	b.lastRuneSize = -1
	return n, nil
}

As I understand the code, bufio.Reader only reads from the underlying reader once at a time, which is not enough to ensure that the buffer is filled.
I think that io.MultiReader should make multi files as one file to the caller. If a file can be read again to fill the buffer, then it should also do it when it is divided into multiple parts and combined by the io.MultiReader.

MultiReader can be used even for TCP streams and many more. If you call Read on anything it must block until it returns at least a single byte (or error).
If you need just this feature for disk operations, you need to make your own buffer. MultiReader is not good enough fix, in case you would use a single Reader (without MultiReader) you would have the same problems.

@polomsky maybe you are right

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group


eip1898: add unit tests to secure all conversions and impl

Full path under serde is used instead of import under cfg flag.
Some functions are simplified.
Unit tests are added to protect all implementations and conversions that can be tricky to mix.

Proposed changes are sensible and does not appear to contain breaking changes. Tests look good, ran tests against main and verified there were no regressions.

@zerosnacks Yes exactly, that's exactly why I was doing this PR. My constant was that there were a lot of conversions in this file and that simplifications could be made in order to end up with exactly the same code while having a less wordy and error-prone implementation.

So I simplified the places that could be simplified by using all the available conversions as much as possible.
I took care to normally add all the unit tests covering all situations because these unit tests are very quick to run and they allow two things:

Check that this PR doesn't break anything and that everything is correct (conversions, string manipulations...)
Be sure that we don't break anything in the future as soon as we touch this EIP.

cc @yash-atreya pending final review, LGTM


Large Projects

My understanding it that the entire "client" will be loaded into the browser. Has anyone tested how large a Blazor project can get before the browser starts having issues?
I have a large project that I would like to test out in Blazor but it is too large (I think) for a single SPA. Has there been any discussion around the ability to maybe have some kind of modules that could be loaded from a URL server side?
Is there any way to do this kind of thing currently? So just as an example my project might have a Sales module, Customers module, Inventory module etc.  Each module could be loaded into the browser as it's own "Blazor" client side package. Some how the Server would need to know what "module" to send to the browser when it receives a request for one.

Lazy loading is tracked here: #25.
Hopefully it will solve our problem. I also have a big LOB application with more than 100 modules (client-server desktop app written in C++). I spend at least a few hours a day working with Blazor. I'm impressed and I believe that it will change the world.

Agreed, I believe Razor has a bright future and the time to get involved is now. I will check out the Lazy loading thread, maybe start contributing as well. Thanks.

yes, but... The way I'd go about this in Angular or Aurelia is not to build a single SPA but rather a set of SPAs, much as the last line of your comment indicates.  A set of Blazor front-end applications with a Shared project should work much the same way, especially if served from the same CDN, and of course, you still get the benefit of sharing code between the client and server.  And a single language to maintain.
Seems to me that regardless of the language or framework they're written in, browser apps are browser apps and follow a certain set of patterns.  These obviously aren't identical to those we use in monolithic LOB application due to the constraints of the client so some adjustments must be made.

@WayneHiller i guess the issue#25 may answer your questions. Just check the last comment from @danroth27

I'll close this as a duplicate of #25.


Fix playground blank screen and formatter

All changed packages have been documented.

:white_check_mark: @typespec/playground

Show changes
@typespec/playground - internal ✏

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


support serfor

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 37 lines in your changes missing coverage. Please review.

Project coverage is 0.22%. Comparing base (e0ec56e) to head (8e36f0e).
Report is 3 commits behind head on main.

Files with missing lines
Patch %
Lines

R/get_hotspots_data.R
0.00%
32 Missing :warning:

R/utils.R
0.00%
4 Missing :warning:

R/get_forest_fire_data.R
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##            main      #6      +/-   ##
========================================
- Coverage   0.24%   0.22%   -0.02%
========================================
  Files         12      13       +1
  Lines        407     443      +36
========================================
  Hits           1       1
- Misses       406     442      +36

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


Add ReadMe file and image

Add ReadMe.md with Liat's resume and a PNG image.

Nice job! please upload a screenshot to Google Classroom if you haven't done so already


Retarget ALS-Refactored

I tried to retarget skelenton from als-refactored to paragon or other characterf ollow the instruction from your tuttorial.
https://www.youtube.com/watch?v=DEpwDjWY2wI&list=PLslFX7TZAr89enRHIfJwUJJ_hJV8wX_8v&index=5
For other plugins like als-community or als original everything works great. But since the developer als-refactored has customized the virtual bone in the original, I have a hard time creating the constraint bone.
I dumbed ske's virtual bone in the als-refactored build:
((VirtualBoneName="VB foot_root",SourceBoneName="root",TargetBoneName="root"),(VirtualBoneName="VB foot_l",SourceBoneName="VB foot_root",TargetBoneName="foot_l"),(VirtualBoneName="VB foot_r",SourceBoneName="VB foot_root",TargetBoneName="foot_r"),(VirtualBoneName="VB hand_l_to_hand_r",SourceBoneName="hand_l",TargetBoneName="hand_r"),(VirtualBoneName="VB hand_r",SourceBoneName="VB hand_l_to_hand_r",TargetBoneName="hand_r"),(VirtualBoneName="VB hand_l_to_ik_hand_gun",SourceBoneName="hand_l",TargetBoneName="ik_hand_gun"),(VirtualBoneName="VB ik_hand_r",SourceBoneName="VB hand_l_to_ik_hand_gun",TargetBoneName="hand_r"),(VirtualBoneName="VB hand_r_to_hand_r",SourceBoneName="hand_r",TargetBoneName="hand_r"),(VirtualBoneName="VB hand_l",SourceBoneName="VB hand_r_to_hand_r",TargetBoneName="hand_l"),(VirtualBoneName="VB hand_r_to_ik_hand_gun",SourceBoneName="hand_r",TargetBoneName="ik_hand_gun"),(VirtualBoneName="VB ik_hand_l",SourceBoneName="VB hand_r_to_ik_hand_gun",TargetBoneName="hand_l"))
And here is the constraintbone that I created:
((ConstraintBone="root",ModifiedBone="ik_foot_root"),(ConstraintBone="foot_l",ModifiedBone="ik_foot_l"),(ConstraintBone="foot_r",ModifiedBone="ik_foot_r"),(ConstraintBone="root",ModifiedBone="ik_hand_root"),(ConstraintBone="hand_r",ModifiedBone="ik_hand_gun"),(ConstraintBone="hand_r",ModifiedBone="ik_hand_r"),(ConstraintBone="hand_l",ModifiedBone="ik_hand_l"),(ConstraintBone="ik_foot_l",ModifiedBone="VB foot_l"),(ConstraintBone="ik_foot_r",ModifiedBone="VB foot_r"),(ConstraintBone="ik_hand_gun",ModifiedBone="VB hand_r_to_ik_hand_gun"),(ConstraintBone="ik_hand_gun",ModifiedBone="VB hand_l_to_ik_hand_gun"),(ConstraintBone="hand_r",ModifiedBone="VB ik_hand_r"),(ConstraintBone="hand_l",ModifiedBone="VB ik_hand_l"),(ConstraintBone="ik_foot_root",ModifiedBone="VB foot_root"),(ConstraintBone="foot_l",ModifiedBone="VB foot_l"),(ConstraintBone="foot_r",ModifiedBone="VB foot_r"),(ConstraintBone="ik_hand_l",ModifiedBone="VB ik_hand_l"),(ConstraintBone="hand_r",ModifiedBone="VB ik_hand_r"))
The result is :(( :

What is the name of white bone that is facing upwards?

Nother way to get the VBs working would be to use the force recompressing feature of TTToolbox. Did you try this already?

What is the name of white bone that is facing upwards?

I'm not sure if it is, before I tried to add the bone constraints it was VB ik_hand_r , but now it's VB hand_l_to_ik_hand_gun = (VirtualBoneName="VB hand_l_to_ik_hand_gun",SourceBoneName="hand_l",TargetBoneName="ik_hand_gun") .

@AchimTuran You can download the project here: Is more clearly for understand. My IK_Rig is on the Folder GameContent->CharacterAssets->ALS or Gknight -> Rigs https://drive.google.com/file/d/1jeZhMcq3nxh2ljzmWbJtt2o8R7MqN4lR/

@PaulNguyen88
Thx for uploading the project will have a look at it during this evening and will get back to you once I know what is wrong. 😉

Can you guide me more clearly? I do not really understand that.

You can do a right click on the skeleton asset and let the editor run this script. Of course it only works for already stored anim sequences which are connected to the skeleton asset. I will soon add a new feature where you can run this script on individual anim sequences.

As this feature is not tested a lot across the community, please make a backup of your project in case something breaks. 😉

Okay the hierarchy seems not to be supported by the solver currently. It might be a loop. Further analysis will be done in #21 if I find the time. This was the reason why I closed this one.

I added a correct constraint for ALS Refactored data table row during the merge of #28. Will be available in v0.5 after #29 is merged back to master.

Thanks you bro, I checked your new config for constraint bone, and look quite good in IK_Rigs. But unfortunely, the retarget results still failed.
ALS refactored used control rig, I dont know what is wrong but the ik_hand bone still not mesh up with the Virtual Bone.
Once again, thank you very much for your contribution to the community :D


Multiple carriage returns in a row are ignored, but should not be

Given:
const text = `word1

word2`

The result is

fixed in ca8c501c90bf64bf2e926086d957d817d1c206b0 but accidentally referenced the wrong issue in the commit message.


Null Safety not supported

I am trying to implement this package to calculate the distance between two Geo-points using the code snippets below:
final geodesy = Geodesy();
final l1 = const LatLng(50.06638889, 5.71472222);
final l2 = const LatLng(58.64388889, 3.07000000);

var distanceBetweenTwoGeoPoints =
        geodesy.distanceBetweenTwoGeoPoints(l1, l2, 6371e3);
    print('Distance between two geopoints: $distanceBetweenTwoGeoPoints');

but I am getting this error about null safety:
Error: Cannot run with sound null safety, because the following dependencies
don't support null safety:

 - package:geodesy

For solutions, see https://dart.dev/go/unsound-null-safety

I am using dart version 2.19.6 (stable)

It seems you are using an outdated geodesy Flutter package that does not support null safety. Additionally, please note that the geodesy package is not related to the geofence_flutter package. I hope this information solves your problem. Thanks.


Simulator Should Clearly And Forcefully Complain When Improperly Setup

I've had a lot of problems writing tests where I set the clock wrong or hadn't run the right passes. The error messages I got from the simulator were somewhat cryptic:

"Assertion failed: (inSels.size() == 2), function updateRegisterValue, file interpret.cpp, line 1360." when correct flatten passes hadn't run
"self.valid cannot be found" when I hadn't setup the simulator correctly by calling state.execute() at least once.

Can the simulator add nice error messages for these user mistakes?

Error message 1. no longer exists due to a simulator refactor.
RE error message 2: In my opinion "self.valid cannot be found" is a clear and forceful complaint. That error originates in a utility function that looks up select values. The function is used internally by the interpreter, and is available to the user in the interpreter API. It doesn't know why the select does not have a value, so it's error message can't tell the user what they need to do to fix the error.
The error message could offer a suggestion about possible sources of the error like clang or ghc sometimes do, but I cannot think of a concise, actionable suggestion to add to the warning that would be worth inserting and that covers all of the possible sources of the error.

@dillonhuff, For Error message 1, I am guessing this is going to be in the next PR? Because I am still seeing it for some stuff I am doing in dev.

@David-Durst, Can this be closed?