522 error

hi. im using wstunnel + cloudflare with ssl.  why im getting this error:
failed to do websocket handshake with the server
2024-01-14T19:15:26.306680Z ERROR tunnel{id="018d0962-9c79-7f>
Jan 14 19:15:26 ubuntu-g2-small3-bamdad-1 wstunnel[4073]: Caused by:
Jan 14 19:15:26 ubuntu-g2-small3-bamdad-1 wstunnel[4073]:     Invalid status code: 522
Jan 14 19:15:29 ubuntu-g2-small3-bamdad-1 wstunnel[4073]: 2024-01-14T19:15:29.044866Z ERROR tunnel{id="018d0963-c134-75>
Jan 14 19:15:29 ubuntu-g2-small3-bamdad-1 wstunnel[4073]: Caused by:
Jan 14 19:15:29 ubuntu-g2-small3-bamdad-1 wstunnel[4073]:     Invalid status code: 522
Jan 14 19:15:30 ubuntu-g2-small3-bamdad-1 wstunnel[4073]: 2024-01-14T19:15:30.093408Z ERROR tunnel{id="018d0962-d2ee-76>
Jan 14 19:15:30 ubuntu-g2-small3-bamdad-1 wstunnel[4073]: Caused by:
Jan 14 19:15:30 ubuntu-g2-small3-bamdad-1 wstunnel[4073]:     Invalid status code: 522

Like that I can't tell you much, http 522 means a timeout. Which would mean Cloudflare don't manage to contact your server.
Try with a simple curl, to see if your server receives new connection from yoy through Cloudflare

Recently, I've encountered an issue with Cloudflare on version 9. However, it used to work fine before. Currently, it's working with version 5, but for the past few days, I've been getting 400 and 522 errors with version 9. I need to use Cloudflare because otherwise, the firewall quickly filters my IP.

Like that, I can't tell you. I would need to test with Cloudflare to see

thanks a lot.

I tried with cloudflare this morning, and I haven't got any issues.
I started wstunnel server wss://[::]:443
On cloudflare, I just made sure to enable full encryption

with my domain proxied
be sure to have websocket enabled in the network tab

after that, I was able to start wstunnel client with
wstunnel client -L 'tcp://xxx' wss://my.domain.com

hi. Thank you for your kindness. I used Cloudflare with an Origin Certificate, and I had also saved and used these certificates in the server wstunnel setting (with --tls-certificate and --tls-private-key flags).


Can't train using  coreml-protogen-x3-4 model.

Describe the bug
https://huggingface.co/godly-devotion/coreml-protogen-x3-4/discussions/1
My train file:

The error:

Reproduction
Provided above.
Logs
No response
System Info

diffusers version: 0.9.0
Platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.31
Python version: 3.9.13
PyTorch version (GPU?): 1.12.1+cu116 (True)
Huggingface_hub version: 0.10.0
Transformers version: 4.24.0
Using GPU in script?: <1080TI>
Using distributed or parallel set-up in script?:

Try to use the below link. The model link that you're trying to train is not suitable for training.
https://huggingface.co/geocine/protogen-diffusers


Suppress the removal compilation warning

Description
Suppress the removal compilation warning when using Spring Boot 3.4.0.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

Tracking upgrading in https://github.com/Azure/azure-sdk-for-java/issues/43101


update function duplicates the meter

Hello i am using
gauge.update({ units: 'Miles' });
to update the units to Miles from kilometer on button click and what i found now two needles on the screen it seems this meter is duplicating

Please, state the browser and OS you're using.
Also, please, post an entire code example, which reproduces the problem, because on my environment everything works as expected if to follow the steps you've described above.
Thank you in advance.

Hi
Its not happening with every graph, its actually with speedometer only, by
the way i changed my chart. Thanks for your prompt response.
On Fri, Mar 3, 2017 at 8:41 PM, Mykhailo Stadnyk<EMAIL_ADDRESS>wrote:

Please, state the browser and OS you're using.
Also, please, post an entire code example, which reproduces the problem,
because on my environment everything works as expected if to follow the
steps you've described above.
Thank you in advance.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Mikhus/canvas-gauges/issues/112#issuecomment-283978983,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIEZNhH4hDkX0NL2n6yVVhRZ56BYTxfVks5riC2QgaJpZM4MRyff
.

--
Shubham Pareta
(INDIA)


Bagnon uses a new API for integrating plugins

And this addon is not compatible with that API anymore, so it causes errors.

I believe this is a duplicate of #2 and resolved by PR #3

Fixed


There's no page at this address

After reInstall app, I see this message
There's no page at this address
Check the URL and try again, or use the search bar to find what you need
All ok locally, with ngrok tunnel.
Steps to reproduce the problem

Create Node App
Do simple changes
Deploy to hosting (I used Digitalocean)
Make changes on App Settings page(change URL and redirect URL to hosting urls)
Install app to Development store
Delete the app from the Development store
Repeat  Install app to Development store
Get this error.

I tried submitting this app. And QA engineer wrote:
Your app hasn't implemented authentication through OAuth correctly and can't be reinstalled. Review our OAuth documentation.
Node app use koa-shopify-auth - as I see the app have AccessToken, Oauth working.
What means hasn't implemented OAuth correctly?
I think Shopify doesn't delete some app data from shop on app deletion.

Install app - ROUTE ACTIVE_SHOPIFY_SHOPS {}
Open app -  ROUTE ACTIVE_SHOPIFY_SHOPS { 'trzdevelopment.myshopify.com': 'write_themes,write_script_tags' }
Remove app - ``

┃ Shopify.Webhooks.Registry.register response {
┃   success: true,
┃   result: {
┃     data: { webhookSubscriptionUpdate: [Object] },
┃     extensions: { cost: [Object] }
┃   }
┃ }

I have the same issue and my app was rejected due to this problem :( any solutions please?

I got the same issue :(

Hello @paulomarg, thanks for your reply.
As I wrote above base app working locally, with ngrok.
If I create a base app, without any changes. I can install/delete it many times. I used my app few days and everything was fine.
After deployment to hosting, after adding the app link to the app setting on Shopify Partners Dashboard I'm starting to see this error.

You can try shopify deploy heroku

Sorry, I missed the part where this was happening after deploying.
The scenario where you delete / reinstall quickly is most likely failing because the webhook wasn't called before you reinstalled the app - as I mentioned it can take a little time for the webhook to actually reach your app, as Shopify needs to process the event and fire the request.
The other scenario, where it fails for a few hours, sounds like a case where the webhook handling is failing in your app, and Shopify is retrying it. You can read more about that in our documentation on webhooks. Shopify will retry failed webhook calls once every roughly 2h to see if they go through.

Ok, why webhook not working in the base NodeJS app, after deploying? How I can prevent this? Which code missed in the current version of shopify-node-app?

@paulomarg  I saw /webhooks called, but my await Shopify.Webhooks.Registry.process(ctx.req, ctx.res) can't be resolved. Is there anything I missed ?

@paulomarg  I saw /webhooks called, but my await Shopify.Webhooks.Registry.process(ctx.req, ctx.res) can't be resolved. Is there anything I missed ?

@paulomarg In the tutorial, it's a specified API version, but in my code, I can't do that because Typescript warning, so I don't add that property and register still successfully, Do it affect?
I saw /webhooks called, but my await Shopify.Webhooks.Registry.process(ctx.req, ctx.res) can't be resolved. Is there anything I missed ?
Tutoiral:

My code:

Warning Typescript:

@paulomarg In the tutorial, it's a specified API version, but in my code, I can't do that because Typescript warning, so I don't add that property and register still successfully, Do it affect?
I saw /webhooks called, but my await Shopify.Webhooks.Registry.process(ctx.req, ctx.res) can't be resolved. Is there anything I missed ?
Tutoiral:

My code:

Warning Typescript:

I feel like Shopify.Webhooks.Registry.process is not firing the webhookHandler which was given in afterAuth().
So, this was my possible work around and my app was approved by shopify review today.

@Sanj718 hello,
I tried your solution in two projects, but still seeing this error.

@SerhiDi hello, the problem is that your shop is not beeing deldted from active_shopify_shops. If you restart your server in heroku installation will work again since active shops const will be empty. So, can you try to console log something befores Shopify.Webhooks....Process? And see if webhook is beeing triggered via heroku logs? It should trigger after 5 seconds you have deleted the app.
If yes you shoul be able to find shop name in incoming request as i did.

@Sanj718, thank you.
I added your code example and restart my remote server ().
      const shop = ctx.request.header['x-shopify-shop-domain'];
      delete ACTIVE_SHOPIFY_SHOPS[shop];

      console.log(`Webhook processed, returned status code 200`);

After app deletion, I see this log:
`Webhook processed, returned status code 200``
I tried to reinstall the app a few times, all working now. I will try to send my app to review again.
@paulomarg, can you check this code example?
And make the appropriate changes in the shopify-app-node boilerplate if they are needed.

Note: While writing point 4 above, something came to mind: whenever you reload your server, the Shopify.Webhooks.Registry object is reset, so it won't be able to process webhooks until the corresponding register call is made. So, if you're testing webhooks, please make sure to go to /auth?shop=... before trying to uninstall the app to make sure the handler has been loaded into the registry.

@paulomarg As I understand if the server restarts all information about webhooks will be deleted. What if the server goes into maintenance or restarts for other reasons?
The only thing that comes to mind:  We should fetch all stores from DB and reregister webhooks again. Is this the only way?

webhookHandler seems never called by Shopify.Webhooks.Registry.process

Same issue here, @Sanj718 solution worked for me thank you!

Hello there
I got same error during reinstall app
I am working laravel shopify app
please help me

Whats the resolution for this this issue where reinstalling apps under development fails with "no page at this address"?

Whats the resolution for this this issue where reinstalling apps under development fails with "no page at this address"?

You have to re-register your webhooks everytime you reset your server.

hi there, I was trying to activate the paypal in shopify from the payments section but it gives me this:Check the URL and try again, or use the search bar to find what you need.
and i want from you a solution plz help me and thank you very much

I see the issue come from, when you uninstalled the app webhook not remove session from database and the issue come from.
Solution: try to rollback all sessions and restart you server

I feel like Shopify.Webhooks.Registry.process is not firing the webhookHandler which was given in afterAuth(). So, this is my possible workaround and my app was approved by shopify review today.

@Sanj718  i have app in express js i am getting the same Error can answer on it


Sanitize and escape html

Fixes #44

My understanding is that in general we will jsoup.clean (sanitize) but not escape. If you escape, you're going to see
some text
 everywhere.
Why not the description field?
I notice the StackOverflow client is excluded here. Are there separate changes for those in another PR, or what's the status of that?

3. I notice the StackOverflow client is excluded here. Are there separate changes for those in another PR, or what's the status of that?

I don't remember there ever being nested tags in either W3/GeeksForGeeks in a non-html context (on queries tested so far/sifting through the site).
Same here regarding tags in the descriptions.
It was my understanding that escaping/stripping regarding SO was being done in #63, as per the latest comments.

It's safe to clean/escape things that aren't HTML too. Whether you do so or not should depend on your intent - is the string you're considering untrusted input? In this case, all fields of the Card come from the wild (scraped websites), so they're untrusted, so you'll need to clean or escape everything, regardless of whether you expect there to be HTML or not. If you don't do so, then it's an avenue for a XSS attack. (Maybe you've never seen there being tags in the W3Schools description field, but if that causes you not to sanitize it, then a malicious actor with control of the W3Schools website could carry out an XSS attack on you.)
For SO - noted.
Still confused on why escaping is being done, though - I thought we would only do that if we know the field contains HTML code (as a hack to allow users to search for HTML code).

It's safe to clean/escape things that aren't HTML too. Whether you do so or not should depend on your intent - is the string you're considering untrusted input? In this case, all fields of the Card come from the wild (scraped websites), so they're untrusted, so you'll need to clean or escape everything, regardless of whether you expect there to be HTML or not. If you don't do so, then it's an avenue for a XSS attack. (Maybe you've never seen there being tags in the W3Schools description field, but if that causes you not to sanitize it, then a malicious actor with control of the W3Schools website could carry out an XSS attack on you.)
For SO - noted.
Still confused on why escaping is being done, though - I thought we would only do that if we know the field contains HTML code (as a hack to allow users to search for HTML code).

Gotcha.
After our meeting, I was under the impression that detecting html before escaping was irrelevant and implies the use of regex, which we wanted to rule out (wasn't aware of that hack though, which is a good idea).

I hope I wasn't misleading. Using regex to parse HTML is indeed imperfect, and I recommended not using it to determine whether to sanitize because 1) it's imperfect and 2) the regex engine is less efficient than just sanitizing it anyway, which is a simple (ish) O(n) sweep.
Yes, the hack I think we're doing by escaping HTML when we detect there may be HTML code (the simplest way to do this is probably just checking if the query contains "html"?) does open us up to XSS attacks and is Not Great. But the hope is to protect ourselves in most cases (e.g. when the query does not contain "html") and make sure the app is still usable for HTML queries.

@ iris and I when ready for review!

This PR contains the GeeksForGeeks test which doesn't seem intentional.
My understanding was that you would only escape the code if the query contained 'html' or whatever heuristic is being used, but I see that most fields are unconditionally both sanitized and escaped.

Are you planning to add the other tests in a different PR?

Are you planning to add the other tests in a different PR?

The GeeksForGeeks tests? They're in #78

Are you planning to add the other tests in a different PR?

The GeeksForGeeks tests? They're in #78

I meant the escaping tests for W3Schools

Are you planning to add the other tests in a different PR?

The GeeksForGeeks tests? They're in #78

I meant the escaping tests for W3Schools

Yes, these should be in a separate PR.

Are you planning to add the other tests in a different PR?

The GeeksForGeeks tests? They're in #78

I meant the escaping tests for W3Schools

Yes, these should be in a separate PR.

Can you create an issue for this?


rename credentials fields

Signed-off-by: James Talton<EMAIL_ADDRESS>

/hold

There are these 3 fields in vmware need to change
"vcenter":              "vCenter",
"vmClusterName":        "cluster",
"datastore":            "defaultDatastore",

And also I am not seeing osServicePrincipal.json in PR

/approve

/unhold


pre-commit hook for cargo clippy fails during local development

Describe the bug
When running the following commands locally:
pdm install
pdm run pre-commit install
git add .
git commit -m 'test'

I get this output:
check for case conflicts.................................................Passed
check for merge conflicts................................................Passed
check toml...............................................................Passed
check yaml...............................................................Passed
fix end of files.........................................................Passed
trim trailing whitespace.................................................Passed
ruff.....................................................................Passed
ruff-format..............................................................Passed
cargo fmt................................................................Passed
cargo clippy.............................................................Failed
- hook id: cargo-clippy
- exit code: 101

   Compiling pyo3-build-config v0.20.3
error: failed to run custom build command for `pyo3-build-config v0.20.3`

Caused by:
  process didn't exit successfully: `/Users/florian/git/deptry/target/debug/build/pyo3-build-config-3314ceeaf5b25712/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-env-changed=PYO3_CONFIG_FILE
  cargo:rerun-if-env-changed=PYO3_NO_PYTHON
  cargo:rerun-if-env-changed=PYO3_ENVIRONMENT_SIGNATURE
  cargo:rerun-if-env-changed=PYO3_PYTHON
  cargo:rerun-if-env-changed=VIRTUAL_ENV
  cargo:rerun-if-env-changed=CONDA_PREFIX

  --- stderr
  error: failed to run the Python interpreter at /Users/florian/git/deptry/venv/bin/python: No such file or directory (os error 2)

It seems to look for the Python interpreter under venv, while it is located under .venv. The following command works fine:
pdm run pre-commit run -a

Maybe other contributors will run into the same. Let's find out why this is happening and what we should do to resolve it.

Closing, maybe it was just an issue with my environment. Not sure what caused it and what resolved it. If in the future someone else encounters the same, let's see if we can get a better list of steps to reproduce than I provided (i.e. all steps starting from a git clone).


Add documentation site redirect from "bashcomplete" to new completions page

The google top result for "click shell completions" is https://click.palletsprojects.com/en/7.x/bashcomplete/. However, when I click from that page on the "show me the 8.x/latest version of this doc" link I'm given a 404.
Give that this link is quite popular, it'd be great if the "bashcomplete" page on the 8.x docs existed, and redirected to the main shell completions doc.

Seems to be impossible, Read the Docs won't redirect /bashcomplete/, only /bashcomplete (no trailing slash), but Sphinx generates links with trailing slashes. I'll report it to RTD, but this should eventually solve itself as Google starts preferring results for the newer version.

Never mind, it does seem to be working now.

Thanks for the update @davidism!


Layout updates 3

Codecov Report

Merging #35 (ae8b43d) into main (acee7c8) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   26.98%   26.98%
=======================================
  Files          55       55
  Lines         630      630
  Branches      133      133
=======================================
  Hits          170      170
  Misses        408      408
  Partials       52       52

Impacted Files
Coverage Δ

src/components/analytics/analytics-card.js
8.57% <ø> (ø)

src/components/automation-hub/hub-card.js
5.88% <0.00%> (ø)

src/components/catalog/catalog-card.js
6.81% <ø> (ø)

Continue to review full report at Codecov.

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


Release v5.1.1

Updated dependencies
Refactored some module

Coverage decreased (-0.05%) to 97.06% when pulling 2aad57c95a1549be7cdfcdf0b075787378626a6d on develop into 44bacd6304ac0a1e2251e1c72ad119d685281fb4 on master.

Coverage decreased (-0.05%) to 97.06% when pulling a1a8aa629fff209a08740747e7545a185dd3800d on develop into 44bacd6304ac0a1e2251e1c72ad119d685281fb4 on master.

Coverage increased (+0.22%) to 97.33% when pulling 8e5bc6da5f89d08b8a2db5568369f23481991e48 on develop into 44bacd6304ac0a1e2251e1c72ad119d685281fb4 on master.

Coverage increased (+0.22%) to 97.33% when pulling 8e5bc6da5f89d08b8a2db5568369f23481991e48 on develop into 44bacd6304ac0a1e2251e1c72ad119d685281fb4 on master.

Coverage increased (+0.22%) to 97.33% when pulling 1848ef0f0a1c9726de744cae6f7060b8a9fc63b3 on develop into 44bacd6304ac0a1e2251e1c72ad119d685281fb4 on master.

Coverage increased (+0.22%) to 97.33% when pulling 1848ef0f0a1c9726de744cae6f7060b8a9fc63b3 on develop into 44bacd6304ac0a1e2251e1c72ad119d685281fb4 on master.


[Feature]:  New Actions requirement for Datepicker Widget

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Feature Requirement Proposal
For the Datepicker Widget, we have one action "onDateSelected".
We may need "onMonthSelected" and "onYearSelected" as there are issues in certain scenarios
This requirement is based on a issue faced by a Discord user
https://discordapp.com/channels/725602949748752515/940981446338809886
Feature Requirement Summary :
---Not Finalised --
Why should this be worked on?
The need happens when we want to display the selected date in another place.
Part A
(1) User Displays the selected date in a label/textfield using the 'onDateSelected" action script.
(2) User navigates to the next month and selects somewhere outside, the month gets changed in the datePicker.
(2.1) The date in the label/textfield remains unchanged, so a mismatch is occurring
(3) Similarly when Year is changed the mismatch between the datepicker date and the display date in a label/textfield occurs.
Considering the above scenario we need the implementation.
Alternatively
Part B
(1) To make mandatory to select the modal calendar, else the calendar may not disappear.
(2) Do not change the Month or Year in the datepicker unless a date is selected.
TODO :
To decide which option are feasible and we could go ahead (Part A or Part B or a combination).
To write down the conclusion in the "Feature Requirement Summary" above.

Hi, as discussed on the Support channel, as long as the date displayed is different than the date if the DatePicker.SelectedDate variable, this is a bug, that can lead to a new feature by primarily a bug.

Hey @leotom2000 thanks for creating this issue. I think the solution should be to only update the date when a date is selected and not when a user is browsing the calendar.
@somangshu @dilippitchika can we get this small issue solved quickly?

@Tooluloope can you help us triage this?


Add prometheus annotation to scrape metrics from queue-proxy

/kind feature
Describe the solution you'd like
Need following annotations on InferenceService so that prometheus can scrape metrics from  revision pods.
prometheus.io/scape: true
prometheus.io/port: 9091

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

I want to collect inferenceservice metrics from kfserving for monitoring. I notice that, there is a yaml promethues-patch. Now I have known the promethus' address, how to config kfserving can make me collect these metrics? BTW, is there haved a list file for definition the metircs' name?

Completed via

#2459

/close


verify.sh in 4.5.1 reporting TEDAPI Vitals Capabilities Incorrectly

Just updated from 4.3.2 to 4.5.1. You all have been busy!  Got TEDAPI connection working to get back vitals & alerts (yes!)  After install, verify.sh reports " - TEDAPI Vitals: Not Connected", however docker logs pypowerwall reports "[INFO] TEDAPI Mode Enabled for Device Vitals (hybrid)".
Looks like in pypowerwall stats the value "pw3" was added between "tedapi" & "tedapi_mode", which messes with the sed command. I believe the line
TEDAPI=curl --silent http://localhost:$PORT/stats | sed 's/.*"tedapi": \(.*\), "tedapi_mode".*/\1/' 2>/dev/null
should be
TEDAPI=curl --silent http://localhost:$PORT/stats | sed 's/.*"tedapi": \(.*\), "pw3".*/\1/' 2>/dev/null

Great find, @SCHibbard  - thank you!


ddl: add expression index support

What problem does this PR solve?
Support creating expression index.
According to https://dev.mysql.com/doc/refman/8.0/en/create-index.html,
Expression part can't be used in primary index、 fulltext index and foreign key.
What is changed and how it works?
Modify yacc to support expression part, rename IndexColName to IndexColNameWithExpr to hold ExprNode, and add some test.
And add error code related to expression index:https://dev.mysql.com/doc/refman/8.0/en/server-error-reference.html
Check List
Tests

Unit test

Code changes

Has exported variable/fields change

Side effects
Related changes

mysql> create table t (a int, index ((a)));
ERROR 3762 (HY000): Functional index on a column is not supported. Consider using a regular index instead.
mysql> create table t (a int, index ((d)));
ERROR 3762 (HY000): Functional index on a column is not supported. Consider using a regular index instead.

This error should reported by parser.

Why should be reported by parser？ It seems that this is not a parser error in MySQL

This error report by TiDB is also acceptable.

The variable naming need more discussion, rest LTGM


Lib throws when encounters a Git repo's pubspec.yaml without a version

Yes, apparently it can happen. I had a pubspec.yaml file in my .pub-cache/git that did not have a version.
It was async_await:
name: async_await
description: Compile-time support for "async/await" syntax in Dart.
dependencies:
  analyzer: '>=0.22.4 <0.23.0'
  barback: '>=0.15.0 <0.16.0'
dev_dependencies:
  http: any
  unittest: any

Which throws this error:
Uncaught Error: Invalid argument(s)
Stack Trace:
#0      _JSSyntaxRegExp.firstMatch (dart:core-patch/regexp_patch.dart:142)
#1      Version.Version.parse (package:pub_semver/src/version.dart:111)
#2      GitDirectoryPackageRef.GitDirectoryPackageRef (package:pub_cache/src/impl.dart:119)
#3      PubCache._parse.<anonymous closure> (package:pub_cache/pub_cache.dart:121)
#4      MappedIterator.moveNext (dart:_internal/iterable.dart:390)
#5      List.addAll (dart:core-patch/growable_array.dart:183)
#6      PubCache._parse (package:pub_cache/pub_cache.dart:122)
#7      PubCache.PubCache (package:pub_cache/pub_cache.dart:41)
#8      _doLoadWhenSnapshot (package:dartdoc/resource_loader.dart:38)
#9      _doLoad (package:dartdoc/resource_loader.dart:24)
#10     loadAsString (package:dartdoc/resource_loader.dart:16)
#11     main.<main_async_body> (file:///Users/sethladd/Code/dartdoc/bin/dart_resource_loader.dart:4)
#12     Future.Future.microtask.<anonymous closure> (dart:async/future.dart:144)
#13     _asyncRunCallbackLoop (dart:async/schedule_microtask.dart:41)
#14     _asyncRunCallback (dart:async/schedule_microtask.dart:48)
#15     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96)
#16     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:143)

#0      _rootHandleUncaughtError.<anonymous closure> (dart:async/zone.dart:886)
#1      _asyncRunCallbackLoop (dart:async/schedule_microtask.dart:41)
#2      _asyncRunCallback (dart:async/schedule_microtask.dart:48)
#3      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:143)

Suggest we tolerate pubspec's without versions in them.

This might do it: https://github.com/dart-lang/pub_cache/pull/18

pushed the fix as 0.0.1+1


Normalize file-config cm keys

Kubernetes does not allow / characters in keys

/run in-cluster tests


pkg/ipcache: initialize globalmap at import time

globalMap can be initialized at import time so there's no really need
to call InitIPIdentityWatcher to initialize this map.
Signed-off-by: André Martins<EMAIL_ADDRESS>Avoid panic of uninitialized variable when running with clustermesh

This change is

test-me-please

Coverage decreased (-0.07%) to 41.751% when pulling 3cfbe57b31b6cd09c72b6b7eb0c4fd721755e877 on pr/fix-globalmap-init into 783942ba8dfdef0f4cf1b7fbeed3bbd5c5185edd on master.


Error while running tests

Running jest tests fails, requires babel plugin lodash - but it's not installed in my project
should it be either peer dependency?
Test suite failed to run:
Plugin 0 specified in "/fez-frontend/node_modules/react-google-maps/package.json" provided an invalid property of "wrapped"

non-issue


Agisoft depth

I read your paper in detail these days, and am very interested in it. I noticed you used Agisoft's depth as your reference. I want to know the dark-purple part of depth map is the massive artefacts or the mask.

Thank you for your patience!🥰

Hi,
Thank you for your interest in my work :)
The purple part is where Agisoft was not able to predict the depth. We didn't use masks.

OK, got it! Thank you so much🎉


Feat/add azure blob storage support

Add Support for Azure Blob Storage
Following the code and tests of the Micronaut implementation of the aws S3-client, I've tried to implement something similar for Azure.
Problem

The crud test in class BlobCustomerControllerTest fails and I was not able to solve the issue

If debugged, it fails because of a timeout of the HTTP-Controller that makes the request
If run, it fails after successfully looking up the Azure Storage Account Client with the following error messages

Message: block()/blockFirst()/blockLast() are blocking, which is not supported in thread default-nioEventLoopGroup-2-3
Path Taken: new $CustomerController$Definition$Intercepted(CustomerRepository repository,BeanResolutionContext $beanResolutionContext,BeanContext $beanContext,Qualifier $qualifier,List $interceptors,InterceptorRegistry $interceptorRegistry) --> new $CustomerController$Definition$Intercepted([CustomerRepository repository],BeanResolutionContext $beanResolutionContext,BeanContext $beanContext,Qualifier $qualifier,List $interceptors,InterceptorRegistry $interceptorRegistry) --> new $CustomerRepositoryStoreImpl$Definition$Intercepted([RootProvider rootProvider],BeanResolutionContext $beanResolutionContext,BeanContext $beanContext,Qualifier $qualifier,List $interceptors,InterceptorRegistry $interceptorRegistry) --> new DefaultRootProvider([StorageManager storageManager])
io.micronaut.context.exceptions.BeanInstantiationException: Error instantiating bean of type  [io.micronaut.eclipsestore.DefaultRootProvider]

[default-nioEventLoopGroup-2-3] TRACE i.m.h.s.netty.body.NettyByteBody - Body claimed at this location. This is not an error, but may aid in debugging other errors
java.lang.Exception: null

The client needs to be defined explicitly here because creating the Blob-container in the Azurite-Docker-container is a blocking call and if moved to AzureBlobLocalClient.buildClient() results in an error (for the S3 tests it somehow works).

Thanks this looks great! Could you target the 1.6.x branch since this is a new feature.

I've switched the target branch.
Do you have any guess what could cause this blocking call?
The HttpClient is a BlockingHttpClient and I don't see why the nioEventLoopGroup would fail.
The blocking call that I had to move out of the bean instantiation was also leading to the same error.
According to the error logs when running ./gradlew build
BlobCustomerControllerTest > testCrud(String) > [1] store FAILED
    io.micronaut.http.client.exceptions.HttpClientResponseException at DefaultHttpClient.java:2259

But when I just run the BlobCustomerControllerTest it seems to fail somewhere in this already existing function with
Caused by: java.lang.IllegalStateException: block()/blockFirst()/blockLast() are blocking, which is not supported in thread default-nioEventLoopGroup-2-3

I think https://github.com/micronaut-projects/micronaut-eclipsestore/blob/1.5.x/test-suite/src/test/java/io/micronaut/eclipsestore/docs/CustomerController.java#L23
Needs to be annotated with @ExecuteOn(TaskExecutors.BLOCKING)

Thanks!
Local, all the checks and the build process pass

Is it possible to add this PR to the next 4.6.0 release?
@SimonFrauenschuh thank you for your contribution!

it is scheduled for 4.6 yes


Remove AdditionalRunFuncs from base controller

What this PR does / why we need it:
This is no longer required as there is a With function on the pkg/controller.Builder structure that can be used to trigger additional things that should be run alongside a controller (which is used in the acmechallenges controller to run the challenge scheduler, and the webhookbootstrap controller to continuously ensure the webhook TLS assets are up to date).
Removing this option to a) simplify things a bit and b) make it easier/cleaner to start controllers during integration tests.
Release note:
NONE

/kind cleanup

/retest

/lgtm

Requires #2743
/hold

/hold cancel


Update virtual-machines-disks-getting-started-ultra-ssd.md

Remove all the dollar signs preceding shell variants in the samples of Azure CLI.

@rioriost : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rioriost Thanks for helping to improve our docs, we appreciate you taking the time to contribute to the platform. :)
Everything looks good so:
#sign-off


Not compatible with BS4

Hi Folks,
I upgraded the application from Bootstrap 3 to 4. Facing UI issues after upgradation. Is there any plan for new release with BS4?

plans? sure, but bs4 is in alpha so I'm not in a hurry

Bootstrap 3 'table-condensed' class has been replaced by 'table-sm'.
I added below styles to fix UI issue.
.table-condensed th, .table-condensed td {
padding: .3rem;
}

Hey @Eonasdan - I would love some Bootstrap 4 support as well. Currently I run the package under Bootstrap 4, but its just minor style issues (i.e. class renames etc) that is an issue. Using the css snippet from @anikets43 solves most of the styling issues.

@canfiax Glad to know it helped you!!
Glyphicons are not included in BS4, you can customize the icons in the configuration. Something like below:
`icons: any = {
time: 'fa fa-clock-o',
date: 'fa fa-calendar',
up: 'fa fa-angle-up',
down: 'fa fa-angle-down',
previous: 'fa fa-chevron-circle-left',
next: 'fa fa-chevron-circle-right',
today: 'fa fa-calendar-check-o',
clear: 'fa fa-trash-o',
close: 'fa fa-times-circle'
};
dateFormat: any = {
    'date': {
        format: 'MM/DD/YYYY',
        icons: this.icons
    },
    'datetime': {
        format: '',
        icons: this.icons
    },
    'time': {
        format: 'LT',
        icons: this.icons
    },

};`

Thanks @anikets43 - Can you show some more code examples of how to use the dateFormat variable? And why use LT instead of HHss in format?

The css and icons fixed the issues that I ran into, thanks @anikets43
Here is a jsfiddle of those two changes https://jsfiddle.net/wrLncr06/


new Database() in Web Worker freezing

What happend
adding const db = new Database('test.db') in worker.ts makes process freezing
Environment info
deno 1.40.5 (release, x86_64-apple-darwin)
v8 12.1.285.27
typescript 5.3.3

sqlite version  3.42.0
Macbook Pro 2020
Minimal example
Here is the minimal reproducible example
main.ts
const worker = new Worker(import.meta.resolve("./worker.ts"), {
  type: "module"
});

// Learn more at https://deno.land/manual/examples/module_metadata#concepts
if (import.meta.main) {
  worker.postMessage({
    msg: "msg1"
  });
  console.log("msg posted");
}

worker.ts
import { Database } from "jsr<EMAIL_ADDRESS>const thisWorker = self as unknown as Worker;
const db = new Database("test.db");

self.onmessage = (e) => {
    const { msg } = e.data;
    console.log(`received ${msg}`);
    thisWorker.postMessage({
        msg: `callback for ${msg}`
    });
    thisWorker.close();
  };

run code with the following command
$ deno run -A --unstable-ffi main.ts

then only msg posted is shown on the screen, and process just hang, expecting shown received msg1 and exiting as usual.

I think this might be because this module loads asynchronously, and by the time you postMessage to that Worker, module hasn't really evaluated by then. This is because the dlopen we use from plug module is async and we use top-level await to resolve it. Try posting a message from the worker itself sometime after the import statement for sqlite module, and only when that is received on main thread, you can start sending messages.

tks, I think got it, closing this issue

after new Worker('worker.ts') being evaluated, worker.ts starts runing asynchronously.
but new Database() just takes time, onmessage hasn't been registered by the time.
main.ts starts to postMessage, since there is no onmessage registered in worker.ts, there wont be any reponding

thanks for your detailed explanation.  I think the way that put listener on worker.ts's ready message in main.ts and await it(by new Promise())  may be flawed, cuz by the time onmessage is registered in main.ts,  worker.ts may have already been evaluated, there won't be any ready message after that.
Here I have draw a chart showing my concerning
By chart
As what's showing in this chart,

worker.ts being executed in another thread (Thread A), total evaluation time is  b
main.ts being executed in deno's Main Thread, the evaluation time between new Worker()  and onmessage is a

As they being executed in separated thread in parallel, time a can be greater than time b, in this situation, the efforts done for waiting worker.ts ready are just wasted.

Flawed Code Example
Here is a flawed code example, I use sleep for simulating main thread's slowness.
main.ts
const sleep = (ms: number) => new Promise(r => setTimeout(r, ms));

const worker = new Worker(import.meta.resolve("./worker.ts"), {
  type: "module"
});

// simulate main thread slowness
await sleep(200);

const worker_ready = () => new Promise<void>((resolve, _reject) => {
  const handleEvnet = (e: MessageEvent) => {
    const { msg } = e.data;
    console.log(`main.ts received msg: ${msg}`);
    if (msg === "ready") {
      resolve();
      worker.removeEventListener("message", handleEvnet);
    }
  }
  worker.addEventListener("message", handleEvnet);
});

// make sure worker ready
await worker_ready();
// then post some msg to worker
worker.postMessage({ msg: "main.ts msg"});

worker.ts
const thisWorker = self as unknown as Worker;
thisWorker.onmessage = e => {
    const { msg } = e.data;
    console.log(`worker.ts received msg: ${msg}`);
    thisWorker.close();
}

thisWorker.postMessage({ msg: "ready"});

Conclusion
As what's showing above, the effectiveness of awaiting worker's ready msg in main thread, relies on the gap of evaluation time, which  may be flawed.
There should be a way that we can register initial ready listener for new Worker(),  or maybe I missing something important, the conclusion is just wrong, lol.


Fix broken link

Hi team 👋, I've fixed a broken link. I would be happy if you could review this PR.

I signed it!


Switch Pro Controller errors with SDL and DSU (BetterJoy)

Dolphin version?
5.0-17269
Operating System?
Windows 10
Describe the bug.

[ ] In the JSONs, under the DSUClient/0/BetterJoy entry, the "Share" key is mapped to the "-" button texture. BetterJoy outputs a "Share" press when the Capture button is pressed, and "Touch" when the - button is pressed.
[X]  In the JSONs, the SDL entry for the Pro Controller is listed as "SDL/0/Switch Pro Controller". Dolphin recognizes the Pro Controller as "SDL/0/Nintendo Switch Pro Controller".

I just encountered this issue as well. The best solution I've come to is opening all the JSON files for all the games you wish to play in Notepad++ and you can replace all appearances of "SDL/0/Switch Pro Controller" with "SDL/0/Nintendo Switch Pro Controller" simultaneously. Make sure to save all the files after editing but that has worked for me so far.

maybe this is due to the revision of the controller, i will just write both in the pack json.

SDL/0/Switch Pro Controller should work now
https://github.com/Venomalia/UniversalDynamicInput/archive/refs/heads/main.zip

fix https://github.com/Venomalia/UniversalDynamicInput/commit/f86158ddad2eda3e0f200b9f955ce9adf1d70e63


Unable to connect to postgresql

[ ] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
I am using the docker instance from https://hub.docker.com/r/amancevice/superset/
Version: 0.20.5
Expected results
Connect to postgresql dataset
Actual results
Not possible
Steps to reproduce
I have an RDS postgres instance, when I try to test the connection using the SqlAlchemy url like this
postgresql+psycopg2://user:pass@host:5432/myTable
I have the following error:
Connection failed!↵↵The error message returned was:↵'NoneType' object has no attribute 'get_password_masked_url_from_uri'

Please post the relevant exception is your logs. Otherwise there's not much we can do.

Also good chances it's a dup of #3799

Yes think so, it could be a dupe, is the same error, weird that I did not find it before, was searching for this error on issue tracker

The close this one and post whatever you have in your superset logs to the other.

It depends on where you log them, by default are not written to a file. If you are starting it with system you have them in the journal for the unit.


Sort by string.toLocaleLowerCase

The sorting behavior for strings is case sensitive. I'd like to be able to sort by string.toLocaleLowercase.
sortValueObj requires a number as its return type, so there isn't a convenient way to do this, unless I'm missing something obvious.

heya @joeleaver, I think it's a good suggestion! There are 3 ways to do it that I propose:

Have sortValueObj to not only be able to return number, but also string, so string | number, or
In sortValueObj, have some kind of key called __global that will apply the modifier to all columns (so we don't have to do (1) for all keys), or
Make sortValueObj not only able to receive a Record<string, Function> but make it also Record<string, Function> | SortValueFunction. This SortValueFunction is something like (key: name) => string | number. So, we can do something like:

sortValueObj: (key, value) => {
  if (key === 'date') return parse(`${value}`, 'MMMM dd, yyyy', new Date()).getTime()
  return value.toLowerCase()
}

What do you think? I'm thinking maybe option 3 is better but the key name would be a bit misleading since it can also accept a function. If we go with 3, maybe I'd consider creating another field with exact same type, but the name is sortValueGetter (and deprecate sortValueObj).

Or maybe we can just simply check if the column value type is a string, then make it lowerCase() there so no external process is required.
https://github.com/imballinst/react-bs-datatable/blob/9e37868601a2d58451797cd8be125b6a250bd596/src/helpers/data.ts#L25-L26

hi @joeleaver, could you try v3.13.1 and see how it goes with your use case? https://codesandbox.io/p/sandbox/distracted-thunder-nx87tq?file=%2Fsrc%2FApp.tsx%3A77%2C40
sortProps={{
  columnValueProcessor: (key, row) => {
    const value = row[key];
    if (key === "name") {
      return value.toLowerCase();
    }

    return value;
  },
}}

This seems to work just fine, thanks for implementing it!

Cool, thanks for the confirmation! I'll close this issue, then. Let me know if you need further help.


sdxl instaintID Watermark in image

How to fix it?

just don't generate at 1024x1024 size.
i usually stick on 904x1096 or viceversa wheni want to stay close to a square and eventually i crop the image later, btw the watermark fix is just to stay away from 1024x1024


fixed dependency cycle

fixed dependency cycle when building inside project caused by headers build phase after compiling resources

This pull request has been refused, the Contribution License Agreement must be signed.


Monitor initial Blockchain Sync & detect Problems

From time to time people report that the initial Bitcoin validation stalls - like this:
https://github.com/rootzoll/raspiblitz/issues/756#issuecomment-633209542
It would be great to monitor the sync and if progress stalls for a longer time to try counter measures like restarting bitcoind or system.

like reported here #1859

To get alternative seeds to kickstart in a 0 peer scenario - considering to use DNS seed.bitnodes.io
needed install: sudo apt-get install -y dnsutils
see: https://bitnodes.io/api/#dns-seeder
based on IPv4, IPv6 or Tor use the correct record

Best to use the https API of bitnodes.io (over Tor).

I am not able to call the bitnodes API https://bitnodes.io/api/v1/snapshots/latest/ from Tor --> that triggers a CAPTCHA block by cloudflare what makes the API not usable. So the call is for now always clearnet .. what is okisch for now because the API call does not contain any node id that can tied to the IP ... BUT for v1.7 this should get a better solution:
I mailed the author asking for a .onion addreess. If that is not an option I would recommend a small proxy script that calls once a day https://bitnodes.io/api/v1/snapshots/latest/ and offers a hidden service for raspiBlitz user to call anon.

Or use the bitcoin seeds list.
https://github.com/bitcoin/bitcoin/blob/cddcd22ab326428b461f1937f1db4d1329e46faf/contrib/seeds/README.md

@NilByte the kick-start is for edge cases where bitcoind itself is not able to get started with peers (they use DNS seed lists internally) So using a non DNS method delivering a different peer list should be our best shot to circumvent such edge case situations.

Hello. Having the same issue and trying to follow-up with little experience.
I ran bitcoin-cli getnetworkinfo | grep "connections" | tr -cd '[[:digit:]]'
It returns 0
I understand somehow my node is no longer connected to any nodes anymore (would love to know the most common(s) causes of this is.
Then as suggested to get alternative seeds to kickstart in a 0 peer scenario I need to run:
bitcoin-cli addnode "**IP_AND_PORT_OF_OTHER_NODE**" "onetry"
It is stated to use DNS seed.bitnodes.io using utility tool 'dig' which is installed with the dnsutils by running the following command: sudo apt-get install -y dnsutils
OK. dnsutils is installed.
I look at the link you shared https://bitnodes.io/api/#dns-seeder but don't know what to do with it... I ran the command:
dig TXT x5.seed.bitnodes.io because my Raspiblitz is configured with Tor.
What do I do now? I run bitcoin-cli addnode "**IP_AND_PORT_OF_OTHER_NODE**" "onetry" but with what IP_AND_PORT_OF_OTHER_NODE?
Thank you in advance.

OK. This prevented me from sleeping, but got it resolved in the end:
As I realised that I have a 'prod' node that as been running for some time now I realised I could kickstart my blockchain synchronisation of 'test' node using 'prod' node.
I don't know why bitcoin-cli addnode "**LOCAL_IP_OF_PROD_NODE**" "onetry" was not working (returned immediatly doing nothing it appears) I had to use its Tor address: bitcoin-cli addnode "**TOR_ADDRESS_OF_PROD_NODE.onion**" "onetry" (then the command took a couple of seconds to return).
I am happy to say at the time I am finishing this post my 'test' node has finished synchronising the blockchain and look ready to play with :-)

Testing looks good. Closing for v1.6.3 release.

Testing looks good. Closing for v1.6.3 release.


chore(blog): add august summary

community summary for september

@derberg there you go
waiting on @AceTheCreator on the other section

yo folks this article needs to be out today for the newsletter to be complete
/ptal

/ptal

/rtm

thank you @quetzalliwrites


Minor Mispelling

Enrolment -> Enrollment on the numbered list

Merging this PR. @da5is, Thank you so much for your contribution!


New version: Unity.Unity.6000.WindowsIL2CPP version 6000.0.11f1

Created by 🥟 Dumplings in workflow run #6684.

Log
Updated: 6000.0.10f1 → 6000.0.11f1
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000 version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163619
Handling Unity.Unity.6000.Android...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.Android version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163620
Handling Unity.Unity.6000.iOS...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.iOS version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163621
Handling Unity.Unity.6000.tvOS...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.tvOS version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163622
Handling Unity.Unity.6000.visionOS...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.visionOS version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163623
Handling Unity.Unity.6000.LinuxIL2CPP...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.LinuxIL2CPP version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163624
Handling Unity.Unity.6000.LinuxMono...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.LinuxMono version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163625
Handling Unity.Unity.6000.LinuxServer...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.LinuxServer version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163626
Handling Unity.Unity.6000.MacMono...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.MacMono version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163627
Handling Unity.Unity.6000.MacServer...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.MacServer version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163628
Handling Unity.Unity.6000.UWP...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.UWP version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163629
Handling Unity.Unity.6000.WebGL...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request
Pull request created: New version: Unity.Unity.6000.WebGL version 6000.0.11f1 - https://github.com/microsoft/winget-pkgs/pull/163630
Handling Unity.Unity.6000.WindowsIL2CPP...
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

Automatic Validation ended with:

Installation failed with exit code -1978334960
2024-07-19 21:49:12.601 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!00007FF9D658C11F: (caller: 00007FF9D6318D6F) LogHr(1) tid(1f28) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!00007FF9D6492B08: (caller: 00007FF9D653BC20) Exception(1) tid(1f28) 8A15005E ]
2024-07-19 21:49:20.904 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!00007FF9BCDEC11F: (caller: 00007FF9BCB78D6F) LogHr(1) tid(47c) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!00007FF9BCCF2B08: (caller: 00007FF9BCD9BC20) Exception(1) tid(47c) 8A15005E ]
2024-07-19 22:01:50.042 [CLI ] ShellExecute installer failed: 2

