Fix saturn path issue and use objcopy to convert to elf

This changes the directory layout of the compiler so that the batch files and absolute paths aren't needed. This also includes the change by @Ced2911 to use objcopy to convert the coff to elf. I don't understand why this is necessary for some inputs on certain systems but it seems to work fine locally.

What about the scripts that are used to invoke the compiler? Those still have hard-coded drive letters, right?

This now uses this folder https://github.com/sozud/saturn-compilers/tree/main/cygnus-2.7-96Q3-bin
We just copy that over
    shutil.move(
        f"{COMPILERS_DIR}/saturn-compilers-main/cygnus-2.7-96Q3-bin",
        f"{COMPILERS_DIR}/cygnus-2.7-96Q3",
    )

Since everything is in the same directory it's basically like it's all on the PATH. I thought the headers and crt0 etc. were required but it doesn't seem like it for the way decomp.me works.


Double-click no longer opens the onnx file

Netron app and version: 4.4.6
OS and browser version: Windows, Edge

this used to work in all the previous versions, now it opens Netron but does not load the model
Steps to Reproduce:

Double click on onnx file from explorer

Please attach or link model files to reproduce the issue if necessary.

same question

Thank you, @lutzroeder, for this awesome tool!

and thanks for such quick fix!


Ability to control verbosity of validation errors

This issue has the exact same motivation as https://github.com/fluxcd/kustomize-controller/issues/190
https://github.com/fluxcd/kustomize-controller/issues/190 was resolved by two filter function:

https://github.com/fluxcd/kustomize-controller/blob/5f966d0ff615658dc630a89837abeb1bf459d75a/controllers/kustomization_controller.go#L645
https://github.com/fluxcd/kustomize-controller/blob/5f966d0ff615658dc630a89837abeb1bf459d75a/controllers/kustomization_controller.go#L648

and they handle the apply case perfectly.
I would like that the validate function gets the same treatment too. https://github.com/fluxcd/kustomize-controller/blob/5f966d0ff615658dc630a89837abeb1bf459d75a/controllers/kustomization_controller.go#L565
There, the kustomize output is returned without any treatment: https://github.com/fluxcd/kustomize-controller/blob/5f966d0ff615658dc630a89837abeb1bf459d75a/controllers/kustomization_controller.go#L601
resulting in a long output, where only the validation error is noteworthy:
validation failed: namespace/flux-system configured (dry run)
customresourcedefinition.apiextensions.k8s.io/alerts.notification.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/buckets.source.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/gitrepositories.source.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/helmcharts.source.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/helmreleases.helm.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/helmrepositories.source.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/kustomizations.kustomize.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/providers.notification.toolkit.fluxcd.io configured (dry run)
customresourcedefinition.apiextensions.k8s.io/receivers.notification.toolkit.fluxcd.io configured (dry run)
serviceaccount/helm-controller configured (dry run)
serviceaccount/kustomize-controller configured (dry run)
serviceaccount/notification-controller configured (dry run)
serviceaccount/source-controller configured (dry run)
clusterrole.rbac.authorization.k8s.io/crd-controller-flux-system configured (dry run)
clusterrolebinding.rbac.authorization.k8s.io/cluster-reconciler-flux-system configured (dry run)
clusterrolebinding.rbac.authorization.k8s.io/crd-controller-flux-system configured (dry run)
secret/gitops-repo configured (dry run)
service/fosdem-2021 configured (dry run)
service/notification-controller configured (dry run)
service/source-controller configured (dry run)
service/webhook-receiver configured (dry run)
error: error validating "fffbeaa2-7f24-44ad-9db5-b14f54d552bf.yaml": error validating data: ValidationError(Deployment.spec.template.spec.containers[0]): unknown field "ima  ge" in io.k8s.api.core.v1.Container; if you choose to ignore these errors, turn validation off with --validate=false

I'm happy to take a stab at this, if the need is generally accepted by Flux maintainers. I would repurpose the parseApplyError function and would take into account that the validation step has  (dry run) suffix in each line.

@laszlocph thanks! please go ahead


Add tutorial for HA with keepalived, Failover IPv4 and Cloud vLAN

This adds a tutorial for a (simple) high availability solution using keepalived, Netcup Failover IPv4, Netcup Cloud vLAN and two Netcup vServers or Root Servers. It is intended as a more advanced tutorial where not every step is ellaborated to the smallest detail.

I have read and understood the Contributor's Certificate of Origin at the end of the template and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: Marcel Pabst<EMAIL_ADDRESS>

Hey, I´m seeing the packets sending with this command sudo tcpdump proto 112 -i eth1. Then I turn off my s1. Should I see in SCP any changes then?

Hey, I´m seeing the packets sending with this command sudo tcpdump proto 112 -i eth1. Then I turn off my s1. Should I see in SCP any changes then?

If you followed the tutorial completely you should see the Failover IP assigned to s2 if s1 is not sending adverts anymore/or the adverts do not reach s2; instead of s1 which can be seen in the network section of s2 in the scp.

When I run sudo journalctl -u keepalived I get the following:
Nov 16 08:26:07 v220211184008137980 Keepalived_vrrp[759]: (VI_1) Entering BACKUP STATE (init)
Nov 16 08:26:13 v220211184008137980 Keepalived_vrrp[759]: (VI_1) Entering MASTER STATE

This happened right after restarting s2 while s1 was running. And it is not visible in SCP.

When I run sudo journalctl -u keepalived I get the following:
Nov 16 08:26:07 v220211184008137980 Keepalived_vrrp[759]: (VI_1) Entering BACKUP STATE (init)
Nov 16 08:26:13 v220211184008137980 Keepalived_vrrp[759]: (VI_1) Entering MASTER STATE

This happened right after restarting s2 while s1 was running. And it is not visible in SCP.

Please always refer to the server where the logs are from.
After starting the service on both servers you should see s1 entering the master state Nov 16 09:04:08 v2202104129099154891 Keepalived_vrrp[1843355]: (VI_1) Entering MASTER STATE and s2 entering backup state Nov 16 09:03:58 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Entering BACKUP STATE (init)
If you restart s1 now (which is our master) then you should see something like this on s2:
Nov 16 09:09:00 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Backup received priority 0 advertisement
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Receive advertisement timeout
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Entering MASTER STATE
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) setting VIPs.
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Sending/queueing gratuitous ARPs on eth1 for <IP_ADDRESS>
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:01 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:06 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Sending/queueing gratuitous ARPs on eth1 for <IP_ADDRESS>
Nov 16 09:09:06 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:06 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:06 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:06 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>
Nov 16 09:09:06 v2202104129099154895 Keepalived_vrrp[464409]: Sending gratuitous ARP on eth1 for <IP_ADDRESS>

We receive a priority 0 advertisement and an advertisement timeout, switching to master state right after that, assigning the VIP to ourself and sending some ARP packages. What happens in the background is the notify_masterscript is called and reroutes our Failover IP via a call to the Netcup Webservice. Only the change of the Failover IP can be seen in the SCP nothing else.
When s1 comes online again s2 switches back to backup which should look something like this in the logs of s2:
Nov 16 09:11:03 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Master received advert from <IP_ADDRESS> with higher priority 255, ours 254
Nov 16 09:11:03 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) Entering BACKUP STATE
Nov 16 09:11:03 v2202104129099154895 Keepalived_vrrp[464409]: (VI_1) removing VIPs.

And the whole switching of the Failover IP to s2 and back to s1 looks like this in the SCP logs of s2:

The logs came from s2.
S2 enters the master state 6 seconds after rebooting although s1 is running. I guess I mocked something up with the script.

The logs came from s2. S2 enters the master state 6 seconds after rebooting although s1 is running. I guess I mocked something up with the script.

Alright, s2 shouldn't be master when s1 is running and it receives the adverts from s1 (which you already checked with sudo tcpdump proto 112 -i eth1 on s2). Maybe some configuration error in the keepalived config? Make sure s2 has a lower priority than s1 and they have the same virtual_router_id. I checked the example keepalived configurations in the tutorial again and they look correct to me.

I tried it again with Debian (I was still using Ubuntu) and now it works. 👍

I tried it again with Debian (I was still using Ubuntu) and now it works. 👍

I forgot to check if the changes are visible in SCP and they are not. But when I run sudo journalctl -u keepalived I get the same result as you.

I tried it again with Debian (I was still using Ubuntu) and now it works. 👍

I forgot to check if the changes are visible in SCP and they are not. But when I run sudo journalctl -u keepalived I get the same result as you.

Alright, so keepalived itself works as intended but the routing of the Failover IP is not. The routing of the Failover IP is changed by a call to the Netcup Webservice via our to_master.sh script. Could you please check if curl is installed on your system (if not sudo apt install curl) and then try again. Please also make sure that the keepalived.conf contains the correct path to the to_master.shscript in the notify_master /PATH/TO/YOUR/SCRIPT/to_master.sh section and that the script has execution rights.

I found the error. My password for the webservice contained a < which broke the whole script.

Hi pabstma, thank you for contributing! It's been a while since you opened this PR and it probably is an open secret we're currently overwhelmed with reviewing all the awesome tutorials we got. Please  be patient - we're about to publish tutorials this and next week. Thank you!


Fix extensible arrow sup vertical alignment

Fixes issue #1254

@ylemkimon You make a good point. I should be more nuanced. So the most recent commit adds the upper element depth to the shift only if: (1) upper element depth > 0.25 em, or (2) the arrow type is a left equilibrium. These are the two cases where we get interference between upper element and arrow.
For a visual comparison, the first image is from this PR.  The second is from AMS-LaTeX. In both images, the "g" vertical alignment is not adjusted, but the big array is adjusted.


chore: release main

:robot: I have created a release beep boop
errors: 0.1.0
0.1.0 (2022-05-04)
Features