Automated error analysis suggests -1978334960 may mean Failed to install package dependencies.
(Automated response - build 887.)


Fix wrong name of recv method in serial server.

Addresses #9.  There is an attempt to map socket API functions to serial API functions but the name receive is used instead of recv.

Thanks!


Variables tag not in correct location for (mouse x)

Error: 'variables' tag must exist once before block and shadow tag elements in the workspace XML, but it was found in another location.

and other not stackblocks


video game sales

[Enter feedback here]
please provide sample pbix for all your pages. eg. In this page you refer to video game sales but the corresponding pbix is missing. Its hard to follow  and not possible to practice
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f919f51d-8623-b039-a1fc-0df774225c46
Version Independent ID: 0376c331-0f7e-ec9b-6262-f47a6c413d47
Content: Decomposition tree - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-decomposition-tree.md
Service: powerbi
Sub-service: powerbi-ai
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @TechnicalCode. This issue has been assigned to the doc author for review.

Thanks for your feedback. Unfortunately we do not have .pbix files for all of the content we create, but some sample files are available.


Added PipeOptions.CurrentUserOnly option to the named pipe streams

#2056

The code changes here look reasonable, but 👀 for us to actually understand all of the issues.

@dotnet-bot test Windows Release x64 Build

🆙 📅  No longer spinning up a server if PipeOptions.CurrentUserOnly isn't available.


feat/initial setup

This is the initial set up for nestjs, it contains eslint and prettier rules, pre-commit for formatting and linting before commit.
There is also a global exception filter that has been enabled so that errors can bubbled up and be caught by the exception filter rather than trying to handle error in every part of the code. I have also enabled global pipe transformation, this would aid in transforming request data and in the use of validator and transformation libraries in nestjs.
The format for data response that can be found in src/filters/http-exceptions is:
{
status: "success" | "error",
message: string | undefined,
data: {} | []
}
the same would be true for the rest of the http responses, this would enable us to have a standard response format.
Additionally, I have enables NestJS Config module globally, so it can be used without having to configure it in every file that it is needed, there is also some vscode rules that has been established, so if anyone opens the code in vscode it would automatically kick in.
This is a PR for this issue #52

This is closed, so that I can reopen a new one.


Simplify types

Since typed-query-selector v2.4.0, we can specify fallback type now, so we don't need to create customized Parse type any more.

Is this still necessary? https://github.com/fregante/select-dom/blob/93c48807aa37333a1b89c962f19863f151a3aade/index.ts#L22

I guess, no. I'm not sure.


MRG: Patch quiver3d()

This PR fixes the bug with the axes in plot_alignment(). The visual bug happened because of a pre-processing in the PyVista glyph() filter that merges the coincident points.
I'll see how to avoid this preprocessing if possible in PyVista directly.
Close #8115