add a method to check for operational errors (6a1e9ac)
initial bootstrapping of the errors package (ae787a0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/errors-v0.1.0 :sunflower:


resolvePackageJsonExports doesn't actually work

Bug Report
Using node module resolution doesn't examine exports paths.
adding resolvePackageJsonExports: true is accepted in the tsconfig, but doesn't actually work. It still says ts can't resolve it.
🔎 Search Terms
resolvePackageJsonExports
🕗 Version & Regression Information
Latest
⏯ Playground Link
Can't use npm imports in the playground.
💻 Code
import UI from "@newthink/ui";

{
  "include": ["src/**/*"],
  "exclude": ["src/**/*.spec.ts", "src/**/*.test.ts"],
  "compilerOptions": {
    "baseUrl": "./src",
    "paths": {
      "@/*": ["*"]
    },
    "noEmit": true,
    "declaration": true,
    "outDir": "dist",
    "declarationMap": true,
    "target": "es2017",
    "module": "esnext",
    "moduleResolution": "node",
    "resolvePackageJsonExports": true,
    "noImplicitAny": true
  }
}

🙁 Actual behavior
Cannot find module '@newthink/ui' or its corresponding type declarations.
🙂 Expected behavior
Actually Work

Can't use npm imports in the playground.

You actually can, sort of, do that:
TS playground
Note - please reorder exports.{import,types} to exports.{types,import}. In this case the types condition isn't strictly needed anyway because those files have the same names (just different extensions) so you can rely on the sibling lookup. However, if you want to include the explicit types condition it's best to put it first. package.json#exports are actually order-sensitive

Okay, I am seeing that in the docs
But if types isn't necessary, since it's index.js and index.d.ts then it also should still work even if the order is wrong.

But if types isn't necessary, since it's index.js and index.d.ts then it also should still work even if the order is wrong.

Yeah, it works right now but might be misleading to users or copy-pasted to other projects where the situation will be different.

@Andarist While I understand the order-dependence, does it matter for types?  import and types are, I thought, completely orthogonal (assuming both exist) so it shouldn’t matter which one has “precedence”.

While this might feel odd at first - yes, types shouldn't be treated in any special way here.
What should resolve here and how exactly the algorithm should be written to make the rules clear if the overarching rule of package.json#exports is that they are order-dependent:
{
  "exports": {
    "import": {
      "types": "./foo.d.ts"
      "default": "./bar.js"
    },
    "types": "./baz.d.ts"
  }
}

Of course, the outer types could be used here merely as a "fallback" (if the inner one wouldn't have types condition) but imho that only makes things more muddy when that overarching rule is in place.
Currently, it actually might be used as a fallback but that's considered a bug: https://github.com/microsoft/TypeScript/issues/50762

fwiw I was referring more to how import and types relate to each other when at the same level of nesting, and both refer directly to files.  I guess you’re implying that if the import comes first then the “expected” behavior is that TS would always use sibling lookup

fwiw I was referring more to how import and types relate to each other when at the same level of nesting, and both refer directly to files.

Yeah, but what you describe goes into "more rules" territory whereas the basic rule of order-dependency is pretty simple and using it uniformly helps educating the community at large about this fact. Diverging from it introduces a dangerous precedence and creates more complexity for other tools that try to replicate TS behavior when analyzing packages etc. In every other context those keys are order-dependent so treating types as not fully order-dependent would force other implementers to specialcase this .

I'm not seeing a bug described here.
Generally we don't ship flags that are completely broken out of the gate?

It just doesn't seem to do anything. Or the spec doesn't clarify when it should or shouldn't work.
Basically, the goal would be getting a situation where you can leave off the .js extension, and use packages that don't have main or types entries in the package JSON, using exports only.
Based on the docs, the aforementioned flag seems like it should use exports always regardless of other settings.


Download hint specificially for volo

Hi @wilmoore, can you confirm if this is still working?
https://raw.github.com/carhartl/jquery-cookie/v1.4.1/src/jquery.cookie.js doesn't seems to be working via GET request, it returns 404.

@FagnerMartinsBrack That link works for me.

@carhartl Isn't cached for you due to previous access?

Ah yes, I see, there's 301, the new location is https://raw.githubusercontent.com/carhartl/jquery-cookie/v1.4.1/src/jquery.cookie.js, but that gives a 404... I don't get why.
https://developer.github.com/changes/2014-04-25-user-content-security/

Though the 301 doesn't have anything to do with it. The link is simply broken, it was introduced here:<PHONE_NUMBER>666bada6d1aed655cfaf4e85a4f88b
I say we're going to remove it.

+1 then, it seems nobody is relying on this, otherwise something would have popped up already.

can you confirm if this still matters

I personally do not have a use for this going forward. The one client that was using volo moved directly to npm. I would hope anyone having this problem would just move to npm as well :smile:


Indexer-cli: Make --network argument optional if only operating on single network

Suggestion: query the model when executing command, if only one value for network identifier than don't require the user to supply the --network param.


[WIP]support min,max for decimal data type

Which issue does this PR close?

WIP, this pr is not ready for review.
related #122
Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

This pull request has been split into several small pull requests.


Update to SLDS 2.0.1 (Summer ’16)

SLDS 2.0.1 (Summer ’16) is out, using these versions of the tokens and icons:
    "@salesforce-ux/design-tokens": "3.1.2",
    "@salesforce-ux/icons": "3.5.1",

Done.


Patch to let chef-solo download cookbooks from S3

Hi,
I implemented the support for downloading the cookbooks tarball (option --recipe-url) from S3.
I added 3 settings: aws_key, aws_secret_key and s3_host.
Cheers,
Angelo

Thank you for supporting Chef! Our process has a couple prerequisites before we can merge your contribution. We need to ensure you've completed a Contributor License Agreement (CLA) and a ticket on our ticket tracker for the release workflow. Pull requests are optional, but should always include the ticket number that they're related to for cross-referencing. Please take a moment to review the below wiki page for the appropriate steps:
http://wiki.opscode.com/display/chef/How+to+Contribute


The order of various settings cannot get defined

Describe the bug
The Settings 2.0 API offers for various schemas (e.g. dynatrace_log_processing) the ability to also define their order. The resources offered by the Terraform Provider don't offer any counterpart.
Solution
Resources where the underlying schema in Dynatrace allows for defining an order will get an additional attribute insert_after. That feature enables users to define the order exactly in the same fashion as the Settings 2.0 API.
resource "dynatrace_log_processing" "rule-a" {
  ...
}
resource "dynatrace_log_processing" "rule-b" {
  insert_after = dynatrace_log_processing.rule-a.id
  ...
}
resource "dynatrace_log_processing" "rule-c" {
  insert_after = dynatrace_log_processing.rule-c.id
  ...
}

As an alternative an additional resource dynatrace_settings_order allows for defining the order for previously defined resource instances at once.
resource "dynatrace_log_processing" "rule-a" {
  ...
}
resource "dynatrace_log_processing" "rule-b" {
  ...
}
resource "dynatrace_log_processing" "rule-c" {
  ...
}
resource "dynatrace_settings_order" "log-processing-order" {
  order = [
    dynatrace_log_processing.rule-a.id,
    dynatrace_log_processing.rule-b.id,
    dynatrace_log_processing.rule-c.id
  ]
}

Starting with Release v1.53.0 the resource dynatrace_log_processing offers the attribute insert_after as described above.
We're keeping this ticket open until all resources, that depend on schemas that are eligible for ordering, will have been enriched with the insert_after attribute.

Fantastic, thanks!

We have decided to put the resource dynatrace_settings_order on hold until further notice. Implementing ordering that way would introduce various ways for contradictory behavior.
With the upcoming release all currently supported resources that are are able to get ordered will however contain the insert_after attribute. If not specified the behavior doesn't change from how it's handled currently - the setting will get appended to the end.


Feat/refactor telnet command

refactor telnet command module
separate dubbo and telnet server port
provide telnet and http protocol
add new telnet command,like online service command,offline service command etc,other  maintenance, state-related commands would be addde later
more beautiful UI
and more...

Excellent!  Great convenience for operation and maintenance


Translation of the pageItemsCount

Is there a way to translate the 'of' in '1-8 of 16' from the $refs.paginator.pageItemsCount ?
I'm using i18n for the rest of the app and I'd like to translate that as well..

Need that feature too. Even if I can handle it, could be good to have a proper to do this.
Maybe give data object instead of a full string and let us handle it.

Same problem here using i18n. It is kind of weird to have all words in French but this small "of" on the bottom of my page. Is there any update about this?


How many items maximum can tb.sendTelemetry handle?

Hi,
Prior to void setup(); I've set to ThingsBoardSized<128> tb(espClient);
I'm using ThingsBoard library version 0.6.0.
However the max items that I can send is only 6 as per below:
    const int data_items = 6;

    Telemetry data[data_items] = {
      { "Target", target_m },
      { "Distance", distance_m },
      {"Plantime", plan_mins},
      {"NSpeed", machspeed},
      {"ASpeed", current_mtrpermins},
      {"Downtime", downtime_mins},
    };

    tb.sendTelemetry(data, data_items);

If I want to successfully send 15 items, I'll have to split into 3 sections as the code below:
const int data_items2 = 5;
    const int data_items3 = 4;
    const int data_items = 6;

    Telemetry data[data_items] = {
      { "Target", target_m },
      { "Distance", distance_m },
      {"Plantime", plan_mins},
      {"NSpeed", machspeed},
      {"ASpeed", current_mtrpermins},
      {"Downtime", downtime_mins},
    };

    tb.sendTelemetry(data, data_items);

    Telemetry data2[data_items2] = {
      { "Down1", Dtime1 },
      { "Down2", Dtime2 },
      {"Down3", Dtime3},
      {"Down4", Dtime4},
      {"Down5", Dtime5},
    };

    tb.sendTelemetry(data2, data_items2);

    Telemetry data3[data_items3] = {
      { "Down6", Dtime6 },
      { "Down7", Dtime7 },
      {"Down8", Dtime8},
      {"Down9", Dtime9},
    };

    tb.sendTelemetry(data3, data_items3);

The above code works to send all 15 but when I want to push to Telegram, it won't stitch as one message but separated in to 3 messages.
I would like to know if I can have 15 data send one go from my Arduino Mega with ESP8266 hooked up.
I've also tried  ThingsBoardSized<128,32> tb(espClient); but no difference seen.
Do hope there is a workaround for this.
Best regards,
James

Also to add-on, if I use latest Thingsboard version 0.7.1, I have this error:
C:\Users\James\Documents\Arduino\libraries\ThingsBoard\src/ThingsBoard.h:17:10: fatal error: vector: No such file or directory
Not used: C:\Program Files (x86)\Arduino\libraries\Adafruit_BusIO
#include

Hi anyone?

The old version of the library did not pass the size you set as a template argument ThingsBoardSized<128,32> to the PubSubClient constructor, this has been fixed in later version.
Additionally the first parameter means the amount of characters ('A', 'B', etc.) you need to send to ThingsBoard. I would recommend you to use the ArduinoJson #define called JSON_OBJECT_SIZE(15), 15, if you pass the amount of data points you want to send into that. Meaning 15 it will automatically calculate how much space you need.
The second parameter is, that would be the allowed amount of keys you can send.
You can simply then pass that into the Constructor instead ThingsBoardSized<JSON_OBJECT_SIZE(15), 15>.
Be aware this isn't always perfect, especially if you use long strings or a value is actually an array. For your use case this should work perfectly tough.

Thanks for the tip on the ThingsBoardSized<> constructor!  I've made the change. I was following the example on the thingsboard-arduino-sdk site.
I retrieved and worked with the second fork commit.   Both ESP8266 and ESP32 compiled.   The ESP8266 crashes while in setup().   The ESP32 runs fine with my minimal functionality example (test RPC and subscribed shared attribute functionality along with sendAtttributes functionality)
Order of things in my test sketch in setup():1-Connect to WiFi2-Subscribe to ThingsBoard (fail)Note: using PubSubClient 2.8.0 by O'Leary
For what it's worth, here's the Arduino IDE ESP Exception Decoder output for the ESP8266 crash- Exception 3: LoadStoreError: Processor internal physical address or data error during load or store
PC: 0x40213d26
EXCVADDR: 0x40242f61
Decoding stack results
0x402059a0: PubSubClient::connect(char const*, char const*, char const*, char const*, unsigned char, bool, char const*, bool) at C:\Users\ferguson\Documents\Arduino\libraries\PubSubClient\src\PubSubClient.cpp line 196
0x40100900: umm_init_common(size_t, void*, size_t, bool) at C:\Users\ferguson\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\3.0.2\cores\esp8266\umm_malloc\umm_malloc.cpp line 428
0x40205b88: PubSubClient::connect(char const*, char const*, char const*) at C:\Users\ferguson\Documents\Arduino\libraries\PubSubClient\src\PubSubClient.cpp line 170
0x4020b8b5: PubSubClient::connected() at C:\Users\ferguson\Documents\Arduino\libraries\PubSubClient\src\PubSubClient.cpp line 690
0x40201dc0: subscribeThingsboard() at C:\Users\ferguson\Documents\Arduino\libraries\ThingsBoard\src/ThingsBoard.h line 426
0x40201e95: setup() at G:\workdir\Microcontrollers\Examples\ESP32\ThingsBoard\RPC_and_attribute_update_callbacks_thingsboard_0.7.1/RPC_and_attribute_update_callbacks_thingsboard_0.7.1.ino line 59
0x402086f0: loop_wrapper() at C:\Users\ferguson\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\3.0.2\cores\esp8266\core_esp8266_main.cpp line 198
On Wednesday, December 7, 2022 at 12:54:51 PM CST, MathewHDYT<EMAIL_ADDRESS>wrote:

The old version of the library did not pass the size you set as a template argument ThingsBoardSized<128,32> to the PubSubClient constructor, this has been fixed in later version.
Additionally the first parameter means the amount of characters ('A', 'B', etc.) you need to send to ThingsBoard. I would recommend you to use the ArduinoJson #define called JSON_OBJECT_SIZE(15), 15, if you pass the amount of data points you want to send into that. Meaning 15 it will automatically calculate how much space you need.
The second parameter is, that would be the allowed amount of keys you can send.
You can simply then pass that into the Constructor instead ThingsBoardSized<JSON_OBJECT_SIZE(15), 15>.
Be aware this isn't always perfect, especially if you use long strings or a value is actually an array. For your use case this should work perfectly tough.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Interesting a LoadStoreError seems to happen if the code either attempts to store in IROM or DROM, or alternatively if a load/store is misaligned 8bit instead of 32 (Dereferencing a pointer)
Would it be possible for you to comment the setup method that crashes?

Here's my test code. I apologize but don't fully understand your question about commenting the setup(0 method that crashes.
/* This sketch tests simultaneous     RPC callbacks     Shared attribute callbacks     Attribute sending
   Using ThingsBoard SDK 0.7.1
   Hardware: ESP32 & ESP8266*/
#include "ThingsBoard.h" /// Must make some edits to fix code and prevent duplicate const#include <ESP8266WiFi.h>  /// ESP8266 only//#include <WiFi.h>  /// ESP32 only
/// Set the following four constants to your setupconst char ssid[] = "xxxxx";                     /// SSID of WiFiconst char wifiPassword[] = "yyyyy";        /// Password of WiFiconst char deviceToken[] = "zzzzz"; /// Access token of test deviceconst char thingsboardServer[] = "<IP_ADDRESS>";  /// IP address of ThingsBoard server////////////////////////
bool subscribed = false;bool attrsubscribed = false;
bool toggleSwitchStatus = false;
WiFiClient espClient;ThingsBoardSized<JSON_OBJECT_SIZE(10),10> tb(espClient);
///--------------------------------------------------void setup() {  Serial.begin(115000);  delay(1000);
  initWiFi();  subscribeThingsboard();    Serial.println(F("End of setup()"));} /// void setup()///--------------------------------------------------
/// Set value on switchRPC_Response toggleSwitchSet(const RPC_Data &data){  Serial.println(F("in toggleSwitchSet"));  toggleSwitchStatus = !toggleSwitchStatus;    Attribute attributetogswset[1] = {    { "toggleswitchstatus", toggleSwitchStatus },  };  tb.sendAttributes(attributetogswset, 1);
  return RPC_Response(NULL, toggleSwitchStatus);};
/// Get value on switchRPC_Response toggleSwitchGet(const RPC_Data &data){  Serial.println(F("in toggleSwitchGet"));  return RPC_Response(NULL, toggleSwitchStatus);}
const std::vector<RPC_Callback> callbacks = {  { "toggleswitchset", toggleSwitchSet },  { "toggleswitchget", toggleSwitchGet }};
////// Shared attributesvoid processSharedAttributeUpdate(const Shared_Attribute_Data &data) {/// Next four lines based on Thingsboard example C++98 syntax/// Changed JsonObject to JsonObjectConst to work  for (JsonObjectConst::iterator it = data.begin(); it != data.end(); ++it) {    Serial.println(it->key().c_str());    Serial.println(it->value().as<char>()); // We have to parse data by it's type in the current example we will show here only char data  }
  int jsonSize = JSON_STRING_SIZE(measureJson(data));  char buffer[jsonSize];  serializeJson(data, buffer, jsonSize);  Serial.print(F("Shared attribute buffer: ")); Serial.println(buffer);} /// void processSharedAttributeUpdate()