can you share screenshots or update an example so we see that things are ok on the circleci artifact?

Using #8115 code snippet:
import mne
fname = mne.datasets.sample.data_path() + '/MEG/sample/sample_audvis_raw.fif'
raw = mne.io.read_raw_fif(fname)
with mne.viz.use_3d_backend('pyvista'):
    mne.viz.plot_alignment(raw.info, show_axes=True, eeg=False, surfaces=[])

master
PR


security notices

Adds a security notice in the docs for where to report vulnerabilities, and a more general notice on security within binder

good call, just updated the language, WDYT?


Show alpha/beta/RC marks in environment list

When I view my list of installed interpreters, I see the x.y.z version number.

However, I would expect to see the full version number, including whether it's an alpha/beta/RC release and the serial. For example, my copy of 3.8 above is actually 3.8.0b4, but this is not shown in the list.

Hi , we've marked this as needs decision so we can have a discussion on whether we want this enhancement or not.
I am expecting it would clutter the version for some people, as most users don't expect to see that.

Most users won't install prerelease versions, and those who do should know when they've done so.
The alternative would be to use the registered display name (at least on Windows).

Yes, I see your point. For those who have installed the prerelease versions, we should be showing it.


Period Resampling Master Issue

Various issue surrounding resampling with a PeriodIndex

[ ] #12884 Return PeriodIndex if possible
[x] #12770 recursion error on existing freq (#12874)
[x] #12774 count (#12874)
[x] #12868 empty PeriodIndex (#12874)
[ ] #13212 empty dataframe
[ ] #11983 with Panels?
[ ] #7744 convention issues
[x] #12883 recursion with 'W' freq when upsampling (#12874)
[ ] #13083 ohlc resampling

cc @MaximilianR

I can't edit the post, but I think all apart from these are solved:

#12884 Return PeriodIndex if possible
#11983 with Panels?
#7744 convention issues

@MaximilianR really?
we would need verifying tests only then.

we would need verifying tests only then

They should all have tests based on the PRs. (Your commits closed the issues, not me). Or lmk if I'm missing something

those all looks like one issues

oh sorry - you alll spat from those - I checked the boxes :)

Adding to the list, size loses the Series name, after https://github.com/pydata/pandas/pull/13079:
In [4]: pd.Series([], name='test', index=pd.PeriodIndex([],freq='D')).resample('M').size()
Out[4]: Series([], Freq: M, dtype: int64)

You can mark "#11983 with Panels?" off given Panel deprecation

This is also fixed (although I think never had an issue): https://github.com/pandas-dev/pandas/issues/12871#issuecomment-220170356