const Shared_Attribute_Callback callback(processSharedAttributeUpdate);//*/
///==================================================void loop() {  if (WiFi.status() != WL_CONNECTED)    reconnect();
  subscribeThingsboard();  subscribeRPC();  subscribeSharedAttributes();
  tb.loop();} /// void setup()///==================================================
void initWiFi() {  Serial.println(F("initWiFi()"));  WiFi.mode(WIFI_STA);  WiFi.disconnect(true);  WiFi.begin(ssid, wifiPassword);  while ((WiFi.status() != WL_CONNECTED)) {    delay(20);  } /// while
  Serial.println(F("End initWiFi()\n"));} /// void initWiFi()
void reconnect() {  Serial.println(F("reconnect()"));  WiFi.disconnect(true);  WiFi.begin(ssid, wifiPassword);
  while (WiFi.status() != WL_CONNECTED) {    delay(20);  }  Serial.println(F("End reconnect()"));} /// void reconnect()
void subscribeThingsboard() {  Serial.println(F("subscribeThingsboard(): "));  if (!tb.connected()) {    subscribed = false;    attrsubscribed = false;
    while (!tb.connect(thingsboardServer, deviceToken)) {      delay(10);    }    Serial.println(F("Subscribed to Thingsboard"));  }} /// void subscribeThingsboard()