those are closed as well:
#12884 Return PeriodIndex if possible
#13083 ohlc resampling
#13224 with NaT

Looks like there's only 2 related period-resample issues left in the list (the Timestamp.freq one seems unrelated). I think it's just tractable to manage as individual issues. Closing


check for a given ns istio install before warn

fixes: #28775
rework of https://github.com/istio/istio/pull/28776
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@linsun: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
d69b9b7254f80a0e29a69ddb01aff58687af23d8
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes_istio

@linsun: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-operator-controller-tests_istio
627398c1995881f7cd3c5366a1cd52138ec86ecf
link
/test integ-operator-controller-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-operator-controller-tests_istio


ImportError: libcublas.so.9.0: cannot open shared object file: No such file.

ImportError: libcublas.so.9.0: cannot open shared object file: No such file or directory
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.
[turtlebot3_dqn_stage_4-2] process has died [pid 7852, exit code 1, cmd /home/syzx/catkin_ws/src/turtlebot3_machine_learning/turtlebot3_dqn/nodes/turtlebot3_dqn_stage_4 __name:=turtlebot3_dqn_stage_4 __log:=/home/syzx/.ros/log/de7523e8-4355-11ec-9868-a4bb6d510def/turtlebot3_dqn_stage_4-2.log].
log file: /home/syzx/.ros/log/de7523e8-4355-11ec-9868-a4bb6d510def/turtlebot3_dqn_stage_4-2*.log

I use tensorflow-cpu.why can it have that problem.

Hi @Crawford-fang
I'm sorry but we do not provide support for the error with the tensorflow installation.
Have you followed the instruction from the eManual below?
If you have, make sure that you have selected the correct ROS version at the top of the page.
https://emanual.robotis.com/docs/en/platform/turtlebot3/machine_learning/#software-setup


Add foreign keys!!!!