void subscribeRPC() {  if (!subscribed) {    Serial.println(F("subscribeRPC()"));    if (!tb.RPC_Subscribe(callbacks.cbegin(), callbacks.cend())) {      Serial.println(F(" Failed to subscribe to RPC"));      //delay(5000); /// Add delay to prevent rapid subscribing      return;    }    subscribed = true;    Serial.print(F("subscribed: ")); Serial.println(subscribed);  }} /// void subscribeRPC
///void subscribeSharedAttributes() {  if(!attrsubscribed) {    if (!tb.Shared_Attributes_Subscribe(callback)) {      Serial.println("Failed to subscribe for shared attribute updates");      delay(5000); /// Add delay to prevent rapid subscribing      return;    }
    attrsubscribed = true;    Serial.println(F("End subscribeAttributes()"));  }} /// void subscribeAttributes()///
On Wednesday, December 7, 2022 at 03:34:35 PM CST, MathewHDYT<EMAIL_ADDRESS>wrote:

Interesting a LoadStoreError seems to happen if the code either attempts to store in IROM or DROM, or alternatively if a load/store is misaligned 8bit instead of 32 (Dereferencing a pointer)
Would it be possible for you to comment the setup method that crashes?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Sorry for the confusion with commenting I just meant that you could post them in this GitHub thread, like you did.
But for the future, if you paste code into GitHub. I would recommend you to use code button atop of the writing box, this allows to mark segments as code.

Just use three ` following the coding language used (c++) then a newline with all your code and at the end add a newline and three ` again to close that section.
So for example:
```c++
ThingsBoardSized<JSON_OBJECT_SIZE(10),10> tb(espClient);
```

For the code I'm confused about your thingsboardServer variable with the value <IP_ADDRESS>, because that IP would be a private IP address, do you use the same thingsboardServer value for the ESP32 as well and it works there?
const char deviceToken[] = "zzzzz";
/// Access token of test device
const char thingsboardServer[] = "<IP_ADDRESS>";

void setup() {
    Serial.begin(115000);
    delay(1000);
    subscribeThingsboard();
    Serial.println(F("End of setup()"));
}

void subscribeThingsboard() {
  Serial.println(F("subscribeThingsboard(): "));
  if (!tb.connected()) {
    while (!tb.connect(thingsboardServer, deviceToken)) {
        delay(10);
    }
    Serial.println(F("Subscribed to Thingsboard"));
  }
}

Because according to the LoadStoreError you've pasted previously the PubSubClient seems to crash in this specific connect method:
boolean PubSubClient::connect(const char *id, const char *user, const char *pass) {
    return connect(id,user,pass,0,0,0,0,1);
}

and then even more specifically probably on the result = _client->connect(this->ip, this->port); line.
boolean PubSubClient::connect(const char *id, const char *user, const char *pass,
const char* willTopic, uint8_t willQos, boolean willRetain, const char* willMessage, boolean cleanSession) {
    if (!connected()) {
        int result = 0;

        if(_client->connected()) {
            result = 1;
        } else {
            if (domain != NULL) {
                result = _client->connect(this->domain, this->port);
            } else {
                result = _client->connect(this->ip, this->port);
            }
        }

The thing I'm assuming currently is that the PubSubClient can accept an IP but it can't be passed like it is done in your example. Might it be possible for you to change this line of the script and I will create a overload in a new commit, that allows to connect with a direct IP address as well.
const char thingsboardServer[] = "<IP_ADDRESS>";
// To this
IPAddress thingsboardServer = IPAddress(192, 168, 1, 107);

My apologies for not using proper commenting.  I very much appreciate your patience.
To be clear, the only combination of ESP8266/ESP32 and ThingsBoard.h versions that runtime fail in the Thingsboard subscribing function, using the line below, is ESP8266 + ThingsBoard.h 0.7.1. The ESP32 works.
const char thingsboardServer[] = "<IP_ADDRESS>";

Following your suggestion, I tried using the following two versions to define 'thingsboardServer'
IPAddress thingsboardServer = IPAddress(192,168,1,107);
// and
IPAddress thingsboardServer(192,168,1,107);  // pulled from mqtt_basic example for PubSub.

with the resulting compile error for both versions below
error: conversion from 'IPAddress' to 'const char*' is ambiguous
  154 |     while (!tb.connect(thingsboardServer, deviceToken))

I note that in ThingsBoard.h, the way the connection using PubSub is done is different between v0.6 and 0.7. Could this be the issue? I'm running out of time today but might try to swap v0.6 into 0.7.
    inline const bool connect(const char *host, const char *access_token = PROV_ACCESS_TOKEN, int port = 1883, const char *client_id = DEFAULT_CLIENT_ID, const char *password = NULL) {

Sorry the change didn't work previously, because the code does not have an overload for IP Address class. I've added that with my newest commit to the fork tough, would be nice if you could try it and see if it compiles and especially if the runtime error is gone now or if it still persists.

@imbeacon I think this issue can be closed, previously the ESP8266 had errors with PROGMEM, therefore it is optional now and should work.


Fill out Assistant SDK main README

First beta is released. https://repo1.maven.org/maven2/com/azure/azure-ai-openai-assistants/


Inference Pipeline with Scikit-learn and Linear Learner.ipynb issues

input types
This Inference Pipeline with Scikit-learn and Linear Learner notebook states that csv is the only valid input type, but this is incorrect according to the SM SKLearn docs:

"The SageMaker Scikit-learn model server provides a default implementation of input_fn. This function deserializes JSON, CSV, or NPY encoded data into a NumPy array."

@aaronmarkham The script has a custom implementation of input_fn that handles only CSV in that example, hence that comment. The user can write logic to decode any other format. It is true that the default implementation of input_fn does de-serialize JSON, CSN or NPY.


S3 plugin: don't use a default port

Fixes #247

Should we validate/document that if s3_port is given, s3_host is also non-empty?

I don't have a strong opinion, I'm fine not validating this, but up to you.

Added a comment and validation.


Allow support for output directory

Hey Mathijs,
So I've implemented the new output directory feature as discussed in #57.
Let me know what you think :)

Thanks! Took a swift look at it and looks nice. (pun intended)
Will go through it more thoroughly when I have a little more time and then merge it. Looks like the way to go. :)

Cool, been using it for my project yesterday and it seems to work fine :)

This will be supported when #76 is merged! Thanks again for the PR!


m1 max still have issues in the final step[Usage]

Describe the issue
Issue:
I have enable the m1 chip using the --device mps but still have the errors.
Command:
python3 -m llava.serve.cli \
    --model-path liuhaotian/llava-v1.5-7b \
    --image-file "https://llava-vl.github.io/static/images/view.jpg" \
    --load-4bit --device mps

Log:
/opt/homebrew/lib/python3.11/site-packages/bitsandbytes/cextension.py:34: UserWarning: The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit multiplication, and GPU quantization are unavailable.
  warn("The installed version of bitsandbytes was compiled without GPU support. "
'NoneType' object has no attribute 'cadam32bit_grad_fp32'
[2023-10-30 01:27:00,711] [INFO] [real_accelerator.py:110:get_accelerator] Setting ds_accelerator to cuda (auto detect)
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:02<00:00,  1.28s/it]
USER: hello
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/Users/bidetime/Research/Projects/LLaVA/llava/serve/cli.py", line 125, in <module>
    main(args)
  File "/Users/bidetime/Research/Projects/LLaVA/llava/serve/cli.py", line 87, in main
    input_ids = tokenizer_image_token(prompt, tokenizer, IMAGE_TOKEN_INDEX, return_tensors='pt').unsqueeze(0).cuda()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/torch/cuda/__init__.py", line 239, in _lazy_init
    raise AssertionError("Torch not compiled with CUDA enabled")
AssertionError: Torch not compiled with CUDA enabled
ASSISTANT: %

Screenshots:
You may attach screenshots if it better explains the issue.

macOS support is updated just now, with quantization coming later. Please pull the latest code base and install/run following the instructions here. You may also try https://github.com/ggerganov/llama.cpp/pull/3436.

macOS support is updated just now, with quantization coming later. Please pull the latest code base and install/run following the instructions here. You may also try ggerganov/llama.cpp#3436.

Thnanks very much, I have run the following, to update to the lastest git repo,
git pull
pip install -e .
And have the output, Successfully installed llava-1.1.3
and start the process again, but the same issue persist, text interaction works, once load the pics, the error pops out.
NETWORK ERROR DUE TO HIGH TRAFFIC. PLEASE REGENERATE OR REFRESH THIS PAGE.

@bioinfomagic what about try with the cli?
python -m llava.serve.cli \
    --model-path liuhaotian/llava-v1.5-7b \
    --image-file "https://llava-vl.github.io/static/images/view.jpg" \
    --device mps

@bioinfomagic what about try with the cli?
python -m llava.serve.cli \
    --model-path liuhaotian/llava-v1.5-7b \
    --image-file "https://llava-vl.github.io/static/images/view.jpg" \
    --device mps

Oh, thanks very much Haotian, it works on my mac now.

@bioinfomagic
Btw, how is the speed on M1 Max and how much RAM do you have?

@bioinfomagic
Btw, how is the speed on M1 Max and how much RAM do you have?

The webUI works perfectly, now I can run LLaVA on my mac with similar output as the LLaVA online demo. Just with a slower speed.
I somehow used 85~125GB of ram and the speed is around 2~3 words per second. I can see the CPU usage shows 105% GPU usage is around 35% according to the activity monitor. In regarding of speed, I can see other LLM apps, when I choose metal enabled I can see the CPU load drops to 30% and GPU loads gose up to 100% or 200% and the speed is much faster same as online chatgpt 10 words per second.
For the CLI it seems still have issues, not sure if it is my problem, but webUI works very well.
-m llava.serve.cli --model-path liuhaotian/llava-v1.5-7b --image-file "https://llava-vl.github.io/static/images/view.jpg" --device mps
[2023-10-31 20:28:04,996] [INFO] [real_accelerator.py:110:get_accelerator] Setting ds_accelerator to cuda (auto detect)
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:03<00:00,  1.82s/it]
USER: hello
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/LLaVA/llava/serve/cli.py", line 125, in
main(args)
File "/LLaVA/llava/serve/cli.py", line 87, in main
input_ids = tokenizer_image_token(prompt, tokenizer, IMAGE_TOKEN_INDEX, return_tensors='pt').unsqueeze(0).cuda()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/homebrew/lib/python3.11/site-packages/torch/cuda/init.py", line 289, in _lazy_init
raise AssertionError("Torch not compiled with CUDA enabled")
AssertionError: Torch not compiled with CUDA enabled
ASSISTANT: %
But If I add the --load4bit, it won't load,
python3 -m llava.serve.cli --model-path liuhaotian/llava-v1.5-7b --image-file "https://llava-vl.github.io/static/images/view.jpg" --load-4bit --device mps
[2023-10-31 20:26:52,172] [INFO] [real_accelerator.py:110:get_accelerator] Setting ds_accelerator to cuda (auto detect)
Traceback (most recent call last):
File<EMAIL_ADDRESS>line 563, in from_name
return next(cls.discover(name=name))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
StopIteration
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "//LLaVA/llava/serve/cli.py", line 125, in
main(args)
File "/LLaVA/llava/serve/cli.py", line 32, in main
tokenizer, model, image_processor, context_len = load_pretrained_model(args.model_path, args.model_base, model_name, args.load_8bit, args.load_4bit, device=args.device)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/LLaVA/llava/model/builder.py", line 33, in load_pretrained_model
kwargs['quantization_config'] = BitsAndBytesConfig(
^^^^^^^^^^^^^^^^^^^

It seems that you haven't pulled the latest code base:
https://github.com/haotian-liu/LLaVA/blob/main/llava/serve/cli.py#L87

It seems that you haven't pulled the latest code base:
https://github.com/haotian-liu/LLaVA/blob/main/llava/serve/cli.py#L87

Thank you very much, you are right, I somehow didn't update the code base correctly, now I updated it works perfectly now for both CLI and WebUI.
python3 -m llava.serve.cli --model-path liuhaotian/llava-v1.5-7b --image-file "https://llava-vl.github.io/static/images/view.jpg" --device mps
[2023-10-31 20:50:05,977] [INFO] [real_accelerator.py:110:get_accelerator] Setting ds_accelerator to cuda (auto detect)
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:03<00:00,  1.69s/it]
USER: hey
ASSISTANT: Hello! How can I help you today?
USER: summarize the pic

Hmmm. I previously thought it was because of my poor M2 16GB, so it seems that the MPS still needs some optimization.

Hmmm. I previously thought it was because of my poor M2 16GB, so it seems that the MPS still needs some optimization.

I totally agree, I can see LMstudio has a much better metal support, it runs 70B model much faster, when metal enable, clearly noticed the speed difference.


'str' object has no attribute 'path'

What version of Music Assistant has the issue?
2022.0.b6
The problem
Updated to b6 and obseved this error in the logs. I have not interacted with MA at all since the reboot.
How to reproduce
Not sure...
Relevant log output
Logger: music_assistant
Source: /usr/local/lib/python3.10/site-packages/music_assistant/mass.py:245
First occurred: 10:47:18 AM (1 occurrences)
Last logged: 10:47:18 AM
Job [Library sync for provider Filesystem] failed with error 'str' object has no attribute 'path'.
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 233, in sync_library
await self._process_deletions(deleted_files)
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/filesystem.py", line 242, in _process_deletions
if "." not in file_path.path or file_path.path.startswith("."):
AttributeError: 'str' object has no attribute 'path'
Additional information
No response
What version of Home Assistant Core are your running
2022.7.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

@OzGav
Didn't see this error anymore in the latest beta's.
Same for you? If so we can close the issue

yes looks fixed


feat: support metadata query by userid

git node metadata @srl295 will show up to 10 active PRs and give the current status.
I'm not a frequent contributor to node. But, sometimes I have a couple of different PRs in the works. This helps me find out what the current status is.

@srl295 do you think this is something you still want to work on?

Can't now, job change