You have no FK constraints in your system. Besides being a performance hit, this is of grave concern for maintaining data integrity.  Your data is too easily corrupted without it, and a programmer mistake could miss data between patients with lethal consequences.
Fix this ASAP!

Why did you close this issue?!? You are putting people's lives at risk? Have you no empathy?

Because this project is so open to database corruption, it can mean one person's records might be mixed with another person's records. Which means a strong potential for bad diagnoses and other catastrophic results.
Other people are forking this project and starring it. Whether or not you want it, the responsibility for doing this project correctly is yours. Step up and do your job!

Hi @sumitkumar1503 ....the issue mentioned here by @pydanny is of major concern....as it will lead to many issues in the database in the future and you will not be able to figure out which data belongs to which user. And since its a hospital management project, someone's life can also get affected and later they can sue you because of your software. So I would recommend taking the necessary actions to resolve this as pointed out as soon as possible. I hope you will be responsible and fix them before someone gets hurt.

@sumitkumar1503  Pl respect reviewers comments and do the needful. Its natural that human do mistake and same time, we need to resolve it.
Happy coding.

Wow. When someone tries to help. He shut down that path. If you can't update the code. At least ask help for PR. Maybe someone will create PR to resolve this issue.

Because this project is so open to database corruption, it can mean one person's records might be mixed with another person's records. Which means a strong potential for bad diagnoses and other catastrophic results.
Other people are forking this project and starring it. Whether or not you want it, the responsibility for doing this project correctly is yours. Step up and do your job!

yes I agree with you completely if you are encouraging other bad practices you should take responsibility. for no confusing I don not mean any I mean the creator of the project for accuracy of my criticism thank you Danny for you wise inshgt

Guys Guys...
First of all Thanku very much for your valuable comment.
I assure that this type of mistake will not happen.
i commited this type of mistake because i m very new in python..this is my 2nd project (practice)
Now its solved by @pydanny
Thanku Sir @pydanny


[SPARK-13659] Refactor BlockStore put*() APIs to remove returnValues

In preparation for larger refactoring, this patch removes the confusing returnValues option from the BlockStore put() APIs: returning the value is only useful in one place (caching) and in other situations, such as block replication, it's simpler to put() and then get().
As part of this change, I needed to refactor BlockManager.doPut()'s block replication code. I also changed doPut() to access the memory and disk stores directly rather than calling them through the BlockStore interface; this is in anticipation of a followup patch to remove the BlockStore interface so that the disk store can expose a binary-data-oriented API which is not concerned with Java objects or serialization.
These changes should be covered by the existing storage unit tests. The best way to review this patch is probably to look at the individual commits, all of which are small and have useful descriptions to guide the review.
/cc @davies for review.

Test build #52419 has started for PR 11502 at commit 6381b00.

Test build #52419 has started for PR 11502 at commit 6381b00.

I'm not familiar with BlockManager, the changes looks straightforward to me, only one minor question.
LGTM over all.

Test build #52419 has finished for PR 11502 at commit 6381b00.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52419/
Test FAILed.

Merged build finished. Test FAILed.

I think this failed an unrelated flaky test in CommitFailureTestRelationSuite.

Jenkins, retest this please.

Test build #52438 has started for PR 11502 at commit 6381b00.

Test build #52438 has finished for PR 11502 at commit 6381b00.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52438/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Test build #52478 has started for PR 11502 at commit 6381b00.

Test build #52478 has finished for PR 11502 at commit 6381b00.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52478/
Test FAILed.

Jenkins, retest this please.

Test build #52489 has started for PR 11502 at commit 6381b00.

Test build #52489 has finished for PR 11502 at commit 6381b00.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52489/
Test FAILed.

Jenkins, retest this please.

Test build #52499 has started for PR 11502 at commit 6381b00.

Test build #52499 has finished for PR 11502 at commit 6381b00.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52499/
Test PASSed.

Merged build finished. Test PASSed.

Alright, I've updated this to address the review comments. Note, however, that most of the code that I changed in my latest commit is going to be immediately deleted / rewritten in the larger refactoring patch that will follow this one.

Test build #52503 has started for PR 11502 at commit 3d8f479.

Test build #52504 has started for PR 11502 at commit 910f997.

Test build #52503 has finished for PR 11502 at commit 3d8f479.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52503/
Test PASSed.

Test build #52504 has finished for PR 11502 at commit 910f997.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52504/
Test PASSed.

LGTM I'm merging this into master.


Specific capitalization required for currency iso values.

I was passing in eur to Globalize.currencyFormatter and getting a missing cldr data error, which was pretty confusing to track down. It turned out that I should have been passing EUR and it worked fine. Seems easy enough to do a .toUpperCase() on the currency value for users and/or detect lowercase letters and throw a more specific error.

So, let's improve parameter assertions and throw an error in that case. Then we stick with ISO 4217 for the currency code.

... and encourage a consistent and standard usage of currency codes.


feat(popover): accessible hover popoover

Description

Had a contact card wrapper in Cantina code that handled this only for contact cards. Expanded the solution to momentum Popover component to handle this for all hoverable popovers.

Use ENTER to open hoverable popovers
Use SHIFT+TAB on the first tabindex="0" element, TAB on the last tabindex="0" element, or ESC to close the popover
The tabbable children in EventOverlay are focusable (previous this was not possible since the popover would close on any keydown)

Related Issue

https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-126886
Motivation and Context

Accessibility for hoverable popover
How Has This Been Tested?

Manually tested on web.webex.com
Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Please add example of different props in Popover examples file.

@aimeex3 @andrewholsted Sorry I forgot that I had to add that TAB and SHIFT+TAB cycle between the tabbable children by default when closeOnFocusLeavesContent is false

@bfbiggs can you review? I added examples. Thank you.


Can't paste image to chat or channel

If I copy an image to the clipboard, ctrl+v or right click paste does nothing in Teams. I have confirmed the image is in the clipboard as I can paste it elsewhere, office...slack...  If I copy the image from Word, it will paste.

We hope to fix these clipboard problems in the next few months. It's irritating to us too.

We are tracking this issue internally. Closing this from documentation issues.


restful_api_host did not use in anywhere

'restful_api_host' in MIIConfig class does not use in
https://github.com/microsoft/DeepSpeed-MII/blob/25c33d77a03604c53b5dd0ef642bb72d630abc75/mii/backend/client.py#L33
It uses localhost all the time.

This should be fixed once we merge in #364.


Not playing on android

Im on "react-native-sound": "0.10.1"
Calling play does not produce sound and the callback is invoked with the value false. coming from the onError listener in RNSoundModule.
Happens on both an emulator and a device. sdk 19 and 25.
Before that i was using the version from  TheGhoul21 fork and the sound did play.
The player returns true when I check for IsLoaded() .
and sadly There is no more information i can give. if someone points me in the right direction so I can try editing the js and/or android code to fix the problem.
example file:
https://languagezenstorage.blob.core.windows.net/media0/xgcUXjHhP8.mp3

File name must be all lowercase

Android: Save your sound clip files under the directory android/app/src/main/res/raw. Note that files in this directory must be lowercase and underscored (e.g. my_file_name.mp3) and that subdirectories are not supported by Android.

@njwest
This is not a local file. so the limit is not set in stone, files in the wild are case sensitive.  My memory was that this "feature" of lower casing the input path was removed. it's a real problem otherwise.

I checked the source again, the file name used unchanged (this is good). but It still does not play and no useful information is given as for the problem.

Can you post your code? I'm running an Android app with 0.10.1 right now and it's playing audio without problems.

Sure
this.recording= new Sound(path, MEDIA_URI, (error) => {
        if (error) {
            console.log('failed to create  sound', error);
            return;
        }

        console.log('sound is loaded: ', recording.isLoaded());
    });

Later
  recording.play((success) => {
        if (success === true) {
            console.log('successfully finished playing');
        } else {
            console.log('play  failed due to unknown reason ' + success);
        }
    });

The value of recording.isLoaded() is true before I call play.
I edited the android code to return the error codes and not just "false" when the onError delegate is invoked.
The error Android is throwing is (-38,0),

Thanks - I'm debugging an Android device while trying to load a networked resource, and I'm seeing:
05-12 18:39:23.260 23518-23553/com.rnsounddemo I/RNSoundModule: https://languagezenstorage.blob.core.windows.net/media0/xgcUXjHhP8.mp3
05-12 18:39:34.963 23518-23553/com.rnsounddemo E/MediaPlayer: Attempt to call getDuration in wrong state: mPlayer=0xcf790b80, mCurrentState=2

My suspicion is that the changes introduced in #169 are causing this (removed the .prepare method). I'll investigate.

@ofirgeller Well, suspicion confirmed. Adding prepare if it's a remote URL does indeed make it play. However, at least in debug mode, the whole app is blocked while the HTTP request is made. If you turn off Wifi the whole app pauses for 10-20 seconds. This obviously isn't great