From: Shelley Vohr<EMAIL_ADDRESS>Sent: Tuesday, July 14, 2020 4:48:29 PM
To: nodejs/node-core-utils<EMAIL_ADDRESS>Cc: Steven R. Loomis<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [nodejs/node-core-utils] feat: support metadata query by userid (#397)
@srl295https://github.com/srl295 do you think this is something you still want to work on?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/nodejs/node-core-utils/pull/397#issuecomment-658467113, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGQZMYUA2R7NBNYI3NPY2TR3TVE3ANCNFSM4LQXCC3Q.


Fix Helm Chart Regression: Use Official Image Repo

Looks like some testing code leaked into the final change. The image should be lightbend/kafka-lag-exporter instead of localhost:32000/seglo/kafka-lag-exporter
Git blame shows this commit responsible for this change: https://github.com/lightbend/kafka-lag-exporter/commit/946f90d49b0f7633466b83f5ccbe8946510ea3f3
@seglo can you confirm?

I pushed my own commit to master to fix this.  I released and smoke tested 0.5.4.


question: how to go to the home page

Once the extension is unpacked and loaded, I get the home page.
But how do I get back there again?

Right now, when I reload the extension, I get the homepage:

Once the extension is unpacked and loaded, I get the home page.

So basically, that page which you see is not the home page. Home page is yet to be implemented, which will also double up as an onboarding page for the users.

But how do I get back there again?

Oh that's a great question, Tushar.
The page which you see is known as the options page, which let's the user customise the . It can be accessed by right-clicking on the extension's icon & then choosing "Options" from the menu. (See image below 👇🏼)

Right now, I've kept the setting such that the options page opens automatically as soon as the extension is installed/reloaded.

Thank you, that answers!


helloworld-test-sample failed when rerun

Extension sample
helloworld-test-sample
VS Code version
1.89.0
What went wrong?
I did the following on Mac

clone this repo
cd helloworld-test-sample
npm install
npm test
test was passed

  Extension Test Suite
    ✔ Sample test
  1 passing (1ms)
[main 2024-05-03T20:24:49.474Z] Extension host with pid 94860 exited with code: 0, signal: unknown.
Exit code:   0
Done

When I ran npm test again, I got error:
Found existing install in /Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0. Skipping download
2024-05-03 16:24:55.315 Electron[94994:16431137] WARNING: Secure coding is automatically enabled for restorable state! However, not on all supported macOS versions of this application. Opt-in to secure coding explicitly by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState:.
WARNING: IPC handle "/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/user-data/1.89-main.sock" is longer than 103 chars, try a shorter --user-data-dir
[main 2024-05-03T20:24:55.385Z] Could not delete obsolete instance handle Error: ENOENT: no such file or directory, unlink '/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/user-data/1.89-main.sock'
    at unlinkSync (node:original-fs:1904:3)
    at ke.f (/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0/Visual Studio Code.app/Contents/Resources/app/out/vs/code/electron-main/main.js:112:4193)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0/Visual Studio Code.app/Contents/Resources/app/out/vs/code/electron-main/main.js:112:1240
    at async ke.a (/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0/Visual Studio Code.app/Contents/Resources/app/out/vs/code/electron-main/main.js:112:1127) {
  errno: -2,
  syscall: 'unlink',
  code: 'ENOENT',
  path: '/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/user-data/1.89-main.sock'
}
[main 2024-05-03T20:24:55.386Z] Error: ENOENT: no such file or directory, unlink '/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/user-data/1.89-main.sock'
    at unlinkSync (node:original-fs:1904:3)
    at ke.f (/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0/Visual Studio Code.app/Contents/Resources/app/out/vs/code/electron-main/main.js:112:4193)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0/Visual Studio Code.app/Contents/Resources/app/out/vs/code/electron-main/main.js:112:1240
    at async ke.a (/Users/gkwan/tasks/CNAI/guides/working/openapi-generator/vscode-extension-samples/helloworld-test-sample/.vscode-test/vscode-darwin-arm64-1.89.0/Visual Studio Code.app/Contents/Resources/app/out/vs/code/electron-main/main.js:112:1127)
Exit code:   1
Failed to run tests

I expect no error when run the test again

Then, I removed the .vscode-test and ran the npm test, the error still occured

I found the workaround:

remove the helloworld-test-sam directory that is created by the first run


[metadata prespecialization] Mark tests unsupported on windows.

@swift-ci please smoke test

@swift-ci please test windows platform

I think that it’s just missing COMDAT, I’ll take a look.

#29150 is better than marking the tests as unsupported

Thank you very much, @compnerd .  The plan was to correct the tests on Windows.  Since you have already done that, closing this.


parquet-cascading and the mapred API does not create metadata file

Repro: run a scalding job that writes parquet files to a folder. no _metadata and _common_metadata file is created
Impact: potential performance problem if parquet metadata is read from client side, which is the case for sparkSQL
casue: the metatdata writing logic is in the mapreduce API but not the mapred API of parquet.
Reporter: Tim / @tsdeng
Assignee: Tim / @tsdeng
Related issues:

MapredParquetOutputCommitter does not work in hadoop2 (is a clone of)

Note: This issue was originally created as PARQUET-197. Please see the migration documentation for further details.

Tim / @tsdeng:
reopen this for fixing hadoop 2 API:
https://github.com/apache/incubator-parquet-mr/pull/141

Alex Levenson / @isnotinvain:
Issue resolved by pull request 141
https://github.com/apache/incubator-parquet-mr/pull/141

Alex Levenson / @isnotinvain:
Issue resolved by pull request 131
https://github.com/apache/incubator-parquet-mr/pull/131


Failed to run dev on a newly created project

I used create-preact to start a project. But when I ran yarn run I got this:
VITE v4.4.9  ready in 487 ms
  ➜  Local:   http://<IP_ADDRESS>:3000/
  ➜  Network: use --host to expose
  ➜  press h to show help
Error:   Failed to scan for dependencies from entries:
  /pages/index.page.tsx
/renderer/_default.page.client.tsx

  ✘ [ERROR] Expected ">" but found "value"

    renderer/usePageContext.ts:13:27:
      13 │   return <Context.Provider value={pageContext}>{children}</Context...
         │                            ~~~~~
         ╵                            >

And when I tried to go to localhost I got this:
14:59:08 [vite-plugin-ssr][request(1)] HTTP request: /
14:59:09 [vite][request(1)] Failed to transpile /renderer/usePageContext.ts because:
Unexpected token, expected ","
  11 |
  12 | function PageContextProvider({ pageContext, children }) {
> 13 |   return <Context.Provider value={pageContext}>{children}</Context.Provider>
     |                  ^
  14 | }
  15 |
  16 | function usePageContext() {
┌───────────────────────────────┐
│ Error isn't helpful? See https://vite-plugin-ssr.com/errors#verbose │
└───────────────────────────────┘
14:59:09 [vite-plugin-ssr][request(1)][Warning] No _error.page.js found, we recommend defining an error page, see https://vite-plugin-ssr.com/error-page
14:59:09 [vite-plugin-ssr][request(1)] HTTP response / ERR

Weird looks like usePageContext did not copy with .tsx even though it should 😅

Renamed it. It's working now :)

Sorry, that bug sneaked into 0.2.0 without us noticing; soon afterwards 0.2.1 was published with the fix, but it looks like you have the bad version cached locally.
You should be able to use npm init preact@latest to clear that though.


Validating Complex Models (Model within a model) using SetValidator

Hi @JeremySkinner , I'm having an issue using the SetValidator in my project.
I have a web.api project with FluentValidation installed.
I call my validator like this
PostParamValidator validator = new PostParamValidator();             ValidationResult results = validator.Validate(param);
My Model is like this
    public class PostParam
    {
        public bool IsCreditCard { get; set; }
        public PostParamCreditCardDetail CreditCardInfo { get; set; }
    }

    public class PostParamCreditCardDetail
    {
        public string CardType { get; set; }
        public string CardNumber { get; set; }
        public string CardName { get; set; }
    }

and my validator is like this :
    public class DepositOnlinePostParamValidator : AbstractValidator<DepositOnlinePostParam>
    {
        public DepositOnlinePostParamValidator()
        {
            RuleFor(x => x.CreditCardInfo).SetValidator(new DepositOnlinePostParamCreditCardDetailValidator());

        }
    }

public class DepositOnlinePostParamCreditCardDetailValidator : AbstractValidator<DepositOnlinePostParamCreditCardDetail>
    {
        public DepositOnlinePostParamCreditCardDetailValidator()
        {
            RuleFor(x => x.CardType).NotEmpty().NotNull();
            RuleFor(x => x.CardNumber).NotEmpty().NotNull();
            RuleFor(x => x.CardName).NotEmpty().NotNull();
        }

    }

In this issue https://github.com/JeremySkinner/FluentValidation/issues/401 You mentioned to

Remove the ValidatorAttribute from the AddressValidator

But i dont use the ValidatorAttribute and it still doesnt work.
Thanks,
Ruben

Sorry, seems that there was an issue when web.api binded the parameters. All good now. thanks!


Logarithmic scales for parallel coordinates

... and while we're at it there is no reason not to switch to bokeh with this one as well, to have a somewhat uniform design.

With 1.3.2 parallel coordinates are interactive! Log-axes are a matter of remapping tick-labels in the bokeh-plot in cave/plot/parallel_plot/parallel_plot.py.


Remove sr-arithmetic/fuzzer from workspace to fix windows builds

Currently, builds do not work on windows due to a honggfuzz-rs compile error:
error: honggfuzz-rs does not currently support Windows but works well under WSL (Windows Subsystem for Linux)
 --> C:\Users\Me\.cargo\registry\src\github.com-1ecc6299db9ec823\honggfuzz-0.5.45\build.rs:7:1
  |
7 | compile_error!("honggfuzz-rs does not currently support Windows but works well under WSL (Windows Subsystem for Linux)");
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

We could solve this by either removing the fuzzer from the workspace (which seems much simpler to me) or by conditionally compiling honggfuzz-rs via cfg(not(target_family="windows")).

I would go the conditional compilation way. Otherwise this will not be run.

@bkchr I don't see the benefit in running the code and getting an error saying that windows is not supported by the fuzzer vs that same error occurring in compilation.

I would expect that conditional compilation does not makes the fuzzer error on windows, as it is not run/compiled.

Sorry, to clarify, we could use an alternative main function like:
#[cfg(target_family="windows")]
fn main() {
    println!("Error: honggfuzz-rs does not currently support Windows but works well under WSL (Windows Subsystem for Linux)");
}

and mark the fuzzer main function with #[cfg(not(target_family="windows"))], but I think it makes sense to let the compilation fail instead. Additionally, the fuzzers aren't run via cargo run but via cargo hfuzz, which won't compile on windows either.


Keys Being Set Without TTL

I am creating keys with this client
var secondOptions = new TimeSeriesOptions(1000, 1000, TimeSpan.FromMinutes(1), "ts#s");
temp.SecondClient = TimeSeriesFactory.New(_db, key, secondOptions);

It is creating keys with no TTL. Is this desired behavior. I would expect every key generated by this client would expire ~1 minute from creation?

True !
Thanks for the issue. Fix is included in the lastest nuget.


Corrected serialisation of Z1 and Z2

Hi @paulmillr,
Here are two small fixes for compressG1() and compressG2(). I'm not sure if the isEmpty() function is meant to imply that a point is the point at infinity, but I found if I modified that it broke other tests.
Let me know what you think.
Mark

Thanks!


  This one doesn't fail that much more than, say, mps2_an385 does due to timing issues, so it got sorta hidden in the noise.  My "timing-reliable parallel qemu test" rig is up to 595 iterations of this now with zero failures.

(Just remembered to shut the test down as it crossed 4400 successful tests without a failure)


The code of conduct link on the readme file is broken

Provide a general description of the issue or problem.
The code of conduct link on the readme file is broken

Thanks for the report, this is now fixed.


Added improvments on string copying, string comparation & calculation of next index in case of collision in custom map

Check List:

[x] You have run ./mvnw verify and the project builds successfully
[x] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[x] All formatting changes by the build are committed
[x] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[x] Output matches that of calculate_average_baseline.sh
[x] For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (line number)

Execution time: 3s - 4s
Execution time of reference implementation: 2 min - 3 min

00:04.101 now.

great, thank you :)


Support PiP when viewing a stream enabled camera

Home Assistant Android version:
1.3.1 (77)
Android version:
10
Phone model:
Pixel 4 XL
Home Assistant version:
0.103.3
Last working Home Assistant release (if known):
N/A
Description of problem:
We should support PiP when a camera is opened and the video stream starts.
It also looks like the full screen icon is disabled, that may be the reason why PiP is not working?

Traceback (if applicable):

Additional information:

Yes, please add this feature.

I'm getting the same problem as this as well. Also when turning the phone landscape the header banner with the camera name is pushing down the video chopping off the bottom

@a-jackson that sounds like a separate issue. If you experience that issue in Chrome on Android then please open a bug on the front end repo.  If it only exists on the android app then please open a new issue here for it.

Closed in #471


Class not found while querying database

I have use the example module which is working like a charm.
I have copied the same module to one of my other app to test the realm and  the pubspec.yaml also looking exactly same but as soon as i move to fetchwidget it give me following error like model class and database is not available.
W/System.err( 4031): java.lang.IllegalArgumentException: Class does not exist in the Realm and cannot be queried: Product W/System.err( 4031): 	at io.realm.DynamicRealm.where(DynamicRealm.java:178) W/System.err( 4031): 	at com.it_nomads.flutter_realm.FlutterRealm.onMethodCall(FlutterRealm.java:97) W/System.err( 4031): 	at com.it_nomads.flutter_realm.FlutterRealmPlugin.onMethodCall(FlutterRealmPlugin.java:73) W/System.err( 4031): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:231) W/System.err( 4031): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:93) W/System.err( 4031): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:642) W/System.err( 4031): 	at android.os.MessageQueue.nativePollOnce(Native Method) W/System.err( 4031): 	at android.os.MessageQueue.next(MessageQueue.java:336) W/System.err( 4031): 	at android.os.Looper.loop(Looper.java:174) W/System.err( 4031): 	at android.app.ActivityThread.main(ActivityThread.java:7682) W/System.err( 4031): 	at java.lang.reflect.Method.invoke(Native Method) W/System.err( 4031): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:516) W/System.err( 4031): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950) E/flutter ( 4031): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: PlatformException(Class does not exist in the Realm and cannot be queried: Product, Class does not exist in the Realm and cannot be queried: Product, [Ljava.lang.StackTraceElement;@186d198) E/flutter ( 4031): #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:569:7) E/flutter ( 4031): #1      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:321:33) E/flutter ( 4031): <asynchronous suspension> E/flutter ( 4031): #2      MethodChannelTransport.invokeMethod (package:flutter_realm/src/method_channel_transport.dart:18:16) E/flutter ( 4031): #3      Realm._invokeMethod (package:flutter_realm/src/realm.dart:81:16) E/flutter ( 4031): #4      Realm.allObjects (package:flutter_realm/src/realm.dart:86:7) E/flutter ( 4031): #5      _FetchWidgetState._fetchAll (package:flutterwhatsapp/fetch_widget.dart:67:41) E/flutter ( 4031): #6      _FetchWidgetState.initState (package:flutterwhatsapp/fetch_widget.dart:22:5) E/flutter ( 4031): #7      StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:4355:58) E/flutter ( 4031): #8      ComponentElement.mount (package:flutter/src/widgets/framework.dart:4201:5) E/flutter ( 4031): #9      Element.inflateWidget (package:flutter/src/widgets/framework.dart:3194:14) E/flutter ( 4031): #10     Element.updateChild (package:flutter/src/widgets/framework.dart:2988:12) E/flutter ( 4031): #11     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4243:16) E/flutter ( 4031): #12     Element.rebuild (package:flutter/src/widgets/framework.dart:3947:5) E/flutter ( 4031): #13     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2432:33) E/flutter ( 4031): #14     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:773:20) E/flutter ( 4031): #15     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:283:5) E/flutter ( 4031): #16     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1102:15) E/flutter ( 4031): #17     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1041:9) E/flutter ( 4031): #18     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:957:5) E/flutter ( 4031): #19     _rootRun (dart:async/zone.dart:1126:13) E/flutter ( 4031): #20     _CustomZone.run (dart:async/zone.dart:1023:19) E/flutter ( 4031): #21     _CustomZone.runGuarded (dart:async/zone.dart:925:7) E/flutter ( 4031): #22     _invoke (dart:ui/hooks.dart:259:10) E/flutter ( 4031): #23     _drawFrame (dart:ui/hooks.dart:217:3) E/flutter ( 4031):  I/Surface ( 4031): opservice is null false
Any clue where i am making mistake to register table schema or model class ?

It seems that Realm wasn't correctly setup for that module. Check gradle configs

@mogol I haven't updated the schema tables for android and ios. I did it and worked now so hence closing this issue.


How can I update the count when loading more photo after present?

I fetch 10 images per time.
var browser = SKPhotoBrowser()
    override func collectionView(collectionView: UICollectionView, didSelectItemAtIndexPath indexPath: NSIndexPath) {
        let cell = collectionView.cellForItemAtIndexPath(indexPath) as! GirlCollectionViewCell

        let originImage = cell.ivGirl.image // some image for baseImage
        browser = SKPhotoBrowser(originImage: originImage!, photos: images, animatedFromView: cell)
        browser.delegate = self
        browser.initializePageIndex(indexPath.row)
        presentViewController(browser, animated: true, completion: {})
    }

The browserViewController have shown 10 images, and when I swipe to 10/10 image, I fetch another 10 images in this method:
func didShowPhotoAtIndex(index: Int) {
        print("didShowPhotoAtIndex:\(index)")
        if index + 1 >= self.girlList.count {

            RequestManager.getGirlList(self.index, onSuccess: { (girlList) -> () in
                if girlList.count > 0 {
                    for girl in girlList {
                        let photo = SKPhoto.photoWithImageURL(girl.url)
                        photo.shouldCachePhotoURLImage = false // you can use image cache by true(NSCache)
                        self.images.append(photo)
                    }
                    self.browser.reloadData()
                    self.browser.updateToolbar()
               // ...here's do something else..
                } else {
                    self.collectionView!.mj_footer.endRefreshingWithNoMoreData()
                }
            })
        }

    }

but when I swipe to 10/10 image, the toolNextButton does not enable and the number 10/10 does not update to 10/20.
Please give me some advice.

try it:
for {...}
self.browser.photos = self.images
self.browser.reloadData()
...

Nice! But the code:
self.browser.photos = self.images

should change to:
self.browser.photos = self.images.map({$0 as SKPhoto})

Thanks a lot.


imports missing generating source code

Overview of the issue
Jeddict 5.4 doesn't anymore declare import declarations generating source code.
Expected Outcome
import javax.persistence.Id should be added during the export.
Current Outcome
No class/method import.
IDE can be used to fix imports.
Steps to Reproduce(Only for bug reports)
https://github.com/trendev/jeddict-issue.git
Jeddict Version(s)
5.4 on NB 11.2 / oracle jdk-8u221-macosx-x64

Uncheck the option Remove Unused Imports and Organize Imports fixes the issue.
Should be closed or improved 😎

Hi @skonx,
I can't reproduce the issue, May you try to clean the system cache or use the new setup: https://github.com/jeddict/jeddict/releases/download/5.4/Jeddict_5.4_with_NetBeans_IDE_11.2.zip
Also, that's a very basic feature without it, Travis build shouldn't have been passed.

Hi @jGauravGupta, those 2 options doesn't work well using jeddict source code generator.
I've tested on a from-scratched Netbean 11.2 & jeddict 5.4 config without any settings expect those one.
I guess the issue should be closed.
Thanks for your quick reply.
NB: if you can point the package in your code, I'll have a look.

Hi @skonx,
Apologies I overlooked it, now I can reproduce the issue after enabling the "Remove Unused Imports" option. Jeddict source generation and IDE auto inspection are not in sync although not a blocker issue.
Thanks for reporting it.

Here(https://github.com/jeddict/jeddict/blob/master/orm-generator/src/main/java/io/github/jeddict/orm/generator/util/ORMConverterUtil.java#L222) source snippet generated from freemarker templates copied to file.


Add Weight and FeeRate newtypes

Use of general-purpose integers is often error-prone and annoying. We're working towards improving it by introducing newtypes.
This adds newtypes for weight and fee rate to make fee computation easier and more readable. Note however that this dosn't change the type for individual parts of the transaction since computing the total weight is not as simple as summing them up and we want to avoid such confusion.
Part of #630
Replaces #1607 (I want to get this in quickly and don't want to be blocked on @DanGould's availability.)

Needs https://github.com/rust-bitcoin/rust-bitcoin/pull/1610

Some other methods that we may want to consider:
For both Weight and FeeRate

to/from_le_bytes
Maybe shift left/right?
ops by various integer types (e.g, Mul<u16>)

For Weight

checked/wrapping/overflowing add/mul/sub

I just looked over the U256 code to see what we might want.

Was going to add to_be_bytes&co but decided not to because the unit is not obvious. In both cases there can be at least two units. Explicit construction is better. A similar argument could be made about serde but I guess that would be too annoying.
Shifts definitely not, this is not a bag of bits.
Multiplication by various integers seems to be a bit odd. No other Rust type has it.
I added checked_ operations since those make sense. Others shouldn't be needed when computing weight.

I'm a bit confused about

the meaning of multiplying or dividing FeeRate by a number. I guess it's to "scale a feerate" but why would one do this? Note that the resolution of integers here really become a problem ... the minimum action you can do here is double the feerate. It may make sense to instead have a dedicated bump_by method or something which takes a floating point and adds 1.0, say.
adding Weight to itself. As I said in #1607 I'm suspicious of this operation and think that it's probably incorrect, and it would be better that users need to do some more gymnastics to compose weights.
multiplying Weight by numbers doesn't seem meaningful except in pretty niche cases (e.g. adding several identical-looking inputs to a transaction)

Yeah, scaling. It is true that it is unlikely, I added it as a "standard impl". Could remove it if you don't like it.
Adding Weight is addressed by transaction parts not returning weight. As I said, I want to write a proper predictor.

adding several identical-looking inputs to a transaction

I happen to have exactly this use case. :)

I have a WIP fee prediction function which is very important for me to get in.  I will try to finish it tomorrow morning if that's OK. If not, I can live with it.

Go for it, no stress, its only a "soft" feature freeze :)

Nice PR, I learned a few things (you'll see my new  issues and comments :)

If we want a link to docs in the Weight file we could use: https://en.bitcoin.it/wiki/Weight_units


v0.1

I am very interested in the first Hackage release of servant-swagger, but I am not sure what keeps it from happening and whether I can help it.
Can we write down the list of things to be done before the first release?

I've been traveling for work. I can release a v0 today if you'd like.
I'd still like to do the following.

generics for swagger models
additional information for parameters. So a class for ToSwaggerParam, ToSwaggerQueryParam.
documentation

In it's current state, servant-swagger is usable and I'm implementing it one of my projects for work.
I will put up a v0.

Yes, v0 would be much appreciated!
Once it's out, I'll try it for my project and probably help you with some of the todos!

It's up http://hackage.haskell.org/package/servant-swagger. This API will
change most likely. I want to add ToSwaggerQueryParam and
ToSwaggerCapture etc. So additional meta data can be captured that the
servant API doesn't include (like enum, and required, etc).
On Wed, Nov 4, 2015 at 9:54 AM, Nickolay Kudasov<EMAIL_ADDRESS>wrote:

Yes, v0 would be much appreciated!
Once it's out, I'll try it for my project and probably help you with some
of the todos!
—
Reply to this email directly or view it on GitHub
https://github.com/dmjio/servant-swagger/issues/1#issuecomment-153771922
.

--
Cell: 1.630.740.8204

Can you also expose .Internal module?
I would like to experiment with servant-swagger within my own project and I'd love to have access to things like e.g. SwaggerRouteDescription which is currently hidden from user.
The specific problem I am having right now is that I have sub APIs and it would be nice to have SwaggerRouteInfo SubAPI defined in SubAPI module (which does not know of SuperAPI) and then be able to use it to make SwaggerRouteInfo SuperAPI. Reality complicates things even further with multiple sub API layers.
Probably this problem should be discussed in a separate issue though.

And I also would like SwaggerAccept to be exposed.

@fizruk there might be some things I forgot to expose, like lenses etc.. Can you submit a pull exposing SwaggerRouteDescription and SwaggerAccept (and corresponding lenses)?
In regards to the SubAPI I ran into the same issue. I have an app that has the following setup:
type API = LeftAPI :<|> RightAPI ....
type LeftAPI = A :<|> B
type RightAPI = C :<|> D

Since SwaggerRouteInfo is more or less the exact same thing as ExtraInfo for servant-docs I think we have to specify that A is in API. And it isn't possible to specify a description for A in LeftAPI.

@dmjio we can select what should be exposed in Servant.Swagger later and simply expose .Internal module since it is The Right Thing To Do™ anyway.
I can probably think of a workaround for sub APIs, but later.

This has been done, closing


Feature request: in-IDE netcheck

Coder provides a coder netcheck command in the CLI to check networking and report possible issues. It would be great to provide a similar feature that's accessible through the IDE extension, so users don't need to run a CLI command to perform these diagnostics. Ideally, the diagnostics would run automatically if the IDE has trouble connecting or maintaining a stable connection (or perhaps on launch or periodically, regardless of detected trouble).

to fix your trouble check this fix, i see it in another issue,
https://github.com/aravinda-gamage/x86_64-win64-ranlib/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Remove "Draft comment area found. " message

If I'm publishing DITA content using draft comments and the "args.draft" parameter enabled, the DITA OT console emits this message for each encountered draft-comment:
  [DOTX040I]: Draft comment area found. To remove this message and hide the comments, build your content without using the DRAFT parameter.

I don't think this message is useful, I have explicitly enabled the parameter, so it just pollutes the console log.

I agree it's not useful on each occurrence, but a single warning could be useful, as it's possible someone else turned on draft comments for your build (say, for a review) and forgot to turn them off.
I wouldn't want not to be warned about draft comments if I thought my deliverable might be ready to deploy!

Actually, I can envision a workflow where this setting is always left on and part of the review process is to remove all the draft comments as they are addressed. In such a workflow, it's possible that "by occurrence" warnings would be useful, and at a minimum the warning should indicate whether any draft comments actually were encountered and not just whether the parameter was set.
One possibilitity would be to add new values to the args.draft parameter:
yes - current "yes" behavior, for backward compatibility
log-each - current "yes" behavior, but with a more accurate description
log-first - includes draft comments, but only log a warning for the first one
log-none - includes draft comments, but doesn't warn about them in the log

The reason for current logging goes back to the early beta days, where there were two issues (both addressed by @shaneataylor above):

Fear that somebody would take draft content, publish with args.draft on because that's what the script does, and unknowingly deploy draft comments
The need to find draft comments or required-cleanup items - errors in the log provided an easy summary of topics that explicitly needed review or cleanup.

Ok, you can close this if you want...

Radu suggested we could close this one over two years ago -- with no updates since then I think it's finally time to pay attention.


fix xla dumps and hlo runner

Can you add a description please? thx!

retest cpu-non-pip please

retest cpu-non-pip please

Can you also push a PR to:
develop-upstream
2.14-rocm-enhanced
2.15-rocm-enhanced.
thx

Can you also push a PR to: develop-upstream 2.14-rocm-enhanced 2.15-rocm-enhanced.
thx

It seems latest xla contains some features. https://github.com/ROCmSoftwarePlatform/xla/blob/69f26cf6a03e9e7fc4c45c138dc3917cc3aa36f8/xla/service/gpu/compile_module_to_llvm_ir.cc#L326

For CI, I found error: RPC failed; curl 56 GnuTLS recv error (-9): Error decoding the received TLS packet. error: 3334 bytes of body are still expected

For CI, I found error: RPC failed; curl 56 GnuTLS recv error (-9): Error decoding the received TLS packet. error: 3334 bytes of body are still expected

yeah - seems that node is having issues. Ive marked it offline for now
retest cpu-non-pip please

Can you also push a PR to: develop-upstream 2.14-rocm-enhanced 2.15-rocm-enhanced.
thx

It seems latest xla contains some features. https://github.com/ROCmSoftwarePlatform/xla/blob/69f26cf6a03e9e7fc4c45c138dc3917cc3aa36f8/xla/service/gpu/compile_module_to_llvm_ir.cc#L326 So other branches don't need these?

2.14-rocm-enhaned still does

It seems something wrong with CI network.

retest gpu-non-pip-multi please

Any update?


Allow rounding parameter at Decimal-based validators

Sometimes it's necessary to define which type of rounding should be used at limiting output places. This could be an additional parameter called rounding which should be used as Decimal.quantize() rounding input.

As I have just found out, the rounding mode in the default context of Python's decimal module is not quite what I (and most people?) would have expected: It uses the ROUND_HALF_EVEN method, which rounds a 5 either up OR down depending on the preceding number, rounding to even numbers. (For example, 1.5 would be rounded to 2, but 2.5 would be also rounded to 2, instead of 3.)
I would suggest to set the default value for the new rounding parameter to decimal.ROUND_HALF_UP. This would be a small breaking change, but I'd argue that for most applications this is actually more a bugfix than a change of expected behaviour.
To restore the default behaviour of using the decimal context, the parameter can simply be set to None.
So in a nutshell:

New parameter rounding: Optional[str] = decimal.ROUND_HALF_UP (apparently these constants are strings)
Without the output_places parameter, this parameter has no effect.
Pass the rounding parameter to decimal.quantize()
The documentation needs to be adjusted accordingly (right now it says the default decimal context is used)

Implemented in #99.


Xamarin AFMobileSample Android Panic

Description
When trying to run the sample Xamarin AFMobileSample Android project, creating a wallet causes a panic to be thrown and execution stops.
To Reproduce
Steps to reproduce the behavior:

Clone the agent-framework repo
Build the AFMobileSample solution and deploy the AFMobile.Android project
Click on the "CREATE WALLET AND DID" MainPage button
In output from debug window, a 'panic' is thrown and execution stops.  No exception is raised in Xamarin

Expected behavior
The sample app uses the default external storage location to store the wallet.
Screenshots
Here is the output from the debug window, just after the button is pressed;
12-23 00:10:41.290 D/Mono    ( 5943): DllImport loaded library 'libindy.so'.
12-23 00:10:41.290 D/Mono    ( 5943): DllImport searching in: 'indy' ('libindy.so').
12-23 00:10:41.290 D/Mono    ( 5943): Searching for 'indy_create_wallet'.
12-23 00:10:41.290 D/Mono    ( 5943): Probing 'indy_create_wallet'.
12-23 00:10:41.290 D/Mono    ( 5943): Found as 'indy_create_wallet'.
12-23 00:10:41.505 I/Choreographer( 5943): Skipped 901 frames!  The application may be doing too much work on its main thread.
12-23 00:10:48.175 E/log_panics( 5943): thread 'unnamed' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }': libcore/result.rs:945