@benvium This makes a lot of sense, when I was solving the same problem using Xamarin I had to call prepare to make it work, I then wrapped the call into a different thread to avoid the blocking.,
  var prepreTask = Task.Run(() => player.Prepare());
Can we try doing the same thing in java?

@benvium  I'm giving a shot to fixing it. will update here if I give up

@ofirgeller I've been having a go too - I can get it to work using .prepareAsync instead, but then using require() to play an audio no longer works :-(

How do you use require? from js? can you are the code that stops working so I can test it as well?

Sure, take a look at the branch fix_android_network_sound at https://github.com/zmxv/react-native-sound/tree/fix_android_network_sound
I've been testing with the try-android-download-fix branch of https://github.com/zmxv/react-native-sound-demo/tree/try-android-download-fix
If you pull that you'll need to update the package.json to point to the branch above (haven't done that yet).

I think ill stop now as otherwise we are going to have a merge-fest
If you want you can skype me (ofir.geller) and we can work on it together for a while. if not I prefer waiting until the code is no longer in flex.

Facing the same issue. Trying to load an audio from an amazon s3 URL. Facing issue in Android alone. Error message reads - resource not found. Any solution for this?

Hi guys, can we please merge #184 until it is fixed properly? Blocking is not a big deal for my app since I load all my sounds when it boots, before any UI is rendered. I can see how that would be a problem for a music player, but blocking is better than silence.

I updated #184 to use prepareAsync, which is a better stop-gap for now

Sounds are not working on Android yet. Any updates?

Same issue. Only sounds from the network are working on Android.

I'm giving up on this library for Android, for now at least..

Me too :(

Try testing in different android version. I just found this is not working on android 4.4. So I'll hide our audio feature for those users :(

You can add a folder called raw in android/app/src/main/res and put a copy of the audio file in it.


Feature Request: Customizable source sets per build type, build flavor

It would be great to be able to customize which source folders to be included/excluded automatically, per build type (debug or release), similar to this
E.g. code in src/debug would be included in the classpath for debug builds only, src/release for release builds only, and src/main for shared code that would be included always.
Is this possible? Is it possible to add support for build flavors as well? I am trying to emulate the functionality of the gradle android plugin, and am not sure what is/isn't possible with SBT.

Flavors are already supported. Custom sources for a debug/release build is not.
You can try modifying sources in Compile depending on the value of apkbuildDebug in Android

How can build flavors be specified?

https://github.com/pfn/android-sdk-plugin#new-features-in-13x-last-version-1324


Capture JS errors and logs in the servers handle_info call

Blocking each webdriver call with a call to check for js errors and logs is an expensive and wasteful process. All of this same information is sent to the server process from phantom. We should be able to capture this information using the same logging capture behaviour and then print logs to the console or throw errors in our respective test. This should work correctly because each phantom is scoped to a single test at a time.

@keathley - So I was playing around with this and had a question about how you want to handle javascript errors on the Genserver side?  In the case where we have a javascript error, raising a Wallaby.JSError goes uncaught and crashes the Genserver if you move current logging behavior server side.
We could do something to notify the client, but doesn't that kinda defeat the purpose of this as we'd still be checking for an error on each call in the driver? I still think we can handle logging server side,  but I'm looking to you for ideas on what to change for handling js errors?


Example bot crashes with segmentation fault when added to a group

I tried the example bot from the documentation that just echoes back whatever you write (echo/main.go).
Then I added the bot to a group with myself and tried to talk to it in the group. This is the error I get:
[signal SIGSEGV: segmentation violation code=0x2 addr=0x48 pc=0x102af14f0]

goroutine 34 [running]:
main.handler({0x102bffdf8, 0x1400002a200}, 0x1400014c000, 0x14000310000)
	/Users/marito/Work/cacabot/main.go:34 +0x40
github.com/go-telegram/bot.(*Bot).ProcessUpdate.func1()
	/Users/marito/go/pkg/mod/github.com/go-telegram/bot@v1.2.2/process_update.go:25 +0x6c
github.com/go-telegram/bot.(*Bot).ProcessUpdate(0x1400004e770?, {0x102bffdf8?, 0x1400002a200?}, 0x0?)
	/Users/marito/go/pkg/mod/github.com/go-telegram/bot@v1.2.2/process_update.go:36 +0xbc
github.com/go-telegram/bot.(*Bot).waitUpdates(0x1400014c000, {0x102bffdf8, 0x1400002a200}, 0x0?)
	/Users/marito/go/pkg/mod/github.com/go-telegram/bot@v1.2.2/wait_updates.go:17 +0x6c
created by github.com/go-telegram/bot.(*Bot).Start in goroutine 1
	/Users/marito/go/pkg/mod/github.com/go-telegram/bot@v1.2.2/bot.go:116 +0xa4

Note that I get the same error whenever I run the bot since then. Probably because it's still trying to download the message that causes it to crash.

Line 34 contains this:
ChatID: update.Message.Chat.ID,
This is the entire function:
func handler(ctx context.Context, b *bot.Bot, update *models.Update) {
	b.SendMessage(ctx, &bot.SendMessageParams{
		ChatID: update.Message.Chat.ID,
		Text:   update.Message.Text,
	})
}

You can use handler
func handler(ctx context.Context, b *bot.Bot, update *models.Update) {
	if update.Message != nil {
		b.SendMessage(ctx, &bot.SendMessageParams{
			ChatID: update.Message.Chat.ID,
			Text:   update.Message.Text,
		})
	}
}

Simple examples don't cover all the ways a bot can be used


luv_unix not buildable with esy (dependency issue)

Hey @aantron -- just one more issue I found with luv_unix :)
OS: macOS 11 (Big Sur)
esy version: 0.6.10
Problem: If I put both luv and luv_unix in my package.json like so:
  "dependencies": {
    ...
    "@opam/luv": "aantron/luv:luv.opam#decdaad",
    "@opam/luv_unix": "aantron/luv:luv_unix.opam#decdaad",
    ...
  }

building luv_unix fails with the following message:
    File "src/unix/luv_unix.ml", line 22, characters 9-50:
    22 |       if C.Functions.Os_fd.is_invalid_handle_value os_fd then
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    Error: Unbound value C.Functions.Os_fd.is_invalid_handle_value
    error: command failed: 'dune' 'build' '-p' 'luv_unix' '-j' '4' (exited with 1)

It seems to be because luv_unix is being built before luv, but I don't know how to enforce the order (i.e. make esy know that luv_unix depends on luv being built first)

The order should already be forced by this line:
https://github.com/aantron/luv/blob/decdaad7511855d012e31f22b35538ddd874c08d/luv_unix.opam#L18
have you positively confirmed that they are being built out of order?
I can't seem to eyeball this issue. To rule out inherent problems with luv_unix, are you able to build luv_unix in a development checkout of the Luv repo (i.e. outside esy)?

I can't seem to eyeball this issue. To rule out inherent problems with luv_unix, are you able to build luv_unix in a development checkout of the Luv repo (i.e. outside esy)?

I am, that's actually how I discovered #108 :)

have you positively confirmed that they are being built out of order?

As far as I can tell I have -- I deleted all builds of luv and luv_unix, and when I run esy build, I get
info esy build 0.6.10 (using package.json)
info building @opam/luv_unix@github:aantron/luv:luv_unix.opam#decdaad@d41d8cd9

which seems to indicate that luv_unix is being built before luv
This seems like it might be an esy problem rather than an luv or opam one, but I'm not 100% sure of that

info esy build 0.6.10 (using package.json)
info building @opam/luv_unix@github:aantron/luv:luv_unix.opam#decdaad@d41d8cd9

Is there build output for @opam/luv? If not, it may be because it is cached. In light of that, this paste alone doesn't explicitly show the order. What does it mean that you have deleted all builds? Have you also deleted ~/.esy? If so, can you confirm and/or show the full build output, with the positions of @opam/luv and @opam/luv_unix included? AFAIK if you had actually deleted all cached builds, you would see much more than this after info esy build.

Ah you're right, it looks like I missed one of the cached builds.
I figured out what it was -- luv_unix was requesting luv 0.5.7 from OPAM, which didn't have the function definition. If I move the packages to resolutions, it works, i.e.:
  "resolutions": {
    "@opam/luv": "aantron/luv:luv.opam#decdaad",
    "@opam/luv_unix": "aantron/luv:luv_unix.opam#decdaad"
  }

Sorry for the confusion!

That's inconvenient. I would have hoped esy would build a consistent set of packages. I'll have to note this for future instructions, thanks!


Mesh qualification improvements

Thanks for the review. I fixed the issues and updated the pull request.

Added commit msg to "mesh: fix handling group send with local interface enabled" to provide more information and fixed a typo in the same commit.


Is it possible to output inside style tags instead of an external css file?

Hi,
Is it possible to output the generated css file into a block of internal css between two style tags in an html file instead of generating an external file?
Thanks

This depends on which tools you are using. You should probably just create a file and then inject that into your HTML.