Desktop:

Windows 10 1809
VS Ent 2019 Version 16.0.0 Preview 1.1

Smartphone:

Android Emulator (x86_64)

Additional context
The sample app has READ_EXTERNAL_STORAGE and WRITE_EXTERNAL_STORAGE in the AndroidManifest.xml file, MainActivity.cs has ReadExternalStorage and WriteExternalStorage and the UI permissions prompt is responded to with 'Allow'.
A work around is to provide a StorageConfiguration value in the MainPage, here I'm using the UserProfile storage location;
        async void OnCreateClicked(object sender, EventArgs e)
        {
            ProgressIndicator.IsRunning = true;
            CreateButton.IsEnabled = false;

            WalletConfiguration.WalletStorageConfiguration _storage = new WalletConfiguration.WalletStorageConfiguration { Path = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.UserProfile), ".indy_client") };
            _config.StorageConfiguration = _storage;

            try
            {
                await _walletService.CreateWalletAsync(_config, _creds);
            }
            catch (WalletExistsException)
            {
                //
            }

...

following on from this, a similar situation occurs when attempting to create a pool.  Here is the modified MainPage code;
    public partial class MainPage : ContentPage
    {
        private readonly IPoolService _poolService = App.Container.Resolve<IPoolService>();
        private Pool _pool;

        async void OnCreateClicked(object sender, EventArgs e)
        {
   ...
            try
            {
                var poolGenesisFile = CreateGenesisTxnFile("pool_genesis.txn");
                 await _poolService.CreatePoolAsync("myPool", poolGenesisFile);
            }
            catch (Exception)
            {
            }
           _pool = await _poolService.GetPoolAsync("myPool", 2);
  ...

    public static string CreateGenesisTxnFile(string filename)
        {
            var file = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.UserProfile), ".indy_client", filename);
            var testPoolIp = "XXX.XXX.XXX.XXX";

            var defaultTxns = new string[]{
   ... 4 nodes ...
             };
            Directory.CreateDirectory(Path.GetDirectoryName(file));
            var stream = new StreamWriter(file);
            foreach (var defaultTxn in defaultTxns)
            {
                stream.WriteLine(defaultTxn);
            }
            stream.Close();
            return file;
        }

and the output debug
12-23 01:46:04.322 D/Mono    ( 6367): Searching for 'indy_create_pool_ledger_config'.
12-23 01:46:04.322 D/Mono    ( 6367): Probing 'indy_create_pool_ledger_config'.
12-23 01:46:04.322 D/Mono    ( 6367): Found as 'indy_create_pool_ledger_config'.
12-23 01:46:04.339 E/log_panics( 6367): thread 'unnamed' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 13, kind: PermissionDenied, message: "Permission denied" }': libcore/result.rs:945

Thanks for submitting the issue @infitude. It makes sense that creating pool ledger config will fail as well, since they write to the same location, just different folders.
Just to confirm, are you able to write to the directory location using the .NET APIs?
File.WriteAllText(Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.UserProfile), ".indy_client"), "test");
@tplooker Any ideas what could be causing this?

@tplooker API 27 Android Oreo 8.1 x86_64 Emulator version 27.3.9-4899998 (WHPX 10.0.17763)

@tmarkovski Can you confirm what version we are using for the pre-built binaries for libindy?

https://repo.sovrin.org/android/libindy/rc/1.6.8-45/


create villager textures

We need textures for the following:

[x] Brewer (glass bottles and potions)
[x] pet shop owner (feed, collars)
[x] sage (manuals and books)
[ ] druid (magic plants)
[x] paleontologist (old dirt nerd)
[x] archeologist (think indiana jones)
[x] Redstoner (simple redstone machines)
[ ] necromancer (body parts, undying fragments, dread shards)
[ ] jeweler (jewelry and low value gems)
[x] leatherworker (processed animal parts)
[ ] gourmand (special foods)
[ ] stonemason (chisel and decorators)
[ ] hunter (wild animal parts)
[x] engineer (processed ores and simple machines)
[ ] magician (wizardry mats)
[ ] theurge (thaumcraft and void)
[ ] vagabond (filth and garbage)
[ ] florist (flowers)
[x] archeologist (fossils)
[ ] demolitionist (gunpowder and dynamite)
[ ] hippie (mushrooms)
[ ] geologist (minerals)
[x] tycoon (fuels and railroad)
[ ] schyster (overpriced knickknacks)

we do not need textures for:

[ ] librarian (paper, books, leather, ink, feathers, scrolls)
[ ] fletcher (arrows and bolts)
[ ] cartographer (fast travel scrolls)

revised villager skins.zip

So, we've been betrayed by optifine. It doesn't play nice with secret rooms, so we're scrapping the new model and going back to the original villager model and texture.


WIP Add --prefix to rez-pip install

Hi all,
This PR adds support for --prefix to rez-pip install, for use with releasing a package.
$ rez pip install some-package --release --prefix c:\some\custom\dir

I tried staying as close as possible to rez-build --prefix, however:

Not sure overwriting config.release_packages_path is OK
Should --prefix affect the final path, even without --release?

Superseded by #614


Fix file GitHub branch name

Fix Github branch name to show correct branch (not default master) in  file details page. [OSF-3882]
Fix Github addon file detail page to show the correct branch content, download correct files from actual branch selected (not default master), and no 404 errors when visiting files unique to a non-master branch. [OSF-4611]

:+1:

CR 1 done.

CR2 pass complete. The final move to ready-to-merge will be contingent on Steve approval, as there is one potential edge case in a section of code I'm not deeply familiar with.


Implement the GitHub "go to file" fuzzy match algorithm instead of the VS Code Quick Open algorithm

I always have terrible results with the VS Code Quick Open feature, and excellent results with the GitHub "go to file" feature.
For example, I want to jump to a file djangosite/urls/default.py so I Quick Open and type urldef, but I get this soup in VS Code:

In GitHub with "go to file", I get exactly what I want:

If I try to separate the two fragments with a space url def (and even try reversing them to def url thinking the first fragment might better match the filename and then be narrowed by path), I get better results, but still the best match is 4th down on the list below what appear to be lower quality matches:

Why does a 3 consecutive character match from the start of the parent directory and covering 75% of the parent directory name (urls) not trump 3x single character matches from anywhere inside separate path fragments? The individually matching characters make up a tiny percentage of the overall length of each fragment, and some are not even matching anywhere near the start of the fragment.
This is my experience often with VS Code where I have to ignore and visually scan past a long list of garbage matches to find the one I actually want. Which is infuriating with the GitHub implementation nails it pretty much every time.

I think this is in the realm of this issue: https://github.com/microsoft/vscode/issues/25925
marking as a duplicate.